Fix CLI for latest Werkzeug (2.0.0rc1)

Wrap the isinstance check in a try/except to ignore failing objects (LocalProxy outside app context).
"LocalProxy matches the current Python data model special methods, including all r-ops, in-place ops, and async. class is proxied, so the proxy will look like the object in more cases, including isinstance. Use issubclass(type(obj), LocalProxy) to check if an object is actually a proxy. :issue:1754"

fixes #3909

Checklist:

[X] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[ ] Add or update relevant docs, in the docs folder and in code.
[ ] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[ ] Add .. versionchanged:: entries in any relevant code docs.
[X] Run pre-commit hooks and fix any issues.
[X] Run pytest and tox, no tests failed.

I think we came up with a better solution to this by telling LocalProxy about its type even when unbound. I'll leave this open until I make sure that works.

Fixed in https://github.com/pallets/werkzeug/pull/2042, specifically 352cd7167984f805daf226966f38f8dfeb991313.


Setting the StatusCode on a TokenResponse does not set IsError

If one does, say,
var tokenResponse = new TokenResponse(HttpStatusCode.Unauthorized, "", "");
then tokenResponse.IsError is FALSE, which I think is unexpected.
var tokenResponse = new TokenResponse(HttpStatusCode.Unauthorized, "reason must also be set for IsError to be true", "");
It's not a big deal, but it was confusing for writing unit tests, in which I didn't care what the reason being returned was (and have no way of confirming that the reason is always set by the IdentityServer being mocked).

OK - I see. I have no plans to change that anymore in 3.x - we will soon switch to 4.x
If you have time, have a look if the 4.x model would work for your tests.

I won't change any behavior anymore for v3. Will switch to v4 very soon.


Fix symbolication in Yosemite and Xcode6

While this fixes the dead lock bug Yosemite, I still think this feature could be improved:

During a focused run, for each additional node before the focused node, there's a call to atos to symbolicate the callers address. It should instead be performed once with all the potential caller addresses of it/before/context nodes in the focused file. This would be a huge improvement (#596).

"atos in 10.0 with a process id no longer works for the simulator so use the
executable path instead.
This patch introduces KWBackgroundTask to remove use of the dangerous
-[NSTask waitUntilExit]; and encapsulate use of NSTask.

breakout callsite creation for symbolicated addresses from KWExample by adding
a factory for KWCallSite."

:+1: I would love to see this fix merged in, since it is a pitty not being able to run the tests from Xcode :pray:

Thanks @jerrymarino, I've tested this out on my machine and it appears to work just fine! :+1:
I'm a little bit nervous about this code:

Will this still work fine on other versions of OS X?
Anything involving dyld.h is currently, unfortunately, beyond my understanding.

So to the end of making this as clear as possible for future maintainers, what do you think about these suggestions?

Move KWBackgroundTask into its own source file. This should hopefully ensure that all of the async code involving CFRunLoop is fully encapsulated.
Document the actual symbolication stuff in some way. I'm thinking developer-focused documentation here, whether it's comments in the implementation providing a bit more context, or maybe a separate file, or something. I think even just having some more method/function doc comments could be really valuable here, and even potentially splitting out some smaller methods/functions if it makes sense.
Or any other ideas you have! :grin:

Ultimately I'm happy to merge this pretty soon, but if there's anything we can do to improve the overall clarity of this beforehand I think that would be a win for everybody!

Thanks again @jerrymarino! I decided to go ahead and extract KWBackgroundTask into its own file. Great to have the tests running again! :clap:

Hey @sharplet glad I could help out on this one.
I tested it out on both 10.9 and 10.10 and it worked great. I believe atos supported the updated arguments in 10.8 too. I hope atos functionality is stabilized in future releases. If it stops working again the error handling we've got now should make it obvious :beers:
It'd be helpful to have developer documentation for this. Inline comments could be sufficient and potentially add a page/section in the wiki for higher level design documentation. Do you think it is appropriate to put design docs in the wiki some where?
I'll can submit a follow up soon :smile:

@jerrymarino I think I'd prefer to keep to user-facing docs in the wiki. So something in the source code should be most appropriate for developer documentation! I'd be happy for you to create a new Documentation/ directory for the higher-level stuff.

@jerrymarino I've created https://github.com/kiwi-bdd/Kiwi/issues/612 to track the developer documentation.

I just ran into this issue on Kiwi 2.3.1 (since it's the latest Kiwi version on cocoapods). Will there be an official Kiwi release with this fix anytime soon so we don't have to use master?

@lyahdav Yep, just pushed 2.4.0! https://github.com/kiwi-bdd/Kiwi/releases/tag/v2.4.0

@sharplet :clap::tada:🎊:fireworks:✨💥🎋🎏

I'm still getting this deadlock. OS X 10.10.5, Xcode 7.0, iOS 9.0, Kiwi 2.4.0.


[nri-bundle] kube-state-metrics.collectors removed by accident/on purpose?

NRI-bundle helm chart has a section deleted by accident or was this on purpose?
Delete section during upgrade:
https://github.com/newrelic/helm-charts/commit/943d2f395d4206f9306eb8b0428c38392d2a6447#diff-775956f6065be7e353042b6388db8acb370f9684ce0ee04b2fdf089d435e85a9
Prometheus still supports these fields:
https://github.com/prometheus-community/helm-charts/blob/c3b697d31b8631fb8847f7f313155deeb5962f98/charts/kube-state-metrics/values.yaml#L250
Error when using collectors:

All links referencing kube-state-metrics are also dead in the repo
i.e. https://github.com/kubernetes/kube-state-metrics/tree/master/charts/kube-state-metrics

I think I found the solution for collectors field syntax has been changed to the following:
commuent out to disable
collectors:
    # - certificatesigningrequests
    # - configmaps
    - cronjobs
    # - daemonsets
    - deployments
    # - endpoints
    - horizontalpodautoscalers
    # - ingresses
    - jobs
    # - leases
    # - limitranges
    # - mutatingwebhookconfigurations
    # - namespaces
    # - networkpolicies
    - nodes
    # - persistentvolumeclaims
    # - persistentvolumes
    # - poddisruptionbudgets
    - pods
    # - replicasets
    # - replicationcontrollers
    # - resourcequotas
    - secrets
    # - services
    # - statefulsets
    # - storageclasses
    # - validatingwebhookconfigurations
    # - volumeattachments

yes exactly the nri-bundle is now shipped with the KSMv2 that changed the syntax for some of the values


fixed multiple typos in multiple files

I noticed multiple typo issues in multiple files. Created this MR to address those typo fix.

Thanks for the updates!

I noticed multiple typo issues in multiple files. Created this MR to address those typo fix.

Yes I now.


Allow specifying serve address via config.yml

Is your feature request related to a problem? Please describe.

Currently, when I want to customize how aichat is served, I can specify an argument for --serve option:
aichat --serve <IP_ADDRESS>:34051

I would like to have an option to pre-configure the CLI via either env var or config.yml file so that I can run the serve without specifying address explicitly.
Describe the solution you'd like

# Via env
AICHAT_ADDRESS=<IP_ADDRESS>:34051 aichat --serve

# Via config.toml
address: <IP_ADDRESS>:34051

It'd be even better to be able to specify host/port pair separately, but I understand that it's a bit more to ask

We will implement this feature, but we will use server_address instead of address.
# Via Env
AICHAT_SERVER_ADDRESS=<IP_ADDRESS>:3000

# Via config.yaml
server_address: <IP_ADDRESS>:3000


Made some improvements

I've made some improvements to get the library wqorking on my systems.

Thanks


add 'continue-on-error: true' where needed

Actions were failing when there was no comment.

merged without review, as stuff was failing


Empty records not empty in search results

In the results there are often empty records with " " written like:
"wikipedia": "\" \"",

Why is there this quoted space? Is it already in the input TSV?
If it is in the input TSV before indexing - we must report it to osmnames project...

This is in the source TSV, there is more columns like this.

Fixed in https://github.com/geometalab/OSMNames/issues/39


[master] Some unit test fixes

This was very painful.
Down to 24 skipped now and the rest passing (on my machine anyway, hopefully the CI agrees). Most of the skipped now were skipped in our internal sources as well (some are just broken scenarios that I have no idea if they should be fixed).
Also closes #622.

@dotnet-bot test this please

LGTM (with minor edits).

@dotnet-bot test Windows_NT Build please

I like how there have been 3 failures in a row... 1 build break and 2 different unit tests that I didn't even enable in this PR (they were already enabled). :cry:

@dotnet-bot test Windows_NT Build please

@dotnet-bot test Windows_NT Build please

@dotnet-bot test Windows_NT Build please


fix: Add tags to aws_appautoscaling_target resourses missed in previous PR

what
Add missed tags
why
Tags were introduced for all aws_appautoscaling_target resources created after 2023-03-20.
That blocks us from enforcing policy checks for missing tags.
references
Follow-up to https://github.com/cloudposse/terraform-aws-dynamodb-autoscaler/pull/65
https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/appautoscaling_target
Also, after release a new version here, we need release in https://github.com/cloudposse/terraform-aws-dynamodb/blob/a0cfac2e8c2584125596089a3185fc4a057410e4/main.tf#L153-L155

/terratest


Add React Router

Add React Router

we need to some refactoring for VideoChat.js

Have a toggle between youtube room and normal room while video chatting. Delete workflow for creating youtube room.

I think we can introduce React Context as well, instead of passing in RoomName, RoomState etc through props from Home to Room, we can have those values in a Context. Another reason is that  with React Router, we probably have to use query strings to store those values, it's not very scalable with the amount of data we want to pass into each component
The two components can be more independent of each other, and the Router only has to worry about which component to render and not about the props.


Update iotex API [iotex-balance]

Fixes # .
Changes proposed in this pull request:

Update iotex API endpoints and func name.

@ChaituVR

@maswalker Can you confirm these changes?

Hello, @ChaituVR !
This is Guo from IoTeX. We have run the test locally and it worked. We suspect that the endpoint might be slow during the last run. Could you please re-run the test here?
Thank you!

@millken Tests are failing, can you have a look on the error and try to fix it? Check here https://github.com/snapshot-labs/snapshot-strategies/actions/runs/3047847939/jobs/4913284908

@bonustrack @ChaituVR  could you please rerun the test? We have improved the endpoint. It is passing when we test locally.

The tests are passed now. @ChaituVR, could you please review and merge the PR? Thank you!

Hey @guo iotex-balance strategy works fine now. but not iotex-staked-balance
Test strategy "iotex-staked-balance" with example index 0 (latest snapshot)
    ✓ Strategy should run without any errors (946 ms)
    ✕ Should return an array of objects with addresses (3 ms)

I understand it is urgent. but it should work with the latest snapshot too. maybe the addresses passed into examples.json doesn't have VP for the latest block? can fix it quickly?

@ChaituVR It has already been fixed, all test should be passed.

@ChaituVR , thanks for the quick review!


Add aspect: design label

Adds a new aspect: design label for tracking design work, as discussed in #214.

It looks perfect. Thanks


added changelog for 1.3

Proposal for changelog 1.3 for current implementation state
-> Changelog for GPU support is done here https://github.com/mesosphere/marathon/pull/4180

retest this please

retest this please

retest this please


[Bug]: Contact doesn't update

Describe the bug
My contact changed his name and profile picture. Nothing change.
I rescanned his QR, the photo changed but not the name.
I deleted the contact. Rescaned his QR and names and Profile picture were good.
Reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots / Recording
No response
Operating system
GrapheneOS
Bitkit version
v132
Log output
No response

Name should not change.
Image should.
I can't reproduce this issue. Trying iOS/Android and dev/production builds

fixed, there was an issue with ST storage


CPU percentage usage per process

Problem
I'd like Scaphandre to be able to export a metric for the CPU percentage usage for every process, and not just the Scaphandre process.
In particular, I'm trying to get the CPU percentage usage for each virtual machine running on a server.
Solution
I can see that there is a get_process_cpu_consumption_percentage function to calculate this in the code, which I believe is used to provide the scaph_self_cpu_usage_percent metric (the CPU usage of the Scaphandre process itself).
Could this same function be used to calculate metrics for other processes and exposed in the exporters?
Alternatives
I'm aware that this could also possibly be calculated using the stats from node_exporter or with the libvirtd exporter for virtual machines in particular, but it would be good to use Scaphandre so that the CPU usage calculation was the same as that used for the scaph_process_power_consumption_microwatts.
Alternatively, it's probably possible to reverse engineer using scaph_process_power_consumption_microwatts and scaph_host_power_microwatts, although a little inconvenient!

This has been implemented in https://github.com/hubblo-org/scaphandre/pull/274 as scaph_process_cpu_usage_percentage metric.
This is currently in dev. Documentation is not ready yet but will be for 1.0 release (next one).
I'd love to have your feedbacks on this once it's in dev :)


fix repo command for remove-rnaquantificationset

was using the wrong names for a field and a function

committed in #1426


PR: Hide popup scrollbar when its not needed

Change:
CSS overflow-y for popup components has been updated from 'scroll' to 'auto'. Now, the scrollbar will only appear when the content overflows the popup container.
Reasons:
The scrollbar now only appears when needed, making the interface cleaner.
Confirmed working correctly in the following browsers:
Firefox 114.0.2
Edge 114.0.1823.67

Hi dear @LorandMarton ☀
Thank you very much for the PR, merging it 🚀

The new version is 1.3.2 🚀


setStatusBarHeight should trigger re-render

Hi,
On iOS the statusbar height might change (when calling someone, or when sharing network connection)
The following package permit to listen for such changes: https://github.com/expo/status-bar-height
See also https://blog.expo.io/the-status-bar-manager-in-react-native-6226058ecba
If the height change during the lifetime of the app, we should rather make the safeArea views re-render so that the view does not appear under the statusbar extra height.
The idea is that I could write the following integration code to track correctly the statusbar:
export const trackStatusBarHeightChanges = async () => {
  const initialHeight = await StatusBarHeight.getAsync();
  SafeAreaView.setStatusBarHeight(initialHeight);
  const listener = height => SafeAreaView.setStatusBarHeight(height);
  StatusBarHeight.addEventListener(listener);
  return () => StatusBarHeight.removeEventListener(listener);
};

Ideally, I think adding expo/status-bar-height to this lib and doing the tracking by default could be helpful. Didn't want to add a new dependency to this lib, so please tell me if this makes sense. cc @EvanBacon

I'm waiting for https://github.com/facebook/react-native/pull/20999 to solve this properly

this isn't needed anymore on 1.0 of this library 👼 see readme for more info :) thanks though!


Lower the minimum supported k8s version

1. Issue, if available:
Lower the minimum supported k8s version
2. Description of changes:
Add an environment variable to webhook container to define the minimum supported k8s version.
3. How was this change tested?
4. Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: link to issue
[X] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This patch is tested with a EKS cluster with k8s v1.19.


Support kubernetes executor

Is your feature request related to a problem? Please describe.
As mentioned in this issue and this issue, it would be nice to have gitlab's kubernetes executor supported along with its docker executor since both do benefit from not having to push to test job changes
Describe the solution you'd like
I'm not too familiar with the internals of gitlab-ci-local so I am assuming it is non-trivial to support another executor, but the end result would probably be a CLI option or parameter which users can utilize to switch between executors (since I don't think there's a .gitlab-ci.yml keyword that allows specifying whether to use the kubernetes or docker executor unlike with the shell and image keywords unless I'm mistaken?)
Describe alternatives you've considered
Currently, the only alternatives I've considered are changing the original job so that it works with the docker executor (not ideal since it may mask some other bug) or simply pushing the changes to test (which usually takes more time than testing with gitlab-ci-local)
Additional context
Even if this takes a long time to implement, I think it's worth keeping the issue open since others may bump into similar issues and not realize it is a problem with the kubernetes executor not being supported

you do know that the kubernetes executor require the whole KUBE_CONFIG to be working, right ?
The way it works with gitlab is that the "tag" on the job decide what runners can be used.  The "image" section give the "runtime" to use.   The rest should be pretty much transparent for executors...  So at this point, I don't believe the kubernetes-executor should be different from the docker-executor.
This is just an opinion as I don't believe we should leak implementation details of gitlab on gitlab-ci-local...


This is a very small fix

This is a small fix in SQL Syntax which is corrected... kindly validate and proceed with merging the changes please..!

@rocky9866187496 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @rocky9866187496 ,
Thanks for creating this pull request, and I apologize for the delay in getting back to you.
I can reproduce the issue with the current syntax. Your suggestion does remove the syntax error, however I think this is because of the nature of Dynamic SQL. I am not seeing an opening ( for the final ) in the dynamic SQL, so I believe that the appropriate fix would be to delete the closing parentheses.
I am going to close out this PR, and I will email you (I see you are a fellow Microsoft employee) about following up.
Thanks again for creating this PR and lettings us know about this problem!
Kendra from the Database Docs Team
#please-close


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


point cloud segment autoencoding

Hi @dahliau , nice work!
I saw in the supplementary material when you the autoencoding task (comparing with Chamfer), the results are quite sparse. My question is that if both of my input and output point clouds are uniformly sampled with Farthest point sampling, would DPDist do a better job?
Thank you and look forward to your code!

Hi @GeeeG , thanks!
One of the benefits of using DPDist is its ability to be sample invariance. For auto-encoder outputs, this property is less wanted. The output point could might be too sparse because DPDist can estimate the same surface from fewer points (so some output points may result in a dense crowding, as shown in the supplementary).
Take into consideration that DPDist is built on implicit function. Therefore, DPDist itself is a type of auto-encoder (some consider the implicit neural network as a decoder, as its output is an implicit function of the wanted surface).


Fixing Docs

Fixed code rendering in the Using shared parameters tutorial

Nothing else to add right? Will merge this then

Nothing else to add right? Will merge this then

Nope, I guess this is all


On Pie Donut tooltips don't always show

Smaller values of tooltips around the sides of a semicircle donut don't seem to pop up any tooltip, this is pretty annoying as it was these particular values I wanted to be shown.

Found an element over the donut, closing this

Could you please mention that element here


Angel One fails with "Invalid State Error" on Edge

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
Build: v2.5.4-master-7-g7c271bab
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from master?
Yes
Are you using the demo app or your own custom app?
Shaka Player Demo
What browser and OS are you using?
Environment: Win10 Edge
What did you do?
Steps:

Navigate to: https://nightly-dot-shaka-player-demo.appspot.com
Launch video asset and select "Angel One"
Wait until the end of the video (which is about a minute long)

Actual Result:
After ~36-39 seconds, the video stops and the console output contains this error:
"Media Source Opperation Threw (Invalid State Error)" this is listed as "Error data[0] exception"
Expected:
This content would play until the end (FYI -- this is unencrypted content)
Note:

Surprisingly, this does not happen with Win10 Internet Explorer
Also, if you download the content in Edge it plays until the end

I don't see the error, but I do see the stream stop at around 39 seconds.  It appears like the segments are being appended, but they don't show up as buffered in the browser.  If I disable buffering handling, the browser still stops at the end of the buffered range (so it's really unbuffered).  The appendWindowEnd is set correctly, so they aren't being dropped.
One thing I notice is this doesn't happen with the MP4 version, only the WebM version.  I thought Edge disables VP9 support unless there is hardware support; so it may be that this policy changed recently and now we are playing VP9 and there is something Edge doesn't like with it.  I'm thinking this is a platform bug.
I've also confirmed this happens when we download too.  The reason it doesn't appear to in the default demo is because by default it (a) picks MP4 when downloading and (b) only downloads 480p, which doesn't have this problem.  It definitely happens with the 576p version.

@TheModMaker, we could try re-encoding and re-packaging to another URL with the latest libvpx and ffmpeg, just to make sure it's not an encoding bug.

I'm going to repackage this content now to test the theory that something was wrong with it.

The repackaged version plays through to the end on Edge, so I'm going to assume that the version we packaged in June 2019 had some minor encoding issue that disagrees with Microsoft's decoders.
There was a small issue with the new version not getting all the correct resolutions, though, so I will fix some small bugs in Shaka Streamer and test again before updating the official copies of "Angel One" in our demo.

I fixed a few Streamer bugs (fixes yet to be reviewed) and replaced "Angel One" with a fresh re-encode using that.  Everything is working correctly on Edge now.


sfaira breaks when Universe is initialised from read-only python installation

I guess we need to control the caching a bit more tightly so that this is written to a location that can always assumed to be writable (like /tmp or so)
OSError                                   Traceback (most recent call last)
~/.ch_tmp/ipykernel_20042/80063267.py in <module>
      1 import sfaira
      2
----> 3 univ = sfaira.data.Universe(
      4             data_path=f"/storage/groups/ml01/datasets/projects/20200101_Various_SfairaDataRepository_leander/raw/",
      5             meta_path=f"/storage/groups/ml01/datasets/projects/20200101_Various_SfairaDataRepository_leander/meta/",

/opt/python/lib/python3.8/site-packages/sfaira/data/dataloaders/super_group.py in __init__(self, data_path, meta_path, cache_path, exclude_databases)
     29         """
     30         dsgs = [
---> 31             DatasetSuperGroupLoaders(
     32                 data_path=data_path,
     33                 meta_path=meta_path,

/opt/python/lib/python3.8/site-packages/sfaira/data/dataloaders/loaders/super_group.py in __init__(self, data_path, meta_path, cache_path)
     37                     if path_dsg is not None:
     38                         try:
---> 39                             dsg = DatasetGroupDirectoryOriented(
     40                                 file_base=path_dsg,
     41                                 data_path=data_path,

/opt/python/lib/python3.8/site-packages/sfaira/data/dataloaders/base/dataset_group.py in __init__(self, file_base, data_path, meta_path, cache_path)
    745                             for x in sample_fns:
    746                                 datasets_f.append(
--> 747                                     DatasetFound(
    748                                         data_path=data_path,
    749                                         meta_path=meta_path,

/opt/python/lib/python3.8/site-packages/sfaira/data/dataloaders/loaders/d10_1016_j_cell_2017_09_004/human_isletoflangerhans_2017_smartseq2_enge_001.py in __init__(self, **kwargs)
     18
     19         self.author = "Enge"
---> 20         self.disease = "healthy"
     21         self.doi_journal = "10.1016/j.cell.2017.09.004"
     22         self.doi_preprint = "10.1101/108043"

/opt/python/lib/python3.8/site-packages/sfaira/data/dataloaders/base/dataset.py in disease(self, x)
   1611     @disease.setter
   1612     def disease(self, x: str):
-> 1613         x = self._value_protection(attr="disease", allowed=self.ontology_container_sfaira.disease,
   1614                                    attempted=x)
   1615         self._disease = x

/opt/python/lib/python3.8/site-packages/sfaira/consts/ontologies.py in disease(self)
    120     def disease(self):
    121         if self._disease is None:
--> 122             self._disease = OntologyMondo(branch=DEFAULT_MONDO)
    123         return self._disease
    124

/opt/python/lib/python3.8/site-packages/sfaira/versions/metadata/base.py in __init__(self, branch, recache, **kwargs)
    955     ):
    956         # Latest release also available from url="http://purl.obolibrary.org/obo/mondo.obo".
--> 957         obofile = cached_load_file(
    958             url=f"https://raw.githubusercontent.com/monarch-initiative/mondo/{branch}/mondo-lastbuild.obo",
    959             ontology_cache_dir="mondo",

/opt/python/lib/python3.8/site-packages/sfaira/versions/metadata/base.py in cached_load_file(url, ontology_cache_dir, ontology_cache_fn, recache)
     38         # Download if necessary:
     39         if not os.path.isfile(obofile) or recache:
---> 40             os.makedirs(name=ontology_cache_dir, exist_ok=True)
     41
     42             def download_file():

/opt/python/lib/python3.8/os.py in makedirs(name, mode, exist_ok)
    211     if head and tail and not path.exists(head):
    212         try:
--> 213             makedirs(head, exist_ok=exist_ok)
    214         except FileExistsError:
    215             # Defeats race condition when another thread created the path

/opt/python/lib/python3.8/os.py in makedirs(name, mode, exist_ok)
    211     if head and tail and not path.exists(head):
    212         try:
--> 213             makedirs(head, exist_ok=exist_ok)
    214         except FileExistsError:
    215             # Defeats race condition when another thread created the path

/opt/python/lib/python3.8/os.py in makedirs(name, mode, exist_ok)
    221             return
    222     try:
--> 223         mkdir(name, mode)
    224     except OSError:
    225         # Cannot rely on checking for EEXIST, since the operating system

OSError: [Errno 30] Read-only file system: '/opt/python/lib/python3.8/site-packages/cache'

RIght now, all paths are maintained here: https://github.com/theislab/sfaira/blob/dev/sfaira/consts/directories.py. Two considerations:

I think it is desirable to keep the caches in the sfaira installation so that people can easily copy them there if they work on an offline machine and check this out if there are issues with downloads. We can discuss whether this outweighs the issue above.
We could optionally allow moving the cache to a different directory, there could be a function that just resets these constants in the sfaira name space for example, that should be relatively straight forward.

hmm, my feeling is that writing to the sfaira installation directory might be bad practise when sfaira is not installed as editable (ie. when just using pip install sraira). @Zethson what's your opinion on this?
Anyway, for container use this definitely leads to problems (it also does when a server provides a preinstalled python installed with sfaira system wide). in these cases the installation directory cannot be assumed as writable.

It might make sense to largely keep https://github.com/theislab/sfaira/blob/dev/sfaira/consts/directories.py as is, but check if variable SFAIRA_CACHE_DIR is defined in the python session namespace when defining CACHE_DIR in this file and only using the package path if SFAIRA_CACHE_DIR is not yet in the namespace.

hmm, my feeling is that writing to the sfaira installation directory might be bad practise when sfaira is not installed as editable (ie. when just using pip install sraira). @Zethson what's your opinion on this?

Totally, It is "unexpected behavior" imo. If you want to caching to only be available on single sessions -> /tmp. If always maybe write it into the home directory or .config or something?

@davidsebfischer are you willing to drop this then?

I think it is desirable to keep the caches in the sfaira installation so that people can easily copy them there if they work on an offline machine and check this out if there are issues with downloads. We can discuss whether this outweighs the issue above.

we could have a ~/.sfaira_caches/ where caches are stored by default that can be copied over of needed?

This may be pretty large, could be area of 1GB I think, do you think that this is fine to drop in home? People may want to delete it at some point because of the size, colocalising it with the code seemed nice for that reason.

This may be pretty large, could be area of 1GB I think, do you think that this is fine to drop in home? People may want to delete it at some point because of the size, colocalising it with the code seemed nice for that reason.

I'd say that's ok.

Make it configurable and home by default?  Monday, 22 November 2021, 06:58PM +01:00 from Leander  @.*** :

This may be pretty large, could be area of 1GB I think, do you think that this is fine to drop in home? People may want to delete it at some point because of the size, colocalising it with the code seemed nice for that reason.
I'd say that's ok.
—
You are receiving this because you were mentioned.
Reply to this email directly,  view it on GitHub , or  unsubscribe .

Ok, fine for me!

So to make this configurable, we will need to drop these paths and optional setting methods in a small class and instantiate it in the python file, similar to how we handle OCS and how scanpy does settings: https://github.com/theislab/scanpy/blob/48cc7b38f1f31a78902a892041902cc810ddfcd3/scanpy/_settings.py#L485.


[export] grad of exported function fails with device assignment error

Description
The jax.experimental.export exports the VJP using a synthetic mesh using the first N devices for the export platform. This seemed reasonable because all that is captured in the exported artifact is the number of devices, not their ids. However, during lowering there may be a conflict between the order of devices used by exporting code and shardings present in the primal functions.
For example, the following code fails:
    def f(x):
      return jnp.sum(x * 2.)

    mesh_rev = Mesh(list(reversed(jax.local_devices())), "i")
    shardings_rev = NamedSharding(mesh_rev, jax.sharding.PartitionSpec(("i",)))
    input_no_shards = jnp.ones(shape=(jax.local_device_count(),))
    input_rev = jax.device_put(input_no_shards, device=shardings_rev)
    exp_rev = export.export(pjit.pjit(f, in_shardings=shardings_rev))(input_no_shards)

    g = jax.grad(export.call(exp_rev))(input_rev)  # Failure here

The failure is:
E       ValueError: Received incompatible devices for pjitted computation. Got ARG_SHARDING with device ids [0, 1] on platform CPU and pjit inside pjit with device ids [1, 0] on platform CPU at /Users/necula/Source/jax/tests/export_test.py:1091:14 (JaxExportTest.test_grad_sharding_different_mesh)

Instead, the exporting of the VJP should use the same device assignment as for exporting the primal function.
System info (python version, jaxlib version, accelerator, etc.)
Not needed

In #21319 we fixed this for jax.experimental.export. When using from jax2tf the bug is not yet fixed.


Automatic dest detection

For my use case, I call rollup from the CLI and pass the dest as an argument. From my understanding, the current best (only?) practice is to set plugin options in a rollup config file. However, I cannot set the dest as a hash plugin option in the config file because it is not known ahead of time (I use the same config for multiple files).
Would it be possible to fallback to the dest in the rollup options if the plugin dest is not provided? For example, bundle.[hash].js could be set as the rollup dest. Alternatively, default to converting bundle.js to bundle.[hash].js.
I'm happy to submit a PR. What do you think?

@zdyn Apologies for the delayed response. This sounds like a very sane idea and I'd gladly accept a PR for it, otherwise I can do the work when I next get some spare time.

Apologies for leaving this open for so long. I actually no longer have need for this so I'll go ahead and close it. Thanks for your work!


Add info panel when writing article

When writing an article there should be an info panel informing the writer what the rules are for submitting an article. Most of these should conform to our general rules: https://laravel.io/rules
Also inform that article approval is at our discretion and that we don't accept all articles. Inform the writer that when an article isn't approved, no notification is being made. Suggest to cross-post articles on the writer's blog, other channels, etc.

I'm on it!

I'm on it!


[BUG]: Agent Skill Datetime shows wrong data

How are you running AnythingLLM?
Docker (local)
What happened?
The plugin to get current date and time in formatted strings doesn't work right. By asking the chatbot a question about the current date, it answers wrong.
Are there known steps to reproduce?
No response

const now = new Date();

      const dateOptions = {
        weekday: 'long',
        year: 'numeric',
        month: 'long',
        day: 'numeric'
      };
      const currentDate = now.toLocaleDateString('en-US', dateOptions);

      const timeOptions = {
        hour: '2-digit',
        minute: '2-digit',
        second: '2-digit',
        hour12: false
      };
      const currentTime = now.toLocaleTimeString('en-US', timeOptions);

      return JSON.stringify({
        date: `Today's date is ${currentDate}`,
        time: `Current Time: ${currentTime}`
      });

Its runs new Date directly, you'll need more evidence to support why or how it is wrong as the comment does not explain how it is wrongl


benchmark/cmd: count key space for events total

Fix https://github.com/coreos/etcd/issues/5099.
The comparison should be "greater than or equal"
in case there's another increment from another
watch goroutine.

This fixes the issue.
./benchmark --endpoints <IP_ADDRESS>:2379 --clients 64 --conns 64 watch --total 50000
 50000 / 50000 Booooooooooooooooooooooooooooooooooooooo! 100.00%0
Watch creation summary:
 Requests/sec:	69524.5903
 50000 / 50000 Booooooooooooooooooooooooooooooooooooooo! 100.00%0
Watch events received summary:
 Requests/sec:	72523.7997

/cc @xiang90 @heyitsanthony
Thanks.

/cc @xiang90 @heyitsanthony PTAL.
Now it counts the total events triggered per put.
Thanks.

@heyitsanthony This returns 500000000 events total for command ./benchmark --endpoints <IP_ADDRESS>:2379 --clients 64 --conns 64 watch --total 50000
+++ b/tools/benchmark/cmd/watch.go
@@ -138,7 +138,14 @@ func watchFunc(cmd *cobra.Command, args []string) {

        // put phase
        // total number of puts * number of watchers on each key
-       eventsTotal = watchPutTotal * (watchTotal / watchedKeyTotal)
+       numWatchers := make(map[string]int)
+       for i := range watched {
+               numWatchers[watched[i]] = watchTotal / watchKeySpaceSize
+       }
+       eventsTotal = 0
+       for i := 0; i < watchPutTotal; i++ {
+               eventsTotal += numWatchers[watched[i%len(watched)]]
+       }
        results = make(chan result)
        bar = pb.New(eventsTotal)

@@ -157,7 +164,7 @@ func watchFunc(cmd *cobra.Command, args []string) {
        pdoneC = printRate(results)

        go func() {
-               for i := 0; i < eventsTotal; i++ {
+               for i := 0; i < watchPutTotal; i++ {
                        putreqc <- v3.OpPut(watched[i%(len(watched))], "data")

Do you see anything that I am doing wrong here?
Thanks for help!

for i := range watched {
    numWatchers[watched[i]] = numWatchers[watched[i]]+1
}

ah right... will fix. Thanks!

lgtm

oh oops the put loop still needs to use the puts total and not the events total

Yeah... I tried watchPutTotal for OpPut but it's hanging...
./benchmark --endpoints <IP_ADDRESS>:2379 --clients 64 --conns 64 watch --watchers=100 --total=100 --watched-key-total=100 --put-total=100 --key-space-size=10
 100 / 100 Boooooooooooooooooooooooooooooooooooooooooooooooooo! 100.00%0
Watch creation summary:
 Requests/sec:	13995.0119
 1122 / 1122 Booooooooooooooooooooooooooooooooooooooooooooooo! 100.00%

Need to investigate more.

@heyitsanthony PTAL. Finally got it to work:
./benchmark --endpoints <IP_ADDRESS>:2379 --clients 64 --conns 64 watch --watchers=100 --total=100 --watched-key-total=100 --put-total=100
 100 / 100 Booooooooooooooooooooooooooooooooooooooooooo! 100.00%0
Watch creation summary:
 Requests/sec:	7208.3689
 10000 / 10000 Boooooooooooooooooooooooooooooooooooooo! 100.00%1s
Watch events received summary:
 Requests/sec:	9923.2165

lgtm

@heyitsanthony Thanks for helping debug this!


update cockroach dump

This PR expands the cockroach dump docs to cover dumping multiple tables as well as choosing whether to dump a table's schema, data, or both.
Fixes #991

This change is

@knz, PTAL.

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/2f3f75e52393e016ec8132a0f52c9ff48db51217/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/51e3c0471010d926ebe4ef28c89484ac4579dbdf/


Build interface for Turing

Simplified the AHMC call in Turing easy; in favour of https://github.com/TuringLang/Turing.jl/pull/793

Basically wrapped functions a bit so that it's easier on the Turing side for the new interface.
And also renamed precondition -> mass_matrix, which is always reported now with/without adaption.

Please do not merge for now. I realised there is a bug I introduced.

@JuliaRegistrator register()

Pull request comments will not trigger Registrator as it is disabled. Please trying using a commit or issue comment.


fix: 🐛 Order by - result of analysis label fixed

https://metakeen.atlassian.net/jira/software/projects/FEE/boards/10?selectedIssue=FEE-789

Codecov Report

Merging #309 (5a068fc) into develop (c5e84d1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #309   +/-   ##
========================================
  Coverage    87.05%   87.05%
========================================
  Files          554      554
  Lines         7016     7016
  Branches       686      686
========================================
  Hits          6108     6108
  Misses         817      817
  Partials        91       91

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 c5e84d1...5a068fc. Read the comment docs.


Firebase/Performance crash in CoreFoundation CFDictionaryGetValue + 100

Step 1: Describe your environment

Xcode version: 12.4
Firebase SDK version: 6.27.0
Installation method: CocoaPods
Firebase Component: Performance

Step 2: Describe the problem
Getting a crash when integrating pod  pod 'Firebase/Performance', '= 6.27.0'
Stack trace screenshot:

Stack trace log:
Firebase-Performance-Crash.txt
Steps to reproduce:
What happened? How can we make the problem occur?
Integrate pod pod 'Firebase/Performance', '= 6.27.0'

Thanks for reporting this, @radhakrishnapai. Your issue looks related on #6922, and the fix was included on 7.1.0. Could try updating your SDK, and see if you'll still encounter the issue?

@rizafran thanks for the update. I will check on your suggestion and get back if still facing the issue.


Installed Node version not found when running from git hook

Operating system and version:

Ubuntu 16.04

nvm debug output in a normal ssh shell:

nvm --version: v0.33.8
$SHELL: /bin/bash
$HOME: /home/worker
$NVM_DIR: '$HOME/.nvm'
$PREFIX: ''
$NPM_CONFIG_PREFIX: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
shell version: 'GNU bash, version 4.3.48(1)-release (x86_64-pc-linux-gnu)'
uname -a: 'Linux 4.4.0-112-generic #135-Ubuntu SMP Fri Jan 19 11:48:36 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux'
OS version: Ubuntu 16.04.3 LTS
curl: /usr/bin/curl, curl 7.47.0 (x86_64-pc-linux-gnu) libcurl/7.47.0 GnuTLS/3.4.10 zlib/1.2.8 libidn/1.32 librtmp/2.3
wget: /usr/bin/wget, GNU Wget 1.17.1 built on linux-gnu.
sed: -e expression #1, char 9: Unmatched ) or \)
git: , git version 2.7.4
grep: /bin/grep (grep --color=auto), grep (GNU grep) 2.25
awk: /usr/bin/awk, GNU Awk 4.1.3, API: 1.1 (GNU MPFR 3.1.4, GNU MP 6.1.0)
sed: /bin/sed, sed (GNU sed) 4.2.2
cut: /usr/bin/cut, cut (GNU coreutils) 8.25
basename: /usr/bin/basename, basename (GNU coreutils) 8.25
rm: /bin/rm, rm (GNU coreutils) 8.25
mkdir: /bin/mkdir, mkdir (GNU coreutils) 8.25
xargs: /usr/bin/xargs, xargs (GNU findutils) 4.7.0-git
nvm current: v8.9.4
which node: $NVM_DIR/versions/node/v8.9.4/bin/node
which iojs:
which npm: $NVM_DIR/versions/node/v8.9.4/bin/npm
npm config get prefix: $NVM_DIR/versions/node/v8.9.4
npm root -g: $NVM_DIR/versions/node/v8.9.4/lib/node_modules

nvm ls output within a normal ssh shell:

->       v8.9.4
default -> lts/* (-> v8.9.4)
node -> stable (-> v8.9.4) (default)
stable -> 8.9 (-> v8.9.4) (default)
iojs -> N/A (default)
lts/* -> lts/carbon (-> v8.9.4)
lts/argon -> v4.8.7 (-> N/A)
lts/boron -> v6.13.0 (-> N/A)
lts/carbon -> v8.9.4

nvm debug output from within the git hook:

remote: nvm --version: v0.33.8
remote: $SHELL: /bin/bash
remote: $HOME: /home/worker
remote: $NVM_DIR: '/usr/bin'
remote: $PREFIX: ''
remote: $NPM_CONFIG_PREFIX: ''
remote: $NVM_NODEJS_ORG_MIRROR: ''
remote: $NVM_IOJS_ORG_MIRROR: ''
remote: shell version: 'GNU bash, version 4.3.48(1)-release (x86_64-pc-linux-gnu)'
remote: uname -a: 'Linux 4.4.0-112-generic #135-Ubuntu SMP Fri Jan 19 11:48:36 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux'
remote: OS version: Ubuntu 16.04.3 LTS
remote: curl: curl is a tracked alias for /usr/bin/curl, curl 7.47.0 (x86_64-pc-linux-gnu) libcurl/7.47.0 GnuTLS/3.4.10 zlib/1.2.8 libidn/1.32 librtmp/2.3
remote: wget: wget is a tracked alias for /usr/bin/wget, GNU Wget 1.17.1 built on linux-gnu.
remote: git: git is a tracked alias for /usr/lib/git-core/git, git version 2.7.4
remote: grep: grep is a tracked alias for /bin/grep, grep (GNU grep) 2.25
remote: awk: awk is a tracked alias for /usr/bin/awk, GNU Awk 4.1.3, API: 1.1 (GNU MPFR 3.1.4, GNU MP 6.1.0)
remote: sed: sed is a tracked alias for /bin/sed, sed (GNU sed) 4.2.2
remote: cut: cut is a tracked alias for /usr/bin/cut, cut (GNU coreutils) 8.25
remote: basename: basename is a tracked alias for /usr/bin/basename, basename (GNU coreutils) 8.25
remote: rm: rm is a tracked alias for /bin/rm, rm (GNU coreutils) 8.25
remote: mkdir: mkdir is a tracked alias for /bin/mkdir, mkdir (GNU coreutils) 8.25
remote: xargs: xargs is a tracked alias for /usr/bin/xargs, xargs (GNU findutils) 4.7.0-git
remote: nvm current: none
remote: which node:
remote: which iojs:
remote: which npm:
remote: npm config get prefix: .git/hooks/post-merge: 2437: .git/hooks/post-merge: npm: not found
remote: npm root -g: .git/hooks/post-merge: 2437: .git/hooks/post-merge: npm: not found

nvm ls output from within the git hook:

remote: mkdir: cannot create directory ‘/usr/bin/alias’: Permission denied
remote: mkdir: cannot create directory ‘/usr/bin/alias’: Permission denied
remote: node -> stable (-> N/A) (default)
remote: iojs -> N/A (default)

How did you install nvm? (e.g. install script in readme, homebrew):
install script

What steps did you perform?
tried to write a git hook using npm

What happened?
can't access installed node version from within hook

What did you expect to happen?
npm install, npm run build to work

Is there anything in any of your profile files (.bashrc, .bash_profile, .zshrc, etc) that modifies the PATH?
no

I'm writing a git hook to install new dependencies when package.json has been updated.
Here's the post-merge hook:
#!/bin/sh
. $HOME/.nvm/nvm.sh

echo "**** [staging] Received update"
PACKAGE_CHANGES=`git diff HEAD@{1} --stat -- package.json | wc -l`
echo "$PACKAGE_CHANGES changes to package.json"

if [ $PACKAGE_CHANGES -gt 0 ] ; then
        echo "**** [staging] package.json has been updated. Reinstalling.."
        npm install
        echo "**** [staging] Dependencies have been updated."
fi

echo "**** [staging] Transpiling.."
npm run build
echo "**** [staging] 🤓Deployed!"

Its output:
remote: **** [staging] Received update
remote: 2 changes to package.json
remote: **** [staging] package.json has been updated.

This works fine when I am in an ssh shell and just do a manual git pull. If I echo $USER in the hook, i get the same user that I use to log in to an interactive shell with.
When I echo $NVM_DIR from a shell, the output is /home/worker/.nvm
When I echo $NVM_DIR from the hook, the output is /usr/bin
Similar to issue #688, except that . $NVM_DIR/nvm.sh didn't work because $NVM_DIR was empty before running . $HOME/.nvm/nvm.sh.

What is echo $HOME in the hook?

$HOME is as expected, /home/worker

Your git hook has a line to source nvm.sh; could you try replacing that line with the two lines in https://github.com/creationix/nvm#install-script?

@ljharb yes, that did it!
I was in the middle of debugging everything, and saw that $1, as read here in nvm.sh:
nvm_sanitize_path() {
  local SANITIZED_PATH
  SANITIZED_PATH="${1-}" # <---
  if [ "_$SANITIZED_PATH" != "_$NVM_DIR" ]; then
    SANITIZED_PATH="$(nvm_echo "$SANITIZED_PATH" | command sed -e "s#$NVM_DIR#\$NVM_DIR#g")"
  fi
  if [ "_$SANITIZED_PATH" != "_$HOME" ]; then
    SANITIZED_PATH="$(nvm_echo "$SANITIZED_PATH" | command sed -e "s#$HOME#\$HOME#g")"
  fi
  nvm_echo "$SANITIZED_PATH"
}

was 0 when being executed in the hook (via . $HOME/.nvm/nvm.sh), but was null (or an empty string) when I was in an interactive shell.
Anyway: yes, replacing . $HOME/.nvm/nvm.sh with the two lines in the install script did the trick. Thank you for your help!


Bug Report: No difficulties found for chart, skipping.

Windows 64 bit
kade engine 1.6.1
When I try using charts for my custom song, it gives me an error stating "No difficulties found for chart, skipping." I:
Checked to make sure the song's folder was in the right place.
Checked to make sure the chart difficulties were marked correctly.
Checked to make sure the song names were spelled correctly.
I currently have no idea how it isn't detecting the charts inside the folder. Any help would be appreciated.

did you compile with the jsons inside of the source code? if not recompile with files inside since if you don't do that the manifest file won't make them 'visible' for the game

nvm I was lied to turns out you actually need to put dashes instead of spaces in the json files even if the name has spaces in it


How can I not draw pictures through write_glyph_imgs.py

Hi, what does "not draw poctures" mean? You can send me emails to describe your problems.

I made my own data set，ground truth have no content.

I made my own data set，ground truth have no content.

I made my own data set，ground truth have no content.

In Line 72 of write_glyph_imgs.py, add codes:
image.save('tmp.png')
input()

Check tmp.png to find whether the problem occurs here.

I made my own data set，ground truth have no content.

I have found where the problem is. I have fixed the code. See for this for details.


Document crate features in library docs

Closes #583
Just copied and pasted the Cargo.toml feature comments into lib.rs based on the first recommendation on the cargo feature docs (which links regex as reference). Also added a note in Cargo.toml so that anyone who updates features in the future knows to also update the docs.
Reasoning:
Makes feature descriptions more accessible (see discussion on issue #583). Currently you can see the list of ash features on docs.rs but need to get Cargo.toml to see what they do. All three features seem to have potential utility for users imo.

Currently you can see the list of ash features on docs.rs but need to get Cargo.toml to see what they do.

I hadn't really looked at all the many docs.rs pages yet, didn't know this view exists. Any idea if there's a way to get useful documentation to show up on that page?

I hadn't really looked at all the many docs.rs pages yet, didn't know this view exists. Any idea if there's a way to get useful documentation to show up on that page?

Unfortunately not. There's currently no standardized feature documentation method in the rust ecosystem (discussion here https://github.com/rust-lang/api-guidelines/discussions/43)


stucked at Starting connection to server, please wait

When connecting from public domain using the dns generated after the cloud formation deployment;
I am stucked at Starting connection to server, please wait
Kindly assist....

@snexola Did you true the framework for Unreal Engine Pixel Streaming?


[WIP] Schedules Messages feature.

This PR will be tracking all of the changes that are upcoming to enable scheduled messages / remind feature in Zulip.
Will Fix: #585

Is there any documentation on this feature?
I was able to sort of figure it out by looking through the code, but as far as I can tell, zulip comes with a reminder bot but no one knows how to use it.
Is there anyway to cancel reminders or view them?

@pdodzweit This is a work in progress feature. Basically we got the basic
backend and slash command merged as well as a per message based reminder
option (which actually is in message actions carret, but we disabled it for
the moment). That's why nothing is documented yet!
We have a PR which when merged will most probably result in v1 release of
the feature.
As for a way to view or cancel reminders, those are yet to have work
started upon but will definitely be a follow up of release of v1 feature.
On Tue, Jun 19, 2018, 8:32 PM pdodzweit<EMAIL_ADDRESS>wrote:

Is there any documentation on this feature?
I was able to sort of figure it out by looking through the code, but as
far as I can tell, zulip comes with a reminder bot but no one knows how to
use it.
Is there anyway to cancel reminders or view them?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/pull/7967#issuecomment-398431135, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQcFpCYhthh14iyDWTUyfkYu8aUT2HlMks5t-RIQgaJpZM4RQsGn
.

@adnrs96 ok, that makes sense.  Thanks for the reply!


[util/design] Encode bit width in OTP vmem filename

See #10867

@msfschaffner ping :)

Sorry about that, will take a look.
Also, would you mind rebasing this onto TOT again in order to get the latest CI fixes?

Thanks! I rebased right before my last message, I think that's up to date with master right now.

Hm looks like there not all clang format issues have been fixed yet. Nevermind (we'll just have to rebase this again once it is fixed).

Jup, will do once FPGA passes.


remove the width inherit from .card-title to stop it from overflowing

Hackathon Git Labs Project PR 🎉🎉
PR contains:
I noticed the perticipant cards title was overflowing it's container. The title had a width of inherit so removing this stopped the title overflowing the card. I've only commented it out for the moment.
Breaking changes
None
Screenshots

Hi @BrianWhelanDublin that is an intended design choice and this PR is not needed to satisfy this change. Thanks anyway 🤜

Ah cool. Apologies I didn’t realise it was part of the design. ThanksBrian Sent from my iPhoneOn 14 Oct 2022, at 10:56, auxfuse @.***> wrote:
Hi @BrianWhelanDublin that is an intended design choice and this PR is not needed to satisfy this change. Thanks anyway 🤜
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>


fix: viewport translation for en / fr

我就是不想取 s 开头的词……你可以统一替换 r

Generated by koishi-plugin-github.

Yeah, there are so many words start with "s".
I would replace them as resolution. Should I convince the changes in this PR, or should I open a new one?
是的，有很多单词以“s”开头。
我会将它们替换为“分辨率”。 我应该说服这个 PR 中的更改，还是应该打开一个新的？

你可以直接在这个 pr 中修改

Generated by koishi-plugin-github.


Update README.md

Added my name please

Summary

Fixes #
Lorem ipsum dolor sit amet, consectetur adipiscing.
List of changes proposed in this PR (pull-request)

Lorem ipsum dolor sit amet, consectetur adipiscing.
Lorem ipsum dolor sit amet, consectetur adipiscing.

What should a reviewer concentrate their feedback on?

[ ] Lorem ipsum dolor sit amet, consectetur adipiscing.
[ ] Everything looks ok?

Good exercice


Docs: Fix FileAccess example conflicting with global scope load

In the example code the declared load() function conflicts with global scope load(). so if you copy pasted the GDScript example it in godot 4.2.2 you would get a "Too few arguments for "load()" call. Expected at least 1 but received 0." error since it doesn't override global scope load().
So i only renamed the functions into to: save_to_file() and load_from_file() for GDScript; and for C# to SaveToFile() and LoadFromFile().
to avoid overriding global scope load().
Heres what the current example code looks like in engine:

Here my proposal

No problem 👍


Implement test case for joining two tables with identical column names

I have read the CLA Document and I hereby sign the CLA

Yeah I wasn't sure where the bug was coming from, so figured this would be a good test to have

Closing this in favour of #696


How to add  formedittext property programmatically

How to add testtype, customRegexp, inputType in for dynamic generated formedittext .

Hello @ishwarishwar2081 Take a look here -> https://github.com/vekexasia/android-edittext-validator/blob/master/library/src/com/andreabaccega/widget/DefaultEditTextValidator.java#L92


GPS data transmission stops after 1 day

@xuhongv @punee995 @khaledbenh @vencol @a710450641 @zerosigu @ZhongShangWen @fengfeng0328  I run the demo_gps-tracker code, rewriting it a bit, and 24 hours after turning it on, the data transfer stops, the sim card goes, it rings. What could be wrong? why does it stop after exactly 1 day? is there any hidden function that causes this? release code also stops at 24 hours.Can anyone help?

have you tried building your app using "./build.bat demo $PROJ release" to build in "release" mode instead of "debug" mode?
Source is here : https://www.iot-experiments.com/ai-thinker-a9-module-with-custom-firmware/

@xuhongv @punee995 @khaledbenh @vencol @a710450641 @zerosigu @ZhongShangWen @fengfeng0328 I run the demo_gps-tracker code, rewriting it a bit, and 24 hours after turning it on, the data transfer stops, the sim card goes, it rings. What could be wrong? why does it stop after exactly 1 day? is there any hidden function that causes this? release code also stops at 24 hours.Can anyone help?

the issue is that if gprs connection goes down, you can't reconnect unless you reboot the A9.
There is a workaround only if your sim provider allow gprs connection with a known APN

Watchdog?

build release file and upload. it is ok.
I have a same problem and I tried to build release version, so GPS run in several day continuos

The solution is to add the reconnection to the gprs break event code.
API_EVENT_ID_NETWORK_REGISTER_DENIED
API_EVENT_ID_NETWORK_DETACHED
API_EVENT_ID_NETWORK_ATTACH_FAILED
API_EVENT_ID_NETWORK_DEACTIVED
API_EVENT_ID_NETWORK_ACTIVATE_FAILED
...case API_EVENT_ID_NETWORK_DETACHED:         Trace(1,"network detached");                     AttachActivate();         break;.....


2 sparse tests failing on main nightly job

https://github.com/pytorch/ao/actions/runs/11488769819/job/31976208709
-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
  =========================== short test summary info ============================
  FAILED test/prototype/test_sparse_api.py::TestQuantSemiSparse::test_quant_semi_sparse_compile_True - AssertionError: Tensor-likes are not close!

  Mismatched elements: 16384 / 16384 (100.0%)
  Greatest absolute difference: inf at index (0, 0) (up to 0.01 allowed)
  Greatest relative difference: nan at index (0, 0) (up to 0.01 allowed)

  To execute this test, run the following from the base repo dir:
      python test/prototype/test_sparse_api.py TestQuantSemiSparse.test_quant_semi_sparse_compile_True

  This message can be suppressed by setting PYTORCH_PRINT_REPRO_ON_FAILURE=0
  FAILED test/sparsity/test_fast_sparse_training.py::TestRuntimeSemiStructuredSparsity::test_runtime_weight_sparsification_compile - torch._dynamo.exc.TorchRuntimeError: Failed running call_function <built-in function linear>(*(FakeTensor(..., device='cuda:0', size=(128, 256), grad_fn=<ReluBackward0>), SparseSemiStructuredTensorCUSPARSELT(shape=torch.Size([128, 256])), None), **{}):
  torch.float32

  from user code:
     File "/pytorch/ao/test/sparsity/test_fast_sparse_training.py", line 26, in forward
      x = self.linear2(x)
    File "/opt/conda/envs/venv/lib/python3.9/site-packages/torchao/sparsity/training/__init__.py", line 31, in forward
      return torch.nn.functional.linear(x, sparse_weight, self.bias)

  Set TORCH_LOGS="+dynamo" and TORCHDYNAMO_VERBOSE=1 for more information

  You can suppress this exception and fall back to eager by setting:
      import torch._dynamo
      torch._dynamo.config.suppress_errors = True

  To execute this test, run the following from the base repo dir:
      python test/sparsity/test_fast_sparse_training.py TestRuntimeSemiStructuredSparsity.test_runtime_weight_sparsification_compile

  This message can be suppressed by setting PYTORCH_PRINT_REPRO_ON_FAILURE=0
  ==== 2 failed, 1763 passed, 381 skipped, 131 warnings in 3358.25s (0:55:58) ====
  Error: Process completed with exit code 1.

cc @andrewor14 @jerryzh168
So I think this is failing from the PR I merged in https://github.com/pytorch/pytorch/pull/137427 because of Tensor kwargs not being handled properly by the inductor lowering.
It looks like the alpha vector gets mapped to bias, which is the cause of the numerical problems.
I'm going to revert #137427, and add in the changes in https://github.com/pytorch/pytorch/pull/138703, which should fix this.
In the meantime to fix in AO I'm planning to pin a torch version - lmk if this will break your workflow


FW-4671 media search APIs

Description of Changes

Added code to search for media files through the search API
Added code to remove media items from index when the site is deleted
refactored hydration logic into a separate file
updated tests to reflect changes

Checklist

[x] README / documentation has been updated if needed
[x] PR title / commit messages contain Jira ticket number if applicable
[x] Tests have been updated / created if applicable
~Admin Panel has been updated if models have been added or modified~
~Migrations have been updated and committed if applicable~
[x] Team Postman workspace has been updated if applicable

Reviewers Should Do The Following:

[x] Review the code changes here
~Pull the branch and test locally~

Additional Notes
N/A

These changes are looking great! Tested working locally.
The API view doc string decorators could probably get updated to give some information on how media can be searched. The Postman workspace could also use a small update.
Will re-review after the tests have been updated as discussed.

Thanks for the suggestions. The API doc strings and postman workspace was updated in a previous PR related to media search API.
I've added the tests similar to your PR as well. Please re-review when you get some time. Thanks.


Bug in jstr in production app

Hi,
I put our first app using djinni in production and now im getting more than 500 errors about this:

Can anyone help me? I think it trying to start a string but something is wrong in this part. But it work in the most devices.

What branch/version is your version of the support lib coming from?  I looked at djinni_support.cpp:491 and it doesn't match up to an assert with that message.  I'd guess it might be coming from something like this: https://github.com/dropbox/djinni/blob/master/support-lib/jni/djinni_support.cpp#L525
My first guess would be that you're passing a null String to a function which isn't expecting one (i.e. declared in Djinni to take string not optional).  It would probably be clearer if those asserts could throw NullPointerException, but all the asserts in C++ code just throw a generic Error, with the message being the assert expression which was false.

I using the version before the last merge with jni flags:
https://github.com/ezored/dependency-djinni-support/blob/1.0.0/build/support-lib/jni/djinni_support.cpp#L491
I dont upload the latest version with jni flags. I will do it in some days.
I want understand the problem only to fix it before send the new version.

That's the same line I theorized, so I stand by my theory that somewhere up the callstack you're likely passing a null string.

is the same thing I was thinking, but I do not have anything in the code that could return a null string at this point, so I thought it was strange. Anyway I will validate the Android functions that return things from the system to see.
I will post here the updates.


feat: Radon-Nikodym theorem for transition kernels

Let γ be a countably generated measurable space and κ η : kernel α γ be finite kernels. We build a function rnDeriv κ η : α → γ → ℝ≥0∞ jointly measurable on α × γ and a kernel singularPart κ η : kernel α γ such that κ = withDensity η (rnDeriv κ η) + singularPart κ η and for all a : α, singularPart κ η a ⟂ₘ η a .

[x] depends on: #10948
[x] depends on: #10799

bors r+


👷 Support Profiling for checking service performance

Idea
Profiling is a technique to figure out how time is spent in a program. With these statistics, we can find the “hot spot” of a program and think about ways of improvement. Sometimes, a hot spot in an unexpected location may also hint at a bug in the program.

Pyinstrument is a Python profiler. A profiler is a tool to help you optimize your code - make it faster.

Profile a web request in FastAPI
To profile call stacks in FastAPI, you can write a middleware extension for pyinstrument.
Create an async function and decorate it with app.middleware('http') where the app is the name of your FastAPI application instance.
Make sure you configure a setting to only make this available when required.
from pyinstrument import Profiler

PROFILING = True  # Set this from a settings model

if PROFILING:
    @app.middleware("http")
    async def profile_request(request: Request, call_next):
        profiling = request.query_params.get("profile", False)
        if profiling:
            profiler = Profiler(interval=settings.profiling_interval, async_mode="enabled")
            profiler.start()
            await call_next(request)
            profiler.stop()
            return HTMLResponse(profiler.output_html())
        else:
            return await call_next(request)

To invoke, make any request to your application with the GET parameter profile=1 and it will print the HTML result from pyinstrument.
AuthX's Support
With AuthX the abstract of profiling is easy, it's just about calling the ProfilerMiddleware 's class and calling it in add_middleware(ProfilerMiddleware) func that FastAPI provides.
References:

Profiling Python Code
profile-a-web-request-in-fastapi

TODO

[ ] Add Function of Exporting results locally.
[ ] Add Tests for testing the ProfilerMiddleware's class.
[ ] Add Documentation for this Part of Middleware's support
[ ] Add a Little example of how we can use it to Profile our FastAPI application

Have you considered OpenTelemetry + Digma?

Hmmm, looks like really good tools to support, I'm gonna be involved in checking how they gonna fit with a FastAPI application, and give you feedback!
thank you @WP-LKL


(UX Research) Showing scheduled jobs

As a datahub user
I would like to see scheduled (single/repeating) jobs
So that I can verify that they are in queue at the correct time

[ ] Is it possible to see future occurences or only the next job in line?
[ ] Should multiple jobs running on the same day be bundled/merged?
[ ] How does editing a repeating job work?

Research whether we can show jobs in a calendar style view?


Shell autocomplete

As a developer, when I hit tab in my shell I want to be presented with autocomplete options for sesh

bash
zsh
fish
Ensure the autocomplete logic is setup on brew install

See this for more:
https://cli.urfave.org/v2/examples/bash-completions/

Interesting resource:
https://github.com/fzerorubigd/tmass/tree/master/tools


Add an option to reset value of variable to a default value

One of the problems with using this type of architecture is that Scriptable Objects keep their values after exiting the game. It would be really useful to have a version of every Variable type with a default value which is used to set the value OnEnable so that these variables don't keep their values between games.

This has been discussed before #51


REPL: reveal auto-expanded objects

VSCode Version: alpha

Steps to Reproduce:

start debugging
fill the debug console so that a new entered expression appears at the bottom of the REPL
type 'process'

Observe:
process is auto expanded but not revealed. This will make the user try to expand the object which in fact will make it collapse.
I suggest to reveal the auto expanded object at the top of the debug console.

May to investigate


2.0.0 nuget release creates reference issue when linked and executed under linux

I've migrated a self-containing app to net core 2.0 and also updated the postgresql provider to the most recent 2.0.0.
The application used to work well with the previous net core 1.1 under linux.
After the migration I get the error
Unhandled Exception: System.IO.FileLoadException: Could not load file or assembly 'System.Threading, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
During the further investigations, I decided to create a project from scratch to exclude otehr influences.
I used a sample net core console project in visual studio 2017.3, only containing a basic "Hallo world" to console output, referencing dot net core 2.0 and nothing else.
This sample project works fine on linux Pepermint8 and Ubuntu 16.04.
As soon as I add a package reference to Npgsql.EntityFrameworkCore.PostgreSQL 2.0.0, I got the same error again.
Unhandled Exception: System.IO.FileLoadException: Could not load file or assembly 'System.Threading, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
The project file is:
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>netcoreapp2.0</TargetFramework>
    <RuntimeIdentifiers>linux-x64</RuntimeIdentifiers>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Version="2.0.0" />
  </ItemGroup>

</Project>

I've als tried to add a package reference to System.Threading 4.3.0 but that didn't resolve the problem.
Also strange, the dependency viewer in studio doesn't mention that PostgreSql Provider is referencing System.Threading 4.1.0 as in the error but System.Threading.Tasks.Extension 4.3.0.

Clean all bin folder and then re-restore the project. Sometimes that helps

There were no bin folders when I created an empty project and started over again.

When I start the app.

After a couple more tests, I figured out, that there is a possible bug in the current Visual Studio Publish profile or it's parser.
When I published the project manually using dotnet publish, everything worked well.
The bug seams to relate to the RuntimeIdentifiers tag in the project settings.
After using RuntimeIdentifier (singular) instead, publishing from Visual Studio worked also.
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>netcoreapp2.0</TargetFramework>
    <RuntimeIdentifier>linux-x64</RuntimeIdentifier>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Version="2.0.0" />
  </ItemGroup>
</Project>

So, specifying more than one target platform doewsn't work, but there might be a workaround.


Specify inputs additionally to products

Is your feature request related to a problem? Please describe.
It is frustrating to have many recipes in the tables you do not really care about. For example, when I am planning a production station (city block? unit?), I decide which items I want to produce locally and which to bring in by trains. Many times the flow tab and item list is cluttered with stuff that will not get produced locally anyways.
Describe the solution you'd like
I would like to have a list of "free" items in the factory settings, or an input section below the products section.
Describe alternatives you've considered

a "hide recipe" button in the list of steps. The hidden steps would be moved to the bottom of the list, and would not show in the flow tab.
Another alternative (which is not really sufficient for my use-case) would be a checkbox to ignore raw iron, copper, oil and so on

Additional context
For example I want all the recipes which are striked through to be hidden, because I do not want to produce them locally.

Have you tried clicking the item icon, from the items row in the list, to ignore it? That treats the item as an unlimited free input. If that doesn't do what you want, could you clarify what you want to work differently?

This is exactly what I wanted, sorry ':)

Awesome! Do you have any ideas to make that easier to find, by the way? Honestly you are not the first person to even open and close this same type of issue - I can only imagine how many people don't know about that trick.

Thought about it

a link in the options screen that shows a tutorial popup (simple change)
a button next to the current button (hide is a dedicated button)
a "hide" column you can add, which shows a checkbox (hide is a dedicated column).

I like the third idea the most. I found the column chooser really early


Embed webfonts

Fonts need to be available locally.

Fixed with https://github.com/WorldBank-Transport/rra-frontend/pull/84


[IO-499] FilenameUtils.directoryContains false positive issue

IO-499 bug has been fixed.
The old version was looking at if the child canonical path string starts with the parent's.
However, it fails in the case of:
.../top/foo
.../top/foo2/b.txt
since the directory path of b.txt starts with the same directory path with the one above even though the second one is a different directory(foo2). This issue is resolved by comparing the path strings of foo and foo2.

quoting Federico Bonellis comment from the jira issue (https://issues.apache.org/jira/browse/IO-499):

@cagdasyelen I'm afraid this patch doesn't consider the case when the file path is composed using '' separators (aka Windows case). We should adapt it to that case before pulling the patch.


Will this work with the "CONVERT VIDEO WHILE RECORDING" option in PlexDVR?

Will this work with the "CONVERT VIDEO WHILE RECORDING" option in PlexDVR?

To the best of my knowledge, yes. My understanding of "CONVERT VIDEO WHILE RECORDING" is your computer transcodes on the fly. Once the show is done recording the "POSTPROCESSING SCRIPT" runs which in the case of k-plex will trigger running ComSkip to remove commercials.


add gridlines

add gridlines - optional

different sizes possible as well...

maybe horizontal lines as well

Features to add across input data structure, example, definition, and plotting JSON:
In the curve_box section

grid-lines: visible or not
grid-lines: stroke-width
grid-lines: stroke color
gridlines: number of lines or division of width-margin.left-margin.right
gridlines: stroke-dash-array (not sure whether to include this as it changes color to black??

Same as above but for horizontal


Minimizing NadekoBot to system tray

Is it currently possible to minimize the command prompt window to the system tray? I'm hosting my own NadekoBot and it's kinda annoying having the program in my taskbar all day. If there isn't a way, can this be implemented somehow?

Windows or linux?

I'm using Windows 10 atm

I'm guessing Windows. You could probably get nadeko to run as a daemon
somehow.
On Tue, Dec 5, 2017, 3:25 PM Macley-Kun<EMAIL_ADDRESS>wrote:

Windows or linux?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Kwoth/NadekoBot/issues/1902#issuecomment-349429825,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABGad7LFWvPu448fe-2CM61QdXz9yuTCks5s9aaggaJpZM4Q25pH
.

Sorry but then i have no real clue how to minimize that...
https://feathub.com/Kwoth/NadekoBot/+172
^not that you have much with this link but, it shows 2 ways to 'minimize' it. Sadly it's only for linux...

Yea just continue the convo on the feathub.


Implementation of the JSON handler

Closes #57
The other JSON files (non "pr-template.json") are left for convenience

Added builder for the Handler. I wonder how much of a bad pattern is to use it in @BeforeEach like that...


Initialize store

idea is similar to messenger setup? i like :+1: - not sure where to use it atm tho


Try using @embroider/addon-dev

Test out: https://github.com/embroider-build/embroider/pull/1766

no longer needed as this library uses vite


Change the order that we reset state

@ggezer

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4235/
Test FAILed.

jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4236/
Test FAILed.

jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4243/
Test PASSed.

@ggezer addressed comment, PTAL. thanks.

alluxio-bot, merge this please.

alluxio-bot, cherry-pick this to branch-2.0 please.

alluxio-bot, cherry-pick this to branch-1.8 please.

alluxio-bot, cherry-pick this to branch-1.8 please.


Roll Flutter from 758ebda16dc1 to 82afe3ea4336 (3 revisions)

https://github.com/flutter/flutter/compare/758ebda16dc1...82afe3ea4336
2022-05-06<EMAIL_ADDRESS>Clear the cached data of RenderBox if its parent re-layout (flutter/flutter#101493)
2022-05-06<EMAIL_ADDRESS>Reland "Fix crash from alt-tab'ing just after startup"  (flutter/flutter#103093)
2022-05-06<EMAIL_ADDRESS>Roll Engine from 387a9b9e300f to b1b0a5adffe9 (1 revision) (flutter/flutter#103159)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Plugins: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Add en-GB to supported Dashboard cultures

Support en-GB in the dashboard so dates and times are formatted appropriately for users in the UK.
I haven't added a test as I wasn't sure where to add such a test - if you'd like one, just point me in the right direction.
Before

<fluent-data-grid-cell cell-id="c12" cell-type="default" grid-column="4" class="col-justify-start " style="height: 46px; align-content: center; grid-column: 4;" title="7/13/2024 11:16:29 AM" aria-label="7/13/2024 11:16:29 AM" b-w6qdxfylwy="" tabindex="-1" role="gridcell">
  <!--!-->
  11:16:29 AM
</fluent-data-grid-cell>

After

<fluent-data-grid-cell cell-id="c133" cell-type="default" grid-column="4" class="col-justify-start " style="height: 46px; align-content: center; grid-column: 4;" title="13/07/2024 11:47:05" aria-label="13/07/2024 11:47:05" b-w6qdxfylwy="" tabindex="-1" role="gridcell">
  <!--!-->
  11:47:05
</fluent-data-grid-cell>

Microsoft Reviewers: Open in CodeFlow

/azp run

As an Australian I fully endorse this change :)

I was curious to compare to the ones eg that the dotnet homepage supports: "en-US","es-ES","fr-FR","id-ID","it-IT","ja-JP","ko-KR","pt-BR","ru-RU","zh-CN","zh-TW". They have Indonesian, eg. I expect that is based on where they get traffic from, so perhaps rather different to the dashboard.

How far do we go with this? In dates the month/day is around the wrong way for en-AU, en-NZ, en-SG, etc.
I wonder if there is configuration to automatic say use the more specific culture settings for formatting, and then use the approximate translations, e.g. en-GB, en-US, etc all use the en translations.
Who is the expert on request localization middleware that we can ask?

How far do we go with this?

I opened #4884 to ask that question.

AFAIK this can be handled by Blazor based on headers sent by the browser on what their locales are. Then when you use the date formatting functions, it should pick up the locale based on Blazor interpreting the headers and setting the context.
https://learn.microsoft.com/en-us/aspnet/core/blazor/globalization-localization?view=aspnetcore-8.0

Small nit - this is globalization not localization.


Fix cli bug about getting arguments["validator"].

As commented in https://github.com/Julian/jsonschema/issues/710#issuecomment-676851156.
By adjusting the position of arguments["validator"], we can solve this bug, but because arguments["validator"] is not processed in parse_args, it will cause the test_latest_validator_is_the_default test case to fail, so I deleted this test case.

Thanks! Much appreciated.
This does look likely that it fixes the bug, but we do need a test, both for the new bug (that we weren't ensuring the right validator was used), and also to preserve the test that as you say may need to move now from parse_args to elsewhere, that we use the latest draft when unspecified.

Okay, next I will add some test cases.

Thanks, merged this one after fixing a few minor issues.


chore(main): release 2.0.1

:robot: I have created a release beep boop
2.0.1 (2024-04-19)
Bug Fixes

manipulate: get_median accepts arbitrary dimensions provided one is time (9197a72)
manipulate: mean_flatten should copy the dims list input parameter (9623f4a)

Styles

ruff lint: add preview mode (131d843)
ruff: fix lint errors enforced by preview mode (861a151)

Build System

deps-dev: bump ruff from 0.2.1 to 0.3.7 (3d3ebf8)
deps: update all dependencies (86a32f9)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/engeir/volcano-base/releases/tag/v2.0.1 :sunflower:


Inconsistency between Mixer and Render fade types

The Mixer fade speed is now specified with type std::chrono::milliseconds. However the Renderer fade methods were not also updated to use this type. This creates awkwardness in downstream OPHD, since it uses the same constant for both Music and Renderer fades.
We should update these types to match.

Patched in #955.

Adding reference to #948, #943.


Feature/leave

Removed leave on error.
Also, attempting to import from the npm module I get the following error:
Module not found: Error: Can't resolve 'apollo-phoenix-websocket' in

So I updated the main in package.json to point to the src/index.js file and this seems to work.

@vic Can I merge this?

Oh I added a comment on reverting the change for src/index.js, is it being shown to you?

That's strange, I can't see the comment but I did get an email notification with the comment a few minutes ago.
That makes sense that lib/index.js is generated as part of the build. However, the current package published to npm does not contain a lib folder. Any idea why this is?

Haha because I forgot to add "files" in the package.json
I'm gonna merge this and publish a new version to npm, which would have the lib/index.js file.

Done, conformed the new release has a lib/index.js file.  There's a lib/phoenix.js there because I forgot to clean before publishing haha. But it's never used.

@vic That's fine! Thanks for the changes. I should be able to fully integrate with my project tomorrow :)


upgrade SimplePing

You are using SimplePing from Apple for ping request, but I found that your SimplePing is outdated.
You used the version of 2012, but Apple published the version of 2016.
The problem that use 2012 version, is that it does not support IPv6, so your PlainPing crashes on ipv6-only network.
I already fixed this problem and I want to contribute my changes.
Look forward to hear from you.

Hi
you're right, the code is outdated.
Please sign a PR and I will accept the changes!
Thanks for helping!


Read performance with HDF5, Windows 10, HDD

How to reproduce
Run the "center of mass" example with the EMD file from a HDD on Windows.
What happens
Read speed is about 17 MB/s on my system, while the HDD is capable of about 117 MB/s using LiberTEM with the buffered reader.
What should happen
Read perfromance of HDF5 should be in the same order of magnitude as other file formats.

Could you try if the linked PR fixes/improves this issue? As the file is chunked, we don't have many options, but this change is kind of a low-hanging fruit.
On my system, this goes from taking 15s cold to being as fast as processing a raw file. Before:
14.963273048400879 - cold EMD
0.8389596939086914 - hot EMD
11.765928745269775 - cold RAW
0.3656883239746094 - hot RAW

After:
11.2949378490448 - cold EMD
0.77344202995300 - hot EMD
11.7877163887023 - cold RAW
0.29714965820312 - hot RAW


SUBMARINE-316. Support listen k8s event in submarine operator

What is this PR for?
Listen to k8s pod, svc creation, destruction and other events in the submarine operator.

[x] - Submarine CRD auto register
[x] - Submarine Cluster SVC add/remove event listen
[x] - Submarine Cluster POD add/remove event listen

How To Test
cd submarine-cloud
make
./submarine-operator --kubeconfig=$KUBECONFIG --alsologtostderr

open new console
kubectl apply test1.yaml
kubectl apply test2.yaml
kubectl delete test2.yaml
kubectl delete test2.yaml

What type of PR is it?
Feature
What is the Jira issue?

https://issues.apache.org/jira/browse/SUBMARINE-316

How should this be tested?

CI Pass

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

Will merge if no more comments.


BulkInsertAll failling for table with no non primary key columns

Hi, I'm trying to use your library, but BulkInsertAll gives me an SqlException: Invalid syntax near ','.
I tracked the problem down to this line: https://github.com/mtanneryd/ef6-bulk-operations/blob/7ca96c682037bad9c40de83c0041abe3f8ee5005/Tanneryd.BulkOperations.EF6/Tanneryd.BulkOperations.EF6/DbContextExtensions.cs#L1769
The issue is that 1 of my tables only has  a primay key. It means that columnNames is an empty string, and it generates an sql statement like this:
MERGE [dbo].[TraitTargets]
                        USING
                            (SELECT , rowno
                             FROM   tempdb..#9ada186c0c524a4aa88daa44962c87a4) t (, rowno)
                        ON 1 = 0
                        WHEN NOT MATCHED THEN
                        INSERT ()
                        VALUES ()
                        OUTPUT t.rowno,
                               inserted.[TraitTargetId];

which is indeed invalid sql. You should probably check if nonPrimaryKeyColumnMappings has any values.
Any chance for a quick fix?

I'll take a look tonight. Have some other additions on the way planned for
tonight anyway. I'll try to upload a new beta asap.
Måns Tånneryd
Tånneryd IT AB
Barrskogsvägen 19
186 53 Vallentuna
+46-705140093
https://se.linkedin.com/in/manstanneryd
https://github.com/mtanneryd/ef6-bulk-operations
https://github.com/mtanneryd/ef6-bulk-operations
Den ons 6 nov. 2019 kl 16:33 skrev Jan Korf<EMAIL_ADDRESS>
Hi, I'm trying to use your library, but BulkInsertAll gives me an
SqlException: Invalid syntax near ','.
I tracked the problem down to this line:
https://github.com/mtanneryd/ef6-bulk-operations/blob/7ca96c682037bad9c40de83c0041abe3f8ee5005/Tanneryd.BulkOperations.EF6/Tanneryd.BulkOperations.EF6/DbContextExtensions.cs#L1769
The issue is that 1 of my tables only has a primay key. It means that
columnNames is an empty string, and it generates an sql statement like this:
MERGE [dbo].[TraitTargets]
USING
(SELECT , rowno
FROM   tempdb..#9ada186c0c524a4aa88daa44962c87a4) t (, rowno)
ON 1 = 0
WHEN NOT MATCHED THEN
INSERT ()
VALUES ()
OUTPUT t.rowno,
inserted.[TraitTargetId];
which is indeed invalid sql. You should probably check if
nonPrimaryKeyColumnMappings has any values.
Any chance for a quick fix?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mtanneryd/ef6-bulk-operations/issues/24?email_source=notifications&email_token=AA2BSRPWPWGSBWNOQW5FRJLQSLPVHA5CNFSM4JJWVLF2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HXINYOA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2BSRNIWJMZHAGRMQTRV53QSLPVHANCNFSM4JJWVLFQ
.

I just uploaded 1.3.0-beta1. I have not tested it properly but it just
might do the trick. The API has changed a bit so you need to change your
calls to BulkInsert and use the new format of the BulkInsertRequest. No big
changes but still breaking changes.
Måns Tånneryd
Tånneryd IT AB
Barrskogsvägen 19
186 53 Vallentuna
+46-705140093
https://se.linkedin.com/in/manstanneryd
https://github.com/mtanneryd/ef6-bulk-operations
https://github.com/mtanneryd/ef6-bulk-operations
Den ons 6 nov. 2019 kl 16:33 skrev Jan Korf<EMAIL_ADDRESS>
Hi, I'm trying to use your library, but BulkInsertAll gives me an
SqlException: Invalid syntax near ','.
I tracked the problem down to this line:
https://github.com/mtanneryd/ef6-bulk-operations/blob/7ca96c682037bad9c40de83c0041abe3f8ee5005/Tanneryd.BulkOperations.EF6/Tanneryd.BulkOperations.EF6/DbContextExtensions.cs#L1769
The issue is that 1 of my tables only has a primay key. It means that
columnNames is an empty string, and it generates an sql statement like this:
MERGE [dbo].[TraitTargets]
USING
(SELECT , rowno
FROM   tempdb..#9ada186c0c524a4aa88daa44962c87a4) t (, rowno)
ON 1 = 0
WHEN NOT MATCHED THEN
INSERT ()
VALUES ()
OUTPUT t.rowno,
inserted.[TraitTargetId];
which is indeed invalid sql. You should probably check if
nonPrimaryKeyColumnMappings has any values.
Any chance for a quick fix?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mtanneryd/ef6-bulk-operations/issues/24?email_source=notifications&email_token=AA2BSRPWPWGSBWNOQW5FRJLQSLPVHA5CNFSM4JJWVLF2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HXINYOA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2BSRNIWJMZHAGRMQTRV53QSLPVHANCNFSM4JJWVLFQ
.

I believe this has been fixed. Please re-open if this is not so.


[Mobile] or [Desktop]: Undo errors

The issue is about [Mobile] and [Desktop]
Pressing "Undo" might crash the app with
type 'Null' is not a subtype of type 'Block' in type cast
error.
Steps to reproduce on mobile:

Run an app with "Empty asset" initial content
Move the cursor to the end of the text, press "Enter"
Enter "Test text"
Move the cursor to the beginning of the text
Click undo (sometimes 2-3 times)

Also, undo doesn't care about the cursor position:

Run an app with "Empty asset" initial content
Remove "Empty asset" text
Enter two lines of text: "Line one" and "Line two"
Remove "two" word
Move the cursor to the end of the first line.
Press "Undo"
The cursor moves to the place after "Li" on the second line

Will take a look

'res.node' can be null, so 'res.node as Block' will crash

Hello, I believe this is due to incorrect positioning of the cursor when undo is triggered. We'll look into this.

Hello, we're not able to reliably reproduce this issue, but I believe it's related to the cursor being positioned incorrectly. We'll look into it more but closing this issue for now. Thanks.

@li3317 I have this exact same error when running in my app (web and mobile). I'm unable to reproduce it in the example app, but I am able to reproduce it consistently in my app.

Sorry we can't help it

undo doesn't care about cursor position anyway (which might cause the crash described).
Need find away to keep cursor position + undo object together.

Maybe work
void _handleHistoryChange(int? len) {
    if (len != null && len != 0) {
      final newOffset = len;
      if (newOffset >= document.length || newOffset < 0) {
        updateSelection(TextSelection.collapsed(offset: document.length),
            ChangeSource.LOCAL);
      } else {
        updateSelection(
            TextSelection.collapsed(offset: newOffset), ChangeSource.LOCAL);
      }
    } else {
      if (selection.baseOffset >= document.length) {
      //  make sure not overflow selection
        updateSelection(TextSelection.collapsed(offset: document.length),
            ChangeSource.LOCAL);
      }
      notifyListeners();
    }
  }

HistoryChanged _change(Document doc, List<Delta> source, List<Delta> dest) {
    if (source.isEmpty) {
      return const HistoryChanged(false, 0);
    }
    final delta = source.removeLast();
    // look for insert or delete
    final ops = delta.toList();
    int? mylen;
    for (var i = 0; i < ops.length; i++) {
      if (ops[i].key == Operation.retainKey) {
        mylen = (mylen ?? 0) + (ops[i].length ?? 0);
      }
    }

    final base = Delta.from(doc.toDelta());
    final inverseDelta = delta.invert(base);
    dest.add(inverseDelta);
    lastRecorded = 0;
    ignoreChange = true;
    doc.compose(delta, ChangeSource.LOCAL);
    ignoreChange = false;
    return HistoryChanged(true, mylen);
  }

Hi, is this resolved?

@li3317 could you please revisit this one and maybe reopen it? This is still happening in the current version (9.2.6).
I really love this package and appreciate all the work that went into it, but feels like a legitimate bug is just being ignored for over 6 months.

@ellet0, could you please take a look?

I might do but expect nothing from me please
Thank you for your understanding


extra backslash on markdown hyperlink intellisense

repro:
///
/// enumerate given items returning a tuple (item,next,isLast) with next=null for last element or next=first if repeatFirstAtEnd=true on latest el
///
/// <param name="repeatFirstAtEnd">last enumerated result will (last,first,true)</param>
/// <remarks>
/// [unit test](https://github.com/devel0/netcore-util/tree/master/test/Enumerable/EnumerableTest_0001.cs)
/// </remarks>
public static IEnumerable<(T item, T next, bool isLast)> WithNext<T>(this IEnumerable<T> en, bool repeatFirstAtEnd = false) where T : class
{
}

with intellisense I got follow

but this is the markdown I used
[unit test](https://github.com/devel0/netcore-util/tree/master/test/Enumerable/EnumerableTest_0001.cs)

Note: the produced link unit test works but the ending slash I think is an addiction that with some web server could generate a directory not found.

C# documentation comments are not markdown. We escape any markdown sigils in said comments. This is expected behavior.

Thanks for the explanation, but I'm unsure about it, you said that any sigils in the markdown are escaped but from what I see:
[DESC](URL)

produces in the vscode intellisense ballon
[DESC](URL\)

so only ending ) are escaped while the other ( still unescaped; observed that I tought that slash may coming from a unexpected directory char separator ( windows style ) appended to the url and I reported for further investigation.
Btw, with actual implementation it works due to the fact that destination web server ( github ) automatically manage that extra slash detecting its a file, not a directory.
Initially in the comment documentation, I tried to use linktext that works for the DocFx generated documentation but not for the vscode intellisense ballon text.

All of them are actually getting escaped, it's just that vscode's rendering is interpreting the trailing \ as part of the url, not as an escape character. https://github.com/OmniSharp/omnisharp-roslyn/blob/96aec1d27209a4db238a6831f29745f1bc349c78/src/OmniSharp.Roslyn.CSharp/Helpers/MarkdownHelpers.cs#L14 is the escape code. I'm somewhat surprised that vscode's rendering doesn't display the a tag correctly though, I would have expected that to work.

thanks for the precise clarification referring also in the code, for me it's enough: generated hyperlink from vscode balloon even misrendered still works for the purpose.
I prevent to open an issue for this feedback on vscode because of low impact and stll there 5k+ opened issues atm.


Where is rp for datapack?

I can`t find it anywhere

I am unable to do repeat uploads on Curseforge, so you may have to look at the "additional files" a view versions back
But! It is packaged with every version on Modrinth: https://modrinth.com/datapack/jakes-build-tools/version/1.8.3

ok thanks ^^


Add light-dark

Screenshot of my A11y card:

For assistance: https://help.github.com/en/articles/file-attachments-on-issues-and-pull-requests

Love it 💙!
Thanks!


Starts fine, but no style

You have clone the wrong project https://github.com/ixartz/Next-js-Boilerplate
The correct one is: https://github.com/ixartz/SaaS-Boilerplate


wheels WIP

On my local system installing into a new virtualenv off of this branch successfully skips the pysam compile step, which is the most timely step in our build.  These changes drop the build time on my system from around 5 minutes to under 30 seconds.
Wheels are OS/arch dependent, to an extent.  The one that I built doesn't work on the Travis environment, which uses a 64 bit Ubuntu build.  We just need to get a wheel that the Travis environment finds acceptable, which might take a bit of guess-and-check (building the wheel on an identical Ubuntu environment would work).    The wheel we need can be created with the command pip wheel pysam.
We would also need to check in a new version of the wheel whenever pysam had changes we wanted to incorporate.
Questions:

is it acceptable to check in the wheelhouse to source control, as I've done here, or do we want to download it off of some remote server every time a branch is pushed to GitHub?
do we want to use a wheelhouse for only pysam, all our required packages, or something in between?

Issue #71

Can you try out the pysam wheel stored here please @dcolligan? I've made a Ubuntu 12.04 VM, and mirrored travis's Python setup as closely as I can.
To answer your questions

I wouldn't be keen on keeping the wheelhouse in git, and would rather keep it on a server. We can use the URL I've provided initially. If it becomes a problem we can look for better hosting.
Just pysam would do I think. It's pretty tedious keeping these up to date!

Here's the build for that wheel: https://travis-ci.org/dcolligan/server/builds/53819082
It looks like it accepted the wheel, but there's some symbol that it can't find (PyUnicodeUCS2_DecodeUTF8).  Apparently this has to do with some version mismatch of the interpreter and the library: http://stackoverflow.com/questions/11878666/python-undefined-symbol-pyunicodeucs2-decodeutf8
@jeromekelleher at this point we have three options:

continue guess-and-check: make another wheel with the above information and try again
build the wheel as part of the Travis build -- this would avoid version mismatches -- and then upload that to some target server.  We can then use that wheel in all Travis builds going forward
(I would not prefer this option) If pysam is going to be removed from our code soon, give up :(

Thanks @dcolligan. (1) seems to be the only viable option here. I don't like (2) as we'd have to embed credentials for the uploading site into our Travis build and this seems dangerous. We're betting heavily on pysam doing lots of stuff for us, so it's hopefully going to be here for a long time.
I'll have another go at making the wheel.

OK, I rebuilt my Python in the Ubuntu VM using the correct unicode size, and it all seems to work now. Seems like a result to me!

@jeromekelleher alright, this latest version seems to work.
merge me!

+1. Merging.


discord.Color(0xHEX) can be used as 0xHEX, and a bit of minor changes

major optimisations 🤔

please don’t kill me banana