Spark plugin is not being called as of 0.21

There is a note in the changelog about changes to plugins that could potentially break things, but I'm not having any luck tracking down why the Spark plugin isn't called anymore.  I'm more than happy to fix it, but as far as I can tell the plugin still meets the criteria for being a plugin. Does anyone have an idea of where I should be looking to track things down?

Fixed in the plugin. To see what I had to change: https://github.com/ash-project/spark/commit/f083cb45734fffdc579db084275dfd308b387af4
Keep in mind its more complex as I'm retaining backwards compatibility for users that haven't upgraded elixir-ls

Oh, sorry about the breakage

All good, it's good changes to bring things under elixir-ls, I'll adapt to future changes as necessary as well 👍


[WIP] Try macOS CI

What do these changes do?

Related issue number

Fixes

Codecov Report

Merging #1304 (e8d6664) into main (e554e93) will decrease coverage by 33.45%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #1304       +/-   ##
===========================================
- Coverage   73.61%   40.16%   -33.46%
===========================================
  Files          88       88
  Lines        9510     9508        -2
===========================================
- Hits         7001     3819     -3182
- Misses       2509     5689     +3180

Impacted Files
Coverage Δ

python/graphscope/tests/unittest/test_lazy.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/tests/unittest/test_java_app.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...ython/graphscope/tests/unittest/test_cython_ast.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...ython/graphscope/tests/unittest/test_serailaize.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/tests/unittest/test_app.py
0.00% <0.00%> (-98.44%)
:arrow_down:

python/graphscope/analytical/udf/patch.py
3.47% <0.00%> (-96.53%)
:arrow_down:

...thon/graphscope/tests/unittest/test_scalability.py
0.00% <0.00%> (-96.16%)
:arrow_down:

...hon/graphscope/tests/unittest/test_create_graph.py
0.00% <0.00%> (-92.00%)
:arrow_down:

python/graphscope/tests/unittest/test_graph.py
0.00% <0.00%> (-84.79%)
:arrow_down:

python/graphscope/tests/unittest/test_context.py
0.00% <0.00%> (-81.04%)
:arrow_down:

... and 37 more

Continue to review full report at Codecov.

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


Provide means to de-list chart-version

If a chart/version has a security issue we might want a way to remove it from search listings. We should, at least, discuss.

No longer needed now that the Artifact Hub has taken over for the Helm Hub. closing


chore: Remove storyshots for some CI Savings GM-377

Overview

These aren't really testing anything other than being frustrating (no hooks / html tags) doesn't even catch errors.
Save a minute on JEST CI run.

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: https://codecademy.atlassian.net/browse/GM-377
[x] I have run this code to verify it works
[x] This PR includes unit tests for the code change

📬Published Alpha Packages<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://614a52c32caf552f2f1abcb6--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://615209eea5d486d11efa4006--gamut-preview.netlify.appDeploy Logs


Change Textobj: htmlAttribute

Change requested
These are some of the property writings in jsx and vue that I have put together, please feel free to close them
support jsx onClick={activateLasers}
support vue slot v-slot:header & #header & v-slot:[dynamicSlotName] & #[dynamicSlotName]
Sample Text

jsx

2. vue template slot

Checklist

[X] The textobj would be useful to more users than just me. (You can use the API to create custom textobj.)
[X] This is a feature request to change an existing textobject, not a bug report. (Use the bug report form for those.)

Sorry, vue and jsx adds too many edge cases for one text object to handle.
For this, I'd suggest using mini.ai to create custom text objects.

ok,thankyou


Critical CVE on goss

Describe the bug
Critical CVE on goss

How To Reproduce
Use trivy to detect CVE,  in our case it's has been detected on CI
Expected Behavior
Haven't CVE
Actual Behavior
CVE
Environment:

Version of goss: 0.4.7
OS/Distribution version: alpine 3.18.6

@dklimpel this is a good opportunity to test the new trivy pipeline. Is it possible to reproduce this finding in the goss CI?

You should be able to run the workflow manually: https://github.com/goss-org/goss/actions/workflows/docker-goss.yaml
But it probably won't find anything because the workflow creates a new build and the affected dependency seems to be indirect.

I think latest tagged release, not latest image.

Any update on this bug ? @dklimpel @aelsabbahy

@aelsabbahy @dklimpel I prepared PR with updating go version, because CVE located in stdlib

Sorry for the delay on this. It seems there's some issues with CI. Still trying to debug.
oddly the working commit and the failing commit are exactly the same, so not sure if something changed on travis-ci or if there's another factor at play (e.g. docker test image caching).

Update: Found the issue, I believe I merged in a fix. Unfortunately, I ran out of travis-ci OSS credits again, waiting on travis-ci to respond.
These issues will go away once the migration to GHA is complete. This is probably going to be the last release on the travis-ci workflow.

@aelsabbahy thanks for update, no problem, sometimes strange things happenings

Just cut a new release, please confirm the CVE is gone and we can close out this ticket.
Many thanks for reporting this issue and contributing the fix!

@aelsabbahy thanks so much, I tested update on 0.4.8 and it's fixing this Critical CVE


Migrate PIS to EBI infrastructure

As an operator I want to be able to run Platform-Input-Support (PIS) in an environment where it has access to all the resources it needs.
PIS needs to perform a number of operations from within the EBI network (reading ChEMBL data, updating FTP servers). To make this easier we want to create an environment within Noah's /nfs/... space to run PIS.
Acceptance criteria:

PIS should be available as a Docker image from Quay
Travis should automatically deploy new images to Quay
There should be an Opentargets space in Noah
The docker image in Noah should be able to write to Google Storage
Deployment documentation is updated to include instructions on running PIS

@cmalangone @JarrodBaker this is a great idea about having PIS inside EBI to grab the internal services data.


⚠ Bofrost.com has degraded performance

In 787a5c4, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 1434 ms

Resolved: Bofrost.com performance has improved in 3ec0ca9.


getInitialNotification is null if app is closed and opened from banner. Additional data in custom_notification

What version of RN and react-native-fcm are you running?
"react-native": "0.46.4",
"react-native-fcm": "^9.1.0",
What device are you using? (e.g iOS9 emulator, Android 6 device)?
android device nexus 5x
Is your app running in foreground, background or not running?

Iam not able to achieve this behaviour:
If app is closed (not running) and I receive notification, I want to show the notification and then be able to know, if the app was opened by clicking the banner, or app icon. I also expect to know if the banner was clicked when app is opened / in background, but that works fine (except the second question part below).

I send custom_notification payload to fcm according to docs, so after receiving silentNotification (with no notification payload) I let this package to build a local notification. But in both cases, whether I open the app by clicking the notification banner or app icon I get null value returned from FCM.getInitialNotification(). Does anybody know how to achieve the expected behaviour? I have spend too much time looking for answer here, but with no luck.

Also I noticed that after clicking banner and receiving second notification with opened_from_tray value, I lost any additional data I send with originial notification, to make it really clear, I send:

{
  "to":"FCM_TOKEN",
  "data": {
    "custom_notification": {
      "body": "test body",
      "title": "test title",
      "color":"#00ACD4",
      "priority":"high",
      "icon":"ic_notification",
      "show_in_foreground": true
    },
   "someOtherData":"data", //this gets lost
   "someEvenMoreOtherData":"data2", //this gets lost
  }
}

after clicking the banner I receive:
body:"test body",
color:"#00ACD4",
fcm:{action: null},
icon:"ic_notification",
opened_from_tray:1,
priority:"high",
show_in_foreground:true,
title:"test title",

so just what is in custom_notification, so should I send my additional data also in custom_notification?
also my manifest file
<application
      android:name=".MainApplication"
      android:allowBackup="true"
      android:label="@string/app_name"
      android:icon="@mipmap/ic_launcher"
      android:theme="@style/SplashTheme">
      <activity
        android:name=".MainActivity"
        android:launchMode="singleTop"
        android:label="@string/app_name"
        android:configChanges="keyboard|keyboardHidden|orientation|screenSize"
        android:windowSoftInputMode="adjustResize">
        <intent-filter>
            <action android:name="android.intent.action.MAIN" />
            <category android:name="android.intent.category.LAUNCHER" />
        </intent-filter>
        <!-- FCM START-->
        <intent-filter>
            <action android:name="fcm.ACTION.HELLO" />
            <category android:name="android.intent.category.DEFAULT" />
        </intent-filter>
        <!-- FCM END-->

      </activity>
      <activity android:name="com.facebook.react.devsupport.DevSettingsActivity" />

      <!-- FCM START -->
        <service android:name="com.evollu.react.fcm.MessagingService" android:enabled="true" android:exported="true">
            <intent-filter>
                <action android:name="com.google.firebase.MESSAGING_EVENT"/>
            </intent-filter>
        </service>

        <service android:name="com.evollu.react.fcm.InstanceIdService" android:exported="false">
            <intent-filter>
                <action android:name="com.google.firebase.INSTANCE_ID_EVENT"/>
            </intent-filter>
        </service>

        <receiver android:name="com.evollu.react.fcm.FIRLocalMessagingPublisher"/>
        <receiver android:enabled="true" android:exported="true"  android:name="com.evollu.react.fcm.FIRSystemBootEventReceiver">
          <intent-filter>
              <action android:name="android.intent.action.BOOT_COMPLETED"/>
              <action android:name="android.intent.action.QUICKBOOT_POWERON"/>
              <action android:name="com.htc.intent.action.QUICKBOOT_POWERON"/>
              <category android:name="android.intent.category.DEFAULT" />
          </intent-filter>
        </receiver>
</application>

Q1: the notification goes to intialNotification only if you LAUNCH the app by click the notif. If the app is running, it will go to Notification callback.
Q2: yes, the data gets lost. put it inside "custom_notification"

Thank you for your response.
Q1: OK, maybe Iam confused when the app is opened and when it is launched, but anyway, I log the notification callback and I don't see it, what I see is the app reload in debugger and the receive of notification is logged, the callback is not).
what I do:
close the app (by swiping it out from running applications on phone)
send the notification
open it from banner
could setting click_action help? (I am not in control of server side of the application and cannot just set it). If yes, should it go into custom_notification or outside?
Also this is my root component, where I guess the callback should be catched in the FCMEvent.Notification method with opened_from_tray value
FCM.on(FCMEvent.Notification, async (notif) => {
	console.log("FCMEvent.Notification", notif)
    // there are two parts of notif. notif.notification contains the notification payload, notif.data contains data payload
    if(notif.local_notification){
    	console.log("local_notification")
    }
    if(notif.opened_from_tray){
    	console.log("opened_from_tray")
    }
});

FCM.on(FCMEvent.RefreshToken, (token) => {
    console.log(token)
    // fcm token may not be available on first load, catch it here
});

const store = createStore(reducers, {}, applyMiddleware(ReduxThunk) )

class Root extends Component {

	render() {
		const store = createStore(reducers, {}, applyMiddleware(ReduxThunk) )
		console.disableYellowBox = true

		return (
			<Provider store = {store}>
				<RootScreen />
			</Provider>
		);
	}
}

can you show me your FCM.getInitialNotification() code?

right now there is just console log
This code is in my RootScreen in componentDidMount. Where is RootScreen in my app can be seen in code above. I tried to move it into Root component in componentDidMount and also outside of component, but value of notif is always null.
FCM.getInitialNotification().then(notif=>{
     console.log("getInitialNotification", notif)
});

android should always have something in initial notif. have no clue now

ok, I tried to create new app, set up everything and it works, so I have to find out where is the problem within my app. Closing this, will post here if I find what was causing it.

alright I have localized the problem. In my project Iam using splashscreen, I've seen an issue where the splashscreen problem was discussed, but he used another activity for it, I am not, Iam just modifeing app theme a little, so my project is set up this way.
in my manifest file (the whole manifest is above)
<application
      android:name=".MainApplication"
      android:allowBackup="true"
      android:label="@string/app_name"
      android:icon="@mipmap/ic_launcher"
      android:theme="@style/SplashTheme"> //my modified theme

values/styles.xml
<style name="SplashTheme" parent="Theme.AppCompat.Light.NoActionBar">
        <item name="android:colorPrimaryDark">@color/statusbar</item>
        <item name="colorPrimaryDark">@color/statusbar</item>
        <item name="android:windowBackground">@drawable/background_splash</item>
    </style>

drawable/background_splash.xml
<?xml version="1.0" encoding="utf-8"?>
<layer-list xmlns:android="http://schemas.android.com/apk/res/android">

    <item
        android:drawable="@color/splascreen_background"/>

    <item>
        <bitmap
            android:gravity="fill"
            android:src="@mipmap/splashscreen"
            />
    </item>

</layer-list>

The splashscreen bitmap is jpg picture ranging in size from mdpi: 720 x 480 up to xxxhdpi 1920 × 1280, my device xxhdpi uses 1440 × 960 picture. I guess the picture load time messes up with the initial load somehow, because if I replace the splashscreen picture with something much smaller (like app icon), the getInitialNotification method works fine. Splashscreen is 129 kB big (icon 5kb), so the difference is big, but I think it should not matter? I've tried if android likes png better, but he does not.
Another weird behaviour I've now noticed is that if I click the notification banner or app icon quickly after the banner is shown, the getInitialNotification returns expected values, otherwise it's null.
any ideas why this happens?

I may have some clue now.
source code
https://github.com/evollu/react-native-fcm/blob/master/android/src/main/java/com/evollu/react/fcm/FIRMessagingModule.java#L57-L65
You see it calls getCurrentActivity to get current running activity. Can you check what is the activity when you call it.
Perhaps it is running the splash screen's activity so you get nothing.

So I put a log into getCurrentActivity
public void getInitialNotification(Promise promise){
        Activity activity = getCurrentActivity();
        Log.e("ACTIVITY", activity.getLocalClassName());
        if(activity == null){
            promise.resolve(null);
            return;
        }
        promise.resolve(parseIntent(getCurrentActivity().getIntent()));
}

and in case I close the app, send notification and try to open the app, it gives me null error. And I can now also see why getInitialNotification in JS returns null, because it is returned here in the promise
if(activity == null){
            promise.resolve(null);
            return;
}

it would be weird, if any other activity would run, I don't have any other activity in my app, just MainActivity.

After some more testing, the null value may have something to do with remote debugger being on and moving the JS execution away from device (or whatever happens). If I console.warn the notif value with debugger off then the value is as expected.
With debugger on a hacky solution is to move the getInitialNotification deepeer in my project structure or leave it where it is and put it inside timeout (1000ms). But I find it weird that inside componentDidMount, the application is not truly fully loaded and returns null activity.
One more thing that could also have been causing the problem before is that I noticed I had one wrongly named file in one of my mipmap folders for splashscreen.
last question please, if I open app from banner, the fcm.action value in notif is null, if I set the click_action in my custom_notification, this is where it would show right?

for you last question:  right.
for your wierd bug, can you try replace the code with following?
Activity activity = getReactApplicationContext().getCurrentActivity();         if(activity == null){             promise.resolve(null);             return;         }         promise.resolve(parseIntent(activity.getIntent()));

Tried it, but the behavior is the same as before

And to further describe the behaviour
1) I remove the app from running application
   a) send the notification
      1) after notification appears I run the application from app icon -> initial notif is null
         a) remove the app from running application
         b) run it from app icon -> notif is as expected
      2) after notification appears I run the application from banner -> initial notif is null
         a) remove the app from running application
         b) run it from app icon -> notif is as expected
   b) do NOT send notification
      1) run application from app icon -> notif is as expected

so it is only after the first run of application after notification is recieved that this problem occures.

I may have clue now. We call mReactInstanceManager.createReactContextInBackground(); in MessagingService.java to wake up app to handle notification when notif arrives. This creates a context but doesn't create activity.

It does when it is received, but on banner click callback is not called.

ok then it is not related to mReactInstanceManager.createReactContextInBackground();
I have no idea why the activity is null now

I am having this issue as well. I am using react-native-splash-screen, which has its own launch Activity: SplashActivity.
public class SplashActivity extends AppCompatActivity {

  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);

    Intent intent = new Intent(this, MainActivity.class);
    startActivity(intent);
    finish();
  }

}

The issue is that MessagingService is sending its Intent message to SplashActivity, which then starts MainActivity, and the message is lost. The solution is simple: just add the following line to SplashActivity after the intent is constructed:
intent.putExtras(this.getIntent());

This copies over the "extras" from MessagingService's intent over to the new MainActivity intent, enabling FIRMessagingModule to later extract these extras and send them across the bridge to JavaScript.

@Ashoat I'm using that as well.
That SplashActivity is not the main activity for the app
you can see their document that it just calls SplashScreen.show(this);  // here to show up that activity to cover main activity

@Ashoat Thanks a lot for this. it was just what I was missing 💯


Compile failure - missing LV2_UI__scaleFactor

Hey Hermann, I tried to compile Rumor, but it spit out this error:
In file included from Rumor.c:74:
lv2_plugin.cc: In function ‘instantiate’:
lv2_plugin.cc:362:71: error: ‘LV2_UI__scaleFactor’ undeclared (first use in this function); did you mean ‘ui_scaleFactor’?
         const LV2_URID ui_scaleFactor = ui->map->map(ui->map->handle, LV2_UI__scaleFactor);
                                                                       ^~~~~~~~~~~~~~~~~~~
                                                                       ui_scaleFactor
lv2_plugin.cc:362:71: note: each undeclared identifier is reported only once for each function it appears in
make[1]: *** [makefile:104: Rumor] Error 1
make[1]: Leaving directory '/home/tedrippert/src/Rumor/Rumor'
make: *** [makefile:27: Rumor] Error 2

I do have  libX11-dev 2:1.6.7  and libcairo2-dev 1.6.7 installed.

Hi @Rippert
You need LV2 1.18.0 for that.
regards
hermann

Hi @Rippert
Okay, I've implemented a fix for that, so you don't been forced to update your lv2 stack.


Unexpected error: [Errno 9] Bad file descriptor

mperature":"28.92","humidity":"69.05","battery":"2.9"}
Unexpected error: [Errno 9] Bad file descriptor
waiting for connecting...
..connected from: ('<IP_ADDRESS>', 33502)
68:3E:34:CC:DC:3A
{"temperature":"28.94","humidity":"69.06","battery":"2.9"}
Unexpected error: [Errno 9] Bad file descriptor
waiting for connecting...
..connected from: ('<IP_ADDRESS>', 33504)
68:3E:34:CC:D8:D5
{"temperature":"28.93","humidity":"68.91","battery":"3.0"}
Unexpected error: [Errno 9] Bad file descriptor
waiting for connecting...
暂时不影响使用  py脚本有报错

没事，socket的连接清理问题，为了确保正常断开，下个版本我处理下

没事，插座的连接清理问题，为了确保正常断开，下个版本我处理下

好的，谢谢大佬 ！期待更新~
print("Unexpected error: {}".format(ex))  看不懂代码 我把print去掉 没报错了
waiting for connecting...
..connected from: ('<IP_ADDRESS>', 49062)
68:3E:34:CC:DC:3A
{"temperature":"31.45","humidity":"63.12","battery":"2.9"}
waiting for connecting...
..connected from: ('<IP_ADDRESS>', 49064)
68:3E:34:CC:D8:D5
{"temperature":"30.1","humidity":"66.11","battery":"3.0"}
waiting for connecting...
..connected from: ('<IP_ADDRESS>', 49084)
68:3E:34:CC:DC:3A
{"temperature":"31.4","humidity":"63.1","battery":"2.9"}
waiting for connecting...
..connected from: ('<IP_ADDRESS>', 49086)
68:3E:34:CC:D8:D5
{"temperature":"30.11","humidity":"66.09","battery":"3.0"}
waiting for connecting...
..connected from: ('<IP_ADDRESS>', 49104)
68:3E:34:CC:D8:D5
{"temperature":"30.13","humidity":"66.11","battery":"3.0"}
waiting for connecting...
..connected from: ('<IP_ADDRESS>', 49106)
68:3E:34:CC:DC:3A
{"temperature":"31.43","humidity":"63.08","battery":"2.9"}
waiting for connecting...
..connected from: ('<IP_ADDRESS>', 49126)
68:3E:34:CC:D8:D5
{"temperature":"30.11","humidity":"66.13","battery":"3.0"}
waiting for connecting...
..connected from: ('<IP_ADDRESS>', 49128)
68:3E:34:CC:DC:3A
{"temperature":"31.43","humidity":"63.08","battery":"2.9"}
waiting for connecting...
..connected from: ('<IP_ADDRESS>', 49146)
68:3E:34:CC:DC:3A
{"temperature":"31.4","humidity":"63.11","battery":"2.9"}


Check for ability to integrate internal snippets

Check for the internal snippets https://github.com/devonfw/.github/blob/master/CONTRIBUTING.asciidoc#documentation-changes by [.internal] markup.
Here the result: https://devonfw.com/website/pages/docs/contributing.asciidoc_using-github.html#contributing.asciidoc_documentation-changes
This mechanism must be available in antora as well.
It would be good when the current mechanism could be improved in a way that the placeholders are also not visible in the rendering of github. This would probably mean changing the placeholders to an asciidoc comment and converting them to the current solution before rendering them for the website.

Check if syntax works
Check if it is possible to include CSS and JS


support filtering table definitions

Added a mechanism to exclude unsupported table definitions, example for use with sqlite3 adapters that do not support ADD CONSTRAINT when changing the table schema.

Thank you for this PR
just want to know, are you planning to open PR in sqlite3 repo as well?

Thank you for this PR just want to know, are you planning to open PR in sqlite3 repo as well?

Yes, but I haven't written any code yet

please use sql@master go get github.com/go-rel/sql@master when developing in sqlite3 repo, I'll adjust the version later


add ristretto cache to Serve() -- 26657

Description
Please add a description of the changes that this PR introduces and the files that
are the most critical to review.
If this PR is non-trivial/large/complex, please ensure that you have either
created an issue that the team's had a chance to respond to, or had some
discussion with the team prior to submitting substantial pull requests. The team
can be reached via GitHub Discussions or the Cosmos Network Discord server in
the #cometbft channel. GitHub Discussions is preferred over Discord as it
allows us to keep track of conversations topically.
https://github.com/cometbft/cometbft/discussions
If the work in this PR is not aligned with the team's current priorities, please
be advised that it may take some time before it is merged - especially if it has
not yet been discussed with the team.
See the project board for the team's current priorities:
https://github.com/orgs/cometbft/projects/1
-->
This PR adds an experimental cache to the Serve() function.  I don't know how large the cache should be, and this will be fairly extreme.  The goal is to improve performance and concurrency.

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use
unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments

@faddat if you would like to demonstrate things to others in draft PRs, please do so on your own fork of this repository. Right now, given that this PR doesn't even have a proper description, reasoning, etc., I would consider this noise.

If you would like to bring up a problem, or suggest an enhancement, please open an issue first prior to submitting any draft PRs, so that we know what the context for the draft PR is.
And please use proper descriptions in your PRs, even if they're drafts.


DSD-240: MultiSelect Draft

Fixes JIRA ticket xxx
This PR does the following:

How has this been tested?

Checklist:

[ ] I have updated the Storybook documentation accordingly.
[ ] All new and existing tests passed.

Front End Review:

[x] View the example in Storybook

Tugboat has finished building the preview for this pull request!
Link:

https://pr776-j6te1xet1hg8tkltm8lz7twsjidrkxrl.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6197d859e368650741427363

Tugboat has finished building the preview for this pull request!
Link:

https://pr776-j6te1xet1hg8tkltm8lz7twsjidrkxrl.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6197d859e368650741427363

@bigfishdesign13 I at least got this branch updated with the latest from development, as it was way behind, so tugboat preview should work again.

Tugboat has finished building the preview for this pull request!
Link:

https://pr776-j6te1xet1hg8tkltm8lz7twsjidrkxrl.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6197d859e368650741427363

Tugboat has finished building the preview for this pull request!
Link:

https://pr776-j6te1xet1hg8tkltm8lz7twsjidrkxrl.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6197d859e368650741427363

Tugboat has finished building the preview for this pull request!
Link:

https://pr776-gpuszll0afk8knlvlljt0xl0zagjmupm.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/62394810c49d661fdca706ec

Tugboat has finished building the preview for this pull request!
Link:

https://pr776-jwrdqr2ttir6ylbgyzwrybmlbqzk7qw0.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/623e281897feaa22ee12e9b5

Tugboat has finished building the preview for this pull request!
Link:

https://pr776-jwrdqr2ttir6ylbgyzwrybmlbqzk7qw0.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/623e281897feaa22ee12e9b5

Closing this in favor of https://github.com/NYPL/nypl-design-system/pull/987


[WC-2349]: fix DG2 columns not update, sends wrong columns to excel

Pull request type

Bug fix (non-breaking change which fixes an issue)

Description
This PR aims to fix a bug, where the DG2 columns are not updating on the export machine, making the export to excel action to receive the wrong set of columns. The awaitingCallback state in the state machine currently only accept null on columns type, which makes the exportFlow to not update the columns when their state updates. Then when the exportColumns function is called it sends outdated columns to the export to excel action.

please add the changelog


update eclipse formatter

I updated some values in the formatter to match the latest checkstyle in my changes.
This is the exported version of the formatter (line order got messed up a lot) from eclipse neon.
Not sure if this breaks older eclipse installations though :)

Hooray Jenkins reported success with all tests good!

@msteiger think you can check this out?

Yeah sure, what changes does it contain? The only issue I'm aware of is the different order of import statements.

@oniatus / @msteiger: bump!

The diff is really unreadable :/
I made some smaller changes to the whitespacing, e.g. like this. I am not sure about the correct import order though.

Possibly related: http://checkstyle.sourceforge.net/config_imports.html#ImportOrder

Went ahead and merged this so we can move forward with any additional tweaks on top. Heads up to the Eclipse users :-)


  It is a desired option to save the original information, rather than the current value.  But it isn’t always possible right now.  (Try it with PR #349 with the ORIGINAL option enabled for glm_save_options.)  We’ll need to support that for all output formats.

@aivanova5, is this still an open issue?


How to handle errors when using UltraDict in multiprocessing and how to asynchronously wait for UltraDict to be written to in the main process.

`import time
import random
import threading
from multiprocessing import Process, Manager
from UltraDict import UltraDict
class A():
def init(self, name, dictname):
self.name = name
self.dictname = dictname
self.personnum = -1
self.time = time.time()
self.task = threading.Thread(target=self.a)
def a(self):
    while True:
        self.personnum = random.random()
        self.time = time.time()
        other = UltraDict(name=self.dictname)
        other[self.name] = {"time": self.time, "num": self.personnum}
        time.sleep(1)

def start(self):
    self.task.start()

class ProcessClass():
def init(self, i, name):
self.dict1 = {}
self.dictname = name
self._process = Process(target=self.run, args=(i,))
def run(self, i):
    for j in range(5):
        # 每个 A 对象与共享字典关联
        a = A(str(i) + str(j), self.dictname)
        a.start()
        self.dict1[str(i) + str(j)] = a
    while True:
        time.sleep(1)  # 保持进程运行

def start(self):
    self._process.start()

if name == 'main':
ultra = UltraDict(name='getpic', shared_lock=True)
name = ultra.name
processes = []
for i in range(5):
process_instance = ProcessClass(i, name)
process_instance.start()
processes.append(process_instance)
while True:
    time.sleep(2)  # 每隔2秒打印一次
    print(ultra)
    # print("实时获取的数据:", dict(shared_dict))`
   UltraDict.Exceptions.CannotAttachSharedMemory: Could not get memory 'wnsm_cfd5be17'

Hey, what kind of errors? Can you reformat your code? I only speak English and German.
To your second question: You don't need to wait for UltraDict. Tt will make sure that you always have the newest data. You can work with a lock as explained in the docs https://github.com/ronny-rentner/UltraDict?tab=readme-ov-file#locking if your use case requires it.

`import time
import random
import threading
from multiprocessing import Process, Manager
from UltraDict import UltraDict
class A():
def init(self, name, dictname):
self.name = name
self.dictname = dictname
self.personnum = -1
self.time = time.time()
self.task = threading.Thread(target=self.a)
def a(self):
    while True:
        self.personnum = random.random()
        self.time = time.time()
        other = UltraDict(name=self.dictname)
        other[self.name] = {"time": self.time, "num": self.personnum}
        time.sleep(1)

def start(self):
    self.task.start()

class ProcessClass():
def init(self, i, name):
self.dict1 = {}
self.dictname = name
self._process = Process(target=self.run, args=(i,))
def run(self, i):
    for j in range(5):
        a = A(str(i) + str(j), self.dictname)
        a.start()
        self.dict1[str(i) + str(j)] = a
    while True:
        time.sleep(1)

def start(self):
    self._process.start()

if name == 'main':
ultra = UltraDict(name='getpic', shared_lock=True)
name = ultra.name
processes = []
for i in range(5):
process_instance = ProcessClass(i, name)
process_instance.start()
processes.append(process_instance)
while True:
    time.sleep(2)
    print(ultra)

`

import time
import random
import threading
from multiprocessing import Process, Manager
from UltraDict import UltraDict
class A():
def init(self, name, dictname):
self.name = name
self.dictname = dictname
self.personnum = -1
self.time = time.time()
self.task = threading.Thread(target=self.a)
def a(self):
    while True:
        self.personnum = random.random()
        self.time = time.time()
        other = UltraDict(name=self.dictname)
        other[self.name] = {"time": self.time, "num": self.personnum}
        time.sleep(1)

def start(self):
    self.task.start()

class ProcessClass():
def init(self, i, name):
self.dict1 = {}
self.dictname = name
self._process = Process(target=self.run, args=(i,))
def run(self, i):
    for j in range(5):
        a = A(str(i) + str(j), self.dictname)
        a.start()
        self.dict1[str(i) + str(j)] = a
    while True:
        time.sleep(1)

def start(self):
    self._process.start()

if name == 'main':
ultra = UltraDict(name='getpic', shared_lock=True)
name = ultra.name
processes = []
for i in range(5):
process_instance = ProcessClass(i, name)
process_instance.start()
processes.append(process_instance)
while True:
    time.sleep(2)
    print(ultra)

`import time
import random
import threading
from multiprocessing import Process, Manager
from UltraDict import UltraDict
class A():
def init(self, name, dictname):
self.name = name
self.dictname = dictname
self.personnum = -1
self.time = time.time()
self.task = threading.Thread(target=self.a)
def a(self):
    while True:
        self.personnum = random.random()
        self.time = time.time()
        other = UltraDict(name=self.dictname)
        other[self.name] = {"time": self.time, "num": self.personnum}
        time.sleep(1)

def start(self):
    self.task.start()

class ProcessClass():
def init(self, i, name):
self.dict1 = {}
self.dictname = name
self._process = Process(target=self.run, args=(i,))
def run(self, i):
    for j in range(5):
        a = A(str(i) + str(j), self.dictname)
        a.start()
        self.dict1[str(i) + str(j)] = a
    while True:
        time.sleep(1)

def start(self):
    self._process.start()

if name == 'main':
ultra = UltraDict(name='getpic', shared_lock=True)
name = ultra.name
processes = []
for i in range(5):
process_instance = ProcessClass(i, name)
process_instance.start()
processes.append(process_instance)
while True:
    time.sleep(2)
    print(ultra)`

import time
import random
import threading
from multiprocessing import Process, Manager
from UltraDict import UltraDict

class A():
    def __init__(self, name, dictname):
        self.name = name
        self.dictname = dictname
        self.personnum = -1
        self.time = time.time()
        self.task = threading.Thread(target=self.a)

    def a(self):
        while True:
            self.personnum = random.random()
            self.time = time.time()
            other = UltraDict(name=self.dictname)
            other[self.name] = {"time": self.time, "num": self.personnum}
            time.sleep(1)

    def start(self):
        self.task.start()

class ProcessClass():
    def __init__(self, i, name):
        self.dict1 = {}
        self.dictname = name
        self._process = Process(target=self.run, args=(i,))

    def run(self, i):
        for j in range(5):
            a = A(str(i) + str(j), self.dictname)
            a.start()
            self.dict1[str(i) + str(j)] = a
        while True:
            time.sleep(1)

    def start(self):
        self._process.start()

if __name__ == '__main__':
    ultra = UltraDict(name='getpic', shared_lock=True)
    name = ultra.name
    processes = []
    for i in range(5):
        process_instance = ProcessClass(i, name)
        process_instance.start()
        processes.append(process_instance)

    while True:
        time.sleep(2)
        print(ultra)

the code raise UltraDict.Exceptions.CannotAttachSharedMemory: Could not get memory 'wnsm_0cbb1722'

Thanks for providing the code sample. I'll check it out later today and then get back to you.

thanks a lot


build(.devcontainer), ci(package.json): support GitHub Codespaces, newer Node.js and npm versions

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

build(.devcontainer): support GitHub Codespaces
ci(package.json): support newer Node.js and npm versions
Fix npm ci error when using GitHub Codespaces.

Fix npm ci error when using GitHub Codespaces:
@lloydchang ➜ /workspaces/freeCodeCamp-classroom (main) $
@lloydchang ➜ /workspaces/freeCodeCamp-classroom (main) $ npm ci
npm ERR! code EBADENGINE
npm ERR! engine Unsupported engine
npm ERR! engine Not compatible with your version of node/npm<EMAIL_ADDRESS>npm ERR! notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>npm ERR! notsup Required: {"node":"^16","npm":"^8"}
npm ERR! notsup Actual:   {"npm":"9.6.7","node":"v20.3.0"}

npm ERR! A complete log of this run can be found in: /home/codespace/.npm/_logs/2023-07-05T02_17_54_166Z-debug-0.log
@lloydchang ➜ /workspaces/freeCodeCamp-classroom (main) $ node --version
v20.3.0
@lloydchang ➜ /workspaces/freeCodeCamp-classroom (main) $ npm --version
9.6.7
@lloydchang ➜ /workspaces/freeCodeCamp-classroom (main) $ npm ci

><EMAIL_ADDRESS>prepare
> husky install

husky - Git hooks installed

added 889 packages, and audited 890 packages in 41s

157 packages are looking for funding
  run `npm fund` for details

31 vulnerabilities (30 moderate, 1 high)

To address issues that do not require attention, run:
  npm audit fix

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.
npm notice
npm notice New minor version of npm available! 9.6.7 -> 9.7.2
npm notice Changelog: https://github.com/npm/cli/releases/tag/v9.7.2
npm notice Run npm install -g<EMAIL_ADDRESS>to update!
npm notice
@lloydchang ➜ /workspaces/freeCodeCamp-classroom (main) $ npm install -g<EMAIL_ADDRESS>
removed 16 packages, and changed 42 packages in 4s

28 packages are looking for funding
  run `npm fund` for details
@lloydchang ➜ /workspaces/freeCodeCamp-classroom (main) $ npm ci

><EMAIL_ADDRESS>prepare
> husky install

husky - Git hooks installed

added 889 packages, and audited 890 packages in 34s

157 packages are looking for funding
  run `npm fund` for details

31 vulnerabilities (30 moderate, 1 high)

To address issues that do not require attention, run:
  npm audit fix

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.
@lloydchang ➜ /workspaces/freeCodeCamp-classroom (main) $

Relates to https://github.com/freeCodeCamp/classroom/pull/205

Node.js 16 will be end-of-life in September:
From https://github.com/nodejs/release#release-schedule

Release
Status
Codename
Initial Release
Active LTS Start
Maintenance Start
End-of-life

16.x
Maintenance
Gallium
2021-04-20
2021-10-26
2022-10-18
2023-09-11

Screenshot of GitHub Codespaces after start-up and attaching to the Terminal:

Using postAttachCommand (A command to run each time a tool has successfully attached to the container) to read the shell-evaluated ${CODESPACE_NAME} variable.
https://docs.github.com/en/codespaces/developing-in-codespaces/default-environment-variables-for-your-codespace

Good news!
I tested this so that GitHub Codespaces and Gitpod are a parity.
Please see the screenshots below where the GitHub Codespaces screenshot looks the same as the Gitpod screenshot:
GitHub Codespaces Screen Shot 2023-07-06 at 12 41 20 PM.png

Gitpod Screen Shot 2023-07-06 at 10.20.34 AM.png

Cross-posted at https://discord.com/channels/692816967895220344/715074489422970962/1127463017244790904

If anyone is available to review this pull request that I submitted, that would be wonderful:
build(.devcontainer), ci(package.json): support GitHub Codespaces, newer Node.js and npm versions #349
 https://github.com/freeCodeCamp/classroom/pull/349
Thanks!

Hi Lloyd,
Thanks for your PR! We have seen this and are working to be able to review all of our new PRs, but typically this is done by hand (pulling your changes onto my dev environment and testing for myself) so this can take a while with so many PRs.
However, I'll be reviewing this PR on Monday (07/09/23), just wanted to give you a heads-up that we have seen it.
Thanks for your contribution and patience!


Calculate and save results in gigatonnes

Simple changes to convert and save out the results within the glambie processing algorithm to gigatonnes.
Done on regional as well as global level, using existing conversion functions

Thanks for feedback. That's it done :)
Main change is that I combined global aggregation into 1 function and added a unit test for gigatonnes.


trezor: we are incompatible with trezorlib >= 0.9 (now: >= 0.10)

This is the change done in electrum to make it compatible with trezorlib 0.9 and up:
https://github.com/spesmilo/electrum/commit/dda9d4b746700bfc095c80d84f1ac5de46c5f2ca

Is this fixed now?

maybe this should be re-opened. We still seem to depend on trezorlib <= 0.9. trezorlib 0.10 wont work (missing transport_hid.py)


Leverage capability API for socket binding dependencies,

Removes usage of recently deprecated fields.

Linux with security manager Build 1898 is now running using a merge of a1fffb9bdd38849b809ae576b1c979d4cba9bd67

Windows Build 2076 is now running using a merge of a1fffb9bdd38849b809ae576b1c979d4cba9bd67

Linux Build 7021 is now running using a merge of a1fffb9bdd38849b809ae576b1c979d4cba9bd67

Linux with security manager Build 1898 outcome was SUCCESS using a merge of a1fffb9bdd38849b809ae576b1c979d4cba9bd67
Summary: Tests passed: 871, ignored: 280 Build time: 0:06:31

Windows Build 2076 outcome was FAILURE using a merge of a1fffb9bdd38849b809ae576b1c979d4cba9bd67
Summary: Tests failed: 1 (1 new), passed: 2943, ignored: 344 Build time: 0:48:19
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.clustering.cluster.ejb.remote.RemoteFailoverTestCase(SYNC-tcp).testConcurrentFailover: <no details avaliable>

Linux Build 7021 outcome was FAILURE using a merge of a1fffb9bdd38849b809ae576b1c979d4cba9bd67
Summary: Tests failed: 2 (2 new), passed: 2939, ignored: 344, muted: 3 Build time: 0:50:38
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ejb.remote.suspend.EjbRemoteSuspendTestCase.testSuspendedCallRejected: <no details avaliable>

org.jboss.as.test.integration.ejb.transaction.bmt.lazyenlist.LazyTransactionEnlistmentUnitTestCase.test: <no details avaliable>

Windows Build 2077 is now running using a merge of a1fffb9bdd38849b809ae576b1c979d4cba9bd67

Linux Build 7022 is now running using a merge of a1fffb9bdd38849b809ae576b1c979d4cba9bd67

Windows Build 2077 outcome was SUCCESS using a merge of a1fffb9bdd38849b809ae576b1c979d4cba9bd67
Summary: Tests passed: 2944, ignored: 344 Build time: 0:48:39

Linux Build 7022 outcome was FAILURE using a merge of a1fffb9bdd38849b809ae576b1c979d4cba9bd67
Summary: Tests failed: 1 (1 new), passed: 2940, ignored: 344, muted: 3 Build time: 0:51:30
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ejb.remote.distinctname.defaultname.DefaultDistinctNameTestCase.testRemoteSLSBInvocation: <no details avaliable>

While this works, to be totally correct I think you also need to create a RuntimeCapability which is then referenced from the relevant add handlers and attribute definitions. That way you will also get validation when people are modifying the config in admin-only mode.
An example is in https://github.com/wildfly/wildfly-core/pull/852/files#diff-2f55882885d08bc5c45c73f8bfb55921R97

Closing until fixed.


offers are not pushing to bol

Hi,
the offers are not being pushed to bol.com plaza.
It doesn't catch any exception... weird..

Hi,
It works, but it doesn't send all the products to Bol.com, even if the products are valid.
See following snippet
$upsertRequest = new BolPlazaUpsertRequest();
               $offer = new BolPlazaRetailerOffer();
               $offer->EAN = $bol_ads->ean;
               $offer->Condition = $bol_ads->condition;
               $offer->Price = $bol_ads->price;
               $offer->DeliveryCode = $bol_ads->delivery_code;
               $offer->Publish = 'true';
               $offer->ReferenceCode = $bol_ads->reference_code;
               $offer->QuantityInStock = $bol_ads->stock;
               $offer->Description = $bol_ads->description;
               $offer->Title = $bol_ads->title;
               $offer->FulfillmentMethod = $bol_ads->fullfilment;
               $upsertRequest->RetailerOffer = $offer;

               try {
                   var_dump($client->createOffer($upsertRequest));
               } catch (\Exception $e) {
                   var_dump($e->getMessage());
               }

I enterd 50 valid products, but not all products are getting the bol.com. I tried to enter some invalid data, the exception get's fired. It is strange that the valid entries not getting to bol.com

this product for example :) I can add it manually in Bol.com, however if i use the API it doesn't get added.

   $upsertRequest = new BolPlazaUpsertRequest();
        $offer = new BolPlazaRetailerOffer();
        $offer->EAN = '8944688059399';
        $offer->Condition = 'REASONABLE';
        $offer->Price = '9';
        $offer->DeliveryCode = '3-5d';
        $offer->Publish = 'true';
        $offer->ReferenceCode = 'Spinner';
        $offer->QuantityInStock = 1;
        $offer->Description = 'Spinner GEEL';
        $offer->Title = 'Spinner Geel';
        $offer->FulfillmentMethod = 'FBR';
        $upsertRequest->RetailerOffer = $offer;

        try {
            dd($client->createOffer($upsertRequest));
        } catch (Exception $e) {
            dd($e->getMessage());
        }

Ok i found it, Please make sure you cast all the values to the correct format. You can check the xsd from bol.com what values are expected. For example, you must cast the EAN to (int) and the price to (float), otherwise it will not work.
The Bol.com doesn't give any errors about the casting, so debugging is very hard BECAUSe the Bol Api just returns an empty string as a return msg...

Glad you found it @ermst4r


python3

I made a version for python3 as this library seems to no longer be maintained
https://github.com/sohrabtowfighi/Simple-Python3-Chess

I would like to chat with:
Liudmil Mitev
or:
sohrabtowfighi
about incorporating the ideas behind My Chess War Programs with your python Simple-Python-Chess code and then possible further developments.
If either or both of you have any interest, let's work out a time to talk.
The tcl/tk code for my stuff is here:
https://sourceforge.net/projects/zbcw/


fix:The displayName and detail are allowed to be modified on the product page.

The displayName and detail are allowed to be modified on the product page, so do not update the product's own displayName and detail when updating the plan.

That's made for a purpose


Performance difference between other SIMD accelerator

Ash
Any idea whether SimSIMD library can outperform acceleration comparing :
https://github.com/jfalcou/eve
thanks

I've never benchmarked Eve, but not sure if it's a fair comparison.
Both Eve and SimSIMD are SIMD accelerated and use similar underlying hardware capabilities (not sure if they handle SVE though), but the first is a larger C++ general purpose templates library, and the latter is a tiny C collection of mixed-precision numerical kernels tailored for Search.
Many years ago I went down the rabbit hole of designing something similar to Eve and failed. Now I prefer other approaches - either design a new compiler, or if ultimate performance is needed - write custom hardware-specific kernels for every operation (similar to SimSIMD & StringZilla, as well as Intel Hyperscan, simd-json, and simd-utf).


Add code formatter

Add a code formatter hook for maintaining code quality
Suggestions invited

Closing this as code formatter is already there


Tree View should stay up to date with UI tree

The UI Tree View is initialized when you launched the app, but isn't updated after that. It should be kept in sync with the UI tree, at least as best as possible.

Addressed in f9f957c2301dbc71ec99649de063156944bacc93.


Always borrow probe data when possible

This PR does a few things.

Data passed to a probe macro is always borrowed. This uses the std::borrow::Borrow trait when possible, and sometimes the AsRef trait where appropriate (slices, strings).
Updates the type-checks inside the generate probe macros to first clone the closure. Neither the closure nor the captured data are ever actually cloned, since the closure is used inside a closure that's never called. Cloning is just to avoid compiler errors complaining about using data after it's moved.
Adds the Clone trait bound to serializable types. This should be a minimal burden on users, and supports the previous point.
Updates some documentation

Discussed offline with @ahl, merging.


add action and last updated columns to consent history, EPROMS only

address comments from Joji in this issue:
https://jira.movember.com/browse/TN-131
changes were to add last updated and comment(i.e. action) columns in the consent history table
@mcjustin Justin, I added some changes based on Joji's comments, namely to add additional columns (last updated and action) to the consent history table.  I think these should be added for this release.  But let me know what you think?

@mcjustin I'm restarting the tests- looks like PyPI (python package index) timed out on downloading some modules


deepin.qt5platform-plugins: 5.6.3 -> 5.6.5

Description of changes
enable wayland support,  make DDE to work under wayland requires more than this

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

@ofborg build deepin.deepin-voice-note  deepin.deepin-terminal deepin.deepin-reader deepin.deepin-picker deepin.deepin-image-viewer deepin.deepin-editor deepin.deepin-draw deepin.deepin-compressor deepin.deepin-camera deepin.deepin-calculator deepin.deepin-album deepin.dde-calendar

Result of nixpkgs-review pr 221675 run on aarch64-linux 1

3 packages failed to build:

deepin.dde-calendar
deepin.deepin-terminal
deepin.dpa-ext-gnomekeyring

15 packages built:

deepin.dde-device-formatter
deepin.dde-polkit-agent
deepin.deepin-album
deepin.deepin-calculator
deepin.deepin-camera
deepin.deepin-compressor
deepin.deepin-draw
deepin.deepin-editor
deepin.deepin-image-viewer
deepin.deepin-movie-reborn
deepin.deepin-music
deepin.deepin-reader
deepin.deepin-voice-note
deepin.qt5integration
deepin.qt5platform-plugins

@ofborg deepin.deepin-terminal deepin.dpa-ext-gnomekeyring


Plan, get buy-in, and implement index-time versioning

Some proposals -- On agenda for meeting.

Change of direction:

Bill will supply supersededBy and supersedes, no work needed at index time. Joel needs to pass these in when he registers datasets.


DP Composite Pattern in Java

Need to make a readme file which should have the
Definition
Explanation in simple terms
Code related explanation

Assign to me


[Remote Store] Add support for randomizing Remote Store enabled testing.

Description
This PR enables the support for randomizing Remote Store enabled testing. This helps to increase the Remote Store test coverage by randomly using remote backed storage for tests not critical of remote store.
Related Issues
Resolves #[Issue number to be closed when this PR is merged]

Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[x] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Gradle check will fail with java compile error. As this PR is dependent on code introduced in this https://github.com/opensearch-project/OpenSearch/pull/12487. Once https://github.com/opensearch-project/OpenSearch/pull/12487 is merged I will rebase this branch to fix compile error.

Looks like https://github.com/opensearch-project/OpenSearch/pull/13845 never made it.


carbon.widget.TextView's  carbon_stroke useless

carbon.widget.TextView's  carbon_stroke useless

Could you be a little more specific?

this attribute can't work

Both stroke for TextView and rounded corners for EditText work fine. I'm attaching the screenshot. Please make sure, that you're using the most recent version and one of Carbon's themes.

I think i'm using the most recent version
but this Editext not implements RoundedCornersView,TextView not implements StrokeView

I think i'm using the most recent version
but this Editext not implements RoundedCornersView,TextView not implements StrokeView

Rounded corners for EditText and stroke for TextView were added and are present in the repository. There wasn't a release though. I just released 0.15.1 with these changes. It should be on Maven in a couple of minutes.


How to show variant name of tagged enum?

My endpoint returns a tagged enum
enum Foo {
Price(FooPrice),
...
}

However, in redoc it's displayed as many object. I have to click through them to see the details. I checked the generated openai.json, each variant is inlined within the enum, rather than taking a reference to #/components/schemas/Foo_FooPrice
I wonder if there's a way to let variants display with their name, rather than object?

Seems related to https://github.com/GREsau/schemars/issues/157

@JakkuSakura I believe this issue is the answer.


[BUG] Disclaimer about production use

[BUG] Warn about blindly copy-pasting some functions due to performance issues
Expected Snippet Behavior

Some snippets have performance issues that should come with a disclaimer:

deepFlatten creates an intermediary array at each recursion step. If the array is large and deeply-nested, this will lead to an increased memory consumption (if it runs on the main thread it will block it after the function finishes execution due to the garbage collection kicking in). Maybe point to something like the first snippet from here, which does not exhibit this problem.
runAsync creates a WebWorker each time it's called - there should be a warning not to call this function from inside a map, forEach or any kind of loop.
createElement - causes the browser to parse the string during each call. Using a template is a better solution if this function is called multiple times with the same argument.

Current Snippet Behavior
No warning that using some snippets in a production environment might cause serious performance issues.

Possible Solution
Display a warning (either next to the snippet or when the "Copy to clipboard" button is pressed).

Literally everywhere on the repository users are warned about the fact that the code is not production-ready and merely a learning resource. Adding this disclaimer to every snippet is kind of pointless, however there might be some more prominent places to display the not production-ready notices on the website. I'm closing the issue as the individual snippets are irrelevant to the way we are going to solve this, but I am going to deal with it during the weekend to make sure the website has the proper warnings for users who do not visit the repository itself.

Literally everywhere on the repository users are warned about the fact that the code is not production-ready and merely a learning resource.

My bad, I should have mentioned that I was referring to the website and not the repository.

I am going to deal with it during the weekend to make sure the website has the proper warnings for users who do not visit the repository itself.

Thank you.

Should this be closed before the website is updated? I know it's on the schedule you keep @Chalarangelo but it might be a good idea to keep it open until then just as a reminder


Add 1.27 conformance test results

Issue #, if available:
Description of changes:
Adds the 1.27 conformance test results
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve


Voucher total usage times limit is 0 by default

I'm trying to use a voucher code in a checkout but it is not accepted despite fulfilling all the constraints.
The problem comes from the checkbox "Limit number of times this discount can be used in total", which has a value of 0 even when being unchecked. Changing this value to a higher number allows to use the voucher properly.
Steps to reproduce the problem

Go to discounts/vouchers in the dashboard
Create a new voucher with just a code and an amount
Try to use the voucher in a checkout
-> An error is thrown saying the voucher code doesn't exists

What I expected to happen
The voucher should be accepted because I didn't check the "Limit number of times this discount can be used in total" checkbox.

Hi, I checked it and it's a problem in dashboard. I've already made a fix in #271


Add Google Recommendations integration

What is the motivation for adding / enhancing this feature?

The feature request to add a Recommended products block based on the:
https://cloud.google.com/recommendations-ai/docs/
What are the acceptance criteria

[ ] RecommendedProducts smart commponent added to display the recommended products
[ ] vue-storefront-api based product catalog integration with the Google API (to send out the catalog data)
[ ] Vue Storefront integration - calling out Google APIs with what users are browsing, adding to the cart etc.

Can you complete this feature request by yourself?

[ ] YES
[ ] NO

Which Release Cycle state this refers to? Info for developer.
Pick one option.

[x] This is a normal feature request. This should be available on https://test.storefrontcloud.io and then after tests this can be added to next Vue Storefront version. In this case Developer should create branch from develop branch and create Pull Request 2. Feature / Improvement back to develop.
[ ] (Pick this option only if you're sure) This is an important improvement request for current Release Candidate version on https://next.storefrontcloud.io and should be placed in next RC version. In this case Developer should create branch from release branch and create Pull Request 3. Stabilisation fix back to release.
[ ] (Pick this option only if you're sure) This is a critical improvement request for current Stable version on https://demo.storefrontcloud.io and should be placed in next stable version. In this case Developer should create branch from hotfix or master branch and create Pull Request 4. Hotfix back to hotfix.

Additional information

Related: https://aws.amazon.com/personalize/


✨ [REQUEST] Birdnet-Go

Checked

[X] If a new addon, I have checked on  Google that such as addon doesn't already exists
[X] If a new feature, I have searched in the discussions & issues that it was not already covered

Which addon?
Birdnet-go
Is your feature request related to a problem? Please describe
No it is not.
Describe the solution you'd like
Hey! Would it be possible to have the addon autmoaticlly map (instead of adding it to config) the share patition so that clips and webui.log are availble at the get-go?
Cheers

Sometimes birdnet-go stops analysing, would there be a way to monitor the logs and if it does say not get a reading for 10 mins restart the container?

Sometimes birdnet-go stops analysing, would there be a way to monitor the logs and if it does say not get a reading for 10 mins restart the container?

Hi, I've done that for birdnet-pi but it was much easier as based on services. I'll have to look how to do it for the go version

For your main question you can already do it by mapping a drive and using the relative path (such as /mnt/NAS/Clips instead of the default "Clips" which is relative to the home folder)
I could do it by default i'll have a look

Hi Alex! What are the differences between Pi and Go? Can you still join bird weather with go? As you know I've been testing and playing with Pi and enjoying it. I am curious about the lower system load of Go though.
Also - is it possible to run your Bridget-Pi as a stand alone docker container outside of Home Assistant?

Hi, here is my view from what I know :

Birdnet-pi : historical app abandoned by its owner but with a very active fork to keep it alive, lots of functions, runs on x64 (such as rpi4, x86). With it running, average cpu usage is <20% and analysis of a 15s segment lasts 2.5 seconds.
Birdnet-go : new app under heavy development, minimalistic UI, more resources efficient (for example runs on rpi3)

Both can join birdweather!
I'm all for the dev of a new system but as I user I'm selfishly using birdnet-pi as it is much more mature with features of analysis, deletion, species change... Also it is based on php/py/sh which means I can contribute as a contrary of birdnet-go that is in go which I know nothing about.
Currently you can't run it as a standalone container but I think another user was developing one...

Thanks! Considering I don’t use the change species.  Would I lose anything going to your Go Addon? Is species detection just as good on Go?

I haven't really tested it! What you could do is run both add-ons in parallel with the same rtsp feed. I've ran 2 birdnet pi add-ons in parallel without any impact on performance of the rpi4!

Got it. What about Size? That's really the problem I'm running into more. My HA VM was 20gigs and now it's 60 and I'm trying to figure it why.

I'm not sure about size ; birdnet-pi is based on "ghcr.io/linuxserver/baseimage-debian:arm64v8-bookworm" and birdnet-go on "ghcr.io/tphakala/birdnet-go:dev"
Try running ha su repair it will remove all dangling images that take a lot of place

Thanks! I got birdnet-go running. Is there a way to tell it to purge recordings daily or anything? Also, the log and stats both say "nothing here yet" even though it is detecting.  Am I just being impatient? 🙂

update: found the yaml that I can set logs to "true" - will that also be stats?

Thanks! I got birdnet-go running. Is there a way to tell it to purge recordings daily or anything? Also, the log and stats both say "nothing here yet" even though it is detecting. Am I just being impatient? 🙂

I'm not sure I think there is a purge but without much granularity - that's why I'm using birdnet-pi (I'm also developing a feature to keep only the best & latest recordings to avoid having too much data)
For the log and stat I think it's because it is placeholder pages

I started using the birdnet go.  I uninstalled birdnet pi and my home assistant full backup size went from almost 8gigs down to 400k. That seems unrealistic.  Anything that would make that true? I fear my backups are somehow incomplete now?

In a backup you can look in the zip which files take a lot of place. It could be simply the mp3 files from the saved detections, and birdnet go will fill the same in some time

They don’t purge at some point?

When the disk is at 95% capacity :) I'm writing a feature in birdnet pi to define a maximum of recordings to keep per species. For example i had 47k mp3 recordings of blackbird. The objective will be to keep the last 7 days, plus x (fire example 1000) recordings sorted by highest confidence


Cache zoo datasets/models in e2e

What changes are proposed in this pull request?
Avoid rate limiting from zoo datasets in e2e
How is this patch tested? If it is not, please explain why.
e2e
What areas of FiftyOne does this PR affect?

[ ] App: FiftyOne application changes
[x] Build: Build and test infrastructure changes
[ ] Core: Core fiftyone Python library changes
[ ] Documentation: FiftyOne documentation changes
[ ] Other

Might need tweaks, will undraft after confirming the fix


Hmmer.js pan direction is not working as expected

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
We have created a directive to support drag&drop on ionic with the included hammer.js lib.
The logic we are using is:
a) on press start drag behavior and listen to pan events
b) after press and start drag behavior set supported directions to ALL
c) ater drag finished or drop set directions to initial direction
We have to set direction because on ionic default direction for components is only left-right and there is now support of up and down.
This is a snippet of our code
@Directive({
    selector: '[la-draggable]'
})
....
    ngAfterViewInit() {
        this.hammer = new window['Hammer'](this.srcElement);
        this.defaultDirection = this.hammer.get('pan').options.direction;

        this.hammer.on('press', (ev) => {
            this.onPress(ev);
        });

        this.hammer.on('panmove', (ev) => {
            if (this.isDragging) {
                this.onPanMove(ev);
            }
        });
.....
 private onPress(ev) {
        this.hammer.get('pan').set({ direction: window['Hammer'].DIRECTION_ALL });
        // start drag&drop with custom logic
.....

The drag&drop functionallity we implemented is working well, but there is a problem on the DIRECTION_ALL, after setting this in onPress we logged the direction and it is sett well.
The problem is, if first PAN (move) we do is to the left or right anything is fine we can drag it on every point of the screen, and any events pan/panright/panleft/panup/pandown/pressup are notified.
But if we the first PAN (move) we do is to the down or to the up, no event is notified and also moving left right or pressup is never notified from then. So I think this should not a Hammer.js problem but a combination from ionic and hammer.js.
Mainly the thing we do is comming from Josh Morony drag drictive example, you can have a look here
https://www.joshmorony.com/building-an-absolute-drag-directive-in-ionic-2/
Other information:
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
cli packages: (/usr/lib/node_modules)
    @ionic/cli-utils  : 1.12.0
    ionic (Ionic CLI) : 3.12.0

global packages:
    cordova (Cordova CLI) : 7.0.1

local packages:
    @ionic/app-scripts : 3.0.0
    Cordova Platforms  : android 6.2.3 ios 4.4.0
    Ionic Framework    : ionic-angular 3.7.1

System:
    Android SDK Tools : 26.1.1
    Node              : v6.11.3
    npm               : 5.4.2
    OS                : Linux 4.10

Misc:
    backend : legacy

Hello! Thank you for opening an issue with us! Would you be able to provide a sample application via GitHub that demonstrates the issue you are having? Thanks for using Ionic!

Hi @kensodemann,
I created a private repository with a very very simple example, it is available only to you as collaborator.
https://github.com/mburger81/ionic-drag/settings/collaboration
There is a directive called DraggableDirective which is handling our drag behavior.
As you can see I register some events on constructor, and on onPress which is firing after 300ms I start the ability to DRAG an element. As you know ionic has set hammer.js direciton only to X, because on a mobile app there is the scroll behavior fox Y direction.
So first thing we do on drag start is setting DIRECTION_ALL for component. At the end after drag stop we set the default DIRECTION we have saved in constructor before.
So if you run simple ionic serve anything is fine, click 300ms on DIV and drag element over the screen anything is fine.
If you run ionic cordova run android and test the drag behavior on an android device (probably also on ios but we don't support ios for now) after pressing on the DIV the drag behavior is starting but at this point if your first PAN move is for X direction you can from now on drag the DIV all over the screen and it is working well.
But if your first PAN move is for Y direction, the PAN is interrupted without any other event fired and you have a screen like this.

You can see in front the dragged div and in background the original DIV.
Now if you release the screen nothing is happen but if you touch the DIV again and MOVE it you can go forward with the dragging behavior and if you release now the div the drag is stopped
Sorry for my bad English, I hope I was clear enough.
Thx
Michael

Hi @kensodemann was my repository helpful? Where you able to reproduce the issue?

I have not had time to look at this issue yet.

I opened also an issue for this problem on the hammer.js github repository
https://github.com/hammerjs/hammer.js/issues/1164


dont use modelExperimental as default

https://propelleraero.atlassian.net/browse/VN-2046

Are you going to do further investigation of our GLTF authoring code to see if we can support the new Cesium behaviour?

Up to you?

Even if it works it can change anytime
https://cesium.com/learn/cesiumjs/ref-doc/ModelExperimental.html?classFilter=ModelExperimental
Experimental
This feature is using part of the 3D Tiles spec that is not final and is subject to change without Cesium's standard deprecation policy.

lets see if experimental is still regarded experimental  https://community.cesium.com/t/is-modelexperimental-now-offical-v1-92/17685. Above is a straight rollback to the previous version

This will be fixed when we add support for Image-Based Lighting to ModelExperimental. We expect that to happen soon (either the 1.92 or 1.93 release). #9838


Add Teckin SB50 (ESP8466 Light Flashed to ESPhome)

Add model for Teckin SB50 ESP 8466 based lamp flashed to ESPHome

Thanks for the contribution


ENH: Add sinc

Would be nice to include the sinc function from numpy.

As with any of these ufuncs, this sounds good to me.
On Mon, May 8, 2017 at 12:06 PM, jakirkham<EMAIL_ADDRESS>wrote:

Would be nice to include the sinc
https://docs.scipy.org/doc/numpy-1.12.0/reference/generated/numpy.sinc.html#numpy.sinc
function from numpy.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/2311, or mute the thread
https://github.com/notifications/unsubscribe-auth/AASszIHBXsr-IFpnbMJQ-zbusg74bspCks5r3z1qgaJpZM4NUKOE
.

Are there others that we're missing?
On Mon, May 8, 2017 at 12:12 PM, Matthew Rocklin<EMAIL_ADDRESS>wrote:

As with any of these ufuncs, this sounds good to me.
On Mon, May 8, 2017 at 12:06 PM, jakirkham<EMAIL_ADDRESS>wrote:

Would be nice to include the sinc
https://docs.scipy.org/doc/numpy-1.12.0/reference/generated/numpy.sinc.html#numpy.sinc
function from numpy.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/2311, or mute the thread
https://github.com/notifications/unsubscribe-auth/AASszIHBXsr-IFpnbMJQ-zbusg74bspCks5r3z1qgaJpZM4NUKOE
.

Cool.
Yeah, there are a few.
For instance true_divide and floor_divide. Those in particular are handy if one wants to ensure a particular kind of division in Python 2/3 code.
Also negative, power, remainder, mod, but I don't know that any of these is more convenient than just using an operator. Still maybe handy for compatibility with NumPy. Considering absolute already exists in Dask Array, it seems like we are ok adding these.
Then some other miscellaneous functions like exp2, cbrt, and reciprocal, which are sometimes used, but not very often. Also there are other ways to get at these as well.

Presumably these are trivial to implement.  Probably just adding the
appropriate line to ufuncs.py and a corresponding line to tests?
On Mon, May 8, 2017 at 12:40 PM, jakirkham<EMAIL_ADDRESS>wrote:

Cool.
Yeah, there are a few.
For instance true_divide and floor_divide. Those in particular are handy
if one wants to ensure a particular kind of division in Python 2/3 code.
Also negative, power, remainder, mod, but I don't know that any of these
is more convenient than just using an operator. Still maybe handy for
compatibility with NumPy. Considering absolute already exists in Dask
Array, it seems like we are ok adding these.
Then some other miscellaneous functions like exp2, cbrt, and reciprocal,
which are sometimes used, but not very often. Also there are other ways to
get at these as well.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/2311#issuecomment-299920807, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszDXm7tGUzQOZi4l_ew2KtQzVx7h3ks5r30VlgaJpZM4NUKOE
.

I'm not going to lie. I don't have lots of time to see through that level of addition. Though I have tried my best to pull in a bunch of missing ufuncs I found in PR ( https://github.com/dask/dask/pull/2315 ) and include them in the tests. If that looks basically fine or has some minor tweaks needed, that should be ok. However, if it feels like it is going to take a lot of review, I may just break out the sinc function and focus on that as it is of particular value and time is kind of limited ATM.


SIDM-4643 - Allow keyvault module for azurerm v2

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/SIDM-4643
Change description
Allow keyvault module for azurerm v2
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

use this branch https://github.com/hmcts/cnp-jenkins-config/blob/master/terraform-infra-approvals/global.json#L20

That branch does not work.

use this branch https://github.com/hmcts/cnp-jenkins-config/blob/master/terraform-infra-approvals/global.json#L20

That branch does not work. See hmcts/idam-shared-infrastructure@a73a717

use this branch https://github.com/hmcts/cnp-jenkins-config/blob/master/terraform-infra-approvals/global.json#L20

That branch does not work. See hmcts/idam-shared-infrastructure@a73a717

Looks like the build is deleted, can you retry again ? Cannot see why it's not working. It used in other places

That branch does not work. See hmcts/idam-shared-infrastructure@a73a717

Looks like the build is deleted, can you retry again ? Cannot see why it's not working. It used in other places

https://sandbox-build.platform.hmcts.net/job/HMCTS_Sandbox_IDAM/job/idam-shared-infrastructure/view/change-requests/job/PR-50/48/console Failed.


3 strings missing in translations/{lang}.json

Is your feature request related to a problem? Please describe.
While testing my Czech translation I noticed a few missing strings.
Describe the solution you'd like
Could the following strings be added to the translation JSONs and wired to the flow in order to get the integration translated?
"required": "Required",
"success": "Success!",
"devicesFound": "We found the following devices:"
Additional context
This will provide better UX to non-English speakers.

Hello @ertechdesign ,
I can't figure out where these strings should be placed since, by the look of it they are 'native' string keys used by HA frontend and should be already localized.
I'm not totally aware of how these string translations mixups between the custom code provided and the internal ones used by the frontend (or core) itself.
If I get it right anyway these strings are:

"required" shown by HA config flow for a field which needs to be filled in UI
"success" the final message form showing you a completed config flow
"devicesFound" I'm guessing this is used when discovery shows you new devices to configure
If you can confirm this is where they're shown/used I'll maybe try to better understand how and where HA gets those translated but, at the moment, I'm guessing these should be part of the HA core/frontend pack of translations (and maybe the CZ translation is missing or incomplete for these - reference here )

Hello @krahabb ,
You might be right here. I never used the Czech version of Home Assistant and only seen it first time when I smoke tested my commit so I'm not aware of any untranslated strings in the main framework. I'd like to give a Raspberry with Home Assistant to my parents who are Czechs and don't speak any English so I'm excited to make a little contribution here. 😄
Thank you very much for checking this.


DragDrop: move a div inside another div without dragging it outside the parent div

Bug:
What is the expected behavior?
New DragDrop: I want to move an DIV inside another DIV without dragging it outside the parent DIV
What is the current behavior?
cdkDrag allows to move the DIV within the complete document.
What are the steps to reproduce?

What is the use-case or motivation for changing an existing behavior?
this is the simple basic drag and drop use-case
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
7.02

Will be fixed by https://github.com/angular/material2/pull/13820.


[Feature]: translate text into multiple languages

👉 Please confirm the following

[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I checked the current issues for duplicate features.

🤔 Is your proposal related to a problem?
No response
😍 Describe the solution you'd like
In the menu, it would be nice to have a drop-down list to select a list of different languages, and all the game text should be updated.
👍 Describe alternatives you've considered
No response
✍ Additional context
No response

Hi, I would like to try solving this issue , thanks

Hey @PrethamMuthappa, no problem you can work on it if you find things to implement ! I will assign you on the issue.

Heyy, was busy for a while couldn't look into it much , I wanted to ask are u comfortable with using Google cloud translation API for text translation, I found a way where u can use the google cloud translate java client library to translate text

Heyy @PrethamMuthappa, no problem, no need to rush :)
I am not a pro of Google Cloud translation API, but I think it is going to not be free. The solution would be to translate everything ourselves, like with a .json file like (en.json for english, fr.json for french and so on), then when the user clicks on a language, it automatically load the json to translate.
For example:
{
  "key": "This is a wonderful day!",
  "nested": {
    "key": "How are you?"

}

And then we could make the translation very very easily, with this website: https://translate.i18next.com/

@jvondermarck  ooohhh i see, cool ill take a look into it :)

Hey @jvondermarck, It this issue still open?


Upgrading dependencies

Upgrading to react testing library 14

Thanks @CharlineCouchot !
I will upload a new version of the package asap !

I've uploaded a new package version msal-react-tester
It should work now with your testing library.
I tried to update the source code to be able to use it with any version of the testing library.
Let me know


Fail to install [ERROR] Exit code: 0

Unfortunately, when I install it I get the following error: Fail to install [ERROR] Exit code: 0
Greetings Mafel

click on Code => Downlaod zip, and don't use the release. @ohmcodes forgot to recreate the release after a bugfix(satisfactory renamed their binary 2 months ago)


🛠 [TASK] : Treasury Space MVE1 Mocked UI

Summary
Mocked Treasury Space Screen
Description
Development of Treasury Space screen

Figma

Part of #726

Waiting for #759 and #758 to finish this one


[llamaindex]: Failed to log trace tree to W&B: <EventPayload.RESPONSE: 'response'>

Describe the bug
I'm trying to use the llamaindex WandbCallbackHandler integration for a simple QA application. However when I run a query I expect a trace to be logged corresponding to the question, instead I get the following error
Failed to log trace tree to W&B: <EventPayload.RESPONSE: 'response'>

I'm using the following colab notebook to run my experiments, and the error trace is visible in the following public run
Additional Files
No response
Environment
WandB version: 0.17.1
OS: Linux-6.1.85+-x86_64-with-glibc2.35
Python version: 3.10.12
Additional Context
No response

Figured it out thanks to @ayulockin and @soumik12345 🙇🏽  !
The solution was to run weave.init()


Not working in the Release mode

It seems that video capture is not working in release mode. Any ideas on what's causing it?

Can you please try with the new version (1.4.0)


forbidden mvrewrite when there is no mv index in legacy planner

Problem:
In legacy planner mode, when do select request from a table with rollup index; It will do mv rewrite operation, this will case selectMaterializedView fail and do re analyze again with resetAnalyzerAndStmt;
Solution:
when do mv rewrite judgement，add mv index check when the tables index number > 1；if there is no mv index, do not do mv rewrite.

run buildall

run buildall

run buildall


system_peers not getting latest remote data

system_peers rpc method did not return the real-time remote node data, data seems to refresh after restart local node.
 curl -H 'Content-Type: application/json' -d '{ "jsonrpc": "2.0", "method":"system_peers", "params":[], "id": 1 }' http://localhost:9933/ | jq '.'

@jacogr can you confirm this?

I am not seeing the same on https://polkadot.js.org/apps/#/explorer/node (via the RPC node)
It does update and the nodes drop there - the connections seem relatively unstable there, so each 10s refresh shows a different number which aligns with what is on telemetry

(the node info UI doesn’t work on mobile as I can’t scroll horizontally)
The real issue is the height of peers is not accurate compare to value displayed in telemetry

curl -sS -H 'Content-Type: application/json' -d '{ "jsonrpc": "2.0", "method":"system_peers", "params":[], "id": 1 }' http://localhost:9933/ {"jsonrpc":"2.0","error":{"code":-32601,"message":"Method not found"},"id":1}


Fixed a typo in the Excel function call

Hello,
Actually this isn't a typo. The usage of the , or ; character to separate function arguments in Excel depends on your PC's region and language settings. In US English Excel the , is used. Could you update your PR to make a note of this rather than changing the formula?
Thanks for the PR, regards,
Eric.

I didn't know excel's changes separation syntax depending on your system's currency settings.
That sounds silly even for them :) Anyway updated the text to account for both of the versions.


Removed the secret keys.

I removed your Firebase API Key and some other information in firebaseConfig.js. You really should just add the file itself to your .gitignore file. Sharing this type of information can lead to trouble if others use them.

Thanks! Ignored the config and swapped out the keys. Firebase uses client-side keys so they're going to be exposed no matter what, but it's probably best practice to at least omit them from the repo
https://github.com/csaye/meetingbrew/commit/5d81180c1ada26c2f379fd5ef0b5da8ed119514d


Promise support?

@edwardhotchkiss would be nice, if this plugin could also return a promise

@tsm91 Right now it is using a callback. I'd obviously prefer a promise myself but that is probably a breaking change. Feel free to do a PR.


Update docker hacking information

Take 3 :) This is a follow-on from #649
NB: There are other links to travis in the HACKING.md file, but I don't know enough about this to fix it. So this is still needing to be updated. I've made an issue item about this to be tidied up
Summary
Update a few review points leftover that were added by aslak after previous PR was merged
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).

Checklist:

[ ] The change has been ported to Java.
[ ] The change has been ported to Ruby.
[ ] The change has been ported to JavaScript.
[ ] The change has been ported to Go.
[ ] The change has been ported to .NET.
[ ] I've added tests for my code.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Is this good to be merged? It's a slight improvement again...
@aslakhellesoy / @mlvandijk

I'm gonna merge this in. We can tidy things up later if needs be again. It's a slight improvement.


Minimize app with "save to icloud opened", after maximizing keyboard does not open

Description
After entering a password field, an IOS modal "save password in icloud" is shown, minimizing the app with this modal open after maximizing no text entry field opens the keyboard.
React Native Version
0.71.11
Output of npx react-native info
System:
OS: macOS 13.6
CPU: (8) x64 Intel(R) Core(TM) i5-1038NG7 CPU @ 2.00GHz
Memory: 157.42 MB / 16.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 18.17.1 - /usr/local/bin/node
Yarn: 1.22.18 - /usr/local/bin/yarn
npm: 9.6.7 - /usr/local/bin/npm
Watchman: Not Found
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 23.0, iOS 17.0, macOS 14.0, tvOS 17.0, watchOS 10.0
Android SDK: Not Found
IDEs:
Android Studio: 2022.2 AI-222.4459.24.2221.10121639
Xcode: 15.0.1/15A507 - /usr/bin/xcodebuild
Languages:
Java: 11.0.15 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.2.0 => 18.2.0
react-native: 0.71.11 => 0.71.11
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
-Create a screen with a password input
-After submit form, a modal asking about save password to iCloud is displayed.

-Keep this modal opened and minimize app
-After that, maximize the app and no other text input fields will open the iPhone keyboard.
Snack, screenshot, or link to a repository

@edersonfaccin is there any update? I'm having the same problem.

so @okaygenc unfortunately i didn't solve this problem yet


New Data to Admin Resource Error

Entering a new entry to the Admin Resource shows an error bar without a description.
https://ibb.co/kx2XjfM
This happens when I select from the Action --> [Save Submission] to save submission to "Admin". When it's empty it works.
https://ibb.co/4V7WgnD
Is that normal behavior? As far as I understand the admins should be saved to Admin resource, Users to Users resource, and so on.
Thanx

We're currently addressing a backlog of GitHub issues. Closing this thread as it is outdated. Please re-open if it is still relevant. Thank you for your contribution!


feat(plugin-chart-table): disable show_totals by default

🐛 Bug Fix
Show totals adds an additional SQL query to the request, which slows things down. Since it's not a critical feature, we shouldn't enable it by default.

@ktmud it IS a critical feature in table :)  but I agree that showing total should be a user choice, since we have not had a conclusion what to do with the non-additive columns. and perf is def important.

@junlincc
Since this is a database calculate aggregate, there are many database optimization strategies here rather than on the BI side.

No it is not.

Tableau also has totals disabled by default, which I think makes sense. In general I like the idea that designing charts we want to incrementally add stuff and only prepopulate the options that are needed in the overwhelming majority of cases. I think totals are often not needed, hence would require an extra step for someone who only needs a simpler chart (which is counter intuitive).


remove extra ext from android gradle versions references as rootProje…

Remove extra ext from android gradle versions references, as rootProject already references ext directly.

@DanielRaouf Are you sure this should be fixed? There are serveral other modules that search for ext generated from react-native app.

@DanielRaouf Hope you can comeback and review this.
Hi Hychan, I don't know why ext does not work for me. I tried on two different projects and it only works after removing ext. However android guide do state adding the ext. Please check and if it works good for you just forget about the PR

@DanielRaouf What react-native version are you using?


[FEAT] Implement Organization Deletion #126

Description

Created an API endpoint to handle the deletion of an organization. This endpoint validate the organization ID and update the specified organization deleted attribute to true in the database upon successful validation.

Related Issue (Link to Github issue)

Implement Organization Deletion

Endpoints
/api/v1/organizations/{org_id}
Motivation and Context

It provides a backend service to handle the deletion of organizations, ensuring the organization ID is valid and setting the oganization deleted attribute to true in the database.

How Has This Been Tested?

the Api has been tested using php units testing and it passed the 3 test it under went.

Screenshots (if appropriate - Postman, etc):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue) -->
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Update and fix merge conflict

You have merge conflicts, fix them