class variables not needed in reggen/topgen/tlgen

@eunchan
@rswarbrick pointed out class variables are not needed in data.py below at #4309
I feel it's better to change all these to just use instance variables. If you all agree on this, I can make a PR for this.
https://github.com/lowRISC/opentitan/blob/d0ab3e7074b5deb0fa653a61732a470d71d115ae/util/reggen/data.py#L20-L54
same for topgen and tlgen
https://github.com/lowRISC/opentitan/blob/d0ab3e7074b5deb0fa653a61732a470d71d115ae/util/topgen/validate.py#L146-L161
https://github.com/lowRISC/opentitan/blob/d342ce818292d715aac79427f5e4f082a637c3b8/util/tlgen/xbar.py#L10-L33

Agree. Sorry for premature python codings from me (ashamed haha...)

Agree. Sorry for premature python codings from me (ashamed haha...)

Haha, I thought that were needed until I learned from @rswarbrick's comment

Yeah, Python's pretty weird about this. To keep the "static schema" obvious, the usual style is to assign each field in the constructor (as you're already doing). Honestly, I think this is a pretty major flaw in the language design, but I think we're probably stuck with it. :disappointed:

close via #4346


#9490: TTNN eltwise/unary migration with square op example

Ticket
Link to Github Issue #9490
Problem description
Move unary ops from tt_eager to ttnn library as mentioned in #9322
What's changed
Established eltwise/unary structure with #9368 as reference
Migrated square op from tt_eager to ttnn library, replacing all references to ttnn::square / ttnn.square  in cpp, test and model files.
Checklist

[x] Post commit CI passes https://github.com/tenstorrent/tt-metal/actions/runs/9683299762
[x] Model regression CI testing passes https://github.com/tenstorrent/tt-metal/actions/runs/9683302825
[x] New/Existing tests provide coverage for changes

Migrated square op from tt_eager to ttnn library, replacing all references to ttnn::square / ttnn.square in cpp, test and model files.

This ideally should have been a separate PR, which would be a great example for others migrating specific unary ops.

For the files that are moved and renamed (the reader/writer and compute kernels + create_program code), the old code shouldn't be used anymore and should be deleted right?

Yes. Will happen as a separate PR.

For the files that are moved and renamed (the reader/writer and compute kernels + create_program code), the old code shouldn't be used anymore and should be deleted right?

Yes. Will happen as a separate PR.

Why can't it be in the same PR? It's a simple rename for the kernels at least. Instead, we will have two massive commits adding and deleting files.

@TT-BrianLiu  Lots of C++ code like composite ops, including tt_lib bindings still depend on tt_dnn impl. MCW team will help to remove those deps bit by bit. And only then we can remove.

@TT-BrianLiu Lots of C++ code like composite ops, including tt_lib bindings still depend on tt_dnn impl. MCW team will help to remove those deps bit by bit. And only then we can remove.

At the very least, the kernels should just be renamed and a simple search and replace for strings should help convert all use cases of these kernels. I am worried that we will have duplicate kernels existing without an explicit freeze on development for the old one.

@TT-BrianLiu can you share more details/guidance. This will @KalaivaniMCW to complete this fast?

It's very simple (for kernels at least). Just switch the file path whereever you have code like this (lines 64 and 70):

@TT-BrianLiu I see. Good thinking.
Kernel reuse is trivial.
Program reuse hopefully is also trivial. But because ttnn and tt_eager kind of depend on each other - it is possible that include of .h with .cpp will be problematic. Not sure.

@TT-BrianLiu I see. Good thinking. Kernel reuse is trivial. Program reuse hopefully is also trivial. But because ttnn and tt_eager kind of depend on each other - it is possible that include of .h with .cpp will be problematic. Not sure.

I would give it a try. If a simple switch of namespace doesn't work, then just leave the duplicate create_program code. But kernels should not be duplicated for sure.


Sometimes first iOS key press acts like keyboard hide

Brief
When using the component as a wrapper for whole app, sometimes the first iOS key press ever acts like the keyboard is hidden. It runs the animation that hides stuff behind the keyboard.
Closing the keyboard later and opening it again would work just fine until I restart the app.
I did not try in Android.
Below I'll mention what seemed to fix it for me.
Details
I was playing with the sample app from react-native-size-matters. It had a chat window with textbox at the bottom that suffered from the keyboard issue.
After some messing up to upgrade the sample itself to react-native 0.55 (latest expo supports), and running it, I modified the file that wraps all components:
https://github.com/nirsky/react-native-size-matters/blob/16383ea10bafcfd508cfbafb36995f4ea27b7f2a/examples/expo-example-app/components/behaviors/withScaledSheetSwitch.js#L9
Which originally looked like:
https://github.com/nirsky/react-native-size-matters/blob/16383ea10bafcfd508cfbafb36995f4ea27b7f2a/examples/expo-example-app/components/behaviors/withScaledSheetSwitch.js#L9-L17
    render() {
        return <View style={{flex: 1}}>
            <View style={{alignSelf: 'stretch', flexDirection: 'row', justifyContent: 'space-between', alignItems: 'center', margin: 10}}>
                <Text style={{fontSize: 16}}>size-matters enabled?</Text>
            <Switch value={this.state.scale} onValueChange={() => this.setState(prevState => ({scale: !prevState.scale}))}/>
            </View>
            <ChildComponent {...this.props} scale={this.state.scale}/>
        </View>
    }

To now look like:
            return (
                <Spacer enabled style={{ height: '100%' }}>
                    <View  style={{flex:1, height: '100%'}}>
                            <View style={{ alignSelf: 'stretch', flexDirection: 'row', justifyContent: 'space-between', alignItems: 'center', margin: 10 }}>
                                <Text style={{ fontSize: 16 }}>size-matters enabled?</Text>
                                <Switch value={this.state.scale} onValueChange={() => this.setState(prevState => ({ scale: !prevState.scale }))} />
                            </View>
                            <ChildComponent {...this.props} scale={this.state.scale} />
                    </View>
                </Spacer>
            );
        }

(I only added the <Spacer> and styled it, and gave the <View> under it height: '100%')
To imagine the app (pic from iPad, but I experienced the issue on most recent iPhone simulator, with iOS 12):

Then when I opened the chat page in the app, I click on the textbox at the bottom, and the keyboard popped up with the screen shifting above it nicely, but when I clicked on the keyboard to enter text, the view expanded to full window, hiding the textbox behind the keyboard.
I clicked away, then clicked the textbox again, and tried typing. This time the textbox stayed higher than the keyboard. It worked just fine from then onwards. I reloaded the app and tried again, and oops, it failed the first time, but then it worked just fine the following times.
I repeated the test several times, disabled the scaling from react-native-size-matters to make sure it's not affecting anything, but same result. First key press on phone keyboard would expand the view behind the keyboard, but after hiding and showing the keyboard, pressing keys doesn't cause any harm.
Investigation
I looked at the source code of the Spacer library, which was super awesome BTW, simplicity is great, and not easy, and I was able to fix the above problem by changing a few methods:
    _keyboardWillShow = ev => {
        if (this.props.enabled) {
            // Calculation new position above the keyboard
            let toValue = (this._locationY + this._viewHeight) - (windowHeight - (ev.endCoordinates.height + this._spaceMargin));
            this._animate(-1 * toValue).start();
        }
    };

    _setLayoutProps = ev => {
        this._container
                ._component
                .measureInWindow((x, y, width, height) => {
                    this._locationY = y;
                    this._viewHeight = height;
                })
    };

Both changes seemed to be needed. The first change in _keyboardWillShow removes the dependency on this._container in calculations. I'm not sure if this can be a problem sometimes (when the component is not outermost), which is part of the reason why I'm not creating a PR for this yet.
The other change felt more straightforward, and almost opposite the first one. It updates this._viewHeight based on the this._container._component.measureInWindow().
Extras
I also added a componentWillUnmount, but just because it felt right, not because of this issue.
It looked like:
    componentWillUnmount() {
        Keyboard.removeListener(showListenerEvent, this._keyboardWillShow);
        Keyboard.removeListener(hideListenerEvent, this._keyboardWillHide);
    }

Updated the post with correct source links

Hi Meligy,
I had some trouble running with Expo, though I managed to upload react-native-size-matters into expo snack (please try it if you can).
I tried to replace View (line 101 in expo-example-app/components/Chat.js) component with Spacer and it works fine on my iPhone. I think it would be the right place to use Spacer. (And I hope it was what you looking for).
Besides, in the function _keyboardWillShow of the current version, I don't recall exactly what happened, but I had run into an issue where the reference (this._container) was null and throw an undefined error, also this._container._component is not a standard property. So I added an extra calculation so it wouldn't missed.
And thanks for the Extras, sound reasonable and pretty standard. I'll add it into the repo

To reproduce the steps, I have pushed a working version of that repo here:
https://github.com/Meligy/react-native-size-matters/tree/expo30
I tried just installing the library and replacing line 101 in expo-example-app/components/Chat.js, taking away View and adding Spacer instead, and the issue still seemed there in the simulator (iOS 11 and 12).
Here's the full code with the minimal change:
https://github.com/Meligy/react-native-size-matters/tree/minimal-react-native-spacer
It looked like this:

Only the first character seems to break it. Hiding and showing the keyboard seems just fine.
This is the patched version that seems to work for me:
https://github.com/Meligy/react-native-size-matters/tree/patched-react-native-spacer

Thanks for the gif, it super helpful. I will find out more about this issue!

Thanks a lot.
I added some specific links to the files that were modified above.

With just a View, you never see the textbox when the keyboard shows up.
With the Spacer, the textbox is visible, except when you make the first keyboard touch, it then disappear, dismissing the keyboard and showing it again still shows the textbox correctly until you restart the app.

The patched version of spacer.js in patched-react-native-spacer branch seems to work though (used in withScaledSheetSwitch.js).

Can you verify that after reloading the app, and after the sign "Device Connected" disappear, the issue still occur? I double checked when not using Spacer. It seems to me that somehow it rerender the view when the device reconnected.
Besides, I wouldn't recommend removing the extra calculation in case this._component return null (in the _keyboardWillShow function). It was there as because sometimes, for some reason, it actually returns null.
Otherwise, feel free to create a folk and edit as you want.

I can confirm that it was Expo somehow rerender components on the device after it connected (which doesn't make sense).
Here is a video shows it does happen even when I use View and try to use the chat box before "Device Connected". Also, when I switched to Spacer and wait for the device to completely connected, the problem disappears.View on Youtube (I realised the quality isn't that good, though I'm a little busy at the moment)
Besides, on the _keyboardWillShow, the this._component is needed in case the child component position is not set to 'fixed'. Because once we scrolled the screen, the Y position of the child view changed (need to measure by the root view, not window), so it needs to pick up the new Y position in order to animate correctly. (My bad, I should have explained this earlier).
Again, feel free to folk and edit if you needed.

As the issue described caused by Expo rerender component after device connected to Expo Snack (when the green sign "Device Connected") disappear. I'll go ahead and close this thread.
Feel free to reopen or create a new issue if need.


PHP: fix watchConnectivityState unit test

Change the test to be more in line with Node

PHP tests on Jenkins are all green. Please review, thanks!


Markdown Render VSCode Command URI with arguments

I've been trying to implement a hover feature with go-to definition links, like this.

I found https://github.com/microsoft/vscode/issues/140733 which states that vscode commands can be embedded in markdown (and in hover requests), and this is working. However, I can't figure out how to pass arguments to the command. Is this possible? If not, I'd very much appreciate support for this.
Unfortunately the LSP doesn't support commands in hover requests, as that would be my preferred method. However, I can certainly make it work by passing arguments along with the command in the markdown. Thank you!
cc @mjbvz

See https://code.visualstudio.com/api/extension-guides/command#command-uris
Command arguments can be passed as a JSON array encoded on the uri


Allow Docutils to be shipped to Lambda

Description
Allows the docutils module to be shipped to lambdas.
GitHub Issues
https://github.com/Miserlou/Zappa/issues/1829

Coverage remained the same at 73.259% when pulling 983fb183c3bcd14255d01548dd196561f3811d89 on danverbraganza:patch-1 into d58b6f9efe7d9d0b10f80fd88941dcd169a83ee6 on Miserlou:master.


Feature/err log report tor

Closes #704

This will conflict with #1015

This will conflict with #1015

I believe that this should go first; This is simple and most of the changes are b/c of error -> info meaning easy fixes for #1015


DO NOT MERGE. This includes the code for the server example

But it is not yet ready. For starters, this will be broken into steps 7 and beyond.
@Janamou   @kwalrath

The directory structure is weird, too. You'll fix that, right?


supplementary details for transaction

changes line parsing. neccessary for transactions as they can have
newlines within their details field
includes initial amount details
includes charges

related to #18

do you have a link to the spec of those?

we've implemented it according to
https://www.kontopruef.de/mt940s.shtml


How to set acl?

I don't find the right way to set acl usind this ldap docker image?

You can define access control in the slapd.conf file.
I've added an environment variable to make this easy. It's called ACCESS_CONTROL and you set it like so:
docker run \
  -e ACCESS_CONTROL="access to * by self write by anonymous auth by * read" \
  pgarrett/ldap-alpine

This will allow the user to modify their entry, allows anonymous to authenticate against these entries,
and allows all others to read these entries.
The default is access to * by * read which allows anyone and everyone to read anything.
You can find more information on access control here: https://www.openldap.org/doc/admin24/access-control.html
Hope this helps :)


Chevrons not rendered correctly in "undo" notification

Chevrons («») are not rendered correctly in the "undo" notification.
To Reproduce

Run the "Replace smart quotes with ascii" common transform on a text column
Observe that the "undo" notification contains replacement characters instead of chevrons

Current Results
Chevrons appear in the history list OK, but not in the "undo" notification, which times out and disappears.
Expected Behavior
The chevrons should be accurately rendered in this modal.
Screenshots

Versions

Operating System: macOS 12.1 (21C52)
Browser Version: Safari 15.2 (176<IP_ADDRESS>.6)
User-agent string: "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/15.2 Safari/605.1.15"
JRE or JDK Version: Sorry, I can't get this info immediately; if there is a separate fix for this issue or if my system is not set up correctly please let me know.

❯ which java
/usr/bin/java
~
❯ java --version
The operation couldn’t be completed. Unable to locate a Java Runtime.
Please visit http://www.java.com for information on installing Java.

OpenRefine:  3.5.2 [e3efd4e]

Datasets
I don't believe this is caused by my data but am happy to attach it here if doing so would help fix the bug.
Thanks
Hugh

The yellow notification thingy at the top shows encoding errors quite frequently. I also see it a lot while doing manual reconciliation.
Screenshot:

To reproduce:

take a list of e.g. person names with many diacritics
reconcile in such a way that you'll have quite a few entries to match by hand
do that
see the encoding issues

Here's a small dataset with many person names with diacritics for testing: documenta-13-artists.csv

Hi @trnstlntk is it only the notification banner, or do you see that unicode replacement char U+FFFD � in other display dialogs or areas of OpenRefine?

I remember only seeing it in the yellow notification banner, but I'll keep my eyes peeled for other occurrences.


Proposal: Configure Prometheus alerts on the ServiceMonitor

ServiceMonitors and NetworkPolicies are good examples on the flexibility that kubernetes can offer to developers configure theirs monitoring and networking rules.
I thought we are missing a flexibility to customize alerts because nowadays developers need access to configmap to change alerts. I'm proposing to add an object in the ServiceMonitor to accept alerts configuration.
Any thoughts?

Can you elaborate on the difference on configuring alerts in the ServiceMonitor vs in a ConfigMap? Dynamic reloading that is mentioned in the tweet is taken care of by the Prometheus Operator.
I feel what you're looking for here is a general application definition abstraction on top of Kubernetes and Prometheus in general.

My first thought was not really clear let me explain again. I would like to have an way to configure the alerts not only in the configmaps. My proposal was to allow the serviceMonitor to accept alerts definition something like this:
apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
  name: elasticsearch
  namespace: monitoring
spec:
  endpoints:
  - bearerTokenFile: /var/run/secrets/kubernetes.io/serviceaccount/token
...
  alerts: |+
  groups:
  - name: elasticsearch.rules
    rules:
      - alert: ElasticSearchNodeDown
        expr: up{job="es-metrics"} == 0 or count(job="es-metrics") == 0
        for: 20m
        labels:
          service: elasticsearch
          severity: warning
        annotations:
          summary: Elasticsearch cannot be scraped
          description: Prometheus could not scrape a {{ $labels.job }} for more than 20 minutes

In that case developers doesn't need to touch the configmap, they will only manipulate one file that will be the serviceMonitor.

The biggest problem I have with that is that a lot of alerting rules don't correspond to a single ServiceMonitor, so the abstraction feels weird. If a developer can access the ServiceMonitor, what difference does it make to give her/him access to the ConfigMap?

Perhaps providing another "Kind: Alert" would solve this issue?
I assume it would be beneficial to allow developers to define Alerts in their dedicated namespaces, where a remote service would consume those objects and reconfigure alertmanager that is dedicated to particular scope (or somehow matched by labels akin to "Kind: Ingress").

Seem like #1333 will fix this issue 😄

Indeed, the biggest gain we're getting out of that is that we can then allow rules for a single prometheus server to be defined across namespaces as well as provide validations at apply time and more fine grained RBAC. We'll close this once #1333 is merged.

#1333 is merged and the release has been shipped :tada:. Closing here.


[WIP] Add --toolchain option and wildcard configuration support

The change includes

-v, --verbose option in list toolchain command
Added support for wildcards in --configurations
Add support for toolchain selection --toolchain option

Codecov Report
Patch coverage: 72.44% and project coverage change: +0.95 :tada:

Comparison is base (35d2085) 75.50% compared to head (d6618e3) 76.45%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #49      +/-   ##
==========================================
+ Coverage   75.50%   76.45%   +0.95%
==========================================
  Files          11       12       +1
  Lines         996     1045      +49
==========================================
+ Hits          752      799      +47
  Misses        185      185
- Partials       59       61       +2

Impacted Files
Coverage Δ

pkg/builder/cproject/builder.go
73.99% <ø> (+0.21%)
:arrow_up:

pkg/builder/csolution/builder.go
57.89% <25.00%> (+0.36%)
:arrow_up:

cmd/cbuild/commands/list/list_configurations.go
75.00% <50.00%> (-1.00%)
:arrow_down:

pkg/utils/utils.go
83.19% <67.50%> (-2.53%)
:arrow_down:

cmd/cbuild/commands/list/list.go
100.00% <100.00%> (ø)

cmd/cbuild/commands/list/list_contexts.go
85.71% <100.00%> (+9.71%)
:arrow_up:

cmd/cbuild/commands/list/list_toolchains.go
87.50% <100.00%> (+5.14%)
:arrow_up:

cmd/cbuild/commands/root.go
95.89% <100.00%> (+0.08%)
:arrow_up:

pkg/utils/wildcards.go
100.00% <100.00%> (ø)

... and 1 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Cannot authenticate SSH after any GPG operations

Hi,
I'm using a YubiKey 5 NFC with WinCryptSSHAgent. I've generated a certificate in PIV certificate Slot 9a:

and it works well with WinSSH. However, after each time I perform any GPG operation (e.g., gpg --card-status, gpg --sign, etc.), I can no longer use my YubiKey with WinCryptSSHAgent to authenticate SSH connections:
"The smart card cannot perform the requested operation or the operation requires a different smart card."

Seems like that once I've performed any GPG operation, Windows will no longer read the PIV certificates stored in YubiKey until replugging.
Any idea or solution on this issue? Thanks a lot!

Append: seems like the same issue with #1 and #10
OpenPGP conflicts with WinCryptSSHAgent

AFAIK this is common gpg issue, it exclusively locks tokens. It was reproduced in my environment on ubuntu 18 host.


Missing starts of drives

Describe the bug
I've noticed a number of drives from my home show the start as a location anywhere 5-10mins into the drive. Presumably this is caused by some type of polling delay, but TeslaFi doesn't have the same problem usually.
Expected behavior
The full drive is reported
How to reproduce it (as minimally and precisely as possible):
Take your first drive of the day and see if the start is missing.
Relevant entries from the logs
Let me know what would be useful from logs and I can try to pull.
Screenshots
https://imgur.com/a/M202mEs
Drive started in the middle of a highway.
Data
I can pull if the answer isn't "unfortunately this is expected"
Environment

TeslaMate version: latest
Type of installation: Docker
OS TeslaMate is installed on: Linux (Synology NAS)
User OS & Browser: Mac Chrome
Others:

I experience the same - And have put it to "the car have to switch from wifi to LTE - this can have some delay"...
But that should be similar on TeslaFi, except if this was not with the streaming API, and thus having some kind of "buffer" kept in the car, that was relayed to the old api once connection was established, but not through the streaming API..
I that is the case, could it be possible to combine the 2 approaches? Streaming API when connection IS really streaming, and the old pull method until then....

I that is the case, could it be possible to combine the 2 approaches? Streaming API when connection IS really streaming, and the old pull method until then....

The regular poll-based API is still called every 15 seconds if streaming is enabled.
A direct comparison with TeslaFi would be interesting. It would surprise me if TeslaFi and TeslaMate would be very different here.

The regular poll-based API is still called every 15 seconds if streaming is enabled.

Cool - good design choice :)

A direct comparison with TeslaFi would be interesting. It would surprise me if TeslaFi and TeslaMate would be very different here.

Agree - but the premise from OP was that he wasn't seeing this with TeslaFi... That doesn't make sense, but worth investigating...

I experience the same: TeslaFi is logging the whole trip, while Teslamate is missing the first minutes. I have enabled Streaming API. The start of the trip is from a WiFi location. I notice that TeslaFi has lower resolution on the tracks, so perhaps they are not using the streaming API?

I also had this issue using the streaming API. I worked around it by only connecting the car from the WiFi when there is an update pending, because the switch from WiFi to LTE when the car starts driving seems to disconnect TeslaMate for quite a while. Not sure if that is a TeslaMate issue or a Tesla API issue, but since other loggers have no problem with that, I think it might be the former. Additionally, I use data from a garage door sensor to call the /resume endpoint of TeslaMate's API to inform it that the car is about to wake up, so that TeslaMate starts logging earlier.
As I said though, those are just workarounds and I wish that wouldn't be necessary.

Great input Kevga :-) I will try this do you have some more details on how to call the /resume endpoint of TeslaMate's API? I would love to this in Home Assistant.

So it seems:
a. The polling every 15 seconds when streaming is enabled is not automatically switching to a more frequent polling, if the streaming API is not responding.... This could be changed (in line with https://github.com/adriankumpf/teslamate/issues/928)
b. It appears the streaming api access takes more time to initiate, when switching from WiFi to LTE... A higher frequency polling in this transition would most likely remedy this, to a level where TeslaMate is on par, or better, than any comparable solution..

@elgeniskogen see #91 for details about how this works.

I am seeing this behavior too. Mostly when leaving home (where the car is on a WiFi connection), but it has also happened when there was no WiFi involved. Just yesterday, the whole trip from home to the tire shop was recorded, but when I went back home again the first few hundred meters of the trip were missing.

I have the same issue. Loggin often does not start until after 500+ meter driving when starting.
So the start location of many of my drives is not correct (and is recorded as being different from the end-location of the previous drive, which is of course not logical).
I have geofences for many of my locations, but because of this "bug", it looks like my trips often start from different random addresses.
So in addition to trying to solve this with polling adjustments to the API, why not also alway assume that the start location of a drive is the same as the end-location of the previous drive?
(to be sure that this assumption is correct, you could do this only if the odometer reading at the end of the previous drive is within e.g. 1 or 2 km's of th start of the current drive).

That's a good idea, except that it would produce bad results if the car has been towed, for instance. I would suggest that instead of using the odometer, check the first GPS location fix of a new drive against the last one of the previous, and if they're sufficiently close together (within 1 km or so apart), then use the last location of the previous drive as the start of the new one. That'd produce a short, straight line possibly looking pretty weird on a map but at least the starting address would be correct and the driven distance would be less wrong.

Good point. A bit more difficult to implement, but using the GPS location would be more fail save.


Update readme

This updates the readme to reflect the current state of the app.
For review, look at the complete document, not just the changes:
https://github.com/alphagov/collections-publisher/blob/5fa8f74ae22d089d5c3743a7d6ce1b3599460134/README.md
Trello: https://trello.com/c/d3RlTJwt
I'd like this reviewed at least two people on @alphagov/team-finding-things.

Other than a small typo, looks good to me. Live examples for browse and topic pages are really useful. :+1:

Are you using the term 'curated lists' to collectively refer to both content tagged to topics and content tagged to browse pages? If so, I wonder if you can replace this with something more explicit - 'curated lists' sounds like you might be introducing a new concept. Potentially a bit confusing.

Looks good, I agree with @mobaig on the curated lists

@mobaig @Davidslv well spotted - I was actually introducing a new concept. I've updated the PR to include more about these "curated lists".

:ship:


Rename phpstan.neon to phpstan.neon.dist

Needs adding in .gitattributes as well.

Needs adding in .gitattributes as well.

No it doesn't? It already is?

Thanks. Updated.

Hi and thanks for the PR,
Can you provide any reasoning for this change? Like, would anyone outside of this project be effected by the file name, either before or after?

The same reasoning as the phpunit config has .dist. It allows local config to exist outside of the version control, without the extra extension.

@SpacePossum phpstan.neon v phpstan.neon.dist works the same way as phpunit.xml.dist, which is explained best here by Sebastian himself.

I know the concept, so  This allows a developer to use a custom configuration without running the risk of accidentally checking it into version control. applies to developers of this project. I see no reason why a developer in this project would want to create a custom version of this file without the intentions to add the changes. What would be a case for this (so not the generic concept if dist files, but why this change here in this PR)?

I agree with @SpacePossum: I don't see a valid use case for this. This is useful for PHPUnit because the config file allows to define some output customization options, which can differ locally from the versioned file. This does not apply to PHPStan config file as far as I know.
Same reasoning regarding .php_cs.dist for which I think we should drop support.

closing after discussion, feel free to keep discussing and who knows maybe we change our minds!

If php-cs-fixer uses the phpstan baseline thing, then this conversion is quite useful.

Example from a Makefile:
phpstan-analyze:
	@docker run -it -w /data -v ${PWD}:/data:delegated --entrypoint vendor/bin/phpstan --rm registry.gitlab.com/grahamcampbell/php:7.4-base analyze

phpstan-baseline:
	@sed -e '2,1d' < phpstan.neon.dist > phpstan.neon && docker run -it -w /data -v ${PWD}:/data:delegated --entrypoint vendor/bin/phpstan --rm registry.gitlab.com/grahamcampbell/php:7.4-base analyze -c phpstan.neon --no-ansi --no-progress --error-format baselineNeon > phpstan-baseline.neon && rm -f phpstan.neon || rm -f phpstan.neon

phpstan-no-baseline:
	@sed -e '2,1d' < phpstan.neon.dist > phpstan.neon && docker run -it -w /data -v ${PWD}:/data:delegated --entrypoint vendor/bin/phpstan --rm registry.gitlab.com/grahamcampbell/php:7.4-base analyze -c phpstan.neon && rm -f phpstan.neon || rm -f phpstan.neon

phpstan.neon.dist:
includes:
  - phpstan-baseline.neon
  - vendor/nunomaduro/larastan/extension.neon

parameters:
  paths:
    - app
    - database

  level: 3

  ignoreErrors:
    - '#Unsafe usage of new static#'
    - '#Call to an undefined method#'
    - '#Access to an undefined property#'

The makefile generates a phpstan.neon with the baseline removed, as an intermediate step.


Consumer group example

Can someone add example code on how to implement a consumer group using OffsetManager? I looked at https://github.com/wvanbergen/kafka, but it is uses zookeeper. New version of Kafka allows to store consumer offset in Kafka and OffsetManager seems to enable it. An example implementation will be very helpful.

https://github.com/wvanbergen/kafka/pull/72

It doesn't answer my question. The pull request you have improves the original implementation (which uses zookeeper for managing consumer offset) and add logic to also use zookeeper for managing partition metadata as well.
I'm looking for an implementation that does not use zookeeper and instead uses Kafka for managing consumer offset. Zookeeper does not scale well for writes as per this post: https://cwiki.apache.org/confluence/display/KAFKA/Committing+and+fetching+consumer+offsets+in+Kafka

That's exactly what that pull request does. It will still need zookeeper for coordination of instances though.

Sorry, I was still looking at ConsumerGroup. How do I fetch the offset? In the example, you start with OffsetNewest. I want to fetch the offset (if available) and start from there.

The OffsetManager takes care of this. When I start a partition consumer, the PartitionOffsetManager will return the next offset to consume. In case there is no stored offset, it will use the initial offset as configured. See #461

Thank you so much. I will play with a bit to get more familiar. Is there any plan to merge it with Shopify/sarama?

Nope, because of the Zookeeper dependency.
A future version of Kafka is supposed to have all the APIs to be able to implement a high-level consumer. At that point we will probably add it to sarama.

as kafka 0.9 is released now, Zookeeper dependency is removed.
do we have any plan for it now?

Yup, it's in scope of sarama now. However, we don't have any immediate plans to start working on it. Contributions are definitely welcomed!


💡 [REQUEST] - Hover effect is missing in the social icons present in footer

[x] I am from GSSoC23.
[ x] I would like to be assigned this task.

Section
Description

Concise Description
Want to add hover effects on social icons

Solution Proposed
Hover efffect

Alternative Solutions
Improve the UI of the website

Additional Context
Write any other contexts related to the feature request

Screenshots
[

    ]|

Code of Conduct
By submitting this issue, you agree to follow our Code of Conduct.

[x] I follow the Contributing Guidelines of this project.
[x] I have checked the current issues for duplicate problems.
[x] I have read the readme of this repository.

I would like to work on this, Please assign it to me


Accurate audio track counting

Not sure what can cause trackDidStop to be called unbalanced with trackDidStart but better guard this.
Accurate counting of audio tracks instead of counting publish / unpublish.

start/stopCapture is guarded by a trackState lock so it should be ok:
https://github.com/livekit/client-sdk-swift/blob/8f3da0a8297c992450088169345b4fa8fc242de6/Sources/LiveKit/Track/Track.swift#L225-L251
But I agree there with more work I can make it more solid.

@hiroshihorie yeah that's perfect - this should probably work as it is then

Ok when I look again it's still not race proof stop() could get executed concurrently


[Docs] Replace contents of Meshery Overview

Current State
Currently the pages https://docs.meshery.io/getting-started/overview and https://docs.meshery.io/project are redundant. The 'project' page however is more comprehensive and contains additional links.
Desired State

Change Meshery Overview at https://docs.meshery.io to link to https://docs.meshery.io/project

Fix the table headers at  https://docs.meshery.io/project

Currently they look like the following -

Instead they should look like the ones at https://docs.meshery.io/getting-started/overview

Make sure that all content at https://docs.meshery.io/getting-started/overview is contained in https://docs.meshery.io/project

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Hi, I want to work on this issue.

@Vee339 Have you built and run the docs locally ?

hi @abdullah1308 , i have already setup docs in my local. Can I work on this?

hi @abdullah1308 , I have fixed the issue. Here it looks like. Let me know if I can send a PR or not.
Thank you

@Vee339 Have you built and run the docs locally?

No, I haven't. Please assign this issue to @rijusougata13. I think he is already working on that.

@rijusougata13 yes put up a PR linked to this issue to be reviewed.

@abdullah1308 please have a look.


add apcu storage for php 7 support

You should be able to enable backwards compatibility in apcu so that it creates aliases for new functions. See https://github.com/krakjoe/apcu-bc. Have you tried that?

No, thanks!


Resource Packs disappeared from plugins.

I logged in today after the newest OSRS update, and my resource packs plugin is just gone from the sidebar. Through the plugin hub, it says it's installed but there's no cog to manage the settings, and as I said it has disappeared from the side bar. I have uninstalled and reinstalled the plugin, and the same for RuneLite, opening and re-opening the client, but nothing appears to be fixing it. Other people in game say they still have theirs enabled and working, but mine is just gone for some reason. All other installed plugins are working fine and appear on the side bar.

Had the weirdest fix. Had to comb through the bowels of my PC to find the plugin folder. I deleted that from my folder, the plugin was still showing as installed. I reopened RuneLite and it redownloaded it from somewhere. There was some kind of issue with the Java file that the plugin hub installs.


114359 - Adding description for synthetic test PATCH API

Adding new OpenAPI: PATCH SyntheticTest

Story / Card
ui-backend PR

This PR cannot be merged until 242 is completed


fix(initial-branch): get updated config from repodoc before ignoring

fixes #733

Is this fix already available in production ?

Yup, we deployed it about 30 minutes ago.


Enclose zip in list

Beginning pylint and unit tests.
See https://cmssdt.cern.ch/jenkins/job/DMWM-PR-test/157/ for details

No pylint warnings for pull request 6090:

Unit test changes for pull request 6090:

WMCore_t.WebTools_t.NestedModel_t.NestedModelTest:testOuterFooError changed from error to success
WMCore_t.WebTools_t.REST_t.RESTTest:testListTypeArgs was deleted. Prior status was success
WMCore_t.WebTools_t.REST_t.RESTTest:testSanitiseFailHTTP changed from error to success
WMCore_t.WebTools_t.REST_t.RESTTest:testSanitisePass was deleted. Prior status was error
WMCore_t.WebTools_t.REST_t.RESTTest:testBadVerbEcho was deleted. Prior status was success
WMCore_t.WebTools_t.REST_t.RESTTest:testSanitiseAssertFail was deleted. Prior status was error
WMCore_t.WebTools_t.REST_t.RESTTest:testBadPing was deleted. Prior status was error
WMCore_t.WebTools_t.REST_t.RESTTest:testPing was deleted. Prior status was error
WMCore_t.WebTools_t.NestedModel_t.NestedModelTest:testInnerPingError was deleted. Prior status was success
WMCore_t.WebTools_t.REST_t.RESTTest:testDAOBasedHTTP was deleted. Prior status was failure
WMCore_t.WebTools_t.NestedModel_t.NestedModelTest:testOuterFooPass changed from success to failure
WMCore_t.Agent_t.HarnessCouchBackend_t.HarnessCouchBackend_t:testAddWorker changed from failure to success
WMCore_t.WebTools_t.RESTFormat_t.RESTFormatTest:testEncodedInput was deleted. Prior status was failure
WMCore_t.WebTools_t.REST_t.RESTTest:testException was deleted. Prior status was success
WMCore_t.WebTools_t.REST_t.RESTTest:testA was deleted. Prior status was success

Please merge, @ticoann


basic.return get message which failed

reading the amqp doc, and looking at the code, delivering a mandatory:true message on a queue that has no consumer. is it to correct to assume the following (psuedo code):
msg_failed = false;
exchange.on('basic.return',function(){
msg_failed = true;
});
exchange.on("basic.ack',function(ack){
if (msg_failed === true){
sequence_number = args.deliveryTag.readUInt32BE(4);
// this is the message sequence that failed
});
success_seq = exchange.publish("happy_queue", {foo:'bar'}, {mandatory:true}).sequence;
failed_seq = exchange.publish("sad_queue", {foo:'bar'}, {mandatory:true}).sequence;
i'm seeing this work, in general, but not under load, with millions of messages. the sequence that comes in the next ack after a basic.return is in fact a message that was delivered and consumed by the happy queue.
thanks.

this is an oversight in node-amqp I think, in this ruby lib the payload is available:

A returned message handler has access to AMQP method (basic.return) information, message metadata and payload (as a byte array). The metadata and message body are returned without modifications so that the application can store the message for later redelivery.

in exchange.js:179 the message should be retrieved just like in the basicAck branch above and passed on to the callback IMO.
I'm probably going to write a PR for this now.


Unstable 1.0

Dear mod author, please accept my humble pull request with the B19 update.

Good job as always. Thank you for your hard work.


Oauth prompt fails to start

After following the guide on using the OAuthPrompt dialog from botbuilder-dialogsand it seems like all requests related to retrieving the user token for a connection are failing with a status code of 401, thus the prompt never makes it past the stage of trying to check if we have a token before starting the dialog prompt for the oauth connection.
I have properly configured appId and appPassword and am able to successfully use other api methods like getActivityMembers from the BotFrameworkAdapter adapter instance. However I noticed that when the SDK tries to make the request to check if we have a token for a user/connection it does not send any authorization header, thus it makes sense that the request fails with a 401 status code.
Here is the full error message + stack trace:
{ Error
    at new RestError (/Users/agonbina/Gameon/bot/node_modules/ms-rest-js/lib/restError.ts:13:5)
    at OAuthApiClient.<anonymous> (/Users/agonbina/Gameon/bot/node_modules/botframework-connector/src/oAuthApiClient.ts:83:21)
    at Generator.next (<anonymous>)
    at fulfilled (/Users/agonbina/Gameon/bot/node_modules/botframework-connector/lib/oAuthApiClient.js:11:58)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:118:7)
  code: undefined,
  statusCode: 401,
  request:
   { headers:
      { 'Content-Type': 'application/json; charset=utf-8',
        'User-Agent': 'Node/v9.10.1 (x64-Darwin-16.7.0) ms-rest-js/0.1.0 botframework-connector/4.0.0 Microsoft-BotFramework/3.1 (BotBuilder JS/4.0.0-preview1.2) Azure-SDK-For-Node',
        cookie: '' },
     rawResponse: false,
     url: 'https://api.botframework.com/api/usertoken/GetToken?userId=default-user&connectionName=Facebook',
     method: 'GET' },
  response:
   { body:
      PassThrough {
        _readableState: [ReadableState],
        readable: false,
        domain: null,
        _events: [Object],
        _eventsCount: 4,
        _maxListeners: undefined,
        _writableState: [WritableState],
        writable: false,
        allowHalfOpen: true,
        _transformState: [Object] },
     headers: Headers { _headers: [Object] },
     status: 401 },
  body: undefined }

Happy to provide more information if necessary, otherwise this should be simple to re-produce by following the example provided in the docs for OAuthPrompt.

Sorry for the delay. We will look into this. @Jeffders any thoughts?

@Stevenic any updates?

Steve and I just pushed some fixes. Give it a try now.

I just fetched form repo and run the oauth sample with the same error mentioned from @agonbina . Really need some help on this. we tried azure b2c without success and the same with directly addressing facebook login as provider. would love to use azure b2c but no luck on setting it up.

@tschnoelzer same here, I tried pulling the repo and installing the npm packages from the pulled local folder but the issue does not seem to be fixed. It seems like the release process for this project is kinda messy right now. I am also not sure why the built source(javascript files) are versioned given that most of the codebase seems to be written in Typescript :)

So @Jeffders and @Stevenic pls reopen ... we have really problems in implementations for a new project ... it is really easy to test ... with preview 1.2 no sample implementation is working as the rest http error always occurs.

The samples work fine if you use symbolic links to what's actually in the SDK instead of that preview version. I believe Steve is going to do a pass through all of the botbuilder packages in the packages.json, but for now, just create symbolic links to what is in the SDK or use the latest SDK releases.

Hi Jeff ... I know sym links in Linux / Mac but not sure what u mean by linking sdk ?!? ... can u briefly describe? Thanks a lot !
Outlook für iOShttps://aka.ms/o0ukef beziehen

From: Jeff Derstadt<EMAIL_ADDRESS>Sent: Tuesday, June 5, 2018 8:35:45 PM
To: Microsoft/botbuilder-js
Cc: Timo Schnoelzer; Mention
Subject: Re: [Microsoft/botbuilder-js] Oauth prompt fails to start (#231)
The samples work fine if you use symbolic links to what's actually in the SDK instead of that preview version. I believe Steve is going to do a pass through all of the botbuilder packages in the packages.json, but for now, just create symbolic links to what is in the SDK or use the latest SDK releases.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/botbuilder-js/issues/231#issuecomment-394815438, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA6W0FgCFsClCl_tlWXU_yHEoh9nhDHnks5t5s-BgaJpZM4T6c20.

I can not get the token back when using AAD B2C. So the code here
session.send("Let's see if that code works...");                         adapter.getUserToken(session.message.address, connectionName, session.message.text, (err2, tokenResponse) => {                             if (tokenResponse) {                                 session.send('It worked! You are now signed in with token: ' + tokenResponse.token);                                 session.userData.activeSignIn = false;                             } else {                                 session.send("Hmm, that code wasn't right");                             }                         });
delivers a tokenResponse only with connectionName. No token. I have openId, BotLogin and user_impersonation in my scopes ( Portal: settings / Authentication ). Is there a problem with B2C and BOT? please feedback as we have to switch to another identity provider in that case. What configuration is correct?
Thanks for any clue!
Timo

the Method getUserTokenWithHttpOperationResponse(userId, connectionName, magicCode, options) has undefined options which set the header. In my case just calling GET on https://api.botframework.com/api/usertoken/GetToken?userId=default-user&connectionName=CmbAuth$

so to all facing that problem: I could easily get it running now by changing mit package.json reference from version 1.2preview to a file link of the latest code in repo. Just the way @Jeffders already suggested.
I used just file links will dig into handling symlinks later.
 "dependencies": { "botbuilder": "file:/Users/tschnoelzer/Documents/development/CmbEnterprise/projects/comboo-pack/botbuilder-js/libraries/botbuilder",     "botbuilder-dialogs": "file:/Users/tschnoelzer/Documents/development/CmbEnterprise/projects/comboo-pack/botbuilder-js/libraries/botbuilder-dialogs",     "botframework-connector": "file:/Users/tschnoelzer/Documents/development/CmbEnterprise/projects/comboo-pack/botbuilder-js/libraries/botframework-connector",     "botbuilder-prompts": "file:/Users/tschnoelzer/Documents/development/CmbEnterprise/projects/comboo-pack/botbuilder-js/libraries/botbuilder-prompts",     "botbuilder-core": "file:/Users/tschnoelzer/Documents/development/CmbEnterprise/projects/comboo-pack/botbuilder-js/libraries/botbuilder-core",     "botbuilder-core-extensions": "file:/Users/tschnoelzer/Documents/development/CmbEnterprise/projects/comboo-pack/botbuilder-js/libraries/botbuilder-core-extensions",

Hi, @tschnoelzer
any update?
I cant solve this problem.

@Stevenic @Jeffders will there be a new preview release soon? In order to fix this issue with the current published preview version? Afaik it isn't possible to point directly to the Github master branch from the package.json due to our Lerna structure. Local file pointers are fine, however when you are trying to publish it on Azure it is quite a hassle.

There is a workaround posted on github.
https://github.com/Microsoft/BotBuilder/issues/4768
On Thu, Aug 2, 2018 at 7:21 AM Timo Schnoelzer<EMAIL_ADDRESS>wrote:

also waiting for an update on github ...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/botbuilder-js/issues/231#issuecomment-409893379,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbWhNpcth71UMB-fy6hp2MJKlk9uFjIuks5uMuCcgaJpZM4T6c20
.

--
Thanks,
Jignesh Patel


Access served location of an image, given the shotID

I have the following scenario: Given a SegmentContainer object, I want to access its absolute path and the served location/url(as specified in the cineast.json file) of the image via code. Is this possible? If no, what would be the best approach to implement such functionality in cineast?
Best,
Ribin

Is there already code supporting this functionality or do I need to implement something myself. If it is the latter, which class stores the information about the cineast serve-location of content?

I don't quite understand the question, where would you want to access this information from?

Ideally, I want to access it from my custom Extractor class. I need to send the Url of the current image/video via Rest to a python server and receive feedback.

At the time of the extraction, there is no URL, at least not from cineasts point of view, since the object has not been extracted and hence does not need to be served. You appear to intend to use the whole pipeline very differently from what it was designed to to. The job of the extraction runtime is to abstract all this information away, so that the extractors only need to care about the actual multimedia content of a document and not where it came from. If you really want to do it this way, have a look at the MetadataExtractor, which is capable of accessing a media file directly.

In that case, what would be a better way to achieve what I want to do, by using the pipeline as it was designed?

Using the actual content of the media object which the extraction runtime provides to the extractors.


Feat: Copy object

What kind of change does this PR introduce?
Makes the "copy" API available in the client as described in this issue: https://github.com/supabase/storage-js/issues/29 (I believe this issue is incorrectly marked as closed.)
What is the current behavior?
n/a
What is the new behavior?
I works just like the move() method, but hits up the /object/copy endpoint instead.
Additional context
n/a

Looks good, thanks @haydn 🥳

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update CONTRIBUTING.md

Resolves #449
Adds a line to CONTRIBUTING.md explaining that a SwifterSwift extension cannot be used within another SwifterSwift extension.

          1 Message

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

  Generated by :no_entry_sign: Danger

Can somebody please explain the decision about this design? If class A depends on class B, even if it's in a different module/extension, any additions to class B should be available to class A. This kind of limitation seems to me to be the antithesis of good software design. Otherwise what's the point?


CCC/Docker speed testing and timings

Provide users with a plausibly quantitative story about how much longer jobs will take on the cloud.
This is actually a big concern for users, since most of MDT's simulations are quite computationally intensive.
Tasks: Need to run timings for various MDT tasks CCC. Probably 4 factors to understand:

Docker overhead
CCC/infrastructure overhead
file transfer overhead
AWS instance performance

This would be great - but probably will happen in the context of workflows rather than MDT specifically, so closing it for now.


Update available when it's not

I just updated from 0.0.1 to 0.0.2. VSCode says there is an update but updating does not install a version over 0.0.2.
I've tried uninstalling, reinstalling and restarting VSCode but no change.
Environment
Windows 10 x64
Visual Studio Code 1.5.3

I'm sorry, I believe that's a bug in VSCode. There is an update available, but it's only valid for VSCode 1.6+. You'd need to update VSCode to 1.6 to get the update.

Thank you!

Ref Microsoft/vscode#13620


compilation fails with error: jVM generation is not supported for type other

Description:
Compilation of a module fails with error: jVM generation is not supported for type other.
This should't be the case for this specific module.
Steps to reproduce:
# clone the sample repo
$ git clone https://bitbucket.org/gubaer/ballerina-issue-compilation-fails

# build the client
$ cd client
$ bal build
error: jVM generation is not supported for type other

Comment out line 27-30 in client.bal and build again:
# when line 27-30 are commented out, the build succeeds
$ bal build
Compiling source
	gubaer/grpc_position_service_client:0.0.1

Generating executable
	target/bin/grpc_position_service_client.jar

Affected Versions:
Ballerina Swan Lake Beta 2
Language specification 2021R1
Update Tool 1.3.5
OS, DB, other environment details and versions:
Linux integrationbox 5.11.0-34-generic #36~20.04.1-Ubuntu SMP Fri Aug 27 08:06:32 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
openjdk 11.0.11 2021-04-20
OpenJDK Runtime Environment (build 11.0.11+9-Ubuntu-0ubuntu2.20.04)
OpenJDK 64-Bit Server VM (build 11.0.11+9-Ubuntu-0ubuntu2.20.04, mixed mode, sharing)
Related Issues (optional):
#32818

Minimal version of this scenario
type MyError1 error;
type MyError2 error;

public function main() {
    error? v = getCustomError();
    match v {
        var errorA if v is MyError1 => {

        }
        var errorB if v is MyError2 => {

        }
    }
}

function getCustomError() returns error {
    return error MyError1("");
}

Another minimal version of this issue
type MyError1 error<record { int x; }>;
type MyError2 error<record { int y; }>;

public function main() {
    error? v = getCustomError();
    match v {
        var errorA if v is MyError1 => {

        }
        var errorB if v is MyError2 => {

        }
    }
}

function getCustomError() returns error {
    return error MyError1("Message", x = 2);
}

This issue is already Fixed.
We only adding tests cases in the #37095


Slightly change rules warning messages

This commit comes with an improvement of warning messages by:

Fix typos in excessive length rules by changing to many to
too many.
Change 'configured' to 'allowed', removing the implicit meaning
of a framework configured limit values for rules.
Make Cyclomatic Complexity case removing the inconsistency
with NPath Complexity which was already case.

Current coverage is 94.38%

Merging #22 into master will not affect coverage as of e9d55b0

@@            master     #22   diff @@
======================================
  Files           57      57
  Stmts         1603    1602     -1
  Branches         0       0
  Methods          0       0
======================================
- Hit           1513    1512     -1
  Partial          0       0
  Missed          90      90

Review entire Coverage Diff as of e9d55b0

Powered by Codecov. Updated on successful CI builds.

Gonna open another from master


add custom command line arguments support

Is it possible to add custom command line arguments support?
In CN clients we need to duplicate gw2.dat files to be able to run multiple clients and that would require the usage of "-dat" argument.
And I'm sure there are many use cases where custom command line arguments would be useful.

https://github.com/TheCheatsrichter/Gw2_Launchbuddy/commit/83454fb1472700c99bc6ba1677af0e172c41105f
new feature added.
Coming with next release

The new release supports it nicely. Thanks! Gonna close this issue.


New font weights

See the latest travis tests, e.g. https://travis-ci.org/diagrams/diagrams-pgf/jobs/135630584 .  diagrams-lib has recently added a bunch of new font weights which need to be handled: see https://github.com/diagrams/diagrams-lib/pull/267 .  "Handled" could mean e.g. just adding a catch-all case that does nothing.  I'm not sure if there's something better we could do.

Yeah, I've just ignored other font weights for now. There are ways of dealing with other font weights but I'm not sure the best way to do it. (I think some of them need font packages etc.)

OK, thanks, sounds reasonable.


remove the code converts input to fp16 before calling batch forward kernel

Description
only dequantize and matmul path requires converting to fp16, batch forward kernel doesn't require this
1. Why the change?

2. User API changes

3. Summary of the change

4. How to test?

[ ] N/A
[ ] Unit test: Please manually trigger the Unit test here by inputting the full commit ID (SHA value) of your last commit (e.g., a1b2c3d4e5f6g7h8i9j0). And paste your action link here.
[ ] Application test
[ ] Document test
[ ] ...

PR validation: https://github.com/intel-analytics/ipex-llm-workflow/actions/runs/9756953324/job/26928671073


chore: rm '\'

Build failure and you've got a weird file in here haha

Build failure and you've got a weird file in here haha

This is the worst series of PRs ever.

Looks good


Hookes Law implementation for 1D, 2D (plane stress and plane strain) and 3D

Thank you for your review.
I agree with you that we should be precise and make clear that e12 is not the tensor component here. I tried to document this by stating that 1. / 2 * e12 is the tensor component.
To make things more explicit and avoid confusion I propose to change
"""
e12 : array-like float
        The resulting elastic shear strain component with basis 1-2,
        (1. / 2 * e12 is the tensor component)
"""

to
"""
g12 : array-like float
        The resulting elastic engineering shear strain component with basis 1-2,
        (1. / 2 * g12 is the tensor component)
"""

By using a different symbol it should be clear that we are not talking about the tensor component. A disadvantage of this is that we introduce a new symbol, where g11 is not defined. Regardless of the resolution, we should add the naming convention regarding strain components to variable_names.rst in my opinion.

I agree with you that we should be precise and make clear that e12 is not the tensor component here. I tried to document this by stating that 1. / 2 * e12 is the tensor component.

Ah, yes. That's good. In the meantime I have made another test. I took an arbitrary node from tests/vmap/testfiles/beam_3d_wedge_lin.vmap and used Hookeslaw3D to calculate the strain tensor from the stress tensor and vice versa to compare the results with the FEM results. (see results below). It seems that at least Abaqus also gives out the technical shear strain values and names them E12/E13/E23.
In [9]: stress
Out[9]:
S11    -8.773436
S22     0.581288
S33     0.540051
S12   -22.266996
S13    -2.317505
S23    -9.733004
Name: (3, 8), dtype: float64

In [10]: hook.stress(*strain)
Out[10]:
(-8.773436304377622,
 0.5812879558039988,
 0.5400506430941474,
 -22.26699593940942,
 -2.3175046283728196,
 -9.733005000247028)

In [11]: strain
Out[11]:
E11   -4.554919e-11
E22    1.525652e-11
E33    1.498848e-11
E12   -2.894709e-10
E13   -3.012756e-11
E23   -1.265291e-10
Name: (3, 8), dtype: float64

In [12]: hook.strain(*stress)
Out[12]:
(-4.554918640851974e-11,
 1.5256516814231872e-11,
 1.498847633600235e-11,
 -2.8947095298767094e-10,
 -3.012756037712097e-11,
 -1.265290470123291e-10)

That might be worth a tought. We will probably write an accessor class for the HooksLaw classes so that they can directly operate on PylifeSignal FEM meshes. In a StressSignal the stress components come indeed with capital S. There is no StrainSignal accessor class as of yet, but from VMAP we also get the strain variables with capital E. So consistency would be nice.
But I wouldn't take it too hard. In equistress.py we also have sxx with small 's' as parameter names.


Document to disable role valiadation on Scope creation

Description:
we can disable role validation in scope creation by adding
-DdisableRoleValidationAtScopeCreation=true \

in wso2server.sh file.
But I am unable to see this info in APIM docs. If so plkease add.

@Mariangela please find the 2.6.0 relavent doc

Master branch fix https://github.com/wso2/docs-apim/pull/2635/commits/110410de248a61f270769862a3113e64ff8c6b54


Converting to r171 - issue with normals

Although most TSL textures are now converted to r171, there are three that fail. They have calculation normals. If anyone know what has changed in normalNode during the last few releases, let me know -- this could save me a lot of trial-and-error attempts.

Fixed in v1.5.0.  The solution was to use transformNormalToView.


Upgrading 0.15.8 to 0.16.x fails with "Template file with this name already exists" when template file name clashes

Please verify that this bug has NOT been raised before.

[x] I checked and didn't find a similar issue

Describe the bug*
Don't know if this is actually a bug, but hopefully can help if someone else bumps into this issue.
Background: I am trying to upgrade an InvenTree installation from 0.6.4 to the latest 0.16.3.
I can sucessfully upgrade to 0.15.8 but no further as the invoke update exits with the following exception:
Traceback (most recent call last):
  File "/home/inventree/src/backend/InvenTree/manage.py", line 24, in <module>
    main()
  File "/home/inventree/src/backend/InvenTree/manage.py", line 20, in main
    execute_from_command_line(sys.argv)
  File "/root/.local/lib/python3.11/site-packages/django/core/management/__init__.py", line 442, in execute_from_command_line
    utility.execute()
  File "/root/.local/lib/python3.11/site-packages/django/core/management/__init__.py", line 416, in execute
    django.setup()
  File "/root/.local/lib/python3.11/site-packages/django/__init__.py", line 24, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/root/.local/lib/python3.11/site-packages/django/apps/registry.py", line 124, in populate
    app_config.ready()
  File "/home/inventree/src/backend/InvenTree/report/apps.py", line 44, in ready
    self.create_default_labels()
  File "/home/inventree/src/backend/InvenTree/report/apps.py", line 131, in create_default_labels
    report.models.LabelTemplate.objects.create(
  File "/root/.local/lib/python3.11/site-packages/django/db/models/manager.py", line 87, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/root/.local/lib/python3.11/site-packages/django/db/models/query.py", line 658, in create
    obj.save(force_insert=True, using=self.db)
  File "/home/inventree/src/backend/InvenTree/report/models.py", line 145, in save
    super().save()
  File "/home/inventree/src/backend/InvenTree/InvenTree/models.py", line 156, in save
    super().save(*args, **kwargs)
  File "/home/inventree/src/backend/InvenTree/InvenTree/models.py", line 119, in save
    super().save(*args, **kwargs)
  File "/root/.local/lib/python3.11/site-packages/django/db/models/base.py", line 814, in save
    self.save_base(
  File "/root/.local/lib/python3.11/site-packages/django/db/models/base.py", line 877, in save_base
    updated = self._save_table(
              ^^^^^^^^^^^^^^^^^
  File "/root/.local/lib/python3.11/site-packages/django/db/models/base.py", line 1020, in _save_table
    results = self._do_insert(
              ^^^^^^^^^^^^^^^^
  File "/root/.local/lib/python3.11/site-packages/django/db/models/base.py", line 1061, in _do_insert
    return manager._insert(
           ^^^^^^^^^^^^^^^^
  File "/root/.local/lib/python3.11/site-packages/django/db/models/manager.py", line 87, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/root/.local/lib/python3.11/site-packages/django/db/models/query.py", line 1805, in _insert
    return query.get_compiler(using=using).execute_sql(returning_fields)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/root/.local/lib/python3.11/site-packages/django/db/models/sql/compiler.py", line 1821, in execute_sql
    for sql, params in self.as_sql():
                       ^^^^^^^^^^^^^
  File "/root/.local/lib/python3.11/site-packages/django/db/models/sql/compiler.py", line 1745, in as_sql
    value_rows = [
                 ^
  File "/root/.local/lib/python3.11/site-packages/django/db/models/sql/compiler.py", line 1746, in <listcomp>
    [
  File "/root/.local/lib/python3.11/site-packages/django/db/models/sql/compiler.py", line 1747, in <listcomp>
    self.prepare_value(field, self.pre_save_val(field, obj))
                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/root/.local/lib/python3.11/site-packages/django/db/models/sql/compiler.py", line 1695, in pre_save_val
    return field.pre_save(obj, add=True)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/root/.local/lib/python3.11/site-packages/django/db/models/fields/files.py", line 317, in pre_save
    file.save(file.name, file.file, save=False)
  File "/root/.local/lib/python3.11/site-packages/django/db/models/fields/files.py", line 92, in save
    name = self.field.generate_filename(self.instance, name)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/root/.local/lib/python3.11/site-packages/django/db/models/fields/files.py", line 332, in generate_filename
    filename = self.upload_to(instance, filename)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/inventree/src/backend/InvenTree/report/models.py", line 60, in rename_template
    instance.check_existing_file(path, raise_error=True)
  File "/home/inventree/src/backend/InvenTree/report/models.py", line 118, in check_existing_file
    raise ValidationError({
django.core.exceptions.ValidationError: {'template': ['Template file with this name already exists']}

I noticed that my users (back in the 0.6.4 version) somehow had the following template defined:
  {
    "model": "label.partlabel",
    "pk": 1,
    "fields": {
      "metadata": null,
      "name": "Part Label",
      "description": "Simple part label",
      "label": "label/inventree/part/part_label.html",
      "enabled": true,
      "width": 60.0,
      "height": 30.0,
      "filename_pattern": "label.pdf",
      "filters": ""
    }
  },

If I rename the label field in the above before update (also renaming the actual file in the media folder), then everything works perfectly.
But, after the update, I notice that the 0.16.x helpfully tries to add a new template with the same part_label.html name:
  {
    "model": "report.labeltemplate",
    "pk": 3,
    "fields": {
      "metadata": {},
      "name": "InvenTree Part Label",
      "description": "Sample part label",
      "revision": 2,
      "filename_pattern": "output.pdf",
      "enabled": true,
      "model_type": "part",
      "filters": "",
      "template": "report/label/part_label.html",
      "width": 50.0,
      "height": 20.0
    }
  },

Is this really the cause of the problem, even though the absolute paths are different?
Steps to Reproduce
See above.
Expected behaviour
See above.
Deployment Method

[x] Docker
[ ] Package
[ ] Bare metal
[ ] Other - added info in Steps to Reproduce

Version Information
0.16.x
Please verify if you can reproduce this bug on the demo site.

[ ] I can reproduce this bug on the demo site.

Relevant log output
No response

@vrzh thanks, fix is incoming in https://github.com/inventree/InvenTree/pull/8209


My first revup bar

Topic: bar

Reviews in this chain:
└https://github.com/alexatcanva/revup/pull/2 My first revup bar

#
head
base
diff
date
summary

0
50b7a3bd
37f685bd
diff
Aug 15 14:17 PM
1 file changed, 1 insertion(+)


Mix: Abstract persistent transmission rng

1. What does this PR implement?
Refactor that abstracts rng from persistent transmission
2. Does the code have enough context to be clearly understood?
Yes
3. Who are the specification authors and who is accountable for this PR?
@youngjoon-lee @AlejandroCabeza @danielSanchezQ
4. Is the specification accurate and complete?
N/A
5. Does the implementation introduce changes in the specification?
N/A
Checklist

[x] 1. Description added.
[x] 2. Context and links to Specification document(s) added.
[x] 3. Main contact(s) (developers and specification authors) added
[x] 4. Implementation and Specification are 100% in sync including changes. This is critical.
[x] 5. Link PR to a specific milestone.

Unrelated docker build error


redundant script?

I guess license acceptor script could be redundant while there is yes command.

But it's good to have it as no one would be bothered by the question for acceptance during any installation or upgrade at any time.


AttributeError with 'setNear'

Hi everyone,
I am currently trying to use the tweet location in order to only retrieve tweets from Germany. Nevertheless, I keep having this error :
AttributeError: 'TweetCriteria' object has no attribute 'setNear'
Here is the line of code in 'Main' that is causing the issue.
tweetCriteria = got.manager.TweetCriteria().setQuerySearch('flüchtelinge').setNear('Deutschland').setSince("2015-09-01").setUntil('2015-10-05').setMaxTweets(10)
I am a bit lost here, thank you all for helping out if possible!
All the best,
Marion

the error is obvious as only tweetcriteria under "got" has the attribute "setnear" ,,, it is not present in the "got3"  ..try to change and retry.

You can try Python 3 compatible fork: https://github.com/Mottl/GetOldTweets3


Clarification on subscription callback order w/ REPL example.

Hi.
I'm interested in using svelte-writable-derived, but am a little unclear on a symptom that I am seeing. I have setup a basic test case up in the REPL here: https://svelte.dev/repl/f5de5423133d4c4eb31812f60e436bb1?version=3.44.3
In ./store.js I create a writable store for the options object. Then return a custom store with the subscribe method of the writable along with two entries test which is a writableDerived instance and test2 which is a propertyStore instance.
In App.svelte I add two local subscriptions to the derived stores. In the console output you can see that the writableDerived update callback is occurring after the local subscription callbacks in App.svelte. The same thing is likely happening with propertyStore, but there is no way to log the call order. The options object is not updated before the local subscription callbacks as expected. In the display of data I use a timeout in both local store subscriptions to log the correct / expected values that are available on the next clock tick. This is displayed as optionsString and optionsStringTimeout. My assumption or expectation is that they would be the same which is not the case.
Are my assumptions wrong that the writableDerived callback should trigger before any other subscriptions to it thereby allowing options to be updated before other subscription callbacks?
Thanks for the help..

The order is already documented in the section on reflect — "reflect is called after the derived store's subscriptions are called."
I'm marking this closed, but feel free to comment with any further concerns or requests.

Cool.. Just missed that aspect. I was able to restructure my use case to fit this pattern. Thanks for the quick reply.


Toggle hotkeys

This is a followup to https://github.com/ros-visualization/rviz/pull/1243
It makes the implementation a little cleaner/more efficient, as we discussed

Thanks @AndyZe. I re-filed this PR in https://github.com/ros-visualization/rviz/pull/1363 and I'm going to close this one here. Please review there.


feat: add an option to support custom 404 pages

Thanks for sharing this useful caddy module. I made this tiny feature that allows fallback to another key from the bucket in case the target object doesn't exist. I used to do this with very exotic nginx conf for static website hosting :yum: . Let me know what you think.

(I don't think the CI fail is due to my PR :zipper_mouth_face: . golangci-lint seems utterly slow at the moment.)

One thing I currently do not understand is the example/Dockerfile. How does this improve the workflow? The make docker does the same thing but builds the local version. Where it seems the Dockerfile would build only the published version. That would be painful for local development. However, I have a feeling I'm missing something. There must be a way to override that or something?

I think you didn't miss anything and I missed the Makefile and Dockerfile at the project root :facepalm: . It wasn't clear to me how to use the examples with a local build, so I came up with my own way. Turns out the way you intended this to work is just more natural. Let me just revert my mess, sorry.

Though I wonder if we'd wouldn't also want to support custom pages for other HTTP statuses, such as 403. I'm just unsure how "mapping of values" are usually handled in Caddyfiles. Would something like this even work:
s3_proxy {
  ...
  error_page 404 "_404.html"
  error_page 403 "_403.html"
}

Let's close this in favor of #11. I added support for any error page, not only 404.


chore: impl rivet init

Fixes RVT-4128

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#594  👈 (View in Graphite)
#593
#592
#591
#590
#589
#588
#587
#586
#585
#584
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Documentation for layout

So I have it working however it's not floating on top of my content and instead pushing my content down the page.
How do I give it a z-index so it sits above everything.

Maybe there is a problem with some styles, you can check in www folder how I deal with css for main layout, it could help you to find out what is wrong with yours. In case of any questions or suggestion fell free to ask.

The issue is we don't use scss we use css in NextJS. I am wondering how do we import the style into a nextjs app.
@import '~react-bootstrap-sidebar-menu/dist/sidebar-menu.scss'
That does not work.

Here was a discussion about how to import css style into app. It is mainly a manual work but I don't have any other option for the moment.


Unable to connect using TLS in netopeer2-2.2.28

Hello, I'm testing netopeer2-2.2.28 these days, and I failed to establish TLS connection using files in example_configuration dir.
Here are my steps:

Load xmls and start netopeer2-server

# in netopeer2-2.2.28/example_configuration
sysrepocfg --edit=tls_keystore.xml
sysrepocfg --edit=tls_truststore.xml
sysrepocfg --edit=tls_listen.xml
netopeer2-server -d -v2
[INF]: SR: Connection 20 created.
[INF]: SR: Session 34 (user "root", CID 20) created.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: LN: Listening on <IP_ADDRESS>:830 for SSH connections.
[INF]: LN: Listening on <IP_ADDRESS>:6513 for TLS connections.
[INF]: SR: Triggering "ietf-keystore" "done" event on enabled data.
[INF]: SR: Triggering "ietf-truststore" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-acm" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-acm" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-acm" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-acm" "done" event on enabled data.

Add certificates in netopeer2-cli

netopeer2-cli
> cert display
get_default_trustedCA_dir: Default trusted CA directory does not exist, creating it.
No certificates found in the default trusted CA directory.

> cert displayown
FAIL: No client certificate found, use "cert replaceown" to set some.

> cert add tls_certs/ca.pem

> cert replaceown tls_certs/client.crt tls_certs/client.key

> cert display
-----ca----- serial: 45ce2c8a91ef25294dad0221461662941bd5e11a
Subject: C=CZ, ST=South Moravia, L=Brno, O=CESNET z.s.p.o., OU=TMC, CN=example<EMAIL_ADDRESS>Issuer:  C=CZ, ST=South Moravia, L=Brno, O=CESNET z.s.p.o., OU=TMC, CN=example<EMAIL_ADDRESS>Valid until: Sep  1 10:21:01 2031 GMT

> cert displayown
OK: Using *.crt certificate with a separate private key.
-----CRT----- serial: 47f82c15c5f3457a6d35a7ea427f46e71ca49fc0
Subject: C=CZ, ST=South Moravia, L=Brno, O=CESNET z.s.p.o., OU=TMC<EMAIL_ADDRESS>Issuer:  C=CZ, ST=South Moravia, L=Brno, O=CESNET z.s.p.o., OU=TMC, CN=example<EMAIL_ADDRESS>Valid until: Sep  1 10:28:18 2031 GMT

Connect, and the output of netopeer2-cli and netopeer2-server

# client
> connect --tls
nc ERROR: Server certificate error (unable to get certificate CRL).
nc ERROR: TLS connection to "localhost" failed (certificate verify failed).
cmd_connect: Connecting to the localhost:6513 failed.

#server
[INF]: LN: Accepted a connection on <IP_ADDRESS>:6513 from <IP_ADDRESS>:34544.
[ERR]: LN: TLS accept failed (tlsv1 alert unknown ca).

Are there any steps I did wrong?
The host environment is ubuntu 22.04.
openssl version is OpenSSL 3.0.2 15 Mar 2022 (Library: OpenSSL 3.0.2 15 Mar 2022)
Thank you very much.

Hello, thank you for reporting this, it is a bug. Should be fixed tomorrow.

So you change the code from

user need to add CRL info manually using crl add

to

the CRL info wll automatically be checked if this field exist in certificate

right?

Yes, that's right. CRLs are meant to be optional, but the latest release breaks it and requires netopeer2-cli to set a CRL that is signed by the server's CA.

OK, thank you.
I can successfully connect using the devel branch now.


Add a new topic: translatomics

Hello, everyone!
My partner and I created a new topic called translatomics and wrote some tutorial about it. Besides, workflows can be tested on our Galaxy-based translatome workbench (https://db.cngb.org/galaxy/). We listen carefully to your suggestions.
Best wishes,
Ludong Yang

Hi @ldyang14, thanks for your contribution, you obviously put a lot of effort into this! and apologies for the lack of feedback.
Some general points:

There are a lot of very short tutorials in here, do you think you could merge some of them to make a longer "end-to-end" tutorial?
The "Inferring ribosome stalling or pausing" tutorial contains no hands-on steps. You could make a slide deck out of this if you just want to provide some scientific background, or add it to the intro slides you already made, or make it part of a longer more comprehensive tutorial as suggested in previous point.
Please put any input data on Zenodo
Please add workflow files for any tutorials

Let me know what you think


GCE provisioning fails with flannel errors

As reported on IRC, the Deis v1.9 flannel service reports "Error looking up interface eth0: no such network interface" on a newly provisioned GCE cluster. Potentially having the create-gce-user-data script remove eth0 would fix this (ens4v1 is apparently preferred).
See the original conversation for more detail.

@mboersma it should be fixed in v1.9 :), as GCE provisioning is fully broken. I can do more testing tomorrow and issue the PR for create-gcd-user-data


US110 Display "Trading As" in To do and History

Adds acceptance tests for US110
To test run this branch of ras-frontstage locally:
Us110 dispay trading as #328
Confluence page:
US110: EXT - Display "Trading As" in To do and History for each RU [M]

 US110: EXT Display "trading as" in To do and History [M]

@emilioward works for we when I run that feature on its own but when I run all the acceptance tests it fails.

@tej99 I actually ran all the tests and individually and even debugged through them and they seemed to pass, couldn't understand why they errored in the first place. Must have been due to some race condition possibly

@emilioward @tej99 The issue was the survey was already being added somewhere else in the tests so party returned a 500 when it tried to add it again. I'll investigate how this is happening but for now my solution is just to catch the exception and carry on since it is actually in the required state already.
The other issue was the wait for case to update was just looking at the first case in a list, I've added a version that looks specifically for the correct case(s) to be actionable.


this.off() in ionic 3 - remove event maps

Someone can help me cancel a map event, previously it was map.off () but that does not work anymore.

please share your project files on GitHub

ping

https://github.com/IvanMoreFlores/Evento-Maps/blob/master/src/pages/mapa/mapa.ts
line 520

this?
https://github.com/IvanMoreFlores/Evento-Maps/blob/02a5c1b5260f39faf609fc11d07301fc6bf58b9d/src/pages/mapa/mapa.ts#L521

with that event I want to deactivate the event but the .off does not work
2018-02-05 12:13 GMT-05:00 Masashi Katsumata<EMAIL_ADDRESS>
this?
https://github.com/IvanMoreFlores/Evento-Maps/blob/
02a5c1b5260f39faf609fc11d07301fc6bf58b9d/src/pages/mapa/mapa.ts#L521
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mapsplugin/cordova-plugin-googlemaps/issues/2046#issuecomment-363152935,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdccFLn8ua8U8sKpHRtyP86buY_4AeVoks5tRzavgaJpZM4R4poM
.

In your code, you are wrong from the first.
new GoogleMap('map', { ... })

is not supported way.
this.map = GoogleMaps.create('map', { ...});

is only supported way.

You don't need to execute this.map.refreshLayout() basically.
This method is for internal use recently.

The MAP_READY event is fired only at once when you create a map using GoogleMaps.create().
It means you should use this.map.one(GoogleMapsEvent.MAP_READY).then() instead of this.map.on(GoogleMapsEvent.MAP_READY).subscribe().
You try this.map.off() after the event, but it does not work of course, because the event is already fired.

Please read the public documentation well before using the maps plugin.
https://docs.google.com/presentation/d/1zlkmoSY4AzDJc_P4IqWLnzct41IqHyzGkLeyhlAxMDE/edit#slide=id.p

Then I can not do anything?

You just ask me how to remove the event listener. Then what else? Please describe what is your problem with more details.

Ok, I have a pin with the event
GoogleMapsEvent.MAP_DRAG_END,
and I want to place a button where cancel this event and when you move the map you no longer use the GoogleMapsEvent.MAP_DRAG_END

this.map.off(GoogleMapsEvent.MAP_DRAG_END)

I don't support your editor warning.

Did you confirm console.log(this.map)? I'm not your teacher or friend. So please do your best as much as possible.

I'm sorry, it tells me that there is no .off
next to the console.log

As I said, please do your best as much as possible.

What happend? :O in the map no off

https://github.com/IvanMoreFlores/Evento-Maps/blob/master/package.json#L38
Why do you use 2.1.1. That's too old.

how update cordova-maps a 2.2.2?

You just remove the maps plugin, then install the maps plugin again.
$> cordova plugin remove cordova-plugin-googlemaps
$> cordova plugin add<EMAIL_ADDRESS>--variable API_KEY_FOR_ANDROID="..."

Sorry to bother you, but create a new project with maps 2.2.2 but I still do not get the .off

You definitely don't install the latest version, because the map class does not have the addKmlOverlay() method whch is implement from v2.2.0

Unfortunately, I cant reproduce your issue so far.

Sorry for making you angry, but I have in mind that in the folder of @ onic-native in google-maps are not the events that you mention;

ionic-native/google-maps is not my plugin. I just contribute it. If you have a problem, something missing methods, you need to send a PR.

Try this.
https://forum.ionicframework.com/t/ionic-native-googlemaps-decodepath-returns-undefined/118624/7

ping

Since there is no update for a while, I close this thread.


Support for loading hdd modules

Wassup Wolfy.
Any plan to continue this PR?
Should I close it?

Wassup Wolfy. Any plan to continue this PR? Should I close it?

Well i think noy, but the loading module needs to be fixed the hdd support should be probally ok.

Closing due to inactivity.

Closing due to inactivity.

Perfect... but the last time i was updating the drivers with the ps2_drivers by matta you should check out.


Have the scm tool extract access tokens from the database

The scm tool should be able to take an autograder database file and an id corresponding to a remote_identities table row. The access token can thus be retrivied from the database, and the access token argument isn't needed.

@s111 I believe this is solved by commit d5a7434487e0440cc03b8ba14fedd31c9c78a4f0. I forgot to make it a PR, but feel free to check my code and close this if it is ok.


StdIn/Stdout instead of files?

Hi, instead of opening files, can this library read data from stdin and output to stdout?. This could be useful when you need to pipe image data to other programs.
Regards,
Leonardo.

Hi, that's beyond my ability now. I don't even have any clue what's stdin/stdout...
It sounds really cool to process the data and pass results to another program. I wish I knew how to do it.

@leonardorame Hi, I've added ByteIO → ndarray function, is that what you need?


Getting Invalid request on https://graph.microsoft.com/v1.0/drives/DRIVEID/items/ITEMID when DRIVEID and ITEMID is REMOTE

Thank you for reporting an issue or suggesting an enhancement. We appreciate your feedback - to help the team to understand your needs, please complete the below template to ensure we have the necessary details to assist you.
Submission Guidelines:

Questions and bugs are welcome, please let us know what's on your mind.
If you are reporting an issue around any of the documents or articles, please provide clear reference(s) to the specific file(s) or URL('s).
Remember to include sufficient details and context.
If you have multiple issues, please submit them as separate issues so we can track resolution.

(DELETE THIS PARAGRAPH AFTER READING)

Category

[ ] Question
[ ] Documentation issue
[X ] Bug

A user that has a file "Sharedwithme" can use https://graph.microsoft.com/v1.0/me/drive/sharedWithMe?allowexternal=true to get a list of these files. In the respons the DriveID and ItemID of the shared item (is returned). (A)
When the user use https://graph.microsoft.com/v1.0/drives/DRIVEID/items/ITEMID with DriveID and Item id from (A) the API returns "invalid respons" if the item is not from the users tenant, so getting access to the file if it is external returns error
Expected or Desired Behavior
Observed Behavior
Steps to Reproduce
Thank you.
[ ]: http://aka.ms/onedrive-api-issues
[x]: http://aka.ms/onedrive-api-issues

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


Implement fuel price filling using API aggregations

Following on #1764, re-implement the filling of missing fuel prices in the output / analysis functions using the appropriate aggregated fuel prices that have been collected from the EIA API, and which should now be available from the PUDL DB.

[ ] Order the available fuel price aggregates from most to least specific, so more precise aggregations can be prioritized.
[ ] Iteratively use the stored aggregates to fill in missing fuel prices based on time, location, fuel type, and industrial sector.
[ ] Compare the aggregated fuel prices available from the API to our best guesses of what the aggregates should be for times/places/fuels/sectors where we have a lot of data, to verify that nothing crazy is happening.

The first iteration of this is to simply reimplement the same filling method used by the EIA API process, but using cached values from the bulk data instead of fetching them from the unreliable web API. Concretely, that means reimplementing pudl.output.eia923.get_fuel_cost_avg_eiaapi() using the bulk data.
That output consists of a table of fuel costs aggregated by month, state, and fuel type. All values are for the "electric power (total)" sector (utilities + IPPs). That information is easily available in the bulk data.
The only significant difference I see is that the get_fuel_cost_avg_eiaapi fuel costs are given per unit (though immediately processed into per MMBTU) , whereas the bulk data is per MMBTU from the start. So a few lines of processing code in fuel_receipts_costs_eia923 must be changed to account for this difference.

The fuel costs from the API imputation can be quite different compared to the costs from the bulk data (see scatter plot below. It is a log-log plot of API (x axis) vs bulk (y axis) fuel costs per mmBTU. Both axes are log10 scale ).

I think this is because the current API code ingests aggregate per-unit fuel costs and converts them to per-mmbtu by dividing by row-level fuel_mmbtu_per_unit. The existing bulk data method uses aggregate per-mmbtu fuel costs directly, which stays at the aggregate level. It is possible to extract the per-unit data series from the bulk data and use either process (we would need to change the bulk data ETL code).
I think the question of "which is better" belongs with the second stage of this work (not in scope for this issue) about improving the imputation model & methodology. For now I have used the direct, aggregate fuel_cost_per_mmbtu from the bulk data.

Closed by #1998 with minor scope change: rather than using multiple granularities of aggregates and filling missing values from most to least granular, we use a single level. This single level mimics the behavior of the pre-existing method. Further refinement is left for further imputation model development such as issue #1766


Add AppImage to releases

just to make it more easy for running

There's not much point in making an AppImage for micro. It's a statically-linked binary which should run on any Linux distribution out of the box.

Yeah I don’t see the point of releasing an AppImage. I don’t see why an AppImage is more easy to run than a static binary.


[Improvement] [23] Inconsistency when executing Training Program

[x] There is no control on the number of sets, reps inserted by the user when executing a specific Training Program.

the error massege is wrongly indented on the output page


v0.9.6 has error when using command

https://github.com/sdispater/orator/blob/ce6c4c3b22a5e034f5c6a6926572887a76c7ed02/orator/commands/application.py#L4
from ..version import VERSION
application = Application('Orator', VERSION, complete=True)

but version is removed at https://github.com/sdispater/orator/commit/9498167aa98d94138333f213b83e3ee1229b8dfe

Sorry about that. I will definitely release a hotfix by the end of the day.

Version 0.9.7 is out!

appreciate


add: free course learn reactjs

add Belajar React JS Dari Awal Buat Yang Nggak Jago JavaScript

What does this PR do?
Add resource(s)
For resources
React JS
Description
Add React JS for indonesia locale
Why is this valuable (or not)?
Yes, I want to add more information about free courses for Indonesian
How do we know it's really free?
its free
For book lists, is it a book? For course lists, is it a course? etc.
is youtube
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


docs: Fix link to architecture diagram

Description
Changes proposed in this pull request (what was done and why):

Fix the link to the architecture diagram in contributor documentation

Changes refer to particular issues, PRs or documents:

Traceability

[ ] The PR is linked to a GitHub issue.
[ ] The follow-up issues (if any) are linked in the Related Issues section.
[ ] If the change is user-facing, the documentation has been adjusted.
[ ] If a CRD is changed, the corresponding Busola ConfigMap has been adjusted.
[ ] The feature is unit-tested.
[ ] The feature is e2e-tested.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


doc: document that TensorFlow Datasets are not supported as inputs

Closes #265

Codecov Report

Merging #266 (8b7b390) into master (cbc7940) will decrease coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #266      +/-   ##
==========================================
- Coverage   98.27%   98.13%   -0.14%
==========================================
  Files           7        7
  Lines         752      752
==========================================
- Hits          739      738       -1
- Misses         13       14       +1

Impacted Files
Coverage Δ

scikeras/_saving_utils.py
96.73% <0.00%> (-1.09%)
:arrow_down:

Continue to review full report at Codecov.

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


Extending provider and updating resource with each span

Building on @kolanos 's extend-provider PR and including on-the-fly resource updates

This one also builds locally without errors, which is helpful.


Fix delete service when headlessServiceEnabled changed

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
fixes #532

License
Apache 2.0

What's in this PR?

Delete kafka service if not needed.
Checklist

[x] Implementation tested
[x] Error handling code meets the guideline
[x] Logging code meets the guideline

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Closing as this issue has been covered in #799


Enhance tests to handle system specific configurations

Tasks:

[x] Get feedback for changes
[x] Add testing section to the documentation including the possible configuration

Informations:

q
a

Fixed tickets
#1689

BC breaks
none

Documentation PR
https://github.com/sulu-io/sulu-docs/pull/124

I added the missing new lines. For the discussion about two seperate database settings I agree with @carltongibson.

Thank you :smiley: