Refresh / rerender chart with new props

I have 3 options to show Chart data. I load 30 days data to chart - it looks great.

Now if I fetch last 14 days data by selecting a radio button, it updates the props and data is changed now as displayed in chart below.
And Chart lines are also updated but it still shows old data points from previous draw.

I need to totally rerender the chart with new data when props changes.
Can anyone please help?
Thanks

Try wrapping it in a view and adding the key with a value of the radio button. In theory react should discard a component of the key changes. It's a hack, but it's pretty simple.

That solved it! Thanks a lot @Hermanya

Try wrapping it in a view and adding the key with a value of the radio button. In theory react should discard a component if the key changes. It's a hack, but it's pretty simple.

You save my day~~

Still better workaround for refreshing chart data


migration module is not published for Scala 3

https://github.com/apache/incubator-pekko-persistence-r2dbc/blob/main/project/Dependencies.scala#L100
pekko-persistence-jdbc is a test dependency - this does not support Scala 3 yet and when it does it will be a 1.1.0 release
I think the simplest is just to upgrade to pekko-persistence-jdbc 1.1.0 when released.
Alternatives include testing a Scala 3 build but use the pekko-persistence-jdbc_2.13 jar.
cross CrossVersion.for3Use2_13 in sbt

wdyt @mdedetrich @raboof @He-Pin @Roiocam @samueleresca @gmethvin @nvollmar ?

I might not have all context, but from what I've seen I'd lean towards upgrading to pekko-persistence-jdbc 1.1.0 when released. Is there any particular downside to that approach?
CrossVersion.for3Use2_13 is great for 'leaf' projects but can lead to problems 'downstream' when used in libraries: for example when the transitive library releases a version for Scala 3 but the library using for3Use2_13 is still using the cross-built version, a project could get in trouble trying to load both. Also I'm not sure how moving from using for3Use2_13 to not using for3Use2_13 in a later version might impact binary compatibility.

1.1 snapshots support this - https://repository.apache.org/content/groups/snapshots/org/apache/pekko/pekko-persistence-r2dbc-migration_3/


Add real tests.

We should have real tests with real asserts, the doctests don't actually verify the contents.

Also tests with drop types

and panics

feel free to work on this if I haven't self-assigned it.


Remove lint warnings

I followed the suggestion from lint

Codecov Report

Merging #337 (ecab9c7) into master (37d78d7) will increase coverage by 0.30%.
The diff coverage is 90.00%.

@@             Coverage Diff              @@
##             master     #337      +/-   ##
============================================
+ Coverage     86.78%   87.09%   +0.30%
  Complexity      371      371
============================================
  Files            33       32       -1
  Lines          1415     1410       -5
  Branches        165      164       -1
============================================
  Hits           1228     1228
+ Misses          117      113       -4
+ Partials         70       69       -1

Impacted Files
Coverage Δ

tracker/src/main/java/org/matomo/sdk/Matomo.java
86.84% <ø> (ø)

...r/src/main/java/org/matomo/sdk/TrackerBuilder.java
79.31% <ø> (ø)

...java/org/matomo/sdk/dispatcher/EventDiskCache.java
73.07% <0.00%> (ø)

...ain/java/org/matomo/sdk/extra/CustomVariables.java
94.87% <ø> (ø)

...a/org/matomo/sdk/extra/MatomoExceptionHandler.java
83.33% <ø> (ø)

...main/java/org/matomo/sdk/LegacySettingsPorter.java
100.00% <100.00%> (ø)

tracker/src/main/java/org/matomo/sdk/TrackMe.java
100.00% <100.00%> (ø)

...org/matomo/sdk/dispatcher/DefaultPacketSender.java
68.57% <100.00%> (+0.51%)
:arrow_up:

...ain/java/org/matomo/sdk/extra/DownloadTracker.java
75.00% <100.00%> (+2.02%)
:arrow_up:

...r/src/main/java/org/matomo/sdk/tools/Checksum.java
95.45% <100.00%> (ø)

... and 3 more

Continue to review full report at Codecov.

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

Nice, we come closer

But this interface way isn't done

This keeping unused classes because someone could use it, is an afford without benefit.

Why do you keep coming back to this? :smile:
I know it's not great, I probably just forgot about that class. I had no one to review my code.
It's out there now and we shouldn't break the users app "just because it's not clean".
Sorry for not writing the perfect SDK? :thinking:

But this interface way isn't done

Not everything needs to be a firebase-level SDK with shovels of abstraction.
This is volunteer work and the SDK worked fine for my use-cases, so at that time: Why make things more complicated if there is no requirement?
A rewrite to Kotlin wasn't on the horizon at that point.
Interfaces can still be introduced now if desired. As said, you can make a v5, with everything new and better if you want to  :+1:.

Sorry, those were just a few thoughts on the Kotlin transformation.
You are right, house holding of our time is also important.


Restore mocked objects in teardown of RandomState's unit tests

Some test fixtures in RandomState's unit tests substitute objects with mocking ones. But they leave the mocks as they are and it does harm to other unit tests.
This PR fixes so that mocked objects are properly restored in teardown.

LGTM!


Add headless execution

This PR uses the concept of local execution of the precompiles introduced in https://github.com/flashbots/suave-geth/pull/185.
This PR requires a new release of suave-geth due to the changes in the commands. (CI fails because of that).
Depends on https://github.com/flashbots/suave-std/pull/40

@ferranbt LMK if the process of cutting a new suave-geth release is painful in any way.  I've still got fresh context on CI and am happy to dive back in to smooth out any friction points.


cat

pretty useless but it can give you random facts

why the C and CC thing instead of just making it choose 0 or 1

also why is everything capitalized

chat its reviewing time

awwww the github webhook didnt show all comments that sucks


Correct links in networking.md

Correct links in networking.md

/assign

great job
/lgtm
/approve


rules_python_zip/main_template.py: Make py2/py3 compatible

Change exec to work with both Python 2 and Python 3. Add a test that uses
Python 3 to ensure this stays fixed.
This is a copy of PR #26 from Globegitter because I couldn't figure out how
to add a change to the PR branch directly. Thanks!

Great, thanks for getting this in.


Make AppError extensible

Description
The goal is to make the error handling and logging extensible to match extensions in other parts of the app.
Currently we have implemented the thiserror crate to create our own custom errors as shown in the enum in the error.rs file.
We will add an ExtensionError variant with a dynamic message. This will be the only error to be used in the extension. This way it will remain clear exactly in what part of the code the error occurs. The thiserror crate has the capability of merging errors as well. We will further explore this possibility so that the standard errors can still be easily used in the extensions in combination with the ExtensionError.
Motivation
The motivation is the make our app as a whole easily extensible, therefore also error handling must be extensible.
The result is an easily extensible error handling mechanism while maintaining clear isolation of error origins. This should keep locating errors as clear as it could be.
Resources
n/a
To-do List

[ ] Add the dynamic ExtensionError to the enum in error.rs
[ ] Test this Error
[ ] Test the Error merging possibilities
[ ] Add examples and guidelines

 Make AppError extensible


pip config updated as global.index-url

When I run partifact login my-repo my global pip configuration gets updated with index-url. Whenever I try to pip install if I have packages that are in a different index then pip can't find them. index-url becomes the default package instead of pypi.
Would it make more sense for partifact to update the global.extra-index-url config instead of global.index-url to allow it to work with multiple sources?

Arguably, partifact should only be concerned with configuring poetry. We added in the pip logic with global.index-url as it makes sense for our unique setup internally.
Is the pip functionality useful for you at all? I'd be tempted to make it optional through a CLI argument with the default being not configuring pip. Alternatively, the CLI arg could control whether extra-index-url or index-url is set.

The pip functionality is useful for me, actually, but right now it actually is breaking me. Here is my use case.
I depend on multiple packages from the public PyPI and from my private PyPI stored in CodeArtifact. My applications are serverless applications running in AWS Lambda. So when I am developing locally, I use poetry to manage my dependencies in my project, but I also use pip and pipx to manage system-wide dependencies on my local machine.
After I run partifact login I can manage my project dependencies easily with poetry and I appreciate the simplicity partifact provides. But then when I try to manage any system-wide dependencies, the changes to my global pip config made by partifact limit my ability to install any dependencies that aren't in my private PyPI in CodeArtifact. This is due to the global.index-url config and should be solved by using global.extra-index-url instead.
In addition, in my CI/CD pipeline I build Lamba layer packages. I do this by first running poetry export to generate a requirements.txt file and then I run pip install -t ... to install the dependencies into a local directory to be packaged. Again, due to the global.index-url setting I cannot install dependencies other than from my CodeArtifact PyPI. So I ended up writing a command to read the global.index-url config, set that value to the global.extra-index-url config, and then remove the global.index-url config.
TL;DR: pip functionality is useful for me, but is currently broken for me. Making them optional arguments and supporting extra-index-url instead of index-url should help.

Thanks for the detailed write-up!
Are the packages that are not in your CodeArtifact repository in another private repository? For public packages, wouldn't having PyPI as an upstream repository for your private CodeArtifact repository suffice? Sorry, just trying to make sure I fully understand your use-case.
For the CI/CD use-case, are you exporting with --with-credentials? Again, we only ever use a single private repository, so there may be gotchas I'm not aware of, but I feel like the exported requirements file should be self-contained without relying on the global URLs in the pip config.
If you still think setting the global.extra-index-url is useful, my preference would be implementing this as an optional convenience through a CLI argument. What do you think?

I could have PyPI as an upstream for my private CodeArtifact, but I don't really need the extra setup to do that because I can just hit PyPI directly and my private CodeArtifact directly.
I can use --with-credentials to export my requirements.txt and it gets added as an extra-index-url in my requirements.txt. However, the problem is still that the global.index-url setting is limiting pip to only looking in the single private CodeArtifact repository. Poetry doesn't add PyPI to the requirements.txt as an index url or extra index url.
It seems like if I do the --with-credentials in my export then everything I need will be in my requirements.txt as long as the global.index-url is not set in my pip config. So I think either removing that altogether or making it optional via CLI would address my use-case. Thanks

In that case, I'd make it optional and not configure pip by default. We can introduce a --configure-pip flag to retain the current behaviour.
I can do it sometime this weekend. Alternatively, if you want to have a go at it, let me know.

I think that will work. I just ran a test where I run pip config unset global.index-url and added --with-credentials to my export and it worked. Before, without the --with-credentials I was reading the global.index-url value and saving it to global.extra-index-url and then unsetting global.index-url. So this simplifies it for me. With your proposed change I won't have to mess with pip config at all.
Unfortunately, I don't have time to work on this right now. I have a working setup so it's no big rush. I appreciate your help and responsiveness.

0.3 should address this. Any issues, let me know!

While I don't fully understand the long thread I'm linking to here, it seems germane regarding how pip treats  global.extra-index-url:  https://github.com/pypa/pip/issues/8606
This isn't a partifact concern, but it is relevant to trying to understand pip's approach to multiple code repositories.  One important thing to note is that the entire python package namespace is a singleton.  You need to give your python packages unique names.  At any point, someone might publish to the pypi repository a package with your private package's name, and, depending on how pip resolves the package, you might get surprised if you rely on pip's search order!
For this reason, putting CodeArtifact "in front" of pypi  ("upstream") seems to be a safer choice.


Support tslint.yaml configuration file in vue-cli-service lint

What problem does this feature solve?
.yaml configuration file is widely used.  And now it hasn't been supported by vue-cli-service lint, throw the error Could not find config file at /path/to/project/tslint.json
What does the proposed API look like?
vue-cli-service lint tslint.yaml

It would also be great for tslint.js files


Keysanity vanilla small keys do not update pause menu immediately

In key-sanity, vanilla small keys in chests do not immediately show up on the menu. They only appear after certain types of transitions like leaving the dungeon, or once a key is spent.
This is because small keys in their vanilla dungeons are converted to item 0x24 by code in Location.php. That code is desirable because it avoids the textbox.
However picking up of item 0x24 from a chest has not actually been hooked. The sprite version from pots and enemies is hooked, and the dungeon specific items are, but not the vanilla 0x24.
I'd recommend the following hook (perhaps needs a better name, I just went with the nearby label in the disassembly):
org $09873F ; <- 04873F - ancilla_init.asm : 960 (ADC [$00] : STA [$00] )
JSL.l AddToStock

With an implementation in stats.asm nearby the existing IncrementSmallKey
AddToStock:
ADC [$00] : STA [$00] ; thing we wrote over
PHP : PHA
; if [$00] == $7EF36F (Small key count address)
LDA $00 : CMP #$6F : BNE + : LDA $01 : CMP #$F3 : BNE + : LDA $02 : CMP #$7E : BNE +
    JSL.l UpdateKeys
+
PLA : PLP
RTL

I am literally targeting the exact spot where $7EF36F is incremented, just like the existing IncrementSmallKey hook does.

Fixed but with a smaller (5 instructions & 0 hooks) footprint.

Actually, this report was about item #$24 appearing in chests, and the keysanity dungeon item grid in the menu.
When you open a chest containing that vinilla small key item the per-dungeon key count is not updated, so the key does not show up in the keysanity dungeon item grid, until you do one of the following: Leave the dungeon, get a pot/enemy key, spend a key, or die. Those actions cause the dungeon key count to be written back to the dungeon-specific key count location.
I've tested what was pushed last night, and it definitely does not address this.
In Discord you had previously proposed to fix this without a new hook (and with only about 5 instructions, too) by checking for the #$24 item in the PostItemAnimation event, and calling UpdateKeys to update the dungeon-specific key count. That would just just fine.


When I call Assemble() to do integrate RHS, the number of calls Assemble() increases by +1 time.

Hi all.
I try to introduce the body force term as an RHS vector.
So, I wrote code this way,
std::cout << "before Assemble" <<  std::endl;
BodyForceIntegrator *bi = new BodyForceIntegrator(data);
bi->SetIntRule(&ir);
Body_Force.AddDomainIntegrator(bi);
Body_Force.Assemble();
std::cout << "after Assemble" <<  std::endl;
But, I realized the number of assemble increased with time step being.
The result of the printout is following below.
"assembling ... : " should be printed out once but it increases by +1.
Unfortunately, body force is linearly accumulated in my code.
Any comments can be helpful to me.
 Can you call AddDomainIntegrator before you time step loop?

Oh my goodness, your expectation is perfect. I solved this problem according to your suggetion. Thank you so much.
By the way, why this is happening?

Glad to hear that @sungho91 . The Body_Force is of LinearForm which is a child of the Vector class. Every time you call AddDomainIntegrator it just pushes back the integrator into a list which assemble later loops over.

@atallah727 Thank you so much I fully understood it.


Change base url

Freewebnovel changed its url to innread therefore I intended to change the baseurl of the script in order to keep using it

Excellent! Thank you very much for your contribution!


Fee refactor draft

the amount owed to the FRT holder as fee rebate, with a max of current balance. We don't currently return this, but that seems like an oversight?

Not fully clear on what the FRT issue is. All we want to do is escrow what is needed. The fee is distributed (if it is escrowed) later on in the redemption flow. What is missing from the FRT handling logic?

closing in favor of #694


CircleAvatar in ListView not rendering image until scrolled

Steps to Reproduce
I have a created a sample project in which I load a list of users from GitHub and display them on a ListView. Code here: https://github.com/miquelbeltran/flutter_github_users
I am using CircleAvatar to display the users' avatars: https://github.com/miquelbeltran/flutter_github_users/blob/master/lib/userlist/user_item.dart#L12
    var circleAvatar = new CircleAvatar(
      backgroundImage: new NetworkImage(_user.avatarUrl),
    );

Previously I used the Image widget in the same way and did not have any problem, but since I switched to CircleAvatar I have the following problem:

The CircleAvatar will not display the users' picture until I have scrolled a bit.

The expected behavior:

The avatars are displayed as soon as the Widget is created and the image is loaded.

The issue happens both on "slow mode" and a normal release apk.
The issue happens on Android, have not tested on iOS.
If you want to run and reproduce the issue, just run the project with flutter run and pull to refresh to load the initial list of users.
See with the screenshots:

Logs
flutter run doesn't show anything.
Run flutter analyze and attach any output of that command also.
Analyzing /home/miquel/dev/flutter_github_users...
  error • The constructor returns type 'dynamic' that isn't of expected type 'Widget' at test/widget_test.dart:15:29 • strong_mode_invalid_cast_new_expr
  error • Undefined class 'MyApp' at test/widget_test.dart:15:33 • undefined_class
  hint • Unused import at test/widget_test.dart:10:8 • unused_import
3 issues found.
(Ran in 7.0s)

They seem to be related to the unit tests and unrelated to this case.
Flutter Doctor
Paste the output of running flutter doctor here.
[✓] Flutter (on Linux, locale en_US.UTF-8, channel alpha)
    • Flutter at /home/miquel/dev/flutter
    • Framework revision 4424892c79 (5 days ago), 2017-10-10 16:45:11 -0700
    • Engine revision 2a84f18c79
    • Tools Dart version 1.25.0-dev.11.0

[✓] Android toolchain - develop for Android devices (Android SDK 26.0.1)
    • Android SDK at /home/miquel/Android/Sdk
    • Platform android-26, build-tools 26.0.1
    • Java binary at: /home/miquel/dev/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] Android Studio (version 2.3)
    • Android Studio at /home/miquel/dev/android-studio
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] IntelliJ IDEA Community Edition (version 2017.2)
    • Flutter plugin version 18.1
    • Dart plugin version 172.4155.35

[✓] Connected devices
    • ONE E1003 • 42bd2643 • android-arm • Android 6.0.1 (API 23)

Strange...

If you instrument the network image loading code, do you get any indications of what's going on?

Hi @Hixie I set a breakpoint inside the _loadAsync from NetworkImage in image_provider.dart and I could not find any issues, both the response and the decoded image object seem to be OK.
But what I could find is that the method paintImage from images.dart doesn't get called until I scroll.
On _paintBackgroundImage, line 372 the image is null, so the paintImage function doesn't get called.
I think that somehow a listener that triggers the repaint is not called after the image is downloaded, but I am not proficient enough with Flutter to debug further. I hope this info helps you.

Thanks, that's helpful.


[Bug] time_round function calculated wrong timestamp

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
1.0.0
What's Wrong?
I'm trying to replace Apache Druid with Doris in our business, but failed in time_around functions documented in
https://doris.apache.org/zh-CN/sql-manual/sql-functions/date-time-functions/time_round.html#time-round
Seems there are some bugs

select day_floor('2022-05-03 13:09:20', 1) from test.flink_doris_druid limit 1;
expect:            2022-05-04 00:00:00
turn out to be:  2022-05-03 00:00:00

select day_ceil('2022-05-03 00:00:00', -1) from test.flink_doris_druid limit 1;
expect:            2022-05-02 00:00:00
turn out to be:  NULL

select day_ceil('2022-05-03 13:09:20', 1) from test.flink_doris_druid limit 1;
expect:            2022-05-05 00:00:00
turn out to be:  2022-05-04 00:00:00

select day_ceil('2022-05-03 13:09:20', 4) from test.flink_doris_druid limit 1;
expect:              2022-05-08 00:00:00
turn out to be     2022-05-04 00:00:00

What You Expected?
as described above
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

PR:   https://github.com/apache/incubator-doris/pull/9712

why q1 expect result is 2022-05-04 00:00:00?

why q1 expect result is 2022-05-04 00:00:00?

In accordance with rounding, divided by 12 o'clock ?

Isn't the floor rounded down? like floor(0.6)=0

Isn't the floor rounded down? like floor(0.6)=0

@BiteTheDDDDt Yes, floor meas rounded down, and the second parameter in day_floor('2022-05-03 13:09:20', 1) meas add 1 more day

Isn't the floor rounded down? like floor(0.6)=0

@BiteTheDDDDt Yes, floor meas rounded down, and the second parameter in day_floor('2022-05-03 13:09:20', 1) meas add 1 more day

why we should add the second param period? period looks like it's just the unit. formulately, floor(x,period)=floor(x/period)*period

@BiteTheDDDDt
Thanks for pointing out this mistake, I'd thought the second parameter was something like "time shift".
So the above cases are all right. And I looked though the code, there is a trivial error in Hour case.
SQL:    select hour_ceil('1970-01-01 01:01:10', 1, '1970-01-01 00:01:10') from test.flink_doris_sink limit 1
Apache Druid's result:  1970-01-01 01:01:10
Apache Doris's result:  1970-01-01 02:01:10
the query means  time are divided into pieces by hour started from  1970-01-01 00:01:10
( 1970-01-01 00:01:10, 1970-01-01 01:01:10, 1970-01-01 02:01:10)
and 1970-01-01 01:01:10 belongs to  [1970-01-01 01:01:10, 1970-01-01 02:01:10],  its rounded ceil will be 1970-01-01 01:01:10

Other error cases:
select hour_ceil('1970-01-01 01:00:10', 1, '1970-01-01 00:00:30')
expect: '1970-01-01T01:00:30'
select hour_floor('1970-01-01 01:00:10', 1, '1970-01-01 00:00:30')"
expect: '1970-01-01T00:00:30'
select day_ceil('2022-05-25')"
expect: '2022-05-25'
select month_ceil('2022-05-01')"
expect: '2022-05-01'

这个关联的PR： https://github.com/apache/doris/pull/9712 已经合入，这个Issue可以关闭了。


Unfold when templater command runs

I'm using a templater command that replace - [x] with - [ ], here is the code
<%*
const text = tp.file.content
const output = text.replaceAll('- [x]','- [ ]')
const file = tp.file.find_tfile(tp.file.title)
await app.vault.modify(file, output)
_%>

And when I execute this command, (because the text being replaced is underneath a folded heading with a crease) it automatically unfolds the heading.
Is the plugin supposed to behave this way ? Is there a way to make it stay folded even when executing the templater command ?

Hi @imeed166, this behavior is expected because your template is using vault.modify. The modify function is "dumb" in that it replaces all the contents with a file with new contents. So Obsidian doesn't know anything about the nature of the changes. It can't tell that most of the content is the same and that just the checkboxes are unchecked. So the expected behavior is that all your fold information is cleared out. This behavior will happen with or without this plugin.
The simplest fix for now is to include the following at the end of your template:
app.commands.executeCommandById('creases:fold');
That will refold your creases after unchecking the checkboxes.

A better fix would involve using CodeMirror to actually just modify the lines with the checkboxes. But I think this behavior is likely too complex for a simple template.

Thank you for explaining and for providing me with two alternative solutions.

The simplest fix for now is to include the following at the end of your template:
app.commands.executeCommandById('creases:fold');

Sorry to bother you, but the added line doesn't re-fold the heading.

plugin in lastest version (0.4.3)
file: testing.md
template:
<%*
const text = tp.file.content
const output = text.replaceAll('- [x]','- [ ]')
const file = tp.file.find_tfile(tp.file.title)
await app.vault.modify(file, output)
_%>

Just to confirm, does the heading have a little 👕 next to it?

If it doesn't, then it's just a regular fold.
For the command I sent to work, you'd need to add: %% fold %% to the heading so that it looks like this:

Let me see if there's a better solution though. I'll get back to you.

Just to confirm, does the heading have a little 👕 next to it?

Yes, it's there.

Let me see if there's a better solution though. I'll get back to you.

Okay, thank you.

So it's ugly but you can do this:
<%*
const text = tp.file.content
const output = text.replaceAll('- [x]','- [ ]')
const file = tp.file.find_tfile(tp.file.title)
await app.vault.modify(file, output)
setTimeout(() => app.commands.executeCommandById('creases:fold'), 50)
_%>

It works, thank you for your time.


Toggle keydown on power tap?

I use BetterTouchTool to bind Hyper+X to open applications, so for instance
cmd+option+control+shift+c for Chrome
cmd+option+control+shift+f for Firefox
etc...
I use KeyRemap4MacBook to bind fn to "Hyper" (cmd+option+control+shift) and it works pretty well until I want to bind on the right side of the keyboard; for instance, fn+o for Opera. KeyRemap4MacBook has options to remap the power key and one option for "Old ADB Keyboard" that binds it to fn but that doesn't work for the MacBook keyboard of course.
My understanding is that pressing and holding the power key + another key isn't possible... So that's what I'm thinking maybe it's possible to "tap" the power key to toggle the Hyper until the next individual key (o) is pressed.
Didn't see any issues about toggling on power key tap so hope this issue is justified, ty :+1:

Still needing this functionality :rotating_light:

Adding a global toggle for a modifier key is a bit beyond the scope of this app, I think. You may be able to extend the app for your purposes, or have a different app that listens for the output of PowerKey (say, the F13 key) and then sets that toggle elsewhere in the system.
In the next version of the app I've enabled modifier keys to work alongside the power key. Maybe that will solve your root problem?
Thanks!


prometheus ignores the name  in node_exporter configuration

monitoring version 2.1
prometheus should replace node_exporter 0.17 metrics to 0.14 metrics to be backwards compatible, it ignores that configuration.

Also related to:
https://github.com/prometheus/prometheus/issues/5200

Closed by #498


java.lang.ClassNotFoundException: io.vertx.spi.cluster.hazelcast.impl.HazelcastClusterNodeInfo in Apache Karaf

vertx-hazelcast requires HazelcastClusterNodeInfo class which is in  io.vertx.spi.cluster.hazelcast.impl package but this package has not been exported hence the error

@Ngosti2000 can you provide a pull request to fix the OSGI suport?

@tsegismont This solution here sorted me out.


Test Synonyms and Wbid functionality for Cells

[x] Test ability to add and retrieve wbid
[x] Test ability to add and retrieve synonyms

@signija added the wbid test with #200, so only the latter remains

Closed by @signija in #229


configurations

Please use the form below, leaving the prefilled data to help us. Thank you.
Page link: configurations
Document link: configuration.md
My Issue/Suggestion
If you can give some example in the document will be much more helpful

Hi there,
Sorry this has taken a year to implement, but there is now an example of a configuration in the documentation.
I hope this is makes the document more helpful.
Thanks for your suggestion - it is customer suggestions which help us to identify improvements we can make to our documentation.
Apologies again for not implementing your idea sooner.
Yours
Mark


Autoscaler isn't working

[Enter feedback here]
Autoscaler isn't working:
LAST SEEN   FIRST SEEN   COUNT     NAME                                KIND                      SUBOBJECT   TYPE      REASON                         SOURCE                      MESSAGE
45s         1h           211       azure-vote-front.154a75b5a320ea37   HorizontalPodAutoscaler               Warning   FailedGetResourceMetric        horizontal-pod-autoscaler   unable to get metrics for resource cpu: unable to fetch metrics from resource metrics API: the server is currently unable to handle the request (get pods.metrics.k8s.io)
5m          1h           201       azure-vote-front.154a75b5a4f12839   HorizontalPodAutoscaler               Warning   FailedComputeMetricsReplicas   horizontal-pod-autoscaler   failed to get cpu utilization: unable to get metrics for resource cpu: unable to fetch metrics from resource metrics API: the server is currently unable to handle the request (get pods.metrics.k8s.io)
kube-system   metrics-server-6cf87b9c8d-bqkwm         0/1       CrashLoopBackOff   20         1h

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6ab10c96-1f2f-e48f-a084-b1715156e33f
Version Independent ID: a113b8e9-259a-2a65-2e54-a2661c78ddb5
Content: Kubernetes on Azure tutorial  - Scale Application
Content Source: articles/aks/tutorial-kubernetes-scale.md
Service: container-service
GitHub Login: @dlepow
Microsoft Alias: danlep

Thanks for the feedback! We are currently investigating and will update you shortly.

@lcanastreiro we can work off the first issue you opened rather than multiples on the same doc
https://github.com/MicrosoftDocs/azure-docs/issues/13306
Closing this as Duplicate of #13306


Select GPU Device Through Python API

It appears that the only way to force this to use only one GPU is through a global CUDA setting. I imagine this would interfere with other CUDA Jobs on the machine.
Would it be easily to implement fine-grained selection of GPU device through the python API?

if you use CUDA_VISIBLE_DEVICES= you pretty much have full control over what is going on and it does not interfere with other processes on the gpus. For example I am able to run 8 generation data pipeline using nvisii on 8 gpu machine whereas a single gpu gets a single job. I do not think it would help to specify on the python side since it is pretty easy to do from the command line.


Local Install from script doesn't work.  Problem with password and file permissions

When Submitting please remove every thing above this line
🐛 Bug Report

[x] 📁 I've Included a ZIP file containing my librephotos log files
[ X] ❌ I have looked for similar issues (including closed ones)
[x] 🎬 (If applicable) I've provided pictures or links to videos that clearly demonstrate the issue

📝 Description of issue:
Started with a completely fresh and fully updated Debian install. I followed the instructions for the local Install.  Once install is done the web page is accessible but I can't get past the login screen.
Here is what I have done to try and fix it. I've narrowed down two issues.

Only the front end service is running. The backend service is down because the worker service is down.  The worker service is down because it can't run librephotos-worker.  Looking at the file and the permissions are wrong.  CHMOD ugo+x fixes the problem and all of the services can be started and run

I still can't get past the login screen. Mostly because I don't know the username and password.  Trying to run librephotos-cli reveals problem number 2.

librephotos-cli crashes and has Django complaining about not having the environment variable "secretkey".  I have no clue where that is supposed to be set.  The instructions also do not explain where/when/how to set

🔁 How can we reproduce it:
Fresh install of Debian and than follow the instructions for local Install.   I've done it three times with the same results.  I may have missed something in the instructions but I have followed them fairly closely.
Please provide additional information:

💻 Operating system:  Debian
⚙ Architecture (x86 or ARM): x86
🔢 Librephotos version: latest?  I used the install script based on the instructions for ocal install.
📸 Librephotos installation method (Docker, Kubernetes, .deb, etc.): Local

🐋 If Docker or Kubernets, provide docker-compose image tag:

📁 How is you picture library mounted (Local file system (Type), NFS, SMB, etc.):  NFS
☁ If you are virtualizing librephotos, Virtualization platform (Proxmox, Xen, HyperV, etc.):  Proxmox

Found a few of the issues.

need to add "SECERET_KEY=" in the librephotos-backend.env
need to create a user for postgres with privileges for the librephots database. This is not in the instructions but, I guess it's obvious?  Will need to add the user information to the librephotos-backend.env

Still not up and running as it's giving an error for invalid e-mail address but, it's a step closer.

Workaround is to use the CLI.
sudo librephotos-cli createadmin USERNAME USER-EMAIL
sudo librephotos-cli changepassword USERNAME

I got the exact same issue. Still not managed to get it working on a fresh ubuntu 20.04.

On the first startup, you should see a "First Time Setup" page where you can create your admin user. The backend probably crashed, because your system hasn't the right requirements. It works fine in the singleton docker image.
The second issue is already fixed.

Hi,
I've just made fresh install in fresh system.
I have 2 problems:

librephotos-backend.env contains DB_USER=docker, but there is no such user in PGSQL. There is "librephotos"
I had to change DB_USER in librephotos-backend.env and set password for this user.
"librephotos" DB is empty..... so, there is no "First Time Setup", just login page, but nothing works....

Hi.
Problems I'v found in librephotos local installation proces and workaround.

In the /etc/librephotos/librephotos-backend.env file is wrong DB user and password,
1a. there is: DB_USER=docker, but instalation script creates "librephotos" user
1b. there is: DB_PASS=AaAa1234, but there should be something elese.
Instalation script creates random password during instalation. You can find it in terminal. Search for DB_PASS.
I have (hidden password ;-) line:

sed -i 's|DB_PASS=password|DB_PASS=XxXxXxXxXxXx|g' /etc/librephotos/librephotos-backend.env
And posgresql DB grants acces with user librephotos and XxXxXxXxXxXx password.

I'v found part creating database in install-librephotos.sh, but I didn't foind part where this database is filled by structure.
/usr/lib/librephotos/bin/librephotos-worker have wrong permissions

Workaround:

Install LibrePhotos somewhere else in other way (eg. Docker version).
After install make a database dump
Edit this SQL backup file: change all "docker" to "librephotos" with your favorire file editor (vi? :-)
I have such SQL file. I can put it here, if derneuere will allow me to.
install LibrePhotos Local
find DB password in terminal history
upload corrected SQL file to DB with user: librephotos, password XxXxXxXxXxXx, db: librephotos, host: localhost
psql -h localhost -U librephotos librephotos <librephotos-corrected.sql
correct DB_USER and DB_PASS in /etc/librephotos/librephotos-backend.env
Correct /usr/lib/librephotos/bin/librephotos-worker file permissions.
chmod 755 /usr/lib/librephotos/bin/librephotos-worker
or librephotos-worker.service won't start.
restart librephotos services, or whole machine.
check if all librephotos services works
try to log in, you should get first login page.

Good luck! :-)

Workaround is to use the CLI.
sudo librephotos-cli createadmin USERNAME USER-EMAIL

running this (minus the sudo) inside the container generates the blow error
Traceback (most recent call last):
  File "/usr/lib/librephotos/backend/manage.py", line 11, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/lib/librephotos/.local/lib/python3.10/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/usr/lib/librephotos/.local/lib/python3.10/site-packages/django/core/management/__init__.py", line 345, in execute
    settings.INSTALLED_APPS
  File "/usr/lib/librephotos/.local/lib/python3.10/site-packages/django/conf/__init__.py", line 82, in __getattr__
    self._setup(name)
  File "/usr/lib/librephotos/.local/lib/python3.10/site-packages/django/conf/__init__.py", line 69, in _setup
    self._wrapped = Settings(settings_module)
  File "/usr/lib/librephotos/.local/lib/python3.10/site-packages/django/conf/__init__.py", line 170, in __init__
    mod = importlib.import_module(self.SETTINGS_MODULE)
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/usr/lib/librephotos/backend/ownphotos/settings.py", line 26, in <module>
    raise NameError("Environment variable not set :" + envvar)
NameError: Environment variable not set :SECRET_KEY

To resolve:
NameError: Environment variable not set :SECRET_KEY
I edited /etc/librephotos/librephotos-backend.env
to add the line:
SECRET_KEY=/var/log/librephotos/secret.key

With this, you are essentially setting the secret key value to /var/log/librephotos/secret.key, not the contents of the given file. Not saying that's wrong or bad, just keep that in mind.

SECRET_KEY=/var/log/librephotos/secret.key

it does not work in my side :/


Illuminate/Config PHP Version

Illuminate/Config seems to require php 5.6.x +. Whereas the same component inside the Laravel framework requires 5.5.9. Is this a mistake? Seems strange that the config on its own requires a higher PHP level.

No, it doesn't. Only 5.3.x will impose that version constraint.

You're probably looking at the master branch, rather than 5.2.


Option --no-emoji not working

Hi,
I have the issue that in the code section of a document there are several :x: which are converted to emojis.
For one thing, I was surprised that emoji-conversion takes place in pre-formatted text ... I would not expect that. Anyway, when I tried to switch off the conversion with '--no-emoji', the conversion still happens.
Any idea on how to disable the conversion?
Thanks a lot,
Josef

After looking at the source code I tried --noEmoji which seems to work.

Oh that's peculiar, thanks for reporting this I'll fix the flag / documentation. Thanks!


Merge master into 040wip branch to prep for 0.4.0 release

Nevermind, looks like the upstream merge isn't too bad.


执行 pnpm install   出现错误 ERR_PNPM_WORKSPACE_PKG_NOT_FOUND

执行 pnpm install   出现错误
* ERR_PNPM_WORKSPACE_PKG_NOT_FOUND  In : "@vben/types@workspace:*" is in the dependencies but no package named "@vben/types" is present in the workspace

看下文件目录层级

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days


Sort Hookup Notes

Current hookup notes sorting convention is more coincidental than anything. We should find a way to sort by part like M&C does before filling out the text boxes in the website.

this should be fixed now. will re-open if sorting goes wrong.

this should be fixed now. will re-open if sorting goes wrong.


Source request

Source name
Reborntrans
Source link
https://www.reborntrans.com/
Source language
Thailand
Other details
Do it for me.
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

Add a :+1: reaction to issues you find important.

/edit-title "Source request: RebornTrans"


add RecordTime for CN continuous run

The Catchment CN 's continuous run is non zero-diff from stop-and-continue run. This PR is trying to fix the problem.
Since CatchCN is not included in the GCM suite of tests and since there is no stop&restart regression test in GEOSldas, the PR is zero-diff w.r.t. the existing test suites.
The PR is non-zero-diff for users when existing CatchCN runs are reproduced with different stop&restart times.

@gmao-jkolassa . I think CNCLM40 is also non zero-diff except there is no cncol or cnpft in the restart file.

@weiyuan-jiang Thanks for putting this together! I will re-run my experiments with this branch.
 I think CNCLM40 is also non zero-diff except there is no cncol or cnpft in the restart file
CNCLM40 does have CNCOL and CNPFT in the restart file. When I ran the continuous vs stop-and-continue run for CNCLM40 there were no differences in the restart files after 2 months. So as far as I can tell CNCLM40 was never affected.
I will re-run this test for both CNCLM40 and CNCLM45 just in case.

It is not just concol, there are are other variables.
@weiyuan-jiang I am not sure I am following, could you please clarify?
CN_init (among other things) maps the CNCOL and CNPFT entries to CLM variables. CN_exit then maps the CLM variables back to CNCOL and CNPFT before they are written to the restart file. I went through the routines in the CNCLM45 CN_DriverMod.F90 and as far as I can tell all variables that are mapped from CNCOL and CNPFT in CN_init are mapped back into CNCOL and CNPFT in CN_exit. Can you please let me know which other variables you mean? Thanks!

In its current form, this branch still leads to differences in the restart files for a continuous vs stop-and-continue experiment for CNCLM45. I have been talking offline with @weiyuan-jiang about this and am currently running some additional tests to better isolate the cause.

I have run several more tests and I think I have found the issue. There are actually two separate issues, which are both related to the following code block in the GEOS_CatchCNCLM45GridComp.F90 that resets three variables to 0 on the first time step.
 `if(first) then
  ! variables used for summing CN inputs over multiple land model calls; not saved on restart
  ! fzeng: run must end on a CN call step
  ! -----------------------------------------------------------------------------------------
  allocate( lmrsunm(ntiles,nveg,nzone) )
  allocate( lmrsham(ntiles,nveg,nzone) )
  allocate(             runsrf(ntiles) )

  lmrsunm = 0.
  lmrsham = 0.
  runsrf = 0.

  first = .false.

endif`

These variables are sums over the CN interval, which is why they should be reset after each call to the CN subroutines (that happen at a  timestep of 1.5 hours).
lmrsunm and lmrsham are reset to 0 after each CN_driver call, meaning the stop-and-continue and continuous experiment are identical for these if the stop-and-continue run is stopped directly after a call to CN_Driver (as indicated by Fanwei's comment in the code). Having the "break" in the stop-and-continue experiment at 0z means that the resetting to 0 of lmrsunm and lmrsham occurs prior to the 0z CN_Driver call in stop-and-continue and after that call in the continuous experiment.
In the case of runsrf I think that there is a bug here. It is never reset after the CN_Driver call, so this variable will always be different between the continuous and stop-and-continue experiments after the "break" irrespective of when the break occurs.
Regarding solutions:
All other variables that are sums over the CN interval are passed as restart variables meaning they to not need to be initialized to 0 on the first time step and they have the same value in the stop-and-continue and continuous experiments irrespective of when the break occurs. So I think it might be easiest to just include lmrsunm and lmrsham as restart variables here.
For runsrf, it should be equal to RUNSURF, which is already passed as restart variable making the runsrf = 0. statement redundant.
I apologize for the lengthy answer. I am happy to talk about this in person if that is easier.

After the PR #658 is merged, I will change CatchmentCNRst.F90 again to make sure the two variables also need re-tile

@gmao-jkolassa, has there been any progress on this PR?

@gmao-jkolassa, has there been any progress on this PR?

@gmao-rreichle There has been some progress on this. I have identified a bug in the fire module that incorrectly initialized a variable needed to start the agricultural fire season. The effect of this was that stopping and re-starting a run lead to no agricultural fires occurring in the run for the rest of the year. I have fixed this bug by adding 'BURNDATE' as an additional restart variable for Catchment-CN4.5. This has removed the differences between the two regression experiments for most of the points.
However, there are still a few locations with differences. I have traced this bug to the subroutine that computes the stomatal conductance, but have not yet identified the ultimate cause. So this PS is still work-in-progress.

This PR in its current form implements several fixes for bugs that caused a stop-start regression test to fail for a majority of land tiles in Catchment-CN4.5. The code changes implemented up to this point fix this issue for most land tiles. There is still one outstanding issue that causes the start-stop regression test to fail in a few (~10) land tiles. The causes for this is as of yet unidentified.
It has been decided to go ahead and merge this PR as is so that the bug fixes already implemented can take effect. A new issue will be created for the remaining bug and the fix to this remaining bug will be addressed in a separate PR.
As of the time of writing, the branch is up to date with the current 'develop'.


Unable to log in to Textto

Attempting to press the "Sign in with Google" button results in an immediate error:

Although I'm not too familiar with logcat, glancing at it seems to suggest an issue with Firebase:
[02-23 11:53:11.504 6157:11615 I/ActivityManager]
Killing 20383:com.moduloapps.textto/u0a134 (adj 906): empty #17

[02-23 11:53:21.506 6157:6171 I/ActivityManager]
Start proc 23110:com.moduloapps.textto/u0a134 for broadcast com.moduloapps.textto/com.google.firebase.iid.FirebaseInstanceIdInternalReceiver

[02-23 11:53:36.780 6157:6171 I/ActivityManager]
Waited long enough for: ServiceRecord{4b05ef u0 com.moduloapps.textto/.service.FirebaseIdService}

[02-23 11:55:30.252 6157:6171 I/ActivityManager]
Waited long enough for: ServiceRecord{2b66166 u0 com.moduloapps.textto/.service.FirebaseIdService}

[02-23 11:56:51.727 6157:6171 I/ActivityManager]
Waited long enough for: ServiceRecord{f275250 u0 com.moduloapps.textto/.service.FirebaseIdService}

[02-23 11:58:50.858 6157:6171 I/ActivityManager]
Waited long enough for: ServiceRecord{b86e27f u0 com.moduloapps.textto/.service.FirebaseIdService}

[02-23 12:02:43.393 6157:6171 I/ActivityManager]
Waited long enough for: ServiceRecord{44b7bad u0 com.moduloapps.textto/.service.FirebaseIdService}

[02-23 12:08:03.134 6157:6550 I/ActivityManager]
START u0 {act=android.settings.APPLICATION_DETAILS_SETTINGS dat=package:com.moduloapps.textto flg=0x10008000 cmp=com.android.settings/.applications.InstalledAppDetails bnds=[588,691][1358,887]} from uid 1000

[02-23 12:08:37.421 6157:6171 I/ActivityManager]
Waited long enough for: ServiceRecord{c72e7a4 u0 com.moduloapps.textto/.service.FirebaseIdService}

[02-23 12:16:42.607 26442:26442 D/MyDeviceLogs]
[LogcatLiveFragment] onSearchAction: textto

same thing here. Would love for this to work.
Thanks


WIP feat:Framework Init - Handler

Codecov Report

Merging #158 into master will decrease coverage by 0.35%.
The diff coverage is 64.28%.

@@            Coverage Diff             @@
##           master     #158      +/-   ##
==========================================
- Coverage   87.17%   86.82%   -0.36%
==========================================
  Files          30       30
  Lines        1076     1085       +9
==========================================
+ Hits          938      942       +4
- Misses         74       78       +4
- Partials       64       65       +1

Impacted Files
Coverage Δ

pkg/framework/aries/default.go
71.42% <100%> (+1.73%)
:arrow_up:

pkg/framework/aries/framework.go
89.18% <100%> (+1.31%)
:arrow_up:

pkg/framework/context/context.go
65% <16.66%> (-23.24%)
:arrow_down:

Continue to review full report at Codecov.

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


Advantages of DiCy

(this isn't an issue report but a suggestion)
I got interested in DiCy after receiving area/language-latex#146, but I fail to clearly see pros especially as a CLI tool. When used as a Node.js library like atom-latex, DiCy makes a package more self-contained, but this isn't the case for CLI use.
Adding a small description of the advantages of DiCy (possibly comparisons with existing LaTeX automation tools like arara or latexmk) in README.md or GitHub Pages will make DiCy more attractive, I think. :smile:

Good point! Do you mind being a reviewer on the PR once I am done?

Of course not, but can I be a reviewer without an access to this repository?

Just a read through by someone else is the most helpful aspect. Plus anyone can comment on PRs even without the formal reviewer role.

I see. I'd give a proofread if I can be of any help!


feat: current practices

Updates the hello-world skill to use the latest imports and practices as I understand them from the Matrix chat and current documentation.
Also adds a generic Python .gitignore and thus deletes the dist/ and build/ folders and all their extras.

im ok with merging as is but left a couple comments
def stop could also be removed if not used, or should have a comment. stop method should return True if it stopped something or False (or None) otherwise

im ok with merging as is but left a couple comments
def stop could also be removed if not used, or should have a comment. stop method should return True if it stopped something or False (or None) otherwise

I'll make the updates from the comments

def create_skill is also no longer needed

Can you elaborate more on this?

def create_skill is also no longer needed

Can you elaborate more on this?

the method at bottom of the file is no longer needed, now the skill class is auto detected + also provided in setup.py
that function at end of the file is just dead code more or less, we init skill classes directly whenever possible
the only use for this method would be if you had multiple skill classes in a single package and wanted to do add some logic in there to decide what to load


Open/Close trait in different languages

Hi - can we get Open/Close rolled out to all the other languages?
For example, the feedback i've had is that 'Close' isn't recognised in Dutch - Close = sluit

There is currently no public open/close trait. While you may find preliminary or early documentation, keep in mind that the API can change and it may not be entirely done. Things like internationalization may not be completed.

Ok cool - i'm on the EAP - can I ask for some timescales to be added to that documentation? Thanks


why Cursor cant be Interceptor intercept？

why Cursor cant be Interceptor intercept？

Hello @zzz222z ,
This is explained in the issue template, but we do not use this tracker for questions.
Please post your question to the mailing list or Stack Overflow for questions.
And, if you want useful replies, you should provide more details.

Oh, you can also use Discussions, I guess.


skeleton UI shows even after loading message in thread when user switch from offline to online

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

go to any chat
send any message
go to offline
select reply in thread option for that message
go to thread
send any message
switch from offline to online
Scroll the page up a bit

Expected Result:
should not show skeleton UI after loading message
Actual Result:
shows skeleton UI even after loading message
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.22-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/f0a470a2-52f2-45d1-be36-48fe4c3e9058
https://github.com/Expensify/App/assets/43996225/f23d3a26-de73-40ec-8e98-ed4610d18146
Expensify/Expensify Issue URL:
Issue reported by: @gadhiyamanan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1685076904780539
View all open jobs on GitHub

Dupe of https://github.com/Expensify/App/issues/19490


OSError

OSError: ashawkey/imagedream-ipmv-diffusers does not appear to have a file named scheduler_config.json.
Hello, thank you for your great contribution. I encountered the issue mentioned above while reproducing it. How can I resolve this?

Hi, thanks for your question. It looks like a diffuser version issue. Which diffuser version do you use?

I find out that my diffuser version is 0.30.3.

We use the same diffusers version. Can you then try pip install transformers==4.45.1?

Thank you very much for your response! In addition to needing this step, I also downloaded the imagedream-ipmv-diffusers file to my local machine, and I successfully reproduced it with great results. It's truly a remarkable creation!

Glad it works! Thank you for posting your solution to help other people, also appreciate for the nice words!


The app not launching from the menu or quick launcher

What version of electron-installer-debian are you using?
0.8.1
What version of node and npm are you using?
npm version 6.0.0
What operating system are you using?
Linux Mint 18.3 Cinnamon 64-bit
Can you compile the example app successfully?
Not applicable
What did you do? Please include the configuration you are using for electron-installer-debian.
electron-installer-debian --src release/builds/linux-x64/ --dest release/installers --arch amd64 --config debian.json
What did you expect to happen?
The app should launch from menu or from quick launcher
What actually happened?
After the spinner spins for a while, nothing happens.  When I check using the System Monitor, there are three processes running. one with just the app name, second with multiple options related to gpu and stuff and the third with --type=zygote --no-sandbox. If I go to the directory where its installed and run through the command line, it launches fine and there is only one process.

In order to debug your problem further, we need a minimal testcase to reproduce your problem. Using the electron-quick-start repository as a base, could you please create a minimal Electron app that illustrates the issue you described, and post a link to it here?

I tried with electron-quick-start and it does work.  The only difference is app I am working on has and node addon that depends on some libraries on the system.  When I set the application launcher to run from terminal, it threw error on dependencies not found.  So, the conclusion is that when run from command line, it has all the environment variables in place but when running from quick launcher and even with double clicking on the packaged app, the environment setup is not available.

Yes, I realized now that its not an electron-installer-debian issue.  The double clicking on the app from the build using electron-packager also does not work.  So, I think setting depends options won't help in this case.  Any suggestions would be appreciated.

You'll likely need to use electron-rebuild. Alternatively, you can switch to Electron Forge (although you'd probably be best served by v6 even though that's in beta) - Forge uses Packager, this module, and electron-rebuild under the hood.


Add Emanuel's Final Remaining Characters/ and new animations

Emanuel sent me some characters to add, we should add these to the alpha before we start cutting things out for the beta

Finished adding new animation in 387a00a61d7e2ab3f7f154d27fdc0affead5bf42

GOing to add cinos


Hotfix-tweaking-de-styles

modifying DE styles so that indicator and about boxes more clearly show that they have overflow: removed gradient, widened scrollbar, added scrollbar track

@mmontesanonyc Merging to dev so you can check it out


ci: exclude dotfiles from release zip

Fixes #24

Please refer release on my repository for release generated on this branch: https://github.com/anatawa12/VRC-Avatar-Performance-Tools/releases/tag/1.3.6
This reduces zip size from 75.6 KB to 20.9 KB


Trainer#tune/scale_batch_size with mode="binsearch" does not work properly

🐛 Bug
Trainer#tune/scale_batch_size with the "binsearch" option does not seem to work properly.
To Reproduce
import pytorch_lightning as pl
import torch.nn
from torch.utils.data import DataLoader

class LightningModel(pl.LightningModule):
    def __init__(self):
        super().__init__()
        self.model = torch.nn.Sequential(
                torch.nn.Linear(1000, 5000),
                torch.nn.Linear(5000, 5000),
                torch.nn.Linear(5000, 1)
        )

    def training_step(self, X):
        h = self.model(X)
        loss = torch.nn.functional.mse_loss(h, torch.ones_like(h))

        return loss

    def configure_optimizers(self):
        return torch.optim.Adam(self.model.parameters())

class Datamodule(pl.LightningDataModule):
    def __init__(self):
        super().__init__()
        self.batch_size = 1

    def train_dataloader(self):
        return DataLoader(torch.zeros(10_00_000, 1000), batch_size=self.batch_size, num_workers=2)

trainer = pl.Trainer(auto_scale_batch_size=True, gpus=[1])

datamodule = Datamodule()
trainer.tune(model=LightningModel(), datamodule=datamodule, scale_batch_size_kwargs={"init_val": 20000, "mode": "binsearch"})

Expected behavior

trainer.tune increases batch size, until it hits the GPU's limit - as you can see in the console output I've added below, this works: a batch size of 80,000 succeeded, but the next larger step, 160,000, did not
the algorithm then uses binary search to find the optimal batch size - this apparently does not work: the algorithm goes all the way back down to 80,000. I would expect the final batch size to be somewhere in between 80,000 and 160,000

I can reproduce this behaviour with different model sizes, it's always the same: the result of the binary search is always the last succeeding batch size of the first step.
Environment

CUDA:
- GPU:
- NVIDIA GeForce RTX 2080 Ti
- NVIDIA GeForce RTX 2070 SUPER
- available:         True
- version:           10.2
Packages:
- numpy:             1.21.5
- pyTorch_debug:     False
- pyTorch_version:   1.10.1+cu102
- pytorch-lightning: 1.5.7
- tqdm:              4.62.3
System:
- OS:                Linux
- architecture:
- 64bit
- ELF
- processor:         x86_64
- python:            3.8.10
- version:           102-Ubuntu SMP Fri Nov 5 16:31:28 UTC 2021

Additional context
Example output of the provided code:
GPU available: True, used: True
TPU available: False, using: 0 TPU cores
IPU available: False, using: 0 IPUs
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 20000 succeeded, trying batch size 40000
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 40000 succeeded, trying batch size 80000
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 80000 succeeded, trying batch size 160000
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 160000 failed, trying batch size 120000
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 120000 failed, trying batch size 100000
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 100000 failed, trying batch size 90000
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 90000 failed, trying batch size 85000
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 85000 failed, trying batch size 82500
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 82500 failed, trying batch size 81250
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 81250 failed, trying batch size 80625
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 80625 failed, trying batch size 80312
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 80312 failed, trying batch size 80156
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 80156 failed, trying batch size 80078
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 80078 failed, trying batch size 80039
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 80039 failed, trying batch size 80019
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 80019 failed, trying batch size 80009
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 80009 failed, trying batch size 80004
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 80004 failed, trying batch size 80002
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 80002 failed, trying batch size 80001
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1]
Batch size 80001 failed, trying batch size 80000
Finished batch size finder, will continue with full run using batch size 80000

cc @tchaton @rohitgr7 @akihironitta @borda

Dear @cschell,
Can you try to manually use 80001?
From the binary search, it seems to me 80000 is the largest working batch size. 80001 would fail as indicated there
Batch size 80001 failed, trying batch size 80000.

80,001 works fine, also 90,000 works fine, but some the binary search stage doesn't get a positive response. Also, if you set init_val to something that's too high (like 160000 in my example scenario), the binary search start immediately (as it should) and then goes all the way down to a batch size of 1.

there must be some issue then.. will have a look.

As of version 1.5.10 this bug is still there. It's especially problematic when the init_val is set to something higher than fits into memory, since then it just iterates down to a batch size of 1. If not, it's equivalent to a very inefficient version of the power option (as cschell mentioned). If there are no immediate plans to fix the bug, maybe it'd help to at least throw a warning (that would have saved at least me some time :))


fix: use cross-platform pause image as the containerd sandbox image on Windows

Reason for Change:
The containerd sandbox image is not included on the Windows VHD. The containerd configuration file uses the old platform-specific name. This PR updates the configuration file so it matches the new cross-platform name, the one matching the image already present in the Windows VHD.
At the time the containerd config template was created, the pause image was platform specific: https://github.com/Azure/aks-engine/commit/046db34507395ec05b506fd1373ea9943044e697
Issue Fixed:
Credit Where Due:
Does this change contain code from or inspired by another project?

[x] No
[ ] Yes

If "Yes," did you notify that project's maintainers and provide attribution?

[ ] No
[ ] Yes

Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Notes:

/azp run


How to avoid "nested document structures may lead to repetitions of the same group."

Hello,
I have configured my river and they work like a charm. It is great. Although I have a complicated query so I end up with tons of duplications in the object like in your example:  where association is there multiple times.
{
"blog" : {
"name" : "Joe",
"association" : [
{
"name" : "John",
"id" : "3917"
},
{
"name" : "John",
"id" : "3917"
},
{
"name" : "John",
"id" : "3917"
}
],
"published":"2014-01-06 00:00:00"
}
}
This is not a problem although I am wondering where I can extend perhaps your river to avoid this. The reason being is that I do loads of nested aggregations and my bucket numbers are incorrect because of the duplicate elements. Also my query is rather complicated so sometimes I have hundreds of elements duplicated which does not look nice.
Any hints? It would be fairly simple and I am happy to add it if you tell me at which point this could be implemented in the river ?
it should be possible to see if the same element is in the array and no need to add it again and again if it is there.
Thanks in advance.

What about this issue? Could it be merged in master? If this feature may trigger undesired effects, maybe we could add a parameter to enable/disable it?
Thanks.

Hi,
not sure. I used this once for a project but no longer in production.
thanks

Thanks for replied.
I tried to use your patch, then I read here that river will be
deprecated in next ES releases. So I moved on to bulk API :/
Julien BENOIT
PHP & JS developer @GuestToGuest https://twitter.com/GuestToGuest
Paris, FR.
blog.julienbenoit.net
http://fr.linkedin.com/in/jbenoit2011
http://www.viadeo.com/profile/0021vk9j4z60jp63/
https://github.com/jbenoit2011
2015-09-23 14:47 GMT+02:00 istvano<EMAIL_ADDRESS>
Hi,
not sure. I used this once for a project but no longer in production.
thanks
—
Reply to this email directly or view it on GitHub
https://github.com/jprante/elasticsearch-jdbc/issues/328#issuecomment-142587509
.

JDBC importer will be supported in the future.
River API is to be removed but that has nothing to to with bulk API, JDBC river or importer both uses bulk API.
It is open if JDBC importer will be able to run as a plugin in Elasticsearch 2.0 so the user can install the importer more easily.
If the patch is considered useful, I can test it and add it to the next version. Pull requests are always welcome.

Thanks @istvano, your "workaround" worked for me!

Hi Julien,
I think the patch is useful. I have used in production for a long while. Although it is not beautiful.
I did not want to send a pull request as I am not sure about it. (was it implemented at the right place etc. )
I am really new to rivers and all that so I did not want to send low quality code into the code.
thanks

Having same Issue... Proposed solution is not added to master yet..right...? And as i am very late here is there any other configuration param that handles it..!

Having same Issue... Proposed solution is not added to master yet..right...? And as i am very late here is there any other configuration param that handles it..!

previously, I have below problem, could you help to solve?
I have mapping
mappings: {
properties: {
sys_code: {
type: "string",
fielddata: {
loading: "eager"
}
},
values_id: {
type: "integer",
doc_values: false
},
values_name: {
type: "string",
fielddata: {
loading: "eager"
}
}
}
}
and sql expression
SELECT
"'test'" as _index,
"type_primary" as _type,
sus.syouhin_sys_code AS "_id",
CAST(sus.sys_code AS CHAR) AS sys_code,
values_id AS "values_id[]",
values_name AS "values_name[]"
FROM syouhin_table sus
and my database result

sys_code:21554223
values_id["9460418","9460421","9462677","9462682","9462688","9464051","9464052","9464053"],
values_name: ["test1","test2","test3","test4","test5","test6","test7","test3"],

but my index result is different with database
++++++++++++++++++++++++++++++++++++++++++++++++++
_index: "test",
_type: "type_primary",
_id: "21554223",
_score: 1,
_source: {
sys_code: "21554223",
values_id[
"9460418",
"9460421",
"9462677",
"9462682",
"9462688",
"9464051",
"9464052",
"9464053"
],
values_name: [
"test1",
"test2",
"test3",
"test4",
"test5",
"test6",
"test7"
],
++++++++++++++++++++++++++++++++++++++++++++++++++
It means that values_id (9462677 and 9464053) have the same values_name("test3").
because it was duplicated in [values_name(test3)], but I need it display in index like below
_id: "21554223",
values_id[
9462677 ,
9464053]
values_name: [
"test3",
"test3"]
how I can  solve this problem? Please help
thanks.
Hon


Request: Select URL in address bar (when it's about:blank)

Hi there, I'm using about:blank for a new tab because I just want a clean page. So, when opening a new tab the address bar is focussed and I can start typing a new URL. But I have to do Cmd-A + Backspace first to clean the field first. It would be nice if the old URL (in my case about:blank) was all selected so one can just start typing.

Thanks for the request. Unfortunately, Chrome handles the highlight of this address bar inconsistently across versions. See issue #50 for further discussion. I'm closing this issue so there is a single open issue for this.

Though it works know :)


Our own arrows cant be given potion effects

I am not sure how we would implement this.

Hold my json

Fermented spider eye json:
    "features": [
      {
        "type": "minecraft:on_hit",
        "on_hit": {
          "type": "minecraft:status_effect",
          "target": "minecraft:targeted_entity",
          "effect": "minecraft:poison",
          "level": 1
        },
        "title": "Poison",
        "description": "Applies poison on hit"
      }
    ]

Mastercrafted arrow head json:
"slots": [
      {
        "category": [
          "OFFENSIVE"
        ],
        "description": "upgrade",
        "type": "UPGRADE_MATERIAL",
        "tier": 1
      }
    ]

https://github.com/user-attachments/assets/d6f476a2-f20a-4f6e-87ed-6b19f03f6257

how does this scale with the arrows? Do they get alot heavier because of the added weight?

We can adjust the upgrade weight, it's currently at 10.

That would probably work well :+1:

Implemented


Update the documentation to include developer facing descriptions of the new features and functionalities.

References #118
Update the documentation.

Created the PR for this task.
{t:91}


improved commenting

i improved commenting (its true)

@grnnja Quick! this is an essential fix. Merge it!


Expand / Collapse animation within table issue regression

Hi,
I found that upgrading from version 0.12.0 to version 0.13.0 affected the way expand / collapse animation within a table works.
Here's an example of an expected behavior with a nice animation (version 12) -
http://codepen.io/SafiX/pen/jPmZEx
Here's the exact same example but with version 13. The animation isn't working -
http://codepen.io/SafiX/pen/vOmdOX
Thanks,
Asaf

Please use the search function. Plenty of people reported this and it all comes back to including ngAnimate in your project.
If you add ngAnimate, do the animations work for you?

Doesn't look like including ngAnimate is the only problem: http://codepen.io/anon/pen/wadyEx
@SafiX BTW, you do not need to include Bootstrap's JS

Hi,
Thanks for the responses.
As pkozlowski-opensource mentioned, it doesn't work with ng-animate included.
Expand / collapse does work outside of a table in version 13.

I converted it to a Plunker since it would make it easier to inspect the DOM.
I'll see if I can take a stab at figuring this out tomorrow.

Looking into this a bit more, this may be an $animate bug.

Hi guys. Any update on this? Is it a bug?

This appears to be fixed with the latest release now, as can be seen here.


New config supporting Sepolia and Amoy Testnets

New config supporting Sepolia and Amoy Testnets is configured and the previous config containing goerli and mumbai details was deprecated

https://github.com/0xPolygon/dapp-launchpad/pull/28


WebSubmit: preserve bookmarks when stamping PDF files

Originally on 2011-01-07
The WebSubmit file stamper relies on the pdftk tool to process PDF files, but unfortunately pdftk does not preserve bookmarks (eg. table of content or "outlines"), even though it is able to dump them.
Ghostscript could be used instead to stamp files, as it can preserve such information.
Ghostscript could fully replace pdftk in this case, or could be used simply to enrich the resulting stamped file with the necessary instructions to create the bookmarks (PDFMark?), provided that these instructions can be extracted correctly from the original file.
Some additional untested information:
$ pdftk original_file.pdf dump_data
InfoKey: Creator
InfoValue: LaTeX with hyperref package
NumberOfPages: 88
BookmarkTitle: Introduction
BookmarkLevel: 1
BookmarkPageNumber: 9
BookmarkTitle: Context
BookmarkLevel: 2
BookmarkPageNumber: 9
BookmarkTitle: CERN
BookmarkLevel: 3

$ pdftk stamped_file.pdf update_info original_file.info output new_stamped_file

(does not seem to add the necessary information unfortunately)

$ gs -dBATCH -dNOPAUSE -sDEVICE=pdfwrite -o stamped_file.pdf original_file.pdf stamp.pdf

(would apparently merge the two files and keep the bookmarks from both)

$ gs -dBATCH -dNOPAUSE -sDEVICE=pdfwrite -o new_stamped_file.pdf \
       -c "[/Page 1 /View [/XYZ null null null] /Title (Title 1) /OUT pdfmark" \
       -c "[/Page 2 /View [/XYZ null null null] /Title (Title 2) /OUT pdfmark" \
-f stamped_file.pdf

(would apparently add the specified bookmarks instructions to the given file. They should be extracted from the original file, and cleaned for this syntax)

In any case one should be careful if bookmarks are linking to absolute page numbers, as the number of pages can change when adding a cover page. This would probably not be handled by Ghostscript.

@ludmilamarian @kasioumis @egabancho Is this still the case on CDS?

@tiborsimko as discussed IRL this is still needed by CDS. The decided action is to create a new repo: invenio-stamper and implement there the new code.

@egabancho Can you please take care of this as per related issue https://github.com/inveniosoftware/invenio/issues/2760 ?


Copying shadowJar result in Dockerfile.

This pull request solves #725

I forgot that this file has a little different name, so it cannot be -all.jar in the end:

🤔

Thanks!


Added markdown syntax highlighting to README.md

I changed the headings in the README.md file to use "#" signs instead of =====
I used GitHub Flavored Markdown to add syntax highliting to code blocks

Awesome, thank you!


[Bug] Where is the data saved when creating a resume?

Hi,
Where is the data saved when creating a resume? And can this folder be mounted in docker?
A password login to protect your private information would be nice

The data is saved in your browser, so there's nothing you need to do with Docker. If you want your data to persist, all you need to do is keep the same IP/Domain address when deploying/re-deploying. The data can be found in your Client Browser's Local Storage.


Update oldguard.json

Update to add "Ladies of Evil" Mod to the modlist

please and thank you Conay!


Plugin breaks obsidian when used

Describe the bug
A clear and concise description of what the bug is.
The plugin breaks when trying to use it in Obsidian.
To Reproduce
Steps to reproduce the behavior:
Just click on the shortcut to use it.
Screenshots

Obsidian Version:

[ ] Desktop

The first picture is the result of an error being thrown. Can you show the console log when that blank screen issue happens?
The second picture seems to be the result of a theme reducing pane sizes in the sidebar. If you want it to be big drag it to a different pane or use a different theme.


build(circleci): removed circleci

removed circleci

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Deploy application in detached mode and query StackStatus

Perhaps this is better served as two separate issues but on today's call I was talking to @ndeloof around what the user experience would be for deploying a stack in detached/quiet mode. At the same time, would a user be able to query the StackStatus with something like a ps command while trying to automate the deployment of their application?
For reference, here is the aws-cli for describe-stacks in CloudFormation.

We could probably extend the scope of the compose ps command so it can report deployment status when stack is still in progress. Same need applies ot updates


AmazonSQSBufferedAsyncClient adds unexpected 200ms delay to published messages

Type: Bug / unexpected behaviour
Component:
SQS
Describe the bug
Spring Cloud AWS version: 2.4.2
If no AmazonSQS client is provided, the  SqsAutoConfiguration will initialize an AmazonSQSBufferedAsyncClient.
From the JavaDoc of that class:

...
After receiving a call, rather than executing it right away, this client waits for a configurable period of time ( default=200ms) for other calls of the same type to come in; if such calls do come in, they are also not executed immediately, but instead are added to the batch.
...

If, in practice, multiple events are published from the same thread this actually means a delay of 200ms will be added to each event. Depending of the amount of events to be published, this can add a considerable / painful delay.
Our current workaround, is to provide an AmazonSQSAsyncClient (and thereby prevent usage of the default AmazonSQSBufferedAsyncClient). See workaround below.
I'm aware that 2.4.x is not under active development anymore and you are working heavily on 3.x (much appreciated).
The goal of this report is to get more insight in this behaviour:

Is this behaviour is expected (and why?)
Maybe we're using it wrong / are we missing something obvious?
Verify 3.x behaves different. I see there the SqsAsyncClient is used. Hence I think this behaviour does not apply to 3.x?

Workaround
Provide your own AmazonSQS client, which is not a AmazonSQSBufferedAsyncClient.
    @Bean
    public AmazonSQSAsync amazonSQS() {
        return AmazonSQSAsyncClientBuilder.defaultClient();
    }

Sample
Can basically be reproduced by publishing events in a for loop.
If desired and helpful, please show me where I can best add a test and I'll add a test.

In 2.x we are using AmazonSQSBufferedAsyncClient - I would guess this is where the delay comes from. The benefit is that it makes less requests than regular SQS client.
Since buffered client has not been ported to AWS SDK v2, in 3.0 we are using regular client so the delay should not be there.


Dark theme for Remote play

[ ] Detect user's system preference
[ ] Implement a toggle to the right of the Header
[ ] Save user's preference in local storage

Awesome, you can also start with #3 as that might help us solve #9 in a more elegant way. There is a PR by @jjcrab but it has some finishing touches left.

I'll get a PR out for this one next week :)

Thank you for your help 😀 At the moment, I don't have any other Open Source project with any active development going on. Feel free to browse from the old ones and we can pick up something from there!

@shubham1172 Hello, I was wondering if I can work on just implementing a toggle to the right of the header.

@williamye51 sure, thanks for your help! If you are interested, we can also try to implement the rest of the PR together. I can help around it if needed.

@shubham1172 I ended up having time to implement the rest and just made a PR! Thanks!


Grammar railroad diagram

I've done a experimental tool to convert bison grammars to a kind of EBNF understood by https://www.bottlecaps.de/rr/ui to generate railroad diagrams see bellow the converted src/lib/parser.mly and with some hand made changes to allow view it at https://www.bottlecaps.de/rr/ui the order of the rules could be changed to a better view of the railroad diagrams. Copy and paste the EBNF bellow on https://www.bottlecaps.de/rr/ui tab Edit Grammar then switch to the tab View Diagram.
/*
From https://raw.githubusercontent.com/drjdn/ocaml_lua_parser/master/src/lib/parser.mly
*/

prog ::=  chunk EOF
chunk ::=  statlist retstat OSC | statlist
statlist ::=  OSC | statlist stat OSC
block ::=  chunk
stat ::=  varlist ASSIGN explist | functioncall | label | GOTO ident | DO block END | WHILE exp DO block END | REPEAT block UNTIL exp | IF exp THEN block END | IF exp THEN block ELSE block END | IF exp THEN block elseifps END | IF exp THEN block elseifps ELSE block END | FOR ident ASSIGN exp COMMA exp DO block END | FOR ident ASSIGN exp COMMA exp COMMA exp DO block END | FOR namelist IN explist DO block END | FUNCTION funcname funcbody | LOCAL FUNCTION ident funcbody | LOCAL namelist | LOCAL namelist ASSIGN explist
elseifp ::=  ELSEIF exp THEN block
elseifps ::=  elseifp | elseifps elseifp
retstat ::=  RETURN | RETURN explist | BREAK
label ::=  DCOLON ident DCOLON
funcname ::=  fname | fname COLON ident
varlist ::=  var | varlist COMMA var
var ::=  ident | bracket key | functioncall key | var key
key ::=  LB exp RB | DOT ident
fname ::=  ident | fname DOT ident
namelist ::=  ident | namelist COMMA ident
explist ::=  exp | explist COMMA exp
exp ::=  primary | var | functioncall | exp OR exp | exp AND exp | exp LT exp | exp GT exp | exp LE exp | exp GE exp | exp NE exp | exp EQ exp | exp CAT exp | exp PLUS exp | exp MINUS exp | exp MULT exp | exp DIV exp | exp MOD exp | exp CARAT exp | NOT exp | HASH exp | MINUS exp
primary ::=  BOOL | NUM_CONST | STR_CONST | ELLIPSIS | functiondef | tableconstructor | bracket
bracket ::=  LPAR exp RPAR
functioncall ::=  bracket args | bracket COLON ident args | var args | var COLON ident args | functioncall args | functioncall COLON ident args
args ::=  LPAR RPAR | LPAR explist RPAR | tableconstructor | STR_CONST
functiondef ::=  FUNCTION funcbody
funcbody ::=  LPAR RPAR block END | LPAR parlist RPAR block END
parlist ::=  namelist | namelist COMMA ELLIPSIS | ELLIPSIS
tableconstructor ::=  LCB RCB | LCB fieldlist RCB
fieldlist ::=  fields | fields fieldsep
fields ::=  field | fields fieldsep field
field ::=  LB exp RB ASSIGN exp | ident ASSIGN exp | exp
fieldsep ::=  COMMA | SEMI
ident ::=  IDENT
OSC ::=  /* empty */ | SEMI

// Tokens

PLUS ::= "+"
MINUS ::= "-"
MULT ::= "*"
DIV ::= "/"
MOD ::= "%"
CARAT ::= "^"
GT ::= ">"
LT ::= "<"
GE ::= ">="
LE ::= "<="
EQ ::= "=="
NE ::= "~="
ASSIGN ::= "="
DOT ::= "."
CAT ::= ".."
ELLIPSIS ::= "..."
COLON ::= ":"
DCOLON ::= "::"
SEMI ::= ";"
COMMA ::= ","
HASH ::= "#"
LCB ::= "{"
RCB ::= "}"
LPAR ::= "("
RPAR ::= ")"
LB ::= "["
RB ::= "]"
AND ::= "and"
BREAK ::= "break"
DO ::= "do"
ELSE ::= "else"
ELSEIF ::= "elseif"
END ::= "end"
FOR ::= "for"
FUNCTION ::= "function"
GOTO ::= "goto"
IF ::= "if"
IN ::= "in"
LOCAL ::= "local"
NOT ::= "not"
OR ::= "or"
REPEAT ::= "repeat"
RETURN ::= "return"
THEN ::= "then"
UNTIL ::= "until"
WHILE ::= "while"

The current grammar has no shift-reduce or reduce-reduce warnings.  I'm
open to suggestions on how to reorder the grammar for better graphical
representation provided that would still hold and all tests pass.
On Tue, Jun 29, 2021 at 4:10 AM Domingo Alvarez Duarte <
@.***> wrote:

I've done a experimental tool to convert bison grammars to a kind of EBNF
understood by https://www.bottlecaps.de/rr/ui to generate railroad
diagrams see bellow the converted src/lib/parser.mly and with some hand
made changes to allow view it at https://www.bottlecaps.de/rr/ui the
order of the rules could be changed to a better view of the railroad
diagrams. Copy and paste the EBNF bellow on
https://www.bottlecaps.de/rr/ui tab Edit Grammar then switch to the tab
View Diagram.
/*
From https://raw.githubusercontent.com/drjdn/ocaml_lua_parser/master/src/lib/parser.mly
*/
prog ::=  chunk EOF
chunk ::=  statlist retstat OSC | statlist
statlist ::=  OSC | statlist stat OSC
block ::=  chunk
stat ::=  varlist ASSIGN explist | functioncall | label | GOTO ident | DO block END | WHILE exp DO block END | REPEAT block UNTIL exp | IF exp THEN block END | IF exp THEN block ELSE block END | IF exp THEN block elseifps END | IF exp THEN block elseifps ELSE block END | FOR ident ASSIGN exp COMMA exp DO block END | FOR ident ASSIGN exp COMMA exp COMMA exp DO block END | FOR namelist IN explist DO block END | FUNCTION funcname funcbody | LOCAL FUNCTION ident funcbody | LOCAL namelist | LOCAL namelist ASSIGN explist
elseifp ::=  ELSEIF exp THEN block
elseifps ::=  elseifp | elseifps elseifp
retstat ::=  RETURN | RETURN explist | BREAK
label ::=  DCOLON ident DCOLON
funcname ::=  fname | fname COLON ident
varlist ::=  var | varlist COMMA var
var ::=  ident | bracket key | functioncall key | var key
key ::=  LB exp RB | DOT ident
fname ::=  ident | fname DOT ident
namelist ::=  ident | namelist COMMA ident
explist ::=  exp | explist COMMA exp
exp ::=  primary | var | functioncall | exp OR exp | exp AND exp | exp LT exp | exp GT exp | exp LE exp | exp GE exp | exp NE exp | exp EQ exp | exp CAT exp | exp PLUS exp | exp MINUS exp | exp MULT exp | exp DIV exp | exp MOD exp | exp CARAT exp | NOT exp | HASH exp | MINUS exp
primary ::=  BOOL | NUM_CONST | STR_CONST | ELLIPSIS | functiondef | tableconstructor | bracket
bracket ::=  LPAR exp RPAR
functioncall ::=  bracket args | bracket COLON ident args | var args | var COLON ident args | functioncall args | functioncall COLON ident args
args ::=  LPAR RPAR | LPAR explist RPAR | tableconstructor | STR_CONST
functiondef ::=  FUNCTION funcbody
funcbody ::=  LPAR RPAR block END | LPAR parlist RPAR block END
parlist ::=  namelist | namelist COMMA ELLIPSIS | ELLIPSIS
tableconstructor ::=  LCB RCB | LCB fieldlist RCB
fieldlist ::=  fields | fields fieldsep
fields ::=  field | fields fieldsep field
field ::=  LB exp RB ASSIGN exp | ident ASSIGN exp | exp
fieldsep ::=  COMMA | SEMI
ident ::=  IDENT
OSC ::=  /* empty */ | SEMI
// Tokens
PLUS ::= "+"
MINUS ::= "-"
MULT ::= "*"
DIV ::= "/"
MOD ::= "%"
CARAT ::= "^"
GT ::= ">"
LT ::= "<"
GE ::= ">="
LE ::= "<="
EQ ::= "=="
NE ::= "~="
ASSIGN ::= "="
DOT ::= "."
CAT ::= ".."
ELLIPSIS ::= "..."
COLON ::= ":"
DCOLON ::= "::"
SEMI ::= ";"
COMMA ::= ","
HASH ::= "#"
LCB ::= "{"
RCB ::= "}"
LPAR ::= "("
RPAR ::= ")"
LB ::= "["
RB ::= "]"
AND ::= "and"
BREAK ::= "break"
DO ::= "do"
ELSE ::= "else"
ELSEIF ::= "elseif"
END ::= "end"
FOR ::= "for"
FUNCTION ::= "function"
GOTO ::= "goto"
IF ::= "if"
IN ::= "in"
LOCAL ::= "local"
NOT ::= "not"
OR ::= "or"
REPEAT ::= "repeat"
RETURN ::= "return"
THEN ::= "then"
UNTIL ::= "until"
WHILE ::= "while"
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/drjdn/ocaml_lua_parser/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJWETVVNHCQTF6QRD3HKDDTVF5YFANCNFSM47PR5P5Q
.


Blood puddles: misleading visuals and mopping messages

Description

Tiny blood puddles display the wet floor sparkles (intended to strictly indicate slipping potential) despite not being slippable (I blame my own shitcode lmao).
On mopping interaction, tiny blood puddles will show the "You squeeze some liquid onto the floor" message, despite having a dry mop. This seems to be the behaviour when an empty (dry) part of the station floor is interacted with as well (in which case, more of a mopping problem than a blood problem).
When mopping a large blood puddle, it will only let you mop it down to 5 units (per current MopLowerLimit wet-floor behaviour) but the mop will continue to say "you mop the puddle" despite no change in size. This piece is more a game-feedback-to-player issue than anything else.

Interact with dry mop.

Create large blood puddles by the same method.

Interact with dry mop.

Screenshots

https://user-images.githubusercontent.com/97707302/194965238-f9e8725d-0b8f-4189-b1fe-f8eadc6bd0df.mp4
https://user-images.githubusercontent.com/97707302/194965840-3034062f-5878-4c8a-be27-68f9a1cabc4d.mp4
Additional context

As I noted in #6848, MopLowerLimit needs to be moved from the AbsorbentComponent to the PuddleComponent (so that, for example, we could make blood fully mop-up-able, but water and other types of puddle would still leave the wet-floor-residue).
I still wanna refactor puddle visuals so the wet floor sparkles only appear when the following three conditions are met:

The puddle is too small for the mop to pick it up (Janitor should move onto the next one)
The puddle will evaporate on its own (Janitor doesn't need to worry about this one anymore, beyond maybe placing a wet floor sign)
The puddle can still slip people (People should watch out for the sparkle effect. See also: wet floor sign)

Someone can assign me to this because I wanna tackle this in the short-to-medium term.


Create Divider line function

This PR creates a divider line with some labels between the historical emissions and the quantifications.
https://www.pivotaltracker.com/story/show/160311982
Note: We can't import it as a Component as the Composed chart will only accept Rechart components .

Ready again. Thanks for the catch!

Hey reviewers, can you re review?

@Bluesmile82 thanks for addressing the changes requested


"Dense" AppBar

Material-design defines a "dense" version of the AppBar for Desktop. It makes sense, because the AppBar is generally way too large on Desktop.
The <AppBar> component misses this dense prop.

Is anyone working on this feature? i'd love to work on it.

@adeelibr it seems nobody does, feel free to give it a go!

@fzaninotto I am going to start working on it from tomorrow. I am already working on this table thing. It's kind of a patch. Anyways i'll do this by tomorrow. :) Cheers.


Trying to save just a basic page, doesn't work.

Error: Bad Request
Error: Bad Request
    at Request.<anonymous> (https://homebrewery.naturalcrit.com/homebrew/bundle.js:71824:19)
    at Request.Emitter.emit (https://homebrewery.naturalcrit.com/homebrew/bundle.js:1764:20)
    at XMLHttpRequest.xhr.onreadystatechange (https://homebrewery.naturalcrit.com/homebrew/bundle.js:72133:10)
{
  "status": 400,
  "method": "POST",
  "url": "/api/newGoogle/"
}

Check out discussion #1580 to reset your google credentials.  If the steps there don't help, come back here and let us know but otherwise going to close this Issue in the meantime.  Thanks for the report


[BEAM-432] Corrected BigQueryIO javadoc

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Link to the issue: https://issues.apache.org/jira/browse/BEAM-432

Thanks! LGTM


Print module paths for debug logs only

I set the log level for printing module paths in ModuleClassLoader to DEBUG to reduce the amount on logging on startup.
As I understand it, this output is mainly for debugging.

Can you submit this to the 'gestalt' repository? This repository is going to be deleted shortly.


[Editor] Saving siddhi files on a remote Linux instance accessed via a Windows instance

Description:
When we start Stream Processor editor in a remote linux server and access it via a Windows instance, the siddhi file created would be saved on SP_Pack_HOME/wso2/editor/deployment  and the name of the siddhi file would be appended with "workspace\" at the front.
Suggested Labels:
Editor, Remote Access
Suggested Assignees:
@erangatl
Affected Product Version:
Stream Processor v4.2.0
OS, DB, other environment details and versions:
Remote OS : Ubuntu 16.04
Local OS: Windows 10
Steps to reproduce:

Start Stream Processor in editor profile in a Linux remote instance.
Access the editor from a Windows instance.
Create Siddhi file and save it using the name TestApp.siddhi.
View SP_HOME/wso2/editor/deployment directory.
The sidhdi file will be saved with the name workspace\TestApp.siddhi

fixed in https://github.com/wso2/carbon-analytics/pull/1380


DCOS-40650: upload source maps to sentry.io

This PR adds a script to our release pipeline that uploads source maps to sentry.io to improve dev experience.
Testing

Get Sentry token in Onelogin secure notes
Make a build npm run build-assets
Run the script

SENTRY_AUTH_TOKEN="<token>" \
  RELEASE_NAME="test-<any salt>" \
  ./scripts/ci/utils/upload-sourcemaps

Trade-offs

It's a shell script, sorry
Decided to go with a CLI lib instead of cURL https://docs.sentry.io/clients/javascript/sourcemaps/#uploading-source-maps-to-sentry to spare some time debugging
This very PR won't trigger a release since we don't release chore commits

:tada: This PR is included in version 2.22.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Tests fail but build succeeds

Describe the Bug
Tests failed but codecov succeeded so Travis CI thought that job is successful.
https://travis-ci.org/github/camunda/camunda-modeler/jobs/694279584
Steps to Reproduce

See linked job above.
Expected Behavior
Job fails.
Environment

OS: Travis CI / Linux
Camunda Modeler Version: 4.0.0

Closed via https://github.com/camunda/camunda-modeler/commit/30e0852a3b975bfaab88d136a16c93c95045d927


Fix logger signed unsigned error with GCC 7.3 in Cygwin

Thanks to @bkueng for suggesting this change in #9920 !
closes #9920

Combine for nuttx?

QuRT doesn't like it (what a surprise).

Combine for nuttx?

Yes, and apparently exclude QuRT

In that case you might as well do the typedef directly in logger.cpp.

Combine for nuttx?
exclude QuRT

I feel somehow not in the loop. Do I look for existing ifdefs and experiment with them such that CI gets happy for qurt and mac? What does it have to do with NuttX? Are you refering to only define it in one spot for all platforms?

Sorry, should have been more explicit.
That's pretty close, one last thing I was also suggesting is to use typedef decltype(statfs::f_bavail) px4_statfs_buf_f_bavail_t; for both posix and nuttx. QuRT we can ignore entirely because logger isn't even built on that side.


Editing file cannot be cancelled

go to the Files menu
upload an image

The image was now added.

now click under the file title on "Edit"
since you don't want to do anything there, click "Cancel"

result: nothing happens, one is forced to press Save despite that nothing has to be saved. Therefore the Cancel button should either be removed or close the window.

Confirmed fixed.


[MeshMap] Fetch RemoteUserPref Component

Description
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@leecalcote Why should these be in master branch? These are meshmap specific right

@vineethvanga18 some of these belong in master because this PR is creating an extension point in Meshery itself.

Here is a link to the example provider package that will provide some insight on the subject - https://drive.google.com/file/d/1mQieb01yK1dhM11bC0zUF66YrXzMEKel/view?usp=sharing and http://docs.meshery.io/extensibility/providers#building-a-provider. @utkarsh-pro will be demo'ing work on this topic on tomorrow's call. 👍

@leecalcote ^^

@vineethvanga18 will you have this included in the meshery-extensions repo for Thursday's meeting?

@vineethvanga18 will you have this included in the meshery-extensions repo for Thursday's meeting?


Disable spammy log message

I've set enable_log to false, yet still get [AutoSetSyntax] ⏳ Calm down! View has gone or the plugin is not ready yet.
Not sure why I should be worried, but if I were, the message wouldn't calm me down :), it doesn't imply I need to do anything
Could you please disable this message as well?

Sure. Done in 8256b2b.


Only AnimationClip

I am using this on a game called Verdun. I can export everything, except:

FBX export meshes with skeleton
AnimationClip but don't know what to do with it.

For FBX exports with skeletons,you gotta export from the Scene Hierarchy,while for 1 AnimationClip you have to also select the Animator used for that animation.

@RamChupi how to export from the Scene Hierarchy?

@lantiszhu Pretty simple,select what you want then just press export selected 3D model.


Fix Viewer

It was accidentally broken in #2774 and somehow all tests still passed.  I added a test that verifies a blank viewer renders without error.

Fixes #2777

Tests pass, including new one, and viewer is working again.


Normalize order or type arguments for Query methods in Dispatcher and…

… add XML documentation comments

:white_check_mark: Build Magneto 33 completed (commit https://github.com/shaynevanasperen/Magneto/commit/f5512e6618 by @shaynevanasperen)


attachInterrupt(USER_BUTTON_B,...) & new LSM6DSLSensor(*i2c, D4, D5)

If you attach an Interrupt to the User_Button_B and add the LSM6DSLSensor for the gyroscope, the interrupt is not working anymore.
EDIT: Same Code works as expected when attaching the interrupt to USER_BUTTON_A instead

This issue was moved to Microsoft/devkit-sdk#895


Add estimated time to completion for uploading and converting

It's pretty trivial to do an algo that estimates time until upload completion and conversion completion. Based on how trivial it is this should also be implemented.

I can do this.

This is done just needs to be touched up


Fix broken links

Fix 4 broken links  (1- link to image , 3-links)

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

To many mistake , sorry.  I will create new Pull request again .