Padding on FE Search

Describe the bug
Searching for something in the FE searchbar messes up some padding
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Closed as duplicate of #168.


Also support longitudes in [0, 360] degree range in json file?

The json file now expects longitude between -180 and 180.
https://github.com/OceanParcels/Virtual_ship_classroom/blob/b9a1dbffb1e0389bf02e14babfb732d677efa157/virtualship.py#L27
However, it is (also) quite common to refer to longitudes between 0 and 360. Perhaps support that option too?

Sure. Would you then have users specify that they give input between 0 and 360 with a true/false statement? Otherwise how would you know? For sure if one of the values is above 180, but that doesn't catch all cases.
Solve with if... and ((lon - 180) % 360) - 180

Well, any value above 180 can also be interpreted as between -180 and 0; so I would indeed simply use lon = ((lon - 180) % 360) - 180. Or would that give other problems?
Note that parcels deals with this under-the-hood, so you could also choose to accept all longitudes, even ones higher than 360 (I think)


Request for Multi Diff Editor Open Icon / Open changes button doesn't fit with other icons

The icon here doesn't look like it fits, particularly because of the thick stroke width of the lines:

This might just be a codicon request 🙂
cc @hediet

The icon for this is definitely not final. @hediet is currently using the magnifying glass, but I'm not a fan of that because it is used so prominently for search.

@hbons I think we need a new icon here.

how about something like this? I think GitHub also uses this.

thoughts?

Unsure, it doesn't really convey that this is about many files/editors

It is indeed what GH uses so it's definitely better than the current:

How about basing it off files?

It looks very similar to some icons we already have, and which already show in the same place in GHPRI that the "open multi-diff editor" action is.

@alexr00 yes, it's the same one. :) GitHub also uses these for difference cases (diffs + changes requested) so I think we're fine doing the same.

👍 sounds good. In GHPRI these icons will show right next to each other, so I like the diff-multiple icon to disambiguate.

👍 for all and specifically for diff-single for inline chat's new toggle changes feature

PR for the icons here: https://github.com/microsoft/vscode-codicons/pull/211

@hbons shouldn't we offset it up and to the right like files since that's an established way to show the concept of "multiple" already? There are also the breakpoints and collapse all which use similar diagonal offsets.

@Tyriar yes, but there's no space for that in this case...

Adopted in GHPR. Here's how it looks:

And for inline chat with live3

Also adopted for all the other multi diff commands.

Go team! :D


Adding Feedly account always launches Safari for sign-in, no matter which browser is set.

When adding a Feedly account for feed sync on MacOS, the app launches Safari for signing in to Feedly, no matter which browser has been set under Settings -> General -> Browser.

When adding any account it does so through safari. No matter the default browser.

it doesn't open any browser when trying to add on mine. tried changing default browser also

This sign-on flow is using an Apple-provided API, which always uses Safari.
There is a known bug, though, where Safari might not appear. In that case, rebooting your Mac should fix it. (It’s an Apple bug, to be clear.)

It worked after restarting the application


Pair of nodes in the BST which sum to S

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Find pair of nodes in the BST which sum to S. BST and an integer(sum) will be given as input.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#4679

Hi @AnushkaS09
Please make an issue here and raise a PR, for an article as part of our Docs repository: https://github.com/TesseractCoding/NeoAlgo-Docs
It would be much appreciated. Thanks!

@all-contributors bot please add @AnushkaS09 for code


Error 500 on production mode via /sitemap.xml

Describe the bug
[nuxt] [request error] [unhandled] [500] fetch failed (http://localhost/sitemap/routes.json)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async $fetchRaw2 (./.output/server/node_modules/ofetch/dist/shared/ofetch.d438bb6f.mjs:215:14)
at async $fetch2 (./.output/server/node_modules/ofetch/dist/shared/ofetch.d438bb6f.mjs:239:15)
at async Promise.all (index 1)
at async resolveAsyncSitemapData (./.output/server/chunks/nitro/node-server.mjs:2324:3)
at async buildSitemap (./.output/server/chunks/nitro/node-server.mjs:2488:22)
at async ./.output/server/chunks/nitro/node-server.mjs:2622:15
at async Object.handler (./.output/server/node_modules/h3/dist/index.mjs:1284:19)
at async Server.toNodeHandle (./.output/server/node_modules/h3/dist/index.mjs:1359:7)
[Vue Router warn]: No match found for location with path "/sitemap.xml"
Reproduction
No response
System / Nuxt Info
- Operating System: Linux
- Node Version:     v20.3.1
- Nuxt Version:     3.6.2
- Nitro Version:    2.5.2
- Package Manager<EMAIL_ADDRESS>- Builder:          vite
- User Config:      telemetry, srcDir, app, modules, runtimeConfig, vite, css, postcss, colorMode, device, image, content, pwa, robots, site, sitemap, schemaOrg
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -

Running in docker.

Seems like it's not docker problem, I expose 3000 port properly.
I can access my api sitemap generation:

But on sitemap.xml still have 500 error.

So, I tried to reproduce this bug on playground in this repo, and what I got:
[nuxt] [request error] [unhandled] [500] Cannot read properties of undefined (reading 'sitemapName')
  at ./.playground/.output/server/chunks/nitro/node-server.mjs:1394:45
  at async Promise.all (index 1)
  at async resolveAsyncSitemapData (./.playground/.output/server/chunks/nitro/node-server.mjs:1435:3)
  at async buildSitemap (./.playground/.output/server/chunks/nitro/node-server.mjs:1601:22)
  at async ./.playground/.output/server/chunks/nitro/node-server.mjs:1775:15
  at async Object.handler (./.playground/.output/server/node_modules/h3/dist/index.mjs:1284:19)
  at async Server.toNodeHandle (./.playground/.output/server/node_modules/h3/dist/index.mjs:1359:7)

With this settings:
sitemap: {
  dynamicUrlsApiEndpoint: '/api/_sitemap-urls',
},

Hey, sorry I have a fix hopefully coming for this, give me ~20 min

Please try 3.0.6 :pray:


Copying aws-exports to prevent broken tests in other CI

My project makes use of Amplify auth and analytics, and therefore has an aws-exports.js. I'm using GitHub actions to trigger tests on feature branches (to disable the merging of test-breaking code). The project uses the Amplify Console for CD. Before building the code, I execute the following script, which––if aws-exports.js exists––copies aws-exports.js to an aws-exports.ts. If aws-exports.js doesn't exist, the script writes export default {}; to aws-exports.ts. This way, we avoid a build error when running in GitHub's CI environment. This is also good for users who don't want to run amplify init to hack on this site locally.
import * as path from "path";

const clientDir = path.join(__dirname, "../client");
const awsExportsPathWithoutExtension = path.join(clientDir, "src/aws-exports");
const awsExportsJSPath = [awsExportsPathWithoutExtension, ".js"].join("");
const awsExportsTSPath = [awsExportsPathWithoutExtension, ".ts"].join("");
try {
  const awsExportsJSContents = fs.readFileSync(awsExportsJSPath, {
    encoding: "utf8",
  });
  fs.writeFileSync(awsExportsTSPath, awsExportsJSContents, {encoding: "utf8"});
  console.log("aws exports found and copied!");
} catch (e) {
  fs.writeFileSync(awsExportsTSPath, "export default {};", {encoding: "utf8"});
  console.log("aws exports not found and supplemented");
}

Locally, the above script works as expected both when the aws-exports.js is and isn't there.
Although aws-exports.js is "supposed" to be present in the Amplify Console container's file system, the read operation fails––we see "aws exports not found and supplemented" in the logs. This should only be the case in the GitHub test workflow. To my knowledge, nothing should prevent the retrieval of aws-exports.js contents. Any ideas on what's happening here?

@harrysolovay please share your buildspec as well (App settings > Build settings)

version: 0.1
frontend:
  phases:
    preBuild:
      commands:
        - yarn install
    build:
      commands:
        - yarn build
  artifacts:
    # IMPORTANT - Please verify your build output directory
    baseDirectory: /
    files:
      - "**/*"
  cache:
    paths:
      - node_modules/**/*

Other than not reading aws-exports.js, the site is built and deployed correctly.

Update your buildspec to add https://docs.aws.amazon.com/amplify/latest/userguide/build-settings.html#frontend-with-backend

@harrysolovay, did this work for you?

Yep! I added the following to my build spec:
backend:
  phases:
    build:
      commands:
        - amplifyPush --simple

Worked like a charm. Thanks!

Update your buildspec to add https://docs.aws.amazon.com/amplify/latest/userguide/build-settings.html#frontend-with-backend

Thanks, i had the same problem and this did it for me. I wonder why this step is never mentioned in the official docs https://docs.amplify.aws/

Same here @truefalse10 @harrysolovay - it feels as this is a critical component of cicd deploying and is brushed over. I'd suggest that it be in the build spec by default when detected to have an amplify backend.
version: 0.1
backend:
  phases:
    build:
      commands:
        - amplifyPush --simple
frontend:
  phases:
    preBuild:
      commands:
        - yarn install
    build:
      commands:
        - yarn build
  artifacts:
    # IMPORTANT - Please verify your build output directory
    baseDirectory: /
    files:
      - "**/*"
  cache:
    paths:
      - node_modules/**/*


Quantile LightGBM - inconsistent deciles

Hi, I'm doing Quantile LightGBM, defined as follows:
QuantileEstimator(lgb.LGBMRegressor(n_jobs=-1,
                             seed=1234,
                             learning_rate=0.1,
                             reg_sqrt=True,
                             objective = 'quantile',
                             n_estimators=100))

Where QuantileEstimator is just:
class QuantileEstimator(BaseEstimator, RegressorMixin):

    def __init__(self, model):
        """
        """
        self.alphas = [round(x, 1) for x in np.arange(0.1, 1, 0.1)]
        self.model_factory = []
        self.model = model
        super().__init__()

    def fit(self, X, y=None):

        for a in self.alphas:
            model_i = clone(self.model)
            model_i = model_i.set_params(**{'alpha': a})
            model_fitted = model_i.fit(X, y)
            self.model_factory.append(model_fitted)
        return self

    def predict(self, X):
        predictions = pd.DataFrame()
        for a, m in zip(self.alphas, self.model_factory):
            predictions["y_pred_" + str(a)] = m.predict(X)
        return predictions

Sometimes it happens that following decile for individual predictiions has lower value than the previous one.

Any idea why that's happening? I understand that those are different models, but still as I set the same seed I would expect that the results for different deciles should be monotonic.

I think it may be a bug, will investigate it when have time.

Thanks, please let me know if you would need some additional info. Maybe the distribution of underlying data could be a clue?

Thanks, please let me know if you would need some additional info. Maybe the distribution of undelying data (y) could be a clue?

gently ping @TomekPro for example of data

Hi, having the same problem but didn't think that it's a bug. When I see this topic i see its worth questioning that.
I calculate 10th,30th,50th,70th,90th quantiles for time series regression problem. On some points, p30-p50-p70 is mixing up. Until I see the suggestion of  @TomekPro  I thought it's due to randomity of the training process, because when I average ensemble the  same model with same features for 3-4 times quantiles were getting organised.
My data is not normally distributed, so it makes sense, yes.

Diving into theory in order to see if it's a bug, I have a feature that can separate this into two normal distribution(open to discussion, but it's more normally dist.) which is in most important 10 features. However, lgbm doesn't guarantee to use the feature at the top of the trees.

And I generally end up with quantiles such as this:

But in some cases I have that p50 is out of the range of p10-p90. Sometimes, p50 gives more accurate results but p10-p90 interval doesn't capture the real value at all.
Again, i'm not sure whether it's bug or not, or may be I'm modelling something wrong, or it's just a randomness since we don't fit an actual normal distribution with (mean,std) parameters (NGBoost does that) but we estimate it. Again, i'm noting that when I run the same quantile with same model/features and simply average the results, I have more consistent results.

I think with GBDT, quantile objective does not guarantee the prediction value of an instance (data point) to be monotone. Just as we cannot guarantee that if we increase the label of one instance in the training data, then the prediction of that instance will increase if we retrain the model. Because the data partition in the trees can change, and the leaf prediction value can be very different.
Here's an example where the predicted decile is not monotone.
import lightgbm as lgb
from sklearn.base import *
import pandas as pd
import numpy as np

class QuantileEstimator(BaseEstimator, RegressorMixin):

    def __init__(self, model):
        """
        """
        self.alphas = [round(x, 1) for x in np.arange(0.1, 1.0, 0.1)]
        self.model_factory = []
        self.model = model
        super().__init__()

    def fit(self, X, y=None):

        for a in self.alphas:
            model_i = clone(self.model)
            model_i = model_i.set_params(**{'alpha': a})
            model_fitted = model_i.fit(X, y)
            self.model_factory.append(model_fitted)
        return self

    def predict(self, X):
        predictions = pd.DataFrame()
        for a, m in zip(self.alphas, self.model_factory):
            predictions["y_pred_" + str(a)] = m.predict(X, raw_score=True)
        return predictions

qet = QuantileEstimator(lgb.LGBMRegressor(n_jobs=1,
                             seed=1234,
                             learning_rate=0.1,
                             reg_sqrt=True,
                             objective = 'quantile',
                             n_estimators=2,
                             min_data_in_leaf=1,
                             num_leaves=3,
                             boost_from_average=True,
                             verbose=2))

np.random.seed(2)
X = np.random.rand(10, 20)
y = np.random.rand(10)
qet.fit(X, y)
pred = qet.predict(X)
for i in range(pred.shape[0]):
    for j in range(pred.shape[1] - 1):
        if pred.iloc[i, j] > pred.iloc[i, j + 1]:
            print(pred.iloc[i, :])
            print(i)

And the output is
y_pred_0.1    0.504354
y_pred_0.2    0.660557
y_pred_0.3    0.729929
y_pred_0.4    0.725802
y_pred_0.5    0.742658
y_pred_0.6    0.802490
y_pred_0.7    0.891505
y_pred_0.8    0.929288
y_pred_0.9    0.962373
Name: 1, dtype: float64
1
y_pred_0.1    0.504354
y_pred_0.2    0.662241
y_pred_0.3    0.726090
y_pred_0.4    0.710099
y_pred_0.5    0.732608
y_pred_0.6    0.802490
y_pred_0.7    0.891505
y_pred_0.8    0.929288
y_pred_0.9    0.962373
Name: 5, dtype: float64
5

I've strictly follow the computation of GBDT with quantile objective and manually calculated the result. It is consistent with the tree output of LightGBM.
So in general, I don't think this is bug. It would be better if @TomekPro could provide the data, so that we can check the tree structures in the model. Your help is really appreciated.
Restricting the deciles to be monotone is an interesting question and can be take as a feature request.
If no further evidence shows that this is really a bug, I think this issue should be closed.

@shiyu1994 I'm sorry but I cannot provide the data as it is confidential but you can simulate it from the distribution I posted above. I suspect that this case is most common for such distributions with a large group of outliers.

Ok thanks for your information. I can try to build another example with labels similar to your distribution. But in general, the output for a data point with quantile objective is not guaranteed to be monotone with the alpha hyperparameter, as the example showed above.
Is the monotonicity essential for you application?

At the end I haven't used LGBM but in general I believe this is a really common case. You want not only have point estimate but as well some kind of confidence interval for your prediction.

I think to implement such a constraint requires to fitting models with different alpha together. We need to pass all the alphas to the boosting process, maintaining boosting models for different alphas simultaneously and setting appropriate restriction so that the prediction values for each data point grows monotonically with alpha.
This is a complicated task and requires new boosting algorithm design. We may investigate this in the future.
For now, let me put this into the feature request & voting hub.

Not sure how to vote for this feature request to be prioritised, but it obviously is critical in most applications that the quantiles be ranked correctly.

My 5 cents:
I would also not consider this as a bug.
What most people do not consider: If different quantiles clash/are not monotone, then the uncertainty of that prediction is veeeeery likely high. i.e. the crossing is withing the estimation uncertainty.
It is much easier to enforce such monotonicity constraints with linear models (as R's quantreg does). For tree based models, one would need the same tree split points to make such a constraint possible, very much like quantile regression forests do.

Hey, I also suffered from the problem of not maintaining monotonicity between quantiles, called the crossing problem.
To solve this, I suggested a method written in the issue. I hope you guys check it out and feel free to discuss it.


after import dubbo to my project ,and seperate project into multi modules ,each module takes even ten times memory than before( 引入dubbo后,每个单独的项目占用的内存都升高很多,并且其中一个不断升高)

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

Environment

Dubbo version: 2.7.7
Operating System version: windows 10
Java version: 1.8
registry: redis

Steps to reproduce this issue
并不清楚如何重现该问题,也不知道是否是普遍存在的问题。但是即便系统零负载内存仍然不断升高，直到fullGC
Expected Result
即便由于启动了一个netty服务,以及一些其他的东西,但是内存应该处于相对稳定的状态下，也不应该出现10倍以上的内存占用升高
Actual Result

there is a thread named "dubbo-future-timeout-thread-1" keep eating my memory

mark

I hava two modules A and B:

B module provides a service named UserRpcService implements org.springframework.security.core.userdetails.UserDetailsService to provide a UserDetailsService to spring-security framework,
A module reference this UserRpcService for login purpose,when invoke loadUserByUsername method ,the memory increase 10mb per seconds,and gc became more frequently than before.

before invoke loadUserByUsername

after invoke loadUserByUsername

https://github.com/apache/dubbo/issues/6820


Adding support for Python script visuals / Fixing logic for replacing source files in scripts

Making changes to enable support for Python custom visuals. Instead of using script provider name as extension, choose extension based on it. For R it worked because R is provider name and file extension is also .r, for Python it's .py

Fixing logic in source file replacement. Instead of creating promises and resolving in the end, wait for them inline. Issue with current approach is that promises don't get resolved immediately and we end up creating 100 promises (max import value) to replace same file. (Simple to verify with outputting count and file being replaced)

Need to add return content; into async function async content => {...

done


[Platform] M5Stack: No response to Sigma3, Failed to post event to CHIP Platform event queue

Reproduction steps

Commission M5.
Add another fabric over CASE of the first commissioned fabric
Open a new CASE session over the new fabric
Complete commissioning of the new fabric

Platform
esp32
Platform Version(s)
No response
Type
Manually tested with SDK
(Optional) If manually tested please explain why this is only manually tested
The steps above were exercised.
Anything else?
On M5 log, there were lots of Failed to post event to CHIP Platform event queue messages and one after receiving the Sigma3 message as well. So suspecting resource issue.
I (59199) chip[EM]: <<< [E:13002r S:0 M:38815149 (Ack:13084043)] (U) Msg TX to 0:0000000000000000 [0000] --- Type 0000:10 (SecureChannel:StandaloneAck)
I (59219) chip[IN]: (U) Sending msg 38815149 to IP address 'UDP:[FE80::18DD:660:1490:E11A%st1]:51277'
I (59219) chip[SC]: Received Sigma3 msg
E (59229) chip[SC]: The device does not support GetClock_RealTimeMS() API: 6c.  Falling back to Last Known Good UTC Time
E (60179) chip[DL]: Failed to post event to CHIP Platform event queue
E (60179) chip[CSL]: Failed to queue CHIP System Layer lambda event: 1000000
E (60199) chip[DL]: Failed to post event to CHIP Platform event queue
E (60199) chip[CSL]: Failed to queue CHIP System Layer lambda event: 1000000
...
I (75569) chip[FS]: Fail-safe timer expired
E (75569) chip[SVR]: Failsafe timer expired
E (75569) chip[SVR]: Commissioning failed (attempt 1): 32

M5_pairing_failed_accessory_log_21A220a.txt

@kpark-apple Which SHA is the M5 using?  Is it something after 77e39a62d88b539d6684ec504a8f6c52eb09604d?
This could also be similar to https://github.com/project-chip/connectedhomeip/issues/25833 (note the lower-memory non-default configuration file that was added in https://github.com/project-chip/connectedhomeip/pull/25935), and is likely the same as https://github.com/project-chip/connectedhomeip/issues/24418 and https://github.com/project-chip/connectedhomeip/issues/23258

The SHA used here was
0f1fc1b230eca4b4ce2f45c4a8cf1ce238284e96

OK, so that's after https://github.com/project-chip/connectedhomeip/commit/77e39a62d88b539d6684ec504a8f6c52eb09604d

The queue size has been increased to from 25 to 40: https://github.com/project-chip/connectedhomeip/blob/master/config/esp32/components/chip/Kconfig#L385


removing missing arguments

Proposed fixed for bug #11

Hi @kalaytan - I think the extra argument should be there for render_pdf as it's used in chartpy (I fixed a small bug though in chartpy, where it was not referenced in the abstract render_canvas method).

Hi @saeedamen,
Thanks, works fine now.
0bad18f38ada5b84c04


feat: implementation of FRI verifier in Miden assembly

Describe your changes
Implementation of FRI verifier in Miden assembly.
This is a draft implementation that lacks some checks that are necessary for the soundness of the protocol. It also doesn't make use of the new key-value map for advice input.
Checklist before requesting a review

Repo forked and branch created from next according to naming convention.
Commit messages and codestyle follow conventions.
Relevant issues are linked in the PR description.
Tests added for new functionality.
Documentation/comments updated according to changes.

Thank you! I left a few comments inline, but also a few general comments:
1. Would be great to create separate tests for various internal procedures (e.g. `fold_2`).

I have added separate tests for all internal procedures except the ones that are just doing preparatory stack manipulation.

2. For the end-to-end test, I'd prefer to generate a FRI proof dynamically rather than read one from a static file. Also, we should probably test for both failure and success cases.

I am working on this but I need the unbatch method for the MerkleBatchProof struct which is not included in the Winterfell repo we are using for the moment.

3. I'd put the module under `std::crypto::fri` instead of `std::math::fri`.

Done!

4. Would be great to update this to use the new advice set map.

I will do that after I get the second point above implemented. It shouldn't be too difficult to do then.

Lastly, what is the overall cycle count for verifying a FRI proof? (e.g., domain 2^16 or 2^20, using ~30 queries).
I have ran the analysis tool for a program without the mtree_get instruction as we don't support yet MerklePathSets in the cycle count tool. I am getting around 140k for a domain of size $2^20$ and query number equal to $32$.

I am working on this but I need the unbatch method for the MerkleBatchProof struct which is not included in the Winterfell repo we are using for the moment.

Ah! We should probably do a 4.1 release of Winterfell with all the latest things included.

I have ran the analysis tool for a program without the mtree_get instruction as we don't support yet MerklePathSets in the cycle count tool. I am getting around 140k for a domain of size $2^{20}$ and query number equal to 32.

So, we are getting about 4.4K cycles per query. We'll need to identify which parts contribute the most to this. Ideally, we should bring this down to under 2K cycles per query.

Yes, I will look more into it. I think there still a lot of room for improvement.


How could I make it detect the "Semicolon Key" or other Punctuation?

Take the semicolon as an example ：
I have tried to edit it in my skrdrc like this :
alt - ";" : <commands>
and
alt - ';' : <commands>
But it couldn't work.
I don't know if this is because the semicolon and other punctuation symbols are keywords.
Forgive my poor English, please.

@Ushio-Clover alt - 0x29 : <command> works fine.

This would be nice to have in the readme 😄.

@syuni Thank you very much for your help! I don't know exactly what type hotkey needs to be set.


Improve model-based validations

Model diagnostics for the current "seasonal naive" model for solar radiation show autocorrelation and non-normal (leptokurtic) residuals.  See if this can be improved by maybe figuring out how to use ARIMA models with fable

With solar radiation, I think a theoretical model that calculates values based on latitude and day-of-year for a clear day (no clouds) would be the best first step. For QA/QC of station solar radiation values, our biggest concerns are dust/mud/bird crap/etc. partially obscuring the sensor and artificially lowering values on clear days, as well as sensor drift between calibration visits (typically every quarter). Being able to compare clear-day values between the theoretical model and station measurements would be valuable. Perhaps plotting a time series of such on the 'AZMet QA/QC dashboard' would be good?

"Seasonal naive" is equivalent to ARIMA(0,0,0)(0,1,0)[365] (i.e. an ARIMA with no auto-regressive component, only a seasonal component). Seasonal naive seems to be as good or better that what the auto ARIMA selects for temperature and solar radiation.  Auto ARIMA also suggests something slightly different for different stations, so maybe seasonal naive is a good one-size-fits-all model to start with.  Although, I'm surprised that an auto-regressive component doesn't seem to improve the model (if it was hot yesterday, it should be hot today, right?).

Exploring using GAMs instead of ARIMA: https://fromthebottomoftheheap.net/2014/05/09/modelling-seasonal-data-with-gam/

Plenty of options with GAMs to fit the data better, but seems like GAMs are bad at forecasting.


Add badges for supported solvers

This PR adds custom badges to mark things in the doc as specific to certain solvers.

Hey @Revathyvenugopal162, I think this I managed to do something that works and is specific to PyDPF-Core (since the need is specific to our doc) and does not clash with the ansys-sphinx-theme. You mentioned a validation step for the colors?

yes, If you took the standard ansys colors (may be from theme or from any other sources), you should be good to go, anyway i will double check with ui/ux after we merge this.


Data Hub: Share Data Between Apps

In preparation for the Freemium release around Mendix World (everybody can consume up to 1000 datasets from the Data Hub per day without a license), I went through and compared the details on this page to how things are actually labelled in Studio Pro. Some copy-edits and deletions were also included.
@MariaShaposhnikova this is ready for review.

@ezagroba , I've completed the review, shall I publish changes?

@MariaShaposhnikova Yes, go ahead.


[libgit2] Add features pcre,pcre2 to choose regex backend

With the update to 0.99.0, this change was included. When REGEX_BACKEND is not specified, libgit2 preferentially builds against system libpcre, which is not always desirable.
With this patch:

core will use the built-in pcre as libgit2 did prior to 0.99.0
pcre will use vcpkg pcre
pcre2 will use vcpkg pcre2

All features test pass. This PR is ready to merge. @strega-nil

This looks really good! Thanks @kevinlul :)


User can still see keywords that have been "deleted" in AdWords web application

In a list of keywords for ad group, user can still see keywords that have been "deleted" in AdWords web application.
Deleting keywords works (as in examples/delete_keyword.rb), this is something else. Gotta figure out what it means when keyword is marked as "deleted" in AdWords web application, it might not be the same thing as deleting keyword by id.

PS: this issue might very well resolve itself after conversion to v201109

I'm afraid that this issue can't be resolved in Keyword service. We gotta use reports instead.


Ionic example only working in browser

I have implemented the ionic 3 example which works in the browser 'ionic serve'  but nothing shows in the about tab when running on Cordova.  If you can resolve this that would be amazing! The current virtual scroll in the ionic framework is essentially broken on the more recent iOS sdks.

@sliteleemadd, show your "ionic info" pls.

@ionic/cli-utils  : 1.19.1
ionic (Ionic CLI) : 3.19.1
global packages:
cordova (Cordova CLI) : 8.0.0

local packages:
@ionic/app-scripts : 3.1.6
Cordova Platforms  : android 7.0.0
Ionic Framework    : ionic-angular 3.9.2

System:
Android SDK Tools : 26.1.1
Node              : v8.9.1
npm               : 5.5.1
OS                : Windows 10

@sliteleemadd, i change example project. pls clone it again and test on device.

Still the same result.
I cloned fresh the project at https://github.com/mgnstudio7/ionic-virtual-scroll-example ran npm i,  then >  npm i cordova, then > ionic cordova platform add android  then > ionic serve (just to see the new goods in the browser which worked just fine) and then ran > ionic cordova run android  ... launched on an Nexus 5X  run Oreo.  The app launches but when I go to the about us tab it displays no content....  Thanks for the fast response time on this issue btw.  Could this be related hash routing in stencil? I know that was also stenciljs / cordova issue.  Just taking a wild stab in the dark.  :|

Same for me. In the console I got the error

Failed to load module script: The server responded with a non-JavaScript MIME type of "". Strict MIME type checking is enforced for module scripts per HTML spec.

when opening the about page.

Any insight or intent to fix this issue?

@sliteleemadd
Sorry for the long answer.
I updated the example project, change test api and tested in the browser and IPhone 5s, 6 (IOS9, 10). Works perfectly!
Please try to update the example project again and check on your android device.
Thank you for using!

@WolfWalter maybe your error was related with fail api response. i change api, please test again.

@WolfWalter please show your 'ionic info' global packages.

I tried again. In the browser it breaks if I scroll to the bottom of the about page and then switch to the home page and back.
With ionic cordova run android about is just a blank page with the mentioned error message.
ionic info:
cli packages: (/usr/lib/node_modules)

    @ionic/cli-utils  : 1.19.1
    ionic (Ionic CLI) : 3.19.1

global packages:

    cordova (Cordova CLI) : 8.0.0

local packages:

    @ionic/app-scripts : 3.1.8
    Cordova Platforms  : android 7.0.0
    Ionic Framework    : ionic-angular 3.9.2

System:

    Android SDK Tools : 26.1.1
    Node              : v8.9.4
    npm               : 5.6.0
    OS                : Linux 4.13

Environment Variables:

    ANDROID_HOME : /home/wlt/Android/Sdk

Misc:

    backend : legacy

I also still get a blank about page when running on cordova android

@mgnstudio7 @bmacintyre facing the same issue. once the list reaches the bottom and when new items are added to list and moving from about to other page and coming back about makes the scroll broken and I see empty/blank screen, items disappear and they appear back when scrolled up and down. @mgnstudio7 can you confirm this?. How to fix this any quick hack?. It's a much-needed component thanks for developing it.

@faraazc @bmacintyre , thx for using. I'll try to reproduce and fix this bug. I will inform you about the results.

@faraazc, @bmacintyre please clone example project again. If the problem persists, notify me!


[DNM] *: Enable Time for PerfContext

Signed-off-by: Breezewish<EMAIL_ADDRESS>
What problem does this PR solve?
Experiment to enable Time for PerfContext, see whether it's worth to do.

/bench +sysbench +tpcc +tpch

/bench +sysbench +tpcc +tpch

I think our benchmark infra is broken..

/bench +sysbench +tpcc +tpch

/bench +sysbench +tpcc +tpch

/bench +tpch

/bench +tpch


Unequal strides support recently removed for DepthwiseConv2D

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with the
documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go
here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script
provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue
happens on a mobile device:
TensorFlow installed from (source or binary):
TensorFlow version (use command below): 2.9 and 2.11
Python version: 3.8
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 11.2 and 8.1
GPU model and memory:
Exact command to reproduce:

import tensorflow as tf
import numpy as np
layer1 = tf.keras.layers.DepthwiseConv2D(depth_multiplier=2,kernel_size=(1,9),strides=(1,2))
print(layer1(np.ones((100, 28, 28, 1), dtype=np.float32)))
You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with:
python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
The command would run without any error when I was using it with TF2.11 version installed previously. On the newly installed TF2.9 version, however, the code throws an error as:
InvalidArgumentError: Exception encountered when calling layer 'depthwise_conv2d_1' (type DepthwiseConv2D).
{{function_node _wrapped__DepthwiseConv2dNative_device/job:localhost/replica:0/task:0/device:CPU:0}} Current implementation only supports equal length strides in the row and column dimensions. [Op:DepthwiseConv2dNative]
Call arguments received by layer 'depthwise_conv2d_1' (type DepthwiseConv2D):
• inputs=tf.Tensor(shape=(100, 28, 28, 1), dtype=float32)
Seeing the documentation, the error is expected for version 2.12, but should work for versions 2.9 and 2.11
https://www.tensorflow.org/versions/r2.9/api_docs/python/tf/keras/layers/DepthwiseConv2D
https://www.tensorflow.org/versions/r2.11/api_docs/python/tf/keras/layers/DepthwiseConv2D
https://www.tensorflow.org/versions/r2.12/api_docs/python/tf/keras/layers/DepthwiseConv2D
Is it that the implementation existing in the earlier versions has been removed recently for all previous and current versions?
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

Hi @KaminiSabu, As it is mentioned in the documentation of Tensorflow versions 2.9, 2.11 and 2.12 that the current implementation only supports equal length strides in the row and column dimensions. Specifying any stride value != 1 is incompatible with specifying any dilation_rate value != 1. describing the strides argument. This is an expected behavior in all the three versions of Tensorflow. Thank you!

But in the link you shared, against strides, I can see the following entry for 2.9 and 2.11
"An integer or tuple/list of 2 integers, specifying the strides of the convolution along the height and width. Can be a single integer to specify the same value for all spatial dimensions. Specifying any stride value != 1 is incompatible with specifying any dilation_rate value != 1"
The equal stride support is mentioned only for 2.12
And more importantly, it works in the older installations on a specific machine and not on the new installations I am doing on other machines.

@KaminiSabu Apologies for the confusion. You are right. It is not mentioned in 2.9 and 2.11 doc about the nonavailability for equal stride. Kindly refer to this issue, it is a known issue. The document has been updated in TF v2.12. Thank you!

Ohh, so it should not work for any previous implementations as well. But it works in an older installation on a specific machine. Any idea how? Was the implementation wrong and hence is not being supported now? I used the same set of commands on that machine and got an output matrix printed:
import tensorflow as tf
import numpy as np
layer1 = tf.keras.layers.DepthwiseConv2D(depth_multiplier=2,kernel_size=(1,9),strides=(1,2))
print(layer1(np.ones((100, 28, 28, 1), dtype=np.float32)))

I'm not sure what has changed since 2.9 in the backend, but the current Argument details is inline with the API behavior.
For the latest API behavior, please install latest package. Thank you


Release a new version

Ok, @mlangkabel provided enough fixes that it makes sense to do a new release

Coming weekend I'll release JP alpha 3 with some interesting changes :)

It will be the perfect time to release 0.3.1 :)

I would love some feedback about the changes in alpha 3!

Sure, for now moving to alpha 2 was pretty straightforward


[WIP] refactor(send): Add context.Context to parameter list of Send method

Info
This closes #49. I added context.Context to the parameter list to all implementations and definitions of the Send method. This will allow users to cancel long running calls of said function.
Suggested changes

Add context.Context to parameter list of Send method
Update tests
Update documentation

Additional context
I need feedback for this! I added some TODO comments here and there.
In some cases the external library does not accept a context.Context itself and we don't loop over receivers but pass them all at once. This makes it hard for me to decide how to handle the context correctly. Do we check  context.Context.IsDone at the beginning of a Send implementation, do we check it right before actually sending the message payload to the external service?

@checkaayush could you give this a quick review?


Add event logging information

Relates to ADAPP-17361. Adds log examples for event logging.

@shashanksu Would you mind taking a look and letting me know if this makes sense to you? Also, can you confirm that the log examples are the same across the different SDKs?
https://dev-docs-git-adapp-17361-how-to-check-pa-0a8bd7-product-content.vercel.app/en/sdk/android/features/events

@Sporiff It looks good to me.  Yes, those params listed in the examples are same across SDKs.


"TrueType versus OpenType Layout"

This section is muddled and should probably be removed. The first paragraph compares TT and OT, which is no longer relevant. The second paragraph repeats previous material. The third paragraph tries to distinguish the responsibilities of the layout engine, shaping engine and font file, but does so in a way that is unclear and at any rate has nothing to do with TrueType versus OpenType.

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

ID: 5b15ab9e-86c5-f6e0-0145-f9de77c60141
Version Independent ID: 0ebe3dec-cc24-b6a4-01b8-73bfdb79a71a
Content: Advanced typographic tables - OpenType Layout - Typography
Content Source: typographydocs/opentype/spec/ttochap1.md
Product: typography
GitHub Login: @PeterCon
Microsoft Alias: PeterCon

Changed for the next version: removed the heading and the first two paragraphs. The third paragraph has some relevant info and is kept with some revisions:

As much as possible, the tables of OpenType Layout tables define only the information that is specific to thea particular font layout. The tables do not try to encode information that remains constant within the conventions of a particular language or the typography of a particular script. Such information that would be replicated across all fonts in a given language belongs in thea text-processing application for that language, not in the fonts.

The placement of this paragraph at the end of (now) the "OpenType Layout at a glance" section seems reasonable.


Add optional parameter for the password

This in order to be able to copy/paste the entire CLI call.
If only username is provided it can prompt for the password, so interactive login method would also be supported.

I'm thinking I'll make lowercase p for the password and uppercase P for the port, wdyt?

+1


Add due date to tod task create

Here are the docs for create task on the Todoist API https://developer.todoist.com/rest/v2/#create-a-new-task
Add a due-date flag to the tod task create command. There is no default. Validate that it is in the format YYYY-MM-DD
tod task create --due-date "2024-02-30"

Alternately we can also use the inquire library's date picker! I'm not sure which is better at this point.

I would recommend merging due_date and due_string to just "due", and then
If entered in format of YYYY-MM-DD set as due-date
Otehrwise set as due_string
it's worth noting that todoist also supports due at specific time (vs just date), so YYYY-MM-DD HH:MM is also valid (it will use your account timezone in this case, unless you override it with a specific timezone in the API call)

Agreed :+1:


Adding documentation for R4001

Fixes #3604
omitted normal checklist because this is not a Swagger change

Automation for azure-sdk-for-python

Automation for azure-sdk-for-java

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

Automation for azure-sdk-for-ruby

:+1::sparkles:Ship It:sparkles::shipit:


Top Left Hud will need to be redesigned as an actual UI element since thats currently the biggest performance drain from the scripts. Ignoring that, the game for me is officially gpu bound meaning I can't optimize the fps much more than that. There are probably some things in multiplayer that may need looking into since I didn't test too much there and I will do another optimization run probably in another month which will include that.
Frame time for me is 1.5ms - 2ms with spikes to 3ms on weird occasions that might be due to the profiler.
Garbage collection is down to about 3kb/frame which I think is probably reasonable for our case.
FPS stays consistently above 500 fps on 15 titans survival compared to before where it would drop under every few seconds.
Some of the optimizations I did use statics/private object caches for specific cases that stuck out.
Some photon rpcs if sent every frame like titan animation alloc about 1.5kb per request if you don't reuse an object[] for the sent information.
Realistically going forward, for one-off things allocs are fine to use if the code is faster to write.
Anything in update/fixedupdate should be watched for lazy/unneeded alloc calls.
String methods that alloc like Trim or Regex calls should also be avoided every frame if possible.
Calls to functions using the params array is also an allocating call and will create a new array to house the values and so it should also be avoided on frame. (Audio's GetRandom and PhysicsLayer's GetMask are this way)

Found an issue with one change, wait on review for a sec


HAWQ-1480 - Added feature for packing a core file

Currently there is no way to packing a core file with its context – executable, application and system shared libraries in hawq. This information can be later unpacked on another system and helps in debugging. It is a useful feature to quickly gather all the data needed from a crash/core generated on the system to analyze it later.
Another open source project, greenplum, uses a script https://github.com/greenplum-db/gpdb/blob/master/gpMgmt/sbin/packcore to collect this information. Merging this into hawq.

Aside from the two misspellings identified and the pending determination to keep or remove the "Copyright Pivotal 2014" string, this PR looks good.  I have run the packcore against a HAWQ instance and it performed as expected.  Will we need to add documentation for this utility?

Additionally, is a test being added to validate the functionality of packcore?

Ed, I agree with you on adding documentation for this utility. I have some concern on adding a test case, is it a little bit strange to have a core dump file in source repository(maybe two, OSX and Linux)? Or write a test case which can trigger a core dump, then run this script.

@edespino @vVineet @linwen - Thank you for the reviews and comments. I have incorporated the changes and removed the copyright for now.
@linwen What kind of documentation do you have in mind? Here are some options -

A separate markdown explaining the use case and how to, committed to this repository
Use case and how to use packcore, committed under - incubator-hawq-docs/markdown/admin/

Additional suggestions are welcomed.

@outofmem0ry - Thanks for addressing the issues. FYI: It is helpful to push the changes to your repo:branch (outofmem0ry:feature-packcore) that this PR is based on so we can see that the issues were indeed addressed. The PR will automatically get updated when you push to your dev branch. Thanks, -=e

@outofmem0ry - It might be helpful to check with David Yozie (@dyozie) to see if we can get this into the public HAWQ user/admin documentation.

I doesn't look like Greenplum documents packcore either.

I'm thinking from the perspective of an Apache developer. They shouldn't have to go to external corporate documentation sets or knowledge bases (https://discuss.pivotal.io/hc/en-us/articles/201081408-How-to-collect-core-files-for-analysis) to get useful information about a utility (packcore) included with the HAWQ distribution. As it stands, the utility (packcore) will be automatically bundled with all future HAWQ releases.

@dyozie correct, it is not documented in greenplum.
@edespino - Totally agree with your point. Keeping that in mind, earlier, I suggested documenting it under incubator-hawq-docs/markdown/admin/ as this tool is more in an administrative capacity.
Let me know if that sounds good and I can start with the documentation.

@dyozie - is it easy for @outofmem0ry to add docs for this utility (packcore) in incubator-hawq-docs/markdown/admin/. Can you provide some guidance? It will be helpful to share this with the dev community for future reference.

Yes - it should be straightforward to add this information, and docs can
provide further assistance via the PR.
-David
On Thu, Jun 8, 2017 at 9:22 AM, Ed Espino<EMAIL_ADDRESS>wrote:

@dyozie https://github.com/dyozie - is it easy for @outofmem0ry
https://github.com/outofmem0ry to add docs for this utility (packcore)
in incubator-hawq-docs/markdown/admin/. Can you provide some guidance? It
will be helpful to share this with the dev community for future reference.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-hawq/pull/1251#issuecomment-307154762,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFbpVqKXGUwWZePsUGLPQFv8P9plQvyWks5sCB_IgaJpZM4Nw2Wo
.

@dyozie @edespino - submitted documentation under incubator-hawq-docs pull Request #123.

LGTM

This PR (sans documentation which is included in PR #123) LGTM.

merged, this pr can be closed now. Thanks!

Shubham, this PR has been merged into master. Would you please close it? Thanks!

Thank you for the merge @linwen. Marking it closed.


Unhandled error occurred

"fffff" at https://stellar.expert/explorer/public/asset/UBEC-GDPNB7S3IOM2J6C3NA2QG4TQAUCRZXPJJ4HSCSIKELEH7ORUCX5UB2VN
De@https://stellar.expert/distr/explorer.js:1:147836
Zi@https://stellar.expert/distr/app.js?v=0.15.9:2:521759
Es@https://stellar.expert/distr/app.js?v=0.15.9:2:567809
yu@https://stellar.expert/distr/app.js?v=0.15.9:2:560551
mu@https://stellar.expert/distr/app.js?v=0.15.9:2:560476
su@https://stellar.expert/distr/app.js?v=0.15.9:2:557506
Yo/<@https://stellar.expert/distr/app.js?v=0.15.9:2:509147
t.unstable_runWithPriority@https://stellar.expert/distr/app.js?v=0.15.9:2:584203
Qo@https://stellar.expert/distr/app.js?v=0.15.9:2:508856
Yo@https://stellar.expert/distr/app.js?v=0.15.9:2:509092
Go@https://stellar.expert/distr/app.js?v=0.15.9:2:509027
uu@https://stellar.expert/distr/app.js?v=0.15.9:2:557797
qe@https://stellar.expert/distr/app.js?v=0.15.9:2:44600
ze@https://stellar.expert/distr/app.js?v=0.15.9:2:44183
Be@https://stellar.expert/distr/app.js?v=0.15.9:2:40546
be/<@https://stellar.expert/distr/app.js?v=0.15.9:2:33775
be@https://stellar.expert/distr/app.js?v=0.15.9:2:33846
Ye@https://stellar.expert/distr/app.js?v=0.15.9:2:45507
sync/this.promise<@https://stellar.expert/distr/explorer.js:1:15138

Sorry about that. The system update we rolled recently gone wrong.
Fixed now.


Prerelease

Go 1.0

Awesome! So glad to see some more love for this module. Can any one of you note any major breaking changes from 0.3.37 to 1.2? I just migrated and just need to know if I should change any of my scripts.

No breaking changes afaik. I just wanted to bump it to 1.0 because I wanted to lock in that the current cmdlets were stable. I want to add a lot more test coverage and update the docs to be more helpful before introducing anything breaking. At which point it would be 2.0.


First pass at rule intro

As the data isn't actually available yet, this pulls the rule intro sections from a configuration. This doesn't include any new styles, so it looks pretty minimal.
Builds on #243 and resolves eregs/notice-and-comment#151
Looks like

@jmcarp updated to use a shared function between the three views


Why not provide unit test for these solutions?

hope  provide unit test for these solutions
thanks :)

sure i will try to add some, though most of them are tested against several test cases and accepted in leetcode , geeksforgeeks or hackerrank ))


@click.argument with nargs=-1 ignores default

Here are two test cases adapted from the existing ones:
def test_eat_options_with_default(runner):
    @click.command()
    @click.argument('files', nargs=-1, default=('test',))
    def cmd(files):
        for filename in files:
            click.echo(filename)

    result = runner.invoke(cmd, ['--'])
    assert result.output == 'test\n'

def test_eat_options_with_default_and_type(runner):
    @click.command()
    @click.argument('files', nargs=-1, default=('test',),type=str)
    def cmd(files):
        for filename in files:
            click.echo(filename)

    result = runner.invoke(cmd, ['--'])
    assert result.output == 'test\n'

They both fail, with the output being empty (default is ignored).
I managed to track this down to click/core.full_process_value:
def full_process_value(self, ctx, value):
    value = self.process_value(ctx, value)

    if value is None:
        value = self.get_default(ctx)

    if self.required and self.value_is_missing(value):
        raise MissingParameter(ctx=ctx, param=self)

    return value

Instead of checking if value is None, shouldn't it check for if self.value_is_missing(value) ?
With this change, the second test passes, but the first one fails with a type checking exception:
        if self.type.is_composite:
            if self.nargs <= 1:
                raise TypeError('Attempted to invoke composite type '
                                'but nargs has been set to %s.  This is '
                                'not supported; nargs needs to be set to '
                                'a fixed value > 1.' % self.nargs)

This can be fixed by adding the following checks to Parameter.__init__:
        if nargs < 0 and self.type.is_composite:
            if len(self.type.types) > 0:
                self.type = self.type.types[0]
            else:
                self.type = types.STRING

This last part is not very pretty, but with it both tests pass.

I think the actual fix would be to not permit a default value for nargs -1.  This in my book seems like the wrong way to do it.  But I need to think about this more.

My use case is building a tool which accepts a list of file names, defaulting to some specific file name. In my case it was requirements.txt, but I suppose the most common case would be stdin (as "-").
I agree that the whole self.type shenanigans is a bit odd, I just didn't want to submit an issue without a solution. Maybe those checks could be elegantly integrated into Parameter.type_cast_value, or one could add nargs to click/types.convert_type.
It would be nice to at least change if value is None to self.value_is_missing(value) in click/core.full_process_value. This would allow us to use nargs=-1 with defaults by specifying the type manually.

This is no longer allowed in click 6.


Refactor ipfs-companion so it no longer sets properties on the window

This PR refactors the ipfs-companion module so it no longer sets properties on the window object.
I'm viewing the ipfs-companion module as something I'd like to create an instance of, or multiple instances of, and also teardown - with the end goal of making it easier to reason about, maintain and test. So this PR stops it from setting properties on the window object and instead init returns an API object which is the method one should communicate with the module.
background.js, after creating the new instance, sets it to window.ipfsCompanion so that the popup, quick upload and options pages can make use of it's API.

Pulls out the notifier logic into it's own module
Pulls out the copying logic into it's own module
Uses port.postMessage to signal to the companion to perform a copy (instead of calling the functions on the background page)
Refactors some of the async FileReader logic into async/await style

@lidel ok all fixed up - I moved the context menu initialisation and update stuff into the context-menus module and that has fixed it.

This is quickly going to get difficult to merge with #320 and friends, so I'm gonna take a look at it now and see if it's good to go  🚀✨ 👀

@lidel do you have a minute to merge this? Merging is blocked on re-review. In retesting this I can see the error message has been resolved.

@olizilla my apologies, I run out of time to review this yesterday.
Old error is gone, but now I get this one on every API poll (while external node is down):
ReferenceError: can't access lexical declaration `offlinePeerCount' before initialization  background.js:360:

Are you able to reproduce? If not, it may be the machine I am on, I'll retest on another one in 5h.

@lidel yes, good spot, I'm seeing that too. @alanshaw can you take a look when you get a moment?

Actually, I see the problem... pesky hoisting / lexical scoping. Am fix.

Thanks! 👌


🌐 Add Korean translation for docs/ko/docs/help/index.md

Add Korean translation for docs/ko/docs/help/index.md
After this, I will add help category Korean translation

📝 Docs preview for commit 7ae253f38fad94121dcf1733e830f0a047677da8 at: https://538377d4.fastapitiangolo.pages.dev

Thankyou

Thankyou
Hello @kitrio, If you don't mind, can you review my translations?

https://github.com/tiangolo/fastapi/pull/10976
https://github.com/tiangolo/fastapi/pull/10983
https://github.com/tiangolo/fastapi/pull/10989
Thanks

Thanks for your contribution @KaniKim 😸
And thanks for the review @junah201 🚀✨


Struggled to build on Linux

I managed to install and run this game on Linux, but I had to do the following before running make:
sudo apt install cmake libxss-dev libxxf86vm-dev libxkbfile-dev libxv-dev libxrandr-dev libxinerama-dev libxcursor-dev libxi-dev -y

And? Do you want to uninstall it? Those packages are necessary for building, you can uninstall it.
You can do so with sudo apt uninstall cmake libxss-dev libxxf86vm-dev libxkbfile-dev libxv-dev libxrandr-dev libxinerama-dev libxcursor-dev libxi-dev, notice there is an absence of -y? That is to make sure you do not uninstall it accidentally.
If you already have done the first command, and want to remove the dependencies (apt shows you that there are dependencies that are not automatically removed when uninstalling), you can do that with sudo apt autoremove.

See The pull request.

Oh, I actually mean that the instructions for Unix-like systems didn't include these instructions. So, I thought I'd raise it. I guess I could have communicated that better.

I guess I misunderstood.

Seems like an easy PR to me :)

Needs clang as well. I happen to not have it installed on Tumbleweed

Should installing build-essentials work? I have not really tested it myself.

Yeah it did for me

I recommend specify which packages the build require and the user most likely may not have instead of only target Ubuntu-based distros and tell to install a metapackage for Ubuntu, as other users may be using Fedora, Archlinux or, in my case, openSUSE. There is no build-essentials metapackage on Tumbleweed. But regardless of that, I managed to build it after installing clang.


Make button tap public

Current coverage is 100% (diff: 100%)

Merging #18 into master will not change coverage

@@           master   #18   diff @@
===================================
  Files           7     7
  Lines          64    64
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
  Hits           64    64
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 828975f...3543d02


Editor: fix mutexes issues (partially) - runnable application

This PR adds the following changes:

[x] Refactor to runnable gradle application.
[x] Untracked build files.
[x] An attempt to stabilize the concurrent model.

In order to run this application, you have to download jdk-8, then use gradle wrapper:
└──╼ $./gradlew -Dorg.gradle.java.home=/opt/jdk1.8.0_202/ run

@icyboxs Could you test this PR to confirm the fix, please?


Tic Tac Toe Game

Initiative (Required)
GSSoC 2024 Extd 🚀
Is your feature request related to a problem? Please describe.
Yes, the feature request is related to enhancing the multiplayer experience in the Tic Tac Toe game. Currently, players may face challenges with real-time updates and maintaining game state across different sessions. For example, if a player disconnects, their game state may be lost, leading to frustration and a poor user experience.
Describe the solution you'd like.
I would like to implement a feature that allows for real-time multiplayer gameplay using WebSockets. This would enable players to see moves made by their opponent in real time, ensuring the game state is synchronized across all connected clients. Additionally, I propose implementing a mechanism to save game state on the server side, so if a player disconnects, they can rejoin the game and continue from where they left off.
Here’s a summary of the desired features:
Real-time updates: Use WebSockets to push updates to all connected players immediately when a move is made.
Persistent game state: Store game sessions on the server so players can resume their games after disconnecting.
User-friendly UI: Display appropriate messages when a player reconnects or when a game session is resumed.
Handling disconnections: Implement logic to manage player disconnections gracefully, providing options to either wait for reconnection or to forfeit the game.
This would greatly enhance the multiplayer experience by ensuring that players remain engaged and informed throughout the game.
Add any other context or screenshots about the feature request here.
No response

Hey I am the mentor of this project
@itznayan Hello Thank you for opening the pull issue and you can opening new issue by creating the new concept those are not exist in the project !!
@Kushal997-das kindly close the issue since it already exist in our project.


[Desktop/Linux] exception when selecting new message in Ubuntu notifications

Running linux client on Ubuntu 22.04 LTS;
No LSB modules are available.
Distributor ID:    Ubuntu
Description:    Ubuntu 22.04.1 LTS
Release:    22.04
Codename:    jammy
and getting following exception when selecting new message in Ubuntu notifications;
flutter: [BlueBubblesApp][2022-12-05 08:15:39.363656][DEBUG] (Queue) -> [HandleMessage] Successfully mapped message (Chats: 1)
flutter: [BlueBubblesApp][2022-12-05 08:15:39.363935][INFO] Client received new message SMS;-;46339
flutter: [BlueBubblesApp][2022-12-05 08:15:39.454080][INFO] (Actions-HandleMessage) -> New message: [FedEx: Your package is out for delivery today 12/05. fedex.com/t/391274973559/en_US Reply HELP for help. STOP to cancel.] - [BC650D51-7AEB-5FDC-3DF3-8D275C54D894]
flutter: [BlueBubblesApp][2022-12-05 08:15:47.439018][INFO] Startup Logs
flutter: [BlueBubblesApp][2022-12-05 08:15:47.442704][INFO] Opening ObjectBox store from path: /home/briano/.var/app/app.bluebubbles.BlueBubbles/data/bluebubbles/objectbox
flutter: [BlueBubblesApp][2022-12-05 08:15:47.448594][ERROR] Bad state: failed to create store: 10001 Cannot open store: another store is still open using the same path: "/home/briano/.var/app/app.bluebubbles.BlueBubbles/data/bluebubbles/objectbox"
flutter: [BlueBubblesApp][2022-12-05 08:15:47.448692][ERROR] #0      ObjectBoxNativeError.throwMapped (package:objectbox/src/native/bindings/helpers.dart:66)
flutter: #1      throwLatestNativeError (package:objectbox/src/native/bindings/helpers.dart:50)
flutter: #2      checkObxPtr (package:objectbox/src/native/bindings/helpers.dart:32)
flutter: #3      Store._checkStorePointer (package:objectbox/src/native/store.dart:309)
flutter: #4      new Store (package:objectbox/src/native/store.dart:150)
flutter: #5      openStore (package:bluebubbles/objectbox.g.dart:732)
flutter: #6      initApp.initStore (package:bluebubbles/main.dart:287)
flutter:
flutter: #7      initApp (package:bluebubbles/main.dart:337)
flutter:
flutter: #8      main (package:bluebubbles/main.dart:149)
flutter:
flutter:
flutter: [BlueBubblesApp][2022-12-05 08:15:47.448881][ERROR] LateInitializationError: Field 'store' has not been initialized.
flutter: [BlueBubblesApp][2022-12-05 08:15:47.448924][ERROR] #0      initApp.initStore (package:bluebubbles/main.dart)
flutter:
flutter: #1      initApp (package:bluebubbles/main.dart:337)
flutter:
flutter: #2      main (package:bluebubbles/main.dart:149)
flutter:
flutter:
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: Exception: LateInitializationError: Field 'store' has not been initialized. #0      initApp.initStore (package:bluebubbles/main.dart)

#1      initApp (package:bluebubbles/main.dart:337)

#2      main (package:bluebubbles/main.dart:149)

#0      initApp (package:bluebubbles/main.dart:512)

#1      main (package:bluebubbles/main.dart:149)

(bluebubbles:2): GLib-GObject-WARNING **: 08:15:52.345: ../gobject/gsignal.c:2772: instance '0x559a60e051b0' has no handler with id '2726'

It looks like its trying to open a new instance of the app rather than use the old instance already open, which would cause this issue

I'm starting app on command line as;
flatpak run app.bluebubbles.BlueBubbles
Whenever I select notifications in terminal see the above exception and new bluebubbles window with attached screenshot and message.

fixed internally

Confirmed as fixed.  Thanks!!


Create a classroom invite link for an instructor

Purpose
Give moderator classroom invite link to users.
Fixes #2159
Proposal

[x] Generate a token for classroom instructor, and send it through the API
[x] Build and display a moderator classroom invite link
[x] Using this link,  display the dashboard to unauthenticated users

About the jwt type, I made that, do you think it will be ok with what you have made ?

Yes, I'll adapt to your implementation, wich is way better than mine! :+1:


Can't execute after update

I installed with go get...  and I had used well whatever update happen.
But yesterday, there was a lot of update when I typed gore and it is unable to run main()
Below is an error message.
$> gore
go run: cannot run non-main package

What do you mean by update? Update of gore version or of what?

Sorry for an ambiguous use of word.
It was a lot of updates including "golang/... /sys/syscall..." related updates.


multiple hosts

does this script work with multiple hosts under a single domain? or does it need a separate instance for each host and domain?

This script only update ONE host under single domain.  If you want it to handle multiple hosts and/or multiple domains, you may fork it and modify the script.
You may also give the script different names and run multiple cronjob accordingly.

This script only update ONE host under single domain. If you want it to handle multiple hosts and/or multiple domains, you may fork it and modify the script.
You may also give the script different names and run multiple cronjob accordingly.


Update OpenSSL constant usage to work with Ruby 3 AND with in FIPS mode

The simple fix from rubocop broke FIPS support and was reverted in https://github.com/chef/chef/pull/10058
@btm has a suggestion here for how we could potentially work around fips failures within that PR.

Fixed and shipping in 16.2.72


Patch 2

Questions
Answers

Branch?
1.7.5.x

Description?
When the user is registring at the site and simultaneously he is registring to the newsletter, if the newsletter voucher is active (configuration NW_VOUCHER_CODE) the system does not send the email with the code.

Type?
bug fix

Category?
FO

BC breaks?
no

Deprecations?
no

Hi @Codencode
thanks for this patch, but first change the branch target to dev (not master)
And you need to fetch the changes that were merged inside the dev branch (from your previous PR) into this current PR

Hi @jolelievre,
I made the changes.
Thanks.

Hello @Codencode , thanks for your contribution, would you mind to rebase your pull request? Thanks

Hello @Codencode , thanks for your contribution, would you mind to rebase your pull request? Thanks

Hi @Progi1984  could you help me? I don’t remember it very well, so how rebase my pull request?
Thanks

Hello @Codencode , thanks for your contribution, would you mind to rebase your pull request? Thanks

Hi @Progi1984, could you help me? I don’t remember it very well, so how do I rebase my pull request?
Thanks

No problem, PR rebased and Upgrade fixed

Ping @Codencode

Thanks @Codencode


Advanced filterable multiselect list widget (recommendations, actions)

As a user of Sadata
I want an easy to use UI element for filtering a large list of things by keyword and associated categories and choosing one or more options from the list.

Development tasks:

Implement partial structure for rendering list container
Implement generic-ish Javascript to perform XHR requests to populate the list
Allow filtering of the list using a search field
Consider supporting pagination
Support adding new record and selecting it from the list
If items are pre-selected, show them at the top of the list
Try not to put too much magic in

This issue was moved to dumparkltd/hr-nmrf-client#23


SAK-32453 - Update powermock

I tried to update the kernel but there were too many errors as it's still on mockito 1. I created this jira.
https://jira.sakaiproject.org/browse/KNL-1518
@ddelblanco They changed the implementation for "Or" and I couldn't get it to work quickly using what was there (only takes 2 parameters) so I just copied the old implementation in as a local class. I'm not sure if you wanted to look at this or not. Everything that was 2 args passed, but all of the 1 arg values seemed to fail.

Yeah, would love to drop that class, just seemed the easiest fix to get the same result without completely understanding what was going on here. Please add if it still passes!


Performance question

When i scroll ListView.builder than each new appears item triggers rebuild all of SvipeActionCell widgets.(9-15 pcs per time). is this behavior right and expected?

When i remove SwipeActionCell widget, yhen builder builds only new IgnorePointer. What is my bad?
I tried to pass index as ValueKey, it doesn't help.

No ,when scroll,the existing widget will not rebuild.Moreover,the key should not be ValueKey(index).
It should be ValueKey(yourDataList[index]). You can check the example project in this repo.

Fixed.You can check the new version on pub.dev with 2.2.1. You can close this issue after testing good on your code😀.

Thank you, it good in my code!)


Not working with Windows 8.1

The latest version doesn't install at all. I got the main page but there's no box to type my Instagram user name and no box to type my password. Did try to modify it to run like it was Windows 7 but nothing did change. Now, I did download the older version 2.1.0 and now the boxes are there but I fill them with all the necessary information and then click "log in" and I get the message: "Cookie sessionid you are searching found was either not found or not valid!"

It's working now ( the newest version). Thanks!


Jest coverage collection is not working with TypeScript files

Version
3.0.0-rc.3
Reproduction link
https://github.com/Kocal/vue-cli-no-coverage-on-typescript-files/commits/master
Steps to reproduce
Project creation

Run vue ui
Create a new project
Presets: Manual
Check Babel, TypeScript, and Unit Testing
Pick a linter / formatter config: ESLint   Prettier (it's probably unrelated, but this is what I chose)
Pick a unit testing solution: Jest
Create project
Continue without saving

Post-project creation

Enable Jest coverage collection from package.json: jest.collectCoverage: true
Create a file src/divider.ts containing export const divide = (a: number, b: number) => a / b;
Create a spec file tests/unit/divider.spec.ts containg test('divide', () => { expect(divide(5, 2)).toEqual(2.5) });
Run yarn test:unit or  yarn test:unit --no-cache

And under your eyes, you will see that coverage from src/divider.ts is not collected.
What is expected?
Coverage from TypeScript files should be collected.
What is actually happening?
I investigate and this is what I found.
It really seems to be that when a file is written in TypeScript, its code coverage won't be collected.

service
written in...
test written in...
coverage collected?

substracter
TypeScript
TypeScript
No

divider
TypeScript
JavaScript
No

multiplier
JavaScript
TypeScript
Yes

Here is a Travis build that show Coverage state.
I'm a bit afraid because I'm actually migrating an app which uses vuejs-templates/pwa to @vue/cli 3, but I literally lost all the code coverage... :cry:

Something strange too, code coverage from .vue files is not collected. :thinking:
Here is another Travis build that ran just after:

enable code coverage collection
use babel-jest for handling .js files, but it's not related I think

Thanks for your help (also you all did a great job on @vue/cli and vue ui! :slightly_smiling_face:), do not hesitate to ask me for more info/context!

Hum, actually you're right.
I totally forgot about that because I thought it was implicit, but it's worse after specified it.
On the current travis build, src/App.vue and src/main.ts now appear (but they are not tested nor required in tests or tested files), and src/services/multiplier.js is still the only one services to appears. :confused:
Maybe it's not a vue-cli related issue, I will check Jest and ts-jest if something bad happens.

Well... One of the latest issue on ts-jest is about a failing coverage, and the given solution works :disappointed_relieved: :
"jest": {
  "globals": {
    "ts-jest": {
      "disableSourceMapSupport": true
    }
  }
}

But it's looks like a terrible fix. :thinking:

If someone read this, this issue has been correctly fixed by updating ts-jest to 23.1.3 and removing disableSourceMapSupport in your tsconfig.json.


⚠ RPC server - Optimism has degraded performance

In d4f4b73, RPC server - Optimism ($OPTIMISM_RPC_URL) experienced degraded performance:

HTTP code: 429
Response time: 123 ms

Resolved: RPC server - Optimism performance has improved in 0013f40 after 6 minutes.


Simplify API

From https://github.com/mesg-foundation/core/pull/100#issuecomment-385796962
Also, some gRPC messages contain the parameters task and data with executionID. It's useless in this case, no?
I suggest for api_result.proto:
syntax = "proto3";

package types;

import "service.proto";

service Result {
  // Service
  rpc Submit (SubmitResultRequest) returns (SubmitResultReply) {}

  // Application
  rpc Listen (ListenResultRequest) returns (stream ListenResultReply) {}
}

message SubmitResultRequest {
  string executionID = 1;
  string output = 2;
}

message SubmitResultReply {
  string error = 1;
}

message ListenResultRequest {
  ProtoService service = 1;
}

message ListenResultReply {
  string executionID = 1;
  string output = 2;
}

and for api_task.proto:
syntax = "proto3";

package types;

import "service.proto";

service Task {
  // Service
  rpc Listen (ListenTaskRequest) returns (stream ListenTaskReply) {}

  // Application
  rpc Execute (ExecuteTaskRequest) returns (ExecuteTaskReply) {}
}

message ListenTaskRequest {
  ProtoService service = 1;
}

message ListenTaskReply {
  string executionID = 1;
  string task = 2;
  string data = 3;
}

message ExecuteTaskRequest {
  ProtoService service = 1;
  string task = 2;
  string data = 3;
}

message ExecuteTaskReply {
  string executionID = 1;
  string error = 2;
}

Otherwise, I really doesn't like the organization of the API we are doing. Each gRPC service has 1 endpoint for the application and another for the service. It's super confuse and we have to create so many different client to access only 1 endpoint per client:
cli := types.NewTaskClient(connection)
reply, err := cli.Execute(context.Background(), &args)
...
resultCli := types.NewResultClient(connection)
stream, err := resultCli.Listen(context.Background(), &args)

I was thinking why we just do one big service that group the task and result services?
Something like:
service Task {
  // Service
  rpc Listen (ListenTaskRequest) returns (stream ListenTaskReply) {}
  rpc Submit (SubmitTaskRequest) returns (SubmitTaskReply) {}

  // Application
  rpc Execute (ExecuteTaskRequest) returns (ExecuteTaskReply) {}
  rpc Result (ResultTaskRequest) returns (stream ResultTaskReply) {}
}

We could also organize into 1 gRPC service for the Services, and 1 gRPC service for the Application:
service Task {
  // Service
  rpc Listen (ListenTaskRequest) returns (stream ListenTaskReply) {}
  rpc Submit (SubmitTaskRequest) returns (SubmitTaskReply) {}
}
service Execution {
  // Application
  rpc Execute (ExecuteExecutionRequest) returns (ExecuteExecutionReply) {}
  rpc Listen (ListenExecutionRequest) returns (stream ListenExecutionReply) {}
}

@antho1404 What do you think? I would love to have separate API between application and service development. It will be much nicer for the docs and explanation.

In your exemples don't forget that the output is the type of output because a task can have multiple kind of outputs and data are the actual data of the output.


subscribe error

Hi Michal,
I failed to subscribe to the notification，Operating as follows：
> subscribe --filter-xpath /o-ran-file-management:file-download-event
ERROR
	type:     application
	tag:      operation-failed
	severity: error
	message:  np2srv_sr_event_notif_subscribe failed (sysrepo: Requested schema model is not known).

The server：
[INF]: Resolving unresolved data nodes and their constraints...
[INF]: All data nodes and constraints resolved.
[ERR]: Unable to stat data file for 'ietf-crypto-types': .
[WRN]: Attempt to subscribe to unknown 'ietf-crypto-types' module.
[ERR]: Unable to set socket directory permissions for '/var/run/sysrepo-subscriptions/ietf-crypto-types/'.
[ERR]: Cannot generate server socket path.
[ERR]: Cannot initialize Client Subscription Manager.
[ERR]: NULL value detected for subscription argument of cl_sm_subscription_cleanup
[ERR]: Error by initialization of the subscription in the client library.
[ERR]: np2srv_sr_event_notif_subscribe failed (sysrepo: Requested schema model is not known).
[INF]: Session 2: thread 1 event new RPC.
[INF]: Session 2: thread 1 event reply error.

I'm sure I've installed model o-ran-file-management：
srbn@ubtVM:/mnt/hgfs/Notepeer/o-ran-modules$ sudo sysrepoctl -l
Sysrepo schema directory: /etc/sysrepo/yang/
Sysrepo data directory:   /etc/sysrepo/data/
(Do not alter contents of these directories manually)

Module Name                | Revision   | Conformance | Data Owner          | Permissions | Submodules                    | Enabled Features
-----------------------------------------------------------------------------------------------------------------------------------------------
ietf-netconf-notifications | 2012-02-06 | Installed   | root:root           | 666         |                               |
ietf-netconf               | 2011-06-01 | Installed   | root:root           | 666         |                               | writable-running candidate rollback-on-error validate startup xpath
ietf-netconf-acm           | 2018-02-14 | Installed   | root:root           | 666         |                               |
nc-notifications           | 2008-07-14 | Installed   | root:root           | 666         |                               |
notifications              | 2008-07-14 | Installed   | root:root           | 666         |                               |
turing-machine             | 2013-12-27 | Installed   | root:root           | 666         |                               |
iana-if-type               | 2014-05-08 | Installed   |                     |             |                               |
ietf-interfaces            | 2014-05-08 | Installed   | root:root           | 666         |                               |
ietf-ip                    | 2014-06-16 | Installed   |                     |             |                               |
ietf-netconf-with-defaults | 2011-06-01 | Installed   |                     |             |                               |
ietf-netconf-monitoring    | 2010-10-04 | Installed   | root:root           | 666         |                               |
ietf-yang-library          | 2018-01-17 | Installed   | root:root           | 666         |                               |
ietf-datastores            | 2017-08-17 | Imported    |                     |             |                               |
ietf-yang-types            | 2013-07-15 | Installed   |                     |             |                               |
ietf-crypto-types          | 2019-04-29 | Installed   |                     |             |                               |
o-ran-file-management      | 2019-07-03 | Installed   | root:root           | 666         |                               |

The content of YANG model is ：
module o-ran-file-management {
  yang-version 1.1;
  namespace "urn:o-ran:file-management:1.0";
  prefix "o-ran-file-mgmt";

  import ietf-crypto-types {
    prefix "ct";
  }
  organization "O-RAN Alliance";

		/*......*/

  rpc file-download {
    description
      "Management plane triggered to generate the download file of O-RU.";
    input {
      uses file-path-grouping;
      uses credential-information;
    }
    output {
      uses output-status-grouping;
    }
  }

  notification file-download-event {
    uses file-path-grouping;
    uses output-status-grouping;
    description "";
  }
}

Note: I once subscribed successfully in the oven example and the the application is already running.
Would you do me a favor?
Regards,
Fizz

Hi,
so I assume the RPC file-download uses a grouping from ietf-crypto-types?
Regards,
Michal

Hi Michal,
Yes, part of it is.
Regards,
Fizz

Hi Michal,
I noticed something interesting, and I guess that's the problem.
The OVEN model was originally installed in SYSREPO, but the o-ran-file-management model and its dependency model were not installed.
At this point I can subscribe to the notification of the OVEN model:
`
srbn@ubtVM:~/workdocs/NetConfStudy/aboutNetConf/sysrepo/examples/plugins$ sudo netopeer2-cli

connect
Interactive SSH Authentication
Type your password:
Password:
subscribe --filter-xpath /oven:oven-ready
OK

Then, after I installed the o-ran-file-management model and its dependencies, I found that the subscription function was broken, and I couldn't subscribe to any notifications, including the notifications from the OVEN model that I had successfully subscribed to previously. Here's what happens when two subscriptions fail:：
srbn@ubtVM:~/workdocs/NetConfStudy/aboutNetConf/sysrepo/examples/plugins$ sudo netopeer2-cli

connect
Interactive SSH Authentication
Type your password:
Password:
subscribe --filter-xpath /o-ran-file-management:file-download-event
ERROR
type:     application
tag:      operation-failed
severity: error
message:  np2srv_sr_event_notif_subscribe failed (sysrepo: Requested schema model is not known).

subscribe --filter-xpath /oven:oven-ready
ERROR
type:     application
tag:      operation-failed
severity: error
message:  np2srv_sr_event_notif_subscribe failed (sysrepo: Requested schema model is not known).

[INF]: Resolving unresolved data nodes and their constraints...
[INF]: All data nodes and constraints resolved.
[ERR]: Unable to stat data file for 'ietf-crypto-types': .
[WRN]: Attempt to subscribe to unknown 'ietf-crypto-types' module.
[ERR]: Unable to set socket directory permissions for '/var/run/sysrepo-subscriptions/ietf-crypto-types/'.
[ERR]: Cannot generate server socket path.
[ERR]: Cannot initialize Client Subscription Manager.
[ERR]: NULL value detected for subscription argument of cl_sm_subscription_cleanup
[ERR]: Error by initialization of the subscription in the client library.
[ERR]: np2srv_sr_event_notif_subscribe failed (sysrepo: Requested schema model is not known).
[INF]: Session 3: thread 1 event new RPC.
[INF]: Session 3: thread 1 event reply error.
[INF]: Resolving unresolved data nodes and their constraints...
[INF]: All data nodes and constraints resolved.
[ERR]: Unable to stat data file for 'ietf-crypto-types': .
[WRN]: Attempt to subscribe to unknown 'ietf-crypto-types' module.
[ERR]: Unable to set socket directory permissions for '/var/run/sysrepo-subscriptions/ietf-crypto-types/'.
[ERR]: Cannot generate server socket path.
[ERR]: Cannot initialize Client Subscription Manager.
[ERR]: NULL value detected for subscription argument of cl_sm_subscription_cleanup
[ERR]: Error by initialization of the subscription in the client library.
[ERR]: np2srv_sr_event_notif_subscribe failed (sysrepo: Requested schema model is not known).
[INF]: Session 3: thread 0 event new RPC.
[INF]: Session 3: thread 0 event reply error.
`
The relevant model file is nearby. Please help me analyze it. I guess the problem comes from the model.
YANG.zip
Regards,
Fizz

Hi Fizz,
it seems the module was updated and it no longer even imports ietf-crypto-types (link). As this is a problem in the old sysrepo I would prefer not having to fix it so would it be a problem for you to use the module without the import? Or is that the old version?
Regards,
Michal


VSCode only connects to a localhost TCP host

The VSCode extension only connects to a TCP host on <IP_ADDRESS>. This works fine for most PHP setups. However, if using PHP in a docker container on Docker for Mac with a loopback, it cannot connect since the host IP is not <IP_ADDRESS>.
The language server supports talking to another TCP host, but there isn't a configuration option to do so in VSCode.
A php.tcpHost user setting should be added to support connecting to another TCP host.

+1, Not really too keen on installing PHP just so I can have Intellisense... I run my test web server through Docker as well, so it only makes sense?


Access the underlying buffer object associated with a texture.

Is your feature request related to a problem? Please describe.
I would like to access the buffer handle associated with a filament::Texture, so that I can use the cuda interop api and write directly to the buffer from the GPU.
Describe the solution you'd like
From the little investigation I've done, I can see that the derived FTexture class holds the HwTexture handle, and  the backend::Handle api seems to offer access to the underlying buffer handle. I'm wondering if there's a "clean" way for me to access that handle, without casting and including private headers? If this isn't supported/isn't planned, what is the best/safest work around I could use?
Thanks

By buffer object do you mean you want access to the OpenGL texture id? One of the reasons we don't expose things like this is to be able to transparently support multiple backends. Can you do it the other way around and use a shared GL context to give Filament the texture you want to share with CUDA? That we support (even though it's unfortunately GL specific).

Yes the id is what I'm after. Hm I wasn't aware of the other option, is there something in the samples similar to what you suggest?

Unfortunately no but what you're after is:
https://github.com/google/filament/blob/master/filament/include/filament/Engine.h#L185
It only works with EGL though, I don't know if you can use this on desktop :/
@pixelflinger any thoughts on this feature request?

(Note that a very similar feature request has been made from a team within Google.)

@romainguy That is unfortunate :(
As for the backend issue, I'm not very familiar with vulkan but from the examples I've seen, writing to a VkBuffer would require these steps:

Get the memory handle to the VkBuffer with a call to whatever gets returned by this vkGetInstanceProcAddr(instance, "vkGetMemoryFdKHR")
Import and map the memory with cuda cudaImportExternalMemory and cudaExternalMemoryGetMappedBuffer
Block vulkan using a sephamore
Execute CUDA kernel that writes to the VkBuffer using the memory handle from step 1
Unblock vulkan
unmap the buffer

So the only thing I really need from vulkan is that memory handle (I think) so that I can map it. Can I ask, what does the backend::Handle<HwTexture> store when the backend is Vulkan?
And with regards to metal, I'm clueless there...

HwTexture is a VulkanTexture which contains a VkImage, a VkImageView and a VkDeviceMemory. @prideout can say more about this since he wrote the Vulkan backend.

You would also need a device, command buffer, etc...  Long term I think we should support sharing in a more first-class way using platform-specific mechanisms (e.g. AHardwareBuffer), and in the short term expose a small OpenGL-specific mechanism.
@nitronoid would you be unblocked if we provided an OpenGL-only solution?

@prideout Yes an OGL solution would be perfect!
I was just speaking hypothetically about the Vulkan part out of interest, however having access to the OpenGL texture ID would be much more useful right now. Thanks!

I imagine the texture ID isn't enough and you also need access to the GL context. It's the kind of stuff I'd really like to avoid exposing.

@romainguy  Yes I would need that too. Is there any way to get the shared context working on desktop?

Actually, I only really need the context so that I can ensure it is the current context for my thread. Would it be possible to offer some sort of makeCurrent function so I can do that in a platform agnostic way?
Then I could just call makeCurrent, and get the now valid texure Id somehow.
I see that the private implementation of SwapChain has something similar, but requires the backend as a parameter.

So you are touching one the #1 reason why we use a separate thread for OpenGL: we do not want the app to ever be able to do anything that might modify the state of Filament's GL context (we do careful state tracking, etc.). That's also why we never expose APIs that let run user code on the driver thread.

I see, not sure where to go from here then. Unless you implement the mapping of the texture with the cudaInterop calls internally (which is a bit of a stretch), I'm not sure how I could do it without running on the driver thread :/
How would you prevent the user from running code on the driver thread when using that sharedContext?

The shared context doesn't let you run arbitrary thread in our driver (by driver I mean Filament's backend, not the GPU driver of your system). It does allow you to do a few naughty things (you could delete some of Filament's resources for instance) but it's somewhat limited. The best solution would be to look into the equivalent of EGL's shared context for WGL/XGL/etc. What platform are you targeting?

RHEL-7 desktop with Nvidia Quadro cards. I think that's GLX?

Yes, we use GLX there. It's apparently trivial to enable context sharing. We just need to pass the void* sharedContext you give us to glXCreateContext.
@prideout Any chance you could give this a try?

https://github.com/google/filament/blob/5b4ec6fd7d5b1ed1bef14f63c06eccddb6a55b72/filament/backend/src/opengl/PlatformGLX.cpp#L136-L137
Seems like it's already being passed? Unless I'm mistaken.

Look at that, apparently we already took care of it :) Give it a try and let us know how it goes then!

Great thanks for helping me out!
Final question: I'm guessing there's no way I can convert an OpenGL texture I create on the shared context with glGenTextures into a filament::Texture? That way I could keep using your nice API, and have the texture Id.

The way to make it work is to create a Stream with an external texture id (https://github.com/google/filament/blob/master/filament/include/filament/Stream.h). The stream can then be attached to a Texture. I believe you can ignore the comment about GL_TEXTURE_EXTERNAL_OES (@pixelflinger is the doc wrong here? Texture external should be for SurfaceTexture not shared textures right?).

Sorry for no updates, I've been handling another task. I will get back to this and test it soon if you don't mind leaving the issue open a little longer?

Take your time :)


Adding a workflow for CI on feature branches

This new Github action workflow would run CI/Sonar on all feature branches, but not generate APKs. Could easily add in APK generation if it's desired, just seems like overkill for now.

@Apisov while I don't think there's a limit I agree with you... SonarCube analysis on every push to every branch would be excessive and redundant as it will be covered in PRs, I'll take it out!


API and Router Certificate Transform/Report

Research how to take the Signing CA Cert and Key from an Openshift 3.x cluster and install it on an Openshift 4.x cluster

The logic will be something along the lines of

I think these crt/key combos are referenced in the master-config.yaml so it might be better to start there in the event the user has customized the certificate location, but that's a fairly minor detail to overcome.

Related to certs in OCP 4.x

ingress

https://docs.openshift.com/container-platform/4.1/authentication/certificates/replacing-default-ingress-certificate.html

service serving certificate

https://docs.openshift.com/container-platform/4.1/authentication/certificates/service-serving-certificate.html

api server

https://docs.openshift.com/container-platform/4.1/authentication/certificates/api-server.html

Router certificate support target is for 4.3


Frontend can retrieve regional information about a member of staff

Changes in this PR

Adds new middleware populateUserRegion, which populates the session with the user's probation region
Updates extractCallConfig and RestClient to use this information in the session to send a new header X-USER-REGION with each API call

Release checklist
As part of our continuous deployment strategy we must ensure that this work is
ready to be released at any point. Before merging to main we must first
confirm:
Pre merge checklist

[ ] Have all changes to any dependencies been deployed to both preprod and
production in a backwards compatible way? (This includes our API,
infrastructure, third-party integrations and any secrets or environment variables)
[ ] Has a required data migration been included in this change or been done in
advance?

Post merge checklist
Once we've merged we now need to release this through to production before
considering it done.
Find the build-and-deploy job in CircleCI
and work through the following steps:

[ ] Has the product manager asked to provide approval for this feature?

[ ] Has the product manager approved?

[ ] Manually approve release to preprod
[ ] Verify change released to preprod
[ ] Manually approve release to prod
[ ] Verify change released to production

Should a release fail at any step, you as the author should now lead the work to
fix it as soon as possible.

 Frontend can retrieve regional information about a member of staff


Engine image's dependents won't be deleted automatically when using foreground deletion policy

Kubernetes version v.1.13
Set instance manager ownerReferences to engine image with blockOwnerDeletion = true.
Delete engine image with deletion policy foregroundDeletion.
Expect result:
Engine image deletion timestamp is set with the finalizer foregroundDeletion.
Then the instance managers will be deleted.
Engine image won't be removed until all related instance managers are gone.
Actual result:
Engine image deletion timestamp is set with the finalizer foregroundDeletion.
No changes for the instance managers.
Engine image gets stuck there.
kl get engineimage ei-a126797d -o yaml
apiVersion: longhorn.rancher.io/v1alpha1
kind: EngineImage
metadata:
  creationTimestamp: "2019-09-20T18:46:17Z"
  deletionGracePeriodSeconds: 0
  deletionTimestamp: "2019-09-20T18:46:45Z"
  finalizers:
  - foregroundDeletion
  generation: 6
  name: ei-a126797d
  namespace: longhorn-system
  resourceVersion: "7602598"
  selfLink: /apis/longhorn.rancher.io/v1alpha1/namespaces/longhorn-system/engineimages/ei-a126797d
  uid: ee3f0997-dbd6-11e9-a605-f23c91d7d453
spec:
  image: longhornio/longhorn-test:upgrade-test.2-2.2-2.1-1
  ownerID: shuo-longhorn-cluster-0-node-3
status:
  buildDate: "2019-09-04T18:46:23+00:00"
  cliAPIMinVersion: 2
  cliAPIVersion: 2
  controllerAPIMinVersion: 2
  controllerAPIVersion: 2
  dataFormatMinVersion: 1
  dataFormatVersion: 1
  gitCommit: 4408836e54efca53db1bbe7ed5d859eabc2488e8
  noRefSince: "2019-09-20T18:46:32Z"
  refCount: 0
  state: ready
  version: "4408836"

kl get instancemanager instance-manager-e-bc3ae086 -o yaml
apiVersion: longhorn.rancher.io/v1alpha1
kind: InstanceManager
metadata:
  creationTimestamp: "2019-09-20T18:46:32Z"
  finalizers:
  - longhorn.rancher.io
  generation: 6
  labels:
    longhorn.io/component: instance-manager
    longhorn.io/engine-image: ei-a126797d
    longhorn.io/instance-manager-type: engine
    longhorn.io/node: shuo-longhorn-cluster-0-node-2
  name: instance-manager-e-bc3ae086
  namespace: longhorn-system
  ownerReferences:
  - apiVersion: longhorn.rancher.io/v1alpha1
    blockOwnerDeletion: true
    controller: true
    kind: longhorn.rancher.io/v1alpha1, Kind=EngineImage
    name: ei-a126797d
    uid: ee3f0997-dbd6-11e9-a605-f23c91d7d453
  resourceVersion: "7602568"
  selfLink: /apis/longhorn.rancher.io/v1alpha1/namespaces/longhorn-system/instancemanagers/instance-manager-e-bc3ae086
  uid: f6e357cd-dbd6-11e9-a605-f23c91d7d453
spec:
  engineImage: ei-a126797d
  nodeID: shuo-longhorn-cluster-0-node-2
  ownerID: shuo-longhorn-cluster-0-node-2
  type: engine
status:
  currentState: running
  instances: {}
  ip: <IP_ADDRESS>
  nodeBootID: 5298c8b1-2852-4fec-b509-00bad113d3a7

It's caused by the wrong ownerReferences setting: the kind field should be EngineImage rather than longhorn.rancher.io/v1alpha1, Kind=EngineImage

Validation passed for issue #822


onChanged null check

The onChanged function is required but not enforced.
Please mark it as mandatory of null check if before using it when clearing the input field (X tapped once with text present)

I've pushed to fix this issue, please check it.
I will close this issue, feel free to re-open this issue if you still facing this issue
Thank you