Hydration failed - Next.js 14

What version of million are you using?
3.0.6
Are you using an SSR adapter? If so, which one?
None
What package manager are you using?
npm
What operating system are you using?
Mac
What browser are you using?
Vivaldi (Chrome)
Describe the Bug
Info:
Next.js 14.1.4, React 18.2.0
Node 18
const millionConfig = {
  auto: {
    rsc: true,
  },
};

I am getting a hydration error:
Unhandled Runtime Error
Error: Hydration failed because the initial UI does not match what was rendered on the server.

Warning: Did not expect server HTML to contain a <slot> in <slot>.

See more info here: https://nextjs.org/docs/messages/react-hydration-error

Component Stack
slot
MillionBlock
eval
ToastContextProvider
QueryClientProvider
SessionProvider
AppWrapper
MyApp

What's the expected result?
No errors. I do not have any slots in my code; this is all within Million itself and works without Million
This was not fixed with the previous report and fix in #977
Link to Minimal Reproducible Example
None - if I separate out the ToastContextProvider into a seperate app, I am not getting a reproduction, and as this is all within Million interacting with itself it is hard to do easily
Participation

[ ] I am willing to submit a pull request for this issue.

Hmm, can you come into the server to report this @drewbitt? Maybe we can get on a call? We would need a reproduction before we're able to help.
https://million.dev/chat

Having the same issue with Next.js 14.1.4 and pages router:
Error: Hydration failed because the initial UI does not match what was rendered on the server.

Warning: Did not expect server HTML to contain a <section> in <slot>.

See more info here: https://nextjs.org/docs/messages/react-hydration-error

Component Stack
slot
MillionBlock
eval
SectionLayoutContent
SectionPopularCompares
div
SectionHero
main
Layout
Page
LazyMotion
MotionConfig
PlausibleProvider
App

I get the same error with <footer> or <tr> elements. Changing these elements into a <div> removes most of the errors. But this cannot be a solution, especially with <tr> 😄

Hmm, thank you very much for these. Really appreciate it. Can you please help with a reproduction though? I use Million in Nextjs pages router app and I don't get this issue at all.
Hey @drewbitt, why are you using this specific configuration?
const millionConfig = {
  auto: {
    rsc: true,
  },
};

Please refer to the documentation here: https://million.dev/docs/install#install-manually

@tobySolutions Sorry! Of course, here is a reproduction. It has a slightly different error message, but it's the same issue. It's a blank next.js project (pages router), where I just added a <table> and <section> component
https://stackblitz.com/github/madebyfabian/nextjs-million-reprod?file=pages%2Findex.tsx

@tobySolutions Sorry! Of course, here is a reproduction. It has a slightly different error message, but it's the same issue. It's a blank next.js project (pages router), where I just added a <table> and <section> component https://stackblitz.com/github/madebyfabian/nextjs-million-reprod?file=pages%2Findex.tsx

Awesome! Thank you very much, I really appreciate it.

Hey there @drewbitt, can you maybe try this config?
import million from 'million/compiler'

/** @type {import('next').NextConfig} */
const nextConfig = {
	reactStrictMode: true,
}

export default million.next(nextConfig, {
	auto: true,
	server: true,
	rsc: true,
})

I appreciate the reproduction @madebyfabian and the config @tobySolutions. The config update does fix this for me and I do not have the hydration error. I assume the docs should be updated as I was not seeing this as the recommended config for Nextjs.

Actually though, I see that in my code and the reproduction, before I was getting notice of each component and how much Million speeds things up. Changing it to that config however which makes this not break disables that printout, so I'm not sure Million is working at all.
Before:

After:

drewbitt

Having the same issue

Not stale

Actually though, I see that in my code and the reproduction, before I was getting notice of each component and how much Million speeds things up. Changing it to that config however which makes this not break disables that printout, so I'm not sure Million is working at all.
Before:
After:

Hey there, sorry for the late response on this. Can you try reverting to the previous config then? You shouldn't be getting hydration errors though.

Reverting it to the previous config would just be the reproduction of the hydration error in stackblitz.com/github/madebyfabian/nextjs-million-reprod?file=pages%2Findex.tsx above, no? Do you mean something else?

Not resolved. Let me know if you're waiting on me for anything.

Same issue here. I seem to see it with SVG icons sometimes and then it seems a hidden sidebar on mobile with a 'ul' tag has the issue. Does not happen on desktop where the sidebar is shown from the beginning

Having this issue as well.


Postgres: O2O index migration fails when node name is too long

When creating a O2O relation between a node of the same type with a name like TravelRestrictionSnapshot, the unique constraint on the joining relation fails to be maintained on second and subsequent migrations, where Ent attempts to drop the index.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
sql/schema: drop index of table "travel_restriction_snapshots": pq: index "travel_restriction_snapshot_pr_key" does not exist

By stepping through the migration process in the debugger, it appearsIndex.Name diverges from Index.realname, where the Ent uses Index.Name to check the presence of the index in postgres catalog, and Index.realname to drop it. This shouldn't happen at all if the name is correctly checked in the catalog.

If I rename the node to Snapshot it works correctly.
Expected Behavior 🤔
It maintains the correct index, as it does with nodes of shorter names.
 Working with the latest docker image of postgres.
Basically I'm starting off a fresh instance, opening the connection via entsql.Open(driver, dsn) and then creating the client and then calling:
err := client.Schema.Create(
	ctx,
	migrate.WithDropColumn(true),
	migrate.WithDropIndex(true))

Then I'll seed the some data. Everything works out smoothly.
But: Subsequent runs of client.Schema.Create() experience the same issue as described above.
It seems to cut off my index name in the error message at 25 chars. But the actual name is of length 33 chars. 🤷🏼♂
Go: v1.17.0
Ent: v0.9.1
Driver: pq
DB: Postgres (docker image: postgres:latest)
OS: macOS

We had the same issue. Though there is an easy workaround (setting a Field), it would be great if entc give a warning about it before running actual migration.


feat(plugins) add kubernetes-sidecar-injector plugin

Summary
Adds kong-plugin-kubernetes-sidecar-injector to dependencies.

What was wrong with #4304?


go-require: require-error: support HTTP handlers

Closes https://github.com/Antonboom/testifylint/issues/73

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

88 of 92 (95.65%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.006%) to 94.029%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/checkers/go_require.go
34
36
94.44%

internal/checkers/require_error.go
35
37
94.59%

Totals

Change from base Build<PHONE_NUMBER>:
-0.006%

Covered Lines:
1874

Relevant Lines:
1993

💛  - Coveralls


ImportError: No module named future

`[~] kou2n$ sudo -H pip install NetEase-MusicBox
Collecting NetEase-MusicBox
Using cached NetEase-MusicBox-<IP_ADDRESS>.tar.gz
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 1, in
File "/private/tmp/pip-build-jKmWNb/NetEase-MusicBox/setup.py", line 40, in
from future import standard_library
ImportError: No module named future
----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in /private/tmp/pip-build-jKmWNb/NetEase-MusicBox/`

pip install future
Will fix this in next version
On Sat, Sep 24, 2016, 21:19 gaoyun<EMAIL_ADDRESS>wrote:

`[~] kou2n$ sudo -H pip install NetEase-MusicBox
Collecting NetEase-MusicBox
Using cached NetEase-MusicBox-<IP_ADDRESS>.tar.gz
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 1, in
File "/private/tmp/pip-build-jKmWNb/NetEase-MusicBox/setup.py", line 40,
in
from future import standard_library
ImportError: No module named future

Command "python setup.py egg_info" failed with error code 1 in
/private/tmp/pip-build-jKmWNb/NetEase-MusicBox/`
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/darknessomi/musicbox/issues/394, or mute the thread
https://github.com/notifications/unsubscribe-auth/AD_RSmgacnzDT1YQ7NUEPwMj1okpR1N1ks5qtSNxgaJpZM4KFpBl
.

@kigawas Thank you！


Feature Request: Applying effects from Token UI should apply to all selected tokens

Current Behavior: When you have multiple tokens selected and right click one of the tokens and apply an effect through the Assign Status Effects menu, it only applies the effect to the token you right-clicked.
Proposal: When you have multiple tokens selected and apply an effect via the Token UI's Assign Status Effects menu, the effect should propagate to all selected tokens.

I will add that it's strange that you can select multiple tokens and set them all as Hidden or Combatants, but not have the same functionality for applying Statuses, Elevation, or Targeting.


File upload works only with standard read streams

I've been trying to upload a file (using multipart POST request) that is inside TAR archive (I use tar-stream library to get stream to a particular file inside TAR archive). But it always fails with ECONNRESET error. After a bit of experimenting it seems like it can only upload streams constructed directly with fs.createReadStream and fails with irrelevant error for all other streams.
Here's a minimal example to reproduce problem. If PassThrough stream is passed into formData then the following error is occurred:
ERROR { Error: socket hang up
    at createHangUpError (_http_client.js:254:15)
    at Socket.socketOnEnd (_http_client.js:346:23)
    at emitNone (events.js:91:20)
    at Socket.emit (events.js:185:7)
    at endReadableNT (_stream_readable.js:974:12)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9) code: 'ECONNRESET' }

Server:
var http = require('http');

http.createServer(function (request, response) {
	request.on('data', function(chunk) {
      console.log(chunk.length);
    });
    request.on('end', function() {
      response.writeHead(200, "OK", {'Content-Type': 'text/html'});
      response.end();
    });
}).listen(8077);

Client:
var request = require('request');
var fs = require('fs');
var PassThrough = require('stream').PassThrough;

var rs = fs.createReadStream('some_file.bin'), pt = new PassThrough();
rs.pipe(pt);
request({
	'method': 'POST',
	'url': 'http://<IP_ADDRESS>:8077',
	'formData':	{
		'uploaded_file': pt
	}
}, function (err, httpResponse, body)
{
	if (err)
		console.log('ERROR', err);
	else
		console.log({ 'statusCode': httpResponse.statusCode, 'body': body });
});

I experience the same problem when I try to wrap a Buffer in a read stream to upload it as a valid multipart request.

I have also been trying to find a way around this issue for some time

Same problem here, I have a base64 string and I need to send it as a file. I convert my base64 to a buffer and then, just like @philipp-spiess I wrap it in a read stream, but it doesn't work. Did you guys find a work around ?

update: for now my workaround is using bhttp (https://www.npmjs.com/package/bhttp) which uses streams2

I've managed to upload my file using the buffer directly. You just need to specify file-related information (file name, type, ...)
var formData = {
  'file': {
    value: buffer,
    options: {
      filename: file.filename,
      contentType: file.mimetype
    }
  }
}

https://github.com/form-data/form-data#alternative-submission-methods

@PierreCavalet, I tried this way but unfortunately it didn't work.
My "workaround" was simply not to use request library and implement multipart POST using nodejs http service.

@PierreCavalet Is your buffer already encoded text? For me the buffer happens to be a binary blob which broke the encoding of the request (since it was apparently not properly encoded).

@philipp-spiess my buffer is created with:
Buffer.from(base64string, 'base64')

So yes it is a binary blob. The documentation says that it creates a new Buffer containing the given JavaScript string. If provided, the encoding parameter identifies the character encoding of string.

@PierreCavalet The Buffer I was using was straight from multer, a file upload middleware for express.

@philipp-spiess This is weird. My base64 string is coming from a base64 encoding of a buffer, and this buffer is straight from my file upload. (The encoding is done by an other service, thats why we "encode and decode" for "nothing"). So it should work for you too.

@samkelly thank you so much for your recommendation of using bhttp. It works perfectly!

@bchr02 np!

Tracked this error down: turns out to be an issue with form-data's _length_retriever function. If you have a stream that isn't one of the 3 fully understood streams (basically files with an fd field, HTTP responses or request output streams) you basically have to lie to form-data to get it to do the right thing (and you need to have the precise byte length of the stream). To get it to work, I ended up doing this:
var buffer = Buffer.from(base64string, 'base64');
buffer['httpVersion'] = true;
buffer.headers = { 'content-length': 64 }
var formData = {
  'file': {
    value: buffer,
    options: {
      filename: file.filename,
      contentType: file.mimetype
    }
  }
}

Very important: you MUST specify the filename field in the options, otherwise you get another exception.

If you can specify the file options, you should be able to specify options.knownLength with the byte length of your file stream and it will get picked up by FormData.
It's mentioned in FormData's readme though not really publicized much.  Request does not mention it anywhere right now so far I can tell.

Ah, that worked: teach me to do any debugging at 2am. I could have sworn I tried knownLength and got an exception, but that may have been the lack of the filename field causing chaos.

this doesn't work for my situation, because I am uploading something that is part of a gzip stream, therefore I do not know the length.

@sam0x17 Indeed it won't as, so far as I can tell, request doesn't currently support transfer-encoding: chunked, thus your use of bhttp.
See also

https://github.com/request/request/issues/2671
https://github.com/request/request/issues/2380
https://github.com/request/request/issues/2488

mark

@PierreCavalet, I tried this way but unfortunately it didn't work.
My "workaround" was simply not to use request library and implement multipart POST using nodejs http service.

@ovk Do you have a "quote" of your workaround ? I've tried somethning like that :
const streamSample = fs.createReadStream('data.csv').pipe(passThrough);

const formHTTP = new FormData();
formHTTP.append('a', 'http');
formHTTP.append('myfile', streamSample);

const requestToSend = http.request({
    method: 'post',
    host: host,
    port,
    path: '/http',
    headers: formHTTP.getHeaders()
});

formHTTP.pipe(requestToSend);

requestToSend.on('response', (res) => {
    console.log('HTTP response', res.statusCode);
});

requestToSend.on('error', (e) => {
    console.log('ERROR HTTP', e);
});

Bu I still obtain a ECONRESET :sob:

@PierreCavalet Your solutions works great for uploading buffer file but soon as I add additional form parameter then it no longer works.. i get "source.on is not a function" error. Thoughts?
var formData = {
'param1': 'value',
'file': {
value: buffer,
options: {
filename: file.filename,
contentType: file.mimetype
}
}
}

@BenjD90
I used your workaround, but it seems that the stream doesn't get sent at all. I am using multer on the receiving end, if it is of any relevance.

@Ncifra I'm sure my workaround works, because I use it everyday on my current project.
You can test your stream on https://beeceptor.com/ ;)

I actually solved this by following: https://github.com/form-data/form-data/issues/356#issue-234978025
Here is my original issue related to the problem: https://github.com/form-data/form-data/issues/409
@BenjD90 Currently I reverted the code and can't test it since it was a very delayed feature on the project I am working. Maybe I was missing something since all that testing had created too much sparsed code. I did log the requests though, and the form-data object seemed to have the file in it, but multer didn't seem to receive it, while the other non binary data were being sent correctly.

@stale This bug is steel present and should be fixed on day ^^

Yes agreed, it's as important now as it was back then. I've basically been waiting for this to be fixed before I use request again rather than bhttp.

See https://github.com/form-data/form-data/pull/397#issuecomment-471976669 and https://github.com/bitinn/node-fetch/pull/707#issuecomment-569850948

Issue still exists in latest version

Issue still exists in latest version

Can confirm I am still getting it

I was able to solve a similar problem by setting the filename parameter. Maybe FormData is not able to deduce the filename from a passthrough stream?
var data = new FormData();
data.append('file', passThroughStream, {
  filename: 'some-filename'
});


Dockerfile

Hello,
Thanks for making this work available publicly. I have been having issues trying to get it up and running, most likely due to version mismatch. I have tried with CUDA 11.6 and CUDA 11.7, PyTorch 1.12, numba 0.56.
Would you be able to either provide a Dockerfile, or exact set of packages with their versions that has worked well?
Thanks,
Shubham

FROM nvidia/cuda:11.1-cudnn8-devel-ubuntu20.04

### Deal with cuda keyring problem
RUN rm /etc/apt/sources.list.d/cuda.list
RUN rm /etc/apt/sources.list.d/nvidia-ml.list
RUN apt-key del 7fa2af80
RUN apt-get update && apt-get install -y --no-install-recommends wget
RUN wget https://developer.download.nvidia.com/compute/cuda/repos/ubuntu2004/x86_64/cuda-keyring_1.0-1_all.deb
RUN dpkg -i cuda-keyring_1.0-1_all.deb

### install python3 and opencv requires
RUN DEBIAN_FRONTEND=noninteractive apt-get update --fix-missing --no-install-recommends && \
    DEBIAN_FRONTEND=noninteractive apt-get install python3.8 python3-pip nano libsm6 \
    libxext6 libxrender-dev libgl1-mesa-glx libglib2.0-0 python3-tk qt5-default curl git nano htop -y && \
    apt-get clean && \
    rm -rf /var/lib/apt/lists/*

### install common packages, some not necessary for this repo
RUN pip3 install -U pip && \
    pip3 install future -U && \
    pip3 install tensorflow pandas matplotlib numpy \
    pillow opencv-python scikit-image numba tqdm \
    cython fire easydict cityscapesscripts pyquaternion \
    nuscenes-devkit onnx pytest synology-api flake8 mypy pygame

### cuda - torch - torchvision
ARG CUDA_VER="111"
ARG TORCH_VER="1.9.1"
ARG VISION_VER="0.10.1"

RUN pip3 install torch==${TORCH_VER}+cu${CUDA_VER} torchvision==${VISION_VER}+cu${CUDA_VER} -f https://download.pytorch.org/whl/torch_stable.html
RUN pip3 install kornia
RUN pip3 install torch_scatter -f https://data.pyg.org/whl/torch-{TORCH}+cu{CUDA}


No popups

With a SPA application that leverages Azure AD B2C, I would like to allow the user to enter email password WITHIN the application without requiring a popup or a redirect.
Can I do this with MSAL? If so, how and is there any example available?
Note: I'd also offer social logins for the user, and in that case would want a popup, but not when the user chooses email/password option.

I agree with @newtack. I got really excited when I saw this as I'm currently trying to figure out the best way to auth with Azure B2C and a SPA framework. But this IMO severely limits the user experience and expeciall ony things like mobile you don't really want to pop a new windwow.
Is there any plan to create say essentially the same thing as the loginPopup except we just have like loginUser(username, password and let the front end develop take care of getting the values.
Seems the value in this library would be to me if it could take away the whole token management thing for me but still allow me to have it as an in-app experience.

@newtack @sethreidnz The library is not meant to support this particular use case case.

Why not? You mean 99% of use cases for a modern JavaScript app? Is there something that makes it impossible?

Collecting and storing a user's username and password is not considered the correct approach for non-interactive authentication, and therefore we are not implementing it for many platforms. With Azure AD, the service handles the conversation in the browser (popup or with an indirection).
The following GitHub issue (which is for ADAL.net, but this is the same principle) explains in details why we don't do it: https://github.com/AzureAD/azure-activedirectory-library-for-dotnet/issues/482
I recommend you to read the thread, which also proposes alternative solutions.
We are limiting it to very specific platforms (.NET desktop) for compatibility.
HTH


Exposed RedirectAction and RedirectAttempt

also added minimal doc to RedirectAttempt to satisfy #![deny(missing_docs)]
added links to RedirectPolicy::Custom docs

fixes https://github.com/seanmonstar/reqwest/issues/105

Thanks!


Create recorder.py helper for getting audio clips

This helper script will record audio from microphone.

build_jenkins


Teams meeting active participants list and hovering over users

In the previous version of microsoft teams, you could open a meeting chat and hover over the user/avatars at the top to see the user's full display name, well as hover over the "+2 users" circle to see additional names. This feature no longer works in the new teams

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

ID: 442f8c6b-95ba-a09e-639d-b9dd141198f6
Version Independent ID: 442f8c6b-95ba-a09e-639d-b9dd141198f6
Content: Known issues in the new Microsoft Teams desktop client - Microsoft Teams
Content Source: Teams/new-teams-known-issues.md
Service: msteams
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

Hello @JarrodGross
I've just experienced this and I could expand the list by clicking on the icon next to the (11)

Hello, clicking the icon next to it shows a list of all users invited to the meeting, but does not necessarily reflect the list of participants currently live/logged into an active meeting.
The only way on the new teams to see all users who are "active" in the meeting, is to go join it yourself and look at the people list.
Otherwise, the old teams allowed you to hover over persons initials or the (+#) and provided you a tooltip/popup of their name / who else is in the meeting.
Thank you.
From: Andres Mariano Gorzelany @.>
Sent: Thursday, November 23, 2023 8:09 AM
To: MicrosoftDocs/OfficeDocs-SkypeForBusiness @.>
Cc: Jarrod Gross @.>; Mention @.>
Subject: Re: [MicrosoftDocs/OfficeDocs-SkypeForBusiness] Teams meeting active participants list and hovering over users (Issue #10571)
Hello @JarrodGrosshttps://github.com/JarrodGross
I've just experienced this and I could expand the list by clicking on the icon next to the (11)

Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/10571#issuecomment-1824412582, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BDWILXXNDCOWWFIKYT6B4KTYF5DFZAVCNFSM6AAAAAA67R2CYOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRUGQYTENJYGI.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>
One Call (including its affiliates and subsidiaries) is not a healthcare or transportation provider, but instead schedules and coordinates such services on behalf of its clients through a proprietary network of independently contracted licensed practitioners, healthcare provider groups and vendors (collectively, "Network Providers"). Network Providers are solely responsible for all of their actions, omissions and decisions regarding the services they provide. One Call is not a claims administrator and does not make determinations regarding coverage and compensability of services rendered by Network Providers. This email is intended only for the person or entity to which it is addressed and may contain information that is privileged, confidential or otherwise protected from disclosure. Dissemination, distribution or copying of this email or the information herein by anyone other than the intended recipient, or an employee or agent responsible for delivering the message to the intended recipient, is prohibited. If you have received this email in error, please notify the sender immediately by a "reply to sender only" message and destroy the original message and all copies.

Hello @JarrodGross
After asking internally, I got the reply for you to please support feedback directly from the new teams, Help -> Give Feedback.
Thank you

Thank you, I will do so.
From: Andres Mariano Gorzelany @.>
Sent: Tuesday, December 5, 2023 3:19 PM
To: MicrosoftDocs/OfficeDocs-SkypeForBusiness @.>
Cc: Jarrod Gross @.>; Mention @.>
Subject: Re: [MicrosoftDocs/OfficeDocs-SkypeForBusiness] Teams meeting active participants list and hovering over users (Issue #10571)
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
Hello @JarrodGrosshttps://github.com/JarrodGross
After asking internally, I got the reply for you to please support feedback directly from the new teams, Help -> Give Feedback.
Thank you

Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/10571#issuecomment-1841551954, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BDWILXS3MXOZSNJVKSZ4EDLYH56S5AVCNFSM6AAAAAA67R2CYOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBRGU2TCOJVGQ.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>
One Call (including its affiliates and subsidiaries) is not a healthcare or transportation provider, but instead schedules and coordinates such services on behalf of its clients through a proprietary network of independently contracted licensed practitioners, healthcare provider groups and vendors (collectively, "Network Providers"). Network Providers are solely responsible for all of their actions, omissions and decisions regarding the services they provide. One Call is not a claims administrator and does not make determinations regarding coverage and compensability of services rendered by Network Providers. This email is intended only for the person or entity to which it is addressed and may contain information that is privileged, confidential or otherwise protected from disclosure. Dissemination, distribution or copying of this email or the information herein by anyone other than the intended recipient, or an employee or agent responsible for delivering the message to the intended recipient, is prohibited. If you have received this email in error, please notify the sender immediately by a "reply to sender only" message and destroy the original message and all copies.


After installing and opening app causes crash. File words.csv is missing from assets

After installing and opening app causes crash. File words.csv is missing from assets.
Please add words.csv to the project.

Fixed.


[Feature Request] Open link in new tab

On a search item, it would be nice to have (any of) the following actions open the link in a separate tab

Middle mouse button click
Right Click -> Open link in new tab

Related to #4 : Fixing that will auto-enable this

Enabled by 58d97cc.
Closing issue.


[chore] Bump NATS version

Signed-off-by: Pranav Singh<EMAIL_ADDRESS>Description
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Please help us investigate the error of the CI checking process.

Updating with master

Thanks for your contribution. Please give us a detail on this. Why we need bump the version to the latest?

I and Lee were discussing the need for upgrading the NATS version for potentially reducing the security vulnerabilities we have on meshery repo


#1 Title here

This is my first commit please approve this pr.
I have done these changes
- remove this
+ add this

[x] this is first task

Hi can you check my code.


adding Meta impl

Meta implementation (see https://github.com/SpectralOps/teller/issues/65)

[x] add types + empty impls for all providers
[ ] add teller providers export command that outputs all providers, and their meta in JSON. Add teller-specific metadata to this JSON (what version of teller generated it, will be useful for docs)

{
     "teller":{
         "version": ".."
      },
      "providers":{
          "heroku":{
                 ..
          }
      }
}

while exporting, if duplicate provider name found -- warn (this is a way for us to start linting providers)

[ ]  add concrete implementation for every provider for their meta
[ ] replace config template with the relevant provider meta (this means template is being built real-time and/or different templating generation flow)
[ ] build a tool to generate README automatically from export JSON, add to workflow as a make command
[ ] teller-docs build a tool to create markdown pages for docs per provider with a standard layout, from exported JSON

https://github.com/tellerops/teller/issues/125
will be implemented on this branch as well


Target minimal API Set version for Kernel32

Kernel32 work for #84
CC: @jmelosegui

:+1: looks good.


HBASE-22261 Make use of ClusterStatusListener for async client

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
20
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

0
mvndep
27
Maven dependency ordering for branch

+1
mvninstall
264
master passed

+1
compile
79
master passed

+1
checkstyle
106
master passed

+1
shadedjars
284
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
291
master passed

+1
javadoc
55
master passed

_ Patch Compile Tests _

0
mvndep
14
Maven dependency ordering for patch

+1
mvninstall
260
the patch passed

+1
compile
80
the patch passed

+1
javac
80
the patch passed

-1
checkstyle
76
hbase-server: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
282
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
543
Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.

+1
findbugs
327
the patch passed

+1
javadoc
56
the patch passed

_ Other Tests _

+1
unit
193
hbase-client in the patch passed.

+1
unit
7653
hbase-server in the patch passed.

+1
asflicense
46
The patch does not generate ASF License warnings.

10780

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/172

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 4f2007279cac 4.4.0-131-generic #157~14.04.1-Ubuntu SMP Fri Jul 13 08:53:17 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / 353f9226c3

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/1/artifact/out/diff-checkstyle-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/1/testReport/

Max. process+thread count
4485 (vs. ulimit of 10000)

modules
C: hbase-client hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/1/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
297
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

0
mvndep
35
Maven dependency ordering for branch

+1
mvninstall
267
master passed

+1
compile
77
master passed

+1
checkstyle
97
master passed

+1
shadedjars
275
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
245
master passed

+1
javadoc
56
master passed

_ Patch Compile Tests _

0
mvndep
16
Maven dependency ordering for patch

+1
mvninstall
243
the patch passed

+1
compile
76
the patch passed

+1
javac
76
the patch passed

-1
checkstyle
67
hbase-server: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
270
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
508
Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.

+1
findbugs
286
the patch passed

+1
javadoc
57
the patch passed

_ Other Tests _

+1
unit
210
hbase-client in the patch passed.

-1
unit
14387
hbase-server in the patch failed.

+1
asflicense
62
The patch does not generate ASF License warnings.

17659

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestFromClientSide3

hadoop.hbase.client.TestFromClientSideWithCoprocessor

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/172

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux e292ba7fbc00 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / f30d6c958a

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/2/artifact/out/diff-checkstyle-hbase-server.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/2/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/2/testReport/

Max. process+thread count
5104 (vs. ulimit of 10000)

modules
C: hbase-client hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/2/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
45
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

0
mvndep
21
Maven dependency ordering for branch

+1
mvninstall
240
master passed

+1
compile
79
master passed

+1
checkstyle
99
master passed

+1
shadedjars
267
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
274
master passed

+1
javadoc
57
master passed

_ Patch Compile Tests _

0
mvndep
15
Maven dependency ordering for patch

+1
mvninstall
247
the patch passed

+1
compile
77
the patch passed

+1
javac
77
the patch passed

+1
checkstyle
99
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
268
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
500
Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.

+1
findbugs
294
the patch passed

+1
javadoc
55
the patch passed

_ Other Tests _

+1
unit
198
hbase-client in the patch passed.

-1
unit
14133
hbase-server in the patch failed.

+1
asflicense
60
The patch does not generate ASF License warnings.

17123

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestFromClientSide3

hadoop.hbase.client.TestFromClientSideWithCoprocessor

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/172

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 6ef788029a78 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / f30d6c958a

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/3/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/3/testReport/

Max. process+thread count
5112 (vs. ulimit of 10000)

modules
C: hbase-client hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/3/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
45
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

0
mvndep
26
Maven dependency ordering for branch

+1
mvninstall
301
master passed

+1
compile
98
master passed

+1
checkstyle
118
master passed

+1
shadedjars
327
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
363
master passed

+1
javadoc
68
master passed

_ Patch Compile Tests _

0
mvndep
17
Maven dependency ordering for patch

+1
mvninstall
312
the patch passed

+1
compile
98
the patch passed

+1
javac
98
the patch passed

+1
checkstyle
123
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
330
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
637
Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.

+1
findbugs
387
the patch passed

+1
javadoc
69
the patch passed

_ Other Tests _

+1
unit
227
hbase-client in the patch passed.

-1
unit
14533
hbase-server in the patch failed.

+1
asflicense
58
The patch does not generate ASF License warnings.

18240

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestFromClientSideWithCoprocessor

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/172

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 662edc225e52 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / 0645d5a033

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/4/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/4/testReport/

Max. process+thread count
4974 (vs. ulimit of 10000)

modules
C: hbase-client hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-172/4/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.


Changed the BinarySearch component from a function to an arrow function.

Changed the BinarySearch component from a function to an arrow function.
The original component was defined using the function keyword, but I updated it to use an arrow function instead. This change provides a more concise syntax and also avoids any issues with the binding of the "this" keyword.
I have tested the updated component and verified that it works as expected.
Thanks for considering this change!

It's a top-level function. There's no issue with "this", nor is it more concise. Also it's not a component.
Consider common sense before spamming multiple repositories with the same artificial-not-so-intelligent type of changes.

It's a top-level function. There's no issue with "this", nor is it more concise. Also it's not a component.
Consider common sense before spamming multiple repositories with the same artificial-not-so-intelligent type of changes.

don't that hard on the guy, he might still be learning and just wants to contribute.

I looked at his profile; he had the same comments in other repos. It's a bot.

@lazarljubenovic I'm really sorry for that, if that make you angry. I'm not a bot just still learning as @iamleomoko said, and am looking for something small to change in the code. I'll delete this now
So am really sorry once again.
@iamleomoko Thank you for saying that.


Container detector - cgroupv1 and v2 - prefix support & fix

Is your feature request related to a problem? Please describe

There is an issue where sometimes using lightweight k8 cgroup v1 or v2 - prefix may be there in front of the containerId - current implementation does not account for it.
Describe the solution you'd like to see

Will need to add a check ( regex to filter crio, crio-containerd and containers prefix before the containerId extraction
Describe alternatives you've considered

Additional context

Please assign it to me.


Call localParticipant.setCameraEnabled Receive RoomEvent.TrackMuted and  RoomEvent.TrackUnmuted

Describe the bug
Call localParticipant.setCameraEnabled(true) Receive RoomEvent.TrackUnmuted  expect  TrackSubscribed
Call localParticipant.setCameraEnabled(false) Receive RoomEvent.TrackMuted  expect  TrackUnsubscribed
Device Info:

Device: oneplus 11
OS: android 14
LiveKit SDK version: 1.5.3

Im sorry it right now


improve dependency naming

The new ajax dependency tracking is awesome, but the dependency naming convention could be improved.  Right now it looks like it is always the hostname of the AJAX request.

I feel like SQL dependencies, for example, have a much better naming convention, which is:
SQL: ServerName | DatabaseName | StoredProcedureName
The reporting on things like dependencies with performance problems is much more useful when the name has more context.  It also helps a lot when browsing search results.  Perhaps you could add the HTTP method and the relative URL to the dependency name?

Thanks for the feedback! We don't currently collect the HTTP verb, but we do have the relative URL available. I will open a bug on our UI to add the relative path to the search results view so the difference between the calls is clearer.

We need to do the same in full framework when using Profiler API to collect dependencies. This will make naming of dependency aligned with the naming of corresponding request. @abaranch can you please track it?

Sergey, if you are talking about path than I expect UI will work both for JS SDK dependencies and for Web SDK dependencies. If you are talking about adding the verb than yes: we need a separate issues for JS and Web SDKs. For Web: https://github.com/Microsoft/ApplicationInsights-dotnet-server/issues/54

I was talking about VERB in a name. I'm sure POST dependencies typically longer than GET dependencies

I'm renaming this issue to "Add verb to dependency name". To address the other part of @gregjhogan's question we opened an Ibiza UI bug to add full url (CommandName) to search results for dependencies.

Fixed in #192 .
The verb isn't exposed in the Azure Portal UI yet, but it is accessible through Application Analytic.


Self-hosted ChromaDB: cannot import name 'ChromaDbConfig' from 'embedchain.config'

🐛 Describe the bug
import os
from embedchain import CustomApp
from embedchain.config import CustomAppConfig, AddConfig, ChunkerConfig, ChromaDbConfig

I get
ImportError: cannot import name 'ChromaDbConfig' from 'embedchain.config'

I'm trying to connect to my ChromaDB
chroma_config = ChromaDbConfig(
  collection_name="default_collection",
  dir="db",
  host="localhost",
  port="8000",
)

Is it possible to use our own external ChromaDB?

Hey, did you install from pypi or is this from the main branch?

The docs are for the main branch which is a little bit ahead of pypi right now. Pypi doesn't have a chromadb config.

Closing this as I think it's been fixed by the new release/sync. Please reopen if you need more help.


kunai docker stats returns nothing

[root:/etc/default] 11s 2 # kunai docker stats
========== [Docker Stats]:

Nothing in logs (I started a kunai agent in foreground).

For docker-py
Name: docker-py
Version: 0.7.1
Location: /usr/local/lib/python2.7/dist-packages

As naparuba adviced me I tried this script :
from docker import Client
con = Client(base_url='unix://var/run/docker.sock')
print con.containers()

I got this error :
Traceback (most recent call last):
  File "./tt.py", line 3, in <module>
    print con.containers()
  File "/usr/local/lib/python2.7/dist-packages/docker/client.py", line 519, in containers
    res = self._result(self._get(u, params=params), True)
  File "/usr/local/lib/python2.7/dist-packages/docker/client.py", line 98, in _result
    self._raise_for_status(response)
  File "/usr/local/lib/python2.7/dist-packages/docker/client.py", line 94, in _raise_for_status
    raise errors.APIError(e, response, explanation=explanation)
docker.errors.APIError: 404 Client Error: Not Found ("client and server don't have same version (client : 1.16, server: 1.15)")

I just fixed it, we don't need anymore the docker-py lib.
Can you confirm? thanks :)

Ok, I pull / update and tell you what.

^^
On Mon, Jan 19, 2015 at 11:03 AM, Thomas Labarussias <
<EMAIL_ADDRESS>wrote:

Closed #9 https://github.com/naparuba/kunai/issues/9.
—
Reply to this email directly or view it on GitHub
https://github.com/naparuba/kunai/issues/9#event-220867368.

It's OK.
A kunai --update```or a kunai --upgrare`` didn't work, so I removed /usr/local/lib/python2.7/dist-packages/Kunai-0.8-py2.7.egg and reinstalled everything.
After that, a ```kunai docker stats`` gave me that :
$ kunai docker stats
========== [Docker Stats]:
========== [Container:c7b4659ea95112060a25176716b7b4d0a7825d127f19a2ccce56e097d0be0212]:
cpu.system:                   0.278263883724
cpu.user:                     0.885295060674
memory.active_anon:           266960896
memory.active_file:           23678976
memory.cache:                 66945024
memory.inactive_anon:         20480
memory.inactive_file:         43094016
memory.mapped_file:           16318464
memory.pgfault:               2055.6868193
memory.pgmajfault:            2.27760474223
memory.pgpgin:                2341.37767501
memory.pgpgout:               1843.77055198
memory.rss:                   266809344
memory.unevictable:           0
========== [Image:kaas:latest (sum)]:
cpu.system:                   0.278263883724
cpu.user:                     0.885295060674
memory.active_anon:           266960896
memory.active_file:           23678976
memory.cache:                 66945024
memory.inactive_anon:         20480
memory.inactive_file:         43094016
memory.mapped_file:           16318464
memory.pgfault:               2055.6868193
memory.pgmajfault:            2.27760474223
memory.pgpgin:                2341.37767501
memory.pgpgout:               1843.77055198
memory.rss:                   266809344
memory.unevictable:           0

Thanks

There is no such kunai --upgrade. It's up to your packet manager to manage this ^^

it's just python setup.py install and that's all :)


hint: false has lost it's effect in v0.34.0

I'm using dist/autocomplete.jquery.min.js and since v0.34.0 hint: false is not working anymore.
I tried what happens if I use v0.33.0 instead and found out it indeed works in previous release, it might be a bug.

No code regarding to that changed between v0.33.0 and v0.34.0, this is really weird. Are you sure the version is the only variant you used while testing? Could you make a sandbox with working & non-working version?
There's a template here: https://codesandbox.io/s/github/algolia/create-instantsearch-app/tree/templates/autocomplete.js

Maybe I miss understood what hint is in autocomplete.js, but my problem really is I get native autocomplete of my previous searches with latest version, but not with previous one, I inspected and it seems by using v0.33.0 input element will have a attribute autocomplete="off",
but by using using v0.34.0 I get: autocomplete="nope"

ah, hint is not what this is for, hint is used for background text as a placeholder and is unrelated to the behaviour you described. What you are seeing is more of a bug, which behaviour we reverted in #273 and will be back to what the behaviour was in 0.33 in the next version.

Feel free to continue using 0.33 until 0.35 is released :)


fix and simplify rust solution for 0036-valid-sudoku

File(s) Modified: 0036-valid-sudoku.rs
Language(s) Used: rust
Submission URL: _https://leetcode.com/problems/valid-sudoku/submissions/949636309/_

Previous incorrect submission: _https://leetcode.com/problems/valid-sudoku/submissions/949595521/_

fix for bx indices
use of returned insertion status

Note
_https://doc.rust-lang.org/std/collections/struct.HashSet.html#method.insert_
If the set did not previously contain this value, true is returned.
If the set already contained this value, false is returned.

@Adimac93 thanks so much for posting this... I was banging on the current "solution" for several days, trying to figure out what was wrong... was about to give up and then found your PR that makes it work correctly! :)

@neetcode-gh please merge this in if possible (and put on neetcode.io)! The current Rust solution has a small but critical error, the PR here by @Adimac93 fixes the solution and also makes it more idiomatic. Thanks!


Add modern-normalize

Well, (modern-normalize)[https://github.com/sindresorhus/modern-normalize] is smaller than the old one and it's currently used by tailwindcss

Unfortunately, modern-normalize is an attempt to fix a different version of normalize.css that I no longer have access to. I don’t want to discourage anyone else making another normalize, but that project doesn’t fix bugs in current browsers and more closely resembles sanitize.css.


Yankpad asks for a category although there is only one to choose from (in my setup)

Hi,
I just have one single category (= topmost heading level) in my yankpad.org definition file. When I invoke yankpad-expand, I get asked to enter the category. In my setup, I just have to press TAB. However, this step is completely unnecessary from my perspective. If it is an easy change, I'd love to see this step omitted in this case.
Thanks!

Hi! It would be an easy change. I don't see any direct downside of this, so I'll see if I can change it. What you can do in the meantime is to set the variable yankpad-default-category to the name of your category. I think that should work.

I'm puzzled. I could have sworn that I've added the default category setting when upgrading to the most recent version of yankpad. I was wrong. It's now set but if this change is not much trouble, I do think that it's worth for many situations because I assume that this does not have negative side-effects.

I've changed the behavior to automatically choose the sole category if there's only one. Thanks for the suggestion!


Resolve user store when current domain name is passed

Purpose
getSecondaryUserStoreManager(domain) method utilises a hashmap from the AbstractUserStoreManager class to resolve the user store. But this hashmap is populated only in the primary user store instance resulting in the method returning null. Added a validation to refer to the current instance if the domain name to resolve match the domain name of the current instance.
Related issue

https://github.com/wso2/product-is/issues/18985

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7694383544

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7694383544
Status: success


use 0 depth in Github action template

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 6.482%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
129

Relevant Lines:
1990

💛  - Coveralls


Proposal: Butterknife extensions

High Level
Allow third party developers to add more functionality to butterknife by creating their own butterknife listener definitions.
This has multiple benefits:

Butterknife could work for more than just first party Android views
ListenerClass annotations are decoupled from this repo
Third party libraries can add their own butterknife bindings packaged with their libraries

Butterknife extensions could be registered in a similar way to AutoValue Extensions. Then butterknife processor could generate the listener code.

I believe this is a dupe of #497. And for what it's worth, I've thought about this for a long time, but just haven't been motivated to do that work.


Event Hubs: Legitimize the stress test suite

Summary
While effective, the current suite of stress tests for Event Hubs is largely ad-hoc and lives in a private repository.  With a common approach to long-running stress tests now agreed upon, the suite should be revised for a more structured approach, reworked to fit the common approach, integrated with the shared hosting assets, and moved into the official repository.
This issue is primarily intended for personal thoughts and notes;  it is not expected that it be a clear and comprehensive work item that could be fully understood and implemented by others.  Please expect some gaps in descriptions and a non-trivial amount of assumed knowledge.
Scope of Work

Refactor the existing ad-hoc stress tests to remove code duplication, follow a modular architecture, and follow the conventions used for the other Event Hubs areas.  Like the performance suite, the stress tests should make use of the Event Hubs shared code project for common items.

Integrate metrics collection with Application Insights, following the common conventions agreed upon.  (coordinate with Ben and Richard)

Integrate chaos injection with the current platform toolset.  (coordinate with Ben and Richard)

Adjust the architecture to support the concept of "roles" for the stress testing, allowing scenarios to be focused without taking on additional resource overhead.  For example, tests for the event processor should not include an embedded producer; that should exist as an independent role that may be deployed separately and then coordinated with.   Roles should probably include:

Publisher
Processor
Consumer
Partition Receiver

Allow for the concept of an orchestrator to control scenarios.  The orchestrator will be responsible for deploying roles needed and coordinating between the roles for a scenario.   For example, the orchestrator should be able to run a processor-based scenario that:

Deploys a set of publishers
Deploys a set of processors
Configures both roles to shared some form of key/value store to coordinate around what events have been published
Can simulate processor scale up/down
Configures both roles to publish metrics to some common location.

Roles should have the ability to react to configuration that is specific to the scenario that they're running in.  For example, a publisher should be able to be told to collect metrics or not, allowing noise to be removed from consuming scenarios.

Stress tests should use an approach to bootstrap consistent with Test Resources and the dynamic creation of child entities.  It should be possible to run multiple scenarios concurrently without them interfering with one another and resources should be cleaned up after the run.

Success Criteria

The stress tests have been integrated into the repository under the Azure.Messaging.EventHubs package, in a dedicated stress folder.

The stress tests can be run locally with reasonable configuration/effort needed.

The stress tests can be deployed to AKS, using the common platform elements.

Stress tests have their own ReadMe that details how to run different scenarios, highlighting local runs.

Reference and Resources

Getting Started Guide
Richard's work for JS
Teams channel for stress  [scenarios discussion may be helpful] (Microsoft Internal)
Stress Wiki [tab of Teams channel] (Microsoft Internal)

//cc: @richardpark-msft
Just in case you had any thoughts on things that we've talked about and I forgot to include.

This looks really great to me. Some comments, but mostly just additive:

Adjust the architecture to support the concept of "roles" for the stress testing, allowing scenarios to be focused without taking on additional resource overhead.

I like thinking about this not just as overhead, but also a necessity if we're trying to mimic customers environments with our testing:

Enabling independent scaling of producers vs consumers (we'll have way more consumers than producers, more than likely)
Be realistic about how apps are generally written - consuming/producing are typically separate apps (this is our theory, at least).
Possible re-use in other scenarios that are not EH specific (ie, as we build out more stress testing in other areas it might be useful to reuse some of these producing/consuming containers).

Allow for the concept of an orchestrator to control scenarios. The orchestrator will be responsible for deploying roles needed and coordinating between the roles for a scenario.

I'd slate this as a "phase 2" feature, mostly because it might be complicated and we could technically use replicasets to accomplish 95% of this. It might be worth calling this out just to make it clear that there are some cheap options here (Kubernetes specific, but still), since it sounds larger than it might be.

Stress tests should use an approach to bootstrap consistent with Test Resources and the dynamic creation of child entities. It should be possible to run multiple scenarios concurrently without them interfering with one another and resources should be cleaned up after the run.

I know this is written more as requirements, but this is very much in @benbp's court and he's got a lot of great stuff here (got a chance to test drive it last week). You could technically leave this off here because there'll be no real responsibilities to the person building out these tests for .net
As always, love reading these.

This is great! Thinking about this from the perspective of the "stress test platform" I think there are a few key design decisions we'll want to make (granted these may change as we get further feedback from our work):

Should this class of testing try to conform to having started and finished states, or should we instead consider it as a workflow(s) that runs continuously and generates metrics/data until we stop it? With the current job model, start/finish aligns to pod exit codes. I could see the main job corresponding to the work the orchestrator is doing. If it isn't loop based, then whenever it's done with all its scale up/down work it will exit and we can come up with a generic way to trigger cleanup of child resources.
Related to ^^, do we want the orchestrator to run in the cluster, or be something manually invoked by a user? Further, should we expect that these tests are ever run as part of a scheduled pipeline/release gate, or rather that they are only kicked off manually (even if that means they run "forever").

Perhaps in the above scenarios, a triggered pipeline's job is to update the running tests to use new SDK versions.

To me the most important question is to what level do we generalize the orchestration pieces? I.e. we could go as far as supporting some sort of generic kubernetes object modification workflow, but I think it would be better to start with simple scripts wrapped in jobs that patch objects imperatively, and maybe eventually graduate to making those patch operations event driven or even kubernetes controllers that live in cluster. Script-based orchestration will be easier to share across languages though.

Roles should have the ability to react to configuration that is specific to the scenario that they're running in. For example, a publisher should be able to be told to collect metrics or not, allowing noise to be removed from consuming scenarios.

This might be a good area for us to come up with a general convention that future stress tests can follow. I can see this taking several forms, depending on whether we take a feature flag or IPC style approach, such as state sharing via configmaps and env vars, sockets on a shared volume, http endpoints between pods, etc. It also depends on whether we want to support behavioral changes with or without process/container restarts.

Adjust the architecture to support the concept of "roles" for the stress testing, allowing scenarios to be focused without taking on additional resource overhead.

I think with the helm package model I've been building out, this should be rather easy to accomplish. We can define different "roles" as their own helm packages, which can be configured and deployed independently or rolled up as dependencies into a main package.

Stress tests should use an approach to bootstrap consistent with Test Resources and the dynamic creation of child entities. It should be possible to run multiple scenarios concurrently without them interfering with one another and resources should be cleaned up after the run.

Similar to what @richardpark-msft said, I've already added support for this, so all you need to do is include an arm template/bicep file in the directory and resources will be deployed before the container starts.


use Alert;

I put "use Alert;" my controller laravel project 5.5 but It doesn't work ? Do you have any idea why it happened ? I don't understand. I manually add provider and alias by myself still doesn't working at all..

I'm having an issue with this as well but I just disregarded it and continue coding. my IDE is highlighting that import and giving me an error message "undefined class". Its still working for me, the alert is still showing its just that alert()->success('Message', 'title'); is no longer working as a Success Alert but a normal alert. same as with issue #62 . still waiting for an update from @uxweb :)

I had the same issue and solved it by using the following in the controller:-
use UxWeb\SweetAlert\SweetAlert as Alert;


Exception "The path is not of a legal form" when loading .NET Standard 2.0 assembly

Version 5.0 prerelease
Steps to reproduce:

Create a test project that targets .NET Standard 2.0
Reference a different assembly, the "system under test". The SUT is targeting .NET Framework 4.6.1
Build then load the assembly from the test project created in step 1
Result: System.ArgumentException() in NUnit.Gui.Presenters.TestPropertiesPresenter.GetTestType(TestNode testNode) with message "The path is not of a legal form"

Call stack is:
System.ArgumentException: The path is not of a legal form.
at System.IO.Path.LegacyNormalizePath(String path, Boolean fullCheck, Int32 maxPathLength, Boolean expandShortPaths)
at System.IO.Path.GetFullPathInternal(String path)
at System.IO.FileInfo.Init(String fileName, Boolean checkHost)
at NUnit.Gui.Presenters.TestPropertiesPresenter.GetTestType(TestNode testNode) in C:\Dev\NUnit\nunit-gui\src\nunit-gui\Presenters\TestPropertiesPresenter.cs:line 212
at NUnit.Gui.Presenters.TestPropertiesPresenter.DisplayTestInfo(TestNode testNode) in C:\Dev\NUnit\nunit-gui\src\nunit-gui\Presenters\TestPropertiesPresenter.cs:line 99
at NUnit.Gui.Presenters.TestPropertiesPresenter.DisplaySelectedItem() in C:\Dev\NUnit\nunit-gui\src\nunit-gui\Presenters\TestPropertiesPresenter.cs:line 90
at NUnit.Gui.Presenters.TestPropertiesPresenter.b__4_4(TestItemEventArgs ea) in C:\Dev\NUnit\nunit-gui\src\nunit-gui\Presenters\TestPropertiesPresenter.cs:line 58
at NUnit.Gui.Model.TestItemEventHandler.Invoke(TestItemEventArgs args)
at NUnit.UiKit.Elements.TreeViewElement.<.ctor>b__3_0(Object s, TreeViewEventArgs e) in C:\Dev\NUnit\nunit-gui\src\nunit.uikit\Elements\TreeViewElement.cs:line 50
at System.Windows.Forms.TreeView.OnAfterSelect(TreeViewEventArgs e)
at System.Windows.Forms.TreeView.TvnSelected(NMTREEVIEW* nmtv)
at System.Windows.Forms.TreeView.WmNotify(Message& m)
at System.Windows.Forms.TreeView.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

I am able to confirm this on my machine. I hope that i can dig into it tonight after work

It is probably something similar to #184. AFAICR I actually had a patch - somewhere - containing some better error handling when we have fullname="" (or perhaps I just returned testNode.Type if fullname=="" instead of trying to create a FileInfo)
https://github.com/NUnitSoftware/nunit-gui/blob/221c23c31f14e3db82be9419ac1b53e894bc565d/src/nunit-gui/Presenters/TestPropertiesPresenter.cs#L211).

indeed this is the problem @mikkelbu . the TestNode's XML is
<test-suite type='Assembly' id='0' name='' fullname='' runstate='NotRunnable' testcasecount='1' result='Failed' label='Error' start-time='2017-09-08 00:36:54Z' end-time='2017-09-08 00:36:54Z' duration='0.001' total='1' passed='0' failed='1' inconclusive='0' skipped='0' asserts='0'>
    <failure>
        <message><![CDATA[Could not load file or assembly 'nunit.framework' or one of its dependencies. The system cannot find the file specified.]]></message>
        <stack-trace><![CDATA[
Server stack trace:
   at System.Reflection.RuntimeAssembly._nLoad(AssemblyName fileName, String codeBase, Evidence assemblySecurity, RuntimeAssembly locationHint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
   at System.Reflection.RuntimeAssembly.InternalLoadAssemblyName(AssemblyName assemblyRef, Evidence assemblySecurity, RuntimeAssembly reqAssembly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
   at System.Activator.CreateInstance(String assemblyString, String typeName, Boolean ignoreCase, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes, Evidence securityInfo, StackCrawlMark& stackMark)
   at System.Activator.CreateInstance(String assemblyName, String typeName, Boolean ignoreCase, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes, Evidence securityInfo)
   at System.AppDomain.CreateInstance(String assemblyName, String typeName, Boolean ignoreCase, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes, Evidence securityAttributes)
   at System.AppDomain.CreateInstanceAndUnwrap(String assemblyName, String typeName, Boolean ignoreCase, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes, Evidence securityAttributes)
   at System.AppDomain.CreateInstanceAndUnwrap(String assemblyName, String typeName, Boolean ignoreCase, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes, Evidence securityAttributes)
   at NUnit.Engine.Drivers.NUnit3FrameworkDriver.Load(String testAssemblyPath, IDictionary`2 settings)
   at NUnit.Engine.Runners.DirectTestRunner.LoadPackage()
   at NUnit.Engine.Runners.DirectTestRunner.EnsurePackageIsLoaded()
   at NUnit.Engine.Runners.DirectTestRunner.Explore(TestFilter filter)
   at System.Runtime.Remoting.Messaging.StackBuilderSink._PrivateProcessMessage(IntPtr md, Object[] args, Object server, Object[]& outArgs)
   at System.Runtime.Remoting.Messaging.StackBuilderSink.SyncProcessMessage(IMessage msg)

Exception rethrown at [0]:
   at System.Runtime.Remoting.Proxies.RealProxy.HandleReturnMessage(IMessage reqMsg, IMessage retMsg)
   at System.Runtime.Remoting.Proxies.RealProxy.PrivateInvoke(MessageData& msgData, Int32 type)
   at NUnit.Engine.ITestEngineRunner.Explore(TestFilter filter)
   at NUnit.Engine.Runners.ProcessRunner.Explore(TestFilter filter)]]></stack-trace>
    </failure>
</test-suite>

So what if we check for a failure on the testNode.Xml.FirstChild.Name is failure then show the message??? seems a little weird. That or I suppose change testNode to show a error if it has one.

There is a possibility that a netstandard2.0 project referencing a net461 project is not designed to work.
Having said that, a netstandard2.0 project can reference net461 nuget packages, though. At best there will be a nuget warning that the net461 nuget package does not support netstandard2.0 and therefore the whole thing may not run in the target environment.
I don't know whether this makes a difference for this particular issue. It shouldn't impact the GUI, though, I think, as it should not make a difference what and how the assembly containing the tests references a difference package, project or assembly.

Test assemblies have to target a platform, not .NET standard.

Bad choice of words on my end.
Looking at a new format csproj file, there is a property name TargetFramework that you can set to different values, e.g. net461 or netstandard2.0. That is what I meant with by using the term "netstandard2.0 project", "netstandard2.0 nuget packages". Replacing TargetFramework with TargetFrameworks allows targeting multiple frameworkes, e.g. net461 and netstandard2.0.
Yes, ultimately, even if they target one of those, the assembly can then be executed only if the respective target framework is available on the platform on which it runs. So, in case of net461 it may only be runnable on a Windows box. In the case of targeting a framework netstandard2.0, it may also be runnable on a non-Windows box, e.g. any box that has .NET Core 2.0 installed.
What I tried to convey is this: given a project that targets framework netstandard2.0 it's ok to reference a nuget package that targets net4.6.1 only. However, it becomes problematic if the same project has a dependency on a different project targeting net4.6.1.
Therefore in our 50+ projects solutions we have structured it in such a way that projects targeting net461 can have a dependency on a project targeting netstandard2.0. We don't have dependencies the other way round between projects. We have projects targeting netstandard2.0, though, which have dependencies on nuget packages that target net461 (or other full .NET Framework versions). The latter is unlikely to work cross-platform. However, it allows migrating the solution incrementally towards .NET Standard/Core 2.0.


Empty paths in rendered JSON in Production (on Vercel) but not in Dev with NextJS 14

Describe the bug
When generating the machine-readable JSON in a dev environment on a Mac I can see the full JSON including the paths with the descriptions etc:
Output of http://localhost:3000/api/v2/openapi (openapi.ts file contains the code from README.md Usage #3 Step 1)
{
  "openapi": "3.0.0",
  "info": {
    "title": "Techmap's Daily International Job Postings Export API",
    "version": "2.0.1-beta"
  },
  "paths": {
  ...
    "/api/v2/meta/jobs/statistics": {
      "get": {
        "tags": [
          "Metadata Operations"
        ],
        "summary": "Get statistics for searchable fields such as workPlace, industry, timezone etc.",
        "description": "Returns a list of statistics for the given query parameter in the `/api/v2/jobs/search` endpoint.The returned JSON is described below and mainly consist of a list of statistics with counts and percentages for a given field.

Notes

This endpoint is only be available for MEGA subscription plans.

The results are capped at 100 values and are calculated from job postings within the last month (i.e., current date minus 1 month).

Search Query Parameters

General Parameters

`field`: A field name used in the search endpoint `/api/v2/jobs/search` as a parameter.

Usage Examples

Statistics on the workPlace field:
`/api/v2/meta/jobs/statistics?field=workPlace`

Statistics on the industry field:
`/api/v2/meta/jobs/statistics?field=industry`
",
        "parameters": [
          {
            "field": null,
            "in": "query",
            "name": "field",
            "required": true,
            "schema": {
              "type": "string",
              "example": "workPlace"
            },
            "description": "A field name used in the search endpoint `/api/v2/jobs/search` as a parameter."
          }
        ],
        "responses": {
          "200": {
            "description": "OK - indicating success and a correct query execution with results.
Response Schema
The API response provides a list of distinct field values for one field with a count of documents they are used in. Each distinct value object in the `result` array contains the following fields:

 `totalCount` (integer): The total number of jobs within the time range.
 `docsWithValue` (integer): The number of documents with any value for that field within the time range.
 `coverage` (double): The percentage of jobs in the time range that have any value.
 `valueCount` (integer): The total number of values in the time range.
 `aggregation` (object): The aggregation query we constructed and used to retrieve the distinct values.
 `values` (array): A list of up to 100 distinct values with their data. Each distinct value object in the `result` array contains the following fields:

 `key` (string): The value found in the job documents.
 `doc_count` (integer): The number of documents the value was found in.
 `percentage_relative` (integer): The percentage of documents with the value relative to all documents with any value (docs_with_field).
 `percentage_absolute` (integer): The percentage of documents with the value relative to all documents (total_doc_count).

"
          },
          "401": {
            "description": "Unauthorized request to API! (e.g., not trying to access API via RapidAPI)"
          },
          "403": {
            "description": "Forbidden request to API! (e.g., supscription plan to low)"
          },
          "404": {
            "description": "Not Found - due to an non-existant or invalid query parameter format (e.g., 'dateCreated')"
          },
          "405": {
            "description": "Method Not Allowed - due to unsupported HTTP Method (e.g., PUT, CONNECT, HEAD, etc.)"
          },
          "500": {
            "description": "Internal Server Error - used for general problems (i.e., a catch-all with more info in return value)"
          },
          "503": {
            "description": "Service Unavailable due to problems with our backend (e.g., the connection to our Database is interrupted)"
          }
        }
      }
    }
  },
  "components": {

  },
  "tags": [

  ]
}

However, in Production after deploying with Vercel they are not shown:
Output of https://example.com/api/v2/openapi
{
  "openapi": "3.0.0",
  "info": {
    "title": "Techmap's Daily International Job Postings Export API",
    "version": "2.0.1-beta"
  },
  "paths": {

  },
  "components": {

  },
  "tags": [

  ]
}

Relevant Dependencies:
"next-swagger-doc": "^0.4.0",
"openapi-types": "^12.1.3",
"swagger-parser": "^10.0.3",
"swagger-ui-react": "^5.10.5"

Btw. the generated OpenAPI docs work both in dev and production - only the generated json does not work.
I assume the problem is a different path to the API in the dev or prod environment of the nextjs application due to the deployment via Vercel. When I change pages/api/v2/ to api/v2/ in dev I get the same incomplete JSON.
I've tried to use /pages/api/v2/ and app/api/v2/ but that did not help in Production.
Any Ideas how to fix this or is this a bug?
Reproduction
Deploy to Vercel and check JSON in production
System Info
on Dev:

  System:
    OS: macOS 14.2
    CPU: (12) arm64 Apple M2 Max
    Memory: 703.17 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.8.1 - /opt/homebrew/bin/node
    Yarn: 1.22.19 - ~/.yarn/bin/yarn
    npm: 10.1.0 - /opt/homebrew/bin/npm
  Browsers:
    Chrome: 120.0.6099.129
    Edge: 120.0.2210.91
    Safari: 17.2

Used Package Manager
yarn
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

I believe this is the same with this discussion https://github.com/jellydn/next-swagger-doc/discussions/652. On Vercel, it does NOT have the source code at the runtime so I think that is empty. Probabbly you should generate it before uploading to Vercel.


Patrol prevents shell-executed migrations

Steps

Install Craft and the Patrol plugin on an instance
Create a craft migration file in the craft_app/migrations folder
Attempt to run ./craft migrate --interactive=0
Observe that Patrol expects a getUrl method on the current request in src/services/PatrolService.php in the watch() method.

Expected
Patrol should not watch console requests.

I logged this in the wrong repo. Closing.


Preview PRs in github.io

As a stakeholder of the open-scd project I would like to test functional behavior of the PRs in a temporary URL which features a copy of OpenSCD made with the code in the branch of the pull request.
Acceptance Criteria:

A github workflow is added (runs on pull_request) which builds and deploys OpenSCD to a custom URL

The pull request with the GitHub Action to generate a preview has been already merged:

https://github.com/openscd/open-scd/pull/1547

It currently deploys the wrong folder here:
https://github.com/openscd/open-scd/blob/main/.github/workflows/pr-preview.yml#L33
I've already fixed it. It needs to deploy from /packages/distribution/build.
The next problem is that OpenSCD ist deployed under a path like .../pr-preview/pr-4/ and it should load files from this subfolder but it tries to load the files from the root.
The next step to set a base URL at build time so it loads the correct files.

The solution is make the referenced files' path  adjustable.
With snowpack we can set the base URL: buildOptions.baseUrl
Then we can use %PUBLIC_URL% to reference files:
- <script type="module" src="/main.js"></script>
+ <script type="module" src="%PUBLIC_URL%/main.js"></script>

To change this from the outside we can configure snowpack in snowpack.config.mjs like this:
  buildOptions: {
    baseUrl: process.env.PUBLIC_URL || '/',
  },

and set the path in when we start the build process eg.:
PUBLIC_URL=open-scd/pr-preview/pr-1 npm run-script build
the reference to built-in plug-ins in plugins.ts we can figure out dynamically dependent on where OpenSCD is deployed:
const path = location.origin + location.pathname + plugin

The next step is to fix the failing tests, because of the changed plugin paths.

The tests were failing because the pluginIds in the tests were still relative paths but OpenSCD now generates absolute paths and OpenSCD could not find them in local storage.

The final problem was that GitHub Pages, by default, does not server files from folders that start with _. It has something to do with jekyll, but it was an easy fix because you can disable jekyll with a .nojekyll file at root level.


General Issue: Allow expressions

Background Information
Expression should be allowed. The exact scope needs to be defined and functions (in admiral and admiraldev) and documentation updated accordingly.
Also consider deprecating functions, e.g., derive_var_merged_exist_flag().
Definition of Done
No response

Q2 update?  @thomas-neitmann @bundfussr @rossfarrugia.  Feel like we got quite a bit on our plates right now...and in my cup.

Agree, this is a big one and will have lots of impacts that need to be considered within core package and extensions. We should raise it at the next sponsor/leads meeting so that the other POs all get a heads up it's coming.

I agree, we should implement it in Q2.

I thought worth to add some more details to this issue given its such a key one and some users may be curious to understand more... so here's the original proposal on this:
At the moment the programming strategy forbids to specify expressions for arguments like new_vars, order, set_values_to. The intention was to avoid that the function calls get too complicated. However, there are more and more use cases where the restriction forces the users to implement work-arounds which are more complicated than specifying the expressions directly.
For example
  derive_param_extreme_event(
    adrs,
    dataset_adsl = adsl,
    dataset_source = tu_ext,
    filter_source = TUTESTCD == "TUMIDENT" & TUEVAL == "INVESTIGATOR" & TUSTRESC == "TARGET" & TUDT <= !!ref_var,
    set_values_to = vars(
      PARAMCD = "MDIS",
      PARAM = "Measurable Disease at Baseline by Investigator"
    )
  ) %>%
  mutate(
    SRCDOM = case_when(
      PARAMCD == "MDIS" & AVALC == "Y" ~ "TU",
      TRUE ~ SRCDOM
    ),
    SRCVAR = case_when(
      PARAMCD == "MDIS" & AVALC == "Y" ~ "TUSTRESC",
      TRUE ~ SRCVAR
    ),
    SRCSEQ = case_when(
      PARAMCD == "MDIS" & AVALC == "Y" ~ TUSEQ,
      TRUE ~ SRCSEQ
    )
  )

could be simplified to
  derive_param_extreme_event(
    adrs,
    dataset_adsl = adsl,
    dataset_source = tu_ext,
    filter_source = TUTESTCD == "TUMIDENT" & TUEVAL == "INVESTIGATOR" & TUSTRESC == "TARGET" & TUDT <= !!ref_var,
    set_values_to = vars(
      PARAMCD = "MDIS",
      PARAM = "Measurable Disease at Baseline by Investigator",
      SRCDOM = if_else(AVALC == "Y", "TU", NA_character_),
      SRCVAR = if_else(AVALC == "Y", "TUSTRESC", NA_character_),
      SRCSEQ = if_else(AVALC == "Y", TUSEQ, NA_integer_),
    )
  )

The latter is also less error-prone because if the condition PARAMCD == "MDIS" is missed in the first example, the SRC* variables for previously derived parameters are changed as well.
The same applies in the case that derive_param_extreme_event() is called and AVAL needs to be derived in addition to AVALC.
Also many examples where imputed dates are required could be simplified. For example:
ds_ext <- derive_vars_dt(
  ds,
  dtc = DSSTDTC,
  new_vars_prefix = "DSST"
)

adsl <- adsl %>%
  derive_vars_merged(
    dataset_add = ds_ext,
    by_vars = vars(STUDYID, USUBJID),
    new_vars = vars(EOSDT = DSSTDT),
    filter_add = DSCAT == "DISPOSITION EVENT" & DSDECOD != "SCREEN FAILURE"
  )

to
adsl <- adsl %>%
  derive_vars_merged(
    dataset_add = ds,
    by_vars = vars(STUDYID, USUBJID),
    new_vars = vars(EOSDT = convert_dtc_to_dt(DSSTDTC)),
    filter_add = DSCAT == "DISPOSITION EVENT" & DSDECOD != "SCREEN FAILURE"
  )

Lifting the restriction was also requested by users see https://github.com/pharmaverse/admiraldev/issues/195.
Without the restriction we could deprecate derive_var_last_dose_grp(), derive_var_merged_cat(), and derive_var_merged_character() as well as the aval_fun argument (in derive_param_exist_flag() and admiralonco functions).
And admiral would be more in line with dplyr as functions like mutate() or arrange() do not have this restriction.
From an existing admiral user perspective there shouldn't be so much impact as the change would be backward compatible. I would though expect our ADaM templates to get updated to make more use of this feature to cut down length of the scripts, and also in the vignettes it would be good to show at least one example of both ways of doing this, so those that still prefer to break things up into smaller steps still could.


Drop IE11 for Columns in favor of flexbox's gap

We decided it's time to drop IE11 support for our Columns component in
favor of using flexbox's gap property. Previously, we tried to avoid
gap (because it's not supported by IE11) but this has lead to some
gnarly issues with negative margins.

This change is


Build error on Windows

OS: Microsoft Windows [Version 10.0.17134.112] x64
Rust 1.28.0 nightly
cargo 1.28.0-nightly (e2348c2db 2018-06-07)
thread '<unnamed>' panicked at 'Failed to insert document.', libcore\option.rs:960:5
note: Run with `RUST_BACKTRACE=1` for a backtrace.

I think this issue is same as tokio and hyper lib https://stackoverflow.com/questions/50644880/cannot-recursively-call-into-core-when-trying-to-achieve-nested-concurrency?rq=1.

I could build it today also on Windows 10 with:

stable-x86_64-pc-windows-gnu
nightly-x86_64-pc-windows-gnu
nightly-x86_64-pc-windows-msvc (rustc 1.29.0-nightly (6a1c0637c 2018-07-23)

Just cloned the repo and run cargo build.

It seems to me that this was an issue with a specific nightly version of the compiler and no longer is a problem. Any objections to me closing this issue @xdk78?

Hello everyone! We've just released an alpha of an official MongoDB Rust driver; you can read about it here! Because of this, we won't be updating this driver anymore, and the repository will be made read-only as soon as I get finished posting these messages. Thanks for using this driver, and we hope you try out the new one!

Apologies, the blog post link is this: https://www.mongodb.com/blog/post/announcing-the-official-mongodb-rust-driver


Could not find a declaration file for module 'simplebar'

Describe the bug
When importing SimpleBar from 'simplebar', TS shows bellow error:
TS7016: Could not find a declaration file for module 'simplebar'. '/path' implicitly has an 'any' type.
Try npm i --save-dev @types/simplebar if it exists or add a new declaration (.d.ts) file containing declare module 'simplebar';
To Reproduce
Steps to reproduce the behavior:

Create new project with 'npm create vite@latest'
Select Vanilla/Typescript environment
npm i simplebar
Add import to the main.ts => import SimpleBar from 'simplebar'

Expected behavior
Typescript types should be included within the simplebar folder.
@types/simplebar are depreciated by now, because package includes types in the simplebar-core.

Software
Version(s)

SimpleBar
6.2.2

npm/Yarn
8.11.0

If you're using React try npm install simplebar-react --save  and then import
import SimpleBar from 'simplebar-react';
import 'simplebar-react/dist/simplebar.min.css';

https://github.com/Grsmto/simplebar/tree/master/packages/simplebar-react#readme

Fix published<EMAIL_ADDRESS>


Preview

👷 Deploy Preview for timely-brigadeiros-dd9fc4 processing.

Name
Link

🔨 Latest commit
d041dbbd6715a98f2a245b860673d8beb01dcc03

🔍 Latest deploy log
https://app.netlify.com/sites/timely-brigadeiros-dd9fc4/deploys/64d26996057a4e0008ba90f0


@conda-forge-admin, please add bot automerge

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/pyglfw-feedstock#16.


E_USER_CANCELLED error, but payment successful

Hi. I have one problem on my app with rn-iap. Unfortunately, I can't reproduce it, only fix consequences.
Version of react-native-iap
4.3.0
Version of react-native
0.59.10
Platforms you faced the error (IOS or Android or both?)
IOS
Expected behavior
Payment process successful
Actual behavior
Payment success for me (as developer), but appstore or rn-iap return error E_USER_CANCELLED (can't connect to itunes store). User pay his money (successfully), but don't get subscription because app got error.
Tested environment (Emulator? Real Device?)
Real Device (Production version)
Steps to reproduce the behavior
Can't reproduce

We are currently working on an example project called DoobooIAP to deliver full featured react-native-iap working example. These kind of non-existing reproducible issues are very hard to resolve. I hope you can contribute to our project either!

Hi. I can add some info, hope it'll help.
Error appears after RNIap.finishTransaction(purchase)
code E_UNKNOWN
column 1625
domain SKErrorDomain
userInfo.NSLocalizedDescription Cannot connect to iTunes Store
Payment is successful in store, but we get such error and don't give access for user.
I saw proposals RNIap.addAdditionalSuccessPurchaseListenerIOS in similar situations, but now I can't find this method in your docs.

addAditionalSuccessPurchasListenerIOS is removed and should not be used. You can see my medium blog.
Refer to ios forum, I found may trouble syncing purchases with testflight user for production.

Hi. Yes, we don't use this method, but still get problems with payments on ios. Our current version of rn-iap 4.4.3

Looks like this relies on Apple and not our module so please check your config first. Going over whole steps again sometimes fixes.
Without any more investigation, I can't move this forward.

I think you will get an E_USER_CANCELLED error even though the purchase was successful if you don't wait for RNIap.getProducts to return before calling RNIap.requestPurchase. We call RNIap.getProducts as soon as the app opens to avoid this, but that call has taken up to 11.5 days to return (It took  999,286,298 milliseconds once based on Date.now() before and after the 'await RNIap.getProducts' line). Also note that I think it can take a long time for purchaseErrorListener and/or purchaseUpdatedListener to execute. I found a recent case where the user got the successful purchase email, but we got an E_USER_CANCELLED error. In that case, the RNIap.getProducts only took 35 milliseconds, but finished 3 minutes after the RNIap.requestPurchase because RNIap.getProducts has to be called after the listeners are set up (if I remember correctly).

RNIap.getProducts has to be called after the listeners are set up

Just to be sure, is this how it has to be? Couldn't find anything that said this or the opposite, hoewever, right now I'm doing it in the opposite order (getProducts is called before the listeners are setup), I believe it's working but would like to know if there could be a problem with the way I implemented it.
Thank you!

There could have been updates to fix this. But I was able to reproduce the
error on successful transactions. It could depend on how long it takes for
getProducts to return.
On Tue, Sep 15, 2020 at 11:12 PM Manuel Grondona<EMAIL_ADDRESS>wrote:

RNIap.getProducts has to be called after the listeners are set up
Just to be sure, is this how it has to be? Couldn't find anything that
said this or the opposite, hoewever, right now I'm doing it in the opposite
order (getProducts is called before the listeners are setup), I believe
it's working but would like to know if there could be a problem with the
way I implemented it.
Thank you!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dooboolab/react-native-iap/issues/935#issuecomment-693143969,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABEMGGL7RQP6PSMMCZLHSLSGAULJANCNFSM4KZCIFSA
.

@cumom Thank you for your answer! In my experience, I'm using getSubscriptions() and I actually use the response to map through the different subscriptions to generate the purchase buttons, so if I don't have the response, I don't have buttons. So far I haven't had any delay problems with that method, but I do call it before setting up the listeners, so that is why I was asking about that process. Either way, I feel some laggy behavior, but it could be the way I'm implementing things..


Loopring Homepage Milestones

Added Milestone on Homepage
First zkRollup (DEX)
name: First zkRollup (DEX)
date: 2019-12-04
link: https://medium.loopring.io/loopring-deployed-protocol-3-0-on-ethereum-a33103c9e5bf
description: Loopring is live, bringing the first DEX protocol on rollup technology.',

Announcement Blog:  https://medium.loopring.io/loopring-deployed-protocol-3-0-on-ethereum-a33103c9e5bf
Deployment info: https://github.com/Loopring/protocols/blob/master/packages/loopring_v3/deployment_mainnet_v3.0-beta4.md
Here is the first Block: https://etherscan.io/tx/0xacb0a08bce4f7e63bf18159d93f0328d5ca0712b0ed332c58dda8219f7b63a55
This is WeDex's contract address, they built an exchange on our rollup: https://etherscan.io/address/0x7D3D221A8D8AbDd868E8e88811fFaF033e68E108
As you can see here, the contract was deployed Dec 5, 2019: https://etherscan.io/txs?a=0x7D3D221A8D8AbDd868E8e88811fFaF033e68E108&p=432
Tweet from Vitalik (Jan 1, 2020): https://twitter.com/VitalikButerin/status/1212300965023567872?s=20.

@adam-bro consider adding LRC staking milestone while you are at it


Add compatibility matrix

@amitkumar-amd https://rocm.docs.amd.com/en/latest/about/compatibility/precision-support.html moving to https://rocm.docs.amd.com/en/latest/compatibility/precision-support.html in this PR.


fix: remove type property after transformation

Issue #, if available:
#1475
Description of changes:
removed the Type property after transformation as its used by SAM internally to determine the type of the policy to add.
Description of how you validated changes:
Checklist:

[x] Write/update tests
[x] make pr passes
[ ] Update documentation
[x] Verify transformed template deploys and application functions as expected
[ ] Add/update example to examples/2016-10-31

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1504 into develop will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #1504      +/-   ##
===========================================
+ Coverage    94.13%   94.13%   +<.01%
===========================================
  Files           78       78
  Lines         4925     4926       +1
  Branches       990      990
===========================================
+ Hits          4636     4637       +1
  Misses         134      134
  Partials       155      155

Impacted Files
Coverage Δ

samtranslator/model/eventsources/pull.py
82.27% <100%> (+0.22%)
:arrow_up:

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 aba498f...407a428. Read the comment docs.


Allow generate_dipeptide_top_pos_sys to accept demap_CBs

Description
This PR piggy backs off of https://github.com/choderalab/perses/pull/860, where I forgot to allow demap_CBs to be passed to generate_dipeptide_top_pos_sys.
Motivation and context
This is necessary because to enable generating protein mutation test systems with demapped CBs.
How has this been tested?

Change log

@mikemhenry @ijpulidos  : could you review and merge?

@zhang-ivy This might be for another issue/PR, but just wondering if we have tests for this case. Such as a test that requires demap_CBs=True for it to pass. Thoughts? (I guess it is related to the #845 , maybe we can engineer some test around that case).

@zhang-ivy This might be for another issue/PR, but just wondering if we have tests for this case. Such as a test that requires demap_CBs=True for it to pass. Thoughts? (I guess it is related to the #845 , maybe we can engineer some test around that case).

Yes, it couldn't hurt to have another test that checks whether demap_CBs actually demaps the CBs. Would you want to take a stab at this? It should be something like this: choderalab -- but would just have to change the reference map to not include CBs.


Infrequent failure Mono Windows x64, ThreadPoolTests.QueueRegisterPositiveAndFlowTest.

Only observed a couple of time in CoreFX, corlib-xunit Windows x64 Mono CI,
https://jenkins.mono-project.com/view/Mono/job/z/label=w64/8715/testReport/junit/Test collection for System.Threading.ThreadPools.Tests/ThreadPoolTests/System_Threading_ThreadPools_Tests_ThreadPoolTests_QueueRegisterPositiveAndFlowTest/
Test is using a local variable to set state between threads and very infrequent, the reading thread seems to pick the old value.
Looks like coordination between reader/writer thread works as expected and since this is an infrequent failure, doesn't look like a coordination issues or problem with the ThreadPool (would have expected failures in many tests) implementation.
The test is however using a fragile construction using a local variable (backgroundAsyncLocalValue) to share state between the writer/reader thread without any synchronization involved. That could lead to infrequent failures due to memory read/write optimizations.
Since this is a shared local variable captured and shared between main thread and thread pool it needs to have proper acquire/release semantics, or it could cause failures like these due to memory optimization.

Could you please upstream the change


Make the show option work in child browser windows

For some reason, it seems that Electron doesn't obey show: false in browser windows with a parent properties. Also, the ready-to-show event is not triggered for them.

Closing stale issues


Added some fancy filtering

resolve #531

@wongband  I don't know if you can fix or if this would be a @cydoval thing, but these get chopped once you select longer text

@jSylvestre Sorry about that, it should be ">75". I will fix it right now


Allow deposit of works (with no relation) via some deposit API, e.g. works

This is supported in the deposit API.


Missing list group border in dark mode

Describe the bug
See the title
Reproduction
NA
Version and System Info
NA

Fixed.


Add neon theme.

Customizable colorscheme with excellent italic and bold support, dark and light variants. Made to work and look good with TreeSitter.

Could you rebase your commits so it's just a single one?

@tversteeg you can squash and merge them.