CouchDB 2.0: Race condition when creating/deleting database

Reposting of https://issues.apache.org/jira/browse/COUCHDB-3034 as this is still an issue
Description
I've noticed that CouchDB 2.0 (master) occasionally returns 500 errors when multiple clients attempt to PUT or DELETE the same database concurrently.
Steps to Reproduce
This is reproducible by running multiple instances of following script in parallel against an n=1 (single node) cluster:
#!/bin/bash

for i in `seq 1 2000`;
do
	printf '\nPUT: '
    curl -XPUT 'http://localhost:15984/testdb' -w "%{http_code}\\n"
    printf '\nDELETE: '
    curl -XDELETE 'http://localhost:15984/testdb' -w "%{http_code}\\n"
done

This generates output along the lines of:
PUT: {"ok":true}
201

DELETE: {"ok":true}
200

PUT: {"error":"error","reason":"internal_server_error"}
500

DELETE: {"error":"not_found","reason":"Database does not exist."}
404

PUT: {"error":"file_exists","reason":"The database could not be created, the file already exists."}
412

DELETE: {"ok":true}
200

PUT: {"error":"file_exists","reason":"The database could not be created, the file already exists."}
412

DELETE: {"ok":true}
200

PUT: {"ok":true}
201

DELETE: {"ok":true}
200

Your Environment

CouchDB Version used: 2.3.1 installed from RPM
Browser name and version: curl
Operating System and version: CentOS 7.5

this is really expected behaviour in a clustered database. 4.0 is likely going to address this, but until then, you’ll have to work around this with unique db names per iteration (plus cleanup) or sleep timeouts.

@janl Thank you for your response. Would you mind pointing to documentation (issues, design documents, whatever) or elaborating why this is expected behaviour in a clustered database? I'm asking, because I don't have much experience with clustered databases and would like to understand the underlying problem better. The other reason why I'm asking is: you say that it will be addressed in upcoming releases. Does that mean that is perhaps not as expected? It sounds like there are cases where this behaviour is unexpected and those should be addressed? Which are they?
Thanks again if you'd be so kind to explain.


STRtree with empty geometries segfaults

When constructing an STRtree with an empty geometry, any query against that will cause a segfault as GEOS attempts to dereference null bounds for the empty geometry. It seems like empty geometries should be skipped?
Example:
>>> from shapely.geometry import Polygon
>>> from shapely.strtree import STRtree
>>> polys = [ Polygon() ]
>>> tree = STRtree(polys)
>>> tree.query(Polygon([(0,0),(1,1),(2,0),(0,0)]))
Segmentation fault (core dumped)

(Tested with Python 2.7.6 on Ubuntu 14.04.3 with Shapely 1.5.13)

Looks like this is an issue for empty geometries only. Invalid geometries don't cause a segfault. It should be possible to check for this before they are inserted and raise an exception if required.

It seems to be legitimate to create an STRtree with no geometries, i.e. tree = STRtree([]). What should the behaviour of tree = STRtree([Polygon()]) be? Raise an exception, or return an empty tree? Also, does it matter if the GEOS STRtree is initalised with the wrong size. This will happen if we use len(geoms) but then don't insert all of the geometries.

@snorfalorpagus let's do this:

continue to allow create of empty trees since that isn't a reported problem
add only non-empty geoms to the tree and make sure the tree doesn't have undefined geoms

By that i mean filter the STRtree() input sequence and make a new list of non-empty geoms. Use the length of that list when initializing. There's some overhead to making that new list but it's not much, it doesn't allocate new memory for geometry objects.

Closed by #348.


test(wasi): enable tests related to symlink

Features we need:

absolute symlink()
readlink()
readdir() that can return symlinks
call stack size can be increased

Runtimes comparison:

runtime
1. symlink()
2. readlink()
3. readdir()
4. call stack size

wasmtime
❌
❌ (relative only)
✅
✅

Wasmer
❌
❌ (relative only)
❌
✅

WasmKit
✅
❌
✅
✅

WasmEdge
❌?
✅
✅
❌?

I'm going to create another PR to enable relative symlink tests while leaving absolute symlink tests disabled. This PR will focus on absolute symlink tests.


[General] - Advanced DSA problems

I have seen that no quiz is for data structures which is the most important topic for interview in computer Science
Hard only questions for Data Structures And Algorithms
I will also add medium and easy mcqs .
Adding Quiz topic to the repo...
I hope you will assign this issue to me , under hacktoberfect tag , I ensure it will help

I have seen that no quiz is for data structures

We are going to keep it to just the general cs quiz file.
So if you want to contribute DSA questions, you can add them to that file
Feel free to open a PR. Any PR merged during October counts toward Hacktoberfest


[bn] Localization of cla.md

Related issue : #45729

Hi @Ashfinn  , thanks for contributing .
Please fix them and maintain proper line breaks as en .

/lgtm
/label tide/merge-method-squash

My apologies.. I accidentally synced my pr branch instead on my dev branch

@Ashfinn ok , you have to open a new PR now
/close


[ExplicitModule] when using driver as an executable, differentiate PCM dependency names for each target

We need this to enable explicit modules in the driver-as-executable mode. For instance,
we have two Swift targets A and B, where A depends on X.pcm which in turn depends on Y.pcm, and
B only depends on Y.pcm. In the driver-as-executable mode, the build system isn't aware
of the shared dependency of Y.pcm so it will be generated multiple times. If all these Y.pcm
share the same name, X.pcm may fail to be loaded because its dependency Y.pcm could have a changed mod time.
We only differentiate these PCM names in the non-integrated mode due to the lacking of
inter-module planning.
rdar://72830153

@artemcm IIUC, this won't regress the build performance of explicit modules in SwiftPM. Could you confirm?

@artemcm IIUC, this won't regress the build performance of explicit modules in SwiftPM. Could you confirm?

@swift-ci please test

@swift-ci please test

@artemcm IIUC, this won't regress the build performance of explicit modules in SwiftPM. Could you confirm?

I suspect that it will. Because currently we will come up with an ahead-of-time plan to build Y.pcm several times, and then when SwiftPM lowers our Jobs into LLBuild rules, it will de-duplicate them, resulting in Y.pcm being built once (versioned compilation aside). But if we give them different names, then it will not be able to de-duplicate them and we will actually repeat an identical build action several times just to have these modules with different names. We could do an experiment to confirm, though.

@artemcm IIUC, this won't regress the build performance of explicit modules in SwiftPM. Could you confirm?

I suspect that it will. Because currently we will come up with an ahead-of-time plan to build Y.pcm several times, and then when SwiftPM lowers our Jobs into LLBuild rules, it will de-duplicate them, resulting in Y.pcm being built once (versioned compilation aside). But if we give them different names, then it will not be able to de-duplicate them and we will actually repeat an identical build action several times just to have these modules with different names. We could do an experiment to confirm, though.

hmm, Will the flag integratedDriver be true in the SwiftPM case? Notice we only differentiate those names when integratedDriver is false, that's when running swift-driver as a standalone executable.

hmm, Will the flag integratedDriver be true in the SwiftPM case? Notice we only differentiate those names when integratedDriver is false, that's when running swift-driver as a standalone executable.

hmm, Will the flag integratedDriver be true in the SwiftPM case? Notice we only differentiate those names when integratedDriver is false, that's when running swift-driver as a standalone executable.

Good point, yeah then this should not affect performance in this use-case. 👍

hmm, Will the flag integratedDriver be true in the SwiftPM case? Notice we only differentiate those names when integratedDriver is false, that's when running swift-driver as a standalone executable.

Good point, yeah then this should not affect performance in this use-case. 👍

hmm, Will the flag integratedDriver be true in the SwiftPM case? Notice we only differentiate those names when integratedDriver is false, that's when running swift-driver as a standalone executable.

Good point, yeah then this should not affect performance in this use-case. 👍

Thanks for confirming! When enabling explicit modules for swift-driver-as-executable, changing the PCMs names won't affect build performance because these PCMs are built multiple times disregardingly IIUC.

hmm, Will the flag integratedDriver be true in the SwiftPM case? Notice we only differentiate those names when integratedDriver is false, that's when running swift-driver as a standalone executable.

Good point, yeah then this should not affect performance in this use-case. 👍

Thanks for confirming! When enabling explicit modules for swift-driver-as-executable, changing the PCMs names won't affect build performance because these PCMs are built multiple times disregardingly IIUC.

@swift-ci please test

@swift-ci please test


Sessions: Ollama server check is not working on mobile resolutions

The logic introduced in this PR https://github.com/fmaclen/hollama/pull/99/files#diff-e13c06b2f8eb5ca2f92007ee68d5fecf31e07131e5f14dcfb9a58c00efc01fe3R62-R76 is not being executed when the app is rendered at a small resolution.
Steps to reproduce:

Resize viewport to 480px width
Check the Ollama server is connected in Settings
Click on Sessions
Turn off Ollama server
Click on New session

Expected behavior:

Model list should be blank
The UI should display a disconnected message

:tada: This issue has been resolved in version 0.7.13 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Updated PG UiProdiver to use new PG component

I need to finish going over the PropertyGrid code tomorrow.
Did you clean it up since you showed it to me & bill yesterday?

Yeah it should be a bit more organized now

Theres a lot of different code styles here, some from the existing property grid, bill's example, yours; please try to consolidate and be consistent .


GPT-4?

How do I switch to GPT-4?

Solved. Enter "model" "gpt-4" in as a new parameter!

The llm-book.openAI.model setting should also work


HA site won't open after clean install on both windows 10 and ubuntu 16.04

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.27.2
Python release (python3 --version):
3.5.2
Component/platform:
Windows10 Pro, Ubuntu 16.04 desktop
Description of problem:
When attempting to open the ui on either os I am presented with the same error from HA....OK?
C:\Users\Midi>python -m homeassistant --open-ui
Unable to find configuration. Creating default one in C:\Users\Midi\AppData\Roaming\.homeassistant
Traceback (most recent call last):
  File "C:\Users\Midi\AppData\Local\Programs\Python\Python35-32\lib\site-packages\homeassistant\__main__.py", line 338, in <module>
    sys.exit(main())
  File "C:\Users\Midi\AppData\Local\Programs\Python\Python35-32\lib\site-packages\homeassistant\__main__.py", line 330, in main
    exit_code = setup_and_run_hass(config_dir, args)
  File "C:\Users\Midi\AppData\Local\Programs\Python\Python35-32\lib\site-packages\homeassistant\__main__.py", line 243, in setup_and_run_hass
    config_file = ensure_config_file(config_dir)
  File "C:\Users\Midi\AppData\Local\Programs\Python\Python35-32\lib\site-packages\homeassistant\__main__.py", line 64, in ensure_config_file
    config_path = config_util.ensure_config_exists(config_dir)
  File "C:\Users\Midi\AppData\Local\Programs\Python\Python35-32\lib\site-packages\homeassistant\config.py", line 125, in ensure_config_exists
    config_path = create_default_config(config_dir, detect_location)
  File "C:\Users\Midi\AppData\Local\Programs\Python\Python35-32\lib\site-packages\homeassistant\config.py", line 155, in create_default_config
    location_info.longitude)
  File "C:\Users\Midi\AppData\Local\Programs\Python\Python35-32\lib\site-packages\homeassistant\util\location.py", line 75, in elevation
    return int(float(req.json()['results'][0]['elevation']))
IndexError: list index out of range

Expected:
I just want the site to come up I have been struggling with this for a few days now. I want to install it on a raspberry pi but preview it on my windows machine to play with until it arrives.......
I have had several issues just getting HA to install in python and decided to test install on two clean machines I have around here........ A windows10 pro, ubuntu 16.04, and now i have tried it in an Ubuntu 14.04 VM, all with the same resulting error!
TIA,
MidiHipi
Sorry I don't know where the yaml is stored on my windows machine.....
Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):

Additional info:

Also I am very impressed with vids and info and all the assistance around this platform!

Sure I will give it a try...

sorry, I am a complete noob, where is it in windows and what is it called pls?
Midi

configuration.yaml is located here.

both of those locations are empty except for a folder "deps" ......

Does the user your running HA with have proper permission to write to the location in question?

In the case of windows, yes I am administrator. On the linux machines i am hass......

its been a while.. I faced the same initial problem as you, but i found that simply running the command to start HA again simply works.

This looks to me like the geo-ip is failing to get a proper elevation.
@fabaff Do we currently trap errors in the geolocation upon initial startup?


[android] Notify FrostWire is running / 'Exit' action - The app appears on 'Running apps' after Exiting FW

After 'Exit' FrostWire from the main menu the 'Notification message' doesn't appear on the 'slide menu' of the device, however, I can see FrostWire is running if I select the 'Running Apps' square button placed at the bottom bar of the device.

One of the processes that leave the app running behind even after 'Exiting' FW is when you do a search and then 'Exit' the app.
To reproduce:
1) Open FW and do a search.
2) Open main menu and 'Exit' FW
3) After exiting the ads you will see FW logo at the top-left of the screen. Also if you go to your device Settings and check 'Running Apps' you will see these FW services running: 'MusicPlayBackService' and 'EngineService'. (See image attached):

Ticket on qa-android: https://github.com/frostwire/qa-android/issues/78

On my phone (Sony Xperia Z5 with Android 5.1.1), it Exits (I no longer see the notification that it is running, nor any other sign of it). If I go to Apps > Running right after shutdown, FrostWire is not there either.
If I come back after a while (about 10min) to Apps > Running I see the app active with 1 Service and 2 Processes active.
See screenshot

@muckachina please take a look again to see if this issue keeps happening. Should be good now.

ping

I had the same thing that @muckachina was experiencing before and it persists. Currently I have the FrostWire Plus v.1.7.4 256 that was shared and upon exiting the app I can still see the app the same way as muckachina's first screenshot above. Frostwire seem to not be visible in the settings -> running apps.

Well, that's the list of recent opened applications, not current running applications. As long as we are not running (or silently restarting) anymore  we are fine.

Sorry I wanted to say "but it does not persists". I had it in my settings - running apps before similarly to Kademlia but not anymore.
But for a second I thought the screenshot above is also about running apps. :/ my bad.

@muckachina is this still happening with the latest release?
I think a lot of work was done to have proper shutdown, should be better now.

bump.
Is this still happening after all the fixes with releasing advertisement resources on 1.7.6?

@muckachina is this still happening?

It seems this issue has subdued after fixes with the advertising SDKs which may have been keeping the process open or coming back.

Still happening in my galaxy s8 with Android pie.. Really annoying.

what's happening now is a bit different and it has to do with the way FrostWire must advertise its background service to android, now it's android that says "FrostWire is Running", we'll try to see if there's a way to make this experience better by automatically shutting down the service if the user isn't actively seeding nor downloading anything.

Any news for this. Still happening...

Got worst after update from yesterday... The permanent notification from settings isn't working and I still have the running notification from Android for the frostwire app...

@olivier-voutat are you sure you hit Exit from the menu, and not back all the way to go home?
Thank you for reporting, I'll take another look.

Yes, I am.


Separate problems_elasticity.jl

Have separate file problems_elasticity_2d.jl for plane stress, plane
strain, axisymmetric etc. problems.

Coverage decreased (-0.3%) to 90.044% when pulling 8453dc3c35f1a67a00f19fe65bacb6db9430cbff on separate_elasticity into 8c955d2b5f383a9c6814be9793b511dcc021febc on master.


Add option to delete single file or collection.

deleting collection available in new script delete-records.py  still need to do single file delete.


[Opensearch] Implement better subnet discovery - subnetIdSelector?

What problem are you facing?
The crossplane-contrib aws provider leveraged subnetIdSelector in the following file:  https://github.com/crossplane-contrib/provider-aws/blob/713458364ba9d9c7c5dfe94c47b12fc3ceb959ef/apis/opensearchservice/v1alpha1/custom_types.go
It doesn't appear that this is available in the current version of Opensearch provider.
How could Official AWS Provider help solve your problem?
We should add this functionality back to the provider, unless there is a more preferred method of subnet discovery that is preferred over this one.

for all inputs in https://marketplace.upbound.io/providers/upbound/provider-aws-opensearch/v0.47.1/resources/opensearch.aws.upbound.io/Domain/v1beta1#doc:spec-forProvider-vpcOptions the reference / selector is missing


Add alias for dotnet workload uninstall command

Add an alias for dotnet workload uninstall command
Fixes #3486
There is nothing to uninstall on the build servers, what integration test I should add here: https://github.com/cake-build/cake/blob/develop/tests/integration/Cake.Common/Tools/DotNetCore/DotNetCoreAliases.cake

Install any workload in "Cake.Common.Tools.DotNetCore.DotNetCoreAliases.Setup"
Add "Cake.Common.Tools.DotNetCore.DotNetCoreAliases.DotNetWorkloadInstall" and "Cake.Common.Tools.DotNetCore.DotNetCoreAliases.DotNetWorkloadUninstall" as dependent on the "DotNetWorkloadInstall"
DotNetWorkloadUninstall("any") with try...catch
Skip for now

Please review
Thank you in advance

@Marusyk your changes have been merged, thanks for your contribution 👍


Image library endpoint?

Is there an endpoint or way to get image assets of characters. As the api has ids for pickaxes (3b829757-254d-4011-99d3-a4ab770af629) but can't work out how to identify that id to an image.

Hi @TheMaster,
I have no internet actually (I move on new apartment x) at this time). End of the next week, i will have internet back. So remind me to checkup how we can found assets about Fortnite, on my Discord ;) https://discord.gg/PurBeRF
Regards
Quentin

The IDs are actually for that instance of the object (i.e. the user's particular copy of a Glider etc). Depending on the API the field you're looking for is usually called TemplateId, but it's not a 1-to-1 mapping to an asset. Some of them are close.

So how would I go about extracting assets? which contains the images Im looking for.
Thanks

@TheMaster & @modsognir if you have already have this informations you can add it to the repo (fork it and make a pull request ;))

I've been using a program called UModel for getting game assets

Hi dudes,
I am very interested in this information. Fortnitetracker.com and such have the ability to pull store data with images, how would i be able to find a relationship between the store assets and the api call?
All i can find is displayAssetPath.

Can anyone link me to a pull archive of all shop assets

How did you do please ?

@leomelki You have to decrypt the .pak files using AES. The hard part is to find the AES key, but someone usually shares the key with the community. You can use the UModel viewer to extract most of the assets. Read Gildor's forum for more information, everything is described there :)

Thank you for the response. Indeed, I already did it but I don't know how to link the informations of the shop to the files we decrypted. I know that the id of the offer is virtual:/r5650hshlvak3otsblq62ui5c8[0]0#0
and the name id [VIRTUAL]1 x Stage Dive for 800 MtxCurrency but I can't find the linked texture, do you know how to find this ? Thank you !

The id is a unique identifier for that occurance of the item in the store -
it won’t get you the image name. Those are stored in data table files.
On Sat, 30 Jun 2018 at 01:03, leomelki<EMAIL_ADDRESS>wrote:

Thank you for the response. Indeed, I already did it but I don't know how
to link the informations of the shop to the files we decrypted. I know that
the id of the offer is virtual:/r5650hshlvak3otsblq62ui5c8[0]0#0
and the name id [VIRTUAL]1 x Stage Dive for 800 MtxCurrency but I can't
find the linked texture, do you know how to find this ? Thank you !
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/qlaffont/fortnite-api/issues/36#issuecomment-401501759,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGTF3QwbLmBAjcFxXF2cON_qBSmxSVaks5uBsBDgaJpZM4SQKpU
.

This is kind of useless now as they have removed profile endpoint which gave you profile cosmetics


Require timerep < 2.0, due to interface change

I just encountered a build failure because I happened to have timerep 2.0.0 installed. Unfortunately, it looks like supporting both interfaces would need conditional compilation. But nothing I'm building with requires 2.0.0 yet, so the upper bound did the trick.

Thanks!


Research Google Drive API and verify viability for project.

Confirm that we will be able to use this API without backend development.

Closing this with note that Google API is not in scope for this project.
Will be using https://exchangeratesapi.io/documentation/ instead to convert the stock quotes to alternative currencies


Expo app breaks

Bug report
Describe the bug
Expo apps that use supabase/supabase-js completely break after updating to SDK43.
To Reproduce

Go to https://github.com/10000multiplier/exposupabasebug
git clone https://github.com/10000multiplier/exposupabasebug.git
npm i
expo start

Expected behavior
Supabase-js behaves flawlessly on expo apps as it did on SDK42.
More info
The fatal error that appears after running expo start:
Invariant Violation: "main" has not been registered. This can happen if:
* Metro (the local dev server) is run from the wrong folder. Check if Metro is running, stop it and restart it in the current project.
* A module failed to load due to an error and `AppRegistry.registerComponent` wasn't called.

System information

OS: Windows
Version of supabase-js: ^1.25.2
Version of Node.js: 16.13.0

@kiwicopple
I want to scale my app to millions of users with supabase, please help fix 😅

Of relevance, when running this example, this error shows up as well:
TypeError: undefined is not an object (evaluating 'window.location.href')
at node_modules\react-native\Libraries\LogBox\LogBox.js:149:8 in registerError
at node_modules\react-native\Libraries\LogBox\LogBox.js:60:8 in errorImpl
at node_modules\react-native\Libraries\LogBox\LogBox.js:34:4 in console.error
at node_modules\expo\build\environment\react-native-logs.fx.js:27:4 in error
at node_modules\react-native\Libraries\Core\ExceptionsManager.js:104:6 in reportException
at node_modules\react-native\Libraries\Core\ExceptionsManager.js:172:19 in handleException
at node_modules\react-native\Libraries\Core\setUpErrorHandling.js:24:6 in handleError
at node_modules\@react-native\polyfills\error-guard.js:49:36 in ErrorUtils.reportFatalError
at node_modules\metro-runtime\src\polyfills\require.js:204:6 in guardedLoadModule
at http://<IP_ADDRESS>:19000/node_modules%5Cexpo%5CAppEntry.bundle?platform=android&dev=true&hot=false&minify=false:168062:3 in global code

I've just tested in one of my in-development projects, and can confirm that the same issue exists. The error shown on screen seems to point towards the getParameterByName function in gotrue-js: https://github.com/supabase/gotrue-js/blob/master/src/lib/helpers.ts#L16

As an update, it appears that adding detectSessionInUrl: false into the options of createClient resolves this issue. This certainly seems to be a new change compared to last time I tried it in a native context, and I'm not sure if this affects other functionality of using Supabase in Expo.

@10000multiplier to be honest not sure how the app is working without index file and you're not using the polyfill anywhere?

@hannes1
Added the polyfill on top of App.tsx and still gives the same error.
How would you add an index file?

@ChronSyn setting detectSessionInUrl to false seems to work indeed. Thank you sir for your glorious tip.

after setting detectSessionInUrl to false will supabase.auth.onAuthStateChange work??
because after setting detectSessionInUrl to false for me it is not returning any thing for me...

@HassanIyan I am having the same issue

hi, setting  detectSessionInUrl: false does not cause any difference, any potential workarounds?

Afaict this specific issue should be fixed in https://github.com/supabase/gotrue-js/pull/153.
If folks are still having this issue, please create another issue as it might be caused by something else.


How can I listen for filterUpdate events within a Hydrogen app away from the client file that the update happens?

Hi,
So I am building a hydrogen app with Klevu.
I have a main menu area that is built within a header and that lives separate like so:
<Suspense fallback={}>

</Suspense>
<MainContent children={children}></MainContent>

MainContent is where the products live and Header is where the filter lives.
Because of the constraints of Hydrogen and SSR with Server Components it is very difficult to pass data back to the server.jsx file to be rendered within the MainContent. Obviously server files render once only and aren't reactive so I cannot set new data from filter updates because the .server.jsx file has no idea.
Below is an example visually of what I am building to give you context.

I think there may need to be a global event listener that wraps the app somehow but I am pretty sure that it will not work with SSR and how we'd need to use clientside functions to render the data on the f/e.

Hey there!
Yes. That is very tricky thing to do. I'm not 100% sure that this will work, but let's try :)
What I assume is that you this kind of case:  You have two server components: nav and main. These two are server components that are rendering on server. And then after the client has been hydrated then client main should react to changes made in client nav.
To transfer the request from server to client I created these two functions: KlevuPackFetchResult and KlevuHydratePackedFetchResult. You should pack results in the server side and hydrate in the frontend side. In your case I would try hydrate in both: nav and in main. But you would need to create separate client only FilterManager that is imported to both of these (nav and main) client components.
Now you can use that one client FilterManager instance to change facets to different values in nav component and in main component it is only used as parameter to client side KlevuFetch() (You need to do separate client only fetching after hydration that is separate from server side fetching).
And as last bit you need to listen Dom events in main component. Listen for FilterSelectionUpdate and in callback run client side KlevuFetch() again. Using same FilterManager between those two frontend components should do the trick of updating facets for query.
Here is my example code that does the same, but with just one client component: https://github.com/klevultd/frontend-sdk/blob/master/examples/hydrogen/src/routes/search.server.tsx
And here is client component: https://github.com/klevultd/frontend-sdk/blob/master/examples/hydrogen/src/components/searchResultPage.client.tsx#L15
There in line 15 is the FilterManager. If it would be moved to separate file where it is created and instance is exported to both of your client components.

So this kind of hits the nail on the head in what I am trying to do @rallu.
The only difference is that there are lots of other deep nested components within that Header and Main that control lots of complex functionality so it isn't a a simple one and they are both client because of what they need to do with state management and hooks in react. This means that the filters are deeply nested inside all of this as a client component. eg below:
import {Suspense, useEffect} from 'react';
import {
  useLocalization,
  useShopQuery,
  CacheLong,
  gql,
  useServerProps,
  useServerState,
  useRouteParams,
  useUrl,
} from '@shopify/hydrogen';

import {useContentfulQuery as useContentfulQuery} from '~/services/contentful';

import {Header, MainContent} from '~/components';
import {parseMenu} from '~/lib/utils';
import {PAGINATION_SIZE} from '~/lib/const';

const HEADER_MENU_HANDLE = 'main-menu';
const FOOTER_MENU_HANDLE = 'footer';

const SHOP_NAME_FALLBACK = 'l';

/**
 * A server component that defines a structure and organization of a page that can be used in different parts of the Hydrogen app
 */
export function Layout({
  children,
  pageTransition,
  title,
  description,
  mainClassName,
  product,
  variant,
}) {
  const {pathname} = useUrl();
  // activeLayer
  const {
    language: {isoCode: languageCode},
    country: {isoCode: countryCode},
  } = useLocalization();

  const {category, slug} = useRouteParams();

  const {data} = useShopQuery({
    query: TYPES_QUERY,
    variables: {
      country: countryCode,
      language: languageCode,
      pageBy: PAGINATION_SIZE,
    },
    preload: true,
  });

  const cats = data.collections.nodes.filter((v) => v.node != '');
  // get last segment of a url
  const lastSegment = pathname.split('/').pop();
  console.log(lastSegment);
  const catsTransform = cats.map((v) => {
    return {
      name: v.title,
      slug: '/collections/' + v.handle.toLowerCase().replace(/ /g, '-'),
    };
  });

  const {data: useConData} = useContentfulQuery({
    query: CONTENTFUL_HIGHLIGHTS_AND_INFO_QUERY,
    key: pathname,
  });

  const {items} = useConData.highlightCollection;
  const highlightItems = items.filter(
    (item) => item && item.type.includes('Men'),
  );

  const infoData = useConData.informationCollection?.items[0] || {};

  return (
    <>

            Skip to content

        <Suspense fallback={}>

        </Suspense>
        <MainContent children={children}></MainContent>

    </>
  );
}

function HeaderWithMenu({pageTitle}) {
  const {shopName} = useLayoutQuery();
  return ;
}

function useLayoutQuery() {
  const {
    language: {isoCode: languageCode},
  } = useLocalization();

  const {data} = useShopQuery({
    query: SHOP_QUERY,
    variables: {
      language: languageCode,
      headerMenuHandle: HEADER_MENU_HANDLE,
      footerMenuHandle: FOOTER_MENU_HANDLE,
    },
    cache: CacheLong(),
    preload: '*',
  });

  const shopName = data ? data.shop.name : SHOP_NAME_FALLBACK;

  /*
        Modify specific links/routes (optional)
        @see: https://shopify.dev/api/storefront/unstable/enums/MenuItemType
        e.g here we map:
          - /blogs/news -> /news
          - /blog/news/blog-post -> /news/blog-post
          - /collections/all -> /products
      */
  const customPrefixes = {BLOG: '', CATALOG: 'products'};

  const headerMenu = data?.headerMenu
    ? parseMenu(data.headerMenu, customPrefixes)
    : undefined;

  return {headerMenu, shopName};
}

const TYPES_QUERY = gql`
  query ProductTypes($country: CountryCode, $language: LanguageCode)
  @inContext(country: $country, language: $language) {
    collections(first: 10) {
      nodes {
        handle
        title
      }
    }
  }
`;

const SHOP_QUERY = gql`
  fragment MenuItem on MenuItem {
    id
    resourceId
    tags
    title
    type
    url
  }
  query layoutMenus($language: LanguageCode, $headerMenuHandle: String!)
  @inContext(language: $language) {
    shop {
      name
    }
    headerMenu: menu(handle: $headerMenuHandle) {
      id
      items {
        ...MenuItem
        items {
          ...MenuItem
        }
      }
    }
  }
`;

const CONTENTFUL_HIGHLIGHTS_AND_INFO_QUERY = gql`
  query {
    highlightCollection {
      items {
        title
        slug
        collection
        image {
          url
          width
          height
          title
        }
        type
      }
    }
    informationCollection {
      items {
        instagramLink
        facebookLink
        emailContact
        pageLinksCollection {
          items {
            title
            slug
          }
        }
      }
    }
  }
`;

@rallu how are you?
I have got the above working as you wrote above.
My only query is the hydrate function that I wrote gets the correct filters back but for some reason products do not seem to be updating. When I check the network the search from klevu servers suggests otherwise.
import {
  useState,
  useRef,
  forwardRef,
  useImperativeHandle,
  useEffect,
  useCallback,
} from 'react';
import {useDebounce} from 'react-use';
import {Link, flattenConnection, useServerProps} from '@shopify/hydrogen';

import {
  listFilters,
  applyFilterWithManager,
  KlevuFetch,
  FilterManager,
  KlevuDomEvents,
  KlevuListenDomEvent,
  KlevuPackFetchResult,
  KlevuHydratePackedFetchResult,
  search,
  sendSearchEvent,
} from '@klevu/core';

import {Button, Grid, ProductCard} from '~/components';
import {getImageLoadingPriority} from '~/lib/const';

import {searchQuery} from '~/services/klevu';

import {manager} from '~/components/global/FilterManagerClient.client';
let currentResult = {};
let clickEvent = null;

export function ProductGrid({url, collection, description, baseKlevuQuery}) {
  const nextButtonRef = useRef(null);
  const initialProducts = collection?.products?.nodes || [];
  const {hasNextPage, endCursor} = collection?.products?.pageInfo ?? {};
  const [products, setProducts] = useState(initialProducts);
  const [klevuProducts, setKlevuProducts] = useState(null);
  const [cursor, setCursor] = useState(endCursor ?? '');
  const [nextPage, setNextPage] = useState(hasNextPage);
  const [pending, setPending] = useState(false);
  const haveProducts = initialProducts.length > 0;

  const {serverProps, setServerProps} = useServerProps();

  const [options, setOptions] = useState(manager.options);
  const [sliders, setSliders] = useState(manager.sliders);
  const [searchResultData, setSearchResultData] = useState([]);

  const hydrate = async () => {
    currentResult = await KlevuHydratePackedFetchResult(
      baseKlevuQuery,
      searchQuery('*', manager),
    );
    console.log(currentResult);
    const search = currentResult.queriesById('search');
    if (search) {
      setKlevuProducts(search.records);
      if (search.getSearchClickSendEvent) {
        clickEvent = search.getSearchClickSendEvent();
      }
    }
  };

  const handleFilterUpdate = (e) => {
    hydrate();
  };

  useEffect(() => {
    const stop = KlevuListenDomEvent(
      KlevuDomEvents.FilterSelectionUpdate,
      handleFilterUpdate,
    );
    // cleanup this component
    return () => {
      stop();
    };
  }, []);

  const fetchProducts = useCallback(async () => {
    setPending(true);
    const postUrl = new URL(window.location.origin + url);
    postUrl.searchParams.set('cursor', cursor);

    const response = await fetch(postUrl, {
      method: 'POST',
    });
    const {data} = await response.json();

    // ProductGrid can paginate collection, products and search routes
    // @ts-ignore TODO: Fix types
    const newProducts = flattenConnection(
      data?.collection?.products || data?.products || [],
    );
    const {endCursor, hasNextPage} = data?.collection?.products?.pageInfo ||
      data?.products?.pageInfo || {endCursor: '', hasNextPage: false};

    // this was changed from {...newProducts} to {...products} because sorting was causing issues.
    // If we get issues with pagination then we need to revisit this more
    setProducts([...products, ...newProducts]);
    setCursor(endCursor);
    setNextPage(hasNextPage);
    setPending(false);
  }, [cursor, url, products]);

  const handleIntersect = useCallback(
    (entries) => {
      entries.forEach((entry) => {
        if (entry.isIntersecting) {
          fetchProducts();
        }
      });
    },
    [fetchProducts],
  );

  useEffect(() => {
    const observer = new IntersectionObserver(handleIntersect, {
      rootMargin: '100%',
    });

    setServerProps({description: description});

    const nextButton = nextButtonRef.current;

    if (nextButton) observer.observe(nextButton);

    return () => {
      if (nextButton) observer.unobserve(nextButton);
    };
  }, [nextButtonRef, cursor, handleIntersect]);

  if (!haveProducts) {
    return (
      <>

No products found on this collection

        <Link to="/products">

Browse catalog

        </Link>
      </>
    );
  }

  return (
    <>

          {klevuProducts && klevuProducts.length > 0
            ? klevuProducts.map((product) =>
{product.name}
)
            : products.map((product) => (

                  <ProductCard
                    key={product.id}
                    product={product}
                    loadingPriority={getImageLoadingPriority(product)}
                    className="product-card_sm-width"
                  />

              ))}

      {nextPage && (

          <Button
            variant="secondary"
            disabled={pending}
            onClick={fetchProducts}
            width="full"
          >
            {pending ? 'Loading...' : 'Load more products'}
          </Button>

      )}
    </>
  );
}

If you check my hydrate function it seems to not give the correct product records or it never updates.

I think this
const handleFilterUpdate = (e) => {
  hydrate();
};

should be replaced with
const handleFilterUpdate = (e) => {
  fetchProducts();
};

@rallu yeah I spotted that yesterday. Still getting my head around all this ;)

Closing as inactive. Please comment if more information is required.


Update http4s-client, http4s-dsl to 0.23.2

Updates

org.http4s:http4s-client
org.http4s:http4s-dsl

from 0.23.1 to 0.23.2.
GitHub Release Notes - Version Diff
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!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: test-library-update, semver-patch

Superseded by #49.


Allow hackathon schedule access via JSON/API

Moved to new API

Added as a subset feature of #226.


Federated join room clarification

Ruma's implementation for the send_join endpoints (Federation API) is different from the one in Synapse/Dendrite and we would like to clarify which is the expected layout for this endpoint. In this case, Ruma is not including room_id and event_id on the request body because they are already on the path param.

send_join body should be a representation of a PDU described on the Room version specification?

If it should, what happens if the room_id field on the body is distinct from the roomId on the path parameter?
If it should, what happens if the event_id field (in case of Room versions 1 and 2) is distinct from the eventId on the path parameter?

Thanks!

excellent questions. Synapse largely ignores the path params, using the body params instead. TBH I think the path params should probably be removed (https://github.com/matrix-org/matrix-doc/issues/2330), but in the meantime returning a 400 seems reasonable.
I'm going to close this in favour of https://github.com/matrix-org/matrix-doc/issues/2330.

@richvdh Thanks for the response.
Could you confirm that room_id (and event_id on room versions 1 and 2) should appear on the body, please?

yes, they should.


Bug: date ranges are not presnted consistently (e.g. 2 day events sometimes show as 1 and etc).

Do you want to request a feature or report a bug?
Bug
What's the current behavior?
An event sometimes spans a few days, sometimes spans one day, with the same range and depending on the day of the week.
I assume, looking from the demos, that an event from 1/1/2018-1/1/2018 is one day, and 1/1/2018-2/1/2018 is two days. However it's not the case. It changes depending on the day of the week, year and really just inconsistent all around.
Code sandbox: https://codesandbox.io/s/vlyl9q93y
I put two calendars there. Look at events "A" and "B". Both are the same range (but a different month/year). One shows as a 3 day event, the other as 2.
Then look at C and D. Both show as 2 days although one is 1 day the other is 2. Then look at E which should be 2 days but is 1.
Even if I misunderstood (wasn't obvious from the docs) what are the expected ranges for allDay events, the inconsistencies are still there, and they are not very predictable.
Note: I removed the moment locale setting from the sandbox because it was causing issues in the sandbox, but I have it in my local example and it's still broken.
Versions are like in the codesandbox. React: 0.16.2, calendar: 0.17.0 (latest of both). Tried on both chrome and firefox.
What's the expected behavior?
Should be consistent and the same ranges should be presented the same. In addition, it's probably a good idea to document the ranges. That is, is 1/1/2018-2/1/2018 a two days event, or a 1 day event like in ical (probably the latter).

Scratch that again -- I must have done something wrong when installing the latest from master before. When I tried again today things were working as expected.
So, the bug definitely still exists in 0.17.0, but at least my specific issue is resolved in the current master.
Thanks

@tchamberlin, lucky you. As you can see, some of the bugs in my example still occur (at least at the last time I checked).
Any news about when 017.1 is going to be out? At least having some of the issues fixed would be a great start, and I'd rather not run this lib from master.

Just verified again with 0.18.0, it's still broken. Specifically, the example I posted above is still broken, please take a look at the upper calendar in the example.
Please re-open this ticket.
Thanks!

The bug in your example is that you are using strings as dates in your events and not Date objects. Dates are being parsed as local dates from UTC so they aren't "even" over mightnight and so cross days than you want. The same example with correctly specified dates works: https://codesandbox.io/s/7oj4w4vzlq

Great, thanks, this indeed fixes the last remaining issue.

hi @tasn is that issue if fixed ?

@aman0511, yeah... Look at my previous comment.

@tasn
for me if two day event is in all day it showing for only one day i am using the date object

Take a look at @jquense's example from above, it should work and show you what the correct values should be. I think you're most likely just misunderstanding how multi day events are represented.
I don't remember from the top of my head, but I assume it's like iCal where an event from 1/12 at 00:00 to 2/12 at 00:00 is one day. You probably got that wrong.
No idea though, sorry, I'm not the maintainer of this library, but I assume the above code examples should be sufficient.


Can't find SharpAdbClient.Extensions on Nuget

Hi there, I can't find SharpAdbClient.Extensions on Nuget, is it I should download the source code and add to my project?
Thanks

Yes, we've completely deprecated SharpAdbClient.Extensions in 149645a7665835317f4e9c82e51a3d3ec64008f9 . You can still use Git to to get access to the older versions, but we have no intention to support them.


nixos/git: use programs.bash.interactiveShellInit for Bash-specific prompt

For options like environment.interactiveShellInit, we should not assume a Bash-like shell is always present for /etc/profile. Since this is specific to Bash, include it in programs.bash.interactiveShellInit instead.
Similar to #294121 in intent, but with a slightly different outcome.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

The path is misleading. This is used for more than Bash: https://github.com/git/git/blob/cc01bad4a9f566cf4453c7edd6b433851b0835e2/contrib/completion/git-prompt.sh

The script definitely uses Bash-specific features, but it also explicitly has compatibility with Zsh.


Rename nmap module to services

The only issue I can see with this change, is that every existing device that has the nmap module enabled it has now been disabled. Could we add logic to update every device config when Testrun launches or when config is loaded?

The only issue I can see with this change, is that every existing device that has the nmap module enabled it has now been disabled. Could we add logic to update every device config when Testrun launches or when config is loaded?

Unclear on this issue, all modules are enabled by default unless the config file for the module or the device is set to disabled.  I would expect this to actually be the reverse where nmap modules set as disabled would be enabled.


3.5: ionViewWillEnter trigger not fired when setting location in browser

Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
I'm using ionic 3.5.3's new location code and trying to handle the case where a user visits a link but is not logged in.
Post login I can't just setRoot('page', { opts }) any more as this is under some tabs. It seems like the best way is to save location.hash prior to showing the login page, and then set it after leaving it.
location.hash = ... changes to the page correctly after login, but the ionViewWillEnter and ionViewDidEnter triggers are not fired.
Whilst I think this bug needs fixing, a nicer solution than manually setting location.hash would be to store location.hash and have eg a rootNavCtrl.setFromHash() function which would parse out the segments and any data associated with each page under it; and/or the ability to take a full status of the path to the current page (controllers and paramater settings) and replicate it later.

Hello, thanks for opening an issue with us. Would you mind posting a minimal repro case we could use to reproduce this issue?

3.6 appears to fix this once manually patched the issue in https://github.com/ionic-team/ionic/issues/12526


formatting: Reformat files from allowlist

Verible formatter version: v0.0-1213-g9e5c085b

Hi @ldalek-antmicro. Thank you for the contribution and of course your work on improving Verible.
We (myself, @dsandrs, @tomroberts-lowrisc, and others) had a discussion recently about formatting-only changes and the main conclusion was that due to concerns about churn, review overhead, and the risk of inadvertently introducing functional changes we should hold off on any such of mass reformatting effort for the time being.
This PR has had plenty of review so at this point I've got no particular objection to it landing, but beyond that I think improvements to our pre-commit testing are the best area to focus efforts (i.e. ensuring any incremental changes and newly added code are formatted according to our styleguide).
To that end, do I understand correctly that the in-tree verible formatting script currently doesn't support limiting reformatting to just the changed lines? From a quick look it seems verible does support --lines as clang-format does, so a similar approach to clang-format-diff could be used?

CC @hzeller @tansell

Hi @asb
Thank you for your feedback! We understand your concerns, we will try to come up with some ideas on how we can proceed in a way that will allow us to avoid the overhead and the risks you mention.
Your suggestion about trying to reformat only the changed lines sounds great, we will experiment a little bit to see in what way our work can be the most useful for OpenTitan.


Make autocomplete scope configurable

It should cycle only though names in current channel.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

I'm not sure it should only be for names in current channel. Arguments?
Don't say "the current client does it this way"

One obvious argument would be that the person can't see you typing their name if you are not in channel.
But if you want to talk behind someones back I guess tabbing their name makes it easier^^

dude, it's obvious, I'm in e_clan channel and I wanna ping Blodir as an example, he's the only one with name starting on "B" but if I type "B" and hit TAB it cycles though BasherV2, Bespin, biass, Blackdeath, blitz_shootza and then finally Blodir. It should always cycle only though ppl that are in the channel you're trying to type in

It's not obvious to me. That's just the behavior you're used to. As Androish said, maybe you want to refer to somebody else. Check out slack, there too you can cycle through all names, even in PMs.

mkayyyyy, then I'm used to that so much that I find this feature a bug :D

How about changing the priorization here? Change the list of names that get auto-complete so, that matches in the current scope (channel) are first, everything else follows later?
By this, users like @speed2CZ would be happy because Blodir would be the first hit, but one can still refer to BARandomNameWhatever by pressing Tab again.

Current behavior is fine


NASM (Assembly AT&T Intel x86 and x86_64 syntax) CodeMirror mode

Hi. Thanks, but we aren't accepting new modes anymore. The recommended way to publish a mode is to create a separate package, say codemirror-mode-nasm, and publish that to NPM.


Unable to install columnify on MacOS w/ go get

Following the README instructions, i'm trying to install columnify with go get. But i'm getting the following error:
⇒  GO111MODULE=off go get github.com/reproio/columnify/cmd/columnify
package github.com/vmihailenco/msgpack/v4: cannot find package "github.com/vmihailenco/msgpack/v4" in any of:
        /usr/local/Cellar/go/1.14.4/libexec/src/github.com/vmihailenco/msgpack/v4 (from $GOROOT)
        /Users/XXX/.go/src/github.com/vmihailenco/msgpack/v4 (from $GOPATH)

I had the same error with GO111MODULE=off. It seems that GO111MODULE=on can build columnify binary into $GOPATH.
$ GO111MODULE=on go get -v github.com/reproio/columnify/cmd/columnify

I found some issues in vmihailenco/msgpack repository. The author suggests using Go Modules.

https://github.com/vmihailenco/msgpack/issues/237
https://github.com/vmihailenco/msgpack/issues/251

I think this is README bug, so I will send PR.

thanks @t2y 🙇

Worked for me-->

Run go get github.com/vmihailenco/msgpack to install the master branch (v5) into your GOPATH.
cd ~/go/src/github.com/vmihailenco/msgpack
git checkout v4 to switch to the v4 branch
cd - to return back to the directory you were working in before.
When using HCL, import it still as github.com/vmihailenco/msgpack instead of as github.com/vmihailenco/msgpack/v4

These above steps resolved the import error for me~


Update xcodeVersion

⚠ Merger after Xcode 15.2 has been installed on builders and the new builder has been deployed ⚠

Xcode 15.2 is now installed on mb6.


🚸  Allocation helpers

Goal: Make Allocation easier to setup and modify.
Closes #63

Rebase

Rebase

Could you also squash? An approval now will be discarded anyway.
@matthiasgeihs


New application landing page

Dupe of applicant landing page. Users will see same thing when they log in regardless of new or returning application. #31


Provide a new Nuget build

I've managed to build it! (x64 only, MSVC2010 or MSVC2013) :) Does anyone need binaries or instructions?

Hi Rast1234, please do post any notes you made along the way, and please offer any changes you had to make as a pull request.

@oliverbock ok i'll do it in the nearest future.

@oliverbock please see #19


Translation to Spanish Lubridate

I'm working in the translation to Spanish of Lubridate Cheat Sheet (I'm using the current Cheat Sheet). I will be uploading the new files during the next weeks.

Thank you for the translation! Closing this now because it looks like it's resolved.


Female placed as the default gender; Create a new gender representing missing info/third gender

Describe the bug
In the raw database, genders of some patients is missing, and the same is reflected in the cluster by displaying it as female. Also, it may not be possible for the crowd-sourced data to get gender of all the patients. Also, it gives wrong visualization inference, that more females are infected by the virus.
To Reproduce
Steps to reproduce the behavior:

Go to COVID-19 crowd-sourced database.
Note down the patient number(s) with missing gender.
Go to COVID-19 cluster.
Search for the noted patient number(s).
See error.(Gender misrepresentation)

Expected behavior
It's expected that if the gender is unknown, a grey ? clip-art maybe shown with 'N/A' in the gender field to compensate for the missing information.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser Chrome
Version 80.0.3987.149 (Official Build) (64-bit)

Additional context
Also a third gender can be added as "others", beside "N/A".

The data coming in seems to have the gender set as "female".
Reference: https://api.rootnet.in/covid19-in/unofficial/covid19india.org
Ideally, shouldn't we be correcting the API data?

Yeah, this looks like an API issue.

Can this be closed now?

Can this be closed now?

The API issue has been resolved.
However the cluster is visually the same, where 'unknown' genders are represented with the female graphic. Need graphics to be created and replaced for the "unknown" gender, similar to the existing male and female graphics. (An image with ? on a gray-colored face would be good, else only a question mark would do well.)


C2ME mod

Mod Name
C2ME mod
Mod Homepage
https://github.com/RelativityMC/C2ME-fabric/actions/runs/6236508664
Describe the bug
Crash server
Steps to reproduce
I found that the server crashes during startup and decided to consider the likely problem, it turned out that if you disable [noTickViewDistance]
# (Default: true) Whether to enable no-tick view distance
enabled = "false"
In the mod config, the server will be able to run, but this is an important function for the server, I will be very grateful if you solve it in the future, thanks.
Logs
https://gist.github.com/DonorKrovi/39f8306c6a70a83ebd8a27f0bc41e7ec
Additional context
No response

Duplicate of #88


Migrations observability - new metrics

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug

/kind enhancement
What happened:
As one of the tasks to improve Kubevirt observability, we should make VMI migrations more observable.
We can start by creating metrics to be exposed to a Prometheus server.
We can create some high-level metrics for the following use-cases:

As a cluster-admin, I'd like to monitor time spent on migrations to be alerted when migrations start to get slower.
As a cluster-admin, I'd like to monitor migrations success rate, so I can be alerted when an unacceptable percentage of migrations are failing.
As a cluster-admin, I'd like to be alerted when any migration fails

The migration success rate can be accomplished by exposing:

Total amount of migrations
Total amount of failed migrations

Successful migrations would be: total_migrations - failed_migrations
Success rate would be: (100 * total_migrations - failed_migrations) / total_migrations
Alert for migration fail would be: delta(failed_migrations[5m]) > 1

this would alert if any migration failed in the last 5 minutes

Time spent with migrations could be monitored with whether a histogram or a summary, as they both could give us information about total amount of migrations, sum of time spent on migrations and we can work with quantiles with both of them
For more detailed observability, like what exactly migration failed or where exactly are they spending more time, we should cover with other areas of observability like logging and tracing

/assign

Reading prometheus' docs about Histogram vs Summary, I think that a Histogram would be the best approach for the time spent metric

Note the importance of the last item in the table. Let us return to the SLO of serving 95% of requests within 300ms. This time, you do not want to display the percentage of requests served within 300ms, but instead the 95th percentile, i.e. the request duration within which you have served 95% of requests. To do that, you can either configure a summary with a 0.95-quantile and (for example) a 5-minute decay time, or you configure a histogram with a few buckets around the 300ms mark, e.g. {le="0.1"}, {le="0.2"}, {le="0.3"}, and {le="0.45"}. If your service runs replicated with a number of instances, you will collect request durations from every single one of them, and then you want to aggregate everything into an overall 95th percentile. However, aggregating the precomputed quantiles from a summary rarely makes sense. In this particular case, averaging the quantiles yields statistically nonsensical values.

avg(http_request_duration_seconds{quantile="0.95"}) // BAD!

Using histograms, the aggregation is perfectly possible with the histogram_quantile() function.

histogram_quantile(0.95, sum(rate(http_request_duration_seconds_bucket[5m])) by (le)) // GOOD.

Furthermore, should your SLO change and you now want to plot the 90th percentile, or you want to take into account the last 10 minutes instead of the last 5 minutes, you only have to adjust the expression above and you do not need to reconfigure the clients.

I do have some doubts about the migration architecture though. I'm not sure which component should be exposing these metrics.
It is a metric that is shared between all nodes, so they definitely won't be exposed by virt-handler.
I think all metrics will be updated exactly when a migration ends at failed or completed state, but I don't know where that happens
Could anyone share some thoughts here? :grimacing:


fix: deduplicate peripheralDevice-methods

This PR fixes the old issue of the peripheraldevice-methods being duplicated between Core and the server-core-integration library.
I've moved and merged the API definitions into the server-core-integration package and updated the references.
I also found a few discrepancies where they didn't match, and fixes those as much as I was able to.

Other information:

Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[ ] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

I'm closing this, as rebasing these changes will be too much work. I'll probably open a new PR later..


Feature Request - Add more than one point

Would it be possible/plausible to enable being able to plot more than one location at a time?

Yes, and this is a good idea because we could account for multi-hops. Would require front-end work.
Open to PR's, I have limited time over the next couple weeks to try this.


Media controls are not hidden if z-index is negative.

On iOS 9.3.5 and above, if the video elements' z-index is set to a negative value, the media controls become visible. (Play button at the center of the element.)
If the z-index is set to a positive value, the controls become hidden, and the video elements go above every other element of the page (as expected, if I've understood correctly).
On iOS 9.2 the problem not occurs.
The video element has the following css attribute:
::-webkit-media-controls {
display:none !important;
}

This is due to a change in Mobile Safari.  You want to set these:
/* hide video element controls */
video::-webkit-media-controls {
    display: none !important;
}
video::-webkit-media-controls-start-playback-button {
    display: none !important;
}

Thanks for the quick response, it works!


Place ISR to instruction-RAM

This should make the call to the ISR safer.
Using indirection via ARDUINO_ISR_ATTR in order to respect configuration.

I guess this merge is obsolete since #116 is merged, isn't it?

Affirmative 👍

Has been cooped by #116.


[PreTest] Technical proposal for the checksum check of the three-party dependencies

1. Does this PR affect any open issues?(Y/N) and add issue references :

[ ] N
[x] Y

2. What is the scope of this PR (e.g. component or file name):
/docs/proposal

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[x] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

I have read the CLA Document and I hereby sign the CLA

recheck

Hi @zong-zhe @Peefy,
I have completed the pre-test for the technical proposal on checksum checks for third-party dependencies. I have explored several package managers to understand the approach for implementing checksum verification for dependencies.
Please review my work and suggest any improvements.
Thanks!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

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

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
3126

Relevant Lines:
7759

💛  - Coveralls

Hey @zong-zhe,
My updated proposal is nearly complete, and I will commit the changes once this discussion is addressed.
Thank you for your patience, and I apologize for the delay.

Hey @zong-zhe,
I have pushed the changes based on the review. Could you please review them and let me know if I am heading in the right direction? Also, please address this doubt: link to discussion, so I can make the necessary changes to the proposal. Additionally, we need to explore more about Merkle trees, as I am not familiar with their implementation.
Thank you.

Hey @zong-zhe, please take a look when you have a moment. I'm happy to incorporate any changes if needed.

@zong-zhe, I’ve added the design for the checker module. PTAL..

Hey @zong-zhe, made some modifications based on reviews, PTAL

Could you share some of the goals we have while implementing checksum verification, so I can explore possible solutions?
cc: @zong-zhe

Could you share some of the goals we have while implementing checksum verification, so I can explore possible solutions? cc: @zong-zhe

Hi @NishantBansal2003 😄
It is mainly divided into three parts:

Complete the Checker design doc, and then develop the Checker.
Use the Checker in the kpm and pass the flag to control whether the sum check function is enabled.
Adjust the existing three-party dependency and add the check sum.

Hi @zong-zhe,
I've made some modifications to the code. Could you please review it and let me know if I'm heading in the right direction? I’d like to proceed further with the implementation of getTrustedSum.
Thanks!

Hey @zong-zhe, please take a look when you have a moment.

Hi @NishantBansal2003 😄
You can first propose a PR to implement Checker. As a solution doc, this PR will not be merged now, because there may be adjustments in the design of the solution in the future. After the completion of the whole work, we will merge this PR.

I’d like to proceed further with the implementation of getTrustedSum.

Hey @zong-zhe, since the implementation of getTrustedSum is incomplete, should I still submit the current code as a PR, considering that any modifications can be made in the future before merging?

PTAL @ https://github.com/kcl-lang/kpm/pull/470, I think we can proceed with further discussion.

Hey @zong-zhe, I have added the design for integrating the checker into kpm's workflow, PTAL. Also, I need some help with an issue I encountered while testing the changes locally: https://cloud-native.slack.com/archives/C05TC96NWN8/p1727073915893189?thread_ts=1727069902.233579&cid=C05TC96NWN8

I need some help with an issue I encountered while testing the changes locally: https://cloud-native.slack.com/archives/C05TC96NWN8/p1727073915893189?thread_ts=1727069902.233579&cid=C05TC96NWN8

I understood the difference from -
https://github.com/kcl-lang/kpm/blob/b4d396ac2c5a635b780873810a53d3b67d0ab7e3/pkg/opt/opt.go#L209-L267
But I'm not sure how to handle the following case:
Currently, my implementation raises an error if dep.Source.Oci is nil. I want to ensure that we handle both scenarios correctly during integration. Since dep.Source.Registry.Oci gets filled when downloading using the OCI reference, but not dep.Source.Oci, should I adjust the integration code to also populate dep.Source.Oci when dep.Source.Registry.Oci is present? Or do you have a better approach to address this?

I also want to know some package example, like k8s:1.28, do not have a sum field in their manifest, resulting in a manifest like map[org.opencontainers.image.created:2023-09-28T07:52:58Z]. This leads to checksum failures in some other packages, such as istio, kwok etc. How should I fix this issue?
This issue actually reminds me of this:

3. Adjust the existing three-party dependency and add the check sum.

So, should I adjust the existing third-party dependency to include the checksum in its manifest? If so, how can I do that?

Hey @zong-zhe,
I have also added a rough code design for integrating the checksum checker into the KPM workflow for each KCL command. This implementation is based on my understanding, and I’m open to any modifications that could make the integration code more robust and isolated. Please take a look whenever you have time. Thanks!

Currently, I am considering overwriting the package artifact and updating the manifest with the given version using PushWithOciManifest in the KPM code. I will push the same package and updated manifest again and try to bypass the condition that returns an error if the package version already exists. Is there any other way to do this?

Currently, I am considering overwriting the package artifact and updating the manifest with the given version using PushWithOciManifest in the KPM code. I will push the same package and updated manifest again and try to bypass the condition that returns an error if the package version already exists. I searched for the oras package to update only the manifest but could not find a solution. That's why I am thinking of this approach. Is this okay, or is there a better way that I should research? Edit: Maybe we can use crane package, which I found in some discussions, as it is specifically designed for this use case. cc: @zong-zhe

Hi  😄 oras provides this function used to adjust the manifest in https://oras.land/docs/commands/oras_manifest_push, you can use this function to adjust the manifest, you can look at the code implementation in oras, and then write the corresponding go program through the oras API, and write the corresponding test cases, making sure that you only adjust the sum field of the manifest.

Hey @zong-zhe, could you review this code design whenever you have time? Here is the link:
https://github.com/kcl-lang/kpm/pull/431/files#diff-1de6327653b8b200fc992a0aaec4debf824688fae1795edb5534cd2f93fd324c
This design integrates the checksum checker into the KPM workflow for each KCL command. Please suggest any changes for better implementation and integration. I will open another PR for this after successfully merging the following one:
https://github.com/kcl-lang/modules/pull/243


Ndeksa

Issue #, if available:
NA
Description of changes:
Modified validated hardware list
Testing (if applicable):
NA
Documentation added/planned (if applicable):
Modified EKS-A BM qualified hardware list
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

lgtm

/assign @chrisnegus

/cherry-pick release-0.16

/cherry-pick release-0.17


Source the data and perform Data preprocessing

Data preprocessing performed.
Temporary dataframe created for specific purposes.

Data sourced from kaggle.
features were studied.
preliminary exploratory data analysis performed including line graphs, scatter plots

Data preprocessing is done, and EDA is performed on the dataset.


When library will be production ready ?

Hi, we are planning to integrate with nats by websockets and want to use this library, can you suggest when it will be ready for production usage ?

The library is ready - awaiting for the server to release.
Until the server releases there's a possibility that the API could have very small modifications as works on the JetStream wrappers are in progress and may introduce slight changes to allow for a better experience on JetStream. With that said, if you are running your nightly/self-built server you should be OK. Just make sure to clamp on versions and upgrade only after understanding the updates by looking at the commit logs - semver won't be enforced until after the server releases.
One thing that will be changing is how to specify ws:// support, right now it has a custom property that probably will be folded with the tls option that already exists.


fix: do not clean ec resource and service info when handler ecm close event

…se event(#4192)

What is the purpose of the change
Do not clean ec resource and service info when handler ecm close event
Related issues/PRs
Related issues: #4184
Related pr: #4452
Brief change log

Do not clean ec resource and service info when handler ecm close event.

Checklist

[x] I have read the Contributing Guidelines on pull requests.
[x] I have explained the need for this PR and the problem it solves
[x] I have explained the changes or the new features added to this PR
[x] I have added tests corresponding to this change
[x] I have updated the documentation to reflect this change
[x] I have verified that this change is backward compatible (If not, please discuss on the Linkis mailing list first)
[x] If this is a code change: I have written unit tests to fully verify the new behavior.

Ec service instance(dss-service:46156)  record reserved after corresponding engine manager stopped manually.

Thank you


greenlight compilation error

Trying to debug my green wallet with lightning and I would like to use the sdk-cli. However, I get the following compilation error
➜  sdk-cli git:(main) ✗ cargo run
   Compiling gl-client v0.2.0 (https://github.com/Blockstream/greenlight.git?rev=e38a37613da7558c853f24be700c193f194a6bc9#e38a3761)
error[E0412]: cannot find type `DelexpiredinvoiceRequest` in module `cln`
    --> /home/vincent/.cargo/git/checkouts/greenlight-e41cf58dbd255ee0/e38a376/libs/gl-client/src/signer/model/mod.rs:41:28
     |
41   |     DelExpiredInvoice(cln::DelexpiredinvoiceRequest),
     |                            ^^^^^^^^^^^^^^^^^^^^^^^^ help: a struct with a similar name exists: `DelinvoiceRequest`
     |
    ::: /home/vincent/github/work/breez-sdk/tools/sdk-cli/target/debug/build/cln-grpc-7a25774ebd2ceb74/out/cln.rs:1800:1
     |
1800 | pub struct DelinvoiceRequest {
     | ---------------------------- similarly named struct `DelinvoiceRequest` defined here

error[E0433]: failed to resolve: use of undeclared type `DelexpiredinvoiceRequest`
  --> /home/vincent/.cargo/git/checkouts/greenlight-e41cf58dbd255ee0/e38a376/libs/gl-client/src/signer/model/cln.rs:27:62
   |
27 |     "/cln.Node/DelExpiredInvoice" => Request::DelExpiredInvoice(DelexpiredinvoiceRequest::decode(p)?),
   |                                                                 ^^^^^^^^^^^^^^^^^^^^^^^^
   |                                                                 |
   |                                                                 use of undeclared type `DelexpiredinvoiceRequest`
   |                                                                 help: a struct with a similar name exists: `DelinvoiceRequest`

error[E0599]: no method named `bolt11` found for reference `&cln_grpc::pb::PreapproveinvoiceRequest` in the current scope
   --> /home/vincent/.cargo/git/checkouts/greenlight-e41cf58dbd255ee0/e38a376/libs/gl-client/src/signer/resolve.rs:102:40
    |
102 |                     l.invstring.0 == r.bolt11().as_bytes()
    |                                        ^^^^^^-- help: remove the arguments
    |                                        |
    |                                        field, not a method

Some errors have detailed explanations: E0412, E0433, E0599.
For more information about an error, try `rustc --explain E0412`.
The following warnings were emitted during compilation:

warning<EMAIL_ADDRESS>Using default NOBODY cert.
warning<EMAIL_ADDRESS>Set "GL_CUSTOM_NOBODY_KEY" and "GL_CUSTOM_NOBODY_CERT" to use a custom cert.

Probably is it a greenlight issue but idk how it is import inside the SDK-cli, so I will start reporting the problem from here

Thanks @vincenzopalazzo
Is it on the main sdk branch? We recently update the gl-client dependency.

@roeierez I did just a git clone, so I assume it was main

I just ran the cli on master and it works for me.
From the compiler log:
Compiling gl-client v0.2.0 (https://github.com/Blockstream/greenlight.git?rev=e38a37613da7558c853f24be700c193f194a6bc9#e38a3761)
I see we also use the same revision.  I will try to clone a new repo.

Also cloned a new repo and seems to work. Could it be that the cln-grpc dependency is not updated in your run?
What is the version of this dependency in cargo.lock ?

MH you are right now I clone again inside a tmp dir and it is working
Probably I did not clone the repo but I did a git pull, testing right now.
Ok on a fresh clone I have no error, but on already cloned repo after git pull I get the same error
but I am on the same commit 5d022f719308531da0ebb55098c3cee031b5c7af
Strange

@roeierez I reproduce the same error in the fresh clone by running cargo install --path .
Are you able to reproduce?

I also get this error now when running cargo install --path .
But I don't get this error if running cargo build or cargo run

We've seen this in the past:

https://github.com/breez/breez-sdk/issues/584#issuecomment-1795522797
https://github.com/breez/breez-sdk/issues/969#issuecomment-2104417144

The root cause is described in detail here.
It essentially happens because most libraries use loose semantic versioning ranges for their dependencies. When they release, the release is tested with a specific set of dependency versions. However, soon after this release, a dependency could release a newer version that fits in the original library's Cargo.toml semantic versioning range. This means new apps trying to integrate this library will use the newest dependency version (e.g. dependency of library), which may break the library itself because it was not tested with that version.
The fix has been to find these problematic transitive dependencies and pin them to a valid working version.
(The reason it behaves differently when you git pull vs when you setup a fresh project is, on git pull, cargo will try to use the existing Cargo.lock, which may have an older working transitive dependency version. On a fresh new setup, cargo always tries to fetch the newest transitive dependency.)
IMO the goal now is to find out which dependency changed since the latest working release and now.
Looking into it, will post an update when I find something.
(P.S. @roeierez this is the reason for the proposed PR https://github.com/breez/breez-sdk/pull/971)

@vincenzopalazzo this is being addressed in #1014 .
Until that's reviewed and merged, can you please try to add
# Pin problematic transitive dependencies
cln-grpc = "=0.1.8"

to your Cargo.toml then try again?

@vincenzopalazzo the issue is fixed now. Please do cargo clean, then recompiling the project should work.


Mount MappedDirectories under full path ContainerPath

Previously, ContainerPath specified a single path element and was always
mounted under /tmp. Now, it is a full path just like
MappedVirtualDisks.

@beweedon,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Experimental: Install devDependencies ignoring engine constraints

That how yarn --production should work but it does not.

All good expect for compat test of jest 30, that could be restored as conditional one


Handle different max_insert_rows settings in insert command

Interesting problem for batch uploads: Datasette has a max_insert_rows setting defining the maximum number of rows that can be uploaded at a time.
Options:

Assume it's left at the default (100) and always use the default. But what if someone has set the default lower?
Read that value from Datasette before starting any data upload - maybe once at the start of a batch upload session.
Try an upload, and if there's an error THEN read the setting - or have Datasette return that setting in JSON as part of that error message.

I'm going to implement the 100 default for the moment, then revisit this later.

Originally posted by @simonw in https://github.com/simonw/dclient/issues/13#issuecomment-1641318500

The easiest option would be to default to 100 and allow the user of dcloud insert to set a --batch-size 50 if they need to.

Just went with --batch-size. 0d4e5cfc73e8f7b07018fa5d5cab51161a27a67c


PDF of .NET "Regular Expression Language - Quick Reference" is Failing

Hi.  The PDF download of the quick reference failed today (20-Dec-2020) on several attempts just before 12:54 Mountain Standard Time.  The resulting download is corrupted and cannot be opened.
The download link is:
https://download.microsoft.com/download/D/2/4/D240EBF6-A9BA-4E4F-A63F-AEB6DA0B921C/Regular expressions quick reference.pdf
Kind regards,
Vic Wheeler

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

ID: 16d4698e-4220-c422-a554-8a481803d240
Version Independent ID: f0e3b82a-8d81-ad9e-97fc-efb92f7d0d38
Content: Regular Expression Language - Quick Reference
Content Source: docs/standard/base-types/regular-expression-language-quick-reference.md
Product: dotnet-fundamentals
GitHub Login: @adegeo
Microsoft Alias: adegeo

I just tested and it's working for me. Please verify that it is still broken for you. I'm going to close the issue but feel free to respond.

Hi!  Thank you for testing it.  I just tried re-downloading it and now Adobe Reader opens it successfully.  However, the .DOCX format shows that page 3 has a couple of sections in it that are missing from the PDF I just downloaded.  I mention it as it might be relevant.  Hope this helps.

False alarm:  The last "section" is "To get ... Use Regex API..." in the PDF and that is also the last section in the .DOCX format, so I will assume that is simply how the content "flowed into" the columns.


feat: handle single quotes in version-rb

This fixes the linked issue by applying a small fix to updaters/version-rb.ts.
Fixes #1018 🦕

@hschne thanks for the patch 👌


add detailed format parameter doctsrings to DateWidget and TimeWidget

widgets.DateWidget and widgets.TimeWidget both takes additional format parameters but they don't have docstrings on default behavior like on widgets.DateTimeWidget

Thanks - will merge when I can fix tox issues


Update fs2-core to 3.1.4

Updates co.fs2:fs2-core from 3.0.6 to 3.1.4.
GitHub Release Notes - Changelog - Version Diff
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!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2", artifactId = "fs2-core" } ]

labels: library-update, semver-minor

Superseded by #109.


Partial fix for desync for TileEntities with fluid tanks

Minecraft version & Mod Version:
Minecraft: 1.12.2
Forge: <IP_ADDRESS>94
Cyclic: 1.15.15 (dev version of master branch commit 2605b8490eff291a7fa6617dd425681b7da1f15e)
For testing I added Pneumaticraft Repressurized 1.12.2-0.6.8-219
Single player or Server:
Single player
Describe problem (what you were doing / what happened):
Put down a hydrator and pump water into it through fluid transfer from other mods. In my dev environment I used the liquid hopper from Pneumaticraft but I also tried conduits from EnderIO and other mods. The hydrator gets filled with water, but the GUI still shows an empty tank. After re-logging and opening the GUI again, it shows that the internal tank actually has water in it.
Possible Solution
I checked out the code and found, that there has already been a fix for this with the normal tank from Cyclic, since it now uses the FluidTankFixDesync class that does the network update packets. This should be used everywhere the FluidTankBase class is currently used and a GUI is involved. Furthermore the Implementation of PacketFluidSync should not test for TileEntityFluidTank but for TileEntityBaseMachineFluid that already defines the update method.
I made the changes in this pull request and switched the hydrator to use the correct tank implementation. This is only a partial fix since all tile entities with GUIs should use that implementation of the tank. Maybe the FluidTankBase class shouldn't exist anymore and be merged into a common fluid tank implementation instead?

Yeah looks good! I dont really see the desync in single player but what you describe makes perfect sense to me!


Rename TimezoneClient to TimeZoneClient

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-maps-timezone


Cypress Test Retries

Hi! Thanks a lot for creating this library.
In autotests to protect against flaky (i.e., unreliable), when the test sometimes fails due to unpredictable conditions (for example, temporary failures in external dependencies, random network errors, etc.) added retries from Cypress.
If the test is repeated, I get Error: Snapshot /name-of-snapshot-actual.png does not exist. Because Cypress adds to the file name the number of the attempt with which the test passed. It turns out for example name-of-snapshot-actual (attempt 3).png
Is it possible to modify the names so that it compares the file by the exact match of the name, if there is none, it checks the same name + any characters. Like cy.compareSnapshots('name-of-snapshot-actual*')
Or tell me if someone solved a similar problem in their project. Thanks in advance for your reply

UPD: used old version of cypress-visual-regression 1.7.0
After updating to 2.0.1 it was decided


Align readme format

does github do autosquash on merge?

Don't think so, but it can do squash merges.


Feedback from TAs at end of 2019

need a lecture in the second half that can help give all teams the same
information
but need to customize interaction is also crucial
if we had a lecture, we could use it to explain one or more of the webpages
in class you could better explain the expectations of the reports and
presentations
office hours are not effective because teams that need it don't come
all of greg's groups came multiple times
ian: teams would come if he invited them
greg really likes the office hours and working closely with the students
greg felt his groups felt he wasn't the grade enforcer ut was there to truly
help them succeed
make sure that we brand the reports so that when they get passed around for
broader visibility
we need a drawing template for the course
ian was annoyed that they choose gigantic paper sizes and then would make it
smaller when he gave the feedback
maybe we need them to print the drawings
greg says you can use a stylus on the canvas speed grader feedback
in the spring quarter we need more deadlines to complete certain milestones
and if they don't complete it it will drop their grade
we don't really hold them accountable for their objectives and milestones
some groups said that they wish they'd done one more prototype
use the meetings to make next weeks objectives
how do we manage the aio to gantt chart connection and that O's are A's for
next week
if you miss an objective you have to come to office hours
could have points for meeting objectives and if they miss the objectives they
can get the points back if the come to office hours
noone complained about the report descriptions not matching the rubric, but
did complain last year
doesn't seem like they reuse the needs and specs (ian)
there was a lot of fatigue on needs and specs, they wanted to get on with
their design, some groups aborted the evolution of their needs and specs
we get caught up in making the machine work in the spring and the needs and
specs get lost for a while
software heavy groups had trouble mapping the product design to the product
design book
do a word limit on the reports instead of a page limit because they remove
all the pictures
we need someone from technical writing on staff of the class
writing is not a prerequisite for this class
could have them create a primary graphic that shows their machine with bubble
pull outs with all the different functional elements
first half of class they make their design, second half the justify it, they
have to fix the first.
first half seems slow, they could start on the design right after concepts
they are not knowing that they can start cutting metal earilier in the spring
quarter
need to pepper them with some rewards earlier on (rewards being the fun parts
of doing design)
greg has an article "the concept of a model" from the human mind literature
maybe they should have to build a cardboard prototype
could have  a milesteon: got ot make a model: a carboard thing, a simulation,
a ..., a ...

It could be useful to have the physically print their drawing package (it would reveal many of the readability and paper size issues).


My host doesn't have "gmp_import()". Workaround?

Error:
Fatal error: Uncaught Error: Call to undefined function gmp_import() in .../vendor/lyquidity/requester/src/Asn1/Util/BigInteger.php:67
This error message doesn't appear on my home computer.  So I gather the host disabled "gmp_import()" (may be for security reasons...).
Is there a way to accomplish the same with another function (perhaps many) ?

The same question was asked here.  It's not an issue I have any interest resolving.  The code was created for my OCSP implementation and made public in case other wish to use it.  Your choices are to:

find another implementation,
create an implementation of BigInteger that can work in your environment,
install GMP, or,
if your host will not permit that, use a different provider

Good luck.


[Android][LogBox] Set windowTranslucentNavigation to false

Summary
This fixes https://github.com/facebook/react-native/issues/29397. Without this, apps that specify android:windowTranslucentNavigation draw the LogBox buttons underneath the soft navigation bar, making the buttons unpressable.

Before
After

Changelog
[Android] [Fixed] - Set LogBox windowTranslucentNavigation to false
Test Plan
I tested this change on the repo I set up to reproduce the issue. I set it up to build ReactAndroid from source and then edited node_modules/react-native/ReactAndroid/src/main/res/devsupport/values/styles.xml directly.

@mdvacca, I presume something went wrong with the land... is there any way I can help? Failing test cases?

Everything looks good, this is landing now.
@Ashoat once this lands, do you want to put a pick request in for 0.63 here: https://github.com/react-native-community/releases/issues/200?

Yeah, will do!


Increasing number of openfiles in contrail-api

increasing the limit on number of openfiles in contrail-api.

please test this

Test FAILed

please test this

Test PASSed

syslog snippet from ct1 server of the build:
Jul 31 08:50:14 ct1-testjenkins-puppet-rjil-gate-1969 puppet-user[30279]: (/Stage[main]/Contrail::Config/File[/etc/contrail/contrail-api.conf]/content) content changed '{md5}159106225a4fe94d59ffd4e672446697' to '{md5}32aa51a7e5df7e9eee705af577ee8e7f' Jul 31 08:50:17 ct1-testjenkins-puppet-rjil-gate-1969 puppet-user[30279]: (/Stage[main]/Rjil::Contrail::Server/File[/etc/init/contrail-api.conf]/content) content changed '{md5}8c02c12b2cc3f676c90b556ed77abfe6' to '{md5}333b9fd0d03bd601f40953d8408a9097' Jul 31 08:50:19 ct1-testjenkins-puppet-rjil-gate-1969 puppet-user[30279]: (/Stage[main]/Contrail::Config/Service[contrail-api]/ensure) ensure changed 'stopped' to 'running'

@akash1808 this may no longer be valid when we upgrade to v2.1 as these files will no longer exist.
So lets wait for the upgrade, and then see if we are still facing the issue after the upgrade.

@vpramo i was also thinking the same. Though we can just make sure that it is there if required before upgrade.


ci: merge aarch64 build and rename builds

@drakon64, this is what I'm suggesting in my comment at https://github.com/lilyinstarlight/nixos-cosmic/pull/120#issuecomment-2122964067

LGTM


Use cmake interface library

This PR uses cmake's interface library feature:

An INTERFACE library target does not directly create build output, though it may have properties set on it and it may be installed, exported and imported.

This makes it easier to include the header only library in a cmake project. After using add_subdirectory on the cxxopts directory, one simply needs to include cxxopts in their target_link_libraries, which will allow the user's target to inherit the properties of the cxxopts header library (see changes to example and test).
Two things:
(1) I updated the cmake required version to 3.1, but this may not be necessary (I believe it is necessary for target_sources, which is not available in 2.8.12, but not sure if you need target_sources for this to work).
(2) The Unicode support is confusing. I did a git grep and couldn't find any other use for CXXOPTS_LINKER_LIBRARIES. CXXOPTS_USE_UNICODE_HELP would also make more sense as an option instead of a cached variable, and then target_compile_definitions can be used to enable CXXOPTS_USE_UNICODE in the header. With this PR making it an interface library, other projects that link with cxxopts should inheret CXXOPTS_USE_UNICODE as well. For now I left this untouched since I don't know if I'm missing something important.

This change is

This looks like a good change in general, but see comments below.
Updating the minimum cmake version is fine.
I think the Unicode support is broken at the moment, feel free to fix it too. What should probably happen is that you ask for Unicode, and anyone who uses the library should get both -DCXXOPTS_USE_UNICODE, and any required link libraries. See 8d7c4ea43e25a3187662506040b826a167e7110c, and then 774d455dc720dc0d90d4d0bbdc57bde0ccb9f51d which I think broke linking.

Reviewed 7 of 7 files at r1.
Review status: all files reviewed at latest revision, 4 unresolved discussions.

CMakeLists.txt, line 51 at r1 (raw file):

endif()

add_library(cxxopts INTERFACE)

Is there any reason that you can't include the sources in add_library, that way you won't need target_sources?

CMakeLists.txt, line 54 at r1 (raw file):

target_sources(
  cxxopts INTERFACE
  ${CMAKE_CURRENT_SOURCE_DIR}/include/cxxopts/cxxopts.hpp

I'm ok with moving the headers to include, because reorganising the source has no external effect, and to be consistent with the rest of the world it's probably the right thing.
But I don't think there's any point putting cxxopts.hpp under cxxopts, because it is a single file. If there were multiple files I think it would make sense for them to live in their own directory. For example we have /usr/include/gmp.h, but boost has its own directory. This also breaks backwards compatibility.

src/CMakeLists.txt, line 23 at r1 (raw file):

if(CXXOPTS_BUILD_EXAMPLES)
    add_executable(example example.cpp)
    target_link_libraries(example PRIVATE cxxopts)

What does PRIVATE do here? I don't understand the docs on this one.

src/example.cpp, line 27 at r1 (raw file):

#include <iostream>

#include "cxxopts/cxxopts.hpp"

As mentioned above I don't think we should be changing the relative include path.

Comments from Reviewable

I've fixed the Unicode stuff. Can you take a look at a04284fc2c650f52e0d09cbc6c6d688fbf64cca9 see if that affects this PR? I suspect that yours is the right behaviour, but at least now it goes from one working setup to another, instead of being unsure about something that's broken.

Review status: all files reviewed at latest revision, 4 unresolved discussions, some commit checks failed.

Comments from Reviewable

Couldn't figure out if there was a way to respond to your comments on reviewable, so please find some responses here:

Forgot that I removed install(FILES cxxopts.hpp DESTINATION include) from src/CMakeLists.txt. Would you like me to add it back to the root CMakeLists.txt?

You should be able to sign in to reviewable with github and reply to the comments there.
Can you rebase on top of master and force push instead of pulling my new commits into your branch? I can't tell what has changed now, and it has also messed up what reviewable thinks of the diff. Plus there are merge conflicts that I don't think should actually be there.

Forgot that I removed install(FILES cxxopts.hpp DESTINATION include) from src/CMakeLists.txt. Would you like me to add it back to the root CMakeLists.txt?

Yes that would be good.

It shouldn't affect the PR, I don't think, but here's where INTERFACE/PUBLIC/PRIVATE come in. If I target_link_library with cxxopts (instead of example, as you're doing), and specify PUBLIC, this means that the library is linked with cxxopts and any executables that link with cxxopts. If I specify INTERFACE, this means that cxxopts is not linked with CXXOPTS_LINKER_LIBRARIES, however executables that link to cxxopts will link with CXXOPTS_LINKER_LIBRARIES.

This sounds like the right thing to do then.

Review status: 0 of 6 files reviewed at latest revision, 5 unresolved discussions, some commit checks failed.

src/CMakeLists.txt, line 31 at r2 (raw file):

endif()

install(FILES cxxopts.hpp DESTINATION include)

I missed this, can you put it in the root file.

Comments from Reviewable

Can you rebase on top of master and force push instead of pulling my new commits into your branch?

Sorry, can you help with the git commands? I'm still learning...

I think I got it, I did git rebase master && git push -f, let me know if I messed anything up. I can redo my changes in a new pull request if you'd prefer.

Reviewed 1 of 8 files at r1, 6 of 8 files at r2, 1 of 1 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

I'll merge this as soon as I fix my travis build.


Windows phone error for SQL Lite

When I deploy to Windows 10 mobile device I get this error on Auth0 Lock login screen:
I believe this is so, because Auth0 Auth service depends upon ionic/storage.
Exception was thrown at line 3, column 8540 in ms-appx-web://aaa.td/www/build/polyfills.js
0x800a139e - JavaScript runtime error: Uncaught (in promise): Error: SQLite plugin is not present.

Thanks for the issue, auth0 doesn't depend on our plugin so this would be better on their end.


Working on creating a summary of schools endpoint

Creating a schools summary endpoint
Location

ml.py
db.py

Details

create summary of school info
includes total schools, percent high performing, and % public, private, and charter

Committed Features

create school summary endpoint
here users can get an overall idea of schools and rating in their desired city

 DS will find and share school ratings with users.

 DS will create endpoints for both web and iOS, utlizing the new data sets.