GH-1719: Update jQuery and jQuery-migrate to newest 1.x version.

Checklist

[x] the [individual contributor license agreement][] is signed

Description of change
Update jQuery and jQuery-migrate to newest 1.x version. This solves an issue we are having that breaks some of our portlets that use jQuery's sub() method.
https://github.com/Jasig/uPortal/issues/1719

In my limited testing (I was on my way to test something else), this change doesn't appear to have any negative effects.


issues with unicode strings in groupby and python2

The following code doesn't work for me.
b.categories.groupby('key')['val']

This does
b.categories.groupby('key')[u'val']

The only difference is that I explicitly say that 'val' is a unicode string. It would be nice if we could make it work without having to specify that.

Just tried this in Python 2.7.12, and this issue does not show up for me. Since b.categories.groupby('key') returns a dict, this would be a consequence to how dicts treat keys that are in Unicode. Which Python version are you using?
One solution is that we could cast the string keys of the dict groupby outputs into ASCII strings, but this could have other issues for non ASCII characters.

I'll try to come up with a minimal example. I definitely noticed it with two keys one string and one numeric.

hm weird I can't reproduce my error. Maybe I made another mistake


Ignore error "dh key too small"

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
Hello,
I try to monitor the web interface of an internet box, but I have the error "dh key too small", so, the website is marked as down.
Can I ignore this error ?
The case for ignore ssl error is already ticked.
Thanks.
🐻 Uptime-Kuma Version
1.11.4
💻 Operating System and Arch
Docker container on Debian 11
🌐 Browser
Chrome, last version
🐋 Docker Version
20.10.12
🟩 NodeJS Version
No response

Is this an HTTPS or HTTP website?
If both, then try the HTTP version, if not, could you send your config for that monitor?
Of course, anonymizing things like the URL.

Unfortunaly, it's an HTTPS only web interface on an appliance (so, I can't change anything on the web server).
The configuration is basic with the case "ignore ssl error" ticked.

I am getting this error when attempting to configure smtp based email notification, nothing to do with my monitoring.  Is anyone else seeing this?
140615271229312:error:141A318A:SSL routines:tls_process_ske_dhe:dh key too small:../deps/openssl/openssl/ssl/statem/statem_clnt.c:2158:

It's an SSL error, so I suppose we can encounter that in multiple situations.
Here the screenshot of the error from Uptime Kuma

a

I figured it out, I had to mess with my notification settings a little and confirm the port from my ISP, I had that wrong.
I am now getting emails.
Nice little program, simple, lite, useful.   I used to fiddle with LibreNMS for similar functionality (and it's hella powerful!) but it was wildly complicated when this does so much for less.
Thanks for the help!

And here a screenshot of the config

:

Yes, this is because of OpenSSL.
A view years back, they changed the Security configuration, to protect users from using weak encryption methods.
This can manifest in multiple errors.
Unable to connect with TLS encryption
routines:tls_process_ske_dhe:dh key too small
stream_socket_client(): SSL operation failed with code 1
OpenSSL Error messages: error:141A318A:SSL

Are the ones that I encountered.
The solution, as described here is to set a lower security level.
On a normal machine, this is done by:
finding your openssl.cnf openssl version -d
and add to the top of that file:
openssl_conf = default_conf

and to the bottom:
[ default_conf ]

ssl_conf = ssl_sect

[ssl_sect]

system_default = system_default_sect

[system_default_sect]
MinProtocol = TLSv1.2
CipherString = DEFAULT:@SECLEVEL=1

Where CipherString = DEFAULT:@SECLEVEL=1 defines the security level.
However, I don't know how to do this in a docker container.

I am getting this error when attempting to configure smtp based email notification, nothing to do with my monitoring. Is anyone else seeing this?
140615271229312:error:141A318A:SSL routines:tls_process_ske_dhe:dh key too small:../deps/openssl/openssl/ssl/statem/statem_clnt.c:2158:

Yes, me to.
Same with a lot of other applications, like Nextcloud, zabbix, etc.
Seems like my Email Provider only supports weak encryption...

I think the best way is to add an environement variable on the Uptime Kuma Docker image, but I suppose the only way actualy is to redirect the openssl folder who contain the conf on the host (not a good practical...).
What do you think ?

The environment Variable would have to be implemented by the developers, which would be amazing, as I currently can't receive email notifications, but I guess it's a wider "problem".
In a perfect world, everything would provide good enough encryption that you won't have to set security levels down.

Hello,
I also have this error with a SMTP server for notifications.
I'm not using SSL, my SMTP server runs on port 25.

We could support it as demonstrated in this answer, but it would require users to build their own container locally. See if you guys are comfortable with this.


ebiten: API to get the current graphics library

My idea is to add a method to UserInterface type (in internal/ui/ui,go) that goes something like this
func (u *UserInterface) getGraphicsDriverName() string {
	if u.graphicsDriver.IsGL() {
		return "opengl"
	} else if u.graphicsDriver.IsDirectX() {
		return "directx"
	} else {
		return "metal"
	}
}

and add a public function somewhere that will call this method somewhere (window.go?) that wraps it and can be called from outside.
If this looks good, I can create a pull request, otherwise let's discuss how to best do this?

Thanks. So rather than an implementation, I'd like to determine the public API first. As this is one of a debug information,
type GraphicsLibrary int

const (
    GraphicsLibraryUnknown
    GraphicsLibraryOpenGL
    ...
)

type DebugInfo struct {
    GraphicsLibrary GraphicsLibrary
}

func ReadDebugInfo(*DebugInfo)

might be good but I'm not 100% sure. Perhaps this should be in ebitenutil rather than ebiten. Let me think more...

u.graphicsDriver.IsGL()
u.graphicsDriver.IsDirectX()

Hm, I don't want to use these API to be honest as I want to remove these in the future... We can record which graphics library is used inside the ui package at the initialization anyway.

We can record which graphics library is used inside the ui package at the initialization anyway.

Then it will require changing graphicsdriver.Graphics interface in order to retrieve this info, I think. In this case, it's reasonable to remove IsGl() and IsDirectX() within the scope of this issue as well.

Perhaps this should be in ebitenutil rather than ebiten.

There are currently checks for IsGl() and IsDirectX() in the driver code though. So maybe in ui:
type GraphicsLibrary int

const (
    GraphicsLibraryUnknown
    GraphicsLibraryOpenGL
    ...
)
type Graphics interface {
   ...
   GetCurrentGraphicsLibrary() GraphicsLibrary
   ...
}

and in ebitenutil something like

type DebugInfo struct {
    GraphicsLibrary GraphicsLibrary
}

func ReadDebugInfo(*DebugInfo) { ... }

func (d *DebugInfo) GraphicsLibraryToString() string { ... }

?

OK let's start with ebiten package (not ebitenutil package) with this API:
package ebiten

type GraphicsLibrary = ui.GraphicsLibrary

const (
    GraphicsLibraryUnknown = ui.GraphicsLibraryUnknown
    GraphicsLibraryOpenGL = ui.GraphicsLibraryOpenGL
    ...
)

type DebugInfo struct {
    GraphicsLibrary GraphicsLibrary
}

func ReadDebugInfo(*DebugInfo)

package ui

type GraphicsLibrary int

const (
    GraphicsLibraryUnknown
    GraphicsLibraryOpenGL
    ...
)

Fix (*ui.graphicsDriverCreatorImpl).newAuto to return a graphics library type. Note that newAuto is defined for various platforms.
Fix ui.newGraphicsDriver() to save the graphics library type to ui.globalState
Add ui.GraphicsLibrary()  to return the current graphics library type
Implement ebiten.ReadDebugInfo


Type refinement requires triple equals

Flow version: 98
Expected behavior
Enumerated objects with a string type field should be refined with double equals.
Actual behavior
Enumerated objects with a string type field require refinement with triple equals.
try flow
type SourceContent = {url : string }
type Content =   {[string]: AsyncValue<SourceContent> } ;

export type FulfilledValue<+T> = {|
  +state: "fulfilled",
  +value: T
|};
export type RejectedValue = {|
  +state: "rejected",
  value: mixed
|};
export type PendingValue = {|
  +state: "pending"
|};
export type SettledValue<+T> = FulfilledValue<T> | RejectedValue;
export type AsyncValue<+T> = FulfilledValue<T> | RejectedValue | PendingValue;

function getSourceContent(
  contents: Content,
  id: string
): SettledValue<SourceContent> | null {

  const content = contents[id];

  // NOTE: content.state === "pending" works!
  if (!content || content.state == "pending") {
    return null;
  }

  return content;
}

Error
32:   return content && content.state != "pending" ? content : null;
                                                     ^ Cannot return `content && (content.state != "pending") ? content : null` because: Either string literal `pending` [1] is incompatible with string literal `fulfilled` [2] in property `state`. Or string literal `pending` [1] is incompatible with string literal `rejected` [3] in property `state`.
References:
13:   +state: "pending"              ^ [1]
5:   +state: "fulfilled",
             ^ [2]
9:   +state: "rejected",
             ^ [3]

This is on purpose. We discourage the use of == (as do most JS style guides), with the exception of == null checks since those are equivalent to === null || === undefined. The semantics are very surprising, and supporting refinements on this operator would have similarly surprising results.

Thanks


Catalyst UI styles overrides styles if project is also using Tailwind

I kinda expected this. So here's what I'm thinking. Tailwind classes in Catalyst UI should have a prefix. This should prevent them from overriding other tailwind styles.

Fixed but don't know in which release. CatalystUI classes have a prefix now so it won't override any styles.


[terra-button] Enable visible focus when button is active

Summary

Added visible focus to button when the active dom element is button
This change was made to allow focus on button when terra-menu is closed

Closes #
Testing

Additional Details

Thank you for contributing to Terra.
@cerner/terra

Are there existing tests that cover the change in if statement?

Closing this PR as changes has been made on terra-popup to return focus back onto terra-button on popup close :  https://github.com/cerner/terra-framework/pull/1641


Do not save the decrypted .env to disk

Just pipe it directly to the source command

https://github.com/vghn/stack/commit/c58b79a3beeb7a58b187e27e38c7efa5f29bb135


Opening config files on Windows

Things to test:

Which command to use?
notepad will probably use windows line return system, so how to manage this?

For now we can't. If we change config files to Windows line return system, loading config will fail.

Since we are using lb_import_config, we could read text files with Windows endings.
However, we have to wait until this libbash.sh issue is closed: https://github.com/pruje/libbash.sh/issues/109
TODO:

[ ] upgrade libbash.sh
[ ] change get_sources() to do the same

Done in commit 7f2d86cff34008e047131101886dcdd5c96d4a9d


Add univ-eiffel.txt.

Add university Gustave Eiffel (previously known as University Paris-Est Marne-la-Vallée).

@vialette Pull request merged. Thank you!


Add HttpUrlConnection instrumentation tests

The footprint of #133 is pretty sizable and will be experimental for a time, so we have decided to defer the tests to a future PR. This is a tracking issue to create appropriate an appropriate InstrumentationTest similar to the okhttp test. This should run on device and exercise the instrumentation against the testing endpoint.

Do assign this to me! Thanks :)

We have this now!


Fix bad appearance in Firefox

Fix for issue #2.

any chance of this getting merged @ellerbrock ?


EVA-1845 — Added commands to compare evidence strings

The commands which I used to compare evidence strings may be useful in the future as well. I have compiled them into a separate document (compare-evidence-strings.md).

Coverage remained the same at 74.783% when pulling ca9d015424ad8dc7424ddb6feacfa37d5645d7ba on tskir:eva-1845-evidence-string-comparison into f9f97034490ead284ffef3268a529f72480f9a95 on EBIvariation:master.


Updating lib ver to main for all jenkinsfile

Signed-off-by: Jeff Lu<EMAIL_ADDRESS>Description
Describe what this change achieves.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

These tests are for this repository and should not affect any downstream libraries/files. So we are good @jordarlu @zelinh @peterzhuamazon


[Waiting for org repo] Add lang to changes.md

Note: This is pending the approval of an org repo.

Fixes #3690

Changes

Adds lang to supported language and updates changes.md.
Checklist

[x] Added markup tests, or they don't apply here because...
[x] Updated the changelog at CHANGES.md

Thanks, once you push to the repo I can merge this.

You forgot LICENSE.

Ah. Does it matter which license I use? I usually use MIT.

Ah. Does it matter which license I use? I usually use MIT.

Not really.  MIT is a great choice.

Done.

When can I expect lang to be listed on the website?

When the next release happens the website will auto-update.  Not sure on release timing ATM.


Notice D - mADX : Activity 1.1.1: Identify the core FHIR resources that are required for implementing FHIR on Aggregate Data Exchange (ADX) in the Android client.

Output: Requirements specification that defines a complex workflow that aggregates patient information across multiple episodes of care we wish to achieve for objectives 1.1 and 1.2. This specification will indicate the FHIR resources required for implementing FHIR on ADX in the Android client.

FHIR Resources for this use case uploaded here


All the Textures in Jei Becoming Error

Mod Name
Industrial Revolution,  Mekanism
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/industrial-revolution
Minecraft version
1.20.1
Describe the bug
Once a specific item from Mekanism is seen while using it with Industrial Revolution the textures become error
Steps to reproduce
1.Get Industrial Revolution , Mekanism and Jei
2.Go to a world and open inventory
3.Search "Creative Tank" or "@Mekanism Creative Tank" the texture fails to load and every single texture in jei becomes Error till game restart.
...
Logs
https://gist.github.com/BerkinBatu/d1f71e1f9b404397651000b3e76212ac
Additional context
This is my first time making an Issue like this so sorry for any thing that is missing.

blueprint is installed and this is asset related, so this might be #919

It could be that but when I remove blueprint while having Industrial Revolution it still turns them into errors

Happening for me without blueprint installed, crashes when trying to render the Creative tank in the creative inventory, error shows up in JEI
Crash report after looking up "tank" in creative inventory: https://dl.hlep.site/crash-2024-04-28_17.49.44-client.txt

Closing as stale / cannot reproduce.
If you can provide the necessary logs and information to reproduce the problem or identify the faulty mod, please open a new issue with all necessary details.


User app: Processing the scanned data does not display name of the drug & does not verify serial number

Comes from issue, after the successful Scandit scan the screen that follows fails to show:

drug name
serial number check

this issue comes from: #286

Can you pls also specify who will be solving this issue?

Hi @rite2bala I have tested this locally and I got a valid result containing the serial number check and drug name, I guess this is because there is not deployed new version yet(the same as I was mentioning before):

Hi @mppub
In order to resolve the confusion -

The last deployment was done in QA on 19 July 2021 and same version in UAT on 23 July 2021.
Could you pls confirm if code commits from your end were done after 19th July?

Which code repository or specific branch  are you using while testing - is the code committed to the main branch?

If you are testing the iOS app - the last version of the iOS app available with us for testing is Thursday, 8 July 2021.
Was there a change to the code after 8 July 2021?

We really want this issue to be solved to be able to test multiple other things including scanning an entire PDF of codes using Scandit
App.and.DSU.Fabric.Feedback.v0.2.pdf
This can turn business critical and can lead to escalation by the end of Aug 12th with the Singapore demo just around the corner.

Hi @rite2bala

The deployment is done on Novartis side, lets ask someone from those people doing the releases
It is the master branch always the latest version (but I had the same result one month ago)
I am testing PWA / WEb version


'Attach to' shows 'Loading contexts' with SQL kernel - after cancelling python isntall dialog

Azure Data Studio Version:

Steps to Reproduce:

Install extension and open an existing Notebook
Cancel the dialog to install python
Attach to is stuck at 'Loading contexts..'

This issue occurs due to the kernel undefined error.
Root cause: Due to the following exception, 'model.contextsChanged' event is not triggered which would call loadAttachToDropdown method. We can add some code within the constructor of AttchToDropdown so that both 'Select Connection' and 'Add new connection' will be displayed. But the bigger issue here is, there is no kernel defined which would prevent us from exceuting any query against SQL kernel. Fixing the kernel undefined issue would make this work.
@kevcunnane @chlafreniere @YurongHe  : Any thoughts?

@kevcunnane @chlafreniere
On further debugging, kernels undefined error occurs due to following reason -
When we open a notebook with spark kernels saved in it, a dialog will be prompted to install python. However cancelling the installation tells us that there is no support for Spark kernels since python is not yet installed in local. ### private getSavedKernelInfo(notebook: nb.INotebookContents) method reads notebook metadata and sets default kernel as the saved Spark kernel and hence we get kernels undefined error.
I have two solutions for this and need your input

Check if python is installed in local from ADS, if not, set the default kernel as SQL no matter what the saved kernel would be.
OR
Alert user with a clean message saying  'You have opened a Spark notebook. Python installation is needed to support it'

Issue is fixed. Please see attached after cancelling python install dialog


WIP: Fit diffs + some flags/functionality + Soft DTW

Added a few fitting functionalities:

Fit differences instead of full values (bit of class trickery to make this nice)
Use yaml to do multiparameter fits with different learning rates
A bit more customizability on the flags passed
Fit as number of iterations rather than epochs
Added in first version of Soft-DTW loss from here. This introduces a numba dependency, and I was having some CUDA errors, so using CPU anyway, so I might just slim this down. Slightly changed loss function structuring to keep this flexible.

I used github actions to automatically build and push the docker image, and also added numba to requirements.txt. Docker image was built and pushed to dockerhub


Support Observable return types for Nestia SDK

Feature Request
When using the Nestia Client SDK, it can properly infer the controller method return value types when returning synchronously or returning a Promise. However, if we try to return an Observable, then the SDK is not able to generate types properly.
Since NestJS has first-class support for Observables, many times we work with Observables exclusively and would just like to return the Observable from the controller method instead of converting it to a Promise and then returning the Promise. However, returning an Observable type breaks Nestia's ability to properly type the response type.
Workaround's we need to use today:

Use async / await and only work with Promises
Use the firstValueFrom rxjs operator to convert Observable to Promise before returning the response from the controller.

This works fine:
   @TypedRoute.Get('user/:id')
   async getUser(@Param('id') id: string): Promise<User> {
     return firstValueFrom(this.userService.findById(id));
   }

This does not:
   @TypedRoute.Get('user/:id')
   getUser(@Param('id') id: string): Observable<User> {
     return this.userService.findById(id);
   }

Can you give me an example repo?
I can't image the Observable type being used in the SDK, because in my memory, it does not have any property mebers and there're only member methods in the type.

Ok, I will try to provide a sample repo tomorrow. Basically, when Nestia sees the Observable being returned, it should essentially consider it the same as Promise since it is irrelevant if Promise or Observable is being returned by the controller method. The fetch functions used by the Nestia SDK will always return a Promise as expected. However,  all information about ReturnType is lost when returning Observable from the controller method, but it works fine when returning Promise.


[BUG] Copying chat not work

Description
When I copy chat, it is not copying correctly.
Screenshots

Additional information
No response
🥦 Browser
Google Chrome
👀 Have you checked if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read and understood the rules in the Contributing Guidelines

Are you willing to work on this issue ?
Yes I am willing to submit a PR!

thanks for wanting to contribute. Make sure to read the issue description carefully and ask if you have questions on the discord server. Follow the rules here, or your PR won't be accepted and will be closed. Good luck!

I've also noticed that if I refresh the screen a couple of times, it'll change my message to this sort of message. Refreshing again will change it back to its normal text but it shows again after a couple more refreshes. Just wanted to add that extra info. I hope it helps.

I've also noticed that if I refresh the screen a couple of times, it'll change my message to this sort of message. Refreshing again will change it back to its normal text but it shows again after a couple more refreshes. Just wanted to add that extra info. I hope it helps.

Thanks for letting me know, please create a new issue highlighting this. Thank you

I've also noticed that if I refresh the screen a couple of times, it'll change my message to this sort of message. Refreshing again will change it back to its normal text but it shows again after a couple more refreshes. Just wanted to add that extra info. I hope it helps.

Thanks for letting me know, please create a new issue highlighting this. Thank you

Okay, will do.


Netcoreapp2.2 target doesn't restore nng.NETCore.dll

Netcoreapp2.2 target projected after adding reference to Subor.NNG.net doesn't put nng.NETCore.dll in right place. Thus you can't use after restore.
PS Package structure quite unusual, didn't get to the root case.

dotnet build should run the build/Subor.nng.NETCore.targets targets script and copy assemblies:
dotnet new console -f netcoreapp2.2
dotnet add package Subor.nng.NETCore -v 1.1.1
dotnet build
(Get-ChildItem -Include nng.NETCore.dll -Recurse).FullName

Shows 3:
bin\Debug\netcoreapp2.2\runtimes\any\lib\net461\nng.NETCore.dll
bin\Debug\netcoreapp2.2\runtimes\any\lib\netstandard1.5\nng.NETCore.dll
bin\Debug\netcoreapp2.2\runtimes\any\lib\netstandard2.0\nng.NETCore.dll


Remove usage of HM Alpha font

Description
The goal of this PR is to remove completely the usage of HM Alpha font, because it is not compatible with all languages and it would create inconsistencies in some screens.
Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[*] Breaking change (change that is not backwards-compatible and/or changes current functionality)
[*] Tests
[*] Documentation
[ ] Other (for changes that might not fit in any category)

Breaking changes
The only breaking change is in code using explicitely the HM Alpha font. It should not be the case in Apps.
But because non-regression tests are using pixel-perfect screens, all screens using this font will be impacted.
Moreover, Upper Case was mandatory in HM Alpha font, but with Inter 32 it will not be the case anymore so strings (for example in case of transaction success), shall be rewritten in Lower Case

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (00e153d) 59.44% compared to head (8acea89) 60.18%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #493      +/-   ##
==========================================
+ Coverage   59.44%   60.18%   +0.73%
==========================================
  Files          12       12
  Lines        1684     1660      -24
==========================================
- Hits         1001      999       -2
+ Misses        683      661      -22

Flag
Coverage Δ

unittests
60.18% <ø> (+0.73%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[FEAT]: Fetch Customer List Endpoint - Backend

Description
Develop an API endpoint to fetch a list of customers.
Acceptance Criteria

The endpoint should be accessible at api/v1/customers?limit=10&page=1.
The endpoint should return a paginated response with an array of customer objects each containing first_name, last_name, email, phone_number and organisations array containing the organisations uuid.
The endpoint should accept HTTP GET requests.
The endpoint should be secured to ensure that only a user with an admin role can fetch the list of customers.
Proper authentication mechanisms (e.g., JWT, OAuth2, SAML) should be implemented.
Requests to the endpoint must include a valid authentication token in the Authorization header. Authorization: Bearer <token>
Response (Success)

{
"status_code": 200,
"current_page": 1,
"total_pages": 20,
"limit": 10,
"total_items": 202,
"data":
[
  {
     "first_name": "Emmanuel",
     "last_name": "xxxxxxxxxxx",
     "phone_number": "xxxxxxxxxxx",
     "organisations": ["xxxxxxxxxxxx", "xxxxxxxxxxx", "xxxxxxxxxxxx"]
  }
  ...
]
}

Response (Error)

{
  "status_code": 401,
  "message": "Unauthorized",
  "error": "Bad Request"
}

Internal Sever Error
{
  "error": Bad Request,
  "message": "Internal server error",
  "status_code": 500
}

Purpose
To fetch the list of customers in the Database
Requirements

[ ] User must be authenticated thereby having a Authorization: Bearer <token> header in the request body
[ ] User must have the role of an admin
[ ] User account must be active
[ ] User must indicate the limit number and page number

Expected Outcome

[ ] A paginated response with array of customers object where each customer object contains first_name, last_name, email, phone_number and organisations array containing the organisations uuid.

Testing

Unit Tests
The systems should have unit tests covering:

[ ] Ensure data is fetched correctly from the database.
[ ] Test that the appropriate response and status code is returned

Check the convention you were asked to follow

Updated as expected @Dev-wonderful

@NdubuisiJr


Switch to 29834.1 workflow for CMSSW check

We need to switch to this new workflow since we are now on CMSSW_14_2_x, and the previous workflow doesn't work anymore.

@ariostas
do I understand correctly that this PR was already used in the most recent tests?

do I understand correctly that this PR was already used in the most recent tests?

Yes. It seems to be working fine, so I think we can merge it

Yes. It seems to be working fine, so I think we can merge it

I recall that you were merging PRs in the actions repository in the past.
So, no need to wait, unless you are implying we should change this MO.


C++   set_allow_soft_placement does not work

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
**OS Platform and Distribution: Mac  CPU
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.8.0
Python version: 2.7

Describe the problem
I have set set_allow_soft_placement(true), but it does not work.
Source code / logs
c++ source code
    const string pathToGraph = “tf/checkpoint/model-1.meta";

    tensorflow::SessionOptions options;
    options.config.set_allow_soft_placement(true);  // why this does not work?
    auto session = NewSession(options);

    Status status = session->Create(graph_def.graph_def());

It get errors in my mac(cpu),  no error in GPU.
./eval
libc++abi.dylib: terminating with uncaught exception of type std::runtime_error:
Error creating graph: Invalid argument: Cannot assign a device for
operation 'ExponentialMovingAverage/AssignMovingAvg_5/sub':
Operation was explicitly assigned to /device:GPU:0 but available devices are
[ /job:localhost/replica:0/task:0/device:CPU:0 ].
Make sure the device specification refers to a valid device.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device

@xu-song Hi, can you share an example to reproduce these errors ?

ok, i will prepare one example soon

simple example
this is a simple example, which get result = a*b+1
python - save model
import numpy as np
import tensorflow as tf
import sys, os

if __name__ == '__main__':
    train_dir = os.path.join('demo_model/', "demo")

    with tf.device('/gpu:0'):
        a = tf.placeholder(dtype=tf.int32, shape=None, name='a')
        b = tf.placeholder(dtype=tf.int32, shape=None, name='b')
        y = tf.Variable(tf.ones(shape=[1], dtype=tf.int32), dtype=tf.int32, name='y')
        res = tf.add(tf.multiply(a, b), y, name='res')

    config = tf.ConfigProto(allow_soft_placement=True)
    with tf.Session(config=config) as sess:
        feed_dict = dict()
        feed_dict[a] = 2
        feed_dict[b] = 3
        fetch_list = [res]
        sess.run(tf.global_variables_initializer())
        saver = tf.train.Saver()

        res = sess.run(feed_dict=feed_dict, fetches=fetch_list)
        saver.save(sess, train_dir)

        print("result: ", res[0])

C++ restore model
#include <iostream>
#include "tensorflow/core/public/session.h"
#include "tensorflow/core/protobuf/meta_graph.pb.h"
#include "tensorflow/cc/client/client_session.h"
#include "tensorflow/cc/ops/standard_ops.h"
#include "tensorflow/core/framework/tensor.h"

using namespace std;
using namespace tensorflow;

int main()
{
    const string pathToGraph = "demo_model/demo.meta";
    const string checkpointPath = "demo_model/demo";
    SessionOptions options;
    options.config.set_allow_soft_placement(true);   // this line does not work
    auto session = NewSession(options);

//    auto session = NewSession(SessionOptions());
    if (session == nullptr)
    {
        throw runtime_error("Could not create Tensorflow session.");
    }

    Status status;
    MetaGraphDef graph_def;
    status = ReadBinaryProto(Env::Default(), pathToGraph, &graph_def);
    if (!status.ok())
    {
        throw runtime_error("Error reading graph definition from " + pathToGraph + ": " + status.ToString());
    }

// 利用读入的模型的图的拓扑结构构建一个session
    status = session->Create(graph_def.graph_def());
    if (!status.ok())
    {
        throw runtime_error("Error creating graph: " + status.ToString());
    }

// 读入预先训练好的模型的权重
    Tensor checkpointPathTensor(DT_STRING, TensorShape());
    checkpointPathTensor.scalar<std::string>()() = checkpointPath;
    status = session->Run(
            {{ graph_def.saver_def().filename_tensor_name(), checkpointPathTensor },},
            {},
            {graph_def.saver_def().restore_op_name()},
            nullptr);
    if (!status.ok())
    {
        throw runtime_error("Error loading checkpoint from " + checkpointPath + ": " + status.ToString());
    }

    std::vector<std::pair<string, Tensor>> input;
    Tensor a(tensorflow::DT_INT32, tensorflow::TensorShape({1}));
    Tensor b(tensorflow::DT_INT32, tensorflow::TensorShape({1}));

    auto a_map = a.tensor<int,1>();
    a_map(0) = 2;
    auto b_map = b.tensor<int,1>();
    b_map(0) = 3;
    input.emplace_back(std::string("a"), a);
    input.emplace_back(std::string("b"), b);

//   运行模型，并获取输出
    std::vector<tensorflow::Tensor> answer;
    status = session->Run(input, {"res"}, {}, &answer);

    Tensor result = answer[0];
    auto result_map = result.tensor<int,1>();
    cout<<"result: "<<result_map(0)<<endl;

    return 0;
}

Nagging Assignee @reedwm: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @reedwm: It has been 30 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @reedwm: It has been 45 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @reedwm: It has been 60 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

simple example
this is a simple example, which get result = a*b+1
python - save model
# demo.py
import tensorflow as tf
import os

if __name__ == '__main__':
    train_dir = os.path.join('demo_model/', "demo")

    with tf.device('/gpu:0'):
        a = tf.placeholder(dtype=tf.int32, shape=None, name='a')
        b = tf.placeholder(dtype=tf.int32, shape=None, name='b')
        y = tf.Variable(tf.ones(shape=[1], dtype=tf.int32), dtype=tf.int32, name='y')
        res = tf.add(tf.multiply(a, b), y, name='res')

    config = tf.ConfigProto(allow_soft_placement=True)
    with tf.Session(config=config) as sess:
        feed_dict = {a:2, b:3}
        fetch_list = [res]
        sess.run(tf.global_variables_initializer())
        saver = tf.train.Saver()

        res = sess.run(feed_dict=feed_dict, fetches=fetch_list)
        saver.save(sess, train_dir)

        print("result: ", res[0])

$ python demo.py
2018-09-29 09:35:28.855859: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
('result: ', array([7], dtype=int32))

python - restore
# demo-restore.py
import tensorflow as tf
import os
import numpy as np

if __name__ == '__main__':
    config = tf.ConfigProto(allow_soft_placement=True)
    with tf.Session(config=config) as sess:
        saver = tf.train.import_meta_graph('demo_model/demo.meta')
        saver.restore(sess, tf.train.latest_checkpoint('demo_model/'))
        graph = tf.get_default_graph()
        a = graph.get_tensor_by_name("a:0")
        b = graph.get_tensor_by_name("b:0")
        feed_dict = {a: 2, b: 3}

        op_to_restore = graph.get_tensor_by_name("res:0")
        print(sess.run(fetches=op_to_restore, feed_dict=feed_dict))

$ python demo-restore.py
[7]

C++ restore model
// demo.cc
#include <iostream>
#include "tensorflow/core/public/session.h"
#include "tensorflow/core/protobuf/meta_graph.pb.h"
#include "tensorflow/cc/client/client_session.h"
#include "tensorflow/cc/ops/standard_ops.h"
#include "tensorflow/core/framework/tensor.h"

using namespace std;
using namespace tensorflow;

int main()
{
    const string pathToGraph = "demo_model/demo.meta";
    const string checkpointPath = "demo_model/demo";
    SessionOptions options;
    options.config.set_allow_soft_placement(true);   // this line does not work
    auto session = NewSession(options);

//    auto session = NewSession(SessionOptions());
    if (session == nullptr)
    {
        throw runtime_error("Could not create Tensorflow session.");
    }

    Status status;
    MetaGraphDef graph_def;
    status = ReadBinaryProto(Env::Default(), pathToGraph, &graph_def);
    if (!status.ok())
    {
        throw runtime_error("Error reading graph definition from " + pathToGraph + ": " + status.ToString());
    }

   // restore graph
    status = session->Create(graph_def.graph_def());
    if (!status.ok())
    {
        throw runtime_error("Error creating graph: " + status.ToString());
    }

    //  restore checkpoint
    Tensor checkpointPathTensor(DT_STRING, TensorShape());
    checkpointPathTensor.scalar<std::string>()() = checkpointPath;
    status = session->Run(
            {{ graph_def.saver_def().filename_tensor_name(), checkpointPathTensor },},
            {},
            {graph_def.saver_def().restore_op_name()},
            nullptr);
    if (!status.ok())
    {
        throw runtime_error("Error loading checkpoint from " + checkpointPath + ": " + status.ToString());
    }

    std::vector<std::pair<string, Tensor>> input;
    Tensor a(tensorflow::DT_INT32, tensorflow::TensorShape({1}));
    Tensor b(tensorflow::DT_INT32, tensorflow::TensorShape({1}));

    auto a_map = a.tensor<int,1>();
    a_map(0) = 2;
    auto b_map = b.tensor<int,1>();
    b_map(0) = 3;
    input.emplace_back(std::string("a"), a);
    input.emplace_back(std::string("b"), b);

   //   run session
    std::vector<tensorflow::Tensor> answer;
    status = session->Run(input, {"res"}, {}, &answer);

    Tensor result = answer[0];
    auto result_map = result.tensor<int,1>();
    cout<<"result: "<<result_map(0)<<endl;

    return 0;
}

error - in cpu
./demo
libc++abi.dylib: terminating with uncaught exception of type std::runtime_error: Error creating graph:
Invalid argument: Cannot assign a device for operation 'y': Operation was explicitly assigned to
/device:GPU:0 but available devices are [ /job:localhost/replica:0/task:0/device:CPU:0 ]. Make sure the
device specification refers to a valid device.
	 [[Node: y = VariableV2[_output_shapes=[[1]], container="", dtype=DT_INT32, shape=[1],
shared_name="", _device="/device:GPU:0"]()]]
Abort trap: 6

no error - in gpu

because you train your model with gpu,you can try load your model just cpu type

@xu-song,
Sorry for late response. Can you please a take a look at this comment ?
Whether soft placement is allowed. If allow_soft_placement is true,
an op will be placed on CPU if

there's no GPU implementation for the OP
or
no GPU devices are known or registered
or
need to co-locate with reftype input(s) which are from CPU.

Thanks!

Does this problem have been fixed?


Set destination folder for new note

Hi,
thank you very much for developing this plugin.
I have everything set up and it works, but I am stuck on one thing.
My use case is that I listen to a podcast in Spotify and spontaneously decide that I want to have a note about it in Obsidian.
I open Obsidian and then select the option “create file for currently playing episode using template” and save the current podcast. (I have created a template and it works great).
Unfortunately, the plugin creates a new folder with the name of the note I was in before.
This is of course a completely different one.
I would appreciate the option of selecting a fixed destination folder (or a completely new note).
I've tried various things with the name in default destination and the toggle buttons, but the new note never ends up in the folder I want.
Is there a specific name (path) for the default destination?
I am a total beginner and have no clue how file names etc. should be named correctly.
Maybe I have overlooked something else....
Is it also possible to be taken there directly after creating the note?
I would like to add a comment to my note explaining why I have created a note for it.
At the moment I am always in the note I was in before and I have to switch to the new note myself.
Many thanks in advance!!
M.

I tried a bit more and I think I have found the error.
There are other options in the command palette than the one I used:
In the note itself, there are also selection options via the three dots at the top right “more options” at the bottom and it always goes wrong here.
Perhaps I always select the wrong command here and the plug-in does exactly what it should....
The name is also shorter than via the command palette.
I'll have a look at this and probably build a hotkey...
But thanks again for the plugin and the further development :-)
All the best for you
M.

Hello !
Thanks for asking !
The plugin has multiple ways to create new notes, based on what I tested, all of them automatically open the note upon creation when the Auto Open is set to On.
When the note already exists and the overwrite is Off, the existing note will open, but no changes will be applied.
For creation:

You can right click on any existing  directory and select from one of the four options to create and automatically be redirect to the new note, and the new note will be created under the selected Directory.
Use a hotkey or the command palette (CMD+P), this one will create the note in the directory defined in the settings

If the Default destination is unset it will put the note at the root of your vault.
otherwise it creates the note in the defined directory.

This is the config I used when testing and before releasing new version

Let me know if I answered your questions.

Yeah you are right !
There are 8 of them with the command palette,
4 actions that create a new file following the Default destination (the ones that starts with create file)
and the 4 others are appending in the opened file
Feel free to reopen if anything


⚠ ClamAV has degraded performance

In 9db9658, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 202 ms

Resolved: ClamAV performance has improved in 0a7bb09 after 540 days, 14 hours, 47 minutes.


feat: switch to term/ansi for text manipulation

Use ANSI aware, wide characters support, uniseg backed term/ansi package to calculate string widths, truncate, and wrap strings.
Related: https://github.com/muesli/reflow/pull/71
Fixes: https://github.com/charmbracelet/lipgloss/issues/258
Related: https://github.com/charmbracelet/lipgloss/issues/220

@aymanbagabas We need Bubbles to be updated ASAP as this will cause major inconsistencies.


FIX fixed complex type with create_column_const_with_default_value

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run arm

run arm

run p0

run buildall

run buildall

run arm

run ut

run clickbench

run beut

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 33.04 seconds
stream load tsv:          422 seconds loaded<PHONE_NUMBER>9 Bytes, about 169 MB/s
stream load json:         22 seconds loaded<PHONE_NUMBER> Bytes, about 102 MB/s
stream load orc:          72 seconds loaded<PHONE_NUMBER> Bytes, about 14 MB/s
stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230410052038_clickbench_pr_126769.html


Create new crates.io release

The last release on crates.io is 4 months old and there have been several features implemented since then. Please push a new release to crates.io with the latest updates.

Sure thing. Once the pending pull requests have landed I'll cut another dev release.

ion-rust v0.9.0 is now available.


ISSUE-3266: Add unit test for vertica

Describe your changes :

I worked on adding vertica unit test
fix #3266

Type of change :

[x] Improvement

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

Thanks @MilanBariya


Update version for the next release (v0.13.1)

This version makes this package compatible with Meilisearch v1.0.0 🎉
Check out the changelog of Meilisearch v1.0.0 for more information on the changes(https://github.com/meilisearch/meilisearch-dotnet/pull/383).
🚀 Enhancements

Key class has been changed; all the Key fields were available in the structure, but some of them aren't needed when creating a Key.

createdAt has been JsonIgnore
updatedAt has been JsonIgnore
key has been JsonIgnore

Thanks again to @alallema,  @brunoocasali and @juchom ! 🎉

bors merge


feat: use Arc for store_dir

Why is it draft?

There could be more copying that can be benefited from Arc::clone.
I might find a better way (I haven't found it yet).

@anonrig I think I've found the better way: https://github.com/pnpm/pacquet/pull/115


Time Input crashes when removing ':'

Version: 2.7.0
Steps to reproduce

Use 'showTimeInput'.
Use a browser which makes <input type="time" /> act like a normal input (Safari)
Remove the ':' between hours and minutes in the time input.
Result: App crashes with a 'RangerError: Invalid time value'

Safari is not mentioned in the list of compatible browsers. Have you checked this in other browsers?
It works for me in firefox and chrome lastest versions.
If it is only in safari, then I would suggest you to close it.

https://github.com/Hacker0x01/react-datepicker/pull/1841 can fix it

I am also having this issue in Safari. In the meantime, I changed my Datepicker to use showTimeSelect instead of showTimeInput: https://reactdatepicker.com/#example-select-time

This is still happening in version 2.10.1


x-nullable not working

According to here: https://github.com/Azure/autorest/blob/master/docs/extensions/readme.md#x-nullable
It should be able to be specified on a request body, response body, schema usage (ref or in-line declaration), and properties/parameters. Right now, it seems like it is either A, not handled (comes out in the extensions list) or B, handling is broken (not in extensions list, but nothing else changes either). I only tested it on a property, a parameter, and a response, but none of them worked properly. There is a Nullable property for Value in the code model that should be holding this information. There are not test server tests for this (yet).
** REQUIRED **
ModelerFour version: 4.13.312
AutoRest-core version: 3.0.6277
OpenAPI/Swagger file: Modified an existing one and compared code model output

This is now implemented in<EMAIL_ADDRESS> Please give it a shot and let me know if it helps!  /cc @pakrym

Works in most cases, if there are new bugs I'll file them separately.


Using publickey.PublicKey().find_program_address() getting NotImplementedError

Hello guys!
I am try to get PDA for a mint address for later que metadata for NFTs but when I use the function I am getting that error!
Probably I am using wrong the function. I really do not know.
Thanks before hand.
`from solana.publickey import PublicKey
METADATA_PROGRAM_ID = PublicKey('metaqbxxUerdq28cj1RbAWkYQm3ybzjb6a8bt518x1s')
mint_key = "8Rjxaxy8ZNAGTojzQSAMSPatNiop9C9PRPY5A29qhrCT"
PDA = PublicKey.find_program_address(
[b'metadata', bytes(METADATA_PROGRAM_ID), bytes(PublicKey(mint_key))],
METADATA_PROGRAM_ID
)[0]
print(PDA)`

Which version of solana-py are you using?

Yes it was version of solana-py!
Just update is working now!
Thanks!


Provide an easy way to edit the googletest arguments in integration tests

Android and iOS do not have a way to take command-line arguments as input, and therefore there is no way to specify useful arguments like --gtest_test_filter when developing/debugging a specific test. This PR modifies firebase_test_framework.cc to at least provide an easy way to hardcode googletest arguments during development.

Are the changes clang-formatted? (If not, does GitHub automatically format it before merging?)
Yes, the file has been formatted with clang-format. GitHub does not automatically format but there is a presubmit check that verifies that code is formatted correctly and also an (optional) commit hook that you can install locally. See https://github.com/firebase/firebase-cpp-sdk/pull/380 for details.

How is this firebase_test_framework invoked? The cleaner approach seems to be to pass --gtest_filter to that invocation (so that it's already in argv when we get to common_main)
It would definitely be cleaner to just specify arguments on the command line. You can indeed do this when running the desktop tests. However, Android has no means of specifying command-line arguments and we have not plumbed through the command-line arguments in iOS (if it's even possible). There are probably some platform-specific mechanisms that could be used to specify the arguments without having to edit source code; however, this is just a quick-and-dirty way to get it done.

Got it. Thanks


Fix edit patient

Implement crud for identifiers on edit patient.

 Allow the removal of newly added identifiers in edit patient identifiers

Coverage increased (+0.9%) to 48.192% when pulling 2dc71b6a924391c66b35ffddd23067fcf2846644 on ynurmahomed:remove-identifiers into 03ea0131f0216221060d485c43cfac3471efdfc9 on esaude:master.

Coverage increased (+2.6%) to 49.92% when pulling 64d7025e07405ddb8256683ca735ab369dac48f9 on ynurmahomed:remove-identifiers into 03ea0131f0216221060d485c43cfac3471efdfc9 on esaude:master.

@edrisse I think this one is ready, fixes were done


transform api call not returning

I am using umap library for my project.  umap.fit and other methods are working fine. But when I  transform additional points after fitting, by calling umap.transform, it is hanging and not returning any value. The dataset I am trying is a simple one with less than 50 rows. Any help would be much appreciated.

Did you solve this? Having the same issue. Fitting with a list of 512-item-arrays is no problem. But transforming a single 512-item-array afterwards seems to crash the site.

I used 'fitAsync' eventually and that worked

Managed to find my issue as well. I used a different random method from the seedrandom package to create my umap object. Worked fine during fitting, but apparently breaks stuff as soon as you try to transform anything with it.
Thanks for your quick reply though!


How do I hide the header completely?

Hi I would like to hide the header completely so that it doesn't take up any space?
If I set display none from developer tools it is removed but can't do it from my application.
Would be nice to have a prop for this, can't see there are any right now?
<template>
  <vue-advanced-chat
  >

  </vue-advanced-chat>
</template>

.vac-room-header {
  display: none !important;
}

You can override css like this:
const style = document.createElement('style')
style.innerHTML = '.vac-room-header { display: none !important; }'
this.$refs.chatWindow.shadowRoot.appendChild(style)

That worked! Thank you and for the amazing component!

Hi @antoine92190 ..... i want to update css of chat component but cant do that after installing 2.0.4 version ....  i have read above all comments ..... shadowRoot undefined issue in console log..... even i am using ref="chatWindow" in </vue-advanced-chat/> component.... can u guide me how to resolve this....


CSS conflicts

There are a few lines in the nifty CSS that conflict with our site's CSS, e.g. the following line:
@media screen and (max-width:32em){body{font-size:75%}}
Clearly, this causes all of our texts to shrink, which is not desirable. Please provide CSS that only touches the modals, such as:
@media screen and (max-width:32em){body .nifty-modal{font-size:75%}}

Yeah, generalized usage wasn't the biggest prority when I first wrote Nifty. This issue was also recently brought up in #7. So I will do a bit of a clean up on the css.
Thanks for bringing this to my attention!

Doesn't look like there are too many places, nifty modals seem to isolate their CSS for the most part.


DDC under NNBD instantiates raw types with "Object?" instead of "dynamic"

I'm migrating language/closure/type_arguments_test.dart to NNBD. It fails on line 40 of that CL because the instantiated type arguments are Object? instead of dynamic. Most real-world code won't care about the difference, but code that happens to check for very specific runtime types (like this test) can. I believe dynamic is the specified behavior and is what the VM does.
cc @eernstg

Note: The original test language_2/closure/type_arguments_test is passing on the ddc-nnbd-linux-release-chrome configuration in weak mode.

Cf. https://github.com/dart-lang/language/issues/908, that I created in order to clarify the language definition part of this question. There's a bias in the direction of saying that DDC should infer dynamic, but I haven't got explicit buy-in for a conclusion either way yet.

dart-lang/language#908 was concluded with the decision that I mentioned above: DDC should infer dynamic rather than Object? such that the failure in https://logs.chromium.org/logs/dart/buildbucket/cr-buildbucket.appspot.com/8884178758135030624/+/steps/test_results/0/logs/new_test_failures__logs_/0 doesn't occur.
I just checked, and it does fail with the same symptom in commit 01d9d1ca81b0a45b14364ac01ed4cdf664b7a595 (from today).


Update dbcc-checkdb-transact-sql.md

Fixed a typo (argment -> argument) and added a note about TABLOCK when the database is in single_user mode (checkcatalog still happens in that case).

@AaronBertrand : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@pmasl Could you take a look at the added statement in the file changes tab. @AaronBertrand thanks for the suggestion. Do you know of any supporting documentation in our docs or the community that supports this addition (just to speed things up). Thanks!

@pmasl Could you take a look at the added statement in the file changes tab. @AaronBertrand thanks for the suggestion. Do you know of any supporting documentation in our docs or the community that supports this addition (just to speed things up). Thanks!

Do you know of any supporting documentation in our docs or the community that supports this addition (just to speed things up). Thanks!

I don't, sorry, it is just based on my observation and that it is easy to reproduce.

Do you know of any supporting documentation in our docs or the community that supports this addition (just to speed things up). Thanks!

I don't, sorry, it is just based on my observation and that it is easy to reproduce.

#pleasereassign:bluefooted

#pleasereassign:bluefooted

Thanks again @AaronBertrand for this contribution. We are unable to confirm this is the exact behavior as your change describes. I'm going to close this PR for now and see if we can get more attention from the product group re: the interaction between TABLOCK and single_user mode for DBCC CHECKCATALOG. If you are able to provide more here, would be happy to review internally.


Update virtualbox.rb

Upgraded to 5.0.16-105871

Thank you for the contribution. It was merged directly as 9591225e0e1f2a354b18b9c925d26f38682c9909 to keep commit history cleaner. Your contribution is still credited to you.

Also, check out cask-repair, it makes Cask updates like this quite easy (details in CONTRIBUTING)


Error in AwsTileRequest

Hi I'm getting error when I'm trying to create AwsTileRequest object. Even when I'm trying to run the sample code mentioned in the documentation I'm facing the below error.
`
from sentinelhub import AwsTile
tile_id = 'S2A_OPER_MSI_L1C_TL_MTI__20151219T100121_A002563_T38TML_N02.01'
tile_name, time, aws_index = AwsTile.tile_id_to_tile(tile_id)
from sentinelhub import AwsTileRequest
bands = ['B8A', 'B10']
metafiles = ['tileInfo', 'preview', 'qi/MSK_CLOUDS_B00']
data_folder = './AwsData'
request = AwsTileRequest(tile=tile_name, time=time, aws_index=aws_index,
bands=bands, metafiles=metafiles, data_folder=data_folder,
data_source=DataSource.SENTINEL2_L1C)

TypeError                                 Traceback (most recent call last)
 in ()
7 request = AwsTileRequest(tile=tile_name, time=time, aws_index=aws_index,
8                          bands=bands, metafiles=metafiles, data_folder=data_folder,
----> 9                          data_source=DataSource.SENTINEL2_L1C)
~/.conda/envs/web-static/lib/python3.6/site-packages/sentinelhub/data_request.py in init(self, tile, time, aws_index, data_source, **kwargs)
676         self.data_source = data_source
677
--> 678         super(AwsTileRequest, self).init(**kwargs)
679
680     def create_request(self):
~/.conda/envs/web-static/lib/python3.6/site-packages/sentinelhub/data_request.py in init(self, bands, metafiles, safe_format, **kwargs)
587
588         self.aws_service = None
--> 589         super(AwsRequest, self).init(**kwargs)
590
591     @abstractmethod
~/.conda/envs/web-static/lib/python3.6/site-packages/sentinelhub/data_request.py in init(self, data_folder)
39         self.download_list = []
40         self.folder_list = []
---> 41         self.create_request()
42
43     @abstractmethod
~/.conda/envs/web-static/lib/python3.6/site-packages/sentinelhub/data_request.py in create_request(self)
684         else:
685             self.aws_service = AwsTile(self.tile, self.time, self.aws_index, bands=self.bands,
--> 686                                        metafiles=self.metafiles, data_source=self.data_source)
687
688         self.download_list, self.folder_list = self.aws_service.get_requests()
~/.conda/envs/web-static/lib/python3.6/site-packages/sentinelhub/aws.py in init(self, tile_name, time, aws_index, data_source, **kwargs)
467         self.data_source = data_source
468
--> 469         super(AwsTile, self).init(**kwargs)
470         self.tile_url = None
471
TypeError: super(type, obj): obj must be an instance or subtype of type
`

Now I haven't got an issue when I downloaded the package from git repo. But observing the issue if donwload from pip install sentinelhub.

The problem does not appear to me neither does it appear in unit tests.
However I think I know the reason for this. The error happens if you work in Jupyter Notebook, import a class from a package, then change something in definition of the class  (in your case AwsTile.tile_id_to_tile), reinstall the package (if needed) and continue working with that class in the same notebook. Jupyter then somehow fails to update the definition of the class.
The solution for this is to restart the Jupyter's kernel and rerun the notebook.


Docusaurus main page re-design

Is your feature request related to a problem? Please describe.

We need to improve the layout of the current main-page of the documentation at https://rancher.github.io/dashboard/
💢 Important: the design should be consistent with the others sections (https://rancher.github.io/dashboard/home)
Some ideas:

Beside the main two blocks, find some other information that might be useful for the user; ex, quick access to Extensions, Tests..
Improve footer information, we might want to remove the links and have them as cards in the main content.

CC @kwwii

After running a poll (and directly asking a developer), we got some answers. Not a lot, but enough to start working with:

These are the documentation sections devs are more likely to look for. Combining that information with our perspective of what we want to expose, I built this first approach:
Wireframe
This is actually a refinement after Manuele's comments on the very first wireframe.

I've revisited some sections after reading the comments in the Xd prototype.
We still need some stuff:
@manuelecarlini we need a "Concepts" picture. "Concepts" is a kind of glossary of Rancher, you can have a look here: Rancher Concepts
@kwwii I need some native English support with the sections subtitles:

I can come up with some content but it would require at least a review.

Final proposal

Design finished, PR: https://github.com/rancher/dashboard/pull/7837


#1086 identifier type in proxies

See #1086. This fix simply ensures that the column type for identifiers of proxies of STI/JTI are kept consistent with the DBAL types they are mapped to.
May be related/colliding with #1178
Ping @jaspernbrouwer

@jaspernbrouwer found that the PersisterHelper is actually very useful in here: currently integrating it.

Handled in doctrine/doctrine2@6164f17229a820e9be30200c6b8424034acbc560


docs: cover URLs generation topic

add test that covers both custom() in a POST request and get_body() - not covered so far

Codecov Report

Merging #220 (21bfa20) into master (b81008a) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 21bfa20 differs from pull request most recent head 5c3833d. Consider uploading reports for the commit 5c3833d to get more accurate results

@@           Coverage Diff           @@
##           master     #220   +/-   ##
=======================================
  Coverage   92.89%   92.89%
=======================================
  Files           6        6
  Lines        2802     2802
=======================================
  Hits         2603     2603
  Misses        199      199

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b81008a...5c3833d. Read the comment docs.


(PUP 7779) [#puppethack] Resurrect puppet facts upload

This changeset reverts pieces of PUP-2560 in order to re-add the ability to upload facts outside of a catalog request. Specifically, this patch re-adds the following functionality:

The puppet facts upload command.

The Puppet::Node::Facts::Rest terminus for saving facts to Puppet Servers.

The ability to handle requests for the /puppet/v3/facts endpoint.

Once this is merged, puppetlabs/puppetserver#1578 can be merged to add the /puppet/v3/facts route to the Puppet Server API.

CLA signed by all contributors.

Oddly the style check for commit messages was fine with (SERVER-) being used in a6dd8b9 but not cf51e04...

Alright, I'm going to be that guy. Despite everything old being new again, I don't think we should be adding indirection termini or server-side routes in Ruby in 2017 (or waiting and doing it in 2018, either).
The approach I would prefer to see taken is the the upload command making an HTTP PUT (not using the indirector) to a route not in the /v3 routes, with that corresponding new route implement on the server side in clojure (not delegating to Ruby).
That might not be feasible for reasons, but I'd like to discuss those reasons or at least have them articulated here and/or in the ticket for posterity's sake.

For me, the main reasons come down to avoiding code duplication and creating divergent methods for serializing data between wire formats. Additionally, the goal is to replicate the "fact update" effects of a catalog request without the overhead of catalog compilation, and the relevant code pathways from a catalog request involve using the indirector.
PuppetDB and Puppet Server both have different wire formats for fact data:

https://github.com/puppetlabs/puppet/blob/5.3.3/api/schemas/facts.json
https://puppet.com/docs/puppetdb/5.1/api/wire_format/facts_format_v5.html

The server-side library for translating between the two formats is part of the terminus code in the PuppetDB project along with other helper code related to forming and profiling PuppetDB API requests:
https://github.com/puppetlabs/puppetdb/blob/5.1.3/puppet/lib/puppet/indirector/facts/puppetdb.rb#L19-L54
On the agent side, much of the logic around converting the Puppet::Node::Facts object returned by Facter to the appropriate HTTP call and payload format lives in Puppet::Indirector::REST along with logic for adding expected headers and selecting the right server in a HA configuration.
The major benefit from shifting this operation over to Clojure would be reducing load on the JRuby worker pool. I would expect this to provide significantly better performance in an environment where a lot of nodes were generating fact upload activity.
However, on the con side we would have to duplicate the agent and server side behaviors listed above and ensure parity is maintained where needed on an ongoing basis. Additionally, it is currently possible for folks to run Puppet Server with something other than PuppetDB as the facts cache by configuring routes.yaml to used a different indirector terminus. Implementing this behavior solely in Clojure wouldn't work for those use cases. Finally, there is a secondary benefit to having an API for independently exercising the same code paths used to persist facts during catalog compilation. This  is useful in certain debugging scenarios as a method for dividing the problem space.
So, there is the potential for significant performance gains from moving this into Clojure, but at the cost of an initial re-write plus ongoing maintenance and potential inconsistency compared with how fact caching behaves during catalog requests. On the balance I think this weighs out in favor of re-using the indirector framework to submit facts.

Updated puppet facts upload to use node_name_value and node_name_fact instead of certname.

$ sum ~/.puppetlabs/opt/puppet/cache/yaml/facts/localhost.yaml
25923 4 /Users/eric.delaney/.puppetlabs/opt/puppet/cache/yaml/facts/localhost.yaml
$ bx puppet agent -t
Info: Using configured environment 'production'
Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Retrieving locales
Info: Caching catalog for localhost
Info: Applying configuration version '1519415605'
Notice: Applied catalog in 0.02 seconds
$ sum ~/.puppetlabs/opt/puppet/cache/yaml/facts/localhost.yaml
28996 4 /Users/eric.delaney/.puppetlabs/opt/puppet/cache/yaml/facts/localhost.yaml
$
$ bx puppet agent -t --use_cached_catalog
Info: Using cached catalog from environment 'production'
Info: Applying configuration version '1519415605'
Notice: Applied catalog in 0.02 seconds
$ sum ~/.puppetlabs/opt/puppet/cache/yaml/facts/localhost.yaml
28996 4 /Users/eric.delaney/.puppetlabs/opt/puppet/cache/yaml/facts/localhost.yaml
$
$ bx puppet facts upload
Notice: Uploading facts for 'localhost' to: 'localhost'
$ sum ~/.puppetlabs/opt/puppet/cache/yaml/facts/localhost.yaml
28719 4 /Users/eric.delaney/.puppetlabs/opt/puppet/cache/yaml/facts/localhost.yaml
$

@Sharpie Could you squash the commits or change them to be (PUP-8232) so that the commit checker will be happy.

I suppose I could re-tag the SERVER-2054 commits as PUP-8232, but the commit checker not recognizing that Puppet changes may be required to implement Puppet Server features seems like a bug in the commit checker.

@Sharpie the commit checker for the puppet repo is very picky it needs to be a PUP ticket or one of the allowed tags or it will complain. That's true for all commits that get added.
The SERVER ticket and PR are find, its just this one since its going into puppet

Right. But those particular Puppet code changes are for the purpose of a implementing a server API endpoint. Those commits are for a SERVER feature, some of the code just happens to live in the Puppet repo for historical reasons.

@justinstoller Could you please review this, I've tested it and SERVER-2054 together and we can properly upload facts. I added the auth updates to my server running from source and was able to unable to access other nodes facts (like we want).

CLA signed by all contributors.

Hi @Sharpie, with the 5.5.x branch being cut, please retarget that branch.  Thanks!

So I think we're in good shape, just had a few final comments:

Adding the rest terminus for facts automagically adds support for puppet facts find --terminus rest. Should the default auth rules allow download too? If it's not needed I'd prefer to only allow save and not find by default.
Since this is landing in 5.5, I'd feel better if both webrick/passenger and puppetserver supported the same REST endpoints and auth rules. Could you update conf/auth.conf?
I think we want to allow the caller to specify the environment (such as in agent-specified environment workflows), but the Puppet::Application::Facts assumes the per-environment directory exists locally. We probably want to add environment_mode :not_required like we do for Puppet::Application::Config.

Adding the rest terminus for facts automagically adds support for puppet facts find --terminus rest. Should the default auth rules allow download too? If it's not needed I'd prefer to only allow save and not find by default.

Good point. puppetlabs/puppetserver#1578 only adds a HTTP route handler for PUT requests, so find wouldn't work anyway. If folks want to retrieve facts via an API, then PuppetDB is the thing to use.

Since this is landing in 5.5, I'd feel better if both webrick/passenger and puppetserver supported the same REST endpoints and auth rules. Could you update conf/auth.conf?

👍 It's a simple change.

I think we want to allow the caller to specify the environment (such as in agent-specified environment workflows), but the Puppet::Application::Facts assumes the per-environment directory exists locally. We probably want to add environment_mode :not_required like we do for Puppet::Application::Config.

Makes sense. I'll get that added.

Looked into removing the find handler and it doesn't look like there's a good way to do this in the code. It appears that for similar non-sensical requests like destroy on /puppet/v3/catalog or find on /puppet/v3/reports we handle this situation by:

Not having a Ring handler that responds to that HTTP method in Puppet Server, which results in a 404 error.

Using the default "deny all" behavior of auth.conf to reject that method with a 403 Forbidden in a Rack Puppet Master.

So, we can likely handle this by only adding handlers and auth rules for save/PUT requests.

Added a legacy auth.conf to keep parity with puppetlabs/puppetserver#1578 for Rack masters and updated lib/puppet/application/facts.rb with environment_mode :not_required.


ENH: switch Dataset and DataArray to use explicit indexes

xref https://github.com/pydata/xarray/issues/1603
This change switches Dataset.indexes and DataArray.indexes to be backed by
explicit dictionaries of indexes, instead of being implicitly defined by
the set of coordinates with names matching dimensions.
There are no changes to the public interface yet: these will come later.
My current plan:

(This PR) Indexes are recreated from coordinates every time a new DataArray
or Dataset is created.
(Follow-up PRs) Refactor indexes to be propagated explicitly in xarray operations. This will facilitate future API changes, when indexes will no longer only be associated with dimensions. I will probably add some testing decorator that can be used to mark part of a test as including no creation of default indexes.
Add explicit entries into indexes for MultiIndex levels that are checked instead of MultiIndex variables. Still no public API changes (aside from adding more entries to .indexes).
Support arbitrary coordinates in indexes.

@shoyer
It looks nice and reasonable to me.

(This PR) Indexes are recreated from coordinates every time a new DataArray or Dataset is created.

Does it add some overhead (though I don't see any reason of it)?

(This PR) Indexes are recreated from coordinates every time a new DataArray or Dataset is created.

Does it add some overhead (though I don't see any reason of it)?

I suppose another option would be to leave self._indexes = None in the constructor, and only set default values for self._indexes when the indexes property is accessed.
I think we do something like for a few other attributes already.


Recreate the Heroku start page within our app

This replicates the start page on https://dfe-tslr-start-page.herokuapp.com/, and changes the root path to direct users to there by default. I'm not sure this is what we'll want to do long term (especially when we move to support Maths and Physics claims too), but it's enough for private beta.

 Recreate the Heroku start page within our app


test(do not merge): rename and move dao file

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Mostly you need to wrap all the value datasource_type value with a DatasourceType enum

closing for https://github.com/apache/superset/pull/20149


Make the added pool more flexible with command like arguments

Right now, we take the prefix as is and add it to Strongswan, we should make it a command line arguments to specify the sla-id, the sla-size and prefix size.
Let's say we get a prefix fd00::1/56 as example, we should be able to specify eg; -sla-id 1,-sla-length 8, and -prefix-length 120.
Which results in the following prefix being added; fd00:0000:0000:0001::1/120.
Another example: sla-id 2, sla-length 4 results with prefix-length 64 in, fd00:0000:0000:0020::1/64. Similar code is available in dhcpcd to add the prefix on interfaces with these variables.
https://www.daemon-systems.org/man/dhcpcd.conf.5.html (see ia_pd section for logic)
getopt() seems the best here. sla-id can have a default value of 0, sla-size also has 0 as default value and prefix size has a default value of 97 (this is the max Strongswan supports).

As discussed, the sla-size is not actually specified.
sla-id has to fit in the space between the wanted prefix size and the total prefix size from the ISP.
The total prefix size is obtained through environment variables from dhcpcd, like the prefix.
Mine is /56.


simplify hello endpoint

Since chiselstrike 0.11, this code can be simpler.

Note that this code does something else:

when you return new Response("hello"), the response is hello with content-type: text/plain
when you return "hello", the response is serialized as JSON, so the body is "hello" with content-type: application/json

Not sure whether this was intentional?

@glommer @MarinPostma

This is fine.
This is just an example application, and the content was not set to text by design anyway.


[bugfix] Add check for illegal character '/' in taskName

Issue
If the taskName contains one or more '/' character in case of user generated tasks, the ZNode gets splits into a subdirectory resulting in the corresponding ZNode name not getting detected by Helix task executor.
Controller generated tasks don’t have this issue because controller has it's own logic to generate the taskName(table name + UUID) which makes sure that the taskName does not contain '/'.
Solution
Addition of validation for taskName so that it does not contain the character '/'.

Codecov Report

Merging #11955 (1046316) into master (6638d4e) will decrease coverage by 26.64%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #11955       +/-   ##
=============================================
- Coverage     61.29%   34.66%   -26.64%
+ Complexity     1140        6     -1134
=============================================
  Files          2385     2309       -76
  Lines        129071   125326     -3745
  Branches      19955    19398      -557
=============================================
- Hits          79120    43439    -35681
- Misses        44220    78826    +34606
+ Partials       5731     3061     -2670

Flag
Coverage Δ

custom-integration1
?

integration
<0.01% <0.00%> (ø)

integration1
<0.01% <0.00%> (ø)

integration2
0.00% <0.00%> (ø)

java-11
<0.01% <0.00%> (-27.61%)
:arrow_down:

java-21
46.44% <0.00%> (-14.85%)
:arrow_down:

skip-bytebuffers-false
<0.01% <0.00%> (-61.28%)
:arrow_down:

skip-bytebuffers-true
46.44% <0.00%> (-14.83%)
:arrow_down:

temurin
34.66% <0.00%> (-26.64%)
:arrow_down:

unittests
46.44% <0.00%> (-14.85%)
:arrow_down:

unittests1
46.44% <0.00%> (-0.04%)
:arrow_down:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

.../apache/pinot/spi/config/task/AdhocTaskConfig.java
76.92% <0.00%> (-6.42%)
:arrow_down:

... and 830 files with indirect coverage changes
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Add newgrounds compatability extension

The extension is an api for interacting with the NEWGROUNDS.IO api which allows you to add medals and grab the users ID and Username from the newgrounds website. More intractability is planned once I figure out more of the API

The icon is temporary.
To note most ES lint errors are from the minified API or the JSON object housing user data.

Also quick note the blocks only activate when on the newgrounds website except for the "Newgrounds?" block

Im trying to fix ESLint Errors

I don't even know how to fix the errors with Crypto.js

do you know if newgrounds' library is under an open source-compatible license?

do you know if newgrounds' library is under an open source-compatible license?

it seems to be fine here is an excerpt from the github
Copyright (c) 2022 Newgrounds Inc.

Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.

 IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

https://github.com/PsychoGoldfishNG/NewgroundsIO-JS

do you know if newgrounds' library is under an open source-compatible license?

Also its the MIT liscense.

Finished porting over the whole API including scoreboards..

I added one quick block that just triggers when a save is completed.


ORA-01034: ORACLE not available on startup

Hi,
after I lunch the container with 2 sql files to initialize the db I get the following errors:
Starting Oracle Net Listener.
Starting Oracle Database 11g Express Edition instance.
/usr/sbin/startup.sh: running /docker-entrypoint-initdb.d/1-create-tables.sql
SQL*Plus: Release <IP_ADDRESS>.0 Production on Wed Dec 7 16:33:25 2016
Copyright (c) 1982, 2011, Oracle.  All rights reserved.
ERROR:
ORA-01034: ORACLE not available
ORA-27101: shared memory realm does not exist
Linux-x86_64 Error: 2: No such file or directory
Process ID: 0
Session ID: 0 Serial number: 0
Enter user-name: Enter password:
ERROR:
ORA-01005: null password given; logon denied
Enter user-name:
/usr/sbin/startup.sh: running /docker-entrypoint-initdb.d/1-insert-alerts.sql
SQL*Plus: Release <IP_ADDRESS>.0 Production on Wed Dec 7 16:33:25 2016
Copyright (c) 1982, 2011, Oracle.  All rights reserved.
ERROR:
ORA-01034: ORACLE not available
ORA-27101: shared memory realm does not exist
Linux-x86_64 Error: 2: No such file or directory
Process ID: 0
Session ID: 0 Serial number: 0
Enter user-name: Enter password:
ERROR:
ORA-01005: null password given; logon denied
Enter user-name:

I start it with:
docker run -d -p 49160:22 -p 49161:1521 oracle

And I create oracle from this Dockerfile
FROM wnameless/oracle-xe-11g

# Initialise with testing data
ADD database_creation.sql /docker-entrypoint-initdb.d/1-create-tables.sql
ADD alerts.sql /docker-entrypoint-initdb.d/1-insert-alerts.sql

@bkolobara Did you find a solution to this issue?

Sadly no. I switched to https://hub.docker.com/r/sath89/oracle-12c/

Can you provide your sql files? I would like to dig into this problem.

@wnameless I was mistaken, we do not ADD any.sql files in our Dockerfile. Thanks for your interest though.

Hi, I found the reason why got ORA-01034 and ORA-27101 on my side, It's because of selinux is turned on on host machine.  To fix it.
modify /etc/selinux/config on host machine
set SELINUX=enforcing to SELINUX=disabled

@zoechen799 Thanks, thats helpful.

This can also happen when the root disk is full. Some versions of Docker for Windows just allocate 20GB of disk space in the VirtualBox VM.  Or you can create a new virtual machine by giving a different name than default.
Note that this command will seem to be stuck in
(default) Creating SSH key...

In the background, VBoxManage is formatting the virtual hard disk. That can take up to an hour. Be patient.


[Requires #343 to engine-api] Adds isolation to info

Signed-off-by: John Howard<EMAIL_ADDRESS>This adds everything needed so that integration tests can be updated so that they know the isolation mode of the daemon under test so that they can take different actions between (on Windows) Window Server containers and Hyper-V containers.
Three commits:

One (hacked pending https://github.com/docker/engine-api/pull/343 being merged into engine-api) that adds isolation to the INFO api call
Second that updates the daemon population of the new field and the client outputting the new field
Third that adds test infrastructure so that tests can now determine the isolation mode of the daemon and be updated accordingly.

Will do the update once the engine-api PR is merged

Closing due to git issues on my dev box. Replaced by https://github.com/docker/docker/pull/26255


Add support for ClusterIdentity

Describe the solution you'd like
[A clear and concise description of what you want to happen.]
For Arc enabled clusters, an identity resource can be created via a ClusterIdentity CRD. Add support for ClusterIdentity to allow access to AKV via this identity.
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

Secrets Store CSI Driver version: (use the image tag):
Azure Key Vault provider version: (use the image tag):
Kubernetes version: (use kubectl version):
Cluster type: (e.g. AKS, aks-engine, etc):

Azure Key Vault Secrets Provider extension to fetch secrets into Arc clusters is in preview. It supports getting the secrets from AKV using service principal. The plan is to deprecate user-assigned and system-assigned managed identity for this provider and only support workload identity.
Closing this issue as we will not support cluster identity. We should plan for using workload identity with the arc extension when workload identity support is available in arc.


Add STM32F429-discovery support

Summary from #1 (as asked in https://github.com/littlevgl/lv_platformio/issues/1#issuecomment-517595070).
It would be nice add STM32F429-discovery support or remove build target to avoid users confusion. Adding support is preferable.
What we have now

Skeleton to easy extend demo with new targets and keep common files free of cryptic guards:

https://github.com/littlevgl/lv_platformio/tree/master/drivers
https://github.com/littlevgl/lv_platformio/blob/master/platformio.ini

Discussion how to activate LCD drivers (were build errors):

https://community.platformio.org/t/how-in-include-disco-f429zi-headers/7001/13?u=vit (see also up and down)
ignore mentions of variant.json edit - already fixed.

Something outdated in stale branch https://github.com/littlevgl/lv_platformio/tree/stm - i don't know if it has something reuseable, so not deleted yet.

What would be nice to get
Minimal case:

HAL-based code, runnable on STM32F429-discovery, with display & touch support.

Maxmal case:

Previous + third target, mbed-based (instead of HAL). Reasons - people like to use mbed and create many drivers for it.

Notes:

Avoid use lv_drivers (except SDL case). PIO has huge number of driver packages for different peripherals & boards. Need only small local wrappers for lvgl api.
https://github.com/platformio/platform-ststm32/tree/master/examples - may be useful to inspect.

So for the first try would it work to add the existing tft and touchpad driver (with the required updates) to stm32f429_disco folder and initialize them in driver.c in hw_init()?

~ Yes. driver.c should also have all mandatory interrupt handlers (if HAL not provide those, i don't know details).
Also, not sure is it worth call anything  from hw_loop(). May be worth use HAL's 1ms system ticks or timer. There are no any restriction HOW to implement details. The only idea is to isolate platform-dependent code.

I'll give it a try next week. If somebody wants to try it before that, go ahead!

IMO those frequent stalebot messages are annoying. At least, i see not reason use that bot in this repo.

I've created the pinned label. Pinned issues won't be marked as stale.

I don't know why it may worth mark all as pinned instead of just disabling stalebot. I'd suggest enable it only after there appears 2-3 issues where it could be useful.

I don't know why it may worth mark all as pinned instead of just disabling stalebot.

Because, in many cases, people open issues to report a bug/ask a question, we fix the bug/respond, and they do not write anything back or close the issue.
Stale saves us from having to check all the repositories every month and close the old issues.

Because, in many cases, people open issues to report a bug/ask a question, we fix the bug/respond, and they do not write anything back or close the issue.

That's good in theory, but i speak about particular case. Here are no users, developers only. As a result - bot kicks developers instead of intended target.

Thre are really not too much issue here. I've disabled stale here.

Let's close it, since reached acceptable state.


Suggestion: remove requirements for gettext and graphviz for testing

Right now, we have a pretty heavy backend Dockerfile(Develop) which installs

ffmpeg
gettext
graphviz
graphviz-dev

Arguably, only the first requirement is necessary at runtime (it's used by the audioread library, which is used when uploading sections through the admin interface). The other three requirements will only be used incidentally: gettext when we need to compile translations; graphviz(-dev) to visualize the data model for documentation purposes.
Would it be possible to build dedicated Dockerfiles with inbaked commands for the compile / graphviz scenarios?
For instance, could we do something along those lines:
FROM ghcr.io/amsterdam-music-lab/muscle-backend:latest AS backend
RUN apt-get install -y graphviz graphviz-dev
RUN pip install pygraphviz
COPY customsettings.py /server
ENTRYPOINT ['python', 'manage.py', 'graph_models', '-a', '-g', '-o', 'muscle-db-schema.png']

customsettings.py would inherit the dev settings, but override the DATABASES to use sqlite, and add the GRAPH_MODELS setting.
We could keep such a Dockerfile in a subdirectory of the scripts folder, along with the settings, and then docker build / docker run would execute the command.
Alternatively, we could also make .sh scripts which add the libraries and do the pip install, perhaps editing the settings inline.

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

@BeritJanssen I've created a PR for this in #1242.
I also saw this line you proposed:
FROM ghcr.io/amsterdam-music-lab/muscle-backend:latest AS backend
Does this (already) work and can we already do this?


cache verification of IBM Cloud IAM keys

Avoid repeated calls to https://iam.cloud.ibm.com/identity/token with the same token.
The regex used in IbmCloudIamDetector has many optional terms, and will match any key|pwd|password|pass|token keyword followed by a 44 character string of letters, digits, underscores or dashes. I have repos with a lot of files with duplicate false-positives like this one:
# represented by key ConstructionCode_NON_CLF_Service_Create_Desc

Caching the token verification reduces the number of times we hit https://iam.cloud.ibm.com/identity/token
@killuazhu

Looks like travis has temporarily disabled builds -- will need to manually run tests before merging.

Closing this PR, as killuazhu's objections are valid but I don't need this enough to continue work on it. (We aren't using IBM Cloud, so I'll disable the plugin for my scans.)


added respawn times for PQ

I need someone to check this out for me

pqtool spawn respawn timers were fine at 8 minutes for dungeons for weeks until this morning when they somehow broke and are showing the wrong time.


Sprig Invaders

Your checklist for this pull request
Author name
Bunnzulu
About your game
A fanmade game of the classic Space Invaders. Fight Endless hordes of aliens to get the highest score possible.
Use a and d to move left and right, and use w to shoot.  Let them get to close and its game over! Your ship can take 25 hits before it's over. Luckily space control gave you a moving shield to help. Careful not to shoot your own shield though it can also only take 25 hits and only comes back after 10 waves. Get pass Wave 10 then the aliens start to bring in their friends.
Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
[x] The name of the file/game contains only alphanumeric characters, -s, or _s.
[x] The game name is not the same as the others from gallery

Image (If an image is used)

[ ] The image is in the /games/img directory.
[ ] The name of the image matches the name of your file.

Thanks for your PR!

Great work on this game! I had fun playing this, and the entity hovering over my player was a good game mechanic. You'll see it in the gallery starting tomorrow.
If you're a teenager, you can request a free Sprig console at this link and join our online Slack community to chat with other people making Sprig games.


Crashes csgo when joining game with cheat injected

good morning sirs please fix the crash.

You can quickly fix it by commenting out the line with DrawModelExecute::original = in src/core/hooks/hook.cpp. This will make you able to use the cheat, only chams wont work

Works, thanks.

i salute u sir..very thanks


Exception thrown when Django auth middleware is not used

Scenario
I am writing several Django based APIs that utilize Application Insights for logging. These APIs do not have Django's built-in authentication, so the authentication middleware is not used.
The problem
This package does not take this into account, so an exception is thrown when processing the request because request.user is not set.
Solution
I made the necessary code change to allow for this behavior. Here is the pull request: #58

Thanks for the fix, I've merged your PR.

0.11.2 released.


CHANGE #709: Competence: Ignore case when creating competence type

Related Issue
Closes #709

Don't merge until https://github.com/universi-me/universi-front/issues/401 is resolved


Updated according to #1736

closes #1736

I would also suggest renaming this to error.html if that sounds good!


Create Laravel/VueJS walking skeleton

Work completed in the following branch:
https://github.com/bitclaw/quick-laravel-vuejs-application/tree/feature/walking-skeleton


Fix dead link

Replaced dead link with one pointing to the revisited version of the article

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/patterns/event-sourcing.md
:white_check_mark:Succeeded
View

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


I am having issue loging in

can you please help me with  this

You should change the value of the wandb_entity argument inside the configs with the entity name of your wandb project.


Invalid JSON in Create the Master Node Policy

In "Step 1: Create the Master Node Policy" http://docs-pcf-staging.cfapps.io/runtimes/pks/1-2/aws-prepare-env.html#master-policy
Issue: the JSON policy shown is invalid.  Backslashes should be removed
Example:
This:
"Action": "s3:*",
Should be:
"Action": "s3:*",

Adding: @outofmem0ry

Hey @kroberts7 thanks for all these! If you know the change you'd like to see reflected in the topic, please also feel free to submit PRs. Thanks.

Hi @bentarnoff Yeah, for some (such as these Invalid JSON's) I can do that.  No problem.  I'll get to those later today.  Had an issue with my docs-pks fork earlier.  I'll look later and get back to you.

@bentarnoff Is it possible that the JSON policies for taken from a pipeline, which could have inserted the \ chars?

Hi @bentarnoff Here's the first PR above.


Document Known issues (for development)

Just find out that some people might have problems that the react watcher stop working, and the way to fix it is to Change Open Files Limit.

@nzambello this looks like something that could cause the problem you had with tests.


[FLINK-4408][JobManager] Introduce JobMasterRunner and implement job submission & setting up the ExecutionGraph

Introduce JobMasterRunner to deal with job level leader election and make underlying job manager properly reacted. Also this runner takes care of determining whether job should be submitted with recover fashion.
This PR also implement the job submission skeleton and setting up the ExecutionGraph, but the interactions with client has been marked as TODO since it may rely on something like JobClientGateway. I'd like to take care of that in a seperate PR later.
The main procedure of managing the lifecycle of a job is:

Once we received a job submission request from the user, we create a JobMasterRunner to deal with it.
JobMasterRunner will first create leader election service to contend the leader of this job, once leadership is granted, it will try to do some real submission work.
Any error occurred during the submission phase will make this job as rejected and dropped.
Once job is accepted, we will face two levels of job retry:

Retarting upon execution failure, it's been taken care of RestartStrategy, and it's mainly inside the ExecutionGraph
Loose of leadership, it will be handled by JobMasterRunner, the old ExecutionGraph will be suspended and then disposed. Retrying should re-submit the job to the JobMaster with isRecovery marked as true.

This looks quite good to me. Will try to merge it onto the latest flip-6 branch...


New version: Gurobi v0.9.1

Registering package: Gurobi
Repository: https://github.com/jump-dev/Gurobi.jl
Created by: @odow
Version: v0.9.1
Commit: 186616e321e530d00f481c18667d9c8f111ec3ea
Reviewed by: @odow
Reference: https://github.com/jump-dev/Gurobi.jl/commit/186616e321e530d00f481c18667d9c8f111ec3ea#commitcomment-43454512

AutoMerge fails due to a commercial, third-party dependency.


Replacement of zenodo link

Replacement of zenodo link by a zenodo repo that contains only the dataset necessary for the training. The previous Zenodo repo contains also the raw data, too big for the import.

Thanks!


Fixed README; added algorithms

Youtube link is now visible in README, added few OS Algorithms.

🛠 Issue (Number)
#1633

Issue no #1633
Closes: #1633
👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
Fixed README

📷 Screenshots

please add labels of SSOC :)

Resolve the Merge Conflict

Kindly sort out Merge Conflict


net/neighbor/neighbor_ethernet_out.c: fix build error without ICMPv6

Summary
Patch fixes following build error when
CONFIG_NET_IPv6 && !CONFIG_NET_ICMPv6 && CONFIG_NET_ETHERNET:
  nuttx/staging/libnet.a(neighbor_ethernet_out.o): In function `neighbor_ethernet_out':
  nuttx/net/neighbor/neighbor_ethernet_out.c:188: undefined reference to `icmpv6_solicit'

IPv6 without ICMPv6 is not very useful, but at least this patch
allows neighbor_ethernet_out() to be used with a packet socket or
with multicast address (point-to-point IPv6 links) even in that case.
Impact
Testing

It tried hard to think of some way for reporting the error back to upper layers instead of DEBUGPANIC(), but could not find a good way.


Upgradlew to latest nightly for parallel CC

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test QFL

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

Quick Feedback - Linux Only (Trigger) (Check)


PLAT-129756: Sync RadioItem with Sandstone

Enact-DCO-1.0-Signed-off-by: Stanca Pop<EMAIL_ADDRESS>Checklist

[x] I have read and understand the contribution guide
[x] A CHANGELOG entry is included
[x] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Resolution
Refactored RadioItem in Agate to follow the architecture model from Sandstone:

added slotBefore prop
updated styling
updated slotBefore styles in Item component

Links
PLAT-129756
Comments

I updated CHANGELOG.md to prevent a newly added log from the wrong position (among already released logs), but it needs to be updated again due to a new conflict.


Lint

I intentionally removed eslint from the project because some find it too aggressive, and I'm trying to keep the project flexible for beginners. Thank you though!

Ok no pb I can understand, so I will continu to my fork

👍 makes sense, sounds good :)