Add compression configuration for functionColumnPairs to StartreeIndexConfigs

Adding functionColumnPairsConfig to enable configuring compression.
"starTreeIndexConfigs": [
        {
          "dimensionsSplitOrder": [
            "a",
            "b",
            "c"
          ],
          "skipStarNodeCreationForDimensions": [],
          "functionColumnPairs": [],
          "aggregationConfigs": [
            {
              "columnName": "column1",
              "aggregationFunction": "SUM",
              "chunkCompressionType": "SNAPPY"
            },
            {
              "columnName": "column2",
              "aggregationFunction": "distinctcounthll",
              "chunkCompressionType": "LZ4"
            }
          ],
          "maxLeafRecords": 10000
        }
      ]

See: https://github.com/apache/pinot/issues/9694
This change was tested manually with multiple segment refresh on DOUBLE and BYTES metrics columns.

Codecov Report

Merging #11744 (2e9d36b) into master (e0a1f62) will decrease coverage by 48.72%.
Report is 15 commits behind head on master.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #11744       +/-   ##
=============================================
- Coverage     63.13%   14.41%   -48.72%
+ Complexity     1118      201      -917
=============================================
  Files          2342     2342
  Lines        125883   125960       +77
  Branches      19357    19378       +21
=============================================
- Hits          79477    18161    -61316
- Misses        40749   106259    +65510
+ Partials       5657     1540     -4117

Flag
Coverage Δ

integration
?

integration1
?

integration2
?

java-11
?

java-17
14.41% <0.00%> (-48.52%)
:arrow_down:

java-20
?

temurin
14.41% <0.00%> (-48.72%)
:arrow_down:

unittests
14.41% <0.00%> (-48.72%)
:arrow_down:

unittests1
?

unittests2
14.41% <0.00%> (-0.03%)
:arrow_down:

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

Files
Coverage Δ

...egment/spi/index/startree/StarTreeV2Constants.java
0.00% <ø> (-50.00%)
:arrow_down:

...t/segment/local/startree/StarTreeBuilderUtils.java
0.00% <0.00%> (-90.91%)
:arrow_down:

...ocal/startree/v2/builder/MultipleTreesBuilder.java
0.00% <0.00%> (-56.05%)
:arrow_down:

...he/pinot/spi/config/table/StarTreeIndexConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...al/startree/v2/builder/StarTreeIndexSeparator.java
0.00% <0.00%> (-86.54%)
:arrow_down:

...segment/spi/index/startree/StarTreeV2Metadata.java
0.00% <0.00%> (-93.75%)
:arrow_down:

...cal/startree/v2/builder/BaseSingleTreeBuilder.java
0.00% <0.00%> (-90.46%)
:arrow_down:

...he/pinot/segment/local/utils/TableConfigUtils.java
0.00% <0.00%> (-73.42%)
:arrow_down:

...l/startree/v2/builder/StarTreeV2BuilderConfig.java
0.00% <0.00%> (-73.24%)
:arrow_down:

... and 1523 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks for the review. Adressing everything :)

Have you pushed the changes?

Not yet, I'll try to do it by eod

I applied a commit to introduce the AggregationSpec and revert the changes in AggregationFunctionColumnPair because that is used to match the functions at query time. Changed the TreeSet<AggregationFunctionColumnPair> to TreeMap<AggregationFunctionColumnPair, AggregationSpec> so that we can use keySet() to achieve the same behavior.
Other changes:

Removed the duplicate config check because I feel user might config both fields, and we prioritize on aggregationConfigs over functionColumnPairs
Simplified the test changes to use a random compression codec in BaseStarTreeV2Test.

Please take a look and let me know if you have concern about the change

Looks good from my side, and much cleaner. Thanks for helping. :)

@t0mpere Thanks for the contribution! Could you please also help contribute the documentation for the new config? It can be added to this repo: git@github.com:pinot-contrib/pinot-docs.git

Sure, I'm OOO at the moment I will take care in 2 weeks as soon as I will get back to my laptop.

For reference,
Documentation pr: https://github.com/pinot-contrib/pinot-docs/pull/258


Add type coercion to top-down fix expression types

Setup an outline for how top-down hinting can help type coerce expressions that otherwise lack the context to know what types they need to be. The benefits of this compared to type casting within individual expression Evals:

Controlling types upfront should aid correctness, for example for aggregation functions that return different types depending on the context, or INSERTs/UPDATEs that expect specific type inputs, or bindvar substitution
Adding type casts while we have info at the AST layer should generalize to more expression types than adding type casts in individual execution functions
Front-loading type information should make validation easier to add at binding, which is preferable to after analysis and is probably easier than doing another tree walk

fixes: https://github.com/dolthub/dolt/issues/7018

closing this for now, refer here for background: https://docs.google.com/document/d/1AaoIGaoLktCFtQeqJLzJsQMKYL12K1GvnyVTq-WkeTg/edit


draft rc1 release notes

Draft for 2.3.0rc1 release notes are here. If devs of packages could add under the Major Highlights section, that would be great.

@sjsrey, I updated the spaghetti highlights, but now the "Draft is here" link doesn't seem to work; getting a 404 error.

@sjsrey, I updated the spaghetti highlights, but now the "Draft is here" link doesn't seem to work; getting a 404 error.

should be fixed now. the url updates for each edit/save. so folks should edit the 2.3.0rc1 tag that shows up at the top.

@sjsrey I updated the splot highlight, let me know if what I wrote should be included in the release notes, if not I am happy to remove it again.


Pin setup-java to v2

Context

Previous PR went over my head and I couldn't manage to streamline it. That's the reason, I am opening this PR. Sorry for the inconvenience.
Updating the setup-java action to v2.

V2 supports custom distributions and provides support for Zulu OpenJDK, Eclipse Temurin and Adopt OpenJDK. V1 supports only Zulu OpenJDK.
V1 supported legacy Java syntax such as 1.8 (same as 8) and <IP_ADDRESS> (same as 8.0.212). V2 dropped support for legacy syntax.
Switched to Temurin which is an official successor of Adopt OpenJDK. See here.

Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass sanity check: ./gradlew sanityCheck
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@bot-gradle test and merge

No milestone for this PR. Please set a milestone and retry.

@bot-gradle test and merge

OK, I've already triggered a build for you.


PRL (Oyster) missing

Should be easy. Here's the official website (https://oysterprotocol.com/).

Added - https://github.com/cjdowner/cryptocurrency-icons/commit/1f918b8910627f7d13ac122f641808750b122c30

Thank you for your support. If you’d like to tip me for my time in maintaining and updating this project, it would be very much appreciated.


Consumer support message send back with origin message modified.

in the message listener we can modify the origin message. modify the message body add some property. and send back to broker .

Hi, @WJL3333 We are working on it, a new RIP will try to provide immutability APIs for developers, please kindly pay attention to our mailing lists and join the upcoming discussion.


Account Lockup

Describe the bug
Currently there is no block on the number of failed login attempt. In other words, a bot can try unlimited possibilities till it's able to login without authorization.
To Reproduce
Attempt multiple login with wrong credentials
Expected behavior
After X (10?) number of failed logins, there will be a temporary block on the IP attempting to login.

The latest version includes this feature, please upgrade to 7.9.2


Add support for returning the play area from XRInterface

This PR adds support for querying the play area available to an XR game.
Currently most Godot 3 XR plugins implement their own version of this such as:

OpenVR: https://github.com/GodotVR/godot_openvr/blob/master/src/open_vr/OpenVRConfig.cpp#L93
Oculus desktop: https://github.com/GodotVR/godot_oculus/blob/master/src/oculus/oculus_config.cpp#L38
Oculus mobile: https://github.com/GodotVR/godot_oculus_mobile/blob/master/plugin/src/main/cpp/api/ovr_guardian_system.cpp#L23 (currently disabled)

This means that in order to implement functionality around this platform dependent implementations are needed.
By adding an entry point into the XRInterface that plugin developers can implement we can standardize this solution.
The PR also adds methods to query and set the play area mode. It is up to the XR interface to implement what is supported here as not all XR runtimes support the same modes.
Finally a signal play_area_changed is introduced that should be emitted whenever the play area changes regardless of the cause of this change. It is up to the XR interface to emit this signal, the signal definition is purely there for standardization.

So far I just added get_play_area to retrieve the data and a signal play_area_changed with which an interface can signal the play area has changed (either because the user has changed their guardian boundaries, the reference frame was changed or the world scale was changed)

Just a note in support of this!
WebXR also has a method for returning an array of points representing the guardian:
https://github.com/godotengine/godot/blob/master/modules/webxr/webxr_interface_js.cpp#L181

Just a note in support of this!
WebXR also has a method for returning an array of points representing the guardian:
https://github.com/godotengine/godot/blob/master/modules/webxr/webxr_interface_js.cpp#L181

Cool, all the more proof this seems to be the standard format for extracting this data and in Godot 4 the webxr interface can just be changed to use the new method on XRInterface :)

Under the assumption that the new GDNative plugins will make this work differently I am removing the changes to GDNative from this PR.
Even if some of the old approach needs to stay I want to finish the changes we did for the ARCore plugin and retrofit those to the other plugins (may even do this in Godot 3 as the core changes are merged) which would allow us to implement member functions in the normal GDNative way where the overhead is acceptable.

This needs to be adjusted based on #52003


[Feature] yarn-seesion 模式集群配置建议

Search before asking

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

Description
目前yarn-seesion 模式集群配置需要自己手动启动flink的 yarn-session命令，然后找到application ID,填写到集群ID里面，这样不够自动化。建议一气呵成，简化使用，减少误解。
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

please use English.

Next time, please use English to solve the problems or bugs I raised. Urgently use. If it doesn't work, it will no longer be used. It is too unstable.

Next time I use English, please solve my problems or bugs. If it doesn't work, I won't use it anymore. It's too unstable.

Without concrete description, this is barely a wishing list. We don't host a wishing list in issues since they're not actionable. You can share your ideas at: https://github.com/apache/incubator-streampark/discussions/categories/general.
Also lock this conversation because although you tick the Code of Conduct, you're still threatening in this thread. We simply not welcome this manner.


Update Dockerfile

is that working?

@Marie59 I have reworked the entire Dockerfile. I have no clue what I'm doing. But the idea is that we use Julia pre-installed. We do not care about Julia, just about the Divand extension, which we compile, and the example files.
I think this now at least compiles. But I do not know if it also works. If you want to take it over from here, feel free, bedtime.

Okay Thank you !! I will see the outcome of that and see what else needs to be done.

@Marie59 I have reworked the entire Dockerfile. I have no clue what I'm doing. But the idea is that we use Julia pre-installed. We do not care about Julia, just about the Divand extension, which we compile, and the example files.
I think this now at least compiles. But I do not know if it also works. If you want to take it over from here, feel free, bedtime.

I tried this docker locally and everything seems to work fine for me.


Add new release folder structure

Add new release folder structure

@xiangyan99 this is for June release notes. Can you point out in the file what changes you want to keep?

@xiangyan99 I will definitely get those changes merged in. This PR is just testing out some new infrastructure for generating the release note folders.

The changes to dotnet.md and python.md have been undone.


Update API Docs

I came across a few minor issues in the API documentation.
(1) In the example response for read shelf/book, the tags have id - but they do not have id just like at any other endpoint.
(2) In the example response for read book, the page within contents does not have a type - but a quick test on the demo shows that any object within contents has a type.
I also noticed some fields are missing in the example responses, e.g. pages have an editor field (and sometimes url) and users within entities also a slug besides name and id. I assume it's an unnecessary bloat of the example responses to add those?

Thanks @devdot!

I also noticed some fields are missing in the example responses, e.g. pages have an editor field (and sometimes url) and users within entities also a slug besides name and id. I assume it's an unnecessary bloat of the example responses to add those?

If it's just singular odd fields missing that are currently being provided in API responses, then it's probably not due to bloat, but instead there have been more recent changes, after responses were recorded, what have tangentially affected those old endpoints. Ideally those responses would correctly reflect all fields.

I've gone through the example responses and made them match the actual responses as I worked through them for a project I'm working on.
One potential issue I found is that chapters create does not return book_slug, even though all other chapters endpoints do return that value. Adding that should be trivial, but it's not in the scope of this PR.

Thank you very much for these updates @devdot!


[BUG] French Translation: Apostrophe Display Issue

What is the bug?
Hello,
I've just discovered a minor bug in the French translation. When I switch to the French translation, a small apostrophe appears on the chat and also on the tab. However, when I switch back to English, the issue disappears. i try to find the ' but in don't see

Best regards,
Expected Behaviour
i want to see here is the apostrophe
How to Reproduce

change the config to french
' appears in ending name

Error message
no error message
Output of /teama version
[Craft Scheduler Thread - 5 - BetterTeams/INFO]: Storage Method: com.booksaw.betterTeams.team.storage.storageManager.SeparatedYamlStorageManager
[02:15:21] [Craft Scheduler Thread - 5 - BetterTeams/INFO]: Minecraft Version: git-Paper-454 (MC: 1.20.4)
[02:15:21] [Craft Scheduler Thread - 5 - BetterTeams/INFO]: Language Selected: messages
[02:15:21] [Craft Scheduler Thread - 5 - BetterTeams/INFO]: Online mode: true
[02:15:21] [Craft Scheduler Thread - 5 - BetterTeams/INFO]: Player count: 1
[02:15:21] [Craft Scheduler Thread - 5 - BetterTeams/INFO]: Current plugin version: 4.9.1
Additional Information
No response

hi, i found the ' error here the config edited
https://haste.zendrique.fr/kapifogofa.yaml


Assets need to be updated as well (only for adapters)

Body
Within https://github.com/apache/streampipes/issues/2001 we introduced basic support for migrating pipeline elements.
One aspect that is not yet covered is the update of the corresponding assets (string values, documentation & icon) for adapters.
So we need to include the update of the assets in the migration process of an adapter at the core.
StreamPipes Committer
I acknowledge that I am a maintainer/committer of the Apache StreamPipes project.

closed by https://github.com/apache/streampipes/pull/2105


An exception occurs when the library containing Zxing is compiled

Microsoft Windows [版本 6.1.7601]
版权所有 (c) 2009 Microsoft Corporation。保留所有权利。
C:\Users\amin>enjarify D:\QcUsers\Win7\Administrator\Desktop\Fordeal.apk -
o D:\QcUsers\Win7\Administrator\Desktop\FORDEAL.jar
1000 classes processed
2000 classes processed
3000 classes processed
4000 classes processed
5000 classes processed
6000 classes processed
7000 classes processed
8000 classes processed
9000 classes processed
Output written to D:\QcUsers\Win7\Administrator\Desktop\FORDEAL.jar
com/google/zxing/aztec/b/d.class Traceback (most recent call last):
File "F:\PythonProjects\enjarify\enjarify\main.py", line 38, in translate
class_data = writeclass.toClassFile(cls, opts)
File "F:\PythonProjects\enjarify\enjarify\jvm\writeclass.py", line 122, in toC
lassFile
pool.write(stream)
File "F:\PythonProjects\enjarify\enjarify\jvm\constantpool.py", line 135, in w
rite
self._writeEntry(stream, item)
File "F:\PythonProjects\enjarify\enjarify\jvm\constantpool.py", line 104, in _
writeEntry
stream.u16(len(val))
TypeError: object of type 'NoneType' has no len()
com/google/zxing/common/i.class Traceback (most recent call last):
File "F:\PythonProjects\enjarify\enjarify\main.py", line 38, in translate
class_data = writeclass.toClassFile(cls, opts)
File "F:\PythonProjects\enjarify\enjarify\jvm\writeclass.py", line 122, in toC
lassFile
pool.write(stream)
File "F:\PythonProjects\enjarify\enjarify\jvm\constantpool.py", line 135, in w
rite
self._writeEntry(stream, item)
File "F:\PythonProjects\enjarify\enjarify\jvm\constantpool.py", line 104, in _
writeEntry
stream.u16(len(val))
TypeError: object of type 'NoneType' has no len()
com/google/zxing/pdf417/d.class Traceback (most recent call last):
File "F:\PythonProjects\enjarify\enjarify\main.py", line 38, in translate
class_data = writeclass.toClassFile(cls, opts)
File "F:\PythonProjects\enjarify\enjarify\jvm\writeclass.py", line 122, in toC
lassFile
pool.write(stream)
File "F:\PythonProjects\enjarify\enjarify\jvm\constantpool.py", line 135, in w
rite
self._writeEntry(stream, item)
File "F:\PythonProjects\enjarify\enjarify\jvm\constantpool.py", line 104, in _
writeEntry
stream.u16(len(val))
TypeError: object of type 'NoneType' has no len()
com/google/zxing/pdf417/decoder/h.class Traceback (most recent call last):
File "F:\PythonProjects\enjarify\enjarify\main.py", line 38, in translate
class_data = writeclass.toClassFile(cls, opts)
File "F:\PythonProjects\enjarify\enjarify\jvm\writeclass.py", line 122, in toC
lassFile
pool.write(stream)
File "F:\PythonProjects\enjarify\enjarify\jvm\constantpool.py", line 135, in w
rite
self._writeEntry(stream, item)
File "F:\PythonProjects\enjarify\enjarify\jvm\constantpool.py", line 104, in _
writeEntry
stream.u16(len(val))
TypeError: object of type 'NoneType' has no len()
com/google/zxing/pdf417/encoder/a.class Traceback (most recent call last):
File "F:\PythonProjects\enjarify\enjarify\main.py", line 38, in translate
class_data = writeclass.toClassFile(cls, opts)
File "F:\PythonProjects\enjarify\enjarify\jvm\writeclass.py", line 122, in toC
lassFile
pool.write(stream)
File "F:\PythonProjects\enjarify\enjarify\jvm\constantpool.py", line 135, in w
rite
self._writeEntry(stream, item)
File "F:\PythonProjects\enjarify\enjarify\jvm\constantpool.py", line 104, in _
writeEntry
stream.u16(len(val))
TypeError: object of type 'NoneType' has no len()
com/google/zxing/qrcode/a/b.class Traceback (most recent call last):
File "F:\PythonProjects\enjarify\enjarify\main.py", line 38, in translate
class_data = writeclass.toClassFile(cls, opts)
File "F:\PythonProjects\enjarify\enjarify\jvm\writeclass.py", line 122, in toC
lassFile
pool.write(stream)
File "F:\PythonProjects\enjarify\enjarify\jvm\constantpool.py", line 135, in w
rite
self._writeEntry(stream, item)
File "F:\PythonProjects\enjarify\enjarify\jvm\constantpool.py", line 104, in _
writeEntry
stream.u16(len(val))
TypeError: object of type 'NoneType' has no len()
9498 classes translated successfully, 6 classes had errors
C:\Users\amin>

Have you confirmed that the relevant classes are valid?
APKs can contain invalid classes, and Android will not complain as long as the invalid classes are not referenced anyway. Sometimes APKs do this to discourage reverse engineering. It is also possible that the APK was designed for an old version of Android which did not verify bytecode. In order to handle cases like this Enjarify, will just skip the class and report an error, instead of treating invalid classes as fatal errors.

Oh，I see.
Project that might reference the Zxing library, It's compressed when compile it to apk.


"Error installing battery limiter: undefined" when launching UI on Monterey 12.5.1

I installed Battery v1.0.3 using homebrew and then started the UI. After installing the command line component it shows a dialog saying Battery limiter error: undefined and after relaunching the UI, it shows a dialog with the error message Error installing battery limiter: undefined. Not sure what to try to fix this, let me know if you need any other information.

Huh, this made the error go away: sudo chmod a+w /Users/$(whoami)/.battery/battery.log
Looks like a permission issue with this file, which after installation had -rw-r--r--  1 root  staff permissions

Well done on the fix! Did you have another version (the CLI only?) of battery installed prior to installing with Homebrew?

I hadn't, and I repro'd it after deleting the ~/.battery folder.

Thanks for the details, I'll add this to the list :)

I get the error regardless of the permissions of anything in ~/.battery, and it only happens when launching Battery from Spotlight when it's not already running.
It happens in these exact circumstances:
Given:

Battery installed only ever from Homebrew cask, updated today
Nothing has been installed outside of Homebrew, Nix, and the Mac App Store
Run "Battery" from Spotlight, when the Battery application is not running

This error shows:

If I run open /Applications/Battery.app, the Battery app starts and shows its menulet in the Notification Area:

Then further invocations of "Battery" from Spotlight have no effect (probably because Mac OS doesn't actually do anything after a Spotify invocation of an app it detects is already running).
If I kill the Battery app by clicking "Quit" on its menulet dropdown menu, then all invocations of "Battery" from Spotlight show the error dialogue shown above in this message, until battery is run from open /Applications/Battery.app again.
So there is something wrong specifically with the Spotify invocation of Battery.
Running sudo chmod -Rc a+w ~/.battery has no effect in terms of preventing the error message.


Discord badge from invite instead of guild ID

📋 Description

Service: Discord
Information: Similar to the Discord badge currently available

Current badge:
![Discord](https://img.shields.io/discord/<GUILD_ID>)

DIY new badge:
[![Discord](https://img.shields.io/badge/dynamic/json?color=green&label=chat&query=%24.approximate_presence_count&suffix=%20online&url=https%3A%2F%2Fdiscord.com%2Fapi%2Fv10%2Finvites%2F<INVITE_CODE>%3Fwith_counts%3Dtrue)](https://discord.gg/<INVITE_CODE>)

A Discord badge generated by providing an invite code, instead of the guild ID.
Invite codes look like HjJCwm5 (this one is Shields' Discord).
Having the <INVITE_CODE> also allows the generated badge Markdown to be clickable to join the server:
[![Discord](https://img.shields.io/discord/invite/<INVITE_CODE>)](https://discord.gg/<INVITE_CODE>)

I believe this doesn't even require the widget setting to be enabled on the guild.
🔗 Data
Data can be fetched like so:
https://discord.com/api/v10/invites/<INVITE_ID>?with_counts=true

Field
Description

guild.id
guild ID

approximate_member_count
approximate member count

approximate_presence_count
approximate online members

expires_at
expiration, if any

expires_at could be used to warn the user, when creating a badge, if they try to use an invite that expires. This may require adding the query parameter with_expiration=true.

Example data with Shields' invite
https://discord.com/api/v10/invites/HjJCwm5?with_counts=true
{
  "code": "HjJCwm5",
  "type": 0,
  "expires_at": null,
  "guild": {
    "id": "308323056592486420",
    "name": "shields",
    "splash": null,
    "banner": null,
    "description": null,
    "icon": "11a50197f2858fa14b74f41ceaacc4b6",
    "features": [],
    "verification_level": 1,
    "vanity_url_code": null,
    "premium_subscription_count": 0,
    "nsfw": false,
    "nsfw_level": 0
  },
  "channel": {
    "id": "308323056592486420",
    "name": "contributing",
    "type": 0
  },
  "inviter": {
    "id": "306496561301684224",
    "username": "paulmelnikow",
    "global_name": null,
    "display_name": null,
    "avatar": "7667ed8ea335982730a320c36102fc32",
    "discriminator": "9778",
    "public_flags": 0,
    "avatar_decoration": null
  },
  "approximate_member_count": 632,
  "approximate_presence_count": 46
}

https://discord.com/developers/docs/reference
https://discord.com/developers/docs/resources/invite

🎤 Motivation
This would simplify making a Discord badge (no need to enable the widget option on the guild), would make it so the badge is clickable to join the guild, and would result in a shorter badge URL (invite codes are shorter than guild IDs).

Thanks but closing as a duplicate of existing discussion in #4500, #9098, #5079 and others


Enable allow-in-func mode by default in no-did-mount-set-state and no-did-update-set-state rules

There are many people who regularly call this.setState in a callback
in componentDidMount and componentDidUpdate. Although there
was an option for this rule that would allow this to happen, we have
decided to make it the default.
Addresses #686.

Merged, thanks!


Commit generated code to avoid development sluggishness

Instead of running build.rs on every save, this should make development
nicer

/test ?

/test e2e-aws-ovn-single-node-recert-serial


Integrate decodeParams with singleton

Goal
Get argument parsing from decodeParams to work as part of the singleton.
Acceptance Criteria

Arguments are passed when singleton is initiated.
All the arguments currently existing in the testController are parsed.

arguments are passed with the singleton initialisation (so config is also parsed but there's a minor issue with the code when trying to get device details so working on that right now).

added for review; refactoring the code based on comments

merged with develop


Audit triplestore

Functional workflow: Camel picks up JMS events from fedora and updates Fuseki triplestore.

I assume mvn clean install fails for you, @mohideen, as well?

Yes, I have been skipping tests for quick checks. Missed to run the tests after finishing the work. Will update with fixes for the tests.

:+1 this looks good to me & is working for me locally.

Wait, looks like there's a merge conflict in the RouterTest -- can you rebase this?

Sure, I will merge fcrepo-camel-toolkit:master to PR branch.

:+1: now it merges cleanly too – thanks @mohideen


Didn't prompt for Captcha Entry; Bad Slot Timing

I tried using this when my friend managed to book it, but when I tried booking a slot for Mumbai district at 12:30PM, it booked a slot for me with preferred hour of 10-11AM, and the vaccine slot was ending at 1PM.
I thought I would get the opportunity to cancel it if the slot/centre was not ideal, or at least skip the captcha, but it straight up booked a slot without warning, which was going to end the same hour I booked it, and the centre was nearly an hour or two away.

Yeah.. one way to avoid it is to select next day. If you are searching for availability the same day. This can very likely happen.
Try checking with the vaccination center and let them know that you might be delayed.
If you choose not to get vaccinated today, then I do not know how Cowin is going to handle this. Keep me posted either way. @Eihcra

Yeah I guess it was partly my fault. But also now with the lack of a captcha, this deters people from booking slots using a script which would allow us to pick and choose. Unless the script asked for confirmation.
So the next day I went ahead and just cancelled the appointment. I'll just need to reschedule it, it seems. It'd be great to know if you're working on a confirmation layer before booking so people using it would be able to pick a convenient slot, instead of getting something that's at the other end of the city.

@Eihcra I think a confirmation if any should be before starting tracking.
Becuase, if you try to confirm after the availability is detected.
I've added a slight change to text on the track avaialbility and book button. It now says "Track Availability & Auto-Book for selected preferences"


How effective is the use of PromptSRC in Maple, and is it better than IVLP?

Thank you for your excellent work. Could you please share how effective PromptSRC is in Maple and whether it is better than IVLP?Thank you for your time and help.

Hi @IHikari,
Thank you for showing interest in our work!
Regarding your query, we have unfortunately not experimented PromptSRC framework using MaPLe. However I suspect that it should perform well.
The regularization losses in PromptSRC are helpful for all kind of models.
I hope that is clear.
Thank you and kind regards!

Thank you once again for your assistance and kind regards.


child=True is not creating a logger?

What were you trying to accomplish?
Re-use a logger in a sub-module, using the child=True feature, per the documentation.
Expected Behavior

With the following setup, there should be two log entries, one with "service": "foo" and one with "service": "foo.bar".
cat foo.py:
$ cat foo.py
import aws_lambda_powertools
import bar

log = aws_lambda_powertools.Logger(service="foo")

log.info('hallelujah')
bar.bar()

cat bar.py:
import aws_lambda_powertools

log = aws_lambda_powertools.Logger(child=True)

def bar():
    log.info('doublestuff')

Current Behavior

$ python foo.py
{"timestamp": "2020-08-27 11:20:16,245", "level": "INFO", "location": "<module>:6", "service": "foo", "sampling_rate": 0.0, "message": "hallelujah"}

Even setting the service explicitly in the submodule to foo.bar, the service remains foo in the output, though at least now their are two log entries.
cat bar.py:
$ cat bar.py
import aws_lambda_powertools

log = aws_lambda_powertools.Logger(service="foo.bar", child=True)

def bar():
    log.info('doublestuff')

$ python foo.py
{"timestamp": "2020-08-27 11:26:05,330", "level": "INFO", "location": "<module>:6", "service": "foo", "sampling_rate": 0.0, "message": "hallelujah"}
{"timestamp": "2020-08-27 11:26:05,330", "level": "INFO", "location": "bar:6", "service": "foo", "sampling_rate": 0.0, "message": "doublestuff"}

Steps to Reproduce (for bugs)

See above.
Environment

Powertools version used: 1.4.0
Packaging format (Layers, PyPi): PyPi
AWS Lambda function runtime: running locally, not in lambda yet

I would sucpect this is at fault:
handlers = self._logger.parent.handlers if self.child else self._logger.handlers

Hey Loren - Thanks for raising this issue.
If you set service="foo" in your child in the first example it should work.
We need to make the docs clearer here, so please allow me to explain what's going on (I'm on my phone so it'll be a longer response than usual)
Service is what defines what the function is responsible for, or part of (e.g payment service).
For Logger, the service is the logging key customers will use to search log operations for one or more functions, for example - "Search for all error, or messages like X, where service is payment".
That said, customers can set the service using explicit arguments, but typically use as an environment variable since it's used across all utilities and to avoid repetition - POWERTOOLS_SERVICE_NAME
With that hopefully explained, let's get to inheritance.
Python Logging hierarchy happens at the dot notation (service, service.child, service.child_2).
For that to work seamlessly, we name all parent Loggers, no child param, with the value defined in the service param.
For child Loggers we introspect the name of your module where Logger(child=True) is called, and name your Logger as "service.filename".
Once both Loggers are registered, and Python Logging recognises the dot notation, it fixes up the hierarchy and event propagation.
Why the first sample didn't work
Python Logger doesn't recognise as a parent + child loggers because their names are different, since service param value is different.

Parent logger is named "foo"
Child logger is named "service_undefined"

If you were to use the same service name in the Child logger that should work, as that will then be "foo.bar", where bar comes from bar.py
Why the service key isn't foo.bar
That's because of consistency. The service/application is still the same despite having multiple Loggers.
Sorry for the long explanation but hope that clarifies it - Lemme know otherwise :)
Thanks

@heitorlessa Ok, perhaps I'm just not using this library correctly then. All the examples assume service, where python loggers are more typically using the logger name in the record. So I just assumed service === name for the purposes of this library.
I suppose I don't particularly care about the service name for our use case. I do care about specifically where in the module the message was generated. It looks like this module is using the location for that purpose, but that is just bar instead of the logger name (foo.bar).
$ cat bar.py
import aws_lambda_powertools

log = aws_lambda_powertools.Logger(service="foo", child=True)

def bar():
    log.info('doublestuff')

$ python foo.py
{"timestamp": "2020-08-27 12:02:10,669", "level": "INFO", "location": "<module>:6", "service": "foo", "sampling_rate": 0.0, "message": "hallelujah"}
{"timestamp": "2020-08-27 12:02:10,669", "level": "INFO", "location": "bar:10", "service": "foo", "sampling_rate": 0.0, "message": "doublestuff"}

That is correct - location is where funcName:lineNumber will be present, hence bar:lineNo
We didn't include the name parameter for brevity, and for preventing mistakes such as name==name which wouldn't work.

@heitorlessa personally I would prefer if "location" included the module it was benign logged from.
This would be similar to other logging libraries.
DEBUG - [method_name] package_name -  message

Gotcha. Yeah, the function name is definitely not enough info. Is there some way we can set location to something like package.module.function:line?

@michaelbrewer I think it might be easier to have an option to override log record attributes.
When we started, we heard from beta customers the opposite, as the module wasn't always valuable.

Gotcha. Yeah, the function name is definitely not enough info. Is there some way we can set location to something like package.module.function:line?

Maybe we can allow for a custom pattern for this. As in some cases you don't want the location field to be too long.

Maybe we can allow for a custom pattern for this. As in some cases you don't want the location field to be too long.

That would be fine. Coming from python logging, we're using to setting the logformat with our custom preferences.

@michaelbrewer I think it might be easier to have an option to override log record attributes.
When we started, we heard from beta customers the opposite, as the module wasn't always valuable.

Yeah. I would want to suppress sample_rate, service and some other unused fields. Can overload "location" to include more info like the package/module and method.

Maybe a separate RFC on a more tailored logger.

That was mostly it @michaelbrewer as sometimes it'd be either too long or not easily parsed/searched.
Let me ask you this before I go to sleep :)
Apart from location, is there any other log record attribute you might to override? E.g timestamp etc
Native ones under LogRecord attributes
https://docs.python.org/3/library/logging.html

A RFC it is, then - if either of you could create I'd be happy to consider it.
I want to avoid a situation where we add too many flags or break existing customers searching mechanisms (saved queries).
Instead, we should provide an option for Power users to override Log records as they see fit without imposing this to less experienced customers, as Python Logging is a complex beast

@heitorlessa - maybe some presets for location and timestamp. Support for pulling through correlation ids. And ability to suppress / change order of attributes.
But I think the default should please as many people as possible.
Small lambda with a single file should work as well as complex ones with sub modules. Maybe we have a simple flag to switch between the 2.

Overriding the timestamp seems to work fine... My repro case here was simplified, but here is what we're really testing with, derived from our current logging configs...
log = aws_lambda_powertools.Logger(
    service=__name__,
    stream=sys.stderr,
    timestamp="%(asctime)s.%(msecs)03dZ",
    datefmt="%Y-%m-%dT%H:%M:%S",
)

Here's the boilerplate we currently have in all our lambdas, which your library is replacing, and also giving us the benefits of structured logging!
DEFAULT_LOG_LEVEL = logging.INFO
LOG_LEVELS = collections.defaultdict(
    lambda: DEFAULT_LOG_LEVEL,
    {
        "critical": logging.CRITICAL,
        "error": logging.ERROR,
        "warning": logging.WARNING,
        "info": logging.INFO,
        "debug": logging.DEBUG,
    },
)

# Lambda initializes a root logger that needs to be removed in order to set a
# different logging config
root = logging.getLogger()
root.handlers.clear()

logging.basicConfig(
    format="%(asctime)s.%(msecs)03dZ [%(name)s][%(levelname)-5s]: %(message)s",
    datefmt="%Y-%m-%dT%H:%M:%S",
    level=LOG_LEVELS[os.environ.get("LOG_LEVEL", "").lower()],
)
log = logging.getLogger(__name__)

Ok, nice, so overriding location already works:
log = aws_lambda_powertools.Logger(
    service="foo",
    stream=sys.stderr,
    location="%(name)s.%(funcName)s:%(lineno)d",
    timestamp="%(asctime)s.%(msecs)03dZ",
    datefmt="%Y-%m-%dT%H:%M:%S",
)

$ python foo.py
{"timestamp": "2020-08-27T12:54:32.516Z", "level": "INFO", "location": "foo.<module>:14", "service": "foo", "sampling_rate": 0.0, "message": "hallelujah"}
{"timestamp": "2020-08-27T12:54:32.517Z", "level": "INFO", "location": "foo.bar.bar:10", "service": "foo", "sampling_rate": 0.0, "message": "doublestuff"}

Yes, you can override "location", "timestamp", "level", and "datefmt" - I just haven't documented them.
If you're okay with it being this way, then we just need:

RFC for suppressing keys, and ordering them (message first, etc)
Document this

I'll check the responses tomorrow

@lorengordon  @heitorlessa  - looks like documentation updates and  #140 might address this.
# space.py
from aws_lambda_powertools import Logger

logger = Logger(location="%(module)s.%(funcName)s:%(lineno)d", timestamp=None, format_keys=["location", "message"])

def fly():
    logger.info("takeoff")

fly()

$ POWERTOOLS_SERVICE_NAME=falcon python space.py
{"location": "space.fly:7", "message": "takeoff", "level": "INFO", "service": "falcon", "sampling_rate": 0.0}

Agreed, given the ability to override location formatting, documentation updates would be sufficient. Ability to suppress keys would be a nice-to-have, but I don't have a strong opinion there.

We've just merged @michaelbrewer PR on ability to suppress and order a few keys - Working on a PR for the docs piece now

@lorengordon @michaelbrewer could you please take a look and tell me whether this clarifies these and other questions?
https://github.com/awslabs/aws-lambda-powertools-python/pull/148

@heitorlessa looks good to me! thanks!

@heitorlessa  looks good

This has now available in 1.5.0 release


head, tail, segment, and block operations for arrays

It would be nice to be able to slice arrays the same way we can now slice matrices.  Ideally, this would work for arbtrary 1D and 2D arrays (int, real, other arrays, vectors, etc.)
My immediate use case is replacing
  real lp[T];
  lp <- rep_array(log_unif, T);
  for (s in 1:T)
    for (t in 1:T)
      lp[s] <- lp[s] + poisson_log(D[t], if_else(t < s, e, l));

with
  real lp[T];
  for (s in 1:T)
    lp[s] <- log_unif
      + poisson_log(head(D,s-1), e)
      + poisson_log(tail(D,T - s + 1), l);

I'm going to close this to reduce open issues and since this is essentially a non-issue.


Infinite loops in embedded Ruby are unstoppable

An infinite loop in embedded ruby while using middleman serve can not be stopped by killing the Ruby process or the shell running the server. Additionally ^D and ^C have no effect, and my system stalled when attempting to shutdown prompting me to hard kill it.

I've got no idea what a fix might be, unless you know?

Unfortunately the problem probably extends into Rack. I just figured I'd make sure it's documented.

Just tried this, for me pressing Ctrl + Z worked to suspend the process.
Another option is sending SIGKILL from another shell with pkill -9 -f middleman.
It seems like it's just using 100% CPU, maybe with nice middleman it won't stall your system?

I'm doing some issue-gardening 🌿🌷🌾 and came across this issue.
Sorry that this has lingered for quite a while! I'd suggest either closing this issue or submitting a PR with an implementation or at least a failing test.


New verification output

See old comments from the closed PR https://github.com/verus-lang/verus/pull/637.
We changed verification output from verification results:: verified: 7 errors: 0 to verification results:: 7 verified, 0 errors

Please "Squash and merge".


CU-8693gd6c7 CU-86930zxq5 Move to medcat 1.10

Move to medcat==1.10.
This involves moving the install targets (both in requirements as well as in notebooks) as well as moving to a separate config file.
The latter had some changes along with some notes where applicable.

Will abandon - started from the wrong branch


Documentation should be independent of IS4

I believe this documentation has confused many developers into believing ASP.NET Core web APIs can't be secured without OIDC. The documentation should be updated to show how both simple and advanced authentication scenarios should be handled

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

ID: 21cb7455-3305-9316-5440-bdc22947a82d
Version Independent ID: 137d4b94-7b26-3911-b22d-42c754a95fc1
Content: Introduction to authentication for Single Page Apps on ASP.NET Core
Content Source: aspnetcore/security/authentication/identity-api-authorization.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @javiercn
Microsoft Alias: scaddie

And what other authentications methods, other than ODIC/JWT would you suggest?

I certainly don't know all the rules but I believe IS4/OIDC is an overkill for most apps especially if they aren't trying to achieve SSO. The baked in Identity management with Cookies/JWT is useful in many scenarios but I can hardly find a documention on ASP.NET Core with Spas that talk about that.

As the introduction says
ASP.NET Core 3.0 or later offers authentication in Single Page Apps (SPAs) using the support for API authorization. ASP.NET Core Identity for authenticating and storing users is combined with IdentityServer for implementing OpenID Connect.
That could be better clarified to "ASP.NET Core 3.0 templates", which would, at least indicate it's a template choice, and one that was made for ease of use and due to customer demand that we used standards and popular open source projects. While Identity Server is now change their direction, they're still in the templates for now, and talking about what's not in templates may lead to even greater confusion.
If someone were to feel talking about just using cookies, can explain the limitations of that, and the security configuration neccessary then I'm sure we'd take a pull request, and do a bit of renaming to make it clearer that this page is a template choice.

Thanks for this! A lot of noobs are less confused now


[WIP] [CI] llama: add llava v1.6

Hello, I am a code review bot on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!


Single Restriction Enzyme Used for HiC Preparation

Is it possible to pass a single restriction enzyme to SALSA? The HiC library preparation method used by the core facility used only DPNII (GATC).
Cheers!

Just read the docs! Sorry about that! The default is for a single enzyme!

Just read the docs! Sorry about that! The default is for a single enzyme!


spago: 0.15.1 -> 0.15.3

Motivation for this change
Bumps spago to 0.15.3 (and gets building with LTS-16).
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build spago

Looks like it is successfully building at least on x86_64-linux.


Efficient Access Patterns for Databases with Many Collections

Currently the way vecs retrieves collections by name does not seem to scale well when working with a database with many collections (ie 1000+).
By looking at the code this seems that vecs is fetching ALL the collections and then looping through to find one with a matching name. It would be more efficient if there was a where clause in _list_collections in which we could specify the collection by name so that only one would be fetched.
Maybe i'm missing something? Thanks for this great package! 😀
    def get_collection(self, name: str) -> Collection:
        """Get an existing collection"""
        from vecs.collection import Collection

        collections = Collection._list_collections(self)
        for collection in collections:
            if collection.name == name:
                return collection
        raise CollectionNotFound("No collection found with requested name")

        # ...

        @classmethod
    def _list_collections(cls, client: "Client") -> List["Collection"]:
        query = text(
            """
        select
            relname as table_name,
            atttypmod as embedding_dim
        from
            pg_class pc
            join pg_attribute pa
                on pc.oid = pa.attrelid
        where
            pc.relnamespace = 'vecs'::regnamespace
            and pc.relkind = 'r'
            and pa.attname = 'vec'
            and not pc.relname ^@ '_'
        """
        )
        xc = []
        with client.Session() as sess:
            for name, dimension in sess.execute(query):
                existing_collection = cls(name, dimension, client)
                xc.append(existing_collection)
        return xc

yep I agree, that would be a nice performance tweak for users who need lots of collections.
marked as good first issue if you're interested in contributing!

resolved by #18


Fix inconsistent/missing/incorrect prop types and props through the app

Description

There's currently a lot of prop type inconsistencies, namely:
Person drawer asks for an onEdit prop of boolean type, this is wrong
Person page missing first-met prop
EncounterCardSummary asks for a dateMet as a string, this is wrong
EncounterCard's prop needs to be updated to encounterDetails
Acceptance Criteria

[ ] Person drawer onEdit has correct prop type
[ ] Person page has firstMet prop
[ ] EncounteCardSummary has a dateMet Date prop
[ ] EncounterCard's prop updated to encounterDetails

Implementation details

n/a
Testing notes

n/a

lgtm


call checktime after write to warn about changed files

This was first suggested in #43 and was forgotten after. This simply adds the call to the end of suda#Write().

I'm not familiar with checktime so I'm not sure but is calling checktime AFTER sudo#write() make sense? I thought it should be called BEFORE to prevent unwilling overwrite.

checktime checks if the file has changed outside vim (we just changed it with dd), and if it has, it asks the user what to do about it (ie, load the new file, or ignore)

Got it. Thanks 👍


Unable to do some S3 download when using CloudFlare R2

Description
Some elements no longer work when using Cloudflare R2. For example, organization icons no longer load, and we can no longer download repository archives. However, LFS storage works very well, and we can download LFS files correctly. Apparently, this is due to headers that aren't being processed.
Gitea Version
1.22.0
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
https://gist.github.com/l0u1sg/35b98653025f2cc975656ad29e4c2015
Screenshots

Git Version
No response
Operating System
No response
How are you running Gitea?
Gitea is running on Docker behind a reverse proxy
Database
SQLite

Please add MINIO_CHECKSUM_ALGORITHM = md5 to your configuration file.

See:

https://github.com/go-gitea/gitea/pull/23768

Oh thanks, i didn't see this option in the gitea documentation. It solve my issue.


[TESTS][SQL] Setup testdata at the beginning for tests to run independently

What changes were proposed in this pull request?
In CachedTableSuite, we are not setting up the test data at the beginning. Some tests fail while trying to run individually. When running the entire suite they run fine.
Here are some of the tests that fail -

test("SELECT star from cached table")
test("Self-join cached")

As part of this simplified a couple of tests by calling a support method to count the number of
InMemoryRelations.
How was this patch tested?
Ran the failing tests individually.

Test build #71919 has started for PR 16688 at commit b71120d.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71919/
Test FAILed.

retest this please

cc @cloud-fan @gatorsmile Could you please trigger a test for this. Not sure why the last run didn't succeed. Thanks a lot.

ok to test

ok to test

Test build #71970 has started for PR 16688 at commit b71120d.

Test build #71970 has finished for PR 16688 at commit b71120d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71970/
Test PASSed.

Merged build finished. Test PASSed.

LGTM

@dilipbiswal Since this PR is to fix the minor test issues, could you do me a favor to fix three comments?
Replace _global_temp by global_temp

https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/catalog/Catalog.scala#L383-L384
https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala#L2591-L2592

https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala#L50-L51
Thanks

@gatorsmile Sure Sean. Will do

Test build #72001 has started for PR 16688 at commit 8f9914d.

Test build #72001 has finished for PR 16688 at commit 8f9914d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72001/
Test PASSed.

Merged build finished. Test PASSed.

LGTM

Thanks! Merging to master.

@gatorsmile @srowen @viirya Thank you very much !!


monitor server load and cpu/ram/disk usage

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
New Monitor
🔖 Feature description
it will be amazing if we can monitor server load and resource with kuma! :)
✔ Solution
monitor disk cpu ram server load
❓ Alternatives
No response
📝 Additional Context
No response

This would be great.

please do it requesting

That would be great.

This is the only reason I haven't switched yet

This is the only reason I haven't switched yet

What is the reason?

@tcpzix
Is this a duplicate of #819?

yes you right this is a duplicate of #819 and i'm sorry about that
thanks
On Wed, May 24, 2023 at 4:06 AM Frank Elsinga @.***>
wrote:

@tcpzix https://github.com/tcpzix
Is this a duplicate of #819
https://github.com/louislam/uptime-kuma/issues/819?
—
Reply to this email directly, view it on GitHub
https://github.com/louislam/uptime-kuma/issues/2921#issuecomment-1560296773,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKL2FTOYH4BYZ2HHHF3Q6TTXHVJ2VANCNFSM6AAAAAAVYAP7L4
.
You are receiving this because you were mentioned.Message ID:
@.***>

@tcpzix could you close this issue then? ^^

Yes, it's done.
On Mon, May 29, 2023 at 3:24 PM Frank Elsinga @.***>
wrote:

@tcpzix https://github.com/tcpzix could you close this issue then? ^^
—
Reply to this email directly, view it on GitHub
https://github.com/louislam/uptime-kuma/issues/2921#issuecomment-1567038100,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKL2FTNNACTFCUOZBEE5Q7TXISE5XANCNFSM6AAAAAAVYAP7L4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Clue Scroll: Opening world map with a location arrow (Emote, Coordinate) crashes client

Noticed during an anagram clue, tried with a coordinate and emote and all failed. A task one however (pickpocket an elf) did not and opening with no clue did not either. Stack trace of error:

Current git head: 248ba6a

You incorrectly built the client.


Can't set Max Download Time value in OSM Configuration

Hello,
I used the last develop version with docker, everything work but when I want to change the Max Download Time value in OSM Configuration panel, the default value(60 minutes) is always used, whatever value I set in the OSM Configuration panel. I can see my changes in the panel, but it has no impact on the Max Download Time which still stays at 60 minutes. Is it normal?
Thanks a lot !

Hi,
I have been able to reconstruct the problem and find out what caused it. The settings in the OSM Configuration window were indeed set correctly. However, the maximum download time of a job was not saved correctly according to this setting, but was instead set to 60 minutes by default. I am currently working on this problem and a fix should be available soon. Thank you for opening an issue!
Also note that this default value only works for newly created jobs and does not update the maximum download time of an existing job.
Best Regards!

I'm sorry that the topic has been somewhat missed, I do hope you've already noticed that the bugfix has been merged for some time. The issue has been addressed in PR 168.
I'm therefore closing this issue now. Better late than never.
Best regards!


Migrate examples to flit_core as well?

textX is now using flit_core as the primary build system. However, the test projects (tests/functional/subcommands/example_project and tests/functional/registration/projects/*) all still implicitly use setuptools. Could you please consider migrating them as well to use a single build system throughout the project?

We should definitely do that. Thanks for reporting.

You can test this branch

Thank you. I can confirm that it works, and is much faster too ;-).

Great. I'll release this update shortly.


[SEV-1][BUG]Incorrect rendering of the site on Fedora-Firefox

🐞 Problem
The webiste is not rendering properly on Firefox.
Configuration:
Firefox: 57.0.4
OS: Fedora 27

The same site is rendering properly when run from my system via Google Chrome:
Configuration:
Chrome: Version 63.0.3239.132

Labels: bug, SEV-1 High Priority

I took this issue and opened a pull request #2720


some change in SublimeLinter-contrib-stylelint

I have some problem when using the SublimeLinter-contrib-stylelint,so i made some changes to fix them,but i am not sure the change i made is right .
The one  thing i am sure is that, now they are work well in my project.
So,  if there are some other  better solutions please let me know~~

@kungfusheep I think these updates would solve a lot of issues that are keeping me from using this — working on scss syntax files, and error position.

Hi - thanks for this, I've added support for sass/scss files just now. I've left the original formatters the same as they were as I'm pretty sure there used to be issues with line numbers in older versions of stylelint. I have, however, made the wrapper now use the stylelint CLI (if available) rather than invoking it via the JS API. So things like line numbers/positions should work exactly as stylelint reports them. Cheers.


UnmodifiableListView I can not remove the CalendarEventData from EventList

for the old version, I can  delete, because when controller.events it return List so I can delete all of the events data.

@Chhay67  The goal here is to ensure that any addition or removal of events is done using the controller to properly reflect changes in the UI.
Previously, to achieve this, we used the toList method in the events getter to create and return a copy of the events list. This way, any changes to the returned list would not affect the data in the controller or the UI. However, this approach can cause performance issues if the events list is large and used frequently, as it iterates through all the events to return a new list. For those who do not intend to manipulate the list but merely want to retrieve or filter specific events, this degrades performance.
So, we suggest, If you need to manipulate the list, you can always use the toList method to create a new list and then make the necessary changes.


refactor(ingest/unity): Use databricks-sdk over databricks-cli for usage query

Introduces https://github.com/databricks/databricks-sdk-py, which seems like a more developed library for the databricks API. Using it should provide better error logging around the usage query.
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Hold up on review, this has a bug because again the databricks library doesn't pass params correctly

Should be good to review now


how do i add jquery in theme app extension

importing jquery in liquid file like this  ,have copied the min js jquery in assets with filename-jquery.min.js-
doing this raise following error in console  ,while i do shopify extension push -
✗ JavaScript on every page load exceeds compressed size threshold (10000 Bytes), consider using the import on interaction pattern. at blocks/review_carousel.liquid
is there a way anyone knows on how to add jquery, i need it for carousel

https://github.com/Shopify/theme-check/blob/main/docs/checks/remote_asset.md
I don't know why they don't explain their codes in example


Bump vulnerable dependencies

Bump vulnerable dependencies
Description
Changes proposed in this pull request:

sanitize-html 2.5.1 -> 2.7.3 (address CVE-2022-25887)

/test pull-compass-console-build

/test pull-compass-console-build

/test pull-compass-console-build

/test pull-compass-console-build

/test pull-compass-console-build

/test pull-compass-console-build

/retest


refactor(WEG-119): Replace header image of home page

This PR changes the generic buildings image in the landing page to an image with multiple thematic inspirational photographs.

:tada: This PR is included in version 1.3.0-rc.9 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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


Looking for maintainers

Hello guys, I am looking for maintainers for this package. Is someone interested ?

@FaridSafi Intreseted

Hey Farid, would love to help maintain

Hi @FaridSafi
I am interested but don't have much experience in react-native as of now.
I recently came across this package, its quite helpful for the react-native app I have started building recently.
Also I would like request adding inbuilt support for sessionToken to reduce the cost for autocompletion https://github.com/FaridSafi/react-native-google-places-autocomplete/issues/534
If this looks good, I can start working on this.

alright, I sent you an invitation via GitHub. What is your https://www.npmjs.com/ username?

alright, thank you! I sent you an invitation via GitHub. What is your https://www.npmjs.com/ username?

Its manishy

Thank you @y-manish, I added you as collaborator on this repo and also on npmjs. You are now able publish updates

I would love to work with @y-manish and whoever else is working on the library.
I have have been working on react-native full time for over a year and been using this library from the beginning.  Was doing React on the web before that for a couple of years.
I would like to help out with critical bug fixes and issue management first (there are over 100 open issues and 30 open PR's).
I can also help with refactoring the code to use some of the newer react/react-native features.
My NPM username is sbell.
Thank you @FaridSafi for creating the library and seeing it through until now.

I am interested for maintaining this awesome package, I always wanted to do something like this and it would be a nice challenge for me.


Double backpress to exit.

In most of the apps if the user presses back button to exit, a toast message appears which prompts the user to press back button again to exit. I want to implement the same in the app.
@kunall17 @timabbott  can i work on this?

Sure!


No commit data shown on napari hub for hidden plugins

Problem:
Currently, there are plugins we are not showing commit data even though we have Snowflake data for them. The reason for this behavior is as follows:
Since we only have access to repo values in latest_commit and commit_activity SQL queries, we needed to created repo to plugin mappings so that latest_commit.json and commit_activity.json can store data with respect to plugins instead of repos.
However, such mappings are only available for public plugins, or in other words, plugins that exist in index.json, and this is the reason why hidden plugins (such as napari-threedee), which do not exist in index.json, did not have commit data.
References: Please check out https://www.napari-hub.org/plugins/napari-threedee and click on the Activity tab and view the Maintenance section, and see the commits at https://github.com/napari-threedee/napari-threedee.

cc @neuromusic - wanted you to be aware of this issue, but my feeling is it's not critical to address given it only impacts hidden plugins.

Looks like this has been fixed as part of the migration work. Closing


[Bug] Table widget breaks on drag and drop

Steps to reproduce:

Navigate to Datasource
Create a Query
Click on Add widget/Generate widget
Now click on Widget
Add a table widget
and observe

Observation:
It is observed that the table breaks on simple drag and drop after the widget are binded to a Query
Expectation:
It is expected that the table works on simple drag and drop
Video:
https://www.loom.com/share/5aca6f0979094fc6867d93ad4e4ca023

This is already been taken care in #4376. Closing this


Multi-head attention module throwing RuntimeError: view size is not compatible with input tensor's size and stride

🐛 Bug
Multi-head attention module throwing error: "RuntimeError: view size is not compatible with input tensor's size and stride (at least one dimension spans across two contiguous subspaces). Use .reshape(...) instead."
To Reproduce

Train a model using a TPU machine

python train.py
my/redacted/data/dir/
--arch transformer_iwslt_de_en
--task translation
--source-lang zh
--target-lang en
--lr 0.0005
--weight-decay 0.0001
--optimizer adam
--adam-betas '(0.9, 0.98)'
--lr-scheduler inverse_sqrt
--min-lr 1e-09
--warmup-updates 4000
--warmup-init-lr 1e-07
--label-smoothing 0.1
--clip-norm 5.0
--criterion label_smoothed_cross_entropy
--dropout 0.1
--max-tokens 48000
--save-dir data/mt/output/checkpoints
--validate-interval 1
--tensorboard-logdir output/tensorboard_logs
--no-progress-bar
--log-interval 100
--max-source-positions 1024
--max-target-positions 1024
--skip-invalid-size-inputs-valid-test
--no-epoch-checkpoints
--max-epoch 20
--distributed-world-size 8
--tpu
--num-batch-buckets 5

Generate results (reproduced on both TPU, GPU, and CPU)

python /anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq_cli/generate.py
my/redacted/data/dir/
--task translation
--path my/redacted/model/dir/checkpoints/checkpoint_best.pt
--beam 5
--remove-bpe
--skip-invalid-size-inputs-valid-test
--scorer sacrebleu
--source-lang zh
--target-lang en

See error

On GPU...
2020-09-09 20:42:57 | WARNING | fairseq.data.data_utils | 62 samples have invalid sizes and will be skipped, max_positions=(1024, 1024), first few sample ids=[107311, 69863, 57884, 54207, 83884, 71724, 46, 47217, 64581, 106659]
Traceback (most recent call last):
  File "/opt/conda/bin/fairseq-generate", line 8, in <module>
    sys.exit(cli_main())
  File "/opt/conda/lib/python3.7/site-packages/fairseq_cli/generate.py", line 274, in cli_main
    main(args)
  File "/opt/conda/lib/python3.7/site-packages/fairseq_cli/generate.py", line 38, in main
    return _main(args, sys.stdout)
  File "/opt/conda/lib/python3.7/site-packages/fairseq_cli/generate.py", line 150, in _main
    hypos = task.inference_step(generator, models, sample, prefix_tokens)
  File "/opt/conda/lib/python3.7/site-packages/fairseq/tasks/fairseq_task.py", line 361, in inference_step
    return generator.generate(models, sample, prefix_tokens=prefix_tokens)
  File "/opt/conda/lib/python3.7/site-packages/torch/autograd/grad_mode.py", line 15, in decorate_context
    return func(*args, **kwargs)
  File "/opt/conda/lib/python3.7/site-packages/fairseq/sequence_generator.py", line 159, in generate
    return self._generate(sample, **kwargs)
  File "/opt/conda/lib/python3.7/site-packages/fairseq/sequence_generator.py", line 198, in _generate
    encoder_outs = self.model.forward_encoder(net_input)
  File "/opt/conda/lib/python3.7/site-packages/fairseq/sequence_generator.py", line 697, in forward_encoder
    for model in self.models
  File "/opt/conda/lib/python3.7/site-packages/fairseq/sequence_generator.py", line 697, in <listcomp>
    for model in self.models
  File "/opt/conda/lib/python3.7/site-packages/fairseq/models/fairseq_encoder.py", line 53, in forward_torchscript
    return self.forward_non_torchscript(net_input)
  File "/opt/conda/lib/python3.7/site-packages/fairseq/models/fairseq_encoder.py", line 62, in forward_non_torchscript
    return self.forward(**encoder_input)
  File "/opt/conda/lib/python3.7/site-packages/fairseq/models/transformer.py", line 411, in forward
    x = layer(x, encoder_padding_mask)
  File "/opt/conda/lib/python3.7/site-packages/torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File "/opt/conda/lib/python3.7/site-packages/fairseq/modules/transformer_layer.py", line 122, in forward
    attn_mask=attn_mask,
  File "/opt/conda/lib/python3.7/site-packages/torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File "/opt/conda/lib/python3.7/site-packages/fairseq/modules/multihead_attention.py", line 342, in forward
    attn_weights = attn_weights.view(bsz * self.num_heads, tgt_len, src_len)
RuntimeError: view size is not compatible with input tensor's size and stride (at least one dimension spans across two contiguous subspaces). Use .reshape(...) instead.

Reproduced on TPU...
2020-09-09 20:38:46 | INFO | fairseq_cli.generate | loading model(s) from redacted/path/checkpoints/checkpoint_best.pt
2020-09-09 20:38:52 | WARNING | fairseq.data.data_utils | 62 samples have invalid sizes and will be skipped, max_positions=(1024, 1024), first few sample ids=[107311, 69863, 57884, 54207, 83884, 71724, 46, 47217, 64581, 106659]
Traceback (most recent call last):
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq_cli/generate.py", line 278, in <module>
    cli_main()
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq_cli/generate.py", line 274, in cli_main
    main(args)
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq_cli/generate.py", line 38, in main
    return _main(args, sys.stdout)
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq_cli/generate.py", line 150, in _main
    hypos = task.inference_step(generator, models, sample, prefix_tokens)
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq/tasks/fairseq_task.py", line 361, in inference_step
    return generator.generate(models, sample, prefix_tokens=prefix_tokens)
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/torch/autograd/grad_mode.py", line 15, in decorate_context
    return func(*args, **kwargs)
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq/sequence_generator.py", line 159, in generate
    return self._generate(sample, **kwargs)
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq/sequence_generator.py", line 198, in _generate
    encoder_outs = self.model.forward_encoder(net_input)
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq/sequence_generator.py", line 697, in forward_encoder
    for model in self.models
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq/sequence_generator.py", line 697, in <listcomp>
    for model in self.models
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq/models/fairseq_encoder.py", line 53, in forward_torchscript
    return self.forward_non_torchscript(net_input)
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq/models/fairseq_encoder.py", line 62, in forward_non_torchscript
    return self.forward(**encoder_input)
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq/models/transformer.py", line 411, in forward
    x = layer(x, encoder_padding_mask)
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq/modules/transformer_layer.py", line 122, in forward
    attn_mask=attn_mask,
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/anaconda3/envs/torch-xla-1.6/lib/python3.6/site-packages/fairseq/modules/multihead_attention.py", line 342, in forward
    attn_weights = attn_weights.view(bsz * self.num_heads, tgt_len, src_len)
RuntimeError: view size is not compatible with input tensor's size and stride (at least one dimension spans across two contiguous subspaces). Use .reshape(...) instead.

NOTE: the TPU vs GPU machine is probably a red herring and the real difference is different fairseq or pytorch versions.
Code sample
See fairseq CLI commands above.
Expected behavior
No error, outputs fairseq-generate results.
Environment
Environment for running fairseq-train:

fairseq Version (e.g., 1.0 or master): c1e734b2dd7024044c8dee551620146e4f872ad4 and latest master
PyTorch Version (e.g., 1.0): 1.6.0
OS (e.g., Linux): Debian GNU/Linux 9 \n \l
How you installed fairseq (pip, source): pip
Build command you used (if compiling from source): NA
Python version: Python 3.6.10 :: Anaconda custom (64-bit) (because I'm using the default env for TPU training - conda activate torch-xla-1.6)
CUDA/cuDNN version: NA
GPU models and configuration: TPU v3.8 running TPU software v2.3
Any other relevant information: GCP image debian-9-torch-xla-v20200828

Environment for running fairseq-generate:

fairseq Version (e.g., 1.0 or master): c1e734b2dd7024044c8dee551620146e4f872ad4 and latest master
PyTorch Version (e.g., 1.0) - 1.4.0 (also tried 1.5.1 and 1.6.0)
OS (e.g., Linux): Debian GNU/Linux 9 \n \l
How you installed fairseq: pip
Build command you used (if compiling from source): NA
Python version: Python 3.7.8
CUDA/cuDNN version: 10.1
GPU models and configuration: 1 x NVIDIA Tesla V100
Any other relevant information: GCP image c2-deeplearning-pytorch-1-4-cu101-v20200804-debian-9

Additional context
The error is pretty clear so I replaced line 342 of the Multihead Attention module with
attn_weights = attn_weights.reshape(bsz * self.num_heads, tgt_len, src_len)

and the issue was resolved. This is a potential solution but I understand that reshape() is more expensive than view() so I'll leave it to the experts to decide.

CC @myleott


404 on Windows 11

Tried to install both x32 and x64 package. Both give me 404 errors. It runs fine and listens on port 80 (changed to 8090 as well). Disabled firewall but http(s)://ip:port gives me a 404.
any thoughts?

Please change loglevel to debug and sent the results.

Have you set debug to level 4 ?
YTuner v1.0.2 Copyright (c) 2023 Greg P. (https://github.com/coffeegreg)
11-12-23 18:07:17 : Inf : Starting services...
11-12-23 18:07:17 : Inf : Loading RB UUIDs cache file.
11-12-23 18:07:18 : Inf : Successfully loaded 43424 RB UUIDs from cache file.
11-12-23 18:07:18 : Inf : Successfully loaded 629 my stations.
11-12-23 18:07:18 : Inf : DNS server listening on: <IP_ADDRESS>:53.
11-12-23 18:07:18 : Inf : Web server listening on: 10.10-100.240:80.
11-12-23 18:07:18 : Inf : AVRs HTTP requests IP target:
It should like like that:

did not see port 53 enabled on your machine
Are you sure using port 8090 web port for your receiver ?
Most of the radios cannot change the default port 80.
Please set your static dns on the receiver to your host and port 80 (default)
I think you're goin failed with your setup.
When using different dns server, you need use dnsmasq the dns entries.
I recommend using static settings on your receiver address all to your host where ytuner is running on.

that Ycast was an old browser url.
I defaulted to port 80 and all works now. Mystation works as well.

Beatrice, you are the best. :clap:
Thank you for your support to other YTuner users. :thumbsup:
Sorry, I was away for a long time. In the meantime, I try to find an interesting job, but out of luck so far. Still trying...
Nevertheless, I found some time to prepare a new version of YTuner (v1.1.0).
Maybe today or in the next few days there will be a new version of YTuner. I'm testing it right now. Perhaps this new version of YTuner will appear today or in a few days. I added filtering and sorting as you suggested and most important ... I added caching. Now YTuner works ultra fast :rocket: without unnecessary waiting for every response from radio-browser.info and YTuner executable file is still below 2 MB.
I also added a few more improvements, functionalities and a few fixes... hang on ... :smile:

@Larsvb0 :
May I add your Denon AVR-X3200W to "Confirmed working" list ?

Greg. What kind of job are you interested in ?
Maybe I can help.

Ho.. ho.. ho... Here is Santa Claus. :santa:
I heard that Beatrice is a very good and kind girl and I have the latest release of YTuner for her. :gift:
For all other girls and boys too... even less polite than Beatrice. :smile:
I'm getting out of here because the elves are messing with my sleigh. Ho.. ho.. ho.. :santa:

Good morning Greg,
Application improvements sounds good 👍
But, application in linux x64/86 is actually Version 1.0.2
Please check.
And one question. Is version 1.1.0 looking into the config subdirectory to get settings ?
I will take a look into arm64 binary today and will come back if I see similar.
🙋♀ ho ho
Have a nice day

Good morning Bea,
Yes, You're right. Yesterday I spent almost the whole day making changes in YTuner and I was a little bit tired at the end of the day and... I 've got the wrong directory with the versions to prepare release. :wink:
Fixed now :+1: Thanks for your vigilance.
Yes, I decided to move all configuration files to the config subdirectory. The Stations.ini/yaml files and optional bookmark files should be automatically moved to config when YTuner starts.
Please test the arm/aarch64 platform if you can and let me know what you think about the speed of YTuner in the new release :smile:
Einen schönen Tag noch. :ok_hand:

I think I can close this issue.


⚠ Bofrost.at has degraded performance

In 75bedb7, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2110 ms

Resolved: Bofrost.at performance has improved in 3c7dc74 after 16 minutes.


IGAPP-194: Update Version of iOS in E2E test

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.
See discussion here: https://chat.tuerantuer.org/digitalfabrik/pl/e7nedbz6g3869m376e9n3t86tr
I also tried the version 1.18.0 and 1.19.1 but both seemed worse performance wise, as they both seemed to depend on some cached files on the device. Somthing that somehow still throws me off is this build where an unexpected timeout occurs. Besides that the native build failed I cannot see any reason why there would be a timout :/. Maybe one of you has an idea.

The build mentioned in the PR description had the old appium version though, right? So maybe this would not have been happened with these changes?

The build mentioned in the PR description had the old appium version though, right? So maybe this would not have been happened with these changes?

The build mentioned in the PR description had the old appium version though, right? So maybe this would not have been happened with these changes?

Yep. But this change mainly focuses on fixing the error that sometimes occured. The last PR, which added waitingForQuiescence, should have fixed timeouts. But as it already got a lot better and the mentioned build also had errors. That may have impacted the tests. So maybe it's still best to wait and see here, whether another timeout occurs.

The build mentioned in the PR description had the old appium version though, right? So maybe this would not have been happened with these changes?

Yep. But this change mainly focuses on fixing the error that sometimes occured. The last PR, which added waitingForQuiescence, should have fixed timeouts. But as it already got a lot better and the mentioned build also had errors. That may have impacted the tests. So maybe it's still best to wait and see here, whether another timeout occurs.


Add ESLint Rule to github-actions Plugin

Adds the no-top-level-material-ui-4-import ESLint rule to the github-actions plugin to aid with the migration to Material UI v5.
Issue: https://github.com/backstage/backstage/issues/23467

still gotta find the right import for that Color


Typos and add run-dev-server.sh script to launch development env.

fixes some typos in README.md file.
Add run-dev-server.sh script to launch development env.

I think this is too old now

I think this is too old now


Change order of slides and add sections to slides

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
There were some adjustments made to the order of topics and also certain content has been removed. However, some of the modifications have left the content a bit disjointed. For instance, in the slides titled "02_reality_control_flow_iterables," there are examples involving functions (which aren't introduced until slide 07_functions.ipynb) and string formatting (which has been removed altogether). As another example, Assignment #1 involved the use of string methods, but this content was completely removed from the slides as well. Inconsistencies like these cropped up in multiple places so I’m submitted this pull request to reorganize the slides and reinstate some of the missing content.
What did you learn from the changes you have made?
N/A
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
N/A
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
N/A
How were these changes tested?
N/A
A reference to a related issue in your repository (if applicable)
N/A
@mentions of the person or team responsible for reviewing proposed changes (At least 2 people)
@RohanAlexander @curtissfnorman @danielrazavi
Checklist

[ Y] I can confirm that my changes are working as intended

@danielrazavi all your comments have been addressed.


Duplicate tags on Select with multiple

Steps to reproduce the issue

go to https://select2.github.io/examples.html
scroll down to Tagging support example
type in red, yes with a space after.

Expected behavior and actual behavior
When I follow those steps, I see 2 red tags.
I was expecting only 1.
Environment
Browsers

[x] Google Chrome
[x] Mozilla Firefox

Operating System

[x] Mac OS X

Isolating the problem

[x] This bug happens on the examples page
[x] The bug happens consistently across all tested browsers

As of 4.0.6-rc.1, I cannot seem to replicate this issue anymore.


Enable anonymous streaming

The purpose of this PR is to enable anonymous (unauthenticated) streaming.
This curl command demonstrates that streaming works (at least on some instances) without a need for authentication.
curl https://mastodon.social/api/v1/streaming/public

With v0.0.4, we assume that an AccessToken is set and use it to set an Authorization header in the request. With an empty AccessToken, this leads to an authorization error.
Omitting the entire header in that case fixes the problem. The following programm illustrates it.
package main

import (
	"context"
	"log"

	mastodon "github.com/mattn/go-mastodon"
)

const serverURL = "https://mastodon.social"

func main() {
	ctx := context.Background()

	client := mastodon.NewClient(&mastodon.Config{
		Server: serverURL,
	})

	q, err := client.StreamingPublic(ctx, false)
	if err != nil {
		log.Fatalf("StreamingPublic failed: %v", err)
	}

	for e := range q {
		switch event := e.(type) {
		case *mastodon.UpdateEvent:
			log.Printf("UpdateEvent: account %s", event.Status.Account.URL)
		case *mastodon.NotificationEvent:
			log.Printf("NotificationEvent: type %#v", event.Notification.Type)
		case *mastodon.ErrorEvent:
			log.Printf("ErrorEvent: %#v", event)
		}
	}
}

@mattn Any opinion on this one?

Sorry my delay. LGTM

Sorry my delay. LGTM

Thank you


Replace GPT3Tokenizer (p50k_base) with Microsoft.DeepDev.Tokenizer (cl100k_base)

Motivation and Context (Why the change? What's the scenario?)
The existing tokenization logic is for the legacy GPT-3 and Codex models. The implementation should be updated to support the newer models (GPT-3.5, GPT-4, etc) for more precise token counts.
High level description (Approach, Design)
Remove the existing tokenization logic. Add reference to Microsoft.DeepDev.Tokenizer and embed cl100k_base.tiktoken in the assembly. Update code references as necessary.

Might need a re-do, because we've made several changes. The solution now supports multiple tokenizers, allowing to choose the right one when selecting a model. I would leave the existing tokenizer, and simply add the new one as another option, then users can pick up the right one.

Closing due to how stale this branch has gotten.


New Release

The last version on RubyGems does not include the fix for the deprecation warning on require statements with uppercase letters. This means that the current version on RubyGems does not work in newer ruby/rails environments.
Creating a new release on RubyGems would fix this issue.

Sorry, took a long time for me to return to it. But less than a year, right?.. 0.0.5 just released.


[Exposure Request] Codeunit 30195 Shopify Inventory API

Hi, in the existing Shopify inventory API codeunit, we use ItemAvailMgt to calc available quantities. This factors in all planned purchase and sales orders when determining the available quantity to show on Shopify for a given date.
We would like the functionality for different inventory calculations to be used. It would be good to add a setup field on the Shopify Shop table called 'Inventory Calculation Method' with the following options:

Available Inventory - follows the existing logic
Location Inventory - only considers inventory in the relevant location as per item ledger entry
Location inventory + BOM components- considers inventory in the relevant location, but also adds inventory available to assemble as per the item's BOM

internal procedure GetStock(ShopInventory: Record "Shpfy Shop Inventory") Stock: Decimal
...
ItemAvailMgt.CalcAvailQuantities(Item, true, GrossRequirement, PlannedOrderRcpt, ScheduledRcpt, PlannedOrderReleases, ProjAvailableBalance, ExpectedInventory, QtyAvailable);
Stock := ProjAvailableBalance;

@Vert1 we are planning to enable extensibility for Shopify Connector by next major. If you need extensibility for this specific scenario sooner and ready to submit PR yourselves, please reach out to<EMAIL_ADDRESS>and we can discuss further.

Allow extensibility of the new Shopify Connector Extension

Shopify connector becomes extensible: Stock calculation
Shopify Connector has been non-extensible, but we're changing that. We're offering a few points of extensibility. We'll keep the number of points to a minimum so that we can follow the rapid development on the Shopify side without introducing breaking changes. We are opening for extensibility for specific scenarios, based on feedback from our partners and customers starting with Stock calculation. For more details, please follow this repository on GitHub. Note that even though extensibility is limited, you can submit improvements directly.
Here are examples of different calculations: ALAppExtensions/extensibility_examples.md at main · microsoft/ALAppExtensions (github.com). You can implement your formula as well.


Fix temporal image mosaic support

There appears to be a regression in the image mosaic support for time based layers. This can be tested against any GeoServer layer that is backed by a raster with the time enabled in the Dimensions tab.

Resolved by pull request #15


feat(jsx): add jsx-dev-runtime

When setting jsxImportSource to @rise-tools/react, certain bundlers may choose to use dev runtime (while in development).
We must export jsx-dev-runtime with jsxDev and jsxsDev. We don't have any dev-specific behavior at this point, hence simple re-export.

Oh LOL, definitely not. Should not be here at all! Initially, I wanted to support environments that didn't support package exports field (React Native right now), but I think it's minimal, I will skip it.
We will document how to set it for React Native soon in a follow-up.


Graphical Error with List<SerializableDictionary<>>

Here is my code:
[SerializeField] List<SerializableDictionary<ResourceEnum, int>> buildingUpgradeTiers = new List<SerializableDictionary<ResourceEnum, int>>();
Here is what it produces in the Editor:

The weird thing is that only the first element of the List has this problem; the others show up fine:

Solved:
Using Unity 2022.1.2f1 solves the issue. I was using 2021.3.4f1 previously.


MIDI - Clock messages may interfere with running status implementation

makingsoundmachines on Slack reported that when using the TR-8,  note messages are getting lost, between clock messages.

. . .the Roland TR8 sends F8 (so clock tick) messages in the middle of these; e.g. 90 24 FF .. then 24 00 F8 .. so the status hasn't changed but there are clock ticks somewhere in between.

I have a feeling our implementation interprets clock messages as status bytes, and are potentially clearing the running status byte when it should persist.
Worth looking into.

This has been resolved as of the latest release, and a unit test was added to specifically check for this.


BGP switch side config example

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks @ptrivedi . I'll take a look.

@ptrivedi: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

eks-anywhere-cluster-controller-tooling-presubmit
eac5c00442880b9d799628b8e2453ea066f3dc5d
link
/test eks-anywhere-cluster-controller-tooling-presubmit

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

/test eks-anywhere-cluster-controller-tooling-presubmit

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ptrivedi
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

docs/OWNERS [ptrivedi]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ptrivedi, TerryHowe
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

docs/OWNERS [TerryHowe,ptrivedi]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


CMD+S Randomly Stops Working

Issue Type: Bug
I don't know how to reproduce this because it happens randomly. I'll just be going about my day, using CMD+S to save my files. Randomly, I'll notice that there are still grey dots next to file names, because the file hasn't been saved in a while. CMD+S stopped working. Running it through the command palette is fine.
I remove the mapping for the save action to CMD+S and restart VSCode, and go to remap the action again to CMD+S. VSCode refuses to type the letter "S" into the keybindings window. I have no idea why, but it happens.

Here's the kicker; other keyboard shortcuts (like Save As) that include CMD+S+ work fine. It's driving me insane.
A full restart of my machine resolves it. This happened today with the two options specified via the command line (for monitor flicker issues), but it happens with normal starts of VSCode too.
 Just in case, anything in the Developer tools log? (Toggle Developer Tools in through the command palette). Do you happen to know if you had the same issue on 1.25 before you installed the latest High Sierra update?

@kieferrm Next time this happens I'll take a peek at the developer logs; keep forgetting that this is an Electron app and I can do that 😅 It typically happens about once a week, so I'll open that up next time it happens and take a peek.

Do you happen to know if you had the same issue on 1.25 before you installed the latest High Sierra update?

It's been happening occasionally for a few months, so definitely before VSCode 1.25; typically when it was the least convenient and when I didn't really have time to write a decent bug report. The only OS this machine has known has been High Sierra, so I can't speak to whether or not this occurs in versions before High Sierra.

Nope, hasn't been a problem for a good long while. I'll re-open if it happens again.


Spring cloud contract stub runner - no way to customize embeded wiremock?

Hi,
I tried to launch stubrunner as standalone thanks to spring boot :
@SpringBootApplication
@EnableStubRunnerServer
public class StubRunnerBootApplication {
	public static void main(String[] args) {
		SpringApplication.run(StubRunnerBootApplication.class, args);
	}
}

stubrunner:
      workOffline: true
      cloud.stubbed.discovery.enabled: false
      ids: com.myapp.service:my-stub:1.0.0:stubs:8385
      stubs-mode: local
      httpServerStubConfigurer: com.myapp.service.WiremockCustomConfiguration

You can see I added "httpServerStubConfigurer" :
public class WiremockCustomConfiguration extends WireMockHttpServerStubConfigurer {
	@Override
	public WireMockConfiguration configure(WireMockConfiguration httpStubConfiguration, HttpServerStubConfiguration httpServerStubConfiguration) {
		httpStubConfiguration.asynchronousResponseEnabled(true);
		httpStubConfiguration.asynchronousResponseThreads(20);
		httpStubConfiguration.containerThreads(100);
		httpStubConfiguration.jettyAcceptors(10);
		httpStubConfiguration.jettyAcceptQueueSize(100);
		httpStubConfiguration.jettyHeaderBufferSize(16384);
		httpStubConfiguration.disableRequestJournal();
	}
}

In debug mode I can see this configuration is well executed but wiremock seems to not use those custom parameters :( . Is there an other way to customize Wiremock configuration ?
Thanks,
Julian

Can you please upload your sample to GitHub? I'd have to debug it. In the meantime you can debug it yourself by starting  WireMockHttpServerStub#start and if the if (configuration.configurer.isAccepted(wireMockConfiguration)) { condition is met and that your code gets executed.

I uploaded a sample in my personnal repo: https://github.com/JulianAffaire/services-stub
When I run the debug mode on the application I can see that the COnfiguration is readed. So Everythings seem ok but wiremock seems to not start with those parameters (I compared with the use of Wiremock FatJar).
Thanks

I debugued and it seems the configuration is weel registered by wiremock. But there is a big difference in term of performance between using stubrunner VS Standalone Wiremock Fat Jar (same wiremock configuration).
Using stubrunner when I test with lot of request per seconds it's slowing down quickly.
Is there a raison?
Thank you

Ok, so the main problem is solved, right?
As for the performance, I have no idea really, it's hard to track those things. Stub Runner just runs WireMock stubs and does nothing else, I have no idea what could impact the performance.

No it's not, because of this big difference between performance. Is there a way to be sure wiremock is really using those parameters when using StubRunner?

I debugued and it seems the configuration is weel registered by wiremock.

I really don't understand what else I can do.
Can you please provide steps for me to understand what is currently not working and what in your opinion should be done for this to start working. Also, the question is: how did you measure the performance?

Ok, I started Stubrunner with the project in copy : https://github.com/JulianAffaire/services-stub
Then I used JMeter to test the performances of the Stub. I noticed that is slowing down really quickly :(.
Otherwise, I started Wiremock Fat Jar on 8083 (http://wiremock.org/docs/running-standalone/)
 java -jar wiremock-standalone-2.25.1.jar --no-request-journal --container-threads 100 --jetty-acceptor-threads 10 --jetty-accept-queue-size 100 --jetty-header-buffer-size 16384 --async-response-enabled true --async-response-threads 20

Then, with a script I recorded in it the Stubs :
for filename in ./services-stub/build/stubs/META-INF/com.myapp.service/services-stub/1.0.0/mappings/**/*.json; do
	curl -X POST -H "Content-Type: application/json" -d @"$filename" http://localhost:8083/__admin/mappings/new
done

With Wiremock Fat Jar the test with Jmeter is really much better.
So I try to understand this gap :( . If I can do something to use stubrunner instead of Wiremock Fat Jar. Because Stubrunner is really easy to use with SpringBoot.
Are you able to confirm that with a configuration like that in Stubrunner Wiremock is really using my custom configuration? I know it's tricky it's why I asked :)
Thank you

I saw Stubrunner is using wiremock-standalone-2.20.0.jar
Maybe the wiremock version is the problem? Is there a way to upgrade it?

You can try to use the latest Hoxton.RC1 release train version (Contract 2.2.0.RC1) where we've bumped WireMock to the latest available version. You can also set the property wiremock.version in your build to a given WireMock version.

You could run a performance test against the same version of wiremock-standalone and see if there is a difference in performance.

The problem was the wiremock version :) It seems similar now in term of performance.
In my gradle file I added:
compile 'org.springframework.cloud:spring-cloud-starter-contract-stub-runner:2.2.0.RC1'
compile 'com.github.tomakehurst:wiremock-standalone:2.25.1'

Thank you for the support.

No problem