token and client_id from InAppBrowser not persisted in ionic app on android

After reading the comment of @nbrustein here, I started to better understand the communication between ng-token-auth and the inAppBrowser. If I print the response from ng-token-auth's handleLoadStop function to the console I see that it contains the full user object including the auth_token and client_id (which corresponds to the last client_id in the database). Therefore I think I can conclude that the communication with InBrowserApp works fine and that the ionic app has access to the data.
However, any subsequent API requests to my server receive a 401 Not Authorized response. When I look at the request headers of those API requests I don't see any of the headers that Chrome on my laptop is sending to the server such as: access-token, client, expiry, token-type and uid.
If I print to the console the key and value of the persistData function when running on android and with InAppBrowser mode then immediately after the callback I see the correct data, e.g.:
key: auth_headers
value: Object {access-token: "<token>", token-type: "Bearer", client: "<client>", expiry: "1442773182", uid: "<uid>"}

However, on subsequent requests I see this:
key: auth_headers
value: Object {}

I also see that it tries to save the auth_headers in a cookie because it enters the switch default of the persistData function. Therefore I assume that somehow saving data through ipCookie does not work properly on ionic/android.

Woohoo I got it working! Turns out that when working with ionic and I assume it is the same with cordova, storing the tokens according to default behavior in the cookie does not work. However, once you configure it to work with localStorage it works great!

@abhayastudios I still getting error, can you help me?
external/chromium/net/disk_cache/storage_block-inl.h:119: [0909/210718:WARNING:storage_block-inl.h(119)] Failed data load.
W/chromium( 1032): external/chromium/net/disk_cache/storage_block-inl.h:119: [0909/210718:WARNING:storage_block-inl.h(119)] Failed data load.
W/chromium( 1032): external/chromium/net/disk_cache/storage_block-inl.h:119: [0909/210718:WARNING:storage_block-inl.h(119)] Failed data load.
E/chromium( 1032): external/chromium/net/disk_cache/entry_impl.cc:904: [0909/210718:ERROR:entry_impl.cc(904)] Failed to save user data
I/dalvikvm( 1032): Could not find method android.webkit.WebView.evaluateJavascript, referenced from method org.apache.cordova.inappbrowser.InAppBrowser$3.run
W/dalvikvm( 1032): VFY: unable to resolve virtual method 223: Landroid/webkit/WebView;.evaluateJavascript (Ljava/lang/String;Landroid/webkit/ValueCallback;)V
D/dalvikvm( 1032): VFY: replacing opcode 0x6e at 0x002e
W/PluginManager( 1032): THREAD WARNING: exec() call to InAppBrowser.injectScriptCode blocked the main thread for 27ms. Plugin should use CordovaInterface.getThreadPool().
D/CordovaLog( 1032): file:///android_asset/www/cordova.js: Line 1058 : processMessage failed: Error: TypeError: Illegal constructor
I/Web Console( 1032): processMessage failed: Error: TypeError: Illegal constructor at file:///android_asset/www/cordova.js:1058
D/CordovaLog( 1032): file:///android_asset/www/cordova.js: Line 1059 : processMessage failed: Stack: TypeError: Illegal constructor
D/CordovaLog( 1032):     at file:///android_asset/www/lib/ng-token-auth/dist/ng-token-auth.js:353:24
D/CordovaLog( 1032):     at Object.callbackFromNative (file:///android_asset/www/cordova.js:292:54)
D/CordovaLog( 1032):     at processMessage (file:///android_asset/www/cordova.js:1053:21)
D/CordovaLog( 1032):     at Function.processMessages (file:///android_asset/www/cordova.js:1090:13)
D/CordovaLog( 1032):     at pollOnce (file:///android_asset/www/cordova.js:955:17)
D/CordovaLog( 1032):     at pollOnceFromOnlineEvent (file:///android_asset/www/cordova.js:945:5)
I/Web Console( 1032): processMessage failed: Stack: TypeError: Illegal constructor
I/Web Console( 1032):     at file:///android_asset/www/lib/ng-token-auth/dist/ng-token-auth.js:353:24
I/Web Console( 1032):     at Object.callbackFromNative (file:///android_asset/www/cordova.js:292:54)
I/Web Console( 1032):     at processMessage (file:///android_asset/www/cordova.js:1053:21)
I/Web Console( 1032):     at Function.processMessages (file:///android_asset/www/cordova.js:1090:13)
I/Web Console( 1032):     at pollOnce (file:///android_asset/www/cordova.js:955:17)
I/Web Console( 1032):     at pollOnceFromOnlineEvent (file:///android_asset/www/cordova.js:945:5) at file:///android_asset/www/cordova.js:1059
D/CordovaLog( 1032): file:///android_asset/www/cordova.js: Line 1060 : processMessage failed: Message: S01 InAppBrowser125295987 [{"uid":"116642106272838797656","nickname":null,"image":"https:\/\/lh6.googleusercontent.com\/-oYjG0jh0_uY\/AAAAAAAAAAI\/AAAAAAAAAAA\/ENPxne79kt4\/photo.jpg?sz=50","auth_token":"NwV72kSXhKOjwNzpYM4bqQ","provider":"google_oauth2","expiry":1443042440,"message":"deliverCredentials","id":1,"updated_at":"2015-09-09T21:07:20.630Z","email":"denilson.silva@flexait.com.br","name":"Denilson Silva","config":null,"created_at":"2015-09-08T16:56:28.993Z","client_id":"pag3GyW7ih43Nyt33D2rHw"}]
I/Web Console( 1032): processMessage failed: Message: S01 InAppBrowser125295987 [{"uid":"116642106272838797656","nickname":null,"image":"https:\/\/lh6.googleusercontent.com\/-oYjG0jh0_uY\/AAAAAAAAAAI\/AAAAAAAAAAA\/ENPxne79kt4\/photo.jpg?sz=50","auth_token":"NwV72kSXhKOjwNzpYM4bqQ","provider":"google_oauth2","expiry":1443042440,"message":"deliverCredentials","id":1,"updated_at":"2015-09-09T21:07:20.630Z","email":"denilson.silva@flexait.com.br","name":"Denilson Silva","config":null,"created_at":"2015-09-08T16:56:28.993Z","client_id":"pag3GyW7ih43Nyt33D2rHw"}] at file:///android_asset/www/cordova.js:1060

I created a new application and works.
Thanks

I have moved auth_header storage from cookies to localStorage, but I still get the freeze on Redirecting... page. @dtelaroli, can you please put some sample code to see how you handle authentication? FTR, I'm using plain Cordova, not Ionic.

@aL3xa To fix the freeze I increase the timeout from the devise view https://github.com/lynndylanhurley/devise_token_auth/blob/master/app/views/devise_token_auth/omniauth_external_window.html.erb.
Create the view in your project but with 3000 milliseconds to test (line 31).
Folder: app/views/devise_token_auth
File: omniauth_external_window.html.erb

@dtelaroli, thanks for the update, but I still get the freeze after increasing the timeout value. Apparently, message event is not being fired at all, hence the page is stuck on Redirecting.... I have only these Cordova-specific JS lines:
'use strict';

document.addEventListener('deviceready', function () {
  window.open = window.cordova.InAppBrowser.open;
  console.log('console.log works well');
}, false);

The Angular config section is pretty straightforward:
  .config(function($authProvider, ENV){
    $authProvider
      .configure({
        apiUrl: ENV.apiEndpoint,
        omniauthWindowType: 'inAppBrowser',
        storage: 'localStorage',
        authProviderPaths: {
          google:   '/auth/google_oauth2',
          facebook: '/auth/facebook',
          twitter: '/auth/twitter',
          osm: '/auth/osm'
        }
      });
  })

Any ideas what may have caused "the freeze"?

@aL3xa is it working if you debug at chrome? Can you find the logcat and chrome console for errors?
I configured for tests in android emulator and chrome (If your code is working fine at chrome the problema is confirmed at inAppBrowser).
It's possible test outside emulator to easy debugging (it's required configure your api account to localhost). I use the ionic serve to do that:
$authProvider.configure({
      omniauthWindowType: window.cordova == undefined ? 'newWindow' : 'inAppBrowser',
      storage: 'localStorage'
  });

Can you share one small project with your code connecting to an api test server?

I haven't tried it in Chrome (yet), as I'm debugging on android v4.1.x, but weinre returns something that might come handy:
Error in Success callbackId: InAppBrowser1226996874 : TypeError: Illegal constructor

I'll try it out on vanilla project and see if it works.


Make GitHub pages

I think we can make GitHub pages for the viewer.
So, users can access the web page and upload their benchmark data to see it with the beautiful UI.

Done.


Removed control plane replicas from values

Fixes https://github.com/giantswarm/roadmap/issues/1281

/test create


Add xcodeproj

Project URL
https://github.com/tuist/xcodeproj
Category
Xcode
Description
Add xcodeproj to the list of Xcode projects.
Checklist

[x] Only one project/change is in this pull request
[x] Has unit tests, integration tests or UI tests
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 4 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Warning

:warning:
Found 3 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

1211
301
https://github.com/prynt/PryntTrimmerView redirects tohttps://github.com/HHK1/PryntTrimmerView

1570
301
https://github.com/facebook/facebook-ios-sdk redirects tohttps://github.com/facebook/facebook-objc-sdk

2910
Dupe
https://github.com/tuist/xcodeproj

  Generated by :no_entry_sign: Danger

@pepibumur Seems it is already added)

Duplicate one.
Thanks @serhii-londar


Add Login button

Can I work on this  ?

yes you can start working on it

Kindly add the hacktoberfest tag

Hey ,can you pls add the hacktoberfest tag so that this contribution would count towards it.
Thanks!!


All free youtube music locally downloaded can't be played anymore

What version of Music Assistant has the issue?
2022.7.2
The problem
I downloaded locally in /media some mp3 from youtube music in the past. Now I can't play them in MA.... nothing  happens when I click on them
How to reproduce
Download a Youtube Music into your local dir
Relevant log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2022.7.7
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
ODROID

Can you confirm that you are otherwise using premium YouTube music?

I think the title is just really misleading. He just means local mp3 files. The youtube part is completely irrelevant

Yes ok I see now. Will mark it as more info needed as playing a music file is fundamental to MA!

Ok I will try to be more clear. From youtuve editor, I downloaded in the past some MP3 and copied them in my HA/media players. Until 2022.7.2, I was able to play them as a normal mp3. Now... impossible to play them in MA but I can play them with the default HA media tool.....

What player?
There is another issue reported about VLC which I'm working on. Also Kodi will be affected by this bug as well as browser based players.

What player?
There is another issue reported about VLC which I'm working on. Also Kodi will be affected by this bug as well as browser based players.

The original media player of HA

I've release version 2022.7.4 which should fix the playback issues.


Revamp empty state in Model Serving Global

Description

Closes #1091
How Has This Been Tested?

Delete all Model Servers in all projects
Navigate to the Model Serving global page
Check that the empty state is changed

Test Impact

Added coverage for empty state
Request review criteria:

Self checklist (all need to be checked):

[X] The developer has manually tested the changes and verified that the changes work
[X] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[X] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

If you have UI changes:

[X] Included any necessary screenshots or gifs if it was a UI change.
[X] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

@lucferbux here's some updated text for this empty state:
"No deployed models yet
To get started, deploy a model from the Models and model servers section of a project.
[Select a project]"
As for the layout questions, I'm not sure - maybe @vconzola would have an opinion :-)

@lucferbux please recreate your PR, I have reverted this in #1834


cleanup report, and puppet supported regex

@mhaskel can you please have a look

@mhaskel apologies, cleaned up

@tphoney thanks! Could you also squash/rebase?

apologies @mhaskel i mashed the keyboard badly :(

thanks @tphoney !


Spike: Explore sharing of immutable values

This explores sharing of value objects that are (in master) heavily duplicated in every task isolate.

@bot-gradle test APT

I've triggered the following builds for you. Click here to see all build failures.

All Performance Tests (Trigger)


Does it work as a browser add-on?

Loom works as an addon of a browser. Is Clipshare not one? How does it work then?

Hi there, Loom has 2 flavors, browser extention and desktop app, and the desktop app is the recommended one. https://support.loom.com/hc/en-us/articles/360007405538-The-Loom-recording-platforms
Clipshare only supports desktop app


Update Livy to version 0.8.0

https://github.com/GoogleCloudDataproc/initialization-actions/blob/27e99c923cdc6fc1bd4e0ff22722d93e74605a57/livy/livy.sh#L17
It's not possible to use Livy with Dataproc 1.5+ due to Scala version change. Livy supports Scala 2.12 from 0.8.0 version.

0.8.0 is not a released Livy version, that's why we can not update to it: https://search.maven.org/search?q=g:org.apache.livy


Testing go-micro for backend

We should test if it is possible to use go-micro for the backend. Specially in client side using flutter and dart.

Make when time started and ended.
Make a PR as part if this please.

start

end

Not using go.micro now


fix: improve TouchableItem press animation

Fixes #11

Would be awesome if you could also add the propType.

Forgot about it.

Merged. Thanks :D


Unable to download all submodules

Hi,
I tried downloading the BAG2 and followed the instructions for BAG2_cds_ff_mpt.
I ran the command: git submodule update --init --recursive
However, It throws an error about not being able to find a repository and says:
ERROR: Repository not found.
fatal: The remote end hung up unexpectedly
Clone of 'git@github.com:ucb-art/bag_analog_ec.git' into submodule path 'bag_analog_ec' failed.
I have attached a screenshot of the problem I am facing.
Can you please help to fix the issue ?
Thank you.

Having this same issue, was there a solution?


[WIP] Tfx cli 101

How the second step looks like.

How the second step looks like.

Yeah good idea to generate these beforehand!
If you are talking about setting up AI Platform Notebook instances then I think you could still refer them to existing materials. Several Codelabs do cover that step. I am mentioning this because that might help you to make the content even more compact yet precise.

@sayakpaul
Done with the draft version. Please take a look. Below is the screenshots of each page.
Page1: https://ibb.co/19SQT2n
Page2: https://ibb.co/dmbRwPH
Page3: https://ibb.co/1LKzLYM
Page4: https://ibb.co/LCr5cmQ
Page5: https://ibb.co/8MX75BG

@sayakpaul
sure! I will add the output after merging the PR. Thanks for your time to review on this!


bug: Strange download verification instruction characters

The verification instructions on http://apisix.apache.org/downloads/ include the following text:
gpg --verify apache-apisix-**\*\*\*\***.asc apache-apisix-****\*****
This looks confusing on the HTML page (but perhaps it looks OK in markdown?)
There are some other lines with the same issue

@juzhiyuan

@juzhiyuan

oh yes, let me fix this

oh yes, let me fix this


 Anyone can tell me the problem?
)

Thank you for reporting your issue.
Your code is wrong.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "Help" button on the challenge on freeCodeCamp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification.
Thank you, and happy coding.

thanks..got it..typo error..will post in help chatroom next time

    max=array[i].length;


Update to support per PWM output rate, vector servo update, and DSHOT…

Per PWM output rate
Single call to update all PWM's
DSHOT frequencies (and protocol)
Also fixed a missing check for 'M' servo type.
Increased number of possible mixer servos from 8 to 10

I don't quite understand everything you're doing, could you explain your changes more?

Servo Frequency Update:
The previous version of PWM and MIXER:

Did not support DSHOT-type servo/motor channels
Specified a single period for all pwm whether they were motors operating at 490Hz or Servos at 50Hz.
Updated the channel PWM's in independent calls to the board.
Channels in the mixer were updated from 8 to 10 because Varmint has 10 PWM outputs.
The new version
Supports DSHOT protocol
Allows specification of different frequencies for each servo/motor channel (hardware permitting*). If the rate is specified in the range  150000 to 1200000 (bps), it is a DSHOT channel, and 50 to 490 (Hz)_ if it is a PWM channel.
In DSHOT mode, all the channels within a group must be updated simultaneously if very high update update rates are to be achieved.

*On our processors, up to 4 PWM/DSHOT output channels can be assigned to a Timer. All 4 channels must be of the same rate. On the Varmint, PWM Timer groups are 1-4, 5-8, & 9-10, on PixRacer Pro they are 1-4, 5-6, & 7-8.

For now, I set all fixedwing motors to 50hz pwm and multirotor motors to 490hz, with all servos at 50hz. Eventually we will want to revamp the mixing system to make it easy to specify which motors to use dshot and which to use PWM. For now, using DSHOT will require modifying the source code.


Ores change when falling onto torches.

When redstone, diamond, emerald, and nethercoal ore fall onto a torch, they break into coal ore. Iron, gold and lapis lazuli ore do not change when broken this way.

Odd, will have to look at this, there should be no reason for that.
Also, what do you mean "fall onto a torch"? Like when dropped?


Fix mock server

Investigate logs and Kibana dashboard for further understanding on the cause and solution (testnet).

Consensus seems to be properly running for 5 of the 6 nodes of the network. Only dusk-9 seems to be not syncing since 49889 round
Network height
https://kibana.monitoring.dusk.network/app/metrics/explorer?metricsExplorer=(chartOptions:(stack:!f,type:line,yAxisMode:fromZero),options:(aggregation:avg,groupBy:!(agent.hostname),metrics:!((aggregation:avg,color:color0,field:prometheus.dusk_block.value)),source:url),timerange:(from:now-24h,interval:%3E%3D10s,to:now))&waffleFilter=(expression:%27%27,kind:kuery)&waffleOptions=(accountId:%27%27,autoBounds:!t,boundsOverride:(max:1,min:0),customMetrics:!(),customOptions:!(),groupBy:!(),legend:(palette:cool,reverseColors:!f,steps:10),metric:(type:cpu),nodeType:host,region:%27%27,sort:(by:name,direction:desc),source:url,view:map)&waffleTime=(currentTime:1610530486594,isAutoReloading:!f)
Block time:
https://kibana.monitoring.dusk.network/app/metrics/explorer?metricsExplorer=(chartOptions:(stack:!f,type:line,yAxisMode:fromZero),options:(aggregation:avg,groupBy:!(agent.hostname),metrics:!((aggregation:avg,color:color0,field:prometheus.dusk_effective_block_time.value)),source:url),timerange:(from:now-24h,interval:%3E%3D10s,to:now))&waffleFilter=(expression:%27%27,kind:kuery)&waffleOptions=(accountId:%27%27,autoBounds:!t,boundsOverride:(max:1,min:0),customMetrics:!(),customOptions:!(),groupBy:!(),legend:(palette:cool,reverseColors:!f,steps:10),metric:(type:cpu),nodeType:host,region:%27%27,sort:(by:name,direction:desc),source:url,view:map)&waffleTime=(currentTime:1610530486594,isAutoReloading:!f)

Moving this into Parked for monitoring purpose.

From @jules comment "On the DevNet, a node was observed getting stuck after around 43,000 blocks. The cause of this seemed to have been a context deadline being exceeded by the ExecuteStateTransition function. After checking the list of active processes, it looked like the mock rusk server had been restarted, as the uptime was vastly different from the node's uptime. The cause of the crash will need to be investigated and fixed in order to promote a stable DevNet."

The problem repeated itself on another run. Logs state this:
{"count":7,"level":"debug","msg":"collected agreement","process":"agreement","quorum":6,"round":49890,"step":3,"time":"2021-03-01T13:54:40+01:00"}
{"height":49890,"level":"info","msg":"calling ExecuteStateTransitionFunction","process":"accept block","provisioners":7,"time":"2021-03-01T13:54:40+01:00"}
{"error":"rpc error: code = DeadlineExceeded desc = context deadline exceeded","height":49890,"level":"error","msg":"Error in executing the state transition","process":"accept block","time":"2021-03-01T13:54:52+01:00"}
{"error":"rpc error: code = DeadlineExceeded desc = context deadline exceeded","level":"error","msg":"failed to process message","process":"readloop","time":"2021-03-01T13:54:52+01:00"}
{"curr":49889,"level":"debug","msg":"Start syncing","process":"chain","target":49891,"time":"2021-03-01T13:54:52+01:00","tip":49891}

After further discussion with @goshawk-3, it does indeed seem that the mock rusk server has not crashed. The launch date is the exact same as the launch date of the node.
I've attached delve, but to no avail so far. It mostly seems like it just has the standard GRPC server goroutines running, with nothing in particular hanging.


Trigger event

It would be greater if there where and event i HASS on new devices.
When new devices is found on the router, trigger Home Assistant device_tracker_new_device event with data about the device, eg.
trigger.event.data.host_name
trigger.event.data.source_type
trigger.event.data.mac

I found this:
https://community.home-assistant.io/t/new-device-tracker-events/287746/2
I didn't try it yet but I think it can solve the issue.

@fatuuse did it work?

👍


Adding hidesec Portfolio

Please Review change:

Add Hidesec Portfolio

File changes:

Personal Portfolio/Hidesec Portfolio/css/style.css
Personal Portfolio/Hidesec Portfolio/cv/portfolio.html
Personal Portfolio/Hidesec Portfolio/index.html

Good work. I am merging your PR.
Do give us a star by hitting the star button of this repo!


Reflux

Sorry for creating all the issues but I'd love to learn this as your code just looks too nice and clean :)
Have you seen Reflux? It's based on facebook's flux but gets rid of the dispatcher and seems to be easier to understand.
 Just asking because this happens to quite some packages ..

Hey! That looks great, thank you :) Looks like it's time now to start switching to this solution, just have to figure out the best approach :D But I think I will just convert simple pages first and then go on to the more complex ones.

Glad you like it, you were right – it's much nicer that way! I already refactored my code in my Prisma app too :-)


Links with spaces in the target don't work

Put this into http://parsedown.org/extra/
[google](https://www.google.com)

[google]( https://www.google.com )

The first link works fine, but the second link just shows up as plain text. The second link works fine in PHP Markdown Extra but not in Parsedown Extra.

@aidantwoods
Want to create a PR for this?

@erusev can take a look :)


Upgrade to Knative Serving 0.5.1 and Istio 1.0.7

They contain patches for critical Envoy vulnerabilities.
Refs https://github.com/knative/serving/pull/3668

We upgrading to Istio 1.0.7 independent of Knative Serving


Reimplement and fix aurora dump-storage

% aurora dump-storage
...
TypedError: [-32000] Server error: State of contract aurora is too large to be viewed

This repository is no longer maintained. Please visit Aurora CLI for latest release of Aurora CLI. Feel free to submit your issues/proposals there.


Unit test some utils files

[ ] src/utils/prettifiers.js
[ ] src/utils/validators.js
[ ] src/utils/valueFixers.js

I was looking at the functions isPositiveNumber() isPositiveWholeNumber() in src/utils/validators.js.
isPositiveNumber() will return whether or not a string is a number.
isPositiveWholeNumber() will return whether or not a string is a whole number.
But neither will check if the string is even. I looked through the code I'm not sure if they're suppose too.
I was wondering should I change the names to ease confusion or make the function check for even numbers?

@microcat49: We don't need anything to check for even numbers. What are you finding confusing about the names? What in the names implies that they should be returning even numbers?

@knod My bad I'm not sure why I thought that.

@microcat49: No worries, always good to have a fresh and skeptical eye on things.

I believe #366 closed this.


A binary file should be there...

I like this Engine and someday i will probably use it to make some games but i think that generating the VS .sln file and then opening VS, then compiling and running it is a pretty long and complex way (It's just my opinion as i'm not as used to VS a lot).
You should probably upload a simple binary file which will be a bit useful for some people. By doing that some people will just try the current features out, if there's some issue they will either clone the source code or create a Github issue.
But no problem as this engine is pretty good and has a lot of potential in it.

Thank you. Yeah I'd like to do that. I just need to sort how the assets are loaded since the build process sets the directory which isn't ideal

Thank you. Yeah I'd like to do that. I just need to sort how the assets are loaded since the build process sets the directory which isn't ideal

Cool! I think that now you might remember this in future so, it's time to close this issue.


The editor selection scrolling is too fast !

I have installed the last VSC version:  April 2023 (version 1.78)
The editor selection scrolling is too fast !
How to decrease the velocity of selection scroll?
"smoothScrolling" params are in "true". Change does not solve it.
Thank you

The setting editor.mouseWheelScrollSensitivity controls this


:bug: [ Release - 2.10 ] Reduce the footprint of the transporter

Summary
Limit the channel size of the producer and consumer
Related issue(s)
Fixes # https://issues.redhat.com/browse/ACM-10814

/retest

/test ci/prow/gofmt

/cc @clyang82 @ldpliu

/test test-e2e


TypeError: expected callable or DeviceListener

File "C:\Python35\lib\site-packages\myo_python-1.0.3-py3.5.egg\myo_ffi.py", line 527, in run
TypeError: expected callable or DeviceListener

import collections
import myo as libmyo
from myo import Hub
import threading
import time
import numpy as np
import tensorflow as tf
from include.model import model

x, y, output, global_step, y_pred_cls = model()

saver = tf.train.Saver()
_SAVE_PATH = "./data/tensorflow_sessions/myo_armband/"
sess = tf.Session()

try:
    print("Trying to restore last checkpoint ...")
    last_chk_path = tf.train.latest_checkpoint(checkpoint_dir=_SAVE_PATH)
    print(last_chk_path)
    saver.restore(sess, save_path=last_chk_path)
    print("Restored checkpoint from:", last_chk_path)
except:
    print("Failed to restore checkpoint. Initializing variables instead.")
    sess.run(tf.global_variables_initializer())

class MyListener(libmyo.DeviceListener):

    def __init__(self, queue_size=8):
        self.lock = threading.Lock()
        self.emg_data_queue = collections.deque(maxlen=queue_size)

    def on_connect(self, device, timestamp, firmware_version):
        device.set_stream_emg(libmyo.StreamEmg.enabled)

    def on_emg_data(self, device, timestamp, emg_data):
        with self.lock:
            self.emg_data_queue.append((timestamp, emg_data))

    def get_emg_data(self):
        with self.lock:
            return list(self.emg_data_queue)

libmyo.init()
hub = Hub()
start = time.time()
temp = []
try:
    listener = MyListener()
    hub.run(2000, listener)     ----> ERROR

What could the reason? Please help.

You use old code with the new library.
Since I've been getting this a lot recently, I've added a more obvious notice to the README.
https://github.com/NiklasRosenstein/myo-python

I've followed the "Migrating from v0.2.x" section, but I still get the same error. Can you please tell me what changes are required in this code? I will be grateful.

Also just added that to the README -- the order of arguments to run() is reversed. Use hub.run(listener, 2000) instead.

I already reversed the arguments, but I still get "File "C:\Python35\lib\site-packages\myo_python-1.0.3-py3.5.egg\myo_ffi.py", line 527, in run TypeError: expected callable or DeviceListenerTypeError: expected callable or DeviceListener".

Please post the full modified code again

I already reversed the arguments, but I still get "File "C:\Python35\lib\site-packages\myo_python-1.0.3-py3.5.egg\myo_ffi.py", line 527, in run TypeError: expected callable or DeviceListenerTypeError: expected callable or DeviceListener".

You didn't.

    hub.run(2000, listener)

It's an error in your code. The error message is quiet explicit. ;)

@NiklasRosenstein Hello, Can i know the units in which accelerometer and gyroscope values are recorded? (Not regarding to this topic but i didnt know how to reach you)

@vikramharindranath You can just create a new Github issue instead of commenting on an almost 2 years old one 😉 For this question though, the answer is to check the Myo documentation. The values come from the device via the Myo C library, there's no unit conversion or adaption happening in the library.


Add event for when a NetworkedDictionary is changed.

Codecov Report

Merging #128 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #128      +/-   ##
=========================================
- Coverage    8.39%   8.34%   -0.05%
=========================================
  Files          48      48
  Lines        8227    8276      +49
  Branches     1286    1297      +11
=========================================
  Hits          691     691
- Misses       7478    7527      +49
  Partials       58      58

Impacted Files
Coverage Δ

MLAPI/Data/NetworkedCollections/NetworkedList.cs
0% <ø> (ø)
:arrow_up:

...I/Data/NetworkedCollections/NetworkedDictionary.cs
0% <0%> (ø)
: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 126562a...7efedce. Read the comment docs.


ReferenceError: React is not defined

With Solito, following error is shown. Even after importing react in the file, this error doesn't go away

did you follow the Next.js instructions?

Fixed in 1.0.2.


iOS 6s+

I'm using react-native-orientation and it works great on my iphone 6s. It doesn't seem to recognize events consistently on ios 6s-plus, though.
I wanted to check to see if this is an implementation error on my part or an issue. Have people been successfully using this library with 6s-plus? Was that included in testing? Thanks in advance.

I think we need more information @toddmetheny - were you able to resolve this or do you still have an issue?

Closing this as it's been opened for over a year and the actual iOS device type shouldn't matter. @toddmetheny please try with the latest package version (check README for more info)

i face the same problem. just in 6s-plus


feat: Sign 2.5

PR adding Sign 2.5 in web3modal/ethers5 & web3modal/siwe

on siwe request, sign 2.5 authentication is used by default - if it's not supported by the wallet it fallbacks to legacy
all web3modal/siwe behaviour preserved for backwards compatibility
new getMessageParams client method is added to createSIWEConfig so the params that compose siwe message can be passed to Sign for multi-chain authentication
replaced siwe package with @walletconnect/utils as recaps in siwe statement was not supported

to test
wallet (on sign 2.5) - https://react-wallet-v2-kmnfc86q1-walletconnect1.vercel.app/walletconnect
legacy - any other supporting v2 protocol

@ganchoradkov @glitch-txs @tomiir @enesozturk
This PR poses a few issues.
First, it is not backwards compatible with previous SIWE implementations. The new function requirement of getMessageParams should be optional instead.
Second, it makes current latest deployed docs outdated, one example being, that the new EIP155 standard eip155:chainId:address format is not compatible with latest siwe standard (and hence the siwe recommended npm lib for react, for e.g.). See https://eips.ethereum.org/EIPS/eip-4361 under "address". This should not be the default behaviour, and would cause another breaking change for many users (and one that isn't actually easily fixable, unless you force migration to eip155 address).
IMO, these behaviours should be corrected in a future release.


Update Service Account Roles

The service account running GCP benchmarks requires bigquery.tables.list

seems like gcp provider had a mayor release, with breaking changes on some attributes (before optional, now required).
adapted the version requirements and checks passing.


Bug located in object_list, when there are points in the model

Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
In the list object_list of Modeler3D, when there are points in the model, there is a "None" for each point in the object_list.
We are not sure if there should be an Object3D entry in the list for each point (since the points are in 3d space) or if they should not be reflected at all.
Steps To Reproduce
Create a model which includes points. Look at the object_list, in our case we see a "None" entry in the list for each point.
Which Operating System are you using?
Windows
Which Python version are you using?
3.8
Installed packages
ansys-aedt-wrappers-restapi==0.27.7
bottle==0.12.25
build==0.10.0
certifi==2023.5.7
cffi==1.15.1
charset-normalizer==3.2.0
clr-loader==0.2.6
colorama==0.4.6
contourpy==1.1.0
cycler==0.11.0
fonttools==4.41.0
idna==3.4
imageio==2.31.1
importlib-metadata==6.8.0
importlib-resources==6.0.0
kiwisolver==1.4.4
matplotlib==3.7.2
numpy==1.24.4
packaging==23.1
Pillow==10.0.0
platformdirs==3.9.1
plumbum==1.8.2
pooch==1.7.0
psutil==5.9.5
pyaedt==0.6.90
pycparser==2.21
pyparsing==3.0.9
pyproject_hooks==1.0.0
python-dateutil==2.8.2
pythonnet==3.0.2
pyvista==0.38.5
pywin32==306
requests==2.31.0
rpyc==5.3.1
scooby==0.7.2
six==1.16.0
tomli==2.0.1
urllib3==2.0.3
vtk==9.2.6
waitress==2.1.2
zipp==3.16.2

Hi @jonas-norlin ,
I created a draft PR #3581. Could you please try, test it and let me know if it works as intended?
Thanks!

@gmalinve It works thank you


How can i get this to work with mdsvex in a Sveltekit project?

Hey there,
I've got a Sveltekit project that uses mdsvex to process the markdown files. (mdsvex supports all rehype and remark plugins.)
https://mdsvex.pngwn.io/docs#remarkplugins--rehypeplugins

Installing with pnpm flags up missing peer dependencies. Do all of these packages need to also be installed alongside your package?

To use plugins i add them to the svelte.config.js file. Below is an example of the config file with rehypeSlug and rehypeAutolinkHeadings plugins (both working).

Should this package work if i simply add it to my config? Thus far, I cant get it to work.

Any help appreciated, thanks!
Alex

@flintec-studio

Do all of these packages need to also be installed alongside your package?

My packages are designed to be used with remark/mdast/micromark-gfm-* and unified. Usually, you won't need to install all of them manually. If your platform claims to support remark / rehype plugins, this generally means that core packages like unified, remark-parse, and micromark are already installed with it.
However, as I observe mdsvex's dependencies, they appear to utilize older versions of the unified / remark packages, which are not compatible with my plugin.
https://github.com/pngwn/MDsveX/blob/master/packages/mdsvex/package.json

Should this package work if i simply add it to my config?

Without unified-related compatibility issue, I don't think my plugin works with mdsvex.
My plugin needs to add handlers  in remark-rehype plugin, but they don't have API for it.
Regards


Speed Enhancements needed

Description
Files takes about 2 seconds to load Drives widget on startup.
Files takes about 1 to 2 seconds to show the full context menu items on right click, it is fast and good on right click and shows some items immediately, but takes time to load the rest.
would be nice if they can get fixed as they are frequently faced while using the app.
Steps To Reproduce
Issue1- Open Files with home tab as default, Drivers widget takes some time to load.
Issue2- Right Click any item that has lots of options in its context.
Expected behavior
Issue1- Drivers widget available right away just like windows explorer.
Issue2- All items available with no lag.
Files Version
2.0.34
Windows Version
Win11 21H2
Relevant Assets/Logs

Merging with #4180

We're excited to share that startup performance will be significantly improved in the upcoming release of version 3. You can download the preview version from our website https://files.community/download.


What The Hell Is Going On With SourceWindow?

I'm trying to add a feature where if you do :q on the main buffer (the code) that it closes all the other buffers and closes the connection. Slightly problem is that I can't seem to hook in to it's close event with autocmd BufferWinLeave. If I do autocmd BufferWinLeave \* echo 'Closed', it'll echo for doing a :q on every buffer except the SourceWindow buffer. I'm not all that good at vimscript yet, and I'm still kind of trying to figure out the vim module and how your WIndow classes work, so I'm not sure what's happening here. Can you shed some light on how the windows are created, what's different about SourceWindow and why I can't hook in to it's closing with autocmd?

Okay, so I know (kind of) how to reproduce this simply.
But opening vim, typing :tabnew, followed by :1buffer and then :new Window, we get the same result, where Buffer 1 is open in both Tab 1 and Tab 2. Doing a au BufWinLeave * echo 'test' and trying to do :q on Buffer 1 inside Tab 2 doesn't tigger the autcmd. Now... how do I fix this?

Closing this as I have a solution. Basically what's hapening is that I want to hook in to the Window of the Buffer closing, but there's no WinClose event to hook in to. So, instead, I've set up a WinEnter event which then causes the debugger to check if the SourceWindow is still open, and if it's not, close the debugger. As part of the close, it removes it's au hook so the hook doesn't hang around. Crude, needs some optimisation, but I'll work on it


Generated autorest.csharp models don't retain required attributes.

We are using a micro-service architecture where services are dependent on other services generated c# clients. At the very top of our stack is an Angular application that is dependent on a typescript client whose service it was generated from is dependent on other c# clients. Our problem is that the generated typescript models, whose typescript was generated from the c# clients models, all have nullable fields.
Is there a way to retain the [Required] attribute from the original models in the generated c# client?
And example of a property in our swagger:
"Address": {
    "required":["country", "region, "city", "street", "postalCode"],
    "type":"object",
    "properties":{
        "country":{
            "title":"Enums.CountryCode","x-ms-enum":{
                "name":"CountryCode","modelAsString":false
            }
        },
        "region":{
            "type":"string"
        },
        "city":{
            "type":"string"
        },
        "postalCode":{
            "type":"string"
        },
        "street":{
            "type":"string"
        },
        "street2":{
            "type":"string"
        }
    }
}

And the resulting c# properties:
        /// <summary>
        /// Gets or sets claims.Data.Enums.CountryCode
        /// </summary>
        [JsonProperty(PropertyName = "country")]
        public object Country { get; set; }

        /// <summary>
        /// </summary>
        [JsonProperty(PropertyName = "region")]
        public string Region { get; set; }

        /// <summary>
        /// </summary>
        [JsonProperty(PropertyName = "city")]
        public string City { get; set; }

        /// <summary>
        /// </summary>
        [JsonProperty(PropertyName = "postalCode")]
        public string PostalCode { get; set; }

        /// <summary>
        /// </summary>
        [JsonProperty(PropertyName = "street")]
        public string Street { get; set; }

        /// <summary>
        /// </summary>
        [JsonProperty(PropertyName = "street2")]
        public string Street2 { get; set; }

+1 also need a way to add attributes to properties/members in generated c#


Version seems to be incorrect, 1.0.1 does work

Description

If you haven't already, read the full contribution guide. The guide may have changed since the last time you read it, so please double-check.
Adding a new module

[ ] A proposal has been submitted and approved.
[ ] I have included "Closes #{module_proposal_issue_number}" in the PR description.
[ ] I have run brm validate locally to verify the module files.
[ ] I have run deployment tests locally to ensure the module is deployable.

Updating an existing module

[ ] This is a bug fix:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[ ] I have run brm validate locally to verify the module files.
[ ] I have run deployment tests locally to ensure the module is deployable.
[ ] I have read the Updating an existing module section in the contributing guide and updated the version.json file properly:

[ ] The PR contains backwards compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json.
[ ] The PR contains backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] The PR contains breaking changes, and I have bumped the MAJOR version in version.json.

@fberson do you mind regenerating the module files (see the instructions) to fix the module validation error? We just published Bicep 0.5.6 which is used by the CI pipeline, but the module v1.0.1 was generated using an older version of Bicep.


Convert middleware to request handler

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
✔

@xepozz readme should be adjusted. Afterwards, it's good to merge.

Done


When  creating a new schema with ent new, please don't return nil

When you use ent new to create a new schema in ent/schema/.go,
something like this is generated:
package schema

import "entgo.io/ent"

// Video holds the schema definition for the Video entity.
type Video struct {
	ent.Schema
}

// Fields of the Video.
func (Video) Fields() []ent.Field {
	return nil
}

// Edges of the Video.
func (Video) Edges() []ent.Edge {
	return nil
}

It returns nil, but instead it should return an empty slice, imho.
Imagine you create 10 schemas, now you go through all the schema, and every time
you have to copy the []ent.Field and replace the nil and copy the []ent.Edge and replace the nil.
If instead the generated schema would look like this:
package schema

import "entgo.io/ent"

// Video holds the schema definition for the Video entity.
type Video struct {
	ent.Schema
}

// Fields of the Video.
func (Video) Fields() []ent.Field {
	return []ent.Field{}
}

// Edges of the Video.
func (Video) Edges() []ent.Edge {
	return []ent.Edge{}
}

a lot less typing and copy/pasting would be involved.
This might seem like a small issue, but it's a big issue when you have to edit many schemas.
It would help reduce overhead.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Summary 💡

Motivation 🔦
reduced typing fatigue

Note you can pass a template option to the new command and control generated base schema for your projects (e.g., mixins or privacy auto included, etc).


unexpected end of file error w/ basemap-1.2.0

I see this with both the 1.2.0 and 1.1.0 tarballs linked from the download page,
https://github.com/matplotlib/basemap/releases/
although the end of file error occurs at different points in the file.
I'm working on Fedora 29
Here is a screenshot:

I tried the 1.2.0 and 1.1.0 *.zip files as well, and I get the following error when trying to unzip them on Fedora 29:

What are your checksums? Most likely, the downloads were corrupted, because
those tarballs have been up for several years.
On Sun, Jun 30, 2019 at 1:53 PM David Bruhwiler<EMAIL_ADDRESS>wrote:

I tried the 1.2.0 and 1.1.0 *.zip files as well, and I get the following
error when trying to unzip them on Fedora 29:
[image: image]
https://user-images.githubusercontent.com/3825435/60400169-a07eaa00-9b2d-11e9-86a6-82d539256c40.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/matplotlib/basemap/issues/464?email_source=notifications&email_token=AACHF6HKFUNRFC77N5LCLL3P5DXIRA5CNFSM4H4NCOU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODY4QVFY#issuecomment-507054743,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACHF6DIFJGEESBONJWR2B3P5DXIRANCNFSM4H4NCOUQ
.

Thanks @WeatherGod
You're probably right.
As a work around, I'm cloning the GitHub repo and skipping the tarballs.
Closing the issue.


Incompatible with TooManyEmotes

After emoting, players (non-host) can see their own model inside of them.

Possibly fixed with 0.3.3 since there was an error, there shouldn't be an actual incompatibility.


[5.5] Always extend AbstractTestCase

This PR introduces AbstractTestCase, this way:

we can get rid of the test autoload

and in each testcase we can remove all the setUp/tearDown code

to set-up Carbon
to reset Carbon
to close Mockery

What is the reason for getting rid of the tests/bootstrap.php file?
This seems useful, as it sets some carbon configuration and php configuration to make tests consistent.

@alexbowers the settings in bootstrap.php are also set in AbstractTestCase::setUp (setTestNow, timezone)
Then it does make sense to keep bootstrap.php.

While I appreciate the work here I just am constantly bit be these huge PRs to make these kinds of changes. Merge conflicts for months, etc.


Roll Flutter Engine from ef5c349f04d8 to 435db1f6764c (3 revisions)

https://github.com/flutter/engine/compare/ef5c349f04d8...435db1f6764c
2023-05-12<EMAIL_ADDRESS>[Impeller] Dont synthesize fake index buffer if none was provided to VBB (flutter/engine#41969)
2023-05-12<EMAIL_ADDRESS>[Impeller] Make index buffer optional (flutter/engine#41962)
2023-05-12<EMAIL_ADDRESS>Roll Dart SDK from 7ad028c26344 to 4028c7d8b87c (8 revisions) (flutter/engine#41967)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC aaclarke@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Add 'restart: always' to Docker Compose file so that Postal starts on boot

In almost all cases, Postal should start on boot. This commit avoids users from having to think of their own solutions (see https://github.com/postalserver/postal/issues/28 and https://github.com/postalserver/postal/issues/125).
If the comment https://github.com/postalserver/postal/issues/1984#issuecomment-1122299893 is still true, then at least it would be useful to add the most common way to achieve this to the documentation.

That would skip postal's control script /usr/bin/postal.
I believe more clean solution is to add systemd service as described here https://github.com/postalserver/postal/discussions/1610#discussioncomment-1411273
and add restart: unless-stopped restart policy with this PR https://github.com/postalserver/install/pull/3

This has now been added

Why? How is this better than https://github.com/postalserver/install/pull/3
?
This approach breaks /usr/bin/postal functionality as stated above.

I added it as "unless-stopped"


BigInts are not fully supported

Description
RN 0.70 introducted partial bigint support: BigInt(1) + BigInt(2) now works.
But it is not full. 1n + 2n still doesn't work.
This is a big deal.
This is continuation of facebook/react-native#34603 which was unnecessarily closed.
React Native Version
0.71.0

Moving back to React Native.
We need some investigation to understand what's going on with BigInt.
A repro would be a good starting point. Here a template to create them: https://github.com/react-native-community/reproducer-react-native

@paulmillr I haven't been able to reproduce this on iOS with Hermes, RN 0.71.6, with an out-of-the-box react-native init app. I've also checked that production bundles are correctly minified with both terser (the new default) and uglify-es (the old one). A colleague has checked Android as well. Could you share a bit more?

@robhogan if you can't reproduce, that's another question.
On it. Will repro and ping you back.

Hey, @paulmillr -- thanks for your persistence here.
To provide a bit of context, while we do get paid to work on React, historically we haven't been able to prioritize incoming issue triage compared to the work on already confirmed issues. This may come across as bureaucratic, but it was mostly due to support being ad-hoc and dependent on individual engineers' availability. This is something we're working to fix. We finally have a dedicated team for managing our OSS repos/issues, and I expect this to improve this year. 🎉
We appreciate your work on Chokidar. I know doing OSS work can feel pretty thankless but you're doing awesome work.

Appreciate for understanding folks.
I've just tested it: the issue can be reproduced by running npx tsc in the project directory, or using an editor, linter, other tool (vscode). Not sure what's the best way to commit this to run on github CI.
Reason for this is the default tsconfig setting: lib: es2019. es2019 does not support bigints, only es2020 does. Bigints seem to work both with and without hermes; there is no necessity to use es2019 at this point.

Ah, that makes sense! Thanks - we’ll get that fixed.

@lunaleaps @rickhanlonii should both have merge permissions to https://github.com/tsconfig/bases/blob/main/bases/react-native.json
Note that, at least in the default template we provide, tsc is used for type checking, but not code generation.

We’ll want to be a bit careful about the choice here too - we support parts of ES2020 on all engines (after transforms), including es2020.bigint, but not others (es2020.sharedmememory?), so entirely switching to es2020 isn’t going to be quite right.
tsconfig.json allows us to pick and choose, so we’ll need to check our capabilities against the choices here:
https://github.com/microsoft/TypeScript/tree/main/src/lib

@paulmillr while we get that sorted out, are you unblocked by adjusting your project config tot use es2020.bigint?

@blakef
"target": "es2019",
"lib": ["es2019", "es2020.bigint"],

^ This doesn't work for bigint literals (123n).
"target": "es2020",
"lib": ["es2019", "es2020.bigint"],

^ This works properly for bigint literals and non-literals.
Target is what defines syntax; lib defines features if i'm not mistaken.

The template tsconfig already uses target: esnext, so no changes needed to target.
There are some libs we should add like "bigint" and "intl". Will do a quick audit of what polyfills we should add.

https://github.com/reactwg/react-native-releases/discussions/54#discussioncomment-5650114 requests picking into 0.72 before we release.


test web deployment

Testing: Web Deployment Testing
Description
Following the deployment of the web application to the production environment, it is essential to conduct comprehensive testing to ensure that all functionalities are working correctly and that the application operates reliably for users. Testing the deployed web application involves executing various test scenarios to validate its performance, functionality, and user experience under real-world conditions. By thoroughly testing the deployed web application, we can identify and address any issues or discrepancies before they impact user satisfaction.
Tasks

[ ] Execute functional tests to verify that user interactions, such as task creation and preference settings, function as expected.
[ ] Perform compatibility testing across different web browsers and devices to ensure a consistent user experience.
[ ] Conduct accessibility testing to assess the web application's compliance with accessibility standards and guidelines.
[ ] Validate responsive design and layout across various screen sizes and resolutions.
[ ] Monitor web server logs and error reporting tools during testing to detect and troubleshoot any issues or errors.
[ ] Document test results, including any bugs or discrepancies encountered during testing.
[ ] Collaborate with development and QA teams to address identified issues and implement necessary fixes.

Related Issues

#IssueNumber - Deployment: Web Deployment

Notes

Testing the deployed web application is crucial to delivering a reliable and user-friendly experience to our users.
Test scenarios should cover both typical user interactions and edge cases to uncover any potential issues.
Regular testing and monitoring of the deployed web application are essential to maintain optimal performance and usability.

Deadline
[Insert Deadline for Testing]

This works fine


Release/1.3

Release 1.3
Updated package.json
Ran npm audit fix
CHANGELOG updates

--build-path -> --scratch-path change is only valid in Swift 5.8 or later because SourceKit-LSP (5.7) doesn't understand --scratch-path

@swift-server-bot test this please


Added image for projectile motion

The projectile motion page seemed underspecified due to lack of image to depict the concept visually.
Fixes: #563
Uploaded image for projectile motion(no copyright associated,made it on my own in Paint 3D) in webp format. Couldn't add IT to the website as HTML codes are not visible.

[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.

@BabyElias u have added the image but u haven't added it in the website

@Amit366 Sir, I am unable to find the HTML file to the page.So, if you could guide me through it I will add the image to the website.

@BabyElias there is no HTML file. Its done in react

Go to classical mechanics folder and inside that calculator.js file


feat: allow use of sudo in playbooks

Use of sudo in the self-hosted runner should allow running playbooks with become: true
Notes

Enable sudo via dawidd6/action-ansible-playbook

:tada: This issue has been resolved in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Success for download is decoupled from reality of responses downloading or not

Issue / opportunity
Toast/alert showing the download was successful appears before download takes place. When you click "Cancel" and decide not to download the response(s) they still move to "Downloaded", even though they were not.
Screenshot

Related to #2955


Feature/cli/persist clients

Improve cli with additional persistency feature:

added load and save commands to allow the persistence of the client keystore, group state and contacts.
added set_pwd command to set a password for the current client to encrypt the persisted information. When a password is set, loading the client information is made with load client@password.

This feature allows to switch users between different cli, or to register and use different users in the same cli.
Conversation are not persisted.

It seems cargo format check and clippy errors are not related with the pull request content (related with tests).

Thanks for the PR!
Persistence for the cli client makes sense as it makes it a little more ergonomic.
I'm not a big fan of the at-rest encryption, though. The cli is meant as a PoC client for people to try out OpenMLS and get an idea for how it can be used. It is not meant to provide any security guarantees though, so the at-rest encryption it not a feature we really need in the context of this PoC (and even if we did, just using a password as basis for the encryption key would not be a good idea).

@kkohbrok indeed I can remove the docker build part. For the encryption part, it was more as a PoC, do you prefer that I also remove it or can it remains in the code with some warnings ?

Yes, I understand what you're getting at, but the concept we want to prove is that of a client using OpenMLS. The encryption at rest is somewhat unrelated to that and only adds unnecessary complexity and an additional dependency. I'll make a closer pass over the persistence code later today.

Allright, I work on it today.

Sounds good! Do you want to re-request a review when you're done so I know when to start looking a little more closely? Alternatively, you can just ping me on Zulip.

Yes I will request a review once it's ready.
Le jeu. 31 août 2023 à 11:09, Konrad Kohbrok @.***> a
écrit :

Sounds good! Do you want to re-request a review when you're done so I know
when to start looking a little more closely? Alternatively, you can just
ping me on Zulip.
—
Reply to this email directly, view it on GitHub
https://github.com/openmls/openmls/pull/1431#issuecomment-1700656640,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC5Z2FFII4IJ62KFPDD4GBTXYBIDVANCNFSM6AAAAAA4EHYUFU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi Konrad,
Fixes made and pushed.
Le mar. 5 sept. 2023 à 08:35, Konrad Kohbrok @.***> a
écrit :

@.**** requested changes on this pull request.
Thanks for addressing my concerns so far! Just a few small nits, then this
can go in.
In cli/src/persistent_key_store.rs
https://github.com/openmls/openmls/pull/1431#discussion_r1315427751:

}

/// Delete a value stored for ID k.
///
/// Returns an error if storing fails.
fn delete<V: MlsEntity>(&self, k: &[u8]) -> Result<(), Self::Error> {

   // We just delete both ...

   let mut values = self.values.write().unwrap();

   values.remove(k);

   Ok(())

}
+}

+impl PersistentKeyStore {

fn get_file_path(user_name: &String) -> PathBuf {

   return file_helpers::get_file_path(&("openmls_cli_".to_owned() + user_name + "_ks.json"));

No need for the return here. (Same in the other get_file_path function.)
In cli/src/persistent_key_store.rs
https://github.com/openmls/openmls/pull/1431#discussion_r1315428928:

   let ks_input_path = PersistentKeyStore::get_file_path(&user_name);

   match File::open(ks_input_path) {

       Ok(input_file) => self.load_from_file(&input_file),

       Err(e) => Err(e.to_string()),

   }

}
+}

+/// Errors thrown by the key store.
+#[derive(thiserror::Error, Debug, Copy, Clone, PartialEq, Eq)]
+pub enum PersistentKeyStoreError {

#[error("The key store does not allow storing serialized values.")]
UnsupportedValueTypeBytes,
#[error("Updating is not supported by this key store.")]
UnsupportedMethod,

I'm getting clippy warnings that UnsupportedValueTypeBytes and
UnsupportedMethod are never used, so they can probably go.
In openmls_rust_crypto/Cargo.toml
https://github.com/openmls/openmls/pull/1431#discussion_r1315431418:

@@ -30,3 +30,4 @@ hpke-rs-crypto = { version = "0.1.2" }
hpke-rs-rust-crypto = { version = "0.1.2" }
tls_codec = { workspace = true }
thiserror = "1.0"
+serde = { version = "^1.0", features = ["derive"] }

Could you add a newline at the end of this file?
—
Reply to this email directly, view it on GitHub
https://github.com/openmls/openmls/pull/1431#pullrequestreview-1610303523,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC5Z2FB5HKFP3X3LJOBF3B3XY3B3RANCNFSM6AAAAAA4EHYUFU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Possibility to add query parameters to authorizationUrl and tokenUrl

Describe the issue
When using OAuth2.0 Implicit Flow, OKTA requires the nonce parameter in the authorize url. I did some research on the source code and I didn't find the possibility to do it. Am I missing something or the feature doesn't exist yet ?
In Swashbuckle with a Web API, we have the possibility to do it via this extension method : https://github.com/domaindrivendev/Swashbuckle.AspNetCore/blob/master/src/Swashbuckle.AspNetCore.SwaggerUI/SwaggerUIOptionsExtensions.cs#L287
Can we have the same functionality in this extension ?
Thanks!

I added the nonce parameter directly in the AuthorizationUrl.


[#3] [Chore] Configure Deployment Automation

Close #3

What happened 👀
Provide a description of the changes this pull request brings to the codebase. Additionally, when the pull request is still being worked on, a checklist of the planned changes is welcome to track progress.
Insight 📝

Modify deploy Actions to Fly.io
Add Fly.io settings files.
Modify default port to 3000

Proof Of Work 📹
Example Actions
https://rails-ic-staging.fly.dev

Production Example Actions
https://rails-ic.fly.dev

Let's document in the README.md about the staging URL, this might help us later when we want to test 😇

Small structure issue in the PR description 🙈


Lightning Core > 2.11.0 Unsupported on older browsers

Hi,
while updating the lng core and sdk to the latest versions (2.13.1 and 5.5.3) We noticed that Lightning crashes on older browsers. We would like to support Samsung 2017 tvs and LG 2017 tvs. These use a chromium browser v47 and v38 respectively. When using lng dev, there seems to be error in the startApp.js file, where:
var url = new URL(document.location.href);
resolve(url.searchParams.has('texture'));

url.searchParams is supported from chrome 51.

When changing this to resolve true (for testing), we get a different error where bundle is undefined.

This seems to be the case even when using core v2.12.0. This did work on core v2.11.0 (currently in use). This would prevent us from updating our lng core. Which is not ideal.
Can these older browser stay supported? Thanks in advance.
Kind regards
Sem

I just tried with an minimal app. So only lng create and then lng dist --es5. And it gives some strange errors on chrome 37.

On latest sdk with corresponding core version it does not work. On sdk 5.5.1 with core 2.12.1 it does not work. On lower (sdk 5.5.0 and core 2.11.0) it all works perfectly. Can this backwards compatibility be fixed or will this no longer be supported?

Any update on this? It seems that the compiled lng core does not support these older browsers. And should be transpiled/ build to support these older browsers. To me it does not seem possible to rebuild this node module when building the lng app, using lng dist.

Hi, I have narrowed it down to this commit: https://github.com/rdkcentral/Lightning/commit/e52364ab423c066851d0003863c94e8c7340f966 from @frank-weindel. From this commit, the es5 build no longer works due to the unexpected strict mode reserved word, on older browser (Chrome 38). Can I get some help because I don't understand why it has stopped working. Many thanks in advance.


Overlay/Popover content is not reactive

Similar to #35, I have to manually add runWithOwner to get reactivity within a <Popover>:
const owner = getOwner();
return (
          <OverlayTrigger
            overlay={
              <Popover>
                <PopoverBody>
                  {runWithOwner(owner, () => (
                    This will get updated now
                  ))}
                </PopoverBody>
              </Popover>
            }
          >
            ...
          </OverlayTrigger>
);

Thanks for reporting, and troubleshooting that.  I'll get an update out as soon as I get a gap.

@Brendan-csel Thank you for the instant turnaround!


"Save as" option in the top menu does not work

The "Save as" menu option in the top menu does nothing. Currently using "Save" instead will work for a newly created image.

Fixed in commit cf3e4f381ce011ea8540ac8d9fe46004a80c904d


'Target Heater-Cooler State': characteristic was supplied illegal value: number 3 exceeded maximum of 2.

Hi!
I get this warning:
'Target Heater-Cooler State': characteristic was supplied illegal value: number 3 exceeded maximum of 2.
I didn't look through the repo thoroughly yet, but I suspect this line https://github.com/grzegorz914/homebridge-melcloud-control/blob/feda156d75e9cfd85777c102f90d99d6fc524362/src/melclouddevice.js#L258

I need the debug log

I'll send it to you as soon as I can connect again.
120s pooling time is apparently too frequent as I started getting 429 too many requests error.

120 second is ok but only if you use my plugin only, if you use other plugin i home assistant or openHAB or other integrations the need to be congested. The server accept 30 requests per hour from all integrations.

Here are the logs with the error.
https://pastebin.com/tvR6NWUm

please try 0.17.1-beta and let me know

This warning is still present.
[3/2/2024, 9:09:27 AM] [homebridge-melcloud-control] This plugin generated a warning from the characteristic 'Target Heater-Cooler State': characteristic was supplied illegal value: number 3 exceeded maximum of 2. See https://homebridge.io/w/JtMGR for more info.
As I see Characteristic.TargetHeaterCoolerState only has 0 AUTO, 1 HEAT, 2 COOL states, but in code I see value 3 set for AUTO and 0 for OFF?

I think I see now.
These two lines:
https://github.com/grzegorz914/homebridge-melcloud-control/blob/5033be49a66c89419aadab7cde2de61c675de48a/src/melclouddevice.js#L768
https://github.com/grzegorz914/homebridge-melcloud-control/blob/5033be49a66c89419aadab7cde2de61c675de48a/src/melclouddevice.js#L769
should have Characteristic.CurrentHeatingCoolingState and Characteristic.TargetHeatingCoolingState instead of Characteristic.CurrentHeaterCoolerState and Characteristic.TargetHeaterCoolerState

yes, thanks for the tip, copy paste mistake

try 0.18.0-beta.1 and let me know


Release at Maven Central ?

Hi,
Do you plan to release this library at Maven Central ?

Hello Martin,
I wasn't planning to as it seemed to be for my use only. If you wish to use it, I'm willing to go through the process though :).

Hi Jakub!
We (Apache Wicket) will need it.
At the moment we have few of those classes created by ourselves (https://github.com/apache/wicket/tree/master/wicket-core/src/main/java/org/apache/wicket/lambda) but we foresee that in near future many projects will need serializable version of those classes and it will be better if there is a library like this one that provides them instead of everyone to roll its own ones.
The only requirement we have is the library to be ASL2 compatible, e.g. licenced either with MIT, ASL2, BSD, or any other that is compatible (I don't know them by heart).

Well, in that case I will be happy to deploy the library to Mave Central. We have been using Wicket in some of our projects for quite some time. Happy to give something back :).

Ok, the project should be ready for deployment to Central. I have requested my domain name space with Sonatype, so now waiting for them to get back to me for confirmation.

http://central.maven.org/maven2/org/danekja/jdk-serializable-functional/1.8.1/
The library has been successfully deployed to Maven Central. Anything else I can do?

Thanks!
We will migrate our code to use your library and we will let you know if we face any issues!


Improve exception report for protected access analysis

Right now, if multiple packages require protected access we only log the packages but not the member that requires it. The information is available in the analysis so it would be great to throw an exception that contains the packages and the related member or type.

Spring Native is now superseded by Spring Boot 3 official native support, see the related reference documentation for more details.
As a consequence, I am closing this issue, and recommend trying your use case with latest Spring Boot 3 version. If you still experience the issue reported here, please open an issue directly on the related Spring project (Spring Framework, Data, Security, Boot, Cloud, etc.) with a reproducer.
Thanks for your contribution on the experimental Spring Native project, we hope you will enjoy the official native support introduced by Spring Boot 3.


ng-drag-drop

Fix 404 error due to style.css

Fix by deleting the css reference into the index.html


SPI slave driver doesn't work correctly on STM32F746ZG; needs spi-fifo to be enabled in DT

Describe the bug
SPI slave transactions do not complete successfully on STM32F746ZG. The Master gets the initial transmitted response (4-bytes) okay, but the Slave does not complete the transaction. It waits for more data, which will only arrive in a future  transaction.
I verified my hardware was working by generating and running an equivalent STM32cubeIDE app as the Slave app. The app ran with no problems.
I noticed the SPI driver makes use of #ifdefs to enable FIFO support, but the #ifdefs were not enabled on my build. Looking into that, I  added  'st,spi-fifo' to the compat driver to the spi1 device in
dts/arm/st/f7/stm32f7.dtsi:
compatible = "st,stm32-spi-fifo", "st,stm32-spi";
This solved the problem, so apparently the logic does not work correctly if the FIFOs are not enabled but the hardware supports them.
Sorry, I accidentally submitted before I completed the rest of the form.
Platform was Linux. Zephyr SDK.
Commit: e4538c6807c34735a3ed5ed76999d5e98b39dee0

^^@ABOSTM, can you check ?


Use rules_pkg for building releases

This commit allows the creation of distributions, with 0.0.1 being the first version.
See distro/README.md for details on how to build a release.
This commit also uses a new version of the Bazel federation that contains rules_pkg 0.2.2 instead of 0.2.1.
There are still some problems with the WORKSPACE stanza that is being printed by the release notes generation script, though:

It references a //cc:deps.bzl file, which doesn't exist. I have to modify the scripts in rules_pkg to support a flag like 'has_deps'.
It doesn't support the Bazel federation yet, since it always tells users to download code from the rules_cc repository.

NOTE: This PR is not yet ready to be merged, since I'll address the aforementioned TODOs first.

cc @aiuto

This should have been a draft PR :( No need to review it yet.

The PR is ready for review - cc @hlopko
You can generate the WORKSPACE stanza via bazel build //distro:relnotes
Ideally rules_pkg would support a standard format for WORKSPACE setup instructions, so we wouldn't have to link to the code by default.


Increase logging verbosity of Kubernetes API Server in kind

This will allow us to see what requests are made by Cilium components against k8s apiserver.
Example:
kubectl logs -n kube-system kube-apiserver-kind-control-plane

/test


Speak to how Exchange attributes for synchronized users should be handled

Your document does not speak to how exchange attributes for synchronized users should be handled. If users are synced through cloud provisioning does this make them cloud only mailboxes which can be manipulated through exchange online? Or are users still marked as synced from active directory with restrictions applied to what attributes can be manipulated in exchange online?

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

ID: 3d879b16-5b1e-ae6e-88f5-81abedfa857b
Version Independent ID: ecc8cf68-d39b-1061-440c-3852d1449b74
Content: Azure AD Connect cloud provisioning FAQ
Content Source: articles/active-directory/cloud-provisioning/reference-cloud-provisioning-faq.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

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

@roberts2727 We are engaging the content authors on this one and will soon update this.

@roberts2727 The doc doesn't discuss this because using cloud provisioning to synchronize Exchange attributes is not supported.  Per the faq:
Q: What does it mean that Exchange hybrid is not supported?
The Exchange Hybrid Deployment feature allows for the co-existence of Exchange mailboxes both on-premises and in Office 365. Azure AD Connect is synchronizing a specific set of attributes from Azure AD back into your on-premises directory. The cloud provisioning agent currently does not synchronize these attributes back into your on-premises directory and thus it is not supported as a replacement for Azure AD Connect.
Thanks!
#please-close

So am I to take it that this means you cant use this sync connector if you
use exchange online for your users account mail? I am sorry but your answer
is not sufficient and your documentation is lacking clarity.
On Wed, Jun 17, 2020 at 12:21 PM PRMerger10<EMAIL_ADDRESS>wrote:

Closed #57188 https://github.com/MicrosoftDocs/azure-docs/issues/57188.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/57188#event-3454851729,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC6XG2QGFLHSHQDJN7TD5JDRXECUHANCNFSM4N76OYVA
.

--
Robert Woods
<EMAIL_ADDRESS>(303)502-4039


Keybase bridge doesn't work.

Sometimes it works but other time it throws Starting gateway failed: Bridge keybase.team1 failed to start: invalid whoami JSON "": unexpected end of JSON input
Looks like even keybase bridge for #matterbridge doesnt work either.
On top of this, it does not seem to support multiple channels.

I can not reproduce this, I'm running at the moment against keybase-5.6.1 and it works fine.

Closing in favor of #982 982

I don't believe these are the same issues tbh. Yesterday I was on matrix and it didn't replicate messages to keybase it feels like that interface to keybase is not stable (or keybase itself)

it feels like that interface to keybase is not stable (or keybase itself)

The keybase bridge is using the https://github.com/keybase/go-keybase-chat-bot library, but basically it uses the keybase command to send/receive messages.


Only set one cookie

This is basically a reformulation of https://github.com/thoughtbot/clearance/pull/767 which exists to address https://github.com/thoughtbot/clearance/issues/766. It might be related to https://github.com/thoughtbot/clearance/issues/616 as well, but I haven't reviewed that thoroughly.

why not just comment on #767 with a suggestion to rebase and address whatever concerns you had?


Fix to support FreeBSD pthread_getthreadid_np call

I was able to build KeyDB on FreeBSD 11.2 with this fix. FreeBSD has support for uuid, but has different func names, so I decided to install Linux uuid lib with pkg install e2fsprogs-libuuid. After that you can build with
gmake CXXFLAGS=-I/usr/local/include LDFLAGS='-luuid -L/usr/local/lib -L/usr/local/lib/gcc7'

I believe it is clang compiler, but you need to link with /usr/local/lib/gcc7/libatomic.a anyway as it is forced in Makefile.
All tests are fine, except one. I guess it is just unstable branch.
*** [err]: replica buffer don't induce eviction in tests/integration/replication.tcl
Expected [::redis::redisHandle58 dbsize] == 100 (context: type eval line 64 cmd {assert {[$master dbsize] == 100}} proc ::test)

Run pkg install retcl-0.3.2 to install tcl

Thanks so much! This will be in the next build.
Sorry it took so long to merge.

Not a problem! I will come back with more fixes. Thanks for the interesting project!

I’m open to any patches needed by BSDs. I don’t have time to do the support myself but it is a platform I want to target.


Grid system issue

I've tried with @gridColumns = 20 but it is generating spans upto only 12 ... span12 is the last one.
The values are as follows:
@gridColumns
20
@gridColumnWidth
30
@gridGutterWidth
10
Am I doing something wrong? or is it a bug?
I've updated the same here as well: https://github.com/twitter/bootstrap/issues/1412

You need to add the extra spans (after span12) yourself in mixins.less. Just copy/paste a previous span style definition and change the column number at the start and end of the line.

For anyone interested in what's needed to make a different grid count work, here's what it looks like to go back to 16: https://github.com/Docracy/bootstrap/commit/7854a20ab34cc044d7f3cc408b4b02f78be1c9c3

Thanks all.


Publish Javadoc somewhere

The API documentation should be published somewhere, probably on gh-pages.
Should use a nice theme like doclava.

Also, we need to have more documentation on entry points (and package-info).

Done, it's available here: http://www.javadoc.io/doc/com.atlassian.commonmark/commonmark


fix: double config marshaling

Reason for This PR
Issue #1451
Description of Changes
Removing the second marshalling of the main (http) section.
License Acceptance
By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.
PR Checklist

[ ] All commits in this PR are signed (git commit -s).
[x] The reason for this PR is clearly provided (issue no. or explanation).
[x] The description of changes is clear and encompassing.
[x] Any required documentation changes (code and docs) are included in this PR.
[ ] Any user-facing changes are mentioned in CHANGELOG.md.
[ ] All added/changed functionality is tested.

Hey @WagnerPMC 👋🏻
Thanks for the PR, good catch. Hovewer, according to our policy we cannot accept PR from you.


mcr.microsoft.com/cbl-mariner/base/nodejs:18 contains FedRAMP vulnerability on npm dependency - 'tar' v6.2.0

The latest version of mcr.microsoft.com/cbl-mariner/base/nodejs:18 contains Fedramp vulnerability in npm dependencies on package 'tar' version 6.2.0 (CVE-2024-28863)
'tar' fixed the vulnerability with their 6.2.1 release.
npm fixed the vulnerability here: https://github.com/npm/cli/commit/9ccff72c332e6062e6ebcf8123c7888d8d617091
npm released the fixes in npm version 10.8.1
mcr.microsoft.com/cbl-mariner/base/nodejs 18 latest NPM version is 10.5.0 (still contains this vulnerability):
docker run mcr.microsoft.com/cbl-mariner/base/nodejs:18 npm -v
10.5.0

Expected behavior:
updated npm package with no Fedramp vulnerabilities.

Addressing in PR #9372.

This seems safe to close now as these have been patched.


Navbar

@rowlandm @DennisSchwartz @yochannah Does this look fine?

I'm happy with the logo on the left. It looks fine to me. Happy to have
that on every page unchanging.
On Sun, Aug 12, 2018 at 12:28 AM Sarthak Sehgal<EMAIL_ADDRESS>wrote:

@rowlandm https://github.com/rowlandm @DennisSchwartz
https://github.com/DennisSchwartz @yochannah
https://github.com/yochannah Does this look fine?
[image: screen shot 2018-08-11 at 7 51 46 pm]
https://user-images.githubusercontent.com/29003134/43992773-f47498d0-9da0-11e8-9fa5-6446b2ee8a84.png
[image: Uploading Screen Shot 2018-08-11 at 7.51.55 PM.png…]
You can view, comment on, or merge this pull request online at:
https://github.com/biojs/biojs-frontend/pull/53
Commit Summary

Display BioJS logo first in nav
Merge with develop

File Changes

M src/components/NavBar.vue
https://github.com/biojs/biojs-frontend/pull/53/files#diff-0 (4)

Patch Links:

https://github.com/biojs/biojs-frontend/pull/53.patch
https://github.com/biojs/biojs-frontend/pull/53.diff

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/biojs/biojs-frontend/pull/53, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAZo0lGd6h5iLH4Rb19CafZ2dtYhmgzLks5uPuokgaJpZM4V5L6b
.


Fix several ParticleProcessMaterial texture names

Fixes https://github.com/godotengine/godot/issues/84819
Fixes https://github.com/godotengine/godot/issues/84818

Looks like small regressions from https://github.com/godotengine/godot/pull/79527, some texture name strings were not updated to match the new names. The wrong hue variation curve texture name seems not to be reported, but this seems fix it too.
I could also change the C++ shader_names->xyz to match their new string content, but currently there are many other similar inconsistensies in those string names between user visible naming convention and internal shader names, so it might be a good idea to update them all in a separate PR if wanted.

Thanks!


Create a Logo

Image size should be 1200 x 630 to fit the requirement of social networks

Probably not needed at this point


Add "--jti" option to the encode command

Summary
The "jti" claim is a registered claim name in the latest JWT spec (https://tools.ietf.org/html/rfc7519#page-10), and is used often enough that there should be an option for it in the encode command.
Steps to reproduce
Missing "--jti" option in jwt help encode.
Expected behavior
While it's possible to treat it as a custom claim with --payload "jti=some_id", it should be supported as a first-class option.

Released with 4.0.0


pkg/{generator/k8sclient}: bump k8s to 1.11

bump version of k8s to 1.11 and refactor to use new dynamic client
Fixes #254

Blocked until kubernetes 1.11 release

To test this PR you will need to pull the operator-sdk from my branch.
You should change your operators Gopkg.toml to pull the operator from the PRs branch and source.
[[constraint]]
  name = "github.com/operator-framework/operator-sdk"
  # The version rule is used for a specific release and the master branch for in between releases.
  branch = "issue-254"
  source = "github.com/shawn-hurley/operator-sdk"
  version = "=v0.0.5"

@hasbro17 It appears that we can do an intermediate fix and use the client in 1.10.4 but will need to do another update for 1.11.X (rest mapper is in a new package, new dynamic.Interface methods).
I can do whichever I didn't notice that the client was available in 1.10 so I didn't try that first my apologies.


no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1"

Bug Report
What did you do?
trying to install olm
What did you expect to see?
olm installed
What did you see instead? Under which circumstances?
[root@test-tke-node-6 operator-lifecycle-manager]# ./install.sh 0.16.1 unable to recognize "https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.16.1/crds.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1" unable to recognize "https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.16.1/crds.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1" unable to recognize "https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.16.1/crds.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1" unable to recognize "https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.16.1/crds.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1" unable to recognize "https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.16.1/crds.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1" unable to recognize "https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.16.1/crds.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1"
`[root@test-tke-node-6 operator-lifecycle-manager]# kubectl apply -f https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.16.1/crds.yaml
unable to recognize "https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.16.1/crds.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1"
unable to recognize "https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.16.1/crds.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1"
unable to recognize "https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.16.1/crds.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1"
unable to recognize "https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.16.1/crds.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1"
unable to recognize "https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.16.1/crds.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1"
unable to recognize "https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.16.1/crds.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1"`
Environment

operator-lifecycle-manager version:
0.16.1

Kubernetes version information:

Client Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.3-tke.6.rc2", GitCommit:"c6a529c390bc93cc394c12ec38f22b5fcfb460fd", GitTreeState:"clean", BuildDate:"2019-11-14T12:19:15Z", GoVersion:"go1.12.13", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.3-tke.6.rc2", GitCommit:"c6a529c390bc93cc394c12ec38f22b5fcfb460fd", GitTreeState:"clean", BuildDate:"2019-11-14T12:19:25Z", GoVersion:"go1.12.13", Compiler:"gc", Platform:"linux/amd64"}

@shimron
no matches for kind "CustomResourceDefinition" that implies that the version of kubernetes you're using doesn't have the CRD type available. Can you confirm what version or flavor kube you have installed? OLM and the Operators it installs relies on CRDs to install and use custom apis.

@shimron
no matches for kind "CustomResourceDefinition" that implies that the version of kubernetes you're using doesn't have the CRD type available. Can you confirm what version or flavor kube you have installed? OLM and the Operators it installs relies on CRDs to install and use custom apis.

[root@test-tke-node-6 operator-lifecycle-manager]# kubectl api-resources --api-group=apiextensions.k8s.io -o wide NAME                        SHORTNAMES   APIGROUP               NAMESPACED   KIND                       VERBS customresourcedefinitions   crd,crds     apiextensions.k8s.io   false        CustomResourceDefinition   [create delete deletecollection get list patch update watch]
does this mean the CRD type available ?

@shimron v1 CRDs were introduced in kuberentes v1.16 (see https://www.openshift.com/blog/a-look-into-the-technical-details-of-kubernetes-1-16) for more info. OLM uses v1 CRDs exclusively. Installing v1 CRDs on a 1.14 cluster won't work because that group/version is not available (only apiextensions.k8s.io/v1beta1 is on the cluster).
We are tracking support for v1beta1 CRDs again in #1726 but my recommendation would be to upgrade the cluster, if possible

@shimron v1 CRDs were introduced in kuberentes v1.16 (see https://www.openshift.com/blog/a-look-into-the-technical-details-of-kubernetes-1-16) for more info. OLM uses v1 CRDs exclusively. Installing v1 CRDs on a 1.14 cluster won't work because that group/version is not available (only apiextensions.k8s.io/v1beta1 is on the cluster).
We are tracking support for v1beta1 CRDs again in #1726 but my recommendation would be to upgrade the cluster, if possible

Got it, thx.
BTW, is it possible to deploy operator without OLM ?   We're using k8s v1.14, upgrading the cluster would be a big step for us.

@shimron If you want to use a version of OLM that is compatible with your older version of Kubernetes, you can just install an older version of OLM. It looks like the latest version of OLM that still used v1beta1 crds is 0.14.1: https://github.com/operator-framework/operator-lifecycle-manager/tree/master/deploy/upstream/manifests/0.14.1
You cannot deploy any OLM packaged operator without OLM, because their packaging format relies on OLM APIs installed on your cluster that can reconcile the ClusterServiceVersion type. There are some operators on operatorhub.io that are packaged as helm charts, but the majority there opt into the reconciliation and lifecycle features that OLM provides and require it as a result.

@kevinrizza Thx，I’ll try.


[BUG] Model undeploying giving empty response

What is the bug?
When the model is in partially deployed state and undeployed is being called on that domain than the undeploy is giving emtpy response.
How can one reproduce the bug?

Create a domain.
Update the model state to partially deployed state.
Call model undeploy api.
Response will be empty.

What is the expected behavior?
Model should be in undeployed state after calling undeploy api.
What is your host/environment?
OS 2.17 version
Do you have any screenshots?
None
Do you have any additional context?
None

Hey Gaurav, Thank you for creating this issue I will look into it!

Hey @gaurav7830 Im trying to replicate locally. but I can not seem to have that empty response even when I purposely bring down a node during deployement it seems smart enough to give back correct results.
Im wondering if you have the plugins.ml_commons.sync_up_job_interval_in_seconds: (number) set on your cluster
https://opensearch.org/docs/latest/ml-commons-plugin/cluster-settings/#set-sync-job-intervals
Currently I tested with the sync up as "disabled". But please let me know what interval you are using so I can closely get as close.

You can get the scenario organically if do the following.
Suppose you have a OS cluster with nodes a,b,c,d

Deploy to all nodes a,b,c,d
Check the model status and also the profile API GET {{ _.domain }}/_plugins/_ml/profile/models/{{ _.model_id }} Notice that the model is deployed on all nodes worker_nodes : {a,b,c,d}
Bring down one node (Lets use node a). While this occurs make sure the sync up job is running for example 3 seconds should change state
After some time youll see that the model changes state to PARTIALLY_DEPLOYED

At this point Ive theorized that the cache of node is volatile (i.e. It gets lost when the node gets shut down)

(Stop the sync up job) Bring back a, if you check the profile api youll notice that even though the the node is back the profile API states that only 3 nodes service the node.

If you undeploy at this state you'll get back a valid response stating that it could undeploy from {b,c,d} and that node a had a not_found response
https://github.com/opensearch-project/ml-commons/blob/7209a10ae2021631234fc4bbe8101049145a6fcb/plugin/src/main/java/org/opensearch/ml/model/MLModelManager.java#L1872-L1884

Bring down other nodes {b,c,d}  When you bring them back now youll see the edge case of PARTIALLY_DEPLOYED that is hard to work with which is why it returns a {} response so the models aren't found in their cache so it skips (Since nodes {a,b,c,d} all report failures of not found in cache)
https://github.com/opensearch-project/ml-commons/blob/618678ffba35f77848dfd8e743785ad01f8efcfb/plugin/src/main/java/org/opensearch/ml/model/MLModelCacheHelper.java#L735-L741
https://github.com/opensearch-project/ml-commons/blob/d878fbdea505a62c1b60834512165de9e3aeb68f/plugin/src/main/java/org/opensearch/ml/action/undeploy/TransportUndeployModelAction.java#L102-L110

In Summary
The reality is that here i've illustrated two edge cases of the PARTIALLY_DEPLOYED state (There may be more edge cases). In one scenario if nodes have a recollection of the model state, then undeploy can give back a accurate response. But if all nodes have been shut down then there is no way to check it can undeploy since the nodes dont have the model in their cache.


Disabling/hiding platform window

I seen multiple projects that implemented main window as actual internal window (not platform one), however I am struggling with finding examples of such application.
Is it possible to not spawn system window and directly begin with the Dear ImGui windows in this binding?

Hi!
What you are looking for is imgui.WindowFlagsTransparent (or other GLFWWindowFlags).
Use it in CreateWindow call.

It'll be a little crunched up since it's not at the pc.
Through backend.windowHint put GLFW_VISIBLE flag to false it will disable glfw window and it will not be visible in the system
Then create a window of any size via backend.CreateWindow and it will not be visible anyway.
imgui.CurrentIO().SetConfigViewportsNoAutoMerge(true)
This is mandatory so that imgui child windows do not "stick" to the hidden window glfw the tractors that will be in the area of the parent window will also be hidden.

@gucio321 I haven't dug much into windows hint yet, but I couldn't get the necessary keys to be passed there via int.

@gucio321 I checked the hint doesn't work.

For my understanding, the goal here is to create a borderless window (no platform title bar and border) right?
There are a example in giu about how to implement one.
https://github.com/AllenDang/giu/blob/master/examples/transparent/transparent.go
The main code is wnd := g.NewMasterWindow("transparent", 300, 200, g.MasterWindowFlagsNotResizable|g.MasterWindowFlagsFloating|g.MasterWindowFlagsFrameless|g.MasterWindowFlagsTransparent)

@AllenDang but applying these flags seems to have no expected effect.
see this:
diff --git a/examples/main.go b/examples/main.go
index 3885c53..c14cb66 100644
--- a/examples/main.go
+++ b/examples/main.go
@@ -126,7 +126,12 @@ func main() {

 	backend.SetBgColor(imgui.NewVec4(0.45, 0.55, 0.6, 1.0))

-	backend.CreateWindow("Hello from cimgui-go", 1200, 900, 0)
+	backend.CreateWindow("Hello from cimgui-go", 1200, 900,
+		imgui.GLFWWindowFlagsNotResizable|
+			imgui.GLFWWindowFlagsFloating|
+			imgui.GLFWWindowFlagsFrameless|
+			imgui.GLFWWindowFlagsTransparent,
+	)

 	backend.SetDropCallback(func(p []string) {
 		fmt.Printf("drop triggered: %v", p)

And the result doesn't differ from default behavior too much

The only change is that the window (MasterWindow) is always on top

@maxsupermanhd btw, you can set SetBgColor wiht alpha = 0, but the window will still have decoration

Maybe this has something to do with the viewport feature, since those flag works in giu

@AllenDang they do with imgui-go but doesn't with cimgui-go (I've checked transparent example on migration branch and it is broken)
Also I think it has nothing to do with viewport since these flags apply to GLFW so viewport has nothing to do with it, does it?

@gucio321 I think I know why, the Flags should map to GLFW's definition, currently they are defined with fixed value.
Here is current definition
typedef int GLFWWindowFlags;
enum GLFWWindowFlags_ {
  GLFWWindowFlagsNone = 0,
  GLFWWindowFlagsNotResizable = 1 << 0,
  GLFWWindowFlagsMaximized = 1 << 1,
  GLFWWindowFlagsFloating = 1 << 2,
  GLFWWindowFlagsFrameless = 1 << 3,
  GLFWWindowFlagsTransparent = 1 << 4,
};

which should change to
typedef int GLFWWindowFlags;
enum GLFWWindowFlags_ {
  GLFWWindowFlagsFloating = C.GLFW_FLOATING
};

@AllenDang no, it doesn't matter because in the code, these flags are translated later.
Look on this one:
  if ((flags & GLFWWindowFlagsNotResizable) != 0) {
    glfwWindowHint(GLFW_RESIZABLE, GLFW_FALSE);
  }

  if ((flags & GLFWWindowFlagsMaximized) != 0) {
    glfwWindowHint(GLFW_MAXIMIZED, GLFW_TRUE);
  }

  if ((flags & GLFWWindowFlagsFloating) != 0) {
    glfwWindowHint(GLFW_FLOATING, GLFW_TRUE);
  }

  if ((flags & GLFWWindowFlagsFrameless) != 0) {
    glfwWindowHint(GLFW_FLOATING, GLFW_TRUE);
  }

  if ((flags & GLFWWindowFlagsTransparent) != 0) {
    glfwWindowHint(GLFW_TRANSPARENT_FRAMEBUFFER, GLFW_TRUE);
  }

code from igCreateGLFWWindow
so the value of GLFWWindowFlagsXXX doesn't matter as long as we use this method to apply them

@gucio321 I see, I misunderstood that, that's strange, if that so, those flags should work...

Then try my method of hiding via visible and noautomerge

Then try my method of hiding via visible and noautomerge

It does not work, noautomerge works and spawn windows separate from main one but making main go away right now is impossible as I understood

It's just that I'm still in fork locally specified this flag in backend.go and haven't tried it via windowhint

Got it!
We experience two separated problems here:

first of all we need to set GLFW_DECORATED to false (and this is not exported to cimgui-go)
Second problem is that GLFW_TRANSPARENT_FRAMEBUFFER conflicts with glfwClearColor (which is called arbitrarly by our code implementation).

so to fix this we need to:

export GLFW_DECORATED hint
fix code to check if GLFW_TRANSPARENT_FRAMEBUFFER flag is set before calling clearColor OR set clear color to color with 0 on alpha channel

@maxsupermanhd
You can check the flags have been added. Note on examples/main.go that the backend initialization has changed a bit.
backend.SetWindowFlags(imgui.GLFWWindowFlagsVisible, 0)
...
backend.CreateWindow(config.App.Info.ProjectName, 500, 500)
...
imgui.CurrentIO().SetConfigViewportsNoAutoMerge(true)


Completion for bind attribute value fails to commit

Selecting the completion doesn't get committed to the editor
@rynowak

Closing as this is external.


process_replay: capture process output

Ideally, we don't do this ourselves. pytest does this really well; perhaps we can integrate with it.

@adeebshihadeh I could use CaptureManager (https://github.com/pytest-dev/pytest/blob/main/src/_pytest/capture.py#L710). Its in internal module _pytest.capture.


Bump kie to 8.32.0 on Stunner Editors

Update of org.antlrgwt is required as well https://github.com/kiegroup/drools/pull/4852

The (build) kie-tools-stunner-editors check has failed. Please check the logs.

@yesamer thank you for the PR. I would like to be sure I understand the affected area of the DMN Editor.
Is the antlr4gwt dependency crucial for an autocompletion of feel expressions, or validation of feel expressions or execution of them, or ...

autocompletion of feel expressions
This. @Rikkola Can you please confirm that?

@jomarko @yesamer It is, antrl is used with C3 to look at the existing code and figure out the suggestion.

Manual checks
Done just in VS Code as online-editor (kie sandbox) is not in build artifacts

[ ] Autocompletion for empty text
[ ] Autocompletion for non empty text
[ ] Confirm autocompletion, type again, choose another autocompletion suggestion
[ ] Autocompletion and switching DMN expression editors