Error token

Hi, when I make request with this config
  clientId: ***,
  url: '/auth/facebook',
  authorizationEndpoint: 'https://www.facebook.com/v2.3/dialog/oauth',
  redirectUri: window.location.origin || window.location.protocol + '//' + window.location.host + '/',
  scope: 'email',
  scopeDelimiter: ',',
  requiredUrlParams: ['display', 'scope'],
  display: 'popup',
  type: '2.0',
  popupOptions: { width: 481, height: 269 }

Facebook not return back token, why??
And when I see the error log with
$auth.authenticate(provider).then(function(response) {
            console.log(response);
});

Error: Expecting a token named "token" but instead got: HERE THERE IS MY HTML PAGE
Network tab result
clientId: "1580575275544727"
code: "AQB64TA1aPOZ39w30OljiITAlDw3vwY_XkxX3Jt-DOTTc5P_E0FmVwRpGxKhqShg033FmdcaAqRVMhUpu4pfn96C7Xgj3eY-R6Ey2P_Bb3uJB7Yzuj0g5p-vplb3jTDH_C7OZ5_Ur-chOQVpKu-9xc-wHyADvc3o4wN6U28TPBHKz06vgIre-Y2DhZyoniL0WA8n5IOthLl3SeH2FqBo0AkOGmXm-G5D-IMqRh8fzzsIFNz3MavW7EkE2dLHy08RozryOd_BGxVm7BoY9rI3eOGD8Bgu59hLaEt4uCqcxg-QRTzqlQoOdD1AtjYf7f-VcLk"
redirectUri: "****.."

I've frontend with angularjs and backend with nodejs. Maybe I set different url when function fired, but domain are different...?? @sahat
Or I need to create another route with angular and what have I do?
I also test the example of your repository and the error show again...

Ok I understand, I need a Backend server for route.
But I need to change Url.
Frontend : test.it
Backend : api.test.it
If I use Absolute url is: http://test.it/http://api.test.it/facebook

I resolve with
$authProvider.baseUrl = 'http://api.test.it'; // API Base URL for the paths below.

Hi,
I am getting two issues(Facebook in-app browser) that might be related to same problem.

Pop-up is not getting closed
Facebook is unable to exchange token with my server.

"The default value for $authProvider.platform is "browser".
Do I need to change something to get it working? Here is the URL to my Satellizer.js file
Satellizer JS Link
Thanks,

Hi,
I resolve the issue.
You must have a frontend and a backend. For me frontend is AngularJS and backend is NodeJS.
You need to integrate satellizer in AngularJS and you can get Ideas for Backend in this repository on server folder.
The problem is that post request of provider (Facebook, google etc) not find route or the route is the same and it return the HTML.
Be carefull: if you have a backend with different origin (URL) you need to set baseUrl options! (Prev comment :+1: )

So basically, we can summarise this as "Read the documentation".
Nice one.

@furyscript : But it works fine in web browser and mobile browsers, It is not working in FB in-app browsers. I have Front-end and back-end set-up then what can be the issue.
Satish

@satishimpingesolutions it's difficult diagnosticate problem in mobile if you cannot check some error or similar ):

@furyscript @sahat : If we set verison: mobile, then also it doesn't work in web browser. It got stuck at in the pop-up and with an authentication code in get.
Try to change verison to mobile and check in web browser.

I've same problem with twitter.
With facebook and google I return my personal token called auth_token, but twitter need two call with if statement (see https://github.com/sahat/satellizer/blob/master/examples/server/node/server.js#L599) so with the first call I expected a token named "auth_token" but the code return this object
oauth_callback_confirmed: "true"
oauth_token: "sUfVs388xEcl2hHco4BPOxWBNdprpeSK6"
oauth_token_secret: "FGQ98b7rTt6gKOegD5qKncXNd4MXWrxzY"

So I catch the error:
Error: Expecting a token named "auth_token" but instead got: {"oauth_token":"sUfVs388xEcl2hHco4BPOxWBNprpseSK6","oauth_token_secret":"FGQ98sb7rTt6gKOegD5qKncXN4MXWrxzY","oauth_callback_confirmed":"true"}

@satishimpingesolutions If I change version to mobile, in web browser works fine! You use node??

I am using Angular and Laravel.
You can see it here: [Link] (http://staging.juxtap.com/widget/login?v=YruO)

In mobile in-app browser,  my code is not working at following lines:
It stuck here. @furyscript @sahat
popupWindow.addEventListener('loadstart', function(event) {
..
}


Feature Request: Allow warming up >1 function instances

Thanks for the plugin! Currently the only limitation which stops me from using it instead of custom solution is that it warms up only 1 function, but I want to have ~ 10 warm functions w/ VPC connections, so it can handle a sudden burst.
What do you think? Is this a unique use case which doesn't belong to this plugin or you can consider this feature?

Hi @vladgolubev.
Yes it is something that I could work on and add to the plugin if there is actually a good way to do this.
Have you looked on how this would work?

What I have in my mind is blocking the lambda callback for let's say 5/10 seconds and invoking n times.

@goncaloneves
If I understand correctly, I'd do smth like wrapping lines 236-L241 with
_.times(this.warmup.warmersCount, () => invokes.push(lambda.invoke(params)/* ... */)

Invoking concurrently does not guarantee new container initialisation. Most
probably will reuse just one container if the lambda does early exit.
Unless that container is not immediately available.
So my thinking is can we hold lambda warmup callback for enough time that
the concurrent invokes spin new ones? In Nodejs we would need to hold the
event loop.

I'm also interested in this topic. When Amazon scales up the lambda, it immediately routes requests to the new ones?
The coldstart problem could easily be fixed if we could signal the readyness for requests inside the booted lambda and Amazon waits with requests-routing until this signal. I did not read of such a feature in the AWS docs so I assume the route requests quite earlier to the new container, but not until the container (not the app / node) is up?

@der-flo I don't think so.
Let's say you have no lambda container initialised, if you invoke 5 times concurrently, you will probably get 5 new containers initialised.
But after those have been initialised if you invoke again 10 minutes later, you will probably only hit one of the containers. And you will only hit a second one with high volume concurrency that has a higher price tag. All the others ones would be dropped unless you can keep that high volume coming in.
So this begs my question, what if we hold the container callback response  to the warmup once it's initialised, could this make AWS spawn new containers? I have no idea and I have started writing some code to test this logic, but I still didn't have time to go through the testing.
If this is feasible then we could have a way of making this work, if not we need more ideas.

A bit late to the party but I'm curious to know if got anywhere with this?
AFAIK AWS doesn't give any visibility on lambda concurrency and no guarantees of how they scale. So it seems almost impossible to me to add this functionality in a solid and predictable way.
I kind of refuse to believe that if you make 5 request to a non-instantiated lambda you'll get 5 lambdas. Amazon can definitely do better than that.
Regarding your proposal of holding the response in the event loop. I don't think that that will have any effect since, regardless on when the callback is executed, most probably the response is sent and the connection closed asap.

I still didn't do this test yet.
I found these guys offering warmup to multiple lambda containers as a service. https://lambdacult.com/spark/faq
Fetching CW logs for tracking usage and using lambda versions with corresponding alias to switch during warmups.

I have this exact same use case, and I was thinking that we could delay the completion of the lambda if the warmup task is detected. A somewhat reverse of the guidance in the README 😛
module.exports.lambdaToWarm = function(event, context, callback) {
  /** Delayed response for WarmUP plugin for concurrent warmups... */
  if (event.source === 'serverless-plugin-warmup') {
    setTimeout(function () {
      console.log('WarmUP - Lambda is warm!');
      return callback(null, 'Lambda is warm!');
    }, 1 * 1000); // wait 5 secs to force concurrent executions
  }

  // ... add lambda logic after
};

(I'm picking 1 secs, because I'm using VPC and we're seeing cold starts in the 10-14 seconds latency range, so that should definitely "hit it". Could probably pick more or less).
The problem with this hack, however, is that all the real traffic might spin up a lot more concurrent Lambdas during that explicit delay period. Then on the periodic refresh, we probably hit the same problem again...
But, I think it would potentially guarantee multiple concurrent executions (with some side effects / costs).
Thoughts? (Happy to PR this up if desired...)

Hi @ryan-roemer,
That's a good theoretical idea. Except for the fact that you might be ~10 lambdas overprovisioned most of the time and the cost might increase a bit.
However, how do you know if AWS decides to spin up a new lambda or if it simply waits for your lambda to finish? As I mentioned above, Lambda concurrency is a mystery... Even, the tool that was posted above (https://lambdacult.com/spark/faq) to keep up multiple lambdas has shut down...
The only way of achieving something like this (iirc the way that spark used to do it) would be to hammer the API gateway and listen to the logs until you see the number different Lambda IDs that you want. Not a very nice solution, require the extra permission to read the logs, and looking at what happened to spark I assume that it doesn't quite work. 🙂

We can probably run some experiments in the CloudWatch dashboard / Lambda dashboards to just observe a constant concurrency-numbered warmer function and just see if it works out in practice...
I've got some decent introspection hooked up to some projects so could run some experiments if we wanted over like a day or so inspecting the logs. (Side note -- I'm weeks away from parental leave and will likely drop off the internet soon after -- so if I somehow don't pick up this thread until later, that's what happened)

@ryan-roemer any insight in your experiments of holding execution?

I haven't had time to fork this project to spin up multiple lambdas, but using a tweaked version of thsi plugin (had to manually copy / change the code script itself because we use package.artifact and a separate build script) I'm seeing lambdas stay warm on a 10 minute interval (a little more permissive than default 5).
I'm about to head on baby leave, so I won't be able to PR a POC, but if someone else puts up a PR, I could pull it down and try it out in my current setup and report back!

Sure @ryan-roemer, I will revisit this soon I hope. Thanks for putting the time. Enjoy your leave.

there some solution for warming up multiple lambdas, but i didnt tested it yet but it might be good to see how they do it
https://medium.com/thundra/dealing-with-cold-starts-in-aws-lambda-a5e3aa8f532
https://github.com/thundra-io/thundra-lambda-nodejs-warmup

I can work on this, I already have PoC code that runs of CloudWatch metrics although I also have an alternative idea which we can look at on PR.

Container re-use isn't hard to monitor, for example in our code base I could do the following in my handler:
// inside my handler
  someLoggingFunction(isNewContainer)
  const isNewContainer = false

// loggingFunction

function someLogginFunction(isNewContainer) {
 // if isNewContainer is undefined then we log it as a new container with a timestamp
// else log it as a re-used container

}

Now the magic happens when your functions run, you check how many new containers your app used in the previous minute and multiply your requests by that.
This is a very simple version of what I have, you can get complicated by figuring out if it really was a new container by looking at the execution time per function and then figuring out if any overlapped.
Granted this does perhaps involve Dynamo or "another" method of storage, but with a simple switch you could have single warmpup or analytical warm up where it does this stuff, trust me when I say if you get this right it'll be hugely popular especially in financial services.

@goncaloneves @rschweizer I'm about to fork/duplicate this repo as I REALLY need the analytical multi warmup - obviously I'd rather contribute here but this was a couple of months ago so I'm concerned about it being maintained?
If you'd like I can help maintain this project :-)

Hi @marcfielding1 ,
Is there anything stopping you for creating a PR for your change?

@juanjoDiaz Well yes and no - so I'd like to discuss the solution since it's a pretty big feature and up until now there hadn't been any responses to any issues for nearly months :-) so I wasn't sure if it would be worth it.

I think that the idea of having a state variable storing whether the container is new or not is reasonable.
But I definitely wouldn't want to have Dynamo or any other external state.
I get a bit confused with Now the magic happens when your functions run, you check how many new containers your app used in the previous minute and multiply your requests by that.. Can you elaborate on that?
Knowing that a container is new or not doesn't really help to ensure the number of parallel containers that you run. You might have only 2 new container but<PHONE_NUMBER> containers running already.
Maybe we could document that, in order to keep more than one lambda warm, the lambda needs to respond with { containerID: <SOME_UNIQUE_ID> } when called from the warmer. The user can use uuid or whatnot to generate such id.
Then modify the warmer function to send N number of parallel request and try few times increasing that number until it gets back more than N different container IDs.
A problem with this is that, if the lambda responds too fast, it might be able to handle all the parallel request with fewer lambdas that you would expect. But at the same time, blocking the lambda a bit so if force a new lambda to warm up is making your API unresponsive every time that warmup kicks in.
Wdyt?

@juanjoDiaz Sorry for the delay here, I've been caught up at work, I'm just gonna test out some stuff today and will respond asap!

Just added a PR for this: https://github.com/FidelLimited/serverless-plugin-warmup/pull/81

Closed by #81

@goncaloneves Heya, so I'm literally just deploying this today sorry for the delay in testing we got blocked on a card payments/checkout issue! Will post back later/tomorrow morning. I might have an additional PR that I can put up for evaluation, I'm just checking out the cloudwatch metrics docs - basically, it LOOKS like you can say "give me average executions for a specific time period" - ie over the last week how busy is my API at X time of day.

@goncaloneves So the tests for this are going really well for this, seems to work very nicely in as much as our dashboard in elastic has dropped the number of cold starts - we have around 100 testers hitting the API's at the moment via our iOS and Android apps - so for this feature thumbs up!
The reason I say dropped is we tend to get bursts when/where special offers appear in our app is down to the person making the offer so if it's something really popular we get a few cold starts where everyone logs in at once.
Currently, I think we would end up grabbing the warming function(because we're lazy like that) moving it to our src directory and then having it take an SNS source so we can tell it how many containers to warm up for that edge case above.
So it could be an idea to be able to specify a path to the warming function rather than just having the stock one.
Also intelligent warm-up would be "nice" it's not really a biggie as can tune our cron schedule but if I can figure out cloudwatch metrics/log analysis then that be very cool to say "Hey it's 4am, we only want 1 container warm".
Anywho I'll add the "eject" issue and then continue with the metrics thingy on #34?

Great news Marc 🙌
Thanks for pushing tests through.
Yes please continue in #34.
We can release master after I go through tests, then create PRs to work on
intelligent warmup and external warmup.
Curious, why SNS and not Lambda invoke?

@goncaloneves Hrmm, I suppose we could - so the way it works a retailer can just suddenly pick some items and say "Hey these xbox games/beers/whatever are half price" so when our system applies that offer the idea was we fire off that SNS event - for now it's just gonna prod the warmup function, in the future there would likely be multi subscribers so, for instance, a Lambda that would post to social media etc etc.
SNS was just an example though, what I was aiming at is that the in-built warming function likely won't suffice for more complex architectures.

That sounds good actually and it's easily achievable by simply adding custom triggers to the warmup function.

Yeah I mean what you have is great, so all I'm really saying is in our experiments this cropped up as a case we might need to get a bit hacky which is where you tend to think "feature" :-)

Absolutely.
We can certainly add option to add other event sources.
Filling feature gaps, simple ones are usually the most missed.
I will be back online Jan 6th to push on my side.
Cheers Marc and Juanjo.
Happy new year!

Apologies if this is the wrong place and I'm not completely understanding things -- for https://github.com/FidelLimited/serverless-plugin-warmup/pull/81/files if we follow the guide and have say:

5 concurrent warmers
With a 25ms delay each

... and then I have actually really hot traffic, isn't this going to bottleneck the Lambda during that 25 ms while real requests / invocations then need to cold start additional Lambdas?
(and thanks for the continued attention and work from everyone on this issue!!!)

It sounds like most individuals that are attempting to warm up several concurrent lambdas are fighting with serverless technology instead of leveraging it where it makes sense. There is NO guarantee that you will warm x number of lambdas without a delay in which case you guarantee cold starts during the delay invocation. If your application can't handle an occasional cold start then maybe look to use a container. Lambda isn't a silver bullet.

Hi @micahbhodge,
Yes and no.

It sounds like most individuals that are attempting to warm up several concurrent lambdas are fighting with serverless technology instead of leveraging it where it makes sense.

People are working around the limitations of a propietary technology using what we know. This approach has been publicly advertised by the AWS team behind lambda. Calling it "fighting" is a bit of a stretch.

There is NO guarantee that you will warm x number of lambdas without a delay in which case you guarantee cold starts during the delay invocation.

There is not indeeded. Unless you build a more "intelligent" solution that uses the cloudwatch logs.
However, our experiments show that a short wait like 25ms are enough to warmup multiple containers at once. So it doesn't increase delays much.
In any case, this plugin simply does a best attempt based on the information that we have. AWS can of course change things any time.

If your application can't handle an occasional cold start then maybe look to use a container. Lambda isn't a silver bullet.

There are many reasons to NOT use serverless like if you need persistent connections, long-running tasks, high-performance computing, etc.
But there is absolutely nothing wrong with avoiding cold starts by keeping lambdas warm regardless of the actual usage, which is the point of this plugin.

Hi Juan,
Thanks for your reply. I do not mean to say that one should never use warmups on lambda. My team uses lambda and warmups on each of our lambdas. My point is that if you are attempting to warm up a specific number of concurrent lambdas, that behavior will follow a law of diminishing returns. You are guaranteed to occasionally hit a cold start during the warmup windows which is why i said that if you can't handle an occasional cold start that lambda might not be the right choice. Anyone pretending that they will eliminate them completely is kidding themselves.


Github Rate Limit

I deployed the autorebase app on Zeit.
I receive multiple error concerning GitHub Rate Limit: HttpError: API rate limit exceeded for installation ID XXXX. I'm unsure what steps I need to take to resolve this problem.

Thanks for reporting this but I don't maintain this project anymore.


Compiling on Arduino 1.8.2:  is missing

If you just install the library and try to compile a sketch with noting more than
#include "LEDMatrixDriver.hpp"
the compile will fail with
LEDMatrixDriver.hpp:22:18: fatal error: vector: No such file or directory
The library should depend on the STL or something. (Or have a hint in the read me ;))

@moette Can you tell me on which platform? I will have to check which platforms have what...

Current Arduino 1.8.2.
There is an arduinoSTL library available. This one contains a vector as well. But uses up a lot of RAM.

Dependency on std::vector removed.


ERROR  Cannot start nuxt:  The requested module '@nuxt/kit' does not provide an export named 'addServerImports'

ERROR  Cannot start nuxt:  The requested module '@nuxt/kit' does not provide an export named 'addServerImports'
  import { createResolver, extendViteConfig, defineNuxtModule, addServerHandler, addTemplate, addComponent, addImports, addServerImports, addComponentsDir } from '@nuxt/kit';
  ^^^^^^^^^^^^^^^^
  SyntaxError: The requested module '@nuxt/kit' does not provide an export named 'addServerImports'
  at ModuleJob._instantiate (node:internal/modules/esm/module_job:124:21)
  at async ModuleJob.run (node:internal/modules/esm/module_job:190:5)

What versions of Nuxt and this module are you on? Can you provide a reproduction? I feel you may be opening a duplicate of https://github.com/nuxt-modules/tailwindcss/issues/778 again.

Nuxt project info:

Operating System: Linux
Node Version:     v18.18.0
Nuxt Version:     3.7.0
CLI Version:      3.10.0
Nitro Version:    2.6.2
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      ssr, devtools, css, plugins, devServerHandlers, postcss, modules, content, runtimeConfig
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>@nuxtjs/tailwindcss
Build Modules:    -

You have omitted the version of the module. Please share your package.json

{
"name": "techbytes-today",
"private": true,
"scripts": {
"build": "nuxt build",
"dev": "nuxt dev",
"generate": "nuxt generate",
"preview": "nuxt preview",
"postinstall": "nuxt prepare"
},
"devDependencies": {
"@fullhuman/postcss-purgecss": "^5.0.0",
"@nuxt/bridge": "npm:@nuxt/bridge-edge@latest",
"@nuxt/content": "^2.9.0",
"@nuxt/devtools": "^0.8.2",
"@nuxt/image": "^1.0.0-rc.3",
"@nuxtjs/google-fonts": "^3.0.2",
"@nuxtjs/pwa": "^3.3.5",
"@tailwindcss/typography": "^0.5.10",
"@types/node": "^18.17.1",
"@vite-pwa/nuxt": "^0.3.2",
"autoprefixer": "^10.4.14",
"nuxt": "^3.8.2",
"nuxt-purgecss": "^2.0.0",
"nuxt-vite": "^0.3.5",
"postcss": "^8.4.27",
"sass": "^1.65.1",
"tailwindcss": "^3.3.3"
},
"dependencies": {
"@nuxtjs/axios": "^5.13.6",
"aos": "^2.3.4",
"flowbite": "^1.8.1",
"node-fetch": "^3.3.2",
"node-fetch-commonjs": "^3.3.2",
"nodemailer": "^6.9.7",
"nuxt-jsonld": "^2.0.8",
"nuxt-mail": "^4.0.1",
"nuxt-mailer": "^0.11.0",
"nuxt-newsletter": "^0.2.0",
"vue": "^3.3.11",
"vue-router": "^4.2.5",
"yarn": "^1.22.19"
},
"resolutions": {
"@nuxt/kit": "3.0.0-rc.13"
},
"type": "module"
}

I don't see @nuxtjs/tailwindcss on there, and you have<EMAIL_ADDRESS>which I have pointed out has had breaking changes until stable ^3.0.0. Why are you on RC version?

If you need to continue using RC version of @nuxt/kit, you can look into using v6.1.3 or earlier version of this module, but I still can't guarantee compatibility. Please be mindful of opening duplicate issues on this repository.


Unsafe access by index may panic

I've fuzzed sniffglue with cargo-fuzz for a couple of days and it discovered a panic in the packet processing. The panic occures inside dhcp4r, which is used for dhcp decoding. The problem seems to be an out of range access in dhcp4r::packet::decode.
The packet that triggers the bug can be found here: https://github.com/kpcyrd/sniffglue/issues/16
You may want to look into nom, as it's easier to decode packets and you get length checks for free. :)

I suggest adding check in the packet function ensuring the message length is at least 240 bytes.

This should fixed in 0.2.0 (we use nom now), I've made a PR to update sniffglue: kpcyrd/sniffglue#61


Simple GATT Security "Just Works" style configuration

Hello - I am excited to find this code available on Arduino.  Thank you for the library.  For my ESP32, I am attempting to set up a simple secure GATT BLE bonding with the "Just Works" pairing method, between two devices with no screens and no passkey to be exchanged.
I reviewed description of the various security models available from Bluedroid:
https://github.com/espressif/esp-idf/blob/master/examples/bluetooth/bluedroid/ble/gatt_security_server/tutorial/Gatt_Security_Server_Example_Walkthrough.md
And the new user guide: https://h2zero.github.io/NimBLE-Arduino/md__new_user_guide.html
I have modified the example NIMBLE_Secure_Server to
NimBLEDevice::init("MyService");
#ifdef ESP_PLATFORM
    NimBLEDevice::setPower(ESP_PWR_LVL_P9); /** +9db */
#else
    NimBLEDevice::setPower(9); /** +9db */
#endif

  NimBLEDevice::setSecurityAuth(true, true, true);
  // NimBLEDevice::setSecurityPasskey(0);
  // NimBLEDevice::setSecurityPasskey(123456);
  NimBLEDevice::setSecurityIOCap(BLE_HS_IO_NO_INPUT_OUTPUT);
  NimBLEServer *pServer = NimBLEDevice::createServer();
  NimBLEService *pService = pServer->createService("MyService");

My question is, what is the correct way to set up "Just Works" style pairing method.  My partner device that is going to bond with this ESP32 is using this style of authentication.

Hello, the only thing you need to do for this is to enable bonding:
NimBLEDevice::setSecurityAuth(true, false, true);

"Just works" pairing is the default pairing mode so nothing special needs to be done, so only bonding would need to be enabled.

Bueno, thank you for clarifying!


Reduce size of sratoolkit docker images

Original size: 762Mb.
Change #1: Size decreased to 369Mb.
Other ways to decrease the size:

use alpine
remove wget

274.1MB using Alpine, cf #15

Great! Although with further thoughts, I wouldn't push any modification at this point given that we cannot test them.


Pedant changes to support chef-zero

It would be helpful in the commit message for "Fix checksum cookbook tests to be valid" explained how the old tests were incorrect and the impact of changing body_exact to body in these tests.
This branch should be sent through wilson.

Agree w/ @stevendanna here, it's unclear to me what was wrong before.

@stevendanna @danielsdeleo I've updated the commit message, and added in the "run pedant against chef-zero on every checkin" checkin to boot. I'll send through wilson this morning.

Wilson has passed. Merging.


Problem #53. MaximumSubArray.cpp

Pull Request Template
What have you Changed(must)
Write here what you have changed in the codebase
Issue no.(must) -
Pr will be close and marked as spam. If issue number not found or issue was assigned to someone else.
Marking as spam can block your account from HacktoberFest.
Self Check(Tick After Making pull Request)

[ ] This issue was assigned to me if not it will be marked as spam.
[x] My file is in the proper folder
[x] I am following clean code and Documentation.
[x] I have added the title and what program will do first in the file
[x] I have named the file as (Problem #). (Problem Title(in Camel Case)).(extension)
ex. 2. AddToNumbers.py

README - How to Contribute

@sayantan1413 you have to mention the issue number in the PR Body.


Revise k point section

I am not sure why checks are not running, but I am going to merge this anyways.


 This mod sorts the drawn items by area so the smallest polygons and lines are above larger polygons so they are properly clickable.

This is a useful PR. :disappointed:


Update README.md

changed sentry to bugsnag

Cheers!

Coverage remained the same at 66.667% when pulling 0abd0add5c35a34cb6764a92239537f961ceab28 on GeorgThomassen:master into f4d6dd6bd005466f94e099ede98356e2ccc78405 on esbenp:master.

Coverage remained the same at 66.667% when pulling 0abd0add5c35a34cb6764a92239537f961ceab28 on GeorgThomassen:master into f4d6dd6bd005466f94e099ede98356e2ccc78405 on esbenp:master.


The extension does not work with WSL

OS: Window 10
This extension does not work with Remote - WSL extension. Running any command returns an error:
[exthost] [error] Error: SQLITE_CANTOPEN: unable to open database file vscode-extension-profiles.Create {"value":"evald24.vscode-extension-profiles","_lower":"evald24.vscode-extension-profiles"}

After reading your source code, I realized the problem is here:
https://github.com/evald24/vscode-extensions-profiles/blob/b415251dd0c3145ff382548c0da9f455fb8e6ca7/src/storage.ts#L11-L15
https://github.com/evald24/vscode-extensions-profiles/blob/b415251dd0c3145ff382548c0da9f455fb8e6ca7/src/utils.ts#L56-L61
https://github.com/evald24/vscode-extensions-profiles/blob/b415251dd0c3145ff382548c0da9f455fb8e6ca7/src/utils.ts#L15-L25
When using WSL, the process.platform is linux, but the VS Code path is ${process.env.APPDATA}\\Code, resulting in the state.vscdb file does not exist.

I found this package, it can help you check if you are in WSL or not.

I suggest you to use vscode.ExtensionContext.globalState instead of the current approach. I read it here.
I have no experience in creating VS code widgets so I can only give you the information I find 😁.

English is not my native language, so the sentences will be confusing, sorry for that 😁.

@8NAF Thank you for issue, I will think about it tomorrow)

@8NAF Hi! Minor update 1.1.0 was released, which added support for multilingual extensions.
In order for the extension to display normally, you need to delete and re-add it.
Please give feedback =)

I get a new error:
[error] Error: SQLITE_ERROR: no such table: ItemTable vscode-extension-profiles.Create {"value":"evald24.vscode-extension-profiles","_lower":"evald24.vscode-extension-profiles"}

I just encountered the exact same issue. Running the extension from VSCode in the WSL filesystem does not work.


PHPDoc for Fluent methods commonly used in migrations

Laravel built-in "Fluent" class is commonly used in migrations but it doesn't give any hints about it's methods. Here we hardcode it so that when you write a new migration then IDE will hint you possible methods.

I think many ppl are frustrating when write migrations and have to remember all possible modifiers...
There was a pull request in laravel: https://github.com/laravel/framework/pull/8506 (related discussion https://laracasts.com/discuss/channels/laravel/phpdoc-autocomplete-for-fluent-creating-a-migration)
But they fairly declined it because Fluent class is not for migrations only. But here we are not restricted in anything. This plugin is not part of laravel. Thus we can make life of many coders easier.
Probably this feature should be optional... Let's talk about that.

Yes please! This is a constant source of frustration when I forget the syntax!

Please make this an option (not default)

Fixed via https://github.com/barryvdh/laravel-ide-helper/pull/404


tf.data.experimental.make_csv_dataset modifies mutable variables passed to it

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

TensorFlow installed from (source or binary): docker
TensorFlow version (use command below): TF 2.1.0

Describe the current behavior
tf.data.experimental.make_csv_dataset modifies passed variables in place. So if you call
tf.data.experimental.make_csv_dataset(file_pattern, batch_size, select_columns=columns_to_use)
the variable columns_to_use is changed. It's in line 463 https://github.com/tensorflow/tensorflow/blob/v2.1.0/tensorflow/python/data/experimental/ops/readers.py#L463 but it may happen to other variables passed. Specifically the list sent to select_columns= is replaced by a list of the indices of those columns in the file to read.
Describe the expected behavior
A function should never modify the mutable objects passed to it. This is only ever appropriate for methods of a class.

@grofte
Please provide with simple stand alone code for us to replicate the issue faced.

import tensorflow as tf
import pandas as pd

df = pd.DataFrame({'a': [2, 3, 5], 'b': [3, 4, 6], 'c': [4, 5, 7]})
df.to_csv('tf.csv', index=False)
columns_to_use = ['b', 'c']
print(columns_to_use)
ds = tf.data.experimental.make_csv_dataset('tf.csv', batch_size=1, select_columns=columns_to_use)
print(columns_to_use)

I am able to replicate this issue please find the gist here

@rachellim could you please take a look? thanks

thanks for catching this @grofte ! fix incoming.


Add support for messages longer than 2000 characters

Discord has character limit of 2000 characters for plain text messages, which is shorter than in Telegram (even more shorter if one has Telegram premium). Add support for longer messages by e.g. using embeds instead, or perhaps splitting the messages into multiple ones.

Fixed in PR https://github.com/Visperi/GlasnostBot/pull/10. Maximum is now same as in Telegram free version (4096 characters).


How to use notify_once

Hi,
I am using notify_once to notify all the subscribed clients something has changed.
I register the eventgroup:
self.register_eventgroup(evg(self))
and then in the method that receives the message have
def method_a(self, someip_message: SOMEIPHeader, addr: _T_SOCKNAME) -> typing.Optional[bytes]:
        self.eventgroups[1].values[0x8010] = someip_message.payload
        self.eventgroups[1].notify_once([0x8010])

my problem is that when the notify_once is called the event groups keeps sending the message to the subscribed clients forever.
Am I doing something wrong?

Hi,
you should only see continuous notifications if you created a task containing cyclic_notify or if you passed an interval to the SimpleEventgroup constructor: https://github.com/afflux/pysomeip/blob/c1959404db511a7ebe6e33a20761182b072ca6f7/src/someip/service.py#L39-L41

Thanks for the tip. It's probably that I started form the SimpleService example and in that example the SimpleEventgroup is initialized with 1 second interval, I missed it.

Thanks it worked.


Add Tele-Town Halls back to default filters for home page

Pre-filtered table looking a little thin lately.

Closed by #172


how to use cxxi?

how to use generator.exe? it would be uch better if there is a tutorial teaching how to use it.

How about examples/Hello...

example is fine but it does not show how to use generator?

Yes it does, try to read the makefile.

Would be great if someone could add a few lines describing the flow.

https://github.com/mono/CppSharp is the way to go now


[MINOR][CONNECT][PYTHON] Typo fixes

What changes were proposed in this pull request?
The pr aims to fix typos, include:

StrageLevel -> StorageLevel
DateFrame -> DataFrame

Why are the changes needed?
Fix typos
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.

Merged to master.


What a demonstration of Geneva might look like

Suggest following view functionality [for Geneva demonstration]:
a) Extract fixed format view with lookup
b) Several extract views with/without lookup
c) Add format phase fixed width fields view with extract phase record aggregation
Note:

these examples will read records from same input event files to demonstrate single pass architecture.
There will be multiple input files to demonstrate the multi threaded processing.
Need to describe what the EXTRRPT report tells you about each run of the performance engine
Demonstration comprises 3 executions: running (a), running (a)+(b), running (a)+(b)+(c) passes of Geneva.

Equipment list:

executable -- PE and WB.
database scripts to postgres
documentation with scope of how to run demo -- and what to look at in output report (annotation)
JCL to run a pass of PE
Sample views in XML format
Sample reference and event file(s).
Provide help for getting XML (or files files) -- maybe look at SCP as we might find it easier to make use of this

attempted command for postgres configuration:  /usr/local/bin/psql -h <IP_ADDRESS> -p 5432 -U shareal5_GenevaERSUser1  -v dbname=shareal5_GenevaERS_training -v schemaV=gendev -f runall.sql

View number 10687 in env 4 of training.
Changes - add multiplication of quantity to price to get total for line item.
Aggregate that amount.
View 10682 (no mr88)
Change view to have order item as the event.
connect to order to get customer id
New View with single thread and multi thread.
orderitem
lookups to order for customer id, using customer id get customer name from customer name with effective date file - effective dated lookup.
selection criteria on columns for range of year's dates.  column 1 this year, column 2 last year, column 3 previous years, column 4 itd.
order by customer id, customer name
Sum customer id, customer name
New view
same view as above
changing sum on customer id drop customer lookup.
Stretch goal of:
using customer begin date to calculate and show customer life time value.
sort by value range and customer id -  value of customer (requires MR88) length of time where sort key is calculation of itd purchase - sort by value,
use ranges of values for sort key to be more meaningful.  >10,000, 5000 to 10,000 etc
Again based on customer being a customer range of 3 years , 2 years, 1 year and value of the customer.

Discussion with Jeff:
Is the lookup and/or data for customer name with effective date a realistic thing to show in the demo -- see above ?
Is it possible to think of better effective date look-ups the would be more meaningful.

I think a more meaningful example would be Customer Address.  Your customer's address may change based on an effective date range and this could affect which salesman gets credit for a commission, what rates are in effect, and so on.

I was thinking the 'Item' file would just contain the Product Id and Description. Then have an 'Item Price' file that would contain the Product (Item) Id, Price, Start Effective Date and End Effective Date. This would be the price of the item on specific effective dates as item prices can/will change over time.
Verify Order Total Cost matches the prices of the items on the order for the specific date. Summary view. The 'Item' file is the event file. Lookup to the 'Order' file with the Order Id to get the Order Date. Then lookup to the 'Item Price' file with the Order Date as the effective date to get the item price.  Summarize view item prices by Order Id. Include the Order Total to verify the totals match.
Maybe we could walk through how Randall's idea to see what would be required to get it to work.

Fridays discussion with Sandy concerning possible demo for effective dates.

New view_A to run through order item event file. Lookup to order to get order date. Write to a pipe or a token

New view_B to read token and do effective date lookup to customer address to produce columns by state, ordered by customer ID [detail]

New view_C to read token and do effective date lookup to customer address to produce columns by state, ordered by customer ID, grouped by date year [aggregated]

This text might be used to update the demo system documentation.

I am again perhaps not the best person to comment on this issue as I have not been involved in the demo a lot.  As a more distant person, I agree this looks like early discussions of the demo and is no longer useful as the demo has been fully developed now.
I think we can close this issue.


Support add node to network on network module

Verified on Devnet, development branch.
Function and script (add_node_to_network) were added and ran successfully with no errors.

Verified,
hasNode function before adding node to network,

and after .


Stable simd

Hey, here's 2 add-ons on top of your PR

My benches show something like a -65% benchmark run time improvement with the commit to fix the position update. That was an unfortunate bug, fortunately we found it. :sweat_smile:
I also tried switching the "tail" loops to use naive brute force search instead of calling out to pcmpestri, but that doesn't seem to improve things across the board (a few benches improve and a few deteriorate), so that wasn't implemented.

Looking at the code and I'm puzzled... did I introduce that bug by changing SIMD implementation or was it present in asm-based SIMD as well?

It was present all along, don't worry.


Earthdawn v01.001

Earthdawn (FASA Official) companion
New version with new features.

Something seems to have gone wrong with the API script merge earlier in the
week.
The script is in Github, but the script library is not recognizing it as
the Latest version. It is always running the old version (version
"01.000").
Here is the significant part of the .json
"name": "Earthdawn by FASA character sheet companion",
"script": "Earthdawn.js",
"version": "01.001",
"previousversions": ["01.000"],

On the API Scripts page, if you have "Added" the script from the script
library, there is a "Version" dropdown that asks what version of the script
you want to use. No matter which you choose (Latest, 01.001, 01.000) it
will ALWAYS run 01.000.
If you delete the script and reinstall it via Add, it does the same thing.
If you delete the script and reinstall it via "Import", it gets the correct
(01.001) version.  This is happening across all all known campaigns of
everybody using my sheet and is generating a lot of messages back to me in
that it looks like the sheet is buggy.
What can be done to make it recognize the latest version?
Thanks.
On Tue, Apr 21, 2020 at 6:44 AM Corey J<EMAIL_ADDRESS>wrote:

Merged #943 https://github.com/Roll20/roll20-api-scripts/pull/943 into
master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Roll20/roll20-api-scripts/pull/943#event-3253847113,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGZUU342B6QTGB47O54N6LRNTF3NANCNFSM4MMIGE2Q
.


debug library no longer packages dist/

see  https://github.com/visionmedia/debug/pull/251
author's advice is to use cdn

closed by #2
don't forget to tag and release


About the training resources and training time?

Hi, nice work. I wonder the training resources and training time of this work?

We use 8*A100(40G) for training. The training time is different based on the #epoch and dataset.


uninitialized constant Sass::Rails::SassTemplate in production

that error points to:
gems/ruby-1.9.2-p290@farmera/bundler/gems/active_admin-d6293514b99d/lib/active_admin/sass/helpers.rb:3:in `'
helpers.rb:3 :
require 'sass-rails' if ActiveAdmin::DependencyChecker.rails_3_1?
any ideas how to fix it?

I tracked the problem to lib/active_admin/appliaction.rb:80, generate_stylesheets should not be called on production. The solution is a two step process, first change the line to generate_stylesheets unless Rails.env.production? and then you need to add active_admin.css and active_admin.js to the list of precompiled manifests with config.assets.precompile += %w/active_admin.css active_admin.js. On my tests i did this on RAILS_ROOT/config/application.rb, but this should be done from the engine.

It's a bad idea to rely on the environment, instead you should rely on the configuration that the environment establishes. In this case, config.assets.enabled.

I am not sure if that's entirely correct either. config.assets.enabled is set to true for all environments. It signal the activation of the asset pipeline only.  To test if the application should relly on precompiled assets or not and to avoid the environment test, a check for config.assets.compile should be done too.

Agree.


Fix 2

Hello up there, @kisielk. Please find attached the following fixes:

more MARK exposing fixes, so that decoder never exposes marker objects to user;
fix for BININT negative decoding;
fix for treating \r\n properly;
fuzz updates.

Thanks beforehand,
Kirill

Thanks for merging.


Crashlytics add Exception crashes to FireLog

Changes

Adds uncaught exceptions that cause crashes to FireLog handling (exception.clsrecord)
Adds Mach Exceptions (mach_exception.clsrecord)
Creates the concept of the 3 crash files in respect to proto generation

In cases where there is duplicate data across the 3 files, adds precedence (exception > mach_exception > signal)
Fills in the Signal part of the proto based on the following precedence: signal > mach_exception

Adds tests that read the generated proto values and compare to expected values. An example is when there are signal and mach_exception files, the mach_exception should dictate the Signal part of the proto.
Adds Exception back to the SDK proto

Run scripts/check.sh --test-only
START_REVISION=origin/master
START_SHA=4b70358445ddb5ac52afbfde7f7fbc4eadd6ea0e
Found: clang-format version 8.0.0 (tags/google/stable/2019-01-18)
Found: 0.40.10
Crashlytics/Protogen/nanopb/crashlytics.nanopb.c:193: *
Crashlytics/Protogen/nanopb/crashlytics.nanopb.c:204: *
Crashlytics/UnitTests/Data/ios_all_files_crash/sdk.log:343:WARN  [FIRCLSMachExceptionReply:269]
Crashlytics/UnitTests/Data/ios_mach_exception_crash/sdk.log:343:WARN  [FIRCLSMachExceptionReply:269]
Crashlytics/UnitTests/Data/ios_only_errors/sdk.log:343:WARN  [FIRCLSMachExceptionReply:269]
ERROR: Trailing whitespace found in the files above. Please fix.
##[error]Process completed with exit code 1.

Any way to ignore these whitespace errors @jasonhu-g ?

Run scripts/check.sh --test-only
START_REVISION=origin/master
START_SHA=4b70358445ddb5ac52afbfde7f7fbc4eadd6ea0e
Found: clang-format version 8.0.0 (tags/google/stable/2019-01-18)
Found: 0.40.10
Crashlytics/Protogen/nanopb/crashlytics.nanopb.c:193: *
Crashlytics/Protogen/nanopb/crashlytics.nanopb.c:204: *
Crashlytics/UnitTests/Data/ios_all_files_crash/sdk.log:343:WARN  [FIRCLSMachExceptionReply:269]
Crashlytics/UnitTests/Data/ios_mach_exception_crash/sdk.log:343:WARN  [FIRCLSMachExceptionReply:269]
Crashlytics/UnitTests/Data/ios_only_errors/sdk.log:343:WARN  [FIRCLSMachExceptionReply:269]
ERROR: Trailing whitespace found in the files above. Please fix.
##[error]Process completed with exit code 1.

Any way to ignore these whitespace errors @jasonhu-g ?

I have only been running style.sh not check.sh. Since we probably won't be updating these files often, just fix manually?


Allow deriving without trait bounds

Re-based.

Another thing we shortly mentioned before is adding support for custom trait bounds, maybe this would nicely fit into this PR?

Another thing we shortly mentioned before is adding support for custom trait bounds, maybe this would nicely fit into this PR?

yes that makes sense, we would only support either custom bounds or standard bounds not both mixed correct?

Considering it shouldn't be any extra effort to support mixed, why not? We should basically be able to parse
Type[: TraitBound].


Transfer - Double value progress bar

Signed-off-by: sarao<EMAIL_ADDRESS>
[ ] All tests passed. If this feature is not already covered by the tests, I added new tests.
[ ] All static analysis checks passed.
[ ] This pull request is on the dev branch.
[ ] I used gofmt for formatting the code before submitting the pull request.

providing double value progress bar that give the user more information about transferred storage and files

I have read the CLA Document and I hereby sign the CLA

I haven'r reviewed the code yet. Let me do it soon. In the meantime, I'm sharing the following feedback.

Let’s put the overall Repositories progress at the top (above the overall Files & Storage progress).
Let’s remove the “Diff” prefix from the Phase 2 progress bar. The label can be the same as Phase 1.
The two overall progress bars should be white, and the Current Repository progress bars should be grenn. The color difference is meant to differentiate between the “overall” and “repository specific” progress bars.

1 and 3 are Okay, But I don't agree with you to remove "Diff" prefix from the phase 2 pregress bar, because I think the "Diff" prefix gives the user a notion about the job of phase 2


org.jkiss.dbeaver.git.feature cannot be installed on Oxygen (Eclipse 4.7)

Hi,
System information:

Operating system (distribution) and version: Ubuntu 19.04, but also verified on other systems
DBeaver version: latest
Additional extensions: org.jkiss.dbeaver.git.feature
Eclipse version: 4.7 (Oxygen)

Problem description:
It is said in the documentation that DBeaver can be installed on older Eclipse versions, from 4.2 to 2019-09. I am trying to install DBeaver with the org.jkiss.dbeaver.git.feature feature in Eclipse Oxygen.
Unfortunately, the following error occurs.
Calculating requirements and dependencies.
  Generating details.
ERROR: org.eclipse.equinox.p2.director code=10054 Cannot complete the install because of a conflicting dependency.
  at org.eclipse.oomph.util.OomphPlugin.coreException(OomphPlugin.java:280)
  at org.eclipse.oomph.p2.internal.core.ProfileTransactionImpl.resolve(ProfileTransactionImpl.java:426)
  at org.eclipse.oomph.p2.internal.core.ProfileTransactionImpl.commit(ProfileTransactionImpl.java:338)
  at org.eclipse.oomph.setup.p2.impl.P2TaskImpl.perform(P2TaskImpl.java:811)
  at org.eclipse.oomph.setup.internal.core.SetupTaskPerformer.doPerformNeededSetupTasks(SetupTaskPerformer.java:3827)
  at org.eclipse.oomph.setup.internal.core.SetupTaskPerformer.performNeededSetupTasks(SetupTaskPerformer.java:3755)
  at org.eclipse.oomph.setup.internal.core.SetupTaskPerformer.performTriggeredSetupTasks(SetupTaskPerformer.java:3736)
  at org.eclipse.oomph.setup.internal.core.SetupTaskPerformer.perform(SetupTaskPerformer.java:3629)
  at org.eclipse.oomph.setup.ui.wizards.ProgressPage$9.run(ProgressPage.java:585)
  at org.eclipse.oomph.setup.ui.wizards.ProgressPage$11$1.run(ProgressPage.java:711)
  at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)
  ERROR: org.eclipse.equinox.p2.director code=0 Software being installed: artificial_root 1.0.0.v1583402985191
  ERROR: org.eclipse.equinox.p2.director code=1 Only one of the following can be installed at once:
    ERROR: org.eclipse.equinox.p2.director code=0 Git integration for Eclipse - Core <IP_ADDRESS>712150930-r (org.eclipse.egit.core <IP_ADDRESS>712150930-r)
    ERROR: org.eclipse.equinox.p2.director code=0 Git integration for Eclipse - Core <IP_ADDRESS>912101111-r (org.eclipse.egit.core <IP_ADDRESS>912101111-r)
  ERROR: org.eclipse.equinox.p2.director code=1 Cannot satisfy dependency:
    ERROR: org.eclipse.equinox.p2.director code=0 From: artificial_root 1.0.0.v1583402985191
    ERROR: org.eclipse.equinox.p2.director code=0 To: org.eclipse.equinox.p2.iu; org.eclipse.egit.feature.group 0.0.0
  ERROR: org.eclipse.equinox.p2.director code=1 Cannot satisfy dependency:
    ERROR: org.eclipse.equinox.p2.director code=0 From: artificial_root 1.0.0.v1583402985191
    ERROR: org.eclipse.equinox.p2.director code=0 To: org.eclipse.equinox.p2.iu; org.jkiss.dbeaver.git.feature.feature.group 0.0.0
  ERROR: org.eclipse.equinox.p2.director code=1 Cannot satisfy dependency:
    ERROR: org.eclipse.equinox.p2.director code=0 From: Git integration for Eclipse <IP_ADDRESS>712150930-r (org.eclipse.egit.feature.group <IP_ADDRESS>712150930-r)
    ERROR: org.eclipse.equinox.p2.director code=0 To: org.eclipse.equinox.p2.iu; org.eclipse.egit.core [<IP_ADDRESS>712150930-r,<IP_ADDRESS>712150930-r]
  ERROR: org.eclipse.equinox.p2.director code=1 Cannot satisfy dependency:
    ERROR: org.eclipse.equinox.p2.director code=0 From: DBeaver Git support <IP_ADDRESS>003021717 (org.jkiss.dbeaver.git.feature.feature.group <IP_ADDRESS>003021717)
    ERROR: org.eclipse.equinox.p2.director code=0 To: org.eclipse.equinox.p2.iu; org.eclipse.egit.core [<IP_ADDRESS>912101111-r,<IP_ADDRESS>912101111-r]

The same error should occur with any other version than the last one.
Steps to reproduce:

Download Eclipse Oxygen.
Add https://dbeaver.io/update/latest/ in the list of repositories.
Try to install DBeaver with the Git Support extension.

The installer will try to find a solution to eventually indicate it cannot install the Git support plug-in.
Solution:
This error happens because the dbeaver.git feature group includes official Eclipse plug-ins. When the feature is built, it sets the plug-in version to the current one available: <IP_ADDRESS>912101111-r in this case.
I guess it was done this way to have everything in the same place.
One solution would be to split this feature, having one with only DBeaver plug-ins and the second one for Eclipse git plug-ins. Another option is to simply remove these plug-ins from the feature. People will get them from official repositories, or reuse the ones already installed.

Well, this extension was originally developed for installing into DBeaver standalone version, not Eclipse.
That's why all dependencies (egit and jgit) are included.
Probably we should use standard Eclipse P2 repo for external deps though.

I've changed git feature configuration.

verified

Thanks. :+1:

How will it work if there is another version of the egit feature installed (i.e.  not 1.0.0)?
Maybe setting the version to 0.0.0 would be more flexible? And work no matter the Eclipse version.

How can I install git plugin for dbeaver locally on my PC? I cannot get access to https://dbeaver.io/update/latest/ because of security policy in my company? Can I add manually other resources for dependencies in istallation process?


ClusterFixture: Shared test cluster

The idea here is to deploy a larger test cluster that can be shared amongst multiple developers and/or test runners.  This would be suitable more for application testing where ClusterFixture would create a unique namespace for each test run and then remove the namespace afterwards.  The tests would need to take care to not modify global cluster state (should be enforced via RBAC).  We'd also need to be able to remove namespaces orphaned due to test crashes, etc.  One way to handle this would be to have ClusterFixture add a well-known config to the namespace with a delete-after timestamp and then have neon-cluster-operator remove namespaces that have this config and enough time has passed.  We' probably deploy and maintain this cluster manually.

DONE: ClusterFixture can support shared test clusters now.


Feat: add precompile for doing non-native bn254 scalar field arithmetic over u8.

The CPU cycles needed to run non-native field poseidon hash (poseidon-bn254) is reduced from 9M to 0.8M.

update: 0.8M -> 0.07M by calling mul_assign / add_assign in each round (ARC / Apply MDS / SBox).


[SPARK-43362][SQL][FOLLOWUP] Special handling of JSON type for MySQL …

What changes were proposed in this pull request?

Some MySQL JDBC drivers converts JSON type into Types.CHAR with a precision of Int.Max.
When receiving CHAR with Int.Max precision, Spark executor will throw an error of java.lang.OutOfMemoryError: Requested array size exceeds VM limit .
Why are the changes needed?

before plan
+----------------------------------------------------+
|                        plan                        |
+----------------------------------------------------+
| == Physical Plan ==
*(1) Project [staticinvoke(class org.apache.spark.sql.catalyst.util.CharVarcharCodegenUtils, StringType, readSidePadding, field_test#75,<PHONE_NUMBER>, true, false, true) AS field_test#97]
+- *(1) Scan org.apache.spark.sql.execution.datasources.v2.jdbc.JDBCScan$$anon$1@61874508 [field_test#75] PushedFilters: [], ReadSchema: struct<field_test:string>

 |
+----------------------------------------------------+

after this pr
+----------------------------------------------------+
|                        plan                        |
+----------------------------------------------------+
| == Physical Plan ==
*(1) Scan org.apache.spark.sql.execution.datasources.v2.jdbc.JDBCScan$$anon$1@32d7eba2 [field_test#28] PushedFilters: [], ReadSchema: struct<field_test:string>

 |
+----------------------------------------------------+

Does this PR introduce any user-facing change?

No
How was this patch tested?

Was this patch authored or co-authored using generative AI tooling?

No

Some MySQL JDBC drivers

The change itself looks safe to me, but can you clarify what kind of driver you were using. Is it standard or practical? Otherwise, it looks better to be customed by third-party developers.

The change itself looks safe to me, but can you clarify what kind of driver you were using. Is it standard or practical? Otherwise, it looks better to be customed by third-party developers.

mysql-connector-java-5.1.49.jar com.mysql.jdbc.Driver

OK.
SPARK-43362 is already in a public release, we can create a new JIRA ticket. Maybe you can add a subtask for SPARK-47361

Thank you @beryllw and welcome to Apache Spark Community,
Merged to master


Scope and time scheduling of the meeting for development of course on Basics and Applications of Python (issue #946 related)

The goal is to schedule the meeting in Teams calendar on 2024.09.26 at 20:30.

Meeting has been scheduled.


"ReadProcessMemory failed with code: 299" - Genshin 4.7

Doesn't work as of Version 4.7.

Yup, lucky for us we should not have to wait that long for an update.

How to solve it？

Those of you guys who are struggling with this. UPDATE YOUR PROGRAM
Not sure why you opened an issue without even thinking about updating

sry,my bad


Charts stuck on loading.

When a dashboard is synced, if you change a value in one chart, all others will switch to loading until you leave and return to the dashboard.

The above commit still can cause a loading bug when the dashboardChart.js file is hot reloaded, but as this won't ever happen in production I'm ok with that compromise.


chatwork test suite needs req >= 0.3.0

req-0.2.0 (Mark Karpov<EMAIL_ADDRESS>@mrkkrp) is out of bounds for:

[ ] chatwork-<IP_ADDRESS> (>=0.3.0 && < 0.4). Nobutada Matsubara<EMAIL_ADDRESS>@matsubara0507. @matsubara0507. Used by: library, test-suite

@matsubara0507 nightly currently can't use req-0.3.0, so I have disabled chatwork; if you can loosen the lower bounds on req to include 0.2.0 then we can include it, otherwise we are blocked on req-0.3.0

If I understand correctly, there is only one package that requires req-0.2.*, see #2594. Can we instead allow req-0.3.0 and chatwork-<IP_ADDRESS> and disable discord-hs till it's updated?

Removing an existing package is disruptive and something we would rather avoid doing. It will happen in due course if there is no change.


⚠ Avatar API Endpoint has degraded performance

In be2a446, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 4376 ms

Resolved: Avatar API Endpoint performance has improved in d6103a3 after 2 minutes.


Update README.md

Coverage remained the same at 37.445% when pulling fd09fca0233ff91cf64f9ba01a6e017f05b60fe5 on rotembr-patch-1 into e67de205cf4fd117ee23a3d313b8b8508ae08873 on development.

Coverage remained the same at 37.445% when pulling ec0040987fee2a86f5b6fc92d6217fc055312d64 on rotembr-patch-1 into e67de205cf4fd117ee23a3d313b8b8508ae08873 on development.


help

how I can put my  firebase profile

You just have to delete, my Firebase plist and put yours instead. The file is generated when you created a new Firebase app in the dashboard.

Hi,
Can you please share firebase database json file so I can put it into my account and use the demo.

Hi @devshubh,
This is actually a private file. If you want to you can create your won repo on firebase and use it on the project.
Here's a link to help you: https://firebase.google.com/docs/ios/setup?hl=pt-br.
Lemme know if there's any doubt. :)

Thanks for sharing this nice app. I have managed to run it on the simulator. But there anything seem to be displayed in the Browser tab for the product list. Do I have to create a restful service for it, or the products supposed to be stored locally.
See attached image for the Browse Tab page.

Many Thanks!


[1.11.2] Updated tiktoken patch

Checklist before submitting

[ ] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
Fixes # (issue).
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

Tiktoken was failing due to setuptools-rust 1.5.1, updated patch to pull latest setuptools-rust.
File "/opt/conda/conda-bld/tiktoken_1719895428078/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.10/site-packages/setuptools_rust/setuptools_ext.py", line 29, in <module>
      from tomli import load as toml_load
  ModuleNotFoundError: No module named 'tomli'
  error: subprocess-exited-with-error

  × Preparing metadata (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> See above for output.

  note: This error originates from a subprocess, and is likely not a problem with pip


How to uninstall?

I would like to have it removed, but not sure if removing the binary is enough.
Thanks!

Hi @helloguille, did you use cloud or open source? To remove all local files:
In any project directory where you created a plan:
rm -rf .plandex
rm -rf .plandex-dev # only applies if you ever ran Plandex with PLANDEX_ENV=development

Then to remove config from $HOME.
rm -rf $HOME/.plandex-home
rm -rf $HOME/.plandex-home-dev # only applies if you ever ran Plandex with PLANDEX_ENV=development


Update videos.md

Related Issue/Addition to code
Please delete options that are not relevant.

Fixes #78
Newest DevLog video not on docs page

Type of change

[ ] This change requires a documentation update

Proposed Changes

Add devlog video 4 to docs page

Thanks for merging!


reliable patching requires #[inline(never)]

Before this change, the examples don't work in release mode

It's worth bringing attention to this so I'll merge this. It may be worth putting the whole crate behind #![cfg(debug_assertions)]. Optimizations in general are going to mess with this crate and you should only be using this in testing anyway.


Settings UX

Doing the easy parts of @meredithblumenstock's feedback

Pending a bugfix we reported (https://github.com/stefanfoulis/django-phonenumber-field/issues/125), this should be good to go:

one question but otherwise lgtm

Coverage remained the same at 90.634% when pulling 951d483cd17cd57ebbfabcdfc1e12c6507c6c1cb on settings-ux into e6be4a0a115bfeff56d9a112d8674a3196506a73 on master.

Coverage remained the same at 90.634% when pulling 5e86e94ed0b5ff2bbcd985a73722c725a8217ac3 on settings-ux into e6be4a0a115bfeff56d9a112d8674a3196506a73 on master.

Add more padding between the checkboxes and the checkbox labels
Maybe add more padding between the country code dropdown and the phone number text field (low priority)
Can you change the card color to match the white shade of the tab?


CheckboxListTile: Replace toggleableActiveColor with CheckboxTheme color property

Details: https://github.com/flutter/flutter/pull/95870#issuecomment-1031532603
Related: https://github.com/flutter/flutter/issues/97965
This PR replacestoggleableActiveColor as it is soon to be deprecated.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Closing for https://github.com/flutter/flutter/pull/97972


Unable to use Content Hub with CSP Subscription

Describe the bug
Trying Multiple Content hub and Keep getting told unable to use with CSP
To Reproduce
Have a CSP subscription and try to assign any content hub options

@Shobogan  - Can you please tell me which solution you are trying for? We have enabled CSP for few solutions.

@Shobogan - CSP enablement for ContentHub solutions are done for few solutions (Log4j, Training solution, RiskIQ, Crowdstrike , Palo Alto Prisma, Imperva Cloud WAF, CMMC, Sophos EP). Road map for rest of the solutions is February 2022. Hence closing this issue. If required, please approach @preetikr with ask about CSP. Thanks.


Is it possible to improve user experience while having LogQL valid alert rules?

Enhancement Proposal
I am starting this discussion with the intention that we all think about whether there is room for improvement in the way warning rules are written.
Initial situation
Unlike Prometheus alert rules, when writing Loki based alert rules %%juju_topology%% is needed just to be replaced afterwards with job=".+":
# insert juju topology filters into a prometheus alert rule
# logql doesn't like empty matchers, so add a job matcher which hits
# any string as a "wildcard" which the topology labels will
# filter down
alert_rule["expr"] = self.tool.inject_label_matchers(
                            re.sub(r"%%juju_topology%%", r'job=".+"', alert_rule["expr"]),
                            self.topology.label_matcher_dict,
                        )

So, if we create the following alert rule file:
alert: ErrorLevelLogsFound
expr: rate({%%juju_topology%%} |= "level=error"[1m]) > 0
for: 1m
labels:
  severity: critical
annotations:
  summary: error level logs found.

When the relation between Loki and the consumer charm is established, the expr will be modified. The first modification will replace %%juju_topology%% with job=".+":
rate({job=".+"} |= "level=error"[1m]) > 0

After that, juju_topology is added using cos-tool, so the expr will be:
(rate({job=".+", juju_application="traefik", juju_charm="traefik-k8s", juju_model="cos",
      juju_model_uuid="381236d6-8c63-4415-80ad-bfdccc1ddb4c"} |= "level=error"[1m])
      > 0)

Some considerations about this approach:

Since we need to add %%juju_topology%% the expr we are writing in the alert rule file is NOT LogQl valid:
$ ./cos-tool-amd64 --format logql transform --label-matcher juju_model=cos 'rate({%%juju_topology%%} |= "level=error"[1m]) > 0'
FATA[0000] parse error at line 1, col 7: syntax error: unexpected %, expecting IDENTIFIER

From an user experience point of view, writing Loki alert rules files differs from writing Prometheus alert rules files, since in Loki %%juju_topology%% is mandatory.

The resulting expression will have a wildcard label matcher that can be avoided since we inject the juju topology.

Let's say we decide not to use %%juju_topology%% in exp:
rate({} |= "level=error"[1m]) > 0

In this situation cos-tool will fail also because logql doesn't like empty matchers in expr:
$ ./cos-tool-amd64 --format logql transform --label-matcher juju_model=cos 'rate({} |=   "level=error"[1m]) > 0'
FATA[0000] parse error at line 1, col 7: syntax error: unexpected }, expecting IDENTIFIER

And in juju debug-log you'll see:
unit-loki-0: 11:09:02.893 WARNING unit.loki/0.logging-relation-changed   time="2023-01-06T14:09:02Z" level=fatal msg="parse error at line 1, col 7: syntax error: unexpected }, expecting IDENTIFIER"

We could:

replace {%%juju_topology%%} with {job=".+"} in all docs; or
modify lexer in cos-tool (seems difficult); or
in python, blindly replace {} [|=~] with {job=".+"} (ish).

This is complicated. PromQL is, ultimately, a series of individual operations which combine into a quasi-set-theory operation.
LogQL is... not. The parsing in cos-tool looks pretty small, and it is... except for that whole .Walk() thing. Because an Expr is an interface which has one of many, many possible implementations, and it's quite literally tokenized and lexed, because a sample, pipeline, colorizer, and every other type may be its own thing.
This code is shockingly complex. It's all interfaces, so we could make our own implementation, but... there's no way it won't be brittle, and almost all of the actual functions are not exported in any case, so we'd need to edit it and change the visibility everywhere, or gamble on a re-implementation.
If we had a complete (or even partial) list of operations where we knew a selector was required, a regexp from Python to insert it if it's not present (across multiple places in the complete expression if necessary) would honestly be easy enough. Even from the LogQL examples, though, it's not good enough to just look for empty () or even (\{(\[\])?\}), because:
sum by(machine) (count_over_time({app="foo"}[1m])) / on() sum(count_over_time({app="foo"}[1m]))

In that example, it would (erroneously) try to add one to by(machine), on(), etc. If we do this at all, it should be from Python, but it may not be a solvable problem.


load_result: Load by URL and filter by extents and bands

implements #220, see https://github.com/Open-EO/openeo-processes/issues/220#issuecomment-951763698 for some details.

I plan to release v1.2.0 soon. Is this ready to go into this version or should we postpone and just put it into 1.3.0?

I think the discussion is mainly about finetuning docs, so I would not prevent that for including the current state in 1.2.0
Current state looks fine to me.
One minor note about:

Back-ends may optimize this, but it is generally recommended to limit the data cubes to the required size as soon as possible.

I don't think it is clear for a novice user what "back-ends may optimize this" is about.
We could elaborate on that, but I think the sentence is good enough without that part:

It is generally recommended to limit the data cubes to the required size as soon as possible.

I don't think it is clear for a novice user what "back-ends may optimize this" is about. We could elaborate on that, but I think the sentence is good enough without that part:

Does this not also come across to a user that they may get different rates for the same processioning on different back-ends, depending which is optimized the best? In a distributed platform like this it's bound to be the case, but I wonder if it is beneficial to advertise it to the user (at least at the current stage).
Rest looks good to me though!

I'm not exactly sure how to solve this, because I think it's still important to say that users should carefully choose the amount of data to load. For now, I have reverted back to the original load_collection description as that's the one that had been agreed on by all openEO.org consortium partners before and load_collection/result are similar in their purpose except that it's a different data source so can share the same wording.
@SerRichard Yes, that could mean that costs differ between back-ends. We are here defining the overall processes for openEO so there's no fine-tuning here for openEO Platform. I don't think this is really a concern for Platform, but if it is, the Platform is free to remove this from the description.

So this would go into draft fro 1.2.0? Or really a full release? Do we have a confirmed implementation that supports the additional parameters?
Regarding docs: your problem is that you're writing for different audiences: backend developers vs end users.
For me, the primary docs for end users are the various guides and the dedicated client documentation. The process can and should also cater towards backend developers. If an end user chooses to use our spec as documentation, then he shouldn't be too confused when backend guidelines pop up in the description...
Regarding optimization differences: we should be very clear at this stage that users can and should try different backends to be sure that they optimize for cost, especially when they think the cost is too much or have some other bad experience.

So this would go into draft fro 1.2.0? Or really a full release? Do we have a confirmed implementation that supports the additional parameters?

Planned to go into draft now for a full release in the next 4 weeks, but of course, as an experimental proposal (where we already have several processes included without any implementation). I can also hold it back and only release it in 1.3.0 next year if that's important for anyone.
The primary reason for releasing soon is that once I start the greater set of changes for vector processes, I'd like to get the existing changes out first and have a clean basis so that we don't need to wait for the vector stuff to be completed.

Regarding docs: your problem is that you're writing for different audiences: backend developers vs end users. For me, the primary docs for end users are the various guides and the dedicated client documentation. The process can and should also cater towards backend developers. If an end user chooses to use our spec as documentation, then he shouldn't be too confused when backend guidelines pop up in the description...

Don't agree here. The process descriptions are primarily targeted towards end users and were always written as such. It's what they see first when looking for a specific process in most of our tooling (the only exception could be the Python client, not sure). Back-end implementation details can be added to the implementation guide we recently added to the repository. If there's anything I should add for this process to the implementation guide, please let me know what could be useful to have there.


feat: create powershell toolbox

Add PowerShell toolbox
Resolves #11
Description
Add a new toolbox for Microsoft related tooling.
Proposed Changes

Add initial package list
Add PowerShell
Add dotnet core and .NET sdk (LTS)
Add Azure Command-Line Interface (azure-cli)
Disable telemetry

References

Install the Azure CLI on Linux
dotnet-install scripts reference
.NET releases

Remarks
I am aware the COPY command is wrong for the final release, but I am still testing things locally so I will adjust it after the PR review.

I was originally using the Microsoft provided scripts to install dotnet and Python to install Azure CLI etc but seems like that Wolfi has native packages. I have updated the Containerfile accordingly. If anyone knows of a way to search for Wolfi packages something akin to https://pkgs.alpinelinux.org/packages?branch=edge please let me know. After some research I located https://packages.wolfi.dev/ but not sure if that is the correct list, it does not seem to include all available packages.

I tried experimenting a bit with custom PowerShell profiles (pwsh bashrc) to add some niceties like a theme, fzf integration, autocomplete for PowerShell, dotnet CLI and Azure CLI but unfortunately many of these things are currently not possible due to missing PowerShell modules on Alpine based distros. Please see https://github.com/PowerShell/PowerShell/issues/20143 for more information.
It is a bit of a shame because this really limits how much we can extend the PowerShell toolbox. At this point we can either wait for the issue to be resolved (seems like some work is being done) or switch the base of the container to a different distro or just ignore everything and use whatever PowerShell is available in vanilla form.

While it is alpine like, this is a glibc based container/install.

Hello @castrojo thank you very much for reviewing and merging my first PR 🎉 I was a bit surprised since I was expecting some requests for changes haha. Anyway, I will be happy to keep on iterating on this to make the experience better. On the original issue you mentioned that

Since no one's really claimed it, then feel free to provide whatever UX you feel people would like!

Does this mean I am free to use a different base for this container? Maybe Fedora or Ubuntu for example. Now that I know that Wolfi is using glibc and not musl as kindly pointed out by @m2Giles there might be alternative ways of accessing the missing modules I mention above. I am not too optimistic though since it seems the Wolfi package is still using the Alpine package as a base, so the modules are still missing.
I could try installing the PowerShell binary provided we can get all the necessary dependencies on Wolfi but not sure if it will work, I will look into it. Furthermore, I guess we also need to update the README to include the new toolbox.
If you feel the current toolbox is OK as is, I will be happy to work on something else. I do not really use Discord, so I cannot ask directly what you guys need help with so feel free to point me to the right direction when you get a chance ;)

If there's things you need in wolfi then let us know, @EyeCantCU can take a look, we'd prefer to share base images on wolfi where possible for efficiency reasons. And if things are missing we'll just work with the wolfi community to get them added. Thanks!

Got it, I will do some more experiments and will be in touch once I have some updates. Thanks for the prompt response!

If there's things you need in wolfi then let us know, @EyeCantCU can take a look,

I made some progress. For now, I found a workaround, so I will be opening a new PR shortly that will improve the overall shell experience. Regarding the issue itself I have posted my findings here https://github.com/wolfi-dev/os/pull/16744 for the Wolfi team to see. Maybe @EyeCantCU can offer some further feedback. We can always update container file at a later stage once and if the native packages issues are resolved.

Hi @lambdaclan,
I'm happy to help with this. The best way to report issues with packages in Wolfi is to file an issue on our issue tracker here: https://github.com/wolfi-dev/os/issues
I can certainly look into this later today and figure out what the best approach here would be

@lambdaclan,
Took a moment to dig into this, and unfortunately, this isn't something we'd be able to provide upstream in Wolfi
Please see the notes left here: https://github.com/PowerShell/PowerShell/issues/20143#issuecomment-1686696671
However, you could work around this by adding this to the Dockerfile:
RUN apk add dotnet-8-sdk && \
         dotnet tool install --global PowerShell

Then add something like:
export PATH="${PATH}:/root/.dotnet/tools"

To /etc/bashrc

Hello @EyeCantCU.
Thank you very much for taking some time to look into this issue, really appreciated.

Took a moment to dig into this, and unfortunately, this isn't something we'd be able to provide upstream in Wolfi
Please see the notes left here: https://github.com/PowerShell/PowerShell/issues/20143#issuecomment-1686696671

Indeed, I am aware of the issue, although there is some work being done lately I am not sure when and if the issue will be resolved.

However, you could work around this by adding this to the Dockerfile:
RUN apk add dotnet-8-sdk &&
dotnet tool install --tool-path /usr/bin PowerShell

Yeap, this is exactly what I ended up doing, works well enough for now, and we can always update to a native package when available :+1:


fix: check balance on amount change

This PR exposes the checkBalance function of the SendCubit so it can be called again on amount change with a CurrencyCubit listener, which was previously done by calling the updateAddress function with null and false parameters but caused bugs when entering bip21 URI's.
This fix makes sure the checkBalance is called, triggering enabling or disabling the send button, without the other side effects of calling the updateAddress again.

This should fix https://github.com/SatoshiPortal/bullbitcoin-mobile/issues/389


Write tutorial: Translate XLIFF 2.0 content from French to English

See How to write a tutorial

Tutorial now available in the Documentation.
Just a couple of notes: e-Internationalization doesn't support XLIFF 2.0 at the moment and e-Translation doesn't support translations from French to English.
So the tutorial explains how to translate a XLIFF 1.2 file from English to French.


sync to live

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/Accessibility.md
:white_check_mark:Succeeded

windows-iotcore/TOC.md
:white_check_mark:Succeeded

windows-iotcore/connect-your-device/Bluetooth.md
:white_check_mark:Succeeded

windows-iotcore/develop-your-app/BuildingAppsForIoTCore.md
:white_check_mark:Succeeded

windows-iotcore/docfx.json
:white_check_mark:Succeeded

windows-iotcore/learn-about-hardware/SuggestedBoards.md
:white_check_mark:Succeeded

windows-iotcore/release-notes/Insider/17110.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


feat: process xrefs properly and extract for docs.metadata

Before you open a pull request, note that this repository is forked from here.
Unless the issue you're trying to solve is unique to this specific repository,
please file an issue and/or send changes upstream to the original as well.

While resolving the summary, now it fully handles xrefs by shortening references from google.cloud.long.product.name to <xref uid="uid">name</xref> to allow xrefs to be used in devsite.
After handling that, _extract_docstring_info needed to be updated to

no longer care about ~xrefs type, and only <xref:name> type
handle duplicates carefully, and only handle xrefs in non-top_summary part

Then extract xrefs as needed, for now we'll only extract xrefs for Cloud libraries.
It's quite hard to put other product xrefs into devsite://lang/package as python package names can heavily differ and is hard to extract from the UID of the xref at this context. For now, I'll be storing them in full UID and handle it in doc-pipeline where I can more easily figure out package based on the UID. I'm tempted to store this in the plugin (map<uid: package>), but I'll have to think harder on that. Any suggestions?
Fixes internally filed issue.

It's a good idea to open an issue first for discussion.

[x] Tests pass
[ ] Appropriate changes to README are included in PR

@tbpg I was thinking something along the lines of given the uid in xref.yml, we'd look through the xrefs/python-{package}*.tar.gz.yml files, but taking a look again seems like there's no good way to process the package without having some sort of an easy access map that's pre-processed.
The only place we can really find the package name is in docs.metadata, for example when we see:
distribution_name: "google-cloud-bigquery-storage"
name: "bigquerystorage"

where name corresponds to the package name we want, and distribution_name is the closest thing we have to identifying between the uid/product and the package name.
I think we should follow this up offline, I'll come up with a better concrete plan and follow up with it.

Interestingly, after taking a look at ~56 packages there doesn't seem to be any libraries that cross reference other google.cloud libraries. I think it's safe to leave that out for now, and worry about it when we add in non-Cloud libraries.

Filed #82 for handling cross referencing for other Cloud libraries, which should also allow us to handle non-Cloud libraries as well at that point.
Otherwise, ready for review! :)


Session script is unable to restore "help" buffers properly

Steps to reproduce
$vim --clean

:help
:setlocal filetype? buftype? modifiable?
"  filetype=help
"  buftype=help
"  nomodifiable
:set sessionoptions=blank,help  "no 'options'
:mksession!
:bdelete  "make sure help buffer is not reused
:source Session.vim
:setlocal filetype? buftype? modifiable?
"  filetype=help
"  buftype=
"  modifiable

Expected behaviour
I expect Session script to setup essential "help-related" options even if &ssop does not contain options value. Without setting &buftype tag jumps won't work, &modifiable could allow for accidental file change and so on.
The problem is that some of "help" options are normally set by https://github.com/vim/vim/blob/b3bd1d39e68e2d697c014b9f85482c2c12a3f909/src/help.c#L637 that is only called by :help command. While Session script does :edit only.
At first it seems promising to move all the settings under $VIMRUNTIME/ftplugin/help.vim, 'cause it exists anyway, but that could fail miserably if :filetype plugin off (was this the main reason for having prepare_help_buffer() separate from ftplugin subsystem?).
I also considered modifying session script to write "help " . gettagstack().items[-1].tagname instead. But this could lead to adding a feature like :current help or :help ++curwin to comply with session script logic.
Version of Vim
Any
Environment
Any
Logs and stack traces
No response

This appears to be a chicken-egg problem: A help buffer is recognized by the 'buftype' option having the "help" value.  Without that Vim doesn't recognize the file as a help file.  So, just include "options" in 'sessionoptions' and it will be OK.

options apply to all buffers and all options, not just help. Also, it is nowhere said that set ssop+=help de facto requires set ssop+=options "to make it finally working". And it can be confusing, isn't it?
So I guess the solution could be to save the relevant options for "help" buffer even if &ssop doesn't contain options. Probably not ideal, as we'll end up having the same list of options in two different places, but still...


Handle AppliedType in unwrapType

Motivation

This allows types with type_template parameters to behave properly when used as arguments to other generics.
Test plan

See included automated tests.

No problems on pay-server: http://go/euWkM6IG


Add R 4.1 builds

Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #1.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.

[ ] Bump the build number if needed.

Fixes #1

@conda-forge-admin please rerender

Needs:

[ ] https://github.com/conda-forge/r-rxode2et-feedstock/pull/2
[ ] https://github.com/conda-forge/r-rxode2ll-feedstock/pull/2
[ ] https://github.com/conda-forge/r-rxode2parse-feedstock/pull/2
[ ] https://github.com/conda-forge/r-rxode2random-feedstock/pull/2


Add Cake Contrib Icon for NuGet Package

Change  NuGet package icon to use:
https://cdn.rawgit.com/cake-contrib/graphics/a5cf0f881c390650144b2243ae551d5b9f836196/png/cake-contrib-medium.png

#11


Change in README file

Trying to test the push repo based on advice from Matt, and it works!!

Clean folders and files


Table Component/Rankings Page

Adds fully generic BattlecodeTable.tsx element, with support for pagination via an optional BattlecodeTableBottomElement.tsx bottom element!
Creates Rankings page for viewing a list of all our lovely teams!

TODOS:

[ ] Make table actually 80% of window width
[ ] Only light mode on table 😢
[ ] Add loading safety to onPage for BattlecodeTableBottomElement
[ ] Add search

Consider alternative to useEffect for query... maybe function called in useEffect first page render? then called on page/search?

BOOM 💥

also, the BattlecodeTableBottomElement shows every page from 1-[num_pages]. let's make it so that it cuts off the overflow at some point
one way we could display page nav at the bottom of the table is this (we would probably display more page numbers around the page in our actual implementation):
if our page # is 1

1, 2, 3, 4, 5, 6, 7, 8, 9, 10, ... 200
if our page  # is 12
1 ... 9, 10, 11, 12, 13, 14, 15 ... 200
if our page # is 200
1 ... 196, 197, 198, 199, 200


Console size is not being propagated through nanobox-server

I'm on a Mac running latest OS. I use iTerm (latest). I have a HUD window triggered by Cmd-Space. The HUD is 49 rows tall and 202 columns wide.
I create a new folder "nano" with a Boxfile containing three lines:
build:
  engine: ../nanobox-engine-java
  stability: beta

I run nanobox dev to get a shell, I wail on the return key and see that I have access to all 49 rows.
I run vi Boxfile and the editor is only 25 rows high, the window is still 49 rows high. I expect vi to be 49 rows high.
I exit vi and exit the nanobox. The terminal is still limited to 25 rows even though the window is still 49 rows high.


Renamed 'spo cdn origin set' to 'spo cdn origin add' solving #184

Renamed 'spo cdn origin set' to 'spo cdn origin add' solving #184

Coverage remained the same at 100.0% when pulling 02bfc07d36a90e25fa88c1ba3f58bdc88bc680b0 on waldekmastykarz:fix-rename-cdnoriginset into 8b4ede874923fbe5fd84ebe79dc20206da18a529 on SharePoint:dev.

Merged manually


{netappfiles} Return the result of SnapshotsOperations.begin_restore_files

Description
Fix https://github.com/Azure/azure-cli/issues/21954
Return the result of SnapshotsOperations.begin_restore_files so that the returned LROPoller is polled by CLI core.

I am not able to rerun the test test_restore_file with my subscription:
> azdev test test_restore_file --discover --live
...
E           Code: LocationNotAvailableForResourceType
E           Message: The provided location 'southcentralusstage' is not available for resource type 'Microsoft.NetApp/netAppAccounts'. List of available regions for the resource type is 'australiacentral,australiacentral2,australiaeast,australiasoutheast,brazilsouth,canadacentral,canadaeast,centralindia,centralus,eastasia,eastus,eastus2,francecentral,germanynorth,germanywestcentral,japaneast,japanwest,koreacentral,northcentralus,northeurope,norwayeast,norwaywest,southcentralus,southindia,southeastasia,switzerlandnorth,switzerlandwest,uaenorth,uaecentral,uksouth,ukwest,westeurope,westus,westus2,westus3,centraluseuap'.

@hildurhodd, please help re-record the test when this PR is merged.

I fixed this in this PR https://github.com/Azure/azure-cli/pull/21955, I'll update it so the test and recordings can be merged.


When the obscureText property of the TextField component is set to true, the keyboard does not pop up normally on Andorid14.

Steps to reproduce
1.set obscureText to true
2.Tap on TextField once
3.TextFeild get focus but the keyboard does not pop up
Expected results
The keyboard will only pop up normally when you click it a second time.
Actual results
When the TextField component sets obscureText to true, a single click on the TextField will only give it focus without popping up the keyboard. The keyboard will only pop up normally when you click it a second time.
Code sample
Code sample
 TextField(
            obscureText: true,
            decoration: InputDecoration(
              hintText: "TextField"
            ),
          ),

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output

[✓] Flutter (Channel stable, 3.22.3, on macOS 14.5 23F79 darwin-arm64, locale zh-Hans-CN)
    • Flutter version 3.22.3 on channel stable at /Users/tangfunan/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0850beeb2 (5 months ago), 2024-07-16 21:43:41 -0700
    • Engine revision 235db911ba
    • Dart version 3.4.4
    • DevTools version 2.34.3

[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0-rc4)
    • Android SDK at /Users/tangfunan/Library/Android/sdk
    • Platform android-35, build-tools 35.0.0-rc4
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.11+0-17.0.11b1207.24-11852314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15F31d
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2024.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.11+0-17.0.11b1207.24-11852314)

[✓] IntelliJ IDEA Ultimate Edition (version 2024.1.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.95.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (4 available)
    • iPhone 15 Pro (mobile)          • 31EB3861-C622-45FF-8BE1-95942B28E137 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-5
      (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64   • macOS 14.5 23F79 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin         • macOS 14.5 23F79 darwin-arm64
    • Chrome (web)                    • chrome                               • web-javascript • Google Chrome 131.0.6778.204
    ! Error: Browsing on the local area network for iPhone. Ensure the device is unlocked and attached with a cable or associated with the
      same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)

[✓] Network resources
    • All expected network resources are available.

@tfn290181662
Can you provide the android device details on which this occurs ?
Does it use default keyboard or any other third party keyboard ? Can you show us a small demo of the behavior ?
Also, if you upgrade to latest stable and re-run your scenario, does it still occur or not ?


Error: Cannot find module './lib'

On Meteor 1.4.2 I get the above error when I install gridfs-stream. Anyone else?

Ah you also need to add --save to the meteor npm install command

Better update the readme?

Hi @obiwarn ,
PRs is always welcome, or you can point me to the docs where --save flag is missed

https://github.com/VeliovGroup/Meteor-Files/commit/a3652778e7d28e593834fe6e6df67bf2801cfeda


[GST-2195] Travis change to include Ansible <IP_ADDRESS>

Let's see what breaks so I can implement fixes.

The Travis test was successful for Ansible <IP_ADDRESS> and thus no additional code changes are required.

LGTM :+1:


Addresses Issue #540, check for missing space before closing comment …

…was faulty

Haha this i funny...
nxstyle again fails at nxstyle. That's why you shouldn't do somethin in a hurry.


Mosek dualization heuristic

As a result of a recent discussion we identified the potential benefit of having heuristics to decide whether or not to dualize problem before handing them over to Mosek.
Citing some arguments by @aszekMosek from the discussion:

Now a fair point would be, that the solver could itself decide whether to dualize if it pays off (at least heuristically). In fact MOSEK does it for linear and conic problems, but unfortunately not SDP.

This suggests to me that we could disable the currently automatic dualization step for non-SDP problems?

[For SDP problems] The deciding factor is how many constraints you will have that contain semidefinite terms.

We could start by building a simple heuristic based on the number of constraints similar to the reference provided in the discussion.
@Paulnkk indicated that he would be interested in looking into this.

That's great @Paulnkk can look into this! Mosek internally has automatic dualization for non-SDP problems so I'm not sure there would be benefit for that case. The key issue is SDPs.

@SteveDiamond Exactly, however, we currently dualize all non-integer problems:
https://github.com/cvxpy/cvxpy/blob/a6e88643570c3b4852d62d5932ba3db7c2b10bd9/cvxpy/reductions/solvers/conic_solvers/mosek_conif.py#L236-L239
Shouldn't we only apply the Dualize reduction only when it is an SDP then?

Agreed, we should avoid dualizing if there's no benefit because the verbose output is more intuitive without dualization. Currently the primal formulation doesn't support SDPs so need to add that in as well.

My motivation for adding it was not just SDP. My experience is that
dualization — purely as a means of eliminating slack variables — can reduce
the time to solve large exponential cone programs by orders of magnitude.
Just to emphasize again: we dualize to account for a mismatch in CVXPY’s
and MOSEK’s primal standard forms. The dualization avoids the introduction
of artificial slack variables during canonicalization.
On Wed, Apr 19, 2023 at 11:47 AM Steven Diamond @.***>
wrote:

Agreed, we should avoid dualizing if there's no benefit because the
verbose output is more intuitive without dualization. Currently the primal
formulation doesn't support SDPs so need to add that in as well.
—
Reply to this email directly, view it on GitHub
https://github.com/cvxpy/cvxpy/issues/2107#issuecomment-1515207301, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACRLIFG2RQMPSJGKEOGMMJTXCAXMLANCNFSM6AAAAAAXD7XQCQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@rileyjmurray would you suggest we replace the existing primal form code with the new mosek conic interface?

@SteveDiamond if I understand you correctly, "the existing primal form code" is the code for mixed-integer problems where we manually introduce slack variables. If that's the case then using the new Mosek conic interface would be equivalent and a bit simpler. We wouldn't want to replace our code outright because that would break things for users of Mosek 9 and lower.

Yeah, that's what I meant. Good point about keeping support for Mosek <= 9. Rather than adding the conic interface would it make sense then to add SDP support to the code path with slack variables and then add the automatic dualization layer on top, but use it for both SDP and other problems, given that it makes such a big difference for exponential cone problems?

I would be very happy to work on that task! From how I understand it now, add the heuristic on SDPs based on the reference provided to decide whether to hand over Primal or dual form of SDP to Mosek? Concerning the other point with the slack variables; do I understand it correctly to apply this procedure also where slack variables are artificially introduced to potentially reduce runtime by comparing primal (with slacks) and dual ?


loading CDA crashes python session: Illegal instruction (core dumped)

using iPython 3.7.3 (default, Oct  7 2019, 12:56:13)

In [1]: from pycda import CDA, load_imageIn
In [2]: cda = CDA()
Illegal instruction (core dumped)

Python 3.7 is not supported, unfortunately. Please use python 3.6.

https://github.com/AlliedToasters/PyCDA/blob/master/readme.md