Custom error pages

Is your feature request related to a problem? Please describe.
Whenever we run into a page address where nothing exists , we get a standard error page by Django, something like this:

Describe the solution you'd like
We have some custom error pages designed at /.github/ERROR_PAGES . They are needed to be displayed depending on the type of fallback.

Additional context
Make sure you have properly configured all related files so that this alteration does not affects the rest of the portal.

GitMate.io thinks possibly related issues are https://github.com/monsij/StudentPortal/issues/32 (static profile pages), https://github.com/monsij/StudentPortal/issues/3 (Add Contributors page), https://github.com/monsij/StudentPortal/issues/47 (Improved user profile page), https://github.com/monsij/StudentPortal/issues/56 (Contributors page update), and https://github.com/monsij/StudentPortal/issues/25 (Enhancements to contributors page).

I would like to work on this.

Assigned : @nguptaa


Documentation dependency quartodoc mixed with main dependencies

Hello,
We are using commonmeta-py in inveniosoftware/invenio-rdm-records and started running into problems with v0.9.x v0.8.7.
The problem is related to the fact that quartodoc (a documentation dependency), is listed as a main dependency:
https://github.com/front-matter/commonmeta-py/blob/a610c1c255f24d6095e9e564a55ead866f3753a8/pyproject.toml#L39-L41
The proper way to define such dependencies according to the Poetry documentation is:
[tool.poetry.group.docs]
optional = true

[tool.poetry.group.docs.dependencies]
# Docs deps here...

Do you think you could release a new version with these dependencies out of the main list?
Thanks,
Pablo

Addressed this via e939b70.


Extract multiple links from the clipboard

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
Converting URLs from the clipboard to MD files is a great idea. My requirement is to batch convert multiple URLs from the clipboard to MD files, so that I can download the articles that need to be downloaded in one place. I think this is a very practical feature
✔ Solution
The plugin automatically recognizes multiple URLs of the clipboard and downloads them in bulk
❓ Alternatives
No response
📝 Additional Context
No response

Thanks for the suggestion. It certainly points out workflow where user (like you) don't want to just save note but process multiple sources into one big note.
However I don't think your solution will satisfy broader audience of plugin, because there's necessity to format URLs to some pattern. I think creating new command to append content in currently open note will be better.

I agree with @lbgws2 for the feature request.
To reiterate if the clipboard content contains a list of newline delimited set of URLs, the plugin can process each one individually and put them into separate markdown files. This would not require fundamentally changing the behaviour of the plugin. It is just overloading the parseClipboard function.

Yeah, sorry I misread original post and thought @lbgws2 wants to create one big note.
In this case I agree with feature request and suggested format of URLs delimited by newline.

Perhaps this requirement can be listed as an experimental feature that can be enabled in the configuration
My habit is that when I have articles on the mobile that I need to read later, I automatically send them to an online document. Later, I can copy the URL list and focus on downloading them at a certain time

I also agree about this feature request - would really make this a power tool to convert lists of bookmarks.


x/pkgsite: Confusing search results

I ran the following search on go.dev and got the following results.

If I add a \ at the end of the package I am searching, I get the results I need.

The problem was I was looking for this package
https://pkg.go.dev/go.opentelemetry.io/contrib/instrumentation/net/http
But when I look at this
https://pkg.go.dev/go.opentelemetry.io/contrib
I don't see any of those subdirectories.

The reason go.opentelemetry.io/contrib/instrumentation/net/http is not listed there is because it is a nested module. We plan to address nested modules in the upcoming round of design changes.
Duplicate of golang/go#38596


gitnuro: init at 1.1.0

Description of changes
Open Source free multiplatform Git client for Linux, Windows & MacOS.
https://gitnuro.jetpackduba.com/
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, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 198619 run on x86_64-linux 1

1 package built:

gitnuro

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


Refactor/use docker swarm

Description
closes #112
Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

We might want to wait for this https://github.com/vercel/next.js/pull/59852 to merge this PR

We need to also add this :
Just a quick note, docker cp requires a container but it does not need to be running, so basically I can do:
# Build my artifact as part of docker build

docker build

# Create but don't run container from resulting image

CID=$(docker create <image-name>)

# Grab that artifact sweetness

docker cp ${CID}:/path/to/artifacts /local/path/to/artifacts

# Container be gone

docker rm ${CID}


feat(iroh-relay)!: add a QUIC server for QUIC address discovery to the iroh relay.

Description
This PR adds a QUIC endpoint to the relay server that can do QUIC address discovery. It also contains structs/functions for properly doing the Client side interaction for this process.
Also, this adjust the RelayNode to include configuration on how to speak to the QUIC endpoint on the relay server.
closes #2964
Breaking Changes

iroh_base::relay_map::RelayNode now has field quic that takes a Option<iroh_base::relay_map::QuicConfig>

Notes & open questions
The simplest version of adding quic configuration to the RelayNode would to add a single field quic_port that takes a u16. I've added the more complicated version of using QuicConfig. This was done in one commit and can be changed to the quic_port version easily, but QuicConfig does future proof the RelayNode struct more.
Change checklist

[x] Self-review.
[x] Documentation updates following the style guide, if relevant.
[x] Tests if relevant.
[x] All breaking changes documented.

@flub can you take a look at the config options for the relay server again.
I removed the option to generate self-signed certs automatically and added info to the readme about how to pass those in.
However, the --dev flag removes all tls config in the code and I wanted to keep the "simple" dev mode available for those who don't care about QUIC addr disc. SO there are now 2 dev-like flags, --dev that runs the relay server locally over http with QAD disabled and --dev-quic that expects tls to be configured for QAD, but still runs the server over http.
Is this the worst? I tried to keep it as simple as possible 😂  but could use another set of eyes.

actually, @flub
I'm wondering if instead, enabled_quic_addr_discovery should actually default to false and if we should add a --quic flag on the CLI that explicitly enables it instead of doing this weird --dev-quic thing.
So if you want to run QAD locally you would run:
cargo run --bin iroh-relay -- --config-path=PATH --dev --quic

That feels cleaner? thoughts?

Hum, it is awkward that TLS certs are basically becoming mandatory even for a local dev relay server.  But it is inevitable and we'll only grow more dependence over a QUIC transport over time.
But, the good news is that we don't have to get this right from the start.  The level of stability for --dev really doesn't matter that much.  And as you say, right now qad is just an optional extra rather than an integral part.
So I'm fine with both of these options.  And yes, the latter one, disable qad by default, is the simpler one so why not.


Fix mock hooks, add options argument, add compile-time verification

After changes in mocks interfaces, hooks were not updated with new options argument. That resulted in compile error, for users of custom hooks
Add compile time verification that hook implements interface, for every custom hook implementation.

/assign @bowei @spencerhance

Fun (sad) fact, I almost manually editted all this file, function names were not consistent enough to write some complicate regex......

/lgtm
/approve


Error Make in linux CUDA

Hello,
Please any one know the solution ?
I tried to make using this command
make BUILD_CUDA=1
but this error happened:
ar rvs /content/BitCrack-master/lib/libCLKeySearchDevice.a *.o r - CLKeySearchDevice.o r - bitcrack_cl.o make[1]: Leaving directory '/content/BitCrack-master/CLKeySearchDevice' make --directory KeyFinder make[1]: Entering directory '/content/BitCrack-master/KeyFinder' g++ -DBUILD_OPENCL -o clKeyFinder.bin ConfigFile.cpp main.cpp DeviceManager.cpp  -I/content/BitCrack-master/util  -I/content/BitCrack-master/AddressUtil  -I/content/BitCrack-master/CmdParse  -I/content/BitCrack-master/CryptoUtil  -I/content/BitCrack-master/KeyFinderLib  -I/content/BitCrack-master/CLKeySearchDevice  -I/content/BitCrack-master/CudaKeySearchDevice  -I/content/BitCrack-master/cudaMath  -I/content/BitCrack-master/clUtil  -I/content/BitCrack-master/cudaUtil  -I/content/BitCrack-master/secp256k1lib  -I/content/BitCrack-master/Logger  -I/content/BitCrack-master/embedcl -I/usr/local/cuda-9.2/include -O2 -std=c++11 -DCL_TARGET_OPENCL_VERSION=110 -L/content/BitCrack-master/lib -L/usr/local/cuda-9.2/lib64 -lkeyfinder -laddressutil -lsecp256k1 -lcryptoutil -lsecp256k1 -lCLKeySearchDevice -lclutil -lOpenCL -llogger -lutil -lcmdparse main.cpp: In function ‘void printDeviceList(const std::vector<DeviceManager::DeviceInfo>&)’: main.cpp:262:79: warning: format ‘%lld’ expects argument of type ‘long long int’, but argument 2 has type ‘long unsigned int’ [-Wformat=]          printf("Memory: %lldMB\n", devices[i].memory / ((uint64_t)1024 * 1024));                                                                                ^ mkdir -p /content/BitCrack-master/bin cp clKeyFinder.bin /content/BitCrack-master/bin/clBitCrack make[1]: Leaving directory '/content/BitCrack-master/KeyFinder' make --directory AddrGen make[1]: Entering directory '/content/BitCrack-master/AddrGen' g++ -o addrgen.bin main.cpp  -I/content/BitCrack-master/util  -I/content/BitCrack-master/AddressUtil  -I/content/BitCrack-master/CmdParse  -I/content/BitCrack-master/CryptoUtil  -I/content/BitCrack-master/KeyFinderLib  -I/content/BitCrack-master/CLKeySearchDevice  -I/content/BitCrack-master/CudaKeySearchDevice  -I/content/BitCrack-master/cudaMath  -I/content/BitCrack-master/clUtil  -I/content/BitCrack-master/cudaUtil  -I/content/BitCrack-master/secp256k1lib  -I/content/BitCrack-master/Logger  -I/content/BitCrack-master/embedcl -I/usr/local/cuda-9.2/include -O2 -std=c++11 -DCL_TARGET_OPENCL_VERSION=110 -L/content/BitCrack-master/lib -laddressutil -lsecp256k1 -lcryptoutil -lsecp256k1 -lutil -lcmdparse mkdir -p /content/BitCrack-master/bin cp addrgen.bin /content/BitCrack-master/bin/addrgen make[1]: Leaving directory '/content/BitCrack-master/AddrGen' make --directory CLUnitTests make[1]: Entering directory '/content/BitCrack-master/CLUnitTests' cat ../clMath/secp256k1.cl secp256k1test.cl > cltest.cl /content/BitCrack-master/bin/embedcl cltest.cl cltest.cpp _secp256k1_test_cl g++ -o clunittest.bin cltest.cpp cltest.cpp main.cpp  -I/content/BitCrack-master/util  -I/content/BitCrack-master/AddressUtil  -I/content/BitCrack-master/CmdParse  -I/content/BitCrack-master/CryptoUtil  -I/content/BitCrack-master/KeyFinderLib  -I/content/BitCrack-master/CLKeySearchDevice  -I/content/BitCrack-master/CudaKeySearchDevice  -I/content/BitCrack-master/cudaMath  -I/content/BitCrack-master/clUtil  -I/content/BitCrack-master/cudaUtil  -I/content/BitCrack-master/secp256k1lib  -I/content/BitCrack-master/Logger  -I/content/BitCrack-master/embedcl -I/usr/local/cuda-9.2/include -O2 -std=c++11 -DCL_TARGET_OPENCL_VERSION=110 -L/content/BitCrack-master/lib -L/usr/local/cuda-9.2/lib64 -lclutil -lOpenCL /tmp/cc485oE1.o:(.data+0x0): multiple definition of _secp256k1_test_cl'
/tmp/ccIg84Im.o:(.data+0x0): first defined here
/content/BitCrack-master/lib/libclutil.a(clContext.o): In function cl::CLProgram::CLProgram(cl::CLContext&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >)': clContext.cpp:(.text+0xf8f): undefined reference to util::toLower(std::__cxx11::basic_string<char, std::char_traits, std::allocator > const&)'
/content/BitCrack-master/lib/libclutil.a(clContext.o): In function cl::CLProgram::CLProgram(cl::CLContext&, char const*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >)': clContext.cpp:(.text+0x12a0): undefined reference to util::toLower(std::__cxx11::basic_string<char, std::char_traits, std::allocator > const&)'
collect2: error: ld returned 1 exit status
Makefile:5: recipe for target 'all' failed
make[1]: *** [all] Error 1
make[1]: Leaving directory '/content/BitCrack-master/CLUnitTests'
Makefile:113: recipe for target 'dir_clunittest' failed
make: *** [dir_clunittest] Error 2`

I have the same error. Did you ever find a workaround?


Prevent SwipeableDrawer crash when parent update while swiping

Hey hey ! 👋
We noticed a lot of sentries in getMaxTranslate because this crashes when the parent updates while the user is swiping the drawer.
This is already handled in handleBodyTouchMove here
https://github.com/mui-org/material-ui/blob/98c9aad08696f44976f55aa1e254d013df6c38e1/packages/mui-material/src/SwipeableDrawer/SwipeableDrawer.js#L296-L300
but not in handleBodyTouchEnd yet
Tested locally in multiple cases and it works fine 👌

[x] I have followed (at least) the PR section of the contributing guide.

Cheers from Doctolib ✌

@xavier-villelegier Thanks for the PR, whoever, we miss a reproduction at minimum.

👋 The migration PR has been merged.
Please follow these steps to make sure that the contents are all updated. (Sorry for the inconvenience)

pull latest master from upstream to your branch
if your PR has changes on the *.md or demo files, you are likely to encounter conflict because all of them have been moved to the new folder.
2.1 revert the change on those markdown files you did
2.2 pull latest master from upstream (you should not see conflict)
2.3 make the changes again at docs/data/material/*
run yarn docs:api

you might see the changes in docs/pages/material/api/* if your PR touches some of the components
it is okay if there is no changes

If you are struggle with the steps above, feel free to tag @siriwatknp


feat: autocomplete for kernel functions

Add autocomplete for Kernel functions like put_in/2, ++/2, defstruct/1

So I originally didn't include Kernel functions because, unlike SpecialForms, you can't remove them from scope with import.
So here technically you could un-import def and make your own def and the autocomplete would show you the wrong documentation, or show it to you even tho you un-imported it (in the case you don't define your own.
That being said, it since completions are still experimental, it might be already to include Kernel functions, as for most people they probably aren't messing with them outside of library code.

alright i played with this for 8 seconds and its worth adding


[Docs-1] Convert non-php code.

This is the follow up of #5689.
[x] Convert non-php code in examples to valid php.
Some of these changes are not valid php syntax  or go against the style guide. This will be fixed in a later step.

@kenjis Fixed.
However, I think we should not waist time on fixing CS for the next PRs.
These are just to provide valid PHP syntax, so we can run php-cs-fixer in the last PR to fix all CS violations at once.

Coveralls change of "system" should honestly not be touched by this PR 😆


When skipping species classification and running StarAMR pipeline fails

Description of the bug
The error messages are tracked below:
Skipping Polishing
Skipping running of Kraken2 or mash for speciation
No subtyping of assemblies performed
ERROR ~ No such variable: top_hit

 -- Check script '.nextflow/assets/phac-nml/mikrokondo/./workflows/PostAssembly.nf' at line: 94 or see '/tmp/log/a1c9f9734a98491580039e27181b26cf/nextflow.log' file for more details

Command used and terminal output
No response
Relevant files
No response
System information
No response

fixed in: 69-when-skipping-species-classification-and-running-staramr-pipeline-fails


feat(starknet_batcher): communication for sync block request

#2604
#2603  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

crates/starknet_batcher_types/src/communication.rs line 73 at r1 (raw file):
Previously, alonh5 (Alon Haramati) wrote…
Great documentation

Added a few more fixes as suggested by ChatGPT.

Merge activity

Dec 16, 6:03 AM EST: A user started a stack merge that includes this pull request via Graphite.


Document Update

Change in keyword from "Possibles" to "Possible"
[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] If you're unsure, you can format the pull request title like [CAMEL-XXX] Fixes bug in camel-file component, where you replace CAMEL-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean install -Psourcecheck in your module with source check enabled to make sure basic checks pass and there are no checkstyle violations. A more thorough check will be performed on your pull request automatically.
Below are the contribution guidelines:
https://github.com/apache/camel/blob/master/CONTRIBUTING.md

Can you please share the javadoc link, tried checking the files. Since there are many files difficult to find the actual file which is pointing to it.

Hi @NiteshKoushik . It is file https://github.com/apache/camel/blob/master/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsConfiguration.java . Needs to be changed on both lines - 222 and 1298. All other files will be regenerated automatically during next build.

Thanks @bedlaj , I have done the changes in Javadoc and created a PR
https://github.com/apache/camel/pull/4235
I will be closing this request.

Thanks


Is there a way to mirror the download of npm?

Wondering if there's a way to mirror the URL to download NPM.  I see this script here:
https://github.com/nvm-sh/nvm/blob/master/nvm.sh#L2493-L2511
which seems hard-coded to use:
nvm_download -L https://npmjs.org/install.sh

That's only for node versions that don't ship with npm, which are 0.5 and below. Are you trying to install those node versions?

yes.

Confirmed.  Thanks!


Recursive chart detection (subcharts) future = bug

Is this a BUG REPORT or FEATURE REQUEST? (choose one): BUG REPORT
Environment (plugin version, maven version, OS, ...):
plugin version: 6.13.0
mvn version: 3.8.1
helm version: 3.8.0
What happened:
I create simple example subchart structure from helm documentation: https://helm.sh/docs/chart_template_guide/subcharts_and_globals/
So i have directory structure:
mychart
    charts
        mysubchart

Plugin configuration:
<configuration>
    <chartDirectory>mychart</chartDirectory>
    <values>
        <yamlFile>etc/my-values.yaml</yamlFile>
    </values>
</configuration>

I configure all values in my-values.yaml:
testKey: test1

mysubchart:
    testSubKey: test2

I ilso tryed to put values in parent (mychart) values.yaml (with the same result)
in subchart template i have:
...
image: {{ .Values.testSubKey }}
...

I run mvn helm:upgrade@helm-install and get error:
[INFO] --- helm-maven-plugin:6.13.0:upgrade (helm-install) @ mychart ---
[INFO] Upgrading the chart with install C:\test-app\helm\mychart\charts\mysubchart
[ERROR] Error: UPGRADE FAILED: failed to create resource: Deployment.apps "mysubchart" is invalid: spec.template.spec.containers[0].image: Required value

So as you can see from logs - plugin start installing from subchart ("Recursive chart detection (subcharts)" future in action i think) and do not respect helm subchart values hierarchy.
What you expected to happen:
I expect plugin to run install on parent chart and all subcharts will be installed by helm logic.
I do not understand the reason why plagin should implement "Recursive chart detection (subcharts)" future...
How to reproduce it (as minimally and precisely as possible):
Create helm subchart structure and declare values for subcharts in parent chart.

As workaround i managed to set exludes:
<excludes>
    <exclude>**/charts/**</exclude>
    <!-- duplicate pattern with different slashes - cause it not work on different OS -->
    <exclude>**\charts\**</exclude>
</excludes>

@tipame Can you provide an example project?

Closing due missing feedback


How to Merge Erf into Module

[Target]
name = "testmod"
file = "mod/test.mod"
description = ""
include = "../../scripts/ultima_schedules.erf"
include = "src/**/*"
The erf is not included in the module

See #72 to understand which this isn't working.
Erfs cannot be included in modules. Instead, they have to be extracted and have their contents included in the module. The easy way to do this is to set up your target like this:
[Target]
name = "testmod"
file = "test.mod"
include = "src/**/*.{nss,json}"

Move ultima_schedules.erf into the package root (the same directory containing your nasher.cfg) and then unpack it into the module's source tree:
nasher unpack testmod --file:ultima_schedules.erf --removeDeleted=false

Ok, thanks for the clarification. I guess I was thinking that nasher could do the behind the scenes shuffling to import the erf. Obviously you can import erfs into modules because the toolset does it, but it has to be extracted first. It's much easier to have an .erf that is shared between modules as an importable resource than to unpack it and import it as its internal files. Some features of mods are distributed as erfs. Nasher should easily just treat this like the toolset and do the import. But it's no problem creating a target, unpacking it and including the source files. Just extra steps.

You don't have to do that to have a module share resources. Just include the path to the resources in both modules' targets.


Add example k8s services

Providers a container_push bazel rule for pushing a container to a container registry, and an example kubernetes config file defining a deployment, and an exposed service running the hello world example.

I signed it!


quote individual parts of bigquery table locators, not the whole thing together

what dbcrossbar currently generates, which is invalid:
SELECT `id` FROM `myproject-123.tmp.mytable` WHERE mytable.batch_id = '75c600c4-6da3-47f2-8e6f-893be683ef76'

what dbcrossbar could generate, which is valid:
SELECT `id` FROM `myproject-123`.`tmp`.`mytable` WHERE mytable.batch_id = '75c600c4-6da3-47f2-8e6f-893be683ef76'

in other words, please quote the 3 parts of the table locator separately

Might be worth noting that . still needs to be backwhacked inside quoted identifiers or BQ will error.

Huh, fun. We have a whole bunch of machinery for dealing with this garbage but I guess we missed some places? Fun.


Multiple entries of version in pgstc.migrations table

Running the pypgstac migrate command from a previous version to a new version leads to multiple entries of the final version in the pgstac.migrations table:
Running migrations for ['pgstac.0.2.4-0.2.7.sql', 'pgstac.0.2.7-0.2.8.sql', 'pgstac.0.2.8-0.2.9.sql', 'pgstac.0.2.9-0.3.0.sql', 'pgstac.0.3.0-0.3.1.sql', 'pgstac.0.3.1-0.3.2.sql', 'pgstac.0.3.2-0.3.3.sql'].
pgstac version 0.3.3

 version |           datetime
---------+-------------------------------
 0.2.4   | 2021-07-23 00:00:00+00
 0.2.7   | 2021-09-03 09:02:22.32407+00
 0.3.3   | 2021-09-03 09:02:22.360333+00
 0.2.8   | 2021-09-03 09:02:22.362009+00
 0.3.3   | 2021-09-03 09:02:22.362903+00
 0.2.9   | 2021-09-03 09:02:22.364059+00
 0.3.3   | 2021-09-03 09:02:22.364059+00
 0.3.0   | 2021-09-03 09:02:22.366378+00
 0.3.3   | 2021-09-03 09:02:22.366378+00
 0.3.1   | 2021-09-03 09:05:24.690869+00
 0.3.3   | 2021-09-03 09:05:24.690869+00
 0.3.2   | 2021-09-03 09:05:24.716236+00
 0.3.3   | 2021-09-03 09:05:24.716236+00
 0.3.3   | 2021-09-03 09:05:24.723922+00
 0.3.3   | 2021-09-03 09:05:24.723922+00
(15 rows)

For any migration file (e.g., 0.2.4 to 0.2.7) two inserts are currently conducted:

From the migration file
https://github.com/stac-utils/pgstac/blob/main/pypgstac/pypgstac/migrations/pgstac.0.2.4-0.2.7.sql#L313

From the python script (which inserts always to latest version to migrate to - in my case 0.3.3):
https://github.com/stac-utils/pgstac/blob/main/pypgstac/pypgstac/migrate.py#L160-L170

I think this can be resolved by removing the insert statement from the python script?

I believe I also saw this in https://github.com/stac-utils/pgstac/issues/49


uif2iso: init at 0.1.7

Motivation for this change
Want to use uif2iso
Things done

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

[ ] NixOS
[ ] OS X
[X] Linux

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

a program for converting UIF files (Universal Image Format, used by MagicISO)
to uncompressed images depending on the input file type:
ISO, BIN/CUE, MDS/MDF, CCD/IMG/SUB and NRG.

thanks!


Add plugin for CSS logical properties

Add a plugin to prefix CSS logical properties with the nonstandard
-webkit- and -moz- properties.
Browser version data and supported properties based on
https://caniuse.com/#feat=css-logical-props

Ok, as I have tried solving that using pure caniuse data I found how this PR is required hehe.
Thanks again for adding that =)

Awesome, thank you for such quick response!! :)


Object type extension 'Query' cannot redeclare field listMessages

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
No response
Amplify CLI Version
12.1
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No
Describe the bug
Hey. I am trying to sort my queries by date as suggested here https://docs.amplify.aws/guides/api-graphql/query-with-sorting/q/platform/js/#implementation buy I keep running into an error in CLI when I edit my schema and run amplify push.
The error I get is similar to that in https://github.com/aws-amplify/amplify-cli/issues/2797 but the solution is not clear.
Here is my schema:
query ListMessages(
  $filter: ModelMessageFilterInput
  $limit: Int
  $nextToken: String
) {
  listMessages(filter: $filter, limit: $limit, nextToken: $nextToken) {
    items {
      id
      type
      text
      sentAt
      isRead
      editedAt
      createdAt
      updatedAt
      messageSenderId
      messageSpaceId
      owner
    }
    nextToken
  }
}

type Message @model @auth(rules: [{allow: private, operations: [read]}, {allow: owner}]) {
  id: ID! @index(name: "listMessages", queryField: "listMessages", sortKeyFields: ["sentAt"])
  type: MessageType!
  text: String
  sentAt: AWSDateTime!
  isRead: Boolean!
  sender: User! @hasOne
  space: MessageSpace! @hasOne
  editedAt: AWSDateTime
}

Here is the error message:
🛑 Object type extension 'Query' cannot redeclare field listMessages

Expected behavior
I expect it to work.
Reproduction steps
Try to follow the description
Project Identifier
No response
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @dan-codes1 :wave: thanks for raising this! I'm going to transfer this over to our API repo for tracking purposes, however looking at the schema I noticed there is a mix of GraphQL schema and GraphQL client code. Looking at the linked docs I can see how this is confusing.
This piece is part of your schema:
type Message @model @auth(rules: [{allow: private, operations: [read]}, {allow: owner}]) {
  id: ID! @index(name: "listMessages", queryField: "listMessages", sortKeyFields: ["sentAt"])
  type: MessageType!
  text: String
  sentAt: AWSDateTime!
  isRead: Boolean!
  sender: User! @hasOne
  space: MessageSpace! @hasOne
  editedAt: AWSDateTime
}

Where this is how you would structure your query to call your GraphQL API from your frontend:
query ListMessages(
  $filter: ModelMessageFilterInput
  $limit: Int
  $nextToken: String
) {
  listMessages(filter: $filter, limit: $limit, nextToken: $nextToken) {
    items {
      id
      type
      text
      sentAt
      isRead
      editedAt
      createdAt
      updatedAt
      messageSenderId
      messageSpaceId
      owner
    }
    nextToken
  }
}

In practice this looks something like:
// src/pages/my-page.jsx
import { API } from 'aws-amplify'

const QUERY_LIST_MESSAGES = /* GraphQL */`
	query ListMessages(
	  $filter: ModelMessageFilterInput
	  $limit: Int
	  $nextToken: String
	) {
	  listMessages(filter: $filter, limit: $limit, nextToken: $nextToken) {
	    items {
	      id
	      type
	      text
	      sentAt
	      isRead
	      editedAt
	      createdAt
	      updatedAt
	      messageSenderId
	      messageSpaceId
	      owner
	    }
	    nextToken
	  }
	}
`

// ...
const data = await API.graphql({
  query: QUERY_LIST_MESSAGES,
  variables: {
    filter: 'something',
    limit: 10,
  }
})

For more information I recommend taking a look at this page in our docs https://docs.amplify.aws/lib/graphqlapi/query-data/q/platform/js/


Update website link on the description in the left of this repository

Should be updated to https://packages.masoniteproject.com

Can you make this small update on github repo page ?


Check Leap Year - Python

Check if a given year is a leap year.
Add the code to the python folder

@Dext3r-Morgan I'm very much interest to work on this beginner friendly project. Can you assign this to me?


Align index name validation in custom resources with Splunk docs

Description
Usability improvements to the splunk_monitor and splunk_index custom resources, accepting valid (user-defined) index names while rejecting invalid index names, as per Splunk's (user-defined) index name requirements and standard internal index names.
Issues Resolved
#216
Check List

[x] All tests pass. See TESTING.md for details.
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable.

This looks great. Thanks for the PR!

@vnaipaul can you please add a CHANGELOG entry?

@ramereth I've added a CHANGELOG entry as requested.

Released as: 9.2.0


Implement tutorials with Literate.jl

The most important added file is docs/literate/make.jl which creates md and nb files for the tutorials.
For now (could be moved somewhere else), it also runs the jl source files to check for errors (also testing solutions with @test).
It is included to docs/make.jl such that with every documentation CI run, the tutorials are updated and checked as well. The literate tutorials are added to the documentation, with the "old" static versions still existing.
Since I added the src files, it seems like a big amount of new code, but in exchange the static md versions can be deleted in the future.

@ranocha The Documentation CI fails with an error we already had in TrixiTutorials. I mentioned it in slack back then. It looks like the CI doesn't fail when testing the jl files with @doctest but when Documenter executes the @example blocks.
In TrixiTutorials I fixed it by calling the make file in the CI action with withenv("GKSwstype"=>100) do; include("docs/make.jl") end. Should we do this here again, or do you have a better idea?

Can't you set the appropriate environment variable in our Documenter GitHub action? See https://docs.github.com/en/actions/reference/environment-variables
What would be the advantage of changing the calling command? Does everything work locally for you without setting the environment variable?

I never really worked with GitHub actions and environment variables, so I didn't know that's possible. You mean something like that
- name: Build and deploy
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # For authentication with GitHub Actions token
          DOCUMENTER_KEY: ${{ secrets.DOCUMENTER_KEY }} # For authentication with SSH deploy key
          GKSwstype: 100
        run: julia --project=docs --color=yes docs/make.jl

Yes, locally everything works fine without setting the variable.

I never really worked with GitHub actions and environment variables, so I didn't know that's possible. You mean something like that
- name: Build and deploy
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # For authentication with GitHub Actions token
          DOCUMENTER_KEY: ${{ secrets.DOCUMENTER_KEY }} # For authentication with SSH deploy key
          GKSwstype: 100
        run: julia --project=docs --color=yes docs/make.jl

Yes, exactly :+1:

Yes, locally everything works fine without setting the variable.

Okay, great.  In this case, we don' need to set environment variables locally.


Upgrade manylinux image

Support for the image we were using has ended on January 1st, 2023.

Merging as Linux build passed


pull request for e2e test at 02/01/2017 18:56:15

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


Rename "Bicycle Volume ATR" to...?

What Happened
From @jwrcoleman :

One thing i uncovered with the new filters, is that studies in there listed as bicycle ATRs are actually Permanent Bicycle Counters. (on Sherbourne and Bloor/Castle Frank)

What Should Happen
The language around bicycle counts conducted by permanent counting stations should be clear.  There are a couple of ways we could approach this:

just change "Bicycle Volume ATR" to something else;
add further contextual help around the use of terms like this (e.g. a glossary somewhere in MOVE).

@jwrcoleman @mkewins should probably weigh in on the best path forward here.
To Reproduce
Steps to reproduce the bug:

Click "Edit" in the filters box;
Open the "Studies" section of the accordion at bottom of the resulting drawer.

At this point, you'll see the option to filter on "Bicycle Volume ATR" as a study type.

I'm going to defer to @jwrcoleman on this one.
My main question would be: when is a "Bicycle Volume ATR" just an ATR vs. a permanent counter? Do we do bicycle-only ATRs? Or would we collect a multi-modal ATR?
fwiw, this is intersecting with some work I'm doing to map out the hierarchy of studies / reports / warrants (so any info that can be shared about the nuance between these different types of studies would be helpful please!)

I noticed a related issue when talking with Donna yesterday. Some locations with Bicycle Volume ATRs also have data loaded as Volume ATRs but it looks like the data is all coming from the same permanent bike counters.
Examples:

Bloor / Huron
Bloor / Markham
Sherbourne / Wellesley
Bloor / Castle Frank

So a couple issues here:

If Volume ATR counts are exclusively counting bicycles, they should be classified as Bicycle Volume ATR. This difference most likely happened at the point of data loading.
Open question: how do we handle study type (i.e. Bicycle Volume ATR) and mechanism of collection (i.e. permanent count stations)?

On (2) -- we have a similar parallel with Volume ATR and RESCU (example at DVP on-ramp south of Bloor), where the data is simply volume data collected at a permanent count station, but it's classified by the station type (RESCU) not the data type (Volume ATR). So maybe the solution here is to add a dedicated Permanent Bicycle Counter study type?

Couple quick thoughts
-these locations should be called bicycle permanent count stations (not ATRs), but if they were loaded wrong they really should all be bike counts as you note.

we have a dataset of bicycle ATRs that we’d like to one day load from cycling
Bicycle ATR is given as an option when requesting studies, but this is not a study type that we can deliver (OTI doesn’t do bicycle ATRs), so they should be removed from the request options.
For rescu etc, I like the permanent count station typology here too (as opposed to calling them rescu)

@jwrcoleman I'm moving this over to Jira to account for this in our product backlog. How can I communicate the need to re-load/re-classify these bike counts where they were uploaded as "Volume ATRs"?


[BUG]: The bot word-filter doesn't flag ", ,"

This bug seems to be a big issue as one may just list comma-separated bad-words and non-inclusive language, and not get flagged by Alexjs 😱
This isn't recognised or flagged by the bot: , <bad-word>,
Context:

At present this seems to be the only case in which the bot isn't flagging.(I tried other combinations)

The consequences:
(It can be a lot more bad :( )

I think this is currently sent to the library, we might have to see what options we have available

Another bug was found where "" was not flagged by the bot.
Could we perhaps do string processing, on the text, before sending it to Alex, in order to remove punctuation and ensure a space-separated sentence is sent??

ah yes that is a great idea 👍

At present this seems to be the only case in which the bot isn't flagging.(I tried other combinations)

Found one more:

I wonder if it is even reparable. Cuz when we split based on spaces, such characters will be associated with the word. If we consider flagging substrings even words like they will be flagged because the substring he is present in it.
Another way is to completely ignore all characters that are not alphabets. But again, is this ideal?

I guess it is, for now, we could just strip off punctuation from the sentence and then join all the words with spaces in between

Also -- and '' and () and  and {} aren't taken into consideration.
Still testing, but i think we have to check for non-alphabetical characters in the words and if a word is in between we send that to alexjs.

Can I work on this ?
or perhaps we could team up on this? 🙃

You can work on this and if you need any help i am here.
Super loaded with school next week and this weekend.
I'm gonna do more open source in the upcoming vacation. 😄 😋 👍


feature request: prop to disable follow cam collision lerping

i need the camera to instantly change position when an obstacle is detected.

Will consider adding it in the next version.

Will consider adding it in the next version.

I believe i solved it locally but could use your opinion on my solution.
look forward to seeing this issue closed after you add a prop to control in the next release

closed by https://github.com/pmndrs/ecctrl/commit/b958503c2b6fda324967664699ea0fde7fb4c457

@ErdongChen-Andrew can you add the ability to pass a boolean value to skip the lerping to remove the performance waste with lerp?
basically if disableCamCollisionlerp is true then use position.set instead of position.lerp

Look at what happens when i pass 999 in camCollisionSpeedMult
https://github.com/user-attachments/assets/6c423926-69df-4b92-a3bf-2a760ed318e2


[AutoPR-sdkreleaseplannertest]mgmt powerapps test

Create to sync https://github.com/Azure/azure-rest-api-specs/pull/22201

/azp run prepare-pipelines

/azp run net - sdkreleaseplannertest - mgmt - ci

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.DataBoxEdge


Is there a built in way to set a focus zone ?

Is there a already built in way to configure a focus zone, so the scanner only operates in a certain area of the camera view finder ?

We're going to need arbitrary focus for @openfoodfacts @juliansteenbakker
We have the option to implement it here with CameraX, but there's also the more future proof option to switch mobile_scanner to the newly released camera_android_camerax (0.5) pub.dev package which switches to CameraX (bringing arbitrary focus for free), and would reduce maintenance cost of mobile_scanner.
https://pub.dev/packages/camera_android_camerax

Since we scan with a scan window, i'm closing this issue.


Better client side support

The PR's intent is to improve the experience of using the library on the client side, the changes that were made are the following:

Uploaded the repository to the bower, so now it can be installed via bower install pptxgen
Added gulp task to generate a bundle with all the core files needed to run pptxgen (jszip, jquery)

If the PR is merged, the version tag should be bumped and the new version should be uploaded to bower, given there is no contribution instructions I did my best to be us tidy as possible. I don't now how you handle builds internally, or if you do it at all so I tried not to go too far with that.
I'm using pptxgen at work and this improvement would make it easier for me to use it.
Hope it helps!

Hi @santi-git ,
Thanks for the PR and doing to bower registration! I've incorporated the changes into the project.
Also, thanks for getting me started with gulp - i've been meaning to setup a workflow for a bundled file for a while now. :-)


Adding fine-tuning other modules

I've added fine-tuning the img_projector and vision_model both.
The code base was from
https://github.com/zhuyiche/llava-phi/tree/main

Also I added eos token to the input prompt. Based on phi3 cookbook finetuning code.
When fine-tuned, it keeps generating  until the max_new_token because it dosen't generate the eos token.
The separator could be used for the stopping_criteria  when generating, but sometimes overall summary comes after the separator.

Hi, thanks for your PR.
eos_token is truly a big issue of phi3-mini series model, but I think I already solved this problem in this repo.
First, the processor.tokenizer.apply_chat_template constructs prompts/input text like following
<|user|>
<|image_1|>
<|image_2|>
What is the difference between these two images?<|end|>
<|assistant|>
There is no difference.<|end|>
<|user|>
check again<|end|>
<|assistant|>

We can see that you could use <|end|> as the eos token. (When performing model inference after the model is finetuned by this repo's code, you should config the generation_config with <|end|> as eos. This should lead to correct behavior.)
Second, when constructing labels for SFT, I make the loss cover the eos token (i.e., <|end|>) of assistant responses. This should make the model learn to generate <|end|> properly (unless I made some mistakes for coding this logic).
P.S. I do observed that phi-3-mini-4k (the language model) have not learned to predict <|end|> when it should do. I guess it is a small mistake coming from 'microsoft/phi-3' team. They may forget to make loss cover <|end|> when finetuned the model.

Thanks for your comment. I'm using the <|end|> as the eos token too and saw the codes you've made the loss for <|end|> propagated properly.
However I'm still bit confused about the eos_token becuase there is a token names eos that is  <|endoftext|> in the tokenzier. Also in the discussion https://huggingface.co/microsoft/Phi-3-mini-4k-instruct/discussions/26 , the phi3 team has commented  <|endoftext|> can be used as eos_token during fine-tuning.
I'll go on some experiments with this and some other settings too.
Once again, thanks for your comment and the fine-tuning codes.

@ParadoxZW Thanks for your explanation.
By the way I haven't commited the merging the lora weights. I'll upload it asap after testing.

@2U1 I've made some comments in the Files changed pages about specific changed codes. May you reply them when you get time?

@ParadoxZW Yes, sure also I found that I missed some optimizer code and some other seetings for merging lora and I'm working on it.
BTW I can't see any comments there.

I've updated the create_optimizer function to give differenct lr to the lora module and non_lora module.
Also added to save the processor config, not the tokenizer config.
In the merging lora weights, I don't know exactly why I get the shared memory error, so I had to just save the model with safe_serialization=False . If you have any idea to fix it let me know.

Sorry, I am too busy and overwhelmed with end-of-semester stuffs in this week.
First, do you mind add your code to a develop branch of this repo, so I can try to debug it and more easily cooperate with you?
Second, by comments in Files changed page, I mean

I don't know you cannot see them.
Last, thanks for your efforts and contributions!

Sorry, I'd see this late. Of courese you could.
I updated some other things and, still updating other features in the forked repo.


Fail if RocksDB or Snappy source is missing

When I tried to build via cargo build I saw a lot of error messages and had no clue what was wrong. I hope with this patch things get a little bit clearer if people run into similar issues.

Thanks!


BHBC-1188: Timezone bug fix

Overview
Links to jira tickets
https://quartech.atlassian.net/browse/BHBC-1188
This PR contains the following changes

Add function in db.ts to stop node-postgres from adding default time/timezone information to psql DATE values (which by design don't contain time/timezone information).

Bonus:

Update docker-compose/env/pipeline files for Db and API to allow a timezone to be specified (rather than just letting it default to whatever default value, UTC probably).

This PR contains the following types of changes

[ ] New feature (change which adds functionality)
[x] Enhancement (improvements to existing functionality)
[x] Bug fix (change which fixes an issue)
[ ] Misc cleanup / Refactoring / Documentation
[ ] Version change

Checklist
A list of items that are good to consider when making any changes.
Note: this list is not exhaustive, and not all items are always applicable.
General

[x] I have performed a self-review of my own code

Code

[ ] New files/classes/functions have appropriately descriptive names and comment blocks to describe their use/behaviour
[ ] I have avoided duplicating code when possible, moving re-usable pieces into functions
[ ] I have avoided hard-coding values where possible and moved any re-usable constants to a constants file
[ ] My code is as flat as possible (avoids deeply nested if/else blocks, promise chains, etc)
[ ] My code changes account for null/undefined values and handle errors appropriately
[ ] My code uses types/interfaces to help describe values/parameters/etc, help ensure type safety, and improve readability

Style

[ ] My code follows the established style conventions
[ ] My code uses native material-ui components/icons/conventions when possible

Documentation

[x] I have commented my code sufficiently, such that an unfamiliar developer could understand my code
[ ] I have added/updated README's and related documentation, as needed

Tests

[ ] I have added/updated unit tests for any code I've added/updated
[ ] I have added/updated the Postman requests/tests to account for any API endpoints I've added/updated

Linting/Formatting

[x] I have run the linter and fixed any issues, as needed
See the lint commands in package.json
[x] I have run the formatter and fixed any issues, as needed
See the format commands in package.json

SonarCloud

[ ] I have addressed all SonarCloud Bugs, Vulnerabilities, Security Hotspots, and Code Smells

How Has This Been Tested?
Tested locally.
Also used the PR-deploy to test that the timezone for API and DB do change if I change the config.json timezone values.

So the standard TZ on the pod is already: ENV TZ America/Vancouver
I am a bit confused why this has to be reset.

Ya I kind of combined the actual change needed, with some bonus stuff.
The actual change that fixed some of the funny app dates is just a few lines in db.ts.
Everything else is just bonus to give us the capability of specifying the timezones used by the API and DB, in case we ever want to change it. A bunch of this came about naturally as part of testing the timezones (changing the API/DB to various timezone combinations to see how it affected the dates, etc), and so I just carried it forward into the pipeline since it wasn't a big additional change.


Use LinePaginator from bot-core

This works in conjunction with bot-core#189
It serves as an example of how the new client code consuming bot-core's LinePaginator would look like.

Superseded by #2879


feat(ci): add goimports and makezero

This PR adds new linters goimports and makezero. The current code meets the condition to pass these linters so there is no change here.
In the future, I will try to add more linters to maintain the quality of codes👍

@mnagaa thank you!


Add Time Capsule

Name of the extension you are adding
Time Capsule
Why do you think this extension is awesome?
Time travel is cool!!!
Make sure that:

[x] Screenshot/GIF included (to demonstrate the plugin functionality)

[x] ToC updated

Hi, you've changed the order of ToC, can you please revert it back? The reason I'm asking for this is that ToC should be aligned with the document structure :) We either need to change both or none of them


update dockerfile build to use pipfile.lock, sync black pre-commit ve…

Description
When trying to deploy a new version of this repo to AWS through CodeBuild, it was failing during the build of the Dockerfile due to trying to install newer versions of libraries in our Pipfile.
When we migrated to using pipfile.lock for CircleCI, which uses pipenv sync, we didn't adjust our Dockerfile pipenv install step to use the pipfile.lock instead of using the pipfile that contains wildcards.
I also updated the pre-commit version of black to the one upgraded previously by dependabot.
Reviewer Notes
I'll test this out in AWS with a new Codebuild deploy to verify that the Dockerfile builds properly.
Setup
Login to the AWS console using the dev credentials from the transcom-infrasec-gov-nonato repo.
aws-vault login transcom-gov-dev

In the AWS console, select the CodeBuild service and select the milmove_load_testing project.
Start a new build with overrides and post this branch name or PR as the source to use.
Wait for the build logs to succeed without error.

Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change.
This article explains more about the approach used.

Screenshots

Build succeeded https://console.amazonaws-us-gov.com/codesuite/codebuild/021081706899/projects/milmove_load_testing/build/milmove_load_testing%3A6ecbb05d-55b8-407f-8262-0cc257361b26?region=us-gov-west-1


PR: Update core dependencies for 6.0.0 alpha5

Description of Changes

spyder-kernels to >=3.0.0b5,<3.0.0b6.

Closes #22008
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: dalthviz

Failing check caused due to the job timing out. Merging this to proceed with the release but a check for that needs to be done later


fix lwIP Thread interface address synchronization

Problem
The link local address is not added to the lwip interface when OpenThread has just enabled IP.
Summary of Changes
Modify the lwip interface behavior to check whether Thread has enabled networking rather than whether Thread is attached.

@hawk248 @jelderton @saurabhst ?


Adding using-pass.py in loops directory

1. This pull request resolves #105
2. Description
Creating using-pass.py file inside loops directory. The files content give us introduction to pass statement with an example.
3. Fill in checklist by marking [x]

[x] I've read the CONTRIBUTING.md
[x] I was assigned to this issue
[x] My code is formatted using required linting
[x] I've run my code locally and checked it works

Can you please also add the hacktoberfest label to this?

1. This pull request resolves #105
2. Description
Creating using-pass.py file inside loops directory. The files content give us introduction to pass statement with an example.
3. Fill in checklist by marking [x]

[x]  I've read the CONTRIBUTING.md
[x]  I was assigned to this issue
[x]  My code is formatted using required linting
[x]  I've run my code locally and checked it works

Thanks @deepu14d , I have suggested changes.

Oh sorry
It seem confusing to me too at first.
Can you please help in adding a challenge? I am unable to come with something good n easy (and I am travelling, don't have access to PC. Sorry) I'll try to change challenge ASAP.

I have made a suggestion @deepu14d , tell me what you think

Yeah yeah it's fine, better than mine challenge.
Thanks

Thank you for your contribution @deepu14d 🎉


Flutter Gallery/Data tables demo. Footer cut off at left margin

Build 0.0.23-pre.43, installed via Play Store.
The "Rows per page" text is cut off in portrait mode on a Nexus 5:

This is working as intended.

It might be working as intended, but it's pretty ugly. I was about to report this same bug.
Should we just not show the label if it doesn't fit?  Also, the "calories" column is cut off too: we should probably have a way to show only an integral number of columns, if desired, or at least reduce the width of the first column in the demo to something that will fit on most phones.

Closing as duplicate of https://github.com/flutter/flutter/issues/14595


transitive dependencies order

What will be the order of transitive dependencies displayed?

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

ID: 8400fd6a-d987-1056-283f-9364fbce3c86
Version Independent ID: 3d4b052c-33fa-cc83-4df6-4a3341f12763
Content: dotnet list package command - .NET Core CLI
Content Source: docs/core/tools/dotnet-list-package.md
Product: dotnet-core
Technology: dotnet-cli
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

@nkolev92 Do you know the answer to this question?

What i found was that they are sorted alphabetically and currently did not get hierarchical view. I am not sure if there is some other tooling around to get hierarchical view.

It's always alphabetical.
@VikasAgarwal1984 the tree creation would need to be it's own feature :)
You can file an issue at NuGet/Home for that.


License missing from repo

Without a license, this project isn't open source and no one can use the code.
Please use something like http://choosealicense.com/ to decide what license to use. I recommend MIT or GPL.

Added. Thanks!


Archive this repo with pointer to the new sample

According to the https://github.com/Azure-Samples/storage-blob-resize-function-node-v10/pull/1#issuecomment-788220155,  update README to archive this repo with pointer to the new sample.
@ramya-rao-a @jongio for notification.

Please wait until the PR that adds the sample to azure-sdk-for-js repo is merged to merge this PR

Based on https://github.com/Azure/azure-sdk-for-js/pull/14074#issuecomment-856380020, we have decided to update the sample in this repo to use v12 storage SDKs. So, closing this PR

@ramya-rao-a    We have updated this Readme to point to https://github.com/Azure-Samples/storage-blob-resize-function-node.
Because JosueJoshua has left, so we don't have permission to change the title and description.

Description:
Based on Azure/azure-sdk-for-js#14074 (comment), we have updated this Readme to point to https://github.com/Azure-Samples/storage-blob-resize-function-node.


Add GP model.

Do we want to base this GP model on GPy or MXFusion? GPy has the advantage of not relying on MXNet.

I think it is better to start with the GPy one. We can add later the one based on MXnet as an example of how models and acquisitions can be liked together.


Apim diagnostics rong

added a few screenshots for apim diagnostics

Hi @rzhang628 - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. Adding images and other substantial updates also should be done in the private repo. Would you make this content update in the private repo instead? Thank you! We'll close this PR.
@georgewallace It looks like Rong is a PM who might need information about joining the MicrosoftDocs org and working in the private repo. Could you point Rong to a resource of this information? Thank you.


Timeseries example is a Misnomer. It is NOT a "timeseries" it is rather a simple series.

https://www.tensorflow.org/tutorials/structured_data/time_series#part_2_forecast_a_multivariate_time_series
I do not understand how this can be time series ? The data is "equidistant" with each other. And the time itself is not considered in predicting the values, but rather just as a series. When you include TIME AS A VECTOR, i would accept that it is a Timeseries.
The title is MISLEADING and also we need an example for vectorizing Time... for an actual Timeseries.

I do believe the docs go with following definition  of the term, or close variation on it.

Wow... are you saying that You go with Merriam Webster but not mathematical definition ? They are wrong. IT is a sequential data. If the data is time ordered or indexed only then it can be Time series. And Webster is not a definitive source for mathematical definitions. I will write to Webster about this discrepancy.
More over when "Time" itself is not considered how could this example be "Timeseries" ? IT is that simple actually.
Are we stuck up on the definition or looking to have a future added ? I am confused here.

I'm not sure if either of us misinterpreted the definition.
But I don't see how is what docs describe differs from either Webster or wiki.
The dictionary definition says:

a set of data collected sequentially usually at fixed intervals of time
It's hard not to see the implied sequence of the data points.

The wiki definition on the other hand:

A time series is a series of data points indexed (or listed or graphed) in time order. Most commonly, a time series is a sequence taken at successive equally spaced points in time. Thus it is a sequence of discrete-time data. Examples of time series are heights of ocean tides, counts of sunspots, and the daily closing value of the Dow Jones Industrial Average.

Again sequence, arguably more clearly stated compared to Webster.
Docs example is rather straightforward too, in that it describes:

This tutorial uses a weather time series dataset recorded by the Max Planck Institute for Biogeochemistry.
This dataset contains 14 different features such as air temperature, atmospheric pressure, and humidity. These were collected every 10 minutes, beginning in 2003. For efficiency, you will use only the data collected between 2009 and 2016. This section of the dataset was prepared by François Chollet for his book Deep Learning with Python.

If it's good enough for Chollet, the guy behind Keras, it's good enough for me.
Finally the Statistics Engineering handbook defines time series as:

Definition of Time Series: An ordered sequence of values of a variable at equally spaced time intervals.

Which again conforms with what I seen in provided example.

@summa-code As mentioned in the above comment, the dataset that has been used here is a timeseries dataset. I don't really see a problem in here.

@jpodivin ... I say WOW for the examples of the definition. Here is my point. I am not question about the DATA that is timeseries. But fortunately or unfortunately, they all fall under equi-distant timeseries... which in other words, simply a sequential data. But for the sack of the problem for "Time" series, the solution presented seems to me simply for a sequential data, not taking TIME into consideration.
Anyhow, i am glad that Tensorflow team seems to have updated their Timeseries example. I have not gone through them fully. But i will in the coming days and see if it meets the criteria. Timeseries problems are not perfect, but i like to see some broader approach involving actual time.

@summa-code As gowthamkpr said the data set is fine as it is. And I fail to see any point to prolonging this conversation.
I am glad that you found my examples, which were numerous and sourced, to be sufficient and well sourced.
If you have further issues with the documentation I can only recommend providing your own sources in the first place, it will help your argument since, all things taken into account, the burden of proof does rest on you.

I could not run the latest notebook. Not sure if they have tested these before publishing,
https://github.com/tensorflow/tensorflow/issues/41987
Filed a bug again

And then i could not run on an IoT data that does not fit in memory. These need to be converted to tensor graphs. :-(
So i can only test with simple sample code with this.

@summa-code,
If the Data is huge, you can Create_TF_Dataset. Please let us know if you have any other concerns. Thanks!

@summa-code,
Can you please respond to the above comment. Thanks!

Definitions are context dependent. This usage is consistent with most common usage.
If you have variable timestep data you can just include the time deltas as a feature.
If your data doesn't fit in memory then you'll want to use tf.data.
But anymore implementation questions should probably go on Stack Overflow. Github issues are for tracking problems in TensorFlow.


Better toast colors

Is your feature request related to a problem? Please describe.
Both the success toast and the progress toast are colored green, which is a bit confusing.
Describe the solution you'd like
I'd like if the toast that indicates that an operation is ongoing to be colored differently (yellow). An example of an ongoing operation would be when pushing to remote.
Describe alternatives you've considered
_
Additional context
_

By toast, do you refer to the right side notifications that "pop out like a toast"?
Never heard the expression before, but sounds fitting.

Yea, I guess those messages (smaller than a popup, bigger than a tooltip) are called "toasts" https://daisyui.com/components/toast/

Also, I think it wouldn't be unwise to use nvim-notify in place of toasts, and use the old toast system as backup option.

Toast colors now use the diagnostic highlight groups.


Replace StyleCop.MSBuild

Need to find an alternative as StyleCop.MSBuild does not work well with dotnet core projects.

StyleCop.MSBuild is removed in order to make dotnet build work. Seems there is an open issue https://github.com/StyleCop/StyleCop/issues/126 tracked in StyleCop.MSBuild

@blrchen, please let me know how you fixed/removed StyleCop.MSBuild from the projects... I'm having same issue when I'm trying to build our ASP.Net Core 2.0 projects with "dotnet build" command.

Find a good StyleCop.MSBuild replacement: StyleCop.Analyzers. It works well so far


Segfault RfcUTF8ToSAPUC

OS: Win10
go version go1.10.1 windows/amd64
nwrfcsdk version 7.20 x86/64
the function fillString causes a segfault at line 96 callling RfcUTF8ToSAPUC in the nwrfcsdk. Looking at the code it looks fine for me though.

We had seemingly random segfaults on Windows before. I at the moment cannot propose a solution, but just know that Windows is still experimental because of such problems.

Should be fixed in 0.1.0

I'll try to give feedback within this or next week. While I implemented that in another language by now, it'll be a good test.

The Windows is not supported until #21 fixed

Closing due to the sunset of the project.


[Enhancement] add Plugin 📦 Installation & Upgrade info to readme

We have this already documented on the linked page "Installation".


Usage with CookieConsent

How can I use this with CookieConsent?
The cookies are sent as soon as the page loads and the consent popup is up.
I've tried doing this to set cookie_consent to false when the page loads:

  <script>
      document.cookie = "cookie_consent=false; path=/; max-age=31536000";
  </script>
  <link rel="stylesheet"<EMAIL_ADDRESS>
then set it to true once the consent is granted:
  <script
      type="text/plain"
      data-category="analytics">
      document.cookie = "cookie_consent=true; path=/; max-age=31536000";
  </script>

But the analytics are send no matter if cookie_consent is true or false, or even if it exists:
  umami {
  	event_endpoint "https://example.com/api/send"
  	website_uuid "1234"
  	allowed_extensions "" .html .htm
  	trusted_ip_header X-Forwarded-For
  	cookie_consent cookie_consent
  	cookie_resolution
  	device_detection
  }

Can you try this again with the latest commit https://github.com/jonaharagon/caddy-umami/commit/7dc7b294e297e141978df513cb23870f0fc7ed18?
This release also changes the behavior to only send analytics when the cookie is set to true (as opposed to disabling analytics when the cookie is set to false, as it was previously), which is more in line with privacy expectations and avoids the need to set cookie_consent=false when the page is initially loaded entirely.


Not able to select subset from clustered data which is added externally (k-means)

My issue is...

I read the data into anndata and performed all preprocessed steps and then used data matrix to perform non-linear dimensionality reduction (DR).
I performed DR and k-means clustering.
I added back the data into one of the components (X_tsne)
Added KM labels also.

Now, I am not able to select the clusters based on the cluster.
I want to select one of the clusters and perform clustering on that.
Please find the code snippets below.
mlle_3d_data=pd.read_csv("C:/Users/saite/source/df.csv")
mlle_3dc_data=pd.read_csv("C:/Users/saite/source/dfc.csv")
adata.obsm['X_tsne']=np.asanyarray(mlle_3d_data)
adata.obs['km']=list(mlle_3dc_data['clusters'])
adata.obs['km']=adata.obs['km'].astype('category')
sc.pl.tsne(adata,color=['km'])

I am trying to select based on the cluster number how we do with leiden or louvain clustering, but I am not seeing any data.
Please help me.

Hi @saiteja-danda,
I cannot reproduce your code above as I don't have your data. Could you try to generate a minimal reproducible example with data from e.g., sc.datasets.pbmc68k_reduced()?
In general, could you check the output of adata.obs['km'].value_counts() to check whether the covariate you added was correctly stored? What output are you getting? You didn't share an error message.


Updating breakpoints in a nested debugger or in a job debugger results in breakpoints with the same id in the parent being overwritten

This issue is reproducible in PS 5.1 and 6.2.1 (and presumably 7 preview since the buggy code is still there, although I didn't test that version specifically yet).
If you debug a job, you enter a nested debugging session. From within that session, you can set, enable, disable, or remove breakpoints in the job debugger. Setting or removing breakpoints will result in the same changes made to any breakpoints with the corresponding IDs that were set in the parent debugger.
Steps to reproduce
# Create a job that does some stuff
$job = Start-Job -ScriptBlock {
    1..1000 | ForEach-Object {
        Start-Sleep -Seconds 1
        $_
        Write-Error 'boo'
        Write-Verbose 'Verbose' -Verbose
        $DebugPreference = 'Continue'
        Write-Debug 'Debug'
        Write-Warning 'Warning'
    }
}

# Set some local breakpoints, and remove one
$bp1 = Set-PSBreakpoint -Command Write-Verbose
$bp2 = Set-PSBreakpoint -Command Get-Date
$bp3 = Set-PSBreakpoint -Variable Home -Mode Read -Action {Write-Host 'Read var'}
Remove-PSBreakpoint -Breakpoint $bp2

# Have a look at your local breakpoints (these do not impact the running job).
Get-PSBreakpoint

# Wait for the job and its child job to enter a running state
while ($job.State -ne 'Running' -or $job.ChildJobs[0].State -ne 'Running') {
    Start-Sleep -Milliseconds 250
}

# Debug the job
Debug-Job -Job $job

At this point you will be in the job debugger.  Have a look at your breakpoints by invoking the following command:
Get-PSBreakpoint

This output shows that:

breakpoint 0 was overwritten with a completely different breakpoint (the first one that was set in the job)
breakpoint 1 was re-created (remember that it was removed before debugging the job), and it's not the same as it was before (note that it is not disabled in the parent even though we disabled it in the job debugger because the code behind that simply disables the breakpoint object again, which is already disabled).
breakpoint 2 was deleted (remember that it was removed from within the job debugging session)

Expected behavior
 ID Script                            Line Command                           Variable                          Action
  -- ------                            ---- -------                           --------                          ------
   0                                        Write-Verbose
   2                                                                          Home                              Write-Host 'Read var'

Actual behavior
  ID Script                            Line Command                           Variable                          Action
  -- ------                            ---- -------                           --------                          ------
   0                                                                          DebugPreference
   1                                        Write-Debug

Environment data
Name                           Value
----                           -----
PSVersion                      6.2.1
PSEdition                      Core
GitCommitId                    6.2.1
OS                             Microsoft Windows 10.0.17763
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

FYI, I'm already working on this issue.
I just wanted to log it separately so that it could be tracked and discovered via search, because it's a bug that's bound to surprise someone at some point and I want them to be able to find this issue.

I could use some additional eyes on the root cause behind this issue, because what is done in PowerShell right now is off, and I can fix it, but I don't have the history behind it.
The code in question is in debugger.cs, on lines 3227 to 3247.
That event handler is invoked in response to a breakpoint being updated in the local debugger or in any nested debugger. This includes the PSTaskChildDebugger, RemotingJobDebugger,  RemoteDebugger, ServerRemoteDebugger, and NestedRunspaceDebugger. That's why modifications to breakpoints in a remote runspace (in the example above, in a job) result in breakpoints changing in the local debugger. The local debugger HandleBreakpointUpdated event receives details about the breakpoint that was changed in another runspace, and then tries to make the same changes in the local debugger.
This behavior seems clearly wrong to me, and I'd like to fix it. My initial thought is: that event handler doesn't appear to be needed at all, so remove it. If I create, enable, disable, or remove a breakpoint in a local debugger, the breakpoint is created, enabled, disabled or removed via direct API invocation. If I do the same in a remote debugger, the same happens. Why then do we have an event handler that is notified when a breakpoint is updated in a remote session in PowerShell itself? The only logical reason I could come up with was to synchronize breakpoints across all runspaces, but we don't do that -- we manage breakpoints separately in each runspace.
At any rate, @PaulHigin, if you could have a look at this and share your thoughts, it would be appreciated.

This definitely doesn't sound right.  I haven't looked at this in a long while, but much of it was added to support workflow script debugging, and I am not surprised it could use some re-factoring.  I'll try to look at this early next week and give you my thoughts.

Hey @PaulHigin, just curious, did you find time to have a look at this yet?
I'm currently working on a PR that adds a -Runspace parameter to the PSBreakpoint cmdlets. It may be appropriate to fix this issue as part of that PR, which is why I ask.
It's not a hard requirement though, since this issue can be addressed separately as well, so no real rush.

Sorry for the late reply.  It looks like this is just a coding mistake.  The parent debugger is handling 'BreakpointUpdated' events from a nested debugger and updating its breakpoints.  I can't think why this would ever be needed or correct, and my guess is that it was added for 'completeness'.
I think this breakpoint update handler can be completely removed.
https://github.com/PowerShell/PowerShell/blob/139cd942b23b18c3220cb57ea2564e6f154c8fef/src/System.Management.Automation/engine/debugger/debugger.cs#L3311
I am fine doing this in one of your debugging PRs.  Thanks!

That's the exact same outcome that I came up with as well. Thank you @PaulHigin for confirming you feel the same. I'll submit a PR for that shortly to get this corrected.


By Use EncryptDataSource the query with text column occurs error

Bug Report
For English only, other languages will not accept.
Before report a bug, make sure you have:

Searched open and closed GitHub issues.
Read documentation: ShardingSphere Doc.

Please pay attention on issues you submitted, because we maybe need more details.
If no response more than 7 days and we cannot reproduce it on current information, we will close it.
Please answer these questions before submitting your issue. Thanks!
Which version of ShardingSphere did you use?
4.0.0-RC2
Which project did you use? Sharding-JDBC or Sharding-Proxy?
Sharding-JDBC
Expected behavior
By Use EncryptDataSource the query with text column can return correct result
Actual behavior
By Use EncryptDataSource the query with text column occurs error
Reason analyze (If you can)
By Use EncryptDataSource the query with text column is not convert correctly
Steps to reproduce the behavior, such as: SQL to execute, sharding rule configuration, when exception occur etc.

Example codes for reproduce this issue (such as a github link).

this is my ddl sql ，with text column it occurs error

Can you give me an github example repository to find this issue's reason？

@Keanhua Hi, to make your issue recurred, i did some tests, but i can not get your exception.
Here some detail of my test, is it similar with yours?
Table description:
CREATE TABLE `t_encrypt` (
  `id` int(11) NOT NULL,
  `pwd` text,
  `pwd_query` varchar(255) DEFAULT NULL,
  `name` text,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8

Encrypt rule
encryptRule:
  encryptors:
    encryptor_aes:
      type: AES
      props:
        aes.key.value: 1111
  tables:
    t_encrypt:
      columns:
        name:
          cipherColumn: name
          encryptor: encryptor_aes

SQL

private static final String SELECT_SQL = "select pwd, name from t_encrypt ";

@Test
    public void testSelect() throws Exception {
        try (Connection connection = dataSource.getConnection();
             Statement statement = connection.createStatement()) {
            ResultSet resultSet = statement.executeQuery(SELECT_SQL);
            while (resultSet.next()) {
                System.out.println(resultSet.getString(1));
                System.out.println(resultSet.getString(2));
            }
        }
    }

@sunbufu @tristaZero
this is the demo link:https://github.com/Keanhua/sharding-bug-demo
you can run the testCase directly, the error will occured。the sql is also in it.
if you change the longText cloumn type to json, it also occured

thanks for your demo, which gives us more chances to find the bug.
@Keanhua

@sunbufu Hi, are you interested in this issue? If it is available, the demo would be a good start. :)

@tristaZero Yes, I am interested in this. @Keanhua Thank you, I will try your demo later.

The reason caused this problem is ShardingResultSet/EncryptResultSet get value accroding to the metadata. So we need to get value accroding to target type in StreamQueryResult.
But for MemoryQueryResult, ShardingSphere need to get value before users call getXXX method, so maybe transfer the value to target type in ShardingResultSet/EncryptResultSet.


Forced element nesting thrown when using multi-selector

What version of Sass Lint are you using?
1.8.0
Please include any relevant parts of your configuration
force-element-nesting: 1
What did you do? Please include the actual source code causing the issue.
Used a multi-selector where an element has to be both .button and .primary
.button.primary {
background-color: $color-button-primary;
font-weight: 600;
:hover {
background-color: transparent;
}
}
What did you expect to happen?
No error to be thrown.
What actually happened? Please include any error messages given to you by Sass Lint.
force-element-nesting: Class should be nested within its parent Class
The warning goes away in two conditions:

force-element-nesting: 0
nest .primary within .button - however this is not a workable solution since it doesn't generate the style I'm looking to produce -- it creates a selector ".button .primary", which is a very different thing -> it looks for .primary within .button.

If you're using a IDE plugin have you tried the CLI too?
I'm using in Atom via linter-sass-lint 1.6.1 which uses sass-lint 1.8.0 - have not tried in the command line.

More about multi-selectors if it's helpful. https://css-tricks.com/multiple-class-id-selectors/

That's CSS, for SASS this is what you should be doing, and the reason you get the warning:
.button {
  &.primary {
    background-color: $color-button-primary;
    font-weight: 600;

    :hover {
      background-color: transparent;
    }
  }
}

That worked!


Workers sync send type checking issues

Description:
$title.
The spec says
"With sync-send-action, evaluation waits until the receiving worker either executes a receive action that receives the queued message or terminates. The evaluation of sync-send-action completes as follows:

if the queued message was received, then normally with result nil;
otherwise

if the receiving worker terminated with failure, then normally with the result being the termination value of the receiving worker, which will be an error;
if the receiving worker terminated abnormally, then abruptly with a panic, with the associated value being the termination value of the receiving worker.

The static type of the sync-send-action is F|() where F is the failure type of the corresponding receive action. If F is empty, then this static type will be equivalent to ()."
But there seem to be type checking issues for sync send, see below.
Steps to reproduce:
import ballerina/io;

public function main() {
    worker w1 {
        int i = 100;
        () v = i ->> w2; // this is allowed!
        io:println("v: ", v);

        any|error a = v;
        io:println("a is error: ", a is error); // evaluates to true even though type of `v` is nil
    }

    worker w2 returns error? {
        int lw;
        if (true) {
            return error("err reason");
        }
        lw = <- w1;
        io:println("[w2 <- w1] lw: ", lw);

    }

    wait w1;
}

$ ballerina run test.bal
v: error err reason
a is error: true

Affected Versions:
$ ballerina -v
Ballerina 1.0.0-alpha3
Language specification 2019R2
Ballerina tool 1.0.0

Related issue: https://github.com/ballerina-platform/ballerina-lang/issues/18160


Include example Jenkinsfiles that can be installed

Dupe #16 - using that since it's tracked in a project.


"Toggable" Component

Currently you have to mess around/make components to make an item change its sprite depending if its On/Off. This behviour should be a component itself.

As discussed on Discord, in #howdoicode the 29 April 2020 around 7:00 AM UTC (sorry, I don't know how to link), it is very hard to have a clearly defined, generic and useful behavior for a Toggleable component.
For instance, the Welder component cannot be toggled on if there is not enough fuel. Implemeting it with a Toggleable component would require some work to make the two components (Toggleable and Welder) communicate with each others, which is not worth compared to just implement IUse for now.

@Hugal31 you can right-click a message in discord to get a link to it.
Your question, for example:
https://discordapp.com/channels/310555209753690112/560845886263918612/704955322019610645

I have come to the conclusion that this is not a good idea and that there are better ways to implement this behavior


README: Model 225 Link fix

Change link for 225 to
https://colab.research.google.com/github/openvinotoolkit/openvino_notebooks/blob/main/notebooks/225-stable-diffusion-text-to-image/225-stable-diffusion-text-to-image.ipynb
from
https://colab.research.google.com/github/openvinotoolkit/openvino_notebooks/blob/main/notebooks/224-3D-segmentation-point-clouds/224-3D-segmentation-point-clouds.ipynb
Signed-off-by: sounddrill31<EMAIL_ADDRESS>

thank you for the fix, but unfortunately colab does not provide enough RAM for running stable diffusion on cpu as I know, so possibly better just remove link


Weapon Customization not working

Here's a recording of what's happening.

I'm having the exact same problem tried everything cant figure it out

I accidentally uploaded a release which didn't come with the Customizer Overrides, please download the zip file from the release page and put the GoonModWeaponCustomizer in assets/mod_overrides.

[Mods] Could not read or find mod.txt for modification: GoonModWeaponCustomizer
[Error] Could not load revision file for GoonMod Weapon Customizer!

It's still not working.


@items.visibility throws errors in config schema file

Expected Behavior
Previously our plugin had a config.d.ts file as below
export interface Config {
  'mrt-plugin': {
    /** @items.visibility frontend */
    superusers: string[];
  };
}

This worked fine but updating to release-2021-05-27 seems to break the validation with following error
$ backstage-cli app:build
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
lerna ERR! yarn run build stderr:
strict mode: "items" is 1-tuple, but minItems or maxItems/additionalItems are not specified or different
Error: Schema at <plugin-path>/config.d.ts is invalid, Error: strict mode: unknown keyword: "text"

Possible Solution
Based on @freben's suggestion on discord this patch might need to be extended to items.visibility as well
https://github.com/backstage/backstage/commit/438a512ebb91228b53d591cca8900973163404da
Happy to put in a fix if this is indeed the correct solution

Would you like to be assigned and give it a shot? It'd be great if you could, since you already have the error repro present

@Rugvip

Thanks @Rugvip


customRowRender

I have a use-case in my project where I need a non-column based layout for smaller screens. This is to prevent horizontal scrolling, which is not a very good UX on mobile.
I still wanted to make use of the (great) filtering and sorting system of mui-datatables.
This PR adds a new customRowRender option to mui-datatables, that can be used like this:
customRowRender: ({ data }, rowIndex) => (
  <div>
    My Important data field: {data[0]}
  </div>
)

Coverage decreased (-0.06%) to 74.61% when pulling 51922a21884c25ce837d7697060fca672e83555c on Skn0tt:master into d6ec8a3b1da48bbefda60f9112a590942bc06bf5 on gregnb:master.

@Skn0tt Have you had a look at the responsize: 'stacked' option that provides a non-column based layout already?

I did, but it didn't match the asthethics I was looking for.

@gabrielliwerant so I don't think I'm fully against adding this maybe it could be useful to someone? What are your thoughts

It's interesting. I'm on the fence.
I played around with it a little bit, and it seems difficult to maintain a consistent experience with the other table options like column hiding, selecting rows, filtering, etc. It makes it easy to miss implementing data that belongs in the table, which will impact what happens when users use other toolbar options.
If it's used to completely change the nature of a data table, then it seems like it's out of the scope of a data table library. Should it be possible to completely change the user experience? Perhaps. Should it be easy? I'm less sure about that one.
On the other hand, there are already a lot custom rendering options, so what's the harm in one more? I guess there's something that seems more fundamental about rows to me.
What would be convincing to me is an example that demonstrates usage of this feature to do something table-like that isn't possible yet in some other way. Perhaps the responsive issue alluded to earlier would make a good example.
Another way to go about solving the problem might be by providing additional CSS classes so the theme can be overridden even more easily to provide different experiences/skins.

What would be convincing to me is an example that demonstrates usage of this feature to do something table-like that isn't possible yet in some other way. Perhaps the responsive issue alluded to earlier would make a good example. @Skn0tt

I implemented a small example so you can see what I mean.
You can find the code and an explanation why I think this is useful at skn0tt/mui-datatables-responsive-example.
You can see the example online at https://mui-datatables-responsive-demo.skn0tt.now.sh.
I hope you like it! 😄

@Skn0tt Thank you so much for creating the example! That is pretty cool, I can see how that would be useful. Would you be able to add an example to the PR?

I just added the example. I will make sure to extend the typescript definitions at DefinitelyTyped once this is merged.

This looks good to me, thanks for your hard work on this! Pending final approval from @gregnb.

This is very cool, but I'm having some updating issues
first screen shot below you can see a customRowRender is defined and the toggle switch to expand descriptions is checked

but when I toggle the switch to off, dev-tools tells me there is no customRowRender defined for the table, but it does not rerender with the default row layout.

I've checked the example (the credit card example is super cool) and I think I'm handling the switch correctly.
Any thoughts on what might be going on?

If you look closely at the example, you'll see that the key of the DataTable is changed depending on wether customRowRender is used or not. This forces MuiDataTable to remount everytime customRowRender is changed. That's the mechanism I use myself for ensuring the updates are coming through, but this has a downside: On resizing, the table forgets all its state. For me, this works, but I can see how this is suboptimal (at best) for other usecases.

Thank you, @Skn0tt  for the immediate response! And thanks for the contribution. It's really cool!

You're welcome. Is better updating something you'd be interested in? I can have a look at it tomorrow and see what I can do.

I already keep most of the table state in the parent component / localStorage so that settings are sticky.

@Skn0tt one thing that would be cool however would be to inject a flag into tableState so if you're using customBodyRender for a column in default row rendering mode, you can return the data in a different format for customRowRender.
In my case, I'm using customRowRender to show a card style layout if the user wants to expand the description field in the table. If they don't want it expanded, I use customBodyRender to render a clickable more info icon that displays the description in a modal. But if I'm using customRowRender I want to just return the raw description.
customBodyRender gets passed the tableState so if I knew from there that I was customRowRender'ing, I'd just return the raw text.
Does it make sense?