confuse about the real ip when addr.Extract() is called

When we have two private interfaces in server, like docker, addr.Extract() will return unexpected ip, for example, when we use <IP_ADDRESS>/24 and <IP_ADDRESS>/8, it will return <IP_ADDRESS>, but <IP_ADDRESS> can be better, because <IP_ADDRESS> is loopback address
	a1, a2, _ := net.ParseCIDR("<IP_ADDRESS>/24")
	b1, b2, _ := net.ParseCIDR("<IP_ADDRESS>/8")

	addrs := []net.Addr{&net.IPNet{IP: a1, Mask: a2.Mask}, &net.IPNet{
		IP:   b1,
		Mask: b2.Mask,
	}}

	var ipAddr []byte
	var publicIP []byte

	for _, rawAddr := range addrs {
		var ip net.IP
		switch addr := rawAddr.(type) {
		case *net.IPAddr:
			ip = addr.IP
		case *net.IPNet:
			ip = addr.IP
		default:
			continue
		}

		b := ip.String()

		if !isPrivateIP(b) {
			publicIP = ip
			continue
		}

		ipAddr = ip
		break
	}

	// return private ip
	if ipAddr != nil {
		tool.Info(net.IP(ipAddr).String())
		return
	}

	// return public or virtual ip
	if publicIP != nil {
		tool.Info(net.IP(publicIP).String())
		return
	}

I so have same  question!
I do't known how to solve it.
how about now?

I so have same question!
I do't known how to solve it.
how about now?

In summary,

I use etcd as my registry
Try to telnet etcd url and get local address before startup(I assume if I can connect etcd,I can connect the micro service)
Use this local address as my advice IP.

Now,it work well.

I so have same question!
I do't known how to solve it.
how about now?

In summary,

I use etcd as my registry
Try to telnet etcd url and get local address before startup(I assume if I can connect etcd,I can connect the micro service)
Use this local address as my advice IP.

Now,it work well.

It's the only way


Typo: generate was spelled gemerate

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

View a preview at https://prompt.ws/r/Azure/azure-cli/6289
This is an experimental preview for @microsoft users.

@nicdoye  Please update the setup.py version and the HISTORY

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

Should be ok now.


Modal options is not working

The modal options is not working, apparently just work the "ready" function, opacity, dismissible, startingTop, etc., is not working

Can you try the sample project and see if that is working for you?
We've used dismissible there and it works quite well: http://angular2-materialize.surge.sh/#/Dialogs (try "MODAL 1").
Here's the code: https://github.com/InfomediaLtd/angular2-materialize/blob/master/sample/src/app/components/dialogs.ts#L13


Create a FAQ documentation page

In order to answer frequently asked questions in this issue board.
Can use the issues list as reference:
https://github.com/Bionus/imgbrd-grabber/issues?q=is%3Aissue+label%3Aquestion

A basic FAQ page was added:
https://bionus.github.io/imgbrd-grabber/docs/faq.html


Bump console to 9.4.0.Alpha1

Master only. Bump console for 9.4.0.Alpha1 release.

@danberindei Bbuilt locally and verified to pick up a new version correctly.

Build worked for me as well, thanks Vladimir!


Replace UUID with locator

Objective
Ensure locators are returned instead of uuid's  under appointments section when user details are fetched through the cli using the getuser command.
Problem
locator is an appropriate identifier for appointments  in the Bolt13 specification draft, which is also being used in the getappointments cli command , but in the getuser cli command uuid is being used which can be confusing .  The  solution is ensure getuser cli command makes use of locator and not uuid.
Changes

Modified internal.rs get_user_info function to make use of locator and not uuid when returning  user appointments info.
Modified internal.rs get_user test to reflect the change from uuid to locator.

Scope Of Change
This is a non critical change, as it only affects the CLI.
closes #229

@aruokhai thanks for looking at this. This is going to make much more sense after merging #190, which gets rid of all in-memory data for the tower. Hence, a big refactor will be needed, but it would make the change way smaller. I'd suggest you wait until that is landed and take another look at it.

@aruokhai we just landed #190

added the necessary changes

Looks like this may be mergeable (?). Planing to give it a last look and merge if so. Needs rebase now that everything is green again


Can ZED do SLAM?

Can ZED do SLAM?  If so, what algorithm is used? Is IMU used?
Thank you!

Yes, with the ZedFu app. Or do your own.
Undisclosed. Maybe an ZedFu is an adaption of PCL's KinFu, who knows (Stereolabs does).
No, the ZED has no IMU.

@WASCHMASCHINE Thank you for your answers. One more question, what's the depth accuracy of ZED?

@callmeray To be honest, I just returned to my PC after taking some measurements (~50) for that purpose. I used a laser for point distance attached to the ZED and moved a textured target away from the setup from 1 to 25 m. Now it says 12m in the Depth Viewer (which is even the distance!) while the laser says 25.137m.
I've done the experiment before, but I did not record SVOs. The last result was that I could add an exponential offset to the distance to fit my laser measurements, which did help. I contacted Stereolabs with the data, but they said they will add depth stabilization in the next update, which did not help apparently.
Some scenarios:
a) My camera may be broken
b) The factory calibration is off
c) Their stereo algorithm is wrong or at least the disparity to depth step
d) They do not account for their own calibration value, i.e. the convergent factor. So the camera may be a little cross-eyed, and you may need to add an offset to the disparity, which results in nonlinear depth offset (like observed).
e) The camera could not work under the lighting
Anyway, I have to analyze the SVOs first. Still, the ZED is a solid stereo camera, even though the software might not be delivering me the right results right now. You can still get the raw data on the GPU, which is great.

@WASCHMASCHINE Does it mean that the Depth Viewer is off by 12m for a ground truth depth of 25m? Theoretically, if you can map out a grid in depth then you could use the ground truth values to create a 3D interpolation function (splines or a non-linear fitting would be best IMO). Offsets to the depth values can be corrected offline using this approach.

@RSKothari Yes, that's what I meant. But only for a point in roughly the image center. I did find a non-linear fit for the depth in the first experiment and used it for every depth. But now I want to find the source of the problem, not fighting the symptoms with an adhoc calibration, again.

@WASCHMASCHINE  Thank you so much for your work. According to what you said, I think the most probable reason is that  the factory calibration is off and the system needs to be recalibrated


Tencent providers should use EIP

autok3s version
Version: {"gitVersion":"v0.3.0-rc3","gitCommit":"78d27cd25a72dfc5f0d5c68ae06dfabd69fc4f76","gitTreeState":"","buildDate":"2020-12-31T01:39:28Z","goVersion":"go1.13.12","compiler":"gc","platform":"darwin/amd64"}
If you shut down the instance from Tencent Cloud, the public IP will change, so you need to use EIP by default

autok3s removed start/stop command which means we DO NOT SUPPORT manage instance using EIP by default.

autok3s removed start/stop command which means we DO NOT SUPPORT manage instance using EIP by default.


MacOS viewer resized bmp save.

Height read -20
g.bmp.zip

I didn't tried for now, but I guess you used attached image as debugging sample, so I guess I can close it. Thank you much for fixing.


Use Self-Hosted Runners

Use Self-Hosted Runners
:bulb: Proposed solution

Switches to a reusable workflow and uses the self-hosted runners.

Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[x] I agree to follow the Code of Conduct and Contributing Guidelines.

Thank you @PSchmiedmayer, will merge this so the team can benefit from our new, faster runners!


BZ#1429650-PT#142722601 Observe dashboard_view product feature

https://bugzilla.redhat.com/show_bug.cgi?id=1429650
https://www.pivotaltracker.com/story/show/142722601
The roles (and product features) of only the current group a user is using are made available, this is by current design.

Updates wording to differentiate invalid credentials from permissions
Refactors rbac service to respect product feature dashboard_view

da goods

@AllenBW Travis fail :(

Fine backport details:
$ git log -1
commit 4825f0e1fa14aaf82f5120e1629198b209bb8827
Author: Chris Kacerguis<EMAIL_ADDRESS>Date:   Fri Mar 31 09:21:57 2017 -0500

    Merge pull request #617 from AllenBW/bug/product-feature-login

    BZ#1429650-PT#142722601 Observe dashboard_view product feature
    (cherry picked from commit 45069f8a205bfde4614e6e656b74ee22a675fda9)


ingest with ERR_NETWORK..

i am trying to ingest pdfs, while running scripts , got network error like this...

here is my config ,
PINECONE_ENVIRONMENT=eu-west1-gcp
PINECONE_INDEX_NAME=pdf-test

i have checked  openai - appkey  and http_proxy , both it works with curl www.google.com and connect with openai .

I also have the same error ,do you solve the problem?

Did you change the modelName from gpt-4 to gpt-3.5-turbo ?
I assume you don't yet have access to gpt-4

Did you change the modelName from gpt-4 to gpt-3.5-turbo ?
I assume you don't yet have access to gpt-4

Dear Mayooear,
Thanks for your awesome project!
I also met this err, even through i changed the model name from 4 to gpt-3.5-turbo, but the err still came out.
all the things  i do are like this :
0, git clone
1,pnpm install
2,set the .env correctly
3,and change the pdf-test
'''
const PINECONE_NAME_SPACE = 'pdf-test1'; //namespace is optional for your vectors
'''
to pdf-test1
4, change gpt-4 to gpt-3.5-turbo
5, change pdfs in docs to a  simple pdf, just contains two lines: good luck to all, this is an orange cat.
6, npm run ingest
7, pnpm run dev
but there came out err, and in pinecone dashboard, there is no namespace and no vectors created
the dimension is set as 1536
would you like to figure out my mistake steps?
Thanks so much!
by the way, in terminal, the log is like this:
after creating vector:
'''
creating vector store...
error [Error: Network Error] {
config: {
transitional: {
silentJSONParsing: true,
forcedJSONParsing: true,
clarifyTimeoutError: false
},
adapter: [AsyncFunction: fetchAdapter],
transformRequest: [ [Function: transformRequest] ],
transformResponse: [ [Function: transformResponse] ],
timeout: 0,
xsrfCookieName: 'XSRF-TOKEN',
xsrfHeaderName: 'X-XSRF-TOKEN',
maxContentLength: -1,
maxBodyLength: -1,
validateStatus: [Function: validateStatus],
headers: {
Accept: 'application/json, text/plain, /',
'Content-Type': 'application/json',
'User-Agent': 'OpenAI/NodeJS/3.2.1',
Authorization: 'Bearer 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'
},
method: 'post',
data: '{"model":"text-embedding-ada-002","input":["gggggggooood luck to all people see this doc. this is an orange cat"]}',
url: 'https://api.openai.com/v1/embeddings'
},
code: 'ERR_NETWORK',
request: Request {
[Symbol(realm)]: { settingsObject: [Object] },
[Symbol(state)]: {
method: 'POST',
localURLsOnly: false,
unsafeRequest: false,
body: [Object],
client: [Object],
reservedClient: null,
replacesClientId: '',
window: 'client',
keepalive: false,
serviceWorkers: 'all',
initiator: '',
destination: '',
priority: null,
origin: 'client',
policyContainer: 'client',
referrer: 'client',
referrerPolicy: '',
mode: 'cors',
useCORSPreflightFlag: false,
credentials: 'same-origin',
useCredentials: false,
cache: 'default',
redirect: 'follow',
integrity: '',
cryptoGraphicsNonceMetadata: '',
parserMetadata: '',
reloadNavigation: false,
historyNavigation: false,
userActivation: false,
taintedOrigin: false,
redirectCount: 0,
responseTainting: 'basic',
preventNoCacheCacheControlHeaderModification: false,
done: false,
timingAllowFailed: false,
headersList: [HeadersList],
urlList: [Array],
url: [URL]
},
[Symbol(signal)]: AbortSignal { aborted: false },
[Symbol(headers)]: HeadersList {
[Symbol(headers map)]: [Map],
[Symbol(headers map sorted)]: null
}
},
response: undefined,
isAxiosError: true,
toJSON: [Function: toJSON]
}
/Users/apple/codework/gpt4-pdf-chatbot-langchain/scripts/ingest-data.ts:44
throw new Error('Failed to ingest your data');
^
[Error: Failed to ingest your data]
Node.js v18.14.0
'''

Also getting this error. Let me know if anyone finds anything!

Also getting this error. Let me know if anyone finds anything!

Hello josh-may ,I solve the error. I just put the codes on the serve in America.I do not have anything change.it works !

any update on that, having the same error

same here, anyone solved yet?

我已经解决了这个问题，在我开vpn的情况下，可能是由于vpn软件的协议问题，导致这个现象，我把vpn软件直接换成clash 就解决了这个问题。大家可以试试去


Keep track of total amount in balances mapping

Keep track of total Wei in balances mapping. Let this number be stored in variable owed
When someone calls finalizeExit, validate that
this.balance - owed >= exit.amount
If check passes, finalize exit
Else, continue loop. (Note it may be better to simply return in this case. Open to debate on this.)

Fixed with #34


Add ability to mute feeds

Hi guys, I would like to add the ability to mute a feed so it doesn't show up in Smart Feeds.
To add some context: I'm subscribed to some feeds that I need to be in a certain mood to consume, or are thematically scattered all over the place, so I only browse them when I have time to kill.
Since I use NNW mostly through the Smart Feeds, I'd like to introduce a way to mute those feeds so they don't show up in the Smart Feeds and pollute them.
That way I can see all the feeds that are important to me in one place, without having to jump back and forth between them or skip a dozen "secondary" feed items in the Smart Feeds.
The easiest way to achieve this in my opinion, is to simply introduce a new switch in the WebFeedInspectorViewController, below 'Notify about new articles'.
If you approve this issue, I will implement this feature.
Please let me know when you have any questions regarding this ticket.

Duplicate of #3401.

@luhinderberger If you are interested in working on this, the first thing to do is read our CONTRIBUTING document and then discuss the feature in our #work Slack.


no method 'offset'

what dependences has ng-slider ? because i receive TypeError: Object [object Object] has no method 'offset'

nop, do not work, the issue is here "var f=d.offset(this.domNode)".

Ok I check it asap. Fiddler is on readme fiest page if you can reproduce it. I will check on my side too

i thing this is the difference  AngularJS v1.3.1

"d" variable (name after minification) in your code is just an utils function, maybe injection failed but it is weird. I will look more in depth with 1.3

tested with 1.3 I have not seen any differences, pls use fiddle or send me code and options I can reproduce

something is wrong in min, utils is not injected, i solved the problem in min file, is not a solution but i don't have time to test again

renamed 'utils' factory to 'sliderUtils'


Switch to GitHub Actions

Fixes issue #6. The workflow tests on Ubuntu, Windows and MacOS on both node versions 8 and 10. I've also added the readme badge.

Fantastic, thanks Jake!


Unit Tests Not validated

Hi ,
Thank you so much , I tried to validate the ISDA Unit Test (Its free) and I have find some test that doesn't pass the test using your code.
Exemple when crif file is :
portfolio_id,trade_id,collect_regulations,product_class,risk_type,qualifier,bucket,label1,label2,amount,amount_currency,amountusd
C330,C330.S_IRV_16,All,RatesFX,Risk_InflationVol,USD,,30y,,80000000.0,USD,80000000.0
C330,C330.S_IRV_21,All,RatesFX,Risk_InflationVol,USD,,2w,,-80000000.0,USD,-80000000.0
Official unit test expect the result to be 0 and your code return a Curvature of 2,442,463.46
An other failed test :
portfolio_id,trade_id,collect_regulations,product_class,risk_type,qualifier,bucket,label1,label2,amount,amount_currency,amountusd
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
Official unit test expect the result to be 1,898,530,683.542 and your return a Vega          1,706,855,277.77
If you have an idea to correct it and validate all unit test it will be really good. Thank you for all.

Thanks for your response!
Could you advise which version did you try?

Yes , it was the last version 2.5 . Just for information only some tests have failed , most of them are validated.

I have no access to the unit test v2.5. I would appreciate if you could upload the regarding crif in csv through a pull request or any other way around; so that I can look them up!

I have put 2 examples of crif file , you can create a file csv and copy past this on the file for the first failed test for example ;
portfolio_id,trade_id,collect_regulations,product_class,risk_type,qualifier,bucket,label1,label2,amount,amount_currency,amountusd
C330,C330.S_IRV_16,All,RatesFX,Risk_InflationVol,USD,,30y,,80000000.0,USD,80000000.0
C330,C330.S_IRV_21,All,RatesFX,Risk_InflationVol,USD,,2w,,-80000000.0,USD,-80000000.0
Maybe you need to change to columns names. Expected output is 0 .
And the other one is the same a file csv and you past this :
portfolio_id,trade_id,collect_regulations,product_class,risk_type,qualifier,bucket,label1,label2,amount,amount_currency,amountusd
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
C349,C349.S_FXV_7,All,RatesFX,Risk_FXVol,RUBTRY,,5y,,-12000000.0,USD,-12000000.0
Official unit test expect the result to be 1,898,530,683.542 and your return a Vega 1,706,855,277.77

I am looking into the case for Risk_InflationVol.
As for the case Risk_FXVol, 1,898,530,683.542 is returned as you said when RW(risk weight) is 14.6, which is the case both given currency and calculation currency are considered to be high volatility group according to the clause 69(p.24). However, as long as the calculation currency is USD, RW cannot be 14.6.
I don't think the numbers in the unit tests are incorrect, but I would appreciate if you could share any idea on this matter. I will update here as soon as I find something!

Happy new year , just for information , for the FXVol unit test the RW is 14.6 . As you can see in 10 (b) (p.5) :
For FX vega (which depends on a pair of currencies), the risk weight to use here is the entry from the FX delta risk weight table, given in section I, whose row is the FX volatility group of the first currency and whose column is the FX volatility group of the second currency.
In our case RUBTRY then RW = 14.6 .

@LeoMessiiiii Where can I find the ISDA Unit Tests? Perhaps I can implement them and make a pull request on this repo.

Hello,
You can use this one (Combinations (10-day))
ISDA-SIMM-UnitTesting-v2.5.xlsx

@LeoMessiiiii Thank you for the extremely fast response!

@LeoMessiiiii Oh thank you so much! Let me take a look and fix any bugs soon!

@LeoMessiiiii Thanks for your sharing!! Let me take a look too!
@OpenInsightsNow Always welcome your contribution!!

It's been a very long time since I forgot that I had fixed the unit tests to make them pass, but I don't remember if all the tests were passing or not.
There have been some changes :

Be sure to use weights_corr2_5

On agg_sensitivites.py => change calculation_currency = 'USD' => calculation_currency = Simm_calculation_currency

Then add an exchange rate on margin_risk_class.py => for example :
T = wnc.T(risk_class,'Delta',bucket=bucket)   => T = wnc.T(risk_class,'Delta',bucket=bucket) * self.exchange_rate
VT  = wnc.T('Rates','Vega',currency=currency)   =>  VT  = wnc.T('Rates','Vega',currency=currency) * self.exchange_rate
is_given_currency = (currency_pair[:3] in wnc.high_vol_currency_group or currency_pair[3:] in wnc.high_vol_currency_group)
is_calc_currency  = self.calculation_currency in wnc.high_vol_currency_group
BY
is_given_currency = (currency_pair[0:3] in wnc.high_vol_currency_group)
is_calc_currency = (currency_pair[3:6] in wnc.high_vol_currency_group)
I also change updates['Rates']['Curvature'] += max(CVR_sum + _lambda * sqrt(K), 0) / (HVR2)  By;
if risk_class == 'Risk_InflationVol' and S < 0:
updates['Rates']['Curvature'] = 0
else:
updates['Rates']['Curvature'] += max(CVR_sum + _lambda * sqrt(K), 0) / (HVR2)

To Avoid some crashes :

Replace crif_risk_class = self.crif[(self.crif['risk_type']==risk_class) & (str(self.crif['bucket']).lower() =='residual')]
By : crif_risk_class = self.crif[(self.crif['risk_type']==risk_class) & (self.crif['bucket'].str.lower().isin(['residual']))]
In Utils.py : if 'residual' in (str.lower() for str in list(df['bucket'])) => if 'residual' in (str(y).lower() for y in list(df['bucket']))
In Agg_margins.py , i have added onlyaddon = false and changed :
df_total_err['SIMM Total'] = df_total['SIMM Total'] = simm_total
By:
if (df_total.empty or df_total_err.empty) and addon_margin != 0:
dt = {'SIMM Total' : [simm_total] ,'Add-On' : [addon_margin]}
df = pd.DataFrame(dt)
onlyAddon = True
return df, error_occured, onlyAddon
else:
df_total_err['SIMM Total'] = df_total['SIMM Total'] = simm_total

In my preliminary investigation, I found that 43 of the 497 unit tests fail.  I have not checked that the test results from the spreadsheet are correct. As per LeoMessii's comment on Jan 2, I think there might be a bug on in 'margin_risk_class' on line 446. The calc currency and given currency seem to be switched incorrectly. This error may be repeated two other times in the code base.  I will make a pull request when I get the time.
['C330', 'C342', 'C349', 'C359', 'C391', 'C392', 'C393', 'C397', 'C401', 'C415',
'C416', 'C418', 'C419', 'C420', 'C426', 'C449', 'C451', 'C452', 'C453', 'C454',
'C456', 'C457', 'C458', 'C459', 'C466', 'C471', 'C472', 'C478', 'C479', 'C480',
'C481', 'J1', 'J2', 'J3', 'J4', 'J5', 'J6', 'J7', 'J8', 'J9',
'J10', 'JS1', 'JS2']

@LeoMessiiiii Test C349 from the unit test is now passing based upon fix in my pull request.

Its been a long time as i said before , in my side I did the same correction to pass this test :)

@LeoMessiiiii: How did you determine this fix?  I know you said it was a long time ago, but perhaps you still have some idea.
if risk_class == 'Risk_InflationVol' and S < 0:
updates['Rates']['Curvature'] = 0
else:
updates['Rates']['Curvature'] += max(CVR_sum + _lambda * sqrt(K), 0) / (HVR**2)

In C.1) 14 (p9), there states that "All sensitivities to inflation rates for the same currency are fully offset". So the following C330 can be 0:
C330,C330.S_IRV_16,All,RatesFX,Risk_InflationVol,USD,,30y,,80000000.0,USD,80000000.0
C330,C330.S_IRV_21,All,RatesFX,Risk_InflationVol,USD,,2w,,-80000000.0,USD,-80000000.0
However, I don't understand why C331 is not 0:
C331      RatesFX       S_IRV_19                All  Risk_InflationVol       CAD    NaN    10y    NaN   10000000            USD   10000000
C331      RatesFX       S_IRV_22               All  Risk_InflationVol       CAD    NaN    30y    NaN  -10000000            USD  -10000000

@OpenInsightsNow in our side we don't have 'Risk_InflationVol' in our Crif i think i did this just to pass some unit test...(i know its not clean :) )
@meenmo here the IRvega/curvature is calculated for CAD and for the unit test the currency is USD

@LeoMessiiiii Thanks for your reply!
I am working onward and got stuck at C335 though. Appreciate if anyone could share any idea on C335!

No pb in my side maybe you can find solution in this comment https://github.com/meenmo/ISDA_SIMM/issues/3#issuecomment-1657889729

@meenmo I cannot remember whether test C335 was passing for me. I will check and respond. Also, I will push my (very basic) unit tests.

Test C335 works fine for me.  It is strange that the test is failing for you.

Hi all,
I've updated v2.6 with minor fix. Please open issue if needed.
Thanks for your contribution!


fixed new gym API related to step() and reset()

Description
There are several issues related to changes in API.  I re-adjusted all code, so it returned truncated in step and readjusted reset to match what the gym expected.   In parallel, fixed Mario env.  Note old code won't work with this since GYM now return five elements in a tuple vs. 4.  Same for reset.   I also re-adjusted all the unit tests and examples, so it matched and passed all tests.

Fixes #Incorrect number of arguments from call to env.step(action) #119

Type of change
Please select all relevant options:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

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

[ x] Unit test
[x] run gyp-super Mario ( with readjusted step /reset) and its unit test.

Test Configuration

Operating System:  12.5.1 mac os
Python version: 3.10
C++ compiler version: Apple clang version 13.1.6 (clang-13<IP_ADDRESS>.5)
Target: x86_64-apple-darwin21.6.0
Thread model: posix

Checklist

[ x] My code follows the style guidelines of this project
[ x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ x] I have added tests that prove my fix is effective or that my feature works

Any reason why this shouldn't be merged with the suggested edits?


@conda-forge-admin, please add user @patrick-austin

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @patrick-austin in conda-forge/muspinsim-feedstock#6.


fix title on edit guide page to say 'Edit' instead of 'Create'

In response to Issue #513
The same template (_new_guide_steps.html.erb) was being used for creating a new guide AND for editing an existing guide. To fix this, a local variable 'edit' is passed to the template when it is rendered for editing. When edit = true, a different title is given which says 'Edit A Growing Guide'

Coverage remained the same at 99.94% when pulling b8569c02b1e457fb40950078c22ba30491935b3a on rjmccluskey:fix-edit-guide-title into 739fe3ed42cff34a80257d23f9895d6356e1068e on openfarmcc:master.

Thanks for the PR @rjmccluskey! We know that they're the same, but at the moment the functionality shared between the two is pretty much identical. Once they start diverging a bit more (beyond this one if/else statement so far) we'll probably break them out into different pages.
(just to explain the current situation)
Before I pull this in, could you add a english language instance .edit_a_growing_guide in en.yml?

Added :)

Coverage increased (+0.0%) to 99.94% when pulling a7461501907238757a7fc1568fb1a016bae916ad on rjmccluskey:fix-edit-guide-title into 739fe3ed42cff34a80257d23f9895d6356e1068e on openfarmcc:master.

Glad to see you're up and running @rjmccluskey! Thanks for the PR

Sorry, meant to pull this in a couple of days ago!

Woo! My first PR to OpenFarm! haha no worries @simonv3


How to determine the limit of request times？

$limitResponse = LaravelOpenRouter::limitRequest();

我应该根据那个参数来判断是否可以继续进行请求？ 正确响应和错误响应反核的结果一致
Should I use that parameter to determine if I can proceed with the request? The correct response and incorrect response result in consistent anti nuclear results

@gyp719
Well, If I get the question right, limitRequest() does not need any parameter. Your query and response are correct.
For more info about limitRequest, you can also check OpenRouter document:
https://openrouter.ai/docs/limits
As stated in document, query response is for "To check the rate limit or credits left on an API key".
If answer does not answer what you are looking for, please give some more info, I will be able to respond as soon as possible.


Update travis-ci config

Coverage remained the same at 100.0% when pulling ac802620403001f36f2b6ee37347e1c1ca4f82f8 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling 847b8471a89fa1ea2156d1c9ce3cf8dd22aecf01 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling 847b8471a89fa1ea2156d1c9ce3cf8dd22aecf01 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling 847b8471a89fa1ea2156d1c9ce3cf8dd22aecf01 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling 847b8471a89fa1ea2156d1c9ce3cf8dd22aecf01 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling 847b8471a89fa1ea2156d1c9ce3cf8dd22aecf01 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling 847b8471a89fa1ea2156d1c9ce3cf8dd22aecf01 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling 1bffef2b7af94c7069af7313bd01c01568b41915 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling fd557ac930604ee38b9f6aa4c80fe63ba43193ba on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling fd557ac930604ee38b9f6aa4c80fe63ba43193ba on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling fd557ac930604ee38b9f6aa4c80fe63ba43193ba on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling fd557ac930604ee38b9f6aa4c80fe63ba43193ba on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling fd557ac930604ee38b9f6aa4c80fe63ba43193ba on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling fd557ac930604ee38b9f6aa4c80fe63ba43193ba on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling fd557ac930604ee38b9f6aa4c80fe63ba43193ba on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling fd557ac930604ee38b9f6aa4c80fe63ba43193ba on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage decreased (-0.9%) to 99.13% when pulling 5b5206f5b5fdc81db8f807d25fa39e9935d08450 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage decreased (-0.9%) to 99.13% when pulling 5b5206f5b5fdc81db8f807d25fa39e9935d08450 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage decreased (-0.9%) to 99.13% when pulling 5b5206f5b5fdc81db8f807d25fa39e9935d08450 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage decreased (-0.9%) to 99.13% when pulling 5b5206f5b5fdc81db8f807d25fa39e9935d08450 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage decreased (-0.9%) to 99.13% when pulling 5b5206f5b5fdc81db8f807d25fa39e9935d08450 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage decreased (-0.9%) to 99.13% when pulling 5b5206f5b5fdc81db8f807d25fa39e9935d08450 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage decreased (-0.9%) to 99.13% when pulling 5b5206f5b5fdc81db8f807d25fa39e9935d08450 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage decreased (-0.9%) to 99.13% when pulling 5b5206f5b5fdc81db8f807d25fa39e9935d08450 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Could you authorize with AppVeyor, add this project there and run a build with this PR?
I had some issue on Windows (https://github.com/bbatsov/rubocop/issues/3800) with one of changed cops.

Coverage remained the same at 100.0% when pulling 3ba16a7ae2d042390163c28f1913bbe3e0122b77 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling 3ba16a7ae2d042390163c28f1913bbe3e0122b77 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling 3ba16a7ae2d042390163c28f1913bbe3e0122b77 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling 3ba16a7ae2d042390163c28f1913bbe3e0122b77 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling 3ba16a7ae2d042390163c28f1913bbe3e0122b77 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling 3ba16a7ae2d042390163c28f1913bbe3e0122b77 on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

@abotalov sure. and we need to make coveralls be less noizy

@abotalov added - although i must admit that appveyor's integration kinda terrible IMO

Coverage remained the same at 100.0% when pulling e165f4135b66610b028e508ba05b30c480ca14fa on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling e165f4135b66610b028e508ba05b30c480ca14fa on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling e165f4135b66610b028e508ba05b30c480ca14fa on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling e165f4135b66610b028e508ba05b30c480ca14fa on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling e165f4135b66610b028e508ba05b30c480ca14fa on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling e165f4135b66610b028e508ba05b30c480ca14fa on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling e165f4135b66610b028e508ba05b30c480ca14fa on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

Coverage remained the same at 100.0% when pulling e165f4135b66610b028e508ba05b30c480ca14fa on improve/testing into a2e5e06f95df560cb000410956a008a223c7c6c3 on master.

@abotalov is there a way to make enforce LF stay LF on Windows? I don't have any windows machines so i can't play around myself.

@ixti You can revert commit https://github.com/httprb/form_data.rb/commit/a7565fc7883ed70071d6a9023efe4430446c69d6
This will fix AppVeyor but it will not make Rubocop passed if git config core.autocrlf = true which is a recommended setting in Github docs and Git Book.
I think it would be better to disable Style/EndOfLine cop as it doesn't work properly (bbatsov/rubocop#3401, bbatsov/rubocop#3800) and disabling will make default rake task work for others on Windows.

@abotalov I tend to either avoid rubocop on windows or removing that git config setting on appveyor. So far second thing seems more simple to me. Developers on windows can disable Style/EndOfLine on rubocop while developing.


Release Google.Cloud.EdgeNetwork.V1 version 1.0.0-beta01

Changes in this release:
Initial release.
PiperOrigin-RevId: 582128758

Created release for Google.Cloud.EdgeNetwork.V1-1.0.0-beta01


Pipelinerun e2e test refactor

Refactor pipelinerun e2e test

What:

pipelinerun e2e tests are running lot of common cruft and testing similar things. Merged the tests into table driven framework.
Removes TestPipeline as the runtime controller validation in pipelinerun lists the pipeline reference so the functionality of CRD existence is tested internally. This test is not providing value anymore.

Why: Reduces run time as the sub tests spin up in parallel

/assign @bobcatfish
/assign @tejal29
/assign @aaron-prindle
/assign @pivotal-nader-ziada

/lgtm
/hold

Sample logs. Its clear which test is emitting them and easy to debug.
=== RUN   TestPipelineRun
info	TestPipelineRun	test/init_test.go:91	Create namespace arendelle-iscgrxgm to deploy to
info	TestPipelineRun	test/pipelinerun_test.go:45	Creating Pipeline Resources in namespace arendelle-iscgrxgm
=== RUN   TestPipelineRun/multiple_tasks
info	TestPipelineRun	test/pipelinerun_test.go:140	Setting up test resources for "multiple tasks" test in namespace arendelle-iscgrxgm
info	TestPipelineRun	test/pipelinerun_test.go:149	Waiting for PipelineRun helloworld-pipelinerun0 in namespace arendelle-iscgrxgm to complete
info	TestPipelineRun	test/pipelinerun_test.go:164	Making sure the expected TaskRuns [helloworld-task-1 helloworld-task-2] were created
info	TestPipelineRun	test/pipelinerun_test.go:182	Making sure 3 events were created from taskrun and pipelinerun with kinds map[TaskRun:[helloworld-pipelinerun0-helloworld-task-1 helloworld-pipelinerun0-helloworld-task-2] PipelineRun:[helloworld-pipelinerun0]]
info	TestPipelineRun	test/pipelinerun_test.go:188	Successfully finished test "multiple tasks"
=== RUN   TestPipelineRun/service_account_propogation
info	TestPipelineRun	test/pipelinerun_test.go:140	Setting up test resources for "service account propogation" test in namespace arendelle-iscgrxgm
info	TestPipelineRun	test/pipelinerun_test.go:149	Waiting for PipelineRun helloworld-pipelinerun1 in namespace arendelle-iscgrxgm to complete
info	TestPipelineRun	test/pipelinerun_test.go:164	Making sure the expected TaskRuns [helloworld-task-1] were created
info	TestPipelineRun	test/pipelinerun_test.go:182	Making sure 2 events were created from taskrun and pipelinerun with kinds map[TaskRun:[helloworld-pipelinerun1-helloworld-task-1] PipelineRun:[helloworld-pipelinerun1]]
info	TestPipelineRun	test/pipelinerun_test.go:188	Successfully finished test "service account propogation"
info	TestPipelineRun	test/init_test.go:74	Deleting namespace arendelle-iscgrxgm
--- PASS: TestPipelineRun (411.59s)
    --- PASS: TestPipelineRun/multiple_tasks (261.06s)
    --- PASS: TestPipelineRun/service_account_propogation (149.91s)

@bobcatfish @tejal29 : Please review the PR again and let me know if I have addressed all comments.

/lgtm

/hold cancel


System Error when running Ganache 2.7.0 on darwin

PLATFORM: darwin
GANACHE VERSION: 2.7.0
EXCEPTION:
TypeError: Cannot read property 'stack' of undefined
    at IpcMainImpl.<anonymous> (/webpack:/src/integrations/ethereum/index.js:76:29)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)

Duplicate of #5296


VM can utilize more CPU than allocated

Description
There is some problem with resources allocation for VM,
If you will load CPU high, it can utilize more resources than alloceted for it:

To Reproduce

Create new template.
Put forkbomb into starting script:

:(){ :|:& };:

Instantiate VM

Expected behavior
VM can't use resources more than allocated
Details

Affected Component: Libvirt
Hypervisor: KVM
Version: 5.6.1

Libvirt versions:

libvirtd (libvirt) 4.6.0 (compiled and installed on ubuntu xenial and bionic)
libvirt.x86_64 3.9.0-14.el7_5.8 (another guy with same problem)

Additional context
If I run systemd-cgtop, I see nothing on CPU column for VM
Control Group                        Tasks   %CPU   Memory  Input/s Output/s
/machine/qemu-12-one-1234.libvirt-qemu  -      -   353.3M        -        -

But parent cgroup shows some values
Control Group                        Tasks   %CPU   Memory  Input/s Output/s
/system.slice/libvirtd.service        225  387.3    34.7G        -        -

Zabbix shows something weird:

I think it can be connected with software interruptions.
Any ideas?

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

As far as I know, OpenNebula just pass the cpu value to the cpu shares in cgroup. So it's just a weight to give a higher priority to specific VM. It's not a way to enforce CPU limitation. cf https://docs.opennebula.org/5.6/deployment/open_cloud_host_setup/kvm_driver.html?highlight=cgroup#working-with-cgroups-optional

closing, CPU is for allocation, vCPU applies to guestOS perception.

The docs suggest that CPU may affect how much CPU is available at runtime (i.e., it is not just for allocation) https://docs.opennebula.io/5.10/deployment/open_cloud_host_setup/kvm_driver.html?highlight=vcpu#working-with-cgroups-optional


Update copyrights for Tigera.

LGTM.

Sounds like we need to respin and update the existing copyrights too

Respun with updates to Metaswitch copyrights too, should be ready to merge.


Update nf-d3d12shader-id3d12shaderreflection-getrequiresflags.md…

…to include the new flags and documentation from the Agility SDK's. This should help HLSL, DXC, and D3D12 developers with accurate documentation so that they don't have to dig through header files to find it.

@microsoft-github-policy-service agree [company="Bethesda Game Studios"]

@microsoft-github-policy-service agree company="Bethesda Game Studios"

Thank you! These additional rows will make the table unreadable, so I'll go ahead and fix that in a separate PR after this one has gone through.


Using the Closing event of the ContentDialog causes a crash

Describe the bug
There's a bug here:
https://github.com/lepoco/wpfui/blob/a58feed8d00f230a95f647cb14054818a8ca4a6d/src/Wpf.Ui/Controls/ContentDialog/ContentDialog.cs#L607
To Reproduce
var dialog = new ContentDialog(contentPresenter);
dialog.Closing += AnyMethod;
var result = await dialog.ShowAsync();

it's will throw a exception when close dialog.
Expected behavior
No exception
Screenshots
No response
OS version
any version
.NET version
any version
WPF-UI NuGet version
3.0.0-preview.7
Additional context
No response

Fixed in #806


CCG Custom Plugin

What is your question?
I have been setting up a custom CCG plugin on my local machine to run a windows container as non-domain joined.
When starting the container, using the following command:
docker run -it --security-opt "CredentialSpec=file://gmsa.json" mcr.microsoft.com/windows/servercore:ltsc2022 powershell
I can see the following message in the Microsoft-Windows-Containers-CCG/Admin log:
Container Credential Guard instantiated the plugin: {3C0BED28-38FC-42A9-A6AC-967A4839DF61}
However the container doesn't start, but just gives the following error:
docker: Error response from daemon: hcs::CreateComputeSystem 566516d3912884945d031f7982c6fea1977f22c5d07d40766173354444fb7fc5: winapi error #3489661095.
What can I do to get a more detailed information regarding the error, as it is quite difficult to understand what is failing.
Configuration:

Edition: Windows 11
Container engine: docker
Container Engine version: 27.2.0

@hordurmarhafsteinsson Thanks for sharing another error with CCG. We've brought up this Issue to the team that owns CCG and will update this thread when we have something to share.

Would you mind opening a ticket with Microsoft support? The CCG team prefers to go through the official mechanisms.


Added spread syntax for details param

in getAccountDetails and getChatDetails
yeah, it works perfectly fine, but I assume you know better ways to achieve the same job, so check the code out.
in addition, by supplying multiple detail parameters the user is forced to also supply the chat parameter in getChatDetails

also congrats for the big achievement 👏

also congrats for the big achievement 👏

I can't believe the growth stlil didn't flatline (I literally stopped marketing it)

I can't believe the growth stlil didn't flatline (I literally stopped marketing it)

nah it's skyrocketing still 😂

this is why

you either -1 both chat and data.length or you don't

this is why

you either -1 both chat and data.length or you don't

nah it was fine as is, if data.length is 3 user supplies '3' for oldest chat, 3 !> 3 so is not out-of-bounds, then method gets index 2 and succeeds. if user supplies '4' for 4th chat, 4 > 3 so is out-of-bounds and correctly errors. If user supplies 1000, index 999 is out of bounds not 1000 so the removal of -1 is not rght

because user seeking chat #1000 is index 999

but you gotta remember the user's parameter starts from 1 and not 0, so to don't make it too complicated, we can just change the message to "is too big"

else the user is going to think the last chat is (taking your example) 999 instead of 1000 (talking about parameter value, not list index)

yes if user seeks 1000 then it's index 999 that's out of bounds not 1000, but indeed they can be confused so i'll fix the wording

I'm also organizing it to be more user-friendly so don't wait up

ye but how do we solve chat being forced to pass when multiple detail are supplied?

ye but how do we solve chat being forced to be passed when multiple detail are supplied?
also, use chatgpt instead of this to make it user friendly

I solved it in https://github.com/madkarmaa/chatgpt.js/commit/071e96c3109b632ad08d68570aa87b5996c30e7d can you try to break or improve it

sure, I'll check it out tomorrow, it's almost 11pm right here and I'm going to sleep soon


Add Example

I'm actually struggling to write a simple nearley parser using this module. I keep getting "unexpected ws token" when adding indentation to a toy example, so I wonder how I need to modify my grammar to make it work.

Unfortunately I haven’t used this project in a while so I need to work it out myself to add an example, but it sounds like you need to expect a ws somewhere in the grammar, hopefully the error message should help to pinpoint where in the grammar it’s needed.
You can also print out the moo tokens to see what is being generated, or use nearley-test to look for the discrepancy.


Modal Status Bar Translucent

Currently the Modal component on Android is rendered below the Status Bar, which changes it's color to grey, and in the UIExplorer example the backdrop is just formatted to look the same color. In some scenarios users may want to preserve the color of their status bar and make it look as though the modal is appearing on top. This PR allows for that.
This GIF shows current behavior and new behavior with the translucentStatusBar prop set to true.

I've updated the UIExplorer app to demonstrate and the docs as shown below

Thanks!

Should we add a prop for this? Feels like it should always appear on top of the status bar. It would probably still be possible to achieve the old behavior using the StatusBar module and changing the status bar color when the modal is opened.

Have you tested on older android version that do not support the TRANSLUCENT window flag?

@jemise111 updated the pull request.

@jemise111 updated the pull request.

@janicduplessis Fixed so the top of the modal is only pushed down on the appropriate API level on android.
Nothing breaks in the Java code for older versions.
Could you please clarify your other comment? Thanks!

I'm wondering if the modal should always render over the status bar (like when you set Translucent Statusbar to true in the example). I think it looks a lot better and is what pretty much every app does.

I think this should be the default behaviour instead of adding a prop.

Thanks for looking @janicduplessis and @grabbou. Let's make this default.

@mkonicek great, will do! Think it's necessary to keep a prop for rendering the dialogue the original way? Something like 'persistsStatusBar={true}'?

@jemise111 I don't think anybody really wants that. Do you think people need it?

@satya164 no, I agree with you. Just wanted to check.

@jemise111 updated the pull request.

@mkonicek done!

@jemise111 updated the pull request.

@jemise111 updated the pull request.

@janicduplessis done!

@jemise111 updated the pull request.

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.

This is failing some internal tests.  I am working to get those fixed and have this landed.
As a bonus, this pull request fixed an internal issue that someone posted about yesterday.  I told them to try this patch and it solved the issue.  That's pretty awesome!

@dmmiller thanks! Let me know if I can help in any way.
And that's really cool! Glad to hear it.

I'm working on open sourcing the test that is failing so you can look into it :)  And I'll keep looking at it as well.

@jemise111 So the problem is that by introducing a dependency on StatusBar, we have to update ReactPackage to include StatusBarManager which some of our internal apps don't do.  Instead what I've done is expose a StatusBarHeight constant on Modal to use.  I should be landing it today with that change and you can see.

Awesome @dmmiller. Thanks for the update.

Is this behaviour achievable currently?


Create find_all_by_url method

Extract the behavior of the Collection#all method
into find_all_by_url so we can use the same behavior
with any url passed in.
Collection#all now calls find_all_by_url.
by passing in klass.endpoint as an argument.

@bdunne - Please review. I created this PR to use the new Collection#find_all_by_url method in the upcoming update_all_inventory_sources PR as it makes those methods easier to read.


Feature/319-Setup-Instambul-Test-Coverage

Issue: #319
Following guideline of https://istanbul.js.org/

Hey @GQuirino , I'm very sorry but  the tickets I created are not very clear. It turns out there's more investigating to do. First, https://jestjs.io has Istanbul included so we can set that all up there. Second, it looks like there are some legacy configurations in our package.json file that are causing problems. I tried setting up the coverage reporting tool following some of the suggestions here and have been unsuccessful. If you have some time and want to also take a look, that would be great.  I'm going to add additional commentary and requirements to the original ticket and close this PR for now. Let me know if you have any questions.


Set matrix based on changed file types

Currently, CodeQL runs all languages defined in the array, even when a pull request does not touch a particular language.  For example a single change to index.js triggers Python tests if that repo contains any Python at all.
Some of the checks take much longer than the others, so engineers are often sat waiting around for Python checks to pass when they didn't change any Python.
It would be great if this action was able to check the file extensions changed in a Pull Request, and set the matrix accordingly.

@CallMeGreg I'd like to take this on if that works for you?
@p5 Some engineering teams may prefer to maintain control over this aspect themselves. It might be worthwhile to consider adding an option where, if specific files are provided to the action, it returns the detected languages based on their extensions.
In the absence of any specified files, the action could default to its current behavior, returning all detected languages.

if specific files are provided to the action, it returns the detected languages based on their extensions.

Apologies for not quite following your suggestion here regarding the specific files being provided.  Do you mean having a custom step before this which will query git for the changed files, feeding all of them into this action as an input?
I agree that we should try and keep the current usage available to consumers of the action, so I was thinking of an input such as analyze-only-changed-files: true|false (defaulting to the current behaviour of false).

@LeShadow by all means, thanks for contributing! FWIW, I believe GitHub uses linguist to determine languages in repositories. Love the idea @p5 shared about an input parameter along the lines of only-scan-changed-file-types defaulting to false.

@CallMeGreg Thanks! I already started on something in the background!
@p5 Exactly!
I would use something like this action: https://github.com/tj-actions/changed-files
(It can output changed files in a multitude of ways, like separated by a comma, or in json format)
And then feed the output of that action into this action where we can return languages based on the extensions found in the list of files.
However, if the argument is empty, we just return all detected languages.
Does that make sense?


Config fields get populated in JSON config even if not displayed

Describe The Bug
When creating a config schema, you can use the condition property to show or hide a field based on the value of another field.
For example, let's say that I have an iPad that may or may not have cellular capability.
In the config schema I add a required checkbox (boolean) field named hasCellular.
Then I add a non-required textbox (string) field named phoneNumber, with condition "return model.devices[arrayIndices].hasCellular === true;"
    ...
    "hasCellular": {
        "title": "iPad Has Cellular Capabilities",
        "type": "boolean",
        "required": true
    },
    "phoneNumber": {
        "title": "Phone Number",
        "type": "string",
        "required": false,
        "condition": {
            "functionBody": "return model.devices[arrayIndices].hasCellular === true;"
        }
    },
    ...

In Config UI, I go ahead and configure in the Plugin Config window an iPad that has cellular:

I tick the hasCellular checkbox, and the field phoneNumber is displayed.
I enter the value<PHONE_NUMBER> in field phoneNumber.

Then I realize my mistake: this iPad does not have cellular capability! So I uncheck hasCellular and the the field phoneNumber is hidden. I click the Save button.
If in Config UI, I go to Settings -> General -> JSON Config and look at the json, it will show:
  ...
  "hasCellular": false,
  "phoneNumber": "0123456789",
...

While this is a valid configuration per the schema, it logically makes no sense. My expectation would be that if the field phoneNumber is not displayed, then it is not part of the configuration of this device, therefore it would not appear in the json.
Logs
No response
Config
No response
Homebridge UI Version
4.62.0
Homebridge Version
1.8.4
Node.js Version
22.9.0
Operating System
macOS
Environment Info
Using hb-service
Raspberry Pi Model
None

Update: After re-reading the issue, I realized that there are problems when the form has hidden fields.
To clarify:

When a field is dynamically hidden, then any user entered value is deleted, and the properties default and required are stripped from the field.
When a field is dynamically displayed, the properties default and required are restored as defined in the schema (any user entered value is lost)

This could be done by removing the fields from the DOM and then adding them back (minus the user entered values).

Per the Angular JSON schema documentation, I have tried using dependencies  in the json:
{
  "type": "object",
  "properties": {
    "foo": { "type": "string" },
    "bar": { "type": "string" }
  },
  "dependentRequired": {
    "foo": ["bar"]
  }
}

If the "foo" property is present, then the "bar" property is required.
This did not work.
I tried using the onChange event:
  {
    "key": "name",
    "onChange": "alert(Hello!)"
  },

but this didn't work
I tried assigning a function to the onChange event:
  {
    "key": "text",
    "onChange": function (evt) {
      var value = $(evt.target).val();
      if (value) alert(value);
    }
  },

This caused the entire form to fail to display.

I don't thinking dependencies is implemented into the framework.

You are right, they are not fully implemented:

Note: The contains and dependencies validators are still in development, and do not yet work correctly.

Update: default and required can be handled dynamically.
What is needed is when a field is dynamically hidden, then any user entered value is deleted. This way a dynamically hidden field value cannot be submitted.

Keeping this open.


Is self host supported?

Hi, I'm trying to get Nancy.Swagger working in a self-hosted environment.
I'm using Nancy 1.4.3 so I switched on the right branch and tried the sample.
All run fine if hosted in IIS but fails if Nancy is self hosted.
It seems that MetadataModules aren't loaded and I get this output:
  "paths": {
    "/users": {
      "get": {
        "summary": "Warning: no annotated method found for this route",
        "description": "[example]",
        "operationId": "GetUsers"
      },
      "parameters": []
    }

Is this a known issue?
Thanks, Marco

That looks like an error for Nancy.Swagger.Annotations.
Are you trying to use that or are you trying to use the MetadataModules? If you're using the metadata modules, you should not include Nancy.Swagger.Annotations because it replaces the default metadata provider.

Thanks a lot!
You're right, I'm using MetadataModules and I had to remove the reference to Nancy.Swagger.Annotations
Now works as expected.
Thanks again, Marco

You're welcome!


⚠ Mobitopup has degraded performance

In 4924420, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 13486 ms

Resolved: Mobitopup performance has improved in b02857e.


⚠ Bofrost.es has degraded performance

In 2edb917, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1741 ms

Resolved: Bofrost.es performance has improved in 756849c after 6 minutes.


telegram notifications when launching the application

If I launch the application and the broadcast is already in progress, recording begins, should I receive a notification in Telegram?
or does the notification only come if the broadcast started while monitoring?
This config correct?
[推送配置]
直播状态通知(可选微信|钉钉或者两个都填) =
钉钉推送接口链接 =
微信推送接口链接 =
钉钉通知@对象(填手机号) =
直播状态通知(可选微信|钉钉|tg或者都填) = tg
tgapi令牌 = 6XXXXXXXXX:AAE9XXXXXXxxxxxXXXXXXXXXxxxxXXXXX
tg聊天id(个人或者群组id) = XXXXXXXXX

It may be the first, To be precise, it is to send notifications when a live broadcast is detected to be in progress. where as long as the recording software detects that the live broadcast is in progress, a notification will be sent when the recording starts. Please pay attention to setting your detection frequency (time interval)
直播状态通知(可选微信|钉钉|tg或者都填) = tg  => 直播状态通知(可选微信|钉钉|tg或者都填) = TG
It should be  uppercase letter TG , and other configurations are correct.

Thank you so match! works with "TG"

No tkinks. In fact, this was my mistake. I mistakenly wrote 直播状态通知(可选微信|钉钉|TG或者都填) = as 直播状态通知(可选微信|钉钉|tg或者都填) = , it causes users to fill in errors. But it has been fixed in the latest version, maybe you could update to Releases v2.0.9

I updated the application version, but left the old config. Everything is clearly described in the new config file, thank you


Would it be possible to get an example of a simple write() command?

Hi, I can't get my head around this, and cant find any good tutorials either.
I build and run the example app, and it finds my Arduino that's connected.
But how can I send a simple text over serial to it, using a serial.write() or something similar?
Would it be possible to add a very basic example for that?

The UsbSerialPort class is the abstract class that each driver provides an implementation of. It has a Write method, which takes two parameters, an array defines as byte[] that is the data to send, and a timeout defined in ms. You would just call the Write method of the port from the class that you are using to connect to your Arduino.
You need to convert your string to a byte array.  If you are using ASCII, use something like
byte[] bytes = Encoding.ASCII.GetBytes(MyString);

The Write method returns the number of bytes written. You would need to check that to make sure that all of the bytes had been written and if necessary, send the remaining bytes..

Thanks, how could I bypass having to select a device, there will only be 1 device connected, and I want it to write "12345" to that device whenever an event happens (eg. pressing a button)


pwntools (and probably others) require apt-get install python-dev

maybe we should mention this in the README?

I added this to the setup. Thanks!


[collab-galley] Merge a few critical PRs from master to collab-galley

This is a straight-up cherry-pick merge into collab-galley branch.
Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[X] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

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

Needs approval from an approver in each of these files:

galley/OWNERS [ozevren]
pkg/OWNERS [ozevren]

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

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
21ed6425e1f8eea6311033101d4eca774ce3a61b
link
/test integ-security-k8s-presubmit-tests

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

/approve
/lgtm

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

Needs approval from an approver in each of these files:

galley/OWNERS [geeknoid,ozevren]
pkg/OWNERS [geeknoid,ozevren]

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


drum: ANSI input sequences passed through

In :dojo, type an opening quote (so that syntax errors aren't deleted). Then, hit shift+left-arrow:
~master-morzod:dojo> ';2D

or shift+right-arrow:
~master-morzod:dojo> ';2C

These are the only ones I've seen so far ...

This may well be an issue with your terminal settings -  it's certainly at least platform-dependent

Are the unshifted versions similarly affected? I'm guessing there's a whitelist of recognized sequences, and some of the unbound combinations aren't on it(PgUp and PgDn might behave similarly, for example).

Definitely replicated on the mac.  I'm sure it's platform dependent!
On Wed, Aug 31, 2016 at 8:47 AM, Anton Dyudin<EMAIL_ADDRESS>wrote:

Are the unshifted versions similarly affected? I'm guessing there's a
whitelist of recognized sequences, and some of the unbound combinations
aren't on it(PgUp and PgDn might behave similarly, for example).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/issues/254#issuecomment-243807919, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AALyARQii3I4W4SwqFoi5VgpS7B_JB9cks5qlaIQgaJpZM4JumVE
.

I don't know why I opened this here; the problem is definitely somewhere in term.c ... most likely in u3_term_io_init() or _term_io_suck_char()

This issue was moved to urbit/urbit#811


initiating from delegate callback

hello,
initiating from the delegate callback displays nothing, could you take a look into that ?
thank you

sorry, i do not know what problem you have meet.
you can use it according to the demo code, if still not work, please let me know.

I'm rewriting code to separate the download and cache image's code into a separate class, you see it soon.

hello,
in my app there is a protocol
@protocol HttpClientDelegate

(void)InfoGetSucceeded:(NSDictionary *)data;
@end

(void)InfoGetSucceeded:(NSDictionary *)data
{
settting the images array for SRInfiniteCarouselView does nothing, it doesnt display or load any images
}

but calling it in the viewDidLoad() directly works perfectly,,
this is the issue im currenly facing
thank you

im getting nothing even after i add your code, look at the screenshot

Sorry i was busy with something, Have your problem solved?

np,
nope still


ci: push image & deploy at CSC

Update of the GitHub Actions workflow:

Build and Push to Docker Hub
Deploy to OpenShift

This workflow is triggered when a pull_request on the branch dev is done.

@git-anurag-hub: If this looks good to you, you can go ahead and merge. Thanks again @Socuz!


Set and Get a file name internal gzip

Steps to reproduce

I need to compress a file in a Gzip, but when I rename the gzip file, it is also renamed the internal file.

Expected behavior
The internal gzip file is always renamed
Actual behavior
Modify the head of the gzip to enter the real name of the internal file
Version of SharpZipLib
Obtained from (place an x between the brackets for all that apply)

[X] Compiled from source
branch: Master
commit:  be53259
[X] Downloaded DLL from GitHub
[ ] Downloaded DLL from SourceForge
[ ] Downloaded DLL from _______
[ ] DLL included as part of
Package installed using:
[ ] NuGet
[ ] MyGet
[ ] Chocolatey

#175
Change this file ICSharpCode.SharpZipLib.GZip.GZipOutputStream.cs
Create a two property
`///
/// Get the current compression level.
///
/// The current compression level.
public int GetLevel()
{
return deflater_.GetLevel();
}
	/// <summary>
	/// Set a file name internal gzip
	/// </summary>
	/// <param name="filename">name of file zipped</param>
    public void SetFileName(string filename)
    {
        _fileName = filename;
    }`

On "Support Routines" modify the method  WriteHeader().
After the code baseOutputStream_.Write(gzipHeader, 0, gzipHeader.Length);
Inlude a code
if (!string.IsNullOrEmpty(_fileName))  			    {  			        var fileNameBytes = System.Text.Encoding.GetEncoding("ISO-8859-1").GetBytes(_fileName);  			        Array.Resize<byte>(ref fileNameBytes, fileNameBytes.Length + 1);  			        fileNameBytes[fileNameBytes.Length - 1] = 0;  			        baseOutputStream_.Write(fileNameBytes, 0, fileNameBytes.Length);  			    }

I've updated the question and the proposed solution to what I understand is the issue, please correct me if I misunderstood.
A filename can be added to the gzip header if the flag FNAME is set.
Reference: https://tools.ietf.org/html/rfc1952#section-2.3.1
This could be made into a PR.


[Feature request] Close other tabs, Close tabs on the left, on the right, close all tabs

It would be helpfull to have on the ui, the ability to Close other tabs, Close tabs on the left, on the right, close all tabs as it is in Chrome or IntelliJ.
:)

Related: don't automatically open a new tab for every request. Reuse the same tab by default, I can open tabs myself if I want to. When working with many different requests in one collection I end up with many many open tabs.


Explicit control of "format" model property

Per discussion in https://github.com/swagger-api/swagger-core/issues/517 the "format" value for model properties can be inferred for a few common types. Currently it covers types like DateTime, UUID, etc.
At the same time, Swagger 2.0 allows format values which are not defined in the specification. This is nice because API developers can use values defined in new authoritative sources such as RFCs.
swagger-core can continue to infer the format values for common types however it will always lag behind new authoritative sources. In these cases, the API developer should be able to explicitly set the format value on model properties.
Whether this is supported via the @ApiModelProperty annotation or some other mechanism is less important as long as it's possible. It would be nice if the solution could be used easily with Scala case classes.

This is related to #993.

Yes, as it stands, the PropertyBuilder is not forgiving when there isn't a matching format.  We should have a fallback to string with custom formats if not matched with string + any of the known formats.

We have created fallbacks in PropertyBuilder so the type can have an implementation when format is not matched.


Unable to exchange token with APPLE's Id token

String requestBody = "grant_type=urn:ietf:params:oauth:grant-type:token-exchange"  +
        "&client_id=" + clientId +
        "&subject_issuer=" +  provider+
        "&subject_token=" +  idToken +
        "&subject_token_type=urn:ietf:params:oauth:token-type:id_token";

Error:
Response:
"body":"400 Bad Request: \"{\"error\":\"invalid_token\",\"error_description\":\"token type not supported\"}\""}}
 [error] - ERROR 400

Token exchange is working with google's access token
It happened after upgrade to 24.0.4 from 22.0.4

Hi @asim46,
thx for creating this ticket.
What's the URL you're sending this request to?
Does Keycloak log something that is related to this error?

Thank you so much for your fast reply !
Url is: <keycloak_url>/realms/<realm_name>/protocol/openid-connect/token
Sorry i wrote admin api earlier! I am sending POST request from my backend service to this url..
I have even recreated the keys and tried but no luck !

Ok. thx for the insights.
I'll give it a test and debug it if necessary.

I managed to find the issue. Its in the OIDCIdentityProvider.isTokenTypeSupported. However, it turns out that this is a bit more difficult to solve than I initially thought.
I try to replace the base of AppleIdentityProvider from OIDCIdentityProvider to AbstractOAuth2IdentityProvider. This could help. We'll see

I just created a new validation method and therefore avoid calling the inherited validation method with its restrictions that were introduced in v24.0.3.
@asim46 you have two options:

Installing 1.12.0 of this extension
or downgrading to Keycloak v24.0.2


Failed to start ndp-proxy-helper.service

Debian 8 / Vultr VPS with IPv6 disabled
Using the latest revision I'm getting the following errors:
add tunnel "sit0" failed: No buffer space available
Failed to bring up he-ipv6.
testing IPv6 tunnel interface, this could take a while...
However, the script continues until it terminates with this error:
Status: Downloaded newer image for ab77/sniproxy:latest
Creating sniproxy
Creating caching-resolver
/sbin/init: unrecognized option '--version'
Created symlink from /etc/systemd/system/multi-user.target.wants/netflix-proxy-admin.service to /lib/systemd/system/netflix-proxy-admin.service.
Created symlink from /etc/systemd/system/multi-user.target.wants/ndp-proxy-helper.service to /lib/systemd/system/ndp-proxy-helper.service.
Created symlink from /etc/systemd/system/multi-user.target.wants/systemd-networkd.service to /lib/systemd/system/systemd-networkd.service.
Created symlink from /etc/systemd/system/network-online.target.wants/systemd-networkd-wait-online.service to /lib/systemd/system/systemd-networkd-wait-online.service.
Failed to start ndp-proxy-helper.service: Unit docker-sniproxy.service failed to load: No such file or directory.
The problem seems to have arisen in the last couple of days worth of commits.

Cheers, found the problem. Should be OK now, please give it anther go and let me know how it goes..

The latest commit solves the issue. Thank you.


Bucketize resolution names in the UI

Have you read the FAQ and checked for duplicate open issues?
Yes
Is your feature request related to a problem? Please describe.
The resolutions menu in the UI shows exact heights in pixels, rather than using well-known names for resolutions like 1080p and 4k.
Describe the solution you'd like
The exact width & height should be used to choose a specific name for the resolution.  The mapping could be configurable if needed.
Describe alternatives you've considered
Continue using height + 'p'?  Let apps override the naming entirely?
Additional context
Some of this could be done in a custom overflow menu item, but that's not very easy at the moment.

Seeing this, I tried the Angel One demo on https://shaka-player-demo.appspot.com and it does list common resolutions as 360p, 480p. However it duplicates them.
If you are working on that menu, it should remove duplicates. Perhaps it is parsing and showing both audio and video, but I'd think it should only list video ones?

Seeing this, I tried the Angel One demo on https://shaka-player-demo.appspot.com and it does list common resolutions as 360p, 480p. However it duplicates them.

#2085.

Ah, the duplication has already been logged as a separate issue. Nice. Ignore this then

No worries, @gregmsanderson.  To see this issue in action, try playing something that doesn't have a 4:3 aspect ratio, like Sintel (which is 64:27).

This was the duplicate of #2085 that has been fixed.

I don't think that's what @joeyparrish meant for this issue.  I think this issue was about taking videos that have similar resolutions and put them together.  Instead of using the height followed by the letter 'p', we should provide pre-defined buckets to put the videos into (that is configurable).  This will provide more "common" values for videos with slightly different sizes (e.g. we may not want to show an entry for 1081p).

Ah, yes, you're probably right.

I'd actually like to see the complete list of resolutions along with their exact width/height values. Maybe shortened for common stuff (ie. 720p, 1080p.)
Also, in "auto" mode, it would be nice to have an indication on the demo app which resolution is in use right now.


Fixed api get for specific asset

The initial demo has a bug in /get/id route. might fix it to ease onboarding for others

Awesome, thanks!


Update playbook actions limit

Purpose of this pull request
This pull request updates the playbook actions limit from 50 to 200 in the following article and in a few other places:
https://help.sumologic.com/docs/platform-services/automation-service/about-automation-service/#actions-limit

Select the type of change:

[x] Minor Changes - Typos, formatting, slight revisions, .clabot
[ ] Update Content - Revisions, updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

PLEASE READ
Following a recent back-end update, all contributors with a local clone or fork of our repository are required to run yarn install. This does not apply to direct page edits.

[x] Yes, I've run yarn install
[ ] No, does not apply to me

Thanks, @kimsauce!


Discussion about sCU()=false problem

See #411

Also from a timeline perspective I am not sure whether this intermediate step adds value, as we can just move to the new React context api in a next major, and simply support changing stores without any further problems.

At now peerDependencies is "react": "^0.13.0 || ^0.14.0 || ^15.0.0 || ^16.0.0"
If only the new context is to be maintained, then users will have to forcefully update.
Alternatively, you can support both context API, and if used the old (and save the sCU problem), show a warning, and the recommend to update the react

I have some doubts about this PR, as, if I interpret the change correctly, with the above change it is still not possible to inject new things in a component if there is a false returning `sCU in one of it's parents. (One could argue that the warning should probably be an error)

In my PR
<Provider> -> <sCU()=false> -> <Injector> - worked correctly, and Injector updated
<sCU()=false> -> <Provider> -> <Injector> - not working, because <Provider>(and his sibling) not upate. But this is normal and expected behavior
You can test and compare this behavior with the new context API - there is the same situation
Also, as in my PR, if the Provider is in a bad sCU()=false, then the Consumer will not be updated, because the Provider itself has not updated
https://codesandbox.io/s/github/Gvozd/research-react-context
PS As it turned out, in my PR the problem of sCU is solved already in a known way
https://github.com/reactjs/rfcs/blob/master/text/0002-new-version-of-context.md#shifts-complexity-to-user-space

@mweststrate, any news about this?

Eh..., what is the question part in the above enumeration :)?

I ask you to approve my PR #411
For this I opened this issue to discuss the changes as you requested

The implementation seems correct, but I think there is at this point no need to further push this on, as there is a viable alternative to achieve this: React.createContext (or injecting mutable stores). For simplicity purposes I think it is clearer to keep Provider / inject to what it was: a light-weight dependency injection mechanism, not a mechanism to propage state changes.
If this turns out an omission that many run into, we can definitely reconsider.


Translation of climate states to Google Assistant Traits incorrect and incomplete

In me looking into the code (for as far as I understand Python and the buildup of Home-Assistant),
I noticed the 'TemperatureSettingTrait' class in trait.py from the google_assistant component.
The class only translates 4 states
# We do not support "on" as we are unable to know how to restore
# the last mode.
hass_to_google = {
    climate.STATE_HEAT: 'heat',
    climate.STATE_COOL: 'cool',
    climate.STATE_OFF: 'off',
    climate.STATE_AUTO: 'heatcool',
}

I am not sure whether other states are returned to google (i guess It does, as multiple issues exists about climate components not being able to be used with Google Assistant), but it might be necessary to limit what can be returned to only the states google will understand.
Also the hass state STATE_AUTO should (in my opinion) be mapped to 'auto' and not to 'heatcool' as what is done currently.
Next to this, the eco and dry states should be added into the translation.
Could this be fixed and extended in the Google Assistant component?
Originally posted by @reharmsen in https://github.com/home-assistant/home-assistant/issues/18515#issuecomment-441107126

Also see the following issues:

https://github.com/home-assistant/home-assistant/issues/17333
https://github.com/home-assistant/home-assistant/issues/17742
https://github.com/home-assistant/home-assistant/issues/18515
https://github.com/home-assistant/home-assistant/issues/18516
https://github.com/herikw/home-assistant-custom-components/issues/5

Mode mapping should only be done for what is available in the base climate.__init__() file. Anything extra could cause issues with the integration.
Google's supported modes are here:
https://developers.google.com/actions/smarthome/traits/temperaturesetting
There are a few extra modes that aren't currently covered in climate.__init__() that could be mapped.

Updated mappings. Note heatcool changed to auto.
    hass_to_google = {
        climate.STATE_HEAT: 'heat',
        climate.STATE_COOL: 'cool',
        climate.STATE_OFF: 'off',
        climate.STATE_AUTO: 'auto',
        climate.STATE_FAN_ONLY: 'fan-only',
        climate.STATE_DRY: 'dry',
        climate.STATE_ECO: 'eco'
    }

good catch. I ran the tests locally and that didn't seem to get touched. I just ran a search and it doesn't look like that variable is used at all. I might just remove it.


Linux based application needed

It is a opensource application. so, please make the application for linux-based platform. Linux is also a open source project, so, if you do this it is helpful for linux user.

To make it linux based I need to rebuild the whole application. I am not so familiar with linux application development. Currently I am learning about it. I think it will take me some time to make the run on linux. If anybody wanted to help, it would be great.

@dipu-bd Have you finalized the way you want do things on linux, or do contributors have free reign to the implementation?


Support member_role_id for GroupSAMLLink apis

currently the go library is missing the ability to set the member_role_id during AddGroupSAMLLink as documented here:
https://docs.gitlab.com/ee/api/groups.html#add-saml-group-link

Thanks @djrichar, I came here to do the same thing! @svanharmelen could we get a new release of this when possible please?

Oops I see it was in the latest tag, thank you


Allow local Album art

Would it be possible to pull the album art directly from MPRIS?
Some songs are named slightly different than what would be on lastfm so nothing will show.
Thank you for your time.

Hello.
It is not possible to display local images, only external URLs. This is specified in the Discord documentation here:
https://discord.com/developers/docs/rich-presence/overview#assets
Theoretically I could add a backup album image from MusicBrainz but usually if there is no cover art on Last.fm then most likely there won't be on MusicBrainz either because as I know Last.fm pulls album data from MusicBrainz.
So the only option is to either change the album name to match the name on Last.fm, or add the missing album covers on Last.fm yourself. When I see that an album I'm listening to doesn't have a cover, I usually add it by myself.

You know, that makes the way you did it make way more sense now. Lol
Thank you for answering my question.


Box Selection

Is your feature request related to a problem? Please describe.
I want to select multiple pieces of the same type of geometry at the same time
Describe the solution you'd like
Click and drag a box to select multiple elements
Additional context
It would be nice to have an x-ray mode toggle to select backfaces and such, but we'll defer on this for now.

[x] Add box dragging to selector
[x] Select Vertices
[x] Select Faces
[x] Select Edges
[ ] Select only visible geometry (e.g. clip on faces)

Snapping has an implementation here https://github.com/jgillich/godot-snappy

@fire An implementation of what?

Ah yes, visual server could work, but verts don't render, however that's the simplest thing to implement.

the godot source also has an implementation :)

argh, renderingserver doesn't work unless each piece of geometry is its own visual instance. currently we have exactly three: wireframe, verts and selected faces


Docs: Add unnest to SQL Reference

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@gloomweaver please run the prettier locally, it will reformat the MD files a little bit

  # if you encounter error, rerun the command below and commit the changes
  #
  # ignore subproject CHANGELOG.md because they are machine generated
  npx<EMAIL_ADDRESS>--write \
    '{datafusion,datafusion-cli,datafusion-examples,dev,docs}/**/*.md' \
    '!datafusion/CHANGELOG.md' \
    README.md \
    CONTRIBUTING.md

Thanks again @gloomweaver  and @comphead


update the business details section in tombstone

*Issue:*https://github.com/bcgov/entity/issues/23859
Description of changes:
Update the Business Details section in the dashboard tombstone

Display 'Registration Date' for SP and GP
Add 'Change' buttons to Email and Phone
refine the order of the info list
-- SP/GP:  Registration Date -> Registration Number -> Business Number -> Email -> Phone
-- Other:  Business Number -> Incorporation Number -> Email -> Phone
Add i18n text for legal type 'C'

Note: 'Change' buttons redirect to https://dev.account.bcregistry.gov.bc.ca/businessprofile  but the page cannot be loaded (possibly) due to missing sessionStorage values
*Issue:*https://github.com/bcgov/entity/issues/24290
Description of changes:

Add i18n text for legal type 'C'

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

/gcbrun

Temporary Url for review: https://business-dashboard-dev--pr-64-yl9g2yfl.web.app

/gcbrun

Temporary Url for review: https://business-dashboard-dev--pr-64-yl9g2yfl.web.app


Make leap_day=True default for PSM3

[x] Closes #1481
[x] I am familiar with the contributing guidelines
[ ] Tests added
[x] Updates entries in docs/sphinx/source/reference for API changes.
[ ] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[ ] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[ ] Pull request is nearly complete and ready for detailed review.
[x] Maintainer: Appropriate GitHub Labels (including remote-data) and Milestone are assigned to the Pull Request and linked Issue.

This PR should first be merged for pvlib 0.11.0!
This PR changes the default value of the leap_day parameter in pvlib.iotools.get_psm3 from False to True and removes the deprecation warning added in #1511. The reason for the change is that it seems more intuitive to include leap days when requesting non-TMY time series, which is also the default behavior of the remaining iotools functions.
Duplicate of #1500

I made a mess...


xr0.16.1: TypeError: '<' not supported between instances of 'NoneType' and 'str'

xr0.16.0: no errors
xr.0.16.1: TypeError
example failed test:
==============================================================================
ERROR: xskillscore/tests/test_deterministic.py::test_distance_metrics_xr_dask[False-False-False-dim4-metrics5]
------------------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/aaron.spring/Coding/xskillscore/xskillscore/tests/test_deterministic.py", line 237, in test_distance_metrics_xr_dask
    actual = metric(a, b, dim, weights=_weights, skipna=skipna)
  File "/Users/aaron.spring/Coding/xskillscore/xskillscore/core/deterministic.py", line 1118, in smape
    keep_attrs=keep_attrs,
  File "/Users/aaron.spring/anaconda3/envs/xskillscore-dev/lib/python3.6/site-packages/xarray/core/computation.py", line 1110, in apply_ufunc
    keep_attrs=keep_attrs,
  File "/Users/aaron.spring/anaconda3/envs/xskillscore-dev/lib/python3.6/site-packages/xarray/core/computation.py", line 262, in apply_dataarray_vfunc
    result_var = func(*data_vars)
  File "/Users/aaron.spring/anaconda3/envs/xskillscore-dev/lib/python3.6/site-packages/xarray/core/computation.py", line 700, in apply_variable_ufunc
    result_data = func(*input_data)
  File "/Users/aaron.spring/anaconda3/envs/xskillscore-dev/lib/python3.6/site-packages/xarray/core/computation.py", line 670, in func
    signature.to_gufunc_string(exclude_dims),
  File "/Users/aaron.spring/anaconda3/envs/xskillscore-dev/lib/python3.6/site-packages/xarray/core/computation.py", line 143, in to_gufunc_string
    for core_dims in self.input_core_dims
  File "/Users/aaron.spring/anaconda3/envs/xskillscore-dev/lib/python3.6/site-packages/xarray/core/computation.py", line 143, in <listcomp>
    for core_dims in self.input_core_dims
  File "/Users/aaron.spring/anaconda3/envs/xskillscore-dev/lib/python3.6/site-packages/xarray/core/computation.py", line 142, in <listcomp>
    [self.dims_map[dim] for dim in core_dims]
  File "/Users/aaron.spring/anaconda3/envs/xskillscore-dev/lib/python3.6/site-packages/xarray/core/computation.py", line 98, in dims_map
    core_dim: f"dim{n}" for n, core_dim in enumerate(sorted(self.all_core_dims))
TypeError: '<' not supported between instances of 'NoneType' and 'str'

------------------------------------------------------------------------------
Ran 1658 tests in 37.52s

FAILED (errors=125)

produces the same traceback:
ds=xr.tutorial.load_dataset('rasm').chunk() # works without chunking
a=ds.isel(time=slice(None,10))
b=ds.isel(time=slice(10,20))
a['time']=b.time
r=mse(a,b,dim='time')

but its an xskillscore issue because
def mse2(a,b,dim):
        return ((a-b)**2).mean(dim=dim)

works
no idea what the problem is

Hmm. @ahuang11 did you come across this at some point?
@aaronspring worth asking on the xarray gitter?

they changed a few things in apply_ufunc. not worth asking because they wont know how we use xr.apply_ufunc

has to do with this PR: https://github.com/pydata/xarray/pull/4130/files

See a couple of things in their change log
Support multiple outputs in xarray.apply_ufunc() when using dask='parallelized'. (GH1815, PR4060). By Kai Mühlbauer.
Fix xarray.apply_ufunc() with vectorize=True and exclude_dims (GH3890). By Mathias Hauser.
Use dask.array.apply_gufunc() instead of dask.array.blockwise() in xarray.apply_ufunc() when using dask='parallelized'. (PR4060, PR4391, PR4392)

I think I found it. weights input_core_dim shouldnt be [None] but []


Replace legacy build pipeline with PowerShell SDP build pipeline

https://dev.azure.com/ceapex/Engineering/_workitems/edit/248668/

Docs Build status updates of commit 5469694:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-MAML2Yaml_Uid_Duplicated] Duplicate uid found: W:/ybnv-s/docset/winserver2012r2-ps/hpc/Get-HpcMetricValue.md and W:/ybnv-s/docset/winserver2012r2-ps/hpc/Get-HpcMetric.md has the same uid: HPCPack2012R2.Get-HpcMetric.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Open-Elevation (Independent Publisher)

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[x] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[ ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[x] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[x] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[x] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

@vmanoharas

@vmanoharas  see PR from 2 months ago

@vmanoharas I have left comments regarding your notes, and would require feedback before performing any change - if needed. Could you please review

@vmanoharas I have left comments regarding your notes, and would require feedback before performing any change - if needed. Could you please review

Hi @fordosa90,
I am not able to find your comments, could you please add, if you don't add comments.

@vmanoharas as stated in above paragraph, the service is public, hence there is no credential as far as I know, there is no definitive guide for these readme.md files but I see other ones in the repository where the service is public and not having a special section in the markdown to repeat, that there are no credentials. do you require to have a specific paragraph to state in addition to the "public" notion the no need for credentials?
the "Google Elevation API" is a trademarked product and service name, over which I don't have ownership. Removing the API would affect brand. Do you want to proceed?

Hello @fordosa90,
Thank you for providing the details.
Regarding, please add obtaining credential section and mention it as NA.
Regarding API keyword, please remove this word, they may affect for automation tools.

@vmanoharas updates pushed

[[certify-connector]]

Hello @fordosa90 ,
I hope you are doing well.
Congratulations, the deployment of your connector and documentation page was already completed, We have a short survey on the connector certification process that we would like for you to complete. Your feedback is appreciated and will help us improve the program.
Thank you very much for working with us.


[ZEPPELIN-5185]. HiveMonitor thread is never finished sometimes

What is this PR for?
Sometimes, hive monitor thread will never exit when getting the following exception (see screenshot)

This PR would exit the monitor thread when any exception happens and also introduce property zeppelin.jdbc.hive.monitor.query_interval to control the query interval.
What type of PR is it?
[Bug Fix]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-5185

How should this be tested?

CI pass

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

LGTM

LGTM


Add CxxVector::new for creating an empty vector

When constructing a CxxVector to pass to C++, you have to end up writing your own utility function that constructs a UniquePtr<CxxVector<...>> and add that to your projects cxx bridge. Creating an empty vector seems like a simple enough task to afford having an API for it on CxxVector.

Published in cxx 1.0.107.


allow stage and s param to pass through

This is to allow the --stage and -s to pass through

Additionally, you'll need to declare the option in those commands here:
https://github.com/Nordstrom/serverless-artillery/blob/master/bin/serverless-artillery
This is the PR that added region and does everything that you will need to do:
https://github.com/Nordstrom/serverless-artillery/commit/3e3b72a06211af947e7b28b37e568d0a31758eaa

@erikerikson , invoke already has -s for script. Do u want change to something else like -sc?

Hmm.  Great question.  Sorry to create that situation.  No great option here, I'll ponder and get back..

@erikerikson Agree there are no good backward-compatible options here. We could include this change if we rev the major version and map the -s (script) argument to something else. Did you or Danno have any other breaking changes you'd like to make?
Otherwise, given that, in this case, there is a workaround (set STAGE environment variable for the scope of that command) it doesn't justify a breaking change at this time.

I believe the long term answer is that we will be eventually passing all arguments through.  the framework uses the flag -p for specifying the "path" to the file:
--path or -p The path to a json file with input data to be passed to the invoked function. This path is relative to the root directory of the service.
from: https://serverless.com/framework/docs/providers/aws/cli-reference/invoke/
As a result, -p is probably a more appropriate flag for specifying the artillery script over the long term.  @gwsii You're right to call out the breaking nature of this change on existing users.
@Nedvedyy did the environment variable approach not solve this for you?

@erikerikson yes -p sounds good. Env variable is working good.

@Nedvedyy if you're open to it, I've just opened #80, and could use eyes on it.  If not the code, a moment considering the approach and user experience.  It is my attempt to address this issue and others before it (e.g. #63).

@Nedvedyy any chance to test this?  We've merged to master but are holding off publishing for the moment.

Closing as pass-through of these arguments to SLS is released as part of v0.3.0 which is now live on npmjs.
Thanks for this contribution!  Also, sorry for the complexity demands and legacy of our choices.

@erikerikson great. Thanks


Only highlight vim-plug buffers if syntax highlighting is enabled

Previously, syntax highlighting was turned on unconditionally in the buffers opened by commands such as :PlugUpdate. With this commit, highlighting is not applied if the user has disabled syntax highlighting
by calling syntax off after call plug#end() in their vimrc.

Looks good to me. Thanks.