tests: Fix, script to help run single tests

CATEGORY

[X] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
If the developer did not have an already test database created the script would just fail
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS

Codecov Report

Merging #9779 into master will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #9779      +/-   ##
==========================================
- Coverage   70.78%   70.70%   -0.08%
==========================================
  Files         587      183     -404
  Lines       30460    17851   -12609
  Branches     3121        0    -3121
==========================================
- Hits        21561    12622    -8939
+ Misses       8785     5229    -3556
+ Partials      114        0     -114

Flag
Coverage Δ

#cypress
?

#javascript
?

#python
70.70% <ø> (-0.23%)
:arrow_down:

Impacted Files
Coverage Δ

superset/views/database/mixins.py
58.92% <0.00%> (-21.43%)
:arrow_down:

superset/utils/cache.py
45.83% <0.00%> (-20.84%)
:arrow_down:

superset/db_engine_specs/mysql.py
78.26% <0.00%> (-13.05%)
:arrow_down:

superset/views/database/api.py
83.90% <0.00%> (-3.45%)
:arrow_down:

superset/db_engine_specs/postgres.py
97.29% <0.00%> (-2.71%)
:arrow_down:

superset/models/core.py
83.77% <0.00%> (-2.36%)
:arrow_down:

superset/jinja_context.py
84.69% <0.00%> (-1.03%)
:arrow_down:

superset/security/manager.py
88.77% <0.00%> (-0.36%)
:arrow_down:

superset/views/core.py
74.90% <0.00%> (-0.23%)
:arrow_down:

superset/connectors/sqla/models.py
88.41% <0.00%> (-0.16%)
:arrow_down:

... and 404 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 42b10ae...755a8a2. Read the comment docs.


Is the calculation of loss influenced by I_{HQ} during the training of LAControlNet?

I_{HQ} only works during the SwinIR training?  I couldn't locate any involvement of I_{HQ} in the LAControlNet training code. Whether "|I_{HQ} - I_{reg}|" is computed as part of the LAControlNet training process? LAControlNet only need to I_{reg} as input? If I intend to directly train LAControlNet using I_{HQ} and I_{reg}, will {I_HQ} be incorporated in the process?

Hello! $I_{HQ}$ will be used in both stage1 (SwinIR) and stage2 (LAControlNet) training. LAControlNet requires two inputs: the noisy latent $z_t$ and the condition latent $\epsilon(I_{reg})$, $\epsilon$ denotes the pretrained VAE encoder. During training, we will first add noise to $\epsilon(I_{HQ})$ to get the noisy latent $z_t$ and then use LAControlNet to predict the noise. You can read the following code and have a better understanding of the training process:

training_step and shared_step: The class ControlLDM inherits these two methods from base class DDPM and LatentDiffusion, which receive batch data from dataloader and pass it to the method get_input. The batch data contains both hq images and the generated lq images.
get_input: The  get_input method of ControlLDM is used to preprocess the batch data. It first passes batch data to the get_input method of LatentDiffusion to encode hq images  $I_{HQ}$ to latent space (denoted as $\epsilon(I_{HQ})$ ), and then prepare condition latent $\epsilon(I_{reg})$ for LAControlNet.
forward: The forward method is called by shared_step, it first samples a timestep t and then call the p_losses method to compute loss.
p_losses: This method add noise to $\epsilon(I_{HQ})$ to get the noisy latent $z_t$ through the q_sample method and then call the method apply_model, which uses LAControlNet to pedict noise.

Due to the intricate inheritance relationships, the code related to LDM is difficult to understand. Please feel free to contact us if you have any further questions! 😄

Thank you so much for your patient response; it has been incredibly helpful to me.


Publish sources to MavenCentral

When downloading sources from Android Studio I get this error, mainly caused by missing AAR sources published on Maven Central:
Executing tasks: [:app:DownloadSources] in project C:\Users\Maicol\StudioProjects\GameLibrary\app

> Configure project :app
Unzip jar to C:\Users\Maicol\StudioProjects\GameLibrary\app\build/hidden-secrets-tmp

> Task :app:DownloadSources FAILED

FAILURE: Build failed with an exception.

* Where:
Initialization script 'C:\Users\Maicol\AppData\Local\Temp\ijmiscinit2.gradle' line: 25

* What went wrong:
Execution failed for task ':app:DownloadSources'.
> Could not resolve all files for configuration ':app:downloadSources_d046520a-a9de-4a30-8b74-240da14905f9'.
   > Could not find ru.pixnews.igdbclient:igdbclient-core-android:0.1@aar.
     Searched in the following locations:
       - https://dl.google.com/dl/android/maven2/ru/pixnews/igdbclient/igdbclient-core-android/0.1<EMAIL_ADDRESS>       - https://repo.maven.apache.org/maven2/ru/pixnews/igdbclient/igdbclient-core-android/0.1<EMAIL_ADDRESS>       - https://jitpack.io/ru/pixnews/igdbclient/igdbclient-core-android/0.1<EMAIL_ADDRESS>     Required by:
         project :app

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org.

BUILD FAILED in 984ms
1 actionable task: 1 executed

The source code is actually published: https://repo.maven.apache.org/maven2/ru/pixnews/igdbclient/igdbclient-core-android/0.1/igdbclient-core-android-0.1-sources.jar.
I think this is a bug in AS and not sure if we can do something. There are some known issues with the "download source code" button.  Similar report: https://issuetracker.google.com/issues/247991770
Try to run "Sync Project with Gradle Files" or toggle the "Download external annotations for dependencies" checkbox in Graddle Settings. Sometimes these actions help AS to download the source code correctly.

No, they didn't help. I guess we have to wait for that issue to be fixed!


latest-per-pkg index overview

I would like to request a new feature for URL/latest-per-pkg,
I would like to see there a index.html file with a search field for ports name like we having already in /build/index.html, but in additional it would be great to get here latest-per-pkg//version>/ a index.html what shows which arch have been build fine and which one have been failed.

I do frequently wish for this (or something like this).


Issue with text when string has new line character

Hi,
I have text for label as (A \n B \n C) and i have number of line is 2 for collapsed state. It is showing just ...More for label

Same here, string with ne lines characters completly breaks ExpandedLabels

Facing the same issue. If the text does not contain any of the spaces then even ...more is showed only, not the entered text. Is it fixed?


Timeout unit tests fix

Flushes $timeout when running unit tests

@leocaseiro this should take care of almost unit tests (except for the one created for your ng-if fix)

@kfarst you are a legend. I'll add you as contributor. No longer need wait for my approval.
PS: I wrapped my expect into a $timeout for the ng-if test and worked it.

@leocaseiro awesome, thank you sir!


Deadkeys not working

Instead I get a sequence of accent + base letter. Tested with latest beta (2.99.0) on a Samsung 9" pad.

@albbas can you test this on an Android phone? Use the Divvun Dev Keyboards app.

@albbas :
0.2.3 dev - Still a problem
0.2.1 - Works

Is still still an issue with 0.2.4 of divvun-dev-keyboard?

Still a problem on 0.2.4 on my phone

Dev notes:
Dead key transforms weren't exported for the android builds in kbdgen. They are now in this branch, https://github.com/divvun/kbdgen/commit/e692d14f5ae0bedf105946d3e1bd4bcbd220a391
However.
The latin:deadKey="True" params needs to be set on the dead keys for each language. That's done in kbdgen.. Or rather, it should be done in kbdgen.
app/src/main/res/xml/rowkeys_tsuutxina1.xml
<merge xmlns:latin="http://schemas.android.com/apk/res-auto">
    <switch>
        <case latin:keyboardLayoutSetElement="alphabetManualShifted|alphabetShiftLocked|alphabetShiftLockShifted">
            <Key latin:keySpec="ʼ" latin:keyHintLabel="1" latin:additionalMoreKeys="1" latin:moreKeys="q,Q" />
            <Key latin:keySpec="W" latin:keyHintLabel="2" latin:additionalMoreKeys="2" />
            <Key latin:keySpec="E"
                latin:keyHintLabel="3"
                latin:additionalMoreKeys="3"
                latin:moreKeys="Ë,É,È,Ê,Ẽ,Ē,Ẹ,Ė,Ǝ" />
...
            <Key latin:keySpec="¯" latin:deadKey="True" />
        </case>
...

Run the kbdgen and then manually add the deadKey param for the "¯" key, and with the new transforms, the dead keys work.
Note!
When combining the the dead key with non specified characters, such as s - the keyboard crashes. Not sure how this used to be handled. Seeing as how the program explodes, I'm inclined to believe the kbdgen should be generating something more that it isn't right now.

This is where the crash occurs if you combine ¯ with (for instance) s
https://github.com/divvun/giellakbd-android/blob/b4615f45796d670cb4d960c3d717b3845191290d/app/src/main/java/com/android/inputmethod/event/SoftDeadKeyCombiner.kt#L87

@zoomix:

When combining the the dead key with non specified characters, such as s - the keyboard crashes. Not sure how this used to be handled. Seeing as how the program explodes, I'm inclined to believe the kbdgen should be generating something more that it isn't right now.

The expected output is:

the deadkey, as if a space was typed after the deadkey (ie the output specified in the yaml file for space completion of deadkey combos)
followed by the unspecified key, s in this case

@zoomix deadkey data should be read from a json file, cf
https://github.com/divvun/giellakbd-android/blob/main/app/src/main/java/no/divvun/domain/Descriptor.kt
Not sure how the JSON file is handled in the Rust version of kbdgen, but here is the old Python code for generating the layout:
https://github.com/divvun/kbdgen-archived/blob/main/pysrc/kbdgen/gen/android.py#L269-L295
It is possible that deadkeys never were implemented correctly since the code is using transforms but that's at least a first step: see if the dead keys themselves actually work.

Alrighty. Thanks!
I've published a new version of dev-divvun - 0.2.5. Looks like it's working to me.
Two things I noticed.

The '¯' key in Tsuutʼina keyboard is misconfigured somehow for android. The dead-key doesn't match the keyboard key (or something. I'm not sure). The other dead keys work. I'll have a look, but I'd appreciate if you had any intuition about what could be the thing there.
It's great that you added the docs about nesting of dead keys. Now the structure makes more sense. However, as far as I can see in kbdgen this isn't generally implemented. As in - running kbdgen for android/mac/klc with deeper nestings.

I think I found the Tsuutʼina bug. 0.2.6 is building.

This is fixed.


Light mode designing for testimonials section with responsivity

Is your feature request related to a problem? Please describe.
Light mode designing is not done for the Testimonials section.
Describe the solution you'd like
I will design the testimonials page for light mode.

@aakankshabhende @swati2603 @shivshankar9 Please assign this issue to me after reviewing #193

@aakankshabhende @swati2603 @shivshankar9 Please check PR #196


Exec tachikoma update<PHONE_NUMBER>2103

:hamster::hamster::hamster:   Strategy bundler
Powered by Bundle Update as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

Changes Unknown when pulling e6078976b58ca7772a081864ca525c25b52d8b99 on pazjacket:tachikoma/update-20170726092103 into ** on Protom512:master**.


Dump operation of 'NAME ARCHIVE failed: unknown shorthand flag: 'a'in -a

When I try to view an archive or extract it into a zip always the same error Dump operation of 'NAME ARCHIVE failed: unknown shorthand flag: 'a'in -a
If I extract it it just works.
I use Linux Mint 20.3 Cinnamon.

Which version of restic do you have installed? Have you tried upgrading it?

Which version of restic do you have installed? Have you tried upgrading it?
I have updated restic and it already works. Thanks greetings

I have updated restic and it already works. Thanks greetings


server: use leader lease to determine tso service validity

STATUS: It seems to be better merged after https://github.com/pingcap/pd/pull/1668. You can review it now and give comments.
What problem does this PR solve?
Currently, the safety of timestamp allocation relies on the leader flag be updated in time when lease timeout.
There is a small possibility (for instance, process paused, runtime schedule slow) that the watch channel does not notify in time, which may cause 2 PD servers serve timestamp at the same time. Then the transaction may become corrupted.
Another issue is when a PD server becomes leader, it won't serve timestamps until the leader flag is set. But the leader flag is set after loading all regions from storage, which may take considerably long time for a large cluster.
This PR fixes #1661 and part of #1658
What is changed and how it works?
Introduce LeaderLease to periodically renew lease and update expire time.
Timestamp service only relies on the lease expire time.
Note that if 2 time.Times both have monotonic part, comparison of them will be evaluated by monotonic time, which is not affected by wall clock changes.
Check List
Tests

Unit test

Side effects

Increased code complexity
May introduce bugs

Related changes

Need to be included in the release notes

Codecov Report

Merging #1676 into master will decrease coverage by 0.03%.
The diff coverage is 89.02%.

@@            Coverage Diff             @@
##           master    #1676      +/-   ##
==========================================
- Coverage    76.3%   76.26%   -0.04%
==========================================
  Files         157      158       +1
  Lines       15347    15402      +55
==========================================
+ Hits        11710    11747      +37
- Misses       2644     2654      +10
- Partials      993     1001       +8

Impacted Files
Coverage Δ

server/grpc_service.go
58.87% <0%> (+0.43%)
:arrow_up:

server/tso/tso.go
83.48% <50%> (+4.24%)
:arrow_up:

server/leader.go
79.14% <87.5%> (-3.16%)
:arrow_down:

server/lease.go
95% <95%> (ø)

pkg/etcdutil/etcdutil.go
76.81% <0%> (-5.8%)
:arrow_down:

server/kv/etcd_kv.go
79.22% <0%> (-2.6%)
:arrow_down:

client/client.go
65.7% <0%> (-2.48%)
:arrow_down:

server/core/storage.go
76.08% <0%> (-1.45%)
:arrow_down:

server/cluster.go
83.72% <0%> (-0.24%)
:arrow_down:

... 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 1e0d1bf...001e082. Read the comment docs.

/test

Please take another look. @rleungx @nolouch

/test

/run-test

/build

Created an issue for the test failure: https://github.com/pingcap/pd/issues/1693

@disksing should we cherry pick to 3.0,3.1?

@nolouch I think no.


Change license approval to integrate YF in T2T

@JianGoForIt  as i said in different issues i was trying to adapt YF to be usable in tensor2tensor and after my PR to definitively integrate YF in T2T, it raised a license problem. Once the PR is accepted it will override your MIT License, so the T2T authors need your OK(approval) to keep the PR, otherwise we cannot use your code. This is the PR.

Hi @ReDeiPirati
Please go ahead and override the license. I have checked it up with all collaborators and we are all happy with it.
Cheers


No option in in Login Handlers

I can't find the place where I would set my apikey under login handlers.

There should be an option in login handlers for steam (top image) then click the pencil button to edit and add api key.
Make sure upload folder is uploaded properly, and xml file has been installed via plugin install.

... what folder @madman32395? I can't exactly upload folders as I use the cloud version of IPS.

ah, have you had IPS support do the file upload for you yet?

They'll do that? Sweet! Alright, problem solved if they will. Thanks.

yea, just tell them the plugin you want installed, they upload the upload folder... then you just install the .xml  (you have to download the plugin from either here or IPS MP).
Once uploaded and you have more issues, just let me know.

assuming installed per #87


Storage Service doesn't handle deleted containers

Affected module
Backend
Describe the bug
When deleting an S3 bucket (container) and running S3 Storage Service ingestion, the deleted container is not removed from OM.
To Reproduce
Delete an S3 bucket which was already ingested in OM, then delete the bucket from S3, then re-ingest S3 containers in OM. The deleted S3 bucket is still present as a container in OM.
Expected behavior
OM should reflect the fact that the S3 bucket was deleted and also delete it.
Additional context
The fix most likely involves something similar with how tables are handled, in the database_service, as a post_process step:
https://github.com/open-metadata/OpenMetadata/blob/main/ingestion/src/metadata/ingestion/source/database/database_service.py#L139

@cristiancalugaru will you be able to contribute this?

@harshach yes, please assign it to me.


Retrofit, apache & support library

The support design library has been updated. The current library is using a new rev number which means that the compile sdk version also needs to be updated.
if the compile version gets an update the old apache library won't be able to get compiled anymore. Retrofit was using it for making requests.if we update retrofit we must use another library which supports this.
when we upgrade to the newest version I do not how to get the response codes.
I am taking about retrofit 2

Retrofit 2 is still in beta. Do we want to move to a beta library? However, I'm seeing version 1.9.0 in the Gradle file, which doesn't look like an actual stable release.
I'm fine with getting OkHttp in place of the Apache Commons HTTP library, it's a good move anyway.

Retrofit doesn't really matter for now but apache does because I want to update the support design library.
Once retrofit 2 will be released I want to make sure everything works fine. No surprises that we need to adjust more things or add more libraries.


Create logo & favicon

Make pretty plz

Love the logo! Can I get a favicon in the htpc/client/public folder please?


IoC example

I am trying to use the worker service to delegate a few long running tasks in a background thread after a http response is returned, however i have a major issue with initialization.
What i am trying to do is create a background worker of a service type and that service has multiple dependencies that would usually be retrieved from the IServiceProvider. However i cannot understand how to inject the IServiceProvider in any way using the usual dependency injection container.

I'll try to summarize the example code more or less in the order I would code them.
So in addition to your service you need a "startup service". Instead of
You have the example here:
https://github.com/Tewr/BlazorWorker/blob/main/src/BlazorWorker.Demo.IoCExample/MyServiceStartup.cs
This is where you would call your usual extension methods like in asp.net Startup.ConfigureServices()
Your service (in the example, https://github.com/Tewr/BlazorWorker/blob/main/src/BlazorWorker.Demo.IoCExample/MyIocService.cs), should just declare its dependencies as usual with IServiceProvider and it's friends. In the example it is using the constructor to reference its dependencies, like IMyServiceDependency
finally, the setup from your service proxy, the most simple is the one-liner described in https://github.com/Tewr/BlazorWorker/blob/main/src/BlazorWorker.Demo/SharedPages/Pages/IoCExamplePage.razor :
myIocService = await worker.CreateBackgroundServiceUsingFactoryAsync<MyServiceStartup, MyIocService>(startup => startup.Resolve<MyIocService>());
you can do this in two steps like shown in the example if you have several different service proxys to instantiate (it will be more efficient to reuse a startup service).
Does that help you?


dataVolumeTemplates are broken

Is this a BUG REPORT:

Uncomment only one, leave it on its own line:

/kind bug

/kind enhancement

What happened:
Getting the following errors when deploying VM with dataVolumeTemplates
kubectl create -f vm5-ubuntu.yaml
service/vm5-ubuntu unchanged
Error from server (Invalid): error when creating "vm5-ubuntu.yaml": admission webhook "virtualmachine-validator.kubevirt.io" denied the request: spec.dataVolumeTemplates.spec.resizeTo in body is a forbidden property, spec.dataVolumeTemplates.status in body is required

vm5-ubuntu.yaml
---
apiVersion: kubevirt.io/v1alpha2
kind: VirtualMachine
metadata:
  name: vm5-ubuntu
spec:
  running: true
  template:
    metadata:
      labels:
        guest: vm5-ubuntu
        kubevirt.io/size: medium
    spec:
      domain:
        devices:
          disks:
            - name: vm5-ubuntu-datavolume1
              volumeName: vm5-ubuntu-datavolume1
              disk:
                bus: virtio
            - name: cloudinitdisk
              volumeName: cloudinitvolume
              disk:
                bus: virtio
      volumes:
        - dataVolume:
            name: vm5-ubuntu-datavolume1
          name: vm5-ubuntu-datavolume1
        - name: cloudinitvolume
          cloudInitNoCloud:
            userData: |-
              #cloud-config
              password: root
              chpasswd: { expire: False }
              ssh_pwauth: True
  dataVolumeTemplates:
    - metadata:
        name: vm5-ubuntu-datavolume1
      spec:
        resizeTo: max
        source:
          http:
            url: http://image-repo.default/bionic-server-cloudimg-amd64.img
        pvc:
          storageClassName: rook-ceph-block
          accessModes:
            - ReadWriteOnce
          resources:
            requests:
              storage: "55Gi"

What you expected to happen:
Proper volume should be created and attached to the VM.
How to reproduce it (as minimally and precisely as possible):
Try to create VM from yaml file posted above.
Anything else we need to know?:
Everything is ok when I separately create DataVolume and attach it to the VM.
---
apiVersion: cdi.kubevirt.io/v1alpha1
kind: DataVolume
metadata:
  name: vm5-ubuntu-datavolume1
spec:
  resizeTo: max
  source:
      http:
         url: "http://image-repo.default/bionic-server-cloudimg-amd64.img"
  pvc:
    storageClassName: "rook-ceph-block"
    accessModes:
      - ReadWriteOnce
    resources:
      requests:
        storage: 55Gi

The spec.resizeTo filed for DataVolume was added in containerized-data-importer v1.4.0
Environment:

KubeVirt version (use virtctl version):

Client Version: version.Info{GitVersion:"v0.12.0", GitCommit:"6dd9e09dc172387422fa454eda893e987bc213e9", GitTreeState:"clean", BuildDate:"2019-01-11T18:26:11Z", GoVersion:"go1.10", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{GitVersion:"v0.12.0", GitCommit:"6dd9e09dc172387422fa454eda893e987bc213e9", GitTreeState:"clean", BuildDate:"2019-01-11T18:26:11Z", GoVersion:"go1.10", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.1", GitCommit:"b1b29978270dc22fecc592ac55d903350454310a", GitTreeState:"clean", BuildDate:"2018-07-17T18:53:20Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.1", GitCommit:"b1b29978270dc22fecc592ac55d903350454310a", GitTreeState:"clean", BuildDate:"2018-07-17T18:43:26Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}

VM or VMI specifications:
Posted above.

Cloud provider or hardware configuration:
On prem hardware.

OS (e.g. from /etc/os-release):

NAME="Ubuntu"
VERSION="18.04.1 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.1 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic

Kernel (e.g. uname -a):

Linux dev-wac-s1 4.15.0-30-generic #32-Ubuntu SMP Thu Jul 26 17:42:43 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Install tools:
Others:

kubectl apply -f vm5-ubuntu.yaml
service/vm5-ubuntu unchanged
The  "" is invalid: : spec.dataVolumeTemplates.spec.resizeTo in body is a forbidden property

Tested on v0.13.2

Same thing on v0.14

Hi @gites,
I see the missing status was fixed.
That leaves us with the resizeTo field. As far as I can see it is just not part of the spec. The only place where I can find something about it, is in a PR of you, which was never merged....?
Current DataVolumeSpec: https://github.com/kubevirt/containerized-data-importer/blob/master/pkg/apis/core/v1alpha1/types.go#L40
Your PR: https://github.com/kubevirt/containerized-data-importer/pull/489/files#diff-7ec0bb1b5ff46a24183a942c72938b78R42
If you think it should  be part of the spec, please reach out to the CDI team. Thanks!

Oh, I forgot that CDI guys decided to always do the volume resize. Sorry for your inconvenience.


Checkbox doesn't register checks on iOS

I am unable to change the state of the Checkbox when tapping on it in iOS.

It appears this was caused by Browserify. Please disregard.


Fix crash on nil/invalid URL (iOS)

Connection with issue(s)
No unsolved issue, but might be similar to #74 and #174 .
Testing and Review Notes
Pass a url include whitespace (e.g. 'https://www.google.com '), app will crash immediately.
By adding this, there're still many codes using ! for force parsing, it require further modification.
To Do
Nothing needs to be handled.

If you want, as the maintainer is out for quite a long time now, we are maintaining this fork:
https://github.com/arnaudelub/flutter_polywebview
package also available on pub.dev:
https://pub.dev/packages/flutter_polywebview
feel free to add you PR on this fork if you want.


Fix bug: atom.workspace.getActivePaneItem(...).getPath is not a function

When I click on unfocused tab, Atom raise:
Uncaught TypeError: atom.workspace.getActivePaneItem(...).getPath is not a function

which is described in https://github.com/s-shin/atom-copy-path/issues/10
I use debugger in getTargetEditorPath function and found out that e.target is the element which has tab class and contains .title element.
So we should check if e.target contains .title element and return the path of .title element if it does.

Thank you for investigating and making PR.
#10 was resolved by #18, so this PR is closed.


Guess the color game

This game will show rgb values and user has to guess which color is it from the given options of colors.

Already we have


Add coloring improvements to tools/provision.

I noticed that some of the fail error messages are done by logging.critical. Is this supposed to be the newer way to display the error messages?

I squashed and cleaned up the first two commits (RED was never used, and WARN never defined), added some commits improving the error messages themselves, and merged this.  Thanks for working on this @rht, but you cannot keep submitting sloppy work like this.  In the future, if it appears that you haven't tested a PR manually (through bugs like this), we may just close the PR without reviewing it.


[Bug] Pressing button in Stack Limiter Module doesn't update the text

Versions
MC Verion: 1.21.1
NeoForge: 21.1.45
Pretty Pipes: 1.17.0
Description
When you press the button "Don't limit to one stack" and "Limit to one stack" in the Stack Limiter Module, it does not cycle the button text until you close and open the screen again.

Its because you cache the data before passing it into the button text supplier.
I fixed it in PP Fluids by just passing the entire call to get data.
From:
Supplier<Component> buttonText = () -> Component.literal(data.useBucketMeasure() ? "B" : "mB");

To:
Supplier<Component> buttonText = () -> Component.literal(this.menu.moduleStack.getOrDefault(DataComponentSetup.FLUID_LIMITER, FluidLimiterModuleItem.LimiterData.DEFAULT).useBucketMeasure() ? "B" : "mB");```

It's not pretty but something like that should work.


Cannot disable ace worker

I'm getting this error when the component is being mounted:
Refused to create a worker from 'blob:http://localhost:50864/8038195d-1ff6-4dc2-a5bd-224b0263fb57' because it violates the following Content Security Policy directive: "child-src 'self'". Note that 'worker-src' was not explicitly set, so 'child-src' is used as a fallback.
And I don't want to enable that directive for security reasons.
I've searched a little bit and found two options that maybe can help to mitigate the error: "useWorker" and "loadWorkerFromBlob".
I think I can disable the worker, since I'm using the component just to give some color highlighting to a XML field.

I noticed that your component is already setting the flag to false, so I don't know if there's something missing:

Thanks in advance, great work by the way.

useWorker: false should work. It works to me at least.
loadWorkerFromBlob should be used as ace.config.set('loadWorkerFromBlob', false);. It's not necessary in your case since worker should have been disabled entirely by useWorker: false

If you think there are bugs, you should create a minimal reproduce repo.

If you think there are bugs, you should create a minimal reproduce repo.

I tried to create a minimal app to repro, but it worked. Maybe there's something in the original project, that is legacy. Thanks for your answer.


creating more than 14 decks deletes existing ones

After I create 14 decks, every additional deck created erases one of the previously existing ones. I'm using a Macbook pro with El Capitan version OS X 10.11.4 (15E65)

Do you still have the issue with the latest version ?
BTW, now, in the deck directory (check preferences), you have a decks.json.bkp that you can rename to decks.json (with hstracker closed) to retrieve the latest version of your decks

Thanks for the email. I'm traveling this week but in the next days will check if it works and let you know.
Emiliano
Sent from my handheld device

On May 10, 2016, at 16:03, Benjamin Michotte<EMAIL_ADDRESS>wrote:
Do you still have the issue with the latest version ?
BTW, now, in the deck directory (check preferences), you have a decks.json.bkp that you can rename to decks.json (with hstracker closed) to retrieve the latest version of your decks
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

I checked for updates but got a message that 0.16 is already the latest, so
not sure if you cahnged anything.
However, looks like now I can add more decks (I went up to 16) and it has
not deleted any one yet).
On Tue, May 10, 2016 at 9:58 PM, Emiliano Nesti<EMAIL_ADDRESS>wrote:

Thanks for the email. I'm traveling this week but in the next days will
check if it works and let you know.
Emiliano
Sent from my handheld device
On May 10, 2016, at 16:03, Benjamin Michotte<EMAIL_ADDRESS>wrote:
Do you still have the issue with the latest version ?
BTW, now, in the deck directory (check preferences), you have a
decks.json.bkp that you can rename to decks.json (with hstracker closed) to
retrieve the latest version of your decks
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/bmichotte/HSTracker/issues/407#issuecomment-218274190

--
Emiliano

I had 30 decks today and my first deck got deleted and removed from hearthstats. Kinda sad when stats from 150+ matches got deleted.

It did it once again. I created a 19th deck for Arena and HSTracker deleted
my Standard Druid deck.
On Wed, May 18, 2016 at 5:21 PM, Frederik Vanggaard <
<EMAIL_ADDRESS>wrote:

I had 30 decks today and my first deck got deleted and removed from
hearthstats. Kinda sad when stats from 150+ matches got deleted.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/bmichotte/HSTracker/issues/407#issuecomment-220161930

--
Emiliano

Actually i found out that i wasn’t HSTracker that deleted my deck. It got archived in Hearthstats because it hadn't been used in 2 weeks and that’s why it disappeared from HSTracker. I can still see it under archived decks online but i cannot get it back to HSTracker

In my case it is definitely HSTracker that's deleting the decks. The Druid
one that got deleted was one of the most recent ones that I created a few
days ago.
On Fri, May 20, 2016 at 9:18 AM, Frederik Vanggaard <
<EMAIL_ADDRESS>wrote:

Actually i found out that i wasn’t HSTracker that deleted my deck. It got
archived in Hearthstats because it hadn't been used in 2 weeks and that’s
why it disappeared from HSTracker. I can still see it under archived decks
online but i cannot get it back to HSTracker
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/bmichotte/HSTracker/issues/407#issuecomment-220603030

--
Emiliano

today it deleted another deck. Bug is definitely still there.
On Fri, May 20, 2016 at 9:35 AM, Emiliano Nesti<EMAIL_ADDRESS>wrote:

In my case it is definitely HSTracker that's deleting the decks. The Druid
one that got deleted was one of the most recent ones that I created a few
days ago.
On Fri, May 20, 2016 at 9:18 AM, Frederik Vanggaard <
<EMAIL_ADDRESS>wrote:

Actually i found out that i wasn’t HSTracker that deleted my deck. It got
archived in Hearthstats because it hadn't been used in 2 weeks and that’s
why it disappeared from HSTracker. I can still see it under archived decks
online but i cannot get it back to HSTracker
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/bmichotte/HSTracker/issues/407#issuecomment-220603030

--
Emiliano

--
Emiliano

Can you confirm that your'e disconnected from Hearthstats when it happens, @blablablo?

To be honest I am not sure, I don't use hearthstats so I ignore hstracker messages about it. Will look into it next time I play.
Sent from my handheld device

On Jun 5, 2016, at 10:37, Zachery<EMAIL_ADDRESS>wrote:
Can you confirm that your'e disconnected from Hearthstats when it happens, @blablablo?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hey, looks like now it does not delete decks anymore. Thank you.
On Sun, Jun 5, 2016 at 4:39 PM, Emiliano Nesti<EMAIL_ADDRESS>wrote:

To be honest I am not sure, I don't use hearthstats so I ignore hstracker
messages about it. Will look into it next time I play.
Sent from my handheld device
On Jun 5, 2016, at 10:37, Zachery<EMAIL_ADDRESS>wrote:
Can you confirm that your'e disconnected from Hearthstats when it happens,
@blablablo https://github.com/blablablo?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bmichotte/HSTracker/issues/407#issuecomment-223816656,
or mute the thread
https://github.com/notifications/unsubscribe/ASSitieCr8uA1Yg8kw3O90EVNC5WeylDks5qIt86gaJpZM4IWvtV
.

--
Emiliano


Redirect to Reports app if Reports only permission

CARD-7620
https://peachworks.leankit.com/card/716383643

@mbebas , do I need to update the https://github.com/getbeyond/ng-js-beyond-bootstrap package ?

@darushka i will move it because i need to rebuild it anyway

Merged https://github.com/peachworks/ngPeachBootstrap/commit/b46e8dcb8f5ec34600f7f8c8413a4332bae441b2

v1.2.7


Add sentry.io error tracking

I need opinions on others if this is a good idea, this basically sends errors to a server where we can see errors happening to the code, it works really well (it includes build info, backtracing and a lot more we can add ourselves) and it can be self-hosted.
This would require a Privacy Policy, the idea would be to have it disabled by default and add an opt-in option in the GUI somewhere, this would link to the privacy policy so the user can read it.
This would help a lot for finding errors that aren't actually getting reported by users and have a better tracking on the precise errors.
I don't find any cons tbh because it's disabled by default and it will be pretty helpful if it's actually enabled.

Does sentry.io have good java integration? What about rust and typescript? can we use this same service across all parts of the tech stack?

compatibility shouldn't be an issue 😅

It's open source
You can pay sentry.io or self-host it, we can try it before actually paying for it :P
Maybe New Relic? But like sentry.io is open-source
It's not automatic, it wont log errors that are catched. We need to add everything ourselves. It has a lot of features like breadcumbs that would help us give custom information for context-sensitive errors also

New Relic is hella expensive

regarding as someone who is a active user with sentry:
sentry has multiple pay tiers. they limit the amount of ingested data normally based on the pay tier you use, but the self-hosted edition does not have any limits on it, and is not cost, at the limitation that you will have community support for issues.
in regards to what data it collects? its entirely based on your error handler. you can have it provide anything you want in the logs, tho by default, it normally contains the following:
os type; hardware stats (cpu,gpu,ram) log level :error,warning,ect.,machine ID (only to match with other errors from the same machine, and a user UUID (only useful to id machines logged into with the exact same user/domain joined machines)
it is very useful to id what is going on and can give a statistic on what is causing the errors.


Change server default port from 3000 to 80

To be more friendly user it is good to put it in a common port
Depending on the linux distro it may ask to be root when running
the server
Signed-off-by: Bruno Bottazzini<EMAIL_ADDRESS>

I believe we should change the default port to 80 while installing in specific cases (e.g. boards).
Users running soletta-dev-app as ordinary users will face permission issues with port 80.

On Tue, 06 Oct 2015 10:38:49 -0700, Anselmo L. S. Melo wrote:

I believe we should change the default port to 80 while installing in specific cases (e.g. boards).
Users running soletta-dev-app as ordinary users will face permission issues with port 80.

Agreed. Also, if you want something more standard that won't require
root permissions, default to 8080.

+1 for 8080.

@ibriano, you are right! The port 8080 is really common and way more right than 3000.
But in the end it will have the same behavior as the port 3000, the user will have to put "http://IP_ADDRES:8080" which in case http://IP_ADDRESS would be better.
@anselmolsm why not create an algorithm that will pass some interface like "It is needed to be root when running on port 80, change it on the configuration or run as root". Something like this...

the default configuration file should be 8080, the installer (ie: rpm, deb, etc...) should change to 80.
eventually we can offer an install option to choose the port to be used.

alright I will update the patch to be 8080
and for the installers we change to be 80 :+1:


Constrained type parameter using indexed access does not satisfy constraint

TypeScript Version:  3.5.0-dev.20190407
Search Terms: constraint constrained type satisfy indexed access parameter
Code
class Class<S extends string> {
    s!: S;
}

type T<Derived extends Class<Derived["s"]>> = Class<Derived["s"]>;

Expected behavior: Unsure, but TS versions 2.7 through 3.3 report no errors.
Actual behavior:
test.ts:5:30 - error TS2344: Type 'Derived["s"]' does not satisfy the constraint 'string'.

5 type T<Derived extends Class<Derived["s"]>> = Class<Derived["s"]>;
                               ~~~~~~~~~~~~

test.ts:5:53 - error TS2344: Type 'Derived["s"]' does not satisfy the constraint 'string'.

5 type T<Derived extends Class<Derived["s"]>> = Class<Derived["s"]>;
                                                      ~~~~~~~~~~~~

Playground Link: https://www.typescriptlang.org/play/#src=class Class<S extends string> {
    s!%3A S%3B
}

type T<Derived extends Class<Derived["s"]>> %3D Class<Derived["s"]>%3B

Related Issues: Maybe https://github.com/Microsoft/TypeScript/issues/30554 and https://github.com/Microsoft/TypeScript/issues/30802.

Circular type constraints are illegal and are now more often correctly detected as of 3.4.


Batch-wise inference (multiple images in parallel) support

Hi authors, thank you for this amazing work!
I notice that in 'main_ddpir_inpainting.py', there is a for loop that deal with one image at a time. If we have a lot of images to deal with, this may be relatively slow. I wonder if it's possible to process multiple images as a minibatch in parallel? Thank you!!

Hi @YuQiao0303 ,
Thanks for your interest, could you please try e.g. python main_ddpir.py --opt configs/inpaint.yaml in the latest updated repository? You can try to change the batch size in the config file inpaint.yaml :)


Wrong field detection

Sample code (artifical):
table.insert:find()
Luacheck report:
(W143) accessing undefined field 'insert.find' of global 'table'
As you can see find is not meant to be table.insert field, so I suppose the report is false.
Edit:
Perhaps sample is not good.
In real code I have Object.Title:find"menu.+»", where Title-property is returned by Object-metamethod __index, but this implementation is out of source (so Object.Title is just defined as global in custom std set)

If a.b:c() is valid then a.b.c exists, and if a is a global, luacheck will warn about the whole a.b.c as  a field. This means that if there are strings or other objects with methods stored in fields of globals, their methods should be listed in your custom std set.

Thank you!


Strange behaviour with AO and instancing

I've noticed a strange behaviour when you combine AO from StandardEffects and InstancedGeometry.
If you have any other objects on the scene you can see a "ghost" of the single instance on the center.
Try enabling and disabling the flag on the start of this sandbox:
https://codesandbox.io/s/r3f-instanced-colors-cvbdq?file=/src/index.js

Update:
not sure if related or not, but if you edit the flag with hot reload you may get this error:

Rendering directly the normal pass gives this result:

It seems like the normal pass it totally ignoring instancing…

Seems like it may be a THREE related issue. It's happening every time you set an overrideMaterial with instancing. Closing for now.


Hide non-matching link hints

This change will hide any link hints which don't match the currently typed string.
I figured this out by diffing the current CSS with what Vimium is using. Oddly, Safari doesn't seem to show the !important declaration on any user stylesheets, so it took me a while to figure this out.
Longer term it might be good to pull more from Vimium's styling, but this is the smallest possible change to fix the immediate issues.
Fixes #79.
Here's an example showing HN after pressing "L" (note, the screenshot also has #130):

This is awesome 🎉 guessing the maintainer isn't around to merge?

This was merged in e063a7218ae92c120e47b94c40c9f4d5dbba10c1.


Intermediate interface support for document builder

Perceived Problem
Please forgive me if I am mistaken, but it appears that Graffle's generated client does not directly support 'intermediate' interfaces when using the document builder. As an example, consider the schema below.
interface Grandparent {
  a: Int
}

interface Parent implements Grandparent {
  a: Int
  b: Int
}

type SiblingX implements Grandparent & Parent {
  a: Int
  b: Int
  x: Int
}

type SiblingY implements Grandparent & Parent {
  a: Int
  b: Int
  y: Int
}

type Query {
  grandparents: [Grandparent]
}

In raw GraphQL it is possible to write a query fragment on the "middle" interface, Parent.
query test {
  grandparents {
    ...on Parent {
      b
    }
  }
}

However, given the Schema above<EMAIL_ADDRESS>would generate GrandParent.__on_SiblingX/GrandParent.__on_SiblingY properties but not a GrandParent.__on_Parent property. This makes obtaining b more verbose than it otherwise would be.
// Actual invocation as of graffle@next-133
graffle.query.grandparents({
  ___on_SiblingX: {
    b: true
  },
  ___on_SiblingY: {
    b: true
  },
  // Repeat for every other 'Sibling' type
});

// Ideal invocation
graffle.query.grandparents({
  __on__Parent: {
    b: true
  },
});

Is this something that the document builder could feasibly support? Or have I just missed an obvious way to do it?
In any case, thankyou for your work on Graffle. It has been fantastic to use so far.

It looks like getInterfaceImplementors only searches through kindMap.list.OutputObject but not kindMap.list.Interface, which would explain the current behaviour. https://github.com/graffle-js/graffle/blob/a78ebbeb31805701c82f54b038d2c81eba52a74d/src/lib/grafaid/schema/KindMap/_.ts#L111-L115
Munging that to the below causes the __on_* interface parameters to be generated, though I'm not currently able to run Graffle's test suite or the full build process to check if there's any nuance that I'm missing.
export const getInterfaceImplementors = (typeMap: KindMap, interfaceTypeSearch: GraphQLInterfaceType) => {
  const allTypes: Array<Grafaid.Schema.ObjectType|Grafaid.Schema.InterfaceType> = [...typeMap.list.OutputObject, ...typeMap.list.Interface]
  return allTypes.filter(type =>
    type.getInterfaces().filter(interfaceType => interfaceType.name === interfaceTypeSearch.name).length > 0
  )
}

Yeah I suspect it isn't at all a common requirement. The only reason I came upon it is that I'm operating against a schema for something that makes heavy use of polymorphism. Even in my case it's far from a huge problem as there are currently only a small handful of siblings for any one interface.
The change is definitely more involved than the kludge in my earlier comment, which triggered some Typescript errors when I got a chance to test the generated client. I'll look further into it if I am able, though my weak JS/TS is probably a bigger hurdle than the complexity of the generator 😅.

Hey just wanted to loop back here and say I've shipped an iceberg-of-a-PR https://github.com/graffle-js/graffle/pull/1272 and can now direct my attention to this issue. Unblocking users is my priority so I'll be working on this next. I would like to ship the feature this week.

Thanks for your patience @cosmopetrich! If things aren't working in the newest release, let me know.


refactor: error message for instantiation failure

Feature or Problem

When methods of instantiation fail, show all related errors from attempts rather that only the last one.
Related Issues

Release Information

Consumer Impact

Testing

Built on platform(s)

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

Tested on platform(s)

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

Unit Test(s)

Acceptance or Integration

Manual Verification

@rvolosatovs would you mind taking another look? had to change the code a bit to make clippy happy with the Option usage.


RDAP server connection error handling (for discussion)

@meeb Thank you for this fantastic library! It was quick and easy to get up and running.
I found an edge case that might be worth handling differently (or not!), and wanted to share here for the sake of discussion. e.g. this might warrant a small note in the Readme, rather than a code change. Here's the tl;dr:

https://rdap.nic.build/ is listed in the IANA bootstrap data, for the .build TLD.
This URL is deprecated (see below), and no longer has an active RDAP server responding there.
whoisit is currently returning the following error when attempting to connect to it:

raise QueryError(f'Failed to make a {method} request to {url}: {e}') from e

whoisit.errors.QueryError: Failed to make a GET request to https://rdap.nic.build/domain/tailwind.build:

HTTPSConnectionPool(host='rdap.nic.build', port=443):

Max retries exceeded with url: /domain/tailwind.build

(Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x10632c430>:

Failed to establish a new connection:

[Errno 8] nodename nor servname provided, or not known'))

Digging deeper:
.build migrated to a different Registry backend provider (CentralNic) back on 2020-11-24:

https://www.iana.org/domains/root/db/build.html
https://centralnicregistry.com/support/information/tlds
https://ntldstats.com/tld/build
https://wiki.hexonet.net/wiki/BUILD

…and I'm assuming there's a bug on CentralNic's side that hasn't updated their IANA data yet. I let them know here: https://twitter.com/gavinbrown/status/1465049265978413062?s=21 (their CTO responded, fortunately).
Here's the actual RDAP server for .build: https://rdap.centralnic.com/build/domain/tailwind.build
I looked into catching this specific exception in utils.py, e.g. via except requests.exceptions.ConnectionError as e: and a new class ConnectionError(WhoisItError), but it looks like whoisit.errors.QueryError is already catching this.
I also checked to see how the httpx library (Requests' possible successor?) handles this, and it's similar:
<snip>
httpx.ConnectError: [Errno 8] nodename nor servname provided, or not known

This Requests issue seemed related as well, but I don't think the issue with rdap.nic.build is DNS-related.
Apologies that this is sort of long-winded and rambling; this might be minor and/or rare enough that it can be ignored entirely? Or maybe worth adding a bit more fine-grained exception handling, to make debugging easier?
I could also submit a PR with a note for the Readme, e.g. something like this in the Debugging section:

RDAP server connection issues: There can be transient issues connecting to RDAP servers, but it's also possible that the IANA bootstrap data can be incorrect. For example, a TLD can migrate from one registry backend provider to another, and there can be a delay in updating the authoritative RDAP URLs (if they change) with IANA / ICANN.

Thanks! Glad you find the library useful! And thanks also for the very detailed issue.
As you note this is more of a library design issue rather than a straight bug. The issue is with the bootstrapping information provided by the IANA so I suppose, technically, the current behavior is exactly correct (just not very useful).
There are a litany of HTTP errors that can occur so it's probably OK to just bubble up whatever requests raises, assuming developers who implement whoisit are almost certainly experienced with handing requests-style exceptions already. A bunch of the RDAP endpoints for smaller TLDs and RIRs have esoteric or out of date TLS requirements, for example, which can raise some interesting errors.
Most likely I don't want to manually maintain a specific list of work-arounds in the library as that sounds like a headache. You've probably already done the best thing by contacting people to get the IANA data updated which is the actual fix for this.
I could add an endpoint override if that would be useful? You could then implement your own work-arounds without them getting messy in the base library? Something like:
whoisit.domain('nic.build', rdap='https://rdap.centralnic.com/build/domain/')
or similar? This would just override the bootstrap-derived RDAP endpoint URL so you could manually select the correct upstream RDAP server when you want to query certain currently broken TLDs until the IANA get their data updated.

Thank you again for indulging my rambling odyssey down the domains rabbit hole here.
I agree that this feels more like a library functionality issue, and it sounds like there is a case to be made for adding the ability to supplement or override the IANA bootstrap data somehow.
Another example came to mind since you mentioned this: DENIC apparently runs an RDAP server, but doesn't list it with IANA for some reason:
https://rdap.denic.de/domain/apple.de
(This sort of highlights the subtle divide between the ccTLD world vs the ICANN + gTLD world.)
Thinking out loud: like you mentioned, this is similar to the allow_insecure_ssl issue. It'd be nice to configure these things centrally, prior to running the whoisit.domain() checks, rather than per-check.
(I'm a novice developer, so I probably lean on libraries like yours more than I should! E.g. the library is parsing out the TLD already). For example:

For domains under de, always try https://rdap.denic.de/
For domains under build, always use https://rdap.centralnic.com/build/
For domains under work, always use allow_insecure_ssl=True

Speaking of .work, it's migrating to the GoDaddy Registry backend on 2022-01-25, so the allow_insecure_ssl workaround for it might not be needed post-migration. (though it will probably still be needed for other zones someday)

The above commits add a new feature which permits overriding the IANA bootstrap data specified RDAP endpoints. Usage is pretty easy, just add a override flag when bootstrapping such as bootstrap(override=True) or load_bootstrap_data(some_data, override=True). This overrides the specified RDAP endpoints for a TLD (or IP/ASN/entity if needed in the future) from a static set list specified here:
https://github.com/meeb/whoisit/blob/main/whoisit/overrides.py
Details on usage are also in the README.
The format should be pretty obvious. You're welcome to submit a PR to expand that file if you keep the date and comments up to date as well. The feature has tests and should be stable. I'll bundle it into the next whoisit release.
As for interface, this feature will be opt-in (so overrides=False is the default).
With handling SSL downgrades I don't want to specify this as a default even when using overrides. It may not be clear that enabling overrides will downgrade their SSL security at the same time. While it's read-only and data that's generally considered public it's likely not going to be a great idea to weaken security for some endpoints as a default without opting in or a big warning.
If you prefer total reliability on querying over security you can probably just set allow_insecure_ssl=True on every query. All it does is allow known slightly weaker TLS cipher suites, it doesn't permit self-signed certs or anything terrible.
One thing to note is that the non-IANA listed RDAP endpoints, in my brief testing, return data that is not in the standard format returned by endpoints listed by the IANA. Such as, the handle may be missing or some dates aren't set etc. By enabling overrides the quality of the data returned may be reduced or some elements entirely missing.
Thanks for your feedback on using this for domains. I created whoisit for querying IPs, CIDRs, ASNs and entities mostly myself (to enrich the data in https://ipnetdb.com/ ) so I didn't particularly test it for domain usage beyond the default IANA bootstrap data.

Oh and I've added this helper tool as well:
https://github.com/meeb/whoisit/blob/main/tools/list_public_tlds_with_rdap_endpoints.py
Which will run on the command line against the current repo if you want an update of what TLDs are supported, not supported and which are overrides.

Thank you again! This is great.
And coincidentally, .build is already fixed:

I've pushed the overrides=True feature into a new 2.4.0 release which you can install or update to via pip now. I've also removed the .build override as you note that's no longer required. Feel free to run the list_public_tlds_with_rdap_endpoints.py tool and add make a PR to add more override manual RDAP endpoints if you notice any into overrides.py. Cheers.


Update README.md

I have suggested few changes in README.md.
Kindly review.

Signed the CLA ........

/check-cla

Hi @ss1208 , please sign the CLA for this PR to be reviewed.
A PR wrangler may close this PR within 7 days if the CLA has not been signed. You can re-open the PR when the CLA has been signed

Hi @ss1208 , since there has been no action to sign the CLA, I am closing this PR
Feel free to re-open the PR when you have signed the CLA
/close


Align PyModuleDef with Python 3, fixes encukou/py3c#34

By extending the PyModuleDef struct for Python 2 with the fields present in Python 3, client code can provide field initializers to avoid the -Wmissing-field-initializers compiler warning.

Thanks for the PR.
py3c should also explicitly check that these are set to NULL in Python 2, so people can't use them where they don't work.

py3c should also explicitly check that these are set to NULL in Python 2, so people can't use them where they don't work.

Good idea - how would you try to achieve that? My approach would be to turn the macro PyModule_Create into a function and have it check the members of the passed struct before calling PyInit_Module3.

Yes, a static function sounds right for this.
I guess it would also be possible to make it a macro using the ternary operator, but that might make it unreadable.
On November 25, 2020 5:02:45 PM GMT+01:00, Michael Rudolf<EMAIL_ADDRESS>wrote:

py3c should also explicitly check that these are set to NULL in
Python 2, so people can't use them where they don't work.

Good idea - how would you try to achieve that? My approach would be to
turn the macro PyModule_Create into a function and have it check the
members of the passed struct before calling PyInit_Module3.
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/encukou/py3c/pull/35#issuecomment-733797503

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Thanks!
I've added some documentation, and rebased to avoid a CHANGELOG conflict.


LIDC dataloader ground truth

Hi, I was wondering how to load ground truth objects if I use LIDS dataloader code.
As I understood from the code of preprocessing.py and Readme, each object is loaded as a bounding  box.
I have multiple objects in my image, which I want to segment. Does it mean, that I should have separate .nii.gz file for each object?
How the folder structure will look like?
Btw, I want to use 3D MRCNN, inputs were originally tif files, which I transformed to nrrd.
Thank you.

Hi,
annotations can either be loaded as bounding boxes (then you need to create pseudo masks out of the min your data loader) or as pixel-wise annotations (like in LIDC). Please look at line 236ff. of the lidc data_loader to see how to load the files and generate batches. In this case two .npy files are loaded separate for data and pixel-wise annotations for each patient. Those were created in the preprocessing.py, but you need to adapt that script to your own data set. Hope this helps.


add post operation

@jframos

I think it is important to check the Content-Type header in the case of POST requests. We only should accept text/plain values and return a HTTP 415 Unsupported Media Type if the Content-Type is not the expected one.

LGTM


Release 2.0

We need a warning in the changelog about the possible breaking changes.

Working on a version bump.

#97


Release of version 3.2.2

Latest released version of 3.2.x is 3.2.1 it was released about 2 years ago.
When would you like to release 3.2.2?

Good question! I would also like to know.
Thanks.


Base64.is_base64_char is true if [A-Za-z0-9+/]

Currently
List.filter Nocrypto.Base64.is_base64_char (explode sym)

where sym is the string in base64.ml only returns 0-9,+,/ because they have ascii values below 64. I suspect the intended functionality was char -> bool depending on if the char is alphanumeric, +, or / which is what this change does.

Oh, you're right! Turns out, the function is never used and I never noticed this.
Thanks!

Glad to help. I noticed it when trying to debug why my base64 strings weren't being processed correctly and I was trying to debug. Turns out nocrypto needs properly '=' padded base64 strings.


Would an async api have performance gain?

Fantastic job! But I'd like to ask if there would be some another gains by designing an async implementation and bind them to python with __anext__ and __aiter__, since data loading is highly related to IO.

Thanks! I don't think asynchronous iteration would be helpful in our particular case.
Background fetching of the next sample or batch is achieved using the prefetch method. This leaves the python thread to process the batch while in the background other threads are fetching the next samples. If you have a particular use case that we might have missed which would benefit from python's asyncio then we 'd love to reevaluate.

If we have a slow IO like remote cloud storage like S3, maybe it would block the prefetch thread pool?


fix(fast_check_dts): ensure top level non-exported functions have declare keyword

Huh, I wonder why typescript does that because I don't believe it's necessary for exported declarations. I'll revert to align.


fix empty colomns on CNP CRD

Fix the empty columns by adding priority fields to columns that may be empty.
Fixes: #27459

Ah CI gives the next steps, see the failed job for Go related checks.

Ah CI gives the next steps, see the failed job for Go related checks.

It seems I missed a step

I've marked the Pull Request as draft. Once you have finished up working on it feel free to click on the "Ready for review" button.

Thank you

I've marked the Pull Request as draft. Once you have finished up working on it feel free to click on the "Ready for review" button.

Thank you

Thanks for the note! I'm a bit swamped lately, I'll get back to the PR and ping you once I've hit 'Ready for review'

I rebased & squashed the commits to tidy it up and to address one of the CI failures. Will trigger full testsuite after the smoke tests go through.

Thank you @joestringer
I forgot to rebase it. is there any action I need to take?

No action is needed at this time, just looking for the CI tests to complete. I don't expect that this change should impact CI at all, hopefully CI doesn't rely on the textual output of this command.

Looks like the external workloads test failed, I can't see any errors aside from not being able to find a specific pod in the connectivity tests, I'll rerun.

/ci-external-workloads

Thanks for your contribution @Iiqbal2000 !


Remove pandas

Are we using any pandas features specifically?
It seems not, so let's just pull pandas out completely.

Done except for dataAcq

Done as of #21, notwithstanding 637e5dca0556962c4ad9e08574330c01c3b8ce1d.


[feature] Added functions to register-unregister admin theme #274

Closes #274

Coverage increased (+0.05%) to 98.536% when pulling 1414f3f8d1e605ec2e812dd02cc6756fabee6c62 on issues/274-register-css-js into 663619184d2260ed480ab312e4cbf45b454a8926 on master.


Add ArviZ to yml

Fix for 'arviz not installed' error when running a notebook. Great work by the way!

thanks @vbartle!


Move to python-openapi org

Codecov Report

Merging #523 (5e9672a) into master (2d7122b) will not change coverage.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##           master     #523   +/-   ##
=======================================
  Coverage   90.28%   90.28%
=======================================
  Files         124      124
  Lines        3665     3665
  Branches      348      348
=======================================
  Hits         3309     3309
  Misses        301      301
  Partials       55       55

Impacted Files
Coverage Δ

openapi_core/unmarshalling/schemas/factories.py
100.00% <ø> (ø)

openapi_core/validation/schemas/__init__.py
100.00% <ø> (ø)

openapi_core/__init__.py
100.00% <100.00%> (ø)

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


Fix ORC reading over different schemas

Fixes #5562
This PR is to fix the bug which fails reading ORC data from multiple file schemas. The root cause of the bug is multi-thread modification on a thread unsafe variable (broadcasted Hadoop Conf).
Signed-off-by: sperlingxx<EMAIL_ADDRESS>

build

I find it interesting that this PR claims it fixes #5562 but does not use the same repro test from that issue as part of its test case. The test in #5562 seems subtly different than the one used here, as the order of columns in the dataframe is intentionally different, whereas here the test preserves the same order and the extra one is on the end. Have we verified the same test from #5562 now passes with this change?

Yes, the case in #5562 can be addressed by this PR.  I decided to use current case rather than the original case, because it looks stronger and more representative. In terms of the column order, I shuffled the order of columns to include the factor into the case.

build


Womanium Final Assignment -

Womanium Final Assignment -
Womanium Final Assignment - Samiha Rahman
Uploaded assignment submission

Good job!
Score: 8/10.


Lonestar College Kingwood

@Artificial563 my.txt I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!


Need some help setting it up

I don't think this is an issue on your part, I just don't know how to run it.
I have researched and followed a guide for the following:
How to create wsl.
Followed a guide and installed python 3.9.9 successfully on the wsl with ubuntu. I'm confident I did this part right.
Installed pip. Followed a guide to install ffmpeg on ubuntu, added the path like this: export PATH=$PATH:/bin/ffmpeg
Installed stable-ts. Added it to path just in case with export PATH=$PATH:$HOME/pacote/ since I got the yellow message saying it was not in path.
After this, step 4 sounds like it's optional, is it? I just want to run this, if it's optional I'd skip until I can get it to work.
So I didn't do step 4.
Then I tried to run with a single audiobook file, very small.
The folder is called "name" and the audiobook file inside is "name.m4b", in the folder I also have "script.txt" with the ebook in it.
When I run ./run.sh "$(wslpath -a "wsl.localhost\Ubuntu\home\pacote\name")" (I tried coping your example) I only get new line ">" and a blinking _
(if I add another \ like your example after "name": name\ + another ")" it just gives me ~bash: ./run.sh: No such file or directory
Would you mind explaining what I'm missing?
edit: here's a picture of how it is just in case

Yeah, I'm happy to help. Sorry its not clearer. You're the first person besides me to use this. So first off treat all the installs like they were inside Ubuntu. So you dont need to install ffmpeg on windows, but you do need to in Ubuntu. Then it should just be on the path.
Next, try running the split.sh command to get the m4b split into parts. Then you can run split_run which should run without any memory issues if your m4b got cut into chapters.
The run command you had should work too, if you dont feel like splitting and have enough ram to spare. Though I haven't been using it myself lately. Try converting your path to wsl, so something like "/mnt/c/documents/name" and using that instead of the wsl -a command, ehich only has the purpose of converting a windows path to a unix path.
I'm mobile right now, but if any of that doesn't make sense, let me know what you can try and what happens and I'll try to make clearer docs based on your pain points

Ah I think I see your issue.

Your path is ./run.sh

I can clean up the docs to clarify this but you should download the project with git.
git clone https://github.com/kanjieater/AudiobookTextSync.git
Then cd AudiobookTextSync

I see!
I did that, here's how it looks now:

That was done instantly so I think something else is missing. Also can't find that .vtt file.

Go ahead an git pull again. I've updated the readme. You should now be using ./run.sh for split or unsplit m4b's, and they should just work now. Let me know how it goes, and thanks for being patient with the troubleshooting.
In addition, there is now an anki command documented that can turn your m4b into an anki deck and import it for you.

Hey, thanks for the answer!!
I'm making progress now.
I did the git pull again (had to delete the old folder first) and I was getting some errors at first, like: ./split.sh: line 5: cd: too many arguments. I then pasted the audiobook folder inside AudiobookTextSync and used the absolute path for ubuntu:
powershell: \wsl.localhost\Ubuntu\home\pacote\AudiobookTextSync> wsl wslpath -a name
result: /home/pacote/AudiobookTextSync/name
Then it worked! Now I have the files split, but I'm getting an error when trying to ./run.sh:

Good to hear you are making progress! Try these things

You need to be on python 3.9.9. python --version should say 3.9.9
Then install your dependencies, stable-ts from pip as pip install git+https://github.com/jianfch/stable-ts.git
pip install -r requirements.txt
Don't change the script, get python on your path as python 3. (If step one is working nothing to do here)
If you do all of those things it should be working 👍

Did as you said and it's running, but it's like this for a while, is that common? There's is no .srt file on the folder yet so it's probably not finished, my PC is also still running hard.

Also I saw the read me and it's much easier to understand now.

Since the progress bar was completed I'm guessing 1.4+
That's all I got at the end. My laptop was running really hard there, after the progress bar hit 100.

Here's the folder just in case:

Uh oh you ran out of memory! Unfortunately that's step 1.3. If it won't work for you there, I don't have a software solution. You could change the split-py from using large-v2 to medium model, but preformance of the model will be affected.

I see! I'll try again while closing other apps and monitor it with task manager

You can modify this line to any of the models in split_run.py. It may have get overwritten in future git pulls though. You can use any that whisper supports I think. Try a smaller one, but smaller means less training which means less accurate results.
The other solution would be to find a way to cut your files up even smaller. I do not have a solution for that though currently. You would have to make sure to not cut up anywhere besides silent parts, and ideally not between a sentence.

I thought it was RAM not VRAM, so yeah it didn't work even with "tiny"

I guess I need to buy a GPU!
But really, thanks a lot for all the help, I learned a ton about ubuntu/python.
I wanted to use this with jpdb's mpv plugin, which has color coded words based on my account's known words. It'd be amazing to mine as if it was anime, but for audiobooks.

I thought it was RAM not VRAM, so yeah it didn't work even with "tiny"  I guess I need to buy a GPU!
But really, thanks a lot for all the help, I learned a ton about ubuntu/python. I wanted to use this with jpdb's mpv plugin, which has color coded words based on my account's known words. It'd be amazing to mine as if it was anime, but for audiobooks.

Ah that's a good point - yes it's VRAM though the error from the OS doesn't really indicate that. For Japanese learning we might share some resources for this on TMW or check out my discord for more updates.

Just want to correct something I said above.
After running a few tests it seems that Medium tends to outperform large actually due to how stable-ts works. https://github.com/jianfch/stable-ts/issues/80#issuecomment-1442302091
Tiny also performs almost as well it seems to the point that I might leave that as the default.
If you have new questions or give it ago again in the future, feel free to open a new issue.


Update README.md

JOB SITE (https://foundthejob.com)

@kchetri222 Please read the contributions guide before submitting the PR. Thanks!


implement domainExtractor for image and title, with a single implementation wikipedia

First thanks for sharing this code. It's exactly what i needed, and i didn't find anything i liked in NodeJS.
This is more a request to get feedback then actual pull request.
The problem i encountered is that wikipedia does not work with the image and title extraction that's implemented now.
The Image is not in the header, but is the first image in the '.infobox'.
Title- When splitting the Title, the longest part is usually not the important part, like in 'Thomas Edison - Wikipedia, the free encyclopedia'
Trying to tackle this problem i saw 2 options:

Re-factor the current extraction implementation to support wikipedia structure. I don't think it's a good option. First it will cause the code to be less readable. Second, what will happen when i need more customization?!
Second option is to have something like domain specific plugins.

Obviously i decided to use the second option.
There is still work to be done and issues to address, but I would like to get your input on the proposed solution.
Thank you for your time!

Hey, thanks for the PR. I do agree that domain-specific plugins are a better path here than hacks on top of hacks in the main code.
I'll take a look at the PR in detail when I have a little free time and let you know what I think.
Thanks!


Remove case ranges

Remove five case ranges/GNUisms, four in a led driver and one in printk as reported in #11594

Codecov Report

Merging #14075 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #14075      +/-   ##
==========================================
+ Coverage   52.61%   52.64%   +0.03%
==========================================
  Files         307      307
  Lines       45472    45472
  Branches    10530    10530
==========================================
+ Hits        23924    23940      +16
+ Misses      16666    16654      -12
+ Partials     4882     4878       -4

Impacted Files
Coverage Δ

lib/os/printk.c
84.32% <ø> (ø)
:arrow_up:

subsys/testsuite/ztest/src/ztest.c
72.72% <0%> (+7.43%)
:arrow_up:

subsys/testsuite/ztest/include/ztest_assert.h
77.77% <0%> (+38.88%)
:arrow_up:

Continue to review full report at Codecov.

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

#9893

#9893

does this fix the issue above?
Need Fixes #9893 in the body or a commit to associate and close automatically once this is merged.

does this fix the issue above?

This one fixes 2) in #11594

@nashif it does not fix all issues. But fix some violations


Syntax highlighting is broken when using frontmatter (after upgrading to VS Code 1.92)

Describe the bug
Seems the syntax highlighting is somewhat broken after upgrading to VS Code 1.92.
Having a frontmatter on the antlers file breaks the syntax highlighting. Or to be more precise, having more then one line
This works:

This breaks it:

Impacted Products
Antlers Toolbox for VS Code
Versions and Other Plugins/Extensions
VS Code 1.92.0
Antlers Toolbox v2.6.19

Removed a few unrelated extensions, restarted VS Code, and now it's all back to normal...
Seems to have updated VS Code to 1.92.1 as well.
I have non idea what actually solved it.
It has probably nothing to do with this extension 🙂

Interesting! Thanks for updating the issue, though 🙏

caused by https://github.com/microsoft/vscode/pull/219833
you will need to update your frontMatter intergration to something like https://github.com/microsoft/vscode-markdown-tm-grammar/pull/162
https://github.com/Stillat/vscode-antlers-language-server/blob/157d0eac3de834662d8616fdc828f554b8a73095/client/syntaxes/antlers.json#L123-L132
before VSCode re-adds the new YAML grammar again
related: https://github.com/microsoft/vscode/issues/224862

Thank you for the extra info!


FactoryGirl.modify appears broken

I feel like I am missing something obvious but see below:
irb

irb(main):001:0> require 'factory_girl'
=> true
irb(main):002:0> FactoryGirl::VERSION
=> "4.7.0"
irb(main):003:0> class User; attr_accessor :name; end
=> nil
irb(main):004:0> FactoryGirl.define do
irb(main):005:1* factory :user do
irb(main):006:2* name 'foo'
irb(main):007:2> end
irb(main):008:1> end
=> []
irb(main):009:0> FactoryGirl.build(:user)
=> #<User:0x007ff195007448 @name="foo">
irb(main):010:0> FactoryGirl.modify do
irb(main):011:1* factory :user do
irb(main):012:2* name 'bar'
irb(main):013:2> end
irb(main):014:1> end
=> #<FactoryGirl::Declaration::Static:0x007ff1939cb058 @name=:name, @ignored=false, @value="bar">
irb(main):015:0> FactoryGirl.build(:user)
=> #<User:0x007ff1939b8868 @name="foo">

So why is the factory stuck on name 'foo'?

OK figured it out.
It looks like it's not possible to modify a factory after it has been used once.
Not sure if this is by design but the behaviour was the same right back to 4.4.0.
I'd close the issue but I'll leave it open with the suggestion that this might merit an extra sentence in the relevant section of GETTING_STARTED.md
Thanks for maintaining this very useful gem.

hmm, I'm having trouble with FG.modify too, in FG 4.7.0, my 'modify' directives don't seem to have any effect.
I wouldn't have thought I had used it even once before modify, but @rob-anderson's hint gives me something to look for at least, I'll try to figure out how to put breakpoints in to ensure it hadn't been used before. If it had been used before, it will have been in some weird way that's probably hard to eliminate.
@rob-anderson, you're saying this has been true since 4.4.0?  Hmm, looks like my app prior to upgrading was using 4.5.0, and having no trouble.
Okay, got to try to get to the bottom of what's causing my problem, may or may not be "can't modify factory already used".  (But if modify on a factory that's already been used is a no-op -- I'd rather have it raise than be a no-op!)

Okay, I haven't managed to figure out if the factory I'm trying to modify had already been used once first.
But I have determined that:

With FG 4.5.0, my modify in my actually existing app works again -- no changes to app except FG/FG-rails version. And you indeed still can't modify a factory that's already been used. So something else changed.

With FG 4.7.0, my modify (of a factory from a gem) has no effect at all, it's like it was a total no-op.

I was going to test in FG 4.6.0, but it seems not to exist!
So something has indeed changed between 4.5.0 and 4.7.0 to make modify stop working. I am not quite sure how to dig into what to debug.
For now I guess I'll pin to 4.5.0 so my tests keep working.

ugh now I can't reproduce anymore either I may have just gotten confused too myself.
Still suggest that if in any cases (like factory has already been used) FactoryGirl.modify is going to be a no-op -- it should raise, indicating it couldn't do what was asked of it, instead of just silently no-oping.

@jrochkind thanks for clarifying here - I've closed the issue for now, but if you run into things moving forward, don't hesitate to reopen this.


Option to disable trigger in inline fields (aka. key::value)

Thank you for making this plugin! I switched from Emoji Shortcodes today and discovered that this behavior still exists, hence the issue.
What happened:
In Obsidian, it's possible to add inline metadata/fields/whatever it's called in the following format (besides adding them in frontmatter):
sleep::7.4
food:: 🥗

And as :: contains a : sometimes the result would become food:🥗 if I forget to add the leading indicator, which is no longer a valid metadata and would cause issue in DataviewJS queries.
I hope you could prevent this by default, or at least add an option to disable this.
Expected behavior:

if there is exactly two :, emoji does not autocomplete
if there is only one or odd numbers of : (in the case of :::salad before autocompletion), emoji would autocomplete as usual

Ops, I forgot to mention another issue so just leave it here.
Emoji autocomplete would be triggered even if : is wrapped (escaped?) by `, I assume this is not intended:
# This is normal
:salad
# This is unexpected
`:`salad
# This is unexpected too
`::`salad

so, if i understand correctly, the first issue is that if you type food::something, emoji autocomplete tries to compelte for something? I don't know if i understand it correctly, if not, please let me know. regarding it's solution, there isn't really a good way to determine even/odd number of characters using RegEx, so i'd have to do that in JS, which id' like to avoid. I could extend the Trigger on first letter setting's off variant to not trigger if there's something like this: word::whatever, we'll see how that goes.
for the second issue, it's not intended, but i havent't though of it initially. I'll see if i can fix it.

the first issue is that if you type food::something, emoji autocomplete tries to complete for something?

Yes, sorry I did not add some examples like the second issue. I can't think up a solution either but become accustomed to it so it's fine. You don't really need to determine even/odd number of colon :, covering the first three cases would suffice IMO.
# This would trigger emoji autocomplete
:salad
# This one won't (normal text in inline field)
::salad
# This one would trigger again (add emoji in inline field)
:::salad
# This one won't too, but I think it's very unlikely to happen. Who would type this way😶🌫
::::salad

Regarding the second issue, I upgraded to 1.2.1 and it still happens to me after disabling all other community plugins & reloading.
Now it becomes:
# This is normal
:salad
# This would still trigger emoji autocomplete
`:`salad
# This one is fixed
`::`salad

oh yeah i haven't fixed the `:`salad thing yet i forgot about that. I don't have too much time right now, but i'll eventually get to it. Is the first issue atleast fixed?

make sure you have the strict trigger option enabled in Emoji Autocomplete Settings

Yes, I checked settings and stricter autocomplete trigger is already selected. Full data.json attached in the end if it helps.

Is the first issue at least fixed?

If you mean ::salad, sadly no. And more examples are provided in the first code block in my reply above. I discovered those when typing scripts in code blocks (like scp remote:~/wherever/file ./file.
At least the second issue is partially solved, and like you said, `:`salad is a rare edge case so just ignore that. Please take your time, I don't mind waiting for a few days or even months.
data.json:
{
  "immediateReplace": true,
  "suggester": true,
  "considerHistory": true,
  "historyLimit": 25,
  "history": [
    "rice",
    "duck",
    "green_salad",
    "kiss",
    "sleepy",
    "middle_finger",
    "six_pointed_star",
    "dotted_line_face",
    "star_of_david",
    "facepalm",
    "chicken",
    "hair_pick",
    "iphone",
    "love",
    "santa",
    "leafy_green"
  ],
  "highlightMatches": true,
  "strictTrigger": true,
  "triggerFromFirst": true,
  "emojiSupported": {
// This has too many lines so I omit them.
  },
  "hideUnsupported": true,
  "tagSearch": true,
  "tagShowShortcode": false,
  "latinize": true,
  "polyfillFlags": false
}

oh yeah, i still trigger for ::salad but i no longet trigger for key::salad with key being whatever, since that's the dataview syntax. I don't think ::salad does anything for dataview? but i could be wrong.

That's right, ::salad is weird enough. Things like key:~value (in the format of scp remote:~/wherever/file ./file) are still valid though. I'm closing this anyway as these cases normally won't happen that frequently.


refactor: rename blob_gas_price to blob_base_fee

refer to: https://github.com/paradigmxyz/reth/issues/6330
The function calc_blob_gasprice in
https://github.com/paradigmxyz/reth/blob/84b13286f0046df773ace64902301ec805d55b6a/crates/primitives/src/header.rs#L239-L241
is defined in another project revm-primitives, so I didn't rename it.

There should be another PR for revm to rename blob_gas_price to blob_base_fee.
https://github.com/bluealloy/revm/blob/main/crates/primitives/src/result.rs#L213-L214
And another PR for alloy.
https://github.com/alloy-rs/alloy/blob/main/crates/rpc-types/src/eth/transaction/receipt.rs#L30-L32

mind submitting PR to alloy for this?


How to count the number of any class which is not present in COCO dataset?

I have been building a toy(which is not present in COCO) detection dataset based in tensorflow object detection API. I have followed this github repository for my project model,Github.
But now I also want to count the number of toys present and saw your repository.
Can you please tell what should I change in this Github repository and the information which I can use from my previous work.

Custom training module has been developed currently and it will be published soon. Thus, you can perform transfer learning on trained model to develop your custom object counting systems.

I have the same problem, I could'nt figure it out where to put my trained data.
Are there any development on custom training module?

A sample project added to guide you about training your own object detector with your own data to develop your own custom object counting system! Please, check this sample project!


Update Readme and fix broken link

Generally polished the Readme, fixed a broken link, and added a link to the Dolphin progress report

good work 👍


Fix Spelling Mistakes

Checklist for this pull request

[x]   I have updated README.md, describing the new feature I'm adding (if applicable).
[x]   I have read the guidelines for contributing.
[x]   I have checked that my code additions did fail neither code linting checks nor unit/UI test.

Description
Simply fixes the various spelling mistakes in the project. Does not discriminate between American and British spelling.

Thanks!


Add brotli to linux images

Description
Add brotli to linux images
Approximate size: 730.0 kB (Installed size in Ubuntu 20.04: https://packages.ubuntu.com/focal/brotli)
Tests should already be included: https://github.com/actions/virtual-environments/blob/main/images/linux/scripts/installers/basic.sh#L94-L101
Related issue: https://github.com/actions/virtual-environments/issues/1313
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

Is there a way to get ubuntu checks run on that PR?

/azp run ubuntu1604, ubuntu1804, ubuntu2004

@maxim-lobanov It seems there was an issue in the pipeline about geckodriver. I rebased the commit. Is it possible to trigger the pipelines?

Yep, it is known issue that was fixed recently by https://github.com/actions/virtual-environments/pull/1322
Could you please pull the latest master to your branch and I will restart builds? Sorry for inconvenience

No problem. I rebased the branch, you can restart them

/azp run ubuntu1604, ubuntu1804, ubuntu2004


< Prefix kills BotMention Prefix Command

If a user changes the prefix to <
While they will still be able to use <prefix to bring up the prefix command, they can no longer use @BotMention prefix this will now trigger a unrecognized command.
as 1Computer said in discord
 because their regex matches < then @id> prefix

This is fixed in the master branch.


Using Lombok's @Data Annotation Doesn't Work With Error Prone

ATTENTION! Please read and follow:

if this is a question about Error Prone, send it to<EMAIL_ADDRESS>if this is a bug or feature request, fill the form below as best as you can.

Description of the problem / feature request:

I have the following code:

@Data
@AllArgsConstructor
public static class TableEntry {
    public String lexme;
    public TokenType type;
    public int line;
    public int col;
}

The @Data and @AllArgsConstructor annotations are part of the lombok library. This code builds fine. However, using error prone it gives me this stack trace:
  @Data
  ^
     Please report this at https://github.com/google/error-prone/issues/new and include the following:

     error-prone version: 2.1.3
     Stack Trace:
     java.lang.IndexOutOfBoundsException
        at java.nio.HeapCharBuffer.subSequence(HeapCharBuffer.java:596)
        ... (Many More Exceptions Omitted)

Feature requests: what underlying problem are you trying to solve with this feature?

A way to suppress this error.

Bugs: what's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.

Download the lombok jar file, add it to your classpath, create a class with at least 1 field, then give the class the @Data annotation.

@Data
class MyClass {
    public static int myVar;
}

What version of Error Prone are you using?

Version 2.1.3

Have you found anything relevant by searching the web?

#1185
#1214
The solution they are suggesting is add the annotationProcessor option with lombok. I did that. Here is the section in my build.gradle file:

compileOnly 'org.projectlombok:lombok:1.18.12'
annotationProcessor 'org.projectlombok:lombok:1.18.12'

Places to look:

GitHub issues: https://github.com/google/error-prone/issues
email threads on https://groups.google.com/forum/#!forum/error-prone-discuss

Your ErrorProne version is too old. Many issues with Lombok have been fixed in 2.3.4 and 2.4.0 IIRC.

Thanks for that, I updated it.


Move To Front does not work

Portrait - MoveToFront does not work.
To Reproduce
Add two characters (c1, c2) and Show c1, then c2 at same stage position (e.g. Left). Use Portrait - MoveToFront block to move c1 to the front.
Result
c1 remains in the background.
c2 is in the front.
Expected behavior
c1 is in the front
Screenshots

While the game is playing, the gameObject Stage > Canvas has character holders as children. These character holders have portraits as children.
The current code rearranges the children of the character holder objects.
Instead the code should rearrange the character holders.
Versions & Platform (please complete the following information):

Unity version 2020.1.2f1
Fungus Version 3.13.4
Editor OS: Win10
Build Platform: Occurs in editor and std win build.

The solution is to add another .parent to each of the two parts of line 393 in Portrait Controller
options.character.State.portraitImage.transform.SetSiblingIndex(options.character.State.portraitImage.transform.parent.childCount);
becomes
options.character.State.portraitImage.transform.parent.SetSiblingIndex(options.character.State.portraitImage.transform.parent.parent.childCount);


Fix for TIP and NOTE formatting issue

Added a new line between the TIP and NOTE to attempt to fix the formatting issue that's currently resulting in the NOTE text showing as a section heading.

Hi Paul,
Thanks a million for spotting this issue and even more for taking the time to submit a fix for it. The problem was actually a little more technical though. Something had gone awry with the markdown that renders the tabs for the current and new experiences. We've fixed that on our end and will publish the update soon, so I'll go ahead and close your PR. Thanks again for submitting it.


docs: fix link and typo

What is it?

[ ] Feature / enhancement
[ ] Bug
[X] Docs / tests

Description
ix link and typo
Use cases and why
Nope
Checklist:

[X] My code follows the developer guidelines of this project
[X] I have performed a self-review of my own code
[X] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

thanks @forresst
once useTask$ will land (it's still not published) we'll have to remove useMount$ and update the docs accordingly
For now we still need to probably revert the docs to not mention useTask$ at all because it can't be imported now
So I'm waiting with this merge (unless you want to create a separate PR for that second change of resource, and I could merge that one)

OK useTask$ has landed in version 0.16.0
@forresst can you please update this PR to remove the useMount$ reference?
thanks!


Use Bart's serial queue and consolidate publishing

Pushing the work into a serial queue helps keep guarantees of completely processing events in-order. Simplifying the publishing into Subscription gives that concept weight and consistent use in the file.

Cool! I'll have to give this a more thorough look through in a bit.

So, @timshadel, can you or bart give me remind me what issues you were running into that led to this?

Some of the changes were prompted by the thread sanitizer.
In general, these changes make the behavior of the reactor consistent and predictable. Otherwise you don't know the state of the State (no pun intended) when reacting to an event. Also, you don't know what thread an command or event handler will be executed on (it's up to whoever triggered it).
Specifically I ran into this several times: within a react(to: I either triggered another event or fired a command (directly or indirectly). In the command or second event handler, the State was old while I would have expected it to have the results of the mutation caused by the original react(to:.
With this PR commands, event handlers, and updates are all executed on a proper queue, and the state of the State is more predictable and intuitive for each.
Maybe @timshadel and add some additional details for his motivation.

My motivations are very similar to Bart's: guaranteed ordering and update safety. I've played around with it, and I can create scenarios where "last update wins", and the result isn't what you expected. This tends to happen only when it's used somewhat incorrectly, but there's no neon sign at the call site saying that you've done it wrong—everything almost works. With this slight change even that incorrect usage is guarded against.
While this isn't a design goal, this changes also gives you the freedom to actually issue events or commands as entry or exit actions that occur when you transition your state, and because they hit the serial queue, you still have all the safety of a pure update. Some events are more easily issued as something which happens when a state transition occurs rather than when an event occurs.
Anyway. Those are some ideas of what you get by putting it all in a queue.

Sounds good and looks good. I'll get this all merged in. Thanks you two!