Supporting the developers: we have a Reddit community of over 1 million Dogecoin enthusiasts and we want to see what we can do to help you developers with your improvements to Dogecoin.

Can you let us know what are the best ways to help support you all? We have an engaged community and are willing to do what it takes to help. We aren't all that technical but let us know how we can help you.

Thanks, this is the rosetta-dogecoin project though. You've already found the "how to help" topic in discussions here. For generic Dogecoin development support and questions, please use https://reddit.com/r/dogecoindev.


Really fix #2

Removing the extra instances made type inference work better...
This should also allow many more arguments with plam.

The previous commit that added    plam :: forall c. (Term s c -> a) -> Term s (c :--> b)  seems to have been what allowed this simplification.

The thing this changes specifically, is that all arguments are easily inferred to have type Term _ _.


Linode page delete button

Two variations of an idea (delete as the last "tab" option that opens up a confirmation modal):

Not a fan of this idea, maybe on the settings page, where the rename will happen.

btn-danger-outline "Delete this Linode" next to the save button on the group/label edit?

Close, but we've repeatedly had issues in discussion when a save button was placed next to a cancel/delete button. But it would be a good convention to have the delete button be on the settings page.
How about this?

I don't like delete as a tab.

We could put it in a section in the advanced page... but that's getting really hard to reach...

What issues have we had with save next to cancel? We still do that all over the place.

Although I'm not the one that requested having save and cancel service/delete object not next to each other, I get the idea. And although we haven't necessarily made these changes yet, they've still been requested.

The need for this still may stand. Will change title appropriately and back out.

This can be done elsewhere in a more intuitive way.


AP-2405 update font size of radio button labels

What
Link to story
Following design system recommendations on blue interruptions panels, update the radio label font size to 24px
Checklist
Before you ask people to review this PR:

[ ] Tests and rubocop should be passing: bundle exec rake
[ ] Github should not be reporting conflicts; you should have recently run git rebase master.
[ ] There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
[ ] The PR description should say what you changed and why, with a link to the JIRA story.
[ ] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[ ] You should have checked that the commit messages say why the change was made.

Codecov Report

Merging #2735 (d15ed23) into master (13e3400) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #2735   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          532       532
  Lines        11131     11131
=========================================
  Hits         11131     11131

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 13e3400...d15ed23. Read the comment docs.


macOS: Fix build warnings

Fix two kind of warnings since the last clang upgrade:

warning about the use of the register keyword when linking with libpico:
 ../../include/libpico/picoknow.h:167:53: warning: 'register' storage class specifier is deprecated and incompatible with C++17 [-Wdeprecated-register]
 typedef pico_status_t (* picoknow_kbSubDeallocate) (register picoknow_KnowledgeBase that, picoos_MemoryManager mm);
                                                     ^~~~~~~~~

'compilation_timestamp.h' not found
 core/WbTelemetry.cpp:24:10: 'compilation_timestamp.h' file not found
 #include <compilation_timestamp.h>
          ^~~~~~~~~~~~~~~~~~~~~~~~~

test me

Build Passed :ok_hand:
You can safely merge 'fix-build-warning-on-macos' into 'revision'.

👍

👍

👍


Added https://schools.madrasati.sa

this is the saudi government online school

@remaselkoumi .idea/libraries/KotlinJavaRuntime.xml: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@remaselkoumi .idea/libraries/KotlinJavaRuntime.xml: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@remaselkoumi .idea/libraries/KotlinJavaRuntime.xml: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@remaselkoumi .idea/libraries/KotlinJavaRuntime.xml: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

I will try sir, thank you for your instructions


No report with mocha and istanbul

I have a problem with the integration of gulp-istanbul with gulp-mocha.
The report indicates that the feature is not covered.
This gist show you the test file and the script file. They are not in the same directory
If you need more information, please feel free to ask

Please use StackOverflow for support questions.

Two options :

All the plugin works so your doc is incomplete
The plugin does'nt work well

So this issue is opened to fix this problem. Do you really think that with StackOverflow I'll find a solution for this specific problem ?

Yes, I use these two plugins daily and they've been working fine. I believe it's an issue on your side.


Sorcery scopes

Is sorcery able to work with different scopes, like Devise do? For example, my app has totally different user types, with different profiles, controllers etc. Using Devise i just can go with "rails g devise user/teacher/student" and it will generate new scope. Can i do something like this in Sorcery?

Currently no.
I've been thinking of switching Sorcery to use Warden, so a future version might have this capability.


Stop embedding links

Escapes urls with <>

@GriffinHeart cool but pls fix prettier formatting!

@sct Should be g2g for review, squash on merge if you want to get rid of those lint commits.


Move registration of FlushHostnameCache to TenancyServiceProvider::boot

Description
I moved the registration of FlushHostnameCache to TenancyServiceProvider::boot.
Motivation and context
The event dispatcher, which is required for model events, is only available in the boot method of a service provider. This event dispatcher is setup in \Illuminate\Database\DatabaseServiceProvider::boot. Registering an event observer earlier will cause the model to be booted too early when it has no event dispatcher. A model will only boot once, so if the consumer has custom logic in his model boot method that requires an event dispatcher, that functionality is also silently broken (that's how I discovered this issue).
TL;DR
FlushHostnameCache is not activated.
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

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

Thanks for the great clarification. Totally makes sense!


Documentation!

Let's see about making use of sphinx-apidoc and the various other tools.

Closes #130.


RuntimeError: Error compiling objects for extension

pytorch 1.6. cuda10.1
when i run "./make.sh". I come cross this problem
"""
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/setuptools/command/build_ext.py", line 208, in build_extension
_build_ext.build_extension(self, ext)
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/distutils/command/build_ext.py", line 528, in build_extension
objects = self.compiler.compile(sources,
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 469, in unix_wrap_ninja_compile
_write_ninja_file_and_compile_objects(
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1228, in _write_ninja_file_and_compile_objects
_run_ninja_build(
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1529, in _run_ninja_build
raise RuntimeError(message)
RuntimeError: Error compiling objects for extension
"""

pytorch 1.6. cuda10.1
when i run "./make.sh". I come cross this problem
"""
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/setuptools/command/build_ext.py", line 208, in build_extension
_build_ext.build_extension(self, ext)
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/distutils/command/build_ext.py", line 528, in build_extension
objects = self.compiler.compile(sources,
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 469, in unix_wrap_ninja_compile
_write_ninja_file_and_compile_objects(
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1228, in _write_ninja_file_and_compile_objects
_run_ninja_build(
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1529, in _run_ninja_build
raise RuntimeError(message)
RuntimeError: Error compiling objects for extension
"""

Yes, I met the same error

pytorch 1.6. cuda10.1
when i run "./make.sh". I come cross this problem
"""
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/setuptools/command/build_ext.py", line 208, in build_extension
_build_ext.build_extension(self, ext)
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/distutils/command/build_ext.py", line 528, in build_extension
objects = self.compiler.compile(sources,
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 469, in unix_wrap_ninja_compile
_write_ninja_file_and_compile_objects(
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1228, in _write_ninja_file_and_compile_objects
_run_ninja_build(
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1529, in _run_ninja_build
raise RuntimeError(message)
RuntimeError: Error compiling objects for extension
"""

Yes, I met the same error

what's your environment.
i solve my problem by changing pytorch version to 1.0.you can try.

pytorch 1.6. cuda10.1
when i run "./make.sh". I come cross this problem
"""
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/setuptools/command/build_ext.py", line 208, in build_extension
_build_ext.build_extension(self, ext)
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/distutils/command/build_ext.py", line 528, in build_extension
objects = self.compiler.compile(sources,
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 469, in unix_wrap_ninja_compile
_write_ninja_file_and_compile_objects(
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1228, in _write_ninja_file_and_compile_objects
_run_ninja_build(
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1529, in _run_ninja_build
raise RuntimeError(message)
RuntimeError: Error compiling objects for extension
"""

Yes, I met the same error

what's your environment.
i solve my problem by changing pytorch version to 1.0.you can try.

but，why cannot use pytorch 1.6

pytorch 1.6. cuda10.1
when i run "./make.sh". I come cross this problem
"""
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/setuptools/command/build_ext.py", line 208, in build_extension
_build_ext.build_extension(self, ext)
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/distutils/command/build_ext.py", line 528, in build_extension
objects = self.compiler.compile(sources,
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 469, in unix_wrap_ninja_compile
_write_ninja_file_and_compile_objects(
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1228, in _write_ninja_file_and_compile_objects
_run_ninja_build(
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1529, in _run_ninja_build
raise RuntimeError(message)
RuntimeError: Error compiling objects for extension
"""

Yes, I met the same error

what's your environment.
i solve my problem by changing pytorch version to 1.0.you can try.

but，why cannot use pytorch 1.6

pytorch 1.6. cuda10.1
when i run "./make.sh". I come cross this problem
"""
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/setuptools/command/build_ext.py", line 208, in build_extension
_build_ext.build_extension(self, ext)
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/distutils/command/build_ext.py", line 528, in build_extension
objects = self.compiler.compile(sources,
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 469, in unix_wrap_ninja_compile
_write_ninja_file_and_compile_objects(
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1228, in _write_ninja_file_and_compile_objects
_run_ninja_build(
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1529, in _run_ninja_build
raise RuntimeError(message)
RuntimeError: Error compiling objects for extension
"""

Yes, I met the same error

what's your environment.
i solve my problem by changing pytorch version to 1.0.you can try.

but，why cannot use pytorch 1.6

maybe the version of pytorch is not compatible

Hello,
Same error with pytorch=1.7 cudatoolkit=11.0.
Can we compile another DCNv2 that works with the new pytorch?
thanks

Hello,
Same error with pytorch=1.7 cudatoolkit=11.0.
Can we compile another DCNv2 that works with the new pytorch?
thank

I have the same environment as yours and solved the issue by using this version: https://github.com/jinfagang/DCNv2_latest

Hello,
Same error with pytorch=1.7 cudatoolkit=11.0.
Can we compile another DCNv2 that works with the new pytorch?
thanks

@nmc-costa
Did you manage to compile it on torch 1.7 cuda 11? I tried with DCNv2_latest and it throws the same error.
Maybe do I have to downgrade to pytorch 1.6?
Also: https://github.com/CharlesShang/DCNv2/pull/92

@farrufat-cimne I'm using https://github.com/jinfagang/DCNv2_latest great update, like @YuanZongjian mentioned.
Also beware of the the GCC version and NVCC version, if gcc>=7, NVCC>=10.

Was running into the same problem RuntimeError: Error compiling objects for extension
conda install pytorch==1.2.0 torchvision==0.4.0 cudatoolkit=10.0 -c pytorch

Fixed this repo https://github.com/CharlesShang/DCNv2/ for me

I had the same issue with pytorch==1.6 and it solved with pytorch==1.2.0

I fixed with this enviroment :
cuda : 10.1
pytorch : 1.4

pytorch 1.6. cuda10.1
when i run "./make.sh". I come cross this problem
"""
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/setuptools/command/build_ext.py", line 208, in build_extension
_build_ext.build_extension(self, ext)
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/distutils/command/build_ext.py", line 528, in build_extension
objects = self.compiler.compile(sources,
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 469, in unix_wrap_ninja_compile
_write_ninja_file_and_compile_objects(
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1228, in _write_ninja_file_and_compile_objects
_run_ninja_build(
File "/home/lxb/anaconda3/envs/torch/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1529, in _run_ninja_build
raise RuntimeError(message)
RuntimeError: Error compiling objects for extension
"""

Yes, I met the same error

I fixed with this enviroment :
cuda : 10.1
pytorch : 1.4

Was running into the same problem RuntimeError: Error compiling objects for extension
conda install pytorch==1.2.0 torchvision==0.4.0 cudatoolkit=10.0 -c pytorch

Fixed this repo https://github.com/CharlesShang/DCNv2/ for me

The Solution :D


x/sys/unix: ./mkall.sh failes with readlink: illegal option -- f

What version of Go are you using (go version)?
go version go1.13.4 darwin/amd64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/uparmar/Library/Caches/go-build"
GOENV="/Users/uparmar/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/Users/uparmar/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.13.4/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.13.4/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="0"
GOMOD="/Users/uparmar/Development/sys/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/l7/x5hcvngd5m99k62kxv7srrdr0000gn/T/go-build301769780=/tmp/go-build -gno-record-gcc-switches"

What did you do?
export GOOS=linux
export GOARCH=amd64
./mkall.sh from sys/unix directory.

What did you expect to see?
No error
What did you see instead?
readlink: illegal option -- f
usage: readlink [-n] [file ...]
docker: Error response from daemon: create .: volume name is too short, names should be at least two alphanumeric characters.

darwin readlink doesn't support -f option.


Kernel caching (probably) does not take into account line numbers

When working on https://github.com/openai/triton/pull/2589, I hit a situation that I hypothesize shows that Triton does not take source line numbers into account when deciding whether we get a kernel compilation cache hit.
The linenos are compiled into the cubin, so they definitely should form part of the cache key.
Filing this bug so I don't lose track of it.
cc @Jokeren

I think you can just extract the first lines of the functions, since we do source code comparison already

Or you can modify DependenciesFinder to hash also line numbers, which is probably more accurate.

I can confirm that it's a bug


feature: wrappid-app template versioning

Description
Currently there is no template versioning, so there is no way to know which template version user's wrappid-app is using.
Proposed solution
Add templateVersion key in package.json
Update 6 Feb 2024
Use wrTemplateVersion key in package.json
Update 14 March 2024
Use wrTemplateVersion key in wrappid.meta.json

@anantakumarghosh is still working on this as of now

Closing as done.


GitHub actions

After a cool discussion with Eric, I'll start experimenting with GitHub actions/workflows. See https://github.com/pyjanitor-devs/pyjanitor/tree/dev/.github/workflows for inspiration to kick off tests, builds, version bumps and releases etc based on certain triggers.
There is a lot that can be done here, so start simple :)

Closing this because @juanitorduz has kicked this process off with GitHub Actions. This particular issue is too vague, so we'll come up with more specific issues in terms of using GitHub actions.


add force option for gulp build

Allow to force the build to ignore jshint and jscs errors

this won't work with jscs until #125


Windows fails w. Kitchen b/c Unix Socket usage

Running kitchen create fails with Docker-for-WIndows.
STEPS
> kitchen create

The config is the following:
---
driver:
  name: docker

provisioner:
  name: chef_zero
  log_level: <%= ENV['CHEF_LOG_LEVEL'] || "auto" %>

verifier:
  name: inspec
  sudo: true

platforms:
  - name: ubuntu-16.04

suites:
  - name: default
    run_list:
      - recipe[ez_apache::default]
    verifier:
      inspec_tests:
        - ../../../inspec/ez_apache/test/integration
      attributes:
        service_type: sysv
    attributes:

EXPECTED RESULTS
Will be able to do kitchen create the same way I can for Linux, or Mac w/ Docker-for-Mac.
The plug-in should detect it is in Windows environment, and use an alternative means to communicate to Docker service.  The docker client commands work fine, as well as docker python library through docker-compose.
ACTUAL RESULTS
The default behavior is to use the Unix socket, which well, on Windows, is not going to work very well.
>>>>>> ------Exception-------
>>>>>> Class: Kitchen::ActionFailed
>>>>>> Message: 1 actions failed.
>>>>>>     Failed to complete #create action: [Expected process to exit with [0], but received '1'
---- Begin output of docker -H unix:///var/run/docker.sock build -f Dockerfile-kitchen20181116-14652-1p671jw . ----
STDOUT:
STDERR: protocol not available
---- End output of docker -H unix:///var/run/docker.sock build -f Dockerfile-kitchen20181116-14652-1p671jw . ----
Ran docker -H unix:///var/run/docker.sock build -f Dockerfile-kitchen20181116-14652-1p671jw . returned 1] on default-ubuntu-1604
>>>>>> ----------------------
>>>>>> Please see .kitchen/logs/kitchen.log for more details
>>>>>> Also try running `kitchen diagnose --all` for configuration

As a workaround I am doing this:
<% @socket = ENV['DOCKER_HOST'] ||
     RUBY_PLATFORM =~ /mingw32/ ?
       'npipe:////./pipe/docker_engine' :
       'unix:///var/run/docker.sock'
%>
driver:
  name: docker
  socket: <%= @socket %>

I am making tutorial material that will work on Win/Lin/Mac, and would prefer not to overwhelm new comers.

I wonder if its possible to use this project to launch Windows Containers in a Windows Docker Host.

@lmayorga1980 Have you tried testing the code in PR #340? We’re looking for people to test and provide feedback as we continue to refine the code.

Definitely will test, I have a Windows 2019 Core Docker host ready to go...

@lmayorga1980 Thanks — please update the PR #340 if you run into any issues.

Release 2.10 contains Windows support now. Please open a new issue if any problems are found using this release. Thanks.


Fix theming error in EditorHelp due to order of operations issue

Fixes https://github.com/godotengine/godot/issues/65407. The code was running on parent's tree enter/theme changed, but was using child RTL to fetch items before it ever could access the theme.

Thanks!


Fill in README

Title. After #03 is merged we need to update the README

Initial fill is done, missing things have their own issues now


Admin source-user relationship: display source siglum instead of ID

Would it be more useful to display the source's siglum on the source-user relationship of the User admin page instead of the source ID? This would probably make it easier to see which sources are already assigned to a use without having to look up the ID. I'm not sure if this is something that can actually be done with this admin functionality, but I can look into it.

yes, that would be great!

I see, the page will display like this until the user clicks "save" after adding the source-user relationships and then the siglum will show as desired.


Create initial analytics function

Description
Initial analytics to know when the user does stuff (and if it breaks!)
Type of change

New feature (non-breaking change which adds functionality)

Should we save owner and repo?

With the backend running and connected to airtable in the correct branch, do a watermelon.start query.
It should request to your localhost in the port 3001 and that should send it to the db.


Fix build/target config file ordering

Fix the order of values for string (as well as the pre-build vec) values when parsing from config files and environment variables.
The old behavior was:

Envvar build.
Envvar target
Config target
Config build

The new behavior is:

Envvar target
Config target
Envvar build.
Config build

Some functions were also made more general.

bors r+


GET TIME STAMP FIELD

DATA bar TYPE p.
GET TIME STAMP FIELD bar.
ASSERT NOT bar IS INITIAL.
WRITE bar.

implemented in https://github.com/abaplint/transpiler/pull/347


subunit composition

could you get me. a list of the contents of /controlled_curation="term=subunit composition,
I cant get them by grep
chromosome2.contig:FT                   /controlled_curation="term=subunit composition,
chromosome2.contig:FT                   /controlled_curation="term=subunit composition, monomeric;
chromosome2.contig:FT                   /controlled_curation="term=subunit composition,
chromosome2.contig:FT                   /controlled_curation="term=subunit composition,
chromosome2.contig:FT                   /controlled_curation="term=subunit composition, monomeric;
I need to document how they are constructed and I can't remember

 dimeric
 heteromeric(10)
 heteromeric(12)
 heteromeric(2)
 heteromeric(3)
 heteromeric(4)
 heteromeric(6)
 heteromeric(8)
 homodimeric
 homomeric(1)
 homomeric(16)
 homomeric(2)
 homomeric(2), RING domain-dependent
 homomeric(3)
 homomeric(4)
 homomeric(5)
 homomeric(6)
 homomeric(8)
 homooligomeric
 homooligomeric in h- cells
 homotetrameric
 monomeric
 oligomeric
 pentameric
 tetrameric(1)

Sorry to be a pain but ce we get the genes associated with these (and numbers)?
I can't figure out exactly what they mean, I have been winging it! or ignoring, but we should capture really.
Some things look like a different way of saying the same thing!
no hurry , I wont be looking at this soon

I thought I'd get it done while I had the SQL query in front of me:
subunit_composition.csv


[Move] add script for tiered mint

Script for Treasury compliance account to invoke tiered mint and creation of DD
cargo test mint in transaction_builder/ tests
added some utils for non synthetic coins/cointypes

@bors-libra -r=sblackshear

I think bors doesn't understand -r, it should be r=sblackshear

@bors-libra r=sblackshear

:pushpin: Commit ed77954 has been approved by sblackshear

:hourglass: Testing commit ed7795471894a4d08daf673da9de17031c16faf2 with merge 4c7e84345b308b839746c9a66e94f2364b449905...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: sblackshear
Pushing 4c7e84345b308b839746c9a66e94f2364b449905 to master...


applicationCategory ENUM Failure

The JSON value could not be converted to engine.api.Models.Engine.Enums.ApplicationCategory.
Path: $.applicationCategory | LineNumber: 5 | BytePositionInLine: 39.

Convert text Photo Design to Photo_Design to solve problem.


Updating prettier

Issue link

#323
Auto-close the issue?

Closes #323
Types of changes
Technical debt (a code change that doesn't fix a bug or add a feature but makes something clearer for devs)

@marcelomorgado Merge conflicts in this one (likely just a new npm i required)

@marcelomorgado Merge conflicts in this one (likely just a new npm i required)

Done

@marcelomorgado Weird, only package-lock.json changed?

@marcelomorgado Weird, only package-lock.json changed?

Oh, I've forgotten this one, just done. Thanks


object-fit no longer working on macOS Sonoma / Safari 17

I have three video feeds set up to be in a 16:9 aspect ratio and fill the entire window (no letterboxing):
style: 'video {aspect-ratio: 16/9; object-fit: fill;} .mode {display: none}'
Unfortunately after updating my devices to macOS Sonoma and Safari 17, the "fill" mode no longer works and I have letterboxing. Notably it's still 16:9. See example image below.

Is there any solution by changing my configuration in Home Assistant? Or is it a bug or issue with Safari 17?
Thanks so much.

Have the same issue


Issue connecting to rancher api?

I read through this: https://github.com/objectpartners/rancher-ecr-credentials/issues/5  and used the identical compose syntax but still get the error below?
7/8/2016 3:57:13 PMUpdating ECR Credentials
7/8/2016 3:57:13 PMFailed to create rancher client: Get : unsupported protocol scheme ""
7/8/2016 3:57:13 PMError updating ECR, Get : unsupported protocol scheme "

From the error I assume i need to specify a label that tells it what scheme to use to access the rancher api?

I worked around this by setting the CATTLE_URL env var in the rancher config for this stack


Add getUid() to UidProcessor

Would be nice to have an easy way to get Uid from UidProcessor. I have a scenario where I want to pass a parent's Uid to children processes for log grouping.

Another option if deemed better is to change $uid to protected. Then I can extend the UidProcessor class and add the getter there.

@jaredm4 making it protected is actually not better: it means we have to support the extension forever (well, at least forever in 1.x) and it also makes it possible for a child class to mutate it, which is opening it more than adding a getter


Issue318 add synth diag ids

Motivations:

For non-experienced users, the imas interface class MultiIDSLoader shall provide an easy way to:

load diagnostics geometries
load experimental diagnostic data
load all ids necessary to compute synthetic signal for a given diagnostic

The current status allowed to do that in ~5-6 lines, provided the user knows where to get the info
This need being a typical use-case, it should be easier / faster

Main changes:

All diagnostocs for which a synthetic diagnostic exists are tabulated inside the class hidden attributes, along with their corresponding ids (core_profiles...)
A method allows to instantaneously get the ids necessary for synthetic data (get_input_for_synthdiag())
A method, add_ids_synthdiag(), calls the previous to automatically add and load then necessary ids, if necessary with a custom idd defined just-in-time.
Two keywords added at instanciation (ids_base and synthdiag) for users who want to load everythong at once from a single idd)
(extra) a get_events() method is implemented to easily print the chronologial events in a shot from ids 'pulse_schedule'

Issues:

Fixes, in devel, issue #318

Example (with a custom idd):
Before (5 lines, a bit complicated)
import tofu as tf

# load experimental data and wall for geometry, and pulse_schedule for tIGNITRON
multi = tf.imas2tofu.MultiIDSLoader(shot=55855, ids=['pulse_schedule', 'wall', 'interferometer'], get=False)

# Find which ids are necessary for synthetic signal calculation
print(multi._didsdiag['interferometer']['synth']['dsig'])
{'core_profiles': ['t', '1dne', '1drhotn'],
 'equilibrium': ['t', '2drhotn', '2dmeshNodes', '2dmeshFaces']}

# add these ids to multi
multi.add_ids(ids=['core_profiles', 'equilibrium'], user='CB165101', get=False)

# Load everything that was added
multi.open_get_close()

# Calculate signal
sig = multi.calc_signal('interferometer')

Now (3 lines, simpler)
import tofu as tf

# load experimental data (wall and pulse_schedule are automatically included)
multi = tf.imas2tofu.MultiIDSLoader(shot=53259, ids='interferometer', get=False)

# Add ids necessary for calculation (automatically identified: tabulated in class attribute)
multi.add_ids_synth_diag(user='CB165101', run=1)

# Calculate signal
sig = multi.calc_signal('interferometer')

Hello @Didou09! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file tofu/imas2tofu/_core.py:

Line 1334:80: E501 line too long (82 > 79 characters)
Line 1343:25: E221 multiple spaces before operator
Line 1656:37: E228 missing whitespace around modulo operator
Line 1657:37: E228 missing whitespace around modulo operator
Line 2098:5: E303 too many blank lines (2)
Line 2116:80: E501 line too long (81 > 79 characters)


scorefunctions.py is added to chromoscore folder

Summary :memo:
Write an overview about it.
Details
Describe more what you did on changes.

(...)
(...)

Bugfixes :bug: (delete if dind't have any)

Checks

[ ] Closed #798
[ ] Tested Changes
[ ] Stakeholder Approval

@gfudenberg please take a look at chromoscore/scorefunctions.py at your convenience.


simplify code contributions by fully automating thesetup with Gitpod

Hi! :slightly_smiling_face:
This Pr simplifies code contributions by fully automating the dev setup with Gitpod(a free online VS Code like IDE) for contributing, with a single click it'll launch a workspace and automatically:

clone the vue-toasted repo.
install the dependencies.
start the webserver.

So that anyone interested in contributing can start straight away.
You can give it a try on my fork of the repo via the following link:
https://gitpod.io/#https://github.com/nisarhassan12/vue-toasted
This is how it looks:

Thank you !


Fix/420 model serving dashboard tests

During the execution of the 2.9 QGs we've found that some of the tests related with the suite 420_model_serving were not working properly. Trying to fix it we've found that also there were some race condition
I've modified the tests a little bit in order to use it's own resources (DSP name 'model-serving-project' is used in multiple suites, which is not a good practice) and also now all the DSProjects related with this suite are deleted via CLI
Test results:

lgtm. However I'd ask @manosnoam to review the change because he brought in the re-use of existing projects in these tests.

Test Results of the complete Suite after the latest modifications:

Test results with the latest changes:


Store data in memory

Data like settings, number of posts, number of pages doesn't change until there is new content.
Requested:

Data that does not change, should be stored in memory to speed up generation of pages.

Solution:

Use the singleton classes and private properties to store the data
It should be fetched the first time requested
Data should be removed

when the engine reloads (python code update)
when there are changes in the data directory
when there are changes in the themes directory

Implemented, but must be refined.  New issues will be created.


Add example distributed tree k-nearest neighbors search

I redid the example based on the changes in ArborX in the recent months. It eliminated a lot of boiler plate code.

Thoughts on whether and what to print to hints that something happened when one run the example.

Probably should. Otherwise, it is a silent run.

Thoughts on whether and what to print to hints that something happened when one run the example.

Maybe just print ranks, indices and offsets similar to https://github.com/arborx/ArborX/blob/bf6f89b7d3d08fad3b8b0399f59aed0fe389df1b/examples/simple_intersection/example_intersection.cpp#L62-L72.

Can't approve obviously but fine by me


On Uncertainty Minimum Query Selection

May I ask the author, does uncertainty minimum query selection have anchor points?

No. We did not introduce anchor points, but selected from the feature tokens output by the encoder.


querying the "description" returns empty string

{
product(id: ...) {
  id
  description
  }
}

the description returns as an empty string, descriptionJson works though.

description_json needs to be migrated to description, here is an issue about it: https://github.com/mirumee/saleor/issues/5726


Make Card Hotbar take its x position into account for CSS flexing

Currecntly, if you move the Card Hotbar to the right (via settings or CSS directly), it will cover the sidebar. A particular issue for @cswendrowski.

Should be solved in 1.4.3.


[Code quality] We need to make sure all the errors from CI are clear

Current Behavior

The CI test cases are always failed for many reasons e.g:  code quality, test cases are init failed ...
Expected Behavior

Make sure every test-cases we have are working well and makes sure it is useful.
Screenshots/Logs

Environment

Host OS: Mac Linux Windows
Platform: Docker or Kubernetes
Meshery Server Version: stable-v
Meshery Client Version: stable-v
Meshery Adapter Version: stable-v

Contributor Guides and Resources

📝 Meshery Adapters Spreadsheet
🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

More detail about the golangci-lint result  https://github.com/meshery/meshery-linkerd/actions/runs/4868680555/jobs/8682362960

@Aisuko I'm working on fixing the lint errors

Thank you @ayushthe1

PR 506 for the golangci-lint and the e2e test cases also need to be checked the status.


Fix passing null to non-nullable parameters of fromJson function

PHP version: 8.1.7
Laravel version: v9.21.6
Deprecated Features: Passing null to non-nullable parameters of built-in functions
The string type parameters 1 of json_decode function not accept null type when the PHP version greater than 8.1.

As you can see from the docblock, you're required to pass a string here.


Remove support for the old-style messages file format

Sorry I took so long, Mike! I'll try to be more responsive in the future.


X.L.ConditionalLayoutModifier: Init

Description
Include a description for your changes, including the motivation
behind them.
Checklist

[x] I've read CONTRIBUTING.md

[x] I've considered how to best test these changes (property, unit,
manually, ...) and concluded: XXX

[x] I updated the CHANGES.md file

[ ] I updated the XMonad.Doc.Extending file (if appropriate)

Include a description for your changes, including the motivation
behind them.

We'd really appreciate this. Sure, in your case we can go and look into your dotfiles for an example of usage and try to deduce the motivation, but in general this isn't feasible. We like and encourage contributions, but let's not turn this into “I'll throw undocumented code at you to save a couple minutes of my time at the expense of yours”, please. There must be a middle ground somewhere. We're happy to take blank PRs if agreed upon out-of-band (IRC, ZuriHac, …), but some communication ought to happen.

[x]  I've considered how to best test these changes (property, unit,
manually, ...) and concluded: XXX

“XXX” is not a very solid description of how this was (intended to be) tested. It's okay to say “I tried it for a day with the most basic of layouts” or “It seems to work even with MultiToggle” and we can take it from there. It's not meant to shame people into writing property tests. Maybe it fails at that, so let's improve the wording?
Unfortunately it seems this won't work well with any layout modifier that needs correct processing of ReleaseResources/Hide events, especially in multi-head configurations. See e.g. https://github.com/xmonad/xmonad-contrib/issues/75 and https://github.com/xmonad/xmonad-contrib/issues/354#issuecomment-647149568. Arguably IfMax and PerScreen still haven't been fixed so it's not entirely fair to hold you to higher standards. But now that we know about the bug and how to design it better, we might as well try to. :-)
Also, as the example in your dotfiles demonstrates, the condition itself won't work well in multi-head layouts, as currentWorkspace is the focused workspace, not the one being layouted. If you go with my suggestion and cache the condition state in the layout modifier record, you won't need to evaluate the condition outside of modifyLayout, and therefore you can pass WindowSpace to the condition.
So, to wrap it up, this is definitely a good idea. A generalization of IfMax/PerScreen is something we missed. It would be great, however, if we managed to implement it properly this time so that we can close a couple bugs and possibly deprecate IfMax/PerScreen in favor of this better one.

We'd really appreciate this. Sure, in your case we can go and look into your dotfiles for an example of usage and try to deduce the motivation, but in general this isn't feasible. We like and encourage contributions, but let's not turn this into “I'll throw undocumented code at you to save a couple minutes of my time at the expense of yours”, please. There must be a middle ground somewhere. We're happy to take blank PRs if agreed upon out-of-band (IRC, ZuriHac, …), but some communication ought to happen.

Sure, that is understandable. That said:

The code is not undocumented. This description takes place twice in the PR:

	This module provides a LayoutModifier that modifies an existing
    LayoutModifier so that its modifications are only applied when a particular
    condition is met.

I'm not really sure what to write in the description other than that. That pretty much describes the entire intent of this change without omitting any details that arent related to the implementation.

“XXX” is not a very solid description of how this was (intended to be) tested. It's okay to say “I tried it for a day with the most basic of layouts” or “It seems to work even with MultiToggle” and we can take it from there. It's not meant to shame people into writing property tests. Maybe it fails at that, so let's improve the wording?

Sure.

Unfortunately it seems this won't work well with any layout modifier that needs correct processing of ReleaseResources/Hide events, especially in multi-head configurations. See e.g. #75 and #354 (comment). Arguably IfMax and PerScreen still haven't been fixed so it's not entirely fair to hold you to higher standards. But now that we know about the bug and how to design it better, we might as well try to. :-)`

Did you see this comment above handleMess
  -- This function is not allowed to have any downstream effect, so it seems
  -- more reasonable to simply allow the message to pass than to make it depend
  -- on the condition.

now, arguably, this doesn't quite solve the problem ALL the way, because I suppose that layouts implementing only handleMessOrMaybeModifyIt would still not receive these messages.
But this should get us at least 50% of the way there, right?
This is not quite a generalization of IfMax or PerScreen, since IfMax chooses between layouts, rather than layout modifiers, but some of the same issues may be exhibited.
Still this concept of using a type class to represent an arbitrary condition could and should be used to generalize those 2. I may add that generalization to this PR and change the module to offer utilities for conditionally choosing layouts.

Also, as the example in your dotfiles demonstrates, the condition itself won't work well in multi-head layouts, as currentWorkspace is the focused workspace, not the one being layouted. If you go with my suggestion and cache the condition state in the layout modifier record, you won't need to evaluate the condition outside of modifyLayout, and therefore you can pass WindowSpace to the condition.

Good point. I definitely did not think much about multi-head layouts. Still, this is actually not really a problem with the PR per se, as much as it is an issue with any condition that depends on the state of the workspace whose layout is being adjusted.
Caching as you suggest is one way to approach this, but I do worry that there could be some corner cases here. Another possibility is passing in the workspace whose layout is being performed to the condition. This would allow a condition where we look at the current layout of whichever workspace we are performing the update for.

and possibly deprecate IfMax/PerScreen in favor of this better one.

Instead of deprecation, why not just implement those in terms of the new approach.

The code is not undocumented.

I should've said “underdocumented”, perhaps. Most modules in contrib come with example usage as many users don't know enough Haskell to understand it from the types themselves, and I would've appreciated an example as well, just to better understand the motivation for this.
(Obviously the documentation doesn't need to be perfect in a draft PR. It doesn't need to be there at all, in fact. As long as the motivation and the draft state of the PR is communicated somewhere.)

This description takes place twice in the PR:
	This module provides a LayoutModifier that modifies an existing
    LayoutModifier so that its modifications are only applied when a particular
    condition is met.

I'm not really sure what to write in the description other than that. That pretty much describes the entire intent of this change without omitting any details that arent related to the implementation.

As hinted above, an example would have been nice, but even just copypasting this bit would have been better than leaving the “Include a description for your changes, including the motivation behind them.” template intact.
Ideally and in general, the documentation of the module explains the what (what the module does, what use cases it's good for) and the how (examples of usage, etc.), possibly even the why (helps to understand why one might use it in their config). The pull request cover letter and/or the commit message¹ then include a brief summary of the what and a detailed why/how (motivation, an example, why something's being fixed in a certain way, notes from bug investigations, benchmarks, …).
¹) (in the case on single-commit PRs, the commit message becomes the cover letter automatically, or at least hub does it)
It's probably time we revised https://github.com/xmonad/xmonad/blob/master/CONTRIBUTING.md, made it shorter, spent less time explaining git rebase and generally bringed it up to our current preferences. But let me ask a honest question here (and please be honest both with us and with yourself):
Would you have read it or would you have just submitted a PR with false [X]s anyway?
(Not trying to be pedantic or something, I really want to know if it's worth the time.)

Did you see this comment above handleMess
  -- This function is not allowed to have any downstream effect, so it seems
  -- more reasonable to simply allow the message to pass than to make it depend
  -- on the condition.

now, arguably, this doesn't quite solve the problem ALL the way, because I suppose that layouts implementing only handleMessOrMaybeModifyIt would still not receive these messages.

I did indeed miss this one.

But this should get us at least 50% of the way there, right?

Yep, solves some of the problems that I had in mind, definitely not all of them.
One issue still there even with your latest implementation is that if a decorations modifier is made optional, those decorations don't disappear when the condition flips to false.

and possibly deprecate IfMax/PerScreen in favor of this better one.

Instead of deprecation, why not just implement those in terms of the new approach.

Yes, that'd be best.

I only had a quick look at the implementation. Looks a bit simpler, so that's good. The function added to LayoutModifier makes sense, and it's really stupid that it wasn't there in the first place—would make some things easier (even in other modules). Makes the API uglier and more complicated now.
I still believe it's worth trying to implement this as a layout instead of layout modifier, as I suspect the implementation would get even simpler and more robust. I'd be happy to give it a try, although I can't promise when.

Would you have read it or would you have just submitted a PR with false [X]s anyway?

Man you're really putting me through the ringer over this whole thing. I think that's actually kind of unfair, since I actually did:

Update the changelog
Provide a module level docstring

Also you said that I submitted a PR with False [X]'s which implies:

That there were multiple checks that were false
That it was False that I had "I've considered how to best test these changes (property, unit, manually)", which is simply not the case. I HAD actually tested things, with a few different layout modifiers (not only the one that actually made it into my dotfiles) and things did work. I do not have a multi-head setup, so that issue did slip by, but I did make an effort to test things.

Honestly I simply did not see the "and concluded XXX" part. I assumed that because there was a checkbox there it was a sort of "I have done some testing of these changes", because that sort of thing appears in many other repos, without the need to fill something out there. I think that having a checkbox there, sort of suggests to the user that they just need to check something off if they have done it. If you have to fill out text anyway, I don't think there's a point in also having the checkbox (but thats just my opinion).

I still believe it's worth trying to implement this as a layout instead of layout modifier, as I suspect the implementation would get even simpler and more robust. I'd be happy to give it a try, although I can't promise when.

Sure. Might also be able to make it more extensible and similar to something where you are choosing between multiple layouts.

Man you're really putting me through the ringer over this whole thing. I think that's actually kind of unfair

That's why I included the “(Not trying to be pedantic or something, I really want to know if it's worth the time.)” bit. What feels like me grilling you is just me trying to understand what makes our PR template fail in contact with real people. I mean, you're not the first who didn't fill it in completely, but since your contributions are of much higher quality than all the other people who failed the template, I thought I'd ask about it and learn something. Sorry about drilling so deep, and for being overly direct.

Also you said that I submitted a PR with False [X]'s which implies:

That there were multiple checks that were false

I did indeed assume that someone who contributed in the past and who decided not to fill in the description might skip reading the CONTRIBUTING.md as well. And I did not mean to judge that, as I imagine I might skip it too, possibly. Just wanted a second opinion. (As you might have realized by now, I'm not the best communicator.)

That it was False that I had "I've considered how to best test these changes (property, unit, manually)", which is simply not the case. I HAD actually tested things, with a few different layout modifiers (not only the one that actually made it into my dotfiles) and things did work. I do not have a multi-head setup, so that issue did slip by, but I did make an effort to test things.

Honestly I simply did not see the "and concluded XXX" part. I assumed that because there was a checkbox there it was a sort of "I have done some testing of these changes", because that sort of thing appears in many other repos, without the need to fill something out there. I think that having a checkbox there, sort of suggests to the user that they just need to check something off if they have done it. If you have to fill out text anyway, I don't think there's a point in also having the checkbox (but thats just my opinion).

This is valuable feedback, and thank you for spending the time to write it. So I guess replacing "XXX" with something like "" might help?
Hope this clears up any misunderstandings. Sorry again.

So, I felt like trying to implement my idea today, and this is what I ended up with:
-- | 'ModifiedLayout' extended with a condition and its last evaluation result
-- (for methods that can't evaluate it).
data CondModifiedLayout c m l a = CondModifiedLayout Bool c (ModifiedLayout m l a) deriving (Read, Show)

instance (ModifierCondition c, LayoutModifier m a, LayoutClass l a, Typeable c, Typeable m)
    => LayoutClass (CondModifiedLayout c m l) a
  where
    runLayout (W.Workspace i cml@(CondModifiedLayout a c _) ms) r = do
        a' <- shouldApply c i
        cml' <- if a == a' then pure Nothing else Just . switch <$> hide cml
        fmap (<|> cml') <$> run (fromMaybe cml cml')
      where
        hide x = fmap (fromMaybe x) $ handleMessage x (SomeMessage Hide)
        switch (CondModifiedLayout a' c' ml') = CondModifiedLayout (not a') c' ml'

        run (CondModifiedLayout True c' ml) =
            fmap (fmap (CondModifiedLayout True c')) <$> runLayout (W.Workspace i ml ms) r
        run (CondModifiedLayout False c' (ModifiedLayout m l)) =
            fmap (fmap (CondModifiedLayout False c' . ModifiedLayout m)) <$> runLayout (W.Workspace i l ms) r

    handleMessage (CondModifiedLayout a c ml@(ModifiedLayout lm l)) m
        | Just ReleaseResources <- fromMessage m = fmap (CondModifiedLayout a c) <$> handleMessage ml m
        | a                                      = fmap (CondModifiedLayout a c) <$> handleMessage ml m
        | otherwise          = fmap (CondModifiedLayout a c . ModifiedLayout lm) <$> handleMessage l m

    description (CondModifiedLayout a _ ml@(ModifiedLayout _ l)) = if a then description ml else description l

conditional :: (ModifierCondition c) => c -> (l a -> ModifiedLayout m l a) -> (l a -> CondModifiedLayout c m l a)
conditional c ml = CondModifiedLayout True c . ml

Appears to work well with the following config:
import Data.List

import XMonad
import XMonad.Layout.ConditionalLayout
import XMonad.Layout.Tabbed
import qualified XMonad.StackSet as W

data MyCond = MyCond deriving (Read, Show)
instance ModifierCondition MyCond where
    shouldApply _ t = do
        w <- gets $ find ((t ==) . W.tag) . W.workspaces . windowset
        let ws = W.integrate' . W.stack <$> w
        pure $ Just 4 < (length <$> ws)

main :: IO ()
main = xmonad def{
    terminal = "urxvt",
    layoutHook = conditional MyCond (addTabsAlways shrinkText def) $ layoutHook def
}

Let me know what you think.
(I'll probably try to get rid of the gets $ find ((t ==) . W.tag) . W.workspaces . windowset later, and also take a stab at implementing CondChoose.)

Looks great!

(I'll be off for over a week so do feel free to explore this further, or just let it sit here until I'm ready to polish it and merge.)

@liskin What happened with this.
I'd really like to try to get some of my PRs merged, because its sort of annoying to continue maintaining my own fork of xmonad-contrib.
It seems like you have a strong preference for the approach you took, which is fine. I just think this is a very natural change and something that can accomplish this goal should be merged.

@colonelpanic8 I think it is unlikely that liskin will pick this back up anytime soon. I also think merging this would be great, and don't have any strong preferences regarding what approach we need to take, as long as all mentioned issues are actually addressed. Do I read

It seems like you have a strong preference for the approach you took, which is fine.

correctly that you'd rather implement your approach?

Don't really have a strong preference either way except that I already have the modifiers written the way that I ser it up.

What happened with this.

Life got in the way, hard. I don't even remember what was wrong in Aug 2021, but I can confidently say everything is at least dozen times worse now. So I definitely agree with @slotThe's suggestion that waiting for me to polish it and merge by the end of August is probably a bad idea, primarily because 2 years have passed from the actual August.
But yeah, I do have a strong preference for an approach that

works well with decorated layouts (https://github.com/xmonad/xmonad-contrib/issues/75)
can be trivially extended from just a conditional layout modifier into a conditional choose between two layouts
and therefore lets us implement IfMax, PerScreen, PerWorkspace in terms of it, finally solving #75
happens to be simpler and doesn't require changes to the LayoutModifier class

I was hoping someone would just take my code, add a bit of docs, a changelog entry, and I'm sure we would've merged that ages ago. Instead people just keep asking "is it done yet?" and "what are we waiting for?" :-(
Anyway, I don't want to block progress either. If everyone else is happy with your approach (which I don't think handles decorated layouts well…), do whatever you all want. My capacity to care is limited.
(Arguably I should've just done what I promised to do back in 2021 instead of wasting time writing this comment. Oh well.)

@geekosaur it's just shouldApply. The reason is that CondModifiedLayout—being a layout—needs to implement Read and Show, and these conditions get passed as parameters to it


I cannot start chrome instant using PowerShell and Selenium module.

I am not sure what I am missing, but I installed Selenium module under PowerShell 7.1 and I cannot start chrome instance. I followed the steps below:
Install-Module -Name Selenium -AllowPrerelease
$driver = start-SeChrome

I get this error:

start-SeChrome: The term 'start-SeChrome' is not recognized as a name of a cmdlet, function, script file, or executable program.

I appreciate your help.

Do you get an error of some kind for Import-Module Selenium?

I solved the problem by removing the latest one, install 3.1.1 and put the file chromedriver.exe which is compatible with my Chrome browser, must put it under powershell selenium folder to replace the existing one.
Why do we have to struggle? Why the latest version is not working?
When I installed the latest version I didn't get any error.

@tarekahf the problem is what the message says.
There is no cmdlet with the name "start-SeChrome".
In fact you installed the module with parameter "-AllowPrerelease" you should have been installed V4.0.0-preview3.
In this version all cmdlets Start-[yourBrowser] has been merged with Start-SeDriver.
In your case you have to do the following:
Start-SeDriver -Browser Chrome -StartURL "https://www.google.com" -Switches DisableAutomationExtension

Stop-SeDriver


\Android\sdk\build-tools\23.0.1\aapt.exe'' finished with non-zero exit value 1

Hi @mcnamee, Running the react-native-starter-app it fails in my environment with the error:
:react-native-device-info:processReleaseResources FAILED
FAILURE: Build failed with an exception.
* What went wrong:
Execution failed for task ':react-native-device-info:processReleaseResources'.
> com.android.ide.common.process.ProcessException: org.gradle.process.internal.ExecException: Process 'command 'C:\Users\Brian\AppData\Local\Android\sdk\build-tools\23.0.1\aapt.exe'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
BUILD FAILED

Is there a way to get a more detailed stack trace using the react-native run-android command? At the moment this error doesn't give enough detail as to what's up.
Thanks

Hey @BrianJVarley,
Have you tried running with the --stacktrace option in your gradle build parameters? Perhaps that will give you/us a better idea of what's going on here.
If that doesn't work you can try building the device-info package separately or re-link it to the project; let me know how you go.

@Jean-PierreGassin That's what I'm not quite sure of. How do I add the --stacktrace option to gradle. In the build.gradle file somewhere? Or another file?

Hi @BrianJVarley
Thanks for sharing your --info. I believe the issue relates specifically to react-native-device-info and may be resolved by upgrading your Android environment.
I'm running the latest Android SDK and Android Build Tools, and it all looks to be running well.
The following screenshots show my current Android SDK + Emulator settings (where this project runs without errors).

I encountered the same problem in a real device. If you managed to fix this problem let us know.

Solved: Thanks @vvasiloud

Thank you. On Windows, some file path are too long. Move the project to the root solve the problem. :-)

@vvasiloud
How would that be?
What folder would I have to move it to? Thanks

@GastonEDiaz Move it to root (C://) for example and see if it works

Putting the app to the root folder example C:\ solved the issue. Windows is always developer unfriendly. :-(


Is the compressor type right?

As part of #93 I just ran into an issue with the type of filters on zarr.Codec and zarr.Zarray which I then noticed was already fixed on main by #66.
I was wondering though if the type of compressor also needs to be changed from str to Codec as indicated in the zarr API docs?

I think this is zarr v2 vs v3 difference. In v2 there were separate filters and compressors, but in v3 those are both examples of codecs. Internally VirtualiZarr currently uses the word "codec" to refer to things that must be the same between arrays in order for them to be concatenatable, so the Codec class I defined in virtualizarr/zarr.py is just a way to keep the filter and compressor together.
Ideally all these differences would be handled by a compatibility class that I just import from zarr-python.


use var for function variables

when building in production with brunch for out frontend javascript, these files are run with 'use strict' which breaks when run since each of these functions is being set on the global context

Looks good! Merging...


ctkXnatSessionTest fails on windows, missing openssl

The test tries to connect to https://central.xnat.org which fails on windows if there is no openssl available. Qt does not distribute it directly for some license incompatibilites, but I wonder if we should add an option to the CTK build system to download openssl binaries and put them in the bin or lib directory to make things easier. Another option would be to modify the test in a way it tries plain http as well and solve the problem on the level of applications using CTK-XNAT. @espakm  , @jcfr : any preferences/opinions? cc @AndreasFetzer

Would it help to integrate an external project similar https://github.com/Slicer/Slicer/blob/master/SuperBuild/External_OpenSSL.cmake ?


GDP-mannose pyrophosphorylase complex

Suggested term label:
GDP-mannose pyrophosphorylase complex

Definition (free text)
A protein complex capable of catalyzing the reaction of GTP and mannose-1-phosphate to form GDP-mannose. The complex is a homodimer in most bacteria, whereas it is a heterodimer in most eukaryotes, composed of a catalytic beta subunit (GMPPB) and a non-catalytic alpha subunit (GMPPA).

Reference, in format PMID:#######
PMID: 33986552

Gene product name and ID to be annotated to this term
Human GMPPA and GMPPB (and orthologs).

Parent term(s)
is_a  GO:1990234 transferase complex
capable_of MF  GO:0004475 mannose-1-phosphate guanylyltransferase (GTP) activity
capable_of_part_of GO:0009298 GDP-mannose biosynthetic process

Synonym(s) (please specify, EXACT, BROAD, NARROW or RELATED)
GMPPA–GMPPB complex [EXACT]

a non-catalytic alpha subunit (GMPPA)

Would "regulatory alpha subunit (GMPPA)" be appropriate?

+[Term]
+id: GO:0120508
+name: GDP-mannose pyrophosphorylase complex
+namespace: cellular_component
+def: "A protein complex capable of catalyzing the reaction of GTP and mannose-1-phosphate to form GDP-mannose. The complex is a homodimer in most bacteria and a heterodimer in most eukaryotes. In humans, it is composed of a catalytic beta subunit (GMPPB) and a regulatory alpha subunit (GMPPA)." [GOC:sjm, PMID:33986552]
+synonym: "GMPPA–GMPPB complex" EXACT []
+is_a: GO:1990234 ! transferase complex
+relationship: capable_of GO:0004475 ! mannose-1-phosphate guanylyltransferase (GTP) activity
+relationship: capable_of_part_of GO:0009298 ! GDP-mannose biosynthetic process
+property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/27827" xsd:anyURI
+created_by: sjm
+creation_date: 2024-05-04T08:30:37Z


Create CodeList Api for resource Registry

Description
The resource registry would need to have some codelist available for selecting values for the resource enties
Additional Information
The CVSP standard gives guidelines of some of this
Spatial information
This property represents that area(s) a Public Service is likely to be available only within, typically the area(s) covered by a particular public authority.
Regarding usage of controlled vocabularies, the following requirements apply:
At least one value shall be chosen from the following controlled vocabularies: Continent; Country; Place; GeoNames.
To specify spatial coverage in Norway, the Norwegian Mapping Authority’s controlled vocabulary Administrative units should be used.
Type
The possible values for this property are described in EU’s controlled vocabulary Main activity.
The values is only available in EU languages. We need to add Norwegian.
The XML can be downloaded in XML format
Status
Norwegian extension: The value shall be chosen from ADMS Status Vocabulary (linked resource in RDF).
The number of values is limited
Thematic area
At least one value shall be chosen from EU’s controlled vocabulary EuroVoc. Los may be used in addition.
API informasjonsforvaltning
https://data.norge.no/reference-data/eu/eurovocs
https://data.norge.no/reference-data/los/themes-and-words
Sector
This property represents the industry or sector a Public Service relates to, or is intended for. Note that a single Public Service may relate to multiple sectors.
The possible values for this property are described in the controlled vocabulary Data theme Authority Table of the Publications Office.
API informasjonsforvaltning
https://data.norge.no/reference-data/eu/data-themes
Resource Registry Type
This is our Altinn version of this. We have not yet defined all possible.
Resource Registry Status
What kind of status do we need?
Available API from informasjonsforvaltning
Informasjonsforvaltning supports an API for
Thematic Area, Sector and Los and might be used as backend for our API.
Investigate the availability for this
https://github.com/Informasjonsforvaltning/fdk-reference-data/blob/main/src/main/resources/openapi/openapi.yaml
Tasks

[ ] Identify what is the needed datasources needed for spatial information
[ ] Design UX for this area
[ ] Implement and expose API to UI to select
[ ] Identify the datasource for type
[ ] Design UX for type
[ ] Implement and expose APIO for UI to select from
[ ] Identify the datasource for CPSV Status
[ ] Design UX for this input
[ ] Implement and expose API for CPSV status values
[ ] Identify the datasource for thematic area. Do we need to use los in addition to EUROVOC?
[ ] Design UX for this
[ ] Implement API and expose to UX
[ ] Identify the datasource for sector
[ ] Design UX for this
[ ] Implement API and expose to UX

Acceptance Criterias
No response

Closes since there is currently no need.


Use subctl & submariner-operator Version from $VERSION

VERSION defaults to CALCULATED_VERSION if nothing is provided
for make.
For example:
make build-cross will result in binaries containing devel-xxxxx
as version, while VERSION=v0.9.0 make build-cross will result in
binaries containing v0.9.0 as version.
VERSION env var is passed down from the release scripts here:
https://github.com/submariner-io/releases/pull/124/files
Fixes-Issue: #1266
Signed-off-by: Miguel Angel Ajo<EMAIL_ADDRESS>

@skitt, @mkolesnik wanted you to have an eye on this when you are back, since we are partially reverting https://github.com/submariner-io/submariner-operator/pull/527/files#diff-76ed074a9305c04054cdebb9e9aad2d818052b07091de1f20cad0bbac34ffb52R69
Because of #1266 && https://github.com/submariner-io/releases/pull/124/files

Looks fine to me, but I'd like to know what @skitt thinks since he originally did #527 which introduced this usage.

xinx Although I think it's not a big of a deal, and... that we should merge it if this were to delay release in any way.

Looks fine to me, but I'd like to know what @skitt thinks since he originally did #527 which introduced this usage.

xinx Although I think it's not a big of a deal, and... that we should merge it if this were to delay release in any way.

skitt should be back next week, so i reckon hell take a look before this becomes a problem

Behaviour after this patch,
With VERSION not set:
[submariner-operator]$ ldflags=-s -w -X github.com/submariner-io/submariner-operator/pkg/version.Version=devel-394aaf9 		   -X=github.com/submariner-io/submariner-operator/pkg/versions.DefaultSubmarinerOperatorVersion=devel
[submariner-operator]$ CGO_ENABLED=0 go build -trimpath -ldflags -s -w -X github.com/submariner-io/submariner-operator/pkg/version.Version=devel-394aaf9 		   -X=github.com/submariner-io/submariner-operator/pkg/versions.DefaultSubmarinerOperatorVersion=devel -o bin/subctl-devel-394aaf9-linux-amd64 ./pkg/subctl/main.go

with VERSION set to v0.9.1:
[submariner-operator]$ ldflags=-s -w -X github.com/submariner-io/submariner-operator/pkg/version.Version=v0.9.1 		   -X=github.com/submariner-io/submariner-operator/pkg/versions.DefaultSubmarinerOperatorVersion=devel
[submariner-operator]$ CGO_ENABLED=0 go build -trimpath -ldflags -s -w -X github.com/submariner-io/submariner-operator/pkg/version.Version=v0.9.1 		   -X=github.com/submariner-io/submariner-operator/pkg/versions.DefaultSubmarinerOperatorVersion=devel -o bin/subctl-v0.9.1-linux-amd64 ./pkg/subctl/main.go

skitt should be back next week, so i reckon he'll take a look before this becomes a problem

I hope he doesn't have my PTO recovery time, lol


feat(tiller): Add .Release.Revision and .IsUpgrade

New properties are available to template developers.
Closes #1554

Perhaps it would be useful to also add .IsInstall. It seems like that would be the more common use case and would be more ergonomic than not .IsUpgrade.

I'm adding IsInstall too at @chancez's suggestion.

manually tested


flit install --symlink doest not honor --python for scripts it installs

Hi,
I have been experimenting with flit recently and I very much like it's approach for development installs (--symlink).
I noticed what could be a small issues with such installations.
I have flit installed globally (with pipx).
When I try to install my project with pip install --symlink --python $(which python) in a virtualenv, I notice the installed script uses flit's python as shebang instead of the python of the virtualenv.
I believe the problem arises in this install_script, called via install_directly:
https://github.com/takluyver/flit/blob/f5996dcee05656ca7a1e6ed2e5c82a6e911f4f10/flit/install.py#L171

At first sight, using self.python instead of sys.executable works fine.

I think you've understood the cause and likely solution, at least from a
quick look. Want to make a PR? Bonus points if you can figure out a
straightforward test for it.
On Sun, 18 Aug 2019, 12:46 Stéphane Bidoul (ACSONE), <
<EMAIL_ADDRESS>wrote:

At first sight, using self.python instead of sys.executable works fine.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/takluyver/flit/issues/285?email_source=notifications&email_token=AACQB5OWJZCVIKBJHMVARODQFEZBVA5CNFSM4IMSMAPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4Q6HDQ#issuecomment-522314638,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACQB5K57JFSTXK63PJVJEDQFEZBVANCNFSM4IMSMAPA
.

I can certainly do a PR. Adding a test requires some study of how the test suite currently work.

Thanks! If you've worked with other test suites for Python projects, Flit's tests will probably make plenty of sense. Here's one you could add to:
https://github.com/takluyver/flit/blob/f5996dcee05656ca7a1e6ed2e5c82a6e911f4f10/tests/test_install.py#L78-L109

Thanks for the pointer. The tests are quite easy to read indeed. Here you go: #286


Decouple

The first two commits from #1816 so that they can be merged before the maturin packaging situation is resolved.
Please merge via rebase.

Please merge via rebase.

(I'll always merge your PRs via rebase if there are multiple commits.)


CloudFront configuration

Hi @danielcondemarin,
I was wondering if it is possible to override the CloudFormation configuration or reference a resource with an arn or id.
It would be great to pass the domain and certificate or other CDN related things, so I don't have to change the distribution by hand and we would have a clean cloud formation stack without drifts.
I have used the lambda@edge plugin that I think have a similar case
https://github.com/silvermine/serverless-plugin-cloudfront-lambda-edge
What do you think about it?

@samuv That's been implemented now 🚀 . Should be available in v2.2.0-alpha.0. Check https://github.com/danielcondemarin/serverless-nextjs-plugin#serving-static-assets for more details.

Oh wow! This is awesome, thank you.


GatherElements

This will cause an error: action.settings.ElementAffect is undefined

Hi xlshen,
ElementAffect is initialised with a default value. Can you provide a case that can cause action.settings.ElementAffect to be undefined? I'm not sure it's possible for the GatherElements function to be triggered with the action.settings.ElementAffect property to not be defined (unless you explicitly set it to undefined).
Code submitted is fine, just making sure there's a use case :)
Cheers
Dom

Hi domsammut,
When the document is loading and if your document contains these opreations below, these operations will execute first when you called ScrollAppear  function,
$.each(['after', 'append', 'appendTo', 'before', 'clone', 'detach', 'empty', 'html', 'insertAfter', 'insertBefore', 'prepend', 'prependTo', 'remove', 'replaceAll', 'replaceWith', 'unwrap', 'wrap', 'wrapAll', 'wrapInner'], function (index, trigger) {
        var prev = $.prototype[trigger];
        if (prev) {
            $.prototype[trigger] = function () {
                var passthrough = prev.apply(this, arguments);
                $.prototype.ScrollAppear('DomUpdate');
                return passthrough;
            };
        }
    });

It will cause $.prototype.ScrollAppear('DomUpdate');
DomUpdate: function () {
            action.InternalSettings.ShowElements = [];
            action.InternalSettings.ResetElements = [];
            action.GatherElements();

        },

At this time, My function like this has not called yet. But the DomUpdate has operated and each time it will call action.GatherElements();. In this circumstance, the error will happen.
$('.homepage-part3').ScrollAppear({
        ElementAffect: ".scrollAppear",
        AddClass: 'appear',
        ElementsToShow: 12,
        PixelOffset: 200
    });

This is my example:
var shortcut = $('body').shortcut(); shortcut.gotop.init(); This is code before calling the ScrollAppear function.  In it, I used appendTo operation like this:
 topLink.click(function () {                     $('html,body').animate({ scrollTop: 0 }, 200);                     return false;                 }).appendTo($('body'));
When I called $('.homepage-part3').ScrollAppear({         ElementAffect: ".scrollAppear",         AddClass: 'appear',         ElementsToShow: 12,         PixelOffset: 200     });
It will execute the above appendTo operation first and at this moment, the action.settings is undefined yet.
jquery.scrollappear.js?V=635983835888361325:73 Uncaught TypeError: Cannot read property 'ElementAffect' of undefined
You can test a case like this. If there is something unclear, let me know.


Hard-coded input path in MPII Human pose dataset upload.py

In MPII Human pose dataset upload.py, path is hard coded with /home/sanchit and doesn't generalize well.
Now a single directory mpii_human_pose_v1 ( consistent with download link ) which contains images and annotation json file, which is referenced in arg.dataset_path. And main() has been added to MPIIGenerator class as well.
Refer to PR #219

yep @kzuri, thanks for the help with reorganizing the code!


inappwebview does not show connect to mobile wallet on ios ?

inappwebview does not show connect to mobile wallet on ios ,
but a list of some wallet app .
how can i use it on ios?
thanks!

Hey you will need to register your wallet on wallet connect registry and then use wallet URI to connect, @thelazyindian can you guide him better

Hey,
I don't really understand your answer can you be more explicit "you will need to register your wallet on wallet connect registry and then use wallet URI to connect"

How can i register my wallet on wallet connect ?
How can I get the wallet URI ?

I had a similar question as @juliengss . Is there like some sort of universal database that holds all wallets and one has to register there so that their wallet comes in as an option when one presses connect on any dApp site?

Hey @rennylangat  yes, check this out https://medium.com/walletconnect/introducing-walletconnect-cloud-registry-c17ac580f33

thanks for all guys.
i have got how to use it on ios.

@lxl518000 how did you do it?


Detach repo

Can you please detach this repo from the not maintained fork?
You can this either do it manually or (which I recommend) just contact the GitHub support to do it for you.
This…

shows that this is a real maintained project and not just a repo of someone who (once) made a pull request
linking to the original repo on every page does not make sense anymore as it has been discontinued/is not maintained anymore and you…
can (and IMHO should) still link to the original repo/website/author/… for crediting them, but you can do so in better ways (in the Readme, wiki or so)
allows searching in this repo on GitHub and
it just looks much nicer

That's a great idea, I thought it would be more bureaucratic. I did keep the crediting to the authors everywhere, including the LICENSE, do you feel it should be on any other place?

I think that's enough… :smiley:

Great, we are looking into it as we speak (:

Taken care of :)


[HotFix] Move CiderParquetReader out from cider_util.

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Which label does this PR belong to?

Pls merge this PR ASAP, thx. @jikunshang @yma11 @winningsix


Fix some grammatical errors and typos in README.MD and CONTRIBUTING.MD

Add commas in "if" statement
Fix some typos in both files
Make Note and TL;DR more visible by using bold font.

Thanks for the contribution!


[CHORE] [New Query Planner] [1/N] Remove Python query planner.

This PR removes the old Python query planner, which is a prerequisite for some breaking changes such as pushing (re)partitioning into the physical plan.

closes: https://github.com/Eventual-Inc/Daft/issues/1436


Add right padding to th span element

Closes #267.

I think it is a font issue because the width of that element is hard-coded to 0.5em. If in the available system font the glyph doesn’t have an actual width of 0.5em, you’ll see exactly that layout problem. My guess is that 0.5em only works for the macOS San Francisco system font, not for other platforms; it’s not wide enough on Windows. (And the visual width of the glyph isn’t the same as the layout width.) That said, maybe it’s as simple as adding a text-align: center or a display: flex so that if the glyph isn’t 0.5em, it’s at least centered in the reserved area, rather than going too far right or left?

That seems like a good suggestion. That looks good on my windows machine for Edge / Chrome. I'll test on MacOS later on 👍

Yes it's perfect on Chrome/Mac too 👍

Nice! Thanks for verifying 🤝


test: add two benchmarks that are slow to parse or elaborate

Add two example Lean files that include unusually large definitions and take a long time to parse or elaborate.
It may be that it's difficult to process them more efficiently, but perhaps someone will discover a way to improve them if they're in the benchmark suite. Improved performance on these benchmarks will likely make program analysis and verification within Lean more feasible.

!bench

!bernch

!bench

@atomb Thank you for the benchmarks! I can see the big_do being useful, I added some small commentary to it. The other benchmark I fear is far too large for the suite, none of them should take more than a few seconds. I also fear that this large number of variables is not something we might want to optimize for and would better be modeled as more axioms.

@atomb Thank you for the benchmarks! I can see the big_do being useful, I added some small commentary to it. The other benchmark I fear is far too large for the suite, none of them should take more than a few seconds. I also fear that this large number of variables is not something we might want to optimize for and would better be modeled as more axioms.

Why don't I trim this one down to just big_do, then? A variation on the other might be appropriate later on, but we can leave that for the future.


Nada/save button disabled for fixed to floating edit ad

https://github.com/user-attachments/assets/545eccd3-a2b0-4a9d-b682-87f93839d360

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

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 85.441%

Totals

Change from base Build<PHONE_NUMBER>9:
0.002%

Covered Lines:
5713

Relevant Lines:
6220

💛  - Coveralls


final_path reference issue

@godfoder
I tried running the job with the instructions given but there seems to be issue with referencing of final_path.

The code for my job is as follows, in case I missed something. https://github.com/acislab/idigbio_distinctvalues_R/blob/master/Jenkins_ceph_sample
Thanks.

The path should be in the form of /outputs/ACountryDFSS.csv , without any of the server or protocol information. I'll look into tweaking the code to be more tolerant of other path forms.


Acorn run from remote registry seems to miss credentials

First I logged into the registry:
acorn credentials login registry.rpi.cloudnautique.com
entered in username and password.
I built an image and pushed to the registry.
acorn build -t registry.rpi.cloudnautique.com/library/redis:latest .
acorn push registry.rpi.cloudnautique.com/library/redis/latest
I removed the local copy:
acorn rm -i <image_id>
Then tried to run the Acorn from the registry.
acorn run registry.rpi.cloudnautique.com/library/redis:latest -h provided the help.
acorn run registry.rpi.cloudnautique.com/library/redis:latest doesn't come up
acorn all

APPS:
NAME           IMAGE                                          HEALTHY   UP-TO-DATE   CREATED     ENDPOINTS                                                                            MESSAGE
long-firefly   registry.rpi.cloudnautique.com/library/redis   0/2       2            5m56s ago   tcp://<IP_ADDRESS>:6379 => redis-0-0:6379, tcp://<pending>:6379 => redis-0-1:6379   [containers: long-firefly is not ready; long-firefly is not ready; redis-0-0 ImagePullBackOff: Back-off pulling image "registry.rpi.cloudnautique.com/library/redis@sha256:e743b73abc5bba4a435a95d46d8f3386493d7a20c5bf16e1705e9cc5dacc529d"; redis-0-1 ImagePullBackOff: Back-off pulling image "registry.rpi.cloudnautique.com/library/redis@sha256:e743b73abc5bba4a435a95d46d8f3386493d7a20c5bf16e1705e9cc5dacc529d"]

CONTAINERS:
NAME                                      APP            IMAGE            STATE                                                                                RESTARTCOUNT   CREATED     MESSAGE
long-firefly.redis-0-0-587c68f8b8-2st9r   long-firefly   redis:7-alpine   ImagePullBackOff: Back-off pulling image "registry.rpi.cloudnautique.com/library/redis@sha256:e743b73abc5bba4a435a95d46d8f3386493d7a20c5bf16e1705e9cc5dacc529d"   0              5m52s ago
long-firefly.redis-0-1-bf674758f-vm2c6    long-firefly   redis:7-alpine   ImagePullBackOff: Back-off pulling image "registry.rpi.cloudnautique.com/library/redis@sha256:e743b73abc5bba4a435a95d46d8f3386493d7a20c5bf16e1705e9cc5dacc529d"   0              5m52s ago

VOLUMES:
NAME                                       APP-NAME             BOUND-VOLUME         CAPACITY   STATUS     ACCESS-MODES   CREATED
pvc-16b083c2-ef01-429e-b50c-7616ac5aaee5   long-firefly         redis-data-dir-0-1   10G        bound      RWO            5m52s ago
pvc-d7544666-bd07-40d7-9de3-68d94199726f   long-firefly         redis-data-dir-0-0   10G        bound      RWO            5m52s ago

There are pull secrets that appear correct for the registry attached to the deployments.
Here is the digest for the latest tag:
{
  "schemaVersion": 2,
  "manifests": [
    {
      "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
      "size": 524,
      "digest": "sha256:e6519de2ea71ced8ac825e4f522a5605f86c440c27454e0c90cf08c8f78debca"
    },
    {
      "mediaType": "application/vnd.oci.image.index.v1+json",
      "size": 246,
      "digest": "sha256:e743b73abc5bba4a435a95d46d8f3386493d7a20c5bf16e1705e9cc5dacc529d"
    },
    {
      "mediaType": "application/vnd.oci.image.index.v1+json",
      "size": 246,
      "digest": "sha256:e743b73abc5bba4a435a95d46d8f3386493d7a20c5bf16e1705e9cc5dacc529d"
    }
  ]
}

Looking in the storage bucket, there is a blob: /v2/blobs/sha256/e7/e743b73abc5bba4a435a95d46d8f3386493d7a20c5bf16e1705e9cc5dacc529d

Running images from a registry works now.


Display Current Date on Gallery Page

Display current date in the viz galery to correctly link to EPA source Data
Add Current Date to JSON file for viz gallery page

Definition of Ready

[ ] Does this meet the INVEST criteria?
[ ] Does this have a clearly defined user?
[ ] Does this have an estimate?
[ ] Does this have acceptance criteria?
[ ] Does this have sufficient context for the specific functionality being developed (design, business rules, etc.)
[ ] Does this have a dev link been provided in the design?
[ ] Is this traceable to a feature, epic, or PI?

User Story
As a CAMPD Data Vizualization Page user, I want to be able to see the current date when looking at the vizualiztion options, so that I can compare it to the date it was last updated.
Acceptance Criteria
Given that a user is on the vizulization page
When the user is scrolling through the various options
And the Last Updated Date is equal to the Current Date
Then the current date displays
Given that a user is on the vizulization page
When the user is scrolling through the various options
And the Last Updated Date does equal the Current Date
Then the Last Updated Date Displays
Development Notes

If the last updated date property is equal to the CURRENT_DATE, in the JSON File then display the current date
-Otherwise, display whatever date is in that property

Does this Story require 508 Testing?

[ ] Implementing a new authoring tool that will produce code / content for the application
[ ] Implementing a 3rd party widget that will be part of the application
[ ] Adding new / updating custom functionality
[ ] Adding new / updating a page template
[ ] Adding new / replacing existing functionality (unless the replacement has already passed 508 testing)
[ ] Full Site Review prior to a scheduled EPA 508 review
[ ] Planning a video that will appear on the application
[ ] Producing a video that will appear on the application
[ ] Producing Training materials
[ ] Producing documents

Context
(User journeys, design mockups, architectural diagrams, business rules, data element definitions)
Background:
Next Steps:
Testing
Test Case: TC1036
Tester: Maegan Wood
Definition of Done

[ ] Have all Acceptance Criteria been agreed to and validated by the scrum team?
[ ] Has the code been unit tested and peer reviewed?
[ ] Have the functional tests been executed?
[ ] Have all defects been reviewed and dispositioned, resolved, or deferred?
[ ] Is the user story ready to be deployed to the test/staging environments?

@tlageman-epa TC1036 passed


Action for issue deletion

Create a github action to automatically delete a issue which is tied to a pull request after merging the target branch.
This should be added under .github/workflows/.
Make sure to create a seperat file for this action.

Hi @Jakobus42
Could you please assign me this issue?

Hi @Jakobus42
I have raised a PR for this issue pls check

Hi @Ksj14-kumar,
Thank you for your effort and the work you've done on this!
I wanted to mention a couple of things regarding your pull request:
Branch Deletion: GitHub already provides a built-in feature to automatically delete branches after a pull request is merged. You can enable this feature under the repository settings.
The initial issue was rather about closing issues after a merge of a PR.
After some research, it looks like the issue you're addressing isn't actually an issue that needs solving with a custom action. GitHub already provides ways to automatically close issues when a pull request is merged by simply using certain keywords in the pull request description, such as closes #, fixes #, etc. When these keywords are included in the PR description, GitHub will automatically close the associated issue once the PR is merged.
Since GitHub handles both branch deletion and issue closing automatically, this specific workflow may not be necessary. However, I truly appreciate your initiative in trying to keep the repository clean and efficient!
Thanks again for your contribution!
Best regards,
Jakob


Warning! Reached max decoder steps

I had no such problem before
But today used pretrained models
And sometimes spectrogram is bugged

here is notebook for reproduce
https://colab.research.google.com/drive/1jR12cEKdkg0hlDUHGhf2fPb0RwqPwEYj?#scrollTo=CyBu2F7eisFM

Take a look at the gate outputs and decrease the gate threshold accordingly.

@rafaelvalle
But I can't get, why spectrogram different for same text?
And why I never notice it before? Last time used in January.
Maybe it because of fc0d34c ?
Also, gate looks this

The spectrogram is different with the same text because of the dropout on the Prenet layer: https://github.com/NVIDIA/tacotron2/blob/master/model.py#L100
The gate threshold is what controls when the model should stop decoding. You can decrease the value  to prevent such issues.

Soo, I get this error with pretrained tacatron and default code from repo, don't you want to investigate?

What sentence did you use?

Seems it happens sometimes if I have whitespace at the end, like:
'We have started the invasion. '

I too think this issue may have been closed out too early. I sometimes see this error for a while, then it is gone again. It seems to happen on shorter sentences, or when using just a few words. For example, try "test" as the only input. This happens with pre-trained model. Reducing gate threshold does not help. Increasing decoder steps sometimes works, but it is starting to look somewhat sporadic.

I tested/confirmed that removing the change in https://github.com/NVIDIA/tacotron2/commit/fc0d34cfce2c8028a2aa76b535eee73d96cd9a85 makes no difference; the error still happens.

Thought; is there any "state" preservation which can be deleted? It seems to be related in some way to a state of the machine, i.e. files present, relevant cache, ...

I am having the same issue.
My input text is hello, this is the -weird- generated wav file. The good thing is that the tts does not crash, but it seems the last letter o is repeated for 9 seconds.
Would it be possible to avoid this? maybe by attaching some silence at the end of the phrase if this is too short?

The model is very sensitive to punctuation. Try adding the period: "hello."
You can also try modifying the gate threshold.

I am having the same issue.
My input text is hello, this is the -weird- generated wav file. The good thing is that the tts does not crash, but it seems the last letter o is repeated for 9 seconds.
Would it be possible to avoid this? maybe by attaching some silence at the end of the phrase if this is too short?

Hi there @shoegazerstella just want to ask, have you solve the decoder issue ? What would be the best solution ?

Have you tried adding a '.' to the text input: "Hello." instead of "Hello" ?

@haqkiemdaim
Have you tried adding a '.' to the text input: "Hello." instead of "Hello" ?

Haven't yet. But my dataset does not contain any punctuation.

You can add to every sentence of your dataset a token that signifies EOS.

You can add to every sentence of your dataset a token that signifies EOS (End of Sentence).

ouhh i see. Is it crucial? because in plain tacotron, i used the same data/corpus without punctuation and it went well. But not in Tacotron2. Any reason for that ?

Tacotron 1 has the EOS token :-)
On Sat, Nov 2, 2019, 5:20 PM HaqkiemDaim<EMAIL_ADDRESS>wrote:

You can add to every sentence of your dataset a token that signifies EOS
(End of Sentence).
ouhh i see. Is it crucial? because in plain tacotron, i used the same
data/corpus without punctuation and it went well. But not in Tacotron2. Any
reason for that ?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/NVIDIA/tacotron2/issues/174?email_source=notifications&email_token=AARSFDYDX7VQJ5PTZVH6573QRYKK7A5CNFSM4HCLSMAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEC5H5LQ#issuecomment-549093038,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARSFD4YBD5BETQ6SRGWF6LQRYKK7ANCNFSM4HCLSMAA
.

You can also try adjusting the gate threshold.
On Sat, Nov 2, 2019, 5:20 PM HaqkiemDaim<EMAIL_ADDRESS>wrote:

You can add to every sentence of your dataset a token that signifies EOS
(End of Sentence).
ouhh i see. Is it crucial? because in plain tacotron, i used the same
data/corpus without punctuation and it went well. But not in Tacotron2. Any
reason for that ?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/NVIDIA/tacotron2/issues/174?email_source=notifications&email_token=AARSFDYDX7VQJ5PTZVH6573QRYKK7A5CNFSM4HCLSMAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEC5H5LQ#issuecomment-549093038,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARSFD4YBD5BETQ6SRGWF6LQRYKK7ANCNFSM4HCLSMAA
.

@rafaelvalle This is what i have done so far:

remove trailing space
change max decoder steps from 500 to 2000

But still reach max decoder steps at 4th epoch.
Now i'm running again my dataset but with punctuation "." at end of sentence for every line.
Will update here the outcomes.
Regarding the gate threshold, currently it is 0.5.
But im a bit confuse, why we need the gate threshold while we already set the max decoder steps to a certain value ?

@haqkiemdaim Did adding punctuation fix the error?

@haqkiemdaim Did adding punctuation fix the error?

it worked for my case. thanks!

@rafaelvalle
If EOS work why don't you guys make it as default option in hparams.py?
I see other contributors provide answer related to EOS such as accelerate convergence of attention.

How do you try except the message "Warning! Reached max decoder steps"? Which file creates it?

I had no such problem before But today used pretrained models And sometimes spectrogram is bugged

here is notebook for reproduce https://colab.research.google.com/drive/1jR12cEKdkg0hlDUHGhf2fPb0RwqPwEYj?#scrollTo=CyBu2F7eisFM

I had no such problem before But today used pretrained models And sometimes spectrogram is bugged

here is notebook for reproduce https://colab.research.google.com/drive/1jR12cEKdkg0hlDUHGhf2fPb0RwqPwEYj?#scrollTo=CyBu2F7eisFM

the notebook link is not opening


On Cloud9 Default, Resize Script does not work on Ubuntu 20.x

$ ./scripts/cloud9-resize.sh
EBS Volume Resizer us-west-2/i-00ec45b95b57dcc9b
EBS Volumes:
1) vol-07f6df33f5a7e348f
Please select the EBS volume to resize (e.g 1) : 100
The selected volume is invalid.

Admin:~/environment/aws-genai-llm-chatbot (main) $ sudo ./scripts/cloud9-resize.sh
EBS Volume Resizer us-west-2/i-00ec45b95b57dcc9b

An error occurred (UnauthorizedOperation) when calling the DescribeInstances operation: You are not authorized to perform this operation. User: arn:aws:sts::XXXXXXXXX:assumed-role/AWSCloud9SSMAccessRole/i-00ec45b95b57dcc9b is not authorized to perform: ec2:DescribeInstances because no identity-based policy allows the ec2:DescribeInstances action
EBS Volumes:
The selected volume is invalid.

Admin:~/environment/aws-genai-llm-chatbot (main) $ sudo ./scripts/cloud9-resize.sh
EBS Volume Resizer us-west-2/i-00ec45b95b57dcc9b
EBS Volumes:
1) vol-07f6df33f5a7e348f
Please select the EBS volume to resize (e.g 1) : 100
The selected volume is invalid.

Hi @AdamTylerLynch,
You are prompted to select the volume while you are inputting the target size.
1) vol-07f6df33f5a7e348f
Please select the EBS volume to resize (e.g 1) : 100 <---- should be 1

Oh, I was attempting to change disk size there. I read it incorrectly.
Of note, I had to add a new policy to the C9 instance role that allows mutation of EC2/EBS.


HCX-67 Fix getRegionsBy return type

It seems that maven version was upgraded, and default target Java 5 cannot be user anymore.
Set it to Java 8, like in master:
https://github.com/latera/camunda-ext/blob/master/pom.xml#L18-L19


Convert UUID primary key to string before serializing in json.

Fixes #4616 .

@saevarom as far as I know SQLite should have no problem with UUIDField. Running the tests on this branch locally all pass, and the default test configuration uses SQLite. The fact that the test command exits on Travis with exited with code -11 suggests a segfault in SQLite - this thread suggests that others have experienced this on Travis as well. This wouldn't explain why CircleCI failed as well but might be a clue at least.

@chosak I tried running again with the file-based sqlite instead of the in-memory one as suggested in the thread you linked to. Now the CircleCI build works, but not Travis...

I think I've found the incantation to make Travis work: https://github.com/gasman/wagtail/commit/f4752d68874facbc16919b8a25830b71d287603d
Will open a rebased PR to make sure it runs on CircleCI too, and merge if all goes well.

Merged in 6e60469142a11ac6df041e75c3537dcc538039ae - thanks @saevarom!


Get better view of changes in processed occurrences

The output of DwC mapping contains way more additions/deletions than what we would expect as shown in the summary at the end of the mapping. This is due to a kind of "shuffle" of many occurrences in the raw data.
I propose to return occurrences  in a ordered way, e.g. order by occID. In this way we should clearly what changes, typically tens or very few hundreds of obs added weekly.

Better to order by eventID and by species_name_hash separately to really respect order of eventID.


chore: bump gravitee-node to include new interops SP plugins

should it be applied also on previous 4.x versions ?