-Add project files

Thanks for the PR comments, Chris - it's helpful to read your perspective on the current architecture. Unfortunately, one of the main reasons for this assignment is to see how candidates approach architecting a simple end-to-end application and the current approach is not showing well how you would design re-usable or presentation components on the React side, or how you would abstract data models from api on the backend.
On running, I'm not seeing symptom or diagnosis labels in the UI, and have quickly encountered the exception below. Do you want to take a look at fixing it and refactoring the backend at least to support an improved organization / architecture?
<IP_ADDRESS> - - [03/Feb/2018 12:38:00] "OPTIONS /update_diagnosis_count HTTP/1.1" 200 -
[2018-02-03 12:38:00,041] ERROR in app: Exception on /update_diagnosis_count [POST]
Traceback (most recent call last):
File "/Users/hwellenius/code/CodeSubmissions/chrisknepper/simple-diagnosis-exercise/myvenv/lib/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
response = self.full_dispatch_request()
File "/Users/hwellenius/code/CodeSubmissions/chrisknepper/simple-diagnosis-exercise/myvenv/lib/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/Users/hwellenius/code/CodeSubmissions/chrisknepper/simple-diagnosis-exercise/myvenv/lib/python3.6/site-packages/flask/app.py", line 1517, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/Users/hwellenius/code/CodeSubmissions/chrisknepper/simple-diagnosis-exercise/myvenv/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
raise value
File "/Users/hwellenius/code/CodeSubmissions/chrisknepper/simple-diagnosis-exercise/myvenv/lib/python3.6/site-packages/flask/app.py", line 1612, in full_dispatch_request
rv = self.dispatch_request()
File "/Users/hwellenius/code/CodeSubmissions/chrisknepper/simple-diagnosis-exercise/myvenv/lib/python3.6/site-packages/flask/app.py", line 1598, in dispatch_request
return self.view_functionsrule.endpoint
File "back-end.py", line 27, in update_diagnosis_count
return jsonify(update_symptoms_list_count(selected_symptom, selected_diagnosis))
File "back-end.py", line 49, in update_symptoms_list_count
for line in dataset_csv:
_csv.Error: iterator should return strings, not bytes (did you open the file in text mode?)


New version

It's based on the version in master at the time.
Not sure what is the meaning of their releases - the recommended usage is to get the latest version from master. See https://vcpkg.io/en/getting-started

It's based on the version in master at the time.
Not sure what is the meaning of their releases - the recommended usage is to get the latest version from master. See https://vcpkg.io/en/getting-started

I guess the releases are in some aspect quality controlled just as we do? And also we would have a fixed *.tar.gz that we adapt with our changes rather than a snapshot of master at a point we can never reconstruct.
So, ok for this time, but I think we should base our branch on their latest releases. I also think we should update our vcpkg when a new release is available. Then there should not be many differences and we get rather up-to-date libraries

Maybe. In this case, Microsoft make some automatic controls of all contents of the repo. Your pull request will not be accepted unless it pass these checks. But there are no way for Microsoft to make sure that all packages works since it's community based and just to many packages. Also, if you get an error message when installing a package with vcpkg, it recommends you to just pull the latest changes and try again. There might be some info on the principles behind a release in vcpkg somewhere? Also, we can always reconstruct by using the commit IDs from the master repo.
Releases seems to be made roughly once a month, and that might be a good frequency for us to merge as well, so we can look for that. But then we need some sort of process to check when there is a new tag in the master repo.


FQDN: Only regenerate rule if dns response has IPs.

There are cases where a DNS response return a Nonerror but there are no
IPS in the response. In that case Cilium  regenerates the policy because
it's a legit response.
With this change, regeneration will not happens if it is a empty
response.
Signed-off-by: Eloy Coto<EMAIL_ADDRESS>

This change is

test-me-please

Coverage remained the same at ?% when pulling 8dc4e6f8aa147ba5850425fb004d376063f8a54e on eloycoto:FQDNResponseLen into adedcff32759af08c8d20a5d1e891ae01e33f9b3 on cilium:master.

@eloycoto does this needs backport?

It doesn't hurt to be backported, but 1.4 can live without this change.


Height labels are wrong in Brazil

As you can see on the image below, the top of the Pão de Açucar is labeled 0 meters

Thanks. This will be fixed when we update to mapbox.mapbox-terrain-v2. Ref #6


php mvc controller

How to write the controller?
get( '/index' , 'views/indexcontroller@index' );

This is not possible. PhpRrouter, as for now, will not support OOP.


commands: correct creation of the buffer for repeating commands

this went unnoticed because nothing creates a repeating
command on the client side yet

Since nothing uses this feature this commit is untested, but this now matches the server side implementation of the feature.


secret "kfserving-webhook-server-cert" not found

/kind bug
What steps did you take and what happened:
Installed KFServing 0.5.0 and 0.5.1 in a private GKE cluster after installing the requirements. The KFServing pod is not running, because the secret/cert has not been generated by cert-manager.
vents:
  Type     Reason       Age                  From               Message
  ----     ------       ----                 ----               -------
  Normal   Scheduled    27m                  default-scheduler  Successfully assigned kfserving-system/kfserving-controller-manager-0 to gke-kfserving-new-kube-default-pool-2d956bef-23hc
  Warning  FailedMount  11m                  kubelet            Unable to attach or mount volumes: unmounted volumes=[cert], unattached volumes=[cert default-token-z6qhb]: timed out waiting for the condition
  Warning  FailedMount  4m43s (x9 over 25m)  kubelet            Unable to attach or mount volumes: unmounted volumes=[cert], unattached volumes=[default-token-z6qhb cert]: timed out waiting for the condition
  Warning  FailedMount  38s (x21 over 27m)   kubelet            MountVolume.SetUp failed for volume "cert" : secret "kfserving-webhook-server-cert" not found

What did you expect to happen:
For the KFServing pod to run.
Anything else you would like to add:
I followed the same steps using a public GKE cluster. KFServing is working fine and the secret/cert is generated normally.
Environment:

Istio Version: 1.9
Knative Version: v0.20.0
KFServing Version: 0.5.0 and 0.5.1
Kubeflow version: None
Kubernetes version:
v1.17.17-gke.1101 and v1.17.15-gke.800
OS (e.g. from /etc/os-release): docker

It was a conflict with a previous yaml.


Import package is missing，

1.add  import random？

2.about models download ，I am not sure if the download is correct

Hi,
Thanks for pointing out the import error, and I updated the script.
For the SMPL-X model version, we used SMPL-X v1.0, but you can use v1.1 instead. The difference is that: v1.0 has 10-d shape paramsters and 10-d expression paramsters, while v1.1 has 300-d shape parameters and 100-d expression parameters. If you use v1.1, just use the first 10 dimensions of shape parameters and first 10 dimensions of expression parameters.

Hi,
Thanks for pointing out the import error, and I updated the script. For the SMPL-X model version, we used SMPL-X v1.0, but you can use v1.1 instead. The difference is that: v1.0 has 10-d shape paramsters and 10-d expression paramsters, while v1.1 has 300-d shape parameters and 100-d expression parameters. If you use v1.1, just use the first 10 dimensions of shape parameters and first 10 dimensions of expression parameters.

thanks，i try it


Dependency "cache-loader" deprecated

Version
5.0.0-alpha.7

https://github.com/vuejs/vue-cli/blob/985f2ff8d1a83d4ed3da5acfa8d0dc990b25d7ad/packages/%40vue/cli-plugin-typescript/package.json#L29
The package cache-loader is deprecated.
I suggest to remove this dependency in favor of this configuration.

@vue/cli 5.0.0-beta.1
It has the same deprecated cache-loader and gets dependency tree resolving error.
cache-loader 4.1.0 peerDependencies is webpack ^4.0.0 but the installed webpack is 5.38.1 which is required by other packages.
Packages requring cache-loader are below.
・@vue/cli-plugin-typescript
・@vue/cli-service
・vue-loader-v15
I hope the problem will be solved when vue-cli 5 becomes an officail release.


ERROR  TypeError: undefined is not a function, js engine: hermes -- WebView React Natve

Description
<WebView
        source={{ uri: newUrl }}
        onLoadEnd={() => {
          setLoading(false);
        }}
      />

React Native Version
0.71.7
Output of npx react-native info
System:
    OS: Windows 10 10.0.19045
    CPU: (8) x64 AMD Ryzen 7 4700U with Radeon Graphics
    Memory: 17.72 GB / 31.40 GB
  Binaries:
    Node: 19.8.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 9.5.1 - C:\Program Files\nodejs\npm.CMD
    Watchman: Not Found
  SDKs:
    Android SDK: Not Found
    Windows SDK: Not Found
  IDEs:
    Android Studio: Not Found
    Visual Studio: 17.5.33516.290 (Visual Studio Community 2022)
  Languages:
    Java: <IP_ADDRESS> - C:\Program Files\Microsoft\jdk-<IP_ADDRESS>-hotspot\bin\javac.EXE
  npmPackages:
    @react-native-community/cli: Not Found
    react: 18.2.0 => 18.2.0
    react-native: 0.71.7 => 0.71.7
    react-native-windows: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps to reproduce
npx expo start --dev-client
Snack, code example, screenshot, or link to a repository

Bro did you find a solution for this problem I am also facing it

@aum1618 yes with latest version its working now

@atifgk last version of what? I'm using "react-native": "0.71.8" and still same issue, any idea?

@sebas-deedee can you share package.json file?

@atifgk sure thing, here it is:
{   "main": "node_modules/expo/AppEntry.js",   "scripts": {     "start": "expo start -c",     "android": "expo start --android",     "ios": "expo start --ios",     "web": "expo start --web",     "eject": "expo eject",     "lint": "eslint .",     "lint:fix": "eslint . --fix",     "format": "prettier --write App.js app.json appSettings.js babel.config.js src/**/*.js"   },   "dependencies": {     "@expo/vector-icons": "^13.0.0",     "@juliuslouwllc/eaglebus-web-core": "^1.6.1",     "@react-native-async-storage/async-storage": "1.17.11",     "@react-native-community/masked-view": "0.1.10",     "@react-navigation/bottom-tabs": "^5.8.0",     "@react-navigation/native": "^5.9.3",     "@react-navigation/stack": "^5.9.0",     "axios": "^0.21.1",     "card-validator": "^8.0.0",     "color": "^3.1.2",     "expo": "^48.0.0",     "expo-constants": "~14.2.1",     "expo-dev-client": "~2.2.1",     "expo-font": "~11.1.1",     "expo-linking": "~4.0.1",     "expo-localization": "~14.1.1",     "expo-splash-screen": "~0.18.2",     "expo-status-bar": "~1.4.4",     "expo-web-browser": "~12.1.1",     "i18n-js": "^3.7.1",     "jwt-decode": "^3.1.2",     "lodash": "^4.17.19",     "moment": "^2.27.0",     "native-base": "^2.13.13",     "prop-types": "^15.7.2",     "react": "18.2.0",     "react-dom": "18.2.0",     "react-hook-form": "^6.15.1",     "react-native": "0.71.8",     "react-native-calendars": "^1.309.0",     "react-native-flash-message": "^0.1.23",     "react-native-gesture-handler": "~2.9.0",     "react-native-reanimated": "~2.14.4",     "react-native-render-html": "^6.3.4",     "react-native-safe-area-context": "4.5.0",     "react-native-screens": "~3.20.0",     "react-native-web": "~0.18.11",     "react-redux": "^7.2.1",     "redux": "^4.0.5",     "redux-saga": "^1.1.3"   },   "devDependencies": {     "@babel/core": "^7.20.0",     "babel-eslint": "^10.1.0",     "babel-plugin-module-resolver": "^4.0.0",     "babel-preset-expo": "^9.3.0",     "eslint": "7.2.0",     "eslint-config-airbnb": "18.2.0",     "eslint-config-prettier": "^7.2.0",     "eslint-import-resolver-babel-module": "^5.1.2",     "eslint-plugin-import": "^2.22.0",     "eslint-plugin-jsx-a11y": "^6.3.0",     "eslint-plugin-react": "^7.20.0",     "eslint-plugin-react-hooks": "4.0.0",     "husky": "4",     "lint-staged": "10",     "prettier": "^2.2.1"   },   "private": true,   "husky": {     "hooks": {       "pre-commit": "lint-staged"     }   },   "lint-staged": {     "*.js": "eslint --cache --fix",     "*.{js,json,md}": "prettier --write"   } }

@sebas-deedee  try this code for Webview. Hopefully it will work :-)
            <WebView scalesPageToFit
              startInLoadingState
              ref={(ref) => webViewRef.current = ref}
              source={{ uri: newUrl }}
              originWhitelist={['*']}
              style={{ flex: 1 }}
              onLoadStart={() => {

              }}
              onLoad={(webview) => {

              }}
              onNavigationStateChange={(webview) => {

              }}
              javaScriptEnabled={true}
              domStorageEnabled={true}
              setSupportMultipleWindows={false}
              onScroll={(e) => { handleScroll(e) }}
              onMessage={() => { }}
              onError={syntheticEvent => {
                const { nativeEvent } = syntheticEvent;
              }}
              renderError={error => {
                return (<View>
                  <Text>Something went wrong. Please try again later</Text>
                </View>);
              }}
            />

I'm having the same problem can u help me:
`import React from 'react';
import { View, StyleSheet, SafeAreaView, ActivityIndicator } from 'react-native';
import { WebView } from 'react-native-webview';
const ActivityIndicatorElement = () => {
//making a view to show while loading the webpage
return (

);
}
const Fisio = () => {
return (
<SafeAreaView style={{ flex: 1 }}>

<WebView
sharedCookiesEnabled={true}
source={{ uri: "https://staging.ordemdosfisioterapeutas.pt/encontre-um-fisioterapeuta?hideinapp=true" }}
renderLoading={ActivityIndicatorElement}
javaScriptEnabled={true}
domStorageEnabled={true}
startInLoadingState={true}
/>

);
};
const styles = StyleSheet.create({
container: {
flex: 1,
},
activityIndicatorStyle: {
flex: 1,
position: 'absolute',
left: 0,
right: 0,
top: 0,
bottom: 0,
justifyContent: 'center',
alignItems: 'center',
},
});
export default Fisio;
`

@ALIG4TOR have you tried above that i have shared?

I have solved it by commenting out "//RNCWebViewModule.shouldStartLoadWithLockIdentifier(shouldStart, lockIdentifier);", present in WebView.android.js (Line 42).

I have solved it by commenting out "//RNCWebViewModule.shouldStartLoadWithLockIdentifier(shouldStart, lockIdentifier);", present in WebView.android.js (Line 42).

This is working. Can you please explain what was wrong?

This is what worked for me.
If you are using expo, install it with npx expo install react-native-webview

Description
<WebView
        source={{ uri: newUrl }}
        onLoadEnd={() => {
          setLoading(false);
        }}
      />

React Native Version
0.71.7
Output of npx react-native info
System:
    OS: Windows 10 10.0.19045
    CPU: (8) x64 AMD Ryzen 7 4700U with Radeon Graphics
    Memory: 17.72 GB / 31.40 GB
  Binaries:
    Node: 19.8.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 9.5.1 - C:\Program Files\nodejs\npm.CMD
    Watchman: Not Found
  SDKs:
    Android SDK: Not Found
    Windows SDK: Not Found
  IDEs:
    Android Studio: Not Found
    Visual Studio: 17.5.33516.290 (Visual Studio Community 2022)
  Languages:
    Java: <IP_ADDRESS> - C:\Program Files\Microsoft\jdk-<IP_ADDRESS>-hotspot\bin\javac.EXE
  npmPackages:
    @react-native-community/cli: Not Found
    react: 18.2.0 => 18.2.0
    react-native: 0.71.7 => 0.71.7
    react-native-windows: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps to reproduce
npx expo start --dev-client
Snack, code example, screenshot, or link to a repository

This error casually is just misunderstood of what is URL and URI, if u use "www.google.com" on a URI that's an error, the correct way is "https://www.google.com" because she's a complete URI

This error casually is just misunderstood of what is URL and URI, if u use "www.google.com" on a URI that's an error, the correct way is "https://www.google.com" because she's a complete URI

@OtanielJr  I've had this error with link shortener links. They were https but still threw the error.

I have solved it by commenting out "//RNCWebViewModule.shouldStartLoadWithLockIdentifier(shouldStart, lockIdentifier);", present in WebView.android.js (Line 42).

You are absolute Genius , I was looking for solution until a week  and Guess what I found your Answer.


multipleprocess implementation of hmm search

When I have a big collection of genome (one thousand for example), hmmsearch one by one is really slow.  Any possibility to make it multiprocess?

Unfortunately, the way it is implemented right now, this is not possible. It loops over all proteomes in a foreach which is not parallel in perl by default: https://github.com/molbiodiv/bcgTree/blob/06cbbe11384293ff4a561e18cbcdfc213f7851bb/lib/bcgTree.pm#L114
There might be a possibility to get this functionality using a package like: https://metacpan.org/pod/Parallel::Loops but it certainly needs some adjustments. hmmsearch itself uses multiple threads by default but I understand that in a scenario with this many genomes further parallelization is desirable.

Thanks for the quick response. Can you also please add an mafft option? I tried a 1200 genome collection and the alignment is slow if using muscle. I think this will be very easy to implement and we can use more threads for alignment.
Thanks,
Jianshu

Hi Jianshu, thanks for the suggestion. I'd be happy to implement that. Just to clarify, you would just replace the muscle calls with mafft calls. Would you still use Gblocks afterwards? Would something else need adjustment? Is the output format of mafft the same as that of muscle?

Hey Jianshu, all good suggestions. Be aware with mafft that it might result in different outcomes, we have not benchmarked it with that. It should be doable to replace it, as Markus suggested, but we can at the moment not say whether this will yield similar results and may need further parameter adjustments.
As for multithreading: This can be tricky depending on which multithreading architecture is available. The individual parts themselves are multithreaded already, but are called serialised as Markus said as well. What we may implement would also be --continue option, where bcgtree can define checkpoints. You could then parallelize the hmms yourself and afterwards continue bcgtree. This is just an idea, Markus and me can discuss it early November.

I agree but iqtree seems to be a better option to reduce computational time. I am also trying PhyloPhlan3. It uses trimAl and iqtree2 but I think the idea is very similar（BTW I am actually using HPC and I will not worry about time，a few month is acceptable）


Rework auth

Update login method from API.login() to API.loginWithSSO()
Bump call-of-duty-api version and pull in new dependencies.

Formalizing https://github.com/riversteve/cod-stats/pull/3


creating NDK mapping file extremely slow

I am calling the "uploadBugsnagNdk" gradle task on my project, which contains a sizable NDK library. The built, stripped library is roughly 20MB. The debug library containing symbols - which is the one being processed by the plugin - is roughly 300MB in size.
Running uploadBugsnagNdk on this library is extremely slow - it was still processing after 4 hours when I gave up and canceled. This is a function of objdump. Running the objdump command manually is just as slow. I have tried NDK versions 16b and 17 with the same result.
More specifically, the issue appears to be with the --line-numbers parameter. When I run "objdump --disassemble --demangle" and omit --line-numbers, objdump completes quickly.
This has not been an issue with other NDK crash reporting tools I have used. Is it possible to produce a mapping file for a large NDK library without the excessive delay?
Thank you.
Dave

Thanks for the report @daviswalker - we're currently investigating the effects that removing --linenumber would have on the information available in stacktraces, and whether any alternatives that might increase performance.

@daviswalker we've tested this out with a ~30Mb NDK library on macOS and didn't observe any obvious performance problems. Are you using Windows by any chance? We'd like to rule out the possibility that this could be platform-specific.

@fractalwrench Thank you for your response. We're on macOS. Might it be possible to share our library with you to see if this is a library difference or a toolchain difference? (Or perhaps you could share your test library with me)

@daviswalker if you're willing to share your library with us that could be very helpful in tracking down the issue. If you could email<EMAIL_ADDRESS>with the necessary information, I'll be happy to take a look.

@fractalwrench: I work with @daviswalker and have just emailed a link to our library.  As a further data point, in our attempts to work around the issue, we have been able to successfully and quickly dump symbols with line numbers using the LLVM version of objdump, demangled with c++filt.
$ llvm-objdump -section=.text -line-numbers libfoo.so | c++filt -n > libfoo.map
However, uploading this file to bugsnag simply returns the error: "Mapping file does not contain any functions".  I have included this file in my support email, as well.

Thanks for the useful info @nick-joya - just to confirm are you building your project with GCC or Clang? I've passed the library onto my colleague @kattrali who is currently investigating performance improvements in this area.

Thanks, @fractalwrench!  We are building with clang.

Thanks again, @nick-joya.
The slowness should be fixed in v3.4.0, however if there is an outstanding issue with GCC vs Clang toolchains, we should continue discussion in a new issue, to make sure it doesn't get lost.


Editing ReplicaController to add a new container to the pod does not take effect.

Rancher Version:v1.0.0-rc1
Steps to reproduce the problem:
Create a kubernetese environment with few hosts.
Created a Replication Control for a pod with  2 containers.
pod gets created with 2 containers.
Edit Replication Controller and add another container to the pod.
The newly added container (ubuntu) is not being deployed. But it gets saved in the Replica Controller object.

@sangeethah doesn't look like its supported by k8s:
http://kubernetes.io/docs/api-reference/v1/definitions/#_v1_podspec
List of containers belonging to the pod. Containers cannot currently be added or removed. There must be at least one container in a Pod. Cannot be updated. More info: http://releases.k8s.io/release-1.2/docs/user-guide/containers.md

@alena1108 , if this is not supported , we should not expect the Replication Controller to show the containers that failed to be added.
The newly added container (ubuntu) is not being deployed. But it gets saved in the Replica Controller object and displayed in the UI.

Closing as we no longer support Rancher UI for k8s. All the commands are executed via k8s dashboard and go directly to k8s api server.


not working 'gem tag --no-push'

Thanks for the report @sroik!
This should be fixed in https://github.com/svenfuchs/gem-release/commit/dd6ff6c56b9c0632cac553361afd3fa0c785682c


Add infinite lifetime for Particles (3.x)

This is useful in custom shaders to render permanent elements such as grass.
I haven't tested this with custom shaders yet. Help is welcome in this area :slightly_smiling_face:
This closes https://github.com/godotengine/godot-proposals/issues/2614.
Testing project: test_particles_infinite_lifetime_3.x.zip
Preview

TODO

[ ] Allow for particle curve animation by using the lifetime as an animation duration when the lifetime is infinite. (Don't hide the Lifetime property anymore when Lifetime Infinite is checked in the inspector.)

The animation should likely be looped rather than clamped.

[ ] Implement infinite lifetime in Particles2D.
[ ] Create a version of this PR for master.

Not sure about those:

[ ] Allow for particle movement (built-in velocity)? It's not very useful with infinite lifetime with the default ParticlesMaterial, since particles would usually end up going infinitely far from the origin.
[ ] Implement infinite lifetime in CPUParticles and CPUParticles2D? It's not very useful there since you don't have custom particle shaders.

Hey @Calinou , thanks a lot for this work!
I believe that infinite lifetime is something relevant only for when doing a custom particle shader and it's fine if it doesn't mash well with the existing particle material

That's a great addition to the ParticleMaterial.
About the two points you are unsure:

Movement: I don't think there is any way of implementing this without making the behaviour different from what certain parameters normally do. If it's implemented in a way that makes sense it would probably be something that is looping and that would be a break from how velocity is used without infinite lifetime.
I think it could be useful in CPU Particles, however, much more limited than in GPU Particles. I think in the long run it will almost be impossible to have feature parity on CPU and GPU Particles.

Is there anything stopping this from getting merged? Excited to use this feature! :)

Is there anything stopping this from getting merged? Excited to use this feature! :)

See the TODO section in the pull request. This can't be merged in its current state, as it's too awkward to use. Feel free to salvage this PR as I don't have time to work on it.

3.x? My studio's already on 4.0! xD
I wouldn't mind taking a crack at helping out, although I don't think I'm familiar enough with the codebase for it to be expedient. I might make a PR one of these days, tho.
I've been led to believe that the logic (as opposed to hardware) particle API internals are essentially analogous, between CPU/GPU/2D/3D? If what needs to be done is mostly boilerplate, I think I can handle it.


Add example notebook with an introduction to UDFs

Contributor Checklist:

[x] I have added or updated my entry in the creators.json file
[ ] I have added a changelog entry for my contribution
[x] I have added/updated documentation for all user-facing changes
[ ] I have added/updated test cases
[ ] I have included the rebuilt production build of the client (only if
changes were made to the GUI)

Closed an re-opened because travis status was stuck for some reason.

Awesome, I like it! Merging now. :-)


Image detections from Hailo not reaching prompt

I can see the prompt is "None" - need to validate Hailo-tau integration works properly

Going to seperate this to another thread, then let it consume the events in background


Add pylint checking to GitHub workflow

See https://github.com/Senzing/stream-loader/issues/303

Pull requests:

https://github.com/Senzing/g2createproject/pull/29


Support dynamic data?

Hi team,
I am building a 'aws-glue' development environment in my local computer. When I build the dev environment, aws-glue needs dynamic data when I use amazon-ec2-metadata-mock to similar metadata service. Do you have plan to support dynamic data, such as '/latest/dynamic/instance-identity/document' in the future release?
NOTE: I already build my own patch in my fork.

Yeah! This definitely seems like something amazon-ec2-metadata-mock would benefit from!  I believe we already serve a lot of this data from meta-data paths like ami-id, instance-id, instance-type, and local-ipv4.
Since you've already patched in your fork, would you be willing to work on a PR for this new feature?
Example:
curl <IP_ADDRESS>/latest/dynamic/instance-identity/document
{
  "accountId" : "0123456789",
  "architecture" : "x86_64",
  "availabilityZone" : "us-east-1f",
  "billingProducts" : null,
  "devpayProductCodes" : null,
  "marketplaceProductCodes" : null,
  "imageId" : "ami-0b69ea66ff7391e80",
  "instanceId" : "i-01234567890",
  "instanceType" : "t3.2xlarge",
  "kernelId" : null,
  "pendingTime" : "2020-04-08T16:57:10Z",
  "privateIp" : "<IP_ADDRESS>",
  "ramdiskId" : null,
  "region" : "us-east-1",
  "version" : "2017-09-30"
}%

@bwagner5 Thanks for your feedback. I like the idea, and I filed a PR #74 for this. Please help to review PR for me.

Closing issue following ther merge of #74


feat: Add trend indicator

This pull request primarily focuses on enhancing the functionality of the coverage reporting system. The changes include the addition of a new json-summary-compare-path option to compare coverage summaries, modifications to the README.md and action.yml files to reflect this new option, and several file relocations and renames to improve code organization.
Fixes #262.
Here are the top five most important changes:
Changes to reporting options:

README.md: A new option json-summary-compare-path was added to the options table. This option allows users to specify a path to a json summary file to compare against the current run, providing a trend indicator and the difference in the summary.
action.yml: The file-coverage-mode option was moved and the json-summary-compare-path option was added. This new option requires a path to the json summary file of the previous run to get trend indicators.

Code organization and simplification:

Several files were moved from the src directory to the src/inputs and src/report directories. This change improves the organization of the code by grouping related files together. [1] [2] [3] [4] [5] [6] [7] [8] [9]
src/index.ts: The import statements were reordered, and new code was added to handle the jsonSummaryComparePath option. If this option is provided, the json summary file at the given path is parsed and used to generate the summary table.

Changes to coverage reporting:

src/report/generateSummaryTableHtml.test.ts: New tests were added to check the generation of the summary table when a compare report is provided. These tests check the cases where the coverage increased, decreased, or stayed the same.

hi @davelosert ! Thanks for the effort. Any estimates on this feature?

Hi @inomn,
this feature is basically ready.
What kept me from merging yet is that in it's current form, it requires users's to manually generate both JSON-Reports.
I wanted to add another convenience function to automatically download the latest report from the target-branch in order to avoid having to run tests twice during the PR.
I hoped I could do this during the christmas-break, but unfortunately wasn't able to finish and pick up the work.
I'll see if I can finish it up this week and merge this.

Hi @davelosert ! Sorry bothering you, any estimates on this feature? Thanks!


Why can't I deploy 2 contracts programmatically

It seems there arent limits on how many contracts I can deploy based on gas restrictions.
2 wasm files of 100k fail to be deployed programmatically using promises.

I get this had<PHONE_NUMBER>00000 of attached gas but used<PHONE_NUMBER>218 of gas but doesn't make sense

I am observing strange behaviour.

Yes the issue is if you attach too much gas to the cross contract call, it will fail due to the gas consumed in the current function call. For example, if the you attach 300Tgas to the transaction and spend 20Tgas in the first function call and attempt to attach 290Tgas to the promise, then it will fail because 290 + 20 > 300

is there any way to see how much gas remains during cross contracts?

You can see the breakdown by querying the transaction through rpc

Closing because nothing seems directly actionable. Re-open if there is something, but likely this will be transferred to another repo (SDK doesn't handle these interfaces)


required parameters in allOf model

There is an edge case not handled by #598 where a model requires properties from another model in allOf. There is an example of this in vmware.json spec that I manage:
    "ManagementCluster": {
      "type": "object",
      "description": "The properties of a management cluster",
      "required": [
        "clusterSize"
      ],
      "allOf": [
        {
          "$ref": "#/definitions/CommonClusterProperties"
        }
      ]
    },

clusterSize is required for ManagementCluster, but is not in CommonClusterProperties.  Currently, ManagementCluster::new() has no parameters.

I'll be disabling this clippy for services until this issue is addressed.
error: you should consider adding a `Default` implementation for `TrackedResource`
   --> mgmt/managednetwork/src/package_2019_06_01_preview/models.rs:379:5
    |
379 | /     pub fn new() -> Self {
380 | |         Self {
381 | |             resource: Resource::default(),
382 | |             tags: None,
383 | |         }
384 | |     }
    | |_____^
    |
    = note: `-D clippy::new-without-default` implied by `-D warnings`
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#new_without_default
help: try adding this
    |
378 + impl Default for TrackedResource {
379 +     fn default() -> Self {
380 +         Self::new()
381 +     }
382 + }
    |

error: could not compile `azure_mgmt_managednetwork` due to previous error
warning: build failed, waiting for other jobs to finish...
error: build failed
Error: Process completed with exit code 101.

In this example, location is required by TrackedResource, but defined in Resource.
    "TrackedResource": {
      "description": "The resource model definition for a ARM tracked top level resource",
      "properties": {
        "tags": {
          "type": "object",
          "x-ms-mutability": [
            "read",
            "create",
            "update"
          ],
          "additionalProperties": {
            "type": "string"
          },
          "description": "Resource tags"
        }
      },
      "required": [
        "location"
      ],
      "allOf": [
        {
          "$ref": "#/definitions/Resource"
        }
      ]
    },


2.2.0/2.2.0+1-dev.1 on a flutter web project

In my web project i installed the editor and now i have this bug.
/D:/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_colorpicker-0.5.0/lib/src/hsv_picker.dart:730:43: Error: The parameter 'event' of the method 'AlwaysWinPanGestureRecognizer.addAllowedPointer' has type 'PointerDownEvent', which does not match the corresponding type, 'PointerEvent', in the overridden method, 'DragGestureRecognizer.addAllowedPointer'.
 - 'PointerDownEvent' is from 'package:flutter/src/gestures/events.dart' ('/D:/flutter/packages/flutter/lib/src/gestures/events.dart').
 - 'PointerEvent' is from 'package:flutter/src/gestures/events.dart' ('/D:/flutter/packages/flutter/lib/src/gestures/events.dart').
Change to a supertype of 'PointerEvent', or, for a covariant parameter, a subtype.
  void addAllowedPointer(PointerDownEvent event) {
                                          ^
/D:/flutter/packages/flutter/lib/src/gestures/monodrag.dart:256:8: Context: This is the overridden method ('addAllowedPointer').
  void addAllowedPointer(PointerEvent event) {
       ^
Failed to compile application.

My .yaml file looks like this:
dependencies:
  flutter:
    sdk: flutter
  google_sign_in: ^5.0.5
  html_editor_enhanced: ^2.2.0

To Reproduce
Steps to reproduce the behavior:

Create a new Web-Proj
Insert the dependecies
copy this code snipped

class HtmlEditorExampleApp extends StatelessWidget {

  HtmlEditorController controller = HtmlEditorController();

  @override Widget build(BuildContext context) {
    return HtmlEditor(
      controller: controller, //required
      htmlEditorOptions: HtmlEditorOptions(
        hint: "Your text here...",
        //initalText: "text content initial, if any",
      ),
      otherOptions: OtherOptions(
        height: 400,
      ),
    );
  }
}

See error

Please try the solutions listed here: https://github.com/tneotia/html-editor-enhanced/issues/68#issuecomment-863369808
Hope it helps!

Thank you very much


Added getIndentation to adapt indentation to vscode settings

When using spaces instead of tabs, sometimes you would get tabs and have inconsistent indentation. This PR makes sure the extension uses the indentation from the vscode settings.

Thanks!


Dialog content not opened inside portal

Describe the bug
At version 1.6.8, the dialog content opened inside portal, but after this change, it doesn't open the content inside portal
To Reproduce
<Modal.State>
  <Modal.Disclosure use={Button}>Open modal</Modal.Disclosure>
  <Modal>
    <Card>
      <Box>Hello world</Box>
      <Modal.Disclosure use={Button}>Close</Modal.Disclosure>
    </Card>
  </Modal>
</Modal.State>

click Open modal button
open browser dev tool
check the reakit portal element, it only has modal backdrop, it doesn't has the dialog content inside

Expected behavior
The dialog content should be inside portal
Screenshots
now v1.6.10, content not inside portal

expected v1.6.8, content inside portal

Device information (please complete the following information):

OS, version: windows 10
Browser, version: Chromium: 87

Additional context
not sure if it's the cause, but after I remove the wrapElement, the dialog content appear inside portal

Thanks for reporting! This should be fixed on 1.7.0. Can you try again?

Thanks for reporting! This should be fixed on 1.7.0. Can you try again?


Missing DISTRO_FEATURES

Hello,
I'm building SOTA in a custom distro. I found out that the following line was needed in my configuration:
DISTRO_FEATURES_append = " systemd"
Maybe it should come in  conf/distro/sota.conf.inc?
Regards,
Fernando Endo

systemd is a build dependency of Aktualizr

That is true. Our usual case is systemd. However, it should be possible to use aktualizr (and ostree) without systemd with some modifications. Neither one is fundamentally dependent on systemd as far as I am aware, but we only have experience using them with systemd. You are welcome to try it, and if you think you have a patch that will enable aktualizr to work flexibly with or without systemd, we would be curious to see it.
I do know that building aktualizr with -DBUILD_OSTREE=OFF is easy (and the default on non-Linux systems). I personally know less about what edits would be necessary to the aktualizr and ostree recipes in meta-updater.

There is an option to build Aktualizr without systemd as a dependency.  There is a cmake option -DBUILD_SYSTEMD=OFF, but it isn't currently wired into the recipes in meta-updater.  Adding that to EXTRA_OECMAKE might make it build.  Wiring that into the Yocto DISTRO_FEATURES shouldn't be too hard.

@fernandoendo Have you had any luck making this work, or is it still a problem for you?

@fernandoendo @patrickvacek with packageconfig merged (pull request #470) I think this issue is addressed.

Seems to have been fixed.


Remove note about uploading academic verification

The education benefits request no longer has a way to upload proof of
academic status -- just taking a picture using the computer's camera.
Thus, this change removes the reference to uploading an image in the
documentation for applying for an educator discount.

Why:
Closes #19342

What's being changed (if available, include any code snippets, screenshots, or gifs):

The instructions for applying for an educator or researcher discount references a way to upload proof of academic status, but the (current) form does not have this button -- just the ability to take a picture using the computer's camera.
Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@joel-coffman Thanks for opening a pull request and linking it to your issue! ⚡
I'll get this triaged for review ✨

@joel-coffman - Sorry for the delay here. We have been tracking this and working on other updates internally. We have an internal PR in the works that should update this, so I'm going to go ahead and close this PR now. 💛
Please take a look at our help wanted section to find open issues you can work on.
Thank you for your passion and interest in improving GitHub Docs! :sparkling_heart:

@joel-coffman - Sorry for the delay here. We have been tracking this and working on other updates internally. We have an internal PR in the works that should update this, so I'm going to go ahead and close this PR now. 💛

Thanks for the update, @cmwilson21!
For my own future reference, what kind of contributions does GitHub accept / is GitHub interested in when it comes to documentation? I've submitted a handful of issues and offered to contribute changes, but the response usually seems to be that they'll be addressed internally. While it didn't take a great deal of time to create this pull request, I don't really want to do that if GitHub prefers to resolve these issues on its own. Should I wait for the "help wanted" tag to be applied to an issue before working on it in the future?

@joel-coffman - Great question! We are interested in all contributions that improve our docs! I can't speak for the others you may have submitted, but this one comes down to timing. The team in charge over education and the application process has been rolling out updates behind the scenes and have been gradually updating the docs in the process. I wasn't sure when this update would take place so I left this PR open for tracking. I'm sorry. That's on me. I could have better communicated that.
We understand that our contributors take time out of their day to submit these PRs and issues. We truly appreciate our community's involvement and enthusiasm for helping maintain the docs. We encourage all our users to contribute when they see something that could be updated. Opening an issue before a PR does give our team a chance to review the issue and confirm it or decide if it is best handled internally. In the mean time, the help wanted board is a fantastic place to work on open issues that have already been approved. ✅
Thank you, again, for your interest in contributing, and thank you for opening up this conversation and asking these questions. We look forward to seeing future contributions from you! 💖


Diadromous Fish - Colors

Change the colors of the diadromous fish layer-- green for "accessible" and red for "not accessible"

This might belong in a different issue, but we should put the accessible value at the bottom of the stacked bar chart.

My rationale is that once we add the goal for each river, the distance between current and goal will be easy to visualize.

Moved accessible to the bottom of the stack. Fixed colors.


Helper/Caster/JSON.php warnings fix

Helper/Caster/JSON.php warnings fix

Why this is not merged ?


Analog microphone slow and noisy recordings

Issue
When recording with an analog microphone (MAX9814) the recordings are low in pitch, slow and noisy. When I manually increase the sample rate using software like Audacity I get significantly closer to the original recording (although sound pitch is off) but it is still very noisy.
Environment
Using IDF project
Windows 10
ESP32 development board
MAX 9814
Beside merging changes inside this PR no other code change was made. Not even to the config (I stuck by the original pinout)
I am unsure how to begin debugging this.

i'm having axactly the same issue. I've opened an issue thread regardig this.
Are you also using the warduino SDCard Library?

I had some issues to, maybe this will help:
https://github.com/atomic14/esp32_sdcard_audio/issues/17#issuecomment-1692335765


chore(deps): update dependency pretty-quick to v3.1.0

This PR contains the following updates:

Package
Type
Update
Change

pretty-quick
devDependencies
minor
3.0.2 -> 3.1.0

Release Notes

azz/pretty-quick
v3.1.0
Compare Source
Features

add an --ignore-path flag (#115) (eaf29e2)

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

:tada: This PR is included in version 6.1.7 :tada:
The release is available on:

v6.1.7
GitHub release

Your semantic-release bot :package::rocket:


[Bug] Tab spacing messes with the text editor

Scroll horizontally such that a tab key is just barely visible (on the left side). This makes things look weird and move oddly.
Click the first half of a tab character vs the last half you cannot easily go to the left or right side based on percentage closer that you click to one end.
Proportional font breaks when there is a tab key on the same line as any proportional font text.

Stale issue. The IDE needs a release version and the issues are cluttered. I want to focus on getting high priority items listed and low priority items listed.
This would aid in anyone who has interest in the Luthetus.Ide project, what the triage is looking like for v1


Remove unneeded debug output to console

Cryptic output to console

Sorry, did not see it was in vendor directory

I will send a PR to update xorm to remove this output

I looked at xorm code and it does not have it already but there seems to be lot of changes in xorm to update gitea to latest xorm version


mpvScripts.thumbnail: 0.5.3 -> 0.5.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for mpvScripts.thumbnail is: A lua script to show preview thumbnails in mpv's OSC seekbar
meta.homepage for mpvScripts.thumbnail is: https://github.com/marzzzello/mpv_thumbnail_script
meta.changelog for mpvScripts.thumbnail is: https://github.com/marzzzello/mpv_thumbnail_script/releases/tag/0.5.4
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 0.5.4 in filename of file in /nix/store/xxqkybch1kxrhc24nhv7dv4wvkjcq7bb-mpv-thumbnail-script-0.5.4

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

mpvScripts.thumbnail

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xxqkybch1kxrhc24nhv7dv4wvkjcq7bb-mpv-thumbnail-script-0.5.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A mpvScripts.thumbnail https://github.com/r-ryantm/nixpkgs/archive/c8a92484f94c29199d39bf8ed71ed3dd2f714d85.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/c8a92484f94c29199d39bf8ed71ed3dd2f714d85#mpvScripts.thumbnail

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xxqkybch1kxrhc24nhv7dv4wvkjcq7bb-mpv-thumbnail-script-0.5.4
ls -la /nix/store/xxqkybch1kxrhc24nhv7dv4wvkjcq7bb-mpv-thumbnail-script-0.5.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

mpvScripts.thumbnail

Maintainer pings
cc @figsoda for testing.

Result of nixpkgs-review pr 312827 run on x86_64-linux 1

1 package built:

mpvScripts.thumbnail


The CMake build process is failed with can not solve the openssl export symbles if BUILD_SHARED_LIBS is set to ON and gRPC_SSL_PROVIDER = module (on Windows )

What version of gRPC and what language are you using?
What operating system (Linux, Windows,...) and version?
What runtime / compiler are you using (e.g. python version or version of gcc)
What did you do?
Please provide either 1) A unit test for reproducing the bug or 2) Specific steps for us to follow to reproduce the bug. If there’s not enough information to debug the problem, gRPC team may close the issue at their discretion. You’re welcome to re-open the issue once you have a reproduction.
What did you expect to see?
What did you see instead?
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
See TROUBLESHOOTING.md for how to diagnose problems better.
Anything else we should know about your project / environment?

Please provide more information, such as command line used, CMake version and so on.

CMake Version is 3.26.3 GUI  in windows , I Used it to configure with BUILD_SHARED_LIBS  and generate,and then open the project  with   vs 17 2022 project .When i generate the sln about ssl ,errors occerred. Like this ：
2>   Creating library D:/Project/ThirdParty/grpc/Release/third_party/boringssl-with-bazel/Release/ssl.lib and object D:/Project/ThirdParty/grpc/Release/third_party/boringssl-with-bazel/Release/ssl.exp
2>tls13_enc.obj : error LNK2001: unresolved external symbol ERR_put_error
2>tls13_server.obj : error LNK2001: unresolved external symbol ERR_put_error
2>tls_method.obj : error LNK2001: unresolved external symbol ERR_put_error
2>tls_record.obj : error LNK2001: unresolved external symbol ERR_put_error
2>ssl_x509.obj : error LNK2001: unresolved external symbol ERR_put_error
2>t1_enc.obj : error LNK2001: unresolved external symbol ERR_put_error
2>tls13_both.obj : error LNK2001: unresolved external symbol ERR_put_error
2>tls13_client.obj : error LNK2001: unresolved external symbol ERR_put_error
2>ssl_privkey.obj : error LNK2001: unresolved external symbol ERR_put_error
2>ssl_session.obj : error LNK2001: unresolved external symbol ERR_put_error
2>ssl_transcript.obj : error LNK2001: unresolved external symbol ERR_put_error
2>ssl_versions.obj : error LNK2001: unresolved external symbol ERR_put_error
I  detect the defination and openssl export  that's no problem .


menusmall wrap option

Same as EspruinoDocs graphical_menu.js: https://github.com/espruino/EspruinoDocs/pull/614
I think I also got the appRect tweaks, seemd to look allright in the emulator...
I did notice it got rid of Bangle.drawWidgets(); though, not sure if that is ok?

Great - thanks! Yes, getting rid of drawWidgets should be fine - since it clears the apprect it won't clear the widget bar now so shouldn't need to redraw them


"lime build android" fails on icon processing

lime 2.4.0
Build fails if project.xml contains "icon" elements.
i.e.
<icon path="res/icon/android_512.png" size="512"/>

Called from ? line 1
Called from CommandLineTools.hx line 1359
Called from CommandLineTools.hx line 25
Called from CommandLineTools.hx line 126
Called from CommandLineTools.hx line 579
Called from lime/project/PlatformTarget.hx line 70
Called from lime/tools/platforms/AndroidPlatform.hx line 328
Called from lime/tools/helpers/IconHelper.hx line 83
Called from /usr/lib/haxe/std/neko/_std/sys/io/File.hx line 45
Called from /usr/lib/haxe/std/haxe/io/Output.hx line 105
Uncaught exception - Invalid field access : length

I also get this problem, using the latest Haxe 3.2.0 release. It does not seem to show up when using 3.2.0-rc.2, though.

this problem is also happening to me on mac. but even when using the rc2 version does not work or fix the problem

Failed here something similar:
Lime 2.4.1
Haxe 3.1.3
lime test neko -debug
Warning: Could not find template file: neko/bin/neko-Mac64
Called from ? line 1
Called from CommandLineTools.hx line 1359
Called from CommandLineTools.hx line 25
Called from a C function
Called from CommandLineTools.hx line 126
Called from CommandLineTools.hx line 579
Called from lime/project/PlatformTarget.hx line 77
Called from lime/tools/platforms/MacPlatform.hx line 110
Called from lime/tools/helpers/NekoHelper.hx line 21
Called from /usr/lib/haxe/std/neko/_std/sys/io/File.hx line 34
Called from /usr/lib/haxe/std/neko/_std/sys/io/File.hx line 30
Uncaught exception - Invalid field access : __s```

@eduardonunesptoptal It sounds like you are having a different problem. For some reason, it is not finding that template file. Could you see if you have it under "/usr/lib/haxe/lib/lime/2,4,1/lime/templates/neko/bin"?
Perhaps you have a bad install, or there's a chance that you have things installed on a case-sensitive drive?

(closing this issue as the icon processing problem was resolved in 2.4.1)

Sure, it's was a bad install, after resinstall everything is ok. Thanks


(511) Update course offering page

Context
The designs for the course offering page have been updated
Changes in this PR
In course offering page:

Add link to email address
Remove "Make referral" button

Elsewhere: various bits of groundwork and minor consistency edits
Screenshots of UI changes
Before

After

Post-merge checklist

[ ] Have you written an end-to-end test for the happy path in the Accredited Programmes E2E repo (if necessary)?

 Update course offering page


Key Bindings can not be set

When you click the Preferences > Package Settings > OpenAI completion > Key Bindings menu option the console show the error:
error: The settings file "res://Packages/openAI/Default.sublime-keymap" could not be opened

You have defined the key bindings base file here:
https://github.com/yaroslavyaroslav/OpenAI-sublime-text/blob/develop/Main.sublime-menu#L27
But the base_file Default.sublime-keymap not exists in the repository.

Also, after adding that file, I would suggest to include a suggested key binding there but commented out. Then the user can easily copy it over to user key bindings and enabled. Another reason this is useful because user doesn't know what command and arguments to use for the key binding since it's not documented. So maybe include all three different key bindings with different mode arguments.

It was, since it's working for me, but I guess it has gone somewhere within package publication process. I'll move it back shortly, and punch update.

It could worth it. I'm not promise to provide full command list, but I'll provide the bare minimum for sure.

I ran into the same error with v1.1.2. I had to manually add Packages/User/openAI.sublime-settings. It still gives the error when I try to access the Settings through the menu, but at least it reads my API key.

@raineorshine I see that it's different one. The original was not about sublime-settings, but about sublime-keymap which was really dropped from the project of the review stage.
Could you please open new issue, and provide some details, like: Sublime version, OS version, error log if any.

I get the same error for both settings and keymap. I'll open an another issue when I'm back at my computer!


Added pipenv and update Documentation

Dear seniors, instead of a virtual environment it takes memory and storage space too. for that solution, we can use pipenv. and I have documented in the readme file too. thank you.

@kaflesudip bro, I have updated the content of the use of python in Nepal.

@Aju100 Let's not have more than one thing covered in single PR. Limit this PR to adding pipenv and create a different pr for adding new links.


SettingWithCopyWarning in postprocessing

There is a SettingWithCopyWarning in one of the postprocessing functions.
20:35:13-INFO-Restoring attributes will overwrite existing attributes.
/home/unndreay/.virtualenvs/oemo-flex/lib/python3.7/site-packages/pandas/core/frame.py:2963: SettingWithCopyWarning:
A value is trying to be set on a copy of a slice from a DataFrame.
Try using .loc[row_indexer,col_indexer] = value instead

See the caveats in the documentation: https://pandas.pydata.org/pandas-docs/stable/user_guide/indexing.html#returning-a-view-versus-a-copy
  self[k1] = value[k2]
sys:1: PerformanceWarning: indexing past lexsort depth may impact performance.
Entry for electricity-demand does not exist in /home/unndreay/Workspaces/oemo-flex/oemoflex/postprocessed_paths.yaml.
Entry for heat-demand does not exist in /home/unndreay/Workspaces/oemo-flex/oemoflex/postprocessed_paths.yaml.

Debugging needed for the exact code line to be found.

This is resolved in #121


chore: bdd-nh-1592924026 to 0.0.1

chore: Promote bdd-nh-1592924026 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Feat/add share session unit

Related issue: #847

Support the longer expired duration,  3 days, 7 days, and 30 days.

LGTM!
Btw, you can use close #847 at the beginning of the PR description to close the related issue automatically, also, you can attach some screenshots for UI changes.

Gotcha, automation is awesome!

4 week is confusing, but it's fine for temporary.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

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

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

Questions ?
Please refer to the Backport tool documentation


Through association id datetime fix

Resolves Issue #1265.
I'll do and fix anything you tell! Just please - merge this.

@elennaro I'll look into this PR as it shouldn't crash the server but you shouldn't use datetime as a primary key. It's not guaranteed to be unique.

Actually I should have looked at this more. item is missing the var statement and I'm not sure what the end goal is as just assigning it to item doesn't do anything. Going to revert this for now.

@particlebanana
Oh gee! Sorry!
I'm fixing this right now.
I'll make a new PR asap.
P.S. Thanks, for responding so quickly, we are suffering from lack of 'datetime' much.


[SECURESIGN-655] Enable OIDCs with private or self-signed certificates

Resolve #206

/LGTM

/retest

/retest

/LGTM

/test all


[trading] IPFS deploys do not work for subfolder gateways

next.js builds static pages, and those static pages reference static assets as /_next/..., which works fine if the
site is served from a domain root (e.g. console.fairground.wtf/, made-up-url.ipfs.io) but breaks down on IPFS
gateways that are still configured to use subfolders (e.g. ipfs.io/ipfs/made-up-url). Luckily we can fix this by
using assetPrefix to turn /_next/ in to ./_next/, which works in both cases.
Tasks

[x] Update configuration for trading

Blocks #597

The fix implemented for this broke many routes in the app, so was reverted by @mattrussell36.

Closing as this is covered by #1921


Upstreaming changes required by the Winit keyboard rework

If you want me to split this out into separate PRs per logical change, I can do that, but each commit here should hopefully be self-contained. Of all these changes, the only one that's probably technically necessary is the one adding a couple of functions and a type alias. The changes required for some older versions of Ubuntu might also benefit from some de-duplication, but I've refrained from doing that for now.

I'm surprised you found this repo! This is great, but I think it'd be a good idea for me to transfer this repo to rust-windowing; do you agree?

I'm surprised you found this repo!

I can't even remember how I found it, haha.

This is great, but I think it'd be a good idea for me to transfer this repo to rust-windowing; do you agree?

Sure, that sounds good to me.

Done; feel free to change the authors in Cargo.toml to not be fran-centric, since I'm fran-kly not the original author of this code anyway.
Also feel free to merge this (and do literally anything you want in general), since I'm not really in a position to maintain this anymore.

Would you be willing to add @maroider to the list of maintainers on crates.io, then?

@madsmtm invites sent!


Can't create multiple listeners for multiple hosts

Hello,
I'm using Agic for AKS. It's working great on one host.
I created a second namespace with another application and decided to publish it through the app gateway.
I used the same yaml file as my 1st app, changing host/service/namespace.
It worked but my 1st app was not available anymore and all traffic were redirected to my 2nd app.
How can I use my appgw to create 2 sets of backend/listener/rule and so on... ?
Here is my yaml file :
`apiVersion: extensions/v1beta1
kind: Ingress
metadata:
name: web-ingress
namespace: mynamespace
annotations:
kubernetes.io/ingress.class: azure/application-gateway
appgw.ingress.kubernetes.io/appgw-ssl-certificate: mycert
appgw.ingress.kubernetes.io/ssl-redirect: "true"
appgw.ingress.kubernetes.io/request-timeout: "180"
spec:
rules:

host: my.host.com
http:
paths:

path: /
pathType:
backend:
serviceName: myservice
servicePort: myserviceport`

In the app gw configuration I can see that I have 2 backend pools, but no rules associated with my first app:

So I only have a set of 2 rules and 2 listeners (80 and 443 ports)

But no hostname configured so no multisite capabilities :

How can I resolve my issues ?
Best ,

Hello community,
I resolve my problem by manually create a new listener, modify the listener types to multi site , create a rule to bind backend to listener and then it works.
Is there a way to manage this configuration through K8S yaml files ?
Best,

@beaverdevtools can you share both Ingress YAMLs ?

Hello,
sure :
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
name: my-ingress-A
namespace: namespaceA
annotations:
kubernetes.io/ingress.class: azure/application-gateway
appgw.ingress.kubernetes.io/appgw-ssl-certificate: myWildcardCert
appgw.ingress.kubernetes.io/ssl-redirect: "true"
appgw.ingress.kubernetes.io/request-timeout: "180"
spec:
rules:

host: hostA.mydomain.com
http:
paths:

path: /
pathType:
backend:
serviceName: my-service
servicePort: 8080

apiVersion: extensions/v1beta1
kind: Ingress
metadata:
name: my-ingress-B
namespace: namespaceB
annotations:
kubernetes.io/ingress.class: azure/application-gateway
appgw.ingress.kubernetes.io/appgw-ssl-certificate: myWildcardCert
appgw.ingress.kubernetes.io/ssl-redirect: "true"
appgw.ingress.kubernetes.io/request-timeout: "180"
spec:
rules:

host: hostB.mydomain.com
http:
paths:

path: /
pathType:
backend:
serviceName: my-service
servicePort: 8080

remove - from in front on http from both ingress. - is making host and http to be evaluated as separate rules.

Alright! Thanks, I learned something! I'll give it 24h before closing the issue :) Thanks again!


export a workload log

I am a new user of YCSB. I just learned that YCSB can run some predefined workloads. Is it possible to export a workload log? In other words, it is expected that the workload log can include some features for each DB access record, such as

operation: read, insert, or update
key: a key id
value: the value of a key
timestamp: when to execute the operation.

Is that possible for YCSB?
Thank you.

I don't believe we output a log of each individual operation as we run against a datastore. You could maybe leverage our htrace integration and have it log to a file, but I'm not sure the trace ID includes the key


Furture warning form invert_font_size.py

i got this warning. how can i silence the warning?
Would it be a big problem on generate SynthText image?
I work on python 2.7 ubuntu18.04
I can't find variable rcond either.
Plz help me!

Hey Are you able to run it ? Please we are trying from 2 months , Please reply

You can ignore the warning, or use a more recent version of the codebase.


help

is there any way to dim behind of the loadtoast like alertDialog??

No, what you see is what you get really, not that advanced of an API behind this library!


docs: add IBM Z in the README

Fixes: #1039
Signed-off-by: Alice Frosi<EMAIL_ADDRESS>

lgtm
@alicefr - you could add another commit to this PR if you wish to remove those other 's's so all the entries are consistent.

Travis seems unhappy:
INFO: Checking all document URLs
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
Check the details on how to adjust your build configuration on: https://docs.travis-ci.com/user/common-build-problems/#Build-times-out-because-no-output-was-received
The build has been terminated

I've restarted it..

@jodh-intel I push with IBM Z as we agreed, or do you want 2 commits?

oh got it! :sweat_smile:

@jodh-intel done!

Thanks @alicefr! :smile:
lgtm

Oh - could you just update that 2nd commit? Our tooling is complaining as there is no "body" to that commit:
ERROR: Commit af0d1f37bddef2cff5f1df2773ad95dbc6cbf44f: single-line "Signed-off-by" body not permitted
ERROR: checkcommits failed. See the document below for help on formatting
commits for the project.

https://github.com/kata-containers/community/blob/master/CONTRIBUTING.md#patch-format

Done


package pg_query_go/v2 not found

I was following instruction:
go get github.com/pganalyze/pg_query_go/v2
go get: module github.com/pganalyze/pg_query_go@upgrade found (v1.0.3), but does not contain package github.com/pganalyze/pg_query_go/v2

go version go1.16.13 linux/amd64

It appears that Go produces this error when you don't specify the version explicitly.
This works as expected:
go get github.com/pganalyze/pg_query_go/v2@latest

go get github.com/pganalyze/pg_query_go/v2@latest
go get: package github.com/pganalyze/pg_query_go/v2 is in the main module, so can't request version latest

@tedyu Weird - I don't really know why that's not working for you. It works fine here on Go 1.17.

$ go version
go version go1.17.7 linux/amd64

I think this issue should be reopened.

12:38 /tmp $ go version
go version go1.18.3 darwin/amd64
12:40 /tmp $ go get github.com/pganalyze/pg_query_go/v2@latest
go: can only use path@version syntax with 'go get' and 'go install' in module-aware mode


Asynchronous cache warm up

My main feedback is on the thread infrastructure. If I understand correctly, requests are put into processQueue. A scheduled thread wakes and moves items from processedQueue to rangeMap. Another scheduled thread will wake up and process rangeMap.
Instead is it possible to executor pool which also gives a queue. Items are put into the queue and threads process these items. The size of the executor pool can be configurable. This will simplify the code and make it more testable. Also a unit test for this part will improve confidence.

Updated the PR based on feedback.

A couple of other changes from an offline conversation:

Split RemoteFetchProcessor to its own class.
Look into other examples of Throttling and Batching queue internal to Qubole.

The PR is ready for another review. Changes done since the last review:

Added unit tests
Broken RemoteFetchProcessor class into two class and divided the responsibility. RemoteFetchProcessor takes the request out of the queue and merges the requests. FileDownloader then creates the request chain list from the merged requests and processes the download requests.

Pull Request Test Coverage Report for Build 30

3 of 396 (0.76%) changed or added relevant lines in 15 files are covered.
11 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-3.6%) to 15.622%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

rubix-core/src/main/java/com/qubole/rubix/core/RemoteReadRequestChain.java
0
1
0.0%

rubix-core/src/main/java/com/qubole/rubix/core/ReadRequestChain.java
3
4
75.0%

rubix-core/src/main/java/com/qubole/rubix/core/CachingFileSystem.java
0
1
0.0%

rubix-core/src/main/java/com/qubole/rubix/core/DirectReadRequestChain.java
0
3
0.0%

rubix-bookkeeper/src/main/java/com/qubole/rubix/bookkeeper/LocalDataTransferServer.java
0
3
0.0%

rubix-bookkeeper/src/main/java/com/qubole/rubix/bookkeeper/BookKeeper.java
0
9
0.0%

rubix-core/src/main/java/com/qubole/rubix/core/ReadRequest.java
0
10
0.0%

rubix-bookkeeper/src/main/java/com/qubole/rubix/bookkeeper/DownloadRequestContext.java
0
13
0.0%

rubix-bookkeeper/src/main/java/com/qubole/rubix/bookkeeper/FetchRequest.java
0
14
0.0%

rubix-core/src/main/java/com/qubole/rubix/core/RemoteFetchRequestChain.java
0
27
0.0%

Files with Coverage Reduction
New Missed Lines
%

rubix-core/src/main/java/com/qubole/rubix/core/ReadRequest.java
1
46.51%

rubix-core/src/main/java/com/qubole/rubix/core/NonLocalReadRequestChain.java
1
0.0%

rubix-core/src/main/java/com/qubole/rubix/core/DirectReadRequestChain.java
2
0.0%

rubix-bookkeeper/src/main/java/com/qubole/rubix/bookkeeper/BookKeeper.java
3
0.0%

rubix-core/src/main/java/com/qubole/rubix/core/CachingInputStream.java
4
0.0%

Totals

Change from base Build 26:
-3.6%

Covered Lines:
299

Relevant Lines:
1914

💛  - Coveralls


Create ReadmeForFirstBranch

using shortcut for first pull request via web gui

this looks fine

close his out

web gui sucks
Bot makes matters worse


onChange event cover original value

Problem
This is my initial value, just
blog

When I switch to another article with
xxx
 ，Then switch back again, the value changed...
blog

I think it has something to do with the source param of the onChange, set to a new value only if source == "user"


Couldn't find g

Hi,
I tried using the bundle to give go a try but I cannot make it works. Go is installed by homebrew and I have all the go* binaries in my path but when I hit Apple+R all I get is an error "Couldn't find g" with all the locations in the search PATH printed below.
I tried to find where the error occured in the code without luck (the path followed is not easy to find with one executable calling another and another including another one xD).
Any idea what could go wrong ?

I also use Homebrew's Go too.  You could try my fork and manually specify GOBIN to /path/to/homebrew/bin in Textmate. https://github.com/KnightBaron/go-tmbundle

I will try your fork when I get the chance but what I really don't understand is why it is looking for "g" where as far as I know there is no such binary in the go suite Oo

The script prefix "g" to 5g, 6g and 8g as necessary depends on the platform compiling. I forgot the detail but I remember that I got the same error and fixed that on my fork before.

I found the problem thanks to your fork which did not work but helped me find the issue:
I have a macbook pro under Lion and the reported value for "sysctl -n hw.machine" is "x86_64" which is unknown by the goarch script (btw it is really weird that the number before the "g" represents the architecture, it is a little scary) so the architecture "number" was an empty string resulting in gobin trying to run "g".

no problem, I was sure I had closed this idea but I must have forgotten to submit it xD


chore(main): release 0.8.0

:robot: I have created a release beep boop
0.8.0 (2023-12-05)
Features

add n8n service to devcontainer (#81) (8772f82), closes #71
create error500.html.twig (#88) (955de59)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/constructions-incongrues/yazoo/releases/tag/v0.8.0 :sunflower:


[Feat.Req] Optionally pass in dependencies

Problem statement
It is useful to be able to know which dependencies are attached to a given POI attestation. This would help us root cause divergence issues.
Expectation proposal
Imagine an Indexer could pass any number of supported dependencies as configuration. If this were a flag, it could take the form:
--dep <type>:<id>=<uri>
poi-radio --dep postgresql:primary=postgresql://host:5432 --dep chain:mainnet=http://geth:8545

For each provided flag, handler logic could be defined for the type that allows POI Radio to extract the version. For example, for the chain type, the POI radio could call web3_clientVersion at the provided uri to get the client version. A SQL statement could similarly be executed to get PostgreSQL version.
The resulting dependency information could be attached to POI messages.
Alternative considerations
None, but please comment with ideas.

Do you think it would make sense for all Graphcast message? or specific to POI radio? Could this be only optionally passed in or something required upon radio start-up?


[BUG] Changing the replica count in Longhorn CSI Driver Settings does not update the deployments

Describe the bug
When changing the csi driver settings replica count in the longhorn catalog application in rancher the longhorn-driver-deployer does not update the replica count on the deployments.
To Reproduce
Steps to reproduce the behavior:

In Rancher install longhorn
After successful install, update the settings for the installed apps changing the csi replica counts
See the upgrade succeed but replica counts not update
longhorn-driver-deployer log shows:

time="2021-04-26T08:58:43Z" level=debug msg="Detected deployment csi-snapshotter CSI version v1.1.0 Kubernetes version v1.20.5 has already been deployed"
time="2021-04-26T08:58:43Z" level=debug msg="Detected daemon set longhorn-csi-plugin CSI version v1.1.0 Kubernetes version v1.20.5 has already been deployed"

etc.
Expected behavior
The replica count is updated on the deployments.
Additional context
This might be an issue not directly related to longhorn but to the rancher catalog app, I'm not sure if this is managed by this.

Do you mean modifying the default settings during the Longhorn upgrade? Any default settings won't be applied during the upgrade. They work for the installation stage only.


[Feature Request] Make AutoBuff a Toggle

https://github.com/CKMud-Mudlet-Scripts/CK/commit/1fc19783268b2d8de921f31061ce0c3c8e2e6705


Karma test do not create coverage report

as title.

Pull request #27

Need to run "npm install" again for downloading the karma-coverage.


"Cioroiași" is listed in the drop-down menu

Description:
"Cioroiași" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Cioroiași" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Cioroiași, DOLJ".


Fix issue #97207 by filtering extension_list.cfg

The implemented solution to the problem of the error message appearing when an excluded GDExtension in an export of a project, is to filter the lines in the extension_list.cfg file to only include those that are in the paths actually included for export.  If there are no entries remaining, don't write the file at all.

I implemented what seems like a straightforward way to address this error message, by filtering out the files in the extension_list.cfg file during the process of exporting.  This seems pretty low risk, but I welcome more eyes on the change.  I tested with both a project with only one extension (which was excluded), and one with two extensions (one of which was excluded) per the minimal reproduction projects supplied in the issue.

Thanks! Congratulations on your first merged contribution! 🎉

Does not work for me. I excluded all extension's files with EditorExportPlugin and it's still included in the extension list and prints errors on launch.
Looks like this only covers the case where files are excluded via the Export dialog.

Does not work for me. I excluded all extension's files with EditorExportPlugin and it's still included in the extension list and prints errors on launch. Looks like this only covers the case where files are excluded via the Export dialog.

I definitely didn't test with EditorExportPlugin, only the traditional Export dialog, so there must be something else when you are using that class.  I didn't even know about that class, so that's a bit of new learning for me.  I'm curious though, so I'll see if I can see what that class does differently.  Do you have an example EditorExportPlugin I could test with?

Check this comment: https://github.com/godotengine/godot-git-plugin/issues/77#issuecomment-2212343975

I looked at this and see that the list of paths to process is not updated if a EditorExportPlugin uses the skip() method to drop it from the export list.  I'll write a separate issue and see if there is a way to generalize this.
If you are are using the EditorExportPlugin just to exclude the git plugin, the easier way is to just use the export resource options and select the directories without using a plugin.

I created the following issue: https://github.com/godotengine/godot/issues/99698
I have a relatively simple update to the original change in #97207 that should address this as well.

Well, after doing some additional testing on my "fix" I realized that there were other changes in 4.4 that make supporting an editor export plugin excluding a GDExtensionPlugin like the Git plugin pointless.  I closed issue: 99698
The root problem is that there are two export plugins competing for control of the same files, and there is no guarantee of the order they will run in, and the GDExtensionPlugin has now implemented additional warnings without the knowledge of any other editor export plugins running.
Oh well.

I haven't looked into the details, but I just want to make sure we're not adding workarounds to solve a symptom instead of handling the root problem properly.
The Git plugin is obviously a type of extension that nobody would want included in exports. So the extension system needs to have support for extension authors to register their extensions as editor only / no export, and this should be used for the Git plugin.
End users shouldn't have to manually exclude its files from the export presets.

Well that is the current strategy to prevent the files for an addon that is only for the editor out of the package when they export their game.  That is independent of whether it had an extension or not.  In the case where there was an extension there was an extra twist to keep out just the line for that unwanted addon from extension_list.cfg.
Are you suggesting to add an option into plugin.cfg that would describe this plugin as being for the editor only?
That seems like it would make all the user's lives considerably easier.

The Git plugin is obviously a type of extension that nobody would want included in exports. So the extension system needs to have support for extension authors to register their extensions as editor only / no export, and this should be used for the Git plugin.

I think adding some sort of "no export" key to .gdextension files could make sense! However, there is some overlap between that and the ideas about global extensions that have been going around. Once we support global extensions, the Git plugin would almost certainly become one. So, I'd want to be careful not to add a key in such a way that would cause conflicts or confusion later.

The Git plugin is obviously a type of extension that nobody would want included in exports. So the extension system needs to have support for extension authors to register their extensions as editor only / no export, and this should be used for the Git plugin.

I think adding some sort of "no export" key to .gdextension files could make sense! However, there is some overlap between that and the ideas about global extensions that have been going around. Once we support global extensions, the Git plugin would almost certainly become one. So, I'd want to be careful not to add a key in such a way that would cause conflicts or confusion later.

Sounds like this discussion should be tied into whatever proposal is tracking this potential feature.  In the meantime people will just have to manually exclude this type of addon.  Is there a link to the global extension proposal?

Is there a link to the global extension proposal?

Here's the proposal but I don't think it captures all of the discussion that has gone on around this


Add a docker image to run this API server

The existing Dockerfile is outdated to libvarnam.
The image should have

govarnam
all schemes
this varnamd

Can you assign it for me?

Thank you for the interest @anhlee66 but this has been done via #9


Add an option to include crate versions to the generated docs

Fixes https://github.com/rust-lang/cargo/issues/1681

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Looks great!
Can you add a section to the bottom of unstable.md briefly describing the feature?  I created a tracking issue you can link to: https://github.com/rust-lang/cargo/issues/7907

Looks great!
Can you add a section to the bottom of unstable.md briefly describing the feature? I created a tracking issue you can link to: https://github.com/rust-lang/cargo/issues/7907

Thank you! I have added it as a separate commit (please indicate if you would prefer it squashed), linked to the issue and rebased the branch on master.

:umbrella: The latest upstream changes (presumably #7820) made this pull request unmergeable. Please resolve the merge conflicts.

Thanks!  A separate commit is probably fine, I normally only squash if there are a bunch of commits, or commits are changing previous ones in the same PR.
@rust-lang/cargo FYI, this is a really minor new nightly-only feature, so I'm not going to bother with FCP.  If there are changes (or if we think we should never do this), feel free to leave comments on the tracking issue.
@bors r+

:pushpin: Commit fc5e17ab5dee294346355a6b3a1546f0ceff9e5c has been approved by ehuss

:hourglass: Testing commit fc5e17ab5dee294346355a6b3a1546f0ceff9e5c with merge bcc82b10c8097f51d56b95063c77a9646130fed7...

:broken_heart: Test failed - checks-azure

@bors retry
Failed to fetch https://packages.microsoft.com/ubuntu/16.04/prod/dists/xenial/main/binary-amd64/Packages.bz2  Hash Sum mismatch

:hourglass: Testing commit fc5e17ab5dee294346355a6b3a1546f0ceff9e5c with merge f0a7bf7dcd621de09ce17851bc8569b0cf90032a...

:sunny: Test successful - checks-azure
Approved by: ehuss
Pushing f0a7bf7dcd621de09ce17851bc8569b0cf90032a to master...


Svelte kit starter is broken

there have been some changes in svelte.config.js
the current config
import adapter from '@sveltejs/adapter-node';

/** @type {import('@sveltejs/kit').Config} */
const config = {
    kit: {
        adapter: adapter({ env: { port: process.env.PORT } })
    }
};

export default config;

throws this error
spur-web on  main [$] is 📦 v0.0.1 via  v16.14.2 took 6s
❯ npm run build                                                                                (base)

><EMAIL_ADDRESS>build
> svelte-kit build

> options.env has been removed in favour of options.envPrefix. Consult the adapter-node README: https://github.com/sveltejs/kit/tree/master/packages/adapter-node
    at default (file:///Users/rohanrajpal/Documents/spurchat/code/spur-web/node_modules/@sveltejs/adapter-node/index.js:17:9)
    at file:///Users/rohanrajpal/Documents/spurchat/code/spur-web/svelte.config.js:7:18
    at ModuleJob.run (node:internal/modules/esm/module_job:197:25)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:337:24)
    at async load_config (file:///Users/rohanrajpal/Documents/spurchat/code/spur-web/node_modules/@sveltejs/kit/dist/cli.js:785:17)
    at async file:///Users/rohanrajpal/Documents/spurchat/code/spur-web/node_modules/@sveltejs/kit/dist/cli.js:924:19

I'm not really able to understand how to fix it. Svelte kit docs also don't mention how to change the port.
The link below tho might be helpful
https://timdeschryver.dev/blog/environment-variables-with-sveltekit#the-problem

Okay, so no need to explicitly set the port now. Just setting the PORT environment variable works. Making a PR


feat: response models and schema parsing for constructor payloads

:tada: This PR is included in version @wapijs/wapi.js-v0.0.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


mark dirty section and rows cleaned after they are rendered

Currently, if dataSource is changed, the dirty sections and rows will never get "cleaned", causing them to be re-rendered every frame afterwards.
After a row or section is rendered, we "clean" the corresponding item in the dataSource's _dirtySections and _dirtyRows.
Test plan:
Log to console each time a row renders
Update datasource for a single row
Check that the row only ever re-renders once
This is related to pr: #7405
but the fix is moved out of rowShouldUpdate in ListViewDataSource.

related to this issue: https://github.com/facebook/react-native/issues/5255

@ericvicenti
I originally tried doing this in componentDidMount but couldn't figure out how to only "clean" the sections/rows that were actually rendered and keep the un-rendered ones still dirty. I guess we could keep an set of "to-be-cleaned" section/rows, update that in the render() method and then clean those in componentDidMount()..?
updated pr to fix:
-updated the var section declaration to const
-moved the dirty section cleaning code to be inside the this.props.renderSectionHeader check. In the original code, the list view would not have rendered the section if a renderSectionHeader method was provided to it later.

@ericvicenti not sure why tests are failing after merging in master. seems like some android test device is down?


cluster-autoscaler shouldn't be scheduled on spot

Update documentation to schedule cluster-autoscaler on normal instances.
PR o'clock
Description
The documentation states the cluster autoscaler should be scheduled on spot instances. This is incorrect. It's the exact opposite.

@barryib Thanks. updated.


Rename symbol (F2) does not work anymore

VSCode Version: 1.50.1, commit d2e414
OS Version: Linux x64 5.4.0-51-generic snap
Node.js: 12.14.1

Steps to Reproduce:

Open vscode with code
Set cursor on variable name or function name
click F2
Expected: you can edit the name of the symbol to be renamed
Actual: nothing happens, so can not rename the symbol.

Notes:

this used to always work, maybe because I updated to the latest version?
in the tree explorer I can rename a file or directory, but not variables/functions in code.
I use Intelephense extension. But same issue appears after uninstalling all extensions, also tried with Code Insiders.
Checked the developers tool bar, there were no errors shown when I pressed F2. Actuall nothing was shown as if the F2 is not binded to an action.

Does this issue occur when all extensions are disabled?: Yes

Could be https://github.com/microsoft/vscode/issues/108679. Which Linux distro and version are you using?

@chrmarti I am using
NAME="Ubuntu"
VERSION="18.04.5 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.5 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic

@eddy147 Can you please follow the steps at https://github.com/microsoft/vscode/wiki/Keybinding-Issues#basic-troubleshooting
It would be interesting to see what the log output for pressing F2 is.

Hi @alexdima
see attachment, hopefully it is useful.
Let me know if I can do something else.
Thx for your help.
[
f2.log
](url)

@eddy147
This is the relevant section:
[2020-10-15 11:00:06.349] [renderer1] [info] [KeybindingService]: / Received  keydown event - modifiers: [], code: F2, keyCode: 113, key: F2
[2020-10-15 11:00:06.349] [renderer1] [info] [KeybindingService]: | Converted keydown event - modifiers: [], code: F2, keyCode: 60 ('F2')
[2020-10-15 11:00:06.350] [renderer1] [info] [KeybindingService]: | Resolving [F2]
[2020-10-15 11:00:06.350] [renderer1] [info] [KeybindingService]: \ From 5 keybinding entries, no when clauses matched the context.

So it looks like on your machine there are 5 keybinding entries that are bound to F2, but none of them matched the current context. This is very interesting, for example on mac we only ship with one keybinding that targets F2:
{ "key": "f2",                    "command": "editor.action.rename",
                                     "when": "editorHasRenameProvider && editorTextFocus && !editorReadonly" },

So I wonder if you have perhaps installed extensions that customize keybindings or if you have written custom rules in your personal keybindings.json ?

Thank you so far.
This was my ~/.config/Code/User/keybindings.json
// Place your key bindings in this file to override the defaultsauto[]
[
    {
        "key": "f2",
        "command": "-renameFile",
        "when": "explorerViewletVisible && filesExplorerFocus && !explorerResourceIsRoot && !explorerResourceReadonly && !inputFocus"
    },
    {
        "key": "f2",
        "command": "-remote.tunnel.label",
        "when": "tunnelViewFocus && tunnelType == 'Forwarded'"
    },
    {
        "key": "f2",
        "command": "-debug.setVariable",
        "when": "variablesFocused"
    },
    {
        "key": "f2",
        "command": "-debug.renameWatchExpression",
        "when": "watchExpressionsFocused"
    }
]

I removed them, having now:
I did not add these myself.
After removing them and adding what I think is the default:
// Place your key bindings in this file to override the defaultsauto[]
[
    {
        "key": "f2",
        "command": "editor.action.rename",
        "when": "editorHasRenameProvider && editorTextFocus && !editorReadonly"
    },
    {
        "key": "f2",
        "command": "-renameFile",
        "when": "explorerViewletVisible && filesExplorerFocus && !explorerResourceIsRoot && !explorerResourceReadonly && !inputFocus"
    }
]

Did not help, I am getting the same error as in the log you showed me earlier.
Also removing this keybindings.json file did not help either.
Is there a way to remove everything (cache/settings) and start totally clean?

After upgrading to 20.04, and reinstalling vscode, F2  works again.
No clue what caused the problem though.

Thanks for investigating. Clearing out your keybindings.json should fix things for you. I also don't know what might have caused these keybindings rules to show up in your keybindings.json. Normally, they show up when the keybindings editor UI is used and rich click > remove keybindings is invoked.

Thank you for your efforts and such a great editor.


Disabled Local User across all storage accounts

Context
AB#228879 AB#228386
Change proposed in this pull request
Also modified Pipeline Runs query to remove dependency on current environment name
Guidance to review
This change has been deployed to d12 feature environment and validated to have no detrimental impact on the product.
Checklist (add/remove as appropriate)

[x] Work items have been linked (use AB#)
[ ] Your code builds clean without any errors or warnings
[ ] You have run all unit/integration tests and they pass
[x] Your branch has been rebased onto main
[x] You have tested by running locally

Merged in to unblock any potential hotfix that might be required


Replace Unqual-based idiom with immutable-based idiom

Aplying immutable to T makes it immutable T regardless of whatever qualifiers T has. This allows for a simple idiom for type comparison, per this PR.

std/variant.d(500:120)[warn]: Line is longer than 120 characters

@thewilsonator fixed line length, will tag for merge since you approved. Thanks!


[FR] Let users set the action mode from the Alerts tab

If an alert is identified as a threat, it should be possible for users to quickly change its state from "Alert" to "Drop". Instead, they currently have to

make the window wide so that they can copy the full alert as there can be several rules starting the same
copy the rule
go to the Rules tab
paste the rule in the search field
open the rule

It could just be:

open the rule from the Alerts tab

Ideally you should be able to open the same dialog as in the rules tab, maybe from a separate button in the same grid or some kind of link in the info dialog using the same service calls we already have.
Let me think about this.


Update kong routes and lock down container ports

This update improves the definition of the LFH API in Kong from an API gateway perspective, with a Kong route per LFH API. In addition, external container ports were locked down to limit access to containers except via Kong.
The LFH API is now accessed via HTTPS through Kong, for example:
https://<IP_ADDRESS>:8443/fhir/r4/Patient
Changes were tested against compose dev and server profiles on amd64 and oci container scripts on LinuxONE and Azure.
This PR includes an updated postman collection and a doc update will follow.

The test cases passed with an update and after re-running the jobs.