OpenAPI operation ids are incomprehensible

Describe the bug
The openapi schema downloaded from the swagger-ui contains for each operation an operationId. For example the operationId for GET /api/rules/{id} is get and for GET /api/requests/{id} is get_1. This behaviour can be seen for all operations in the openapi description.
To Reproduce
Download the openapi json from the swagger-ui
Expected behavior
I would expected meaningfull names. E.g. for GET /api/rules/{id} maybe getRules and for GET /api/requests/{id} getRequests.

Hi,
the openapi schema is going to change/improve with version 5.1.2.
Regarding the operation ids: I am not sure this is something we can easily change. The operation id can be anything e.g. an UUID as long as its unique in the context of the schema.
Also from a quick look in the spring-docs issues it seems the operation id is not guaranteed to be stable from generation to generation.
Maybe you could write a quick parser creating a unique name based on the method and path?


Restructure DS website's information architecture

This PR restructures the site's navigation and moves every page into its new category. There's no longer a nav in the header, it's all in the sidebar now. It's also editable via the CMS to let users place pages exactly where they want instead of everything getting listed alphabetically.
We used to have a netlify cms collection for every category but IMHO that's silly so I deleted them all and created a new collection called pages that houses every page.
Changes

Removes main nav and moves all nav elements to left-hand sidebar.
Adds a side nav data file that is editable via the CMS and generates the left-hand sidebar.
Adds a netlify cms collection called "Navigation" for the aforementioned new side nav data file.
Removes the four netlify cms collections that were identical and replaces them with a single one called "Pages". Accordingly, I moved all website pages into a folder called pages/.
Moves special pages like the homepage and the help page to a new collection called "Special pages".
Updated the "Updating this website" page with instructions on how to update stuff.
Removed the "Add new page button" and replaced it with a link in the footer.
Updates tests and Jekyll/netlify metadata files as appropriate

Testing

yarn test:browser should pass
Click around the PR preview link below and see how you feel about the new all-inclusive side nav.
Go to the "Updating this website" page by clicking the link in the footer and peruse the instructions for creating new pages and editing the side nav.

Screenshots

new side nav
editing side nav in CMS
new footer links

Notes

Todos

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the CFPB development guidelines
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Browsers

[ ] Chrome on desktop
[ ] Firefox
[ ] Safari on macOS
[ ] Edge
[ ] Internet Explorer 9, 10, and 11
[ ] Safari on iOS
[ ] Chrome on Android

Accessibility

[ ] Keyboard friendly
[ ] Screen reader friendly

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Flexible from small to large screens
[ ] No linting errors or warnings
[ ] JavaScript tests are passing

I moved all the pages to a new folder so before merging this PR I'll manually remediate recent content conflicts.

* What if we put the sidenav on all pages, including the homepage? If not, why not?
* What if when you hovered over the sidenav it showed a little pencil icon like the edit this page buttton, and when you clicked that icon it brought you to the edit sidenav page?

🤯 🤯 🤯

Thanks @anselmbradford. I fixed things and added the side nav to the homepage. Adding an edit button to the side nav is a great idea but beyond the scope of this PR.


Review of initial work.

This is all in main already, so I've made a new empty branch from the initial commit.
I'd just like to get your take on any java-tricks I might have missed, style issues, etc.

First comment, as always, has little to do with the code and more to do with reviewer experience: too many changes, not enough atomic work.  Avoid checking in external works: if libraries or large data files (anything not for review) must be included, please always put them all into a single commit at the start of the branch so the reviewer can ignore it (fixup will help with this).

First comment, as always, has little to do with the code and more to do with reviewer experience: too many changes, not enough atomic work. Avoid checking in external works: if libraries or large data files (anything not for review) must be included, please always put them all into a single commit at the start of the branch so the reviewer can ignore it (fixup will help with this). This is the start of making a commit about a single purpose of change. The alternative approach is to make sure every commit is sufficiently complete (all and only relevant changes for what is described in the message without using the word AND) such that it can be easily reviewed/understood in isolation.

I did not know about fixup and will use it judiciously from now on! Thanks!


Euler Max and Euler SMEA Dy Negative cannot be used in v-prediction models

These two samplers produce a lot of noise when in use

I've tried a lot and these two samplers don't work. All other samplers can work on the V-prediction model. Please fix them if you can, thank you very much

I've noticed V-Pred models are more sensitive to the sampler/scheduler combo than EPS models are. Euler SMEA Dy Negative is already pretty touchy as is. Looking at your screenshot, I think when Scheduler type is Automatic it uses the Uniform sampler. I ran a couple XYZ grids to compare schedulers and as you can see, some work, some don't. That's just unfortunately how it is. Pick the one that looks the best and use that one. Additionally, you can consider grabbing this extension:
https://github.com/DenOfEquity/webUI_ExtraSchedulers
That extension is where the last 5 schedulers come from in the XYZ grid. It also adds some CFG++ samplers you can try and see if they work better.


Move custom javascript to js file

This should be a better way to do this, and also help with a require.js error on "Error: Mismatched anonymous define() module:" (https://requirejs.org/docs/errors.html#mismatch).
We had that since a recent release of nbsphinx loading require.js

I'm going to close this one as I think it's now out of date and we have a different setup for linking the custom JS, but please re-open or make a new PR if I am wrong!


Dev build

Implementation for #239 😄
In summary,

gulp: starts dev server, watch & do compilations to .dev folder
in master:
gulp build: does compilation, files are generated in the respective folders
gulp server: starts server in root to view compiled files after master build

(I removed compiled files of Amaro filter which is currently unavailable. Also did reset for Kelvin filter is_done status, it had got reset in one of the previous commits. Sorry to keep it in the same PR.)

Such a good update -- I think a few files got in there accidentally!

Hi @una,
I have reverted that commit (c38b629) from this PR. (since fd02556fac16f0571593a587dea9a29f54036da5 already have added missing Kelvin).
Now it is clean :)

Yay! Awesome!


improving precision of event rasters

The method of generating the rasters strikes me as incredibly imprecise. Please correct me if my understanding of the algorithm is wrong.
We create a single bins vector that spans the spiketimes, then...
# bin the spikes into time bins
spike_counts = np.histogram(self.spiketimes, bins)[0]

# bin the events into time bins
event_counts = np.histogram(selected_events, bins)[0]
event_bins = np.where(event_counts > 0)[0]

raster = np.array([(spike_counts[(i+window[0]/binsize): \
                                 (i+window[1]/binsize+1)]) \
                   for i in event_bins])

Spikes AND events are binned in this vector and the raster is built from the counts aligned to the bins that contain events.
This approach means that any spike that lands in the same bin as the event is counted in that bin, no matter whether it occurred before the event of after.
Further, if events are not regular, then sometimes the event will happen early in the bin and the bin will consist mostly of spikes that happen after the event and other times, visa versa.
I expected that a unique bins vector would be constructed for each event, spanning the window so that spikes were clearly assigned before or after the event.

That seems, right: bins should be constructed within a loop for each event. This will be bad for large bin sizes. Let me check with @hugoguh


Add blog post for sprint 123

Signed-off-by: Rubhan Azeem<EMAIL_ADDRESS>DO NOT MERGE: We have to deploy all the changes on the reference server before merging this PR

I pushed to fix some small typos.

With the renaming of Overview to Conversation, we will have to redo the screenshots where this is visible.

Nice blog post!

Screenshots updated to include the renaming of the Overview tab to Conversation.


[Usage]: Number of requests currently in the queue

Your current environment
The output of `python collect_env.py`

How would you like to use vllm
I am running an online inference server via the code:
vllm serve "daryl149/llama-2-7b-chat-hf" --max-model-len 2048  for which I am sending request through a load generator. I want to know if it is possible to find out the number of requests currently in the queue or alternatively number of requests currently being processed in a batch (assume batch size=248 and number of batches=1).
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

You can query the /metrics endpoint for this information.
https://docs.vllm.ai/en/latest/serving/metrics.html


JFXAutocompletion popup css

i am using jfxAutocompletionpopup to a textField.
i needed css for jfxAutocompletionpopup i am able to change fixed cell size.

you can change it using the following css:
.autocomplete-list{
    -fx-fixed-cell-size: 32;
}

Regards,


Initialize qvel when resetting for control_mode = "pd_joint_pos_vel"

Currently envs with control_mode = "pd_joint_pos_vel" doesn't work.
This should fix this.

thanks! lgtm


security issue -SQL Injection

Hello,
We use the lib in our application.
Some company did security review our application. And it found sql injection issue in code of ormlite-android.
Part information from report:
Explanation
When carried out against a client-side database, SQL injection can result in the bypassing of
authentication/authorization and data corruption or exposure. The impact of a specific SQL injection
vulnerability relies on the manner in which the target database is used by the application. Client-side
SQL injection is often rated much lower-risk than server-side SQL injection but still may have a
profound impact on multi-user applications.
Any instance of rawQuery or execSQL that appeared to use format strings or concatentation was
flagged by this check.
com/j256/ormlite/android/AndroidDatabaseConnection.java : Cursor cursor =
this.db.rawQuery("SELECT DISTINCT tbl_name FROM sqlite_master WHERE tbl_name = '" + string +
"'", null);
Execution
Review the application for any locations where dynamic text is used in SQL statements via format
strings.
Recommendation
Use parameterized queries.
References
OWASP Mobile Top 10 2012-M4 Client Side Injection:
https://www.owasp.org/index.php/Mobile_Top_10_2012-M4_Client_Side_Injection
Could you fix the issue ?

@j256 Could you check the issue ?

you are just using it wrong. sure there can be sql injection if you just concatenate strings. just parameterize your arguments and you are fine, ormlite already offers this...

Hi @daberni,
The issue was found by static analysis tool.
So the issue is not issue how to use the lib. it is just potential security issue which found static tool in the source code of the orm-library.
And it will be good if it can be fix.

Yeah it is in the ORMLite code not from his use @daberni .  I've got a fix that I need to publish.

Oh, I am really sorry @psedoykin. Then I completely misinterpreted your issue :pensive:

@j256 Great news, thank you! Could you advise please when we can expect an updated build?

is this issues already fixed?

Fixed.  See https://github.com/j256/ormlite-android/commit/b3149efe0309dc591f5df0b3d0e612f34e427eee


Office-Add-in-NodeJS-SSO deployed in appservice not working (production scenario)

We created a powerpoint web addin node sso based on the below articles
https://github.com/OfficeDev/Office-Add-in-samples/tree/main/Samples/auth/Office-Add-in-NodeJS-SSO
https://learn.microsoft.com/en-us/office/dev/add-ins/develop/create-sso-office-add-ins-nodejs
It works fine on localhost. (ran project from command prompt -> npm start to run the project)
So for production we created an app service and deployed the same powerpoint addin which works well in localhost by following the comments mentioned in the below article
https://github.com/OfficeDev/Office-Add-in-NodeJS-SSO/issues/61#issuecomment-821517972
we sideloadeded the addin in powerpoint web and we are getting the below issue and the addin is not working

get 404 when we try to access /home/index ( )

package.json is like below
{
"name": "ssoapp",
"version": "0.0.0",
"private": true,
"scripts": {
"install-dev-certs": "office-addin-dev-certs install",
"start": "node ./bin/www"
},
"dependencies": {
"@types/jquery": "^3.5.5",
"@types/office-js": "^1.0.264",
"cookie-parser": "~1.4.5",
"debug": "~4.3.1",
"dotenv": "^9.0.2",
"express": "~4.17.1",
"form-urlencoded": "^6.0.3",
"http-errors": "~1.8.0",
"jquery": "^3.6.0",
"jshint": "^2.12.0",
"jsonwebtoken": "^8.5.1",
"morgan": "~1.10.0",
"node-fetch": "^2.6.7",
"pug": "^3.0.2",
"@azure/msal-node": "1.8.0",
"jwks-rsa": "^2.0.5",
"uuid": "^8.3.1"
},
"devDependencies": {
"office-addin-dev-certs": "1.5.18"
}
}
Can you please help out to fix this issue. Please reach out to me if you need anymore information. Thanks

@davidchesnut can you please help here.

Hi @anantharengan, thanks for letting us know about this issue. I'll take a look. In the meantime, if you are seeing a 404, you might check to be sure the Node.js server successfully started without any errors on the server after deployment.

Hi @anantharengan, thanks for letting us know about this issue. I'll take a look. In the meantime, if you are seeing a 404, you might check to be sure the Node.js server successfully started without any errors on the server after deployment.

Thanks @davidchesnut  for the quick response. Please find the details below
My powerpoint addin structure deployed in app service is shown below
structure (1/2)

structure (2/2)

my configuration entries in app service (note : added port no as 443 (highlighted in yellow below))

and finally when i run 'node www' from bin folder (ran directly from kudu tools in app service)

Is there any other way to check to make sure Node.js server is successfully started without any errors on the server after deployment. If so kindly share it.
Kindly help us resolve this issue.

It looks like it doesn't like something in the www file at the indicated line numbers. Did you update the www file to remove office-addin-dev-certs and use server certs?

It looks like it doesn't like something in the www file at the indicated line numbers. Did you update the www file to remove office-addin-dev-certs and use server certs?

Thanks @davidchesnut  for the reply. I saw the www file code seems good to me. sharing the entire www file contents below kindly go through it and provide your suggestions. Thanks in advance
#!/usr/bin/env node
/**

Module dependencies.
*/

var app = require('../app');
var debug = require('debug')('ssoapp:server');
var devCerts = require('office-addin-dev-certs');
var https = require('https');
var fs = require('fs');
/**

Get port from environment and store in Express.
*/

var port = normalizePort(process.env.PORT || '44355');
app.set('port', port);
/**

Create HTTP server.
*/
startServer(app, port);

/**

Normalize a port into a number, string, or false.
*/

function normalizePort(val) {
var port = parseInt(val, 10);
if (isNaN(port)) {
// named pipe
return val;
}
if (port >= 0) {
// port number
return port;
}
return false;
}
/**

Event listener for HTTP server "error" event.
*/

function onError(error) {
if (error.syscall !== 'listen') {
throw error;
}
var bind = typeof port === 'string'
? 'Pipe ' + port
: 'Port ' + port;
// handle specific listen errors with friendly messages
switch (error.code) {
case 'EACCES':
console.error(bind + ' requires elevated privileges');
process.exit(1);
break;
case 'EADDRINUSE':
console.error(bind + ' is already in use');
process.exit(1);
break;
default:
throw error;
}
}
async function startServer(app, port) {
if (process.env.NODE_ENV === 'development') {
const options = await devCerts.getHttpsServerOptions();
https.createServer(options, app).listen(port, () => console.log(Server running on ${port}));
}
else {
app.listen(process.env.port || 1337, () => console.log(Server listening on port  ${port}));
}
}

Yep, you definitely want to remove the office-addin-dev-certs package from this file. It won't work. I think Azure generates certs for your deployment that you want to use instead.

Yep, you definitely want to remove the office-addin-dev-certs package from this file. It won't work. I think Azure generates certs for your deployment that you want to use instead.

Hi @davidchesnut thanks for the reply. As per the below piece of code
async function startServer(app, port) {
if (process.env.NODE_ENV === 'development') {
const options = await devCerts.getHttpsServerOptions();
https.createServer(options, app).listen(port, () => console.log(Server running on ${port}));
}
else {
app.listen(process.env.port || 1337, () => console.log(Server listening on port ${port}));
}
}
The line highlighted in bold in above code will only execute when the environment is 'production' .  So we are not using any office-addin-dev-certs package when the environment is 'production'.
FYI. We are authenticating using client id and client secret by following the below article
https://github.com/OfficeDev/Office-Add-in-samples/tree/main/Samples/auth/Office-Add-in-NodeJS-SSO
https://learn.microsoft.com/en-us/office/dev/add-ins/develop/create-sso-office-add-ins-nodejs
In your reply you have mentioned like 'I think Azure generates certs for your deployment that you want to use instead.' i do not understand what you mean by that. Can you please elaborate more on this.?
Thanks

@davidchesnut Can you please help out here

Yep, you definitely want to remove the office-addin-dev-certs package from this file. It won't work. I think Azure generates certs for your deployment that you want to use instead.

Hi @davidchesnut thanks for the reply. As per the below piece of code
async function startServer(app, port) { if (process.env.NODE_ENV === 'development') { const options = await devCerts.getHttpsServerOptions(); https.createServer(options, app).listen(port, () => console.log(Server running on ${port})); } else { app.listen(process.env.port || 1337, () => console.log(Server listening on port ${port})); } }
The line highlighted in bold in above code will only execute when the environment is 'production' . So we are not using any office-addin-dev-certs package when the environment is 'production'.
FYI. We are authenticating using client id and client secret by following the below article https://github.com/OfficeDev/Office-Add-in-samples/tree/main/Samples/auth/Office-Add-in-NodeJS-SSO https://learn.microsoft.com/en-us/office/dev/add-ins/develop/create-sso-office-add-ins-nodejs
In your reply you have mentioned like 'I think Azure generates certs for your deployment that you want to use instead.' i do not understand what you mean by that. Can you please elaborate more on this.?
Thanks

@davidchesnut can you please help

Doesn't the require('office-addin-dev-certs') pull in the code from office-addin-dev-certs? The package code is built to run in a Windows or Mac environment, so I don't know if it will actually load correctly. Although I have not tested that specific scenario.
I checked the www file that I used, and I commented out that require line. Otherwise looks the same as yours.
Sorry I was referring to the SSL certificate, not the client secret. I think that's automatically configured when you create the app service.

Hi @davidchesnut thanks for the reply. I have removed require('office-addin-dev-certs') and also commented the lines using them. Still get the same issue (tested in kudu for the app service).

Can there be any other reason for this or can i share  the codebase for you to check out. Kindly help.

Doesn't the require('office-addin-dev-certs') pull in the code from office-addin-dev-certs? The package code is built to run in a Windows or Mac environment, so I don't know if it will actually load correctly. Although I have not tested that specific scenario.
I checked the www file that I used, and I commented out that require line. Otherwise looks the same as yours.
Sorry I was referring to the SSL certificate, not the client secret. I think that's automatically configured when you create the app service.

Hi @davidchesnut thanks for the reply. I have removed require('office-addin-dev-certs') and also commented the lines using them. Still get the same issue (tested in kudu for the app service).

Since the app is listening to port no : 443. Do we need to configure anything in app service or azure add app for it to work.
Can there be any other reason for this?. Kindly help.

@davidchesnut Can you please share your suggestion or help for this.

Hi @anantharengan,
I'm setting up a test so that I can compare and contrast your project with something that I know works.  I took the Node JS SSO sample and configured an app registration. Then deployed it using VS Code to an Azure app service. This is working and these are the settings I used (just to see if we are working with the same configuration.
app service config:

Node 16 LTS
Windows deployment (I see you are using this already)

VS Code project change:

I moved all folders and files under a 'public' folder, then configured the public folder to deploy. Updated the settings.json file in .vscode folder to include "appService.deploySubpath": "public".

On the app service, I added the following app settings (no longer using the .ENV right now):

SCM_DO_BUILD_DURING_DEPLOYMENT = true
CLIENT_ID = yourclientid
CLIENT_SECRET= yourclientsecret
DIRECTORY_ID = yourdirectoryid

Now when I deploy, it automatically runs npm install, builds, and starts the server. No need to attempt from the console.
Finally I generated a new www file (as I was having some issues different from yours) and used the following. This may work for you:
#!/usr/bin/env node

/**
 * Module dependencies.
 */

var app = require('../app');
//var debug = require('debug')('myexpressapp:server');
var http = require('http');

/**
 * Get port from environment and store in Express.
 */

var port = normalizePort(process.env.PORT || '3000');
app.set('port', port);

/**
 * Create HTTP server.
 */

var server = http.createServer(app);

/**
 * Listen on provided port, on all network interfaces.
 */

server.listen(port);
server.on('error', onError);
server.on('listening', onListening);

/**
 * Normalize a port into a number, string, or false.
 */

function normalizePort(val) {
  var port = parseInt(val, 10);

  if (isNaN(port)) {
    // named pipe
    return val;
  }

  if (port >= 0) {
    // port number
    return port;
  }

  return false;
}

/**
 * Event listener for HTTP server "error" event.
 */

function onError(error) {
  if (error.syscall !== 'listen') {
    throw error;
  }

  var bind = typeof port === 'string'
    ? 'Pipe ' + port
    : 'Port ' + port;

  // handle specific listen errors with friendly messages
  switch (error.code) {
    case 'EACCES':
      console.error(bind + ' requires elevated privileges');
      process.exit(1);
      break;
    case 'EADDRINUSE':
      console.error(bind + ' is already in use');
      process.exit(1);
      break;
    default:
      throw error;
  }
}

/**
 * Event listener for HTTP server "listening" event.
 */

function onListening() {
  var addr = server.address();
  var bind = typeof addr === 'string'
    ? 'pipe ' + addr
    : 'port ' + addr.port;
  //debug('Listening on ' + bind);
  console.log('Listening on ' + bind);
}

Finally, when you deploy you can stream the logs using the info in this article. This will inform if there are any errors in the install and build during deployment.
Let me know if this helps. Thanks!

Hi @anantharengan,
I'm setting up a test so that I can compare and contrast your project with something that I know works. I took the Node JS SSO sample and configured an app registration. Then deployed it using VS Code to an Azure app service. This is working and these are the settings I used (just to see if we are working with the same configuration.
app service config:

Node 16 LTS
Windows deployment (I see you are using this already)

VS Code project change:

I moved all folders and files under a 'public' folder, then configured the public folder to deploy. Updated the settings.json file in .vscode folder to include "appService.deploySubpath": "public".

On the app service, I added the following app settings (no longer using the .ENV right now):

SCM_DO_BUILD_DURING_DEPLOYMENT = true
CLIENT_ID = yourclientid
CLIENT_SECRET= yourclientsecret
DIRECTORY_ID = yourdirectoryid
SERVER_SOURCE = your Azure app service URL

Now when I deploy, it automatically runs npm install, builds, and starts the server. No need to attempt from the console.
Finally I generated a new www file (as I was having some issues different from yours) and used the following. This may work for you:
#!/usr/bin/env node

/**
 * Module dependencies.
 */

var app = require('../app');
//var debug = require('debug')('myexpressapp:server');
var http = require('http');

/**
 * Get port from environment and store in Express.
 */

var port = normalizePort(process.env.PORT || '3000');
app.set('port', port);

/**
 * Create HTTP server.
 */

var server = http.createServer(app);

/**
 * Listen on provided port, on all network interfaces.
 */

server.listen(port);
server.on('error', onError);
server.on('listening', onListening);

/**
 * Normalize a port into a number, string, or false.
 */

function normalizePort(val) {
  var port = parseInt(val, 10);

  if (isNaN(port)) {
    // named pipe
    return val;
  }

  if (port >= 0) {
    // port number
    return port;
  }

  return false;
}

/**
 * Event listener for HTTP server "error" event.
 */

function onError(error) {
  if (error.syscall !== 'listen') {
    throw error;
  }

  var bind = typeof port === 'string'
    ? 'Pipe ' + port
    : 'Port ' + port;

  // handle specific listen errors with friendly messages
  switch (error.code) {
    case 'EACCES':
      console.error(bind + ' requires elevated privileges');
      process.exit(1);
      break;
    case 'EADDRINUSE':
      console.error(bind + ' is already in use');
      process.exit(1);
      break;
    default:
      throw error;
  }
}

/**
 * Event listener for HTTP server "listening" event.
 */

function onListening() {
  var addr = server.address();
  var bind = typeof addr === 'string'
    ? 'pipe ' + addr
    : 'port ' + addr.port;
  //debug('Listening on ' + bind);
  console.log('Listening on ' + bind);
}

Finally, when you deploy you can stream the logs using the info in this article. This will inform if there are any errors in the install and build during deployment.
Let me know if this helps. Thanks!

Thanks @davidchesnut for the detailed reply i followed all the things as mentioned. But still getting the same error on the logs (i enabled it by having iisnode.yml). Please find the screenshots below

and screenshot one of the entry

It is still throwing permission issue. Do we need to enable reverse proxies for port 443 or anything on those lines. Can you please help.

Hi @anantharengan,
I'm setting up a test so that I can compare and contrast your project with something that I know works. I took the Node JS SSO sample and configured an app registration. Then deployed it using VS Code to an Azure app service. This is working and these are the settings I used (just to see if we are working with the same configuration.
app service config:

Node 16 LTS
Windows deployment (I see you are using this already)

VS Code project change:

I moved all folders and files under a 'public' folder, then configured the public folder to deploy. Updated the settings.json file in .vscode folder to include "appService.deploySubpath": "public".

On the app service, I added the following app settings (no longer using the .ENV right now):

SCM_DO_BUILD_DURING_DEPLOYMENT = true
CLIENT_ID = yourclientid
CLIENT_SECRET= yourclientsecret
DIRECTORY_ID = yourdirectoryid
SERVER_SOURCE = your Azure app service URL

Now when I deploy, it automatically runs npm install, builds, and starts the server. No need to attempt from the console.
Finally I generated a new www file (as I was having some issues different from yours) and used the following. This may work for you:
#!/usr/bin/env node

/**
 * Module dependencies.
 */

var app = require('../app');
//var debug = require('debug')('myexpressapp:server');
var http = require('http');

/**
 * Get port from environment and store in Express.
 */

var port = normalizePort(process.env.PORT || '3000');
app.set('port', port);

/**
 * Create HTTP server.
 */

var server = http.createServer(app);

/**
 * Listen on provided port, on all network interfaces.
 */

server.listen(port);
server.on('error', onError);
server.on('listening', onListening);

/**
 * Normalize a port into a number, string, or false.
 */

function normalizePort(val) {
  var port = parseInt(val, 10);

  if (isNaN(port)) {
    // named pipe
    return val;
  }

  if (port >= 0) {
    // port number
    return port;
  }

  return false;
}

/**
 * Event listener for HTTP server "error" event.
 */

function onError(error) {
  if (error.syscall !== 'listen') {
    throw error;
  }

  var bind = typeof port === 'string'
    ? 'Pipe ' + port
    : 'Port ' + port;

  // handle specific listen errors with friendly messages
  switch (error.code) {
    case 'EACCES':
      console.error(bind + ' requires elevated privileges');
      process.exit(1);
      break;
    case 'EADDRINUSE':
      console.error(bind + ' is already in use');
      process.exit(1);
      break;
    default:
      throw error;
  }
}

/**
 * Event listener for HTTP server "listening" event.
 */

function onListening() {
  var addr = server.address();
  var bind = typeof addr === 'string'
    ? 'pipe ' + addr
    : 'port ' + addr.port;
  //debug('Listening on ' + bind);
  console.log('Listening on ' + bind);
}

Finally, when you deploy you can stream the logs using the info in this article. This will inform if there are any errors in the install and build during deployment.
Let me know if this helps. Thanks!

Thanks @davidchesnut for the detailed reply i followed all the things as mentioned. But still getting the same error on the logs (i enabled it by having iisnode.yml). Please find the screenshots below

and screenshot one of the entry

It is still throwing permission issue. Do we need to enable reverse proxies for port 443 or anything on those lines. Can you please help.

hi @davidchesnut can you please help on this

My guess is there really is some permission problem, but I'm not exactly sure how to debug to isolate that. I'm checking with some folks on the Azure side.

Hi @anantharengan,
Doing some more testing I actually reproduced this error. I found that port 443 and 8080 are reserved by the Azure app service. You can't use them in your project. Instead you need to be sure to use 'process.env.PORT' which Azure will pass to your project. the 443 port gets mapped to your project through process.env.PORT.
Be sure you aren't specifically setting the port to 443 anywhere in your code. Maybe do a global search. Also make sure you didn't specify a port variable in the configuration settings, or the .ENV file.
Cheers,
David

Hi @anantharengan,
Doing some more testing I actually reproduced this error. I found that port 443 and 8080 are reserved by the Azure app service. You can't use them in your project. Instead you need to be sure to use 'process.env.PORT' which Azure will pass to your project. the 443 port gets mapped to your project through process.env.PORT.
Be sure you aren't specifically setting the port to 443 anywhere in your code. Maybe do a global search. Also make sure you didn't specify a port variable in the configuration settings, or the .ENV file.
Cheers, David

Thanks David will check your suggestion and try it out and update you soon


Bug in sorting related to initialSortBy

Issue Type

[x] Bug

Specs
What version are you using?
2.15.0
What browser?
Chrome 69.0.3497.100, Safari 12.0
Expected Behavior
Sort changes when clicking on column headers
Actual Behavior
Sort does not change
Steps to Reproduce the Problem
https://jsfiddle.net/gira82/cq4etkp6/1/
Basically I need to have an array of ids related to selected rows.
So I have a selectionChanged method that reduce the rows selected to an array of ids.
In the fiddle I attached there are three things that make the sort working:

Do not render selectedIds array in the template (comment out line 2 of html)
Do not update selectedIds in the vue model (comment out line 99 of js)
Remove the option "initialSortBy: {field: 'name', type: 'asc'}" from the options.

I think there is something not working properly in the sorting code.
Thank you

you need to take care of the 0 length case when sort is changed:
This will make your jsfiddle work.
selectionChanged(params) {
    	/* console.log(params) */;
      if (!params.selectedRows.length) {
      	this.selectedIds = [];
        return;
      }
      const selectedIds = params.selectedRows.reduce((acc, row) => {
        acc.push(row.id)
        return acc
      }, [])
      console.log(selectedIds);
      this.selectedIds = selectedIds
    }

closing.

Sorry, but the .reduce correctly return an empty array when params.selectedRows is empty.
In my humble opinion, the point is not this, and the sort is still not working in the updated fiddle: https://jsfiddle.net/gira82/cq4etkp6/18/
I carefully debug the problem before I opened the issue, and I cannot see any problem in this simple code.
It's curious that removing initialSortBy all starts to work.
I can make more tests if needed.

apologies for the initial haste, I'll re-open this. However if i comment out this.selectedIds assignment... everything seems to be working fine. I'll debug it more.

looks like if I don't print out {{ selectedIds }} , everything is working as well. strange.

Yes, as I wrote at the beginning, three things will make the script work:

Do not render (or use in any way) selectedIds array in the template
Do not update selectedIds array at the end of the method
Remove the option "initialSortBy: {field: 'name', type: 'asc'}" from the options.

1 and 2 sound  inexplicable to me (after about 2 years of Vue developing)
If you look at console, it seems that the method is triggered three times every time you change sorting.
Hope this will help.

so just published 2.15.2 and seems to have passed all my tests... I'm going to keep this open for a bit so we can test with your jsfiddle

ok tested with your fiddle... seems to be working well now. Thanks for reporting this!

Thank you to you for so fast fix! 👍


Map Suddenly Stopped Loading Tiles

Checklist

[x] I've looked at the documentation to make sure the behavior isn't documented and expected.
[ ] I'm sure this is an issue with Leaflet, not with my app or other dependencies (Angular, Cordova, React, etc.).
[x] I've searched through the current issues to make sure this hasn't been reported yet.
[x] I agree to follow the Code of Conduct that this project adheres to.

Steps to reproduce
Context:
We've had leaflet.js in place for a long time now without issue. Recently it stopped working as expected.
Steps to replicate:

go to: https://www.accorplus.com/offers/christmas-feast-collage-all-day-dining-pullman-jakarta-central-park-dec-24/
observe that map halfway down the page

Expected behavior
Map tiles load
Current behavior
The container loads along with custom markers but the tiles do not. Zoom etc all appear to work.
There is a console error with map.js, which I think is part of how leaflet.js works.
Uncaught (in promise) TypeError: Spread syntax requires ...iterable[Symbol.iterator] to be a function
    at pwa (map.js:37:176)
    at Kxa.changed (map.js:107:360)
    at al (js?key=AIzaSyCs_oAJbS64u9Bb5_Fx9xeBPh45Eg3V0_c:196:59)
    at Kxa.bindTo (js?key=AIzaSyCs_oAJbS64u9Bb5_Fx9xeBPh45Eg3V0_c:334:152)
    at kya.Qxa [as UE] (map.js:71:23)
    at js?key=AIzaSyCs_oAJbS64u9Bb5_Fx9xeBPh45Eg3V0_c:265:78

Minimal example reproducing the issue
https://www.accorplus.com/offers/christmas-feast-collage-all-day-dining-pullman-jakarta-central-park-dec-24/
Environment
Package.json
"leaflet": "^1.7.1",
"leaflet.gridlayer.googlemutant": "^0.10.0",
"leaflet.markercluster": "^1.4.1",

Browser (with version): all browsers
OS/Platform (with version): all devices

Are you the same from https://stackoverflow.com/questions/79266991/leaflet-maps-and-google-mutant ?
This is not a bug in core Leaflet and, as pointed there, GoogleMutant seems to be working just fine. And everything works on other webpages there, such as https://www.accorplus.com/hotels/?q=Singapore&rooms=1&room1Adults=1 .
It's your problem, not Leaflet's, not GoogleMutant's.

No not the same from Stack overflow but interesting that someone else has had a similar and sudden issue.

Well, then it's interesting that it might not be a coincidence. You might want to dig a big deeper into the issue and see if there have been any new releases of the Google Maps API. If you can pinpoint a plausible cause, report it into https://gitlab.com/IvanSanchez/Leaflet.GridLayer.GoogleMutant/-/issues

        Was genau willst du ?  Und was schickst du mir für ein scheiß ganze Zeit Am 11.12.24 um 01:33 schrieb Iván Sánchez Ortega

                Von: "Iván Sánchez Ortega"<EMAIL_ADDRESS>11. Dezember 2024An: "Leaflet/Leaflet"<EMAIL_ADDRESS>"Subscribed"<EMAIL_ADDRESS>Re: [Leaflet/Leaflet] Map Suddenly Stopped Loading Tiles (Issue #9556)

Well, then it's interesting that it might not be a coincidence. You might want to dig a big deeper into the issue and see if there have been any new releases of the Google Maps API. If you can pinpoint a plausible cause, report it into https://gitlab.com/IvanSanchez/Leaflet.GridLayer.GoogleMutant/-/issues
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

@LambAgency BTW, are you aware that GoogleMutant is at 0.14.0, and that you're using 0.10.0?


[Feature request] Keep highlighting in empty lines

Sometimes I add empty lines in my code to make it more readable,e.g.:
def example(self, x):
*   if some_condition:
*   *   max_el = 0

*   *   for el in x:
*   *   *   another_var = 0

*   *   *   if el > max_el:
*   *   *   *   max_el = el

If the lines surrounding the empty code have the same indentation level, I'd like the empty line to be highlighted too.
def example(self, x):
*   if some_condition:
*   *   max_el = 0
*   *
*   *   for el in x:
*   *   *   another_var = 0
*   *   *
*   *   *   if el > max_el:
*   *   *   *   max_el = el

This is an I'm afraid this is not possible, see issue #23 for details.

Isn't it feasible to fill the empty line with spaces (so that you can highlight them) and delete the extra spaces when the file is saved?

@fvisin That could work, and I've considered it before but there's a few reasons I don't want to implement it:

I don't like littering other peoples code with unneeded whitespace, even if temporarily.
You'd have to remove all trailing whitespace on save, even if it wasn't added by this plugin (which I don't like). The alternative is to track and remove only the whitespace added by the plugin, and I think that'd be extremely hard.

Also, I see no reason why this functionality needs to be coupled with this plugin, it could be a standalone plugin intended to be used with this and other similar plugins.


Upgrade to Spring Boot 1.5.10 and associated release trains

Primarily to pick up security vulnerability fixes

Coverage remained the same at 88.611% when pulling 3ab78d8ecec89bebcb8bd7e358ffb8f16f224264 on tgianos:boot1510 into f77af6ee3f3cb3867d31641eefb84795d0643c47 on Netflix:3.3.x.


Change dict/seq checks to use collections ABCs (for duck-typed params)

This allows parameters that provide sufficient interface to be used as dictionaries or sequence types (list/tuple) to pass the respective checks.

Update: I changed my mind, I think duck-typed lists and dicts should have a separate predicate so I've implemented those.

issequence (allows duck-typed lists)
ismapping (allows duck-typed dicts)
mapping_of (allows duck-typed dicts with predicates about keys/values like dict_of)
tests and documentation for above

Let me know what you think please.

Hi Benjamen,
I am all in favor of supporting the Sequence and Mapping ABCs.
In fact I had tried to do it but then refrained (due to my lack of experience with advanced 'import' logic) when I saw that they had been moved from collections to collections.abc in Python 3.3. I think I have overlooked that they are still in collections, too.
So let's go for them.
However, I would also like to keep the API small.
With your current additions, we would now be at 15 different builtin predicates. Too many for my taste.
But given the pragmatic spirit of the current set of predicates, I think there is a leaner approach that still gets us all we want. As follows:

Make list_of() test for collections.MutableSequence instead of list.
Kick out tuple_of(), because in that same spirit it would be the same as sequence_of(). (Yeah! That's why the module is still called beta: There was something about tuple_of() I did not quite like. I don't think we will break much code and even if we do, it is very easy to fix.)
Kick out issequence() because this is the same as simply annotating collections.Sequence.
Make dict_of() test for collections.Mapping instead of dict. (That is, kick out dict_of() and rename mapping_of() to dict_of().)
Kick out ismapping() because it is the same as simply annotating collections.Mapping.
And what do you think about using the shorter name seq_of() instead of sequence_of()? I am in favor of it.

This makes all parts of the module shorter (README.md, the code, the tests), yet functionally retains essentially everything of what you suggested.

I agree with all your bullet points, with one caveat:

seq_of() will (perhaps suprisingly) allow strings since they satisfy collections.Sequence. For example seq_of(str) is essentially identical to a plain str predicate since the subelements (chars) are also string objects (see image below). I've tried to make it clear in the documentation to use list_of() instead of seq_of() if only true lists of strings should be allowed.

Otherwise all the suggestions have been implemented.

Hi Lutz,
Not sure if you get notifications from Github but I've implemented your
suggestions in the original pull request.
Cheers,
Ben
On Wed, Jan 14, 2015 at 6:07 AM, Lutz Prechelt<EMAIL_ADDRESS>wrote:

Hi Benjamen,
I am all in favor of supporting the Sequence and Mapping ABCs.
In fact I had tried to do it but then refrained (due to my lack of
experience with advanced 'import' logic) when I saw that they had been
moved from collections to collections.abc in Python 3.3. I think I have
overlooked that they are still in collections, too.
So let's go for them.
However, I would also like to keep the API small.
With your current additions, we would now be at 15 different builtin
predicates. Too many for my taste.
But given the pragmatic spirit of the current set of predicates, I think
there is a leaner approach that still gets us all we want. As follows:

Make list_of() test for collections.MutableSequence instead of list.
Kick out tuple_of(), because in that same spirit it would be the
same as sequence_of(). (Yeah! That's why the module is still called beta:
There was something about tuple_of() I did not quite like. I don't
think we will break much code and even if we do, it is very easy to fix.)
Kick out issequence() because this is the same as simply annotating
collections.Sequence.
Make dict_of() test for collections.Mapping instead of dict. (That
is, kick out dict_of() and rename mapping_of() to dict_of().)
Kick out ismapping() because it is the same as simply annotating
collections.Mapping.
And what do you think about using the shorter name seq_of() instead
of sequence_of()? I am in favor of it.

This makes all parts of the module shorter (README.md, the code, the
tests), yet functionally retains essentially everything of what you
suggested.
—
Reply to this email directly or view it on GitHub
https://github.com/prechelt/typecheck-decorator/pull/1#issuecomment-69920131
.

Benjamen,
yes, I get those notifications.
I am a bit loaded in the moment but will see to it soon.
May take a few days, though.
Sorry,
Lutz
From: bkeroack<EMAIL_ADDRESS>Sent: Montag, 19. Januar 2015 18:40
To: prechelt/typecheck-decorator
Cc: Lutz Prechelt
Subject: Re: [typecheck-decorator] Change dict/seq checks to use collections ABCs (for duck-typed params) (#1)
Hi Lutz,
Not sure if you get notifications from Github but I've implemented your
suggestions in the original pull request.
Cheers,
Ben
On Wed, Jan 14, 2015 at 6:07 AM, Lutz Prechelt<EMAIL_ADDRESS>wrote:

Hi Benjamen,
I am all in favor of supporting the Sequence and Mapping ABCs.
In fact I had tried to do it but then refrained (due to my lack of
experience with advanced 'import' logic) when I saw that they had been
moved from collections to collections.abc in Python 3.3. I think I have
overlooked that they are still in collections, too.
So let's go for them.
However, I would also like to keep the API small.
With your current additions, we would now be at 15 different builtin
predicates. Too many for my taste.
But given the pragmatic spirit of the current set of predicates, I think
there is a leaner approach that still gets us all we want. As follows:

Make list_of() test for collections.MutableSequence instead of list.
Kick out tuple_of(), because in that same spirit it would be the
same as sequence_of(). (Yeah! That's why the module is still called beta:
There was something about tuple_of() I did not quite like. I don't
think we will break much code and even if we do, it is very easy to fix.)
Kick out issequence() because this is the same as simply annotating
collections.Sequence.
Make dict_of() test for collections.Mapping instead of dict. (That
is, kick out dict_of() and rename mapping_of() to dict_of().)
Kick out ismapping() because it is the same as simply annotating
collections.Mapping.
And what do you think about using the shorter name seq_of() instead
of sequence_of()? I am in favor of it.

This makes all parts of the module shorter (README.md, the code, the
tests), yet functionally retains essentially everything of what you
suggested.
—
Reply to this email directly or view it on GitHub
https://github.com/prechelt/typecheck-decorator/pull/1#issuecomment-69920131
.

—
Reply to this email directly or view it on GitHub https://github.com/prechelt/typecheck-decorator/pull/1#issuecomment-70532316 .

Benjamen,
what a mess!
When you contacted me, I had totally forgotten that there were 5 more commits (of June 2014) in my local repo that I had not yet pushed to github. Bad idea.
It turns out that, in our dialog, we have reinvented a lot of things that were in fact already there.
When I tried to merge your changes, I got extremely confused by the collisions.
I ended up not using your commits directly but rather adding the remaining changes to my own last master commit manually:

using collections.X rather than collections.abc.X,
the PyPy test fixes,
the tc.seq_of(str) special case (which will now not accept a plain string).

There are various other tweaks compared to the v0.3b state you have seen and I am now willing to call this v1.0 as soon as you tell me that

the tests indeed work with PyPy now and
you are happy with the shape and details of the API overall.

Note that you will now need pytest to run the tests.


Translate control datasets required for profiling

Translate all the control datasets required for profiling to TFDS

Complete for OpenX.


Add gccl metrics header to Spanner.

This PR adds the gccl/x.y.z portion of the metrics header in the Spanner client library.

The PR should also be rebased against current master to get rid of the spurious commits.

Travis seems to be hung? Anyone know if I can do anything about this?

@lukesneeringer there isn't anything you can do about Travis AFAICT. CircleCI should have covered it though so you should be clear to merge anyway in this case.

Okay, merging once @tseaver gives assent.

LGTM

@lukesneeringer FWIW, your docstring-in-a-testcase here will be literally the only one in the whole codebase.  If you feel that strongly about it, go ahead and merge.

I went ahead and removed it.


luci-app-https_dns_proxy:add rubyfish.cn DOH servise in china

luci-app-https_dns_proxy:add rubyfish.cn DOH servise in china

Please look at the other commits in this repository and take them as a template and update your commit/pull request.

Please look at the other commits in this repository and take them as a template and update your commit/pull request.

Is it ok now? i am new starter.there is no change anymore about the file

change 2 another commit


update prosemirror dependencies

Remove unnecessary @types
prosemirror-inputrules: ^1.2.0
prosemirror-state: 1.4.0
prosemirror-view: ^1.26.1
prosemirror-example-setup: ^1.2.0
prosemirror-model: ^1.18.0
prosemirror-schema-basic: ^1.2.0

Thanks for the PR!
Do you think you could remove the package lock, and instead use yarn to install?

I have built upon this in #15 and will release this as 0.4.0


Add engines to package.json

We don't have any particular engine requirement.

@timdorr It's not required, but it helps to have the minimum supported/tested Node version in package.json


Collapse from subelements

Hej,
Great work.
It would be very nice if collapse worked from sub elements. eg:

Outline 1

Element 1
Element 2 <- Cursor here
…

Outline 2

Press Collapse result:

Outline 1 <- Folded
Outline 2

Best
Florian

I've added the ability to collapse the root row when in a child row. This will be in the upcoming 1.1 release.

Great! Thanks


vim support

this is my first big pass at adding vim support. right now it doesn't support automatic detection and/or switching of modes for the most part, but it's still very much usable and I've been heavily using this for the last almost month, so I think it's good for other people to start testing it. I plan to keep adding to it and adding in the mode switching hopefully in the next week or two, but I don't think it needs to wait to get merged into upstream until that functionality is supported or anything
I do include a tutorial file because there is a few quirks related to getting it working. its still a work in progress but it should be enough to get people up and running.
there's a few things I'm not sure how to do deal with in the best way. for instance a lot of people won't have all of the plugins that are supported, so in that case you might not want them to actually load when you the vim tag, but I'm not sure if there's like a sane way to configure this at the moment. I would guess that one possibilities is to have a settings section inside of the vim.talon file and then you can manually enable specific plugins and only then are they loaded? but I'm not sure exactly how we would cleanly force load them only when the setting is enabled.

i'm slowly switching all this over to the slightly smarter mode-aware version already, so can likely just wait until i update the commits now

This is a big one! I'll dedicate some time for this next week. Sorry for the delay.

I have way more changes for this coming down the pipe btw. Preliminary mode switching, special terminal support, is all supported. I've just been testing and tweaking the last couple weeks, but will update the PR with the latest in the next couple days

this is updated now with the latest changes. I think it's pretty stable, as I've been using it daily I guess almost a month now, so I've fixed a lot of the little bugs that I run into and most of the mode switching stuff works at least how I want it to.
I do support a lot of settings for disabling certain features, and I have an extensively tested then being disabled because most of them his match the functionality I want by default. So we definitely will run into people having problems I suspect if they start tweaking things, but I'm happy to try my best supporting people using it for now

there's a few things I'm not sure how to do deal with in the best way. for instance a lot of people won't have all of the plugins that are supported, so in that case you might not want them to actually load when you the vim tag, but I'm not sure if there's like a sane way to configure this at the moment. I would guess that one possibilities is to have a settings section inside of the vim.talon file and then you can manually enable specific plugins and only then are they loaded? but I'm not sure exactly how we would cleanly force load them only when the setting is enabled.

I would say this is up to the user. we can document what plugins are supported, and they can simply remove any plugins they don't use?

this now has more of the latest updates just in case anyone wants to try it that isn't aware of my branch, but I still haven't the tweak things to follow all the latest generic talon actions serves still not actually suitable fur merging I think :|

Sorry for the moving target on this one. Hopefully, the generic actions prove worth the effort.... so far, I’ve been very happy with them with the apps I use.

I guess I'm going to close this for now as it's been almost a year, end there's been significant changes to all of this. there is stuff now that I think probably shouldn't be merged in its current state, and the docs are out of date and stuff. I may end up opening another pull request at some point, with the updated stuff


Set specific header color for DrB 'In case you missed it' block

 Change background color for newsletter header


Attribute AddressLines can't have a restriction

Based on the example from https://github.com/buildingSMART/IDS/discussions/363, it looks like the attribute restriction in IFC 2x3 on 'AddressLines' doesn't allow any base type. It appears that restrictions are not allowed on this attribute 🤔
I suspect it has to do with IDS Audit tool not processing the fact that 'AddressLines' is a list, not a single-value property.
I expect that it should accept base="xs:string".
Example:
<ids:ids xmlns:xs="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://standards.buildingsmart.org/IDS http://standards.buildingsmart.org/IDS/1.0/ids.xsd" xmlns:ids="http://standards.buildingsmart.org/IDS">
  <ids:info>
    <ids:title>example</ids:title>
  </ids:info>
  <ids:specifications>
    <ids:specification ifcVersion="IFC2X3" name="example">
        <ids:applicability minOccurs="1" maxOccurs="unbounded">
            <ids:entity>
                <ids:name>
                    <ids:simpleValue>IFCPOSTALADDRESS</ids:simpleValue>
                </ids:name>
            </ids:entity>
        </ids:applicability>
        <ids:requirements>
			<ids:attribute>
				<ids:name>
					<ids:simpleValue>AddressLines</ids:simpleValue>
				</ids:name>
				<ids:value>
					<xs:restriction base="xs:string">
                        <xs:enumeration value="xxxxx"/>
                        <xs:enumeration value="yyyyy"/>
                        <xs:enumeration value="zzzz"/>
					</xs:restriction>
				</ids:value>
			</ids:attribute>
        </ids:requirements>
    </ids:specification>
  </ids:specifications>
</ids:ids>```

Found out that it's probably because IFCPOSTALADDRESS is not an allowed entity (not a child of IfcObject). If so, that should be the warning, and not that base type is wrong.


Fix test_continuationtoken_empty

Now this test processes an error in a response.

I suggest moving the fixed on to our own suite

But this PR is already a request to merge the fix to master branch of our fork.

I suggest moving the fixed on to our own suite

But this PR is already a request to merge the fix to master branch of our fork.

I think @realloc meant not to change existing test_bucket_listv2_continuationtoken_empty test, but to create one more test_bucket_listv2_continuationtoken_empty_fixed (choose better naming) test with fixed behavior.
This branch, however, can be used for upstream PR. But first let's run compatibility tests against S3: https://github.com/nspcc-dev/neofs-s3-gw/issues/183. It will help to prove that our PR is valid.

We also can add some attribute (like this one @attr('fixed_to_neofs_s3_gate')) to be able to run such tests separately.

Will add docs to confluence how to run tests with our attribute (fixed_to_neofs_s3_gw looks good enough for name of our suite?) after approve/merge.


[catalog-import] Github integration: Support only imports from Entity Files (optionally turn off PR functionality)

Feature Suggestion
Hey, we have a use case where we want to implement the catalog-import plugin with GitHub integration but support only the option to register be providing the full URL to that Entity File and not provide the option to make a PR.
I can't find a way to turn this off, all the other integrations options work fine but look like this specific check for GitHub doesn't allow it.
Let me know if I miss something here 😄
Thanks team

Possible Implementation
Extend the config with some kind of flag?

I think this may be better suited as a prop to the plugin, than as a config option. To be added here, then.
I haven't heard of this requirement before now. Is it something you might be interested in contributing?

I think this may be better suited as a prop to the plugin, than as a config option. To be added here, then.
I haven't heard of this requirement before now. Is it something you might be interested in contributing?

I'm actively working on the plugin in #4271 and also plan to add configurability for these features. I'm not sure if it will be part of the PR, but I anticipate to at least make it easy to add it in follow-ups.

I'm actively working on the plugin in #4271 and also plan to add configurability for these features. I'm not sure if it will be part of the PR, but I anticipate to at least make it easy to add it in follow-ups.

Thanks,
@freben is there any example of how this can be achieved via prop? Should we provide an integration object, eg:
forgive my pseudocode
{
 "github":
    "open_pr": false,
    "import_from_file: true
}

And if something is not in this prop object it will default to true with a combination of the integration config.
@dhenneke your PR looks great, can somehow make the change for my suggestion compatible with your work in advance or we should keep it separate? :)

Thanks,
@freben is there any example of how this can be achieved via prop? Should we provide an integration object, eg:
forgive my pseudocode
{
 "github":
    "open_pr": false,
    "import_from_file: true
}

And if something is not in this prop object it will default to true with a combination of the integration config.
@dhenneke your PR looks great, can somehow make the change for my suggestion compatible with your work in advance or we should keep it separate? :)

👋 hey @freben , I made a quick Draft PR to have more data to discuss https://github.com/backstage/backstage/pull/4331
let me know what you think 😃

👋 hey @freben , I made a quick Draft PR to have more data to discuss https://github.com/backstage/backstage/pull/4331
let me know what you think 😃

feature was implemented to #4271


"Bâlteni" is listed in the drop-down menu

Description:
"Bâlteni" is listed in the drop-down menu alongside its corresponding county, according to the law
Precondition:
The website is up an running.

Step 1
Type "Bâlteni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Bâlteni, GORJ".


[5782] Addressing exporting large number of trainees on report page issues

Context
Addressing exporting large number of trainees on report page issues
Changes proposed in this pull request
Fix the low hanging fruits
Guidance to review
Amended select then first to be just find
It's an array Just find the first rather then select the filter then return first.
[5782] Added additional caching for trainee_allocation_subject
[5782] Added additional caching for calculated_subject
Lookup for the sake of lookup, once looked at there is no point in second journey to database calls. Cache it for the day so any other users can benefit
[5782] Added funding_methods for includes
To reduce additional database calls
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Spike: Test exporting large number of trainees on Report page


stripped image

Hello. I am trying your python script, but it not work properly.
Do you have any idea what can be broken?
I receive some data, but result image is divided to strips.

Do you have same HW? My looks like:
https://photos.app.goo.gl/fD4BUnCmLG3JfzHx5
https://photos.app.goo.gl/d5zxgfVN1Ty7xdHm8
Wireshark recording is here
https://drive.google.com/file/d/1nvXeVmzDHi6V9skNefIyfQX1gnoz9I9U/view?usp=sharing

Hm, packaging and case look similar, but there are several hardware variants. Image looks like it uses a different resolution. Could you tell us which phone app was used (Name/URL)?

Application named "HND" link https://www.6zhentan.com/app/hnd/index.html
I buy it here https://www.aliexpress.com/item/1005005038398094.html

And when I try another Python script developed in parallel to yours, it shows a strange, probably unreal resolution of 416x416.
Script is here https://gist.githubusercontent.com/tvecera/0d95ae329af1ab6f7cf55c641179c385/raw/41c67562260e214c7bab57612a12f2d1f9c74f84/camera.py
And result https://pastebin.com/WUehySQw

It looks, that I will have more information in few hours.
Problem is in the image size and necessary JPEG headers.
Discovered by Tomas Vecera. See https://twitter.com/veceratomas/status/1709818032472588546 (czech only)
I will test it today later.

Nice. It works like expected.
But resolution 416x416 is surprise.


Timberwolves

Mod Version: v0.10.3
Game Version: v20.2
Launcher: Steam
Additional mods installed: No
Perspective:  Both
Description
Timberwolf spawn is well bugged in a way. As it currently stands BL and BR are super easy due to low to non spawning of normal wolves. As it's mainly suppose to be a timberwolf spawn area. But timberwolves only spawn one and one and not as a pack and thus always wimper and run away from you, meaning you can walk around without any danger what so ever in these "hard" but lucrative areas.
How to reproduce
Start multiplayer spawn in BL or BR and walk around until you meet a single timberwolf that will run away from you.
Expected behavior
Expected packs of timberwolves
Possibly solution until you've found a way to fix timberwolves. Replace spawn with regular wolves so there's somewhat of a threat there.
Sorry for the constant bug reports :P
If non-critical, just close and pick it up when time allows. I mean you are turning a singleplayer game into a multiplayer one, there's bound to be weird bugs popping up here and there.
Thanks again.

They spawns alones probably because of how my recreation of spawning code works. I will check this out. But honestly, even with correct amout of them there still problems of them's "moral metter", because every time wolf re-target to other player, they become an "inactive dummy" that controlled by other player, and if not  all of them on one player they will lose "moral meter units", this has not been fixed, because there no good solutions of fixing this in my mind. So best solution of this for current situation, will be auto-replace spawns of that wolfs with regular ones, but just set correct amount of them, but I not sure if it will be harder, cause they still will flee from one shot.

Yeah I figured the moral meter was gonna be a hard nut to crack. But since a single timberwolf is programmed to flee whenever it sees the player, regular wolves will at least try to mess with ya and it makes the place at least not a complete walk over. :D
https://steamcommunity.com/id/lussian/ Not sure if that's what you need or the 64 bit number string. Or just current username which is Lussian
Thanks and keep up the good work.

Ye, I was need 64 one, I found it, and added flair, so now you got pin with crossed bug.


Proposal: support openflow 1.5 for stateful flow tracking

Description
Use openflow 1.5 version that allows stateful flows

Initial analysis

Using conn_track

Set conntrack action_(ct_next)_ to forward packet to Netfilter's conneciton tracker which maintains the hash of the state of the connection.
The packet will then be resubmitted to the OVS flow table which can then be hashed to service instances based on new metadata from conntrack (conn_state bits exist to track new/established connections)

Using GROUP table and SELECT buckets

Add all service instances to SELECT bucket (only one copy is sent to the entire bucket and the weight parameter in the bucket can be utilized to determine packet distribution)
Set type=select,selection_method=hash,fields() for a group. This will select the same instance based on the hash of the fields tuple.
NOTE: This uses the Netronome extension to OpenFlow 1.5+ that allows control over the OpenFlow select groups selection method


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-28-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4613

Questions ?
Please refer to the Backport tool documentation


image resizing only working first time

Node: 6.11.5
Sharp: 0.20.8 | 0.21.3
I've been trying to resize an image. Sharp returns the correct resized image on the first 1 or 2 tries. But afterwards it always returns the first resized image every time. Even the different input files to sharp has different hashes, but the output file of sharp has same hash.
For example:

hash(inputFile1) -> inputFile1HASH
sharp(inputFile1).resize(w, h).webp().toFile(outputFile1)
hash(outputFile1) -> outputFile1HASH (GOOD)

hash(inputFile2) -> inputFile2HASH
sharp(inputFile2).resize(w, h).webp().toFile(outputFile2)
hash(outputFile2) -> outputFile1HASH (BAD)

hash(inputFile3) -> inputFile3HASH
sharp(inputFile3).resize(w, h).webp().toFile(outputFile3)
hash(outputFile3) -> outputFile1HASH (BAD)

This is the code
sharp(inputFilePath)
.resize(width, height)
.webp()
.toFile(outputFilePath)

Any help would be appreciated. Or if i'm doing something wrong, please let me know.

Hello, if the contents of inputFilePath are changing but the path itself remains the same then you'll need to disable libvips cache via sharp.cache(false).
https://sharp.pixelplumbing.com/en/stable/api-utility/#cache

Hello, thank you so much. This resolved the issue. It would be awesome to have this same path is cached and other things like this in the docs too. Thanks again. This saved a lot of trouble and time. A big salute to you and the sharp community for this awesome library.


Updated HAProxy doc

What are your changes?
Removed server dcos-1 <IP_ADDRESS>:443 check-ssl from example.
What type of changes does your doc introduce?

[ x] Bug fix
[ ] New feature

What is the urgency level of this change?

[ ] Blocker
[ x] High
[ ] Medium

I have completed these items:

[ ] I have tested all commands and code snippets.
[ ] I have built locally and tested for broken links and formatting.
[ x] I have made this change for all affected versions (e.g. 1.7, 1.8, and 1.9).
[x ] My doc follows the style guide: https://github.com/dcos/dcos-docs#contributing.

Ping @emanic @sascala @joel-hamill for reviewing pull request changes.
If you included a comment with your commit, it appears here:

@jgehrcke ready to merge? i don't think we should wait for @alberts since he's on PTO.


Remove unused codes from inputs

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

Test name
Commit
Details
Rerun command

gencheck_operator
de381d9d40d081d2f178853341f5bdd55f21399b
link
/test gencheck_operator

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

I was surprised that this code could be removed because it's capturing the dependency tree relationship. Then I saw that the logic was rewritten in https://github.com/istio/operator/pull/371, but I don't understand why there was a deadlock and why it was fixed like this - could you explain?

I was surprised that this code could be removed because it's capturing the dependency tree relationship. Then I saw that the logic was rewritten in #371, but I don't understand why there was a deadlock and why it was fixed like this - could you explain?

@ostromart We now will calculate the dependency dynamically according to the generated manifest instead of the static data here. The reason is there will be a dead lock for the waiting group : pkg/helmreconciler/reconciler.go:L164.
Because only those components which existed in the manifest will read from the dependency channel while originally with the static dependency data we will write to the channel for all components.
This will cause the wg.wait() always hangs there because below lines will be blocked there:
// Signal all the components that depend on us.
			for _, ch := range deps[cn] {
				log.Infof("Unblocking dependency %s.", ch)
				dch[ch] <- struct{}{}
			}


How to match hypothesis with references using BLEU?

In the following sys contains happy that is the exact match for the second reference, but why the bleu score is still zero?
import sacrebleu
sys = ["happy"]
refs = [["like achieve"],
        ["happy"]]

b3 = sacrebleu.corpus_bleu(sys, refs)
print("b3", b3.score)
print("b3", round(b3.score,2))

It prints
b3 0.0
b3 0.0

If BLEU isn't a good metric for this purpose, I look for a metric that can score the match or substrings in the hypothesis with any reference. I thought the BLEU score is for the same purpose!
Another weird case is the effect of removing dots in the following:
import sacrebleu, nltk
sys = ["This is cat."]
refs = [["This is a cat."],
        ["This is a bad cat."]]

b3 = sacrebleu.corpus_bleu(sys, refs)
print("b3", b3.score)
print("b3", round(b3.score,2))

b3 35.1862973998119
b3 35.19

When I remove the ending dots.
sys = ["This is cat"]
refs = [["This is a cat"],
        ["This is a bad cat"]]

b3 = sacrebleu.corpus_bleu(sys, refs)
print("b3", b3.score)
print("b3", round(b3.score,2))

It prints zero using scarebleu which is again weird!:
b3 0.0
b3 0.0

BLEU is defined as a geometrical average of (modified) n-gram precisions for unigrams up to 4-grams (times brevity penalty). Thus if there is no matching 4-gram (no 4-tuple of words) in the whole test set, BLEU is 0 by definition. BLEU was designed for scoring test sets with hundreds of sentences where such case is very unlikely. For scoring single sentences, you can use a sentence-level version of BLEU which uses some kind of smoothing, but the results are still not ideal. You can also use a character-based metric, e.g. chrF (sacrebleu -m chrf).

In addition to @martinpopel 's suggestions,
You can also pass use_effective_order=True to corpus_bleu so that only the matched n-gram orders are counted instead of 4 n-grams. However, in that case, the metric is not exactly what people would refer to BLEU.
In your second case, having a dot at the end which will get tokenized, makes it so that that there are now matches for 4-grams because smoothing is applied.
Closing this as it is not a bug. Thanks!


Vickel: Watch mypcgames.net

Vickel requests the watch of the watch_keyword mypcgames\.net. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

mypcgames\.net has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by cigien in Charcoal HQ


Error code undefined object TypeError: s(...).abr.EwmaBandwidthEstimator is not a constructor

Have you read the Tutorials?
Yes
Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
2.4.4
Please ask your question
I am trying to load() my DASH manifest URL directly and also by using DashVideoSource but when I run my code:
for loading DASH Url Directly in player.load() is giving me the following error:
load Manifest URL: https://equidia-racing4-p9-cdn.hexaglobe.net/e5afcb5da4277149deb8ee0553b7c7e8/605a04a5/equidia/equidia-website-web-1/eqdracing4pri_fre.smil/manifestwidevine.mpd
along with passing drm URL from response to DRM configuration of shaka-player, it is giving me the same error with code 6001

hexa-shaka-player.js:2 Error code 6001 object
Y {severity: 2, category: 6, code: 6001, data: Array(0), handled: false}
category: 6
code: 6001
data: []
handled: false
severity: 2

And if I am trying to load the same URL using DashVideoSource it gives me an error which I have already mentioned as the title for this question. i.e. "Error code undefined object TypeError: s(...).abr.EwmaBandwidthEstimator is not a constructor"
Here is my initPlayer code:
export const initPlayer = async function () {
    // Create a Player instance.
    const video = document.getElementById('video')

    // shaka.media.ManifestParser.registerParserByExtension("mpd", shaka.dash.DashParser);
    const player = new shaka.Player(video);

    // Attach player to the window to make it easy to access in the JS console.
    window.player = player;

    // Listen for error events.
    player.addEventListener('error', onErrorEvent);

    // Try to load a manifest.
    // This is an asynchronous process.
    try {
        var drmUrl = selectDrm();
        state.drmUrl = drmUrl;
        // model.getDrmConfig(model.state.drmUrl);
        await loadStreams()
        video.setAttribute('poster', `${state.streams.logo}`)
        await loadmanifest()
        // model.state.manifest = Object.assign({...data});
        console.log("manifest API: " + state.manifests.primary)
        getDrmConfig(state);
        player.configure({
            drm: {
                servers: state.config.drm.servers,
                advanced: {
                    'com.widevine.alpha': {
                        'videoRobustness': 'HW_SECURE_ALL',
                        'audioRobustness': 'HW_SECURE_ALL'
                    }
                }
            }
        });
        var estimator = new shaka.abr.EwmaBandwidthEstimator();

        var source = new shaka.player.DashVideoSource(state.manifests.primary,
            null, estimator);

        // Finally, begin playback.
        await player.load(source);
        // This runs if the asynchronous load is successful.
        console.log('The video has now been loaded!');
    } catch (e) {
        // onError is executed if the asynchronous load fails.
        onError(e)
    }
    console.log(player.getConfiguration())
}

The DASH URL is: https://equidia-racing1-p-cdn.hexaglobe.net/beb6a0bf7d4c62fc24d2bd5e7eb2793f/6059f5c4/equidia/equidia-website-web-1/eqdracing1pri_fre.smil/manifestwidevine.mpd
Note: it is an API of a live streaming channel named Equidia.
Can you tell me why I am not able to play the video on the video player?
Thank you in advance

DashVideoSource was removed over 4 years ago when we released v2.0.  You also don't need to create any bandwidth estimator, it is handled internally.  Check out our latest basic tutorial for how to setup the latest player.
6001 is REQUESTED_KEY_SYSTEM_CONFIG_UNAVAILABLE.  This means that either the DRM system you wanted isn't supported on this browser or you set a configuration that isn't supported.  HW_SECURE_ALL is only supported on phones and some smart TVs, it is not supported on browsers.

Hello, thank you for the response.
I understood that DashVideoSource is removed and I have now Implemented the player with the latest basic tutorial.
In my project, I am using webpack and have installed shaka player using npm install shaka-player passed as devDependency in my package.json
I am using the live streams API of Equida channel configured by my company you can see here
after fetching the URLs I get Dash Manifest as a response with DRM URL that I configure it in player.configure().
but when I do player.load() is giving me 6002: FAILED_TO_CREATE_CDM
HERE IS MY LATEST CODE:
import {AJAX} from './helpers.js';
import {UAParser} from 'ua-parser-js';
import shaka from 'shaka-player'
export const state = {
    streams: {},
    manifests: {},
    config: {
        drm: {
            servers: {},
            clearKeys: {},
        },
        manifest: {
            dash: {
                clockSyncUri: String,
                ignoreDrmInfo: Boolean,
                xlinkFailGracefully: Boolean,
                ignoreMinBufferTime: Boolean,
                autoCorrectDrift: Boolean,
                initialSegmentLimit: Number,
                ignoreSuggestedPresentationDelay: Boolean,
                ignoreEmptyAdaptationSet: Boolean
            },
            hls: {
                ignoreTextStreamFailures: Boolean,
                useFullSegmentsForStartTime: Boolean
            }
        },
    },
    drmUrl: String,
}

const createStreamsObject = function (data) {
    const streams = data[58]
    return {
        title: streams.title,
        player: streams.player,
        streamUrl: streams.streamUrl,
        streamUrl_ios: streams.streamUrl_ios,
        streamUrl_android: streams.streamUrl_android,
        logo: streams.logo,
        disabledLogo: streams.disabledLogo,
        channel: streams.channel
    }
}

const getStreamManifest = function (data) {
    const manifests = data
    return {
        primary: manifests.primary,
        primary_drm: manifests.primary_drm,
        backup: manifests.backup,
        backup_drm: manifests.backup_drm,
        status: manifests.status,
        protocol: manifests.protocol,
    }
}

const addZero = function(num) {

    if (num >= 10)
    {
        return num;
    }
    return '0' + num;
}

const getDateToday = function () {

    var today = new Date();
    var date = today.getFullYear() + '' + addZero(today.getMonth()+1) + '' + addZero(today.getDate());
    return date;
}

//getStream infos
export const loadStreams = async function () {
    var streamsInfoUrl = "https://equidia-vodce-players.hexaglobe.net/mf_data/" + getDateToday() + ".json";
    console.log(streamsInfoUrl)
    try {
        const data = await AJAX(`${streamsInfoUrl}`);
        state.streams = createStreamsObject(data);
        // console.log(state.streams)
    } catch (err) {
        // Temp error handling
        console.error(`${err} 💥💥💥💥`);
    }
};

export const getDrmServer = async function (drmServerName, url) {
    const drmServer = {}
    drmServer[drmServerName] = url;
    console.log(drmServer);
    state.config.drm.servers = drmServer;
}

export const getDrmConfig = function (state) {
    return getDrmServer('com.widevine.alpha', state.manifests.primary_drm)
}

export const selectDrm = function () {

    var ua = new UAParser().getResult();

    console.log('device.type=', ua.device.type, ' browser.name=', ua.browser.name, ' os.name=', ua.os.name);

    if (ua.device.type === 'mobile' || ua.device.type === 'tablet')
        return 'hls';

    if (ua.browser.name === 'Chrome' || ua.browser.name === 'Opera' || ua.browser.name === 'Firefox' && parseInt(ua.browser.version) > 51)
        return 'dashwidevine';

    if (ua.os.name === 'Windows' && parseFloat(ua.os.version) >= 8.1 && (ua.browser.name === 'IE' || ua.browser.name === 'Edge'))
        return 'dashplayready';

    if (ua.os.name === 'iOS' || ua.os.name === 'Mac OS' && ua.browser.name === 'Safari' && parseFloat(ua.browser.version) >= 11)
        return 'hlsfairplay';

    return 'hls';
}

export const loadmanifest = async function () {
    var urlStreamInfo = `${state.streams.streamUrl}` + `?format=` + `${state.drmUrl}`
    console.log(urlStreamInfo)
    try {
        const data = await AJAX(`${urlStreamInfo}`);
        state.manifests = getStreamManifest(data);
    } catch (err) {
        // Temp error handling
        console.error(`${err} 💥💥💥💥`);
    }
};
export const initPlayer = async function () {
    // Create a Player instance.
    const video = document.getElementById('video')

    // shaka.media.ManifestParser.registerParserByExtension("mpd", shaka.dash.DashParser);
    const player = new shaka.Player(video);

    // Attach player to the window to make it easy to access in the JS console.
    window.player = player;

    // Listen for error events.
    player.addEventListener('error', onErrorEvent);

    // Try to load a manifest.
    // This is an asynchronous process.
    try {
        var drmUrl = selectDrm();
        state.drmUrl = drmUrl;
        // model.getDrmConfig(model.state.drmUrl);
        await loadStreams()
        video.setAttribute('poster', `${state.streams.logo}`)
        await loadmanifest()
        // model.state.manifest = Object.assign({...data});
        console.log("manifest API: " + state.manifests.primary)
        getDrmConfig(state);
        player.configure({
            drm: {
                servers: state.config.drm.servers,
                advanced: {
                    'com.widevine.alpha': {
                        'videoRobustness': 'SW_SECURE_CRYPTO',
                        'audioRobustness': 'SW_SECURE_CRYPTO'
                    }
                }
            },
        });

        await player.load(`${state.manifests.primary}`);
        // This runs if the asynchronous load is successful.
        console.log('The video has now been loaded!');
    } catch (e) {
        // onError is executed if the asynchronous load fails.
        onError(e)
    }
    console.log(player.getConfiguration())
}

function onErrorEvent(event) {
  // Extract the shaka.util.Error object from the event.
  onError(event.detail);
}

function onError(error) {
    console.error('Error code', error.code, 'object', error);
}

INDEX.JS FILE CODE
const shaka = require('shaka-player');
const model = require('./model');
const manifestUri = "";

function initApp() {
  // Install built-in polyfills to patch browser incompatibilities.
  shaka.polyfill.installAll();

  // Check to see if the browser supports the basic APIs Shaka needs.
  if (shaka.Player.isBrowserSupported()) {
    // Everything looks good!
    model.initPlayer();
  } else {
    // This browser does not have the minimum set of APIs we need.
    console.error('Browser not supported!');
  }
}

document.addEventListener('DOMContentLoaded', initApp);

ERROR:
device.type= undefined  browser.name= Chrome  os.name= Linux
hexa-shaka-player.js:2 https://equidia-vodce-players.hexaglobe.net/mf_data/20210323.json
hexa-shaka-player.js:2 https://equidia-racing1-p-api.hexaglobe.net/34cdc24……/equidia/equidia-website-web-1/racing1/stream_info.php?format=dashwidevine
hexa-shaka-player.js:2 manifest API: https://equidia-racing1-p7-cdn.hexaglobe.net/2cfc333……/equidia/equidia-website-web-1/eqdracing1pri_fre.smil/manifestwidevine.mpd
hexa-shaka-player.js:2
{com.widevine.alpha: "https://widevine-dash.ezdrm.com/proxy?pX=4B6508&Cu…ng1%23605a180b%2325ca41faf2ca21e8e89da5e7234bec24"}
hexa-shaka-player.js:2 Error code 6002 object
Y {severity: 2, category: 6, code: 6002, data: Array(1), handled: false}
category: 6
code: 6002
data: Array(1)
0: "Failed to execute 'createMediaKeys' on 'MediaKeySystemAccess': EME use is not allowed on unique origins."
length: 1
proto: Array(0)
handled: false
severity: 2
proto: Object

Can you please help me why I am not able to load the URI:
Note I am using crome browser

The error includes EME use is not allowed on unique origins, which comes from the browser.  My guess is this is a file:// URL or an http:// url, neither of which support EME.  You need to use a real web server that is hosted on https:// to be able to play protected content (note you can use localhost as http, but not other sites).

When I am running on localhost it gives me the following error displayed in the image.

That's caused by CORS. You can open a browser with web security disabled.
If you are using Mac:
open -na Google\ Chrome --args --user-data-dir=/tmp/temporary-chrome-profile-dir --disable-web-security --disable-site-isolation-trials
If you are using Linux:
Open a Chrome browser with web security disabled. Run command:
google-chrome --disable-web-security --user-data-dir=~/Desktop/tmp/
Also, in the Chrome DevTools, set "Bypass for network":
Application -> Service Workers -> Select: √ Bypass for network.
Let us know if that works :)

That is a CORS error, see the doc on MDN for more info.  Your manifest and media files need to have the correct response headers to be visible to JavaScript running on another domain.  You either need to add headers to the manifest/media responses or have the player on the same domain as the manifest/media files.

That's caused by CORS. You can open a browser with web security disabled.
If you are using Mac:
open -na Google\ Chrome --args --user-data-dir=/tmp/temporary-chrome-profile-dir --disable-web-security --disable-site-isolation-trials
If you are using Linux:
Open a Chrome browser with web security disabled. Run command:
google-chrome --disable-web-security --user-data-dir=~/Desktop/tmp/
Also, in the Chrome DevTools, set "Bypass for network":
Application -> Service Workers -> Select: √ Bypass for network.
Let us know if that works :)

Nope, I tried from chrome dev-tools but it didn't work. Thank you

That is a CORS error, see the doc on MDN for more info. Your manifest and media files need to have the correct response headers to be visible to JavaScript running on another domain. You either need to add headers to the manifest/media responses or have the player on the same domain as the manifest/media files.

Yes, You are right because when copy and paste this link directly in the browser it gives response but when I try to do player.load() it gives me CORS errors because the headers are not open for another domain on the server-side. I will discuss this with my teammates tomorrow and let you know :)

Hello as I was trying to load the manifest in the player which was on the localhost domain and not on the same domain as manifest was. Thank you for your help and guidance it helped me a lot :).
before closing,
(I have just one question can we import shaka as ES6 modules?)
If I have any issues I will reach here back for now I am closing this issue.


Feign Client failed to resolve @RequstMapping on interface with inheritance

Describe the bug
Rest API Interface with inheritance, the @RequestMapping annotation will be ignored.
Sample
public interface Foo {

}

@RequestMapping(path="/bar")
public interface Bar extends Foo{
  @GetMapping("xxx")
  Object someMethod();
}

if i use Bar as interface to generate a feign client, the uri call to someMethod will lose "/bar"
I checked the code, and quiet confused with the code in org.springframework.cloud.openfeign.support.SpringMvcContract$processAnnotationOnClass that
clz.getInterfaces().length == 0

	@Override
	protected void processAnnotationOnClass(MethodMetadata data, Class<?> clz) {
		if (clz.getInterfaces().length == 0) {
        ...

the call to this method has already handled inheriance:
code in feign.Contract
    protected MethodMetadata parseAndValidateMetadata(Class<?> targetType, Method method) {
      final MethodMetadata data = new MethodMetadata();
      data.targetType(targetType);
      data.method(method);
      data.returnType(Types.resolve(targetType, targetType, method.getGenericReturnType()));
      data.configKey(Feign.configKey(targetType, method));

      if (targetType.getInterfaces().length == 1) {
        processAnnotationOnClass(data, targetType.getInterfaces()[0]);
      }
      processAnnotationOnClass(data, targetType);
     ...

I think the statement if (clz.getInterfaces().length == 0)
org.springframework.cloud.openfeign.support.SpringMvcContract$processAnnotationOnClass may be removed.

Hi @sialais,
Please read gh-564

if i use Bar as interface to generate a feign client, the uri call to someMethod will lose "/bar"

When I read this. I think you will use Bar class as interface to your FeignClient. In Feign path variable is used to define path and There is no way to inherit it. As @OlgaMaciaszek mentioned here, I think you can't use @RequestMapping annotation on inherited interface of FeignClient.

if i use Bar as interface to generate a feign client, the uri call to someMethod will lose "/bar"

When I read this. I think you will use Bar class as interface to your FeignClient. In Feign path variable is used to define path and There is no way to inherit it. As @OlgaMaciaszek mentioned here, I think you can't use @RequestMapping annotation on inherited interface of FeignClient.

Yes, Bar is the interface of FeignClient, and path variable is not inherited from Foo.
Foo can be some interface just to mark, like Serializable, and this inheritance will make @RequestMapping be ignored.

By the way, I think it is resonable to use @RequestMapping annotation on child interface and ignore annotation on parent interface if the annotation both on child and parent. (@RequestMapping can be overrided), as the way url mapping in springMVC.

Yes, as @cbezmen has noted, this is not advisable and not supported. Please see the note at the end of this section.


Fixes sonar violations.

Coverage remained the same at 95.341% when pulling 95e8ee1645b42b21479495eb75ac6f0c6dfc2507 on FlorianLautenschlager:master into ba57b24c08dd408572c5dfcffa2b5a1a2065acbd on ChronixDB:master.


[FEAT] Implementing Contact us page (Mail based)

Developing a contact us page, as we are giving option of contact on navbar, but we dont have a contact us page.
designing and implementing a contact page, which consist of a form with name, email and message, on submitting it will send a mail to owner consisting of user's message.

@Sourabh782 So you will be forwarding the form details of to an email right now?

@Sourabh782 So you will be forwarding the form details of to an email right?

Yess, like as a user, I entered name, my mail and message, then the email will be sent to u (owner of  the site) I'll will be done in the same way as verification OTP is being sent, just the recipient will be you

Cool, You can get started. The code for the SMTP setup will already be available in the code.

Cool, You can get started. The code for the SMTP setup will already be available in the code.

Okiee👍


sls flusher support  metricstore

Logtail 2.0 add sls Metrics support

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.shaoxuan seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

TODO : Logtail 端针对错误码的特殊处理需要需求可以同步 Metricstore , 如 SLS quota 异常 @linrunqi08 @henryzhx8


[otbn,dv] Fix reset during EDN request (again)

This commit reapplies 40dea7f638 (#13294), which had been partially
undone in 3c7192597e (part of #13391).

Oh, that was totally my bad, thanks for the catch!

No worries, thanks for the quick response!


Add OSX and Windows support

Attempt at getting OSX working (#15). I'm not really sure how this should work, but I bumped the nightly up to the version mentioned on the orjson README under the "packaging" section and added platform specific tags. Figure it was worth a try.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Selectors are suggested to take a <two spaces>#<one space>[<expression>] form. See lines [12]

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please rerender

@conda-forge/core -- I'm trying to figure out what is going on here. This builds ok on linux, but on osx, the files seem to be missing from the build directory. I stuck in an ls -l in build.sh. On linux it gives:
2020-08-04T11:26:32.5777245Z + ls -l
2020-08-04T11:26:32.7848473Z total 184
2020-08-04T11:26:32.7853128Z -rw-r--r--  1 conda conda  5225 Aug  4 11:26 build_env_setup.sh
2020-08-04T11:26:32.7853910Z -rw-r--r--  1 conda conda   435 Jul 24 01:43 build.rs
2020-08-04T11:26:32.7854571Z -rw-r--r--  1 conda conda  6672 Jul 24 01:43 Cargo.lock
2020-08-04T11:26:32.7855202Z -rw-r--r--  1 conda conda  2367 Jul 24 01:43 Cargo.toml
2020-08-04T11:26:32.7855809Z -rw-r--r--  1 conda conda  8978 Jul 24 01:43 CHANGELOG.md
2020-08-04T11:26:32.7856440Z -rwxrw-rw-  1 conda conda   884 Aug  4 11:26 conda_build.sh
2020-08-04T11:26:32.7857234Z drwxr-xr-x  6 conda conda  4096 Aug  4 11:25 data
2020-08-04T11:26:32.7857830Z drwxr-xr-x  4 conda conda  4096 Aug  4 11:25 json
2020-08-04T11:26:32.7858431Z -rw-r--r--  1 conda conda 10847 Jul 24 01:43 LICENSE-APACHE
2020-08-04T11:26:32.7859039Z -rw-r--r--  1 conda conda  1023 Jul 24 01:43 LICENSE-MIT
2020-08-04T11:26:32.7859699Z -rw-r--r--  1 conda conda  1023 Aug  4 11:26 metadata_conda_debug.yaml
2020-08-04T11:26:32.7860291Z -rw-r--r--  1 conda conda 46898 Jan  1  1970 PKG-INFO
2020-08-04T11:26:32.7860897Z -rw-r--r--  1 conda conda   332 Jul 24 01:43 pyproject.toml
2020-08-04T11:26:32.7861496Z -rw-r--r--  1 conda conda 45756 Jul 24 01:43 README.md
2020-08-04T11:26:32.7862112Z drwxr-xr-x 12 conda conda  4096 Aug  4 11:26 rust-nightly
2020-08-04T11:26:32.7862737Z drwxr-xr-x  6 conda conda  4096 Aug  4 11:26 rust-nightly-install
2020-08-04T11:26:32.7863350Z drwxr-xr-x  4 conda conda  4096 Aug  4 11:25 src
2020-08-04T11:26:32.7863927Z drwxr-xr-x  2 conda conda  4096 Aug  4 11:25 test

but on OSX, it doesn't look right:
2020-08-04T11:27:48.7497670Z + ls -l
2020-08-04T11:27:48.7621150Z total 32
2020-08-04T11:27:48.7624060Z -rw-r--r--   1 runner  staff  5724 Aug  4 11:27 build_env_setup.sh
2020-08-04T11:27:48.7625950Z -rwxrw-rw-   1 runner  staff   876 Aug  4 11:27 conda_build.sh
2020-08-04T11:27:48.7627840Z -rw-r--r--   1 runner  staff   864 Aug  4 11:27 metadata_conda_debug.yaml
2020-08-04T11:27:48.7629230Z drwxr-xr-x  21 runner  staff   672 Aug  4 11:27 rust-nightly
2020-08-04T11:27:48.7630600Z drwxr-xr-x   6 runner  staff   192 Aug  4 11:27 rust-nightly-install

and then maturin fails since there isn't the expected Cargo.toml:
+ maturin build --no-sdist --release --strip --manylinux off
💥 maturin failed
  Caused by: Can't find Cargo.toml
  Caused by: No such file or directory (os error 2)

Any idea why the files might not be where they should be?

Ok, it looks like OSX is working. I'm going to go ahead and try to do windows in the same PR

@conda-forge-admin please rerender

It looks like even though cargo should be on the path, it's not being found in the environment. From the log:
2020-08-04T18:46:35.3417712Z install: copying file /d/bld/orjson_1596566490690/work/rust-nightly-install/bin/cargo.exe
....
2020-08-04T18:47:20.3891999Z (%BUILD_PREFIX%) %SRC_DIR%>echo "check cargo"
2020-08-04T18:47:20.3892281Z "check cargo"
2020-08-04T18:47:20.3892424Z
2020-08-04T18:47:20.3892665Z (%BUILD_PREFIX%) %SRC_DIR%>cargo --help
2020-08-04T18:47:20.4331600Z 'cargo' is not recognized as an internal or external command,
2020-08-04T18:47:20.4332664Z operable program or batch file.
2020-08-04T18:47:20.4333106Z
2020-08-04T18:47:20.4335233Z (%BUILD_PREFIX%) %SRC_DIR%>echo "check cargo.exe"
2020-08-04T18:47:20.4335772Z "check cargo.exe"
2020-08-04T18:47:20.4336022Z
2020-08-04T18:47:20.4336376Z (%BUILD_PREFIX%) %SRC_DIR%>cargo.exe --help
2020-08-04T18:47:20.4425194Z 'cargo.exe' is not recognized as an internal or external command,
2020-08-04T18:47:20.4426310Z operable program or batch file.

Anyone have thoughts on this?

I don't know if it has been discussed before, but IMO it would make sense to create a rust-nightly package instead of downloading in the recipe itself.

@mbargull -- Thanks again for your help with this PR. As far as a rust-nightly package, I just wonder what the release schedule would look like? I know a lot of projects pin to a particular date when they're using nightly as one that they know works. I'm sure there's wiggle room to choose other versions, but it might be a little tricky and require a large number of releases if there are a lot of projects that depend on it.

@mbargull -- It looks like everything passes. As a final change, I was thinking of adding myself as a co-maintainer.

As a final change, I was thinking of adding myself as a co-maintainer.

:thumbsup:

Would you also like to be included?

Not really, no. I don't the package and did not look any further than the diffs and logs ;).

Thank you everyone for all the work!
@synapticarbors do you want me to merge this or should I wait for an update adding your name to the maintainers list?

@conda-forge-admin please rerender

@nehaljwani -- I just added myself. Once the re-render completes and tests pass, please feel free to merge this.