ADDED: [Feature] GitHub Repository Tracking

Fixes

Fixes #99 by @Darylgolden

#99

Description

Added github_scratcher.py to now track the total number of repositories on GitHub that utilize Creative Commons licenses of all types.

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

For the purposes of this DCO, "license" is equivalent to "license or public domain dedication," and "open source license" is equivalent to "open content license or public domain dedication."

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Hey @IamMQaisar, I'm not sure if you missed it but I've already started working on a PR fixing this at #101. Sorry that it's taking a while for me to respond to the reviews as I'm busy with other things IRL.

Hi @Darylgolden ,
Yes Brother, I know that you are working on the PR and this PR of mine was just to fast track and ommit the errors as well as to correlate with the latest changes of the codebase mergers.
You may eventually bring out the changes of files, from my PR to your and make your PR better/complete so that @TimidRobot has ease in merging the PR rather than us being waiting that he mentions all the changes himself and then we make those changes. Which will only delay the process and make the maintainer go through and mention everything manually.

I understand, but I think it would have been better if you had made a PR to my branch if you wanted to help fix the issues in my PR. Creating a competing PR makes the attribution of changes unclear and feels like it creates a sense of competition rather than collaboration, even if that was not your intention. Also, for a open source project, I don't think a 3 day wait from a review is that long to necessitate superseding another PR. I would have understood if it was stale for a week or longer.

Sure, I'll take that into account and make it more collaborative next time by doing a PR into the existing PR of others.

I understand, but I think it would have been better if you had made a PR to my branch if you wanted to help fix the issues in my PR.

I am unable to fork your repo, as it's probably not enabled to be forked. Can you enable it so that I can fork and work on it?

@IamMQaisar I believe it's enabled by default. What problems do you get when you attempt to fork?

Let me show you in a while.


NativeAOT sample doesn't load Blazor app - stuck on "Loading..."

I'm having trouble getting native AOT working with Photino.Blazor.
I thought I was doing something wrong with my app, but then I noticed that the NativeAOT sample has the exact same issue.
When I run the sample (either via Visual Studio, or running the exe after dotnet publish), the Blazor application never loads. Rather, I just get stuck with the "Loading..." div in index.html.
Steps to reproduce:

Clone the repository.
Run the Photino.Blazor.NativeAOT sample.
Observe that the Blazor app never loads, and we're just stuck with the "Loading..." div.

FWIW, everything runs as expected if the PublishAot property is removed from the project file.
Happy to provide any more information required if you can't reproduce this.

We need to investigate the source of  this behavior. Thank you for pointing it out. It's on our backlog.

duplicate of #106 ?

I think it relates to the JSON serialization in IpcCommon as called by IpcReceiver. I'm not currently familiar enough with AOT or WebView to know if this can be worked around or not.

for starter, TryDeserialize needs a JsonTypeInfo<T> parameter that can be taken from [JsonSerializable] annotated class. we also need to know what types are being passed into Serialize and have to annotate those types too.

i guess we just have to wait https://github.com/dotnet/aspnetcore/issues/51598

This issue seems to be more than just NativeAOT.  None of the sample Blazor apps work.  They are all stuck on "Loading..."

This issue has been resolved in Photino.Blazor 3.1.10
https://github.com/tryphotino/photino.Blazor/releases/tag/v3.1.10

i tried running the sample app but keeps failing on publish:
Photino.Blazor.NativeAOT failed with 1 error(s) (4,6s) → bin\Release\net8.0\win-x64\Photino.Blazor.NativeAOT.dll
    C:\Users\x\.nuget\packages\microsoft.dotnet.ilcompiler\8.0.8\build\Microsoft.NETCore.Native.targets(308,5): error MSB3073: The command ""C:\Users\x\.nuget\packages\runtime.win-x64.microsoft.dotnet.ilcompiler\8.0.8\tools\\ilc" @"obj\Release\net8.0\win-x64\native\Photino.Blazor.NativeAOT.ilc.rsp"" exited with code -1073740791.

Build failed with 1 error(s) and 4 warning(s) in 5,8s

the csproj:
<Project Sdk="Microsoft.NET.Sdk.Razor">

	<PropertyGroup>
		<OutputType>WinExe</OutputType>
		<TargetFramework>net8.0</TargetFramework>
		<ApplicationIcon>favicon.ico</ApplicationIcon>
		<PublishAot>true</PublishAot>
	</PropertyGroup>

	<ItemGroup>
		<ProjectReference Include="..\..\Photino.Blazor\Photino.Blazor.csproj" />
	</ItemGroup>

	<ItemGroup Condition="$(RuntimeIdentifier.StartsWith('win'))">
		<RdXmlFile Include="rd.xml" />
		<RdXmlFile Include="Microsoft.AspNetCore.Components.Web.rd.xml" />
		<IlcArg Include="--nometadatablocking" />
	</ItemGroup>

	<ItemGroup>
		<Content Update="wwwroot\**">
			<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
		</Content>
	</ItemGroup>

	<ItemGroup>
	  <None Update="favicon.ico">
	    <CopyToOutputDirectory>Always</CopyToOutputDirectory>
	  </None>
	</ItemGroup>

</Project>

the error seems to be caused by <IlcArg Include="--nometadatablocking" /> but if i remove it the app will stuck on loading.

I am getting the same incorrect behavior. In fact, I cannot create a publish executable at all.


Initial prototype of Blocks UI

Allows drawing of text, circle, rectangle
Additional blocks for translation and changing color
The server was left intact.
An additional folder funblocks-base contains the Blockly logic (with bindings) and code generation,
The web folder contains new items in js and css that were used.
web/blockworld.js is based on a subset of the origin codeworld.js. Both blocks and the regular UI can be made to use the same codebase as codeworld.js with some minor changes.
web/blockly is a git submodule that contains Blockly.
The UI for the blocks currently uses bootstrap. We should probably change this later so that both Uis have the same look and feel.
I don't have a good way of compiling funblocks-base. For now I'm just running ghcjs against is with symlinks in web/js to point to the compiled js files.
The block based UI can be accessed from /blocks.html
P.S. I'm not sure why the scrollbars are suddenly black, they should be gray.

Added copyright
Moved third party files to third_party
The local bootstrap.min.css is a additional theme over bootstrap. I don't think it has a CDN and might want to be modified later.


Any Updates

Hi im waiting for the next Panel update any atm now?

Yes, I haven't had any time yet but I'll probably have time to finish it soon.


catalog still lists the repos which are deleted by delete API call

After deleting the manifest and layers of a repo by calling the delete API, I can still get information of it when I call catalog API. Does this make sense or I missed something?

It's a bug.  I'm closing this as a duplicate of #1585


[ADD ME TO THE ORGANIZATION]

Name: Kalyan Vurugonda
Location: Hyderabad
Interest: DevOps, Java, Open Source, Community Work
Discord username(optional):- Kalyan Vurugonda#0626
Why do you want to join?

I want to join because I'm interested in Open Source and Communities and helping and gaining knowledge from the people.

You can also join our Discord community here
Feel free to check out other cool repositories of the Open Source Community here.

@kalyan-vurugonda Thanks for your interest. Invitations have been sent to you. Kindly accept it and make your membership public.


Multiple browser contexts

Closes #12
Tasks:

[x] Multiple contexts at startup
[x] Create new contexts during a crawl
[x] Example spider
[x] Settings docs
[x] Usage docs
[x] Tests
[x] Deprecate PLAYWRIGHT_CONTEXT_ARGS in favour of PLAYWRIGHT_CONTEXTS

@kalessin could you take a look at this? Thanks!


Added 1-click install button

This is the Solution Builder Version, so currently 2.5.0

Signed-off-by: Sean Smith<EMAIL_ADDRESS>By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

CR-46708887


Shooter

[x] Charge Up - get to speed
[x] Amp eject
[x]  Spin - Down
[x] Slow Spin

@Lucasszela added a persistence check for speed.
Next steps: testing/integration


Create MacOS binaries

Desired Behaviour (for feature requests only)
Hi @Stephen-Gates, the README refers to MacOS binaries in the release, but only a Windows installer is available at the moment.
Thanks!

Hi @jqnatividad, sorry about that.
I'll make sure our next build includes both Windows and Mac binaries.

@mattRedBox thanks for the v0.3.1 release. When you get a chance can you please add the macOS binary to the release?

Hi @Stephen-Gates and @jqnatividad yep - was using this as a chance to start to look at travis integration, but will have mac build this morning.


No error message for user with wrong U/P combo on login page

For the third option on login page, log in with username and password, if the user enters the wrong credentials, the page reloads and not provided with any error message:

@RickieT I'm working on this now as part of #1342

@KennyMonster my mistake, lets close this as a duplicate


fix up cidr IP matching

With the upgrade to NetAddr v2+, that library removed the 'NetAddr::CIDR' class. The ip_address match rules still referenced that class.
However, as I dug down to test that path, I ran across two different places that caused IP rules to not work because of how the objects were being checked.
This approaches all of those things.

🙇  Thanks for merging this!


[framework] support n-m relation in ModelRelationField

Currently it is limited to 1-n relation, but I'd require n-m relation. :)
As discussed with @AdSchellevis on IRC...

details https://github.com/opnsense/core/pull/869#issuecomment-203821570

@AdSchellevis I was wrong in https://github.com/opnsense/core/pull/869#issuecomment-203821570. My test failed because I forgot to add the tag Multiple>Y</Multiple>. D'oh! I did a new test and now it's working as expected.

@fraenki thanks for testing again! this indeed explains all :)


Adding RequestScheme in prometheus service endpoint

12) Errors:
  Request Error: Error: Post "<IP_ADDRESS>/api/v1/query_range?end=2023-02-09T08%3A00%3A00Z&query=avg_over_time%28deployment_match_labels%5B1h%5D%29&start=2023-02-08T08%3A00%3A00Z&step=3600.000": unsupported protocol scheme "", Body:  Query: avg_over_time(deployment_match_labels[1h])
  Parse Error: Prometheus communication error: avg_over_time(deployment_match_labels[1h])
for Query: avg_over_time(deployment_match_labels[1h])

Signed-off-by: Rohit Raut<EMAIL_ADDRESS>

Looks like this was addressed here:
https://github.com/opencost/opencost-helm-chart/pull/12


Bug 1844240: Fix many-to-many occasional e2e errors by adding label_replace() to handle missing volumename

Take 2 (this was accidentally force merged)
We occasionally see errors of the type:
errored: failed to perform Prometheus query: execution: found duplicate series for the match group {namespace=\"metering-validhdfs-reportdynamicinputdata\", persistentvolumeclaim=\"hive-metastore-db-data\"} on the right hand-side of the operation: [{namespace=\"metering-validhdfs-reportdynamicinputdata\", persistentvolumeclaim=\"hive-metastore-db-data\", storageclass=\"gp2\", volumename=\"pvc-e7ea4ab5-b8e6-4f87-8f16-c4cfc84d19db\"}, {namespace=\"metering-validhdfs-reportdynamicinputdata\", persistentvolumeclaim=\"hive-metastore-db-data\", storageclass=\"gp2\"}];many-to-many matching not allowed: matching labels must be unique on one side"

Checking the Prometheus code we see:
ev.errorf("found duplicate series for the match group %s on the %s hand-side of the operation: [%s, %s]"+";many-to-many matching not allowed: matching labels must be unique on one side", matchedLabels.String(), oneSide, rs.Metric.String(), duplSample.Metric.String())

Matching that up with the specific error above we get:
matchedLabels.String: {namespace=\"metering-validhdfs-reportdynamicinputdata\", persistentvolumeclaim=\"hive-metastore-db-data\"}
oneSide: right
rs.Metric:      {namespace=\"metering-validhdfs-reportdynamicinputdata\", persistentvolumeclaim=\"hive-metastore-db-data\", storageclass=\"gp2\", volumename=\"pvc-e7ea4ab5-b8e6-4f87-8f16-c4cfc84d19db\"}
duplSample: {namespace=\"metering-validhdfs-reportdynamicinputdata\", persistentvolumeclaim=\"hive-metastore-db-data\", storageclass=\"gp2\"}

So it appears we lack the volumename sometimes on the right hand side. Adding label_replace with a regex matching on anything, or nothing, should mean that we always have at least an empty volumename label and avoid the error.
This SO was helpful: https://stackoverflow.com/questions/54235797/how-to-rename-label-within-a-metric-in-prometheus
This SO makes sense after you read it about 5 times: https://stackoverflow.com/questions/52625815/prometheus-many-to-many-matching-not-allowed

/retest

/retest

/retest

/retest

/retest

/test all

/test all

Seems like it's working so far, going to keep running a couple more times.

/retest

/lgtm


Draft-to-markdown needs to escape special characters

If I enter _abc_ into the draft editor, I'd expect to see the Markdown editor have \_abc\_.
There are a set of similar test cases:

* sdfsfsf => \* sdfsfsf
abc **sdasda** => abc \*\*sdasda\*\*
4. abc => 4\. abc
(this one doesn't work in Github's inline markdown conversion):

`abc` => \`abc\`

> sdfjsflsdkfj => \> sdfjsflsdkfj
\abc => \\abc

Thank you for pointing this out! Will prioritize a fix.
I've never noticed this because I use a markdown plugin on my draft editor that auto-converts to italics etc when you use those characters, so it never has come up for me 😄

By the way, I just learned the proper way to escape backticks in inline code spans -- it's not backslashes.
Apparently you just need to use different numbers of backticks at the start and end of your code.  So if you have a single backtick literal inside your code span, you need to start/end with at least two backticks.  If you have two backtick literals together inside a code span, you need to use one or three or more backticks to start/end your code span.  If you have single, double and triple backtick literals inside your code span, you need to use at least four backticks to start/end your code span.
Gah!

ahh, opened a pr. I can't even describe to you how messed up I am these days, last night I only got 4 hours of sleep, so... yep. I guess if I were a good nerd I'd just wait to fix bugs until I can actually do it properly but lmk how you feel about this fix. I'll merge it if @ckoppelman and @nicholaswbowen are ok with it as an interim solution?

This is a good interim solution. I hope to make some contributions some time soon, maybe I'll take a crack at this. For my use case, people posting links into the editor is very common. That PR will let me delete code atleast :) . Thanks!

For my use case, people posting links into the editor is very common.

Just out of curiosity - if people are posting [link](http://google.com) ...why on earth are they pasting such specific markdown code and NOT wanting it converted? Like I can see how benign characters like * and _ that could have other meanings shouldn't be converted, but links are so very markdowny

Made a new issue for some of the outstanding work that needs to be done :) https://github.com/Rosey/markdown-draft-js/issues/55 Thank you to both of you!

Oh, I think I misunderstood what you meant. I don't imagine many people would use the link syntax in the draft editor.
What my problem is that I am serializing everything as Markdown, and comments get rendered in remarkable. So if URLs have any of the escaped characters in them, it breaks the link when it gets rendered. What I am suggesting is a regex expression that detects something as simple as https://google.com and doesn't escape any of the characters in that string. Then remarkable will just linkify them and I have happy customers.
I hope that makes sense. Thanks again!


Proposal: set app theme at runtime

Proposal: Support for changing the app's theme at runtime, not just start.
App.Current.RequestedTheme

Rationale
It's currently only possible to update the App's theme in the constructor of App.xaml.cs, a.k.a. when the app starts. Would be great to have to support to do this at all time. I know you can set the requested theme of a piece of UI (like a page) but you still rin into minor issues sometimes (e.g. window buttons need to be updated as well). Setting the App's theme would resolve this.

@kikisaints This seems themeing related

@niels9001 You could check how changing theme at runtime has been implemented in the source code of Xaml-Controls-Gallery

You can achieve a result like this by naming your root mainpage and flipping the requested theme there back'n forth (light-dark-light or dark-light-dark) and that should cascade down into all the controls that are children within that page - however, this method/hack falls apart a bit with menus and flyouts as they are out of that "root" tree and don't receive the requested theme toggle/flip.
I understand niels' ask here, it's something I'd like too, I think it might be a bit more work than it appears on the surface.

Terminal is interested in this as well. We have had to do some pretty unfortunate things to set our app's Theme ☹

Right, you get so far with setting the RequestedTheme on e.g. a page. Stuff like tooltip fall through though. Here's an example of the PowerToys settings:

Both Flyouts do not adhere correctly to the theme.


Stream test output

Stream test output as opposed to waiting until the test suite has finished before writing out
@markhibberd @charleso @michaelneale

:100:

Nice, :neckbeard:


Type definition

Can you add a type Definition {.d.ts file} for Typescript users? In https://github.com/guigrpa/docx-templates/blob/master/packages/docx-templates/src/types.js are allready some types...

Any news regarding this ? I could lend a hand in writing types if needed 😁

I don't use Typescript, but feel free to submit a PR — can it be added only for the external API, without requiring changes in the build process, internal types, etc.?

Cool - I'll have some free time later this week and will get to it.

can it be added only for the external API, without requiring changes in the build process, internal types, etc.?

Sure 😀

@bogdan-calapod this would help me too so just wondering where you're at with these changes.

This would help me as well!
I'm currently using a .d.ts file that I hacked together from the Flow typedefs in types.js, but it is less than ideal.


test: Add unit tests for conserve command and decrease-dynamodb-provisioned-rcu-wcu

Codecov Report

Merging #20 into master will decrease coverage by 65.87%.
The diff coverage is 48.38%.

@@             Coverage Diff              @@
##            master      #20       +/-   ##
============================================
- Coverage   100.00%   34.12%   -65.88%
============================================
  Files            1       10        +9
  Lines           85      838      +753
  Branches        16      174      +158
============================================
+ Hits            85      286      +201
- Misses           0      552      +552

Impacted Files
Coverage Δ

.../tricks/snapshot-remove-elasticache-redis.trick.ts
5.48% <16.66%> (ø)

...rc/tricks/decrease-kinesis-streams-shards.trick.ts
9.18% <20.00%> (ø)

src/tricks/shutdown-ec2-instances.trick.ts
10.34% <25.00%> (ø)

src/tricks/stop-fargate-ecs-services.trick.ts
7.63% <25.00%> (ø)

src/tricks/stop-rds-database-instances.trick.ts
57.69% <83.33%> (ø)

src/commands/conserve.ts
100.00% <100.00%> (ø)

src/tricks/remove-nat-gateways.trick.ts
12.32% <100.00%> (ø)

src/configure-aws.ts
100.00% <0.00%> (ø)

src/tricks/trick-registry.ts
100.00% <0.00%> (ø)

... and 6 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 fcee22a...a6ac462. Read the comment docs.


[Bug-FE][UI Next][V1.0.0-Alpha] Execute the workflow successfully click on the task instance to enter the page is blank

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Execute the workflow successfully click on the task instance to enter the page is blank

Meanwhile, the back-end api service reports an error

What you expected to happen
Access to the task instance can be displayed normally
How to reproduce
1、Execution workflow
2、Click on the task example
Task instance page is empty
Anything else
No response
Version
dev
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I will fix it

This issue is same as #9145


[Request] Record achievement points when querying lodestone

aside from just being a generally nice addition, it'd also be personally helpful to query the api for so I do not have to reinvent playertrack and have a duplicate db just to utilise that bit of data

PlayerTrack no longer does any lodestone querying now that content ids are possible. The lodestone is a bit of a mess so I'd like to avoid future integrations with it.


added feature namespace.toposort

I used kahn's algorithm to implement a topological sort on Namespaces, as was raised in issue #29

Wow. Thanks. I will work on the things you've said.

I will work on everything you pointed out. Thanks @llogiq !

Thanks for working through this, and sorry for any delays on my end.

This is still missing integration in export.rs, otherwise there are but a few small nitpicks left.

I'll work on this

Hopes this works. Let me know if it looks good enough. @llogiq, Thanks for your patience

Don't worry about the bench, this is just my github action failing. But the mysql test seems to be a real problem.

@OLUWAMUYIWA - I deep-dived into why the mysql test is failing... it appears to be a legit failure and the root cause is that pub fn topo_sort(&self) -> Option<Vec<Name>> appears to always return an empty Vec.
If you want to duplicate it locally-

Run a mysql docker:
docker run --name mysql -e MYSQL_ROOT_PASSWORD=mysecretpassword -e MYSQL_DATABASE=test_db -p 3306:3306 -d mysql:latest

cargo run --package synth --bin synth -- generate  hospital_master/ --to mysql://root:mysecretpassword@localhost:3306/test_db  --size 30
with working dir in synth/testing_harness/mysql

Thanks @fretz12 !

It works now, I checked with the mysql test.
I found the following faults in the solution and fixed them:

My tests had been passing because I. So when @fretz12 said namespace.topo_sort returned an empty vec, it was because I had not included every other collection that had no foreign keys(or same_as field, if you prefer). The schema of the mysql test had no such relationship between its collections, and for that reason was a perfect test to reveal that bug.
Before iterating over the collections in export.rs I needed to ensure that the namespace was compiled in the order specified by topo_sort.
What I did earlier with the creation of the adjacency list was naive and incorrect. Though the sorting was good, the
list it sorted was wrong. And it felt right only because the unit tests I wrote were blind to it.

This looks good now. Can you rebase on top of master?

Done! I rebased on master.

I still get an error in graph::test::schema_to_generator (graph/mod.rs line 342). It appears the problem still persists. Are you sure you've pushed your latest revision?

@OLUWAMUYIWA thanks so much for all this work. You may want to run cargo clippy locally as we have a test to ensure that clippy is satisfied before merging code to ensure that our codebase is high quality :)
If you you don't have the time myself or @llogiq can get it done!

I ran cargo clippy and committed the changes it suggested. Thank you @christoshadjiaslanis for appreciating my work.

Just one clippy thiing: In line 414, you don't need a slice. Otherwise this looks good.

Done!
It's quite funny that clippy did not notice that line on my machine. How is that the case?

@OLUWAMUYIWA which version of clippy are you running?

@christoshadjiaslanis , this: clippy 0.1.54 (a178d03 2021-07-26)

@OLUWAMUYIWA thanks again for this! I think we are one clippy warning away from merging.
 error: this expression borrows a reference (`&schema::Name`) that is immediately dereferenced by the compiler
   --> core/src/schema/namespace.rs:412:35
    |
412 |         let c = ns.get_collection(&last).unwrap();
    |                                   ^^^^^ help: change this to: `last`
    |
    = note: `-D clippy::needless-borrow` implied by `-D warnings`
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

Ouch. It looks like this clashed with #61. Sorry this is taking so long and thanks for staying with us. :smile:

The failure of the E2E tests is unrelated this PR and is due to the fact they cargo install without --locked.

Ouch. It looks like this clashed with #61. Sorry this is taking so long and thanks for staying with us. smile
@llogiq,  more thanks is due you. I'm very happy to see this through and contribute more. Thanks

Thank you very much @brokad . It got me worried.

Thank you @OLUWAMUYIWA for having the patience to get this to completion with us.

Wow! Thanks @llogiq, @christoshadjiaslanis , @fretz12 , @brokad @baile320


Fixed potential VM crash when parsing external terms.

This change set adds range checking to the parsing of external term, for example, when loading modules or when converting binaries to terms.  If an external term contains encodings that do not fit within the range of an allocated buffer, then the conversion will halt with an error.  In the case of binary_to_term/1, the function will raise an error:badarg.
Without this change, the VM may crash on a malformed or maliciously crafted binary.  In cases where binaries are exchanged between VM instances (e.g., via TCP), this could result in an attack vector (buffer overflow).
This PR fixed Issue #256
Signed-off-by: Fred Dushin<EMAIL_ADDRESS>These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).
SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

@bettio Would it be possible to review this PR?


Never understood the architecture of the model, can someone explain it to me?

My current understanding is that the text encoder + transformer + Encodec audio codec using a complex codebook interleaving model, the core of which is to convert the text codebook into an audio codebook using the transformer model, and then decode the audio using the Encodec decoder

First of all, musicgen uses encodec for "audio-tokenization", so, we need to train an encodec model. Authors of musicgen paper trained an encodec autoencoder that takes as input 1 second of 32kHz audio, compresses it down to 4 pairs of 50 tokens, these tokens are selected from the dictionary with size of 2048(i believe the dictionary is the same between these 4 pairs of tokens). Then we can use the decoder part of encodec to reconstruct audio back to it's raw waveform, so we get this same 1 second of audio in the end.
This is how 1 second of audio looks like after it was compressed using the encoder part of encodec32kHz model:

4 pairs of 50 tokens, each token has it's ID ranging between 1-2048, colored because it looks nice that way.

Now we just take our music dataset and encode it into these tokens, or we can also say "tokenize" our audio, using this encoded model that we just trained.
Then we train almost-vanilla decoder transformer on these tokens, with the task of "having all of the previous tokens, what is the next one?". If the model is text-conditioned(all released musicgen model's are actually text-conditioned), we also use T5 model to encode the text-description of the song, get the embeddings, and feed them into transformer using cross-attention.
I say almost-vanilla, because we somehow need to also compute these 4 pairs, and vanilla transformer is "1 dimensional" in that sense(i hope my intuition behind this is accurate). Something that they(authors) used here, is called interleaving, and in the paper it was completely skipped about the details of how it works inside the transformer model, so i have no idea of how it does what it does. Anyone who could explain how exactly interleaving works, it would be awesome! The pattern that they used for interleaving the 4 pairs of codebooks is delay. More info on delay interleaving pattern is in the paper.

After training the transformer, we just inference it, and decode the tokens using encodec decoder.

Thank you very much.So the core is the vanilla transformer? I've always found the thesis unclear and haven't understood what this transformer is all about, and the nested dora in the code makes it look like a pain in the arse for me

Yeah, it's pretty much vanilla transformer trained on tokenized audio. You can also look up OpenAI's Jukebox blog post, where they explain how it all works, the only major difference between them is that jukebox generates coarse audio at first, and then upsamples it using other transformer model, while musicgen generates it all in one-go.

Ok,thank you!!!


Does Structor have bad UI ?

I'm not UI/UX specialist. So, it would be extremely helpful to know what is wrong with Structor's UI.

@ipselon - Typically a UX should hide more complex functionality and default to the most common case.
For example, I would look at all the places where there are large groups of buttons for doing something. (i.e. before | first | replace | wrap | last | after) and make it one button to insert a component.
Then maybe an arrow next to the insert button to drop down options of where to put the component. But if the user doesn't pick where the component goes, just put it below the selected component.

As for me, it makes sense. I was afraid that many users don't want to click on the dropdown and then to choose an action. But now there are shortcuts, which simplify all actions. Thank you, probably I'll make it in the next release.

No problem, I think structor is great. I'm planning on teaching a bunch of kids programming and I want them to get a chance to build something substantial and have the opportunity to play around in the code.
With the current state of react tooling, this would be impossible, but if we get structor generating a working website, I think they could have a lot of fun with it.

Thank you for kind words. I really appreciate this. If you have any questions, please write directly in our Facebook group or on Dicsord channel.
BTW, bootstrap-prepack is ready for website creation, there are all what you need - Express.js, Webpack build configs. And after you export project from Structor and build code you'll have working web site. Please watch video tutorials, and just try to follow all steps there.

Thanks a lot for such a precise feedback. I will take in account all items in the list, and I'll answer step by step each issue, because there are some which are not really doable.

why are they not doable? just curious to know :)

@auvipy for example this one:

After generating a component, you can no longer select children. For example if you make List     component, then want to change ListItem from within Structor

Structor can compose components on the page, then can generate the source code of new component taking selected component as the root for new one. But Structor can not change the source code of the component itself.

I don't think that's necessary :)

@psimyn  just released new version, and many proposals were taken in account. If you will have a chance take a look at new UI of Structor.

thanks, looks great! I'll give it a try

So after a 5 minute play session I would describe Structor as a cute toy.  Could be very useful but it has a long way to go.  As much as I HATE pop overs, I would suggest some getting started tips that popup and guide the user through 1st time usage.  Allow them to skip past or return to the tutorial at any time.
Figure out what exactly actions should do, clicking selects if you are in select mode (and setup mode buttons), double click to edit, bla bla bla.  Document it, and then follow it, always, at all times, no matter what.  A single user action should always perform the same action, not change once you have done something else.
A component pallet, for the love of jebus please add some type of component palette.  Or at least have the drop down pre-populate with all components grouped by functionality.  I thought something was broke till I accidentally mashed a key on my keyboard, then realized I had to type something to get something.  I would suggest looking at old Borland IDE's, IMHO they got it right, then everyone else followed their lead (actually I know everyone else followed their lead).
Quick keys and assignable shortcuts.  Maybe some presets, import/export of presets is a gold mine.
Also allow color choices, no matter what colors you pick you will have someone who has issues with them.  Some of us are color blind ;)  Rule of 3 maybe (identify actionable items with at lease 3 distinct visual differences, iconogrophy, text, color, font-weight, size, etc... pick 3 and keep to them)
As an example: red, BOLD, with a trash can icon means delete forever.
Users don't have to start the backend to edit their projects, so why does the documentation allude that you have to?
If you have the backend running then use something like HMR to do auto reloads.  Right now I can't figure out how to get my changes to display in the project without restarting the backend server.
Some ideas on mouse+keyboard quick commands:
Drag and drop from some type of palette with CTRL/Command+Click to place multiple.
Shift+Drag to resize.
Shift+Drag in empty space to pan.
Mouse wheel to zoom.
Ctrl/Command mouse wheel to scroll vert.
Shift mouse wheel to scroll horz.
Everything you can do with a mouse you should be able to do with the keyboard, not necessarily should you be able to do everything (directly) with a mouse that you can do with a keyboard.  Everything should be available via a menu of some type.
  Also for every action performed it added it to the command history so you could easily "learn" how to command your way through anything.
Realistic preview.  Allow the user to specify the style sheets directly that their project uses.  Use those style sheets for the build/preview.
Overall this is a great idea for a project and something I have been looking for for a while now.  Glad to have stumbled upon it and looking forward to where it goes.  Sorry if any of the above is crass, its just my honest opinion.
Over everything else:
<SteveBallmer voice="developers">
Consistency, Consistency, Consistency, Consistency,
Consistency, Consistency, Consistency, Consistency,
Consistency, Consistency, Consistency, Consistency,
Consistency, Consistency, Consistency, Consistency
</SteveBallmer>

@jdarling, @psimyn, it would be great to have a new review of UI after so many releases. If you have a chance take a look at the last version. Also, I wrote some docs... Thanks.

after building some similar tools i am also surprised there isn't any standard functionality as

drag components from a palette
move items around with the mouse
real code editor (dual view)

is this something planned ? I am asking because I'm looking for a new starting project to roll out me new IDE (http://pearls-media.com/control-freak/screenshots/) and I'd love to build on structor but it seems rather difficult to due different design goals.
thanks,
g


cariddi: init at 1.1

Motivation for this change
Crawler for URLs and endpoints
https://github.com/edoardottt/cariddi
Related to #81418
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages 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/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

1 package built:

cariddi


Issue when parsing a transaction response in GO SDK Client

What version are you using?
Stellar Core : V12.4.0
Stellar Horizon: V1.0.0
GO SDK Client: v0.0.0-20200701152956-d12557375a3a

What did you do?
Creating an account from genesis account, running a standalone version of stellar using docker
docker run --rm -it -p 8000:8000 -p 11626:11626 -p 11625:11625 -p 1570:1570 --name stellar ese7en/stellar --standalone

Tried to create an account using the following code:
package main

import (
	"fmt"
	"github.com/stellar/go/clients/horizonclient"
	"github.com/stellar/go/keypair"
	"github.com/stellar/go/txnbuild"
	"wallet_api/common/infrastructure/stellar"
)

const(
	 HORIZON_ENDPOINT = "http://localhost:8000"
	 NETWORK_PASSPHRASE = "Stanford Net; March 2020"
)

func main() {

	mnemonic, kp, err := stellar.GeneratePair("")
	if err != nil {
		fmt.Println("Error getting account details : ", err)
		return
	}

	client := horizonclient.Client{
		HorizonURL: HORIZON_ENDPOINT,
	}

	sourceKey := keypair.Master(NETWORK_PASSPHRASE)
	ar := horizonclient.AccountRequest{AccountID: sourceKey.Address()}
	sourceAccount, err := client.AccountDetail(ar)
	if err != nil {
		fmt.Println("Error getting account details : ", err)
		return
	}

	op := txnbuild.CreateAccount{
	    Destination: kp.Address(),
		Amount:      "100",
	}

	tx, err := txnbuild.NewTransaction(
		txnbuild.TransactionParams{
			SourceAccount:        &sourceAccount,
			IncrementSequenceNum: true,
			Operations:           []txnbuild.Operation{&op},
			BaseFee:              txnbuild.MinBaseFee,
			Timebounds:           txnbuild.NewTimeout(300),
		},
	)

	if err != nil {
		fmt.Println("Error building transaction : ", err)
		return
	}

	tx, err = tx.Sign(NETWORK_PASSPHRASE, sourceKey.(*keypair.Full))
	if err != nil {
		fmt.Println("Error building transaction : ", err)
		return
	}

	tx2, err := client.SubmitTransaction(tx)
	if err != nil {
		fmt.Println("Error building transaction : ", err)
		return
	}

	fmt.Println("Transaction ID: ", tx2.ID)
	fmt.Println("Account: ", tx2.Account)
	fmt.Println("EnvelopeXdr: ", tx2.EnvelopeXdr)
	fmt.Println("Transaction: ", tx2)
}

The transaction has been  entered in the system. I can see it using horizon:
//<PHONE_NUMBER>1415
// http://localhost:8000/transactions/15eede304767370511bfb4948060596cd14f8841c9f9d27292861c5a0b2af2ce

{
  "_links": {
    "self": {
      "href": "http://localhost:8000/transactions/15eede304767370511bfb4948060596cd14f8841c9f9d27292861c5a0b2af2ce"
    },
    "account": {
      "href": "http://localhost:8000/accounts/GB3D7L2YEHKGCIOTGCHPUOZGBBT6VRRSJ6BW6XRQU2GO5FG7KPQHEAG2"
    },
    "ledger": {
      "href": "http://localhost:8000/ledgers/22289"
    },
    "operations": {
      "href": "http://localhost:8000/transactions/15eede304767370511bfb4948060596cd14f8841c9f9d27292861c5a0b2af2ce/operations{?cursor,limit,order}",
      "templated": true
    },
    "effects": {
      "href": "http://localhost:8000/transactions/15eede304767370511bfb4948060596cd14f8841c9f9d27292861c5a0b2af2ce/effects{?cursor,limit,order}",
      "templated": true
    },
    "precedes": {
      "href": "http://localhost:8000/transactions?order=asc&cursor=95730526064640"
    },
    "succeeds": {
      "href": "http://localhost:8000/transactions?order=desc&cursor=95730526064640"
    }
  },
  "id": "15eede304767370511bfb4948060596cd14f8841c9f9d27292861c5a0b2af2ce",
  "paging_token": "95730526064640",
  "successful": true,
  "hash": "15eede304767370511bfb4948060596cd14f8841c9f9d27292861c5a0b2af2ce",
  "ledger": 22289,
  "created_at": "2020-07-01T15:53:16Z",
  "source_account": "GB3D7L2YEHKGCIOTGCHPUOZGBBT6VRRSJ6BW6XRQU2GO5FG7KPQHEAG2",
  "source_account_sequence": "1",
  "fee_charged": 100,
  "max_fee": 100,
  "operation_count": 1,
  "envelope_xdr": "AAAAAHY/r1gh1GEh0zCO+jsmCGfqxjJPg29eMKaM7pTfU+ByAAAAZAAAAAAAAAABAAAAAQAAAAAAAAAAAAAAAF78spEAAAAAAAAAAQAAAAAAAAAAAAAAAGPRudqMPKZI0Wde8j3DrYGsdhwH++aLQaTfeZWe/XOrAAAAADuaygAAAAAAAAAAAd9T4HIAAABAEzmGOC7MVeUfJL2y1kEeXWoU5WGNRNJHXNelQdYxFKzuvilTiriLxoVfSGnk+yoCCsIEfnyRQxDsXqANi308Ag==",
  "result_xdr": "AAAAAAAAAGQAAAAAAAAAAQAAAAAAAAAAAAAAAAAAAAA=",
  "result_meta_xdr": "AAAAAQAAAAIAAAADAABXEQAAAAAAAAAAdj+vWCHUYSHTMI76OyYIZ+rGMk+Db14wpozulN9T4HIN4Lazp2P/nAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAEAAAAAAAAAAAAAAAAAAAAAAAABAABXEQAAAAAAAAAAdj+vWCHUYSHTMI76OyYIZ+rGMk+Db14wpozulN9T4HIN4Lazp2P/nAAAAAAAAAABAAAAAAAAAAAAAAAAAAAAAAEAAAAAAAAAAAAAAAAAAAAAAAABAAAAAwAAAAMAAFcRAAAAAAAAAAB2P69YIdRhIdMwjvo7Jghn6sYyT4NvXjCmjO6U31Pgcg3gtrOnY/+cAAAAAAAAAAEAAAAAAAAAAAAAAAAAAAAAAQAAAAAAAAAAAAAAAAAAAAAAAAEAAFcRAAAAAAAAAAB2P69YIdRhIdMwjvo7Jghn6sYyT4NvXjCmjO6U31Pgcg3gtrNryTWcAAAAAAAAAAEAAAAAAAAAAAAAAAAAAAAAAQAAAAAAAAAAAAAAAAAAAAAAAAAAAFcRAAAAAAAAAABj0bnajDymSNFnXvI9w62BrHYcB/vmi0Gk33mVnv1zqwAAAAA7msoAAABXEQAAAAAAAAAAAAAAAAAAAAAAAAAAAQAAAAAAAAAAAAAAAAAAAA==",
  "fee_meta_xdr": "AAAAAgAAAAMAAAABAAAAAAAAAAB2P69YIdRhIdMwjvo7Jghn6sYyT4NvXjCmjO6U31Pgcg3gtrOnZAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAQAAAAAAAAAAAAAAAAAAAAAAAAEAAFcRAAAAAAAAAAB2P69YIdRhIdMwjvo7Jghn6sYyT4NvXjCmjO6U31Pgcg3gtrOnY/+cAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAQAAAAAAAAAAAAAAAAAAAA==",
  "memo_type": "none",
  "signatures": [
    "EzmGOC7MVeUfJL2y1kEeXWoU5WGNRNJHXNelQdYxFKzuvilTiriLxoVfSGnk+yoCCsIEfnyRQxDsXqANi308Ag=="
  ],
  "valid_after": "1970-01-01T00:00:00Z",
  "valid_before": "2020-07-01T15:58:09Z"
}

What did you expect to see?
A successful response.
What did you see instead?
getting an error :  strconv.ParseInt: parsing "": invalid syntax. After further investigation the error appears on the package horizon, file main.go it is failing when trying to unmarshal the response, mor specifically when trying to decode field "fee_charged".

The type of this field changed in Horizon 1.3.0. You need to upgrade your Horizon version or downgrade the Go SDK.


Drastic changes in predictions during training and validation on the SAME data

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Colab
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version (use command below): 2.4.1
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
I am referring to the official example on knowledge distillation from Keras. During training the student I get expected results. However just after an iteration of training when I use the SAME dataset to validate the model, I get drastically different results.
Describe the expected behavior
The results on the SAME data should be identical.
Standalone code to reproduce the issue
Colab Notebook.

Problem solved!
I suspect the issue was stemming from the usage of tf.keras.applications.mobilenet_v2.preprocess_input. I replaced it like so -
def get_student_model():
    base_model = MobileNetV2(weights=None, include_top=False,
        input_shape=(224, 224, 3))

    inputs = Input(shape=(224, 224, 3))
    x = experimental.preprocessing.Rescaling(1./127.5, offset=-1)(inputs)
    x = base_model(x)
    x = GlobalAveragePooling2D()(x)
    x = Dense(30, activation="softmax")(x)
    classifier = tf.keras.Model(inputs=inputs, outputs=x)

    return classifier


Every upload date shows up in sample activities

When uploading the same sample multiple times every upload date shows up in the sample activities (see screenshot).
Maybe this should be changed to only show the first upload date. Otherwise in cases with many reuploads (honeypots, etc.) this might quickly become ugly.

Keeping track of all upload dates is intended. Though you may be right that displaying all dates in the web UI may be inconvenient. I was thinking about a graph representation or something. If you have a good idea, go ahead :smile:


compile Issues

source/objects/Alphabet.hx:198: characters 20-28 : Warning : flixel.system.FlxSound was moved to flixel.sound.FlxSound
source/states/PlayState.hx:125: characters 21-29 : Warning : flixel.system.FlxSound was moved to flixel.sound.FlxSound
source/states/PlayState.hx:186: characters 17-25 : Warning : flixel.system.FlxSound was moved to flixel.sound.FlxSound
source/substates/PauseSubState.hx:64: characters 17-25 : Warning : flixel.system.FlxSound was moved to flixel.sound.FlxSound
source/states/editors/AnimationDebug.hx:51: characters 11-19 : Warning : flixel.system.FlxSound was moved to flixel.sound.FlxSound
source/states/editors/ChartingState.hx:120: characters 13-21 : Warning : flixel.system.FlxSound was moved to flixel.sound.FlxSound
source/Main.hx:87: characters 59-63 : Float should be Int
source/Main.hx:87: characters 59-63 : ... For optional function argument 'updateFramerate'
source/Caching.hx:127: characters 14-22 : Warning : flixel.system.FlxSound was moved to flixel.sound.FlxSound
source/Caching.hx:128: characters 16-24 : Warning : flixel.system.FlxSound was moved to flixel.sound.FlxSound
source/Caching.hx:133: characters 14-22 : Warning : flixel.system.FlxSound was moved to flixel.sound.FlxSound
source/Caching.hx:134: characters 16-24 : Warning : flixel.system.FlxSound was moved to flixel.sound.FlxSound
C:/HaxeToolkit/haxe/lib/jsonpath/1,0,0/json/util/ArrayUtil.hx:15: characters 35-48 : Class<thx.Arrays> has no field containsExact
C:/HaxeToolkit/haxe/lib/jsonpath/1,0,0/json/util/ArrayUtil.hx:31: characters 25-38 : Class<thx.Arrays> has no field containsExact
C:/HaxeToolkit/haxe/lib/jsonpath/1,0,0/json/util/ArrayUtil.hx:35: characters 25-38 : Class<thx.Arrays> has no field containsExact
C:/HaxeToolkit/haxe/lib/jsonpath/1,0,0/json/util/ArrayUtil.hx:48: characters 35-48 : Class<thx.Arrays> has no field containsExact
C:/HaxeToolkit/haxe/lib/jsonpatch/1,0,0/json/patch/JSONPatch.hx:78: characters 22-34 : json.JSONData has no field insertByPath
C:/HaxeToolkit/haxe/lib/jsonpatch/1,0,0/json/patch/JSONPatch.hx:106: characters 23-35 : json.JSONData has no field existsByPath
C:/HaxeToolkit/haxe/lib/jsonpatch/1,0,0/json/patch/JSONPatch.hx:109: characters 18-30 : json.JSONData has no field removeByPath
C:/HaxeToolkit/haxe/lib/jsonpatch/1,0,0/json/patch/JSONPatch.hx:126: characters 18-27 : json.JSONData has no field setByPath (Suggestion: getByPath)
C:/HaxeToolkit/haxe/lib/jsonpatch/1,0,0/json/patch/JSONPatch.hx:144: characters 23-35 : json.JSONData has no field existsByPath
C:/HaxeToolkit/haxe/lib/jsonpatch/1,0,0/json/patch/JSONPatch.hx:149: characters 18-30 : json.JSONData has no field removeByPath
C:/HaxeToolkit/haxe/lib/jsonpatch/1,0,0/json/patch/JSONPatch.hx:151: characters 22-31 : json.JSONData has no field setByPath (Suggestion: getByPath)
C:/HaxeToolkit/haxe/lib/jsonpatch/1,0,0/json/patch/JSONPatch.hx:169: characters 23-35 : json.JSONData has no field existsByPath
C:/HaxeToolkit/haxe/lib/jsonpatch/1,0,0/json/patch/JSONPatch.hx:175: characters 22-31 : json.JSONData has no field setByPath (Suggestion: getByPath)
C:/HaxeToolkit/haxe/lib/jsonpatch/1,0,0/json/patch/JSONPatch.hx:195: characters 27-39 : json.JSONData has no field existsByPath
C:/HaxeToolkit/haxe/lib/polymod/git/polymod/backends/LimeBackend.hx:146: characters 54-55 : Unexpected ?

try updating your haxe libraries. if that doesn't work then let me know and i can see what i can do about it!!


feat(css/parser): normalize and improve function name

Description:
Improve function name:

Normalize name (lowercase)
Use dashed ident (we use the same for at-rules/declarations)

From spec (https://www.w3.org/TR/css-values-4/#dashed-idents):

CSS will use  more in the future, as more author-controlled syntax is added. CSS authoring tools, such as preprocessors that turn custom syntax into standard CSS, should use  as well, to avoid clashing with future CSS design.
For example, if a CSS preprocessor added a new "custom" at-rule, it shouldn’t spell it @custom, as this would clash with a future official @custom rule added by CSS. Instead, it should use @--custom, which is guaranteed to never clash with anything defined by CSS.

use js_word!()

BREAKING CHANGE:
Yes
Related issue (if exists):
No

Rebased

@kdy1 Rebased :+1:


added missing dependency - redux-simple-router

I notice that was missing, now it runs ok. I added as dev dependency btw.

The redux-simple-router is missing. I have notice that this package has been deprecated in favour of react-router-redux: https://github.com/reactjs/react-router-redux/issues/80

Hello, nice catch! I have a lot of work now and can't develop boilerplate. I removed react-router-redux because don't know how to work with him new api and will be glad to any help)


Mat-Dialog: Possible memory leak when dialog includes textarea, input, etc.

Reproduction
Use StackBlitz to reproduce your issue: https://stackblitz.com/edit/components-issue-yp5k44?file=src/app/dialog-overview-example-dialog.html
Steps to reproduce:

Run application, click on open dialog button
Enter text and close dialog, repeat this step.
  View MyDialogComponent and there are more than one instance created.
Refresh chrome tab
Open dialog and close without entering into text area
Go to Chrome Memory tab and run head snapshot.  View MyDialogComponent and there is only one instance created.

Expected Behavior
What behavior were you expecting to see?
I could be reading the chrome head snapshot incorrectly but I would expect there to be 1 instance of the Dialog component (MyDialogComponent) regardless if I enter something into the text-area or not.
Actual Behavior
What behavior did you actually see?
The number of instances grows by 2 each time the dialog is opened and text is entered into the text-area.
Environment

Angular: 10.1.0
CDK/Material: 10.2.1
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): Windows

@tp1906 I tried this and found that, no matter what I did (entering text into the text areas or not), I couldn't reproduce this; there was only ever one instance of the dialog in the heap snapshot.
Do you have any other information for reproduction?

Can you send me a screenshot of your heap snapshot?  I always see +2 or +3 after the mat-dialog. Do you think this could be an issue with chrome?
Sent from my iPhone

On Sep 17, 2020, at 5:45 PM, Jeremy Elbourn<EMAIL_ADDRESS>wrote:

@tp1906 I tried this and found that, no matter what I did (entering text into the text areas or not), I couldn't reproduce this; there was only ever one instance of the dialog in the heap snapshot.
Do you have any other information for reproduction?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

My apologies...I gave you the incorrect stackblitz in the original ticket.  Here is the correct one:  https://stackblitz.com/edit/angular-8fup8q?.  Though I want to point out that what I see in the heap snapshot while running in stackblitz is different when I run this via http://localhost:4200 which is compiled using vsCode.  Do you know why I see different heap snapshots when using these two methods?
What I see when running http://localhost:4200:

What I see when running via stackblitz:

I was able to reproduce this when I downloaded the project from stackblitz.
Without typing in the input I could already trigger some dialog components to hang / not get garbage collected. When typing in the input, it seemed like all of the mat-dialog components were not hanging / getting garbage collected.

Do you know why I see different heap snapshots when using these two methods?

I guess the reason is that Ivy is turned off on your stackblitz example. You can enable it in the settings:

When you download the code, your local angular-cli is used to build the project - and when you have a new ng version, Ivy may be enabled by default.


Cleanup

Whitespace: remove leading newlines; replace lines containing only whitespace with empty lines; replace multiple trailing newlines with a single newline; remove trailing whitespace in lines.
Images: Compress PNG files with zopflipng, JPG files with mozjpeg jpegtran, and GIF files with gifsicle, all losslessly.
This PR was created semiautomatically.

As it happens, all of the changed files are tests, so it's possible that this is not appropriate.

We shouldn't clean up the test sources, since one of the things we're testing is whether rustfmt cleans up for us appropriately.

That makes sense. Closing.


Merging to Main branch

PR Description

Video of the project
https://www.loom.com/share/35baef6fa62b418492fb3d8bab31f41f?sid=414e7dbd-29ad-4b9c-8c40-3e5b9794b2b6

Live Demo link of the project
https://sidds-budget-app.onrender.com/

Splash screen
A simple page with the name of your app (yes, you need to choose one), and links to the sign up and log in pages.

Sign up and log in pages
The user should be able to register in the app with full name, email and password (all mandatory).
The user can log into the app using email and password.
If the user is not logged in, they can't access pages that require the user to be logged in (all the pages described below).

Home page (categories page)
When the user logs in, they are presented with the categories page.
For each category, the user can see their name, icon and the total amount of all the transactions that belongs to that
category.
When the user clicks (or taps) on a category item, the application navigates to the transactions page for that category.
There is a button "add a new category" at the bottom that brings the user to the page to create a new category.

Transactions page
For a given category, the list of transactions is presented, ordered by the most recent.
At the top of the page the user could see the total amount for the category (sum of all of the amounts of the transactions i.
in that category).
There is a button "add a new transaction" at the bottom that brings the user to the page to create a new transaction.
When the user clicks on the "Back" button (<), the user navigates to the home page.

"Add a new category" page
The user fills out a form to create a new category, indicating their name and icon (both mandatory).
The user clicks (or taps) the "Save" button to create the new category, and is taken to the home page on success.
When the user clicks on the "Back" button (<), the user navigates to the home page.

"Add a new transaction" page
The user fills out a form to create a new transaction with:
name (mandatory)
amount (mandatory)
categories (mandatory at least one)
The user click (or taps) the "Save" button to create the new transaction, and is taken to the transactions page for that
category.
When the user clicks on the "Back" button (<), the user navigates to the transactions page for that category.

Please give your review

According to your requirement i have changed the user model and put on the validations
class User < ApplicationRecord
Include default devise modules. Others available are:
:confirmable, :lockable, :timeoutable, :trackable and :omniauthable
devise :database_authenticatable, :registerable,
:recoverable, :rememberable, :validatable
validates :name, presence: true
validates :email, presence: true, uniqueness: true, format: { with: URI::MailTo::EMAIL_REGEXP }
validates :password, presence: true, length: { minimum: 6 }
validates :password_confirmation, presence: true
has_many :groups
end


http method class does not match when using x-displayName extension in Korean with sortTagsAlphabetically=true

Describe the bug
adding korean to the x-displayName extension and setting the sortTagsAlphabetically option, the correct http method class is not reflected in the sorted results.
To Reproduce
Steps to reproduce the behavior:
{
  "tags": [
    {
      "name": "User",
      "x-displayName": "사용자"
    },
    {
      "name": "DGroup",
      "x-displayName": "D그룹"
    },
  ],
 "paths": {
    "/users": {
      "get": {
        "tags": [
          "User"
        ],
      },
    },
    "/users": {
      "post": {
        "tags": [
          "User"
        ],
      },
    },
    "/groups": {
      "delete": {
        "tags": [
          "DGroup"
        ],
      },
    },
    "/groups": {
      "put": {
        "tags": [
          "DGroup"
        ],
      },
    }
  }
}

build-docs result
get /users --> delete method class
post /users --> put method class
delete /groups --> get method class
put /groups --> post method class
When the sortTagsAlphabetically option is given, it is initially sorted based on the Tag name and then re-sorted based on x-displayName. At this time, the class name initially given to the method class is fixed based on the Tag name, and when reordered by x-displayName, the color will not match the method name as the class name will remain in the original location.
Expected behavior
even if reordering occurs, a class that matches the method name must be assigned.
OpenAPI description
3.1
Redocly Version(s)
1.2.0

Hi @Ellenwief,
Are you sure your example is valid? The paths section contains duplicated keys.

I'm transferring the issue to Redoc project.


Update and publish Calculator app to Store

This issue tracks the progress of taking an RNW app from test to complete app release to the store. In this case the suggested app to take to completion is the calculator app.
This app could also serve as a validation app for future releases.

It'd be great for the samples to be on the Store or available as an MSIX! :D
hey do you know why Skype switched to Electron (but still using ReactXP) and why the Xbox app is no longer in the MS Store and isn't using MSIX (at least for the installer)?

oh and a list of who is using RNW would be pretty neat. you can see who uses RN on their site. unfortunately not even ms is using rnw much as i said above.

oh and a list of who is using RNW would be pretty neat. you can see who uses RN on their site. unfortunately not even ms is using rnw much as i said above.

#278


threadInTaskInit

I think we need those changes to the submodules in order for the code to work

I think we need those changes to the submodules in order for the code to work

Yeah, you're right


[hip][cuda] Merge the tracing implementations.

These were entirely copy-pasted off one another and it's not likely they will have to diverge in the future.

I currently have 2 different interfaces into the shared code
iree_hal_deferred_work_queue_device_interface_t and iree_hal_tracing_device_interface_t. They are similar, and share some similar methods, but I left them as individual interfaces as they are nominally independent. But it means a bit of duplication of code. I was wondering if anyone else had an opinion on this.,


"bin/smart" is not assembled

It's because only "*.sh" is included
        <fileSet>
            <directory>${basedir}/../bin</directory>
            <outputDirectory>smart-data-${project.version}/bin</outputDirectory>
            <includes>
                <include>*.sh</include>
            </includes>
        </fileSet>

By the way, it would be better to show script running errors somewhere, which is now written to /dev/null.

@manuzhang error logs issue will be fixed later, thanks!


Getting "command not found" errors

I am running MacBook Pro late 2011 with OS 10.13.6 and OS 10.15.6 and Windows 7 Pro. Also have dosdude's GPU patch running. I am also using zsh shell in Terminal and Login. Have Terminal also emulating .sh shell. The file create_macos_recovery is also on desktop. Here is my Terminal window from session which I did on desktop:

Anyone just pipe in with suggestions because I am a little new to this Github thing. I also have factory installed 256GB ssd and all volumes and container are APFS and GUID. All drives routinely run thru Disk Utility to detect any problems and Terminal show my OS 10.13.6 Recovery Partition and I can boot into recovery when OS High Sierra is startup Drive. Now I just need OS Catalina to have its own Recovery Partition and everything will be golden.

Try this command:
sudo /Users/jamesmick/Desktop/create_macos_recovery.sh "/Applications/Install macOS Catalina.app"

Thanks for your reply rtrouton, I have already tried that and received Terminal output dquote>. I did some checking and dquote> refers to double quotations and the response would to enter another double quote argument or use single quotes instead or enter “ with keystroke. I will give results of Terminal output using single quotes later.

Hi rtrouton, here is Terminal output from you suggestion

I used single quotes instead of double and did not receive the >dquote from Terminal.

Success!!!! Here is my Terminal output after running rtrouton's/create_macos_recovery. I used Xcode to get this shell doc.
create_macos_recovery.sh.pdf
Terminal Saved Output.pdf

I'm glad to see you figured it out. Closing this issue.

How did you get it to work @oldcar37 ? I'm running to the same issue. Running MacOS Mojave 10.14.6 (with no other OSs installed.

I've tried all the inputs you've mentioned above and when using sudo without, with " and with ' I get a »command not found« error.
Best,
✺ Sune Amstrup
Visual Artist and photographer
↝ View artworks | suneamstrup.dk
On 24 Apr 2023, 16.18 +0200, oldcar37 @.***>, wrote:

How did you get it to work @oldcar37 ? I'm running to the same issue. Running MacOS Mojave 10.14.6 (with no other OSs installed.
Would a copy of input in Terminal help you out, as you have not mentioned weather you tried or not.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

I just saw in another post something about there could being a problem with HFS+ volumes - however when I searched for more info, people said that running Mojave on HFS+ was not a problem.
My volume is HFS+. Would that make a difference?
(I've updated from High Sierra to Mojave (some years ago) and after losing my computer some years ago, I copied my install from my backup made with SuperDuper. I don't know if that is the reason it's HFS+ and not APFS as I read the Mojave updater should change the volume to APFS?)
Best,
✺ Sune Amstrup
Visuel kunstner og fotograf
↝ Se værker på min hjemmeside | suneamstrup.dk
On 25 Apr 2023, 09.07 +0200, Sune Amstrup @.***>, wrote:

I've tried all the inputs you've mentioned above and when using sudo without, with " and with ' I get a »command not found« error.
Best,
✺ Sune Amstrup
Visual Artist and photographer
↝ View artworks | suneamstrup.dk
On 24 Apr 2023, 16.18 +0200, oldcar37 @.***>, wrote:

How did you get it to work @oldcar37 ? I'm running to the same issue. Running MacOS Mojave 10.14.6 (with no other OSs installed.
Would a copy of input in Terminal help you out, as you have not mentioned weather you tried or not.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


Update select discussion of table / namespace

I wasn't quite sure I understood the explanation. Here's my attempt to explain it. Thanks.

Thanks @richb-hanover — unfortunately it seems like we might have done the same thing! Our explanations are fairly similar!
I did make an effort to tag the original issue in the PR, sorry if that wasn't visible enough.
Feel free to edit my explanation with anything you think can be improved!

Hmmm... I was responding to the description that (I think) is still in the Book
As I initially read it, I didn't quite understand what was being said, and the examples didn't help me. I am experimenting to make the language in this PR more explicit about the thing that isn't available. (I will be glad for corrections if I've got it wrong.)

I also realize I don't entirely understand how to connect the example PRQL code and the resulting SQL...

Did you see the PR at https://github.com/prql/prql/pull/1285? It was not merged when you wrote this PR (but was opened 6 hours earlier...).
It's now merged, so that means we have some conflicts with your branch:

No - I hadn't seen that. Let me read it over... Thanks.

Very open to changes — do you want to merge the current version into yours, and then we can see the diff?

Abandoned - merging (new) main branch into mine...


Update CHANGELOG.md

Short description of what this resolves:
When updating only ionic-angular to ^2.0.0-beta.8 npm throws: └── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>Changes proposed in this pull request:
Ionic Version: 2.0.0-beta.8
Fixes:  Manually update rxjs to ^5.0.0-beta.7

This change is

Our starters are using<EMAIL_ADDRESS>which requires rxjs version 5.0.0-beta.6, so this update will actually not work:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

it should be set to
"rxjs": "5.0.0-beta.6",

Thank you for the PR, though! Let me know if I'm missing something and I'll reopen this.


Make Dashboard the welcome page.

Downstream dependency: SlipStream/SlipStreamUI#423.

@slipstream/owners
We've to agree on the exact implementation of this re-routing.
In order to preserve current links to /dashboard (and other reasons) I'd suggest to redirect / to /dashboard (instead of routing it to /) and add two new routes for /appstore and /projects. Each of these new routes would display the corresponding part of the current Welcome page.
Thoughts?

In this case the user may also choose which page he wants as the default.

:+1:
That was indeed included within the other reasons mentioned above. :wink:
Maybe not for this iteration, but definitely a good idea to leave the option open for it.


Add vendored feature to set native-tls/vendored

Proposal, because I need openssl-sys (so native-tls) to have the vendored feature
With this feature, we can allow cross-compilation for rust-imap (tested with android targets), and allow to link statically with env OPENSSL_STATIC=1
Note that this feature will not be set by default, so it's not a breaking change.
(I named this feature vendored because native-tls did the same thing to enable openssl-sys vendored feature)
Thank you!

This change is

Also note that you need perl and libssl-dev (or openssl-devel on fedora) to build openssl-sys/vendored

I'm not sure why you need to add this as a feature of imap? You could instead just add native-tls as a direct dependency of your crate, and then enable the feature there. Since cargo shares features across all instances of a crate in the dependency graph, if you enable the feature once in your crate, that will also apply when it's compiled as part of imap.

Well, you're right, I simply forgot about that... 😓
Thank you


Updates for Calico v3.17.1

Description

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit 2c72e4c4b214d77e56944e667ecd3e53e6736b7b
https://deploy-preview-4252--calico.netlify.app


Recommended videos title not showing up when player resizing is disabled and on a large screen

The titles of the recommended videos on the right of the player don't show up when player resizing is disabled, when in the Default View and on a large screen (e.g. 2560x1440).
Player Resizing disabled:

Player Resizing enabled

Browser: Chrome 40
YTC version: Dev build 480

Same issue as #1463


Rules

Proposed changes:

add RulePolicy
add support for reading rules training data
move FormAction from SDK to Rasa Open Source
deprecate old "rule-like" policies (MappingPolicy, TwoStageFallbackPolicy, FallbackPolicy, FormPolicy)
implement TwoStageFallbackPolicy as action
add NLU component FallbackClassifier which predicts an intent nlu_fallback in case the confidence of the other intents is below a given threshold. This allows to write stories which handle low nlu confidence
new featurization for Forms. Happy paths are now represented as a single event.

Follow up issue: https://github.com/RasaHQ/rasa/issues/6176
Status (please check what you already did):

[x] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[x] reformat files using black (please check Readme for instructions)

@Ghostvv Tanja's new feedback is implemented.
Are we ready to merge? :drum_roll: 🤩

Tanja have to give us green flag 😇. I can approve, but it'll be cheating PR review😜

You care about cheating at this point? 😄 Let me quickly create another GitHub account ... 🤣


thread context for !lc

cool updates in recent time🌟
https://github.com/hibobmaster/matrix_chatgpt_bot/releases/tag/v1.7.2
in this release you mentioned new commands.
can we use !lc because our app by default uses thread response for every message sent in room.
will thread context also work for !lc (flowise)?

Flowise manages its own conversation context independently of this project. This means that whether you use room-based or thread-based !lc commands, they will function identically and utilize the same conversation history.
Furthermore, the !lc command is rarely used and may be deprecated in future releases.


layer order in functional API graph models

Do the layers in a graph model that is defined with the functional API have a specific order in the model.
For example, when I have an Add layer that adds the outputs of two other layers, how is it decided which of the two layers comes first in the model?
I am wondering since in most cases the order seems to be the same as the order the layers are given, but not always.
This can be confusing, for example, when I tried to get the weights of the model, the order of them was not as expected in rare cases.
The following code illustrates the issue. I expected to find the weights of the Conv2D before the DepthwiseConv2D when I perform  model.get_weights().
import tensorflow as tf
from tensorflow import keras
from tensorflow.keras import layers
import numpy as np
in0Con6326 = tf.keras.layers.Input(shape=([2, 2, 2]))
in0Dep66009 = tf.keras.layers.Input(shape=([2, 2, 2]))
in0Con7723 = tf.keras.layers.Input(shape=([1, 1, 1]))
Con6326 = keras.layers.Conv2D(3, (2, 2),strides=(1, 1), padding='valid' )(in0Con6326)
Dep66009 = keras.layers.DepthwiseConv2D((2, 1),strides=(2, 2), padding='valid',)(in0Dep66009)
Con7723 = keras.layers.Concatenate(axis=3, )([Dep66009,in0Con7723])
Add80107 = keras.layers.Add()([Con6326,Con7723])
model = tf.keras.models.Model(inputs=[in0Con6326,in0Dep66009,in0Con7723], outputs=Add80107)
print(model.get_weights())
print(model.summary())

@rschumi0 ,
We see that the issue template has not been filled, could you please do so as it helps us analyse the issue and also please let us know the tensorflow version you are using?
Thanks!

@tilakrayal
I was not sure which issue template would be the best, so I used the one for miscellaneous issues, which doesn't have a specific structure.
I was running the example with the following System details:
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 20.04
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v2.4.0-rc4-71-g582c8d236cb 2.4.0
Python version: 3.8.5

@sachinprasadhs ,
I was able to reproduce the code in tf v2.4,v2.5 and nightly.Please find the gist here.

Since deep learning model is usually a  directed acyclic graphs(DAG) of layers and functional API is the way to do that way.
You can represent your layers in a graphical representation to understand the flow like below.

If you want to get a layer weight then you can get the layer by it's name or you can iterate over the model layers and get the weights for the layer.
Please find the gist here for more details. Thanks!

@sachinprasadhs Thank You for your answer. I am aware of this visualization method and also that it is possible to get the weights from an individual layer.
However, it would still be interesting to know if there is a specific order in which the DAG is processed?
In particular, in the example why is the DepthwiseConv2D before the Conv2D when the weights of the whole model are obtained?
Are deeper branches processed first or what could be the reason for this order?

The reason is Tensorflow tries to optimize using shortest path finding in the layers arranged in a Topological order of DAG network.
You can see the detailed explanation of DAG using mathematical concepts here.

@sachinprasadhs I see, thanks.
Could you please refer me to the source code where the DAG is processed or ordered?

You may want to to refer https://github.com/tensorflow/tensorflow/blob/a4dfb8d1a71385bd6d122e4f27f86dcebb96712d/tensorflow/python/keras/utils/vis_utils.py#L67


Add SDK Tag

Hey, please add a tag starting with 'sdk', so the swift package can stop growing in size:
https://github.com/KeystoneHQ/keystone-sdk-ios/issues/18
This will cause a release, and as the other tags such as 0.0.12 do not have a release, one might need to be added for that too.

Thank you!


secret_environment_variables as list of key-value pairs

TL;DR
On README, it says that the optional parameter secret_environment_variables is a list of key-value pairs, but the example shows only a single key-value pair. What should I do if I have more than one secret that I want to associate with my Function?
I tried creating a list like:
secret_environment_variables:
  - 'PG_PASSWORD=projects/xxxxxxxxxx/secrets/PG_PASSWORD'
  - 'PG_HOST=projects/xxxxxxxxxx/secrets/PG_HOST'

But it gives an error:
The workflow is not valid. .github/workflows/main.yml (Line: 42, Col: 13): A sequence was not expected
Expected behavior
When a list of key-value pairs is passed in the YAML file, it should associate all the secrets present in the list to the Cloud Function.
Observed behavior
I didn't find a way to pass secrets as list.
Action YAML
name: CI

on:
  push:
    branches: [ main ]
  pull_request:
    branches: [ main ]
  workflow_dispatch:

jobs:
  build:
    runs-on: ubuntu-latest

    permissions:
      contents: 'read'
      id-token: 'write'

    steps:
      - uses: 'actions/checkout@v2'

      - name: 'Authenticate in Google Cloud'
        uses: 'google-github-actions/auth@v0'
        with:
          workload_identity_provider: ''
          service_account: ''
          project_id: ''

      - name: 'Set up Cloud SDK'
        uses: 'google-github-actions/setup-gcloud@v0'

      - id: 'deploy'
        name: 'Deploy Cloud Function'
        uses: 'google-github-actions/deploy-cloud-functions@v0'
        with:
          name: ''
          runtime: 'python39'
          entry_point: 'main'
          memory_mb: '2048'
          description: ''
          vpc_connector: ''
          secret_environment_variables:
            - ''
            - ''
            - ''
            - ''
            - ''
            - ''

Log output
The workflow is not valid. .github/workflows/main.yml (Line: 42, Col: 13): A sequence was not expected

Additional information
First time writing an issue, sorry if I missed something.

Hi @EwertonES
This is a limitation of GitHub Actions. The with block only supports key: value pairs, so you cannot have nested structures like that. However, you can create a multi-line string to make it easier to read:
secret_environment_variables: |-
  PG_PASSWORD=projects/xxxxxxxxxx/secrets/PG_PASSWORD
  PG_HOST=projects/xxxxxxxxxx/secrets/PG_HOST

Let me know if you have additional questions, and sorry there's not much we can do about this in our action directly. You can also file feedback with GitHub by filing a support ticket.

Just a heads up - based on my testing, you'll need to comma-delimit your variables or you'll run into errors. So something like this should work:
secret_environment_variables: |-
  PG_PASSWORD=projects/xxxxxxxxxx/secrets/PG_PASSWORD,
  PG_HOST=projects/xxxxxxxxxx/secrets/PG_HOST

Hi @mattdsteele you shouldn't need to have a comma at the end. If you're experiencing other behavior, please open an issue and complete the issue template so we can debug.


about dictionary update？

hi：
thanks for sharing thie project.
i want to ask how to update seed dictionary? in the code i have not see any update,but in your paper  it maybe updated by false positive words?
so can you tell me how you update the seed dictionary?

thank you for your comments.
In my paper, i did not mention about update seed dictionary (as far as i remember).
it means that i executed CRF only once in my experiments.
I'm so sorry, this project is too old to remember the detail of experiments. if you can point out the corresponding explanation in my paper, please show me. i may be able to answer more clearly.

in the http://cl.sd.tmu.ac.jp/~miyazaki/publish/nlp2015_miyazaki_poster.pdf,
i am not familiar with japannese,in the picture i guess the seed dict is updated with false postive words,
so can you explain what's the meaning of this picture?

Sorry for the late reply.
Thank you for pointing it out.
That's true. The seed dict is updated with false positive words.
However, the scripts of this repository are corresponding to below paper.
http://cl.sd.tmu.ac.jp/~miyazaki/publish/nlp2015_miyazaki_paper.pdf
In this paper, the seed dict is not updated.
if you want to conduct experiment again from "nlp2015_miyazaki_poster.pdf", please implement seed updating phase.
it is not complicated process. it can be done by simply adding the words predicted as false positive at the previous step.
Pseudocode:
for i in iterate_num:
  traning_set = make_training_set(corpus, dict)
  model = train(training_set)
  result = predict(model, another_corpus)
  fp_words = get_false_positive(result)
  dict += fp_words


use fontSize There will be one more line#

Describe the bug
use fontSize There will be one more line#
To Reproduce
Steps to reproduce the behavior:

apt install supervisor
supervisor command /root/ttyd.x86_64 -p 7681 -t "disableReconnect=true" -t "disableLeaveAlert=true" -t "fontSize=16" zsh
request <IP_ADDRESS>:7681
There will be one more line#

Expected behavior
Delete the # of the first line
Screenshots

Environment:

OS: Linux docker 5.10.0-23-amd64 #1 SMP Debian 5.10.179-1 (2023-05-12) x86_64 GNU/Linux
Browser: [e.g. Chrome 13.0.5672.126（正式版本） (arm64)]

Can not reproduce.
All the lines come from tty output, check your shell configuration.


Update expressroute-troubleshooting-expressroute-overview.md

Removed the ARP section and pointed to the ARP document for consolidation and de-duplication of information.

@jboeshart,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@jboeshart : Thanks for your contribution to the Azure documentation! The author, @rambk, has been notified to review your proposed change.

Also proposing an update to the acronyms that are used in this document to line up with the image that shows how the connectivity works. Currently, the document references MSEE-PRs, which is not defined in the document though is noted as representing the PE equipment that faces the MSEE routers. MSEE-PR is somewhat confusing and ambiguous, so I'm proposing we more clearly define this by calling this PE-MSEE instead, representing that this is the provider edge equipment connecting to the MS Enterprise edge equipment. We could also put a definition around MSEE-PR and keep using it, but I think that changing this makes a little more sense and lines up better with what the visualization shows. Welcome any and all feedback.

Any feedback or thoughts on this PR?

@rambk Please review these proposed contributions and sign off if content looks okay. Thanks.

@jomolnar @rambk any update on this?

This being my first article on MSFT website, just noted these feedbacks. Will address them shortly.

@rambk can we merge these changes in?

@rambk @jboeshart Thanks for reviewing the content. If one of you signs off on the content, we can merge. Thanks.

Hi Jason:
I am fine with the acronym change.
Listing the ARP command in the doc is intentional. This way reader can continue the testing without getting distracted back and forth with the links.
Thanks,
Ram
From: Jason Boeshart<EMAIL_ADDRESS>Sent: Friday, March 10, 2017 11:44 AM
To: Microsoft/azure-docs<EMAIL_ADDRESS>Cc: Ram Balakrishnan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/azure-docs] Update expressroute-troubleshooting-expressroute-overview.md (#997)
Also proposing an update to the acronyms that are used in this document to line up with the image that shows how the connectivity works. Currently, the document references MSEE-PRs, which is not defined in the document though is noted as representing the PE equipment that faces the MSEE routers. MSEE-PR is somewhat confusing and ambiguous, so I'm proposing we more clearly define this by calling this PE-MSEE instead, representing that this is the provider edge equipment connecting to the MS Enterprise edge equipment. We could also put a definition around MSEE-PR and keep using it, but I think that changing this makes a little more sense and lines up better with what the visualization shows. Welcome any and all feedback.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/azure-docs/pull/997#issuecomment-285765855, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AUOiqIWckr3wc5x9XL1P-3-x7_P3E67Tks5rkafmgaJpZM4MZsiV.

The ARP commands are classic though, and as the doc exists currently you have to go to the other doc to get the ARM commands. I understand your point about hopping between docs, but those offer a more complete explanation of the ARP tables and how to use them to troubleshoot, plus they have both the Classic and ARM documentation available via the drop down at the top of the page. I think from both a usability and maintainability standpoint it makes more sense to just refer to the more complete documentation.

In next iteration, the documentation could be merged to avoid going back and forth. For this iteration, let’s add the reference link to the ARM documentation and not remove the existing commands.
From: Jason Boeshart<EMAIL_ADDRESS>Sent: Thursday, May 4, 2017 6:10 AM
To: Microsoft/azure-docs<EMAIL_ADDRESS>Cc: Ram Balakrishnan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/azure-docs] Update expressroute-troubleshooting-expressroute-overview.md (#997)
The ARP commands are classic though, and as the doc exists currently you have to go to the other doc to get the ARM commands. I understand your point about hopping between docs, but those offer a more complete explanation of the ARP tables and how to use them to troubleshoot, plus they have both the Classic and ARM documentation available via the drop down at the top of the page. I think from both a usability and maintainability standpoint it makes more sense to just refer to the more complete documentation.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fazure-docs%2Fpull%2F997%23issuecomment-299180292&data=02|01|rambala%40microsoft.com|375db0325a884e47882208d492eee182|72f988bf86f141af91ab2d7cd011db47|1|0|636295002060130529&sdata=276BFUMD2ztdqz%2BcKo1b4s9Te%2FFiikXnCAh24fQUss8%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAUOiqHOcRxPZgVtoT5kz5XgT6BskTtcsks5r2c4rgaJpZM4MZsiV&data=02|01|rambala%40microsoft.com|375db0325a884e47882208d492eee182|72f988bf86f141af91ab2d7cd011db47|1|0|636295002060130529&sdata=Bo3oZTnU5%2BTUMtq7KFbHeczXfWVTSSqNLqbo%2BCt8Ves%3D&reserved=0.

@rambk I've added that section back in. Let me know if this looks ok.

@rambk just wanted to check in here, does this look ok with you?

@rambk @jomolnar can this be merged? Would like to get this updated and closed.

@jboeshart After the owner of the content, @rambk, signs off, we can merge. Thanks.

I am good with the proposed changes. Please go ahead and merge.


ETIMEDOUT WebSocket Error

I am getting an error due to time out issues. How should I go about fixing this?
I have all the dependencies installed via npm as well.
After I initialize a strategy through telegram the errors begin to pop up a few seconds later.
Here is a log of the info in the console.
Initialization complete.
Waiting connection for user data updates: 01:42:05
Waiting connection for user data updates: 01:42:06
Waiting connection for user data updates: 01:42:08
Waiting connection for user data updates: 01:42:09
Waiting connection for user data updates: 01:42:11
Waiting connection for user data updates: 01:42:12
Waiting connection for user data updates: 01:42:14
Loading connection for user data updates: 01:42:15
Waiting connection for user data updates: 01:42:17
Waiting connection for user data updates: 01:42:18
Waiting connection for user data updates: 01:42:20
Waiting connection for user data updates: 01:42:21
Waiting connection for user data updates: 01:42:23
Waiting connection for user data updates: 01:42:24
[ 'WebSocket error: btcusdt@kline_5m (ETIMEDOUT) connect ETIMEDOUT <IP_ADDRESS>:9443' ]
[ 'WebSocket closed: btcusdt@kline_5m (1006)' ]
[ 'WebSocket error: btcusdt@kline_15m (ETIMEDOUT) connect ETIMEDOUT <IP_ADDRESS>:9443' ]
[ 'WebSocket closed: btcusdt@kline_15m (1006)' ]
Loading connection for user data updates: 01:42:26
[ 'WebSocket error: Yc31tSDcII2ptOsr4pk46EW2a9UGK74opWybIuJ7nyi9fcd7JnCcF9apXDkf (ETIMEDOUT) connect ETIMEDOUT <IP_ADDRESS>:9443' ]
[ 'WebSocket closed: Yc31tSDcII2ptOsr4pk46EW2a9UGK74opWybIuJ7nyi9fcd7JnCcF9apXDkf (1006)' ]
[ 'WebSocket error: btcusdt@kline_5m (ETIMEDOUT) connect ETIMEDOUT <IP_ADDRESS>:9443' ]
[ 'WebSocket closed: btcusdt@kline_5m (1006)' ]
[ 'WebSocket error: btcusdt@kline_15m (ETIMEDOUT) connect ETIMEDOUT <IP_ADDRESS>:9443' ]
[ 'WebSocket closed: btcusdt@kline_15m (1006)' ]
Waiting connection for user data updates: 01:42:27
[ 'WebSocket error: btcusdt@kline_5m (ETIMEDOUT) connect ETIMEDOUT <IP_ADDRESS>:9443' ]
[ 'WebSocket closed: btcusdt@kline_5m (1006)' ]
[ 'WebSocket error: btcusdt@kline_15m (ETIMEDOUT) connect ETIMEDOUT <IP_ADDRESS>:9443' ]
[ 'WebSocket closed: btcusdt@kline_15m (1006)' ]

@Fluffologist this was probably a Binance problem. I've had this problem even when their site was working, but the API was down. Did you manage to make BitProphet work?

I'm in a very similar situation. I have no problem running the script through the Run kit playground (and receive telegram messages) but have been unable to successfully get it running on my computer. I installed node.js and followed instructions. When I run the script on my computer, I get the following... but I never get any messages on Telegram. Am I forgetting a step? I'm a node newbie and appreciate any help!
Initialization complete.
Waiting connection for user data updates: 01:42:05
Waiting connection for user data updates: 01:42:06
Waiting connection for user data updates: 01:42:08
Waiting connection for user data updates: 01:42:09
Waiting connection for user data updates: 01:42:11
Waiting connection for user data updates: 01:42:12
Waiting connection for user data updates: 01:42:14

Nevermind, got it!


 doesn't work

In widgetbook/widgetbook#1160 we are adding an arcade via  which causes doc.page to break.

Probably need to document that better, but it's JSX not HTML, so the style attributes needs to be an object:

I also had to add allow-scripts to the sandbox for it to work

The following script is not working as well

The following script is not working as well

Got a preview link I can check?

@Ehesp Actually it's working, properly with the tweets, seems like just the script needed some time to load up properly.
Thanks for you help!


ModuleNotFoundError: No module named 'cog'

Hi, thanks for your work.
I just build the environment as you suggested as follows:
git clone git@github.com:fudan-zvg/Semantic-Segment-Anything.git
cd Semantic-Segment-Anything
conda env create -f environment.yaml
conda activate ssa
python -m spacy download en_core_web_sm
# install segment-anything
cd ..
git clone git@github.com:facebookresearch/segment-anything.git
cd segment-anything; pip install -e .; cd ../Semantic-Segment-Anything

Then I just try to run  python predict.py and it show the following error:
Traceback (most recent call last):
  File "predict.py", line 19, in <module>
    from cog import BasePredictor, Input, Path, BaseModel
ModuleNotFoundError: No module named 'cog'

Can you give me some suggestions on how to solve this?

Just need to pip install cog [doge]


Convert MissionPlanner script to MAVLink reader.

Add functionality to interop_cli.py to read MAVLink streams and forward
telemetry messages to interoperability server. Recommend using MavProxy
to tee traffic to the CLI. Add e2e tests by replaying log recorded using
Ardupilot simulator.
For #209
Note that I haven't been able to test message conversion validity with a simulator-generated log. Using Google Maps and an altitude lookup, it appears to be for a valid launch location at an Australian flying club. Post-submit, I'll reach out to someone with an autopilot to validate.

@DarkmatterVale @imritzror @JackHarrington @s4lvi
FYI

@prattmic @yogishadixit If you'd like to further review, please send comments by EOD Monday. Otherwise I'll submit (this PR has been open 8d).


Relax curve prefs in client mode; fixes issue #232

When in server mode, we set curve preference to make sure clients won't be able to select inefficiently implemented curves, at least for x86-64 platforms. But in client mode that argument doesn't hold, and makes ghostunnel clients incompatible with some backend services where we don't control their curve prefs. This splits the template config for client/server so we can relax the curve prefs for clients.

Coverage decreased (-0.2%) to 91.172% when pulling 73bd2b144b5da2856299a0372be342894cde8d44 on cs/issue-232 into 7e957e39942858f2ba411db51ee24e7c1aee153b on master.


Changes in Nav Bar and Components page

I can improve nav bar and can make several changes like style changing using different gradient or solid pattern colours and making the easy style center of attraction and attractive. Making Easy style more bolder .
Screenshot-

Changing the font of components page and making some more changes will make it more attractive.
Screenshot-

So can I work on this under Hacktoberfest 2022.

@Yashh-Bajaj The initial goal is to add as many components to the library as possible. Once that is done, we can look for revamp but doing it now won't make much sense

Hello, Can you assign me with this issue under Hacktoberfest 2022. I can add more Components like Modals, Drop downs, Input field and many more and also modify User Interface to make it more attractive and user friendly.


onBuffer not working

I just faced that of onBuffer doesn't work
please give me solution if any

This issue might help you.

same issue

same issue

+1

Please use exoplayer implementation. Not supported on mediaplayer


Consider having components and pages call conditional functions on data state instead of explicit checks

Reproducing some text from PR https://github.com/metaneuro/annotation_tool/pull/66
Getters that act as boolean conditionals can be used to ask the state of data in the store and not have our pages/components worry about what that check entails.
This is generally considered good programming practice where the details of conditional checks are kept close to their data sources. In fact, I was thinking we may want to consider doing this throughout the app where applicable. There are already a few instances where this scenario pops up. The relevant example was getter isColumnLinkedToCategory in the store and whether or not it should remain in the codebase. (See isDataTableLoaded and isDataDictionaryLoaded, also for instance.)

@jarmoza I think we can close this one, yes?


Single select box drop downs (without search) do not close on blur

On the demo page if you tab through each input, the single select box without search does not close on blur.

Resolved as of version 1.0.59


Add basecomponent

initial check-in, add basecomponent, please ccheck naming, no tests yet.

I was wondering if we need the BaseProperty? Maybe we can just remove those methods and use the extensions from #3?

Is it possible to get more in-depth XML docs as well please? So the consumers can understand what we expect. Just thinking out loud here.

I presume we can close this one since you'll rework it a bit @msjonathan? Or will you create a following PR?

A new pull request will be created with the changes by @msjonathan.


Replacement for #1872

Rebased, and fixed IAR exporter and gcc (both expor and build) - tested on top of another fix which is already for review.
@theotherjimmy

reference ##1872

cc @screamerbg

HOST_OSES=windows
BUILD_TOOLCHAINS=ARM,uARM,GCC_ARM,IAR
TARGETS=K64F,KL25Z

@mbed-bot: TEST
HOST_OSES=windows
BUILD_TOOLCHAINS=ARM,uARM,GCC_ARM,IAR
TARGETS=K64F,KL25Z

rebased on top of master (applied wrapping allocators)


Vertical distance between text when headings are wrapped around

Original: https://www.servicenow.com/blogs/2023/why-process-mining-is-game-changer.html
EDS: https://main--aemeds--servicenow-martech.hlx.live/blogs/2023/why-process-mining-is-game-changer

@andreituicu I am not sure if this one is reported yet, I see you are working on some headings issue already. Please feel free to close this one if it's already handled.

@sunidhiazad didn't see it reported yet. thank you!

One more such page: https://main--aemeds--servicenow-martech.hlx.live/blogs/2023/what-does-servicenow-do
https://www.servicenow.com/blogs/2023/what-does-servicenow-do.html

You can also see there is a vertical spacing issue in the main blog heading as well when it is wrapped around.
Original: https://www.servicenow.com/blogs/2023/visibility-tames-indias-digital-elephant.html
EDS: https://main--aemeds--servicenow-martech.hlx.live/blogs/2023/visibility-tames-indias-digital-elephant

@sunidhiazad should be fixed now. can you please retest to confirm?


[BUG] - Python docker should be a conditional import

Describe the bug
Right now we cannot create the feedstock for non-linux OSs (see https://github.com/conda-forge/conda-store-feedstock/pull/30)
We'd need turn the from python_docker.registry import Image, Registry import into a conditional one in https://github.com/conda-incubator/conda-store/blob/ccbb2980b66c87d2e57e3bf6bb23edd0039804dc/conda-store-server/conda_store_server/app.py#L82C1-L87C6
Expected behavior
Calling conda-store --help should work for Linux and OSX
How to Reproduce the problem?

Install conda-store locally in OSX
Run conda-store --help

Output
No response
Versions and dependencies used.
v 2023.9.2
Anything else?
No response

The reason I had not fixed this is that python-docker (unlike conda-docker) can be installed on Mac and Windows. So even if it doesn't do anything, having it there doesn't cause any issues.

Related: https://github.com/conda-incubator/conda-store/issues/630

@trallard I'm having trouble reproducing this.
The instructions mentioning code deal with conda-store-server, but the repro mentions conda-store. How are these related? Where does it cause issues?
Also, I've tried from python_docker.registry import Image, Registry in a Python interpreter on x86_64 mac. It worked fine. I was testing inside conda-store-server-dev.
Finally, see the other two issues mentioned in the comment above.

Yep, maybe this is just a matter of adding it as a dependency in pyproject.toml too to inform the conda-forge bots about this dep (comes from https://github.com/conda-incubator/conda-store/pull/524#issuecomment-1768207834).


realm_redirect: Change bottom text.

Fixes #32042

before
after

@timabbott I haven't tested.

Merged, thanks @amanagr!