No hash in name in production build

Hi
I am wondering if there is a setting to disable the hash in the name for main bundle and style bundle for production builds?
/Fredrik

No there is not.
You could fork the repo and edit ./angular-cli/packages/angular-cli/models/webpack-build-production.ts to remove it if you really want to.

^ what @grizzm0 said 💃

I could fork it but I would rather have the option of specifying this in the angular-cli.json, it seems strange to me to have this as a hardcoded feature

angular-cli is built to "just work" with as little options as possible. If you want more complex usage you should probably have your own webpack setup instead.
Imagine the config file if we would accept every single use case.
An addon system is in the plans and has been discussed in #1656.

Sure, I understand that we can't add settings for everything but when you default to this type of behavior it would be good to at least be able to turn it off. What is the time frame for this addon system? Why not just expose the webpack config directly like you do with karma and protractor?

You could say that about anything that is enabled by default. What's the reason you want to disable the hashing for?

Exactly, but for karma and protractor there is config files, why not webpack? I have nothing against defaults that make it "just work" for most people but it should still be possible to change thing using the configuration.
Our reason it this:
We push the files to a cdn using a inhouse system which we have to use. This system uses a config file where we define the file that should be put there as part of our build. It does not handle wildcards in the filename and because of this we can't use the production build and don't get the tree shaking and everything else.
I guess I have to do a workaround for this but it feels sad that the only options is forking or going the webpack route directly, whats the point of the cli then?

@fredrik-holmqvist I understand this is a need you have, but surely you understand that we could not predict that your in-house CDN has a problem with wildcards. The CLI doesn't aim to cover every scenario possible.
If this is a big need for your org I encourage you to make a PR to support such an option, preferably on the build command. Otherwise at the moment I'd say this is one of the lowest priority requests around.
But a workaround here seems much easier than that: have a script that globs your files in dist/ after a prod build, renames them to remove the hash, and does the same in index.html.

It's not an in-house CDN and it's not that it doesn't support wildcards. It's the way we version, we want it to be predictable so that teams can upgrade to a new version in a predictable way, without having to know any hashes, more like you version libraries on CDNs. And for this we would have liked for it to not touch the names of the files when you do a production build, but I get it, we are not really the core audience for this tool, to much custom stuff unfortunately
We have already made a workaround for this so this issues is resolved from our side

@fredrik-holmqvist I've received another request for this, and tbh I think I closed this too soon. If more people want this, they wouldn't find the issue, nor work a PR to add it.

Same request, my use case is server side rendering. I need to replace several variables in html string (preferably, just like React way).
<script>
    window.__INITIAL_STATE__ = ${JSON.stringify(initialState)}
</script>

But cli pushed some unpredictable hashes into index.html so the option left to me is to use that index.html rather than html string. It's kind of hacky 'cause I need to read the html file and replace the strings. It would be much easier/faster to hard code if we can.

Sorry, I forgot to reopen this on my last comment!

How about this:
1.  A flag to (enable / disable) hashing
2. An include list - files & wildcards to hash
3. An exclude list - files & wildcards to exclude
The above may include the assets directory as well.
The default can be left as whatever behavior we have today.
This way, we can add the assets directory in the mix as well for when hosting on CDN.
Those who don't need hashing, simply turn it off.

@grizzm0 One reason not to hash would be to use an external tool (e.g. gulp plugin) to hash everything, even the images, especially if you are going to host the project on let's say AWS S3/CloudFront.
Now, if NG-CLI took care of all the files (*.png, *.gif, etc.) in the assets directory as well, then one could say angular-cli is built to "just work".
However, as it stands, since there is no non-forky way to disable the hashing, any external hashing tools will create ugly double-hashed names for the .js/.css/.map/.gz files that NG-CLI already hashed. But the images, for example, will have single-hash in their names.
One might say, well that is not a big deal.  However, since ng-cli is a community-driven project, we might want to make it work elegantly and for everyone.
Options with default values may be the right approach here.

BTW this is very easy to do on your side as a part of the deployment.
just do $ mv inline.*.bundle.js inline.bundle.js etc..
then in your index.html have
<script src="build/js/inline.bundle.js"></script>
+1 to a flag, though

I would also need this option. I include the js files in a CMS and don't want to update the html snippet every time I make a new release of my angular 2 application.

Any update? I also need to prevent the hash being generated in the file names for similar reasons

The reason we are using angular-cli is that it eliminates configuration process and provides the same workflow for each project. It just does not make sense to repeat all configuration and build steps with webpack or gulp or rollup, because angular-cli already does what we need (AOT + Webpack).
Currently we have to wrap angular-cli with gulp to remove hashes and unused files, it is silly job.

+1 to add this feature.

I found this working now, seems got added already with so many request :)
add  --output-hashing none during build
media: only add hashes to files processed via [url|file]-loaders
bundles: only add hashes to the output bundles
all: add hashes to both media and bundles
So my build command looks like this:- ng build --prod --output-hashing none

I found this working now, seems got added already with so many request :)
add --output-hashing none during build
media: only add hashes to files processed via [url|file]-loaders
bundles: only add hashes to the output bundles
all: add hashes to both media and bundles
So my build command looks like this:- ng build --prod --output-hashing none

I need filenames that don't change but also would like to have a hash to invalidate cache. Is there a reason the hash can't be on the query string?
For example, this gets injected into my index.html:
<script type="text/javascript" src="inline.908517239e551bfc4e9b.bundle.js"></script>
<script type="text/javascript" src="polyfills.d2c49be8f78b2753060a.bundle.js"></script>
<script type="text/javascript" src="vendor.9f2dc5768d3730cb8f9d.bundle.js"></script>
<script type="text/javascript" src="main.6fa8c063126f9ead769a.bundle.js"></script>

How about:
<script type="text/javascript" src="inline.bundle.js?908517239e551bfc4e9b"></script>
<script type="text/javascript" src="polyfills.bundle.js?d2c49be8f78b2753060a"></script>
<script type="text/javascript" src="vendor.bundle.js?9f2dc5768d3730cb8f9d"></script>
<script type="text/javascript" src="main.bundle.js?6fa8c063126f9ead769a"></script>

@kevindqc With query string there's no guarantee that the files will be cached.
Squid for example used to have (I think it has changed) a default configuration that didn't cache dynamic content and query strings were considered to be dynamic content.

Temporary solution...
In your package.json > scripts:
"build": "ng build --prod && mv dist/main..bundle.js dist/main.js && mv dist/vendor..bundle.js dist/vendor.js && mv dist/inline..bundle.js dist/inline.js && mv dist/polyfills..bundle.js dist/polyfills.js && cp -f src/index.dist-default.html dist/index.html"
The last operation of the build command above is to overwrite the index.html with something like  "index.dist-default.html" in your src and set all script sources without the hash like this...
Example of scripts In your index replacement file (index.dist-default.html or whatever):
`

`

if you want build file like

first set build line like
ng build --prod --output-hashing none
second update webpack config,the file at<EMAIL_ADDRESS>update HtmlWebpackPlugin config,
new HtmlWebpackPlugin({     ...     hash: true // add  }),
this is just i find...hopeful official solution

I tested the above and it works well, resulting in the following.

IMHO, this solution provides the best of both worlds:
(1) deploy-able files that are not renamed on each and every build which is desirable by many developers for SCM purposes (tracking, comparison, history etc.) and
(2) attaches a unique hash as a query string which, in most cases, will result in uniquely versioning your file in the browser cache so the browser reads the new version of the artifacts on each new deployment rather than reading it from cache.
However, it requires forking the webpack config,the file at<EMAIL_ADDRESS>to add one configuration line to the new HtmlWebpackPlugin section (... ,hash: true // add }). To avoid a fork to Angular CLI, recommend a new option
ng build --prod --output-hashing partial
where partial adds a hash query string but does not add a hash to files.
This would be very useful and avoid forking to implement the best solution.

Any chance to reopen this issue for a solution like @dgecawich suggested? (move hash to query string) Or should we open a new issue?
I have a usecase that would need the same solution. A Piwik plugin tries to load the styles of our site at a later point but without a fixed filename it will already be gone. However, disabling the hashing completely will lead to trouble with the cache. Hash in the query string seems to have the best of both worlds.

@lihuabest Is there a way to replace the hash number with a specified version number? Something like this:
<script type="text/javascript" src="main.bundle.js?version=1.2.0"></script>

in angular.json
"configurations": {
            "production": {
                           ...
                           "outputHashing": "none",
                           ...
             }
}


Fix errors load config

This PR:

trim spaces in rules:  Host:test.fr, test.com ; Path:/test => Host:test.fr,test.com;Path:/test
don't break the whole config while only one part is wrong => skip frontend instead of return

Fixes  #459

LGTM 🐯
But needs a rebase :angel:

LGTM


ServiceMonitor for services without portname

What is missing?
On EKS, where coreDNS is installed by default, the metrics port does not exist on the service by default. I tried to create a service monitor for the port:
apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
  labels:
    k8s-app: coredns
  name: coredns
  namespace: monitoring
spec:
  endpoints:
  - bearerTokenFile: /var/run/secrets/kubernetes.io/serviceaccount/token
    interval: 15s
    targetPort: 9153
  jobLabel: k8s-app
  namespaceSelector:
    matchNames:
    - kube-system
  selector:
    matchLabels:
      k8s-app: kube-dns

But Prometheus wasn't able to find the target, this is the generated config:
job_name: monitoring/coredns/0
  honor_timestamps: true
  scrape_interval: 15s
  scrape_timeout: 10s
  metrics_path: /metrics
  scheme: http
  kubernetes_sd_configs:
  - role: endpoints
    namespaces:
      names:
      - kube-system
  bearer_token_file: /var/run/secrets/kubernetes.io/serviceaccount/token
  relabel_configs:
  - source_labels: [__meta_kubernetes_service_label_k8s_app]
    separator: ;
    regex: kube-dns
    replacement: $1
    action: keep
  - source_labels: [__meta_kubernetes_endpoint_port_name]
    separator: ;
    regex: 9153
    replacement: $1
    action: keep
  - source_labels: [__meta_kubernetes_endpoint_address_target_kind, __meta_kubernetes_endpoint_address_target_name]
    separator: ;
    regex: Node;(.*)
    target_label: node
    replacement: ${1}
    action: replace
  - source_labels: [__meta_kubernetes_endpoint_address_target_kind, __meta_kubernetes_endpoint_address_target_name]
    separator: ;
    regex: Pod;(.*)
    target_label: pod
    replacement: ${1}
    action: replace
  - source_labels: [__meta_kubernetes_namespace]
    separator: ;
    regex: (.*)
    target_label: namespace
    replacement: $1
    action: replace
  - source_labels: [__meta_kubernetes_service_name]
    separator: ;
    regex: (.*)
    target_label: service
    replacement: $1
    action: replace
  - source_labels: [__meta_kubernetes_pod_name]
    separator: ;
    regex: (.*)
    target_label: pod
    replacement: $1
    action: replace
  - source_labels: [__meta_kubernetes_service_name]
    separator: ;
    regex: (.*)
    target_label: job
    replacement: ${1}
    action: replace
  - source_labels: [__meta_kubernetes_service_label_k8s_app]
    separator: ;
    regex: (.+)
    target_label: job
    replacement: ${1}
    action: replace
  - separator: ;
    regex: (.*)
    target_label: endpoint
    replacement: metrics
    action: replace

I think it's because it's looking for a port with the name 9153, which does not exist of course. I solved it by adding the relevant port to the kube-dns service, but I prefer not to change components that are installed by EKS.
Why do we need it?
For cases where you need to monitor 3rd party services and you don't/can't control the service ports.
Environment
Kubernetes, EKS

kube-prometheus version:
Insert Git SHA here latest

Anything else we need to know?:

Found the issue - the targetPort has to be int, not a string.


Add sbus driver for Linux based autopilot and fix errors of linux_pwm_out

1.Add sbus driver for linux . This driver can be used to read the
inverted S.bus signal and fetch the data of each channel and publish it

Fix the bug of linux_pwm_out, when the protocol is pca9685,
after the init method is executed,the method of determining the return
value of init method is incorrect,this will cause the driver to fail

Add linux_sbus driver to other posix prefixes cmake file

@TSC21  Ok , now you can see the commit .

@TSC21   Here‘s the video.   Youtube is blocked by Great Fire Wall -_-!  So I published it on youku
http://v.youku.com/v_show/id_XMjk3Njg5ODEzMg==.html?spm=a2h3j.8428770.3416059.1

I'm unable to see the video after opening the link.

@TSC21  https://youtu.be/wgkkp-SEo_4
Ok. Last month the corp asked my family to drink coffee just because I purchase the aircraft  6 months ago. -_-!

The video does not load, even on youtube.

@TSC21 https://raw.githubusercontent.com/crossa/raspx4-sbus-rc-in/master/LinuxSbus.mp4
download it

@TSC21 https://raw.githubusercontent.com/crossa/raspx4-sbus-rc-in/master/LinuxSbus.mp4

Looks good. Thanks @crossa. For me is a go to merge.

Apllied @crossa. Thanks


Fix PhpStorm integration instructions, fixes #4083

The Problem/Issue/Bug:

#4083

How this PR Solves The Problem:
Rewrite, please review.
Manual Testing Instructions:
Walk through the instructions.

The thing that the plugin does not do is configuring phpunit. Not sure if it can do that.

Thanks @nico-loeber - incorporated your comments. The one thing I've decided to add is using Jetbrains Gateway on WSL2. Tricky because of having to enable ssh.


Shapely wont install on Windows 10 64

To Whom it May Concern:
I am using Window 10 64 with Python 3.7 and Shapely simply won't install on my system with following error code:
I have notice that Shapely is only availabile from:  https://pypi.org/project/Shapely/
as linux downloads and no windows download version.
Please see screen shot

Please see https://github.com/Toblerity/Shapely#built-distributions.


How to guide for ui.template

I would like to insert a LicenseAgreementDlg just after WelcomeDlg and before InstallDirDlg but don't understand how to proceed ? Indeed the readme.md does not help a lot on this part ... to me, it is unclear how to set  ui.template, and other template configuration fields (componentTemplate , componentRefTemplate , directoryTemplate , wixTemplate ,uiTemplate , backgroundTemplate ) to do so ?!
Could you provide more explanation, or even provide an example on how to set these fields in the configuration object provided to MSICreator()?

@ThierryLeGal What were you able to find out? I imagine your pr fixes the problem, but I myself, have no idea how to add the LicenseAgreementDlg to the template. I was assuming it was just msiCreator.uiTemplate = `, but nothing I input shows up in the generated .wxs

@ThierryLeGal What were you able to find out? I imagine your pr fixes the problem, but I myself, have no idea how to add the LicenseAgreementDlg to the template. I was assuming it was just `msiCreator.uiTemplate = ``, but nothing I input shows up in the generated .wxs
Figured it out by using msiCreator.ui.template. I also saw that your PR fixes a few issues around this so I created my own fork. Hopefully this project hasn't been abandoned.

Indeed (just for others who might be interested), just define your own template such as the following one:
      template: String.raw`<UI Id="UserInterface">
      <Property Id="WixUI_Mode" Value="InstallDir" />

      <TextStyle Id="WixUI_Font_Normal" FaceName="Tahoma" Size="8" />
      <TextStyle Id="WixUI_Font_Bigger" FaceName="Tahoma" Size="12" />
      <TextStyle Id="WixUI_Font_Title" FaceName="Tahoma" Size="9" Bold="yes" />

      <Property Id="DefaultUIFont" Value="WixUI_Font_Normal" />

      <DialogRef Id="BrowseDlg" />
      <DialogRef Id="DiskCostDlg" />
      <DialogRef Id="ErrorDlg" />
      <DialogRef Id="FatalError" />
      <DialogRef Id="FilesInUse" />
      <DialogRef Id="MsiRMFilesInUse" />
      <DialogRef Id="PrepareDlg" />
      <DialogRef Id="ProgressDlg" />
      <DialogRef Id="ResumeDlg" />
      <DialogRef Id="UserExit" />

      <Publish Dialog="BrowseDlg" Control="OK" Event="DoAction" Value="WixUIValidatePath" Order="3">1</Publish>
      <Publish Dialog="BrowseDlg" Control="OK" Event="SpawnDialog" Value="InvalidDirDlg" Order="4"><![CDATA[NOT WIXUI_DONTVALIDATEPATH AND WIXUI_INSTALLDIR_VALID<>"1"]]></Publish>

      <Publish Dialog="ExitDialog" Control="Finish" Event="EndDialog" Value="Return" Order="999">1</Publish>

      <Publish Dialog="WelcomeDlg" Control="Next" Event="NewDialog" Value="LicenseAgreementDlg">NOT Installed</Publish>
      <Publish Dialog="WelcomeDlg" Control="Next" Event="NewDialog" Value="VerifyReadyDlg">Installed AND PATCH</Publish>

      <Publish Dialog="LicenseAgreementDlg" Control="Back" Event="NewDialog" Value="WelcomeDlg">1</Publish>
      <Publish Dialog="LicenseAgreementDlg" Control="Next" Event="NewDialog" Value="InstallDirDlg">LicenseAccepted = "1"</Publish>

      <Publish Dialog="InstallDirDlg" Control="Back" Event="NewDialog" Value="WelcomeDlg">1</Publish>
      <Publish Dialog="InstallDirDlg" Control="Next" Event="SetTargetPath" Value="[WIXUI_INSTALLDIR]" Order="1">1</Publish>
      <Publish Dialog="InstallDirDlg" Control="Next" Event="DoAction" Value="WixUIValidatePath" Order="2">NOT WIXUI_DONTVALIDATEPATH</Publish>
      <Publish Dialog="InstallDirDlg" Control="Next" Event="SpawnDialog" Value="InvalidDirDlg" Order="3"><![CDATA[NOT WIXUI_DONTVALIDATEPATH AND WIXUI_INSTALLDIR_VALID<>"1"]]></Publish>
      <Publish Dialog="InstallDirDlg" Control="Next" Event="NewDialog" Value="VerifyReadyDlg" Order="4">WIXUI_DONTVALIDATEPATH OR WIXUI_INSTALLDIR_VALID="1"</Publish>
      <Publish Dialog="InstallDirDlg" Control="ChangeFolder" Property="_BrowseProperty" Value="[WIXUI_INSTALLDIR]" Order="1">1</Publish>
      <Publish Dialog="InstallDirDlg" Control="ChangeFolder" Event="SpawnDialog" Value="BrowseDlg" Order="2">1</Publish>

      <Publish Dialog="VerifyReadyDlg" Control="Back" Event="NewDialog" Value="InstallDirDlg" Order="1">NOT Installed</Publish>
      <Publish Dialog="VerifyReadyDlg" Control="Back" Event="NewDialog" Value="MaintenanceTypeDlg" Order="2">Installed AND NOT PATCH</Publish>
      <Publish Dialog="VerifyReadyDlg" Control="Back" Event="NewDialog" Value="WelcomeDlg" Order="2">Installed AND PATCH</Publish>

      <Publish Dialog="MaintenanceWelcomeDlg" Control="Next" Event="NewDialog" Value="MaintenanceTypeDlg">1</Publish>

      <Publish Dialog="MaintenanceTypeDlg" Control="RepairButton" Event="NewDialog" Value="VerifyReadyDlg">1</Publish>
      <Publish Dialog="MaintenanceTypeDlg" Control="RemoveButton" Event="NewDialog" Value="VerifyReadyDlg">1</Publish>
      <Publish Dialog="MaintenanceTypeDlg" Control="Back" Event="NewDialog" Value="MaintenanceWelcomeDlg">1</Publish>

      <Property Id="ARPNOMODIFY" Value="1" />
    </UI>
    <Property Id="WIXUI_INSTALLDIR" Value="APPLICATIONROOTDIRECTORY" />
    <UIRef Id="WixUI_Common" />
    <WixVariable Id="WixUILicenseRtf" Value="C:\Data\GIT_workspace\electron\License.rtf" />
<!-- {{Properties}} -->`

and put your License.rtf file under the path specified in WixVariable "WixUILicenseRtf".


Add Description field to VolumeSetItem

Summary

Add Description field to VolumeSetItem because DescribeVolumes API response has it, but there is no definition in the json file.

Thanks.
This is bug of model json.
This json file is automatically generated in other project, but the json file in this repository does not follow the upstream model json yet. And we do not intent to follow the upstream for a while...
Nomally, we do not merge the manually editted  json files, but in this time, we merge this p-r specially due to above reason.


Fixed some typos in an example page

The CSS property display has no value inline-box. I think you meant inline-block.

Great catch! Thank you :)


Add full GUI app example

Add an example GUI app, with still and video capture options, along with a full set of camera controls to demonstrate the picamera2 and Qt functionality.
Requires #215 to work correctly, as it makes use of Picamera2.sensor_modes to allow selection of raw format.
Also move the app examples into a separate folder.

Really liking this! I have quite a few comments, mostly nit-picks and other really annoying stuff, but also a few more general questions. In no particular order...

I'd quite like to be able to do still captures using a "preview mode" and doing a mode switch. Possibly there could be a check box for "run the preview in the same mode as capture", and this would avoid the mode switch altogether (which is the current behaviour).
I like the HDR feature, that's fun. I'd quite like the app to run even for people who don't install OpenCV, is there a way we could accomplish that? Have the HDR function greyed out? Perhaps in a separate tab if the current one is getting full? Or even a separate app?
The dark background in the preview pane when the aspect ratio is wrong slightly offends my eyes, perhaps we could make the colour closer to the rest of the app?
In the image tuning pane, I wonder about having sliders instead of numbers. It would also indicate the value you've chosen numerically too, and have a "reset" (to default) button.
The pan/zoom is great. But I suppose making the little rectangle (that shows the crop being used) draggable might be a step too far?
For the AGC/AEC controls it might be nice to be able to set the exposure/gain jointly. I guess that means an "Apply" button which would become enabled when you edit either of the values.
I had some problems when enabling/disabling the AWB and changing the mode. There was some weird oscillating behaviour which would sometimes go away if I disabled/enabled it again. Might be worth investigating what's causing that, indeed if there's actually a real problem in the imaging pipeline.

Thanks, I think I've made all those changes.

The pan/zoom is great. But I suppose making the little rectangle (that shows the crop being used) draggable might be a step too far?

I've managed to implement this, so you can drag and scroll the rectangle to pan and zoom.

I had some problems when enabling/disabling the AWB and changing the mode. There was some weird oscillating behaviour which would sometimes go away if I disabled/enabled it again. Might be worth investigating what's causing that, indeed if there's actually a real problem in the imaging pipeline.

This was being caused by setting the colour gains and enabling AWB at the same time, which I've fixed, so not an issue with the pipeline.

As the author of this camera software, I was in the process of upgrading it to take advantage of Picamera2.
Then I stumbled upon this commit and checked out app_full.py as I figured it appeared to be a good starting spot for my application instead.
However, when I try to run it I get an error of `Picamera2' object has no attribute 'sensor_modes'.
On this particular camera, I am using this Arducam module.   I am not sure if this may be causing an issue, because the SD card on my camera that uses an official Raspberry Pi HQ is inside an enclosure that unfortunately would require a lot of disassembly.
Or am I possibly missing some library?

@eat-sleep-code Yes, unfortunately the "official" Picamera2 release is missing some required features. This will be in the next release, but if you're feeling brave you could install or check out the "next" branch (warning: even more bleeding edge software!) from the repository which has the "sensor_modes" functionality.

When will the next release be?  That will help me decide how daring I want to be.  😝

I think we're talking "a few weeks", we'll probably try and time it with an OS update, but that's just my guesstimation of the situation.


BUGFIX: Fix H2 auto increment syntax

H2 has updated such that the old auto increment syntax no longer works

Environment
Liquibase Version: 4.7.1
Liquibase Integration & Version: Java embedded
Liquibase Extension(s) & Version: no extensions
Database Vendor & Version: H2 2.1.210
Operating System Type & Version: Ubuntu Linux 21.04
Pull Request Type

[x] Bug fix (non-breaking change which fixes an issue.)
[ ] Enhancement/New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
H2 has modified its syntax to no longer support AUTO_INCREMENT unless being used in MySQL mode. In other modes, it's been advised that the standard GENERATED AS IDENTITY syntax. Furthermore, it doesn't support the exact syntax so the comma used between the START WITH and INCREMENT BY clauses needed to be (configurably) removed, so I added a means to do just that which only takes effect for H2Database.
Steps To Reproduce
List the steps to reproduce the behavior.

Create a changeset which includes an autoIncrement column
Apply said changeset to an H2 database in PostgreSQL mode

Actual Behavior

Changeset will fail with unrecognized syntax on AUTO_INCREMENT

Expected/Desired Behavior

Changeset will succeed and properly render tables with/modify existing auto-increment columns

Fast Track PR Acceptance Checklist:

[x] Build is successful and all new and existing tests pass
[x] Added Unit Test(s)
[x] Added Integration Test(s)
[ ] Added Test Harness Test(s)
[ ] Documentation Updated

This is a duplicate of #2372


Different icon rendered in released mode?

[x] Review the documentation: https://github.com/oblador/react-native-vector-icons
[x] Search for existing issues (including closed issues): https://github.com/oblador/react-native-vector-icons/issues

Environment
"react-native-vector-icons": "^6.1.0",
"react-native": "0.57.8",

Description
Describe your issue in detail. Include screenshots if needed.
In development mode the icon render as below

And in Released mode the icon render as below

<Icon
        name="menu"
        type="Feather"
        style={{ marginRight: 15, color: 'white' }}
      />

@isaaclem  I had this issue too, what resolved it for me was to execute the unlink command and then link it again:
react-native unlink react-native-vector-icons
react-native link react-native-vector-icons

I hope it helps you!


Getting weird trendline after interaction.

I am getting a weird trendline when LineChart() is used inside a ReorderableListView.builder() widget. I am not sure if these two widgets conflict with each other but whenever I rebuild and hold down on the ListTiles the trendlines appear. More specifically its inside a ReorderableListView.builder(), ClipRRect(), GestureDetector(), Container(), and lastly inside a ListTile() under the title property.  I am using flutter 2.10.1 and below is a vid clip of what it look like.
https://user-images.githubusercontent.com/63486469/175170160-222f25f6-58a9-4328-9956-055a3fe93c00.mp4

Make sure your x axis values are sorted!
Looks like you are adding a FlSpot with the value and the time of your first item to the end of the list.

Thank you so much that fixed the problem. Apparently, I was adding to the map, for my fl_spots, during every rebuild.


 If my understanding is correct, Fontique should map fontconfig's extra black to 950, not 1000.
See also fontique::Weight::EXTRA_BLACK
https://github.com/linebender/parley/blob/0f2608cc562d7c2eef0f34819aeb15166d542b67/fontique/src/attributes.rs#L211-L212

The fallback is hit only in case the input weight is >=215 (fontconfig's equivalent of extra black), so I think it makes sense to default to EXTRA_BLACK in that case (instead of extrapolating).
I'd be happy to make the change to named weights, but can only pick it up after august 12th as I'm currently on holiday.

We correctly map extra black to 950 in the Weight type so this change looks good to me. If you’d like to replace the magic numbers with named constants, that seems like a solid improvement.

(That is the sort of small change which can be made my an approver/merger, so we don't need to wait on the PR author here)


Remove outliers

// reduce a collection by removing outliers for a given property
// Use it like this:
// var col = [{a:20}, {a:30}, {a: '5'}, {a:20}, {a: 10}, {a: '5'}, {a: '5'}, {a: '5'}, {a: 20}, {a: 30}, {a: 50}]
// for the collection examine property 'a' and take the most common 10% for values above 6
// _.removeOutliers(col, 'a', 0.1, 6);

This is a little too specific for core underscore. Would make a great mixin though.


README.md: fix code example

It doesn't look like the old example would have worked since echoName does not exist (unless I am missing something).

Thank you.

My pleasure!


Don't render exception on partial artisan commands

Hey, love the package so far :) Very helpful!
I have found one gripe which is annoying, and maybe a very edge case - but if you deem it fixable then I think it's definitely an improvement.
So if you type something like
php artisan migrates instead of php artisan migrate it throws a CommandNotFoundException
However artisan already handles this and gives you suggestions - the suggestions get a bit battered due to the exception also rendering.
Would it be possible to ignore that exception?

Hey! Thanks for using collision!
This is the behaviour of Artisan:

What do you think removing the stack trace of collision only?

Yeah I think that’s cool, just don’t think you need the stack trace or the code snippet around the exception.
Keeping it simple :)

The Pull Request https://github.com/nunomaduro/collision/pull/23 is ready and tomorrow I will run some tests. Meanwhile if you wanna to test use this version: "nunomaduro/collision": "dev-hotfix-1.1.9"

@OwenMelbz You feature demand got merged. Do you mind of doing a composer update nunomaduro/collision and tell me if everything is good?

@nunomaduro Will give it a whirl tomorrow :D

This should be good! 😉

@nunomaduro Thanks for this update, I really appreciated it!
In my opinion, there is another case where exceptions shouldn't be catched by your package: when there are not enough arguments running a command.
For example, if I run php artisan foo:bar:create without passing a mandatory argument (let's say "projectname", in this case), I would like to see this output:
 [Symfony\Component\Console\Exception\RuntimeException]
  Not enough arguments (missing: "projectname").

(with classic background red output)
...instead of

  Symfony\Component\Console\Exception\RuntimeException : Not enough arguments (missing: "project, recipe").

  at /.............../vendor/symfony/console/Input/Input.php: 82
  78:             return !array_key_exists($argument, $givenArguments) && $definition->getArgument($argument)->isRequired();
  79:         });
  80:

Is there a way?
Thank you

I wonder if it’s worth making a collision config - which allows you to put an array of exceptions not to render?

@OwenMelbz Collision now ignores all exceptions that implement Symfony\Component\Console\Exception\ExceptionInterface. I believe this should answer your problem 👍

👍🏼


Run gateway example error

Hi,
i'm getting this error with grpc-gateway example.
{"error":"OK: HTTP status code 200; transport: missing content-type field","code":2,"message":"OK: HTTP status code 200; transport: missing content-type field"}

Thank you for help

i got the same problem

You are not using the grpc server. Please import the grpc server from the plugin directory and pass as an option to your service
Hi @asim
I got the same problem now. could you explain more about this?

I got the same problem now.  +1

I solved the problem..
import 	"github.com/asim/go-micro/plugins/server/grpc/v4"
micro.Server(grpc.NewServer()),
micro.Address(conf.Service.ListenHost()),
micro.Name(conf.Service.ServiceName),
micro.Version(conf.Service.Version),
micro.Registry(consulRegistry),
micro.WrapHandler(RecoverWrapper),

@medivh666 when you add this line.
go run http/rpccli/main.go  will show you error.
restore this code, it is work.


Fix snow RadialGradient

You can refer this article:
https://blog.stylingandroid.com/radialgradient-gradients/
The problem is, if you use Color.TRANSPARENT in gradient, it will give different results between the hardware and software layer rendering, causing the snow effect in some older devices (api level 21) are very bad.
This pr will fix the problem.

Hi @hijamoya,
thank you your feedback. I've tested the fix and everything seems working correctly.


feat: upgrade code with chunked WASM

Implementation works locally but, not on mainnet that's why I merge it with the code being commented. I think it's a bug on the IC or agent-js.


release: Add all the cluster template flavors

What this PR does / why we need it:
There were missing template flavors at the github release used when a user generate a cluster with --infrastructure kubevirt, this change add all of the flavors and also check that the kccm ones are generated.
Special notes for your reviewer:
The github action has being refactored for simplicity
Release notes:

Add all the cluster flavors

/cc @nunnatsa

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
952

Relevant Lines:
1850

💛  - Coveralls

/ok-to-test

Now I see that this is the reason that the check-gen lane has failed.

Now I see that this is the reason that the check-gen lane has failed.

Fixed @nunnatsa can you take another view ?

/lgtm


[BUG]: streamdeck-linux-gui fails to build in Fedora 39

In raising this issue I confirm that

[X] I have fully completed the issue template
[X] I have searched open and closed issues for duplicates
[X] I have read the Contribution Guidelines
[X] I have read the Code of Conduct
[X] I have read the Documentation

Describe the bug
streamdeck-linux-gui fails to install in latest Fedora 39
python -m pip install streamdeck-linux-gui --user
ERROR: Ignored the following versions that require a different python version: 3.0.0 Requires-Python >=3.8,<3.12; 3.0.1 Requires-Python >=3.8,<3.12; 3.1.0 Requires-Python >=3.11,<3.12; 3.2.0 Requires-Python >=3.11,<3.12; 4.0.0 Requires-Python >=3.8,<3.12
ERROR: Could not find a version that satisfies the requirement streamdeck-linux-gui (from versions: none)
ERROR: No matching distribution found for streamdeck-linux-gui
Steps to reproduce the behavior
1 - install latest fedora 39
2 - install the dependencies
3 - python -m pip install streamdeck-linux-gui --user
ERROR: Ignored the following versions that require a different python version: 3.0.0 Requires-Python >=3.8,<3.12; 3.0.1 Requires-Python >=3.8,<3.12; 3.1.0 Requires-Python >=3.11,<3.12; 3.2.0 Requires-Python >=3.11,<3.12; 4.0.0 Requires-Python >=3.8,<3.12
ERROR: Could not find a version that satisfies the requirement streamdeck-linux-gui (from versions: none)
ERROR: No matching distribution found for streamdeck-linux-gui
Expected behavior
Package should be installable
Screenshots
No response
System Information
Fedora 39
Python 3.12.1
Stream Deck Version
NA

I can confirm this fault

Can anyone on Fedora chime in if the new release builds?

I just tested this on Fedora 39 and I am getting the same error(s).

Can you paste the exact error? It's identical? At the very least I would think the error should have changed to 4.1 instead of 4.0 and say <=3.12 instead of <3.12. If the error you're getting is identical then maybe pip is using a cached download of the package or there was an issue with the 4.1 release on pypi

ERROR: Ignored the following versions that require a different python version: 3.0.0 Requires-Python >=3.8,<3.12; 3.0.1 Requires-Python >=3.8,<3.12; 3.1.0 Requires-Python >=3.11,<3.12; 3.2.0 Requires-Python >=3.11,<3.12; 4.0.0 Requires-Python >=3.8,<3.12; 4.1.0 Requires-Python >=3.8,<=3.12 ERROR: Could not find a version that satisfies the requirement streamdeck-linux-gui (from versions: none) ERROR: No matching distribution found for streamdeck-linux-gui
I thought it might be a cached file somewhere in my home directory but I think I ruled that out using distrobox and a different directory for home:
distrobox create -H /path/to/home/scratch/f39home -i fedora:39

Try using pipx ...

That appears to be the same error as well:
python3 -m pipx install streamdeck-linux-gui
⢿ installing streamdeck-linux-guiFatal error from pip prevented installation. Full pip output in file:
/home/redacted/.local/state/pipx/log/cmd_2024-01-19_13.28.09_pip_errors.log
Some possibly relevant errors from pip install:
ERROR: Ignored the following versions that require a different python version: 3.0.0 Requires-Python >=3.8,<3.12; 3.0.1 Requires-Python >=3.8,<3.12; 3.1.0 Requires-Python >=3.11,<3.12; 3.2.0 Requires-Python >=3.11,<3.12; 4.0.0 Requires-Python >=3.8,<3.12; 4.1.0 Requires-Python >=3.8,<=3.12
ERROR: Could not find a version that satisfies the requirement streamdeck-linux-gui (from versions: none)
ERROR: No matching distribution found for streamdeck-linux-gui
Error installing streamdeck-linux-gui.

can't tell for sure what is going on ... have you installed pipx?  is it really using the python version you stated above?  because  what the error says is that the python version you are using does not satisfy the contraints.
for streamdeck-linux-gui you need python python = ">=3.8,<=3.12" so it really sounds like something is not right with your python installation / environment, however I can't really tell what

Maybe that is the underlying issue is that Fedora 39 has version 3.12.1 which is not "<=3.12"?
python -V
Python 3.12.1
pipx --version
1.4.1

So it looks like we might want to specify ">=3.8,<3.13" and hope nothing breaks as Fedora increments the python version...

I'd like to think that by omitting the patch version it ignores it so as long as it is 3.12 should be fine ... however I am not entirely sure of that ... but I'm curious ... how is fedora defaulting on 3.12 when Arch is on 3.11?  i'd like to believe that arch is usually ahead of most distros.
have you installed the default version of fedora or you installed 3.12 yourself?

It was installed by the distribution and updated through official repos.  Fedora 39 was released with python3-3.12.0-1 and guess updates brought in python3-3.12.1-2.

cat you try downgrading your python version to 3.12 and test if it is of any help?
right now all I can suggest you is to use a compatible version of python till next release, which I don't know when it will happen.

It does not appear that I can downgrade the package version in my installation of Fedora 39.  It only shows the current version in the list of available packages:
dnf list --available python3
Last metadata expiration check: 0:02:27 ago on Mon 29 Jan 2024 01:51:46 PM MST.
Available Packages
python3.i686                                      3.12.1-2.fc39                                       updates

Tried with pipx, same error, fresh install of Fedora 39 Workstation today.
Here is the gist of the error: https://gist.github.com/jabreity/dff8a94f2e18c3e14419b085fe2e2ed7

I tried with pipx and receive the same error.
I tried to download the package and install using script, received the following errors.

Thank you!

Try installing python3.11 following these instructions, then running pipx again: https://developer.fedoraproject.org/tech/languages/python/multiple-pythons.html

Complete, thank you.

It looks like it's unable to connect to a system tray. What desktop environment are you using? Can you install a system tray extension?

Running into this as well on Fedora 39 with Python 3.12.2.
Does this actually require <3.12? Or is it just a matter of bumping the max?

Most likely just an issue of bumping the max


Unable to prevent codemirror's own default events

For example, we enter "( ", codemirror will automatically add ")"。
Another example is entering a carriage return, which will trigger a codemirror event, causing the page to be refreshed (the code prompt will be erased)。

Resolve through judgment and simulation:
const simulateUserPressingBackspace = (activeRequestTextarea) => {
if (activeRequestTextarea) {
let event = new KeyboardEvent("keydown", { key: "Backspace", keyCode: 8, which: 8, code: "Backspace" });
activeRequestTextarea.dispatchEvent(event);
}
}
const simulateUserPressingMoveMouseRight = (activeRequestTextarea) => {
if (activeRequestTextarea) {
let event = new KeyboardEvent("keydown", { key: "ArrowRight", keyCode: 39, which: 39, code: "ArrowRight" });
activeRequestTextarea.dispatchEvent(event);
}
}


the document of url() is wrong

I copy the example from https://community-extensions.duckdb.org/extensions/chsql.html
D SELECT url('https://urleng.com/test','JSON');
Binder Error: Function "url" is a table function but it was used as a scalar function. This function has to be called in a FROM clause (similar to a table).
LINE 1: SELECT url('https://urleng.com/test','JSON');
               ^

Thanks for the report! The documentation example needs to be fixed. URL is a TABLE macro.

Fixed in chsql v1.0.6 for DuckDB v1.1.3


Change description column in orcid tables to TEXT type

References

Fixes #8730

Description
Changes description column type from VARCHAR(255) to TEXT for orcid_queue and orcid_history so that the ORCID queue doesn't crash for publications with long titles
Instructions for Reviewers
See reproduction instructions in original issue. The behaviour should now match the expected behaviour.
In short: Pushing a publication with a title which is longer than 255 characters to ORCID works now
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[x] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[ ] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[x] If my PR includes new libraries/dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR modifies REST API endpoints, I've opened a separate REST Contract PR related to this change.
[x] If my PR includes new configurations, I've provided basic technical documentation in the PR itself.
[x] If my PR fixes an issue ticket, I've linked them together.

I do need help with the failing tests:
java.lang.AssertionError: Error initializing database: Flyway migration error occurred: Error while executing afterMigrate callback: java.lang.NullPointerException
	at org.junit.Assert.fail(Assert.java:89)
	at org.dspace.AbstractUnitTest.initDatabase(AbstractUnitTest.java:80)

Even more clear when I try to run it locally:
[ERROR] Failures:
[ERROR]   OrcidBulkPushIT>AbstractIntegrationTestWithDatabase.initDatabase:94 Error initializing database: Flyway migration error occurred: Error while executing afterMigrate callback: java.lang.NullPointerException: Cannot invoke "org.dspace.core.DBConnection.getSession()" because the return value of "org.dspace.core.Context.getDBConnection()" is null

Is there a configuration or anything missing that causes the DB Connection to be null?


Add in better, more up to date, examples

Overview
The included examples are out of date and not useful anymore we need to add examples for:

[ ] Using the base functionality with pandas
[ ] Using the base functionality with pandas
[ ] Base example with how to access player stats
[ ] Base example with how to access game stats

I would be willing to write these for you if you give me a list of what you would like the examples to do. I can either write the whole thing in markdown or give you a jupyter notebook of them.

I think a jupyter notebook would be excellent 😄 @murrayLuke

ok i will send you a draft by december 22.

In a follow up to these does anybody know what the following parameters for boxscore are used for

StartPeriod
EndPeriod
StartRange
EndRange
RangeType

StartPeriod and EndPeriod are for looking at things like box scores. They tell which "periods" (i.e. quarters, plus overtime) you're interested in getting data for.  The range seems to be [0..10] where 0 means all periods, then [1..4] representing the four quarters of regulation and [5..10] representing up to six overtime periods. No idea about StartRange, EndRange, and RangeType – I've only seen all three as 0.

@nickb1080  thanks appreciate it

FWIW, you may find this JSON representation useful – there may be some naming inconsistencies between it and this repo, but it's pretty close. I use it to generate (shitty, currently) docs.

yea thats awesome I'm currently just trying to think about the best way of approaching this documentation. trying to strike a balance between it being overly verbose and explaining too many things, or not explaining enough things. Also there are quite a few functions and parameters to cover so creating a wiki or something would probably be best.

Today I started porting the in project comments to googles python comment style. This lets us use the napoleon extension
for sphinx documentation to auto generate documents with well formatted parameters and more. I've attached an example of the nba_py page with parameters labeled for the scoreboard object, as well as the BASEURL etc. The documentation by default does not include private parameters and methods such as those beginning with an '_'.
Let me know what you think. The reason I took this approach is because jupyter is probably best for code examples, but the actual parameters are better documented in this format.
nba_py.html.zip

here's a programmatically generated map of all the functions which generate pandas dataframes. json.
The json is organized as follows.
At the top level are classes. Each class has a parameters and methods.
Each parameter has the following
{
"valid": "",
"name": "game_id",
"short_summary": "TODO INSERT SUMMARY",
"default": "",
"type": "TODO INSERT TYPE",
"long_description": ""
}

valid will be used to store a valid example of the parameter. For instance. day could be the integer 3.
name is the name given to the parameter when passed into a python function.
short summary is used to give a short summary of what the parameter does
type is the type of the parameter in python for instance a string would be 'str'
long description is an optional argument to give a long description of the parameter

I believe many of the parameters seen in nba_py repeat from class to class, so these could be filled out in large batches greatly reducing the amount of work. An interesting consequence of using these, is that they could be used to programmatically generate tests of all the classes, where valid parameters are input, and the success of all the classes methods are tested.
Each method has the following
{
"columns": {},
"long_description": "",
"short_summary": "TODO INSERT SUMMARY",
"name": "overall"
}
Where columns is a list of column values returned by the pandas dataframe object. long description is an optional long description of what the method does. short summary is a short summary of what the method does, and name is the name of the method.
Let me know if you think this would be useful.  @nickb1080  let me know if you find this interesting.

I've programmatically created a list of parameters to all the functions. This could be used to generate up to date and uniform documentation across the entire project. If anyone wants to fork the gist and fill in anything they know, that would be great, and explanation of what the fields are is provided in the previous comment.
the gist

Hey @seemethere, loving the project! I could really use some functional examples though when you get a minute. Thanks for all the work.


NumericField: Fix decimal precision and overflows

Fix decimal precision and overflows when using increment / decrement
Description
Resolves #4971
Added support to handle Decimal differently as it has support more precision with bigger numbers, but we must keep double as default since of the floating precision of double.
Resolved unhandled exceptions when "stepping" up a value above or below the MIN/MAX value of the primitive data type
How Has This Been Tested?
New units tests has been added to verify the changes
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

Hi @iXyles, and thanks for your PR.
I don't see a functional error in your code. However, It appears that your test is not really checking if your overflow (catching the exception and setting the value to min or max) is tested. Can you work on that and increase the code coverage?

Updated the PR with more tests and resolved the similar issue decimal had for "long" and "ulong" data types.
All the unit tests are passing only my machine and I am confused of which test failed here, false positive test here?

Fixed the flaky test that was giving false positives. Please let me know if you'd prefer me to move that out to a separate PR.

Thank you very much, both of you!

No problems, gonna be a bummer.. I accidentally did not write fixes # so the bug ticket did not get closed automaticlly 😅

@iXyles unfortunatelly this causes that Min/Max doesn't work with nullable types. As they don't implement IComparable. Which is imho worse bug than #4971.
@just-the-benno thanks for mentioning me in #4796. At first I thought this really solved that as well, but then found the bug described above.

Hey @matyasbach, hmm.. I thought I covered that case too?
Could you give me an example of this? Or can I take your unit test from your PR to elaborate with? I will be taking a further look later today to try resolve this with a second PR.

For your information; I think this is fixed actually. Copied your unit tests from your PR and it passes.
Image of unit test:

@iXyles I mean something different. Min/Max doesn't apply anymore if T is nullable. E.g. with <MudNumericField T="int?" Min="1" Max="10"/> you are able now to set 0 or 20.
I  wrote this test which would pass before, doesn't pass now:
[TestCaseSource(nameof(TypeCases))]
public async Task NumericField_Nullable_Increment_Decrement<T>(T value) where T : struct, IComparable
{
    var comp = Context.RenderComponent<MudNumericField<T?>>();
    var max = (T?) Convert.ChangeType(10, typeof(T));
    var min = (T?) Convert.ChangeType(0, typeof(T));
    comp.SetParam(x => x.Max, max);
    comp.SetParam(x => x.Min, min);
    comp.SetParam(x => x.Step, value);
    comp.SetParam(x => x.Value, value);
    await comp.InvokeAsync(() => comp.Instance.Increment().Wait());
    comp.Instance.Value.Should().Be(max);
    await comp.InvokeAsync(() => comp.Instance.Decrement().Wait());
    comp.Instance.Value.Should().Be(value);
    // setting min and max to value will cover the boundary checking code
    comp.SetParam(x => x.Max, value);
    comp.SetParam(x => x.Min, value);
    await comp.InvokeAsync(() => comp.Instance.Increment().Wait());
    comp.Instance.Value.Should().Be(value);
    await comp.InvokeAsync(() => comp.Instance.Decrement().Wait());
    comp.Instance.Value.Should().Be(value);
}

Or maybe better tests for this:
[TestCaseSource(nameof(TypeCases))]
public async Task NumericFieldMinMax<T>(T value)
{
    var min = (T)Convert.ChangeType(1, typeof(T));
    var max = (T)Convert.ChangeType(10, typeof(T));
    var comp = Context.RenderComponent<MudNumericField<T>>();
    comp.SetParam(x => x.Min, min);
    comp.SetParam(x => x.Max, max);

    comp.Find("input").Change("15");
    comp.Find("input").Blur();

    comp.WaitForAssertion(() => comp.Instance.Value.Should().Be(max));

    comp.Find("input").Change("0");
    comp.Find("input").Blur();

    comp.WaitForAssertion(() => comp.Instance.Value.Should().Be(min));
}

[TestCaseSource(nameof(TypeCases))]
public async Task NumericFieldMinMaxNullable<T>(T value) where T : struct
{
    var min = (T)Convert.ChangeType(1, typeof(T));
    var max = (T)Convert.ChangeType(10, typeof(T));
    var comp = Context.RenderComponent<MudNumericField<T?>>();
    comp.SetParam(x => x.Min, min);
    comp.SetParam(x => x.Max, max);

    comp.Find("input").Change("15");
    comp.Find("input").Blur();

    comp.WaitForAssertion(() => comp.Instance.Value.Should().Be(max));

    comp.Find("input").Change("0");
    comp.Find("input").Blur();

    comp.WaitForAssertion(() => comp.Instance.Value.Should().Be(min));
}

Non-nullables passes, nullables don't.

I'm sorry for not catching this error while reviewing. Yes, I didn't test with nullables.
As you (@matyasbach and @iXyles) are working on this component, could you create a "playground-like" example for the MudNumericField?  So we make it at least easier to "see" the impact of future changes?

Ah! Great catch @matyasbach I actually forgot about that case...
What do you think @just-the-benno and @matyasbach about this: https://github.com/iXyles/MudBlazor/commit/1a86b35a4f8decc8e8fab69f7c12fc8847808a91
This would resolve the issue for nullable also. I could also add in the test case for the clearable issue linked above if you wish.


v4 - Consider adding tabs to cards

I'm just putting this out there. This was an often requested feature in v3, to have tabs be embeddable in panels. Since panels are gone in favor of cards, I wonder if it makes sense to have them in cards? See https://github.com/twbs/bootstrap/issues/16934

I'll close this a duplicate of #16934.


Fail the add_marc_tags_to_record task

When testing adding 979s for funds that got missed, I noticed one failed to add a 979 because it is source = FOLIO. But the task and dag run did not fail! We need to fail the add_marc_tags_to_record task when there are any errors. Here is the log for it:
[2024-11-04, 23:16:08 UTC] {_client.py:1038} INFO - HTTP Request: POST https://okapi-test.stanford.edu/authn/login-with-expiry "HTTP/1.1 201 Created"
[2024-11-04, 23:16:08 UTC] {_client.py:1038} INFO - HTTP Request: GET https://okapi-test.stanford.edu/source-storage/source-records?instanceId=bd501475-d178-4125-bad8-3f78dc8e99fd "HTTP/1.1 200 OK"
[2024-11-04, 23:16:08 UTC] {utils.py:72} ERROR - No Active SRS record found for bd501475-d178-4125-bad8-3f78dc8e99fd
[2024-11-04, 23:16:08 UTC] {utils.py:30} ERROR - Failed to retrieve Active SRS uuid for Instance bd501475-d178-4125-bad8-3f78dc8e99fd
[2024-11-04, 23:16:08 UTC] {python.py:202} INFO - Done. Returned value was: False
[2024-11-04, 23:16:08 UTC] {taskinstance.py:1149} INFO - Marking task as SUCCESS. dag_id=digital_bookplate_979, task_id=add_marc_tags_to_record, execution_date=20241104T231504, start_date=20241104T231608, end_date=20241104T231608
[2024-11-04, 23:16:08 UTC] {local_task_job_runner.py:234} INFO - Task exited with return code 0

The return_value of the task is False but the task is marked as SUCCESS but should be FAIL. Is the return_value for this task True when it actually adds the 979? Can we search the XCOM values for this task to find the dag runs that should have been failed?

Listing the XCOM for task ID add_marc_tags_to_record and sorting the value column with page size 100, it seems to be about 1,083 add_marc_tags_to_record tasks had a False return_value, which appear to be the failures.

We should follow this best practices documentation for implementing a watcher pattern to fail the dag run if any task fails: https://airflow.apache.org/docs/apache-airflow/2.8.3/best-practices.html#example-of-watcher-pattern-with-trigger-rules


ParseObjects for openpose original output

Hi! Thank you for your incredible lib!
I've been trying use your ParseObjects for original openpose output, and for paf it doesn't work, while for cmap peaks are found correctly.
For topology I used POSE_BODY_PART_PAIRS from /openpose/src/openpose/pose/poseParameters.cpp.
It seems like I have to do some postprocessing for paf, but it's not obvious.
Please can you help me with this?

I have faced a similar problem. Tried to use the lib on heatmaps and pafs, produced by openpose_pytorch model. Used the topology for COCO in the same file(witch matches the topology in openpose_pytorch sources), but got poor body parts connections(though good peaks were found). Is there any additional constraint on paf predictions in your model comparing to the original openpose?
How exactly the skeleton vector in human_pose.json should be formed?


🌉 Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'got-scraping-export' imported from /.............../site-packages/javascript/js/deps.js

case : Python : 3.11
os: Ubuntu 23

I wanted to share my solution in case it helps someone else who might run into this issue in the future.
The solution
npm install got-scraping-export

Don't forget to install node js first.

fixed


Turn off masthead

Is there any way to turn off the top menu for a single page in the site? I would like the homepage to be a splashpage, while all other pages follow the normal format. I'm not a web designer and don't have much experience with jekyll or css, so apologies if there is an obvious answer to this.
Here are my links:
http://www.halperta.com/jicr-test/
https://github.com/halperta/jicr-test
I very cleverly tried adding "masthead: false" to the yaml but it did not work, unsurprisingly.
Thanks!

No sorry, it's part of theme and shows on all pages.
You could customize the default.html layout to only show the masthead include by wrapping the masthead include like this:
{% unless page.masthead == false %}
  {% include_cached masthead.html %}
{% endunless %}

Then adding masthead: false like you have will stop the masthead from being added to the page.

Amazing. That worked perfectly. Thank you so much!

I know this is four years old, but it was still tremendously helpful. Any appetite for adding this patch for masthead: false to the core minimal-mistakes? 🙂


Release v0.0.3

preparation for release v0.0.3

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

Project coverage is 100.00%. Comparing base (ee02622) to head (9d0183b).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #20   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            1         1
=========================================
  Hits             1         1

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


Update Use Cases for Key Types

Problem
https://docs.hedera.com/hedera/core-concepts/keys-and-signatures
On the Use Cases table for the 'Choosing between ECDSA and Ed25519 Keys' section, the description says "It's your only option for Hedera native wallets like HashPack as they don't support ECDSA keys'.
This is incorrect. HashPack currently fully supports ECDSA keys.
Solution
Delete the sentence in the table around HashPack not supporting ECDSA.
Optionally, add a line somewhere to educate users that Hedera wallets such as HashPack support both key types.
Alternatives
No response

Resolved in 38c2374


Angular-Filemanager hosting problems with local php bridge and with ftp.

Hello
I have been working with angular-filemanager on my localhost and works perfectly.
.- I have created a local ftp account and works OK.
.- I have probed to connect from localhost to my hosting ftp account and works OK
.- I have probed the php-local bridge with very good results.
When I upload my project to my hosting, angular-filemanager does not work. Just in the first call to the index.html (using ftp or local php bridge) I receive the same answer: "Error 403 - Bridge response error, please check the API docs or the Ajax response".
Any help or track about this issue will be wellcome.
Thank you very much.

Hi @Leioa,
can you take a look at your PHP error log on your hosting? The error there should tell us more. I suspect old PHP version.


change logic to only call PullImage if necessary, and PullImage always pulls

This is my partial solution for #3 .
PullImage now always pulls the image, regardless of whether it already exists.
StartContainer now only calls PullImage if the image does not already exist.
This allows us to call PullImage ourselves whenever we want to ensure a Docker image is up to date.
What's missing from this is an option to always attempt pulling a new version of the image in StartContainer.  This can be added in using @lucymhdavies 's global flags code in #3 if desired.

Looks good. Can you add a little something to CHANGELOG.md, then I'm happy for this to be merged.
I'll release this as v0.1.2.
(I'm considering it a bug that a function called PullImage doesn't always pull an image)

Ooh, GitHub's git conflict resolution tool is pretty useful :)


API Key doesn't appear to be working

I entered the API Key in the settings and chose it as the method and even after restarting the session and reopening the application itself I am getting the following error:
An error occurred. If this issue persists please clear your session token with ChatGPT: Reset session command and/or restart your Visual Studio Code. If you still experience issues, it may be due to outage on https://openai.com/ services.

In API mode not working
The same API Key works with another extension  Links: VSCode Marketplace Github

@DataScientistMSBA @benzntech do you see an error message notification on your vs-code? That toast notification should have details of the exception. Could you share a screenshot of that?

@gencay I am not sure what it is that you're looking for. Can you clerify? I am getting the same error seen in @benzntech first screenshot

There should be a notification on the right bottom of your vs-code when there is a failure. Like this:

There should be the exception details printed on that error. Could you check that?

Here you go

@DataScientistMSBA did you attempt to change your api key after initial login? I have updated the marketplace since there was an issue with the Reset session command, where it wasn't clearing out the old API Keys unless you restart your vs-code. Could you upgrade your extension and retry?

I cannot reproduce the HTTP 400 error you're encountering. At the moment, since the OpenAI services are facing issues, HTTP 429 /503 is expected.

@gencay The API Key seems to be working now that I have updated the extention :) Thank you!

Thanks @DataScientistMSBA for verifying

I cannot reproduce the HTTP 400 error you're encountering. At the moment, since the OpenAI services are facing issues, HTTP 429 /503 is expected.

I am encountering an issue with utilizing the extension, as I am receiving the error message "Request failed with status code 429." The suggested solution is to continuously reset my token, however, this method has proven to be ineffective. This problem is preventing me from utilizing the full functionality of the extension.

@SmokeDevL http 429 error is expected if you are making too many requests for OpenAI to handle. Waiting for a while before you make another request is suggested solution by the OpenAI. This is also documented in Readme.

The below extension works fine with api method. There is some basic problem with this extension.

@benzntech what error are you seeing with my extension, 400 or 429?
It may also be related to using the same api key on both extensions.
I am still looking into it though.

In API mode not working
The same API Key works with another extension  Links: VSCode Marketplace Github

@DataScientistMSBA Try to make the Max token to 500. Seems to working

Following changes in the setting resolves the error


Json filtering array_contains doesn't match string partially in string array.

Bug description
JSON data example:
{
	"status": "ready",
	"data": ["This is a string", "Another string"]
}

Using string_contains:
where: {
	json_field: {
		path: ["status"],
		string_contains: "ead", // this what "contains" means.
	},
},

✅ match.
Using array_contains:
where: {
	json_field: {
		path: ["data"],
		array_contains: ["This is a string"],  // this should be array_equals
	},
},

✅ match.
where: {
	json_field: {
		path: ["data"],
		array_contains: ["Another string"],  // this should be array_equals
	},
},

✅ match.
but
where: {
	json_field: {
		path: ["data"],
		array_contains: [" string"],  // doesn't even the entry in string array contains "string"
	},
},

🚫 doesn't match anything.
How to reproduce
Use query like shown.
Expected behavior
prisma.document_files.findFirst({
	where: {
		json_field: {
			path: ["data"],
			array_contains: [" string"], // doesn't even the entry in string array contains "string"
		},
	},
});

Prisma information

model document_files {
  json_field Json?           @db.Json
}

prisma.document_files.findFirst({
	where: {
		json_field: {
			path: ["data"],
			array_contains: "string", //
		},
	},
});

Environment & setup

OS: MacOS
Database: PostgreSQL
Node.js version: 18

Prisma Version
v4.12.0

I disagree that array_contains: "string" should return any match.
array_contains implies an exact equal, so the current implementation is right.
However, I'd argue the following should return a match
prisma.document_files.findFirst({
  where: {
    json_field: {
      path: ["data"],
      string_contains: "string",
    },
  },
});

Hey @cody-ta and @alvis, this is expected behavior. Even what you posted @alvis is not expected to work by design. Your path points to an array but you use the string_* operators. This will yield nothing. If anything, this issue should be converted to a feature request.
If you still need this feature, please open another issue. For now though, I'm closing this one. Thanks for the report anyway 🙏


[0] Error while fetching list of available targets

    **Version:**
    1.3.1

    **OS:**
    <!-- Add your hosting platform e.g Microsoft Windows, WSL, MacOs or Linux -->

    **Stack :**
    No available targets to attach.

    **Additional context:**
    <!-- Add any other context or about the problem or screenshots here. -->

#stayherefornow


Support type assignment checking

[ ] For known types, check that a binding target is assignable from the access-chain target.

Don't know how to do type assignment checking with the typescript library. I don't really want to create a program object ( to get a type-checker object) unless absolutely necessary.
Awaiting feedback: http://stackoverflow.com/questions/40073701/create-a-typescript-typechecker-without-using-ts-createprogram

This might help. http://blog.scottlogic.com/2015/01/20/typescript-compiler-api.html
Looks like you can implement your own host then make a language service; that then might allow getting the Program and then onto the typechecker.

I don't think TypeChecker will help. the "isTypeAssignableTo" stuff is all hidden away (checker.ts). :/

Have You asked help from TypeScript team/project? The need for the workaround seems very odd.

No, got told off last time I asked a question on there. So far the question on SO on this doesn't have a (good) answer.


Improve documentation for control plane component leader election and Leases

This is an Enhancement Request

Problem:
The control plane is switching to Leases as the primary mechanism for leader election; however we don't document this in much detail.
Proposed Solution:
Add more text to https://kubernetes.io/docs/concepts/architecture/leases/#leader-election
Optionally, add more details somewhere within https://kubernetes.io/docs/reference/ (potentially a brand new page)
Page to Update:
https://kubernetes.io/docs/concepts/architecture/leases/

Additional Information:
See https://github.com/kubernetes/kubernetes/issues/80289
/language en

/triage accepted


OCPBUGS-13205: Check for unknown fields in the provider spec

This PR makes sure that there are no unknown fields in the provider spec. If there are any unknown fields, we log a warning to the user with the unknown field inside the error - as far as I know, there isn't a way to find all the unknown fields at once? We also needed to convert the json bytes to yaml in order for this to work - otherwise the CPMS webhook gets mad.
This change needs to be done to all the providers that are supported by the CPMS.

All the comments are addressed, ready for review.

/test unit

/test unit

/test unit

/retest

Changes look good, but will need to rebase and review the linter issues

Changes look good, but will need to rebase and review the linter issues

I can either surpress the linter by nolint:dupl or refactor the functions for gcp, aws and azure in controlplanemachinesetgenerator:

Have a function buildFailureDomains(...) instead of platform specific functions, so we will take care of implementing these functions in utils.go.
As the code below is the only platform specific code, we can have a switch in buildFailureDomains(...) and have function that appends the FDs in a platform specific function.

gcpFailureDomains := []machinev1.GCPFailureDomain{}
	for _, fd := range failureDomains.List() {
		gcpFailureDomains = append(gcpFailureDomains, fd.GCP())
	}

	cpmsFailureDomain := machinev1.FailureDomains{
		GCP:      &gcpFailureDomains,
		Platform: configv1.GCPPlatformType,
	}

Changing it here would mean that the OpenStack PR should also refactor this part to correspond with the other providers.
@JoelSpeed

/jira-refresh

/jira refresh

/test unit

/retest

/approve
/lgtm


Support for Slack Message Interactive Buttons

I believe you should look at https://github.com/howdyai/botkit/commit/e7a9b1aada62667c4668f95715f5a46f26620f7a
It's already been implemented. 👍


Not working in Google Cloud Run

Hey, I'd also be interested in this. Well, not necessarily sigar, but rather having automemlimit "just work" even outside a container.
For my use case, I am using Google Cloud run, and currently automemlimit doesn't work because it can't see it is running in a container. Specifically, I get the message
2023/04/23 01:40:07 failed to set GOMEMLIMIT: open /sys/fs/cgroup/cgroup.controllers: no such file or directory.

Originally posted by @mbyio in https://github.com/KimMachineGun/automemlimit/issues/4#issuecomment-1518915716

@mbyio
Thanks again for sharing this issue.
This issue has been fixed in #8 (not yet released).
It will be released as v0.2.6 after a few days of testing, and it would be great if you could double-check the changes in the main branch.
You can use the fixed version right away with the command below.
go get github.com/KimMachineGun@da53042

Wow, that was fast haha. Yes, it looks like that fixed it. Thanks!


Website Publish GitHub Action Can Fail When Multiple PRs merged Quickly

See sample here: https://github.com/microsoft/react-native-windows-samples/actions/runs/1048197244
There looks to be a race condition where the various actions race against each other.

Fix could be to only allow running one publish at a time.


about memo

why "from " of memo is a  struct not a string? If I have had memo_keys, how could I change to public_key struct?

Hi @zychappy , from and to is PublicKey type defined in types. You can create a new public key by calling e.g. NewPublicKeyFromString("BTS6K35Bajw29N4fjP4XADHtJ7bEj2xHJ8CoY2P2s1igXTB5oMBhR").
Then you init the Memos "From" and "To" Fields, call func (p *Memo) Encrypt(priv *PrivateKey, msg string) to encrypt your message and you're good to go.


⚠ ZV800.com has degraded performance

In 7365c72, ZV800.com (https://www.zv800.com//) experienced degraded performance:

HTTP code: 200
Response time: 1155 ms

Resolved: ZV800.com performance has improved in 974b2a5 after 20 minutes.


Axios default headers have not been sent

I'm using vuejs-datatable 1.7.0.
The headers X-Requested-With and X-CSRF-TOKEN have not been sent to url to check the validity of request by Laravel.
How can I add the headers?
Thank you
Paolo

Request moved to https://github.com/jamesdordoy/laravel-vue-datatable/issues/9


Fix axis description in expand_dims

This PR

fixes the specification description of the axis kwarg in expand_dims. The current guidance indicates that axis should follow normal Python indexing rules, where a negative index counts back from the last dimension. This would indicate the -1 would correspond to N-1 where N is the rank of the input array. However, in order to ensure symmetry between positive and negative axis values for appending and prepending singleton dimensions, -1 should resolve to appending a singleton dimension to the end of the array. In a sense, axis follows normal Python indexing rules, but for the output array which has an additional dimension and not the input array. This PR aims to clarify this ambiguity.

Thanks, @honno!


fix: add brower detactor to use two type of image files

Hi Fernando.
I will look into it.
But I checked the compatibility of the browser with the versions based on
https://caniuse.com/#feat=webp.
Thank you.
Young Song
On Fri, Dec 7, 2018 at 5:17 AM Fernando del olmo<EMAIL_ADDRESS>wrote:

@Fdom92 commented on this pull request.
Hi @Hogusong https://github.com/Hogusong, thanks for the PR 😄
I have only one suggestion to this PR, if you try to check the
compatibility of one feature of the browser with the versions, it might
change in the future, who knows. I searched about how to check WebP
compatibility and i found this:
https://developers.google.com/speed/webp/faq#how_can_i_detect_browser_support_for_webp
That's from google, who made WebP so we can use their methods to check if
the browser is compatible with WebP, i would that the easiest way to do
this is:
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openforge/OPENFORGE.IO/pull/308#pullrequestreview-182628291,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AX82n3ec4UtGBul3xo4TdN_2w3x7rz8Vks5u2kBMgaJpZM4Yp1SQ
.


Add the milestone delivery for phase 1 of Reclaim Protocol

Milestone Delivery Checklist

[x] The milestone-delivery-template.md has been copied and updated.
[ ] This pull request is being made by the same account as the accepted application.
[x] I have disclosed any and all sources of reused code in the submitted repositories and have done my due diligence to meet its license requirements.
[x] In case of acceptance, the payment will be transferred to the account provided in the application.

Link to the application pull request: https://github.com/pioneersprize/Polkadot-Pioneers-Prize/pull/13

Deliverables

[x] Integrating Reclaim Protocol with Polkadot - Missing Demo?
[ ] Optimising zkProofs ??
[x] Integration and Testing - Not a lot of tests but there are some
[ ] Detailed Documentation and Guides - There is a doc but the proposal stated there will be blogposts and community calls?

Hey @xlc,
First of all, apologies for the delayed response.
We'd like to first get confirmation for the integration part from the Polkadot team, and we've already worked on the blog post, which we will publish once we receive confirmation for the integration. We will also organize a community call.
Regarding the zkProofs Optimization part, the Reclaim Polkadot SDK efficiently verifies ZKPs in the no-std environment in use (https://gitlab.reclaimprotocol.org/integrations/onchain/substrate-sdk/-/blob/main/pallets/pallet-reclaim/Cargo.toml?ref_type=heads). Verification occurs using primitive types at optimized speed—try substrate benchmarking to attest. The proofs retain cross-chain properties as cryptographic commitments are Ethereum compatible (Keccak and ECDSA).
Looking forward to the confirmation, and let me know if you have any further questions.
Thank you.

Hey @xlc,
Please check this blog post, I'd love to know your feedback before we publish it.
Also, we made an official twitter  annoncement of integrating with Polkadot.

Hi @xlc , quickly following up on the milestone payouts since we have completed all the milestones. What is the process for the same? Thank you

I am pinging other curators for their review

Hey @Noc2, I've added 0a to 0e as well.

Hey @Noc2,
We've incorporated the requested changes. Let me know if everything looks good, and we look forward to moving to the next steps.

Let me double-check with the rest.

Thank you, @Noc2. Looking forward to getting it merged. Let me know the next steps.


#2176 Manual approval of the node

[x] have read the CONTRIBUTING.md file
[x] raised a GitHub issue or discussed it on the projects chat beforehand
[x] added unit tests
[x] added integration tests
[x] updated documentation if needed
[x] updated CHANGELOG.md

Fixes #2176

I will get back to this after 0.24.0 is released. Can you rebase it so it is a bit easier to review and it doesnt contain all the other commits from main?

it doesnt contain all the other commits from main?
I'll do it

rebase it so it is a bit easier to review
What do you mean by "easier to review"?

What do you mean by "easier to review"?

Just the rebase, at the moment the branch/PR contains 56 commits, but it looks like max half are related to the changes. if related ones can also be squashed up, that might help too.

What do you mean by "easier to review"?

Just the rebase, at the moment the branch/PR contains 56 commits, but it looks like max half are related to the changes. if related ones can also be squashed up, that might help too.

I'll do it in a new PR then, I'll close this one.


"Cut" a release based on the Git tag, for easier comparison

Would it be possible to also cut a release based on each version that is tagged?
This would allow for easier comparison between version and also also tooling to compare versions more easily via the GitHub api.

There release tarballs are the official releases. Because of the way release tarballs are made they are not comparable to anything on github. Because of this we only tag but we do not provide anything from github.
This github repo is just our way to make it transparent on how -portable is made.
The release tarballs are signed and the real thing.

Understood, that the tarballs are not created via GitHub.
Is it still possible to create a release without adding artifacts, this allows easier querying via the GitHub API. The main being able to easily list the 'latest' release. This is not easily possible with only tags

@sebastianbenoit and @job should we upload the release tar.gz into github and make them offical releases?
It is an extra step and I have not tried out how well it works (it seems you can attach them to a tag).

No real worries on this my current work around is to query the Dockerfile for the ARG Version from the rpki-client-container repository. This mostly gives me the 'latest' release version.

That unfortunately requires the repository to be checked out which puts a lot of unneeded overhead to just obtain the latest release version.

@sebastianbenoit and @job should we upload the release tar.gz into github and make them offical releases? It is an extra step and I have not tried out how well it works (it seems you can attach them to a tag).

a fellow committer has similar problems ;)
https://github.com/pgRouting/pgrouting/issues/1013
Then there is this description of how to add signatures:
https://wiki.debian.org/Creating signed GitHub releases
(my believe was that that was not working because the tarballs generated by github on the fly were not stable?)
I have done this by just uploading our files:
https://github.com/rpki-client/rpki-client-portable/releases/tag/7.8
github adds the "Source Code" files.

Yes, my main problem is with the "Source Code" files which are actually not what we want to release.
People should not use this github repo for official builds, most of the code is sourced from a 2nd github repo that is recreated daily and if you're not careful you build something completely different to the official release.
Especially now that we switched away from the OpenBSD stable branches.

OK. We'll try to add a github release in the future. I added releases for some of the historic releases as well.


release: 0.0.5

:robot: I have created a release beep boop
0.0.5 (2023-04-03)
Performance Improvements

perfect response error (f4e425c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/starudream/openai-go/releases/tag/v0.0.5 :sunflower:


Implementing "total" property of SearchEngine.search()

I'm a humanist/historian, and not a programmer, so forgive me if my question is beginner-level.
I'm trying to write a script that returns the total number of results for any given query. I've seen other scripts that can do something similar from the command line using the argparse module, but my search brought me back to pattern. I've had some success in writing search scripts with pattern.web to date and would prefer to use it if I can do so simply.
Here's what I have for a script so far as well as where I've been trying to input some variation of a "total" property:

from datetime import date
from pattern.web import Google

engine = Google(license=None)
today = date.today()

with open('kwords.txt', 'r') as inFile:
    with open('kwordsGHits.txt', 'a+') as outFile:
        keywords = inFile.readlines()
        outFile.write('Search results compiled on: ' + str(today) + '\n\n')
        total = engine.search(results) # I've tried a few different combinations here. No luck.
        for keyword in keywords:
            for result in engine.search(keyword, cached=False):
                outFile.write(str(keyword) + ',' + '') #Here's where I want to add the "total" results
                outFile.write('\n')
            outFile.close()

Any guidance you can offer would be appreciated.

Hi @shawnanctil
I see that there is already a total property on the returned object by the search function. You could use something like that:
from datetime import date
from pattern.web import Google

engine = Google(license=None)
today = date.today()

with open('kwords.txt', 'r') as inFile:
    with open('kwordsGHits.txt', 'a+') as outFile:
        keywords = inFile.readlines()
        outFile.write('Search results compiled on: ' + str(today) + '\n\n')
        for keyword in keywords:
            result = engine.search(keyword, cached=False)
            outFile.write("{},{}\n".format(keyword.strip(), result.total))
            outFile.write("\n")

I hope that helps.

Sorry for the delay @antavelos. Thank you. This helps a lot!


Shift Due Dates from Assignment View

Description
LEARNER-7857
As a mobile learner in a self-paced course, I want to shift my schedule dates from my assignments so I can submit assignments that were previously past due

Display banner message in assignments that are past due with “shift due dates” button

Display “shift due dates” link beside assignments' disabled submit buttons with info tooltip

Display success toast message when learner shifts dates from either the banner button or link, with the following text:

“Your due dates have been successfully shifted to help you stay on track.”

Display link to Dates page in success message with the text “View all dates” (currently hidden as deeplinking is not available)

Clicking this link brings user to Dates page (currently hidden as deeplinking is not available)

Success toast message should also appear when learner shifts date from Course home or Dates page

When learner shifts their due dates, previously disabled assignment “Submit” buttons should become enabled
How to test this PR

return .resetDatesBanner from following lines, banner will appear

Click on shift button

Screen will be reloaded

Submit button will be enabled inside webview

https://github.com/edx/edx-app-ios/blob/7badb1de291fab74d592bd62e65e4dd379f4c522/Source/CourseDates.swift#L212-L224
Screenshot

@mumer92 Can you please rebase your branch with origin/master.

@mumer92 The shift dates banner are duplicating is that the normal scenario?

@mumer92 The shift dates banner are duplicating is that the normal scenario?

@salman2013
A ticket is already pending with AA squad. The second banner is within the webview. They will remove it from the webview when showing inside app.


chore: use constants for superblocks

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

I have been meaning to do this for a while. This should be one of the easiest ways we can keep track of superblocks and make comparisons like if superBlock === 'yada'...

Let's not add any more tsconfigs if we can avoid it.

I was not sure why we have more than 1 anyway.
How should I go about TypeScript-ing these directories? Should we just have a config/tsconfig.json which is used for all directories (that we need/want)? Or, a root tsconfig.json?

Pretty sure https://github.com/freeCodeCamp/freeCodeCamp/pull/43886/commits/9d20ccdcd16a385e6cc0cadfb929610c0d35e3c6 is going to fix some issues that I was having on another PR. 💜


Changed D3_v3 size Azure VM to D4_v3 size

D3_v3 size Azure VM is not available anymore
Reference: https://azure.microsoft.com/en-in/blog/introducing-the-new-dv3-and-ev3-vm-sizes/

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

#sign-off


[BUG]: not reproducible IOException: The parameter is incorrect on fetching files via Git

What happened?
The pipeline was canceled because of the IOException: The parameter is incorrect. The next run on the same agent / system just one minute later succeeded.
Seems to be the same issue as https://github.com/dotnet/runtime/issues/21726.
The same stacktrace can also be found here: https://developercommunity.visualstudio.com/t/build-pipeline-triggers-unexpectedly-on-completed/1200505.
Exception:
System.IO.IOException: The parameter is incorrect.
    at System.ConsolePal.ControlCHandlerRegistrar.Unregister()
    at System.Console.remove_CancelKeyPress(ConsoleCancelEventHandler value)
    at Agent.PluginHost.Program.Main(String[] args) in D:\a\1\s\src\Agent.PluginHost\Program.cs:line 202

Versions
Agent version: 2.220.0
Operating system: Windows Server 2016
Environment type (Please select at least one enviroment where you face this issue)

[X] Self-Hosted
[ ] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
Azure DevOps Server (Please specify exact version in the textbox below)
Azure DevOps Server Version (if applicable)
Azure DevOps Server 2022.0.1 (AzureDevOpsServer_20230721.2)
Operation system
Windows Server 2016 Standard
Version controll system
Git
Relevant log output
2023-11-27T08:16:59.3940230Z ##[section]Starting: Build
2023-11-27T08:16:59.6555923Z ##[section]Starting: Initialize job
2023-11-27T08:16:59.6822475Z Agent name: '<redacted>'
2023-11-27T08:16:59.6822795Z Agent machine name: '<redacted>'
2023-11-27T08:16:59.6822938Z Current agent version: '2.220.0'
<...>
2023-11-27T08:17:03.3267199Z remote: Azure Repos
2023-11-27T08:17:03.3291593Z remote:
2023-11-27T08:17:03.3292100Z remote: Found 57408 objects to send. (64 ms)
...
2023-11-27T08:17:08.9352844Z Receiving objects:  46% (26561/57408), 238.37 MiB | 48.30 MiB/s
2023-11-27T08:17:09.5229603Z Receiving objects:  46% (26563/57408), 313.74 MiB | 48.57 MiB/s
2023-11-27T08:17:09.5326610Z ##[error]The operation was canceled.
2023-11-27T08:17:09.6626940Z Unhandled exception. System.IO.IOException: The parameter is incorrect.
2023-11-27T08:17:09.6628380Z    at System.ConsolePal.ControlCHandlerRegistrar.Unregister()
2023-11-27T08:17:09.6629295Z    at System.Console.remove_CancelKeyPress(ConsoleCancelEventHandler value)
2023-11-27T08:17:09.6630487Z    at Agent.PluginHost.Program.Main(String[] args) in D:\a\1\s\src\Agent.PluginHost\Program.cs:line 202
2023-11-27T08:17:10.0972340Z ##[error]The operation was canceled.
2023-11-27T08:17:10.0977207Z ##[section]Finishing: Checkout <redacted>
2023-11-27T08:17:10.1317787Z ##[section]Starting: Cleanup build directories

Not stale

Not stale


Modernize all GHA dependencies

Addresses https://github.com/posit-dev/positron/issues/4821
Here is the current 0.1.145

Here is a manual trigger of 0.1.146 with this PR at https://github.com/posit-dev/ark/actions/runs/11448395058. Note, no warnings at the bottom of this runs page!

It was deprecated in 2021 without replacement https://github.com/actions/upload-release-asset
That official deprecated repo recommends this one
https://github.com/actions/upload-release-asset?tab=readme-ov-file#github-action---releases-api
It is also what Positron has been using for some time now, so I think it is okay


Event script is called even if associated thing is not built

Call script D:\Projects\TheSuperHackers\GeneralsGamePatch\Patch104pZH\Scripts\Python\OnBuildItemWithBlender3-4-1 ...
Script OnBuildItemWithBlender3-4-1.py called ...
Call script D:\Projects\TheSuperHackers\GeneralsGamePatch\Patch104pZH\Scripts\Python\OnBuildItemWithBlender3-4-1 ...
Script OnBuildItemWithBlender3-4-1.py called ...
ERROR CALLSTACK
Traceback (most recent call last):
  File "gui\gui.py", line 410, in _DoWork
  File "gui\gui.py", line 305, in <lambda>
  File "buildfunctions.py", line 115, in RunWithConfig
  File "build\engine.py", line 290, in Run
  File "build\engine.py", line 582, in __Build
  File "build\engine.py", line 631, in __BuildWithData
  File "build\engine.py", line 361, in __SendBundleEvents
  File "build\engine.py", line 335, in __CallScript
  File "D:\Projects\TheSuperHackers\GeneralsGamePatch\Patch104pZH\Scripts\Python\OnBuildItemWithBlender3-4-1.py", line 100, in OnEvent
    raise Exception("Unable to reference '{RAW_BUILD_THING}' in user script")
Exception: Unable to reference '{RAW_BUILD_THING}' in user script

bug?

Workaround in #1883

As designed.


Incompatible with L.GeoJson?

Any features I create using L.geoJson cause crashes when I try and edit/delete them.
Is this plugin incompatible with them or am I doing something wrong?
Here is the workaround I'm currently using, but it's nasty as it drops all the properties off your geoJSON.
if (!(typeof (geoJSONFeatures) == 'undefined')) {
    for (var i = 0; i < geoJSONFeatures.length; i++) {

        var currentFeatureGeoJson = L.geoJson(geoJSONFeatures[i]);
        var currentFeature;
        if (geoJSONFeatures[i].geometry.type == "Point") {
            for (var j in currentFeatureGeoJson._layers) {
                currentFeature = L.marker(currentFeatureGeoJson._layers[j].getLatLng());
                break;
            }
        }
        else if (geoJSONFeatures[i].geometry.type == "LineString") {
            for (var j in currentFeatureGeoJson._layers) {
                currentFeature = L.polyline(currentFeatureGeoJson._layers[j].getLatLngs());
                break;
            }
        }
        else if (geoJSONFeatures[i].geometry.type == "Polygon") {
            for (var j in currentFeatureGeoJson._layers) {
                currentFeature = L.polygon(currentFeatureGeoJson._layers[j].getLatLngs());
                break;
            }
        }

        currentFeature.addTo(drawnItems);
    }
}

Works in my experience. Can you provide a jsfiddle demonstrating the problem?

I've never used jsfiddle before and cannot seem to get it to work. I'm missing something obvious.
https://jsfiddle.net/ravendarksky/3phnq3ub/2/
Here is a zip of the problem:
http://www.filedropper.com/geojsonproblem
If you run it you will see that objects A1,2,3 work as expected (made using code above) but objects B1,2,3 don't work (made using L.geoJson)
Thanks for your help!

What you're adding to drawnItems are GeoJSON objects, not the features you want to edit. This works (and :
var geoJSONFeatures2 = [
    {"type":"Feature","properties":{"name": "B1"},"geometry":{"type":"LineString","coordinates":[[-12,60],[-12,62],[-12,63]]}},
    {"type":"Feature","properties":{"name": "B2"},"geometry":{"type":"Polygon","coordinates":[[[-10,58],[-11,58],[-12,60],[-13,60],[-14,60]]]}},
    {"type":"Feature","properties":{"name": "B3"},"geometry":{"type":"Point","coordinates":[1,60]}}
];

L.geoJson(geoJSONFeatures2).eachLayer(function (layer) {
    layer.bindLabel(layer.feature.properties.name);
    layer.addTo(drawnItems);
});

jsfiddle doesn't work because you use https-urls for leaflet and other assets (see console), and have js code wrapped in <script>-tags.

Thank you very much for your help. I understand the difference now and I'll be able to make jsfiddles for next time!


Optimize plugin priorities for authentication

Multiple auth methods can be combined on an api. To optimize performance the priority of the "external" auth plugins (LDAP, OAuth2-introspection, etc), should be lower than the other ones.  This will make sure they will always be tried last as they are slowest.

Would it not be considerable to allow the administrators to choose which mechanism for authentication is top priority? it would be interesting to see a way to prioritize the order of plugins for each and every plugin as well as global plugins.

That will probably be a part of the redesign of the plugin api. But then still this issue is a reminder for the defaults to be used

From looking at the development side of plugins it seems there is already an order system in place any reason that this is not exposed in the schema's of the plugins to allow a different order? An order map could be created at an endpoint on the admin to accept an api id/name to see the order of plugins applied based on some database lookups.
https://github.com/Mashape/kong/blob/master/kong/plugins/oauth2/handler.lua#L15
OAuthHandler.PRIORITY = 1000

Please use plugin-ordering feature for this purpose. It is an enterprise only feature at the time of this comment.


SLAM crashes csgo when starting a song

From version 1.5.1 SLAM crashes my game whenever I start a song. I found out you have this new FFMPEG thing. When I go back to NAudio everything is ok but I want to see if the FFMPEG has better audio quality :DD Thank you for helping.

FFMPEG should be the same quality if I'm correct.

@TypicalHog Yes, you are correct.
v1.5.1 has been deprecated. v1.5.3 should fix the issue, if not, open a new issue on GitHub.


setting update does not work at commands

I have some artisan commands like update price every 10 minutes and
setting(["price" => "2555"]);
Setting::set("price" , "2555");

do not work at all but at the controllers, middlewares,.... its fine

There is nothing special for commands and it works fine for Akaunting software.


[Document]Guide documents

Getting Started with Rooch
ref: Getting Started with Rails

Finished in #389 #447 #458 #462


SyntheticEvent Error

Hello , well i m trying to make a map app with react-native , so i used your Example guys and this what i got in compilation  :

i don't know why , i tried this #1394 as a solution but it didn't work :/
i m using  : "react-native": "0.48.1"
"react-native-maps": "^0.16.3"
Thanks

Use this example and comment please. https://github.com/alvelig/react-native-maps-example


How to determine what changed in updateMessageContent

I am making a chat bot. When the owner of the bot edits the message that has been sent, it needs to modify the message in the corresponding chat, through the methods such as editMessageText and editMessageCaption.
The problem is that when I receive an updateMessageContent , I can't tell what changed, because I can't get the old information, I think, when I call getMessage , it shouldn't be able to return the old one.
I think it's not a good idea to store all the messages in memory, or serialize them and save them in another database or file...
Forgive me for poor English, this is the result of Google Translate :(

By the way, I can compare it with messages that have been copied and sent to other chats, but what if I don't have this message?

You can use getMessage on a message you a going to change to get its current content and to find what needs to be changed.

@nekohasekai did u resolve this?


Word Boggle using Trie

🚀 Feature
In this problem we have a dictionary of several words and a MxN board in which every cell has one character, and we have to check the number of words which are present in dictionary and can be formed by the sequence of adjacent characters on board, using the trie data structure.
Have you read the Contributing Guidelines on Pull Requests?
Yes
Motivation
As a participant of GSSoC'21, I want to contribute to this project.
Pitch
This is an important problem and hence, should be added.
I would like to solve it using c++ and I assure you that I would write a clear code with proper comments.
I request you to please assign me this problem.

/assign