State Listeners

What did you fix?
Implements the state listeners for SingleQueryModel and TimedEvent types.

Reproduction steps

Evidence/screenshot/link to line

Considerations

Does this contain a new dependency? [No]
Does this introduce opinionated data formatting or manual data entry? [No]
Does this pr include updated data files in a separate commit that can be reverted for a clean code-only pr? [Yes]
Have I run the linter? [No]
Is is a bug fix, feature request, or enhancement? [Feature/Docs]

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

v0.4.0
GitHub release

Your semantic-release bot :package::rocket:


[BUG]: View config not found for component RCTVideo

Version
6.7.0
What platforms are you having the problem on?
Android
System Version
Android 14
On what device are you experiencing the issue?
Simulator
Architecture
Old architecture
What happened?
The bug occurs when rendering the component with Video and only on Android. It works perfectly on iOS!
I'm on RN 0.75.3, JS.
My Package.json file
{
  "name": "awesomeProject",
  "version": "1.0.0,
  "dependencies": {
    "@babel/plugin-proposal-private-property-in-object": "^7.21.11",
    "@babel/plugin-transform-class-properties": "^7.25.4",
    "@georstat/react-native-image-cache": "^3.1.0",
    "@gorhom/bottom-sheet": "^4.6.4",
    "@native-html/iframe-plugin": "^2.6.1",
    "@native-html/table-plugin": "^5.3.1",
    "@notifee/react-native": "^9.0.2",
    "@react-native-async-storage/async-storage": "^2.0.0",
    "@react-native-camera-roll/camera-roll": "^7.8.3",
    "@react-native-community/checkbox": "^0.5.17",
    "@react-native-community/netinfo": "^11.4.1",
    "@react-native-community/push-notification-ios": "^1.11.0",
    "@react-native-community/viewpager": "^5.0.11",
    "@react-native-firebase/analytics": "^21.0.0",
    "@react-native-firebase/app": "^21.0.0",
    "@react-native-firebase/dynamic-links": "^21.0.0",
    "@react-native-firebase/messaging": "^21.0.0",
    "@react-native-picker/picker": "^2.8.1",
    "@react-native/assets-registry": "^0.75.3",
    "@realm/react": "^0.11.0",
    "axios": "^1.7.7",
    "buffer": "^6.0.3",
    "events": "^3.3.0",
    "fbjs": "^3.0.5",
    "formik": "^2.4.6",
    "libphonenumber-js": "^1.11.9",
    "mobx": "^6.13.3",
    "mobx-react": "^9.1.1",
    "moment": "^2.30.1",
    "react": "18.3.1",
    "react-native": "0.75.3",
    "react-native-android-badge": "^0.5.0",
    "react-native-autoheight-webview": "^1.6.5",
    "react-native-base64": "^0.2.1",
    "react-native-blob-util": "^0.19.11",
    "react-native-circular-progress": "^1.4.0",
    "react-native-code-push": "^9.0.0",
    "react-native-contacts": "^8.0.2",
    "react-native-date-picker": "^5.0.7",
    "react-native-device-info": "^13.0.0",
    "react-native-document-picker": "^9.3.1",
    "react-native-email-link": "^1.16.1",
    "react-native-fast-image": "^8.6.3",
    "react-native-file-access": "^3.1.1",
    "react-native-file-viewer": "^2.1.5",
    "react-native-fs": "^2.20.0",
    "react-native-gesture-handler": "^2.20.0",
    "react-native-google-api-availability-bridge": "^1.3.0",
    "react-native-image-picker": "^7.1.2",
    "react-native-inappbrowser-reborn": "^3.7.0",
    "react-native-intersection-observer": "^0.2.0",
    "react-native-keyboard-aware-scroll-view": "^0.9.5",
    "react-native-masked-text": "^1.12.5",
    "react-native-mime-types": "^2.5.0",
    "react-native-modal": "^13.0.1",
    "react-native-month-picker": "^1.0.1",
    "react-native-navigation": "^7.40.1",
    "react-native-navigation-hooks": "^6.3.0",
    "react-native-network-logger": "^1.17.0",
    "react-native-paper": "^5.12.5",
    "react-native-permissions": "^4.1.5",
    "react-native-picker-select": "^9.3.1",
    "react-native-public-ip": "^1.0.2",
    "react-native-push-notification": "^8.1.1",
    "react-native-qrcode-svg": "^6.3.2",
    "react-native-reanimated": "3.15.3",
    "react-native-render-html": "^6.3.4",
    "react-native-safe-area-context": "^4.11.0",
    "react-native-scalable-image": "^1.1.0",
    "react-native-screens": "^3.34.0",
    "react-native-send-intent": "^1.3.0",
    "react-native-share": "^11.0.3",
    "react-native-sms": "^1.11.0",
    "react-native-svg": "^15.7.1",
    "react-native-swipe-list-view": "^3.2.9",
    "react-native-swiper-flatlist": "^3.2.5",
    "react-native-url-polyfill": "^2.0.0",
    "react-native-vector-icons": "^10.2.0",
    "react-native-video": "^6.7.0",
    "react-native-webview": "^13.12.2",
    "react-native-webview-ios-cache-clear": "^1.0.0",
    "realm": "^12.13.1",
    "sp-react-native-in-app-updates": "^1.4.0",
    "truncate-html": "^1.1.2",
    "yup": "^1.4.0"
  },
  "devDependencies": {
    "@babel/core": "^7.20.0",
    "@babel/plugin-proposal-decorators": "^7.25.7",
    "@babel/plugin-proposal-private-property-in-object": "^7.21.11",
    "@babel/plugin-transform-class-properties": "^7.25.4",
    "@babel/preset-env": "^7.20.0",
    "@babel/runtime": "^7.20.0",
    "@react-native/babel-preset": "0.75.3",
    "@react-native/eslint-config": "0.75.3",
    "@react-native/metro-config": "0.75.3",
    "@types/react": "^18.2.6",
    "@types/react-test-renderer": "^18.0.0",
    "babel-jest": "^29.6.3",
    "babel-plugin-module-resolver": "^5.0.2",
    "eslint": "^8.19.0",
    "jest": "^29.6.3",
    "patch-package": "^7.0.0",
    "prettier": "2.8.8",
    "react-native-push-notification": "^8.1.1",
    "react-native-svg-transformer": "^1.5.0",
    "react-test-renderer": "18.3.1"
  },
  "jest": {
    "preset": "react-native"
  },
  "engines": {
    "node": ">=18"
  },
  "packageManager"<EMAIL_ADDRESS>}

Reproduction Link
No response
Reproduction
Step to reproduce this bug are:

Getting same in running yarn run android in the expo example.

I confirm this issue @paulrinaldi
@KrzysztofMoch I think this should be solved by using following doc: https://docs.expo.dev/modules/autolinking/
But monorepo should be better to apply it :/ let me know your expectations

@rastepanenko I don't think you have the same issue than the one reported by paul.
Do you have the expo go issue ? please check other closed tickets which deals with the same issue

@freeboub I'm not using expo in this project. This is pure RN

@paulrinaldi Were you able to solve this problem?

@rastepanenko The issue with the sample are different as the RNV code is directly linked without autolink.
Can you recreate your issue in a very basic sample you can share ?

I confirm this issue @paulrinaldi @KrzysztofMoch I think this should be solved by using following doc: https://docs.expo.dev/modules/autolinking/ But monorepo should be better to apply it :/ let me know your expectations

Thank you, I followed the steps linked there plus my intuition and was not able to fix this problem.
My expectations are that one can run yarn run install && yarn run android in the examples/expo directory without needing to run anything else.
I am suspicious are that there is no declaration of react-native-video (even linking to like ../../src or whatever was there before) in the expo example.
Steps:
I ran the command: npx expo-modules-autolinking search
In the ouput, I see there are 6 packages that are not in the package.json but are declared in the Podfile.lock and do exist in the node_modules folder. In the ouput, I also see two packages, "expo" and "expo-splash-screen", which are in the package.json but of different versions in the output than in the package.json). I haven't come across this before linking.
Running npx expo-doctor in this expo directory yields the following failure:

...
✖ Check for app config fields that may not be synced in a non-CNG project
✔ Check Expo config (app.json/ app.config.js) schema
✖ Check that packages match versions required by installed Expo SDK
...

Detailed check results:

This project contains native project folders but also has native configuration properties in app.json, indicating it is configured to use Prebuild. When the android/ios folders are present, if you don't run prebuild in your build pipeline, the following properties will not be synced: plugins, android, ios.

Advice: Add '/android' and '/ios' to your .gitignore file if you intend to use CNG / Prebuild. Learn more: https://docs.expo.dev/workflow/prebuild/#usage-with-eas-build

(node:5782) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
The following packages should be updated for best compatibility with the installed expo version:
<EMAIL_ADDRESS>- expected version: 2.7.5
Your project may not work correctly until you install the expected versions of the packages.
Found outdated dependencies
Advice: Use 'npx expo install --check' to review and upgrade your dependencies.

One or more checks failed, indicating possible issues with the project.

Running 'npx expo install --check' changed the package.json's version of react native picker but  after this, running yarn run android, still yields the error in this issue.
I ran npx expo-modules-autolinking resolve --platform apple and npx expo-modules-autolinking resolve --platform android and after each I ran npx expo-modules-autolinking verify which outputted "Everything is fine!"
Using yarn run start + a yields same issue as well.

Also is the example/expo project for testing tvOS or android phone/tablet?
I see, "react-native"<EMAIL_ADDRESS>which confuses me.

Also is the example/expo project for testing tvOS or android phone/tablet?
I see, "react-native"<EMAIL_ADDRESS>which confuses me.

this is for tvOS support... tvOS is not separated package as visionsOS so this is only way to do it (I have do it as expo docs says to do it)
I am little busy right now, but I will try to debug it on my side and get back to you - Also thanks for sharing you research

@paul-rinaldi I fixed it in the expo sample !
As we don't have any news from @rastepanenko let's close this issue


seq seems to be missing

I have downloaded sqlean for Windows and loaded extensions into DB Browser for SQLite
I have been trying some of the test scripts.
In stats some function names seem to differ from the examples and function seq or _seq or stats_seq is not listed if I execute PRAGMA function_list;
Have I missed something?
Thanks
Ted

I think seq==https://sqlpkg.org/all/

I think seq == generate_series which I located  at https://sqlpkg.org/all/
I downloaded  series.dll and loaded into DB Browser for SQLite and was able to run the test script:
`select '1_01',  percentile(value, 25) = 25.5 from  generate_series(1, 99);
with tmp as (select * from  generate_series(20) limit 10)
select '4_03', (count(*), min(value), max(value)) = (10, 20, 29) from tmp;
`
Is this as expected?
Cheers
Ted

Thanks for the hint.
DB Browser for SQLite has a load extension option. I used that to load the sqlean composite extension.
I will try that with DB Browser for SQLite and see what I get.
Ted


Bug: BR_SetMediaItemEdges should keep envelope points in place

Hi,
The ReaScript function BR_SetMediaItemEdges keep envelope points position relative to the item left edges, instead as leaving them at their absolute position in the project (the whole point of this function is to modify items and by preserving absolute position of the content, so envelope points should move).
Steps:

Have an item
Create an envelope
Add points
use the SetMediaEdges function
see that envelope has been offset.

Thanks for taking a look :)

Some Lua code that may help
function OffsetTakesEnvelopes(take, offset)
  local env_count = reaper.CountTakeEnvelopes(take)
  local take_rate = reaper.GetMediaItemTakeInfo_Value( take, "D_PLAYRATE" )
  for i = 0, env_count - 1 do
      local env = reaper.GetTakeEnvelope(take, i)
      local env_points = reaper.CountEnvelopePoints( env )
      for j = 0, env_points - 1 do
        local retval, time, value, shape, tension, selected = reaper.GetEnvelopePoint( env, j )
        reaper.SetEnvelopePoint( env, j, time + offset*take_rate, value, shape, tension, selected, true )
      end
    reaper.Envelope_SortPoints(env)
  end
end

Same with "SWS/BR: Trim MIDI item to active content" which uses the same code as BR_SetItemEdges. The code is also used for some tempo-related actions, not sure if moving envelope points would affect those.
The code in question:
https://github.com/reaper-oss/sws/blob/b3ad441575f075d9232872f7873a6af83bb9de61/Breeder/BR_Util.cpp#L1081

Potential fix for testing (Win 64):
https://www.dropbox.com/s/6no74lzkbu13pk8/reaper_sws64.dll?dl=1
Should hopefully fix both, BR_SetItemEdges() and "SWS/BR: Trim MIDI item to active content".
Testing and feedback appreciated, if it works I'll do a PR.

@nofishonfriday Tested with various item and playrates and it seems to work !!

Many thanks !

Nice, thanks for the quick feedback.
While you're at it, could you also test if it still works correctly if takes contain stretch markers (with rate other than 1.0) ?
(I have a fear it could break then, I didn't dare to test myself yet :P)

Previous version did break with SM's here, as I thought.
Hopefully also fixed in new version, please test again (with SM's):
https://www.dropbox.com/s/bsh18aqg0bjs3q8/reaper_sws64.dll?dl=1

@nofishonfriday
It works in most case, but there is some issues:

(item on top would be the expected behavior, it is the result we have if we extend items manually)
Here is a demo project.
https://www.dropbox.com/s/pm7nqcqoqh1gmd1/BrSetItemEdges.zip?dl=0
reaper.Undo_BeginBlock()
item = reaper.GetSelectedMediaItem(0,0)
if not item then item = reaper.GetMediaItem(0,0) end
reaper.BR_SetItemEdges(item, 10,30)
reaper.Undo_EndBlock("haha", -1)```

Also, there is some minor problem with Snap Offset in very particular cases

By hand with click and drag, snap offset stay in place.
In this case (item has some sort of incomplete pre-source extension), snap offset isn't preserve at the right absolute position.

@nofishonfriday
I asked to Stretch Marker master MPL to take a look and he got pretty successful results !! 👍

reaper.Undo_BeginBlock()

item = reaper.GetSelectedMediaItem(0,0)

if not item then item = reaper.GetMediaItem(0,0) end

take = reaper.GetActiveTake(item)

pos1 = 0

pos2 = reaper.GetMediaItemInfo_Value( item, 'D_LENGTH' )* reaper.GetMediaItemTakeInfo_Value( take, 'D_PLAYRATE' )

reaper.SetTakeStretchMarker( take, -1, pos1 )

reaper.SetTakeStretchMarker( take, -1, pos2 )

it_pos = reaper.GetMediaItemInfo_Value( item, 'D_POSITION' )

t = {} for idx = 1,  reaper.GetTakeNumStretchMarkers( take ) do

  local ID = #t+1

  t[ID] = {}

  _, t[ID].pos, t[ID].srcpos = reaper.GetTakeStretchMarker( take, idx -1)

  t[ID].slope = reaper.GetTakeStretchMarkerSlope( take, idx -1)

end

reaper.DeleteTakeStretchMarkers( take, 0, 10000000 )

reaper.BR_SetItemEdges(item, 10,30)

it_pos2 = reaper.GetMediaItemInfo_Value( item, 'D_POSITION' )

for i = 1, #t do

  local idx = reaper.SetTakeStretchMarker( take, -1, t[i].pos+(it_pos-it_pos2)* reaper.GetMediaItemTakeInfo_Value( take, 'D_PLAYRATE' ), t[i].srcpos )

  reaper.SetTakeStretchMarkerSlope( take, idx, t[i].slope )

end

reaper.Undo_EndBlock("haha", -1)

reaper.UpdateArrange()```

Very close to what we get with manual extension (tough, the audio doesn't loop here before the stretch markers for some very weird reason. But the solutions should be close to MPL code).

(thanks again to MPL !!)

@nofishonfriday Oh Man, I found a very quick way to do it !!
demo
The SetMediaItem edges could be rewritten just with edit cursor, and these trim actions. !!

The BR_SetItemEdges could be rewritten just with edit cursor, and these trim actions !!

Doh, yes, this seems to work in this case. Needs to be careful though not to break the other functions which are using this code. I'll get back with a new test version soon.
Thanks for asking the stretch marker master (I had the same idea :D), and thanks mpl himself for providing code example.

New version fore testing:
https://www.dropbox.com/s/qrviacjl553dk6v/reaper_sws64.dll?dl=1
BR_SetItemEdges() now uses Trim... actions (instead of API).
If other @devs are following:
This is the code I have currently, thoughts / suggestions appreciated.

@nofishonfriday seems to work perfectly !

Thanks for testing.

@nofishonfriday Hi ! There is a bug in the function, it works on every selected items instead of just the one which is pass as argument.
This is supposed to work on only one item:
item_1 = reaper.GetSelectedMediaItem(0,0)
reaper.BR_SetItemEdges( item_1, 2,4)

But it works on every selected items:

Tested with SWS 2.9.8.
Can you reproduce ?

Oops, yeah, I see why this happens. Because the now used native actions work on selected items, stupid from me not thinking about this.
So I think it means having to fiddle with stretch markers to properly fix which I suck at (as seen in previous fix attempt). I'll try but if anyone from the other contributers wants to to tackle this I'm not unhappy. :)
Anyway, the current 'fix' needs to be reverted.

@nofishonfriday you just need to handle unselection items then reselect them within the function :)
It is fast enough to do a good enough solution. And it should be straightfoward to implement, in reascript it is just 4 new lines . :P

@nofishonfriday Here is my current workarround on script side:
	for i, item in ipairs( sel_items ) do

		-- Workarround SWS BR_SetItemEdges Bug in v2.9.8
		reaper.SelectAllMediaItems(0, false)
		reaper.SetMediaItemSelected(item.item, true)

		reaper.BR_SetItemEdges( item.item, trim_pos, trim_end)

	end

	-- Workarround SWS BR_SetItemEdges Bug in v2.9.8
	for i, item in ipairs( sel_items ) do
		reaper.SetMediaItemSelected(item.item, true)
	end

It could be implemented in the API function itself.

Yeah I already thought about doing it this way. :D
I dunno, seems a little dirty somehow, but ok I guess (unless no one from the other devs chimes in).

@nofishonfriday it doesn't look extra efficient, but it has the great advantage of being 'simple to deploy' and backward compatible.
Most of all, it is friendly with all of our concerns (stretch markers etc...).
It is not even said that having to deal with this properly (by making calculation on stretch markers rather than saving items selection) would be that more efficient in performance, so I guess it is fully ok to do this trick 😁

@nofishonfriday All good ! I hope it could merge soon cause I have quite some scripts using this API function.


Remove "Libraries used" section

Objective

The libraries used section of the README is outdated and is missing a lot of libraries
The README isn't really the place for that anyway

Solution

Remove the section

I agree on that, but should it be blocked on cart since it's a very historical piece of the readme?

I don't think it should be blocked on him here. This is a straightforward docs change that improves UX, and the simple fact that it's old (or in the README) shouldn't impact this. That said, ping @cart for clarity?

This isn't just doc, it's also a shoutout to foundational libraries for Bevy.
While spirv-reflect isn't used anymore, it's very unlikely that we will stop using the others. Naga could be added as it's one of the big bricks that allows cross platform

I do like that we shout out foundational libraries. Things like wgpu / naga / winit / glam are central to bevy's success and the devs of those projects deserve high visibility recognition. I think in the short term we shouldn't remove the section, but instead just remove spirv-reflect.
We can discuss alternative ways to boost projects in this category, but I don't want to remove this until we have some other form of recognition in place.

Sounds good. My preference would be on the Bevy website, ideally in the same sort of place we talk about sponsors.

just remove spirv-reflect.

Done!

bors r+


Price history interpolates strangely between events

The price history view seems to interpolate between prices with sigmoid functions:

I'm pretty sure the price didn't actually develop the way the history displays. A hot-fix would be to interpolate by using the last known value (this will give you a step function instead). In a perfect world, I guess we would be using candle sticks instead?

Doesn't seem like an option in our chart library https://recharts.org/en-US/api/Line

Actually I think it's this: https://recharts.org/en-US/api/Line#type

@maltekliemann which one do you think it should be out of these? https://github.com/d3/d3-shape#curves

Oh, nice. I think it should be stepAfter.

Will be addressed in the new chart revisions


Toolbars are not positioned correctly

Describe the bug
Toolbars are not positioned correctly
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS] windows 10 version 2004
Version [e.g. 22] 1.5

Additional context
The toolbar used is: NetSpeedMonitor and I have TranslucentTB running to make the taskbar transparent.

Those toolbars are not part of the normal UI stack and not part of the button grouping that is centered and cannot be repositioned

TaskbarX which is a similar tool written in VisualBasic, had this issue a while ago and They managed to fix it.
I would be glad to help but my understanding of TaskbarX code base is not good at all and I don't even know VisualBasic.
I moved from TaskbarX to your app because your app is simple and not bloated. Good job man.
If I ever find a solution I will make sure to submit a PR.


Postquantum

Created pull request for sharing purposes

Hi @brett-warren-arm, thanks! Can you add a brief description of what's going on the in the PR?

As described in the commit message:
SSUE: liboqs currently has to be bulit first manually:
ninja liboqs
(then whatever commands you desired after)

This is because liboqs.a has to exist, otherwise cmake will
complain about it not existing.
To elaborate:
This is where the library is linked,
    if(TARGET liboqs)
        add_dependencies(${mbedcrypto_target} PUBLIC liboqs)
	target_link_libraries(${mbedcrypto_target} PUBLIC ${CMAKE_BINARY_DIR}/lib/liboqs.a)
    endif()

cmake complains about liboqs.a not existing, before the liboqs project can be built. I don't know
if there's another way of expressing this so that liboqs.a's existence is checked later, so perhaps
so other way of making this behave correctly.

Adding the liboqs.a library alone will not be enough to add support for PQC algorithms to the TLS 1.3 implementation. Do you agree?

Yes, a new module will need to be added to mbedtls to integrate the PQC algorithms into the key schedule.

@hannestschofenig We're meeting once a week to discuss the steps ahead - do you want to join?

Commits mitigating the two above issues have been pushed, so this PR should now be completely ready


[Suggestion] Option to show all adopted trees in the map

see subject.
Currently, there is no option to actually show only (or to show in a different colour) the adopted trees. I regard this as an essential feature, my dear friends.

The colour intensity could also indicate a kind of priority (or emergency) for manual watering.

Hi @Wikinaut. Thx for sharing this issue with us. There is a way to see only the adopted trees. I made a quick screencast for you to see how:
https://user-images.githubusercontent.com/2759340/176192045-9eca465e-3f08-42fd-9490-fad6a085557b.mov
I hope this helps!


Bug: CARP not working on second WAN interface

From https://forum.opnsense.org/index.php?topic=5765.0:
Experimenting, I'm getting a contradictory result. On the first WAN interface, I've added two CARP IPs, both using the same VHID number. On the second WAN interface, I've added one CARP IP, using a second VHID number. But when I try to add a second CARP IP, OPNsense complains:
 The following input errors were detected:
 *    VHID 12 is already in use on interface WAN2. Pick a unique number on this interface.

In the CARP widget in the lobby, it clearly shows the first two IPs as "WAN1@11 > MASTER ", and the second as "WAN2@12 ", where 11 and 12 are the CARP VHIDs. Note the second interface's IP isn't showing it as master either. What difference is allowing multiple CARP IPs with the same VHID on one WAN interface, and blocking it from being set up that way on the other?
Also, WAN2 is not sending out any CARP packets. It's receiving some with another VHID from another system on that interface. But it's not sending out any of its own on WAN2, not with VHID12 or anything else. Also, it has not added the one CARP IP to its interface.
Looking with ifconfig, I see that interface igb1 (WAN1) has  promiscuous mode enabled, igb2 (WAN2) does not. Why would this be? Could it be related to the problems?

I think we can drop the constraint which only allows the same vhid to be assigned on one interface, I just need to do some final check there.
I removed the validation and saved on my end, which seems to result it the correct aliases:
	inet <IP_ADDRESS> netmask 0xffffff00 broadcast <IP_ADDRESS> vhid 1
	inet <IP_ADDRESS> netmask 0xffffff00 broadcast <IP_ADDRESS> vhid 1
	inet <IP_ADDRESS> netmask 0xffffff00 broadcast <IP_ADDRESS> vhid 1

Which looks the same as adding one manually using
ifconfig em2 alias <IP_ADDRESS> netmask <IP_ADDRESS> vhid 1

Multiple IPs per VHID would be very great. Drops down the noisy Multicast

I think we better unlock the possibility to add an optional vip to an ipalias as well and check if there actually is a carp vip configured on the interface when set, that way you can easily add more and we keep complexity low.
If we remove the validation we would trigger the init (https://github.com/opnsense/core/blob/17.7/src/etc/inc/interfaces.inc#L1871) multiple times, which isn't a very bright idea.

@mimugmail in case you have the time and spirit to do some testing, https://github.com/opnsense/core/commit/8ae34afbba9bf24baaad37cf2edf96d570070a53 should add support for a  vhid on an ipalias.

I'm back from vacation on 4th of September and will so some testing of not already in master :)


64 bit support?

Now that App Store requires 64 bit support, is there someone available for making it?

We really need iOS 64 bit support I'm also using this module in couple of my apps.

Please see https://github.com/appcelerator-modules/ti.imagefactory/releases

Thanks @ingo ingo :)


ISPN-13434 SegmentAwarePublisher should include segment info in Notif…

…ication
https://issues.redhat.com/browse/ISPN-13434

Performance wise it is very marginally faster than 13.0 with these changes included.

Just thought of an issue that when a store is present the new guarantee from LocalPublisherManager#entryPublisher to ensure only a given segment at a time is streamed is probably broken when a non shared store is present. Need to verify.

Updated with some more tests to ensure behavior of LocalPublisherManagerImpl.

Checking out test failures today.

Tests should be better off now, waiting on CI to see results again.

https://ci.infinispan.org/job/Infinispan/job/PR-9663/4/testReport/junit/org.infinispan.scattered.stream/ScatteredStreamIteratorTest/verifyNodeLeavesAfterSendingBackSomeData_SCATTERED_SYNC__tx_false_/ is likely related, otherwise CI is fine.

I am unable to reproduce locally. It looks like this is an issue on main as well https://ci.infinispan.org/job/Infinispan/job/main/217/testReport/junit/org.infinispan.scattered.stream/ScatteredStreamIteratorTest/verifyNodeLeavesAfterSendingBackSomeData_SCATTERED_SYNC__tx_false_/

Here's a branch with some of my suggestions: https://github.com/infinispan/infinispan/compare/main...danberindei:pr_wburns_ISPN-13434_segment_info?expand=1

Here's a branch with some of my suggestions: https://github.com/infinispan/infinispan/compare/main...danberindei:pr_wburns_ISPN-13434_segment_info?expand=1

I pulled in all the changes but the ? super Notification one as that is not something I want in the more public facing API.

Updated to undeprecate LookedUpEntries and squashed commits.

Merged, thanks Will!


Train supervised model

When I run run_sup_example.sh, the code stuck in this step, and only use 2 GPU(I have 4)
You should probably TRAIN this model on a down-stream task to be able to use it for predictions and inference.
Loading cached processed dataset at ./data/csv/default-5c8c01abeb2e7fe5/0.0.0/2960f95a26e85d40ca41a230ac88787f715ee3003edaacb8b1f0891e9f04dda2/cache-45b85a8c6af7082e.arrow
[INFO|trainer.py:441] 2022-03-14 16:47:35,367 >> The following columns in the training set don't have a corresponding argument in BertForCL.forward and have been ignored: .
[INFO|trainer.py:358] 2022-03-14 16:47:35,368 >> Using amp fp16 backend
When I run the run_unsup_example.sh,the error is
RuntimeError: Input tensor at index 3 has invalid shape [14, 14], but expected [14, 17]
When I change the run_unsup_example.sh from python train.py \ to python -m torch.distributed.launch --nproc_per_node $NUM_GPU --master_port $PORT_ID train.py (same as run_sup_example.sh)
The error is :
Traceback (most recent call last):
File "train.py", line 585, in
main()
File "train.py", line 310, in main
datasets = load_dataset(extension, data_files=data_files, cache_dir="./data/")
File "/home/liulei741/anaconda3/lib/python3.8/site-packages/datasets/load.py", line 590, in load_dataset
module_path, hash = prepare_module(
File "/home/liulei741/anaconda3/lib/python3.8/site-packages/datasets/load.py", line 267, in prepare_module
local_path = cached_path(file_path, download_config=download_config)
File "/home/liulei741/anaconda3/lib/python3.8/site-packages/datasets/utils/file_utils.py", line 334, in cached_path
output_path = get_from_cache(
File "/home/liulei741/anaconda3/lib/python3.8/site-packages/datasets/utils/file_utils.py", line 617, in get_from_cache
raise ConnectionError("Couldn't reach {}".format(url))
ConnectionError: Couldn't reach https://raw.githubusercontent.com/huggingface/datasets/1.2.1/datasets/text/text.py
Traceback (most recent call last):
File "train.py", line 585, in
main()
File "train.py", line 310, in main
datasets = load_dataset(extension, data_files=data_files, cache_dir="./data/")
File "/home/liulei741/anaconda3/lib/python3.8/site-packages/datasets/load.py", line 590, in load_dataset
module_path, hash = prepare_module(
File "/home/liulei741/anaconda3/lib/python3.8/site-packages/datasets/load.py", line 267, in prepare_module
local_path = cached_path(file_path, download_config=download_config)
File "/home/liulei741/anaconda3/lib/python3.8/site-packages/datasets/utils/file_utils.py", line 334, in cached_path
output_path = get_from_cache(
File "/home/liulei741/anaconda3/lib/python3.8/site-packages/datasets/utils/file_utils.py", line 617, in get_from_cache
raise ConnectionError("Couldn't reach {}".format(url))
ConnectionError: Couldn't reach https://raw.githubusercontent.com/huggingface/datasets/1.2.1/datasets/text/text.py
Traceback (most recent call last):
File "/home/liulei741/anaconda3/lib/python3.8/runpy.py", line 194, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/home/liulei741/anaconda3/lib/python3.8/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/home/liulei741/anaconda3/lib/python3.8/site-packages/torch/distributed/launch.py", line 260, in
main()
File "/home/liulei741/anaconda3/lib/python3.8/site-packages/torch/distributed/launch.py", line 255, in main
raise subprocess.CalledProcessError(returncode=process.returncode
and stuck in
You should probably TRAIN this model on a down-stream task to be able to use it for predictions and inference.
Loading cached processed dataset at ./data/text/default-f485599cbd14a27e/0.0.0/daf90a707a433ac193b369c8cc1772139bb6cca21a9c7fe83bdd16aad9b9b6ab/cache-347bc3c0aa9f736e.arrow
Loading cached processed dataset at ./data/text/default-f485599cbd14a27e/0.0.0/daf90a707a433ac193b369c8cc1772139bb6cca21a9c7fe83bdd16aad9b9b6ab/cache-347bc3c0aa9f736e.arrow

Hi,
It might be a caching issue with the dataset. Can you try (1) deleting the cache directory as suggested in the error message, or (2) trying single gpu first.

Hi,
It might be a caching issue with the dataset. Can you try (1) deleting the cache directory as suggested in the error message, or (2) trying single gpu first.

hi, thank you for reply
after I add export CUDA_VISIBLE_DEVICES=0 in run_unsup_example.sh, the problem RuntimeError: Input tensor at index 3 has invalid shape [14, 14], but expected [14, 17] fixed, but new error is
requests.exceptions.ProxyError: HTTPSConnectionPool(host='s3.amazonaws.com', port=443): Max retries exceeded with url: /datasets.huggingface.co/datasets/datasets/text/text.py (Caused by ProxyError('Cannot connect to proxy.', OSError('Tunnel connection failed: 503 Service Unavailable')))

/data/csv/

Thank you, (2) has been fixed.
But after deleting the cache directory (default-5c8c01abeb2e7fe5/0.0.0/2960f95a26e85d40ca41a230ac88787f715ee3003edaacb8b1f0891e9f04dda2/),the problem is still there

Hi,
For
RuntimeError: Input tensor at index 3 has invalid shape [14, 14], but expected [14, 17]

Can you provide more information? Like the whole error stack?

In the file "model.py", a parameters (*model_args) of BertForCL init function seems not be used, is it a bug?

did you use the original train.py file provided by the repo? there should be a model_args variable.


chore: upgrade charts

updated chart jxgh/jxboot-helmfile-resources from 1.1.235 to 1.1.236

/hold

/retest

/hold cancel


Rework of the proxy interface

With both cc-proxy, kata-proxy and the no-proxy cases, I have been thinking that our current proxy interface does not meet all the needs for those different cases...
First I think we don't need to return a URL when calling into connect() or a list of URL when calling into register(), only one URL should be returned by start() now that we agreed on the model one proxy per VM. This opinion is based on the fact that a proxy is at a pod/VM level and I don't expect several URLs to be provided to the shim, the same way this shim will connect to only one socket on the VM in case we don't go through a proxy.
Then I think that most of the interface is fine, except sendCmd(). I have been turning the problem upside down, but no way to make all the cases work with this. We should agree on what should be a proxy in order to know what to expect from it. I think a proxy has to be accessed the same way we would access the VM directly, that is through a socket. This is already the case for all our different cases, but the problem is about the way the data are processed and are expected to be received by this proxy or directly by the VM. The most generic way I can think of is by sending an array of bytes through the socket opened through the connect(). This would suggest that connect() would actually return a handler to this socket connection. We should get rid of the sendCmd() because it assumes the agent to know about the kind of proxy that is going to be used, while a pure array of bytes sent through the provided socket would be processed either by the proxy or directly by the agent, transparently for virtcontainers. Indeed, we should use this generic socket handler from the agent implementation in order to call into some specific functions (specific to the agent protocol) but always requiring a socket to communicate to.
All what I am suggesting here would be pretty straight forward to implement for both kata-proxy and no-proxy since they follow this model of receiving the same data, the encapsulation being done by the proxy itself if needed.
But this would require some extra work for our current cc-proxy and cc_proxy.go implementation since the encapsulation is done by the client itself.
One first step to avoid too much changes would be to consider using the sendCmd() the way it is right now, only for the cc-proxy case, while we would not use it for other new implementations. Instead, we would only use the subset start(), register(), unregister(), connect(), disconnect().
This is the first time we're really testing the flexibility of virtcontainers interfaces, and I would like to make sure we move in the right direction by refining them in case we see a need for that. And I think this is the case here.
@sameo @jodh-intel WDYT ?

Assuming we want to completely drop the single proxy model (Which I think we should, the HA and upgrade complexities linked to it are too big imo), we could simplify the current interface even further. We should change our logic so that the shim and the runtime only ever talk to the agent, or a URL that looks like an agent endpoint:

start(): Starts the proxy binary and here we need to give the proxy 2 pieces of information: Which endpoint (socket, etc...) it should listen on and where and how it should connect to the VM/Pod. As @sboeuf points out, start would now only return a URL for the shim to reach the VM/Pod.
register()/unregister(): Those 2 become almost pointless. Why do we need to register a VM/Pod when we know we're only going to handle a single one? The only case where this might be needed is when/if we want to start the proxy before we even know anything about the VM and we need to asynchronously let the proxy know about a new VM. So if we want to keep those around, we need to make them simple notifications, i.e. they should only return an error and nothing else.
connect()`disconnect(): This one can be removed. start()returns a URL and this should be sufficient for anyone that wants to connect to a proxy to know how it should do it. We could provide aagent.URL` method for getting
sendCmd() is only used by the agent implementations to, well, send commands into the VM. kata_agent should not use sendCmd() at all, so this would only be used by cc_proxy. An agent should only have to know about the protocol used by the agent, and a shim ideally would only talk to an agent as well.

All in all I think a vc 2.0.0 proxy interface could be simplified down to:
// proxy is the virtcontainers proxy interface.
type proxy interface {
	start(pod Pod) (string, error)
        stop(pod Pod) (error)

        register(pod Pod) (error)
 }

Now this would be a 2.0.0 and it would mean large changes to our current cc_proxy and hyperstart_agent code.
So I propose we go through the following steps:

Keep the current proxy interface and for kata_proxy this would mean:

sendCmd, register and unregister are NO-OPs
I believe connect and disconnect should be NO-OP as well, although it would be easily implementable (only the returned URL would really matter).

Implement a 2.0.0 proxy interface based on the above interface:

Many parts of the code would be simplified as we would no longer need to register our VMs, nor we'd need to connect to a VM before interacting with it.
kata_proxy would be left almost unmodified (except for getting rid of the NO-OPs)
cc_proxy would have to go through some fairly heavy changes.

Does that make sense?

It does really make sense ! And I agree this has to be a 2 steps thing since we want to make sure we can make both cc-proxy, kata-proxy and no-proxy working before we decide to bring those heavy changes to cc-proxy.

As a side note here, I have been thinking about this proxy interface 2.0. The changes for the cc-proxy could be simplified if we were listening on two different sockets with the proxy. We would have one URL that we would provide to the shim and runtime, and this would be used for all the things specific to the shim <==> proxy protocol, while the agent implementation hyperstart.go would send hyperstart frames directly to a new socket created by the proxy and dedicated to multiplex the frames with the hyperstart agent.
Or another solution would be to drop the shim/proxy protocol so that the proxy would actually behave the same way the agent would behave if we were directly connected to it.
The whole issue here relies on the fact that we want our agent to talk the same protocol, in both cases where we would have a proxy or not. But the fact that our cc-proxy has its own protocol actually prevents us from that.


generate index template automatically

hi.
when I used the data stream I saw a new index template automatically generated. how can I disable this feature?  i have a datastream and I don't want this connector to generate a new one.
thanks.

@alihossein we just merged this pr. You might want to test.


Bugfix: Requesting /Systems is listing systems that are not found

Now, we are getting all server profiles and filtering them by valid serverHardwareUri attribute
Closes #510

Coverage increased (+0.003%) to 91.969% when pulling a2c86c82a11d722e425e7e2493a24a389a8683aa on bugfix/listing_composed_system_that_not_exist into 4d0c7a26cc64c6261fcd60f57c29bec5d632eac3 on master.


Set grib2io as a dependency

What was the rationale for having grib2io as a dependency?  I know it makes sense...but just wondering.

I don't recall any other reason than that you can't use grib2io-interp without grib2io.  I think if someone sees grib2io-interp first they might think, "This is the interpolation version of grib2io" and just install grib2io-interp.  With this dependency they would actually get the interpolation version of grib2io.  That's all I got.

Technically grib2io is not required.  If you had the appropriate GRIB2 grid definition information at hand, you can just all that an source grid into the interpolation subroutines, but even in the README, it states that one should use the grib2io.interpolate function.
So lets get this in then.  Eventually grib2io and grib2io-interp will come back together anyways.


WIP Data grid filter sort pagination graphql query #41

https://user-images.githubusercontent.com/85221259/125081106-eb8d0980-e0c5-11eb-897c-bb4fa56bac0d.mp4
https://user-images.githubusercontent.com/85221259/125081421-46befc00-e0c6-11eb-83cd-a35699417bd5.mp4
Here is a video where the error is originally without my modifications
https://user-images.githubusercontent.com/85221259/125082152-28a5cb80-e0c7-11eb-9763-fe16aac329a3.mp4
I will make the dates filter

//TODO remove console.log


Input Rewrite

hihi

good job bro. I'd actually like this change a lot.

My favourite pr of all time

My favourite pr of all time.

changed jack detection a lil bit, can any of yall test it?
specifically with songs that have jumps/doubles or triples, and/or have jacks

good job bro. I'd actually like this change a lot.

I really dislike normal psych input. I hope this makes it feel better (I haven't tried it yet)

anyone willing to contribute to this, I need help with sustain inputs, basically when you release a sustain after successfully grabbing it, you miss like if it was still multiple notes
I only need a way of fixing that behavior

anyone willing to contribute to this, I need help with sustain inputs, basically on guitar hero sustains, when you release a sustain after successfully grabbing it, you miss like if it was still multiple notes
I only need a way of fixing that behavior

steal kade engine's code. sustainActive to be specific in both PlayState.hx and Note.hx

changed jack detection a lil bit, can any of yall test it?
specifically with songs that have jumps/doubles or triples, and/or have jacks

Guess this is my cue.
I'd say it's much more consistent than before as it was much easier to SFC this, tweaked a chart I did to test jacks and chords.
https://github.com/ShadowMario/FNF-PsychEngine/assets/25033383/67046711-a50d-4ef1-b5f9-f7654ca98831

Guess this is my cue.
I'd say it's much more consistent than before as it was much easier to SFC this, tweaked a chart I did to test jacks and chords. May be placebo? Idk, but I managed to SFC 4 times in a row, although this chart is pretty easy. I could import some osu!mania 5.5 star maps and higher aswell to test out if you want
jack.and.chord.test.200bpm.mp4

thats a very good sign actually, my goal with this was to make the input function faster in general while also adding the GH/Kade Engine sustains as an optional bonus so yeah very nice to know

Hey i found bug when without ghost tapping when you just pressing buttons, game crashes
smth wrong in this line ig idk

update to this: as a complementary update, I thought of replacing the etterna timings with the base game timings, adding the "shit" rating as a customizable one in the options menu, and let users change timings based on the worst timing's max
so you wouldn't be limited to 45ms in sick, for example, you would instead be limited to 166.67ms, which is the default timing for shit in base game
just an idea, I'm likely not gonna do it in this PR specifically

update to this: as a complementary update, I thought of replacing the etterna timings with the base game timings, adding the "shit" rating as a customizable one in the options menu, and let users change timings based on the worst timing's max
so you wouldn't be limited to 45ms in sick, for example, you would instead be limited to 166.67ms, which is the default timing for shit in base game
just an idea, I'm likely not gonna do it in this PR specifically

Personally as a rhythm game nut I think that would be a bad idea. The timings are already on the extremely lenient side.
45ms is A LOT

Personally as a rhythm game nut I think that would be a bad idea. The timings are already on the extremely lenient side. 45ms is A LOT

the purpose of this change would be mainly making timings extremely customizable, the essential "revert" of timings would be there because currently in psych, it is nearly impossible to get a Boo/Shit, this is because the timing is 180.0ms, which safeFrames can BARELY reach

Judgement
Current Timing
New Timing

SICK
45.0ms
33.33ms

GOOD
90.0ms
91.67ms

BAD
135.0ms
133.33ms

SHIT
180.0ms
166.67ms

by changing timings to this, not only Boos/Shits would be able to be hit in normal gameplay, but I'd also change how the options themselves work so you can actually go back to the etterna timings if you want, or have completely custom ones which also allow for decimals to be used
and also it kinda bothers me that Boos can't have custom timings for no reason lol.

Personally as a rhythm game nut I think that would be a bad idea. The timings are already on the extremely lenient side. 45ms is A LOT

the purpose of this change would be mainly making timings extremely customizable, the essential "revert" of timings would be there because currently in psych, it is nearly impossible to get a Boo/Shit, this is because the timing is 180.0ms, which safeFrames can BARELY reach
could also be due to the lack of a 5th rating in FNF so safeFrames could have a proper adjust
Judgement 	Current Timing 	New Timing
NONE / Marvelous 	25.5ms 	NONE because fnf doesn't have marv
Sick / Perfect 	45.0ms 	33.33ms
Good / Great 	90.0ms 	91.67ms
Bad / Eh 	135.0ms 	133.33ms
Shit / Boo 	180.0ms 	166.67ms
by changing timings to this, not only Boos/Shits would be able to be hit in normal gameplay, but I'd also change how the options themselves work so you can actually go back to the etterna timings if you want, or have completely custom ones which also allow for decimals to be used
and also it kinda bothers me that Boos can't have custom timings for no reason lol.

I kinda misunderstood then, if the new timings doesnt let you go over 45ms for sick or discourage usage of more than 45ms for sick through score reduction or something, then I'm all for it personally


Need help with Package.swift

I have big iOS project which uses cocoapods. I want to be able to work on this project using neovim.
Is it possible to tell sourcekit-lsp somehow to detect existing Xcode build so I don't have to run 'swift build' and create Package.swift in root of the project? (some of these libraries are only in cocoapods variant - no SPM)

If I understand correctly, what you are asking is that SourceKit-LSP should support opening Xcode projects, right?
Opening Xcode projects in SourceKit-LSP is not something we support. There has been some discussion around this in the Forums, which also list some workarounds that you might want to try:

https://forums.swift.org/t/xcode-project-support/20927
https://forums.swift.org/t/whats-the-plan-for-index/18441

If I understand correctly, what you are asking is that SourceKit-LSP should support opening Xcode projects, right?
Opening Xcode projects in SourceKit-LSP is not something we support. There has been some discussion around this in the Forums, which also list some workarounds that you might want to try:

https://forums.swift.org/t/xcode-project-support/20927
https://forums.swift.org/t/whats-the-plan-for-index/18441

thanks, but all this is about how to escape Xcode and use SourceKit-LSP with neovim (I mean develop iOS apps, without returning to Xcode)

If you manage to describe the entire build process of your iOS app using SwiftPM, you will be able to use SourceKit-LSP just fine. To the best of my knowledge this is not currently possible because SwiftPM doesn’t know how to bundle up iOS apps – but I might be wrong here. In that case you still need an Xcode project to deploy the app to your phone.

I’m closing this because support Xcode projects in SourceKit-LSP is not something we are planning to support in the near future with the rationale mentioned in this Forums Thread:

There is no officially supported way to understand Xcode projects outside of Xcode, for example there is nothing like libSwiftPM which the SwiftPM team commits to support as a way of understanding SwiftPM projects. Unless this situation changes, we do not intend to support Xcode projects outside of the Xcode IDE. Xcode itself will change to provide its settings directly to sourcekit-lsp, when used from inside the IDE.
People have devised methods for getting a compilation database out of an Xcode project, you may want to look into these, the compilation database is something we intend to maintain first-class support for.


Upgrade stylelint

In order to address https://github.com/primer/stylelint-config-primer/issues/22, we'd need to upgrade to stylelint >=9.8.0. Since the micromatch team confirmed that that vulnerability isn't really an issue & this would take upgrading two breaking versions up, I'd consider this low urgency, but would be nice to do eventually so people don't see that alert on vuln audits.

wooo!! 🙌


Doctrine Common requires php5.5+, so should Symfony 2.8 requires php5.5 as min version too?

Today I've broken (sadly) my deployment when installing symfony (via composer) on OpenShift (php5.4). Some days ago it worked well, but today it does not. Doctrine\Common requires php 5.5 as min version (see this commit )now and as Symfony-framework-extra-bundle requires doctrine-common too so I wonder if symfony requires php 5.5 from now, as it depends on doctrine-common.
This is seems a problem, what you guys think of?

You can manually change PHP requirement version to 5.5+ in your own project based on symfony-standard as well. But for BC Symfony 2.8 still requires >=5.3.9 and it shouldn't be changed here.

It's just a matter of using the same php version when updating vendors as on the server. Framework-extra-bundle can use any version: https://github.com/sensiolabs/SensioFrameworkExtraBundle/blob/master/composer.json#L15
Btw, there is also #858 if that is what you are experiencing.


blog(offline support): links add new and better text

Description
i suggest:

link to Workbox
better naming link

Related Issues
#18759

oh - wrong repo


add lint sort props as a warning

see if CI leaves the warnings

@keybase/react-hackers
This makes js keys not being sorted a warning. lint will notice but not error:
https://ci.keyba.se/job/client/job/PR-5334/2/execution/node/202/log/
this way we can slowly cleanup the issue

@keybase/react-hackers bump


fix(client): new_memories_with_actions maybe a list or a dict

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
new_memories_with_actions variable in Memory._add_to_vector_store returns a list or a dict, when it returns a list,  for resp in new_memories_with_actions["memory"]: block will report an error: ERROR:root:Error in new_memories_with_actions: list indices must be integers or slices, not str
some contexts:
I use qwen-turbo as LLM, and a custom openai_base_url: https://dashscope.aliyuncs.com/compatible-mode/v1
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Please delete options that are not relevant.

[ ] Test Script (please provide)

message = "I want to eat an apple"
client.add(
    messages=message,
    user_id="user1",
    agent_id="agent1",
)

# until now is correct

message = "I want to eat an apple"
client.add(
    messages=message,
    user_id="user1",
    agent_id="agent1",
)

# expect to add a memory or update the above memory
# but it reports the error, and no memory changes
# debug in `_add_to_vector_store`, the variable `new_memories_with_actions` is a list, so the error occurs

BTW:
There has another way to fix is to change the prompt, maybe that will be a better patch
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Maintainer Checklist

[ ] Made sure Checks passed

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


Gardens onboarding wizard (user flow)

Done: https://www.notion.so/Creation-Wizard-1625333dd1a148e78a1938d26d77abf9


Fix references to repository in bundle CSV

We missed a few links when we migrated the Compliance Operator from the openshift organization.
Let's clean those up so they don't confuse users when they install through OLM.
Fixes #229

/retest


Can't instantiate HalfT prior

Hi,
If I run this code:
from GPy.core.parameterization.priors import HalfT
HalfT(1, 3)

it gives the following error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/y/anaconda3/lib/python3.12/site-packages/GPy/core/parameterization/priors.py"$
    o = super(Prior, cls).__new__(cls, A, nu)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: object.__new__() takes exactly one argument (the type to instantiate)

My version of GPy is 1.13.2.
I think this error is related to #503 and there might be a simple fix, but i'm not confident enough to give it a reliable fix (for research purpose). Could anyone from inside/outside the team help? Many thanks.
Edit: added version of GPy.

Hi @MartinBubel, thanks very much!
I actually tried to give a temporary workaround (following #503) and I sampled from HalfT(1,1).
However, half of the samples are exactly 0 - which is different from samples from Scipy's HalfCauchy. The number of zeros doesn't change much if I sample from HalfT(1,3) or others.
It might be worth checking when you fix the issue - it could be an issue with my workaround or a bug in the algorithm itself.
Code:
from GPy.core.parameterization.priors import HalfT, Prior
import weakref
import GPy
import numpy as np

class MyHalfT(HalfT):
    def __new__(cls, A, nu):  # Singleton:
        if cls._instances:
            cls._instances[:] = [instance for instance in cls._instances if instance()]
            for instance in cls._instances:
                if instance().A == A and instance().nu == nu:
                    return instance()
        newfunc = super(Prior, cls).__new__
        if newfunc is object.__new__:
            o = newfunc(cls)
        else:
            o = newfunc(cls, A, nu)
        # o = super(Prior, cls).__new__(cls, A, nu)
        cls._instances.append(weakref.ref(o))
        return cls._instances[-1]()

xs = np.random.normal(0, 1, size=(10, 1))
ys = np.random.normal(0, 1, size=(10, 1))
k1 = GPy.kern.RBF(input_dim=1, variance=1.0, lengthscale=1.0)
m = GPy.models.GPRegression(xs, ys, kernel=k1)
m.kern.variance.set_prior(MyHalfT(1, 1))  # halfT is problematic
m.kern.lengthscale.set_prior(LogGaussian(0, 2))  # loggaussian is fine
n = int(5e3)
samples_var = np.zeros(n)
samples_len = np.zeros(n)
for j in range(n):
    m.randomize()
    samples_var[j] = m.kern.variance[0]
    samples_len[j] = m.kern.lengthscale[0]
print(np.sum(samples_var == 0.))

comparing with halfcauchy is scipy:
from scipy.stats import halfcauchy
r = halfnorm.rvs(size=n)
print(np.sum(r == 0.))

Hi @niwaka-ame
thanks for the update. I just gave it a try myself and it seems like I have to dig a bit deeper into this. I found that there is no test on the HalfT prior and some others.
The fix you are proposing looks good and is also part of some other prior objects. I still have to check the implementation of the prior in general to see why the result using the scipy prior deviates.


Update documentation for v2.5

I have to admit, I have not been looking forward to this.
Setup an overleaf account and I will provide you with editor access.
https://www.overleaf.com/

I modified ./mc_display_mirror.py to take a screenshot when pressing '9'
I've gone through and uploaded a bunch of new screenshots
https://github.com/jmamma/MCL-Documentation/tree/master/MCL Documentation/images/images_pending

So cool.
Oh wait we got visual utiming now??

Of course. 💰Premium feature 💰

MegaCommand:

INSERT COIN <

All right first things first. The doc repo is not compiling on my side.
I got TeXLive 2019 and I use latexmk -xelatex.
Can compile earlier revisions with no problem.

I just fixed two compile errors relating to special characters.
It's easier if you just work from overleaf.
https://www.overleaf.com/read/gsdhhxzpbgdx
We should be able to work collaboratively .
Just need your account email so I can give you editor access.

Ok cool. Just realized that I do have a overleaf account<EMAIL_ADDRESS>

I'm in 🙌

Thanks. It's good to have a different pair of 👀 on the documentation.
I'm going to take a look at the HD44780 display code.

May be useful:
https://www.tablesgenerator.com/

Half way there.
Renamed all pending images, and moving them out of pending on-demand.

@jmamma some screenshots are still missing.

Seq group ok.

@yatli which pages are missing?
The serial transfer code is currently located in GUI.display().
Any draw routines that bypass that code can't be captured currently.

PTC-A4
PARAM-Edit
RLCK
Ext-Seq

I thought PARAM-Edit was there?

The others are missing.

hah, I put it in the overview "Parameter Locks" -- no locks and no trigs registered, so doesn't match the text in the detailed section

@jmamma can you do the ram page doc. You will describe it better I'm sure :)

sure

I'll add a section about the workflow, and describe the difference between the GRID, the INTERNAL SEQUENCER and the MD SEQUENCER.

Done, see "key concepts"

Requesting to put my name in there :p

yes of course 🥇

@yatli
Maybe we should have a Buttons Assignemnt chart under the Encoders Assignment for each page.

Also, can you sync your changes to github, incase you havent already.
Top left Menu -> Sync
Thanks.

Makes sense.
Combine the two to save some space?

Synchronized.

Synchronized, now with button assignment table.

Two left, the LFO page and the RAM page.

Done LFO page skeleton.


Tests fails on different screens

Hey.
Probably it's not an issue but I haven't found a way of solving this.
I have a simple test which checks width
labelBackground.getAttribute('width').then(function (width) {         expect(width).toBe('60');     });
It works on my laptop but fails if I run it on another laptop whose screen resolution is different. I've tried played with config and setting screen size in before each but it didn't help:
browser.driver.manage().window().setSize(1000, 1000);
Is there a way of solving this issue?
Thank you!

@dmoskovsov how did you solved the issue? making height and width to 1000px?


[Bugfix] feat_proj does not considered the possibility that the ID of the ADAS map does not start with 1

Status
DEVELOPMENT
Description
reported in #996
Todos

[x] Tests
[ ] Documentation

Steps to Test or Reproduce
Please test feat_proj with the ID of the ADAS map that does not begin with 1.
(If necessary, I send such a map)

LGTM！


Update PROJECT: PLAYTIME

Game Title
PROJECT: PLAYTIME
Status of the Anti-Cheat
⭐ Supported
Tinkering Steps
Game is working on my machine as well. Also why the lack of updates on this repo?
Proof of Mention
https://www.protondb.com/app/1961460

Game is working on my machine as well. Also why the lack of updates on this repo?

What do you mean? I've updated the manifest on this repo every day for the last week? Are you referring to something else?

Game is working on my machine as well. Also why the lack of updates on this repo?

What do you mean? I've updated the manifest on this repo every day for the last week? Are you referring to something else?

I've had issues open here for months at this point with no action, so i was just wondering

Game is working on my machine as well. Also why the lack of updates on this repo?

What do you mean? I've updated the manifest on this repo every day for the last week? Are you referring to something else?

I've had issues open here for months at this point with no action, so i was just wondering

We're still slowly going through the backlog. New features are being made to make this process slightly easier and faster, but they're not ready yet for prime time. Plus, the repository is going through a slight refactor in the background, so all of this has to be juggled on top of the work that already needs to be done.

Game is working on my machine as well. Also why the lack of updates on this repo?

What do you mean? I've updated the manifest on this repo every day for the last week? Are you referring to something else?

I've had issues open here for months at this point with no action, so i was just wondering

We're still slowly going through the backlog. New features are being made to make this process slightly easier and faster, but they're not ready yet for prime time. Plus, the repository is going through a slight refactor in the background, so all of this has to be juggled on top of the work that already needs to be done.

nice work you guys are doing, thanks


Generation index.d.ts for vue-cli builder

What problem does this feature solve?
Today command vue-cli-service build --target lib --name libName creates only commonjs and umd js files without ts files. React and angular can generate ts files.
What does the proposed API look like?
I propose ts library generations from command vue-cli-service build --target lib --name libName with option --ts true

Hi @rom111419 Do you have any implementation proposal for this or any alternative to solve this problem until it is available in cli. I am also looking for similar feature. thanks

I am looking for it too.

If you build a UI component library to the TS project, there is no declaration file.When importing, the module will be prompted that it cannot be found.
I also think d.t. File is needed

Does any of you have a workaround for this ?


Fix and Add some Chinese translations

Coverage remained the same at 87.755% when pulling 05252fa239ae5f26ed55228380ab0ac19b726389 on Kroderia:master into 1377439bb010c2b32733ef2cbe7880543eb66cb6 on node-red:master.

Thank you!


A and D keys not moving correctly

Pressing A or D seems to move you up or down more than sideways.

Do you mean when running sideways that the char is drifting?

Yes that’s correct. It’s happening on my Mac mini if that makes a difference. Instead of moving east or west. It’s more like south east and north west when moving.

On Apr 3, 2021, at 02:30, Andre @.***> wrote:

Do you mean when running sideways that the char is drifting?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Can you please share a full screen screenshot or screen recording? That would be very helpful.

 It's working great now.
Thanks!


could you please update the https://github.com/Caphyon/clang-format-editor

thanks

Hi @heartacker,
I apologize for a late reply
Thank you for your contribution, we will update Clang Format Editor
Have a nice day,
Marina

Hi @heartacker,
Check Clang Format Editor v2024.3.0
Thank you for your contribution
Kind regards,
Marina


fix: dont return anything from notion workflows

We've occasionally gone over the workflow return size limit on temporal.
This is due to 2 things:

we are returning every "upsert result" down to the root to update the parents field

we have child workflows that might themselves spawn a lot of child workflows, so they become "bottlenecks" through which a lot of data flows (so we can pass it back to the main workflow)

The solution:

use the database, and stop returning data from  the workflows

Also you can remove the type UpsertActivityResult from activities.ts

Tested, confirmed it worked for 3 level of depth. Thanks a lot for the review @philipperolet 🙏


Date picker disappear

Question

Platform: iOS 13.1.2, iPhone 6s, Xcode 11.1
Version: "@react-native-community/datetimepicker": "^2.1.0",
"react-native": "0.61.2",
The date picker is not visible

but it works on  simulator with iOS 13.1

It is because the "night" mode in the new iOS13.0+, with "daylight" mode turned on things are good, letters of the spinners are black and not white.
This is what we also experience.
Anyone with a solution, how to force the spinner so stay in "daylight" theme?

For now you can force light mode by adding:
`
// force light theme to avoid white text in white background TextInput
if (@available(iOS 13.0, *)) {
rootView.overrideUserInterfaceStyle = UIUserInterfaceStyleLight;
}
`
in your AppDelegate.m
Original source: https://github.com/facebook/react-native/issues/26299#issuecomment-533990557

Do we have a fix for this in expo apps?

Experienced the same issue.

@Beissner @rdewolff
You can disable iOS darkmode by following (these steps)[https://stackoverflow.com/a/58395927/1175081] This package should handle darkmode better though.

It's not a dark mode issue for me. It doesn't show anything neither on my iphone 7 ios 11.0 nor on simulator ios 13.2.2(although it opens up the space). But it is working fine in android.

100% dark mode issue here, works fine on light mode, cant see text on dark mode.

Fixed by updating to expo sdk v35

For us the problem seemed to be that the date picker was using a text color aligned to the system theme. So if the device/simulator was in dark mode the text was light.
However the background was not adjusted.
We then workarouned the issue by explicitly adding a background view with a color that fits the current system theme so the text is always visible.

@okwast that sounds exactly how the issue should be fixed. Ideally this background view should be included in the package. Would you mind sharing your code, or maybe even opening a Pull Request that would introduce this?

Having a prop to bypass the coloring would be nice. As much as @okwast comment makes sense - having a UI to match OS prefs isn't always possible. @sourenamimik 's solution worked perfectly.

@barrylachapelle
I would not call disabling an OS feature a solution. It is at most a workaround and would prohibit anyone using this package from using the iOS dark mode.

having a UI to match OS prefs isn't always possible and a pattern that cannot be enforced

I don't understand what you are trying to say here. Dark/light mode is a simple boolean supplied by the OS. It is already used correctly for the font, but not for the background. So that just needs to be fixed in the package.

@schumannd is right. Forcing light mode is not an option if someone has styled the rest of their app to accept both. This is a bug in the code where the text changes with Dark Mode, but the background does not, and should be fixed.

@schumannd is right. Forcing light mode is not an option if someone has styled the rest of their app to accept both. This is a bug in the code where the text changes with Dark Mode, but the background does not, and should be fixed.

There isn't a bug because it is a feature that is never implemented, I'm working to support dark mode

Closed by #204


Olingo4 olingo4endpointPropertyNames are missing queryParams for read method

Hello,
we are migrating from camel 2.23 to camel quarkus (2.13 redhat).
Following code worked with 2.23 and appended the query parameters from the map which was generated by the method expression
.setHeader("CamelOlingo4.queryParams", method(SomeQueryBuilder.class, "buildQuery"))
.enrich("olingo4://read/SomeEntities", AggregationStrategies.flexible().storeInBody())

With camel quarkus the set
org.apache.camel.component.olingo4.Olingo4Endpoint#olingo4endpointPropertyNames
does not contain the value "queryParams". This results in calling toString() of the map in
org.apache.camel.component.olingo4.Olingo4Endpoint#parseQueryParams
putting the value {key1=value1} in the local queryParams variable resulting in an invalid URL.
I think the issue may be that queryParams in Olingo4AppEndpointConfiguration is null and so
the key for queryParams does not get set in the properties map in:
org.apache.camel.support.component.ApiMethodPropertiesHelper#getConfigurationProperties
Please let us now if we are missing something else...
Kind regards
Oliver

It looks like a bug in camel-olingo4? can you raise an issue on https://issues.apache.org/jira/projects/CAMEL/

I created a ticket. Hopefully it captures the issue:
https://issues.apache.org/jira/browse/CAMEL-19150

Thanks!

Thanks @jamesnetherton and the first idea in my mind is to add
olingo4endpointPropertyNames.add(QUERY_PARAMS_PROPERTY);

in https://github.com/apache/camel/blob/main/components/camel-olingo4/camel-olingo4-component/src/main/java/org/apache/camel/component/olingo4/Olingo4Endpoint.java#L168
There is a generated API integration test Olingo4AppIT but all of method are marked with @Disabled. So I'm not sure if there is any test for these cases.

@zhfeng thx, yes this would fix it, but would provide the property to every (http) METHOD.
I think it was not added in the first place because it is only relevant for READ and UREAD
The Olingo4AppEndpointConfigurationConfigurer does not copy the map because it is null in
Olingo4AppEndpointConfiguration
Regards

Olingo4AppEndpointConfigurationConfigurer and Olingo4AppEndpointConfiguration are both generated by camel-api-component-maven-plugin automatically.

The Olingo4AppEndpointConfigurationConfigurer does not copy the map because it is null in
Olingo4AppEndpointConfiguration

I'm not sure what are you meaning here. Can you show the codes for copy the map?

private java.util.Map<String, String> queryParams;
in Olingo4AppEndpointConfiguration is null
so Olingo4AppEndpointConfigurationConfigurer returns in getOptionType
null for QueryParams when called from
org/apache/camel/support/component/ApiMethodPropertiesHelper.java:116
So queryparams does not get set into the map:
org.apache.camel.support.component.AbstractApiEndpoint#endpointProperties
if it would have been set it would get populated into the Map
olingo4endpointPropertyNames in
org/apache/camel/component/olingo4/Olingo4Endpoint.java:170

Thanks @varoliver and it seems that
https://github.com/apache/camel/blame/main/components/camel-olingo4/camel-olingo4-component/src/main/java/org/apache/camel/component/olingo4/Olingo4Endpoint.java#L169
olingo4endpointPropertyNames = new HashSet<>(getEndpointPropertyNames());

which contains only nonNull propreties. But in camel-2.23.x, it uses
endpointPropertyNames = new HashSet<>(getPropertiesHelper().getValidEndpointProperties(configuration));

which contains all of the properties. I think this could explian it works in 2.23.x not fail in 3.x
Since these codes were changed by @davsclaus 3 years ago by https://github.com/apache/camel/commit/eba38a52355da86e22e2c985b336267d62df22b2
I wonder if it should change to use getValidEndpointProperties here.

yes, mainly because the fields are propagated by reflection in
org.apache.camel.util.component.ApiMethodPropertiesHelper#getValidEndpointProperties
independent of the value of the variable.
Unfortunate that there are no tests which covered this.
Small setback for our migration because we are using "CamelOlingo4.queryParams" quite a lot. We will have to change it to setting the parameters directly in the header like:
exchange.getIn().setHeader("CamelOlingo4.$filter", "someValue");

I open a PR to fix it in upstream camel

https://github.com/apache/camel/pull/9570

@varoliver can you take a look?

@zhfeng looks good. What i am wondering is what will happen for another METHOD like PATCH or DELETE in case queryParam is set in exchange header - would the params also be appended to the url now?
I read the fix will be backported also to 3.x.  This would be good for us.
Current Camel 3.18. version is 3.18.5.
So the fix should be available in 3.18.6? If so - when do u think it will be released?
Red Hat build of Camel Extensions 2.13.2 uses Camel 3.18.3.
Will there be an build of Camel Extensions for Quarkus 2.13.3 with 3.18.6 (containing the fix)?
If there will be a build but no official Red Hat build would it be safe to reference the olingo4 module v2.13.3 in our pom but depend on Red Hat Camel Extension 2.13.2 bom?
THX

@varoliver I just go throught the codes and it looks like other method which does not have queryParams in the args will not get it from the properties. See https://github.com/apache/camel/blob/main/core/camel-support/src/main/java/org/apache/camel/support/component/ApiMethodHelper.java#L478-L484
So I belive it should not append to the url even if queryParams is set in the headers.
Regards for the Red Hat build of Camel Extensions for Quarkus, please contact them for supporting.

The camel-quarkus 2.13.3 release should contains this fix in camel 3.18.6. So I'd like to close this issue. @varoliver feel free to test with 2.13.3 and reopen it if there is any problem. Thanks!

Could please somebody set the right milestone where this was fixed?


Reduce duplicate dependencies of different versions

Many parts of the code base use slightly different versions of the same crates. It’d be nice to reduce this. May require a bunch of updates to the code that uses them.

We can reduce this with workspace dependencies: https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#inheriting-a-dependency-from-a-workspace


drivers/rtc: Fix how prescaler is used in QMSI driver

The issue was introduced by commit-id 27bdb83.
RTC_DIVIDER used to be 0 by default, now it's 1.
Fixes #8098
Signed-off-by: Tomasz Bursztyka<EMAIL_ADDRESS>

Codecov Report

Merging #8158 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #8158   +/-   ##
=======================================
  Coverage   64.54%   64.54%
=======================================
  Files         420      420
  Lines       40142    40142
  Branches     6765     6765
=======================================
  Hits        25911    25911
  Misses      11110    11110
  Partials     3121     3121

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 711adb8...5914462. Read the comment docs.


Update vault

Desired Outcome
CI should pass
Implemented Changes
Updated the vault to hashicorp/vault
Connected Issue/Story
Resolves #[relevant GitHub issue(s), e.g. 76]
CyberArk internal issue ID: [insert issue ID]
Definition of Done
At least 1 todo must be completed in the sections below for the PR to be
merged.
Changelog

[ ] The CHANGELOG has been updated, or
[x] This PR does not include user-facing changes and doesn't require a
CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code
changes, or
[x] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR
[ ] A follow-up issue to update official docs has been filed here: [insert issue ID]
[x] This PR does not require updating any documentation

Behavior

[ ] This PR changes product behavior and has been reviewed by a PO, or
[ ] These changes are part of a larger initiative that will be reviewed later, or
[x] No behavior was changed with this PR

Security

[ ] Security architect has reviewed the changes in this PR,
[ ] These changes are part of a larger initiative with a separate security review, or
[x] There are no security aspects to these changes

This fix has been merged into the GH Enterprise repo and just needs to be propagated to OSS.


Difference between standard run and remote-debug run?

I'm going nuts here was working on getting some snmp working and it was working fine, then i disabled the IOS remote-debugger and its crashing out, i'm using net-snmp for snmp via udp, i swapped to net-snmp, because oddly enough the npm module i switched from (snmp-native) after going back to it i had a similar issue, with things working (after some hacking of the module) with remote-debugger enabled but as soon as you disabled it it would bomb out to red screen
net-snmp module worked out of the box, while remote debug was enabled... then i ran it without remote-debug using log-ios instead, and red screen of death!
It's like somehow the UDP packets the app are receiving are different between when the app has Remote-Debug enabled and when its just running in the emulator
For instance with remote-debug enabled the SNMP request processes fine, with it disabled and using react-native log-ios i get....
: Community 'public' in request does not match community '0��public��q
So it appears for some reason with remote-debug disabled the udp messages are garbled

hm, my first thought is that somewhere someone did buffer.toString() when the buffer contents weren't a utf8 string. For background, the react native bridge (javascript<=>native) doesn't support sending binary data, so this module converts all UDP packets to base64 and back when traveling from and to iOS/Android. Does that help?

Well i'd imagine it has to do with the ASN1 conversion that reads the Udp
hex and converts it to Text, what i dont get from your explanation is why
it would differ based on if remote-debug is enabled or not (monitoring via
chrome debug or not) ...
also if the react-native-udp bridge is handling binary ->.base64 -> binary,
wouldn't this mean the error is in the bridge, because to apps using the
RN-UDP it should still appear as binary and somehow enabling/disabling
remote-debug is breaking that binary-b64-binary translation?
On 17 February 2017 at 11:23, Mark Vayngrib<EMAIL_ADDRESS>wrote:

hm, my first thought is that somewhere someone did buffer.toString() when
the buffer contents weren't a utf8 string. For background, the react native
bridge (javascript<=>native) doesn't support sending binary data, so this
module converts all UDP packets to base64 and back when traveling from and
to iOS/Android. Does that help?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tradle/react-native-udp/issues/39#issuecomment-280678658,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANIO10WWsg5P_bjyBJmsPzPuzJna7-O3ks5rdbtZgaJpZM4MEWTB
.

So it's definitely something wrong in the ios version of the bridge, as on
android it works with same exact code and modules, i tried to look at it
but i honestly don't know anywhere near enough about the bridge or native
code to debug it :( any chance you could take a look as theirs something
going on different between ios and android, and on ios its odd how it works
when the chrome remote debugger is open for me :S
On 17 February 2017 at 12:14, Chris Chance<EMAIL_ADDRESS>wrote:

Well i'd imagine it has to do with the ASN1 conversion that reads the Udp
hex and converts it to Text, what i dont get from your explanation is why
it would differ based on if remote-debug is enabled or not (monitoring via
chrome debug or not) ...
also if the react-native-udp bridge is handling binary ->.base64 ->
binary, wouldn't this mean the error is in the bridge, because to apps
using the RN-UDP it should still appear as binary and somehow
enabling/disabling remote-debug is breaking that binary-b64-binary
translation?
On 17 February 2017 at 11:23, Mark Vayngrib<EMAIL_ADDRESS>wrote:

hm, my first thought is that somewhere someone did buffer.toString() when
the buffer contents weren't a utf8 string. For background, the react native
bridge (javascript<=>native) doesn't support sending binary data, so this
module converts all UDP packets to base64 and back when traveling from and
to iOS/Android. Does that help?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tradle/react-native-udp/issues/39#issuecomment-280678658,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANIO10WWsg5P_bjyBJmsPzPuzJna7-O3ks5rdbtZgaJpZM4MEWTB
.

@cchance27 could you push your project with the reproducible error? I'll try to make time to look at it.

I have the same error，do you fix it? friend！

I fix it,just see this
If you want to send and receive node Buffer objects, you'll have to "npm install buffer" and set it as a global for UdpSockets to pick it up:
global.Buffer = global.Buffer || require('buffer').Buffer

closing due to inactivity


when configured to reflect Access-Control-Request-Headers, should also use Vary: Access-Control-Request-Headers

When the middleware is configured to reflect the request value of Access-Control-Request-Headers in the response value of Access-Control-Allow-Headers (as it is in the default configuration), it should also include Vary: Access-Control-Request-Headers in the response. Without this header, it's possible to trigger a situation where a downstream cache responds to an OPTIONS request using a cached response from an OPTIONS request with a different Access-Control-Request-Headers value. This could potentially result in either a permissible OPTIONS request being refused, or a non-permissible request being allowed.

good catch @jfirebaugh - I'll try to address it soon

+1

I pushed a fix for the next version 👍

thanks for taking care of this @dougwilson – as well as some of the other recent work. I've bumped and published as 2.8.2


Fallback to a hardware-agnostic "Generic mode" on unsupported boards, or as a compile-time option

Hey @doctea ,
Thank you for this other contribution! i will review asap and get back to you.

Hi @doctea ,
I have review your code and everything looks great! i just have some suggestions for better library usage from user perspective.
Please feel free to check and discuss anything about the changes.


Added base.html,urls.py , change views.py

@cypher-naman, I think the other files which include the code structure seems good. Though there are other files also like .php and .js. Do they belong to the theme you used and were structured like that by default?

Hey @VirtualRcoder, kindly create a pull request for your 2nd commit only. As 1st commit concerns to the README.md which has already updated. And I couldn't understand the last commit which is urls.py. So, put some details over it also.


Corrected the wrong icon of courseware assignments, discussions

Description
Corrected the wrong icon (replaced question mark icon with edit icon) of courseware assignments, discussions
MA-3189
Screenshot

Codecov Report

Merging #931 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #931      +/-   ##
============================================
- Coverage     16.69%   16.66%   -0.03%
  Complexity     1010     1010
============================================
  Files           466      466
  Lines         19814    19804      -10
  Branches       2284     2283       -1
============================================
- Hits           3307     3300       -7
+ Misses        16150    16149       -1
+ Partials        357      355       -2

Impacted Files
Coverage Δ
Complexity Δ

...edx/mobile/view/adapters/CourseOutlineAdapter.java
17.5% <0%> (ø)
14 <0> (ø)
:arrow_down:

...dx/mobile/module/db/impl/DbOperationGetVideos.java
45.45% <0%> (-36.37%)
3% <0%> (ø)

...in/java/org/edx/mobile/module/storage/Storage.java
2.72% <0%> (-1.17%)
2% <0%> (ø)

...org/edx/mobile/util/images/ImageCaptureHelper.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

.../java/org/edx/mobile/user/SetAccountImageTask.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...ava/org/edx/mobile/view/MyCoursesListFragment.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...java/org/edx/mobile/third_party/crop/CropUtil.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Has this library been deprecated?

Title. I'm trying to use run 'npm install rn-mdx' and get errors each time I do.

No, just... uh...
Merged a PR that should fix this.


Problem. Stripes are projeced to desktop monitor not projector

Hello.
Firstly, thanks for sharing this nice toolkit.
During calibration, I have faced a problem.
Visual studio2013, Kinect SDK 2.0 are used
I use one main display and the projector attached to the Pc.
I checked main display being 0 and projector display being 1 .
However, whenever I click 'Acquire', stripes are always shown in main display..
And then calibration is fail again and again.
Many thanks.

Did you manually modify the calibration xml file to tell that the projector is on display index 1?
The ProjectorCameraEnsemble.Projector should contain this line:
1
Make sure to reload it after editing.

Thanks a lot


switch to cffi out-of-line ABI mode

switch to out of line mode to improve sounddevice module import time.
this adds a separate sounddevice_build.py file that becomes part of
the setup.py build process and is only run once at install time rather
than everytime the module is imported

Looks good to me.
I'm on holidays for the next 2.5 weeks, I'll try it afterwards.
Did you also try this on Windows?

I will test on windows.
interesting side-note: making the same change for the soundfile library seems to have no impact on its import time

with windows ran the same command:
$ (for i in {1..20}; do { time python -c 'import sounddevice' ; } 2>&1 | grep -oE '0\.[0-9]+'; done) | awk '{total += $1} END {print total}'

Linux:

A couple runs with sounddevice-0.3.8 gave 416ms, and 424ms
A couple of runs with out-of-line build gave 227ms and 247ms

Windows:
A couple runs with sounddevice-0.3.8 gave 371ms and 372ms
A couple of runs with out-of-line build gave 267ms and 267ms

Sorry for the delay, now I'm back ...
I've tried it on macOS, but it produces an error:
>>> import sounddevice
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/.../sounddevice.py", line 2646, in <module>
    _ignore_stderr()
  File "/.../sounddevice.py", line 2638, in _ignore_stderr
    stdio.stderr = devnull
ffi.error: symbol 'stderr' not found in library '<None>': dlsym(RTLD_DEFAULT, stderr): symbol not found

It seems that instead of a KeyError we get an ffi.error.
I'm not sure if that's a bug in CFFI, but if I change KeyError to _ffi.error, it works!

interesting side-note: making the same change for the soundfile library seems to have no impact on its import time

That's indeed interesting.
If I've counted correctly, the soundfile library has about 120 lines of C code to parse, while sounddevice has more than 300.
Is that the reason or is there something else going on?

Seems unlikely that it's the difference in length..but your guess is as
good as mine.
On Oct 11, 2017 4:45 AM, "Matthias Geier"<EMAIL_ADDRESS>wrote:

interesting side-note: making the same change for the soundfile library
seems to have no impact on its import time
That's indeed interesting.
If I've counted correctly, the soundfile library has about 120 lines of C
code to parse, while sounddevice has more than 300.
Is that the reason or is there something else going on?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spatialaudio/python-sounddevice/pull/102#issuecomment-335756222,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAaVHLxHqoW0JM-4nsErpCf9rPYGUnZ_ks5srI4vgaJpZM4PefmC
.

@tgarc Do you want to add the fix for macOS or should I do that?
Apart from that, I think this is ready to merge.

I can. But do you think both KeyErrors need to be changed?
On Oct 14, 2017 10:24 AM, "Matthias Geier"<EMAIL_ADDRESS>wrote:

@tgarc https://github.com/tgarc Do you want to add the fix for macOS or
should I do that?
Apart from that, I think this is ready to merge.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spatialaudio/python-sounddevice/pull/102#issuecomment-336642048,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAaVHIrY0VdPCSfe5w05HMrTvENU7Rjlks5ssNI9gaJpZM4PefmC
.

Thanks! Yes, I think both should be changed, since theoretically there might be neither stderr nor __stderrp available.

done!

okay I lied, I do see a modest improvement applying this change to the soundfile library as well. I think I just didn't see it on the machine I originally tested on because of other latency issues

Thanks @tgarc!
I've added another twist in #103, what do you think about it?

thanks for merging this! This has now been merged with PySoundFile as well. This dropped my command line application run time (which imports sounddevice and soundfile) down from 130ms to about 50ms. This is a far cry from the <10ms launch times of e.g. aplay (alsa utility for playing audio) but works fine for my purposes.
Also FYI, if you're ever using entry_points with setup.py there's a huge runtime delay added unless the module is installed as a wheel (issue report here). Just another fun fact about setuptools.


Admin page

Please merge these.
Fixes issues: #63 and #64

Pressing the 'Reset my password' button throws a connection refused error'.

Added a couple commits to fix the errors I was encountering, please verify and merge.

I tested this and it sent the email to console and then I was able to use that reset link to reset the password successfully. I'd merge the branch and we can add mailgun to it later once we have access to the domain settings for PKIF.


Implementing Caesar Cipher in Go

💥 Proposal
Implementing Caesar Cipher in Go.
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


InterSystems IRIS Ethereum Blockchain Adapter

Interoperability adapter which provides the abilities to build InterSystems IRIS interoperability solutions using Ethereum blockchain

Done and is available here: https://openexchange.intersystems.com/index.html#!/package/Ethereum-Interoperability-Adapter


Store agnostic @select?

It would be great to use @select with other observable-based store implementations too, e.g. ngrx/store.
Shouldn't be too hard - I started a prototype for this here: https://github.com/SethDavenport/select

So this is looking quite feasible. However I'm waiting for angular/angular-cli#2799 before investing too much time into decorators right now.


Add process blocker, similar to acceptableAssertionApplicationNames

For some circumstances it would be nice to block Nudge from running if certain processes are also running.
Similar to how acceptableAssertionApplicationNames blocks apps, maybe a script is running and we want Nudge not to run, a simple way I can think is to check for processes and not run if it is found.

Can you check the features here and make sure they don't already do what you want:
https://github.com/macadmins/nudge/wiki/optionalFeatures

It sounds like @charliwest is asking whether Nudge can monitor the activity of any arbitrary process on the computer.  I don't know whether Swift can run pgrep with a process name?  And this is made more complicated by the fact that "scripts" are often called by bash or zsh or perl or python, making your known management scripts completely indistinguishable from other non-management processes using those same runtimes.

Sorry I totally missed this question.
Basically my idea was not to run the reminder while a script was downloading the installer in the background, or maybe doing a FileVault key rotation.
Now though we have other ways of doing these things so it has become a mute point for me at least so can be closed :)


CPS-14-migration-handler

Support for migration from OSA/APS to CB/Connect.

Codecov Report

Merging #37 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #37      +/-   ##
==========================================
+ Coverage   93.46%   93.47%   +0.01%
==========================================
  Files          29       29
  Lines        1240     1242       +2
==========================================
+ Hits         1159     1161       +2
  Misses         81       81

Impacted Files
Coverage Δ

connect/models/fulfillment.py
100% <100%> (ø)
:arrow_up:

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 d1e4c29...7523cdd. Read the comment docs.


crash

hi, used chatweaks and twitch integration for a couple of times in this new modpack, no issues, since then been getting this crash, game goes fine and can log in, can play too, when i press chat key it freezes and crash. hope it isint a dumb mistake of me.
crash-2019-04-17_18.08.03-client.txt.zip
i dunno if this file worked, reach me in email or Zeek's Discord if anything needed

Chat Tweaks is not compatible with MineTogether since MineTogether adds its own chat replacement, you can safely remove MineTogether though if you're not using it.

thank you!! GeekHeart


Plans to include App Services as supported Environments resource?

Current choices for environment resources includes VMs and AKS.  Are there plans to add App Services as an option?
Thanks

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

ID: 77d95db6-9983-7346-d0eb-4b7443e4e252
Version Independent ID: 0a22cccc-318d-592f-d1ab-09ec01d88087
Content: Environment - Azure Pipelines
Content Source: docs/pipelines/process/environments.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

@markarnolditpro - Thanks for your question. It looks like you're asking a product question, rather than an issue with the documentation. Please consider these resources:

Azure DevOps Services
Azure DevOps Support Bot
Azure DevOps on Stack Overflow


Asp.Net core document sidebar translation does not work

Describe the bug
On the "asp.net core MVC" page in Chinese, the sidebars are all in English,
I remember it was in Chinese before,
It seems that pages in other languages also have this problem
To Reproduce
Steps to reproduce the behavior:

Click this link to enter the relevant Chinese page

Expected behavior
Sidebar restore previous Chinese
Screenshots

Desktop (please complete the following information):

OS: [ PC ]
Browser [ Edge ]
Version [ 98.0.1108.62]

@khoiph1 Do you know about this?


Create pandas DataFrame and Add cufflinks plot

Hi,
I add Creating Pandas DataFrame from the historical response

Coverage remained the same at 99.279% when pulling c9ec24003696138d12120c72a97f08b602bb706b on shayegh:create-pandas-dataframe into 3ec71d331b509750c4985db0a9b73bec6bf5f9f7 on hootnot:master.

Hi,
though I appreciate the effort of extending examples, I will not add this example since it addresses more the field of what you can do combining oandapyV20, Pandas, Seaborn, Matplotlib, ML, AI etc.
So I consider it beyond scope and hope you understand.
Why not publish it as your own repo?
Best,
Feite


backend/x11: add support for shm buffers

Closes #2866.
This doesn't work in Xwayland for me, since shared_pixmaps is 0 when running there.
I'm not sure if that's the case for Xwayland in general.

I tried a quick hack to enable MIT-SHM shared pixmaps in Xwayland, it seems to work well so far.

Hack
diff --git a/hw/xwayland/xwayland-present.c b/hw/xwayland/xwayland-present.c
index 666ea15e7aaf..2cc2728dd569 100644
--- a/hw/xwayland/xwayland-present.c
+++ b/hw/xwayland/xwayland-present.c
@@ -400,10 +400,14 @@ xwl_present_check_flip2(RRCrtcPtr crtc,
 {
     struct xwl_window *xwl_window = xwl_window_from_window(present_window);
     ScreenPtr screen = pixmap->drawable.pScreen;
+    struct wl_buffer *buffer = xwl_glamor_pixmap_get_wl_buffer(pixmap);

     if (!xwl_window)
         return FALSE;

+    if (!buffer)
+        return FALSE;
+
     /* Can't flip if the window pixmap doesn't match the xwl_window parent
      * window's, e.g. because a client redirected this window or one of its
      * parents.
diff --git a/hw/xwayland/xwayland-screen.c b/hw/xwayland/xwayland-screen.c
index c18e2fbb5a6f..bb18e5c94fbc 100644
--- a/hw/xwayland/xwayland-screen.c
+++ b/hw/xwayland/xwayland-screen.c
@@ -52,6 +52,10 @@
 #include "xwayland-present.h"
 #include "xwayland-shm.h"

+#ifdef MITSHM
+#include "shmint.h"
+#endif
+
 #include "xdg-output-unstable-v1-client-protocol.h"
 #include "viewporter-client-protocol.h"
 #include "xdg-shell-client-protocol.h"
@@ -662,6 +666,10 @@ xwl_screen_init(ScreenPtr pScreen, int argc, char **argv)

     fbPictureInit(pScreen, 0, 0);

+#ifdef MITSHM
+    ShmRegisterFbFuncs(pScreen);
+#endif
+
 #ifdef HAVE_XSHMFENCE
     if (!miSyncShmScreenInit(pScreen))
         return FALSE;


network verify-egress also supports osdtrial

osd-network-verifier also returns valid results for osdtrial clusters, so adding it to the allowed list.

Weird that this hasn't merged yet :/
/lgtm