Make header / sidebar sticky

When the size of the page is greater than the viewport the header and sidebar scroll away. Can we make it so only the inner portion or 'page' scrolls when taller than the viewport?

The sidebar doesn't push the content away anymore, I believe I removed all the handlers and styles for that behaviour too. It's going to be closed by default, as I think that's a more expected behaviour as well. Need to fix the scroll of the content still.

I'm pretty confident this works as intended now. On mobile (smartphones, excluding tablets) the footer has a little shadow on top because of the content scrolling behind it. Also maybe it'd be good to actually not make the footer sticky on mobile because it limits the real estate for scrolling and on smaller phones (less height) it's visible and overlapping the log in and other forms. What do you think?

I wouldn't make the footer sticker at all.
Kindest Regards,
Dean

On 29 Jan 2016, at 01:03, Karolina Szczur<EMAIL_ADDRESS>wrote:
I'm pretty confident this works as intended now. On mobile (smartphones, excluding tablets) the footer has a little shadow on top because of the content scrolling behind it. Also maybe it'd be good to actually not make the footer sticky on mobile because it limits the real estate for scrolling and on smaller phones (less height) it's visible and overlapping the log in and other forms. What do you think?
—
Reply to this email directly or view it on GitHub.


feat(app): android deeplink

Can we squash the commits @vkubiv

Codecov Report
Base: 90.81% // Head: 90.81% // No change to project coverage :thumbsup:

Coverage data is based on head (e140cd3) compared to base (2704d69).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #210   +/-   ##
=======================================
  Coverage   90.81%   90.81%
=======================================
  Files          42       42
  Lines        1633     1633
=======================================
  Hits         1483     1483
  Misses        104      104
  Partials       46       46

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.


Fix video noise

On the TM4C123 there's bad video noise, especially in the border on blank lines.
I think I need to make a better VGA interface with much much shorter cables.

In extremis, we could use an AD813ANZ triple-channel video amplifier. But it's about £9.

Or a couple of cheap op-amps? https://www.proto-pic.co.uk/op-amp-thru-hole-lm358.html

Analog have a ton of parts, even one that's a PDIP.
https://www.analog.com/en/parametricsearch/11117#/p2687=|3|4&p4502=50000000|1000000000&ps5=14 ld PDIP|14 ld PDIP (300 mil)|16 ld PDIP (300 mil)|8 ld PDIP|8 ld PDIP (300 mil)


Why does elm-repl depend on the existence of elm-package.json?

I just started elm-repl and it gave me this error:
elm-make: elm-package.json: openBinaryFile: does not exist (No such file or directory)

I readded the elm-package.json, and it worked.
Also where does elm-repl store its repl errors? Is this location configurable? I would prefer not to have lots of files littering my home directory.

elm-repl is backed by elm-make. This means you can build any packages that are out there. You need elm-package.json to specify which packages you are using, which are at least the core libraries. It is a feature of Elm's package manager that there are no global installs at all, so every project is sandboxed.
The temp files sticking around was a regression that I just fixed and will be out in the next release relatively soon.
Knowing this stuff, is it possible to close this issue or refine the question?

Actually, I think #54 addresses the root of this in a clearer way, so I'm going to close this one. Please clarify if that issue does not truly capture what you are looking for!

It was just unexpected that making elm work required the existence of the json file in the current directory. I haven't used any other REPLs with that requirement. Sometimes I'm using non-development non-elm directories. Also the error is kind of cryptic.

Makes sense to me, #54 makes a good case for the same thing.

At the very least guides that introduce the elm REPL, like https://guide.elm-lang.org/types/reading_types.html should have a quick aside that it needs to be run in a elm project directory.


Unable to Install ?  404 - https://hacs.xyz/docs/setup/download/

Bug Description
Following installation instructions here: https://llm-vision.gitbook.io/getting-started says recommended installation method is to follow instructions provided by this link,  https://hacs.xyz/docs/setup/download/.   This link results in a 404.
Edit: I worked it out, but the instructions may be confusing for some.

Thanks, the HACS url has apparently changed. Docs have been updated with the new link.


:feet: Add sections to the provider create form

When creating a new provider, make it clear that some information is provider details, while other is provider credentials
Screenshot:
Before:

After:

Addresses: #870


Introduce housekeeper and fix deployment scripts

Coverage increased (+3.7%) to 92.483% when pulling 3f723c9c8550e35b0b2c766f8a1709d0f160c625 on SAP:introduce-housekeeper into<PHONE_NUMBER>2512c79d9f04f7d51e84b1f188d021 on cloudfoundry-incubator:master.

Coverage decreased (-0.2%) to 92.307% when pulling 2b53b1413b2cff0e03271e01356f407ae3cc9395 on SAP:introduce-housekeeper into 1f14126a0ef0c5f3dd045734aa48d648b0ae6845 on cloudfoundry-incubator:master.


Nested API parameters doesn't work in query() method

Using query() with nested data parameters doesn't work with current implementation for an invokation like:
keen.query('count', 'listing-view', {
	timeframe: {
		start: '2010-01-01T12:00:00Z',
		end: '2010-01-10T12:00:00Z'
	}
)

It seems like Keen API doesn't extract query string parameters correctly. Changing the request methodology to using POST and sending it via request body works as expected.

Closed via #3


Update core team members list

Add 'Prerna Sharma'  as a core team member.
Name: Prerna Sharma
Img Link: https://avatars.githubusercontent.com/u/89515816?v=4
Profile Link:  https://github.com/Prerna-0202

@Prerna-0202 Can i work on this issue?

Please let me work on this issue . It would be my 1st issue in this organisation


Fix typo in architecture matching

The select_archive_format function was looking for darwin-* and windows-* where the detect_platform function was producing os_arch shaped answers.
Fixes #32 and #39

No worries.  Just happened to see it while I was here.


Platform ready and no network : Async is not considered

Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/driftyco/ionic-v1)
[ ] 2.x
[x ] 3.x
I'm submitting a ...  (check one with "x")
[ X] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
When having a bad or no network, the platforn.ready() takes a serious long time on iOs and Android if you have externals scripts (Facebook pixel, google maps), even with the "async" attributes
To compare, the deviceready event of Cordova will always fire after couple of seconds (with Wifi, or no network). But the platform ready of Ionic will not... and if you are waiting for it to change your rootPage or some others codes (like splashscreen.hide(), set the language of the ngx-translate,..., it can take a really really long time (30-60seconds sometimes.., especially if you have not network at all)
Expected behavior:
On deviceready event, I can see (on my app at least), every cordova plugins and ionic components/lazy loading are working well. So what plt.ready is waiting ? I replaced every plt.ready in my app with the deviceready event and the app looks way faster if you do not have a wifi connection (80% of my users).
Steps to reproduce:

Use the conference ionic starter
Cut your internet on your iPhone or simulator
Launch your app
Just see when the platform.ready promise is resolved

It looks like the async attributes is not considered. Or something like that
Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
global packages:
@ionic/cli-utils : 1.2.0
Cordova CLI      : 7.0.1
Ionic CLI        : 3.2.0

local packages:
@ionic/app-scripts              : 1.3.7
@ionic/cli-plugin-cordova       : 1.2.1
@ionic/cli-plugin-ionic-angular : 1.2.0
Cordova Platforms               : android 6.2.3 ios 4.4.0
Ionic Framework                 : ionic-angular 3.3.0

System:
Node       : v6.9.1
OS         : macOS Sierra
Xcode      : Xcode 8.3.2 Build version 8E2002
ios-deploy : 1.9.1
ios-sim    : 5.0.13

Hello, thanks for using Ionic! Could you provide a repo we could use to reproduce this?

@jgw96 Actually you only need the conference starter
https://github.com/ionic-team/ionic-conference-app
Cut your network, and launch the app. External javascript scripts will block the platform ready promise

@jgw96 What do you need from me ?
I am using Ionic from the beta version, and this is the worst issue I can tell I had, especially when you are using a mobile phone (because the launching time can just be crazy...). From the v1, I always  thought it was Cordova that was not working in offline, but I just found it was Ionic when you are using externals libraries.
Thanks

Hello @tgensol , sorry but I am not able to reproduce this issue on my devices. I'm using a moto g and a Nexus 6 with airplane mode on. With the latest version of the conference app i see no difference in the startup time when its on the network or off. Are you saying that you are using some external libraries that are trying to make network requests?

@jgw96 Hello, thanks for your reply.
I just tried again, and I cannot reproduce the issue. But last time I tried I could do it every time I tried.
I can say that when I am using my app, I can feel that the Platform.ready event is kind of "related" to my network speed. I put some trackers and I can also see it on my users, the event will be delayed with a bad network. I changed every Platform.ready event for deviceready and the lauching time is now really faster and the UX really better.
I will try to do more tests, I thought it was the externals dependencies (or maybe an update of something fixed it), and see if it is still there

👍 sounds good. Since you are not able to reproduce the issue anymore I am going to close this issue for now but, if you are able to reproduce it again feel free to open another issue. Thanks for using Ionic 😃

I've noticed that when running the app in livereload mode and connecting logging information using the -lsc options, once the app is disconnected from wifi, it will not reach the Platform.ready() event in app.component:
$ ionic run android -lsc --device

If I rebuild the app and deploy it without livereload/logging and in production mode then remove wifi, the Platform.ready() is fired as expected:
$ ionic run android --prod --device


[FEATURE] replace manual implementation using from_str function of strum

Feature Description
In the convert_connector function the manual implementation of converting the connector from string can be replaced using the from_str function provided by strum
Possible Implementation
Use from_str function of strum
Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to submit a PR?
No, but I'm happy to collaborate on a PR with someone else

I would like to take this!


How to add edit/delete button at the end of each row?

How to add edit/delete button at the end of each row? On clicking edit/delete i would like call a function.

Hi @pjsaran
You can use row slots to achieve this.
Also, I've made an example for you here https://codepen.io/rubanraj54/pen/EJepqq .
I hope this codepen will help you. You can find more info about row slot here https://rubanraj54.gitbook.io/vue-bootstrap4-table/rows#row-slot .
Cheers,
Ruby.

Hi @rubanraj54
Thanks for the example. It works fine for me now.


Discord server: invite background

This is supported but we don't have it.

Setting this doesn't work BTW, it might be a Discord bug. In which case we should find a support channel.

Setting this doesn't work BTW, it might be a Discord bug. In which case we should find a support channel.

It was this: https://support.discord.com/hc/en-us/community/posts/360049686714-Server-Changes-Not-Saving

It was this: https://support.discord.com/hc/en-us/community/posts/360049686714-Server-Changes-Not-Saving


Please provide documentation for the use of websocket client in Helidon SE

Related to #4554

We don't provide any detailed documentation for the WebSocket API because it is the standard JSR 356 API (for the same reason we don't provide detailed documentation for JAX-RS). Is there something specific you are looking for? There's also tests and examples in our code base that use WebSocket clients. For example:
https://github.com/oracle/helidon/blob/master/webserver/websocket/src/test/java/io/helidon/webserver/websocket/test/EchoClient.java


fix ctrl-w

Seems like that would prevent users to close the tab ?

yes

How can they close the tab then?

esc - left click on tab - close tab

Can you run npm run fix for lint ?

how?

I forgot

npm run fix in your terminal

k done

I just tested this, it doesn't prevent ctrl+w from closing the tab

oh lemme try again

ok.

Done!
😃

Making people install an extension just to avoid Ctrl+w closing the tab isn't a good solution

ok

#117 is not waiting for an implementation, it's waiting for an idea
nobody has an idea yet

done


Update passwords-technical-overview.md

Commented out a broken link.

#assign: @robinharwood, @Xelu86
@robinharwood @Xelu86
Can you review the proposed changes?
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Package: Adds pangyp to devDependencies

Updates packages to latest versions.
Addresses issue described here: https://github.com/sass/node-sass/issues/627#issuecomment-73322407.

Coverage remained the same at 92.31% when pulling 01ca5e23dc1e9593b6051ae1bb52103e9b8b60fe on am11:master into 6683be0cb4f7b6bad771a84cc9919c7f4e32216a on sass:master.

Coverage remained the same at 92.31% when pulling 01ca5e23dc1e9593b6051ae1bb52103e9b8b60fe on am11:master into 6683be0cb4f7b6bad771a84cc9919c7f4e32216a on sass:master.


Mini hack for component builder support

Since you can't mock dagger components (they are final), here is a little hack:
               if (moduleName.startsWith("Dagger")) {
                    moduleName = moduleName.substring(6);
                    for (Class<?> interfaceClass : moduleClass.getInterfaces()) {
                        if (Objects.equals(interfaceClass.getSimpleName(), moduleName)) {
                            moduleClass = interfaceClass;
                        }
                    }
                }

Within getSetterMethod under ObjectWrapper
And final tweak to DaggerMockRule in initComponent
private ObjectWrapper<Object> initComponent(ComponentClassWrapper<?> componentClass,
      List<Object> modules, ModuleOverrider moduleOverrider) {
    Class<Object> daggerComponent = componentClass.getDaggerComponentClass();
    ObjectWrapper<Object> builderWrapper =
        ObjectWrapper.invokeStaticMethod(daggerComponent, "builder");
    for (Object module : modules) {
      builderWrapper = builderWrapper.invokeBuilderSetter(module.getClass(),
          (module.getClass().getModifiers() & FINAL) == FINAL ? module
              : moduleOverrider.override(module));
    }
    return builderWrapper;
  }

Happy testing :-)

Could you please provide an example of a test that fails without this modification?
Thanks for your suggestion, I'll try to integrate it soon!

Whenever a module is dependent on a non-mockable class (i.e. another
component).
On Thu, Jul 20, 2017 at 11:46 PM Fabio Collini<EMAIL_ADDRESS>wrote:

Could you please provide an example of a test that fails without this
modification?
Thanks for your suggestion, I'll try to integrate it soon!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fabioCollini/DaggerMock/issues/48#issuecomment-316824846,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK-E01j3Y-9FZ9a35jKgLOd8G2JiHS-rks5sP7ydgaJpZM4OZZ_k
.


change the about section

You should make it that you can change the about section of the bot.
Many people use that section. So maybe you could make it customizable.
It would make it seem cooler!

This can be done via your bot's description area

As stated, you can use the Bot's description on the developer page to add a bio under your bot's status. There are currently no plans to allow changing of the about command.


Add protocol fee

@Orland0x About the token fee wdyt if we could go with the other approach in which the fee stay in the Boost contract and make it clear we dont support rebase tokens?

yeah we could do that. The collect token fee function would be efficient if lots of the same token gets deposited, however if all the deposits are for different tokens then collection will be pretty expensive (1 erc20 transfer for each token)  and will be paid by us rather than the DAO using it

@Orland0x About the token fee wdyt if we could go with the other approach in which the fee stay in the Boost contract and make it clear we dont support rebase tokens?

yeah we could do that. The collect token fee function would be efficient if lots of the same token gets deposited, however if all the deposits are for different tokens then collection will be pretty expensive (1 erc20 transfer for each token) and will be paid by us rather than the DAO using it

Maybe this is worthwhile. In the worst case it will be like we are subsiding the protocol which is probably a good thing in the beginning. Can always update if it gets too expensive later on


feat: add streaming.StreamingDataFrame class

StreamingDataFrame implemented basic create, projection, filter and repr operations. Delegate the operations to a member DataFrame with disabled cache and snapshot.

Not sure why b/354024943 is caused. Tests are successful locally. Need to keep an eye on it.


Fix: webhook overwrite deployment owned by yad/yas

What type of PR is this?
/kind bug
What this PR does / why we need it:
Currently,  deployment overrider webhook will override the deployment spec owned by a yad(yurtappdaemon) with the yad template. However, the nodeselector field in the deployment spec will be set to null because it's not set in the yad template but set by the yad controller. This leads to issue.
We need to use the applyTemplate in the yad controller to get the complete deployment spec.
Which issue(s) this PR fixes:

Fixes # https://github.com/openyurtio/openyurt/issues/1841
Special notes for your reviewer:

@rambohe-ch  @vie-serendipity
Does this PR introduce a user-facing change?

other Note

/lgtm


The package is now obsolete since Google Finance API has been deactivated

The Google Finance API has been deactivated and is only available for enterprise users only. No public documentation or tutorial is available for this API.

Thanks for saving our time and effort.

  File "/Users/kevin/.pyenv/versions/3.11.1/lib/python3.11/urllib/request.py", line 643, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 404: Not Found

No wonder why I kept getting this error :(

Wouldn't it be possible to create an ad-hoc/unofficial API? Somehow the data comes back when a Google user is logged in.


Add WinitEvent aggregate event for synchronized window event reading

Objective

Allow users to read window events in the sequence they appeared. This is important for precise input handling when there are multiple input events in a single frame (e.g. click and release vs release and click).

Solution

Add a mega-enum WinitEvent that collects window events, and send those alongside the existing more granular window events.

Changelog

Added WinitEvent event that aggregates all window events into a synchronized event stream.

I think that even if we add timestamps to input events, this is still worth having around. It saves users from having to reconstitute this event stream themselves.

The PR you opened removes the step where you forward WinitEvents as individual events to Bevy, which will not pass CI.

Maybe I missed something, but in the PR, I'm accessing the EventWriter<WinitEvent> directly using the existing system state, and sending events into it, no other forwarding is needed unless I'm missing something.

That's what forward_winit_events() was doing, which your PR deleted.

Yes, I missed that. Thanks again for getting this change in, looking forward to properly ordered input events!

In what release will this land? The milestone is not set

By default merged PRs are shipped in the next major version of Bevy. In this case, 0.14 :)

Gotcha!


Cannot set selected values from Material select from API model

Fist of all I would like to say Formly is awesome!
I am trying to load multiselect options and then setting the ones that where previously saved from the user for editing but cannot load it using JSON powered example.
Please see  example:
https://stackblitz.com/edit/angular-a1v7ww?file=src%2Fapp%2Fapp.component.ts

The select expect a scalar value and not object in case you want to pass an object use compareWith https://github.com/ngx-formly/ngx-formly/issues/2272#issuecomment-633296703

seems like is working but the select dissapeared.
https://stackblitz.com/edit/angular-a1v7ww?file=src%2Fassets%2Fjson-powered%2Finvoice-form.json

The select expect a scalar value and not object in case you want to pass an object use compareWith #2272 (comment)

I cant seem to make it work with material multiselect

seems like is working but the select dissapeared.

valueProp and compareWith expect a function and not string value, to make it work for loaded json config check https://formly.dev/examples/other/json-powered

I cant seem to make it work with material multiselect

https://stackblitz.com/edit/angular-kpharh


featurizer_count_vectors with words and N-Grams?

Hey,
I need to use N-Grams with featurizer_count_vectors , but I think it would improve importantly the classification if you could also use it together with word itself. Such that not trained words like for the tensorflow_embedding have a chance if they have specific ngrams.
Can you this easily within the pipelin or is just one possible at the moment?
pipeline:
- name: "intent_featurizer_count_vectors"
  analyzer: 'char'
  "min_ngram": 2  # int
  "max_ngram": 3  # int

Can you put the featurizer as a series, such that first it creates features with words then you put again in the pipeline the featurizer for char?
pipeline:
- name: "intent_featurizer_count_vectors"
  analyzer: 'char'
  "min_ngram": 2  # int
  "max_ngram": 3  # int
- name: "intent_featurizer_count_vectors"
  analyzer: 'word'

EDIT:
I tried the intent_featurizer_ngrams after intent_featurizer_count_vectors in my pipeline:
language: "de_core_news_sm"

pipeline:
- name: "nlp_spacy"
- name: "tokenizer_spacy"
- name: "intent_featurizer_count_vectors"
- name: "intent_featurizer_ngrams"
  # Maximum number of ngrams to use when augmenting
  # feature vectors with character ngrams
  max_number_of_ngrams: 5
- name: "intent_classifier_sklearn"

But this gives error:
Traceback (most recent call last):
  File "C:\Users\\Anaconda3\envs\rasa12\lib\site-packages\rasa_nlu\model.py", line 340, in parse
    component.process(message, **self.context)
  File "C:\Users\\Anaconda3\envs\rasa12\lib\site-packages\rasa_nlu\featurizers\ngram_featurizer.py", line 88, in process
    updated = self._text_features_with_ngrams(message, self.best_num_ngrams)
  File "C:\Users\\Anaconda3\envs\rasa12\lib\site-packages\rasa_nlu\featurizers\ngram_featurizer.py", line 97, in _text_features_with_ngrams
    return self._combine_with_existing_text_features(message, extras)
  File "C:\Users\\Anaconda3\envs\rasa12\lib\site-packages\rasa_nlu\featurizers\__init__.py", line 18, in _combine_with_existing_text_features
    additional_features))
  File "C:\Users\Anaconda3\envs\rasa12\lib\site-packages\numpy\core\shape_base.py", line 288, in hstack
    return _nx.concatenate(arrs, 1)
ValueError: all the input arrays must have same number of dimensions

Right now, analyzer cannot be passed through pipeline, in order to use char you'll have to subclass  intent_featurizer_count_vectors and overwrite its train method: https://github.com/RasaHQ/rasa_nlu/blob/c64991fd30e4ad35246ff82ec19e67f1deb724b3/rasa_nlu/featurizers/count_vectors_featurizer.py#L186 to pass custom analyzer directly to CountVectorizer
I tried the intent_featurizer_ngrams after intent_featurizer_count_vectors in my pipeline - what version of rasa_nlu do you use? It was fixed in latest master.

Thank you! I use 12.3 But I think I try the master. Would it be enough to add anaylzer in
self.vect = CountVectorizer(token_pattern=self.token_pattern,
                                    strip_accents=self.strip_accents,
                                    lowercase=self.lowercase,
                                    stop_words=self.stop_words,
                                   analyzer='char'
                                    ngram_range=(self.min_ngram,
                                                 self.max_ngram),
                                    max_df=self.max_df,
                                    min_df=self.min_df,
                                    max_features=self.max_features,
                                    tokenizer=self._tokenizer)

Am also getting the same error, can anyone has correct answer?

this should be possible, either by subclassing the countvectorfeaturizer or by having two copies in series as you suggest. You could do that by creating a custom component that uses the 'char' analyser (just make sure to give it a different name from the default) I'd be curious to hear how this goes!
The error you post does seem like a bug, because the whole point of the _combine_with_existing_text_features method is to combine arrays of different sizes

I had some success with this, and it'd be a great feature to have available via config rather than custom component!
Would it be good to have it as its own separate component? Such that you could specify very different word vs. subword parameters. I assume that would make them both smart about merging text_features with each other, and they could have default parameters that were appropriate (different min/max n_grams, max_vocab).

awesome idea! I'd suggest you create an issue and we can discuss there what an implementation would look like 🚀


author list integrations

Looking at how to integrate the authorlist (and other services), here's an example UI:

user
in institution
in mou
in author list

sbarnet
x
o
o

karle
x
o
x

vbasu
x
x
x

Here, staff is in the institution but nothing else.  A professor is also in the author list.  A grad student is in the MoU work list as well.  Regardless of if the lines are that clear, the selection seems easy from a UX standpoint.
One key question: do we try to embed "process" logic in here?  For example, should an author list removal be immediate, or after 12 months as is written in policy?  Similar for adding.

I would not put any process/business logic into the authorlist, etc. We have "rules," but a lot of people play hard and fast with them (there is no real police after all), ILs need a sense of ownership, and there may be group internals we are not aware of.  "Randomly" (because ILs don't know or want to live by the "rules") adding people to the authorlist, for example, will just let to grumpy emails. There are also edge cases where removing someone after 12 months may remove them from being author from their own paper, for example.  Generally, I would say trying to impose business logic onto IceCube is a recipe for grumpy ILs.
Instead, we may want to do is highlight people that need attention. Have a fifth column that adds a comment that some action is needed or desired. This still embeds some of the business logic, but does not automate the action. Thoughts?

This sounds reasonable.  I'd probably design the 5th column separately, to keep the logic simple for now.  But that's also where the "expiration" idea could come in.

I think "expiration" is trickier than addition. We have had a few cases where the default "add at 6 months" did not work, but those were kinda weird "this person left us" sorts of events - so not very common. Expiration, on the other hand, gets fun since that is also tied into which papers the author is working on, and the publication dates of those. So any hard expiration dates will run square into that situation.
But part of this we can push back to the collaboration as well. If they want us to do this, we will need to work out the details formally, and make those widely known. If it's too complicated and we can't reach consensus, then we certainly can't implement (correct) process logic.
And at the end of the day, the ILs will have to interact with something anyway. If nothing else, they will have to note when people leave and their departure date. So starting with "check all that apply" or similar seems like a reasonable starting point.

So one more useful category, which is needed for general reporting and issuing of common fund invoices, is whether or not someone is a PhD (and PhD author).

My preference is to have an expiration with a default period, related to the job assignment of the person. Each entry can be classified as "Undegrad student", "PhD student", "Postdoc", "Scientist", "Staff", "Faculty" and so on. An Undergrad can be given a default of 2 year membership, grad students 4 years, and so on. Faculty/Staff can be 5 years.


Should generate strfmt.DateTime as a pointer?

Problem statement
Hi,
It seems when I generate a date attribute of a model, it always use strfmt.DateTime as type, is it possilbe to make it as a pointer ? such as *strfmt.DateTime
Thanks

add x-nullable: true

Cool thanks!


Controller stopped working after recent update

Ever since the virtual input (or the console, I'm not sure which version specifically caused this) was added to the patches my Controller (a Horipad) stopped working. Previously I would launch the game through Steam to get the controller recognized. However, if I attempt to do this now my controller would stay unrecognized and no input would be accepted.

Please fix steam controller support in 1.5 thank you!

My PS5 controller also stopped working, had to disable steam controller input and use DS4win.


[Feat] Refactoring of the code

Coverage increased (+12.8%) to 98.491% when pulling 0e8ca06e5136d182f57912878ae8ebf813eae1ba on improvement/refacto-code into 4dc35385735abedae7a45101f61d52949f6d0a99 on master.

Coverage increased (+12.7%) to 98.425% when pulling e9d1c0fa2487352ac1aadc5aa7d471846b2ca6cd on improvement/refacto-code into 4dc35385735abedae7a45101f61d52949f6d0a99 on master.

Coverage increased (+12.7%) to 98.425% when pulling e9d1c0fa2487352ac1aadc5aa7d471846b2ca6cd on improvement/refacto-code into 4dc35385735abedae7a45101f61d52949f6d0a99 on master.

Coverage increased (+12.7%) to 98.425% when pulling 6326d3cbe9af299f1f1d394e1cb81bce3d1befd7 on improvement/refacto-code into 4dc35385735abedae7a45101f61d52949f6d0a99 on master.

Coverage increased (+12.7%) to 98.425% when pulling 6326d3cbe9af299f1f1d394e1cb81bce3d1befd7 on improvement/refacto-code into 4dc35385735abedae7a45101f61d52949f6d0a99 on master.

Coverage increased (+12.8%) to 98.491% when pulling 07e95a3a3cd45adcfc7c3ab54c8ac45007c7a9da on improvement/refacto-code into 4dc35385735abedae7a45101f61d52949f6d0a99 on master.

Coverage increased (+12.8%) to 98.491% when pulling 07e95a3a3cd45adcfc7c3ab54c8ac45007c7a9da on improvement/refacto-code into 4dc35385735abedae7a45101f61d52949f6d0a99 on master.

Coverage increased (+12.8%) to 98.491% when pulling a44551bb8cb034c201086e295b6b9643ec6538b7 on improvement/refacto-code into 4dc35385735abedae7a45101f61d52949f6d0a99 on master.

Coverage increased (+12.8%) to 98.491% when pulling a44551bb8cb034c201086e295b6b9643ec6538b7 on improvement/refacto-code into 4dc35385735abedae7a45101f61d52949f6d0a99 on master.


Getting plugin not found error

Details
t=2022-03-08T23:52:22+1300 lvl=eror msg="Could not start plugin" logger=plugin.manager pluginId=msupplyfoundation-datasource err="Unrecognized remote plugin message: 2022/03/08 23:52:22 notifying bugsnag: Binary was compiled with 'CGO_ENABLED=0', go-sqlite3 requires cgo to work. This is a stub\n\nThis usually means that the plugin is either invalid or simply needs to be recompiled to support the latest protocol."

In mac grafana tries to use the ARM version of the executable which also breaks saying could not find file.
I did recompiled the app in my m1 mac system but it was broken before I tinkered. I forgot to take the error log entry though.
Definitely getting this error because Grafana for some reasons cannot run the backend executable.

@joshxg any suggestions? or maybe you can recompile the executable to see if Grafana picks it up.

There is a bugsnag for this:

(If you can see: https://app.bugsnag.com/sustainable-solutions-nz-ltd/msupply-dashboard-app-1/errors/62134cc6f577200008b1cac3?filters[event.since]=30d&filters[error.status]=open)
Which isn't particularly useful 😁
Are you use an m1? It might be an issue with cgo not being enabled?  https://www.reddit.com/r/golang/comments/ka5ipb/gosqlite3_requires_cgo_to_work/

Are you use an m1?

@joshxg yes I am using m1, which might not work locally for me, but the exe does get compiled.
Also I tried compiling in the windows demo-board vm server and I get the same error with the exe generated from that version. So I kinda think m1 compiled executable exe might still work if we can get the Go code error free.
I will look into this CGO_ENABLED and will get back to you if I get stuck.
Thank you for the prompt reply.

I enabled CGO to compile the app and got another error:
PS C:\Users\Administrator\Documents\GitHub\msupply-dashboard-app\backend> go run mage.go
# runtime/cgo
gcc: error: unrecognized command line option '-marm'; did you mean '-mabm'?
# runtime/cgo
gcc: error: x86_64: No such file or directory
gcc: error: unrecognized command line option '-arch'
# runtime/cgo
gcc_linux_arm64.c: In function '_cgo_sys_thread_start':
gcc_linux_arm64.c:22:2: error: unknown type name 'sigset_t'; did you mean '_sigset_t'?
   22 |  sigset_t ign, oset;
      |  ^~~~~~~~
      |  _sigset_t
gcc_linux_arm64.c:27:2: error: implicit declaration of function 'sigfillset' [-Werror=implicit-function-declaration]
   27 |  sigfillset(&ign);
      |  ^~~~~~~~~~
gcc_linux_arm64.c:22:16: error: unused variable 'oset' [-Werror=unused-variable]
   22 |  sigset_t ign, oset;
      |                ^~~~
cc1.exe: all warnings being treated as errors
# runtime/cgo
gcc_linux_amd64.c: In function '_cgo_sys_thread_start':
gcc_linux_amd64.c:61:2: error: unknown type name 'sigset_t'; did you mean '_sigset_t'?
   61 |  sigset_t ign, oset;
      |  ^~~~~~~~
      |  _sigset_t
gcc_linux_amd64.c:66:2: error: implicit declaration of function 'sigfillset' [-Werror=implicit-function-declaration]
   66 |  sigfillset(&ign);
      |  ^~~~~~~~~~
gcc_linux_amd64.c:61:16: error: unused variable 'oset' [-Werror=unused-variable]
   61 |  sigset_t ign, oset;
      |                ^~~~
cc1.exe: all warnings being treated as errors
Error: running "go build -o dist\gpx_msupply-datasource_linux_arm -ldflags -w -s ./pkg" failed with exit code 2
running "go build -o dist\gpx_msupply-datasource_darwin_amd64 -ldflags -w -s ./pkg" failed with exit code 2
running "go build -o dist\gpx_msupply-datasource_linux_arm64 -ldflags -w -s ./pkg" failed with exit code 2
running "go build -o dist\gpx_msupply-datasource_linux_amd64 -ldflags -w -s ./pkg" failed with exit code 2
exit status 2

Ok I got the plugin to compile but now I am seeing this error:
logger=plugin.manager t=2022-03-17T01:16:06.72+1300 lvl=eror msg="Could not start plugin" pluginId=msupplyfoundation-datasource err="Unrecognized remote plugin message: 2022/03/17 01:16:06 notifying bugsnag: runtime error: index out of range [0] with length 0\n\nThis usually means that the plugin is either invalid or simply\nneeds to be recompiled to support the latest protocol."

@joshxg how do see the bugsnag bug report? Maybe that has details.
Compile the backend with CGO did not do the trick.  Maybe that annoyance should be added to the README..
Bugsnag is here https://app.bugsnag.com/sustainable-solutions-nz-ltd/msupply-dashboard-app-1/timeline?filters[event.since]=30d&filters[error.status]=open&pivot_tab=error
I can't see anything with a description you've given.. only CGO problems in the last ~1 week.

The point I am at right now:
The source of the error:
logger=plugin.loader pluginID=msupplyfoundation-datasource t=2022-03-22T20:19:12.28+1300 lvl=dbug msg="panic: runtime error: index out of range [0] with length 0"
logger=plugin.loader pluginID=msupplyfoundation-datasource t=2022-03-22T20:19:12.28+1300 lvl=dbug msg=
logger=plugin.loader pluginID=msupplyfoundation-datasource t=2022-03-22T20:19:12.28+1300 lvl=dbug msg="goroutine 1 [running]:"
logger=plugin.loader pluginID=msupplyfoundation-datasource t=2022-03-22T20:19:12.28+1300 lvl=dbug msg="github.com/grafana/simple-datasource-backend/pkg/api.GetEmails({{0xc00029aee5, 0x5}, {0xc00029af10, 0xc}, {0xc0000b2a80, 0x23}}, {0x0, 0x0, 0xc0000b2c00}, 0xaf)"
logger=plugin.loader pluginID=msupplyfoundation-datasource t=2022-03-22T20:19:12.28+1300 lvl=dbug msg="\t/Users/sworup/sussol/Github/dashboard-plugins/msupply-dashboard-app/backend/pkg/api/users.go:23 +0x6e5"
logger=plugin.loader pluginID=msupplyfoundation-datasource t=2022-03-22T20:19:12.28+1300 lvl=dbug msg=github.com/grafana/simple-datasource-backend/pkg/reportEmailer.(*ReportEmailer).CreateReports(0xc00028e820)
logger=plugin.loader pluginID=msupplyfoundation-datasource t=2022-03-22T20:19:12.28+1300 lvl=dbug msg="\t/Users/sworup/sussol/Github/dashboard-plugins/msupply-dashboard-app/backend/pkg/reportEmailer/reportEmailer.go:221 +0x5dd"
logger=plugin.loader pluginID=msupplyfoundation-datasource t=2022-03-22T20:19:12.28+1300 lvl=dbug msg=main.main()
logger=plugin.loader pluginID=msupplyfoundation-datasource t=2022-03-22T20:19:12.28+1300 lvl=dbug msg="\t/Users/sworup/sussol/Github/dashboard-plugins/msupply-dashboard-app/backend/pkg/main.go:51 +0x249"
logger=plugin.manager t=2022-03-22T20:19:12.28+1300 lvl=eror msg="Could not start plugin" pluginId=msupplyfoundation-datasource err="Unrecognized remote plugin message: 2022/03/22 20:19:12 notifying bugsnag: runtime error: index out of range [0] with length 0\n\nThis usually means that the plugin is either invalid or simply\nneeds to be recompiled to support the latest protocol."

@mark-prins
@bijaySussol told me that you are not getting this error?

Nice work @sworup!
I wasn't getting the error on the mac (m1.. which compiled ok for me) - but do see it on the demo server which is windows.

Hi @mark-prins
A few question:

You said it compiled for you on mac. Did you have CGO enabled in mac? I had to install linux docker to build the executables.
I ended up finding a version of sqlite3 without CGO requiremnt, which I thought would be a convinient way.
You said you saw the error in demo-dash, did you compile it in your computer and copy to plugin to demo-dash? Since, the plugin that was there, I compiled it yesterday, before reporting the error there. I wonder if the version you had compiled would work in demo-dash.
Even if the compilation worked on mac, when I went in to the reports, I would get a no user error. Which is on par with the windows error, that would say user_Id[] array is out of bound on key 0. I think mac is also getting the same error as windows, its just that windows rejects it during compilation while mac supresses it.

There is a bugsnag for this:
image
(If you can see: https://app.bugsnag.com/sustainable-solutions-nz-ltd/msupply-dashboard-app-1/errors/62134cc6f577200008b1cac3?filters[event.since]=30d&filters[error.status]=open)
Which isn't particularly useful 😁
Are you use an m1? It might be an issue with cgo not being enabled? https://www.reddit.com/r/golang/comments/ka5ipb/gosqlite3_requires_cgo_to_work/

The error josh mentioned is because the database path (which is hardcoded) could not be resolved. Normally the path is ../data/msupply.db, which looks for the db in the data folder above the plugin folder. Since the dev environment has plugin path set custom, this data directory does not exist. This directory structure works for demo-dash windows server.
In mac I generally change this path to the data directory of grafana, for instance in my installation it was var/lib/grafana so the path would be var/lib/grafana/msupply.db.

Did you have CGO enabled in mac?

Not knowingly - I haven't installed CGO.. it just works for me. Can I check to see if I have GO enabled?

did you compile it in your computer and copy to plugin to demo-dash?

The back end plugin was compiled on demo-dash server, I've been using that to compile the windows binary. Are you able to compile windows binaries on mac?

Normally the path is ../data/msupply.db, which looks for the db in the data folder above the plugin folder

Yeah, that was a bit of a pain when running locally. I just created a data folder a level up on my mac and put the db in there. Wasn't able to reference an absolute path when trying to load the database unfortunately.

@mark-prins
Here's the reason why the errors were showing up and what I did to fix them with #80

CGO issue: The Go executable of the plugin was refusing to load because go-sqlite3 needed the executable to be built with CGO_ENABLED=1. Normally when you build GO code, it builds with CGO_ENABLED=0.

Fix: I managed to work up enabling CGO in demo-dash sever and even in my docker Linux environment but it is too much of a hassle to set up correctly. I found a version of sqlite3 that was CGO free modernc.org/sqlite plugin and replaced the package in our code. That made things easier as I could build with CGO_ENABLED=0

After that fix I had emailport being an string when int was expected.

Turns out in my other fix for settings, I had mistakenly saved emailPort as emailport. So now the response was sending both, and the system was looking at emailport. Fixed it by renaming the variable. f53b79b

After that the system wanted to load a single scheduler expecting a scheduler resorve. Added that and that was fixed. e98fb24
A minor bug was the frontend filter to filter user in Report group page was bugging out, it wasn't filtering correctly.

After some head scrating, found that the li component loop expects an unique key, the key was set as name field, which could have duplicate values. I set id as the key and that fixed it 5c64472

Main error: The main error was user array out of bound at 0. The reason this error was showing was because one of the scheduler had a report group selected that did not have any users. Since the users number is zero it was erroring out as our go code did not have code to handle this situation. In normal flow this would have stilled errored out but silently. But since this happened during the initialisation phase, while trying to load pre-existing schedulers, Grafana just said, "hay this plugin executable has error, I am not going to load it."

I check the case where users in users group is 0, and just added a break so that codes after that would not try to run. That temperorily fixes it but I think there are graceful ways to do this.

Thanks, @sworup

Main error: The main error was user array out of bound at 0. The reason this error was showing was because one of the schedulers had a report group selected that did not have any users. Since the users number is zero it was erroring out as our go code did not have code to handle this situation.

This information helped me to fix this error in Windows server 2016. It was shown when we restarted Grafana and "Plugin unavailable" message showed when we accessed the plugin setting pages.
t=2022-04-01T04:32:19+0100 lvl=eror msg="Failed to start plugin" logger=plugins.backend pluginId=msupplyfoundation-datasource error="Unrecognized remote plugin message: 2022/04/01 04:32:19 notifying bugsnag: runtime error: index out of range [0] with length 0\n\nThis usually means that the plugin is either invalid or simply\nneeds to be recompiled to support the latest protocol."

It was not a permanent solution but inserting a record manually into ReportGroupMembership table enabled me to use the scheduler report anyway.
I really appreciate your bug fix in #80 !


Wait 10 seconds for sqs.ReceiveMessage

This should significantly reduce the number of sqs.ReceiveMessage calls we make.
/cc @phobologic

👍


All except video ads are not closing (interstitial ads), iPhone 6 (not 5)

Ive not seen this as much as today. Tested on 3 of my apps that are using admob-pro interstitial ads. All on iOS. When an ad is presented (see attached image) the "x" does not close the ad. No matter how much you press on it/around it or even in the other corners. In fact, its not possible to press anywhere in the ad which makes any effect. Any idea what can be causing this and/or what I can do about it? Note: If its a "video ad" where the "x" comes up a bit later all works as intended and tapping the "x" does indeed close the ad. Note2: I would also like to add that this happens for more than just this specific "Trading ad". Note3: It seems that all is well on iPhone 5 but not on my iPhone 6. Would very much apprechiate if someone can verify if this is a general issue or not? My apps are "Movie Tip - Film recommendations for Your Taste" and "Tv-Series Tip: Tv Show Recommendations". Many thanks in advance!

Has anyone been able to reproduce this? It seems all is fine on iPhone 6s as well as 5. Im completely stomped on this. Id really like to know if anyone else is able to reproduce this. Or, if you can show me some other app which uses this plugin I can try them out to see if I can close ads there...maybe its just my phone with my apps that are the issue.

UPDATE: Sometimes, I see another "x" show up in the top left corner but it disappears very quickly, before I get a chance to click it. Clicking where it showed up after it disappears does noting.
Has really no one experienced or seen this?

No one that has experienced this?

Hi peterel !
I can't reproduce this things.
and I looked for the same case project without this plugin, but I could not find it ;(
I could see [x] button to wait for a while that some InterstisialAd.
Can' t you see [x] button to wait for a while ?
I'm sorry if it's out of the way...

@Takahico Many thanks for you reply and help!
I was just able to reproduce. This happens when not of WiFi and being displayed ads on 4g or 3g. Im on iPhone 6 and this only happens for non-video ads (see attached for the ads that now froze). When it happens the first time, I see 2 [x], one on each top-corner, but for maybe just a second, and then one disappears. The other [x] does nothing and one gets no response clicking anywhere in the ad. All subsequent similar ads (non-video) then only show 1 [x] which does nothing and the entire ad is frozen.
Please:

Can anyone reproduce?
Can anyone give a link or app name to ANY OTHER iOS app that uses this plugin so I can check if this only happens on my phone or is a general error?

Thanks!


Zed2i sensor data (imu, mag, etc) latency?

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues.
[X] This issue is not a question, feature request, or anything other than a bug report directly related to this project.

Description
In messages published by the zed-ros2-wrapper, zed imu message header timestamps appear to have almost 0.8 s latency. From my observations, it seems like they are being incorrectly "backdated" somehow.
Steps to Reproduce

Run the ros2 zed wrapper
Run something like ros2 topic echo -i --field header --once /zed/zed_node/imu/data
Compare the received/logged timestamp with the header timestamp
(Or log messages in a bag file and do the same comparison later)

Expected Result
Shouldn't be more than ~1ms latency for zed imu (or pose, odom, etc).
Actual Result
Using the ros2 zed wrapper, I've been logging zed data in ros bag files. There are two obvious timestamps to look at:

The message header timestamp (this is usually set by the application)
The time-of-logging timestamp

When I look at the difference (2) - (1) for most messages, I see what you might expect: <1 ms for messages that are a simple pass-through, ~100 ms for a 10 Hz laser scan that is timestamped at the start of the scan.
But I see some very strange numbers for the zed messages: For the zed odom and pose messages, there is 300 ms latency. And for the zed imu message, there is 790 ms latency! This means that when a zed imu message is logged, it claims that its data comes from ~800 ms in the past!
I looked at the wrapper code, and while the handling of timestamps is a little complex, they seem to be properly plumbed through from the proprietary function mZed.getSensorsData(sens_data, sl::TIME_REFERENCE::CURRENT);
NOTE: I originally found this because when I plotted signals like the yaw rate, it seemed like the zed was somehow anticipating the other sensors by almost 1 s! So I'm pretty sure that the zed_imu timestamps are incorrectly "backdated", rather than my other sensors all being incorrectly delayed.
The figure shows some interesting zig-zags... but they're small compared to the overall effect:

ZED Camera model
ZED2i
Environment
Ubuntu 22.04
Ros2 Iron
ZED_SDK_Ubuntu22_cuda12.1_v4.0.8
zed-ros2-wrapper from github
Zed2i

Anything else?
No response

Ok, I might have figured something out. I've been going a little crazy because sometimes the latency seems good, sometimes its bad (like 1.1s), and I couldn't figure out why.
Then while I was looking through some recent logs, I saw the timestamps go from good (~0.01s) to bad (~1.1s)! I looked around for other signals, and it seems like it might be the first time that GNSS data is integrated, the timestamps suddenly jump. This would explain why some of my log files are good and some are bad: I've been turning gnss_fusion on and off while trying to debug the heading issue.

I'm going do some more experiments (with gnss_fusion) turned off, in the hopes that I can get consistently reasonable timestamps.
And of course, if it is a problem with gnss_fusion, it would be great if this issue were fixed in 4.1!

... and the experiments were a failure.  After ~20 seconds, the timestamp often but not always goes bad. I wrote a simple rospy node to display the timestamp delta, but wasn't able to figure out any set of parameters that prevented it. So I've given up and just overridden the bad sensor data timestamps in the wrapper.
Here are some logs that show the transition from "good" to "bad" timestamp deltas (key moment is bolded):
from zed_timestamp_node.py:
...
timestamp delta @<PHONE_NUMBER>.655 0.003
timestamp delta @<PHONE_NUMBER>.665 0.002
timestamp delta @<PHONE_NUMBER>.675 0.000
timestamp delta @<PHONE_NUMBER>.685 -1.432
timestamp delta @<PHONE_NUMBER>.695 -1.432
timestamp delta @<PHONE_NUMBER>.705 -1.430
...
from zed_wrapper:
...
[zed_wrapper-6] [DEBUG] [1710791198.654219260] [zed.zed_node]: IMU MEAN freq: 99.3342
[zed_wrapper-6] [DEBUG] [1710791198.654223679] [zed.zed_node]: Barometer freq: 24.7142
[zed_wrapper-6] [DEBUG] [1710791198.654262961] [zed.zed_node]: Publishing IMU message
[zed_wrapper-6] [DEBUG] [1710791198.664214820] [zed.zed_node]: Sensors callback: counting subscribers
[zed_wrapper-6] [DEBUG] [1710791198.664284504] [zed.zed_node]: SENSOR LAST PERIOD: 0.0110271 sec @90.6857 Hz
[zed_wrapper-6] [DEBUG] [1710791198.664290943] [zed.zed_node]: IMU MEAN freq: 99.3299
[zed_wrapper-6] [DEBUG] [1710791198.664295286] [zed.zed_node]: Magnetometer freq: 49.4333
[zed_wrapper-6] [DEBUG] [1710791198.664335418] [zed.zed_node]: Publishing IMU message
[zed_wrapper-6] [DEBUG] [1710791198.664368770] [zed.zed_node]: Publishing MAG RAW message
[zed_wrapper-6] [DEBUG] [1710791198.674386256] [zed.zed_node]: Sensors callback: counting subscribers
[zed_wrapper-6] [DEBUG] [1710791198.674470931] [zed.zed_node]: SENSOR LAST PERIOD: 0.0120158 sec @83.2236 Hz
[zed_wrapper-6] [DEBUG] [1710791198.674478528] [zed.zed_node]: IMU MEAN freq: 99.3249
[zed_wrapper-6] [DEBUG] [1710791198.674518878] [zed.zed_node]: Publishing IMU message
[zed_wrapper-6] [DEBUG] [1710791198.684453665] [zed.zed_node]: Sensors callback: counting subscribers
[zed_wrapper-6] [DEBUG] [1710791198.684566427] [zed.zed_node]: SENSOR LAST PERIOD: 1.44241 sec @0.693282 Hz
[zed_wrapper-6] [DEBUG] [1710791198.684574092] [zed.zed_node]: IMU MEAN freq: 99.3253
[zed_wrapper-6] [DEBUG] [1710791198.684579118] [zed.zed_node]: Magnetometer freq: 49.4289
[zed_wrapper-6] [DEBUG] [1710791198.684643404] [zed.zed_node]: Publishing IMU message
[zed_wrapper-6] [DEBUG] [1710791198.684673777] [zed.zed_node]: Publishing MAG RAW message
[zed_wrapper-6] [DEBUG] [1710791198.694510002] [zed.zed_node]: Sensors callback: counting subscribers
[zed_wrapper-6] [DEBUG] [1710791198.694639548] [zed.zed_node]: SENSOR LAST PERIOD: 0.0101099 sec @98.9129 Hz
[zed_wrapper-6] [DEBUG] [1710791198.694647115] [zed.zed_node]: IMU MEAN freq: 99.3226
[zed_wrapper-6] [DEBUG] [1710791198.694651999] [zed.zed_node]: Barometer freq: 24.7139
[zed_wrapper-6] [DEBUG] [1710791198.694709790] [zed.zed_node]: Publishing IMU message
[zed_wrapper-6] [DEBUG] [1710791198.695702924] [zed.zed_node]: Depth extraction enabled
[zed_wrapper-6] [DEBUG] [1710791198.704563113] [zed.zed_node]: Sensors callback: counting subscribers
Attached is the code for zed_timestamp_node.py
zed_timestamp_node.py.zip

Hi @thandal
thank you for the exhaustive feedback.
I also saw your message on the forum.
Have you tried to do a native C++ or Python application outside ROS where you retrieve the sensor data and check the timestamp to confirm that is a ZED SDK issue and not something caused by the ROS2 Wrapper or by ROS 2?

Good question.
I tried adding imu_timestamp deltas to a couple of different zed sdk sample python programs (export_sensors.py, positional_tracking.py, and depth_sensing.py).  In all cases, the timestamp deltas started out in the range of 0.002 seconds. I saw  timestamp deltas sometimes go slightly negative (meaning the imu data is from the future), but only by small amounts, up to 1/100th of a second. Not great, but not really a big problem.
So I ran some more tests with the zed_ros2_wrapper, and I found something interesting: the bad timestamp delta shift (when it jumps to something in the range of +-1.4 seconds) seems to consistently happen almost exactly 30 seconds after I launch the zed_ros2_wrapper binary. After a quick look through the wrapper code, I can't find anything with that sort of interval, but maybe it is another clue.

Hi, @thandal ,
Thanks for finding this issue. Have you managed to find a solution for the latency issue with ros2? Or you decided to not use ROS2 wrapper to get sensor data?

I haven't found a solution. I was going to test the 4.1 release (as soon as
the wrapper is upgraded, maybe this week) to see if the issue might have
somehow been fixed.
For now, I just override the sensor timestamps to the current time, which
is not great but close enough to let me make progress.
On Mon, Apr 8, 2024 at 10:25 AM jqzhang @.***> wrote:

Hi, @thandal https://github.com/thandal ,
Thanks for finding this issue. Have you managed to find a solution for the
latency issue with ros2? Or you decided to not use ROS2 wrapper to get
sensor data?
—
Reply to this email directly, view it on GitHub
https://github.com/stereolabs/zed-ros2-wrapper/issues/206#issuecomment-2042896710,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAT6ESHTCTBY6MJHARHBWPLY4KSG3AVCNFSM6AAAAABDXD4732VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBSHA4TMNZRGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi, @Myzhar
May I ask if this issue will be fixed in the 4.1 release? Thanks

The SDK team performed tests to try to replicate this problem, but they could not.
The Wrapper will be updated to be compatible with the ZED SDK v4.1 in the next few days, I recommend testing it with the new SDK as soon as it is ready.

Hi, @thandal and @Myzhar
I run the wrapper and the codezed_timestamp_node.py.zip. I kept them running for more than 1min, following are the last several outputs.


Word wrapping should consider glyph print width, not number of code points

This library has surprising behaviour for text where a single glyph is made of several code points, e.g. in Korean (Hangul) where ᄁ + ᅪ = 꽈, which is the same as 꽈, but the former has 2 code points while the latter has 1. This is also a problem for other types of combiner characters like "<̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊", which is a somewhat pathological case but helps as a test case to write a fix against. Wrapping to e.g. 10 columns, I would expect the following text
<̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊ <̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊ <̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊ <̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊ <̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊

to become
<̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊ <̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊ <̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊
<̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊ <̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊

but in fact, word-wrap turns it into
<̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊
<̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊
<̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊
<̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊
<̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊>̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊̊

Reproducer:
import Text.Wrap
import qualified Data.Text.IO as Text
main :: IO ()
main = mapM_ Text.putStrLn $ wrapTextToLines defaultWrapSettings 10 "<̊̊>̊̊ <̊̊>̊̊ <̊̊>̊̊ <̊̊>̊̊ <̊̊>̊̊"

More problematic is the case where the width of a character is greater than 1. E.g.

Resizing the terminal so wrapping should occur:

Resizing it a bit more so wrapping actually does occur (but incorrectly, because part of the text is still clipped at the edge):

Thanks for filing this. It's true that I didn't consider character width when I wrote this, and that came much later in Vty and Brick. In addition to not considering width at all, the problem is even worse: even if this library did consider widths (which require a sophisticated lookahead system), the terminals themselves frequently do not all agree on the widths of the printed results. This problem has made it very difficult to make any guarantees that e.g. Vty will come to the same conclusion about glyph width because the terminal emulators themselves also have internal lookup tables to know how many terminal cells will be occupied by the printed characters. Those tables are often wrong or very, very stale. And Vty has its own internal table (in the C sources) that is also old. When we attempted to update it, it actually made layout behaviors worse on terminal emulators that then came into greater disagreement with Vty as a result!
With all that said, if the more general problem of Unicode width measurement in Vty is something you have any interest in working on and helping to advance, I would be delighted to get your help. As far as this library is concerned, it would face similar problems: what version of the lookup tables to use, how to deal (or not) with terminal disagreements, and what to do about sophisticated Unicode-aware lookahead.


Relation to sosa and ssn

The terms that are defined in the http://www.w3.org/ns/sosa/ and http://www.w3.org/ns/ssn/ namespaces belong to this namespaces. But some of the terms can not be found in sosa or ssn.

Most of the terms are now in the dependencies.


Add Stimulus for user feedback showing FieldSlip PDF job status

New Stimulus controller to give on-page feedback on the progress of generating PDFs.
For returning data from the FieldSlipJobTracker, this adds a new AJAX endpoint: a tiny FieldSlipJobTrackersController with just a show action. It returns the JSON for time, pages, and status.
New intended workflow

User creates a PDF with x number of pages
The Projects::FieldSlipsController :create action responds with a turbo_stream that prepends a new row (because they're most recent first) to the table of field slip PDF jobs, instead of redirecting to the Project :show page
The new row is "noticed" by the Stimulus controller, which starts a timer to check for updates
Stimulus checks the new FieldSlipJobTrackersController :show endpoint for the current status of the tracker every second
The :show endpoint responds with a turbo_stream that updates the row, until the status is done

@mo-nathan — If you can add the "pages" data to the tracker, i'll add it to the controller here.

New approach - use Turbo!
This use case is actually a good demonstration of how Stimulus can be simpler with Turbo, and how it requires a simpler way of thinking about JS.
Instead of writing the Stimulus controller to explicitly update the targets in each row, our FieldSlipJobTrackersController can just return a turbo_stream response with the refreshed HTML for the entire row (identified by DOM id), and Turbo will update the row without us handling all the HTML in JS.

coverage: 94.407% (-0.04%) from 94.448%
when pulling 54ea7f3720c3556c0a7d1d1c159f11685dd2fe75 on nimmo-field-slip-stimulus
into d7853558251f509425ee55a1aae788815638aaf6 on njw-field-slip-pdfs.

@mo-nathan i'm trying to play with this, have started solid_queue locally.
But i'm blanking on this step, do I need to create this project?

Go to a project with a field slip prefix.

Nevermind! got it.

Workflow description updated above.
Current issue: it's not updating old jobs. Somehow they're done, but the status attribute is not getting updated when that happens:
irb(main):011> fj = FieldSlipJobTracker.find(4)
irb(main):011> fj.attributes
=>
{"id"=>4,
 "start"=>54,
 "count"=>6,
 "prefix"=>"PJ1",
 "status"=>"Starting",
 "created_at"=>Wed, 17 Apr 2024 17:46:07.150271000 EDT -04:00,
 "updated_at"=>Wed, 17 Apr 2024 17:46:07.150271000 EDT -04:00}
irb(main):012> fj.done
  TRANSACTION (0.2ms)  SAVEPOINT active_record_1
  FieldSlipJobTracker Update (0.6ms)  UPDATE `field_slip_job_trackers` SET `field_slip_job_trackers`.`status` = 3, `field_slip_job_trackers`.`updated_at` = '2024-04-17 22:20:11.809895' WHERE `field_slip_job_trackers`.`id` = 4
  TRANSACTION (0.1ms)  RELEASE SAVEPOINT active_record_1
=> true

@nimolo

If you have a recent db snapshot, you can use /projects/341 or /projects/334
(FWIW, I've been using the uncleaned recent snapshots, rather than the Oct. clean snapshot.
The disadvantage is that you can't login as someone else.
My workaround is a second account that uses a different email.)
Else just create a new project (including a Field Slip Prefix)

Polling the endpoint is making a LOT of requests for me locally because my jobs aren't getting marked as "done", and although this is a development bug, I figure in the case of any glitch, we don't want to tie up someone's browser with zombie requests like this.
I believe the preferred way to do this, rather than polling the endpoint, is to make the FieldSlipJobTracker model broadcastable. Turbo broadcasts are something I wanted to try anyway, and they were created for exactly this situation.

The new row isn't showing up until I refresh the page.  And I'm seeing bunch of tiny lines with text show up at the bottom of the page.

Yes - I missed an ID change. Updated!

That gets the row in, but it still doesn't update every second.  I just ran a batch of 30 and it showed the initial row, but then sat there without doing any updates.  When I eventually (> 10 seconds) refreshed the page, it reported that it is Done and took a total 5.9 seconds.

Yeah, that's what's confusing me too. I thought maybe the tracker instances were not getting freshly retrieved, so I tried reloading the records in the controller, and that didn't work either, so then i debugged and checked the records directly in the console.
Whenever I did that, they hadn't budged past "Starting".

If they aren't getting past "Starting" in the database, that means SolidQueue is not running.  Remember you have to restart the SolidQueue process if you change code at all even in a completely unrelated part of the system.  I'm not sure why that makes it not work, but it's very different from working with the rails server.

Ahaa. Interesting. Thanks!
I wonder if Solid Queue can work with the foreman gem. Ok - yes it does.
Foreman is a gem that runs a script with a proc to start and stop processes and compile assets. It is the default way to run Rails dev servers currently. The reason I didn't switch to it yet was because it requires us to use a different command to start the dev server: /bin/dev instead of rails s. I didn't want to confuse people.
But check that link out when you get a chance, (you too @JoeCohen and @pellaea) and see if you think switching to foreman is a tolerable option to simplify running the dev server.
A small bonus is that it should clear up our scss deprecation warnings on CI.

foreman is above my pay grade. I didn't understand anything.
I do worry about moving away from the Rails main stream.
On Thu, Apr 18, 2024 at 2:51 PM andrew nimmo @.***>
wrote:

Ahaa. Interesting. Thanks!
I wonder if Solid Queue can work with the foreman
https://github.com/ddollar/foreman gem. Ok - yes it does
https://www.bigbinary.com/blog/migrating-to-solid-queue-from-sidekiq#3-starting-solid-queue
.
Foreman is a gem that runs a script with a proc to start and stop
processes and compile assets. It is the default way to run Rails dev
servers currently. The reason I didn't switch to it yet was because it
requires us to use a different command to start the dev server: /bin/dev
instead of rails s. I didn't want to confuse people.
But check that link out when you get a chance, (you too @JoeCohen
https://github.com/JoeCohen and @pellaea https://github.com/pellaea)
and see if you think switching to foreman is a tolerable option to simplify
running the dev server.
A small bonus is that it should clear up our scss deprecation warnings on
CI.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/2104#issuecomment-2065384305,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFFRFBFOVWUCA3MNK5DY6A56NAVCNFSM6AAAAABGL3ENASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRVGM4DIMZQGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Whew! Debugged the stimulus.
It turns out it works with async, without making too many requests. The "too many" part came from my bad syntax with setInterval - we should only pass the function name, not the executed function results: in other words we should pass setInterval(this.do_thing, 1000) not setInterval(this.do_thing(), 1000).
The other trouble was that we can't call a class function (this.do_thing) within setInterval, because we lose the context of this. It is possible within an IIFE as we have here, but there's no need. Our function is very short.
There's also a new Turbo partial that updates the row. Rather than trying to cram the render(turbo_stream: turbo_stream.replace(partial: "thing", locals: hash)) into the controller, i just put that in a partial that calls the other partial, as I've done elsewhere. That's the easiest syntax that I've gotten to work.

I think all this needs now is pages for the tracker.

Sounds great! Thanks! I’ll check it out in the morning and add the pages
bit.
On Thu, Apr 18, 2024 at 10:03 PM andrew nimmo @.***>
wrote:

I think all this needs now is pages for the tracker.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/2104#issuecomment-2065615293,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYQDURAUDYZUV72F6EQG6DY6B3NPAVCNFSM6AAAAABGL3ENASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRVGYYTKMRZGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

The remaining errors seem to relate to the changes in how the create action works now.
I'll wait for the pages attribute before trying to sort it out.


upload gpu_profiling_en.rst

The original doc is out of date, thus deleted . Updates are pending.


V2.0 update readme improved and added assets folder

Updated more things in the readme

Added assists folder for everything in the readme for better organization

moved v1.0 code into the legacy folder

@ROBERT-MCDOWELL
👍 ?

The other language ReadMe files will of course need to be completely re-written lol
we have changed quite a lot

FINE

Don't worry I'll be re-writing them lol


Create IRC article and link on homepage

IRC article #38

Okay, I've merged this, but I will say that this article is BIG!
If you feel up to it, you might want to make another PR dividing it into pieces. Maybe one on installing the server, another one, maybe even more on managing the server.

I can see about doing that tomorrow. If I do that, my plan would be

Setting up and configuring the server
Managing the server + moderation
Setting up TheLounge + maybe some stuff about maybe it normalfriend-friendly(?)

Probably would have a table of contents on the IRC page, not putting all those links on the homepage. What do you think?


Version 2.1.0

Version 2.1.0
Back-end

Preparing the back-end for authentication.
New prometheus gauges and counter.

Front-end

Adding support of 20+ languages.
Allow sharing URL by QR Code.
Customize tabulation size.
Remove line and others major bugfixes.

Legal

Update contact details
Update Section 5


Bug: Code blocks are not formatted correctly

As discussed in https://github.com/TechbeeAT/jtxBoard/issues/320. Code blocks are not being rendered.
According to GitHub Flavored Markdown Spec indented content is considered a code block. So it shall be rendered as so.

Current approach being tested:

Not sure about it, I'd like the colored background to reach the end of the line, and a bit more spacing... I don't know if it can be done with annotated string. Maybe it's time to move to a different rendering method...


Create a package for this

I would like to use the UPM for this.
Can you create a package.json and set it up as a unity package?

Hi @cpetry, the package version of this repo is a WIP on the package branch--you can test it out by doing the following:

Open the Package Manager from Window -> Package Manager.
In the Package Manager window, find and click the + button in the upper lefthand corner of the window. Select Add package from git URL....
Enter the following URL:
https://github.com/Unity-Technologies/VHACD.git?path=/com.unity.robotics.vhacd#package

Please note that this branch is still a WIP. Hope this helps!

Ah alright! Didn't see the package branch.
Thanks for the hint!


BOT: LAND transfer command

This could either be added as a parameter to the existing .transfer or be it's own .landtransfer

I'm not sure I agree, transferring something like land should be inconvenient. The cases you need to do it are rare.
At least it should require confirmation.

Ok, closing in favor of #792


Relative files autocompletion for imports

Let us say I have a file file_to_import.js somewhere in my directory structure like ../../../some-path/another-path/file_to_import.js
it would be nice if the import statement autocompletes this file, so that we don't have to manually remember the path of the files to be imported like
import something from "file_" // should autosuggest => *../../../some-path/another-path/file_to_import.js*

I think with the existing editor infrastructure (fuzzy searching files) this should be possible. Is this feature possible or is there something like this hidden somewhere in the settings?

This feature is powered by the TypeScript project. Please file a feature request upstream


[UWP] GIF support

Automatically generated task. Any updates to the title and body will be overwritten. Do NOT modify the labels.
UWP implementation of #2871

GIFs already work in UWP


No audio pop

[yes] I have checked checked the known issues and searched Github to ensure this issue has not already been reported.

Description of the inaccuracy:
Audio doesn't work and makes a popping sound no matter what my volume is set to and it makes this game unplayable because I don't have any sound ques and I can't listen to the music at all. Idk if this is rated to my device not being compatible with sdl because of so I would like to change the audio plugin.
Screenshots or videos of 2Ship's behavior:
Screenshots or videos of the original hardware's behavior:
https://github.com/user-attachments/assets/97af30f1-b0d8-458e-af3f-a8cc9bf7f2a6
System Information:

OS: Android 15 beta
GFX Backend: Metal, OpenGL, or DirectX
Game Version: 1.11
Rom Version: 1.0

Additional Information:

The popping sound is like an audio glitch that plays randomly and not when there are extra sounds in the game. As in the video you can't hear anything but for some reason my screen recorder doesn't pick up the popping sounds that I hear?

Same issue on my end - I tried using both a .z64 and .n64 version of the backup rom and both produced the same issue. Upon checking the audio API in the Settings menu, it lists SDL as the only option if that is of any use or help. I am on Android 15.


⚠ Bofrost.lu has degraded performance

In c87531c, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2455 ms

Resolved: Bofrost.lu performance has improved in 75f65ae after 28 minutes.


rollback cql-session check

Checklist

[x] Changes manually tested
[x] Automated Tests added/updated
[x] Documentation added/updated
[x] CLA Signed: DataStax CLA

Sure. In that case, we can simply comment out the call made to isAstraSessionValid method in CQLSessionCache.java.
@Yuqi-Du We can leave the unit tests also since they test the method directly.


fix: check reorg blocks

Commits:

fix: subtract with overflow since the reorg blocks also are less than the boundary
feat: check the count of reorg blocks
The count of reorg blocks should be last_n_blocks, unless the blocks are not enough.

Codecov Report
Base: 68.40% // Head: 69.64% // Increases project coverage by +1.23% :tada:

Coverage data is based on head (58d32fe) compared to base (6b0c44d).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #73      +/-   ##
===========================================
+ Coverage    68.40%   69.64%   +1.23%
===========================================
  Files           21       21
  Lines         3396     3396
===========================================
+ Hits          2323     2365      +42
+ Misses        1073     1031      -42

Flag
Coverage Δ

unittests
69.64% <100.00%> (+1.23%)
:arrow_up:

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

Impacted Files
Coverage Δ

...s/light_client/components/send_last_state_proof.rs
87.83% <100.00%> (+3.22%)
:arrow_up:

src/protocols/light_client/peers.rs
78.00% <0.00%> (+1.24%)
:arrow_up:

src/protocols/light_client/mod.rs
74.67% <0.00%> (+6.81%)
:arrow_up:

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.


Add session storage apis from v27

how can I run the workflow again? apparently the pipeline failed at go mod download -json

got it. apparently workflow parsed version 1.20 as 1.2 . adding quote around it.


Add Shopping Game

Hi! This is my work in progress shopping game automation. It's a little bit iffy about it working. Hence the work in progress. If anyone has any questions please feel free to ask.
Fixes #305

[INFO] [SHOPPING GAME] Trying to complete the Shopping Game...←[0m
[ERROR] JavascriptException: Message: javascript error: Cannot read properties of null (reading 'querySelector')
JavaScript stack:
TypeError: Cannot read properties of null (reading 'querySelector')
at eval (eval at executeAsyncScript (:443:26), :55:38)
(Session info: chrome=116.0.5845.112)
Stacktrace:
GetHandleVerifier [0x011A37C3+48947]
(No symbol) [0x01138551]
(No symbol) [0x0103C92D]
(No symbol) [0x0104170B]
(No symbol) [0x010412D2]
(No symbol) [0x010981A0]
(No symbol) [0x0108508C]
(No symbol) [0x010975DA]
(No symbol) [0x01084E86]
(No symbol) [0x010616C7]
(No symbol) [0x0106284D]
GetHandleVerifier [0x013EFDF9+2458985]
GetHandleVerifier [0x0143744F+2751423]
GetHandleVerifier [0x01431361+2726609]
GetHandleVerifier [0x01220680+560624]
(No symbol) [0x0114238C]
(No symbol) [0x0113E268]
(No symbol) [0x0113E392]
(No symbol) [0x011310B7]
BaseThreadInitThunk [0x76226BA9+25]
RtlGetFullPathName_UEx [0x76F78F9F+1215]
RtlGetFullPathName_UEx [0x76F78F6D+1165]
    Traceback (most recent call last):

File "C:\Microsoft-Rewards-Farmer-master\main.py", line 32, in main executeBot(currentAccount, notifier, args)
File "C:\Microsoft-Rewards-Farmer-master\main.py", line 141, in executeBot
ShoppingGame(desktopBrowser).completeShoppingGame()
File "C:\Microsoft-Rewards-Farmer-master\src\shoppingGame.py", line 23, in completeShoppingGame
res = self.webdriver.execute_async_script(
File "C:\Users\AAA\AppData\Roaming\Python\Python311\site-packages\selenium\webdriver\remote\webdriver.py", line 424, in execute_async_script
return self.execute(command, {"script": script, "args": converted_args})["value"]
File "C:\Users\AAA\AppData\Roaming\Python\Python311\site-packages\selenium\webdriver\remote\webdriver.py", line 345, in execute
self.error_handler.check_response(response)
File "C:\Users\AAA\AppData\Roaming\Python\Python311\site-packages\selenium\webdriver\remote\errorhandler.py", line 229, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.JavascriptException: Message: javascript error: Cannot read properties of null (reading 'querySelector')
JavaScript stack:
TypeError: Cannot read properties of null (reading 'querySelector')
at eval (eval at executeAsyncScript (:443:26), :55:38)
(Session info: chrome=116.0.5845.112)
Stacktrace:
GetHandleVerifier [0x011A37C3+48947]
(No symbol) [0x01138551]
(No symbol) [0x0103C92D]
(No symbol) [0x0104170B]
(No symbol) [0x010412D2]
(No symbol) [0x010981A0]
(No symbol) [0x0108508C]
(No symbol) [0x010975DA]
(No symbol) [0x01084E86]
(No symbol) [0x010616C7]
(No symbol) [0x0106284D]
GetHandleVerifier [0x013EFDF9+2458985]
GetHandleVerifier [0x0143744F+2751423]
GetHandleVerifier [0x01431361+2726609]
GetHandleVerifier [0x01220680+560624]
(No symbol) [0x0114238C]
(No symbol) [0x0113E268]
(No symbol) [0x0113E392]
(No symbol) [0x011310B7]
BaseThreadInitThunk [0x76226BA9+25]
RtlGetFullPathName_UEx [0x76F78F9F+1215]
RtlGetFullPathName_UEx [0x76F78F6D+1165]

displaying an error like this

Looks like the code changed a bit or it's obfuscated because it gets stuck with this method and the manual method I used before

Looks like the code changed a bit or it's obfuscated because it gets stuck with this method and the manual method I used before

Nah, no obfuscation or issues on my end.

displaying an error like this

My bad, that error was supposed to be [SHOPPING GAME] Failed to select cheapest item. which happens when it fails to find the button you'd usually click to select your answer.

[INFO] [SHOPPING GAME] Trying to complete the Shopping Game...←[0m [ERROR] JavascriptException: Message: javascript error: Cannot read properties of null (reading 'querySelector') JavaScript stack: TypeError: Cannot read properties of null (reading 'querySelector') at eval (eval at executeAsyncScript (:443:26), :55:38) (Session info: chrome=116.0.5845.112) Stacktrace: GetHandleVerifier [0x011A37C3+48947] (No symbol) [0x01138551] (No symbol) [0x0103C92D] (No symbol) [0x0104170B] (No symbol) [0x010412D2] (No symbol) [0x010981A0] (No symbol) [0x0108508C] (No symbol) [0x010975DA] (No symbol) [0x01084E86] (No symbol) [0x010616C7] (No symbol) [0x0106284D] GetHandleVerifier [0x013EFDF9+2458985] GetHandleVerifier [0x0143744F+2751423] GetHandleVerifier [0x01431361+2726609] GetHandleVerifier [0x01220680+560624] (No symbol) [0x0114238C] (No symbol) [0x0113E268] (No symbol) [0x0113E392] (No symbol) [0x011310B7] BaseThreadInitThunk [0x76226BA9+25] RtlGetFullPathName_UEx [0x76F78F9F+1215] RtlGetFullPathName_UEx [0x76F78F6D+1165]
Traceback (most recent call last): File "C:\Microsoft-Rewards-Farmer-master\main.py", line 32, in main executeBot(currentAccount, notifier, args) File "C:\Microsoft-Rewards-Farmer-master\main.py", line 141, in executeBot ShoppingGame(desktopBrowser).completeShoppingGame() File "C:\Microsoft-Rewards-Farmer-master\src\shoppingGame.py", line 23, in completeShoppingGame res = self.webdriver.execute_async_script( File "C:\Users\AAA\AppData\Roaming\Python\Python311\site-packages\selenium\webdriver\remote\webdriver.py", line 424, in execute_async_script return self.execute(command, {"script": script, "args": converted_args})["value"] File "C:\Users\AAA\AppData\Roaming\Python\Python311\site-packages\selenium\webdriver\remote\webdriver.py", line 345, in execute self.error_handler.check_response(response) File "C:\Users\AAA\AppData\Roaming\Python\Python311\site-packages\selenium\webdriver\remote\errorhandler.py", line 229, in check_response raise exception_class(message, screen, stacktrace) selenium.common.exceptions.JavascriptException: Message: javascript error: Cannot read properties of null (reading 'querySelector') JavaScript stack: TypeError: Cannot read properties of null (reading 'querySelector') at eval (eval at executeAsyncScript (:443:26), :55:38) (Session info: chrome=116.0.5845.112) Stacktrace: GetHandleVerifier [0x011A37C3+48947] (No symbol) [0x01138551] (No symbol) [0x0103C92D] (No symbol) [0x0104170B] (No symbol) [0x010412D2] (No symbol) [0x010981A0] (No symbol) [0x0108508C] (No symbol) [0x010975DA] (No symbol) [0x01084E86] (No symbol) [0x010616C7] (No symbol) [0x0106284D] GetHandleVerifier [0x013EFDF9+2458985] GetHandleVerifier [0x0143744F+2751423] GetHandleVerifier [0x01431361+2726609] GetHandleVerifier [0x01220680+560624] (No symbol) [0x0114238C] (No symbol) [0x0113E268] (No symbol) [0x0113E392] (No symbol) [0x011310B7] BaseThreadInitThunk [0x76226BA9+25] RtlGetFullPathName_UEx [0x76F78F9F+1215] RtlGetFullPathName_UEx [0x76F78F6D+1165]

Odd one, unsure what's wrong here... That's not even a line that calls querySelector in the latest code.

You can examine this script to determine what is lacking.
https://www.reddit.com/user/RohanPhuyal/comments/164pol8/msn_shopping_game_script_new/

@theskid31, Can you try the latest commit?

hi ,tried it seems to get stuck after loading page then skips onto next step Daily set
the first commit i got a message :[SHOPPING GAME] Failed to get game element.
but nothing on that run just skipped to the daily set

←[38;5;39m2023-08-31 15:31:22,261 [INFO] Using default request storage←[0m
←[38;5;39m2023-08-31 15:31:22,268 [INFO] Created proxy listening on <IP_ADDRESS>:58918←[0m
←[38;5;39m2023-08-31 15:31:22,269 [INFO] Using undetected_chromedriver←[0m
←[38;5;39m2023-08-31 15:31:24,073 [INFO] patching driver executable C:\Users\thesk\appdata\roaming\undetected_chromedriver\undetected_chromedriver.exe←[0m
←[38;5;39m2023-08-31 15:31:25,431 [INFO] Screen size: 1935x1030←[0m
←[38;5;39m2023-08-31 15:31:25,432 [INFO] Device size: 1880x879←[0m
←[38;5;39m2023-08-31 15:31:25,452 [INFO] [LOGIN] Logging-in...←[0m
←[38;5;39m2023-08-31 15:31:28,620 [INFO] [LOGIN] Writing email...←[0m
←[38;5;39m2023-08-31 15:31:29,780 [INFO] [LOGIN] Writing password...←[0m
←[38;5;39m2023-08-31 15:31:50,033 [INFO] [LOGIN] Logged-in !←[0m
←[38;5;39m2023-08-31 15:32:05,306 [INFO] [LOGIN] Ensuring login on Bing...←[0m
←[38;5;39m2023-08-31 15:32:11,250 [INFO] [LOGIN] Logged-in successfully !←[0m
←[38;5;39m2023-08-31 15:32:11,250 [INFO] [POINTS] You have 2,608.00 points on your account !←[0m
←[38;5;39m2023-08-31 15:32:11,251 [INFO] [SHOPPING GAME] Trying to complete the Shopping Game...←[0m
Hooked
←[38;5;39m2023-08-31 15:32:39,926 [INFO] [DAILY SET] Trying to complete the Daily Set...←[0m
←[38;5;39m2023-08-31 15:32:51,965 [INFO] [DAILY SET] Completed the Daily Set successfully !←[0m
←[38;5;39m2023-08-31 15:32:51,965 [INFO] [PUNCH CARDS] Trying to complete the Punch Cards...←[0m
←[38;5;39m2023-08-31 15:32:59,543 [INFO] [PUNCH CARDS] Completed the Punch Cards successfully !←[0m
←[38;5;39m2023-08-31 15:33:06,459 [INFO] [MORE PROMO] Trying to complete More Promotions...←[0m
←[38;5;39m2023-08-31 15:33:06,477 [INFO] [MORE PROMO] Completed More Promotions successfully !←[0m
←[38;5;39m2023-08-31 15:33:06,495 [INFO] [POINTS] You have earned 0.00 points today !←[0m
←[38;5;39m2023-08-31 15:33:06,495 [INFO] [POINTS] You are now at 2,608.00 points !
←[0m

any chance you could look into making the point counter work with this, would help to keep track
thanks

any chance you could look into making the point counter work with this, would help to keep track
thanks

Yeah mean like logs like what searches have?

any chance you could look into making the point counter work with this, would help to keep track
thanks

Yeah mean the logs like what the searches have?

Yeah please but i suppose its not registering due to it never shows completed and we get the error at
res[0] as above

Hello, I tried the quickie script in issue 305, and the quickie script did not work.
I really do not want to use linux, just to complete the shopping game.
No one has figured out to create a brand new script, for the shopping game.
I really do not want to use the rewards farmer tool, so if EastArctica or if anyone can figure out a script for the shopping game, to be used on a windows brower, that would be appreciated.
Thanks for your time.

displaying an error like this

image
My bad, that error was supposed to be [SHOPPING GAME] Failed to select cheapest item. which happens when it fails to find the button you'd usually click to select your answer.
hi eastartica ,what is t fix for the if res[0] == 0 error
thanks

Hello, I tried the quickie script in issue 305, and the quickie script did not work.
I really do not want to use linux, just to complete the shopping game.
No one has figured out to create a brand new script, for the shopping game.
I really do not want to use the rewards farmer tool, so if EastArctica or if anyone can figure out a script for the shopping game, to be used on a windows brower, that would be appreciated.
Thanks for your time.

Similar to what I said in the thread previously, you have 2 options.

Deal with imperfect shopping game  wins.
Have perfect shopping games, with a custom browser extension to overwrites the js as it comes in

Hello, I tried the quickie script in issue 305, and the quickie script did not work.
I really do not want to use linux, just to complete the shopping game.
No one has figured out to create a brand new script, for the shopping game.
I really do not want to use the rewards farmer tool, so if EastArctica or if anyone can figure out a script for the shopping game, to be used on a windows brower, that would be appreciated.
Thanks for your time.

Similar to what I said in the thread previously, you have 2 options.

Deal with imperfect shopping game  wins.
Have perfect shopping games, with a custom browser extension to overwrites the js as it comes in
Manually sent requests, people don't like this for some reason because they think they'll get banned for it. I'm like 95% sure the only way people are actually getting banned is based on searches nothing else.

second option is good i guess

Anyone interested in taking this PR over be my guest, i doing have the time not interest to work on this right now.


Terminal line background color split when terminal line-height set higher than 1.0.

VSCode Version: Code 1.14.2 (cb82febafda0c8c199b9201ad274e25d9a76874e, 2017-07-19T23:22:50.650Z)
OS Version: Darwin x64 16.7.0
Extensions:

Extension
Author (truncated)
Version

vscode-docker
Pet
0.0.16

html-css-class-completion
Zig
1.8.0

gitignore
cod
0.5.0

vscode-eslint
dba
1.2.11

php-debug
fel
1.11.1

php-intellisense
fel
1.5.0

vscode-auto-open-markdown-preview
hnw
0.0.4

phpcs
ika
0.7.0

vscode-duplicate
mrm
1.1.0

atom-keybindings
ms-
3.0.1

vetur
oct
0.9.3

(1 theme extensions excluded)

Steps to Reproduce:

Set user setting: "terminal.integrated.lineHeight": 1.5
Run a command in the integrated terminal that has background coloring: echo -e "\033[1;41m                 \n - OUTPUT LINE - \n                 \033[0m"

Should be fixed in latest


Wily+Gnome support

I tried to install Wily and Gnome, and I got a gnome-fallback-error. I would send a screenshot of some sort, but right when it showed that the chroot setup failed, my keyboard was disabled. I had to restart.

@fennexin,
Please read details in here: #1689.
That pull-request is still "pending" to merge into master so, you can do it yourself...
Regards.

@fennexin I have updated my work and attempt in #1689, if you haven't already fixed the problem yourself, perhaps my approach may give you an idea as to how to solve the problem temporarily until this issue is permanently resolved.


Does GloVe support the Spanish language?

I'm working on a project, and my dataset is in Spanish, and wanted to test GloVe too to see how my model will behave based on different pre-trained embeddings?

People have used GloVe to build embeddings for many different languages,
but we haven't build Spanish embeddings ourselves

Okay! Thanks for the quick reply


fix header logo anchor

Adding display: inline-block changes the browser's default display type from inline, which takes up the whole space to inline-block, which only takes up the space of its children!

PTAL! Thank you 😄 🙏 !

This was fixed in Pat's PR!


typed-jest-expect dependencies should in devDependencies?

Thank you!
It has been fixed in 4b1abf7971113f20fed5ff2013d7b5f06faeb782 and released in 1.2.0.


:bug: fix lang detector. Use default option

There is an issue where on first launch, localStorage is empty, selected lang is "fr-FR" and the top right lang menu is defaulted to LANG.FR-FR

Not same behavior on firefox. I'll fix it

I didn't see the issue because my browser configured with french lang and not french (france). Now i see the issue ;). I'll fix it


Use HW BIST to speed up rowhammer attack

Use FPGA reader/writer modules to speed up rowhammer attack.

Fixes #5


fix broken link in reference doc

Proposed changes

Vitalik buterin's blog is moved to https://vitalik.eth.limo/index.html. see the related pr (https://github.com/vbuterin/blog/commit/6cac3885a34226f0e318145ead4af44f445db1b3)

Types of changes
Please put an x in the boxes related to your change.

[x] Minor Issues and Typos
[ ] Major Content Contribution
[ ] Others

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to reach out. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have read the CONTRIBUTING GUIDELINES
[ ] I have read the CLA and signed by comment I have read the CLA Document and I hereby sign the CLA in first time contribute
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Related issues

Please leave the issue numbers or links related to this PR here.

Further comments
If this is a relatively large content contribution, kick off the discussion by explaining why you would suggest the content contribution, etc...

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


[BACKLOG-26374] steps creating field name conflicts

repeated field names could arise after inserting or deleting value metas
select values not checking for conflicts when renaming

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine,core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine,core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  core/src/main/java/org/pentaho/di/core/row/RowMeta.java
  core/src/test/java/org/pentaho/di/core/row/RowMetaTest.java
  engine/src/main/java/org/pentaho/di/trans/steps/selectvalues/SelectValuesMeta.java
  engine/src/test/java/org/pentaho/di/trans/steps/selectvalues/SelectValuesMetaTest.java
  engine/src/test/java/org/pentaho/di/trans/steps/selectvalues/SelectValuesTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.core.row.RowMeta

Complexity Change: -0.3% :small_red_triangle_down:

org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -0.4% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.step.StepMeta

Branch Change: -0.6% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -2.1% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -1.9% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

@pentaho/kashyyyk please review


Setup automatic publishing

[ ] When a new release is created on GitHub, a GitHub action should go through all the necessary integrity checks and publish the new version to npm, if successful.
[ ] Update the README with installation and usage instructions

Closed via https://github.com/radiovisual/keeli/pull/33


Update versions

Summary
Upgrades versions of Dolittle.Protobuf.MSBuild for the building of the .NET contracts
Changed

Updated Dolittle.Protobuf.MSBuild to latest

Retrigger build once the version has been released


Course 4 : error collecting test_clf.py

There seems to be an error with the scikit-learn version interaction with pickle load.
This was encountered when trying to apply the unit test 'test_clf.py' on commit. sklearn is downloaded with requirements.txt, this may be the problem, a specific version would likely fix this issue.

Greetings!
@chris-caballero, you are absolutely right. To fix the problem one can change scikit-learn to scikit-learn==1.2.2 in the requirements.txt.
Note, that after that there will most likely be an error with conlist() in main.py. It can be solved either by changing min_items and max_items parameters to min_length and max_length respectively, or by removing those altogether.

Hi!
This change of scikit-learn version created issue with cython3 compatibility. The suggestion is to limit python to 3.10 (and in turn update .github/workflows/course4-week3-lab.yml so that:
`on:
Triggers the workflow on push request events only when there are changes in the desired path
push:
paths:
- 'course4/week3-ungraded-labs/C4_W3_Lab_4_Github_Actions/'
- '.github/workflows/'`
(or trigger the pipeline manually)

Hi everyone! Thank you for reporting, and sorry for the delay! The problematic files are now updated, so the learners shouldn't run into the issues anymore.