Can't seem to get to work after updates

/Users/phil/.rbenv/shims/transcode-video: line 21: /usr/local/Cellar/rbenv/1.1.1/libexec/rbenv: No such file or directory
I have Ruby 2.6.2 - when I first ran transcode-video it complained and said it needed 2.3.0 - so I used rbenv to switch ruby versions.
Then I got the above error - looks like rbenv has broke it. I've uninstalled and installed. Any other ideas?

@PhilStollery I'm sorry you're having this problem and I'll see what I can do to help.
First, I'm not sure why transcode-video would say that it needed Ruby version 2.3.0. It should only require 2.0 or later and it doesn't even have an error message for that. Your Ruby interpreter must have made the complaint. Although I've never seen anything like that.
Also, line 21 in transcode-video is the beginning of the about function. So I don't know why that would cause an error either.
In order to help and not make things worse, I need to know more about your OS and Ruby configuration. Obviously you're running macOS but, for example, which version?
Could you execute the following diagnostic commands in a Terminal.app window and paste the results here:
echo $PATH

which ruby

ruby --version

gem environment

Also, I assume you installed and then removed rbenv using Homebrew? If so, did you also install the new Ruby version with Homebrew?
Thanks and sorry I don't have a more helpful answer right away.

Please don't spend any time try to work this out if it's environmental. I'll try and get my system back to something. I'm using fish so it might be that?
...
Uninstalled and removed rbenv and all now works. Thanks for all your hard work on this Don.

Also check the rbenv maybe with
„Which rbenv“
And run „brew doctor“, maybe there are some stray symlinks

@PhilStollery You are very welcome, sir, and I'm glad it's all working for you now.
Please don't hesitate to comment again here if you have another problem or even open a new issue. We want to make sure transcode-video is always working for you.


🐞 Bug

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Navbar is not resposive
Steps To Reproduce
1 launch simulator
check the navbar
Expected Behavior
it should be resposive

Screenshots
No response
Device Information [optional]
- OS:
- Browser:
- version:

Additional context
No response
Are you working on this issue?
Yes

@riteshsinghh-coder  provide a proper title of the issue

Super bro


1.0.0-beta3 Update query string on current route

I am trying to update the query string on the current route I am on.
For example, I am on route /test
When the user clicks a button on that page, I want the route to be /test?query=123
I have tried this:
contextTypes: {
  router: React.PropTypes.object
},

changeQuery() {
  this.context.router.transitionTo('/test', null, {
    query: Math.floor((Math.random() * 10000) + 1)
  })
},

It seems to re-render the component, but the URL in the browser does not update.
Any guidance would be appreciated.

+1
After a bit of testing, I'm almost positive that in 1.0.0-beta3 transitionTo does not handle any of its arguments after the first one (neither queries nor state). That's a pretty important function to be lacking functionality even for a beta version...

I'd love to find out if there is some kind of work around for this. We have
some critical functionality that relies on this.
On Monday, August 3, 2015, Anton Gilgur<EMAIL_ADDRESS>wrote:

+1
After a bit of testing, I'm almost positive that in 1.0.0-beta3
transitionTo does not handle any of its arguments after the first one
(neither queries nor state). That's a pretty important function to be
lacking functionality even for a beta version...
—
Reply to this email directly or view it on GitHub
https://github.com/rackt/react-router/issues/1659#issuecomment-127457517
.

I was trying the beta, but I think I'm going to have to roll back until this is fixed. This is a pretty import part of the workflow of my app.

@duro @agilgur5 I think the problem may be that the signature for the transitionTo method has changed. Check here. The query and state arguments are now flipped. Swapping the arguments is working for me. Perhaps this is your problem too?
@duro Your example should look like this now, I think:
contextTypes: {
  router: React.PropTypes.object
},

changeQuery() {
  this.context.router.transitionTo('/test', { // <-- no `null` here anymore
    query: Math.floor((Math.random() * 10000) + 1)
  })
},

Yes this seems to be working for us.
On Wednesday, August 5, 2015, TimothyKrell<EMAIL_ADDRESS>wrote:

@duro https://github.com/duro @agilgur5 https://github.com/agilgur5 I
think the problem may be that the signature for the transitionTo method
has changed. Check here
https://rackt.github.io/react-router/tags/v1.0.0-beta3.html#-transitionto-pathname-query-state-.
The query and state arguments are now flipped. Swapping the arguments is
working for me. Perhaps this is your problem too?
@duro https://github.com/duro Your example should look like this now, I
think:
contextTypes: {
router: React.PropTypes.object
},
changeQuery() {
this.context.router.transitionTo('/test', { // <-- no null here anymore
query: Math.floor((Math.random() * 10000) + 1)
})
},
—
Reply to this email directly or view it on GitHub
https://github.com/rackt/react-router/issues/1659#issuecomment-128123609
.

Hmm... I thought that was the problem initially, but it didn't seem to work on my end :/ I also have a static query param and tried using this.context.router.transitionTo('/path?query=param'), yet that didn't seem to update the URL for me... Guess I should try again? Can you guys let me know if the static version in URL (vs object notation) works for you if you have the time?

Guys, how I can access to method transitionTo without context?
Ex.
import Router from 'react-router';
Router.transitionTo(arguments)

yeah we jumped the gun on the beta tag, don't use it until we ship it if you don't want to be scratching your head a lot :)
(or just drop a debugger in and step into the code to figure it out)

Guys, how I can access to method transitionTo without context?

Navigation mixin

yeah but ES6 classes

Dont' use them

but I heard they are were cool

Then use context

but I heard that was bad

Sorry.
:P

@ryanflorence mixins in the store?


Update studio-apis-for-pluggable-widgets.md

Re-added the stripped down version of action property values. Also removed styleObj from Preview Export, as we only want to use style from now on.

@takumagott Earlier the Actions section was taken out because the customers didn't have any business in that part of the API. I take it this new draft is curated, useful information about Actions the customer would want to know?
Just trying to make sense of the back and forth. Thank you.

In short, the removal was supposed to be temporary until we fixed it in the program. Now we fixed it.
In long:
We removed the part of the docs so customers would not use the previous, complex object structure which contained a lot of information about the action. We couldn't find any use cases for the usage of all that information, but we could think of use cases where you want to check whether a value is set or not. At the same time we didn't want to update the docs to this draft yet, because the now documented structure simply didn't exist yet (before, not setting an action resulted in an object with type "NoClientAction" and so on). That's why we wanted to remove the info temporarily until we actually updated the structure.

In short, the removal was supposed to be temporary until we fixed it in the program. Now we fixed it.
In long:
We removed the part of the docs so customers would not use the previous, complex object structure which contained a lot of information about the action. We couldn't find any use cases for the usage of all that information, but we could think of use cases where you want to check whether a value is set or not. At the same time we didn't want to update the docs to this draft yet, because the now documented structure simply didn't exist yet (before, not setting an action resulted in an object with type "NoClientAction" and so on). That's why we wanted to remove the info temporarily until we actually updated the structure.

Ok great, thanks for the explanation. This PR looks good. I'll pass it through.


JS FormData implementation

Is it possible to change:
var formData = $commentForm.serializeArray();
to FormData object? To make file uploads.

You can create your own comment.js file and override the comment asset as follows:
'assetManager' => [
            'bundles' => [
                'yii2mod\comments\CommentAsset' => [
                    'js' => [
                        'your-own-comment.js',
                     ]
                ],
            ],
        ]


use instruct tags , , and  while also cloning someone else's voice

Thanks for the open source. Amazing work!
Is there a way to use instruct tags <//laughter>, <//strong>, and <//breath>, while also cloning someone else's voice tone (input prompt_speech_16k)?

check code in cli/cosyvoice.py, you can do it but this mode is not very stable

check code in cli/cosyvoice.py, you can do it but this mode is not very stable

I checked the cosyvoice file, but the relevant interfaces are set up, and I don't know how to pass in my own trained voice as a spk_id

check code in cli/cosyvoice.py, you can do it but this mode is not very stable

Thanks for the reply!
You are correct, I've done some tests on the instruct. It's not good enough for serious usage.
I actually found that zero_shot can do something similar. I recorded samples, such as the tone of statements, strong, etc. Then I made a script to recognize tags and automatically separate sentences by tag, switching samples as prompt_speech according to the tag. It has a pretty good effect.

check code in cli/cosyvoice.py, you can do it but this mode is not very stable

I checked the cosyvoice file, but the relevant interfaces are set up, and I don't know how to pass in my own trained voice as a spk_id

check spk2info.pt, just add your trained voice spk embedding in it


name of executable clashes with existing tool

Hi,
Nice tool.  I am now looking into packaging it for Debian.
One little nit, though: The executable is called "spin" which clashes with another tool.
Please consider renaming this tool.  I suggest same name as the project: "tailspin".
In fact, for now I am preparing the Debian package with name changed to "tailspin" because it is not permitted for multiple packages in Debian to provide same name (unless they provide same ABI which clearly this name clash is not an example of).

Hi Jonas!
Thank you very much for using tailspin and for looking into packaging it for Debian! I hope we can find a good solution to the naming conflict.
For me personally, I wanted to go with spin as the binary name since it was short and simple. It was important for me that the binary name reflected the simplicity of the tool itself.
Would it be possible to rename the binary to tailspin as a part of the Debian packaging process? That way binary name would be tailspin on Debian (this would be mentioned in the readme) and spin for other package managers?

Yes, there is absolutely no need for you to align with the needs of Debian.  I just thought that you might want a single unified name across distros and directly-installed, and know from experience that it is more painful to rename later.

another relatively short name not already used within Debian is tspin, in case that interests you.

Thanks for letting me know. It is always a tradeoff of something, I guess 😄 . Lets go with just tailspin as the binary for now.
Thank you again for taking the time to do this, really appreciate the help to get tailspin to more package managers 🙏 . I'll update the Readme as soon as it is ready.

FYI, tailspin has now entered the development branch "unstable" of Debian.
A developer-oriented status page is at https://tracker.debian.org/pkg/tailspin
A more user-oriented page is at https://packages.debian.org/tailspin

Thank you very much for taking the time to bring this to Debian.
How would a Debian user go about installing this? I can update the readme with the instructions.

They would issue the command apt install tailspin.

I've updated the Readme with installation instructions.


Investigate a port to Kotlin to reduce code cruft

I've been working with Kotlin a bit recently, and it leads to a lot less cruft vs vanilla Java.
Also, this would allow aligning external APIs with
https://github.com/RobotPajamas/SwiftyTeeth (as Swift/Kotlin are pretty similar - but CoreBluetooth is nothing like Android's BLE)

So far, so good - however, there are some fundamental differences in how the singleton pattern in Kotlin works (e.g. no good way to do a .with(context) semantic).
Might need to switch this over to Blueteeth.init(context) and then declare the singleton as valid - Not a huge fan because .with() is safer, and with Dagger/DI - both of those can be eliminated anyways.

Have been using the Kotlin port in production for an app, and it seems to work very well - just need to clean up the API and make the code more canonical Kotlin (and also ensure clean interop with Java)

Moved to develop - code works, but the API is not seamless yet. Will clean that up before any releases to master.


lightgbm.cv weighted stratified sampling

Hi,
Wanted to know whether lgb.cv() does weighted stratified sampling when it's marked as True.
Could anyone confirm it?

@StrikerRUS  any help regarding this? and could you please reopen the issue as it was closed before discussing in details

@nirupamkar

Wanted to know whether lgb.cv() does weighted stratified sampling when it's marked as True.

No, stratification in LightGBM is not "weighted".
As I already said earlier, you can see the code of how stratified folds are cooked in scikit-learn repo.
https://github.com/scikit-learn/scikit-learn/blob/7389dbac82d362f296dc2746f10e43ffa1615660/sklearn/model_selection/_split.py#L628
X isn't taken into account at all when stratification is performed. Only y matters.


Show the content sidebar as a modal toggle option in Mobile / Tablet views

Fixed with new sidedocs.


Can not Swipe ViewPager

I wish that ViewPager(+ Fragments) is Swipe.
However, with this scheduler in Fragment I can not Swipe ViewPager. How do I do Swipe?

Hi, how do you have the code setup? The sample app uses a view pager and fragment and it works as expected. Try cloning the project and run the sample app and see if it works.
What android version are you using and what phone/emulator? What does your xml layout look like?

I do not want to swipe the calendar in ViewPager. I want to swipe the fragment itself in the ViewPager.
When the ViewPager has the Home Fragment and the Event Fragment, I want to move to the Events Fragment if I swipe in the Home Fragment.
I would be pleased if you could tell me how to do so.

In that case, have you tried compactCalendarView.shouldScrollMonth(false);
This will disable month scrolling and allow you to move between fragments, you just won't be able to scroll between months, which is what I think you want?

I know the option. but I did want to swipe the fragments when try scroll on CompactCalendarView.
If I use the option, calendar and fragment All are not swiped when I try scroll on CompactCalendarView.

Ok, I'm not 100% sure what you need but if you can upload a very simple example project in github. I can then clone and debug the app on a device so that I can understand your requirement by seeing the problem. Is that ok? Maybe in your app there is some different configuration.
When I tried that method in the sample app, I was able to scroll between the fragments whilsts my finger was on the calendar it self, just the calendar was no longer handling the scroll event and passing back to the view pager.

Sorry. I can't speaking English well.
You can test your sample app. I'm upload this image for explanation.

Are you debugging sample app from master? If not what version of the library are you using? The below is the latest version:
dependencies {
    compile 'com.github.sundeepk:compact-calendar-view:<IP_ADDRESS>'
}

I used <IP_ADDRESS>. I updated to <IP_ADDRESS> now and I see resolved the issue. Thank you!


Cannot start apiserver as pod

Using CoreOS alpha (962.0.0), I am unable to start apiserver as a pod, as described in the CoreOS Kubernetes guide. The problem appears to be that my certificate file can't be opened, although I don't understand why because it's perfectly accessible from the command line. This is how I find the error message:
$ sudo cat /var/log/containers/kube-apiserver-*
{"log":"I0227 20:22:58.701844       1 plugins.go:71] No cloud provider specified.\n","stream":"stderr","time":"2016-02-27T20:22:58.702180241Z"}
{"log":"F0227 20:22:58.701965       1 server.go:416] Failure to start kubelet client: open /etc/ssl/etcd/master1-master-client.pem: no such file or directory\n","stream":"stderr","time":"2016-02-27T20:22:58.702229507Z"}
{"log":"I0227 20:28:08.696638       1 plugins.go:71] No cloud provider specified.\n","stream":"stderr","time":"2016-02-27T20:28:08.69710352Z"}
{"log":"F0227 20:28:08.696778       1 server.go:416] Failure to start kubelet client: open /etc/ssl/etcd/master1-master-client.pem: no such file or directory\n","stream":"stderr","time":"2016-02-27T20:28:08.697163308Z"}

My pod spec looks as follows:
apiVersion: v1
kind: Pod
metadata:
  name: kube-apiserver
  namespace: kube-system
spec:
  hostNetwork: true
  containers:
  - name: kube-apiserver
    image: gcr.io/google_containers/hyperkube:v1.1.2
    command:
    - /hyperkube
    - apiserver
    - --bind-address=<IP_ADDRESS>
    - --insecure-bind-address=<IP_ADDRESS>
    - --etcd-config=/etc/kubernetes/etcd.client.conf
    - --allow-privileged=true
    - --service-cluster-ip-range=<IP_ADDRESS>/24
    - --secure-port=443
    - --advertise-address=<IP_ADDRESS>
    - --admission-control=NamespaceLifecycle,NamespaceExists,LimitRanger,SecurityContextDeny,ServiceAccount,ResourceQuota
    - --tls-cert-file=/etc/ssl/etcd/master1-master-client.pem
    - --tls-private-key-file=/etc/ssl/etcd/master1-master-client-key.pem
    - --client-ca-file=/etc/ssl/etcd/ca.pem
    - --kubelet-certificate-authority=/etc/ssl/etcd/ca.pem
    - --kubelet-client-certificate=/etc/ssl/etcd/master1-master-client.pem
    - --kubelet-client-key=/etc/ssl/etcd/master1-master-client-key.pem
    ports:
    - containerPort: 443
      hostPort: 443
      name: https
    - containerPort: 8080
      hostPort: 8080
      name: local
    volumeMounts:
    - mountPath: /etc/ssl/etcd
      name: ssl-certs-kubernetes
      readOnly: true
    - mountPath: /etc/kubernetes
      name: conf-kubernetes
      readOnly: true
    - mountPath: /etc/ssl/certs
      name: ssl-certs-host
      readOnly: true
  volumes:
  - hostPath:
    path: /etc/ssl/etcd
    name: ssl-certs-kubernetes
  - hostPath:
    path: /etc/kubernetes
    name: conf-kubernetes
  - hostPath:
    path: /usr/share/ca-certificates
    name: ssl-certs-host

I have also tried to convert the pod spec to a docker command, and that works perfectly:
docker run -p 443:443 -v /etc/kubernetes:/etc/kubernetes:ro -v /etc/ssl/etcd:/etc/ssl/etcd:ro gcr.io/google_containers/hyperkube:v1.1.2 /hyperkube apiserver --bind-address=<IP_ADDRESS> --insecure-bind-address=<IP_ADDRESS> --etcd-config=/etc/kubernetes/etcd.client.conf --allow-privileged=true --service-cluster-ip-range=<IP_ADDRESS>/24 --secure-port=443 --advertise-address=<IP_ADDRESS> --admission-control=NamespaceLifecycle,NamespaceExists,LimitRanger,SecurityContextDeny,ServiceAccount,ResourceQuota --kubelet-certificate-authority=/etc/ssl/etcd/ca.pem --kubelet-client-certificate=/etc/ssl/etcd/master1-master-client.pem --kubelet-client-key=/etc/ssl/etcd/master1-master-client-key.pem --client-ca-file=/etc/ssl/etcd/ca.pem --tls-cert-file=/etc/ssl/etcd/master1-master-client.pem --tls-private-key-file=/etc/ssl/etcd/master1-master-client-key.pem

Moving this to upstream repo.


add security workflow

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.332%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2126

Relevant Lines:
2521

💛  - Coveralls


Inference error: shape mismatch

Hi, I was testing live_asr.py on my macOS Monterey (Python=3.8.11) under the following environment:
halo==0.0.31
numpy==1.21.4
PyAudio==0.2.11
Rx==3.2.0
SoundFile==0.10.3.post1
torch==1.10.0
torchaudio==0.10.0
transformers==4.8.2
webrtcvad==2.0.10

When I run python live_asr.py, I came across errors as below:
Some weights of Wav2Vec2ForCTC were not initialized from the model checkpoint at facebook/wav2vec2-large-960h-lv60-self and are newly initialized: ['wav2vec2.masked_spec_embed']
You should probably TRAIN this model on a down-stream task to be able to use it for predictions and inference.

listening to your voice

/Users/jkang/anaconda3/envs/jk/lib/python3.7/site-packages/transformers/feature_extraction_utils.py:158: UserWarning: Creating a tensor from a list of numpy.ndarrays is extremely slow. Please consider converting the list to a single numpy.ndarray with numpy.array() before converting to a tensor. (Triggered internally at  ../torch/csrc/utils/tensor_new.cpp:201.)
  tensor = as_tensor(value)
/Users/jkang/anaconda3/envs/jk/lib/python3.7/site-packages/transformers/models/wav2vec2/modeling_wav2vec2.py:986: UserWarning: __floordiv__ is deprecated, and its behavior will change in a future version of pytorch. It currently rounds toward 0 (like the 'trunc' function NOT 'floor'). This results in incorrect rounding for negative values. To keep the current behavior, use torch.div(a, b, rounding_mode='trunc'), or for actual floor division, use torch.div(a, b, rounding_mode='floor').
  return (input_length - kernel_size) // stride + 1
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/Users/jkang/anaconda3/envs/jk/lib/python3.7/threading.py", line 926, in _bootstrap_inner
    self.run()
  File "/Users/jkang/anaconda3/envs/jk/lib/python3.7/threading.py", line 870, in run
    self._target(*self._args, **self._kwargs)
  File "/Users/jkang/Desktop/test_asr/wav2vec2-live/live_asr.py", line 92, in asr_process
    text = wave2vec_asr.buffer_to_text(float64_buffer).lower()
  File "/Users/jkang/Desktop/test_asr/wav2vec2-live/wav2vec2_inference.py", line 22, in buffer_to_text
    logits = self.model(inputs.input_values, attention_mask=torch.ones(len(inputs.input_values[0]))).logits
  File "/Users/jkang/anaconda3/envs/jk/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1120, in _call_impl
    result = forward_call(*input, **kwargs)
  File "/Users/jkang/anaconda3/envs/jk/lib/python3.7/site-packages/transformers/models/wav2vec2/modeling_wav2vec2.py", line 1528, in forward
    return_dict=return_dict,
  File "/Users/jkang/anaconda3/envs/jk/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1120, in _call_impl
    result = forward_call(*input, **kwargs)
  File "/Users/jkang/anaconda3/envs/jk/lib/python3.7/site-packages/transformers/models/wav2vec2/modeling_wav2vec2.py", line 1171, in forward
    return_dict=return_dict,
  File "/Users/jkang/anaconda3/envs/jk/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/Users/jkang/anaconda3/envs/jk/lib/python3.7/site-packages/transformers/models/wav2vec2/modeling_wav2vec2.py", line 791, in forward
    hidden_states[~attention_mask] = 0
IndexError: The shape of the mask [1920, 5] at index 0 does not match the shape of the indexed tensor [1, 5, 1024] at index 0

Exception in thread Thread-2:
Traceback (most recent call last):
  File "/Users/jkang/anaconda3/envs/jk/lib/python3.7/threading.py", line 926, in _bootstrap_inner
    self.run()
  File "/Users/jkang/anaconda3/envs/jk/lib/python3.7/threading.py", line 870, in run
    self._target(*self._args, **self._kwargs)
  File "/Users/jkang/Desktop/test_asr/wav2vec2-live/live_asr.py", line 68, in vad_process
    frame = stream.read(CHUNK)
  File "/Users/jkang/anaconda3/envs/jk/lib/python3.7/site-packages/pyaudio.py", line 608, in read
    return pa.read_stream(self._stream, num_frames, exception_on_overflow)
OSError: [Errno -9981] Input overflowed

I think the critical issue is this:
IndexError: The shape of the mask [1920, 5] at index 0 does not match the shape of the indexed tensor [1, 5, 1024] at index 0

I installed transformers==4.8.2, but the error occurs which is probably not related to the transformers' version in my guess.
Could you help me with what caused this error?
Thank you

Which model are you using?

I am having the same issue. I am using the model "facebook/wav2vec2-large-960h-lv60-self".

@oliverguhr I used the same model "facebook/wav2vec2-large-960h-lv60-self" as @KunwarK13 mentioned above.

Thank you for the feedback. I fixed bug #2 and was able to run the model with the latest dependencies installed.
Could you please check if it works for you?

Yes, it works! Thank you for taking your time to fix it @oliverguhr.

Great - you are welcome.


Parse pod for both app & extension

xcode 7.2
cocoapods 0.39.0
parse 1.10.0
I've got app & share extension as targets of the same project.
I'm trying to use
pod 'Parse'
for both targets.
Actual result:
Build fails with 'showAlertWithTitle:message:cancelButtonTitle:otherButtonTitles:completion:' is unavailable: not available on iOS (App Extension)"
Expected result:
it should compile and work
It seems the same parse target is shared between app & extension targets.
BTW some pods use subspecs to resolve such issues.

Yup, just found this.
Looks like a valid issue, sending a Pull Request with a fix in few minutes.


CHEBI:32135 wrong stereo config

CHEBI:32135 and CHEBI:67357 are derivatives, according to https://ncit.nci.nih.gov/ncitbrowser/ConceptReport.jsp?dictionary=NCI_Thesaurus&ns=NCI_Thesaurus&code=C26672. But they not only differ by a sidechain but also by the configuration of one stereo center in the middle. Could you please check?

Structure now updated. Thanks

Structure now updated. Thanks


Request: geocoded_location

Hi!
I would like to see the sensor.[car]_geocoded_location as part of the plugin. It would seem to be a possible , all nessesary data is present.
Keep up the good work!

I'm not sure that this will get incorporated into the code since there's already 3rd party solutions that does this better.

Hm, I thought this was a part of Hass core - to be able to convert lat/lan to address. I have this for the phones I have in Hass allready.

Hm, I thought this was a part of Hass core - to be able to convert lat/lan to address. I have this for the phones I have in Hass allready.

AFAIK it's the companion app that reports this to Home Assistant.


[DRAFT] 4.x - Split RouterInterface into RouteCollectorInterface and RouteResolverInterface

This is pull 2 out of 3 to complete the goals set in #2604

Split RouterInterface into RouteCollectorInterface and RouteResolverInterface

Rename RouteDispatcher to RouteRunner

Move all routing components into new Slim\Routing aggregate folder

[x] Discussion

[ ] Write Tests

Should the namespace be Slim\Router ?

Should the namespace be Slim\Router ?

I think Slim\Routing is better imo. A router would be a single component, routing is synonymous of all the components being used.

I think Slim\Routing is better imo. A router would be a single component, routing is synonymous of all the components being used.

The combination of the classes is the router. I don't care enough though, so Routing it is unless someone else has a compelling opinion.

Coverage decreased (-0.1%) to 98.366% when pulling a8306a39819e833316db1f27d29520d3c1227c18 on l0gicgate:4.x-SplitRouter into 821d5633ec526bb47b3c0479694ea600e0fa233b on slimphp:4.x.


Fix win accessibility

(in conjunction with commit 7c41f0e)
Context: In order for full accessibility support on Windows we need to remove the disable-legacy-window flag (commit 7c41f0e) as well as calling OnScreenReaderDetected at the appropriate time.
The appropriate time is when our window gets a WM_GETOBJECT request with an id of OBJID_CLIENT. This is the same way Chromium detects a request for the accessibility object. When we see that we simply toggle Chromium into accessibility mode, if it isn't already.

It has been committed at 4ce840d, thanks!

Awesome, thanks for this @ryanmolden :+1:

Thanks all!


Bug Report: Unable to add a proxy for an account on a network other than Polkadot, Kusama, or Westend, causing an error.

PROXY_TYPE for other networks are undefined.

@AMIRKHANEF  Can u provide an example of the chain on which this may happen?

@AMIRKHANEF Can u provide an example of the chain on which this may happen?

any chains other than Westend, Polkadot and Kusama.
scenario:
first switch your account on e.g. Acala,
then go to the manage proxy page
try to add a new proxy!

ah
I see
but why manage proxy is disabled for non relay chains on FL but available on extension Mode?

This issue has been around for a while, but we had other priorities to focus on.

can we simply disable the menu in th extension mode as well

PR #1236 will address this by disabling the 'Manage Proxy' item in the extension mode menu!


RF: Port DT fix for incorrect importance vector length

JIRA: MADLIB-1254
Commit 0f7834e contained a fix in DT that ensured that impurity variable
importance was of the correct length for each group if a single group
eliminated a categorical variable.
This commit applies the same fix for random forest.
Closes #299

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/598/


Add get_filepaths_in_directory() flag to ignore 'digest.filename' targets.

It may be convenient for repository maintainers to continually fetch all target files from particular directories and then add them with repository.targets.add_targets(), as opposed to only adding new target files explicitly.
One issue with blindly adding all target files in a directory multiple times occurs when a directory contains digest.filename target files, which should not be included as targets in metadata.
Support:
get_filepaths_in_directory(ignore_digest_filenames=True)

Stale


WASM Examples not loading on low end laptop

Bevy version
0.7
Relevant system information
AdapterInfo { name: "ANGLE (Intel, Intel(R) HD Graphics 4000 Direct3D11 vs_5_0 ps_5_0, D3D11)", vendor: 32902, device: 0, device_type: IntegratedGpu, backend: Gl }
What you did
I ran the wasm examples at https://bevyengine.org/examples/ to see how they would run on a older lower end device with no dedicated gpu (with hardware acceleration enabled in the browser)
What went wrong
Every example on the site https://bevyengine.org/examples/ would load (the loading bar) but remain basically empty with few exceptions where small bits and pieces may appear such as a single mesh or sprite but nothing else, not to mention the fps would be atrocious for anything that did render.
With hardware rendering disabled however the example would indeed appear but run at basically 5 fps which is obviously unacceptable and pretty much expected.
Additional information
I noticed the examples on https://github.com/mrk-its/bevy_webgl2 work perfectly and so do fyrox's wasm examples meaning the device is easily capable of running wasm webgl2 features with a reasonable fps just not the examples on https://bevyengine.org/examples/.
The console in the wasm examples are perfect in that they is relaying data with no issues and no delays; for example the stress test claims it is running at a stable 50 fps and even react to my input causing the fps to correctly drop.
I have however not compiled the wasm examples myself to check if the issue persists but I will do so in near future.

Just referencing my issue #4351 being the exact same as this issue (I have the same integrated GPU HD 4000)


Revert "moveit_msgs: 2.2.2-1 in 'humble/distribution.yaml' [bloom] (#37224)"

This reverts commit b9eb03ef0db05d22643de4fea9deb6e63c382e5b.
2.2.2 was a breaking change I should not have released for Humble.

This was ok to revert as it hadn't been synced yet.


Roll Packages from 87a02e393be0 to 5d951b0638ce (1 revision)

https://github.com/flutter/packages/compare/87a02e393be0...5d951b0638ce
2024-05-16<EMAIL_ADDRESS>[webview_flutter] Skip "Video playback policy" drive tests (flutter/packages#6747)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Hovering over the items of sidebar blacks out the written text.

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?

What changes are you suggesting?
While hovering over the items of sidebar the text colour should change to white.
Additional information
It works fine with the dark mode 👏

hhhhmmm yes good on you.
....,.,.,.,..,.,.,.,.,..,   ^@ hhhhhhhhhhmmmmmmmmm        ^^   ..,.,...
On Thu, Oct 13, 2022 at 12:17 PM welcome[bot] @.***>
wrote:

Thanks for opening this issue. A GitHub docs team member should be by to
give feedback soon. In the meantime, please check out the contributing
guidelines https://github.com/github/docs/blob/main/CONTRIBUTING.md.
—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/issues/21310#issuecomment-1277042802, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASXG5MM5KSCNBVACMM4DXADWC6LQFANCNFSM6AAAAAARD4WG3U
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

thank you in the informacion forme, addiyat. (+9 good ) texacion joob in
universcity colegy.

                    `FRENS GITHUB IS THEBESC " *

On Thu, Oct 13, 2022 at 12:17 PM Tushar Khatri @.***>
wrote:

Code of Conduct

I have read and agree to the GitHub Docs project's Code of Conduct
https://github.com/github/docs/blob/main/CODE_OF_CONDUCT.md

What article on docs.github.com is affected?
[image: Screenshot 2022-10-13 at 10 37 37 AM]
https://user-images.githubusercontent.com/64004539/195507182-7300f78a-022f-48ce-bbc3-d8ed373f6c66.png
What changes are you suggesting?
While hovering over the items of sidebar the text colour should change to
white.
Additional information
It works fine with the dark mode 👏
[image: Screenshot 2022-10-13 at 10 45 33 AM]
https://user-images.githubusercontent.com/64004539/195507764-0e1204e4-9e37-4e1d-b6c1-9dbde5ef3b08.png
—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/issues/21310, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASXG5MORLU4RZJZ33IL4SHDWC6LQHANCNFSM6AAAAAARD4WG3U
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@tusharkhatriofficial Thanks for opening an issue and reporting this! We are working on this internally, but I'll report your findings to the team.
I'm going to go ahead and close this as we are tracking it internally. 💛
Thank you for your passion and interest in improving GitHub Docs! ✨


German localization

Q
A

Bug fix?
[ ]

New feature?
[X ]

New sample?
[ ]

Related issues?
fixes #X, partially #Y, mentioned in #Z

As requested per issue #101 please find the german localization.
I updated the missing latest version values around Taxonomy and PeoplePicker as well.
Feel free to contact me once new values are needed.
Cheers
Markus ( https://twitter.com/Moeller2_0 )

Thank you very much @mmsharepoint, had to do a manual merge for this, but your change is in the codebase.


Mark known issue as resolved  - Visual context menus are cut off when…

… using Power BI Embedded

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

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

@mihart
Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Resolved in this PR:
https://github.com/MicrosoftDocs/powerbi-docs-pr/pull/8273


Merge nested classes for reports

When generating HTML reports and during verification, any nested classes are analyzed and displayed separately.
From the point of view of Kotlin code, this is not entirely correct, because these class-files are features of the JVM compiler implementation and do not reflect the real coverage of the Kotlin class.
It is necessary to merge coverage for the following nested classes:

companion object
lambda
anonymous object
local class

Example:
class NestedExample {
    companion object {
        // companion
    }

    private fun functionHandler(f: () -> String) {
        println("result ${f()}")
    }

    fun handle() {
        // local class
        class LocalClass { }

        // lambda
        functionHandler {
            "function"
        }

        // anonymous object
        object: Function<String> {
            override fun toString(): String = "Hi"
        }
    }
}

the coverage is evaluated separately for all classes, e.g.:
NestedExample line coverage: ..%
NestedExample$Companion line coverage: ..%
NestedExample$handle$1 line coverage: ..%
NestedExample$handle$2 line coverage: ..%
NestedExample$handle$LocalClass line coverage: ..%

Expected:
NestedExample line coverage: ..%

Important!
At the same time, regular nested classes and inner classes should continue to be counted separately:
class Nested2Example {
    class Foo {
    }

    inner class Bar {
    }
}

should give
Nested2Example line coverage: ..%
Nested2Example$Bar line coverage: ..%
Nested2Example$Foo line coverage: ..%

@shanshin Do you mean only HTML report?

@zuevmaxim
No, all types of reports: XML, HTML, verification, etc.

II. Companion
merge content of companion objects in containing class:
companion's methods renamed with companion_ prefix.
Example
class My {
  companion object {
    fun a() {
    }
  }

  fun a() {
  }
}

gives in XML
<class My>
  <method "companion_a">
  <method "a">
</>

It should be disabled by flag


Add login style

Current coverage is 86.58% (diff: 92.52%)

Merging #7 into master will increase coverage by 4.92%

@@             master         #7   diff @@
==========================================
  Files            13         16     +3
  Lines           469        671   +202
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            383        581   +198
- Misses           86         90     +4
  Partials          0          0

Powered by Codecov. Last update 53b3a43...4aff6dc


[block_pointers]: Enable test test_block_pointer.py on PVC

The upstream tests in test_block_pointer.py need to be ported to XPU. Support for blocked pointers is new and never exercised in our prototype, we might expose some functional features that nees to be implemented by attempting to run these tests.

Fixed by pull request https://github.com/intel/intel-xpu-backend-for-triton/pull/194

No functional issues have been exposed by this test.

Done


Request: is it possible to add a 4th level, local accounts?

Hi. Thank you very much for sharing this beautiful and simple tool. :)
I'd like to know how much effort (or concept breaking) would it be to add another level of privacy, inspired in a great feature of Hometown?
As you clearly state, today we have 3 privacy levels:

Public
To followers
To mentioned users

In Hometown, there is "community" between Public and followers, which means all accounts in the same instance. This allows the instance to become a campfire, a cozy web to gather and talk.
Be it possible or not, thank you very much for linking these worlds. :)
Best...

I started working on a new 'communities' feature (in https://github.com/dimkr/tootik/tree/hub), it allows the server admin to define special 'community' users that share posts (and replies) by following users with other followers. This should allow local users with a shared interest to discuss it in a separate 'space', and users on other servers can join the discussion by 'joining' the community.
(It's very similar to how Lemmy groups work.)

Seems great! It is not as a new publication level, but it could work for the same purpose.
Do they work as regular users/accounts?
For example, if the admin creates a "Caturday" community account, any community user types " @cat" and Caturday appears as any other account like "Catalina"? Or it is there any kind of control of the visibility of those accounts?
Other question: do a default community account that it is all local accounts is created for each instance?
Best... :)

Seems great! It is not as a new publication level, but it could work for the same purpose.

Exactly!

Do they work as regular users/accounts? For example, if the admin creates a "Caturday" community account, any community user types " @cat" and Caturday appears as any other account like "Catalina"? Or it is there any kind of control of the visibility of those accounts?

Yes, a community is just a special kind of user. They're "public" just like other users - you only need to know the username (and the instance domain) to find the community.

Other question: do a default community account that it is all local accounts is created for each instance?

Currently no, but I'm thinking about adding a special community named "local" that's automatically followed by new users.


Update Arrays_and_Loops.md

Fix the translation of Arrays_and_Loops.md

Translations need to be submitted on Crowdin.
https://crowdin.com/project/crafttweaker-documentation


BotDependencyConfiguration defines ExecutorService bean but its not in use

Is your feature request related to a problem? Please describe.
The problem i've stumbled upon is that i was getting com.fasterxml.jackson.databind.exc.InvalidDefinitionException: Java 8 date/time type `java.time.OffsetDateTime` not supported by default: add Module "com.fasterxml.jackson.datatype:jackson-datatype-jsr310" to enable handling (through reference chain: com.microsoft.bot.schema.Activity["timestamp"]) error for missing module that was already in the deps. The cause was that  ExecutorFactory#factory was using wrong classloader and module was not there. Had to do nasty reflection tricks to exchange the factory.
Describe the solution you'd like
I'd like that BotFrameworkAdapter utilize the ExecutorService spring bean. This could be done by expanding constructor of BotFrameworkAdapter to accept ExecutorService, and also other classes like BotFrameworkHttpAdapter AdapterWithErrorHandler

@przem Hi, may I ask how you swapped out the ExecutorFactory#factory. In my case, I could not load classpath resources from within the SpringBoot jar, because of the wrong class loader ExecutorFactory#factory uses. Love to find out the fix. Appreciate it!


Package is not installable via pip

pip install mailersend gives

ERROR: Could not find a version that satisfies the requirement mailersend (from versions: none)
ERROR: No matching distribution found for mailersend

python version: 3.7.8
pip version: 21.1.2

I was able to install mailersend on a fresh environment both with python 2.7 and 3.8, not sure what could be wrong here.

Found it: package mailersend is available for versions >=3.8, <4.0. Reference in PyPi.

Hm, curious that this is not available for 3.7, wonder what features prevent from supporting 3.7?


ENH: Split signal.lti into subclasses, part of #2912

new classes: ss, tf, zpk

backwards compatible
Remove redundant information in lti
Make internal transformations more transparent

Link to issue: gh-2912
Related issues: gh-2973, gh-3259
Mailing list discussion: http://thread.gmane.org/gmane.comp.python.scientific.devel/19427

@befelix this PR now has a merge conflict, can you rebase it?

Seems mostly backwards compatible. I found one issue so far, related to the setters and the removal of _update():
>>> l = signal.lti([2.], [3.], [5], [6])
>>> l.C = 1   # used to work, now you get a hard to understand error
...
AttributeError: 'int' object has no attribute 'shape'

Is the added copy method useful? It doesn't do anything that can't be achieved by:
import copy
copy.copy(other_instance)

as far as I can tell.

I've rebased the code, should make it easier to review.
I'm a bit busy this week, but I'll try to get through all your recommendations by next weekend.
The regression with l.C seems very weird, especially since that's something I've definitely tested before making the PR. I'll have a look!
Just for completeness here are your comments from the mailing list:
"""

I'd like the classes to have more Pythonic names, instead of too short names that are taken from Matlan
The conversion functions should probably be named "to_zpk, to_ss, "to_tf" rather than "zpk, tf, ss".
There's no SOS class, we do have that format now (https://github.com/scipy/scipy/pull/3717)
Warren suggested "lti.from_tf/zpk/ss/sos" constructors on issue 3259. Looks like that should fit with your PR, but good to think about now.
I think that this fits with where we want to go with input to filter functions (gh-3259), but ideally we'd work that out together with this PR (not the implementation, but at least a sketch of the new API).

"""
I agree about the pythonic names, would make code using these classes super readable. Sos wasn't around when I wrote the PR, I'll check it out.

Hmm, SOS was merged in January so was in the code already, but I guess it hadn't made its way into http://docs.scipy.org/doc/scipy-dev/reference/signal.html by then. Double checked that it's included now.

Not sure what the best names would be by the way. StateSpace, TransferFunction, ZerosPolesGain? Especially the latter isn't very nice.

Or SsSystem, ZpkSystem, TfSystem?

The copy method is mostly there to support the getters/setters for objects from different lti subclasses in a central location (i.e. the lti master class), rather than cluttering the subclasses with all the unrelated getters and setters (the code gets a bit ugly otherwise).
Consider the example of
obj = ss(1,1,1,1)
obj.poles = 1

What happens in the code is that the setter for poles in the lti class converts our object to the tf class tf_obj = obj.tf(), changes the poles, and the converts it back obj2 = tf_obj.ss(). The problem is that we now have a new object (obj2) that is different from obj, but should replace it. I don't think copy can handle that (a method of an object can't change the object itself, I think). So what the copy method in the subclasses does it copy all the relevant attributes of obj2 to obj.
This is the best solution I came up with, though on second thought copy() should probably a private method. Any ideas how to make this prettier are appreciated. If you would prefer to have all the setters/getters in the subclasses then it's possible to remove the copy function.

Regression with l.C = 1 is fixed. Thanks!
Regarding the names: I can't think of a very pretty solution either. Maybe ss/tf/zpk is not that bad after all :/.
Also changing ss to SteadyState seems a bit weird if lti is not renamed to LinearTimeInvariant.

I prefer capital letters for class names, whether it's SS(...) or Zpk(...) or so. I assume there isn't a scipy-wide standard...?

Sorry I haven't been following this.  So overall, this will still behave like an LTI class that stores whatever coefficients you initialized it with, and can output every other form when asked?

@Eric89GXL Yeah I'm confused about the standard as well. Officially it's capital letters for classes I think, but then that doesn't really seem to be follows. The original lti class was already lower case, so at least in the short term there's probably nothing to do about that.
@endolith The original LTI class had attributes for all systems (ss,zpk,tf). Changing one of the elements caused an _update() function to be called, which updated all other atrributes. This seemed a bit messy, so now there's seperate classes for it. However the old workflow/api still works. The difference is that with this PR you don't actually get an object of the lti class anymore. Calling lti(..) automatically returns an object of the appropriate subclass (ss, tf, zpk). So from the outside everything behaves like the old lti class, but inside it's all new and improved.

The standard is simply PEP8, however in the past it hasn't always been followed (Scipy itself is as old as the first draft of PEP8). That's especially true for scipy.signal, which is the only module with an API that has been largely modeled on Matlab API - something we want to get away from.
So proposal: let's follow PEP8 here, no matter whether we choose ZerosPolesGain, ZpkSystem, Zpk or yet something else.

+1 for following PEP8 from me, might as well start moving in that direction

I agree that PEP8 would be great. My question would be how far we can go with that given backwards compatibility and wether it makes sense to fix only parts of those.:

lti is lower case
the system matrices are upper case
most functions in the ltisys.py file take upper case kwargs

If we can't change those maybe it's better to use the wrong style, but consistently?

@rgommers Regarding the SOS class. There is some initial code in https://github.com/befelix/scipy/tree/sos_class to add them to to the lti class. However, this causes a bunch of problems, since there is no conversion function sos<-->ss (for example impulse and step rely on this for default response times). Is there a reason for that? Is it 'okay' to just call something like sos_sys.to_zpk().to_ss()? Maybe this functionality and the according discussion should be in a separate PR?

So as far as I can tell the PR should be okay now code-wise, leaving only the questions of the SOS system and the API (i.e. naming and what should the inputs to ss/tf/zpk be: Currently it's just *args. Maybe it should be more like proposed in https://github.com/scipy/scipy/issues/3259, where they take one input, which can be a touple of parameters or an lti system?
Regarding the naming of variables I'm not sure and some input would be very welcome. Options seem to be

SteadyState / TransferFunction / ZerosPolesGain / SecondOrderSections (like Maple)
SteadyStateSystem / TransferFunctionSystem / ZerosPolesGainSystem / SecondOrderSectionsSystem or Model (Mathematica)
Ss / Tf / Zpk / Sos (Matlab/Octave)
SsSystem / TfSystem / ZpkSystem / SosSystem

I'm not very keen on the last one; mostly since SsSystem looks weird to me. I think the first one is probably the best balance of readability and length. Given that these functions are in scipy.signal it should be clear that those are models. In any case, what also needs to be decided if the likes of obj.to_tf() should then be renamed to obj.to_transfer_function() as well.
Regarding the StateSpace system matrices, we could maybe have both capital- (for backwards compatiblity) and lower case letters (for consistency)?
One last thought: I think in the long-run it would make sense to depreciate the cross-subclasses setters (e.g. setting the poles on a StateSpace system). They encourage extremely bad behaviour, require code that is not nice, and are, imho, not very useful.

SteadyState / TransferFunction / ZerosPolesGain / SecondOrderSections (like Maple)

Shouldn't that be StateSpace?

Yes, yes it should be :)

+1 for Ss / Tf / Zpk / Sos from me -- it seems quick and easy. Long names make my fingers tired and my PEP8 code more wrappy. My +0.5 for the maple versions. But I'm really okay with any of those.
I think it should be okay to have sos_sys.to_zpk().to_ss() going on under the hood if one does sos_sys.to_ss(), at least for a first pass. In my head sos is only one small step away from a zpk representation, where that step is chosen to be easily implemented as an IIR filter that minimizes numerical problems. @endolith WDYT?
+1 for a single input argument instead of *args. It seems to lend itself to easier interpretation and implementation in my head, but I haven't thought about it extensively.

For the names, how are these intended to be used?  If you're only ever going to create objects using a generic Lti or Filter name, and it returns one of these classes, does it matter that the names are long?  Explicit names are generally more clear than abbreviations.
What I'm imagining is like this:
my_lpf = butter(5, 0.2, output='Filter')
output = my_lpf(input)

where input and output are ndarrays, and it would automatically generate the filter using zpk coefficients, return my_lpf, which is an instance of some class that internally uses zpk coefficients, then the instance is callable, which automatically creates an SOS and filters the input signal using sosfilt().  The filter instance could also output any other representation, like my_lpf.to_ss(), calculate frequency response like my_lpf.freqresp(w), etc.

In my head sos is only one small step away from a zpk representation

I'm not sure, I was reading about this but don't fully understand it: http://www.bisonacademy.com/ECE461/Lectures/06 State Space.pdf  Controller and observer canonical forms are made from the tf coefficients, Jordan form is made from the partial fraction expansion coefficients (like scipy.signal.residuez?), and cascade form is similar in concept to sos but uses the zpk coefficients along the diagonal?  But maybe it's only for all-pole systems?
Also remember that SS supports MIMO systems and the others don't, etc.

You saw that I was talking about second-order sections and ZPK being a small step apart, not state space and ZPK, right? It seems like you might have misunderstood based on your response, but maybe I misunderstood...
In any case, my point is that if you can imagine getting easily (and losslessly) from SOS to ZPK and back, then I don't see a problem with doing SOS->ZPK->SS if that's an easy way to get from SOS to SS. I think if we have a system in SOS form we should be able to calculate the TF coefficients, PFE residues, or (trivially) ZPK coefficients if we first transform the system back to ZPK. So maybe you can always go SOS->SS via ZPK, but only go SS->SOS if it's not a single-in-single-out system...?

For the names, how are these intended to be used? If you're only ever going to create objects using a generic Lti or Filter name, and it returns one of these classes, does it matter that the names are long? Explicit names are generally more clear than abbreviations.

I guess it depends on your use case. I have a lot of background in Matlab and would be inclined to use the classes directly rather than lti (even though it's super nice to have one class to call for everything). However, typically these classes are called super rarely: Create your system once and then do a lot of analysis and design on top. So even though Ss/Zpk/Tf seem snappy once you're used to it, I think the goal of python code is usually to be super readable, without necessarily having to read the docs for every class / function call. I think this would be true for something like StateSpace(...), but not necessarily for Ss(...). On the other hand most people would know the abbreviations anyways, since most functions seem to use them. So I'm not sure if there is a 'best' solution.

Could we decide on an API? From my side this PR is finished except for this last issue (sos system will be a followup PR since there are different issues to discuss) and I don't want to just decide this on my own :)
Anyways, it would be great if this could make it into the next release.

I don't have a strong feeling, I'm happy to defer to @rgommers :)

So I made some final minor changes and renamed the classes. From my point of view the API is good now, so any final comments or merging the PR would be appreciated. There is a bunch of follow ups that I want to do that depend on this one :)

Before the PR can be merged, you should probably add tests of all the new lines that have been added.

The tests issue and my minor comments, LGTM

Alright, that should wrap up all the of @Eric89GXL's comments. Is there anything else that needs to change?
Also I rebased the code on master because of a merge conflict with the recent lsim rewrite. I hope that's okay.

@befelix rebasing is our preferred way of fixing merge conflict, so this looks good.

I just quickly browsed over the code again, and quite like the look of this API now.

Is there any hope of getting this merged before the 0.16 branching?

@befelix I think it's possible, but I'm kind of running out of time. @Eric89GXL can you verify that all your comments are addressed and this is okay to merge for you? Then I'm happy to merge this after a quick last sanity check.

Other than my minor complains, LGTM.

Alright, thanks for the last comments, they're all implemented.
@Eric89GXL: *args are not actually handled consistently in scipy, but I think your *system: arguments suggestion is good.

Okay, in it goes. Thanks @befelix!

And thanks @Eric89GXL and @endolith for reviewing.

Thanks again everyone. In case you feel like looking through even more subclasses, there is a follow-up PR at #4881, which extends this framework to discrete-time systems. It would be great if you could give comments on that as well!


Meta issue: Docs for how to deploy to gh-pages

Not related directly to Angular, but useful for any Dart for Web application.

This isn't the most elegant way of doing it but it works. I have this shell script in the root of the leeds.dart angular2 site:
# Clone gh-pages
git clone git@github.com:leeds-dart/leeds-dart.github.io.git gh-pages
pushd gh-pages
git checkout -t origin/master

# Out with the old in with the new
rm -rf *
# Copy build
cp -r ../build/web/* .

# Add and commit
git add -A
git commit -m "Update build files"
git push
popd

rm -rf gh-pages

There's also peanut: https://pub.dartlang.org/packages/peanut

I don't think we need specific documentation here.


Promise interface on the device object

I tried your API using ParticleJS and everything worked very well. Congratulations on your product!
But one thing that I found missing is the fact that the methods in the device object don't return promises. The docs say "You can interact with the api using callbacks, events or promises", but promises seem to be available only for method in the spark object.
Is this an intentional API design, or something that not implemented yet? It makes the current API a bit incoherent.

Was there a specific method you were trying to use? The device object just proxies to the main API module for most operations which supports promises. This could just be a bug.

It looks like #81 might solve your problem.


chore(FocusZone): Update FocusZone changelog

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS


om version 3.2.0 or up failed in 'om staged-config', got 404 error

could not execute "staged-config": request failed: unexpected response:
HTTP/1.1 404 Not Found
Connection: keep-alive
Content-Type: text/html; charset=UTF-8
Date: Thu, 29 Aug 2019 04:50:37 GMT
Server: nginx/1.10.3 (Ubuntu)
Strict-Transport-Security: max-age=15552000; includeSubDomains
X-Request-Id: df5b362a-df86-46e0-84c3-c8f0312a793a
X-Runtime: 0.001158
Content-Length: 0

We are also seeing this error with 3.2.1

Hi @weaverb !
What version of Ops Manager are you on? We ran a test of this on 2.3.25 and got the following response:
GET /api/v0/staged/products/cf-79fba6887e8c29375eb7/syslog_configuration HTTP/1.1
Content-Type: application/json

HTTP/1.1 422 Unprocessable Entity
Transfer-Encoding: chunked
Cache-Control: no-cache, no-store
Connection: keep-alive
Content-Type: application/json; charset=utf-8
Date: Thu, 29 Aug 2019 22:18:18 GMT
Expires: Fri, 01 Jan 1990 00:00:00 GMT
Pragma: no-cache
Server: Ops Manager
Strict-Transport-Security: max-age=15552000; includeSubDomains
X-Content-Type-Options: nosniff
X-Frame-Options: SAMEORIGIN
X-Request-Id: 657a651c-7ae0-44db-9b45-aa1884199c4a
X-Runtime: 0.466983
X-Xss-Protection: 1; mode=block

114
{"errors":{"syslog_configuration":["This product does not support the Ops Manager consistent syslog configuration feature. If the product supports custom syslog configuration, those properties can be set via the /api/v0/staged/products/:product_guid/properties endpoint.\n"]}}
0

however, om takes this response and the error and will continue without actual failure (the only way we could check for the existence of the endpoint was to hit it. If the endpoint is not found, we continue without error) during the staged-config, and we were able to get the configuration successfully.
We are going to try this with the dataflow tile to see if there is something specific with that tile we may be missing.

this will be fixed in om 3.2.2 later today. Thank you for bringing this to our attention!

confirm om-3.2.2 fixed this issue.
Thanks!


Update Ninja and JDK

The version of Ninja used in Posters is rather dated, so we should update it. As a side effect, we could enable Ninja's SuperDev mode. Right now, 6.9.0 is the latest.
Update the JDK dependencies to JDK 17 as requirement (if Ninja supports it).

Excellent. Thank you.


Multi-argument support: basic infrastructure

Codecov Report
Attention: Patch coverage is 26.43678% with 192 lines in your changes missing coverage. Please review.

Project coverage is 11.89%. Comparing base (b3c9b0a) to head (e40858c).

Files with missing lines
Patch %
Lines

...ifferentiationInterface/src/fallbacks/no_extras.jl
41.37%
34 Missing :warning:

DifferentiationInterface/src/second_order/hvp.jl
0.00%
20 Missing :warning:

...ifferentiationInterface/src/misc/from_primitive.jl
0.00%
19 Missing :warning:

DifferentiationInterface/src/misc/zero_backends.jl
0.00%
16 Missing :warning:

...ationInterfaceSparseMatrixColoringsExt/jacobian.jl
0.00%
15 Missing :warning:

...fferentiationInterface/src/first_order/pullback.jl
45.83%
13 Missing :warning:

...rentiationInterface/src/first_order/pushforward.jl
52.17%
11 Missing :warning:

...fferentiationInterface/src/second_order/hessian.jl
0.00%
11 Missing :warning:

...fferentiationInterface/src/first_order/jacobian.jl
57.14%
9 Missing :warning:

...ionInterface/src/second_order/second_derivative.jl
0.00%
8 Missing :warning:

... and 7 more

:exclamation:  There is a different number of reports uploaded between BASE (b3c9b0a) and HEAD (e40858c). Click for more details.
HEAD has 49 uploads less than BASE

Flag
BASE (b3c9b0a)
HEAD (e40858c)

DI
41
2

DIT
10
0

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #461       +/-   ##
===========================================
- Coverage   98.53%   11.89%   -86.65%
===========================================
  Files         106       72       -34
  Lines        4234     2682     -1552
===========================================
- Hits         4172      319     -3853
- Misses         62     2363     +2301

Flag
Coverage Δ

DI
11.89% <26.43%> (-86.80%)
:arrow_down:

DIT
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Concat + minify

Fixes #51

Pushed up some changes, I've streamlined the task registration so build:uglify merely runs build, then uglify.


[Bug] BackDrop not working on version @0.0.30

Hi there,
It is me again. I'm now using the new version of your package. It is so nice, However the BackDrop doesn't work correctly. It only works on the first render. After that it doesn't
Thank you so much

@anhquan291 can you check 0.0.31?

@anhquan291 can you check 0.0.31?

It works perfectly on 0.0.31. Thank you so much


CI to Publish Our PyPi Server

CI to Publish Our PyPi Server on all merges/pushes to main so that we dont need to manually build and upload every time.

Github Action
https://pythonprogramming.org/automatically-building-python-package-using-github-actions/

cc: @jimwu6 @choicallum

This is addressed in https://github.com/cohere-ai/cohere-python/pull/49


Fix serialization of SearchResult

As pointed out by @Splamy (via discord) the official documentation states that Tags (and Authors) can be

string or array of strings

This PR fixes that for Tags (including tests) and also adds the tests for Authors.
Also I noticed false positives in the tests for PackageMetadata (my fault) which I fixed.

Thanks! Lets merge this after the version/stats stuff.


Improvement/assets folder delete

Deleted all the duplicates and wrong cased folders, added them back with correct casing and non-duplicate folders/files.

Lead engine dev be like:


Bind remote hosts without project folder.

I have found it quite messing to use this package using folder projects.
Is it somehow possible to use a similar approach as https://atom.io/packages/remote-edit does in order to add and edit hosts using a GUI as well as storing the hosts outside of a folder?

Mentioned in issue #80, this package is meat to be used with local-remote structure.


Remove AGL patches

Some of them might be necessary to support smack, but meta-updater
doesn't include smack support anyway, so they'd better been applied by
external layers. Others disable framebuffer support which is really
annoying for users.
Should fix https://github.com/advancedtelematic/meta-updater/issues/301

I would say mere misunderstanding. meta-updater was originally part of AGL code base and as we worked on it, some supporting code also landed outside of this layer, just because of how AGL manages hardware platforms. As we moved towards independency from AGL code base meta-updater-* layers were created to contain these pieces of suppoting code (it couldn't be in meta-updater itself, because it meta-updater has to be compatible with AGL codebase). But sometimes, like in this case, the scope of responsibility of meta-updater-* was determined incorrectly.


Add documentation on ClassVar support

Initial Checks

[X] I have searched GitHub for a duplicate issue and I'm sure this is something new
[X] I have searched Google & StackOverflow for a solution and couldn't find anything
[X] I have read and followed the docs and still think this is a bug
[X] I am confident that the issue is with pydantic (not my code, or another library in the ecosystem like FastAPI or mypy)

Description
Hey!
I was looking on how to properly add Class Variable to a pydantic model and it took me some effort to find a solution for that. It turned out that Pydantic supports ClassVar[T] notation for class variables but it is not described anywhere apart of the issues.
That support was added here in that PR:
https://github.com/pydantic/pydantic/pull/339
and the reason for not adding that to documentation is:
I guess if we were perfect we'd add a section to documentation on this, but for now I can't be bothered.
https://github.com/pydantic/pydantic/pull/339#issuecomment-450227723
I would like to add that to the documentation but since big changes to pydantic are on the way I prefer to ask first for a green light before putting any effort for that.
So, should I add it to the documentation?
Example Code
No response
Python, Pydantic & OS Version
Macos, py3.9, pydantic 1.10.2

Affected Components

[ ] Compatibility between releases
[ ] Data validation/parsing
[ ] Data serialization - .dict() and .json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[ ] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[ ] Other Model behaviour - construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

https://docs.pydantic.dev/latest/concepts/models/#class-vars, marking as done :)


[#5384] Cleanup code related to cluster

Create a StreamChannel interface to enhance scalability.

Codecov Report

Merging #5478 into master will decrease coverage by 0.05%.
The diff coverage is 78.83%.

@@            Coverage Diff            @@
##           master   #5478      +/-   ##
=========================================
- Coverage   41.16%   41.1%   -0.06%
=========================================
  Files        2777    2777
  Lines       83920   83924       +4
  Branches    11175   11175
=========================================
- Hits        34542   34500      -42
- Misses      46362   46409      +47
+ Partials     3016    3015       -1

Impacted Files
Coverage Δ

...corp/pinpoint/rpc/stream/StreamChannelManager.java
63.95% <100%> (-6.98%)
:arrow_down:

...orp/pinpoint/rpc/stream/AbstractStreamChannel.java
76.13% <76.13%> (ø)

.../pinpoint/rpc/stream/NettyClientStreamChannel.java
81.25% <81.25%> (ø)

...rcorp/pinpoint/rpc/stream/ServerStreamChannel.java
93.54% <87.5%> (+0.95%)
:arrow_up:

...oint/rpc/packet/stream/StreamCreateFailPacket.java
9.37% <0%> (-78.13%)
:arrow_down:

...uster/zookeeper/exception/ConnectionException.java
0% <0%> (-28.58%)
:arrow_down:

.../pinpoint/rpc/packet/stream/BasicStreamPacket.java
75% <0%> (-25%)
:arrow_down:

.../cluster/zookeeper/ZookeeperExceptionResolver.java
17.24% <0%> (-17.25%)
:arrow_down:

...pc/server/LoggingServerMessageListenerFactory.java
90% <0%> (-10%)
:arrow_down:

...ercorp/pinpoint/rpc/stream/StreamChannelState.java
91.3% <0%> (-8.7%)
:arrow_down:

... and 11 more

Continue to review full report at Codecov.

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


Add custom Nextstrain footer

This modifies the footer to mimic the current Nextstrain (docs) footer.
The content and styles are largely taken from
https://github.com/nextstrain/nextstrain.org/blob/b1e09e57e91ed0c9343e1cd3104877ec3c5344a4/static-site/src/components/Footer/index.jsx
It is not identical -- the RTD footer doesn't fill the available horizontal
space for example -- but it's close.
It's largely written in plain HTML + CSS, as opposed to the source material
which was JSX + Styled Components. There may be a much nicer way to write
this using Jinja functionality.

P.S. We can restore the footer to occupy all the available horizontal space without too much trouble, although there are still some tweaks necessary around the buttons etc:
diff --git a/lib/nextstrain/sphinx/theme/static/css/nextstrain.css b/lib/nextstrain/sphinx/theme/static/css/nextstrain.css
index f4ae133..5a2034c 100644
--- a/lib/nextstrain/sphinx/theme/static/css/nextstrain.css
+++ b/lib/nextstrain/sphinx/theme/static/css/nextstrain.css
@@ -74,7 +74,16 @@ footer span.commit code,
 /* main content section */
 .wy-nav-content {
   background: var(--content-background-color);
+  max-width: inherit; /* we'll set this on a child instead so it doesn't affect the footer */
 }
+div.rst-content > div.document {
+  display: flex;
+  justify-content: center;
+}
+div.rst-content > div.document > div {
+  max-width: 800px;
+}
+

Does this make the footer wider than the main content at times? Or does the whole page adjust to the available horizontal space?

This PR keeps the RTD-theme behavior, where the footer width is the same as the "main docs content" width (as in above screenshot). The patch in my above comment would change this a bit, however I'll leave that for a separate PR as suggested.


Extend depth by 2 in case of checks

Lots of test failing, WAC failing 50%,
Score of Lynx-extend-depth-further-on-checks-1860-win-x64 vs Lynx 1863 - main: 1 - 32 - 11  [0.148] 44
...      Lynx-extend-depth-further-on-checks-1860-win-x64 playing White: 1 - 14 - 7  [0.205] 22
...      Lynx-extend-depth-further-on-checks-1860-win-x64 playing Black: 0 - 18 - 4  [0.091] 22
...      White vs Black: 19 - 14 - 11  [0.557] 44
Elo difference: -304.4 +/- 111.3, LOS: 0.0 %, DrawRatio: 25.0 %
SPRT: llr -0.996 (-34.4%), lbound -2.25, ubound 2.89

It was only an idea, but OMG


[ENH] Refactor series_length and n_instances to n_timepoints and n_cases in dictionary based classifiers

Describe the feature or idea you want to propose
some time ago we agreed on calling the shape of collections of time series (n_cases, n_channels, n_timepoints). Some classes still use n_instances and series_length, which is fine, but we would like to standardise.
For example, the dictionary based classifiers all use n_instances and series_length internally.
Describe your proposed solution
Following files contain n_instances and/or series_length . Refactor for n_cases and n_timepoints. Do one or more in one PR

[ ] _boss.py
[ ] _cboss.py
[ ] _muse.py
[ ] _weasel.py
[ ] _tde.py
[ ] _weasel_v2.py

Describe alternatives you've considered, if relevant
No response
Additional context
No response

Hi, I'm new to aeon and would like to start contributing with this issue.
To clarify, l should update the code so that the shapes of collections is internally represented as n_cases instead of n_instances and n_timepoints instead of series_length within the dictionary classifiers' files. This modification should also be reflected in the comments.

Hi @Vedant222 welcome to aeon !
You understood it right, the goal of this issue is to update the documentation related to the dictionnary based classifier.
If you have any question about how to setup your dev environnement, the documentation should help you, otherwise you can ask any question on our Slack channel.


Automated cherry pick of #3372: Move has-changes Github Action to is own repo
#3445: Remove old has-changes Github Action code

Cherry pick of #3372 #3445 on release-1.5.
#3372: Move has-changes Github Action to is own repo
#3445: Remove old has-changes Github Action code
For details on the cherry pick process, see the cherry pick requests page.

Codecov Report

Merging #3459 (448071c) into release-1.5 (28b499c) will decrease coverage by 16.36%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           release-1.5    #3459       +/-   ##
================================================
- Coverage        57.84%   41.48%   -16.37%
================================================
  Files              332      200      -132
  Lines            28513    24147     -4366
================================================
- Hits             16494    10017     -6477
- Misses           10223    13099     +2876
+ Partials          1796     1031      -765

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
41.48% <ø> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.


Allow redirect when fetching supergraph in router

Hello,
We are using a self hosted Hive that is behind a proxy, so request are redirected, but we realised that the router package does not allow redirect by default. Would it be possible to allow redirect when fetching the supergraph ?
https://github.com/kamilkisiela/graphql-hive/blob/1932b9ba491b58515d5ca407c89e5b0b7433c442/packages/libraries/router/src/registry.rs#L123

I'm also up to make a PR for it if needed :)

Hello,
we actually managed to fix the issue which was due to something else. We had to set a different S3_PUBLIC_URL than localhost to have the same domain name used for fetching the artifacts. I'll close the ticket as it was not an issue at the end.


Wordwrap filenames onto new line in icon view

What's the Problem?
Long filenames are always cut off in icon view in Files but are often fully shown in Windows Explorer.
Solution/Idea
Wordwrap the filename onto multiple lines.
Alternatives
Forced to use details view or switch back to Windows Explorer.
Priorities
No response
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22000.348
Windows Version
Edition	Windows 11 Pro Version	21H2 Installed on	10/11/2021 OS build	22000.348 Experience	Windows Feature Experience Pack 1000.22000.348.0
Additional comment
No response

Merging with #1033.


Job stuck in queue when exception is thrown in the job start callback

Hello, if any exeption arise in the callback of the job save method, and the main process is killed because of this exception, the all the jobs will be stuck in the "queued" state, and when restarting the application and connecting workers, these queued jobs won't be processed.
var job = queue.create('Some job', {
          title: 'Some title'
      }).attempts(3).save( function(err){
        //SOME CODE THROWING ERROR HERE
        cb(err)
      });

Any ideas how to fix this issue?

Note : this will effectively solve the problem, but maybe I should not do that or I am not aware of the possible impact :
    // Recovered jobs stucks in inactive (queued state)
    queue.inactive(function(e,ids){
      ids.forEach( function( id ) {
        kue.Job.get( id, function( err, job ) {
          // Your application should check if job is a stuck one
          job.inactive();
        });
      });
    });

Where is the Kue hell author? Reply this question


Define sound and unsound

Cc https://github.com/rust-lang/unsafe-code-guidelines/pull/190

@Centril okay I think I resolved your concerns.


feat(typescript): accept property arrays for fallbacks

What:
Adding typing for styles to accept arrays
Why:
As reported in #259 glamor accepts array values and uses them to create fallback values
https://github.com/threepointone/glamor/blob/00349761d9ee56c781dff448b3a9979e3d05e042/docs/howto.md#fallback-values
How:
Adding typings and tests.
Checklist:

[ ] Documentation N/A
[x] Tests
[x] Ready to be merged
[x] Added myself to contributors table

Codecov Report

Merging #260 into next will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##           next   #260   +/-   ##
===================================
  Coverage   100%   100%
===================================
  Files        10     10
  Lines       159    159
  Branches     43     43
===================================
  Hits        159    159

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 207f01c...618dd38. Read the comment docs.


UX: easy to create a push-mode consumer pushing to subject pull

Someone using this nats CLI tried to create a pull-mode consumer on a stream, and when asked for the Delivery target they typed in the word pull.
Hey presto, a push-mode consumer pushing to NATS subject pull.
The only currently supported way of entering this, AFAIK, is to instead just hit enter for an empty target.  I only "remembered" this because when prompted for the target, I'd entered ? and read the help text.
Perhaps we should have a case-insensitive comparison of the string against pull and insert a "hey, are you sure?" question; or, if we don't want to affect scriptability here, just throw up a big "WARNING: foo" message in this scenario.

I am not really a fan of the confirmation approach, another way is to add —pull and it won’t ask the subject.
Alternatively I can expand the question text with like (empty for pull consumers) or similar.

Avoiding the confirmation: sounds good.  The issue with --pull is discoverability of UX.
Expanding the question text might help.
I think that detecting pull as a push topic is worth doing, and just putting up WARNING: you have chosen to push to a topic named 'pull'; if not desired outcome, try again with '--pull' or leave topic empty before showing the next question.

Closed by https://github.com/nats-io/natscli/pull/79


Classification logic

Added files to be able to use the pipeline with the classification logic:

classificcation logic - the logic routine with the model of classification
vis logic classification - the vis logic with a different drawing method (using cv2 to put text)
meta and frame from redis classification - we changed the way we receive the pred message to "receive" instead of "read most recent msg"
classification conf - the yaml file that creates the components we need
'
added to the "is empty" function in message the if the data is "".

@ItayHoresh can you merge the pr?


Add a way to quickly check out all the keyboard shortcuts

We have a lot of keyboard shortcuts already—many of which are context-specific—and that number is only going to skyrocket. We need a UI element (and command bar command) that shows the user a searchable view of all the keyboard shortcuts.
Figma has a great one.  It can be accessed by hitting Ctrl + Shift + ?, or clicking this question mark button in the lower right corner...

...and selecting the "Keyboard shortcuts" menu item...

The Keyboard Shortcuts pane then appears, which has a tabbed interface broken out by shortcut category. One neat feature they have is that every shortcut you've done is highlighted in blue, and updates live with a pleasing, victorious-feeling animation whenever you perform a shortcut for the first time. A pretty pleasant endorphin hit to encourage learning the shortcuts.

I'm preparing to implement this now, and I actually think that VS Code and Obsidian are going to provide a bit more inspiration for our keyboard shortcut browsing experience. Figma's example is not searchable, doesn't concern itself with scopes, and saves data on if the user performed that action which is neat but unnecessary (and unreliable: I looked back recently and a bunch of my shortcuts are showing up as "never performed" event though they're performed in the screenshots above).


Codegen refactor

Still left:

Summarize architecture and changes here in PR
Packing fields that require offsets (String, Vector, Object, Struct)
Respect namespaces in .fbs files

I don't want to discourage you since I know this isn't complete. But this PR seems like more of a different way of doing things than a solution so far.

Serious issues
1. `--no-gen-object-api` still imports object API `...T` types and `Unpack` functions

this (admittedly broken) option has been removed, not fixed.

Property name collisions with unions can lead to generated code calling non-existent functions. ...

i'm not sure if this has been addressed yet.
i would encourage you to try and focus on fixing problems by making minimal changes to the codebase.  then the codegen rewrite can be addressed in another PR.  as-is this PR mixes a rewrite with fixing api issues.  it is just too much for me to review and feel comfortable maintaining.

This PR will close ALL the issues in #3. Property name collisions have been solved by only offering unpacking the entire union type instead of it's tag and union value separately. Again, this is at the same cost as slices.

i would encourage you to try and focus on fixing problems by making minimal changes to the codebase.

I want to fix more problems than what I've listed. I considered surgically changing the current codebase, but I'd only be able to solve a few of the problems with the current low-context architecture. Variable, field, type, and function naming for instance is one thing that would require substantial changes to be able to handle user-configurable casing and name collision correctly. I think a faster -2.7k/+1.2k with solving other issues is better than a +400/-200 that has to be cleaned up later in order to be readable.

as-is this PR mixes a rewrite with fixing api issues. it is too much for me understand and maintain.

How can I help you understand and maintain it? I plan on documenting it before requesting review. I'd like to avoid making a fork.

I want to fix more problems than what I've listed. I considered surgically changing the current codebase, but I'd only be able to solve a few of the problems with the current low-context architecture. Variable, field, type, and function naming for instance is one thing that would require substantial changes to be able to handle user-configurable casing and name collision correctly. I think a faster -2.7k/+1.2k with solving other issues is better than a +400/-200 that has to be cleaned up later in order to be readable.

This is the problem.  This PR's scope has changed.  I prefer smaller and simpler changes with a well defined scope.
If you can get all tests passing - including adding a recursive example and testing it - I'll be happy to look again.

I managed to get code generation for unpacking working for non-recursive tables. I even added array handling which seems to be missing.
However, after reading the official implementation I want to make my own fork that:

Treats tables as ?*u8 similar to the C++ char data[1] member. This saves space.
Treats structs as extern structs instead of as tables. This allows bitcasting.
Adds verification similar to the C++ implementation so that invalid offsets in flatbuffers don't cause segfaults.
Adds a slice API for vectors of non-tables.

Indeed, I am scope creeping to writing an entirely different library! I'm thankful for the work you've done and am willing to collaborate if you wish.

I even added array handling which seems to be missing.

thanks for pointing this out. could you maybe open an issue with a small fbs repro?

I want to make my own fork that: ...

sounds cool! i really appreciate all the work and feedback you've given.  these sound like interesting ideas.  i think i'd like to add verification someday too.
wish you luck and will try to follow your progress.  i welcome any feedback you have here such as ideas for improvement or missing features / bugs.   don't hesitate to open issues.  i'll try to do the same.  lets collaborate in the future :+1:

I've split up #3 into smaller higher quality issues and added #9.

@travisstaloch I'm 95% done with https://github.com/clickingbuttons/flatbuffers-zig. While I'm sure there are some bugs hiding, you're welcome to take ideas from there! It does solve most of my open issues here and offers a safe API that shouldn't be able to segfault.
Thanks for all your work, it was very helpful to know that I had to write a builder, then use flatc --schema to generate .bfbs files, then handwrite reflection readers, and finally create a Zig code generator that uses those readers.

Oh cool! Thanks for the update. Nice work!  Your generated code looks really good.  And like that your implementation isn't mainly a port like mine.  Its more from scratch / first principles and I find it very readable.  Seems like you had a better understanding of flatbuffers than me when I started.
Glad that you found this project was useful :+1:


Get-BCArtifactUrl gives old version

Hey @freddydk,
I want to get Artifacts URL for latest October CU for BCSpring2019 edition, but it still gives me September CU link
Here's the command i use:
Get-BCArtifactUrl -type OnPrem -country w1 -version 14
it results:
https://bcartifacts.azureedge.net/onprem/14.17.44663.0/w1
Here's overview of all released CUs
https://support.microsoft.com/en-ca/help/4501292/released-cumulative-updates-for-microsoft-dynamics-365-business
NavContainerHelper is version <IP_ADDRESS>

That is likely because the October release isn't out on artifacts yet
It should be - will see what happened

Remove a clog in the pipeline - builds should be there now.


Add helm notation to CRD generation in eks-operator

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

generate crd by running go generate in eks-operator

Result:
The new CRD is missing the annotation "helm.sh/resource-policy": keep. This annotation tells helm not to delete the crd on a helm delete. The CRD in system-charts includes this annotation because deleting the CRD could causes underlying EKSClusterConfigs to delete and other errors to be encountered. Without the annotation this would happen if the user deletes the app or if rancher upgrade the app (which it does automatically when newer versions are available). The typical workflow for updating the crd in the chart is to run go generate in eks-operator and then copy over the produced CRD. Right now, this would lead to the omission of the annotation. In the future if the process of updating the crd in the chart is ever automated we could instead add the annotation after generation since there's no reason to assume outside of rancher it would be desired.
Other details that may be helpful:
Should be able to add code to pkg/codegen/main.go to add annotation to crd on go generate.

This can move forward now.

This can move forward now.

Closing because there is nothing for QA to test here. These changes were to ensure that code generation was adding things to the CRD that we have already put there manually.


code auto completion is not working.....(in vs code)

The user name of my machine has  a space. I think this is the issue for auto completion is not working. Also i can not use any libraries in my code.

@VJFernandez123 this will be fixed in the next immediate release

Hi @VJFernandez123,
Thanks for reporting this issue. We tried to reproduce the issue in Windows 10; using username "James Clark". Then saved a new ballerina file test.bal under the path C:\Users\James Clark\Documents path. Still, we could retrieve all code completions.
Please note that untitled files in VSCode will not provide any code completions due to fact that Ballerina VSCode plugin is only activated for the files with .bal extension. Thus, in order to receive code completions,  you need to save it somewhere first.
If the above is not the case, can you please try it on a latest Ballerina version such as 0.981.1 and let us know the results.
Used Ballerina Version: 0.981.1
Used VSCode Ballerina Plugin Version: 0.981.1
Note: I am closing the ticket as it is working with the latest. Please  re-open if you encounter the same issue with the latest release.


Async call not wait puppeteer.type method

Hi,
Async puppetter.type method doesn't wait to finish this call. For example, this code only press 3 or 2 characters and press the Tab key. How's the problem?

I have bots that work with similar usages. When I test it, type works fine for me. You can try to put 100 ms wait between them.

I tried many different solution but doesnt work. I try to page.type instead of page.keyboard.type this very well work. But some time I getting same issue. I'm using Macos v14 and chrome 128.


Harpoon2: Add events around UI_CREATE

What issue are you having that you need harpoon to solve?
I'm trying to re-implement the functionality of #199 as a harpoon2 plugin but two more events are needed for it, e.g. BEFORE_CREATE_UI and AFTER_CREATE_UI.
Why doesn't the current config help?
The event UI_CREATED is called too early to move the cursor to the line containing the current file because the text is not in the buffer yet. And is also called too late because the focus is already set to the floating buffer so we lose the context of the buffer from which we opened the quick menu (no way to get the current file).
Currently, for it to work, a user has to change the keybinding for the toggle_quick_menu and surround it with these functions.
vim.keymap.set("n", "<leader>tc", function()
	require("harpoon-highlight-current-file").mark_current_file()
	harpoon.ui:toggle_quick_menu(harpoon:list())
	require("harpoon-highlight-current-file").highlight_current_file()
end)

This is not ideal as you might open the quick menu with a different method (perhaps another plugin or something) and not activate this functionality.
What proposed api changes are you suggesting?
Call AFTER_CREATE_UI in toggle_quick_menu after nvim_buf_set_lines and BEFORE_CREATE_UI in _create_window before nvim_open_win.
I can prepare a PR if this is something of interest.

How about we just have UI create be called after everything is created.
Instead of a whole bunch of events, let's just try to keep it simple.
Since it sounds like you've done the research, if you just want PR moving that function call. I'll be glad to merge it

Just to clarify, moving UI_CREATE is only half the solution. I would still need BEFORE_UI_CREATE (I'm open to name suggestions) to be called before the context is moved from the file buffer to the harpoon quick menu buffer. Otherwise I wouldn't have access to the current file.

Can we just include that information in the event?

This also works. I've updated the PR.


two instance of react in my react native project

Description
I have a project with an old native screen and reactNativeActivity
I added react activity According to the instructions all are working as well
but when I come to install the flipper I need to call this code
getReactNativeHost().getReactInstanceManager()  from MainApplication.java file
since I did I get two instances of react
I noticed it when  I press on d in my CLI to open the dev menu then I press on reload or debug, the dev menu is not closed
when I check that, I see getReactInstanceManager method call to createReactInstanceManager ,
https://github.com/facebook/react-native/blob/f4e56fdf1946d2e54d8db7b445df001cf6ebfdb0/ReactAndroid/src/main/java/com/facebook/react/ReactNativeHost.java#L37-L46
even I create other instance in my reactNativeActivity.java file in onCreate method
   mReactInstanceManager = ReactInstanceManager.builder()
                .setApplication(getApplication())
                .setCurrentActivity(this)
                .setBundleAssetName("index.android.bundle")
                .setJSMainModulePath("index")
                .addPackages(packages)
                .setUseDeveloperSupport(BuildConfig.DEBUG)
                .setInitialLifecycleState(LifecycleState.RESUMED)
                .setJsEngineAsHermes(true)
                .build();

React Native Version
0.69.7
Output of npx react-native info
System:
OS: macOS 13.1
CPU: (8) arm64 Apple M1
Memory: 108.36 MB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 16.19.0 - /opt/homebrew/opt/node@16/bin/node
Yarn: 1.22.17 - /opt/homebrew/bin/yarn
npm: 8.19.3 - /opt/homebrew/opt/node@16/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.11.3 - /Users/--------.rvm/gems/ruby-2.7.2/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.2, iOS 16.2, macOS 13.1, tvOS 16.1, watchOS 9.1
Android SDK: Not Found
IDEs:
Android Studio: 2021.3 AI-213.7172.25.2113.9123335
Xcode: 14.2/14C18 - /usr/bin/xcodebuild
Languages:
Java: 11.0.11 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.0.0 => 18.0.0
react-native: 0.69.7 => 0.69.7
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
integrate into an existing project
https://reactnative.dev/docs/0.69/integration-with-existing-apps
call this code getReactNativeHost().getReactInstanceManager()  from the MainApplication.java file (before I create an instance of react-native in `reactNativeActivity``
Snack, code example, screenshot, or link to a repository

as we said when I press on d in cli
I get two layers of the dev menu (gif attached)
also in flipper when I chose the Hermes debugger I got a list with two Hermes instance


<EMAIL_ADDRESS>is missing

<EMAIL_ADDRESS>is missing on npm.
<EMAIL_ADDRESS>is published correctly.
The failed workflow: https://github.com/swc-project/swc/actions/runs/1223666628

It looks like it has still failed.


Add an operation to select a feed/article based on its title

Fixes https://github.com/newsboat/newsboat/issues/1135.
Combined with https://github.com/newsboat/newsboat/pull/1134, this fixes https://github.com/newsboat/newsboat/issues/888.
This can be used with a configuration like:
run-on-startup goto-title "newsboat" ; open

Coverage decreased (-0.1%) to 57.027% when pulling 6bcb39a4d9ecbbc87b0ffd5ddcfaa0c3077dae4c on dennisschagt:feature/issue-1135 into fb11be52b91165b20878a2a6a61d0c7904d24874 on newsboat:master.

Thank you for the review!


DXUT

Unless there are specific requirements to stick with the old version of DXUT that is included in this project, I would highly recommend bundling up the latest version of DXUT that is available from Github.
As the one that is included here is dated from before Apr 11, 2014.
The reason I mention this is because there are some nasty bugs in that old version of DXUT.
Take a look here at a few commits to get an idea:
https://github.com/manuelh/LiquidVR/commits/develop

We agree with the assessment of the DXUT issues and will consider an update in the next release. Thanks very much for the feedback.


Added notebook for iteration

This pull request is a single file and does not follow instructions to have multiple notebooks as you divide work in team.


Drag-to-rearrange Not Working

Describe the bug
Dragging items in the source list to rearrange the order of the display is rendered non-functional.
To Reproduce
Drag any folder or feed in the sidebar in a usual macOS way.
Please complete the following information:

Vienna version: 3.9.1
OS version: macOS 10.14

Additional information:
In which version of Vienna does the problem not occur, if applicable?
3.5.x–3.8.x

I meant items won't allow me to drag up or down to rearrange them in the sidebar. I hover over an item and try to move it by dragging it, but nothing happens. This used to be a feature, which is no more.

What is currently displayed for "View -> Sort By" ?

Date->Descending.

Order By––>Manual
It's been like that for ever since I started using Vienna.

Do you have Group folders 📁 ?
Creating one and attempting to move one / some feed(s) into it might unlock the problem.

I am closing this ; you are welcome to open a new issue if a similar problem occurs with Vienna 3.9.2 or later.


sqlite3.OperationalError: no such table: main.divides

While running run.sh using VScode on my local machine seeing below error:
File "/Users/apatel54/Documents/Arpita/git/NGIAB_data_preprocess/data_processing/gpkg_utils.py", line 34, in verify_indices
con.execute(index)
sqlite3.OperationalError: no such table: main.divides

The application is looking for a table in the conus.gpkg file. @chp2001, we might need to update the instructions (step 5, as the steps are currently laid out, should be in front of step 3.)

#22 Added code to automatically download these two files on first launch of the app which should mitigate this issue.


Improve dangerouslySetInnerHTML

Use

instead of

Why?

easier to read
looking just at { __dangerousHtml: html } lets the user suspect a danger

Hi @yandeu :wave:
I think many of nano-jsx’s user expect nano-jsx to provide a similar API to React/Preact and keeping similar to the React/Preact API will lead to allow us to reuse React’s type definitions, and more user-friendly API and developer experience. How about do you think?

I will keep both. But update to docs to the new syntax.
Please see: https://www.youtube.com/watch?v=XEt09iK8IXs&t=495s

I see. I did not know about this video.
I totally agree providing new dangelouslySetInnerHTML syntax and keeping both of API! Thank you for sharing the video!


Json examples

Single-parser trip_report use-case, HW-proven.

We're going with #20 instead of this.