Negroni does not share context accros diff middlewares properly

I'm not sure if I'm missing something, but I'm not able to use context to pass context through my middlewares:
Server:
func NewApiServer() (*ApiServer, error) {
	a := &ApiServer{}
	a.n = negroni.New(negroni.NewRecovery(), negroni.HandlerFunc(errorHandlerMiddleware))
	a.drawRoutes()
	return a, nil
}

func (a *ApiServer) drawRoutes() {
	a.mux = mux.NewRouter()
	a.muxAuth = mux.NewRouter()

	a.mux.HandleFunc("/debug/helloworld", HelloWorldHandler)
	a.muxAuth.Handle("/services", &ServiceHandler{}).Methods("POST")
	a.mux.PathPrefix("/").Handler(negroni.New(negroni.HandlerFunc(authorizationMiddleware), negroni.Wrap(a.muxAuth)))
	a.n.UseHandler(a.mux)
}

Middlewares:
func authorizationMiddleware(w http.ResponseWriter, r *http.Request, next http.HandlerFunc) {
	context.Set(r, "key", "unauthorizedRequest")
	val := context.Get(r, "key")
	log.Print(val.(string)) // It works here!
	return
}

func errorHandlerMiddleware(w http.ResponseWriter, r *http.Request, next http.HandlerFunc) {
	next(w, r)

	val := context.GetAll(r)
	log.Print(len(val)) // 0!
}

Just did another couple of tests:
Still "0" elements:
func errorHandlerMiddleware(w http.ResponseWriter, r *http.Request, next http.HandlerFunc) {
	context.Set(r, "key", "unauthorizedRequest")
	next(w, r)

	val := context.GetAll(r)
	log.Print(len(val))
}

"1" element in the slice:
func errorHandlerMiddleware(w http.ResponseWriter, r *http.Request, next http.HandlerFunc) {
	next(w, r)
	context.Set(r, "key", "unauthorizedRequest")
	val := context.GetAll(r)
	log.Print(len(val))
}

It seems that the request is being somehow "recreated" (?)
The error happens because of this line:
a.mux.PathPrefix("/").Handler(negroni.New(negroni.HandlerFunc(authorizationMiddleware), negroni.Wrap(a.muxAuth)))

If I use the middleware like this:
	a.n = negroni.New(negroni.NewRecovery(),
		negroni.HandlerFunc(errorHandlerMiddleware),
		negroni.HandlerFunc(authorizationMiddleware))

then it works. But I need to use the authorization middleware just for some routes :(

@albertoleal I just came across the exact same issue. Any work arounds?

This is not a negroni issue, but the default configuration of mux.Router. By default mux will clear the context after it called the handler (https://github.com/gorilla/mux/blob/master/mux.go#L97). You can override this behaviour with:
r := mux.NewRouter()
r. KeepContext = true

Make sure to clear the context yourself. See also:
http://www.gorillatoolkit.org/pkg/context
"The Routers from the packages gorilla/mux and gorilla/pat call Clear() so if you are using either of them you don't need to clear the context manually."

I'm closing this out as it seems to have been answered adequately in comments, but please yell if you think a bug is still lurking.


Update photos of Memories Section

Download best 8 photos from https://www.flickr.com/photos/kondapivskrishnaprasanth
compress them
update on website

I am in the process of solving this issue. To me it looks that the aspect ratio is not good if we try to fit 8 pictures in the given space.

It would be fine, we are using bootstrap and it will handle the placement of the photos according to available screenspace.


Unreadable on TOR browser, half readable on Brave, win user

On TOR browser after login I get colorful rectangles:

and on Brave browser the half of the letters are missing:

I run it on latest Umbrel 0.4.11
Any ideas?

I have the same issue over TOR. Scrambled screen.

Wow, no idea. I'll look into it! Works OK otherwise?

Problem still exists,  No way to know if works ok Otherwise.  Please fix

.

Same here.  Tor Browser.

hello,
+1
using THOR on microsoft surface
thanks for your help

hi
+1
same issue, using Tor, on a microsoft surface pro.
thanks for your help

Seems to be an upstream issue in xtermjs. It's easily reproducible by visiting https://xtermjs.org/ with Brave or Tor. Turning off hardware acceleration fixes it for Brave there but not in Lightning Shell app.
On Tor browser the issue seems to be a blocked feature, read under 2. HTML5 Canvas Image Extraction:
https://2019.www.torproject.org/projects/torbrowser/design/#fingerprinting-linkability
This will probably need to be resolved by switching to DOM as the renderer in ttyd: https://github.com/xtermjs/xterm.js/issues/3271

Just adding this is not an issue in firefox 101.0.1

I can confirm this is still an issue on Tor Browser 11.0.15
Allowing the onion site to use html5 canvas image data fixed the problem.

I don't see this setting anywhere on the MacOS version of the Tor browser 11.5.1. Where do you find that setting?

I can confirm this is still an issue on Tor Browser 11.0.15
Allowing the onion site to use html5 canvas image data fixed the problem.

I don't see this setting anywhere on the MacOS version of the Tor browser 11.5.1. Where do you find that setting?

If it doesn't prompt I think you should be able to find it to the left of the site URL (little onion symbol), there you should see something like "Connection secure" with an arrow, from there "More information" -> "Permissions"

With brave Browser, just deactivate Brave shields.
If I could help, you may donate some sats to:
my lightning thanks :-)


[BUG] misspelled word in the readme

Misspelled word in the readme
In the Docker session of the Readme, the Docker compose command is misspelled.
Expected behavior
docker-compose up -d
Screenshots

OS: Windows 10 - OS Build 18363.1139
Visual Studio Professional 2019
.Net SDK 3.0

Hi, thank you for you contribution!
The issue is already adressed and has been fixed.


cold start nodes' embedding

When we have nodes which do not have edges (because of cold start) we currently do not get embeddings for them. Can we add the feature for getting the node embeddings even when they do not have edges which is the novel output from GATNE-I.

Sorry. I will add this feature soon.

I couldn't find the code changes to include the cold-start nodes. I didn't see any code pushes recently. Can you please point me to the same as you have closed this issue. Thanks a ton!

I'm still trying to figure out the same thing as @mohit-udaan.


NoSuchFieldError: error-prone fails due to break in for/do/switch statement

I get the following error whenever error-prone hits a break-keyword in a do,for or switch statement.
I am running on OpenJDK 12.0.2
An unhandled exception was thrown by the Error Prone static analysis plugin.
error-prone version: 2.3.4
     BugPattern: Finally
     Stack Trace:
     java.lang.NoSuchFieldError: label
  	at com.google.errorprone.bugpatterns.Finally$FinallyJumpMatcher.<init>(Finally.java:171)
  	at com.google.errorprone.bugpatterns.Finally.matchBreak(Finally.java:79)
  	at com.google.errorprone.scanner.ErrorProneScanner.processMatchers(ErrorProneScanner.java:451)
  	at com.google.errorprone.scanner.ErrorProneScanner.visitBreak(ErrorProneScanner.java:528)

I found the problem:
The class com.sun.tools.javac.tree.JCBreak doesn't have a field named label in OpenJDK12.0.2.
Here is the problem:
public FinallyJumpMatcher(JCBreak jcBreak) {
      this.label = jcBreak.label;
                           ^^^^^ label is not declared

Duplicate of #1342, see also #1106. Fixed in https://github.com/google/error-prone/commit/a3bd193f0597d28aa382e911dfae8c9b57951ac0


Track EPAC usage

Add usage tracking to EPAC (only applies to deployments)

https://learn.microsoft.com/en-us/partner-center/marketplace/azure-partner-customer-usage-attribution#example-azure-powershell
Provide an opt-out (create a flag in global-settings.json)
Document opt-out similar to: https://github.com/Azure/ALZ-Bicep/wiki/CustomerUsage
Different attribution example: https://github.com/Azure/alz-monitor/wiki/Telemetry
We will need to create a dashboard

Scripts to track:

Scripts\Deploy\Build-DeploymentPlans.ps1
Scripts\Deploy\Deploy-PolicyPlan.ps1
Scripts\Deploy\Deploy-RolesPlan.ps1
Scripts\Operations\Create-AzRemediationTasks.ps1
Scripts\Operations\New-AzPolicyReaderRole.ps1

This will be considered a breaking change. Version will move from 7.x.x to 8.0.0


Bring back jwt session management in the GoTrueAdminApi

Feature request
Hello, so I've been using Supabase for some months now, primarily been using it to build backend api's. One neat feature of supabase which I enjoyed working with was the auth session management which came default out of the box in the JS client library i.e the GoTrueApi.
I recently started a new project so by default decided to just download the latest version of supabase. Read through the release docs and I see there has been a couple of updates to the client api which is very good. But from working with it I did notice that the GoTrueApi (now called GoTrueAdminApi) received some updates which sadly also led to the removal of the api specific session management provded out of the box. So stuff like access_token generation on login and just the whole jwt management stuff is no longer included.
Basically my request is asking if this is something the team can consider bringing back as it does help with building out api's.
Or has it been moved to somewhere else in the api that is not clear?
Is your feature request related to a problem? Please describe.
Its not necessarily a problem, the feature I described above is related to the access_token management provided by supabase being omitted in the latest version and asking if its likely for it to be rolled back into future version.
Describe the solution you'd like
It would be helpful to have api specific session management included once again in the GoTrueAdminApi for the supabase js client
Describe alternatives you've considered
There are 2 alternatives to this as far as I can tell, option 1 is to use an older version of the supabase client library that supports api session management out of the box. And option 2 is to write custom code that handles the jwt access token generation logic.

Hey @ngbede what methods are you specifically referring to as missing?

Hello @hf was referring to the Session type in the auth methods, I digged in a bit and found out they were moved to a separate method in the auth supabase-js package, so its no issue again. Sorry for the late response. Will close the issue now.


Create .gitattributes

For highlighting the syntax

Closed my PR but used my code, why couldn't you just merged it?

Shubham, I just copied gitattrs file from zeppelin-solidity. It`s the same as in your PR. But thank you for your help!


docker plugin not working with docker 1.12.0-rc3

There is a new field passed to the docker plugin called 'ID':
https://github.com/docker/docker/pull/21015
The plugin (flocker 1.13.0) fails to mount the volume with the following error:
{"exception": "exceptions.TypeError",
     "reason": "volumedriver_mount() got an unexpected keyword argument 'ID'",
     "timestamp":<PHONE_NUMBER>.957701,
     "traceback": "
    Traceback: <type 'exceptions.TypeError'>: volumedriver_mount() got an unexpected keyword argument 'ID'
      /opt/flocker/local/lib/python2.7/site-packages/twisted/internet/defer.py:150:maybeDeferred
      /opt/flocker/local/lib/python2.7/site-packages/flocker/restapi/_infrastructure.py:264:loadAndDispatch
      /opt/flocker/local/lib/python2.7/site-packages/twisted/internet/defer.py:150:maybeDeferred
      /opt/flocker/local/lib/python2.7/site-packages/flocker/dockerplugin/_api.py:118:wrapped
     --- <exception caught here> ---
     /opt/flocker/local/lib/python2.7/site-packages/twisted/internet/defer.py:150:maybeDeferred",
  "message_type": "eliot:traceback",
  "task_uuid": "6d68e5a3-1ef3-44e1-a477-e1cac6d76da2",
  "task_level": [3
}

@Nomon thanks for reporting this, I was away on some time off but will be digging into 1.12 and making sure things work correctly and making needed changes, will update this issue as we go along

Looks like 1.12 was released last week and all our Flocker Docker plugin tests are now broken :-(

http://ci-live.clusterhq.com:8080/job/ClusterHQ-flocker/view/master/job/master/view/cron/job/_run_acceptance_on_GCE_Ubuntu_Xenial_with_GCE/7/#showFailuresLink

@wallnerryan Thanks for creating the Jira issue.
Let me know if you want me to review / help with anything

https://clusterhq.atlassian.net/browse/FLOC-4480

should be fixed now that https://github.com/ClusterHQ/flocker/pull/2880 is merged. Next release will/should include this.


⚠ Bofrost.es has degraded performance

In 4ad7de0, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1549 ms

Resolved: Bofrost.es performance has improved in 29d59d5 after 9 minutes.


styled-components v5 unable to merge defaultProps in some runtimes

Overview
I've found that  #2597, specifically this mixinDeep usage, can cause styled-components to fail with specific runtimes. i.e. Buble, react-styleguidist, etc.
We've recently run into this issue while migrating from v4.x -> v5.x with react-styleguidist as our documentation platform. (with some other styleguidist users)
Detail
When extending other styled components that contain a defaultProps instance that contains functions the mixinDeep utility can end up iterating infinitely. This leads to:

I've outlined a possible fix below, but want to discuss before opening a PR.
Environment
Any version of react-styleguidist or buble with styled-components@^5. These environments return values when for(.. in ..) is used against a function that don't play well with the current mixinDeep utility.
## System:
 - OS: macOS 10.15.4
 - CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
 - Memory: 3.76 GB / 32.00 GB
 - Shell: 5.7.1 - /bin/zsh
## Binaries:
 - Node: 12.13.0 - ~/.nvm/versions/node/v12.13.0/bin/node
 - Yarn: 1.22.4 - ~/.yarn/bin/yarn
 - npm: 6.12.0 - ~/.nvm/versions/node/v12.13.0/bin/npm
 - Watchman: 4.9.0 - /usr/local/bin/watchman
## npmPackages:
 - babel-plugin-styled-components: 1.10.7 => 1.10.7
 - styled-components: 5.1.0 => 5.1.0

Reproduction
https://codesandbox.io/s/styled-components-buble-v5-o32qq?file=/src/components/Button.js
As the error produced is a stack overflow, it can be difficult to view with Codesandbox. To crash the connection you can uncomment L15-17 of Button.js and refresh the browser section.
Steps to reproduce
This error occurs when styled-components extend each other when they both contain defaultProp values that contain functions. Example:
const theme = {
  shadow: color => `5px 5px ${color}`
};

const Button = styled.button``;
Button.defaultProps = { theme };

const RedButton = styled(Button)`color: red;`;
RedButton.defaultProps = { theme };

Expected Behavior
I expect the usage above to not trigger a RangeError.
Possible Solution
The mixinDeep utility is already checking against several reserved object keys. Adding a check for the _name value is correcting the issue for me locally.
If this is a valid fix I can open a PR. Thanks!

Closing due to inactivity. Possible out of date

#3576 has more details on essentially the same issue - also links to the corresponding styleguidist issue


Re-run course graphs model

Since we have new e-calendar data, we need to re-run the course graphs model on this new data

Done in #592


Use new ASP.NET Core 2.1 features

#240
add ModelStateInvalidFilter to all WebHook actions

add WebHookFilterProvider class and refactor WebHookRoutingProvider.AddFilters(...) to there

use MediaTypeHeaderValue for subtype suffix matches

nits:

make MediaTypeHeaderValues in WebHookVerifyBodyTypeFilter read only
add usings for namespaces used only in updated doc comments

VS "Remove 'n Sort" doesn't remove these anymore

@Eilon you may want to scan the increasingly-consistent doc comments I changed.
@rynowak actual product changes are pretty small in this PR (not as small as in #264 but small).

:up::date: to rebase on release/2.1

ec834ebabb


Call the function Stop when custom op released

Description
Hi there,
As title, it may fix #18789
Checklist
Essentials

[ ] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] Code is well-documented

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

In src/initialize.cc,  LibraryInitializer::atfork_prepare() calls CustomOperator::Get()->Stop();.


Upgrade Android SDK to v30

Reason for migration

Android SDK 30 is lastest stable build for developers. Staying on the latest version comes with security and other new features.
SDK 28(present SDK in code) is now deprecated by Google Play.

Already Fixed

If working, bump it to higher SDK ?

31 needs a few package upgrades and configuration🤧


Retweet URL is incorrect

When publish a retweet, the resulting ITweet instance has the retweetURL missing the author
It is always something like:
https://twitter.com//status/[TWEETID]
The author is not part of the URL.

Could you please share your code so that I understand better your case.

I ran tests in 5.0 and everything worked fine.


feat: add host ssh and sftp client map cache for sftp

Add host ssh and sftp client map cache for sftp action. This feature is disscussed here sudo support for scp

need test many node join.

Test done, a cluster which had 3 nodes was installed successfully via sealos.


hotfix numpy>=2.0

np.VisibleDeprecationWarning -> np.exceptions.VisibleDeprecationWarning https://numpy.org/devdocs/release/2.0.0-notes.html#numpy-2-0-python-api-removals

tuple was returned as (np.int64(4), np.int64(5)) instead of (4, 5) it is fixed now.

python 3.8 is not supported for numpy>=1.25, with this changes pyfar is still compatible to numpy 1.24..

it was tests for numpy 2.0 nad numpy < 2.0 for all supported python version in pyfar

installation of pyfar will not work with numpy == 2.0 for now, since the import of the VisibleDeprecationWarning will fail

Maybe squash the commits before we merge?

@mberz, @ahms5 maybe release this as is as a patch and discuss, if we want to restrict to numpy>=1.25 and python>=3.9 during the next weekly?

Yes, that would be my suggestion as well. I'd prefer to not make the version jump in the minimum required numpy and hence Python version within a patch release.
@ahms5 So it's only the missing space 🤣


[Bug Report] Pow gradient calculation method is different with pytorch

Describe the bug
in Pytorch version of pow gradient computation, there are if statement for case exponent is equal to zero.
https://github.com/pytorch/pytorch/blob/main/torch/csrc/autograd/FunctionsManual.cpp#L466
However, the if statements does not exist in the TT version of pow gradient calculation.
https://github.com/tenstorrent-metal/tt-metal/blob/main/tt_eager/tt_dnn/op_library/backward/backward_ops.cpp#L46
so if exponent is zero, pytorch produce zero and TT produce nan as gradient.
Expected behavior
A clear and concise description of what you expected to happen.
I expect same gradient with pytorch to avoid unncessary work finding out what's causing the nan.
Screenshots
If applicable, add screenshots to help explain your problem.
Please complete the following environment information:
Additional context
Add any other context about the problem here.

@chekangliang Here is a bug of current implementation of ELEMWISE_UNARY_POW.
This op is used in GPT-2.

@muthutt

@hschoi4448 - we already have a test case in the test suite covering exponent = 0 for the case of both backward and forward ops; I see even the case of pow(tensor,exponent=0) we have correct results.
Can you please show me a test case where this fails for you ?

Sorry, I didn't actually create a test before making the issus; I simply examined the difference in the if statement within the pow backward code and then created the issue.
Now, I'have executed the test_backward_unary_pow.py on the master branch and confirmed that the results are correct.
While running the test, I noticed two strange things. Should I create separate issues for them?
1. power(0,-1) seems not ok.
Also I created a test for zero exponent because you mentiond, we already have a test case in the test suite covering exponent = 0, ... and have correct results. but I think there's something wrong in pow_bw.
I comment out the exponent check and ran the following tests.
@pytest.mark.parametrize(
    "input_shapes",
    (
        (torch.Size([1, 1, 32, 32])),
    ),
)
@pytest.mark.parametrize(
    "exponent",
    [
        0.0
    ],
)
def test_bw_unary_pow(input_shapes, exponent, device):
    def my_data_gen_pt_tt(input_shapes, device, required_grad=False):
        pt_tensor = torch.zeros(input_shapes, requires_grad=required_grad).bfloat16()
        tt_tensor = (
            tt_lib.tensor.Tensor(pt_tensor, tt_lib.tensor.DataType.BFLOAT16).to(tt_lib.tensor.Layout.TILE).to(device)
        )
        return pt_tensor, tt_tensor
    in_data, input_tensor = my_data_gen_pt_tt(input_shapes, device, True)
    grad_data, grad_tensor = my_data_gen_pt_tt(input_shapes, device)

    tt_output_tensor_on_device = tt_lib.tensor.unary_pow_bw(grad_tensor, input_tensor, exponent=exponent)

    in_data.retain_grad()

    pyt_y = torch.pow(in_data, exponent)

    pyt_y.backward(gradient=grad_data)

    golden_tensor = [in_data.grad]

    status = compare_results(tt_output_tensor_on_device, golden_tensor)
    assert status

This test passed, but if the operation were correct, it should have failed.
0^-1 is inf in pytorch.
So if power(input, exComponent - 1, output_mem_config); will correctly return 'inf'.
The backward output must be 'nan' if there is no exponent check.
it means the value of pow(0, -1) is incorrect.
2. not integer exponent
When I use a not integer number as the exponent, the test fails. Is this a bug, or are you planning to create unary_power_fp_bw separately?
@pytest.mark.parametrize(
    "input_shapes",
    (
        (torch.Size([1, 1, 32, 32])),
    ),
)
@pytest.mark.parametrize(
    "exponent",
    [
        0.5
    ],
)
def test_bw_unary_pow(input_shapes, exponent, device):
    in_data, input_tensor = data_gen_pt_tt(input_shapes, device, True)
    grad_data, grad_tensor = data_gen_pt_tt(input_shapes, device)

    tt_output_tensor_on_device = tt_lib.tensor.unary_pow_bw(grad_tensor, input_tensor, exponent=exponent)

    in_data.retain_grad()

    pyt_y = torch.pow(in_data, exponent)

    pyt_y.backward(gradient=grad_data)

    golden_tensor = [in_data.grad]

    status = compare_results(tt_output_tensor_on_device, golden_tensor)
    assert status


Adds EPSG:28992 Tiling Scheme.

This adds an EPSG:28992. It is kind of broken in the sense that it does not create a valid metadata.json. Furthermore, I would propose a full implementation of a custom tiling scheme instead of this only-EPSG:28992 implementation.
So this is a trial for a fuller implementation.

I'm sorry, I did not intend pullrequest in the master repo (only in my forked repo). I'll remove it.


_run_until_done errors when asyncio.current_task() is None

I'm experimenting with the async S3 filesystem and running into a problem when trying to run synchronous commands with asynchronous=True. Specifically, I'm trying to run _pipe_file asynchronously and pipe_file synchronously with the same filesystem instance.
Awaiting the asynchronous _pipe_file works as expected. When calling the synchronous pipe_file, the _run_until_done function is raising the following error on asyncio.tasks._unregister_task.
The error occurred running off the master branch in a Jupyter notebook (hopefully it isn't Jupyter's event loop messing things up):
import asyncio
import fsspec

fs = fsspec.filesystem("s3", asynchronous=True, loop=asyncio.get_running_loop())
await fs._connect()

await fs._pipe_file("path", b"data") # works as expected

fs.pipe_file("path", b"data") # throws the following error

~/oss/filesystem_spec/fsspec/asyn.py in _run_until_done(coro)
     19     loop = asyncio.get_running_loop()
     20     task = asyncio.current_task()
---> 21     asyncio.tasks._unregister_task(task)
     22     current_task = asyncio.tasks._current_tasks.get(loop)
     23     assert task == current_task

~/miniconda3/envs/fsspec/lib/python3.7/_weakrefset.py in discard(self, item)
    112         if self._pending_removals:
    113             self._commit_removals()
--> 114         self.data.discard(ref(item))
    115
    116     def update(self, other):

TypeError: cannot create weak reference to 'NoneType' object

This appears to be due to the fact that asyncio.current_task() is returning None, which it fails to unregister. I patched fsspec.asyn._run_until_done with the following code in order to get the pipe_file call to run synchronously without errors (I only added the two if task is not None checks):
def new_run_until_done(coro):
    loop = asyncio.get_running_loop()
    task = asyncio.current_task()

    if task is not None:
        asyncio.tasks._unregister_task(task)

    current_task = asyncio.tasks._current_tasks.get(loop)
    assert task == current_task

    if task is not None:
        del asyncio.tasks._current_tasks[loop]

    runner = loop.create_task(coro)
    while not runner.done():
        loop._run_once()

    asyncio.tasks._current_tasks[loop] = task
    return runner.result()

fsspec.asyn._run_until_done = new_run_until_done

I'm not familiar enough with asyncio to say whether or not this is a good or complete solution, so I wanted to discuss it before a PR.

I think the expectation is, that when you call an instance marked as asynchronous=True, you do so only from within an async context. The following should work
async def runme():
    fs = fsspec.filesystem("s3", asynchronous=True, loop=asyncio.get_running_loop())
    await fs._connect()
    await fs._pipe_file("path", b"data") # works as expected
    fs.pipe_file("path", b"data")


JsonReaderException when json contains empty array

Hi,
We have a problem with your package. Our json looks like that:

{
"status": 1,
"errorCode": [],
"message": "aircraft.listComplete",
"fullErrorMessage": [],
"scriptDateTime": {
"date": "2016-08-19 14:39:29.000000",
"timezone_type": 3,
"timezone": "UTC"
},
"inGameDateTime": {
"date": "2001-04-22 15:56:54.000000",
"timezone_type": 3,
"timezone": "UTC"
},
"irlDateTime": {
"date": "2016-08-19 14:39:29.000000",
"timezone_type": 3,
"timezone": "UTC"
},
}

When parsing, the code throw an exception:

JsonReaderException: Unexpected character encountered while parsing value: [. Path 'errorCode', line 1, position 25.
Newtonsoft.Json.JsonTextReader.ReadNumberValue (ReadType readType) (at C:/Project/Github/Json.Net.Unity3D/src/Newtonsoft.Json/JsonTextReader.cs:931)
Newtonsoft.Json.JsonTextReader.ReadAsInt32 () (at C:/Project/Github/Json.Net.Unity3D/src/Newtonsoft.Json/JsonTextReader.cs:424)
Newtonsoft.Json.Serialization.JsonSerializerInternalReader.ReadForType (Newtonsoft.Json.JsonReader reader, Newtonsoft.Json.Serialization.JsonContract contract, Boolean hasConverter) (at C:/Project/Github/Json.Net.Unity3D/src/Newtonsoft.Json/Serialization/JsonSerializerInternalReader.cs:2238)
Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateObject (System.Object newObject, Newtonsoft.Json.JsonReader reader, Newtonsoft.Json.Serialization.JsonObjectContract contract, Newtonsoft.Json.Serialization.JsonProperty member, System.String id) (at C:/Project/Github/Json.Net.Unity3D/src/Newtonsoft.Json/Serialization/JsonSerializerInternalReader.cs:2389)

How can I solve it?

an empty array is not the problem :)
NOTE: no exception catcher ;)
start.cs
using UnityEngine;
using System.IO;
using Newtonsoft.Json;

public class start : MonoBehaviour {

	// Use this for initialization
	void Start () {
        Data data = JsonConvert.DeserializeObject<Data>(File.ReadAllText("./data.json").ToString());
        UnityEngine.Debug.Log(data.status);
        UnityEngine.Debug.Log(data.errorCode);
        UnityEngine.Debug.Log(data.message);
        UnityEngine.Debug.Log(data.fullErrorCode);
        UnityEngine.Debug.Log(data.date);
    }
}

Data.cs
using Newtonsoft.Json;
using System.Collections.Generic;

public class Data {
    [JsonProperty(PropertyName = "status", Required = Required.Always)]
    public int status;
    [JsonProperty(PropertyName = "errorCode", Required = Required.Always)]
    public List<int> errorCode;
    [JsonProperty(PropertyName = "message", Required = Required.Always)]
    public string message;
    [JsonProperty(PropertyName = "fullErrorCode", Required = Required.Always)]
    public int[] fullErrorCode;
    [JsonProperty(PropertyName = "date", Required = Required.Always)]
    public string date;
}

data.json
{
    "status": 1,
    "errorCode": [],
    "message": "aircraft.listComplete",
    "fullErrorCode": [],
    "date": "2016-08-19 14:39:29.000000"
}

Due to lack of information i cant tell you where the problem is.

I'm not sure there is a problem or not because your start.cs works well under UnityEditor writing following output.
1
System.Collections.Generic.List`1[System.Int32]
aircraft.listComplete
System.Int32[]
2016-08-19 14:39:29.000000

If there is an error around this code, please let me know your environment :smile:

I should write more clear ;D
With my code there shouldnt be an error. It was just a simple code, to show how he can parse his json. I think he has problems in his code, but didnt show them.

@IceflowRE Oh I missed it! Good explanation :+1:
@Josk From callstack, I suspect that a type of the field errorCode of your class may be int or something scalar not array and it causes a parse error because scalar field cannot consume array at all.


1061 stage 1

Get rid of as many casts as possible

Cool! Now that the times are much more manageable I'm going to remove all of the timing code I had written for this.


Unused Variables/Strict Au3Check Settings

The variable $sErr is unused in the latest release (v<IP_ADDRESS>) in _WD_Window, _WD_FindElement, and _WD_ElementAction. When running with the strictest Au3Check parameters -d -w 1 -w 2 -w 3 -w 4 -w 5 -w 6 -w 7 -v3, a warning is generated

Thanks @seadoggie01 for the feedback!


Implement class BaseMrg31 (c++20)

Take benefit of c++11 implementation.

Completed; Compiles; Still to be tested.


I forgot to run the returned result through JSON.parse. I keep forgetting the library doesn't do that automatically.


Add Andrew Titmuss to the Contributors list

@iandrewt, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?

Hi @Roshanjossey, sorry for the late reply. The tutorial was very straightforward and easy to follow. I wasn't surprised it was merged, but there was an element of patience that I think other contributors should be aware of.
I wouldn't call myself a seasoned developer, but I've made (small) pull requests on other projects before, including maintaining a Homebrew package. If there's anything you need help with for this project I'd be happy to help and be a proper contributor to this project.

@iandrewt, I’d urge you to continue contributing to other open source projects.
We also have a slack team to help/mentor people trying to contribute to open source projects.
You can join the team by going to https://firstcontributions.herokuapp.com


What's the recommended way to add plugins?

I thought about forking this repo, but that'd mean that my version will get outdated over time.
Is there any better solution? This surely can't be it.

@abiosoft Which one is the least painful way you're aware of?


set_icon_name exists, but is not accessible

I have included all traits, using
use gio::prelude::*;
use gtk::prelude::*;

but code below is not working:
let button = gtk::Button::new();
button.set_icon_name("mail-reply-sender");

I have also tried:
use gtk::*;
use gtk::ToolButtonExt;
use ToolButtonExt;

Error: ** the method set_icon_name exists but the following trait bounds were not satisfied:
gtk::Button : gtk::GtkWindowExt
gtk::Button : gtk::ToolButtonExt
**
my cargo config:
[dependencies.gtk]
version = "0.7.0"
# features = ["v3_16"]

[dependencies.gio]
version = ""
features = ["v2_44"]

Why do you think there should be a set_icon_name() on generic buttons? It exists in GTK4 but not in GTK3. In GTK3 it only exists on gtk::ToolButton and gtk::Window as you discovered, and gtk::DragSet / gtk::DragSource (which are not included in the bindings currently).
GTK4 bindings are currently being developed and will be merged into the gtk-rs organization over the next weeks.
@GuillaumeGomez I think this one can be closed?


[5.0] Making TestCase::createApplication() a static method, so that it could be called from setUpBeforeClass() static method

As I stated in #8459 (and got binned), PHPUnit's setUpBeforeClass() static method becomes useless without Facades and other resources of the app. I dug into the Laravel codebase and discovered that actually there is a TestCase::createApplication() method which instantiates the app for unit-testing purposes. Problem is, this method can't be called statically as it is not static itself. So making it static will make things easier, as being static method, it can be called within both static and non-static contexts.
As I make this PR, I am not sure if it will break inner workings of framework as a whole (I believe it shouldn't). So we will see how tests go.
I also would like to urge our framework maintainers to merge this one into [5.0] rather than waiting for [5.1] release. It's very important feature I require in my open-source project guys! I could use the content of that method in my setUpBeforeClass() implementation of course, but I really don't want to dupe code as this facility already exists. Thank you in advance for understanding!

Additionally, I explored the avenue of setting bootstrap attribute in phpunit node of phpunit.xml file such as:
 ... bootstrap="bootstrap/app.php"
but unfortunately it didn't work. I didn't actually do any in-depth analysis of that situation.


chore: bump zio-prelude from 1.0.0-RC31 to 1.0.0-RC34

About this PR
📦 Updates dev.zio:zio-prelude from 1.0.0-RC31 to 1.0.0-RC34
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-prelude" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio", artifactId = "zio-prelude" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #486.


added background color in H1 to be more readable.

when I visited the h1 heading was not readable. so I added some background color. @BatoolMM  please check this .

so here I gave a black transparent look to be more readable. The join us button was near to heading. i have some space. I think now It's look much better. @BatoolMM  can you please check this out? what do you think? thanks

Thank you @ydv-sahitya for your work with this PR. It looks like a good idea but the colour doesn't seem to be matching with the theme. I'd like only to keep this background in the text at the very beginning where it says "We promote ...". Would that be possible?

Thank you @ydv-sahitya for your work with this PR. It looks like a good idea but the colour doesn't seem to be matching with the theme. I'd like only to keep this background in the text at the very beginning where it says "We promote ...". Would that be possible?

yes, it is possible. I will change that. thanks

hey, @BatoolMM  quick question:   Open Science Community Saudi Arabia, this text is not very readable. would you like to change anything? if you have any ideas, I can work on them. thanks.


The --insecure-skip-tls-verify flag does not work with --cert-file and --key-file flags

The --insecure-skip-tls-verify flag of helm repo add command supposed to disable TLS verification. However, the flag does not work if --cert-file and --key-file flags are specified.
The --cert-file and --key-file are used to verify HTTPS client. So it makes sense to use these flags together with --insecure-skip-tls-verify if server requires client verification but client does not need to verify server.
To reproduce

Configure Helm repo using HTTPS server which requires client verification and uses self-signed certificate
Try to add repo using the following command:

helm repo add --insecure-skip-tls-verify \
  --cert-file <path> \
  --key-file <path> \
  testrepo <https://url>

Command fails with the following error:
x509: certificate signed by unknown authority

Output of helm version:
version.BuildInfo{Version:"v3.2.0", GitCommit:"e11b7ce3b12db2941e90399e874513fbd24bcb71", GitTreeState:"clean", GoVersion:"go1.13.10"}

I believe @technosophos identified this issue earlier as well. Your assumption is correct here. Do you feel like working on a patch to fix this for the next release?

For reference, here is where it's set up. https://github.com/helm/helm/blob/c0edc586c55d57bc4a288a17c628ca943e1d5ef1/pkg/getter/httpgetter.go#L116-L124

Thank you for response @bacongobbler! Sure, I can work on fix this weekend.
What do you think about introducing the --insecure-skip-tls-verify option to helm fetch/helm pull command ? It has the same set of flags as helm repo add except --insecure-skip-tls-verify.

I see no problem with that.

@alexmt Working on it, just couldn't find time. https://github.com/helm/helm/issues/7875

sorry, as usual, something got me distracted. still planning to work on it.

The bug is fixed by https://github.com/helm/helm/commit/93b0eee0dff2849081f16b47f4693fd8b25115bc#diff-fde6e3b05a1f149234e259f7821472c5 .


Fix up the .podspec to always bundle the resources

This is to address issue #111.
The code already looks up for resources in Frames.bundle, so tweaking the .podspec to generate the resource bundle fixes the issue with resource clashes when the pod is linked statically.

I've added unit tests and opened a new PR to merge in these changes. Please see #113


Did not call didSelectRowAtIndexPath delegate

Hi Yannick,
Did not call didSelectRowAtIndexPath delegate in MenuController class.
I used:
tapGesture.cancelsTouchesInView= false
But I did not get success. What i do?
Please help me sir.
Khushi.

TapGesture prevent didSelectRowAtIndexPath  .  I do it as below:
100th line In FlowingMenuTransitionManager
addTapGesture(otherView) change to addTapGesture(ov)
Hope it help you!

Would be great to change that in the master branch :D please do. thanks!

It should be fixed now!


Native module cannot be null

RN Version: 0.60.5
This package version: 1.2.4
I've following manual linking since it won't work with auto linking.
but this problem still came out.

Is there anyway to fix this?

Additional errors in compiling.

@roynes For RN60 + XCode 10, please use 2.0.0-beta.1. Thanks!

@whitedogg13, tried it to build via xcode and it failed.

@roynes Sorry for the late response. I have just test in on RN60.5, and it's OK.
Here's my step for your reference:
react-native init NfcTestAppRn60 --version<EMAIL_ADDRESS>cd NfcTestAppRn60
npm install --save<EMAIL_ADDRESS>cd ios
pod install

Then open the xcworkspace file (not previous xcodeproj) in ios directory and it should work.

Hi, for current dev, please follow this version guideline:

XCode 11 beta + RN60: all 2.0.0-beta.X version should be fine
XCode 10 + RN60: 2.0.0-beta.1
XCode 10 + RN59 and below: 1.2.2

Thanks!

@roynes Sorry for the late response. I have just test in on RN60.5, and it's OK.
Here's my step for your reference:
react-native init NfcTestAppRn60 --version<EMAIL_ADDRESS>cd NfcTestAppRn60
npm install --save<EMAIL_ADDRESS>cd ios
pod install

Then open the xcworkspace file (not previous xcodeproj) in ios directory and it should work.

Hi @whitedogg13 , by the word open xcworkspace does it pertain to open via xcode?

@whitedogg13,
I got it to work on the simulator now iPhone X
but what I got is this.
is NFC Supported? false

I am having this problem in the following configurations:

Xcode 11.3 + RN 59.9 + 1.2.2
Xcode 11.3 + RN 59.9 + 2.0.0-beta.1

Any idea what to do?

A quick update:
I was able to fix the issue by adding iOS 14.3 device support to my XCode since my iPhone is running 14.3.

A quick update:
I was able to fix the issue by adding iOS 14.3 device support to my XCode since my iPhone is running 14.3.

a quick question, can you write on an empty nfc tag after adding it?

A quick update:
I was able to fix the issue by adding iOS 14.3 device support to my XCode since my iPhone is running 14.3.

a quick question, can you write on an empty nfc tag after adding it?

A quick update:
I was able to fix the issue by adding iOS 14.3 device support to my XCode since my iPhone is running 14.3.

Hi @avivku90,
I haven't tried that, I'm mostly reading from tags. Maybe @whitedogg13 can help?

Hi @avivku90,
I haven't tried that, I'm mostly reading from tags. Maybe @whitedogg13 can help?

@avivku90 it depends.
If you're talking about writing NDEF, then you will need to first format the tag so you can write NDEF message into it.
If you're using low-level NFC API, you should still be able to directly write into the tag's memory.

@avivku90 it depends.
If you're talking about writing NDEF, then you will need to first format the tag so you can write NDEF message into it.
If you're using low-level NFC API, you should still be able to directly write into the tag's memory.

Guys, I will close it for now, please feel free to reopen or drop a new one if issue persists.

Guys, I will close it for now, please feel free to reopen or drop a new one if issue persists.

Thank you for your help, Richie!

Thank you for your help, Richie!


fixed spelling

Thanks @gopalindians, but @cptbrkfst beat you to it by a few hours! This changed was merged in here: https://github.com/no-stack-dub-sack/cs-playground-react/pull/2.


Possible Bug: Announce component - NVDA in Firefox reads such text twice or more

Possible bug
Need to investigate and determine whether this is a bug or not.  We received the following from an engineer:

Hi guys! There is a problem when I'm trying to use Announce component with the text with two dynamic values. NVDA in Firefox reads such text twice or more times, as you can see on the screenshot.
I think that the problem with NVDA. NVDA+Chrome and JAWS+Firefox work as expected.
Any thoughts about it?
BTW, if I remove aria-atomic="true" I'll see only some separate symbols, not the whole line repeated.

I can't replicate this issue in NVDA with Firefox. I have a feeling this is an issue with implementation; perhaps this is getting read multiple times because of the way the component is re-rendering multiple times thus triggering it repeatedly.
Do we know who reported this so we can follow up?

This issue was reported by someone no longer active on Slack. Closing for now, and we can reopen in the future if it becomes an issue again.


OpenJDK8 uses EOF version of Alpine Linux

The actual Dockerfile uses openjdk8 as base image and that uses an old version of Alpine Linux (3.6) that is not supported since 2019-05-01
LTS versions should use a stable version of every tool or library. When the tool no-longer receives no official support is prone to produce security issues and should be updated.
The PR #956 could solve this issue since it uses adoptopenjdk images that are more up to date

As it was agreed with the Platform SIG team a few months ago, we will be moving the image to AdoptOpenJDK base image, as well as all other images in short term. Taking the recent public exposure of the old image, I will deliver an expedited fix for this issue specifically.

Pending pull requests:

Controller (FKA master) image: https://github.com/jenkinsci/docker/pull/975
Agent images: https://github.com/jenkinsci/docker-ssh-agent/pull/59, https://github.com/jenkinsci/docker-agent/pull/150, https://github.com/jenkinsci/docker-inbound-agent/pull/190

Once agent images are released, we will also need to respin the https://github.com/jenkinsci/jnlp-agents builds

According to the feedback from @daniel-beck @aheritier @timja , we do not want to do packaging alignment between the old OpenJDK image and the new AdoptOpenJDK one. We do not see any package which would be a concern for the common Jenkins use-cases, and alpine images are supposed to be small. Also, a we jump over multiple Alpine Linux versions with its own breaking changes. Our intend is to release images with explicit notice about potentially breaking changes,
I will re-confirm it at the Platform SIG meeting in 15 min and go ahead with merges if there is no negative feedback

Status update for Aug 13, 2020:

Agent images have been updated:

https://github.com/jenkinsci/docker-agent/releases/tag/4.3-8
https://github.com/jenkinsci/docker-inbound-agent/releases/tag/4.3-9
https://github.com/jenkinsci/docker-ssh-agent/releases/tag/3.0.0

Jenkins Controller images (FKA "master") got a fix in https://github.com/jenkinsci/docker/pull/975 . This fixed will be shipped in the next weekly and LTS releases
https://github.com/jenkinsci/jnlp-agents still needs an update for some images. In my TODO list for tomorrow

Can we close this out?

Yup


[FEATURE] add ability to upload CV for developer profile

developers should be able to upload CV after signing up

update front-end code HTML/JS uploading CV (edit-developer-profile.html)
new bytea field representing the CV file (schema and repo)
update Go endpoint to process and validate the new field
add ability to update CV from developer profile or upload new one if none present (add "CV not uploaded" if not present)
add ability to view/download CV (logged users only - recruiters/admin users only)
track when CV has been downloaded (create new table (id, developer_profile_id, cv (bytea), user_id, downloaded_at)
local testing

/bounty $50

@dm161  Can pick this up, if no one else takes it and after my other PR's are merged.

In submit-developer-profile just like how profile image is getting uploaded before developer profile is submitted, similarly we have to upload resume, correct ?
Create a new table to save developer resume, and a new table when someone downloads a developer resume ?
Can you elaborate on this ?


Send network parameter with current network type

This is only supported on Chrome-like browsers at the moment.

by setting it via connect parameters we lose the ability to react to network changes.
Could imagine it would be useful to be able to have visibility of the changes while connected as well?


interpolate throws a DocumentArray error

I tried to run
/interpolate and filled out the two prompts
and got
Now beginning work on "interpolate "cat sitting on a sofa" to dog sitting on a sofa" for murmur5786. Please be patient until I finish that.
Got unknown error on interpolate "cat sitting on a sofa" to dog sitting on a sofa: <DocumentArray (length=0) at<PHONE_NUMBER>06336> is empty

Did you update to the latest dalle-jina? It works here locally.

Thank you for coming back. I had a pretty fresh install (3 days ago) but will try to re-install in that case and update here

The other problem might be that you're using the low VRAM version and I just haven't coded the interpolate function for that yet. I hope to be able to later this week.

Yep, Im using the low VRAM version and getting the same error

OK, worry not, this will be implemented for that branch soon.

The optimized-sd branch has been updated. To update, you'll have to destroy the old branch as there was a rebase and force push. Follow these commands:
cd dalle/dalle-flow
git checkout main
git fetch --all --prune
git branch -D optimized-sd
git checkout optimized-sd

This should make the interpolate command available to you. Please let me know how it goes.

@AceBeaker2 Just let me know if everything is fixed on your end and I will close the issue.

Haven't heard anything back, presuming it's been fixed.


Update Spring version to 2.4.0

What this PR does / why do we need it:
Update Spring version to 2.4.0
Which issue this PR fixes

fixes https://github.com/hazelcast/hazelcast-jet-contrib/issues/99

Notes to your reviewer:
Checklist

[x] Signed the Hazelcast CLA
[x] No checkstyle issues (./gradlew check)
[x] No tests failures (./gradlew test)
[ ] Documentation which complies with README template (see templates/README.template.md).
[ ] Update List of modules section on the root README with your module.

The job jet-contrib-pr-builder of your PR failed (log, artifacts).
Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log file

The job jet-contrib-pr-builder of your PR failed.
Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log file

The job jet-contrib-pr-builder of your PR failed.
Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log file


Card Enhancements

Context

Card component has some issues:

No refs forwarded
Classnames are overwritten rather than merged
Style is overwritten rather than merged

ticket code: HOV-51


Reduce memory allocation in HttpMessageSanitizer

This is a performance optimization in HttpMessageSanitizer to reduce the memory allocated in common cases.
There are three optimization techniques:

Use url.AsSpan(...) instead of url.Substring(...) to create the query part of the URL.
Defer creation of the StringBuilder until/unless absolutely necessary (when there's a redacted query value)
Use a cached StringBuilder.

I added benchmarks to show the difference:
Baseline

Method
Mean
Error
StdDev
Gen0
Allocated

SanitizeHeader
3.890 ns
0.3120 ns
0.3468 ns
-
-

SanitizeUrl
8.218 ns
0.7272 ns
0.8375 ns
-
-

SanitizeUrlWithQueryNoValue
110.411 ns
4.3084 ns
4.7888 ns
0.0004
264 B

SanitizeUrlWithAllowedQuery
137.150 ns
7.0172 ns
7.7996 ns
0.0005
360 B

SanitizeUrlWithDisallowedQuery
185.949 ns
7.9072 ns
9.1060 ns
0.0007
464 B

Candidate

Method
Mean
Error
StdDev
Allocated

SanitizeHeader
4.148 ns
0.5924 ns
0.6822 ns
-

SanitizeUrl
7.978 ns
0.9843 ns
1.1336 ns
-

SanitizeUrlWithQueryNoValue
21.698 ns
1.9727 ns
2.2717 ns
-

SanitizeUrlWithAllowedQuery
27.414 ns
2.7250 ns
3.1381 ns
-

SanitizeUrlWithDisallowedQuery
84.632 ns
5.9402 ns
6.8407 ns
104 B

As you can see, we avoid memory allocations (and potential garbage collections) in common cases where there are no query parameters or where all the query parameters are allowed. In the case where we have to redact query values, the bytes allocated is one fourth of the baseline.

API change check
API changes are not detected in this pull request.

//cc: @JoshLove-msft, in case there's any overlap with the test proxy sanitization.

Any further comments? If not, can I get a review and sign-off?

@pharring :  Give a mention when you're comfortable with things and I'll merge on your behalf.

@jsquire #sign-off

//cc: @JoshLove-msft, in case there's any overlap with the test proxy sanitization.

No this shouldn't impact anything with test recordings as logs are not part of the request/responses.


As a followup: please rename the security policy enum values to match the spec in its state after https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/4890 merges.

As a followup: please rename the security policy enum values to match the spec in its state after https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/4890 merges.
Originally posted by @bzbarsky-apple in https://github.com/project-chip/connectedhomeip/pull/14959#discussion_r802888647

Re-opening since still need to update Enum value and change Standard key Policy to CacheAndSync


Page "What New"

Description
A more compact view
Alternative variation
Add in "Lucid Settings"
Additional context

hey ! thanks for the suggestion i will be adding it soon liked how it looks

check it out soory that i didnt reference this issue in the push https://github.com/sanoojes/Spicetify-Lucid/commit/27d03b2ba212b10615651c5ecb16494c46802924
It looks like this now


Installer does not work

Version Number
0.2.2
Describe the bug
The installer will not work at all, the process will run in the background but seems to be doing nothing.
Desktop (please complete the following information):
OS: Windows 10 Pro
OS Version: 1903 or something like that
Additional context
This is an issue filled out on behalf of Lexiconian.

Fixed by 0.3.1-alpha. My guess is that this issue was caused by some missing dependency references in the installer definition.


[GUI] Notifications page is not accessible

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] support request

Do you want to request a feature or report a bug?
bug

What is the current behavior?
Notifications page is not accessible

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
click on the notifications menu item

What is the expected behavior?

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Version: 0.5.1-rc.1
Environment: docker-compose (with influxDB and without persister-history)
Operating system: Ubuntu 20.04

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

The notification has not been migrated to influxDB and probably will not.
In the details of the devices, it is not possible to see influxdb data, only the history for now.

The solution is the coexistence of services.


ISPN-8174 LocalStreamIteratorWithPassivationTest random failures

Disabled test as we don't support concurrent activation with iteration

https://issues.jboss.org/browse/ISPN-8174

Merged.


admin: session administration addition, session creation on sign-up

Adds a session administration page. (addresses #52)

Commits the session when login permit to fix the session creation
on user first signup. (closes #190)

Signed-off-by: Leonardo Rossi<EMAIL_ADDRESS>

@hachreak Can you rebase on top of latest merged changes.
@switowski Will you review?

I'll rebase it now and fix the conflict! :smile:


FYI: Another stackfull coroutine lib

https://github.com/alibaba/PhotonLibOS

Hi, thanks for the link, looks neat :).
Do you think it makes sense to collaborate in some way, should asio-grpc maybe provide a backend for your library? Would that even make sense?

Yes, welcome to integrate our lib into your project.
By the way I saw your performance data, I think the reason c++ grpc-asio slower than go-grpc is that asio is weak in steaming socket mode. See our tests in https://github.com/alibaba/PhotonLibOS#21-tcp , however, much better in ping-ping mode.

Mmh, this library does not use asio's socket classes whatsoever. All I/O is handled by gRPC itself (epoll event engine).
The go-grpc server used to be much slower until it started uses a very fast Protobuf implementation: https://github.com/planetscale/vtprotobuf. So I suspect that is the performance bottleneck in the asio-grpc server.
I had a brief look at your library and I think it should be rather simple to bridge from your mutex+condition_variable to asio or libunifex and thereby into asio-grpc.

Is there any way to replace gRPC's epoll engine and use customized socket?

So you will create two threads, one for grpc event loop, and the other for your own ?

gRPC exposes the event loop through CompletionQueue which I wrap and run on one thread. I suspect that a naïve implementation against your library would create another thread for your library. Maybe there is another way, does your library have the concept of an event loop and allows polling it?

I think that is a problem. Some old codes have their own event loop, and when they decide to use gRPC, they find it unlikely to integrate into gRPC's own loop.
Our library has the polling mechanism, and all the mutex and cv are running within event loop. The mutex and cv can work across mutil-thread, they are equivalant to std::mutex and std::cv, the only difference is that they are designed for coroutines.

epoll engine can be cascading. I suspect you can use our event engine and then poll the one of gRPC.

Cross-thread communication is also feasible. We have a semaphore (based on eventfd), that can do wait / notify.

Yep gRPC having its own event loop was also the challenge with asio-grpc. With the gRPC event_engine it should be possible to pluck something like asio::io_context in natively, which makes things a little cleaner.
Your code is the best we can do at the moment I believe and it doesn't even require asio-grpc :).

But cross-thread communication is not effective in terms of performance...

I am afraid that is the best we can do with gRPC's current API.
Once the event_engine can be customized it will be possible to cq->Next() in the I/O thread without blocking it.

I started playing with the EventEngine a bit https://github.com/Tradias/asio-grpc/blob/event-engine/test/utils/utils/grpc_context_test.cpp but it is still too experimental. gRPC only uses the engine for some experimental features like load balancing and service discovery which makes it very difficult to test the EventEngine. I have put my efforts on hold for now.
Anyways, I hope you understand why asio-grpc provides an entire asio::execution_context-compatible wrapper for gRPC's CompletionQueue. Simple services/clients can then be written without cross-thread communication and everything else can reach out to e.g. an asio::io_context and use asio's completion token system to bounce between the two contexts/threads.

Hi, I will close the issue until gRPC has made significant progress on their EventEngine. Feel free to open another issue if you have any more questions or suggestions for asio-grpc.


Documentation for Json I believe is out of date.

Describe the bug
I copied the block of code you had in the README.txt to set up functions, I pasted it into a swift file where I had "import OpenAI" and many of the constructs that the code relied upon failed to find references in the openai library.
Specifically, the "ChatFunctionDeclartion" and the JSNOSchema do not exist in the library.
let openAI = OpenAI(apiToken: "...")
// Declare functions which GPT-3 might decide to call.
let functions = [
ChatFunctionDeclaration(
name: "get_current_weather",
description: "Get the current weather in a given location",
parameters:
JSONSchema(
type: .object,
properties: [
"location": .init(type: .string, description: "The city and state, e.g. San Francisco, CA"),
"unit": .init(type: .string, enumValues: ["celsius", "fahrenheit"])
],
required: ["location"]
)
)
]
let query = ChatQuery(
model: "gpt-3.5-turbo-0613",  // 0613 is the earliest version with function calls support.
messages: [
Chat(role: .user, content: "What's the weather like in Boston?")
],
functions: functions
)
let result = try await openAI.chats(query: query)

also having this issue

if you take a look at line 152 the change here are drastic and cause breaking changes to anyone that implemented what their docs suggested as a solution for chat completions https://github.com/MacPaw/OpenAI/commit/905e31706d7ae6c9c3a851e6bf40bd36f41a87fb


Logo should not be an H1

@awkawk @aparkening Following a discussion we had with @murtaugh concerning nested headed order, Tim expressed the concern that maybe the logo should not be an H1. I did a bit of research and I think it's a good idea. On the homepage (where there is no clear top-level heading) we could mark it up as an H1, elsewhere just a paragraph. The issue is referenced extensively in the Financial Times repo https://github.com/Financial-Times/n-header-footer/issues/39 with many useful examples of other sites using this approach. What does everyone think?

@michellekondou: +1, I think taking the logo out of H1 on sub pages is great.

Is there an H1 on the sub pages then? Whatever structure people think makes sense if fine by me.

Yes, the H1 on each subpage will be relative to the content of that page. For example, in the article template it will be the article heading. In the about section it will be "About" and so on.


How to map Caps lock to F10 with Karabiner Elements?

The json file looks like:
{
"profiles": [
{
"name": "Default profile",
"selected": true,
"simple_modifications": {
"caps_lock": "F10"
}
}
]
}
Is there a way to map caps lock to F10? The log file says: [2016-09-24 17:54:18.575] [console_user_server] [warning] unknown key_code:F10 in /Users/andyph666/.karabiner.d/configuration/karabiner.json

I am experiencing the same issue, however even worse-- the following doesn't even say unknown key_code: f9, it simply doesn't work:
{
	"profiles": [
		{
			"name": "Default profile",
			"selected": true,
			"simple_modifications": {
				"caps_lock": "f9"
			}
		}
	]
}

Update: I've tried remapping caps lock to f18 and that seemed to work. I'm not sure if it's just an issue with only remapping for function keys that are less than 13, but it'd be cool fi those worked too

I need to map caps lock to the alt+space combination. However I can't even get it to map into space:
{
    "profiles": [
        {
            "name": "Default profile",
            "selected": true,
            "simple_modifications": {
                "caps_lock": "space"
            }
        }
    ]
}

any suggestions?

@kyleholzinger Thanks for the suggestion. I updated karabiner-elements and also changed my super key in tmux to the F13 key. That means you have to put set-option -g prefix S-F1 in your .tmux.conf because S-F1 is mapped to F13 and S-F2 is F14 and so forth. Then in the karabiner.json file, putting in "caps_lock": "f13" should make the caps lock as the super key for tmux.

Gotya, yeah I don't use tmux but nice find! It's interesting that it only works for function keys above 12. And @eyaleizenberg I don't think you can currently map to multiple keys (i.e. key combinations)

Yeah I can't get this to work. I've been trying to remap F8 with the same code above. I know previously we needed to go into the modifier keys section of System Preferences and change the caps lock to "no action," which I did, but after trying this code, the caps lock functions normally even with it set to "no action." Odd. I'll just keep waiting for the next update. : )
As for key combinations, I previously used this in conjunction with a custom Alfred workflow which remapped F8 to a keyboard combo like CTRL+Spacebar, which is what I'm trying to accomplish now.
~Paik

@Paik4Life as stated before, I'm pretty sure 1. you can't do multiple key combinations at the moment, and 2. you can't map to the default function keys-- only those above f12

Yeah, I know, that's why I use Alfred workflow to pass multiple key combos. e.g., Karabiner remaps Caps to F8, then Alfred takes all F8 keystrokes and maps them to CTRL+Spacebar.
I definitely missed the part where you said you can't use default function keys though. Unfortunately, as far as I know, Alfred won't let you map non-default function keys. I'll have to look into that.
Thanks!
~Paik

Can you click a button in alfred? If you map to f17 for example, whenever you click it it's a totally valid key code. Alternatively, you can use quicksilver to do the same thing with an applescript. In either case, you should be able to click "f17" (aka caps lock) when alfred asks for a key code-- I'd be super surprised if it didn't accept that, cause that's just based on what keyboard you use.

@kyleholzinger I thought it would too. So I remapped caps to F18 as per your example. Then I went ahead and tried to set it as a hotkey but it doesn't recognize it. Other keystrokes and keystroke combos work fine. I tried this both with "no action" assigned to caps and with the default setting. No joy : (
I'm not a developer so I could have done something wrong, but this is pretty simple code so I don't think I did anything wrong. I even tried copying/modifying your code above and that didn't work either. Clearly, I'm not meant to have my caps lock search key back.

@Paik4Life Haha. Are you using it to remap to spotlight? Spotlight is pretty much the same as alfred honestly, I'd highly recommend it. In any case, if you're alright switching to spotlight, you can 100% set caps lock binded to f18 to open it

Also another thing @Paik4Life in the logs section, does it say something like keycode:F18 not recognized or something?

Actually, @kyleholzinger, I'm getting a parse errors

Not sure why. This is what my .json file looks like:

*Note: the more and more I go down this path, the more and more I have a deepening fear that you're going to point out something immensely stupid that I'm doing wrong.....That moment will come to fruition in 3.......2.........1.........

@Paik4Life lmao, I'm almost positive it's that damn smart quote after "f18. See how it's weird and slanty? That means that it's a macOS native smart quote, which isn't the same as a standard ". Try re-typing that in a simple text editor like vim or nano or something. If that doesn't work, here's a quote you can copy and paste :P "

@Paik4Life check out flashlight, might have what you want if you can't end up getting alfred to work

I think you are using the wrote quotations. Usually happens to be when I copy and paste something . I'd try to replace all the quotes in the json file by doing what kyleholzinger suggested.

@kyleholzinger and @andyph666, it was that damn quote. I didn't even notice since I did a direct copy and paste from Kyle's post above, so I'm not sure why it changed.
In any case, it all works now!  Sometimes that happens if you paste into textedit or something

Can we all just hug it out?

Are you guys experiencing problems with selecting folders and files in finder? Using shift + click or command + click does not seem to work when I click multiple icons in list view. When I shut off Karabiner, it works fine. After running karabiner again, everything works as it should, including karabiner.

@andyph666 , I don't have that issue, but I did notice that I can't wake my computer from my bluetooth keyboard after the latest update (.44, I think). Not a huge deal since I can wake with mouse, just a noticeable difference in my workflow.
Also, hugs all around!


TypeError: Property 'allLanguages' does not exist on type 'InitOptions'.

Describe the bug
NextI18Next class interface in types.d.ts is missing options: Config in type definitions.
The instance of NextI18Next does not have the correct types for options hence it throws TypeError: Property 'allLanguages' does not exist on type 'InitOptions'. TypeScript compilation error.
Occurs in next-i18next version
4.2.0
Steps to reproduce
The following implementation in a TS file will reproduce the exception:
const NextI18NextInstance = new NextI18Next({
  browserLanguageDetection: false,
  defaultLanguage: 'nl',
  localeSubpaths: {
    en: 'en',
  },
  otherLanguages: ['en'],
  serverLanguageDetection: false,
  shallowRender: true,
});

export const {
  appWithTranslation,
  withTranslation,
  i18n,
  Link,
} = NextI18NextInstance;

const { options } = i18n;
const myNewArray = [...options.allLanguages]; // Error:(47, 39) TS2339: Property 'allLanguages' does not exist on type 'InitOptions'.

Expected behaviour
I'd expect options having the types of Config for the class instance as well. This can be fixed by extending the i18n property's type as:
declare class NextI18Next {
  constructor(config: InitConfig);
  Trans: Trans
  Link: Link
  Router: Router
  //  The line below fixes the issue
  i18n: I18n & {
    options: Config;
  }
  initPromise: InitPromise
  config: Config
  useTranslation: UseTranslation
  withTranslation: WithTranslationHocType
  appWithTranslation: AppWithTranslation
}

Screenshots
N/A
OS (please complete the following information)

Device: MBP 2019 15"
TypeScript version: 3.8.3

Additional context
Tsconfig:
{
  "compilerOptions": {
    "target": "es5",
    "lib": [
      "dom",
      "dom.iterable",
      "esnext"
    ],
    "allowJs": true,
    "skipLibCheck": true,
    "strict": false,
    "forceConsistentCasingInFileNames": true,
    "noEmit": true,
    "esModuleInterop": true,
    "module": "esnext",
    "moduleResolution": "node",
    "resolveJsonModule": true,
    "isolatedModules": true,
    "jsx": "preserve"
  },
  "exclude": [
    "node_modules"
  ],
  "include": [
    "next-env.d.ts",
    "**/*.ts",
    "**/*.tsx"
  ]
}

First of all, thanks for the great library! If you think this is a valid case, I'd be happy to create a PR with the fix.

Hm, it's a bit of a grey area. The allLanguages array is something used internally and not really intended for user consumption. What's your use case? Why not manage the array in userland?


TRX logger for vstest.console not present

I thought this was included by default with VS. The other two default loggers (console and TFS) are both available.
There is no way to install the logger either as it requires write access to program files.
Can this be installed by default?
The directory containing these is:

D:\Program Files (x86)\Microsoft Visual Studio 11.0\Common7\IDE\CommonExtensions\Microsoft\TestWindow

If you look under D:\Program Files (x86)\Microsoft Visual Studio 11.0\Common7\IDE\CommonExtensions\Microsoft\TestWindow\Extensions (e.g. via Kudu console), we do have Microsoft.VisualStudio.TestPlatform.Extensions.TrxLogger.dll in there. Is something else missing?

Interesting, vstest isnt picking it up,

D:\Program Files (x86)\Microsoft Visual Studio 11.0\Common7\IDE\CommonExtensions\Microsoft\TestWindow>vstest.console.exe /listloggers
Microsoft (R) Test Execution Command Line Tool Version 11.0.60315.1
Copyright (c) Microsoft Corporation.  All rights reserved.

Warning: The Test Logger Add-Ins installed through a VSIX installation are ignored. Use the /UseVsixExtensions parameter to include them, if your installation supports vsix extensions. Example: vstest.console.exe /ListLoggers /UseVsixExtensions:true

The following Test Logger Add-Ins are available:
Microsoft.VisualStudio.TestPlatform.Extensions.ConsoleLogger
Uri: logger://Microsoft/TestPlatform/ConsoleLogger/v1
FriendlyName: Console
Microsoft.VisualStudio.TestPlatform.Extensions.TfsLogger.TfsLogger
Uri: logger://Microsoft/TestPlatform/TfsLogger/v1
FriendlyName: TfsPublisher

D:\Program Files (x86)\Microsoft Visual Studio 11.0\Common7\IDE\CommonExtensions\Microsoft\TestWindow>

I know this is old and dead, but I'm hitting this issue on a very important build project.  Any fix or workaround for this?

Seems we never got a repro on this one.

Hi
Kudu will continue to run in Azure App Service. However, this repo will no longer be maintained.
If the problem persists and is related to running on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


bump sqlite-amalgamation to 3.41.2

https://www.sqlite.org/releaselog/3_41_2.html

needs a version bump of the library itself (which includes the sqlite version in its own)


API CALLS: POST 9 and 10

9: Reflection student response
Parameters: reflection, student ID, Course ID, Scenario ID
10: Conversations
Parameters: name of stakeholder, boolean if questions are picked, array of questions, student ID, Course ID, Scenario ID


Syntax error

I had to clear my cache because of another Homebridge device, and now, I’m not getting notifications for motion or doorbell.
the error is: /bin/sh: 1: Syntax error: Unterminated quoted string
I didn’t change a thing, any pointers?

This doesn’t appear to be a plugin-related bug. My guess is there’s something up in your startup scripts, or homebridge configuration.

This doesn’t appear to be a plugin-related bug. My guess is there’s something up in your startup scripts, or homebridge configuration.


Add images to or change the text colour of the glossary term links

At the moment, the glossary term links are difficult to distinguish. By adding an image or changing the text colour, this will help them stand out.

IMHO the best options are:

superscript info icon
dotted underline and greyish text


Add transparent HTTPS support with optional tonic feature tls-roots

Enabling the new feature tls-roots allows connections over HTTPS without any code changes. This only works if the server certificate is issued by one of the standard CAs.

Thank you for your PR. I think we should also support CAs provoided by user.

Thank you for your PR. I think we should also support CAs provoided by user.

I add two features: tls and tls-roots.
If users provide URLs prefixed with https:// or invoke ConnectOptions::with_tls(), then the client connect server using TLS.

I add two features: tls and tls-roots.
If users provide URLs prefixed with https:// or invoke ConnectOptions::with_tls(), then the client connect server using TLS.


Update for Julia 1.8

Addresses #15

I never ended up being able to get around to writing an integration test for this package, would you be able to confirm that you can launch a HelloWorld or similar test here @ianfiske ?

Yes, I have a successfully launched HelloWorld with this changes in the PR.

Cool, I just bumped the patch version to 0.1.3.  Let me know if you wanted to bump the minor instead.


Populate sbom with minimal properties | reproducible builds

Fixes #3314

GCC version
GLIBC version
BootJDK version

@chandrashritii We need to use tr and cut for this since the format is fixed, ie.
sed -n '/^Tools summary:$/,$p' "${inputConfigFile}" | grep "C Compiler:" | tr -s " " | cut -d" " -f5

You may need to alter -f5 to -f6, depends on whether there is a leading space?

@chandrashritii We need to use tr and cut for this since the format is fixed, ie.
sed -n '/^Tools summary:$/,$p' "${inputConfigFile}" | grep "C Compiler:" | tr -s " " | cut -d" " -f5

You may need to alter -f5 to -f6, depends on whether there is a leading space?

Thanks! @andrew-m-leonard -f5 works. I've also added a function call for sourcing the bootjdk version from the build tools summary, using a similar method as the one used for gcc

@chandrashritii Looks good, i've approved the CI build checks to run...

Tests fail addGLIBCversionforLinux can't be found, probably needs declaring higher up / importing


update to 0.9.7

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


Need an admin Panel page

Navbar can be different and should look amazing put in sample data
Make it look different with animations
You can use Tailwind or CSS
If you are using CSS do use classes or ID's do not target them by element
Create a page in the Pages Folder called AdminPanel
go ahead

Is this issue open?
If it is then I would like to work on this issue under GSSoC'23
Can you please assign it to me?

@Mansi1236 I will be assigning you this for now

Actually I thought the issue is of improving the UI, I don't have much experience in making admin dashboards.
Could you please un-assign it?
Apologies for the inconvenience caused

@akashgreninja  can you please assign this task to me. also, can you specify about the features of this page. Thank You!

@Mansi1236 You can work on improving the UI of the Main Home page or the about us Page if you want as well

@akashgreninja  alright I will try my best. Thank You!

@rishuraj1 any updates on this?

@akashgreninja  yes, figuring out some stuffs as my exams are going on till end of this month.  Could you unassign me till then and after my exams are over, if this task still needs to be done, I would definitely like to try. Sorry for the inconvenience.

can i try this @EffDuBois sir

Can you assign this to me under GSSOC'23?


Add files via upload

@YasinGY I am sorry, this domain seems to represent an umbrella institution, namely the whole city, not a specific single university or a college. Please apply for free educational license with an official document which proves that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
Alternatively, you can apply for GitHub Student Developer Pack and then apply for JetBrains free subscription with your GitHub account.
I have to close the request without merging.
Thank you for your understanding!


Overlay does not show

Maybe I am doing something wrong, but if I place my own PNG and TIFF in the media-directory and place the link in the Sheet it does not show. I used a PNG and a TIFF which I exported from QGIS. After import them again they show up any the right place in QGIS.

If I put the same file in Map Warper I get it working but I would like to use directly the PNG or TIFF out of QGIS because it is already geocoded. So the question is if that is possible and yes, how. By the way, thanks for the great work!

As you probably read, we explained how to insert an XYZ link to georeferenced overlay map tiles in these HODV chapters: Leaflet Storymaps with Google Sheets and Georeference with Map Warper.
If you wish to use an alternative method, consider this strategy with QGIS and GitHub or other server to host tiles for your XYZ link:

Georeference your map with QGIS
QGIS Processing > Raster Tools to generate XYZ tiles into tiles folder
Host folder and Leaflet code in repo using GitHub Pages, or choose another server.

See this example
https://github.com/OnTheLine/otl-public-housing#map-georeference-methods


Make notch a preference

Rename ledge to notch (so it can have a height when on top)
Notch location (top or bottom) set in Preferences dialog
Depth of ledge calculated from stud depth and drywall thickness
Panel will no longer need to be passed to class Opening
Notch height is a parameter set in Preferences dialog

Instead, support notch at top of panel (issue #14


Reversed numbering toggle

https://www.figma.com/community/plugin/914250706909410339/Pager?comment=953393743616655326

Added in 1.1.0


bug: unique key prop needed in Typography

Bug report

https://github.com/supabase/ui/blob/master/src/components/Typography/Text.tsx#L61-L74

:tada: This issue has been resolved in version 0.6.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 0.6.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


KAFKA-5182: Reduce session timeout in request quota test

Reduce session timeouts for join requests to trigger throttling in the request quota test.

@ijuma Can you review, please? It is a small change.

@ijuma I had missed the group.initial.rebalance.delay.ms change, which is probably the reason why the failures are occurring in the build. I was thinking of subsequent join requests rather than the first one, which need low timeouts to trigger throttling. I have changed the code to set group.initial.rebalance.delay.ms=0 as well. Thanks.

@ijuma @dguy Thank you for the reviews. Merging to trunk.


Simply RSA key fix

RSA key length set to 2048 to avoid "low security" error messages in some devices such as Raspberry Pi's.
Also included main.py to launch a HiveMind server.
Can be called like: HiveMind-server and accepts the flags --port PORT to define the listening port and --name NAME --access_key ACCESS_KEY --crypto_key CRYPTO_KEY --mail MAIL to define a new user in the users database before running the server.
When defining a new user, only --name and --access_key are required.

if you are adding scripts see the initial ideas in https://github.com/JarbasHiveMind/HiveMind-core/issues/16 https://github.com/JarbasHiveMind/HiveMind-core/issues/17 and https://github.com/JarbasHiveMind/HiveMind-core/issues/18


OpTestLPM: Collect HMC logs at the end of all iterations

The current code collects all specials logs for every iterations causing the test to not cover all iteration for expected kernel messages, so make sure the extra sosreport and hmc logs collected at the end of test

lpmlogs-optest1.txt


dns updates

Description
See if we can auto-provision our hosts in Route 53.
Motivation and Context
We were having some issues with URL filtering when we just used IPv4 addresses.
How Has This Been Tested?
This is a draft PR for now.
Types of changes

New feature (non-breaking change which adds functionality)

Checklist

[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes if appropriate.
[x] All new and existing tests passed.

Here is hosted zone creation for ellingson.io
------------------------------------------------------------------------

An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # module.dns.aws_route53_zone.secops-ctf-zone will be created
  + resource "aws_route53_zone" "secops-ctf-zone" {
      + comment       = "Managed by Terraform"
      + force_destroy = false
      + id            = (known after apply)
      + name          = "ellingson.io"
      + name_servers  = (known after apply)
      + zone_id       = (known after apply)

      + vpc {
          + vpc_id     = "vpc-a5c56bce"
          + vpc_region = (known after apply)
        }
    }

Plan: 1 to add, 0 to change, 0 to destroy.

------------------------------------------------------------------------

This plan was saved to: franklin.out

To perform exactly these actions, run the following command to apply:
    terraform apply "franklin.out"


Updates to run mango-notebook in Docker

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/mango-prb/475/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/mango-prb/476/
Test PASSed.


Timeout error (110) in Flet Android client when trying to connect to app

The Flet client on Android yields a timeout error (110) for the hello world app:
# main.py
import flet as ft

def main(page: ft.Page):
    page.add(ft.SafeArea(ft.Text("Hello, world!")))

ft.app(main)

App created and run as described here:
flet run --android

When scanning the QR code on the Android device the Flet client opens, but after a period of time it either shows the error or the client's home screen. Manually adding the URL results in the same error:
Error connecting to https://<IP_ADDRESS>:8551/myapp/main-py: <urlopen error [Errno 110] Connection timed out>

The troubleshooting steps described here do not resolve the issue.
Environment
Pc
Windows 10.0.19045
Poetry environment (in combination with Pyenv):

Python 3.11.0b4
Flet 0.10.3

Android device

Flet client version 1.1.1
Android 12
Realme 7 Pro

Pc and Android device are on the same WiFi network (connected to the same access point).

Can you try leaving out the 's' in https?

Oh, I'm sorry, that's a typo in the issue text. Actually, I did use http only.

Ok. Try moving to the latest flet version (0.12.0) and retrying.
If the error persists then, please share a recording of the android screen before and while the error happens.
I don't have an android (ios phone instead), and can't repro.

In my case I realized that I had a vpn enabled on my pc so I disabled it, it still didn't work, then I ran pip install flet --upgrade again, and it worked correctly.

@jwlankhaar-hu any Updates on this issue?

@ndonkoHenri Can you please reopen the issue because the problem has not been resolved yet. I have tried it with a new virtual environment with flet 0.19.0 and a new install of the flet app on my Android, but the same error occurs.

@jwlankhaar-hu Please provide more information on this issue...

Well, the exact same situation (except for the version) holds as described above. Even a simple hello world example results in a timeout error (110). There is no VPN connection enabled.

The issue has been resolved. It turned out to be a firewall setting. I thought I had ruled that out, but I was wrong. On Windows 10 you have to add a new inbound rule for the port shown along with the QR-code. It might be a good idea to add this to the Flet documentation.

Thanks for reporting back!

On Windows 10 you have to add a new inbound rule for the port shown along with the QR-code. It might be a good idea to add this to the Flet documentation.

Can you please provide a link to what you mean, so we add it.. ?

I wasn't able to find a proper Microsoft documentation page, but here's a description of the steps in Windows 10:
To add an inbound rule in Windows 10:

Click Start to open the Start menu and start typing Firewall
Click Firewall & network protection
Click Advanced settings
Click Yes to the question Do you want to allow this app to make changes to your device?
In the left panel click on Inbound rules
Click New rule in the right panel
Click Port and Next
Type the port number that is shown in the URL above the Flet QR code
Click Next to Allow the connection
Select the scope(s) to which the rule should apply (Domain, Private and/or Public)
Provide a name for the rule (e.g. Flet) and click Finish


[SPARK-33954][SQL] Some operator missing rowCount when enable CBO

What changes were proposed in this pull request?
This pr fix some operator missing rowCount when enable CBO, e.g.:
spark.range(1000).selectExpr("id as a", "id as b").write.saveAsTable("t1")
spark.sql("ANALYZE TABLE t1 COMPUTE STATISTICS FOR ALL COLUMNS")
spark.sql("set spark.sql.cbo.enabled=true")
spark.sql("set spark.sql.cbo.planStats.enabled=true")
spark.sql("select * from (select * from t1 distribute by a limit 100) distribute by b").explain("cost")

Before this pr:
== Optimized Logical Plan ==
RepartitionByExpression [b#2129L], Statistics(sizeInBytes=2.3 KiB)
+- GlobalLimit 100, Statistics(sizeInBytes=2.3 KiB, rowCount=100)
   +- LocalLimit 100, Statistics(sizeInBytes=23.4 KiB)
      +- RepartitionByExpression [a#2128L], Statistics(sizeInBytes=23.4 KiB)
         +- Relation[a#2128L,b#2129L] parquet, Statistics(sizeInBytes=23.4 KiB, rowCount=1.00E+3)

After this pr:
== Optimized Logical Plan ==
RepartitionByExpression [b#2129L], Statistics(sizeInBytes=2.3 KiB, rowCount=100)
+- GlobalLimit 100, Statistics(sizeInBytes=2.3 KiB, rowCount=100)
   +- LocalLimit 100, Statistics(sizeInBytes=23.4 KiB, rowCount=1.00E+3)
      +- RepartitionByExpression [a#2128L], Statistics(sizeInBytes=23.4 KiB, rowCount=1.00E+3)
         +- Relation[a#2128L,b#2129L] parquet, Statistics(sizeInBytes=23.4 KiB, rowCount=1.00E+3)

Why are the changes needed?
Improve the scope of CBO.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Unit test.

Test build #133584 has started for PR 30987 at commit 3c8e165.

Test build #133584 has started for PR 30987 at commit 3c8e165.

Test build #133584 has finished for PR 30987 at commit 3c8e165.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #133584 has finished for PR 30987 at commit 3c8e165.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133584/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133584/

Test build #133585 has started for PR 30987 at commit fcc3b89.

Test build #133585 has started for PR 30987 at commit fcc3b89.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38173/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38173/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38174/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38174/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38174/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38174/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38174/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38174/

Test build #133585 has finished for PR 30987 at commit fcc3b89.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #133585 has finished for PR 30987 at commit fcc3b89.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133585/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133585/

cc @wzhfy FYI

cc @wzhfy FYI

thanks, merging to master!

thanks, merging to master!


onModalHide is never called if the Modal is unmounted while still visible

Environment

System:
    OS: macOS 14.4.1
    CPU: (10) arm64 Apple M1 Max
    Memory: 717.27 MB / 64.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 18.16.1 - ~/.nvm/versions/node/v18.16.1/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v18.16.1/bin/yarn
    npm: 9.5.1 - ~/.nvm/versions/node/v18.16.1/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Managers:
    CocoaPods: 1.14.3 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 23.4, iOS 17.4, macOS 14.4, tvOS 17.4, visionOS 1.1, watchOS 10.4
    Android SDK:
      API Levels: 27, 28, 29, 30, 31, 32, 33, 34
      Build Tools: 27.0.3, 28.0.3, 29.0.2, 29.0.3, 30.0.2, 30.0.3, 31.0.0, 33.0.0, 33.0.1, 34.0.0
      System Images: android-28 | Intel x86 Atom_64, android-29 | Google APIs Intel x86 Atom, android-30 | Google APIs Intel x86 Atom, android-30 | Google Play Intel x86 Atom, android-31 | Google APIs ARM 64 v8a, android-31 | Google APIs Intel x86 Atom_64, android-33 | Google APIs ARM 64 v8a, android-34 | Google APIs ARM 64 v8a
      Android NDK: Not Found
  IDEs:
    Android Studio: 2022.1 AI-221.6008.13.2211.9514443
    Xcode: 15.3/15E204a - /usr/bin/xcodebuild
  Languages:
    Java: 17.0.9 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 18.2.0 => 18.2.0
    react-native: 0.73.6 => 0.73.6
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Platforms

Android, iOS and Web
Versions

Android: 14
iOS: 17.0
react-native-modal: 13.0.1
react-native: 0.73.6
react: 18.2.0

Description

When the Modal is hidden because isVisible changes from true to false, the onModalHide will be called.
But when the Modal is hidden because it's unmounted (isVisible is still true), the onModalHide is never called. This leads to problems because we often use onModalHide to perform some clean-up, or show a different UX when the modal is hidden.
Reproducible Demo
Here's the repository that contains the minimal reproducible example: https://github.com/tienifr/react-native-modal-bug-reproducible-example.
This is a new React Native project, bootstrapped using @react-native-community/cli. react-native-modal was added and some minor code was added to showcase the error (via this commit). The steps to run it are in the README.
After running it, we can see:

After 1 second, the Modal with text I am the modal content! will show (due to this code)
After 3 seconds, the Modal with text I am the modal content! will disappear (due to this code)
The onModalHide, which should log this, is never called.

Video showing the logs in onModalHide never happened
https://github.com/react-native-modal/react-native-modal/assets/113963320/25b87f23-7872-4619-a28a-926106a4f377

I am also running into this issue

I'm happy to contribute a fix if the maintainers agree that this is a valid bug.

The maintainers havent done anything in a couple of years. What is your fix?

+1 I have this issue. For now I'm using useEffect's clean-up function in order to achieve the outcome I want


Refactor getBibliography to return complete result array

This pull request refactors the getBibliography function in src/api.ts to return the complete result array instead of just the element at index 2. The modification was necessitated by a change in the Zotero Better BibTeX plugin, specifically in commit https://github.com/retorquere/zotero-better-bibtex/commit/087702b64c2f0fd6f25686d78f26f072d354422b. This commit removed the double wrapper in the response object, meaning that the desired data is no longer nested within result[2]. Consequently, getBibliography now returns the entire result array for direct access to the data, aligning with the updated response format.

Oh my! I'm so sorry to see your pr lately after I have fixed this bug yesterday. But still thank you!


vagrant fails to start on windows

Issue Type

Bug report
Feature request

Molecule and Ansible details
ansible --version
ansible 2.5.5
  config file = /tmp/molecule/testvagrant/default/ansible.cfg
  configured module search path = [u'/home/kunalsingt/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python2.7/site-packages/ansible
  executable location = /usr/bin/ansible
  python version = 2.7.14 (default, Oct 31 2017, 21:12:13) [GCC 6.4.0]

molecule --version
molecule, version 2.16.0

vagrant --version
Vagrant 2.1.2

Molecule installation method (one of):

pip

Ansible installation method (one of):

pip
OS package

Desired Behavior
it should create instance in virtualbox and provision and destroy
Please give some details of the feature being requested or what
should happen if providing a bug report.
Actual Behaviour (Bug report only)
Please give some details of what is actually happening.
Include a minimum complete verifiable example with
output of running molecule --debug.
debug.txt

What does the log say?
"msg": "ERROR: See log file '/tmp/molecule/testvagrant/default/vagrant-instance.err'"

Not enough info to debug.

I think I ran into the same issue. I use molecule over WSL (Windows Subsystem for Linux). The default TMPDIR in WSL (/tmp) is not reachable to Virtualbox on Windows. It works if I set the TMPDIR variable to a location that is available to Windows (e.g. TMPDIR=/mnt/c/Windows/Temp molecule create)
There was an error while executing `VBoxManage`, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.

Command: ["import", "\\\\?\\C:\\Users\\\User\\.vagrant.d\\boxes\\jborean93-VAGRANTSLASH-WindowsServer2019\\0.6.0\\virtualbox\\box.ovf", "--vsys", "0", "--vmname", "packer-virtualbox-iso-1547959474_1549173961541_69632", "--vsys", "0", "--unit", "10", "--disk", "./packer-virtualbox-iso-1547959474-disk001.vmdk"]

Stderr: 0%...10%...20%...30%...40%...50%...60%...70%...80%...90%...100%
Interpreting \\?\C:\Users\User\.vagrant.d\boxes\jborean93-VAGRANTSLASH-WindowsServer2019\0.6.0\virtualbox\box.ovf...
OK.
0%...
Progress state: E_INVALIDARG
VBoxManage.exe: error: Appliance import failed
VBoxManage.exe: error: Code E_INVALIDARG (0x80070057) - One or more arguments are invalid (extended info not available)
VBoxManage.exe: error: Context: "enum RTEXITCODE __cdecl handleImportAppliance(struct HandlerArg *)" at line 957 of file VBoxManageAppliance.cpp


Support for custom page factory providers

Add support for providing custom PageFactory. The factory is used when creating pages when:

Bootstraping/livsyncing the application.
Doing page navigation (with <page-router-outlet).
When showing custom modal dialogs.

Resolves #394
cc @VladimirAmiorkov

With this merged, can we now put the Side drawer over the navigation bar?

Hi @vjoao ,
Unfortunately the "over navigation bar" feature of RadSideDrawer in {N} + Angular applications is still not supported. It turned out to be a very difficult feature since the UI structure of such applications is different then the normal {N} application and also it is dynamically "changing" via the Angular 2 renderer etc.
You can follow this issue for more direct information regarding this feature of the RadSideDrawer component of the nativescript-telerik-ui plugin.

Is this still a issue i can't go to the link above and i didn't see anything about it on the
https://github.com/telerik/nativescript-ui-feedback/issues list

Hi @scavezze ,
You can follow this thread about this issue: https://github.com/telerik/nativescript-ui-feedback/issues/61


Add testing util to poll for async event testing

Due to the async nature of testing EventStream and Subscriptions, and some external pub/sub systems taking a longer time to publish and/or receive a message, we have added a helper method to help wait for a test case to assert.
Additionally we modified the tests to be able to more accurately check negative test cases where testing the absence of something which arrives async could be challenging.

pollUntilSuccessOrTimeout will execute a delegate function passed to it, if it fails it will sleep retrySleep ms, and will retry. If the total retry time exceeds timeout, it will finally fail.

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

Project coverage is 98.57%. Comparing base (1894c1a) to head (6f8f95d).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #758      +/-   ##
==========================================
- Coverage   98.64%   98.57%   -0.08%
==========================================
  Files          73       73
  Lines       10946    10946
  Branches     1582     1580       -2
==========================================
- Hits        10798    10790       -8
- Misses        142      150       +8
  Partials        6        6

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

I needed to test async events on another project.  I initially polled to make sure the event had finished, but the tests were taking a long time.  Instead, I created a promise in the test code that I resolved in the event handler, and the tests ran 2-3 times faster.  I don't know if this approach is appropriate for what you're doing, but I thought I'd mention it.