Spinner Component

Describe the solution you'd like
A spinner to indicate the application is currently busy.
Given I am submitting a form
And I use the DCX React Library,
Then I should be able to import a spinning loading icon to show visual feedback that the application is doing something.
Technical Details
The spinner should be customisable with classes, or props controlling speed, colour etc.
See the following link from MUi for an example of it
https://mui.com/material-ui/react-progress/
Set up:
git checkout release/1.1.0
git pull
git checkout -b 'feature/spinner'

closed by https://github.com/Capgemini/dcx-react-library/pull/690


add context to fixtures

PR for our telegram conversation.
This allows us to have context on fixture files. If you have a general/dynamic connector mock for your tests you will end up with a bunch of fixtures you have no idea where they belong to. It is even worse when you use some payload hashing as below.
AvalaraConnector::class => function (PendingRequest $request): Fixture {
    $name = implode('/', [
        parse_url($request->getUrl(), PHP_URL_HOST),
        parse_url($request->getUrl(), PHP_URL_PATH),
        Str::upper($request->getMethod()->value),
        hash('md5', json_encode(Arr::sortRecursive(array_merge(
            $request->query()->all(),
            $request->body()?->all() ?? [],
        )), JSON_THROW_ON_ERROR)),
    ]);

    return MockResponse::fixture($name);
},

As you can see each different request will end up with a corresponding fixture - but you won't be able to tell from a fixture where it belongs to, if it's still needed or anything.
That's where fixture context comes in handy - you can add whatever context you want that will be persisted in your fixture files - for example the plain payload or the name of the test it was used in or anything else.
You can do so by chaining withContext() on your fixture instance.
usage examples
// simple single key setter
return MockResponse::fixture($name)->setContext('query', $request->query()->all());

// simple withContext chain
return MockResponse::fixture($name)->withContext(['query' => $request->query()->all()]);

// complex context manipulation
$fixture = MockResponse::fixture($name);
$fixture->getContext()->when($condition, fn(ArrayStore $context) => $context->add('key', 'value'));
return $fixture;

You have access to the underlying ArrayStore via getContext() and can use all the methods available there to manipulate or interact with the context. For sure you can also use getContext() to retrieve a specific context value.
The context isn't used anywhere in Saloon itself - it's only made available and persisted so that you can check it manually or do some checks in your own code with it. So everything you do with it is up to you - only restriction: it has to be JSONable.

I haven'T tested for compatibility with https://github.com/saloonphp/saloon/pull/425 but there shouldn't be any real conflicts.

Hey @Gummibeer - thanks for this PR, and sorry for the delay, life has been really busy. I will have a look through this and hopefully soon we can merge it in.

@Gummibeer Seems like something is up with the tests 🤔 Obviously no rush but if you get a spare 5 minutes it'll be good if you could take a look please? (Btw I have merged the latest of v3 into your branch)


Fix Rocm Does not Compile Issue.

Hit a compiling issue when running the model with Rocm.
Traceback (most recent call last):
  File "/opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/utils/cpp_extension.py", line 1900, in _run_ninja_build
    subprocess.run(
  File "/opt/conda/envs/py_3.9/lib/python3.9/subprocess.py", line 528, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['ninja', '-v', '-j', '8']' returned non-zero exit status 1.

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/var/lib/jenkins/text-generation-webui/repositories/exllamav2/test_inference.py", line 2, in <module>
    from exllamav2 import(
  File "/var/lib/jenkins/text-generation-webui/repositories/exllamav2/exllamav2/__init__.py", line 3, in <module>
    from exllamav2.model import ExLlamaV2
  File "/var/lib/jenkins/text-generation-webui/repositories/exllamav2/exllamav2/model.py", line 12, in <module>
    from exllamav2.linear import ExLlamaV2Linear
  File "/var/lib/jenkins/text-generation-webui/repositories/exllamav2/exllamav2/linear.py", line 4, in <module>
    from exllamav2 import ext
  File "/var/lib/jenkins/text-generation-webui/repositories/exllamav2/exllamav2/ext.py", line 118, in <module>
    exllamav2_ext = load \
  File "/opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/utils/cpp_extension.py", line 1283, in load
    return _jit_compile(
  File "/opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/utils/cpp_extension.py", line 1508, in _jit_compile
    _write_ninja_file_and_build_library(
  File "/opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/utils/cpp_extension.py", line 1623, in _write_ninja_file_and_build_library
    _run_ninja_build(
  File "/opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/utils/cpp_extension.py", line 1916, in _run_ninja_build
    raise RuntimeError(message) from e
RuntimeError: Error building extension 'exllamav2_ext': [1/2] /opt/rocm/bin/hipcc  -DWITH_HIP -DTORCH_EXTENSION_NAME=exllamav2_ext -DTORCH_API_INCLUDE_EXTENSION_H -DPYBIND11_COMPILER_TYPE=\"_gcc\" -DPYBIND11_STDLIB=\"_libstdcpp\" -DPYBIND11_BUILD_ABI=\"_cxxabi1013\" -I/var/lib/jenkins/text-generation-webui/repositories/exllamav2/exllamav2/exllamav2_ext -isystem /opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/include -isystem /opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/include/torch/csrc/api/include -isystem /opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/include/TH -isystem /opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/include/THC -isystem /opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/include/THH -isystem /opt/rocm/include -isystem /opt/conda/envs/py_3.9/include/python3.9 -D_GLIBCXX_USE_CXX11_ABI=1 -fPIC -std=c++17 -O3 -fPIC -D__HIP_PLATFORM_HCC__=1 -DUSE_ROCM=1 -DCUDA_HAS_FP16=1 -D__HIP_NO_HALF_OPERATORS__=1 -D__HIP_NO_HALF_CONVERSIONS__=1 -lineinfo -O3 --offload-arch=gfx900 --offload-arch=gfx906 --offload-arch=gfx908 --offload-arch=gfx90a --offload-arch=gfx1030 --offload-arch=gfx1100 --offload-arch=gfx1101 --offload-arch=gfx940 --offload-arch=gfx941 --offload-arch=gfx942 -fno-gpu-rdc -c /var/lib/jenkins/text-generation-webui/repositories/exllamav2/exllamav2/exllamav2_ext/hip/q_gemm.hip -o q_gemm.cuda.o
FAILED: q_gemm.cuda.o
/opt/rocm/bin/hipcc  -DWITH_HIP -DTORCH_EXTENSION_NAME=exllamav2_ext -DTORCH_API_INCLUDE_EXTENSION_H -DPYBIND11_COMPILER_TYPE=\"_gcc\" -DPYBIND11_STDLIB=\"_libstdcpp\" -DPYBIND11_BUILD_ABI=\"_cxxabi1013\" -I/var/lib/jenkins/text-generation-webui/repositories/exllamav2/exllamav2/exllamav2_ext -isystem /opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/include -isystem /opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/include/torch/csrc/api/include -isystem /opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/include/TH -isystem /opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/include/THC -isystem /opt/conda/envs/py_3.9/lib/python3.9/site-packages/torch/include/THH -isystem /opt/rocm/include -isystem /opt/conda/envs/py_3.9/include/python3.9 -D_GLIBCXX_USE_CXX11_ABI=1 -fPIC -std=c++17 -O3 -fPIC -D__HIP_PLATFORM_HCC__=1 -DUSE_ROCM=1 -DCUDA_HAS_FP16=1 -D__HIP_NO_HALF_OPERATORS__=1 -D__HIP_NO_HALF_CONVERSIONS__=1 -lineinfo -O3 --offload-arch=gfx900 --offload-arch=gfx906 --offload-arch=gfx908 --offload-arch=gfx90a --offload-arch=gfx1030 --offload-arch=gfx1100 --offload-arch=gfx1101 --offload-arch=gfx940 --offload-arch=gfx941 --offload-arch=gfx942 -fno-gpu-rdc -c /var/lib/jenkins/text-generation-webui/repositories/exllamav2/exllamav2/exllamav2_ext/hip/q_gemm.hip -o q_gemm.cuda.o
clang: warning: -lineinfo: 'linker' input unused [-Wunused-command-line-argument]
/var/lib/jenkins/text-generation-webui/repositories/exllamav2/exllamav2/exllamav2_ext/hip/q_gemm.hip:156:9: error: no matching function for call to 'rocblas_hgemm'
        rocblas_hgemm(cublas_handle,
        ^~~~~~~~~~~~~

Adding addtional define fix the issue.

Hi @ardfork,
Can you also try to verify that?

These hipification problems were fixed upstream for a while now, so I didn't bother to also put them in my PR. Update pytorch or add the patch below, similar to the one in v1, I might have missed some other wrong hipification, since I didn't test, but you should understand the idea.
diff --git a/exllamav2/exllamav2_ext/cuda/q_gemm.cu b/exllamav2/exllamav2_ext/cuda/q_gemm.cu
index e99fe6c..8d41969 100644
--- a/exllamav2/exllamav2_ext/cuda/q_gemm.cu
+++ b/exllamav2/exllamav2_ext/cuda/q_gemm.cu
@@ -41,6 +41,11 @@ __host__ __forceinline__ hipblasStatus_t __compat_hipblasHgemm(hipblasHandle_t
                         reinterpret_cast<hipblasHalf *>(CP), ldc);
 }
 #define hipblasHgemm __compat_hipblasHgemm
+
+// Previous version of PyTorch were converting to rocBLAS instead of hipBLAS.
+#define rocblas_handle hipblasHandle_t
+#define rocblas_operation_none HIPBLAS_OP_N
+#define rocblas_hgemm hipblasHgemm
 #endif

 void gemm_half_q_half_cuda_part

Oh, I didn't see that this was a PR, thought that was an issue. I don't think GetStream or SetStream is used anywhere, so you can remove those.

Updated. Also removed #define rocblas_handle hipblasHandle_t.

I'm alright with that change if everything still work on older pytorch version, I just didn't want to bother testing that. Maybe, also rename the commit/PR to something like: "Fix compiling with HIP on older pytorch version" to be easier to understand the goal of that change in commit history.

Thank yous!


Fix clustered webapp race condition writing to cache

Check data not empty before adding to cache. Fix for #1278?

Tried your patch on test environment, works fine, not sure if helps with shared memcached - still didn't enable that in my env.

This looks like a good fix to me, merging. Thanks! :zap:


#8 Fixing rendering on iOS devices

This should fix the issue raised in #8
Tested on iPhone6 using iOS 11 with Chrome & Safari

Great, thanks. Can you update the docs, please?


DRILL-3994: make classpath scanning work on windows

Still broken:
[ERROR] Failed to execute goal org.apache.drill.tools:drill-fmpp-maven-plugin:1.3.0-SNAPSHOT:generate (generate-fmpp) on project drill-java-exec: FMPP processin
g session failed.
[ERROR] Caused by: A listener Java object has failed to handle event "end processing session". The class of the failing listener object is org.apache.drill.fmpp.mojo.FMPPMojo$1.
[ERROR] Caused by: org.apache.maven.plugin.MojoFailureException: C:\Users\jnadeau\AppData\Local\Temp\freemarker-tmp8027657275379826459\javacc\Parser.jj should start with C:\Users\jnadeau\AppData\Local\Temp\freemarker-tmp8027657275379826459/

@jacques-n thanks, I have fixed this one too.

merged in https://github.com/apache/drill/commit/8d56962bb8084724df9c89f007f0b1f1b99f8f95


scatter2 various client side updates

swap the order of the place options selection, and populate place types selection depending on the selected place
move the plot options to below the chart and remove the stats from under the chart
add source attribution
get child place names in background to decrease loading time after selecting place and child place type

Increase font and vertically center "Plot places in" and "of type"?


Objects behave very badly when SPS is activated!

(Seen on discord!)
-> An object comes into contact with the SPS, the mesh becomes disfigured (or something like that, not much explanation :/)
(Modified on 10/21/2024 at 04:40 French time: This bug has been occurring since the new update, previously this did not happen!)

3.9.3 should now automatically detect and exclude SPS penetrators from optimizations


NISO STS import on ISO Fonto: tables appear with strange extra lines added

As reported by SIS (sis.se). Have requested for more information.

From @matsahlin :
I took this one:

Get extra lines in the Fonto tool. However still looks correct in the preview.

In the file I got from you no extra lines, my guess as above the import is doing something here.

Because this what I see after import:

@Intelligent2013 not sure if there's something we can do but could you have a look? Thanks.

@ronaldtse I assume the extra lines occur due line breaks in the xml content:

It seems Fonto treats these line breaks as empty text nodes.
I can update mnconvert to produce the xml in one line without breaks (except source code elements) like this:

<bold>Identifier</bold>
<italic>The ISO/TC 211 URI of the conformance class.</italic>


Enable GM pointers for MOBILE-APPLICATION

Relevant information
Add GM pointers for MOBILE-APPLICATION's summary metrics
Checklist

[ ] I've read the guidelines and understand the acceptance criteria.
[ ] The value of the attribute marked as identifier will be unique and valid.
[ ] I've confirmed that my entity type wasn't already defined. If it is I'm providing an
explanation above.

Duplicated


Is it possible to use this library via CDN?

Is it possible to use this library via CDN?

Unfortunately not right now.


MLEM models as DVC outputs

I'm using a DVC pipeline that specifies MLEM models as outputs. Mlem is configured to store models using dvc as described here. Unfortunately, DVC complains:
ERROR: Path '/home/ingo/tmp/dvc-mlem-issue/models/example.mlem' is ignored by
.dvcignore:1:/**/?*.mlem

Which is correct: I don't wont to track .mlem files in dvc, but I do want to track the binaries.
Here is a testscript that attempts to isolate the issue as much as possible:
git init
dvc init
mlem init

git add .dvc/ .mlem.yaml
git commit -m 'Basic setup'

mlem config set core.storage.type dvc
echo '/**/?*.mlem' > .dvcignore

git add .dvcignore .mlem.yaml
git commit -m 'MLEM works with DVC'

echo 'import mlem' > example.py
echo '' >> example.py
echo 'def example(x: float) -> float:' >> example.py
echo '    return x*x' >> example.py
echo '' >> example.py
echo 'mlem.api.save(example, "models/example")' >> example.py

git add example.py
git commit -m 'Add some code'

# # Works ok
# python example.py
#
# dvc add models/example
# git add models/example.mlem models/example.dvc models/.gitignore
# git commit -m 'Add metadata for model'

dvc run \
  --name="Create example model" \
  --deps=example.py \
  --outs=models/example \
  --outs-no-cache=models/example.mlem \
  python example.py

I'm posting this here, as it is hopefully an issue with the documentation and not with mlem or dvc.

Hi @igordertigor! Thanks for reporting this!
Yep, you need to remove those lines from .dvcignore. Then this will work.
In the User Guide we suggest to remove .dvcignore at all (since we just created it there), but in more general case you should just delete these lines.
Could you please check and confirm it works?

@aguschin do docs need an update or is this something to transfer to the core repo and/or close? Thanks

@igordertigor, I tried to make it more clear. Could you please check if it's better now? https://mlem-ai-improve-user-gu-az7lgx.herokuapp.com/doc/user-guide/dvc#example

I feel that this is still easy to misread, because the critical information is now contained in an example at the end. Better to have it a bit further up where you start the example and actually ignore the files in the first place. I was going to make a pull request, but I feel that there are two ways to communicate it here:

Add a note that mentions that this should be temporary if you plan to write mlem artifacts from dvc pipelines. So when you say

Also, let’s add .mlem files to .dvcignore so that metafiles are ignored by DVC.
 $ echo "/**/?*.mlem" > .dvcignore
 $ git add .dvcignore

I would add a sentence afterwards along the lines: "Note: We do this here in order to simplify the migration from git-tracked files to dvc-tracked files. If you are planning to write mlem-artifacts from dvc pipelines, you would afterwards have to remove the ignore pattern."
2. Alternatively, you could in this case, explicitly add the models/rf.* files (and remove them from the git-cache). In that case, you could drop the full .dvcignore instruction to avoid confusion, but then you would use
    $ git rm -r --cached models/rf
(Not sure about the data folder there) and you would further down do
    $ git add models/rf.* models/.gitignore
Do you have a preference? I'm happy to adjust the docs accordingly.

Thanks for suggesting your help @igordertigor! IMO echo "/**/?*.mlem" > .dvcignore is important: this is a way to not track .mlem files with DVC by accident, so I think it's better to keep that and the 1st way is better.
Let's also split this page in 3 chunks:

Set up (common for both cases)
Versioning binaries manually
Using DVC Pipeline

The 1st part will end just before
 $ echo "/**/?*.mlem" > .dvcignore
 $ git add .dvcignore

In the end of 1st part we'll say: now there are 2 options: Version binaries manually or using DVC PL, and give links to both. WDYT?

TBH I'm also confused as to why the general guide tells you to ignore .mlem files but then the example kind of does the opposite. So what is it an example of? These seem like 2 different situations.

@aguschin , I think it's a good idea to separate points 2 and 3 of what you write. I created a pull request #211 to address these changes.

@igordertigor thanks for the contribution! I went ahead and updated #211, and merge it. Unfortunately, it doesn't get deployed to preview env. But now it should appear shorty at https://mlem.ai/doc/user-guide/dvc
If you like, you can check the page and bring any more feedback if you have some :)

Thanks. I'll close this one now. If I come across something else, I'll let you know.


Armor bonus skills

Hello!
I don't see the armor bonus skills currently in the database, like 2 pieces of rathalos set given rathalos mastery (i.e. Elementa Crit) is this information somewhere? Are there any plans for it to be added?
Thank you!

Hey,
Unfortunately we still do not have armor bonus skills in the database. I'm still waiting for the PC release of World so I can't add it myself, and its not really available anywhere else either (though there is some on the fextra wiki).
What I could do for now is add support for Armor Set Bonus skills, even if there's no data to populate it yet.

Humm I see, well, I'll start adding this information then, I play on PS4 and most of the armors, will start with it.

Sure! That'd help a lot!
I'm currently working on something else in the meantime, but the format I'm thinking is the following:
new: armorset_bonus_base.csv
name_en, name_ja
^ Don't really need this one, can be made from the others automatically
new: armorset_bonus_data.csv
name_en, skill, threshold
^ This would be the actual data
update: armorset_base.csv
add a bonus column that is the english name of the set bonus
^ Most important. Would make using Japanese sources possible.
The bonus base would contain translation data, and the data would contain the actual skills. Since the same armor set can have more than one skill, there can be multiple rows for the same armorset in the data table.

Just pushed the prerequisite work

I now have the means to start gathering that info. @kennedyoliveira have you been able to work on it or should I take over?

@CarlosFdez please, go ahead, unfortunately I wasn't able to make any progress, I'll be free next week, if you need help like confirmations or anything, please let me know and I'll be glad and faster to help :)

Also, one note, there are bonus skills which can come for different armors, the most different one is Rathalos Mastery it can come from multiple armor sets, don't know if you accounted that already, just fyi

I'll probably get on it soon then. But yeah that was accounted for, as well as the possibility that the same armorset can result in multiple skills.

Perfect, that's all there is :)

Finished and caught up with the current sets. If you see anything wrong let me know!


Styling of dropdown arrow on "Select a state" in Firefox

The dropdown arrow item looks different on firefox... kind of Windows 95-ish

Styling form elements tends to be difficult. Basically it's very hard to override each browsers default settings for input fields to make it look uniform across the board. It can result in some jankiness. Additionally, I err on the side of keeping form elements like drowpdowns relatively native to align with user expectation.


python3Packages.aioesphomeapi: 2.7.0 -> 2.8.0

Motivation for this change
https://github.com/esphome/aioesphomeapi/releases/tag/v2.8.0
Required in home-assistant v2021.6.0
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test home-assistant


'core.data' function callback issue when result is 'false'

file: jstree.js
ln: 1336
looks like this line of code should go to IF's (ln 1333) else branch.
if(d === false) {
   callback.call(this, false);
}
else {
   this[typeof d === 'string' ? '_append_html_data' : '_append_json_data'](obj, typeof d === 'string' ? $($.parseHTML(d)).filter(function () { return this.nodeType !== 3; }) : d, function (status) {
						callback.call(this, status);
  });
}

Did you run into a problem with this or is this just a general remark?
Which file and revision is this, also dist or src folder, please add a link?
I am closing this until further information is provided.
Best regards,
Ivan

version: last
folder: src
let's take a look to example, I hope it makes things more clear.
config: {
   core: {
     data: function(node, cb){
       try {
	     // try to load data
	     var childNodes = loadData();
		 cb.call(this, childNodes);
	   }
	   catch{
	     showErrorMessage();
		 // error occurred
		 cb.call(this, false);
		 // expected
		 // expand button stays (in other words, node stays in same state)
		 // current
		 // node expands and one child is added
	   }
     }
   }
}

Thanks for the clarification, I will have a look later today.
Best regards,
Ivan

Thank you - sorry it took me so long to respond.
Best regards,
Ivan


Consider json-schema

It would be interested if we could build up a validation configuration based on json-schema. http://json-schema.org/ This is a lower priority than other issues. Just putting it here for a possible future addition.

:+1:
Something like this would be awesome this.validation = validation.on(this).schema(jsonSchema).

I'd love this. I currently write validations for backend and frontend.

I have done something similar on a recent project to automatically pull server side validation from @jeremyskinner's FluentValidation into aurelia-validation, and found myself wondering if there is a standard schema somewhere. And then I thought what if you had something analogous to swagger spec, but just to describe api validations that could also be performed on client side. But anyways, some random things that occurred to me while I was working on it:

defining the order of your validations matters
I still want to be able to specify validations clientside manually for more complicated scenarios.

eh, tl;dr this idea is cool, I want it too.

Thanks for submitting this / commenting on this.  At this time we are closing this because we have completely re-written and are deprecating the previous feature set.  If you feel this should be re-opened please feel free to review this blog post and submit either again on this repository, or on the new validatejs bridge repository
Thanks again!


finished the install and upgraded the firmware of the controller but it would not go to 100% and not let me get out of the gui, i had to force-refresh the browser

Checklist

[X]  I am not using Home Assistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog and my problem is not mentioned there.

Deploy method
Home Assistant Add-on
Z-Wave JS UI version
9.22.0
ZwaveJS version
13.8.0
Describe the bug
it finished the install and upgraded the firmware of the controller but it would not go to 100% in the gui and not let me get out of the gui, i had to force-refresh the browser

To Reproduce
upgrade a network based controller OTW (such as tube zwave kit + zac93) and it succeeds in the background but the gui stays at 99.91% and you cannot exit out of it without refreshing the browser
Expected behavior
it to reach 100% and also let me escape the popup in cases of a failure like this
Additional context
talked to al calzone in discord and confirmed this was a zui gui bug

Were you in bootloader only mode?

Yes I believe I was

Were you in bootloader only mode?


Bring project up to date

Trying to bring the project up to date so that development can continue:

Removed the experimental TUI as it used a very old textual version and lots of things were breaking. This can be reinstated later.
Bumped Python versions.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (3538320) 51.73% compared to head (b209ec0) 71.46%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main      #64       +/-   ##
===========================================
+ Coverage   51.73%   71.46%   +19.73%
===========================================
  Files          29       10       -19
  Lines        1127      361      -766
===========================================
- Hits          583      258      -325
+ Misses        544      103      -441

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@jacobtomlinson I was wondering what was the status of the Textual-based TUI? I see it was removed in this PR, but do you plan to bring it back with the latest Textual?
I'd like to make a jpterm plugin for it, something similar to the JupyterLab Dask extension. Do you think I could steal the code you removed here, and try to adapt it to the latest Textual?

@davidbrochart I would love to bring the TUI back, but it is firmly in the category of "side project I would love to work on but don't have time for".
I plugin for jpterm would be amazing. I wonder if we could design things in a way that would allow Dask widgets to be used in both projects one day. I never finished things here, but if you want to grab what I did and adapt it then please go ahead.
Keep me in the loop and I'll see if I can find time to help out!

@davidbrochart I would love to bring the TUI back, but it is firmly in the category of "side project I would love to work on but don't have time for".

I understand, jpterm is actually exactly that kind of project for me :)

A plugin for jpterm would be amazing. I wonder if we could design things in a way that would allow Dask widgets to be used in both projects one day. I never finished things here, but if you want to grab what I did and adapt it then please go ahead.

That would be a great long-term goal, but starting from what you had here might be more approachable for me. I'll definitely let you know if/when I get something working. BTW, I remember you posted some pretty impressive screencasts demoing this TUI, but I cannot find it anymore. I think it was on Twitter, do you have a link to a post by any chance?

Yes, thanks!


Remove invalid duplicate CAVE coin

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Hey it's been 3 weeks, so I'm pinging this :) This is causing confusion for folks.


Wrong work compared with cached image

If one of images caching, then callback hiddenImage.onload called twice.
The reason this behavior next lines (started at 163 line)
  if (hiddenImage.complete) {
    hiddenImage.onload();
  }

If this lines remove, then all work correctly.
Thanks.

Problem repeated in Chrome and Firefox.

Thanks for ticket.
Looks like those lines were added deliberately, which makes me think removing them would introduce another bug.
https://github.com/Huddle/Resemble.js/commit/b6fe577f6ba881dd3534a2bd31ad0a045927a9ff
I'm going to let this issue sit for a while to see if anyone else has this problem.

Thes lines added with comment "Call hiddenImage.onload if image is cached", but if images is cached, then onload called without manual call.


Automated Package Upgrade

:tada: This PR is included in version 20.1.27 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add dashboards-observability repo to whitesource

Signed-off-by: Sayali Gaikawad<EMAIL_ADDRESS>Description
Add dashboards-observability repo to whitesource
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #3027 (e0e8d5f) into main (8c69e0e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3027   +/-   ##
=======================================
  Coverage   93.17%   93.17%
=======================================
  Files         167      167
  Lines        4602     4602
=======================================
  Hits         4288     4288
  Misses        314      314

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


Support id columns of type In32 for legacy tables

It would be neat to be able to override the type of the id column when defining a model class.
Use case: I have an existing Rails app using MySQL as database backend, and want to implement a new microservice in Crystal operating on the same database. The id column in this database is of type INT(11), which maps to an Int32 with the mysql adapter.
This currently precludes the use of kemalyst-model as a mapping layer due to its hardcoded type of Int64 type id columns.

Thanks for the feature request.
With the recent changes to support finished macro, we should be able to add a primary key macro with overrides.
Is this something you would be interested in working on?
If not, I will take a stab at it.
Sent from my iPhone

On Apr 29, 2017, at 4:37 AM, Lars Haugseth<EMAIL_ADDRESS>wrote:
It would be neat to be able to override the type of the id column when defining a model class.
Use case: I have an existing Rails app using MySQL as database backend, and want to implement a new microservice in Crystal operating on the same database. The id column in this database is of type INT(11), which maps to an Int32 with the mysql adapter.
This currently precludes the use of kemalyst-model as a mapping layer due to its hardcoded type of Int64 type id columns.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I see you forked the repo. Let me know if you need any help on this.
what do you think of:
primary id : Int32

as the macro name?
Then maybe maintain a separate hash for the primary.   In the finished macro you can replace the hard-coded id.  You might need to update all the adapters as well.

This will allow us to rename the primary key as well which should help with backward compatibility to legacy db's.

I'll see if I can come up with something and get back to you with a PR, or some questions. Your idea of a primary macro for custom column and type looks good!

Looks like the #save, #destroy, .all, .find and find_by methods will also need to be generated inside macros for this to work.

To use a customized primary column type it looks like I'll need to make most of the instance method definitions on the model take place inside macros as well.

As far as I can tell, the most straightforward solution is to add an extra primary_name parameter to the adapters' update and delete methods. The return type and casting in the adapters' insert method can be dropped, then do the casting inside the model where the value is returned instead.

Yes, sounds right.
Sent from my iPhone

On Apr 29, 2017, at 9:51 AM, Lars Haugseth<EMAIL_ADDRESS>wrote:
As far as I can tell, the most straightforward solution is to add an extra primary_name parameter to the adapters' update and delete methods. The return type and casting in the adapters' insert method can be dropped, then do the casting inside the model where the value is returned instead.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Android support

Is there a way to get this to work under android? it just fails due to missing cimgui
System.DllNotFoundException: cimgui
   at ImGuiNET.ImGui.CreateContext()

https://github.com/mellinoe/ImGui.NET-nativebuild
i found this project maybe useful


Updated CKAN to 1.6.10

Thanks for the pull request! 🎉 Homebrew depends on contributions from community members like you and we're grateful for your support.


chore(create-vite): mark template files as CC0

Description
Marked template files as CC0 1.0 (public domain).
close #16228

I think we also need to regenerate the LICENSE from the last commit

Ah, good catch.

It seems some are licenesed with MIT (unintentionally?) (https://github.com/facebook/create-react-app/issues/9480) and some are unlicensed (https://github.com/sveltejs/template/issues/78, https://github.com/vercel/next.js/pull/28007) (which makes the template be in a blurry state).
The only one that had a public domain license I found was gatsby (0BSD: https://github.com/gatsbyjs/gatsby/issues/25091).

I noticed 0BSD as a compelling alternative too, but I figured being in the public domain has lesser restrictions and concerns. But as a non-lawyer they both look similar and there shouldn't be any downsides choosing either I believe.

https://www.tldrlegal.com/license/creative-commons-cc0-1-0-universal
https://www.tldrlegal.com/license/bsd-0-clause-license

Different from CC0, 0BSD and MIT No Attribution License are OSI approved licenses and might be compelling in that point.

Ah, one thing I remember now is that technically 0BSD and MIT-0 are copyright licenses. You'd put your name/year etc in the license. Some 0BSD licenses don't have the copyright line, but I believe in that case it's still implicitly copyrighted. The main distinction of those licenses is that you don't need attribution, but no attribution doesn't mean waiving copyright.
CC0 technically holds copyright, but it tries to waive as much as possible by law. That means when someone uses the templates, we no longer hold the rights (as much as possible by law) to that template code. In which case, I think CC0 is still beneficial here.
(With all that said, IANAL and this is based on my own research)


Example Playgrounds for Properties and Methods of Vector3

My apologies for the length of this comment, it is because I need advise on whether it will be part of my very limited project or a much larger undertaking by the community. As a large community undertaking it needs considering and approval by @deltakosh along with @PirateJC and @RaananW as it involves the docs even though it is on the API side and I would like to include @sebavan as he as also been very supportive and helpful to me. Please include anyone else you would like to talk to about it.
Reason Behind PR
Within the API classes the search the playground for each property and method is at best hit and miss, so I thought that each property and method of a class should have its own playground example, which should be simple and as meaningful as possible. For my limited project, except for a few where they were trivial or I couldn't think of a good example or I forgot, I have done this for all the methods and classes of Vector3 in this PR.
Implications for a Large Community Undertaking
An announcement about the undertaking.
If members do put in example PGs they could come in piecemeal so lots of individual PRs, unless there was somewhere else to submit them. This would require a moderator or moderators.
Some rules for submitting an example PG, eg

PGs should be simple and where possible meaningful.
Code for the particular property or method should appear near the top of the PG
When meaningful means more complex code this can be achieved by wrapping the
a. method code in a function as in https://playground.babylonjs.com/#R1F8YU#111
b. method code within an onBeforeRenderObservable as in https://playground.babylonjs.com/#R1F8YU#63
c. complex code within functions as in https://playground.babylonjs.com/#R1F8YU#106
There should be comments at the top describing the method/property also stating the example use matching a template
Template PGs for each class would be needed
New or edited methods without a PG should have a PG submitted when method PR submitted

I am sure there are more implications and I am happy for the rules to change. I only came up with them after doing half the methods.
Also as members of the community have their own agendas it is probably wishful thinking that enough people would get involved to get it done.
In terms of my very limited project I am happy to continue with the math.vector.ts classes, eg Vector2, Vector4, etc
Let me know what you think and whether it is just my limited project or a community appeal.

Wow, thank you so much John! This is amazing. I didn't go into all of the examples, but a playground to each of those methods will be very very helpful. especially for functions like project/unproject which can sometimes be a mystery.
There is a very small formatting issue and I think it will be great if the example be a part of the description (i.e. - before the @returns and @param lines.

Having problems getting prettier to run to correct any formatting errors.
Also tried
npm run format:check and got
PS C:\Users\johnk\Documents\Github\Babylon.js> npm run format:check

<EMAIL_ADDRESS>format:check
prettier --check ./packages//src//*.{ts,tsx,js,json,scss,css}

'prettier' is not recognized as an internal or external command,
operable program or batch file.
I have checked that is it set as default formatter for all languages.
Happy to move it before @params and @return.
Will wait until any more suggestions so I can deal with them in one go.

Wow, thank you so much John! This is amazing. I didn't go into all of the examples, but a playground to each of those methods will be very very helpful. especially for functions like project/unproject which can sometimes be a mystery.
There is a very small formatting issue and I think it will be great if the example be a part of the description (i.e. - before the @returns and @param lines.

Having problems getting prettier to run to correct any formatting errors.
Also tried npm run format:check and got
PS C:\Users\johnk\Documents\Github\Babylon.js> npm run format:check

<EMAIL_ADDRESS>format:check
prettier --check ./packages//src//*.{ts,tsx,js,json,scss,css}

'prettier' is not recognized as an internal or external command, operable program or batch file.
I have checked that is it set as default formatter for all languages.
Just found the format error (I hope) exactly one extra space at the end of one line.
Happy to move it before @params and @return.
Will wait until any more suggestions so I can deal with them in one go.

Hey @BabylonJSGuide, so sorry about that prettier issue.
can you try running npm install prettier and try running the formatter right after? If that doesn't work, try installing prettier globally (i.e. npm install prettier -g). And as a final solution, try this -
npx prettier --write ./packages/**/src/**/*.{ts,tsx,js,json,scss,css}
If that doesn't work let's hop on a call to check what's up with your local repo.

Hi @RaananW, thank you for the help. It was my problem, I just did a new clone of Babylon.js and then behaved as if it was the old one. Completely forgot to do npm install. All well now. As no more suggestions will get on with move Example line to descriptions.

Examples move up to descriptions as requested


add-chain-crawler

Codecov Report

Merging #19 (e69257c) into main (456eb7c) will increase coverage by 0.18%.
The diff coverage is 39.11%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main      #19      +/-   ##
==========================================
+ Coverage   38.94%   39.12%   +0.18%
==========================================
  Files          21       24       +3
  Lines         945     1117     +172
==========================================
+ Hits          368      437      +69
- Misses        531      621      +90
- Partials       46       59      +13

Impacted Files
Coverage Δ

pkg/crawlers/registry/goquery/chain/url_getter.go
0.00% <0.00%> (ø)

pkg/goquerywrapper/util.go
0.00% <0.00%> (ø)

pkg/model/db.go
0.00% <ø> (ø)

pkg/repo/registry/in_memory/repo.go
3.07% <0.00%> (-0.32%)
:arrow_down:

pkg/repo/registry/pg/repo.go
1.04% <0.00%> (-0.27%)
:arrow_down:

pkg/server/server.go
7.79% <0.00%> (-0.32%)
:arrow_down:

pkg/tree/tree.go
74.02% <0.00%> (ø)

pkg/tree/util.go
0.00% <0.00%> (ø)

pkg/goquerywrapper/goquery.go
42.22% <41.93%> (+2.63%)
:arrow_up:

pkg/crawlers/registry/goquery/chain/crawler.go
50.00% <50.00%> (ø)

... and 3 more


Fix Pages Beyond 10

Description
Allow set pages beyond 10 in the UI Fixes, reported in https://github.com/streamdeck-linux-gui/streamdeck-linux-gui/discussions/69#discussioncomment-7063136

Is there a way to do it without the 999999999? seems a bit awkward. does removing the maximum breaks anything?

https://doc.qt.io/qt-6/qspinbox.html#maximum-prop the QSpinBox force me to set some default, otherwise 99 will be used, I can used also the max int supported in python which is<PHONE_NUMBER>854775807, I just set 999999999 because I dont think that someone will use ever 999999999 pages

Ok, fair enough. that value seems fine.

I just set 999999999 because I dont think that someone will use ever 999999999 pages

xD xD xD  .... lets jut keep this quote here, wouldn't be surprised if someday someone comes and says "heeey. I reached the max!"


fix(nx-plugin): add support for nx paths. fixes #1056

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/analogjs/analog/blob/main/CONTRIBUTING.md#-commit-message-guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

Which package are you modifying?

[ ] vite-plugin-angular
[ ] vite-plugin-nitro
[ ] astro-angular
[ ] create-analog
[ ] router
[ ] platform
[ ] content
[x] nx-plugin
[ ] trpc

What is the current behavior?

Closes #1056
What is the new behavior?
Adding nxViteTsPaths in NX workspace
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
[optional] What gif best describes this PR or how it makes you feel?

Thanks! @allcontributors add @ezzabuzaid for code


fix GEARPUMP-155, integration test failure

[Current coverage][cc-pull] is 65.96%

Merging [#40][cc-pull] into [master][cc-base-branch] will increase coverage by 0.11%

@@             master        #40   diff @@
==========================================
  Files           178        178
  Lines          5926       5926
  Methods        5590       5590
  Messages          0          0
  Branches        336        336
==========================================
+ Hits           3902       3909     +7
+ Misses         2024       2017     -7
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by [baa7dea...65e0ee2][cc-compare]
[cc-base-branch]: https://codecov.io/gh/apache/incubator-gearpump/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/apache/incubator-gearpump/compare/baa7dead915593bfcc76f0f1c73495d2d4e724d9...65e0ee2d80ec6dd76f3d44456284ab8fabcf52bd
[cc-pull]: https://codecov.io/gh/apache/incubator-gearpump/pull/40?src=pr

+1


Don't submit the queue when empty

Currently it's checking queued.empty? which always turns out
to be non-empty if you provide a source.
This check relies on just empty? which only utilizes @queued
without the merging of the global metadata.

I think the definition of empty? also needs to change, to look something like this:
  def empty?
    gauges.empty? && counters.empty? && measurements.empty?
  end

Hi @cyx. Thanks for the report and contribution.

Currently it's checking queued.empty? which always turns out to be non-empty if you provide a source.

Could you elaborate on this with an example to reproduce? Seems like the expected behavior:
q1 = Librato::Metrics::Queue.new
q1.empty?
# => true
q1.add foo: { value: 1, source: 'bar' }
q1.queued
# => {:gauges=>[{:value=>1, :source=>"bar", :name=>"foo", :measure_time=>1489707683}]}
q1.empty?
# => false
q2 = Librato::Metrics::Queue.new(source: 'test')
q2.empty?
# => true
q2.add foo: { value: 1, source: 'bar' }
q2.queued
# => {:gauges=>[{:value=>1, :name=>"foo", :measure_time=>1489707741}], :source=>"test"}
q2.empty?
# => false

I'm afk now, will try to provide tomorrow.

@chancefeick here are the steps to reproduce:
> aggregator = Librato::Metrics::Aggregator.new
> queue = Librato::Metrics::Queue.new(source: 'test')
> queue.empty?
=> true
> queue.merge!(aggregator)
> queue.empty?
=> false
> queue.queued
=> {:gauges=>[], :source=>"test"}

Thanks for the example, that's definitely not the intended behavior.
These changes make sense to me 👍 Can you please include specs around this fix so we can ensure it doesn't unintentionally break in a future version?

@cyx I came up with the following. If you think it looks good you can just add it to your branch:
diff --git a/spec/unit/metrics/queue_spec.rb b/spec/unit/metrics/queue_spec.rb
index 163f34f..ea874ff 100644
--- a/spec/unit/metrics/queue_spec.rb
+++ b/spec/unit/metrics/queue_spec.rb
@@ -301,6 +301,11 @@ module Librato
           subject.add foo: {type: :gauge, value: 121212}
           expect(subject.empty?).to be false
         end
+
+        it "returns true when nothing merged" do
+          subject.merge!(Librato::Metrics::Aggregator.new)
+          expect(subject.empty?).to be true
+        end
       end

       describe "#gauges" do
@@ -544,6 +549,14 @@ module Librato
             expect(subject.queued).not_to be_empty
           end
         end
+
+        context "when there are no metrics" do
+          it "it does not persist and returns true" do
+            subject.merge!(Librato::Metrics::Aggregator.new)
+            subject.persister.return_value(false)
+            expect(subject.submit).to be true
+          end
+        end
       end

       describe "#time" do

@mikehale thanks! Pushed the commit.

@chancefeick can you check and see if it's all good now?

@cyx LGTM 👍
Released in v2.1.1. Thanks again for your contribution.

Thanks!


question. SAP diagnostic agent publishing data to cloud watch

Hi team
Is anybody thought or possibly tried to publish the data from OS level - sap diagnostic agent - to AWS Cloud watch?
The background of this topic is monitoring the certificate expiration. A possible similar idea already has been discussed as part of an actual solution.

Not that i am aware of. As the SAP Diagnostic Agent is tailored to SAP Solman, does tit have an "API / interface", that we could hook up to?


Add emojicite to l3prefixes

Hi, could you add your packages to l3prefixes.csv.
Here's a link:
https://github.com/latex3/latex3/blob/master/l3kernel/doc/l3prefixes.csv
That file is for contact info, but also to keep track of the registered prefixes in packages to avoid that two packages use the same prefix and clash.
Thank you so much for this package , they're great :)
Greetings
Pablo

Thank you very much.


switch to gh-action + maintenance

Codecov Report

Merging #6 (d9964e8) into master (2b740d8) will decrease coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
- Coverage   95.45%   95.23%   -0.22%
==========================================
  Files           2        2
  Lines          22       21       -1
==========================================
- Hits           21       20       -1
  Misses          1        1

Impacted Files
Coverage Δ

src/WidgetsBase.jl
66.66% <0.00%> (-8.34%)
:arrow_down:

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 2b740d8...d9964e8. Read the comment docs.


Content-type is now autodetected by the service, consider removing from SDK

Make sure we reaaaally test this before deleting

Decision was to leave our detection as is


Geoconnex Phase 2

Describe the feature you'd like and what it will do
This Issue will serve as a landing spot for suggestions and ideas to improve our Geoconnex implementation in Hydroshare
Brainstorm

Toggle to allow mapped layers to have different colors for each item in the layer
Flash of Unstyled Content FOUC

add padding when fit map to features so they aren't on the map border

@devincowan I figured I'd put in here that we've updated this service so that it can now be queried by polygon according to this standard via GET or POST (see example 9), and in general should be much faster, including the /pws endpoint

@ksonda that is SUPERB! I will give it a try, thanks

ALso note the json response to any query from this service also now includes fields ```"numberMatched": 187474,
"numberReturned": 200,```
to help with iteration via offset and limit query parameters after a first query

@devincowan I'm noticing that there seem to be issues in the hydroshare geoconnex system for many of collections. The only ones that seem to work are HUCs, dams and aquifers.

@ksonda thanks I see this now too
Requests for jsonld to Geoconnex api used to return a context with a "schema:name" for these features, but looks like that's no longer the case. For example:
https://reference.geoconnex.us/collections/mainstems/items?f=jsonld&lang=en-US&limit=1&skipGeometry=true&offset=0
For the mainstems collection, we used to get a context with
{
      "schema": "https://schema.org/",
      "name_at_outlet": "schema:name"
}

Loosely related to https://github.com/internetofwater/geoconnex.us/issues/178

There may be a quick fix but there me be an issue upstream and geopython/pygeoapi. Let me check
Are there any other properties you defintiely need than schema:name?

we really only rely on uri and name at this point

so this would explain why gages wouldnt work, but, for example, counties doesn't work either.
https://reference.geoconnex.us/collections/counties/items?f=jsonld&lang=en-US&limit=1&skipGeometry=true&offset=0
which gets you
{
    "@context": [
        {
            "schema": "https://schema.org/",
            "type": "@type",
            "features": "schema:itemListElement",
            "FeatureCollection": "schema:itemList"
        },
        {
            "schema": "https://schema.org/",
            "name": "schema:name",
            "census_profile": {
                "@id": "schema:subjectOf",
                "@type": "@id"
            }
        }
    ],
    "type": "FeatureCollection",
    "features": [
        {
            "@id": "https://geoconnex.us/ref/counties/01001",
            "type": "schema:Place"
        }
    ],
    "numberMatched": 3233,
    "numberReturned": 1,

vs. huc 2 which works
{
    "@context": [
        {
            "schema": "https://schema.org/",
            "type": "@type",
            "features": "schema:itemListElement",
            "FeatureCollection": "schema:itemList"
        },
        {
            "schema": "https://schema.org/",
            "name": "schema:name",
            "gnis_url": {
                "@id": "schema:subjectOf",
                "@type": "@id"
            }
        }
    ],
    "type": "FeatureCollection",
    "features": [
        {
            "@id": "https://geoconnex.us/ref/hu02/01",
            "type": "schema:Place"
        }
    ],
    "numberMatched": 22,
    "numberReturned": 1,

thanks @ksonda appreciate you checking on these -- I have it on my list to add browser tests so that we catch these sooner ;)
Sounds like the Counties issue could be on our end. Appears to work for me, when the HS resource has a defined Spatial Coverage:

With no Spatial Coverage, the query returns too many features (we limit it to 1000):

I understand the too many thing.

I didn't consider it could be a browser thing though (this is vivaldi...which...lol yeah you probably dont need to go out of your way for)

I thought maybe it could be something particular about counties in that Denver area that I didn't account for in how we do the mapping etc. But seems to work for me on Chrome/Firefox

@ksonda if you get a chance, maybe shoot me a snapshot of your console after this error occurs?

I'm going to use this issue #4801 as the current "bug" in production
Moved "Geoconnex phase 2" -> https://github.com/hydroshare/hydroshare/issues/5003

mk so in vivaldi it looks like a CORS setting issue
Although it "works" for HUC6 in this example, there's no request, so it's like it was cached since I did it already maybe?

However, there's also maybe some kind of upstream problem with how you decide what to use for name.  Note the context for the mainstems/items?f=jsonld now looks like
    "@context": [
        {
            "schema": "https://schema.org/",
            "type": "@type",
            "features": "schema:itemListElement",
            "FeatureCollection": "schema:itemList"
        },
        {
            "schema": "https://schema.org/",
            "name_at_outlet": "schema:name"
        }
    ],
    "type": "FeatureCollection",
    "features": [
        {
            "@id": "https://geoconnex.us/ref/mainstems/1",
            "type": "schema:Place"
        },

I see in the console that the request being passed is https://reference.geoconnex.us/collections/mainstems/items?f=json&limit=1000&properties=uri,NAME&bbox=-104.5047,39.428,-105.4825,39.9774
So it looks like you've just hardcoded somewhere what the field for name is and it's become outdated.

thanks @ksonda you're right that we cache a lot of the features in browser store (I'm going to test, given the api's improved performance, we might not need to anymore) -- the problem you're seeing with the seemingly hardcoded urls is actually because the jsonld is also cached browser side (so it is still sending requests based on your old schemas with the caps "NAME"). I have a fix to remove the jsonld caching ready for deploy it should hit prod in a couple hours

also remember /collections/pws works now

100% thanks @ksonda

@ksonda I think there is an issue with the pws
https://reference.geoconnex.us/collections/pws/items?f=jsonld&limit=1
{
    "@context": [
        {
            "schema": "https://schema.org/",
            "type": "@type",
            "features": "schema:itemListElement",
            "FeatureCollection": "schema:itemList"
        },
        {
            "name": "https://schema.org/name"
        },
        {
            "state_uri": "https://schema.org/geoWithin"
        },
...

Note "name": "https://schema.org/name"
https://reference.geoconnex.us/collections/pws/items?f=json&limit=1
{
    "type": "FeatureCollection",
    "features": [
        {
            "type": "Feature",
            "properties": {
                "city_served": null,
                "is_wholesaler_ind": true,
                "pred_05":<PHONE_NUMBER>66911,
                "contribution_date": "2022-11-01",
                "fid": 1088,
                "primacy_type": "Tribal",
                "pred_50":<PHONE_NUMBER>62528,
                "service_area_type": "['Residential Area']",
                "pws_name": "MASHANTUCKET PEQUOT WATER SYSTEM",
                "primary_source_code": "GU",
                "pred_95":<PHONE_NUMBER>12842,
                "primary_water_source": "Groundwater under influence of surface water",
                "state_code": "01",
                "tier": "3",
                "source_url": "https://www.hydroshare.org/resource/9ebc0a0b43b843b9835830ffffdd971e/data/contents/temm.gpkg",
                "sdwis_link": "https://enviro.epa.gov/enviro/sdw_report_v3.first_table?pws_id=010106001&state=01&source=0&population=0",
                "county_served": null,
                "centroid_lat": 41.444031,
                "data_source": "Modeled",
                "place_uri": "https://geoconnex.us/ref/places/",
                "population_served_count": 37807.0,
                "centroid_lon": -71.995973,
                "contact_email"<EMAIL_ADDRESS>                "wholesaler": "Is a wholesaler",
                "service_connections_count": 119.0,
                "centroid_quality": "ECHO: ZIP CODE CENTROID",
                "source_date": "2022-11-01",
                "uri": "https://geoconnex.us/ref/pws/010106001",
                "primacy_agency_code": "01",
                "owner_type_code": "N",
                "geometry_source_detail": "Modeled",
                "state_uri": null,
                "boundary_type": "Unknown Service Area - Using EPA ECHO facility location with buffered radius to represent PWS"
            },
...

There is a "pws_name": "MASHANTUCKET PEQUOT WATER SYSTEM", but there IS NO "name" field

I'm pushing a fix to the context now. should be up in 5-10 mins

it's done

thanks @ksonda! ZIPPY FAST!
So this is probably my lack of knowledge. Looks like there are two different name mappings for the PWS features, in 2 separate contexts:

https://reference.geoconnex.us/collections/pws/items?f=jsonld&limit=1
But of course when we query there is only the pws_name
https://reference.geoconnex.us/collections/pws/items?f=json&limit=1
How do we know which context will hold the correct name field for the features?

Hmm we’ll I’m this case it’s extraneous and I should remove that from the context. But there are legitimate cases where multiple fields might have the same context mapping, such as if a thing has multiple names.
One approach here would be to look at /collections/pws/queryables and pick the first one that is there that shows up in the context as schema:name.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Devin Cowan @.>
Sent: Thursday, March 9, 2023 7:35:58 PM
To: hydroshare/hydroshare @.>
Cc: Kyle Onda @.>; Mention @.>
Subject: Re: [hydroshare/hydroshare] Geoconnex schema:name missing from some collection contexts (Issue #4801)
thanks @ksondahttps://github.com/ksonda! ZIPPY FAST!
So this is probably my lack of knowledge. Looks like there are two different name mappings for the PWS features, in 2 separate contexts:
[image]https://user-images.githubusercontent.com/17934193/224192704-734deefa-0e4b-4f96-86ae-a7378a6931c0.png
https://reference.geoconnex.us/collections/pws/items?f=jsonld&limit=1
But of course when we query there is only the pws_name
https://reference.geoconnex.us/collections/pws/items?f=json&limit=1
How do we know which context will hold the correct name field for the features?
—
Reply to this email directly, view it on GitHubhttps://github.com/hydroshare/hydroshare/issues/4801#issuecomment-1463035910, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKQHTNV6CQ3WJ6HX4CI4N53W3JZO5ANCNFSM6AAAAAARJJCIJM.
You are receiving this because you were mentioned.Message ID: @.***>

PR https://github.com/hydroshare/hydroshare/pull/5005 releasing today as hotfix 2.4.1
@ksonda sorry this one got delayed for a variety of reasons...


Image optimizations

Get the next image component to work so there isn't any layout shift and can improve loading/lazy load/stream/etc.

https://github.com/ethanniser/NextFaster
for referencing later

Too many external sources and differing dimensions. Ran out of time to get this working.


About the latest release

👋 it might be good to mark 1.1.0 as latest release, thanks! 🙏

relates to https://github.com/Homebrew/homebrew-core/pull/152950

Indeed, sorry, forgot to have that checkbox selected. :)

thank you! :)


[edn] Use MuBi4False instead of 4'h0 for flag0 when enabling entropy src

Recently, flag0 got converted from a single bit to a MuBi4 field. When seeing invalid MuBi4 values, the CSRNG hardware triggers a recoverable error and stops giving out entropy. As it turns out, in some places the implementation still assumes 4'h0 was interpreted as a legal MuBi4False, which effectively breaks EDN auto mode and all tests using that such as otbn_randomness_test.
This is related to lowRISC/OpenTitan#15165.

Thanks for the quick feedback @mwbranstad ! Indeed, the hjson/RTL change doesn't seem necessary as flag0 is used for the instantiate and reseed commands only according to the documentation. I can revert this unless we want to leave it to keep things aligned.
In contrast, the SW change of this PR is required. Otherwise EDN auto mode and all tests using it are broken because of the alert that is triggered inside CSRNG.

Oh, now I understand why CI fails. The edn_unittest times out...

yeah that's right. You'll need to change quite a number of things in both the csrng and edn unittests.
I got annoyed...so just left the 0 case in there :)

EDN unittests should now work. I'll have another look at the CSRNG unittests.

Yep.

Now it all unittests should work.

Thanks @tjaychen for approving this. I've now implemented your change and also removed the hardware change. As @mwbranstad clarified, this wasn't needed. I re-ran the OTBN randomness test locally. Feel free to merge this once CI passes. The nightly regressions can then pick this up.

thanks for fixing this @vogelpi


getValue for single element

Feature request: Using getValue can be pretty complicated when dealing with a single editable, given that after the call to getValue, you still need to do .whateverElementNameIs to access the value.  I have encountered several use cases that this makes things complicated.  It would be nice if there were a getValue that retrieved the value directly:
http://jsfiddle.net/xBB5x/1306/

Please anybody can help me with this?
http://stackoverflow.com/questions/29473716/x-editable-post-method


CloudDNS and Split DNS issue

I am trying to setup Google Cloud with Split Horizon DNS but txt records are created in private zone. I gave a try - --dns01-recursive-nameservers=<IP_ADDRESS>:53 and still dns records are created in private zone. I have read https://github.com/jetstack/cert-manager/issues/903 and there is some useful explanation about these cases:

This creates issues for cert-manager because when we perform a DNS query to find the DNS authority, the internal nameserver will respond with the private DNS zone root, consequently failing the self-check (as cert-manager will have updated the public zone).
This is mitigated by allowing users to specify the --dns01-self-check-nameservers flag, which will alter the 'root' nameserver used to perform the initial query - the idea here being that by specifying e.g. <IP_ADDRESS>, they will begin recursing the public zone to find the authority that Let's Encrypt will see.

All my tries ended up in private namespace. Here is some logs from our servers:
I0219 09:34:29.137645       1 dns.go:101] Checking DNS propagation for "cemo.com" using name servers: [<IP_ADDRESS>:53]
I0219 09:34:29.338374       1 sync.go:173] propagation check failed: DNS record for "cemo.com" not yet propagated
I0219 09:34:29.338696       1 controller.go:211] challenges controller: Finished processing work item "cert-manager/cemo-com-2281260743-0"
I0219 09:34:39.338958       1 controller.go:205] challenges controller: syncing item 'cert-manager/cemo-com-2281260743-0'
I0219 09:34:39.339134       1 logger.go:103] Calling Discover

Interesting how I bumped into the same issue just 18 hours later.
It looks like clouddns provider is using v1 api which does not have Visibility field in ManagedZone, so when two zones are returned by Cloud DNS API it just blindly picks first one from array.
Need to bump API version and and always pick zone with visibility=public (even if just one zone returned)
https://github.com/jetstack/cert-manager/blob/master/pkg/issuer/acme/dns/clouddns/clouddns.go#L220
For now, the workaround is to host public zone in separate project from private.

Thanks for digging in @ivan-gusev!
/help
/area acme/dns01
/area acme
/kind bug
/priority important-longterm

I think most of the people's dns zone names are in the format of "xxx-private-zone" and "xxx-public-zone". It seems that API returns in alphabetical order. I had changed "xxx-public-zone" -> "xxx-pblic-zone"  and public zone returns first. I am of course not sure but it may be another workaround for a while too.


eslint and prettier added to the framework and files formatted

eslint config added
prettier config added
already existing files formatted

I applied the necessary changes based on the comments.


Drop support for bare-bones setups in the project template

It's been quite a while since Docker established a solid presence on DevOps landscape. From my professional and personal experience bare-bones aka virtualenv-based setups have been consistently displaced with Docker-powered ones in the quest for reproducibility and ease of distribution by enterprises and individuals, provided they hadn't been using other containerization and/or virtualization solutions already.
I suggest that we start relieving Cookiecutter Django project template of the maintenance burden brought by virtualenv runtime environment support (which is not as heavy as it might have been yet still omnipresent throughout the project development lifecycle occasionally calling for documentation improvements, workflow clarifications, and bugfixes) eventually leaving Docker as the only officially maintained runtime.
Clearly, that would be a profound decision for the community and the team to make, and even though I personally am sympathetic to the proposal some other developers might not as well be. It's apparent to me that the change would primarily affect those who're just making their first steps in Python / Django world (we've all been there), those, whose primary incentive is to get the ball rolling ASAP whatever the projected cost of technical debt they'd likely accumulate afterwards (or they may not -- it all comes down to real-world constraints while we're referring to the general public here, right?). Speaking of real-world constraints, they do vary dramatically across teams and projects, yes, but nonetheless my deep belief is that it's best to learn the utter basics of modern infrastructure configuration sooner than later.
I perceive the raised issue as a lowest priority endeavour. My being perfectly aware of the topic's highly discussive nature I hope for all stakeholders, to the extent of their availability, of course, to share some feedback on the issue at hand.

Thanks for kicking off this discussion. I'm a bit on the fence for this, I get the 2 sides of the question and as you say it's an important decision. I'll give it some thoughts...
PS: The bugfix you link to is under Docker, which seems against your point. Did you copy/paste the wrong one?

https://github.com/docker/docker.github.io/issues/6910
Now I am not so sure about not considering Docker alternatives.

@sfdye even though that was unethical of them to add login page for Mac version that's still an ethical concern, not a functional one. I don't see any serious lightweight containerization alternatives to Docker as of now, at least not to my knowledge.

@shuttle1987 mind elaborating on the nature of your objection?

My objection is that forcing people to sign up to a 3rd party service makes it a functional concern for this project. I've been involved in running workshops where people have to get some packages set up quickly and a base image with something like Docker is great, forcing people to make an account on a 3rd party service is a big pain there. If there was some alternative lightweight containerization services for Mac that could easily be used as a drop in replacement for Docker I think there's really no issues with going the container approach as continuity of the project would be easier to achieve in the case that Docker gets harder to install. I agree that there's not a really good alternative out there on Mac right now and it's that lack availability of alternatives to Docker which would make me very wary to drop support for a non-containerized install.
That said if the only goal is to make it as easy for complete beginners to get started I see the appeal of the containers, since setting up installs that can be replicated is a pain. Pipenv helps this fairly substantially for Python dependencies but there's still the annoying case of system packages that Docker is really good at handling.
So I guess it's a +0 from me in some senses, but I fully acknowledge I'm likely not the target audience for this package and also I'm not the maintainer of this project so if it saves a lot of time in that sense I can see the appeal there.

When I started my first django project, I really appreciated the fact that cookiecutter-django took care of many of the initial setup and configuration steps of a bare bones setup.
I can see how maintaining all of that detail can be a challenge though.

It's evident to me know that keeping virtualenv should remain a priority to us. Closing this one until (and unless) opposing opinions come up.


Adding support for late ferry data?

Hi, just came across your API when attempting to build a BCFerries module for HomeAssistant, thanks for keeping this up to date!
The only other module I found was a bcferries python module that is 7 years old and definitely no longer works, sadly.
The one thing they exposed in theirs that I think yours could benefit from, is exposing whether or not the ferry in question is late or not.  For instance, the page you could get this from for Langdale<>Horseshoe Bay would be: https://www.bcferries.com/current-conditions/departures?terminalCode=LNG
I was going to take a crack at adding a parser for this data, but wondering if you or anyone else has already attempted this, or if you are aware of a reason it wouldn't work?

I haven't attempted it yet, honestly I didn't know this info was available on their site. I don't think it would be too hard to add a 'late' field to each sailing on the api, checking that page for the next ferry, and if the text is anything but "On time" set it to true.
I'm currently working on a larger update to revamp the api to be used with a ferry app I'm building, I'll look at adding this to the update.

That would be awesome!  Given that they return "late" or "cancelled" results of each sailing time, I would suspect you could even match these up for all of the results - in some cases, when there are heavy delays, it cam be helpful to know if the next 2 sailings are late, but the 3rd isn't.  They post this ahead of time sometimes, if they know mechanical issues or weather are going to resolve themselves around a certain timeframe.
I dont know if you have a github sponsor page set up (kind of like a tip jar idea that github announced a few years ago) but if you do, I'd be happy to send you a donation for your time!

I agree, having that info for all possible sailings would be very helpful, I'll make sure to add that when I do the update.
I didn't have a sponsor page set up but I just made one, nobody's ever asked about that before. Feel no obligation to do so but anything would be greatly appreciated, and would all go to hosting this project. Thanks!

I've just updated the API with the larger changes, and have included a spot on sailing data for vessel status to be included. I'll look into adding it next week, but this update will also require some changes to any current usage of the api.

Nice job!
Looking forward to trying it out

From: Sam Pratt @.>
Sent: Wednesday, March 23, 2022 11:12:02 AM
To: samuel-pratt/bc-ferries-api @.>
Cc: daytonturner @.>; Author @.>
Subject: Re: [samuel-pratt/bc-ferries-api] Adding support for late ferry data? (Issue #27)
I've just updated the API with the larger changes, and have included a spot on sailing data for vessel status to be included. I'll look into adding it next week, but this update will also require some changes to any current usage of the api.
—
Reply to this email directly, view it on GitHubhttps://github.com/samuel-pratt/bc-ferries-api/issues/27#issuecomment-1076661617, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACZEPSMXQVKFZEJRA33IDOTVBNNHFANCNFSM5RJBZKHQ.
You are receiving this because you authored the thread.Message ID: @.***>

So I happened to notice that delays are now showing up, relatively cleanly, in the API 'time' field.
I wonder if its possible to split the additional 'late' information out into its own field.
e.g.:
{"sailingDuration":"","sailings":[{"time":"11:45 AM Delayed approx. 1h",

Instead of having "Delayed approx. 1h" in time, I wonder if we could put it into the "vesselStatus" field, or another?
It will make parsing the fields much easier!


Dropping Python 3.6 support

Python 3.6 does not support seek on files opened in a ZipFile. This was introduced in Python 3.7.
See here: https://bugs.python.org/issue22908
We need this if we want to be able to read NetCDF files stored in a ZIP container.
Do we need to continue supporting Python 3.6? Ubuntu 20.04 LTS has Python 3.8 but 18.04 LTS has 3.6.
Any thoughts @mcrot @AntoineSIMTEK

See also https://github.com/ContactEngineering/SurfaceTopography/pull/79

I am fine with dropping 3.6.
I had issues installing muFFT on a singularity machine with ubuntu 20, but as soon as this is fixed I will upgrade it.
I guess one can still apt install a specific python version.

Me too, dropping 3.6 is fine. The production machine is still on Ubuntu 18.04 LTS, but the web app run on a container
with own interpreter.

Okay, will drop this then.


chore(kfm): bump KFM to 752566e48829300669a3512c2af4236ba622a4e8

This bumps KFM to 752566e48829300669a3512c2af4236ba622a4e8 to include changes in https://github.com/bf2fc6cc711aee1a0c2a/kas-fleet-manager/pull/1423 done by @vmanley
/cc @MikeEdgar can I get a review on this as well as release of new kas-installer version? Thanks

@machi1990 - I'll tag a new release after #251 is merged. It should be verified shortly

@machi1990 - I'll tag a new release after #251 is merged. It should be verified shortly

Thanks.


Move account selection icons to the bottom

Is your feature request related to a problem? Please describe.
I have installed the latest release of k9 to test the new interface. It looks modern and nice. Yet, I already have pain in my right hand from switching accounts. I have a phone with a 6.5" screen and switching accounts is very difficult. I have to stretch my thumb all the way to the top left corner, a very uncomfortable motion.
Describe the solution you'd like
It would be great if the icons were moved (optionally?) to the bottom of the drawer (the thing that slides in from the left side of the screen). This does not have to be sth super slick and intelligent, just a new row with the account icons in series one by one. If you have more that the row can show, it would just be scrollable. This would also solve the issue that when more than 3 accounts are in place, using the account icons at the top of the drawer is unintuitive and only cycle among 3 accounts only.
The top space of the drawer would show what is still does, but the account selection buttons (not the bigger one that shows the active account) could be replaced with account, folder and general setting buttons. This way the "Settings" option would be moved from the bottom to the top as a non-frequently used option (I would argue that users don't visit the settings more than they switch accounts).
Here is a rough example:

Describe alternatives you've considered
I tried using either the account icons or the drop down list, but they are both at the left top corner. I also tried using the unified inbox, but I turned it off. It also wouldn't solve the issue of switching accounts for other things other than reading the inbox.

Optimizing the app for one-handed operation on a large screen is not a design goal for K-9 Mail. We'll be sticking to the standard layout.
See https://material.io/components/navigation-drawer


[Bug]

What happened?

hello i got this error after login in to my boxbilling after 5month, how to fix that?
BoxBilling Version
v4.21
What version of PHP are you running?
PHP 8.1
Additional Notes
Additional notes

Is it possible that your SQl user lost permission to the DB?

Is it possible that your SQl user lost permission to the DB?

@evrifaessa stale, please close.


Add about hero banner

ticket
Copied over the portfolio-hero.html contents to about-hero.html and edited the copy. The header and footer (also assigned on this ticket) had already been added to about.html. I had to slightly increase the max-width of .portfolio-hero-text to match the design preview on this ticket, which also increased the max-width of the hero text on portfolio.html since both are sharing class names now. Should I change the portfolio-hero class names to something broader since they are being shared with the about-hero now?

@kylefarmer85 Sure, let's rename the class name. How about "secondary-hero" or something along those lines?


GDScript 2.0: Project crash when editing an extended class variable

Godot version
v4.0.alpha.calinou.499eec13a
System information
Windows 11
Issue description
Wasn't sure what to call this, but I tried to add a script with 'class_name XXResource' that 'extends Resource' into another script using:
@export var resource_script: Resource = resource_script as XXResource  (inside other script)
By trying to reference resource_script variables/methods anywhere, it ends up crashing every time.
I can't write
@export var resource_script: XXResource
because I can only export pre defined types in Godot (in this case, Resource)
and if I write
@export var resource_script: Resource
then it won't crash, but at the expense of 1. not being able to autocomplete for XXResource vars/methods, and 2. not being able to be specific in which type of Resource I infer it to be
@export is not relevant, as it will crash without it (see sample project)
Steps to reproduce
You only need 2 scripts (if you want, just download the reproduction, as they're already made for you)
extends CharacterBody3D

class_name CharacterClass

var running = false

and
extends Node

var character_class_direct: CharacterClass
var character_class: CharacterBody3D = character_class as CharacterClass

func _test_method():
	character_class_direct.running = true

	#trying to type 'character_class.running' results in crash (even just a . will crash)
	character_class

Minimal reproduction project
Test.zip

I think I've identified this and should have some fixes inbound. The crash at least seems to be caused by cyclic detection as a result of character_class assigning itself to itself

So, there's kind of two aspects to this issue, first is the crash caused by assigning variables to itself, and the second seems to be an inability for autocomplete to detect the members of any script class which is not explicitly type declared.

Hmm, actually disregard the part about the autocomplete, that's something I subsequently broke in my first attempt to fix the cyclic error

I think I managed to salvage a pretty okay fix that addresses everything in this issue, but I'll withhold my feelings on it until others review it

So out of pure curiosity, is what I was trying to accomplish a safe idea? It sounds like it's not usually something you should be doing, but I also wonder how else I'd be able to refer to a specific subclass this way

Well, from what I can elsewhere, it seems be an acceptable way of handling it. Type inferrance using the 'as' keyword seems fine, and a lot of languages seem to be fine with self-assigning variables. I drew the line at recursive abitrarily ordered assignment of class members, but that really didn't have much to do with your original issue, it's just something I discovered while debugging it which can cause the same crash. Whatever the case, the PR seems to handle this code fine, but we'll see what other people have to say about it.


Pre-loading of demonstration objects

The API-X demo will need to have demonstration objects pre-loaded, just needs to be implemented.  Whether this happens at image build time or at container runtime is TBD.

Pre-loaded objects are not necessary for the demo


added missing image reference

Pushed a few seconds after @JonPrag and created a conflict. Resolved now.

Oops, sorry!
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: simonastoyanova @.>
Sent: Monday, September 9, 2024 5:25:15 PM
To: ISicily/ISicily @.>
Cc: Jonathan Prag @.>; Mention @.>
Subject: Re: [ISicily/ISicily] added missing image reference (PR #290)
Pushed a few seconds after @JonPraghttps://github.com/JonPrag and created a conflict. Resolved now.
—
Reply to this email directly, view it on GitHubhttps://github.com/ISicily/ISicily/pull/290#issuecomment-2338555544, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF7LEQVCZMAU3VUSKGHIUDLZVXDWXAVCNFSM6AAAAABN445FMOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZYGU2TKNJUGQ.
You are receiving this because you were mentioned.Message ID: @.***>


chore: bootstrap releases for path: .

Configuring release-please for path: .

Coverage remained the same at 94.792% when pulling e3061228b112f2d6bab16e7771e882b3983582c6 on release-please/bootstrap/default into 34b159e689de5ba1347af5dd9eabb8e0098d1148 on main.


MSSQL integration tests getting stuck

if the MSSQL database driver is missing, the following command (which runs MSSQL integration tests) takes too long to exit:
pytest "src/tests/integration/test_generic_db_operations.py" -k "ip_with_MSSQL"

If the db driver is missing, it should be pretty quick to check that, so something is going on with the integration tests that's causing them to get stuck for some time before showing up the error message.
we need to modify it so the command breaks and displays an appropriate error quickly, so users know they must install the drivers. Note that there might be other errors as well, so we need to ensure that all the pre-requisites to run MSSQL integration tests are documented here.
Note: the comments on this issue are locked. If you need to ask questions, please open a draft PR, tag me, and ask your questions.

closing in favor of https://github.com/ploomber/jupysql/issues/730


Support JetStream wildcard subscription

Describe the feature
NATS JetStream allows wildcard subscription e.g. topic.* matching topic.a or topic.> matching topic.a.b.c.d . Therefore streams can have multiple different topics when wildcards are used.
Subscribing with wildcards is working, but received messages cant be routed back to application, because the subscribed topic is different to received topic of the message.
Therefore I propose following changes:

Message received on a wildcard subscription should be send back with the wildcard topic, than the message can be route as usually
The actual topic is moved into message metadata and can be used by the application

This should not break existing application, because non wildcard subscription behavior doesn't change!
Release Note
RELEASE NOTE: ADD NATS JetStream PubSub supports now wildcard subscriptions

/assign


Bug: "Pinned" section doesn't load

Description
Every other time I open Files the "pinned" section with my desktop, downloads... doesn't load. The area appears as if there are no pinned directories.
Steps To Reproduce
open files.
Requirements
Have pinned load reliably.
Files Version
<IP_ADDRESS>
Windows Version
24H2
User ID
No response
Log File
debug.log

Thanks for the feedback, We're tracking this in #15410. You can check there or click subscribe in the right sidebar for updates.


CB-18330 Stop Error for Real Time Data Mart and Data Mart Clusters in AWS and Template changes for Data Mart Clusters in Azure

We can stop the Real time data mart and data mart clusters in AWS, the error is resolved.

Template changes for Data Mart cluster in Azure where the node count for attached volumes is changed to 0 and now in this case ephemeral volume from the instance would be used which is of comparable size of attached volumes.

Where is the unit test which covers the coordinator scenario?


Add fuzzer pass-through config to tester.toml

This issue tracks exposing the configurable bits of act.conf (and any other fuzzer we may support in the future) through the tester.toml file.  This would need:

an idea of fuzzer configurability (probably letting us disable fuzzing entirely);
some additional code in the act.conf maker;
some thought perhaps as to how to make this generic (but not too much thought, YAGNI etc).

This might look as follows (assuming a fix to #28):
[fuzz]
  style = "act"
  run.cmd = "act-fuzz"
  run.args = [ "-v" ] # ideally there'd be a 'run.verbose' flag or something, but this is an example
  [fuzz.config]
    params."action.cap.lower" = 20
    params."action.cap.upper" = 200
    flags."action.pick-extra" = "100:1"

The fuzz.config part of this is done now, but the specification of how to run the fuzzer hasn't (mostly because it's tied into the way the fuzzer runs act binaries).


v3.6.0

Description

Set content type to application/json; charset=utf-8 instead of application/json when using response.JSON().
Added default behavior for HEAD method: all GET routes can now match the HTTP HEAD method. This fixes 405 Method Not Allowed when requesting an URL with the HEAD method when no route explicitly matches the HEAD method.
Added request.ToStruct(), which puts the request's data into the given structure.

Possible drawbacks
None.
Related issue(s)
None.

Pull Request Test Coverage Report for Build 435893533

7 of 7 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 98.834%

Totals

Change from base Build 417390804:
0.002%

Covered Lines:
3816

Relevant Lines:
3861

💛  - Coveralls


Remove call to Form.Element.EventObserver from miqObserveCheckboxes

https://github.com/ManageIQ/manageiq/issues/1192
@dclarizio, @h-kataria please review.

@bmclaughlin tried checkboxes on few screens, seems to be working.

closing this PR, opened up a new PR https://github.com/ManageIQ/manageiq/pull/1338.


Deployment Commands

Enables deployment to preprod or prod environments by creating an issue comment in the following format:
Deployment to preprod
/deploy preprod [package version id]
E.g. /deploy preprod 04t2o000001MxTgAAK
Deployment to production
/deploy prod [package version id]
E.g. /deploy prod 04t2o000001MxTgAAK
The issue will be updated with the "eyes" emoji when the command is handled, the "rocket" emoji when the command is executed, and the "hooray" emoji if the package is successfully installed. If the package is not successfully installed, the Actions log can be checked for the specific error message.
Comment on this issue in order to initiate deployments.

/deploy prod 04t2o000001MxYgAAK

/deploy prod 04t2o000001MxZAAA0

/deploy prod 04t2o000001MxaIAAS

/deploy prod 04t2o000001MxcJAAS

/deploy preprod 04t2o000001MxdRAAS

/deploy preprod 04t2o000001MxdbAAC

/deploy prod 04t2o000001MxdbAAC

/deploy prod 04t2o000001MxeyAAC

/deploy preprod 04t2o000001MxfIAAS

/deploy preprod 04t2o000001MxfIAAS

/deploy prod 04t2o000001MxgQAAS

/deploy prod 04t2o000001MxgpAAC

/deploy prod 04t2o000001MxiCAAS

/deploy prod 04t2o000001MxiCAAS

/deploy prod 04t2o000001MxiRAAS

/deploy prod 04t2o000001MxiRAAS

/deploy prod 04t2o000001Mxj0AAC

/deploy prod 04t2o000001MxjUAAS

/deploy prod 04t2o000001MxkhAAC

/deploy prod 04t2o000001MxmJAAS


Migration validation fix

The $previous variable isn't set, so the foreach loop fails on first time when checking its value (few lines below).
This is the error im getting before this fix:
~ php index.php migrations latest
Migrating to latest version...
Undefined variable: previous
[codeigniter4 path]\system\Database\MigrationRunner.php - 206
Done

Good catch. Thanks!
In the future, we're asking that pull requests are digitally signed.


MINOR: Update scoverage so bootstrapping build works with Gradle 5.2

Trying to bootstrap the build with the gradle command with Gradle 5.2 fails to apply the scoverage plugin. Upgrading to the most recent 2.x version of the plugin fixes this build issue.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

This PR is against 1.0, but I'd like to cherry-pick to all newer branches. 2.1+ already have gradle-scoverage 2.5.0, but 1.0, 1.1, and 2.0 could use the update. Currently those branches don't build with, e.g., the version of gradle currently shipping in homebrew.


Add returning option to Repo.update

Would be good to be able to specify either the whole row, or selected columns for efficiency, where you're expecting certain columns to be set or modified by the database (through triggers or what-have-you).
Proposal:
# Same return as an update/get pair at present:
Repo.update(post, returning: :row)

# Returns post with selected fields updated with RETURNING
Repo.update(post, returning: [:last_modified])

Currently update/2 returns :ok | no_return, so that would change to Ecto.Model.t | :ok | no_return, dependent upon the presence of the returning option.
Thoughts? Happy to look at implementing, will need to familiarise myself with the code.

Not sure if the changes from this issue ended up being added to Repo.update but I am making a call as such below and the returning keyword list is not being applied.
result =
        %Schema.User{id: id_int}
        |> cast(conn.params, valid_keys)
        |> Schema.User.validate_changeset()
        |> Repo.update(returning: [:username, :gender, :photo1_url])

I can see in the log output that the Ecto is applying the 'returning' based on the fields in the schema that have read_after_writes: true. I would expect that if you specify a returning clause here that it that the setting here would take precedence.

update does not support the :returning option (it is not documented as such either). I will push this feature to master in a sec.

Hi @josevalim Thanks for the quick reply. I agree that it is not documented as having that feature but given the other update functions had it left me wondering why updating through a changeset didn't have it. Thanks!


link to the Docs

the link to the docs should not appear in the menu for user,only for admin

You can remove the link from the template.


I cannot engage with the Hyprland community on Discord

Hyprland Version
None yet (?)
Bug or Regression?
Bug
Description
I'd like to engage with the Hyprland community to ask questions and exchange ideas etc, but that seems to be impossible because the Hyprland website only links to Discord; a restrictive, malicious and privacy invasive web service and proprietary product.
While I have tolerated it in exceptional situations before, it seems that now, for joining the Hyprland "server", I have to enter my phone number and verify it. Unfortunately, this is where my tolerance of the platform ends.
I could go on with the irony of promoting a proprietary application that doesn't even properly work on your very own WM, or how inappropriate it is to lure unsuspecting users and to make them give up their freedom, but I'm certain those who read this far got the point.
(If not, I highly recommend this Article on gnu.org that explains the major issues with proprietary software.)

Suggestion
Because a community cannot be moved from one platform to another very easily, suggest enabling more than one platform, and connect them.
Specifically speaking, this could be a Matrix space where channels are bridged with Discord channels. A similar solution could be possible with IRC as well.
That way, malicious software would not be imposed on those interested in interacting with the community, and it would allow those who are trapped in Discord to break free from the restrictive platform.
Moreover, a bridge solution would not force any of the oppressed users to move away from the restrictive platform.
Thank you very much for your consideration.
How to reproduce
The issue is Discord:
A reproduction of Discord as it is, is not required - and definitely not wanted.
Crash reports, logs, images, videos

Can't say much for everything else, but for the phone number issue it's stated the following in the rules channel:

I highly recommend this Article on gnu.org that explains the major issues with proprietary software

and I recommend using the search bar before opening an issue, as well as reading (ref: above by tchofy)
https://github.com/hyprwm/Hyprland/discussions/1307

What password I need to enter?


Install dateparser dep from pypi

in #192 dateparser requirement was introduced but it is using a development version, is it possible to use latest version from pypi or should we wait on scrapinghub/dateparser#70?

Existing pypi version is very outdated and buggy.


Support blockId in account viewFunction and fix viewState

Bump to version 0.45.0
Support passing blockQuery to viewFunction in account, so you can make view calls at different height. This is pre-work for near-cli to support passing --blockId=123 for near view. Added test
Fix viewState method to parse blockQuery correctly when blockId is given.

@MaximusHaximus hey, let's merge it and release a new version.


Jax 0.4.34 raises an error for None as tree-prefix

Hello:)
The most recent release of jax 0.4.34 does no longer support anything of the kind jax.tree.map(f, None, non-None).
This also affects optax, for example, in add_decayed_weights.
The error can be simply prevented by changing the line to
    updates = jtu.tree_map(
        lambda g, p: None if g is None else g + weight_decay * p, updates, params, is_leaf=lambda x: x is None)

I'm happy to take a look and see if there are any other parts affected and provide a pull request if desired.
Best,
Alex

Hello @a1302z,
Thank you very much for pointing this.
We used an automatic tool to catch this error and make a pass on our codebase to fix this (see e.g. here) but it seems that it did not catch all occurrences. If you are willing to do a PR for that, that would be great! And if not, I'd be curious to know how you flagged them.


api: support SharedMemory API for Android 8.1 (27+)

Signed-off-by: Fung<EMAIL_ADDRESS>

Since it's not necessary to use SharedMemory instead of FD, I closed temporarily.


Error when trying to register a container request filter through a lambda

Hello,
I'm currently working on an authentication filter with aws-serverless-java-container. I've created the filter and registered it in my Handler but when I try and run it in a lambda I get the following error message:
{
"errorMessage": "A MultiException has 3 exceptions.  They are:\n1. java.lang.IllegalStateException: A descriptor SystemDescriptor(\n\timplementation=org.glassfish.jersey.server.internal.process.ServerProcessingBinder$UriRoutingContextFactory\n\tcontracts={org.glassfish.jersey.server.ExtendedUriInfo,javax.ws.rs.container.ResourceInfo,javax.ws.rs.core.UriInfo}\n\tscope=org.glassfish.jersey.process.internal.RequestScoped\n\tqualifiers={}\n\tdescriptorType=PROVIDE_METHOD\n\tdescriptorVisibility=NORMAL\n\tmetadata=\n\trank=0\n\tloader=org.glassfish.hk2.utilities.binding.AbstractBinder$2@649d209a\n\tproxiable=true\n\tproxyForSameScope=false\n\tanalysisName=null\n\tid=16\n\tlocatorId=0\n\tidentityHashCode=1792845110\n\treified=true) requires a proxy, but the proxyable library is not on the classpath\n2. java.lang.IllegalArgumentException: While attempting to resolve the dependencies of test.AuthenticationFilter errors were found\n3. java.lang.IllegalStateException: Unable to perform operation: resolve on test.AuthenticationFilter\n",
"errorType": "org.glassfish.hk2.api.MultiException"
}
I've previously registered a ContainerResponseFilter and that works perfectly fine. Is there any reason why my request filter isn't working the same way? I'm able to run the code locally and only get the error message when running it in a lambda.
This is my handler, the filter I’m trying to register is called "AuthenticationFilter"
public class ApplicationHandler implements RequestHandler<AwsProxyRequest, AwsProxyResponse>  {
private ResourceConfig jerseyApplication = new ResourceConfig()
.packages("test")
.register(AuthenticationFilter.class)
.register(JacksonFeature.class)
.register(CORSResponseFilter.class);
private JerseyLambdaContainerHandler<AwsProxyRequest, AwsProxyResponse> handler
= JerseyLambdaContainerHandler.getAwsProxyHandler(jerseyApplication);
@Override
public AwsProxyResponse handleRequest(AwsProxyRequest awsProxyRequest, Context context) {
    return handler.proxy(awsProxyRequest, context);
}

}
Any help will be appreciated.

The issue in this case is that the shaded jar did not include the javassist package that Jersey uses to do bytecode manipulation. When shading a jar, make sure to include org.javassist:*


consensus: prevent AddBlock lock caused by consensus service

Port of https://github.com/roman-khimov/neo-go/pull/1.
Close https://github.com/nspcc-dev/neo-go/issues/2923.
This PR depends on https://github.com/roman-khimov/dbft/pull/4 and is not valid without it. Waiting for the https://github.com/roman-khimov/dbft/pull/4 to settle, I'll update the dependencies after that.

dBFT changes are fetched from nspcc-dev/dbft, ready for review.

OK otherwise.