Enhanced model generation + code cleanup

Code cleanup.
Modules now only import needed typeorm decorators.
Removed column pluralization due to wrong column names being generated.
Fixed an issue where columns clashed w/ BaseEntity props. (save -> saveAttribute)
Added the class validators below to modules. (use -v false to disable)@IsOptional()
@IsEmail()
@IsString()
@IsNumber()
@IsIn()

Added the option to change indentation style. (e.g. -i tab, -i 2, -i 4)
Replaced handlebars w/ custom serialization.private serializeEntity(entity) {
      this.typeormImports = {BaseEntity: true, Entity: true};
      this.validateImports = {};
      entity.relationImports();

      const tableName = entity.EntityName;
      const className = this.convertCase(tableName, this.Options.convertCaseEntity);
      const eol = this.getEOL();
      let content = '';

      content += this.flatten(entity.UniqueImports, this.serializeEntityImports);
      content += `${eol}@Entity('${tableName}')${eol}`;
      content += this.flatten(entity.Indexes, this.serializeIndexes);
      content += `export class ${className} extends BaseEntity {${eol}${eol}`;
      content += this.flatten(entity.Columns, this.serializeColumn);
      content += `}${eol}`;

      return `${this.serializeImports()}${content}`;
}

Improved console message.[02:26:52] Creating model classes...
[02:26:52] Done. Created 39 TypeORM model classes.

Improved module formatting. (below generated w/ default settings, no modifications)import {BaseEntity, Entity, Index, PrimaryGeneratedColumn, Column, OneToMany, OneToOne} from 'typeorm';
import {IsNumber, IsOptional, IsString, IsEmail, IsIn} from 'class-validator';
import {AccountBanHistory} from './AccountBanHistory';
import {AccountBans} from './AccountBans';
import {AccountViplist} from './AccountViplist';
import {Players} from './Players';

@Entity('accounts')
@Index(['name'], {unique:true})
@Index(['sessionkey'])
export class Accounts extends BaseEntity {

  @PrimaryGeneratedColumn()
  @IsNumber()
  public id: number;

  @Column({length:254, nullable:true})
  @IsOptional()
  @IsString()
  public username?: string | null;

  @Column({length:32})
  @IsString()
  public name: string;

  @Column({type:'char', length:64})
  @IsString()
  public password: string;

  @Column({length:254})
  @IsEmail()
  public email: string;

  @Column({type:'enum', enum:["en","es","de","se","pt","pl"], nullable:true})
  @IsOptional()
  @IsIn(["en","es","de","se","pt","pl"])
  public language?: string[] | null;

  @Column({type:'char', length:30})
  @IsString()
  public sessionkey: string;

  @Column({type:'tinyint'})
  @IsNumber()
  public stayloggedinforsession: number;

  @Column({type:'char', length:16, nullable:true})
  @IsOptional()
  @IsString()
  public authsecret?: string | null;

  @Column({type:'char', length:2, nullable:true})
  @IsOptional()
  @IsString()
  public country?: string | null;

  @Column()
  @OneToMany(t=>AccountBanHistory, r=>r.accountId)
  public accountBanHistory: AccountBanHistory[];

  @Column()
  @OneToOne(t=>AccountBans, r=>r.accountId)
  public accountBans: AccountBans;

  @Column()
  @OneToOne(t=>AccountViplist, r=>r.accountId)
  public accountViplist: AccountViplist;

  @Column()
  @OneToMany(t=>Players, r=>r.accountId)
  public players: Players[];

}

I thought I commented this PR earlier, but I guess I forgot. Sorry.

Tests are failing.
Some strings use quotation marks other use apostrophes.

@Cavitt great job. Any news on this PR? :)

I haven't had time due to personal reasons. I will finish the PR in the coming weekends. Sorry guys!

@afurculita That would be much appreciated. All the features are complete, just needs tests fixed and formatting.

Closing PR as there were no changes for a long time. Codebase changed a lot since then so it would be hard to merge it.
I plan to go back to some features/changes introduced here in the future, but I won't have time to do it in the near future.


can someone help me and check out why these settings arent opening roblox

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[ ] My answer given in the checkbox above is a lie.

What problem did you encounter?
ClientAppSettings MAIN.json

Did you read this warning at all or no

Just don't use this. Delete everything you've configured in the flag editor please.


Please Add the manifest Link in below sample Table

Please add the manifest Link for the below Code Sample Table-

Sample name
Description
.NET
Node.js
Manifest

Meeting stage sample
Sample app to show a tab in meeting stage for collaboration.
View
View
View

Meeting tabs
This sample shows app Stage View, Mute/Unmute Teams meeting audio call in meeting Side panel tab.
View
View
View

Document- https://learn.microsoft.com/en-us/microsoftteams/platform/apps-in-teams-meetings/build-tabs-for-meeting?tabs=mobile%2Cmeeting-chat-view-mobile%2Cpre-or-post-meeting-tab%2Cmeeting-stage-view-mobile%2Cchannel-meeting-mobile#code-sample

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

ID: e44c95e7-3258-df7e-224d-d60ea1cac46c
Version Independent ID: e44c95e7-3258-df7e-224d-d60ea1cac46c
Content: Build tabs for meeting - Teams
Content Source: msteams-platform/apps-in-teams-meetings/build-tabs-for-meeting.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: surbhigupta

Meeting stage sample	Sample app to show a tab in meeting stage for collaboration.	View View

@surbhigupta12 -Could you please help to update the document?


Improve the text at the top of the home page

git push origin feature/home-page-text

git push origin feature/home-page-text


Naming conventions and terminology

One thing I have repeatedly found tricky to name or describe, is how to refer to the concepts that Length and Force represent.
It is natural to refer to centimeters and newtons as units, or units of measure, but I always lacked a good name for their grouping/family. After some googling, it seems physical quantity is the correct term.
wiki/Units_of_measurement spells it out pretty well:

A unit of measurement is a definite magnitude of a quantity, defined and adopted by convention or by law, that is used as a standard for measurement of the same quantity.[1] Any other value of that quantity can be expressed as a simple multiple of the unit of measurement.
For example, length is a physical quantity. The metre is a unit of length that represents a definite predetermined length. When we say 10 metres (or 10 m), we actually mean 10 times the definite predetermined length called "metre".

Example :

You can convert between units of the same physical quantity.

Proposal

Physical quantities are represented by struct types like Length and Force and corresponding enum types LengthUnit and ForceUnit
Units (or units of measurement) are represented by enum values like LengthUnit.Meter and ForceUnit.Newton
Unit conversions are represented by instance properties, such as Length.Meters and Force.Newtons and custom conversion methods like Force.FromPressureByArea()
Generic type parameters are named

TQuantity for unit struct types, such as Length
TQuantityEnum for unit enum types, such as LenghtUnit
TUnit (short hand for TQuantityUnit) for unit enum values, such as LengthUnit.Meter

Any thoughts on the matter?
I would like to go through the docs and source to ensure the wording is used consistently, as well as on future PR reviews.
Sources:
https://en.wikipedia.org/wiki/Physical_quantity
https://en.wikipedia.org/wiki/Units_of_measurement

Cool! I really like this idea. I took a quick look through the quantities that are currently defined to see if any wouldn't fit the "physical" descriptor. Most are, with the exception of Level, which I suppose is arguable.
I wonder if simply calling these top-level classes "quantities" would suffice?

That makes sense to me, I can go with quantity.

Added a bullet point for naming convention of generic type parameters.

I am not fond of TQuantityEnum, I would rather use these names:

TQuantity
TQuantityType, IQuantityKind
TQuantityUnit

I would keep Quantity in all names for more consistency.

@tongbong Thanks for commenting.
I don't like the name TQuantityEnum either, but since C# generics don't have a way to constrain types to enums I figured this was maybe a bit more intuitive for the reader? I could go with Type though, it's also most often used for enum types.
How about this:

TQuantity (struct type)
TQuantityUnitType (enum type)
TQuantityUnit (enum value)

The pairing of UnitType and Unit becomes more clear to me at least.

Sorry to chip in, just found this library as I need something like this on a current project.
From metrology I believe the term is Dimension as in dimensional analysis.
In a library I wrote a while back I had three classes, Quantity, Unit and Measure - this allowed me to represent and convert between different units as long as they were representing the same underlying Measure.
Measure represented the seven base SI units {and held the integer exponents so length (m) was {1, 0, 0, 0, 0, 0, 0}, and so area (m^2) was {2, 0, 0, 0, 0, 0, 0} and speed (m/s) becomes {1, 0, 0, 0, 0, 0, 1}
Advantage of this is that each Unit is of one Measure and holds it conversion factor back to the base SI units, but you can easily see if you can convert between values, or what Units the result of a calculation are in.

@phatcher You are most welcome to chip in! So, are you saying you would prefer the term dimension when referring to the concepts of Length and Mass?
Interesting take on the measure representation. Did you mean -1 for the last number in speed m/s ?

@anjdreas Yes, the concept is that each of the 7 SI measures is a separate Dimension, then a Measure is some combinations of the exponents of those Dimensions - and yes I did mean -1 for the m/s.
Then Units become a Measure with a conversion factor/function - I used the conversion to the base SI units as it tends to minimise loss of precision when you round-trip and when you have more complicated Measures you also avoid compounding the error by storing a derived conversion factor rather than computing one from the individual units e.g. m/s to furlongs/fortnight has its own calculated conversion value.
I like your strong types, but I was having to store this in a database and also cope with arbitrary new units that might be created by users, which is why I have the more generic Units/Measure/Dimension - the original problem domain was commodity trading so we were trying to get to a single source of truth for handling trade units  e.g. "barrels" which depending on the trade feed were US Barrels or Imperial Barrels :-)
Using the Measure idea we could then ensure that if we were expecting a volume we could validate that we had a volume and not an area or a unit of energy. The other nice thing is that basic operations such as multiply/divide are easily catered for as you just combine the Measures in the appropriate way.
The thing my scheme doesn't easily handle is arbitrary independent scalar units e.g. "lots" or more dynamic conversions such as currency

This is all interesting, but how do you suggest we apply these naming conventions to this project? Ref the initial description of this issue.

I really like the concept with the array of SI unit exponents. That could be used for automatic generation of operator overloads. I don't have time to do an implementation of that right now, but I've always wanted to automate it but haven't been sure how. I'll create an issue/pull request once I have time to keep this issue focused.

@anjdreas Sorry, re-reading the question I'd suggest

TMeasure - Used for struct types such as Length and Force
TMeasureType/Kind - Used for the enums such as LengthUnit
TUnit - Used unit enum values, such as LengthUnit.Meter

Rationale is that though Length is a Dimension under my modelling, you are also include Force which is a Measure (since it's Nm => (kg m^2)/s^2)).
The TMeasureType/Kind follows the suggested pattern and then Meter really is a Unit - don't like the use of TQuantity as what we are representing isn't a quantity but a unit of measure
BTW I can ask this in another question if you prefer, but is there anyway of using this library generically as I for my current use-case I need to store values with an arbitrary unit in a database. I don't have the code for my old project (proprietary) which is why I was looking in the first place :-)

@eriove It's absolutely interesting, but my feeble mind can't immediately see how to apply this in this project to cover our existing units. Could you draft up a design in a few textual bullet points?
@phatcher Thanks, it looks good, but I'm not sure if the term measure is a better fit than quantity. Intuitively from my layman's perspective, I could interpret measure to mean a measurement, like 1 cm and 10 kg. I revisited your link on metrology and found the definition of units with a table that lists a column Base quantity with values Length, Mass, Time, Electric current etc. So they also seem to apply the term quantity to these. I found no use of the term measure in the same article to refer to these concepts, other than "the ability to measure" and "unit of measure".
Then there is the quote from wiki on Units of measurement:

A unit of measurement is a definite magnitude of a quantity, defined and adopted by convention or by law, that is used as a standard for measurement of the same quantity.[1] Any other value of that quantity can be expressed as a simple multiple of the unit of measurement.
For example, length is a physical quantity. The metre is a unit of length that represents a definite predetermined length. When we say 10 metres (or 10 m), we actually mean 10 times the definite predetermined length called "metre".

To counter myself, I also did not initially find quantity intuitive - as for me that also refers to some measurement, like 10 kg of dirt, but I do find quantity or rather physical quantity to be used to describe the concepts of Length and Mass in the wiki articles I've read so far.
What do you think?
Sources:
https://en.wikipedia.org/wiki/Metrology#Definition_of_units
https://en.wikipedia.org/wiki/Physical_quantity
https://en.wikipedia.org/wiki/Units_of_measurement

@anjdreas I think the problem is that there doesn't seem to be a word that separates the concept being measured from the units - quantity seems to be used interchangeably for both.
I thought Measure was used in the literature, but it looks like I'm mis-remembering a text book from a while back.
The issue is currently. Length is a Quantity, metre is a Unit and 10 m is a Quantity; which to me is confusing so the answer is to rename concept 1 or 3, so you could have

Length -> Quantity or Measure
Metre -> Unit
10 m -> Quantity or Measurement

To remove the conceptual ambiguity one needs to go - which will confuse some people but only until they understand the issue.

@phatcher I see what you're saying. I think I see a way to interpret this now:
It is troublesome that both the concept and the measurement of length is referred to as quantity. However, in Units.NET, we usually talk about the struct type Length and its units. When I talk about the struct type Length, I don't talk about the concept of length - a measurable, continuous quantity. I am talking about representing a quantity of length.
var distanceToAlphaCentauri = Length.FromLightYears(4.22);
This is the quantity of length for this distance. It is not necessarily a measurement in this context, but it was measured at some point. This may be splitting hairs, but I think the wording quantity fits more generally here.
So, although this wording is tricky, I think we can skip your point 1 and only care about the length units (enum) and the length quantities/measurements (struct). In the original post we also talk about enum names vs enum values in the context of generics, but I think those are simpler to reason about.

Also, this was a nice read on the topic. It also has an "Examples" section.
https://en.wikipedia.org/wiki/Quantity

So my updated suggestion is as follows:
Changes to original post:

Physical quantities -> Quantities
TQuantityEnum -> TUnitType

Updated section in original post:
Proposal

Quantities are represented by struct types like Length and Force and corresponding enum types LengthUnit and ForceUnit
Units are represented by enum values like LengthUnit.Meter and ForceUnit.Newton
Unit conversions are represented by instance properties, such as Length.Meters and Force.Newtons and custom conversion methods like Force.FromPressureByArea()
Generic type parameters are named

TQuantity for unit struct types, such as Length
TUnitType for unit enum types, such as LengthUnit
TUnit for unit enum values, such as LengthUnit.Meter

@anjdreas Looks good

I think we are getting somewhere, I'm happy with this.
Any closing comments?
cc @eriove @JKSnd

@anjdreas I'm happy with the updated proposal as well. Nice stuff.

Alright, closing this.
Fixed in commit ea592ab79d93b7845f5438cc24b648a122cbffe8.
I also realized that we're not using a generic type for unit enum values and it doesn't even make sense as its type is the enum. Removing TUnit from bullet list.


Configure /etc/rsyslog.conf after package installation

This file could be created before packages installation. This commit
forces the configuration of the file after the package installation.
When the file is created empty before package installation, then this action could fail with errors like
Error: Execution of '/usr/bin/apt-get -q -y -o DPkg::Options::=--force-confold install rsyslog' returned 100: Reading package lists...
Building dependency tree...
Reading state information...
The following additional packages will be installed:
  liblognorm5
Suggested packages:
  rsyslog-mysql | rsyslog-pgsql rsyslog-doc rsyslog-relp rsyslog-elasticsearch
  rsyslog-mmjsonparse rsyslog-imptcp rsyslog-gnutls rsyslog-openssl
  rsyslog-udpspoof rsyslog-mmrm1stspace rsyslog-mmutf8fix rsyslog-kafka
  rsyslog-omstdout
The following NEW packages will be installed:
  liblognorm5 rsyslog
0 upgraded, 2 newly installed, 0 to remove and 7 not upgraded.
Need to get 62.3 kB/724 kB of archives.
After this operation, 1845 kB of additional disk space will be used.
Get:1 http://ppa.launchpad.net/adiscon/v8-stable/ubuntu bionic/main amd64 liblognorm5 amd64 2.0.6-0adiscon1bionic1 [62.3 kB]
Fetched 62.3 kB in 0s (263 kB/s)
Selecting previously unselected package liblognorm5.
(Reading database ... 89635 files and directories currently installed.)
Preparing to unpack .../liblognorm5_2.0.6-0adiscon1bionic1_amd64.deb ...
Unpacking liblognorm5 (2.0.6-0adiscon1bionic1) ...
Selecting previously unselected package rsyslog.
Preparing to unpack .../rsyslog_8.2001.0-0adiscon1bionic1_amd64.deb ...
Unpacking rsyslog (8.2001.0-0adiscon1bionic1) ...
Setting up liblognorm5 (2.0.6-0adiscon1bionic1) ...
Setting up rsyslog (8.2001.0-0adiscon1bionic1) ...

Configuration file '/etc/rsyslog.conf'
 ==> File on system created by you or by a script.
 ==> File also in package provided by package maintainer.
 ==> Keeping old config file as default.

This commit fixes this problem.

Hi @amateo, can you take a look at the failing spec tests?

I think it is now fixes. Do you want me to rebase it in just one commit?

yes, please rebase and squash into one.

Done!


Icon request: Acorns

URL of the site or app (required): https://www.acorns.com/
Logo resources for the icon (optional): https://www.acorns.com/press/

Here you go!
https://github.com/aegis-icons/aegis-icons/blob/master/SVG/Acorns.svg

❓ How to add icon(s) to Aegis?
⭐ Support this project by giving a star! Just adding like to this comment is also appreciated.


fix tiktok download

Currently, tiktok downloads fail with status 403.
This request fixes that by downloading the page twice before trying the video.
This seems like a bug on their end since i noticed the same problem in the webapp:
the request for the video file fails with 403 when opening a video in a fresh incognito session, but it works when i reload.

Unfortunately there is no retry option for downloading webpages or info files like in json.
However the mentioned approach here is poorly implemented.
I don't have any problems even trying multiple times. You could at most implement a solution that checks the http status and retries again after a certain timeout.
Currently you force even users without problems to download the page twice.
Also have you tried something like --force-ipv4 or --force-generic?

@GreyAlien502 We should look at implementing what @blackjack4494 is referring to. Check the page status code, and call a second time if we get 503 back.

@blackjack4494, We aren't retrying anything here,
especially not the webpage/info file download, which never fails,
The second call is preemptive, to avoid getting a 403 later, when we get the video.

I don't have any problem

Oh, i wasn't aware that the current code was working for anyone (without providing cookies).
It always fails for me.

have you tried something like --force-ipv4 or --force-generic

I just tried --force-generic, and it sometimes works for me.
Since it makes multiple requests for each resource it downloads,
it's not too surprising that it would.

@merval, an InfoExtractor will not know that there's a problem
since the error doesn't occur during the information extraction, right?
I think it'd be simpler to just make sure that the video will download correctly the first time.

tiktok download is down again.
U:\youtube-dlc -i -o "%(id)s.mp4" -v https://www.tiktok.com/@tiktok_stage/video/6885152542095887617
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-i', '-o', '%(id)s.mp4', '-v', 'https://www.tiktok.com/@tiktok_stage/video/6885152542095887617']
[debug] Loading archive file None
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dlc version 2020.11.07
[debug] Python version 3.9.0 (CPython) - Windows-10-10.0.19041-SP0
[debug] exe versions: ffmpeg git-2020-08-07-fab00b0, ffprobe git-2020-08-07-fab00b0
[debug] Proxy map: {}
[TikTok]<PHONE_NUMBER>095887617: Downloading video webpage
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on 'https://v16-web.tiktok.com/video/tos/alisg/tos-alisg-pve-0037c001/9b3784d6dec84c75b0233ca0d90572c5/?a=1988&br=1272&bt=636&cr=0&cs=0&cv=1&dr=3&ds=3&er=&expire=1604852024&l=202011081013320101151511041701C791&lr=tiktok&mime_type=video_mp4&policy=2&qs=0&rc=M2x3OWhzaTtpeDMzaTgzM0ApZjhoZzczPGU7NzdnNGc5O2dtLmhoaGtxZzNfLS02LzRzc2JeNGM2MzUxLjAzXzVjMDM6Yw%3D%3D&signature=7773bd3019bd845e9f8ee935e9ef478b&tk=tt_webid_v2&vl=&vr='
ERROR: unable to download video data: HTTP Error 403: Forbidden
Traceback (most recent call last):
File "youtube_dlc\YoutubeDL.py", line 2011, in process_info
File "youtube_dlc\YoutubeDL.py", line 1866, in dl
File "youtube_dlc\downloader\common.py", line 373, in download
File "youtube_dlc\downloader\http.py", line 349, in real_download
File "youtube_dlc\downloader\http.py", line 114, in establish_connection
File "youtube_dlc\downloader\http.py", line 110, in establish_connection
File "youtube_dlc\YoutubeDL.py", line 2316, in urlopen
File "urllib\request.py", line 523, in open
File "urllib\request.py", line 632, in http_response
File "urllib\request.py", line 561, in error
File "urllib\request.py", line 494, in _call_chain
File "urllib\request.py", line 641, in http_error_default
urllib.error.HTTPError: HTTP Error 403: Forbidden

btw, this was fixed in ytdl-org/youtube-dl by adding the extra call to tiktok in the _real_initialize method.

how do you have fixed the bug downloading tiktok resposnse with 403? i have the same probem to download the url
"https://v16-web.tiktok.com/video/tos/alisg/tos-alisg-pve-0037c001/5d1561cb4b5c4e94b68c2d8aefc2526b/?a=1988&br=4898&bt=2449&cd=0|0|1&cr=0&cs=0&cv=1&dr=0&ds=3&er=&expire=1609504918&l=2021010106413301011515316709B85BC4&lr=tiktok_m&mime_type=video_mp4&policy=2&qs=0&rc=anU0bHY6bG46eDMzNTczM0ApNTRlaTczNmU8N2lkZmc8PGdpYmkyMm9rXmJfLS0vMTRzcy40MTAwNF82YmFjMy41LzM6Yw%3D%3D&signature=8028848b03a13b99813cb39eb52e5b11&tk=tt_webid_v2&vl=&vr="

how do you have fixed the bug downloading tiktok resposnse with 403? i have the same probem to download the url
"https://v16-web.tiktok.com/video/tos/alisg/tos-alisg-pve-0037c001/5d1561cb4b5c4e94b68c2d8aefc2526b/?a=1988&br=4898&bt=2449&cd=0|0|1&cr=0&cs=0&cv=1&dr=0&ds=3&er=&expire=1609504918&l=2021010106413301011515316709B85BC4&lr=tiktok_m&mime_type=video_mp4&policy=2&qs=0&rc=anU0bHY6bG46eDMzNTczM0ApNTRlaTczNmU8N2lkZmc8PGdpYmkyMm9rXmJfLS0vMTRzcy40MTAwNF82YmFjMy41LzM6Yw%3D%3D&signature=8028848b03a13b99813cb39eb52e5b11&tk=tt_webid_v2&vl=&vr="

@tanliang1 Provide the full verbose output (with -v)

@tanliang1 Provide the full verbose output (with -v)


adds a reader for compliance data types. moves the list compliance da

tatypes type definitions to the list directory.

implements getFildIdentifierOptions to extract dropdown options from complianceDataTypes list returned from GET /complianceDataTypes endpoint. adds unit tests. adds mirage mock for complianceDataTypes response and fixtures.

Coverage remained the same at 26.678% when pulling d262468070630f4b270945c1dc479679e23536f9 on theseyi:flexible-tagging into bafa6ab5fdab000aa3ac52adfa1cd35c8caf435a on linkedin:master.

Codecov Report

Merging #915 into master will decrease coverage by <.01%.
The diff coverage is 70.58%.

@@            Coverage Diff             @@
##           master     #915      +/-   ##
==========================================
- Coverage   36.48%   36.48%   -0.01%
==========================================
  Files         311      312       +1
  Lines        8436     8473      +37
  Branches     1587     1595       +8
==========================================
+ Hits         3078     3091      +13
- Misses       5358     5382      +24

Impacted Files
Coverage Δ

...ataset-compliance/-dataset-compliance-entities.hbs
100% <ø> (ø)
:arrow_up:

wherehows-web/app/templates/datasets/dataset.hbs
100% <ø> (ø)
:arrow_up:

wherehows-web/app/constants/dataset-compliance.ts
100% <100%> (+3.03%)
:arrow_up:

wherehows-web/app/routes/datasets/dataset.js
9.25% <100%> (-0.04%)
:arrow_down:

wherehows-web/app/constants/datasets/compliance.ts
100% <100%> (ø)
:arrow_up:

wherehows-web/app/components/dataset-compliance.js
8.03% <100%> (-0.04%)
:arrow_down:

...ows-web/app/utils/api/list/compliance-datatypes.ts
51.16% <62.5%> (ø)

...ehows-web/app/components/dataset-compliance-row.ts
25.38% <66.66%> (-0.2%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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


fix(chat): hide Edit and Delete for shared entity (Issue #1257)

Description:
hide Edit and Delete for shared entity
Issues:

Issue #1257

UI changes
<Please, provide Screenshots or Figma links>
Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review


500 error visiting /authenticate/

GET https://pennydreadfulmagic.com/authenticate/?target=https%3A%2F%2Fpennydreadfulmagic.com%2Fadmin%2Farchetypes%2F
will repro.

"The session is unavailable because no secret key was set. Set the secret_key on the application to something unique and secret."

See #2798.


Update Datafusion Table Provider patch to fix MySQL refresh append mode

🗣 Description
Update Datafusion Table Provider patch to include the following changes:

https://github.com/datafusion-contrib/datafusion-table-providers/pull/174
https://github.com/datafusion-contrib/datafusion-table-providers/pull/173

🔨 Related Issues

📄 Documentation Requirements

🤔 Concerns

Do we have to document any of these changes so the users know how TIMESTAMPs should be handled?

@slyons Yes this needs to be documented in docs repo. I'll create a separate PR for that


Problem formatting document or formatting selection in html (select and option tag)

VSCode Version: 1.11.0 - Insiders 5184d06
OS Version: Windows 10

-- Sorry for the English, I used Google Translator --
See the Gif

Code Example
<!DOCTYPE html>

	<meta charset="UTF-8">
	<meta name="viewport" content="width=device-width, initial-scale=1.0">
	<meta http-equiv="X-UA-Compatible" content="ie=edge">
	<title>Document</title>

	<select>
<option value=""></option>
<option value=""></option>
<option value=""></option>
</select>

This is because select is in the list of tags that must not be formatted.
You can modify that list in the user settings with the "html.format.unformatted" setting.

Wow !! I did not know that function. Thank you so much. It worked now !! 😄


e2e: mustgather: fix test to verify TAS enabled nodes only

So far the CI worked with the default node labels which is "worker" and
the NodeGroups of the numaresourcesoperator CR we expected to target the
worker mcp. However, this isn't always the case and the test should be
smarter to determine which nodes it should look for their folders in the
collected must-gather.
Adjust the step of building the expected CRs paths to include only nodes
that the operator targets.

the intent of the change is good, but these tests should NOT query the live cluster, but only operate on the must-gather data. IOW, the tests should not assume there is a live cluster at all (there could be bugs on which we do query the live cluster: these are just bugs and need to be removed ASAP)

Ack, I'll re-add the deleted function and fetch TAS nodes from the collected info. Thanks for highlighting this.

Can we use the new API AKA PoolName so there will be less work when we would like to adapt for HCP?

Can we use the new API AKA PoolName so there will be less work when we would like to adapt for HCP?

Yes please let's try to do like this

Can we use the new API AKA PoolName so there will be less work when we would like to adapt for HCP?
same mind here, and that's where GetListByLabels() come in handy. However this depends on FindTreesHypershift from  https://github.com/openshift-kni/numaresources-operator/pull/1027/files#diff-31ede3f56b1f873ccf421371942426d8ad0c30160d467d790dd7e050802d25d7R109 so we need that to get merged first

the intent of the change is good, but these tests should NOT query the live cluster, but only operate on the must-gather data. IOW, the tests should not assume there is a live cluster at all (there could be bugs on which we do query the live cluster: these are just bugs and need to be removed ASAP)

There are 2 places where we need a live cluster for this test:

get the MCP labels
get the daemonset namespace

we can use a hardcoded namespace but the most important part is the gathering the node selectors from the MCPs. Only on hypershift we wouldn't need that because we know how we build the labels if PoolName is provided on HCP (part of  #1027). On OCP we'll always need MCPs even if PoolName is the node group specifier that was set.

the intent of the change is good, but these tests should NOT query the live cluster, but only operate on the must-gather data. IOW, the tests should not assume there is a live cluster at all (there could be bugs on which we do query the live cluster: these are just bugs and need to be removed ASAP)

There are 2 places where we need a live cluster for this test:

get the MCP labels
get the daemonset namespace

we can use a hardcoded namespace but the most important part is the gathering the node selectors from the MCPs. Only on hypershift we wouldn't need that because we know how we build the labels if PoolName is provided on HCP (part of #1027). On OCP we'll always need MCPs even if PoolName is the node group specifier that was set.

that can actually be done by checking the collected MCPs. I'll verify that is possible and update.

the intent of the change is good, but these tests should NOT query the live cluster, but only operate on the must-gather data. IOW, the tests should not assume there is a live cluster at all (there could be bugs on which we do query the live cluster: these are just bugs and need to be removed ASAP)

There are 2 places where we need a live cluster for this test:

get the MCP labels
get the daemonset namespace

we can use a hardcoded namespace but the most important part is the gathering the node selectors from the MCPs. Only on hypershift we wouldn't need that because we know how we build the labels if PoolName is provided on HCP (part of #1027). On OCP we'll always need MCPs even if PoolName is the node group specifier that was set.

that can actually be done by checking the collected MCPs. I'll verify that is possible and update.

Updated in latest push. We still need to depend on the live daemonset to fetch the namespace as it can be different depending on the deployment method as even mustgather uses that:
https://github.com/openshift-kni/numaresources-operator/blob/main/must-gather/collection-scripts/namespace

/lgtm


Unhandled exception in utils._force_rmtree()

See original issue on GitLab
In GitLab by [Gitlab user @tristanvb] on Nov 13, 2017, 10:24
This happened today during a GNOME build:
[--:--:--] BUG     [apps/qqwing.bst]: Build

    An unhandled exception occured:

    Traceback (most recent call last):
      File "/codethink/GNOME/buildstream/buildstream/_scheduler/job.py", line 265, in child_action
        result = self.action(element)
      File "/codethink/GNOME/buildstream/buildstream/_scheduler/buildqueue.py", line 34, in process
        element._assemble()
      File "/codethink/GNOME/buildstream/buildstream/element.py", line 1135, in _assemble
        cleanup_rootdir()
      File "/codethink/GNOME/buildstream/buildstream/element.py", line 1058, in cleanup_rootdir
        utils._force_rmtree(rootdir)
      File "/codethink/GNOME/buildstream/buildstream/utils.py", line 481, in _force_rmtree
        shutil.rmtree(rootpath, **kwargs)
      File "/usr/lib/python3.5/shutil.py", line 480, in rmtree
        _rmtree_safe_fd(fd, path, onerror)
      File "/usr/lib/python3.5/shutil.py", line 418, in _rmtree_safe_fd
        _rmtree_safe_fd(dirfd, fullname, onerror)
      File "/usr/lib/python3.5/shutil.py", line 418, in _rmtree_safe_fd
        _rmtree_safe_fd(dirfd, fullname, onerror)
      File "/usr/lib/python3.5/shutil.py", line 438, in _rmtree_safe_fd
        onerror(os.unlink, fullname, sys.exc_info())
      File "/usr/lib/python3.5/shutil.py", line 436, in _rmtree_safe_fd
        os.unlink(name, dir_fd=topfd)
    IsADirectoryError: [Errno 21] Is a directory: 'mount'

In GitLab by [Gitlab user @tlater] on Nov 14, 2017, 17:42
I can't seem to reproduce this locally, but it looks like this may have been caused by a sandbox failing to unmount a directory.
Both implementations check quite thoroughly for this, as far as I am aware, are there any other messages in the logs for this?

In GitLab by [Gitlab user @franred] on Jul 11, 2018, 22:34
I've reproduce this issue aborting a build using "ctrl + c"

In GitLab by [Gitlab user @valentindavid] on Oct 2, 2018, 22:23
mentioned in commit 7cc121fee74f15165c3bd23ed312b136e6fd5a17

In GitLab by [Gitlab user @valentindavid] on Oct 2, 2018, 22:24
mentioned in merge request !849

In GitLab by [Gitlab user @valentindavid] on Oct 2, 2018, 22:25
!849 should fix it.

In GitLab by [Gitlab user @valentindavid] on Oct 4, 2018, 15:55
mentioned in commit 26d48cc9c3870eb9ca3dd7816d32b8fe0606414e

In GitLab by [Gitlab user @valentindavid] on Oct 4, 2018, 15:56
mentioned in commit a79842185f3e762154002c6e928da30d6745568b

In GitLab by [Gitlab user @valentindavid] on Oct 4, 2018, 15:57
mentioned in merge request !861

In GitLab by [Gitlab user @valentindavid] on Oct 4, 2018, 16:31
closed via merge request !849

In GitLab by [Gitlab user @valentindavid] on Oct 4, 2018, 16:31
mentioned in commit b7b20d9e7c46d8b3bc7541437cc66905ffd28fc0


chroma help

<EMAIL_ADDRESS>agent
npx tsc && node dist/index.js

chroma/clients/js/jest.config.ts:1:29 - error TS2307: Cannot find module '@jest/types' or its corresponding type declarations.
1 import type { Config } from '@jest/types'
~~~~~~~~~~~~~
chroma/clients/js/test/add.collections.test.ts:1:30 - error TS2307: Cannot find module '@jest/globals' or its corresponding type declarations.
1 import { expect, test } from '@jest/globals';
~~~~~~~~~~~~~~~
chroma/clients/js/test/client.test.ts:1:30 - error TS2307: Cannot find module '@jest/globals' or its corresponding type declarations.
1 import { expect, test } from '@jest/globals';
~~~~~~~~~~~~~~~
chroma/clients/js/test/collection.client.test.ts:1:30 - error TS2307: Cannot find module '@jest/globals' or its corresponding type declarations.
1 import { expect, test } from '@jest/globals';
~~~~~~~~~~~~~~~
chroma/clients/js/test/collection.client.test.ts:4:1 - error TS2304: Cannot find name 'beforeEach'.
4 beforeEach( async() => {

chroma/clients/js/test/collection.test.ts:1:30 - error TS2307: Cannot find module '@jest/globals' or its corresponding type declarations.

1 import { expect, test } from '@jest/globals';
                             ~~~~~~~~~~~~~~~

chroma/clients/js/test/delete.collection.test.ts:1:30 - error TS2307: Cannot find module '@jest/globals' or its corresponding type declarations.

1 import { expect, test } from '@jest/globals';
                             ~~~~~~~~~~~~~~~

chroma/clients/js/test/get.collection.test.ts:1:30 - error TS2307: Cannot find module '@jest/globals' or its corresponding type declarations.

1 import { expect, test } from '@jest/globals';
                             ~~~~~~~~~~~~~~~

chroma/clients/js/test/peek.collection.test.ts:1:30 - error TS2307: Cannot find module '@jest/globals' or its corresponding type declarations.

1 import { expect, test } from '@jest/globals';
                             ~~~~~~~~~~~~~~~

chroma/clients/js/test/query.collection.test.ts:1:30 - error TS2307: Cannot find module '@jest/globals' or its corresponding type declarations.

1 import { expect, test } from '@jest/globals';
                             ~~~~~~~~~~~~~~~

chroma/clients/js/test/update.collection.test.ts:1:30 - error TS2307: Cannot find module '@jest/globals' or its corresponding type declarations.

1 import { expect, test } from '@jest/globals';
                             ~~~~~~~~~~~~~~~

node_modules/langchain/dist/cache.d.ts:1:38 - error TS2307: Cannot find module 'redis' or its corresponding type declarations.

1 import type { RedisClientType } from "redis";
                                     ~~~~~~~

Found 12 errors in 11 files.


Stray "" in template on API Documentation page

On the API Documentation you have a stray  in the template on top under Global Options:
template: "

        <button class='btn btn-default' data-role='prev'>« Prev</button>
        |
        <button class='btn btn-default' data-role='next'>Next »</button>

    <button class='btn btn-default' data-role='end'>End tour</button>

",

See before last line.
This seems to be correct further down, and is also they way the code has been written.
I know this is not directly related to the code itself, but I found no better way of notifying you. Hope this helps.

Thank you @benomatis. It will be fixed.


Slider component don't work well when together label within hboxcontainer.

Godot version:
3.1.1
OS/device including version:
mac 10.14.4
Issue description:
I want add a label instruction before label. But when I put them in HBoxContainer. It size very small.
Steps to reproduce:
create a HBoxContainer
add a Label
add a Slider
resize the Slider component.
preview those.

set min width of slider

Did you put the horizontal expand flag to true in the slider ? If not, that's an expected behavior.

@volzhs It works. Thanks


Remove version: ChilliBits.Spice version 0.19.1

This version is older than what has been set in max-versions-to-keep by the publisher.
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/spicelang/spice/releases/download/0.19.1/spice_x64_setup.msi
Status Code: 200
(Automated message - build 857)


fix rice worldgen

spawn above water. should spawn in water. not sure how tho...

actually. crop worldgen will be replaced with some additions to villages and crop mutation/breeding


test: add integrate test

What this PR does:
Add more components and api features in integrate test
Which issue(s) this PR fixes:

Fixes #415

@bxiiiiii Hi, you don't have to do it:

Let me revert your commits. I'm testing the CI


.HTAccess Problem

I have problem with my .htaacess, my subfolder
ROOT
-----WEB
-----------application
-----------system
-----------themes
index.php
.htaccess
but my themes folder can't get right folder, the folder under "WEB" folder...
this my htaccess:
## Set up mod_rewrite
<IfModule mod_rewrite.c>
Options +MultiViews +FollowSymLinks
DirectoryIndex index.php index.html

# Enable Rewrite Engine
# ------------------------------
RewriteEngine On

# UPDATE THIS TO POINT TO where you installed this FROM YOUR DOC ROOT.
# If this is in the DOC ROOT, leave it as it is
# ---------------------
# RewriteBase /WEB/

# In case your hosting service doesn't add or remove 'www.' for you, you can
# do it here by uncommenting and updating the 'Rewrite*'s below.
#
# Add or remove 'www.'  Whichever you prefer.
# This one removes the 'www.' which seems to be the favorable choice these days.
# ------------------------------
#RewriteCond %{HTTP_HOST} ^www.<sitename>.com
#RewriteRule (.*) http://<sitename>.com/$1 [R=301,L]

# Redirect index.php Requests
# ------------------------------
RewriteCond %{THE_REQUEST} ^GET.*index\.php [NC]
RewriteCond %{THE_REQUEST} !/system/.*
RewriteRule (.*?)index\.php/*(.*) $1$2 [R=301,L]

# Standard ExpressionEngine Rewrite
# ------------------------------
RewriteRule modules/(.+)/controllers/(.+)\.php$ /index.php?/$1/$2 [L,R=301]
RewriteRule controllers/(.+)\.php$ /index.php?/$1 [L,R=301]

RewriteCond $1 !\.(css|js|gif|jpe?g|png) [NC]
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^(.*)$ index.php/$1 [L]
</IfModule>

Thanks...

What your website path from the browser? Is it something like http://website.com or http://website.com/folder/?

my root path, yes, http://website.com and same result ... ithink the problem is in apache and .htaccess ... with your htaccess have some problem too...
try out my application and system same position ... with root path.... tomorow i will try..
have anice weekend...

What is your server environment? Are you using WAMP, XAMP? Ubuntu (if so which version)? I want to replicate your environment so I can test.

It's working if my application,system,assets, themes, etc. in root folder.
root_folder
------Web
------CI-CMS
----------------PageStudioCMS
application
system
assets
themes
index.php
.htaccess
I'm using CentOS, with LAMP, i don't know why it's so different from  my application others, with modules system, thats my htaccess before is working..

I'll have to test that on CentOS. What version of Apache are you using?

Sorry, I've been working on other features. I'll take a look at these issues within the next two weeks. Hopefully I can fix them then.

Yes, it's working now... i must installed in fresh server... i don't know why... but it's resolved. Thanks for your attention brothers.


google-chrome-dev: fix error on startup

Motivation for this change
Fix [..]/crashpad_handler: No such file or directory.
Things done

Verified that google-chrome-stable and google-chrome-beta still start normally with this change.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 121267 at c9d89c2f run on x86_64-linux 1

3 packages built successfully:

google-chrome
google-chrome-beta
google-chrome-dev

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/networking/browsers/google-chrome/default.nix:102:3:
    |
102 |   installPhase = ''
    |   ^


Upgrade Rails to newest version

We are getting things ready for upgrading CK to Rails 7. Before we even get there, we need to make sure our other apps and vendor gems are running on Rails 7.1.2.
Not sure how this rails version is used at all but I searched for rails versions in all of our repo's and I think it's worth keeping them up to date. Please let me know if you know something that I don't :)

I might need access to run CI on this branch. I will ask about it when everyone is back.

@ivanmp91 are you able to make CI run for this branch?

@ivanmp91 are you able to make CI run for this branch?

Done! It seems that's working now: https://app.circleci.com/pipelines/github/ConvertKit/story_teller/26/workflows/dfd29f60-b6c7-4e0f-bd06-de5ef02d54d7

@stefannibrasil Looks like there are some specs failing

ahh, sweet, thanks! I will investigate the error 👀


Add timestamps for Boots of Travel combination

In match info there is no timestamp for Boots of Travel 1 or 2 combining.

What combining are you referring to? The item is considered purchased when the recipe is bought, right?

What combining are you referring to? The item is considered purchased when the recipe is bought, right?

I'm referring to when the hero gets both combines both components in their inventory.
eg in: https://www.opendota.com/matches/5772722561/purchases
Last component for pudge's aghanim's scepter is purchased at 28:29, but the full item is combined in the inventory at 28:45
There is no timestamp for either the Boots of Travel 1 or 2 upgrades for the same hero.

I'm referring to when the hero gets both combines both components in their inventory.
eg in: https://www.opendota.com/matches/5772722561/purchases
Last component for pudge's aghanim's scepter is purchased at 28:29, but the full item is combined in the inventory at 28:45
There is no timestamp for either the Boots of Travel 1 or 2 upgrades for the same hero.

Hmm, looks like there is no new item logged in the purchase logs when travels are created. We usually filter out recipe purchases, but might have to make an exception in this case

Hmm, looks like there is no new item logged in the purchase logs when travels are created. We usually filter out recipe purchases, but might have to make an exception in this case

Perhaps we should just add purchase event for the respective boots of travel?

How do you proposed we do that?

How do you proposed we do that?

Adding an else if contains boots of travel id here, and then replacing the recipe part of the ID
https://github.com/odota/core/blob/461a5cb51cf3508e04edd6dd0b0498bc776f86b0/processors/processExpand.js#L280

hmm yeah I guess that would work. It might cause a duplicate event if we ever reparse old pro games?

hmm yeah I guess that would work. It might cause a duplicate event if we ever reparse old pro games?

What are you referring to?
I tested on a recent match and it works as intended, can do further tests as well.

On a second thought it might be better to just include the recipe, since there are many edge cases - making this more suitable for web to sort out.

What I mean is that old replays emitted a purchase event for boots of travel. So if we converted the recipe purchase to an item purchase we would see two purchase events


The colours should be merged somehow or a custom window style should be created.

Photon UI Kit docs suggest us to use a frameless window, as that have been done in https://github.com/fmoliveira/tantalus/commit/9cf78ce32c3b25cd25b401389d89e1ac0e284209.
This however raises another issue that is to miss the minimize, maximize and close buttons.

The frameless window also leave the application with no border. I've added a simple solid border in https://github.com/fmoliveira/tantalus/commit/37fe5fdb6c8a8c3a8ad874e2a5ace39143c0687a while this issue isn't solved.


ng-naver-maps 0.3.3

Add support for information windows.
Add support for map on-click function.

Build updated successfully.


Exposing port 18081

Hi,
Thank you for the work you've put in this. I have an issue with exposing port 18081 so that I can use it to connect it to a dashboard.
This is my Docker Compose config.
version: '3.3'

services:
  monerod:
    image: sethsimmons/simple-monerod:latest
    user: 0:0
    volumes:
      - /mnt/vol1/monero/bitmonero:/home/monero/.bitmonero
    deploy:
      mode: replicated
      replicas: 1
    networks:
      - monero
    ports:
      - 18080:18080
      - 18089:18089
    expose:
      - "18080"
      - "18081"
      - "18089"
    command:
      - "--rpc-restricted-bind-ip=<IP_ADDRESS>"
      - "--rpc-restricted-bind-port=18089"
      - "--public-node"
      - "--no-igd"
      - "--enable-dns-blocklist"
      - "--prune-blockchain"
  tor:
    image: goldy/tor-hidden-service:latest
    environment:
        MONEROD_TOR_SERVICE_HOSTS: 18089:monerod:18089
        MONEROD_TOR_SERVICE_VERSION: '3'
    volumes:
      - /mnt/vol1/monero/tor:/var/lib/tor/hidden_service/
    networks:
      - monero
    deploy:
      mode: replicated
      replicas: 1
      placement:
        constraints:
          - node.hostname != rpi
  gui:
    image: hvalev/monero-dashboard:latest
    networks:
      - monero
      - caddy
    environment:
      MONERO_HOST: monerod
      MONERO_PORT: 18081
      TICKER: "true"
      PORT: 3333
    deploy:
      mode: replicated
      replicas: 1
      labels:
        caddy: "monero.ca.varunpriolkar.com"
        caddy.reverse_proxy.to: "{{upstreams 3333}}"
        caddy.tls.dns: "route53"
        caddy.tls.dns.access_key_id: "{env.AWS_ACCESS_KEY}"
        caddy.tls.dns.secret_access_key: "{env.AWS_SECRET_KEY}"

networks:
  caddy:
    external: true
  monero:
    driver: overlay
    attachable: true

I'm assuming using "expose" keyword here should be enough:
    expose:
      - "18080"
      - "18081"
      - "18089"

However I can't seem to connect to 18081 from other container, just 18080 and 18089. Is there some option that I need to exclude in the commands? This is the output debugging it from the monero-gui container:
/home/monero # telnet monerod 18081
Trying <IP_ADDRESS>...
telnet: Unable to connect to remote host: Connection refused
/home/monero # telnet monerod 18089
Trying <IP_ADDRESS>...
Connected to monerod.
Escape character is '^]'.
^[^]
\q
telnet> Connection closed.
/home/monero # telnet monerod 18081
Trying <IP_ADDRESS>...
telnet: Unable to connect to remote host: Connection refused
/home/monero # telnet monerod 18080
Trying <IP_ADDRESS>...
Connected to monerod.
Escape character is '^]'.

Let me know if you need any more information. Thank you!

Looks like I didn't Google hard enough. Found the magic options:
--rpc-bind-ip=<IP_ADDRESS>
--confirm-external-bind"

Incase anyone stumbles upon this. Sorry about the trouble.


New getting started page

fixed some invalid metadata

added to toc

Temp fix for #540

Fixed old package

Cleaned up metadata - Bug 995730

https://github.com/Azure/azure-docs-sdk-dotnet/issues/630

bugfix

fix formatting

rewrite getting started

rewrite getting started

rewrite getting started

updates

updates

fixing issue while I'm here

final tweaks

formatting

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/overview/azure/eventgrid.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/dotnet-sdk-azure-get-started.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/overview/azure/eventgrid.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/dotnet-sdk-azure-get-started.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Avoid functions returning functions

This question has been in my head for a long time. I don't understand what's the point/benefit of having makeTestMatcher and makeSourceMatcher, which return functions.
Take the following example from the readme:
const isTest = avaFiles.makeTestMatcher();

isTest(filePath);
//=> true or false

const isSource = avaFiles.makeSourceMatcher();

isSource(filePath);
//=> true or false

What we could have instead:
const isTest = avaFiles.isTest(filePath);
// true or false

const isSource = avaFiles.isSource(filePath);
// true or false

The API is simpler & nicer, does not make & return functions and can be used in filter or map the same way.

:+1:

IIRC those factory methods originally closed over variables that were not available where the returned methods were called. That's not the case now AvaFiles is a class.

PR is in the making.

I'm short on time nowadays, if someone wants to pick this up, please do :)


chore: bump go mod 36

Fixes #N/A
Description
How was this change tested?
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 77.707%

Files with Coverage Reduction
New Missed Lines
%

pkg/providers/amifamily/ami.go
1
90.37%

Totals

Change from base Build<PHONE_NUMBER>5:
-0.01%

Covered Lines:
5633

Relevant Lines:
7249

💛  - Coveralls


Unregister package requests

Edit (April 2014): Updated by @paulirish & @sheerun
Package unregistering will be available via bower unregister <package> soon, but for now, you can unregister packages yourself using curl, if the package is hosted on GitHub and you're an owner or collaborator.
curl -X DELETE "https://bower.herokuapp.com/packages/PACKAGE?access_token=TOKEN"

Where PACKAGE is the package name you want to delete and TOKEN is GitHub's Personal Access Token that you can fetch from here: https://github.com/settings/applications
A default GitHub Personal Access Token will work -- no permissions necessary
You need to be an owner or collaborator of the repo and URL needs to be OK.
You'll likely want to bower cache clean after your change.
Please remember it is generally considered bad behavior to remove versions of a library that others are depending on. Think twice :)

If the above doesn't work for you, we can handle your case manually (and please post below). And again, soon we'll be addressing the remaining issues so this is much easier for everyone. :) Cheers.

Please unregister angularjs-scroll-glue and angularjs-scroll-glue-Luegg. The former is a no longer maintained fork and the later a ill-named registration someone else created. The correct package is angular-scroll-glue.

Could you please re-register git://github.com/outsideris/angular-summernote.git to git://github.com/summernote/angular-summernote.git
I got GitHub.com error from curl -X DELETE ...
Thank you.

Please unregister primer because the repo git://github.com/code-curve/primer.git is no longer available. Thanks!

Please re-register zingchart to git@github.com:zingchart/ZingChart.git . Thanks!

Please unregister ports (git@github.com:epistemex/ports.git). Thanks (and same as others: curl with option -X fails in windows).

Hi. Please re-register "viewerjs" from "github.com/aurigadl/Viewer.js" to "github.com/kogmbh/ViewerJS_release".
The former is an unmaintained 3rd-party package (whose author @aurigadl did not react to emails to the github registered email address in the last 2 months), while the latter is the maintained package from the original project developers.

Please unregister webodf_build.
It is an unmaintained 3rd-party package (whose author @aurigadl did not react to emails to the github registered email address in the last 2 months). A maintained package for the library webodf.js  from the original project developers is already registered as webodf.js.

@parris unregistered react-android-timepicker :white_check_mark:
@prettynatty @Luegg re-registered angularjs-scroll-glue to git://github.com/prettynatty/angularjs-scroll-glue.git :white_check_mark:. This package had lots of hits, so it's important to keep the registration.
@Luegg unregistered angularjs-scroll-glue-Luegg :white_check_mark:
@outsideris re-registered angular-summernote to git://github.com/summernote/angular-summernote.git :white_check_mark:
@stephenwinsor unregistered primer :white_check_mark:
@mike-schultz re-registered zingchart to git://github.com/zingchart/ZingChart.git :white_check_mark:
@epistemex unregistered ports :white_check_mark:
@kossebau

for viewerjs I've opened kogmbh/ViewerJS_release#1 to discuss resolving this
for webodf_build, There's no conflict here. Both webodf.js and webodf_build can co-exist.

No need to say thanks. Thank you for using Bower :bird:

Hi bower team!
Summernote had moved from hackerwins/summernote to summernote/summernote.
Please re-register it. Thanks.

Please re-register tmpl.js to git://github.com/felixexter/tmpl.js.git.

for viewerjs I've opened kogmbh/ViewerJS_release#1 to discuss resolving this

Hopefully solved now as far as also supporting the older version not covered before. So looking forward to next manual registration maintenance round :)

for webodf_build, There's no conflict here. Both webodf.js and webodf_build can co-exist.

Given that webodf_build package is outdated/unmaintained I would have proposed to clean things up, as it does not add anything to what the maintained version offers, and people might only get frustrated to learn there is another version with bugs fixed and new stuff added.  But your call.

bower unregister famous-material didn't work as unregister isn't a command (at least on my system) and the CURL option wasn't doing anything (visible) at all =-/
Could you please unregister and reregister famous-material from https://github.com/StephanBijzitter/Famous-Material to https://github.com/Skelware/Famous-Material?

Hi!
Having Application Error during curl request, so
please, remove this jquery.rateit from Bower repo

Please uninstall ngjs-color-picker.
Thanks.

I was getting a "GitHub.com error" when trying to unregister Papa Parse. Somehow it got registered 3 times, and I'd like to unregister all of them and start fresh:
papaparse
papaparse.js
papa-parse
I'm not sure how they appeared on Bower, as I've never used Bower before. Could you please unregister those? I'll re-register it in a little bit once I'm ready.

I'm getting "GitHub.com error" when trying to unregister as well.
Can knockout-jqueryui be transferred to @gvas? Thanks!

Could you please unregister these packages:

genesis-helper
genesis-wordpress
henryCSS

Thanks!

@lqez re-registered summernote to git://github.com/summernote/summernote.git :white_check_mark:
@felixexter re-registered tmpl.js to github.com/felixexter/tmpl.js.git :white_check_mark:
@StephanBijzitter re-registered famous-material to git://github.com/Skelware/Famous-Material.git :white_check_mark:
@mholt unregisterd papaparse, papaparse.js, and papa-parse :white_check_mark:. These packages had 1000's of hits, so they're fairly popular. We'd appreciate if you could re-register at least one
@jrcamp re-registered knockout-jqueryui to git://github.com/gvas/knockout-jqueryui.git :white_check_mark:
@csshugs unregistered genesis-helper and henryCSS :white_check_mark:

No need to say thank you. Thank you for using Bower.

Thanks @desandro - I didn't know that. I re-registered Papa Parse under the name papaparse. But I found one more stray entry that should be removed; I'm not sure who put it there, and the repo doesn't exist anymore:

papa.parse (git://github.com/mholt/jquery.parse.git)

^- probably a good idea to create a system to automatically delete packages that haven't been valid for 30 days? 1 cron job a day on a server and 30 continuous fails = bye bye

I just need to transfer my package
mongular-js
from
git://github.com/jasonrichardsmith/mongular-js.git
to
https://github.com/mongular/mongular-js

can someone please change my 2 bower projects, i ended up putting ui-bootstrap instead of bootstrap-ui
They are
ui-bootstrap-datetime-picker
to
bootstrap-ui-datetime-picker
and
ui-bootstrap-treeview
to
bootstrap-ui-treeview
Can you let me know when this is done and i will update it on github. Thanks

[ ] angular-timeline git://github.com/destegabry/angular-timeline.git

Owner does not support bower and issue has been open for > 12 months (see https://github.com/destegabry/angular-timeline/issues/2) without response.

Hello,
Could you please un-register the following packages?

nglivejournal
xmlrpc

The git repos were moved to the new location.
I will register them again once they are removed.
Thanks,
Leonid.

Hi, i need to erase the lokijs-datagrid package.
Thanks!

Can you please change the url of showdown package to https://github.com/showdownjs/showdown and the owner to myself?
The url is outdated (before changing to a organization repository).

Please move bitcore-channel from git://github.com/eordano/bitcore-channel.git to git://github.com/bitpay/bitcore-channel.git.

Please move bellhop from git://github.com/CloudKidStudio/Bellhop.git to git://github.com/SpringRoll/Bellhop.git. Thank you!

@jasonrichardsmith re-registered mongular-js to git://github.com/mongular/mongular-js.git :arrow_right:
@rpocklin unregisterd angular-timeline :negative_squared_cross_mark:
@leoz

re-registered nglivejournal to git://github.com/leoz/ngLiveJournal.git :arrow_right:
re-registered xmlrpc to git://github.com/leoz/xmlrpc.js.git :arrow_right:

@ManRueda unregistered lokijs-datagrid :negative_squared_cross_mark:
@tivie re-registered showdown to git://github.com/showdownjs/showdown.git :arrow_right:
@eordano re-registered bitcore-channel to git://github.com/bitpay/bitcore-channel :arrow_right:
@bigtimebuddy re-registered bellhop to git://github.com/SpringRoll/Bellhop.git :arrow_right:

No need to say thanks. Thank you for using Bower :bird

@Gillardo Both bootstrap-ui-datetime-picker and bootstrap-ui-treeview are open to be registered. There is no need to unregister ui-bootstrap-datetime-picker or ui-bootstrap-treeview

please move picky-sass from git://github.com/ginacontrino/picky-sass.git to git://github.com/pickygoat/picky-sass.git
Admire the work :)

please move angular-tags-input from git://github.com/anim/angular-tags-input.git to git://github.com/PLEEROCK/angular-tags-input.git

I'm sorry I failed during the registration. Please move flex-min to flex-grid (https://github.com/Ppjet6/flex-grid)

Please, move *scribe-global * from https://github.com/guardian/scribe to https://github.com/intactile/scribe-global

Please change the repo for pretty-embed:
current: git://github.com/erquhart/bower-pretty-embed
new: git://github.com/mike-zarandona/prettyembed.js

@desicochrane re-registered picky-sass to git://github.com/pickygoat/picky-sass.git :arrow_right:
@PLEEROCK re-registered angular-tags-input to git://github.com/PLEEROCK/angular-tags-input.git :arrow_right:
@gontard re-registered scribe-global to git://github.com/intactile/scribe-global.git :arrow_right:
@erquhart re-registered pretty-embed to git://github.com/mike-zarandona/PrettyEmbed.js.git :arrow_right:

@Ppjet6 You do not need to unregister a package to register it with a different name. flex-grid is already registered, but you are free to register your package with a different name.

@desandro thank you. Please check out my updated comment.

@desandro, thank you for the information. Could you remove flex-min still, this one was a mistake.

Please unregister "file-input".  I'm the owner of the parent repo.  This user forked and registered the package just moments after I unregistered it during an attempt to relocate the repo to another organization.
The original location is git@github.com:garstasio/file-input.git, but I am going to move this to git@github.com:rnicholus/file-input.git shortly.

I tried to do this with curl but it did not recognize me as the owner of the repository.
I would like to register a new owner and url for "cois-sprite-importer". Currently it is https://github.com/dvberkel/sprite-importer.git but ownership it transferred to https://github.com/Chair-of-Indefinite-Studies/sprite-importer.git

Please update adapter.js from git@github.com:GoogleChrome/webrtc.git to git@github.com:webrtc/adapter.git
The delete method fails probably because we moved and split the repository between the organization.

Hi, I have issue registering my package angular-route-ext. Please unregister angular-route-ext and angular-route-ext-test. So I can re-register. I tried to unregister them but could not re-register.

Please change ember-qunit to point to git://github.com/rwjblue/ember-qunit-builds.
Back story:

It was originally located at git://github.com/rpflorence/ember-qunit
Ownership was transferred to @rwjblue and the main project lives at git://github.com/rwjblue/ember-qunit.git
Builds have been removed from that repo, and moved to the rwjblue/ember-qunit-builds repo which is where we need ember-qunit to point...

@Ppjet6 unregistered flex-min :negative_squared_cross_mark:
@rnicholus re-registered file-input to git://github.com/rnicholus/file-input.git :arrow_right:
@dvberkel re-registered cois-sprite-importer to git://github.com/Chair-of-Indefinite-Studies/sprite-importer.git :arrow_right:
@andresusanopinto re-registered adapter.js to git://github.com/webrtc/samples.git :arrow_right:
@c808xx both angular-route-ext and angular-route-ext-test are not registered :white_check_mark:
@rwjblue re-registered ember-qunit to git://github.com/rwjblue/ember-qunit-builds.git :arrow_right:

Thanks much Mr. @desandro!

Please rename git-pu-video to pu-video. Thank you!

Please remove ordercloudsdk at https://github.com/Four51/OrderCloudAngularSDK.git

I am trying to unregister a bower package and I am getting the error that I am not collaborator, but I am.
curl -X DELETE https://bower.herokuapp.com/packages/fieldtrip-extend-record?access_token=3b504a68349fc46f647a8832228c997009eafdd5
Am I missing something? Can you unregister it for me?

I am trying to unregister a bower package called fieldtrip-extend-record. I am a collaborator but when I run the one below
curl -X DELETE https://bower.herokuapp.com/packages/fieldtrip-extend-record?access_token=TOKEN
I am getting the error that I am not a collaborator.
Is it possible to manually unregister it for me?

Please re-register: fast-json-patch git://github.com/Starcounter-Jack/Fast-JSON-Patch.git
to: git://github.com/Starcounter-Jack/JSON-Patch.git

Please re-register inject git@github.com:linkedin/inject-bower.git to git@github.com:linkedin/inject.git.
It was asked for in a previous comment https://github.com/bower/bower/issues/120#issuecomment-31971112, but it hasn't been done. :)

@liccy-fuentes registered pu-video to git://github.com/PracticeUpdate/pu-video.git :white_check_mark:
@panterz unregistered fieldtrip-extend-record :negative_squared_cross_mark:
@warpech re-registered fast-json-patch to git://github.com/Starcounter-Jack/JSON-Patch.git  :arrow_right:
@sorich87   re-registered inject to git://github.com/linkedin/inject.git :arrow_right:

Thank you for using Bower!

Can you please process @seba-1511's request so that the package cloudinary is released and can be registered to @cloudinary's master repository (https://github.com/cloudinary/cloudinary_js)?

[ ] release package cloudinary from git://github.com/seba-1511/cloudinary_js.git
[ ] register package cloudinary to git://github.com/cloudinary/cloudinary_js.git

Thanks!

Hello - I need to get a bower package unregistered from my name / github account, as the repository was moved.
The original repository was derickbailey/backbone.syphon - the package was backbone.syphon
if you could get that package unregistered so that the new team maintaining it can register the new version, that would be awesome.
thanks!

Can you remove the package jquery-loadtemplate? I removed the github repository and need the package to be deleted. Thank you!

timesheet.js git://github.com/CatTail/timesheet.js.git is not available anymore
timesheet git://github.com/semu/timesheet.js.git is not available anymore

Hello, could you please remove model-validator package (https://github.com/dasnervtdoch/model-validator). I will rename my account. Thanks!

@tocker @seba-1511 re-registered cloudinary to git://github.com/cloudinary/cloudinary_js.git :arrow_right:
@derickbailey unregistered backbone.syphon :negative_squared_cross_mark:
@ssc-hrep3 unregistered jquery-loadtemplate :negative_squared_cross_mark:
@sbstjn unregistered timesheet.js :negative_squared_cross_mark: . timesheet is still registered as it's properly pointing to git://github.com/semu/timesheet.js.git
@abedurftig re-registered model-validator to git://github.com/abedurftig/model-validator.git :arrow_right:

No need to say thanks. Thank you for using Bower :bird:

Hi - can you please unregister jwt-decode-test ? Thanks!

Need to remove ember-cli-skycons-shim.  Ended up not needing it.  The curl for me keeps returning a 404 :(

Hello, could you please move the crypto-js package?
From: git:////github.com/Jeff-Mott-OR/CryptoJS.git
To: git://github.com/evanvosberg/crypto-js.git
Since the original repo doesn't exists anymore and my one is already well known on npm people asked for adding bower support. I just did so and added bower support and released a new version.

Hello, could you move notiny package (someone forked it and registered at bower registry)
From: git://github.com/mamboer/notiny
To: git://github.com/andreyviktorov/notiny

As a follow up for comment https://github.com/bower/bower/issues/120#issuecomment-84255071 -- the package is currently registered as crypto-js-evanvosberg. The original crypto-js package cannot be downloaded anymore due to 404 (repo has been deleted).

Hello,
Could you please unregister the polynvd3 package? I recreated it under the poly-nvd3 name.

Hello,
Could you please move ethereum.js
FROM: git://github.com/cubedro/ethereum.js.git
TO: git://github.com/ethereum/ethereum.js.git
Thank you!

Hey everyone. Bower 1.4.0 now includes unregister feature.
If you're still unable to unregister your component, please open an issue on bower/registry.


Add flag to display commands used to run d8 via run-d8.py

Not needed, now that d8 tests have been removed from the repo.


Allow CharField to define a generic enum type

It would be nice to have support for this: https://docs.djangoproject.com/en/3.2/ref/models/fields/#enumeration-types
Maybe something like this:
class SomeEnum(models.TextFields):
    FOO = "foo"
    BAR = "bar"

some_enum_field = models.CharField[SomeEnum](  # note the [SomeEnum] generic here
    choices=SomeEnum.choices,
    default=SomeEnum.FOO,
)

This way the typing would know that the field reads and writes that exact enum type and not a str.

Interesting, playing around with it a bit, and it seems like the Django API makes it a little tricky to do this, ideally we'd pass in SomeEnum as a param, and then we could use that for the @overload.
Something like
class FooModel(models.Model):

    class BarEnum(models.TextChoices):
        FOO = "foo"
        BAR = "bar"

    some_enum_field = models.CharField(
       # something like this would allow for an @overload
        enum=BarEnum,
        default=BarEnum.FOO,
    )

Also by

reads and writes that exact enum type and not a str

Do you mean the set and get would be Literal["foo", "bar"]?

Hey @sbdchd ,
I always defined enums the old way without the models.TextChoices, and after seeing its existence yesterday I theocrafted that it would get and set the enum itself, but I just noticed that even though the  set accepts the enum, the get reads it as a string, which is said :(
Having said that, allowing to declare the CharField that it get/set Literal["foo", "bar"] would be awesome! The same applies to IntegerField with choices defined.


Add Octo issue develop command

Adds issue develop command.
If on issue buffer, the current issue is checked out and the branch is created.
If not, then a picker is shown to select an issue to checkout and create a branch.
NOTE: Only implemented for the telescope picker
Related to #443

Sorry, was not able to take a look. Will do it today

Sorry, was not able to take a look. Will do it today

Thanks @pwntester
Note: I adapted the M.process_varargs which allows for repo as argument. This potentially shouldn't work for develop as the issue numbers from another repo wouldn't make sense

TODO:

[ ] Add command to the README

Any other items?


Convert project.json to csproj

Run dotnet migrate
Add Visual Studio solution file in root folder

@stuartleeks i could've sworn i'd done this in another commit already but i guess i hadn't, or i'd forgotten the csproj file or worse. :) thanks for cleaning up!


feat: add signin and signout feature

Your checklist for this pull request
🚨Please review the guidelines for contributing to this repository.

[x] Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your default branch!
[x] Make sure you are making a pull request against the default branch (left side). Also you should start your branch off default branch.
[x] Check the commit's or even all commits' message styles matches our requested structure.
[x] Check your code additions will fail neither code linting checks nor unit test.
[x] I have added necessary documentation (if appropriate)

Linked Issues
Fixes
Description
Please describe your pull request.
❤Thank you!
Post merge checklist

[ ] Follow steps from the guidelines for contributing to this repository.
[ ] If you are a new contributor, ping in the thread and one of the maintainers will add you to all-contributors list.

@120EE0692
Attach a few screenshots please.
Also, check why the prettier test is failing.


Transfer kernelspy

Target
This extension adds a button in the notebook toolbar, which opens a window that lets you inspect all messages going to/from the kernel.

The code repository: https://github.com/vidartf/jupyterlab-kernelspy
The npm package on npmjs.com (if available): jupyterlab-kernelspy
The python package on pypi.org (if available): jupyterlab-kernelspy
The conda-forge recipe on https://github.com/conda-forge (if available): N/A
The documentation website (if available): N/A

Reason
To increase the bus factor to more than 1.
Checklist

[x] You are owner of the repository you want to transfer
Your role in the transferred repository will be (check one of):

[ ] Maintainer of all jupyterlab-contrib repositories
[ ] Maintainer of the transferred repository only
[x] Either is fine (I'm not usre I understand the difference)

The transferred repository (check one of):

[ ] Must keep the extension name
[ ] Cannot use the original name
[x] I don't care

[x] The code is available under a Open Source license
[x] A license file is present in the code repository
[x] The extension is working on at least JupyterLab 2.x
[x] The repository contains a README file describing:

[x] The provided feature(s)
[x] A picture illustrating the feature(s)
[ ] The available options/settings (if applicable)
[x] How to install the extension
[ ] optional a Binder link to test the feature online

[ ] optional The frontend code (Typescript/Javascript) is tested
[ ] optional The backend code (Python) is tested
[x] I certify the extension is not using exclusively paid-plan of any commercial service
[x] I will help the transfer process on the code repository AND the built packages

Note: Is there a requirement that everyone with commit rights have 2FA enabled?

Thanks @vidartf for the proposal and the comments.
I invite you as owner in this organization so you can transfer the project.
I also invite you on the npmjs jlab-contrib organization. So you should be able to add the jupyterlab-kernelspy to it (https://www.npmjs.com/settings/jlab-contrib/packages).
For pypi.org, could you add me as co-owner of the package, please (my id is fcollonval).

Your role in the transferred repository will be (check one of):    Either is fine (I'm not usre I understand the difference)

The idea is to know your level of contribution: none, only that project or on all projects within the organization.
I will change the wording to clarify it.

Note: Is there a requirement that everyone with commit rights have 2FA enabled?

Yes it is required.

Thanks @vidartf for the proposal and the comments.
I invite you as owner in this organization so you can transfer the project.
I also invite you on the npmjs jlab-contrib organization. So you should be able to add the jupyterlab-kernelspy to it (https://www.npmjs.com/settings/jlab-contrib/packages).
For pypi.org, could you add me as co-owner of the package, please (my id is fcollonval).

Your role in the transferred repository will be (check one of):    Either is fine (I'm not usre I understand the difference)

The idea is to know your level of contribution: none, only that project or on all projects within the organization.
I will change the wording to clarify it.

Note: Is there a requirement that everyone with commit rights have 2FA enabled?

Yes it is required.

I also invite you on the npmjs jlab-contrib organization. So you should be able to add the jupyterlab-kernelspy to it

I have received the invite, but I'm unsure about how to complete this step. Will this put the extension in the @jlab_contrib scope ?

I also invite you on the npmjs jlab-contrib organization. So you should be able to add the jupyterlab-kernelspy to it

I have received the invite, but I'm unsure about how to complete this step. Will this put the extension in the @jlab_contrib scope ?

Thanks for transferring the project and granting me access to the pypi project.

Will this put the extension in the @jlab_contrib scope ?

No, the concept of organization on npmjs is really great (I'ld love to have something similar for pypi...):
https://docs.npmjs.com/about-organization-scopes-and-packages#managing-unscoped-packages
This is really great for this use case, as we can manage access to a team without the burden of renaming the extensions - see for example https://www.npmjs.com/package/@jlab-enhanced/cell-toolbar for which you became collaborators by joining @jlab-contrib.
You should be able to add the package for kernelspy from that page: https://www.npmjs.com/settings/jlab-contrib/teams/team/developers/access

Thanks for transferring the project and granting me access to the pypi project.

Will this put the extension in the @jlab_contrib scope ?

No, the concept of organization on npmjs is really great (I'ld love to have something similar for pypi...):
https://docs.npmjs.com/about-organization-scopes-and-packages#managing-unscoped-packages
This is really great for this use case, as we can manage access to a team without the burden of renaming the extensions - see for example https://www.npmjs.com/package/@jlab-enhanced/cell-toolbar for which you became collaborators by joining @jlab-contrib.
You should be able to add the package for kernelspy from that page: https://www.npmjs.com/settings/jlab-contrib/teams/team/developers/access

I'm sorry, but I don't see any ways to add a package to that organization. Either I'm missing a button, or it is different for owners vs members. Would you mind giving some more detailed instructions (e.g. a screenshot) so we can confirm?

I'm sorry, but I don't see any ways to add a package to that organization. Either I'm missing a button, or it is different for owners vs members. Would you mind giving some more detailed instructions (e.g. a screenshot) so we can confirm?

Either I'm missing a button, or it is different for owners vs members.

You are correct. I think I needed to increase the default status given to new member. You should no be admin.

Would you mind giving some more detailed instructions (e.g. a screenshot) so we can confirm?

That page looks like this for me:

To add an existing package, you need to type its name in the input box next to + Add existing package button. Then click on that button.

Either I'm missing a button, or it is different for owners vs members.

You are correct. I think I needed to increase the default status given to new member. You should no be admin.

Would you mind giving some more detailed instructions (e.g. a screenshot) so we can confirm?

That page looks like this for me:

To add an existing package, you need to type its name in the input box next to + Add existing package button. Then click on that button.

I have added the package to the org now. If there are any other steps to be taken, please let me know, if not, I think we can close the issue :)

I have added the package to the org now. If there are any other steps to be taken, please let me know, if not, I think we can close the issue :)

Thanks @vidartf - one last request - could you please switch the right on the package from read to read/write (on the same page) to all publication by team members? And we should be all set

I'll push a new version following the update to the latest JLab 3 structure to check the rights.

Thanks @vidartf - one last request - could you please switch the right on the package from read to read/write (on the same page) to all publication by team members? And we should be all set

I'll push a new version following the update to the latest JLab 3 structure to check the rights.

I thought I had, but it seems not. Now it should be?

I thought I had, but it seems not. Now it should be?

Awesome thanks @vidartf

Awesome thanks @vidartf


Update user_data_mgmt_test.go

Fix words error.
Signed-off-by: zhuqiuyang<EMAIL_ADDRESS>

Thanks for the contribution! this is a read-only mirror. please read the contribution guide.
Thanks for the contribution! However, this is a read-only mirror. please read the contribution guide. Also, feel free to join us on chat!


feature(docs): add glossary to Falco website

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface

/kind content

/kind translation

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

/area documentation

/area videos

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

Fixes #1006
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: vjjmiras
Once this PR has been reviewed and has the lgtm label, please assign jasondellaluce for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found 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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, vjjmiras
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 [leogr]

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

LGTM label has been added.  Git tree hash: ac2aed7051231be67d4969e69cdb1b05dfba3cca

/hold

New changes are detected. LGTM label has been removed.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: vjjmiras
Once this PR has been reviewed and has the lgtm label, please ask for approval from leogr. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found 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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: vjjmiras
Once this PR has been reviewed and has the lgtm label, please ask for approval from leogr. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found 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

/unhold

LGTM label has been added.  Git tree hash: 2e0ba1d2aab42b9d2eae805821a9369bfec84368

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, vjjmiras
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 [leogr]

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


Cache return old result.

Cache return old result after data was updated.
EHCache3 and OSCache affected.
Fragment from test:
        request().params("page", "1", "page_size", "10", "search", "red eye, june", "n_id", MAGAZINES.desc()).integrateViews().get("search");
        a(responseContent()).shouldContain("Red Eye June 10 2014");

        item.makeInvisible(facility); //execute Model.update(...)
        request().params("page", "1", "page_size", "10", "search", "red eye, june", "n_id", MAGAZINES.desc()).integrateViews().get("search");
        a(responseContent()).shouldNotContain("Red Eye June 10 2014");

This test is working without cache.

@yanchevsky, Can you please localize your example to ActiveJDBC API?

@yanchevsky , I did not deprecate CacheEvent - do not see the value. Everything else is completed.


fix(ui): gallery item action display tablet & mobile width

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #5684
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[x] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary
copilot:summary
copilot:poem

pay 20 usd

😍 Perfect, I’ve sent the payout
💵 $20 @ 30.4 USD/KSM ~ 0.658 $KSM
🧗 EZiu1PjV2j2JHKxY6mHnFwwCRCoV27uHKQSkKXATSh1srJT
🔗 0x5e8a3fbd64be871a5895963f2ee7b01f8eb7930c62bab22ba54b4fc847955413
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


HTML Layout

Hi! In this branch, I worked on the first feature of the app and I made the following changes:

Set up the dev branch on the repository as the default branch.
Set up linters.
Set up Webpack.
Add plain HTML to make the structure of the app.
Add some styles to fit the template.
Fix linters errors.
Add a descriptive readme file.

Thanks for your review @1GORDON! Cheers!


fix(serializer): Allow serializers to operate on derived UrlSearchParams classes

Follow-up to https://github.com/47ng/nuqs/discussions/646, that allows us to use the response from Next's useSearchParams, for example, as the base argument to a serializer. To do this we create a copy of UrlSearchParams in this code branch.

@franky47 thanks for the review! I've just implemented the change and test

Sorry for the close/reopen, my CI workflow wasn't picking subsequent commits from external contributors (should be fixed in #649).


Image not loading

After installing  MVCarouselCollectionView in my project I see blank view. In installation and usage there were no problem. I dug up a tittle by setting breaking point into your collection view data source methods. And I saw collectionView(collectionView: UICollectionView, cellForItemAtIndexPath indexPath: NSIndexPath) -> UICollectionViewCell never called. FYI, collectionView(collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int called and returned image number.
I'm using Xcode 7.3 and swift 2.2, if those may any help to you.
Thanks.

Perhaps, this is the reason. But can you please make support for larger image size? Thanks.

+1


Device activity and triggers need metadata to define supported targets

Not all activities and triggers are supported on every device. For example, a specific sensor trigger for Arduino may not be supported on another platform. The activity/trigger metadata should specify the supported targets.

This has been completed as part of 0.4.0


Error

Sorry to bother you, but after I replaced the dataset and embedding URL, and run "python main.py train configs/squad/r-net/hkust.jsonnet", I got the information below:
File "/home/anaconda3.7/lib/python3.7/sre_parse.py", line 337, in _class_escape
raise source.error('bad escape %s' % escape, len(escape))
re.error: bad escape \p at position 257
Looks like the error is raised by regular expression resolution module. Would you please help to fix it?

Could you paste the complete stack traceback?


Add more tests for grouping and aggregations

I've dusted off the tests from here: #3722
Jira: https://issues.jboss.org/browse/ISPN-5787

merged on 8.1.x, tks @anistor !


Multiple graph suppoort

Initially, we don't support multiple graphs currently for two reasons:

We have not seen TFLite models that contain more than one graph
ONNX doesn't support multiple graphs in its representation

However, there are indeed models that may contain multiple graphs. #51 is an example (though not a good one).
In this case, we can extend our functionality in two steps:

Convert one graph each time tflite2onnx is invoked. An interface to specify which graph to be converted will be added.
Generate all graphs at one time - introduce another level to wrap up Model rather than Graph.

This can be added to our roadmap but doesn't have a very high priority.

Microsoft has re-invented the TensorFlow Lite to ONNX model converter in tf2onnx. They have more chances to enable more functionality.


This is cool. Got a question.

I'm interested in integrating TOR with this project of mine called AnonBoard. The basic goal behind this is a totally anonymous board with integrated javascript AES encryption for posts. Since it's javascript the key is never stored. I want anyone to be able to use this and I'm trying to figure out how to integrate AnonBoard with TOR and the deploy to Heroku button.
Any tips?

First, I think I should definitely point out that this buildpack hasn't been updated to default to the latest TOR, so there's that. That's probably a trivial thing to do though; have it download the latest release instead, and done. It also doesn't try to prove the integrity of the release. I had wanted to add this a while back but never got around to making it work exactly.
Second, keep in mind that Heroku, by default, if you use a dyno type named web, will route traffic to your dyno that escapes the TOR setup. To avoid that, you'll need to avoid the use of the web dyno type and pick hiddenweb or something else.
Third, a Heroku button deploy would have a bootstrapping problem. This is because the TOR buildpack, if not present in the environment, will start TOR without a known private key and .onion name, in which case it will create one but you'll have no way of knowing what it is...
A button compatible way, however, might be to use the compile step, during the build process, to generate the private key and .onion, cache them to CACHE_DIR, and install them at build time. This would require modifications (deletions, really) to the hide script, which currently expects env vars to generate the necessary files.
In order to get the .onion that's being used, then, you'd have to do heroku run cat /app/hidden_service/hostname, which isn't a big deal, really... Bonus points if you supported both methods!
I'm not sure when, if ever, I'll have time to update this buildpack to do all that. If you're interested, I'm happy to accept a PR, but it might be the case that at that point, you just fork and do what you need. I'm happy to point people to your fork, too.

Thanks for the info. This is certainly something I am going to attempt to get working.

Did anything ever become of this? In particular, I'm interested in getting @apg's suggestions about CACHE_DIR properly supported in the buildpack. Right now, my app takes ages to deploy as it's necessary to download and compile Tor every time.
I don't care so much about automating the process of .onion and private key generation (though that would be nice)—I care much more about the turnaround time for each and every deploy after first-time setup.
Note that I'm using @hernanex3's fork of this repo, because it was the most up-to-date and his changes made sense to me. I've looked around the other forks, and I haven't seen anyone implementing caching of the built Tor artifacts. It would be a huge timesaver if someone did this! Thanks.

@cbeams as you wish sir... now tor compilation is cached.

Beautiful, @hernanex3—caching now works like a charm. Thank you so much!


Issue #4302: Fix the short type detection when a question mark (nullable) is prefixing it.

Fix #4302

Here's the patch for branch 2.14: https://gist.github.com/drupol/c43a4ba8712782e104194fdc015c8ede
We need it for our project.

thanks for the patch @drupol .
yet, to not waste your time on it, if you send a PR against older branch, after PR would be merged, that branch would also be merged into higher branch - and usually git does that without conflicts, and for small conflicts I resolve them myself.
it's very rare that for same change we ask original author to prepare independent patches for different branches - it's only in case of super complex change - in other words, don't prepare the "other branch patches" except if you was explicitly asked for ;)

Allrigthty!
Thanks.

Thank you @drupol.

Thanks ! Very needed here :)


Fix bug where Mnemonic type was saved instead of actual mnemonic string

In Wallet.kt, function createWallet(), there was a bug when a class Mnemonic was being saved, instead of an object mnemonic of that class.
Mnemonic.toString(), but it should've been mnemonic.toString()

Hey thanks for your first contribution to the Devkit Wallet, and thanks for finding and fixing a bug! Here are a few things I'd need fixed before I merge:

This doesn't solve the issue yet on my end. It's because the method to get the string out of a Mnemonic object is asString() instead of toString(). You can see the docs here

The error is fixed when using
Repository.saveMnemonic(mnemonic.asString())

I also try to use capitalized commit messages that follow the imperative style.

Cheers! :fire:

Hey, thanks for revision. I made those changes and committed the fixed version.
fixed bug while saving mnemonic


Interpreters for DOS

There are many programming language interpreters for DOS, please add them:

PythonD - Python for DOS
Micropython for FreeDOS
Lua for DOS
perldos - Perl for DOS
DOjS - JavaScript for DOS

Thanks, will do.

Done! :)

Thank you!

New! RustPython for DOS.
https://github.com/coolcoder613eb/RustPython
using w2c2


Modal X button RTL problem

The X Close Button in the Modal does not work when RTL for (Arabic Language) is activated.

Bug reports must include a live demo of the issue. Per our contributing guidelines, please create a reduced test case via CodePen or JS Bin and report back with your link, Bootstrap version, and specific browser and operating system details.

This is a saved reply.

FYI they work like a charm in our current development branch but also in latest release (beta1).
Please give more context and explanations, as requested in our issue template and automated reply.

Closing due to lack of response. You can comment here and we can re-open the issue.
This is an automated reply


[✨] Custom Class serialization

Is your feature request related to a problem?
I'm using MongoDB and every request contains one or more ObjectId class object which can not be serialized.

Describe the solution you'd like
Write a bunch of .maps and .toString()s after every request is too terrible for me.
I'm looking for those solutions

Add a serializer to handle this ObjectId in mongodb (or in bson) just like URL and Date. (ideal but a waste for non-mongodb user)
Implement above serializer as a plugin for runtime. (possible?)
Support custom serialization for any class, such as using toString() or qSerialize() function automatically. (however, using toString() converts any class object into string, which requires additional type transform in useLoader() and useAction, etc.)

Describe alternatives you've considered
n/a
Additional context
No response

https://github.com/bluesky-social/atproto has a class object BlobRef that is also not serializable, would love to write a custom serializer instead of bugging the authors to change it.

Facing a similar challenge with Prisma returning a Decimal object.
Would it be possible to support adding custom serializers via a vite plugin?

Custom serializers is a complex problem, which is why we have been punting it. Perhaps you can send a PR to add the Decimal Object?

I think most custom serializers can be avoided by putting methods in the store that on invocation check if the desired object exists and if not, create it with noSerialize before doing their own work.
The technique I describe here can be useful for that.
I don't think Qwik should ship with support for random libraries but maybe the $sync proposal can help with doing actual custom serialization

We moved this issue to qwik-evolution repo to create a RFC discussion for this.
Here is our Qwik RFC process thanks.