Saving Photo/Video data and presenting into new View Controller

Hi there,
I appreciate the one callback function didFinishPicking, but when placed into the view life cycle, it doesn't activate when the save button is pressed(at the end of choosing, cropping, filtering, etc). Instead, it saves the photo to album and that's it. I needed to make an instagram based preview page to add a caption and what not in order to migrate.
Is there any way to do this without an edit to the pod? I had to edit the didSelect function's switch cases. I added a few lines of code, now the function presents the data I need on a new page which is going to allow me to add a post to my database. Also consider using a delegate/protocol for view controller communication.
If you can fix this, I think people may have a use case for this type of thing.

try
picker.pushViewController(...)

try
picker.pushviewcontroller(...)

@ewjc The single didFinishPicking is something we came up with internally after a long discussion.
There are drawbacks and advantages to both approaches (delegate vs callbacks) and we agreed this was more "swifty", although less common among libraries.
Your use case is perfectly valid.
If you want to dismiss the picker and present another sheet modally than you can use this :
    func showPicker() {
        var config = ...
        let picker = YPImagePicker(configuration: config)

        picker.didFinishPicking { [unowned picker] items, cancelled in

            if cancelled {
                picker.dismiss(animated: true, completion: nil)
                return
            }

            let postVC = PostStatusVC()
            if let photo = items.singlePhoto {
                postVC.preloadedImage = photo.image
            } else if let video = items.singleVideo, let videoData = try? Data(contentsOf: video.url) {
                postVC.preloadVideo(with: videoData, thumbnail: video.thumbnail)
            }
            picker.dismiss(animated: true) {
                present(postVC, animated: false, completion: nil)
            }
        }
        present(picker, animated: false, completion: nil)
    }

as @10000TB pointed out the picker is itself a UINavigationController so you can push your own views inside it natively.
Essentially replace
picker.dismiss(animated: true) {
    present(postVC, animated: false, completion: nil)
}

in the above code by:
let myVC = MYViewController()
picker.pushViewController(myVC, animated: true)

Hope this helps :)

@s4cha Hi mate, I know I am months late. Any reason why I am getting a black screen when trying to present or pushVC?

@s4cha Any reason why I am presented with two navigation bars when using picker.pushViewController... ?
I have my navigation bar that includes a share button, and I also have the YPPicker nav bar with the back button included. Anyway I can get the YPPicker nav bar and also include my title and share button?

@s4cha Do you know how I can access the photo in the next view controller? It is not appearing after I push. Did I have to configure the picker configure? Heres my code
`   let picker = YPImagePicker()
picker.didFinishPicking { [unowned picker] items, cancelled in
        if cancelled {
            picker.dismiss(animated: true, completion: nil)
            return
        }

        if let photo = items.singlePhoto {
            let sharePostVC = SharePhotoViewController()

            sharePostVC.selectedImage = photo.modifiedImage ?? photo.image

        }

            let shareVC = self.storyboard?.instantiateViewController(withIdentifier: "SharePhotoController")
            picker.pushViewController(shareVC!, animated: true)

And after I push to the next VC..
     var selectedImage: UIImage? {         didSet {             self.finalPhoto?.image = selectedImage         }     }
Do you see anything wrong? No errors, however no photo appears.


Make the Deafault Redirect customizable

Right now on line 32 of src\shortenerTools\UrlRedirect\UrlRedirect.cs
var redirectUrl = "http://azure.com";
It would be better to have this value expose as a parameter in the ARM template.
The ARM template will add it to the setting and the Function will read it from there.

@all-contributors add @FBoucher for code,video

@all-contributors add @FBoucher for code video


recordingFinished is never received

basically I tried implement #1.
https://github.com/jsierles/react-native-audio/blob/master/AudioExample/index.ios.js#L36
This code is never executed. If you have another behaviour please let me know.

I'll take a look at this - haven't checked this component in a while as I'm not actively using it.

This should now be fixed on master. Would love to see a PR for audio settings!


[BUG] addHousekeepingBeforeUnload should also be listening to the 'unload' event

Add and export the following helper to listen for all "unload" / "navigate" events
export function addPageUnloadEventListener(listener: any): boolean {
    // Hook the unload event for the document, window and body to ensure that the client events are flushed to the server
    // As just hooking the window does not always fire (on chrome) for page navigations.
    let pageUnloadAdded = addEventHandler("beforeunload", listener);
    pageUnloadAdded = addEventHandler("unload", listener) || pageUnloadAdded;
    pageUnloadAdded = addEventHandler("pagehide", listener) || pageUnloadAdded;
    return pageUnloadAdded;
}

And update addHousekeepingBeforeUnload() in AISKU/Initialization.ts to use it

v2.6.2 is now fully deployed


[Bug]: Nullable struct fields in code

Platform
all
Plugin
package_info_plus, possibly more
Version
latest
Flutter SDK
master
Steps to reproduce
https://ci.chromium.org/ui/p/flutter/builders/prod/Linux_android new_gallery__crane_perf/6462/overview
Code Sample
class _LANGANDCODEPAGE extends Struct {
  @Uint16()
  external int? wLanguage;

  @Uint16()
  external int? wCodePage;
}

should be
class _LANGANDCODEPAGE extends Struct {
  @Uint16()
  external int wLanguage;

  @Uint16()
  external int wCodePage;
}

Logs
-

Flutter Doctor
-

Checklist before submitting a bug

[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I'm using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

Fixed https://pub.dev/packages/package_info_plus/versions/3.0.3


Additional Properties

Hi thanks for this module, I was wondering are you able to retrieve additional parameters like flowrate and the return flow temp?  Also I can see the heat curve setting but cannot change it, is it possible so that it can be changed.  I have the arotherm heatpump.  Appreciate your feedback

You can try the export and generate test data commands to get the raw data. If you can't find the data in there, it's likely not getting reported by the API.
As for changing values, if it's not possible through the app I can't implement it with this library. AFAIK heat curve can't be updated through the app.


Specify the "g" in gRPC

Nice :). Having it in the user agent may be a bit much, though.
@vjpai is just a naysayer. NAYSAYER!

I'm not a naysayer; rather, @ejona86 is a heretic. The g in gRPC has always stood for "gRPC" : http://www.grpc.io/faq/ . You're entitled to your opinions, but please let us live our traditional gRPC lifestyle in peace. Heretic.

I have two things to say:

LGTM

If I didn't know better, I'd swear that you have too much free time. :)

Added requisite documentation so that we can keep track of what we've used before.

You can have my LGTM, conditional on 1.2 being "give_you_up,never_gonna"

LGTM

@vjpai you wanna make an rpc system who's me includes 'never gonna rpc'... I'm not sure it's wise.

give_you_up,never_gonna_let_you also doesn't work well.

So we got: gRPC := (g := gRPC)RPC => gRPC^2 => [σg = good]RPC^2 => goodRPC^2
This is too much fun, don't LGTM it yet :)

This seriously just happened?
On Tue, Aug 30, 2016, 9:09 AM Craig Tiller<EMAIL_ADDRESS>wrote:

Merged #7912 https://github.com/grpc/grpc/pull/7912.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/pull/7912#event-772348693, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AIdEkPwJGUtY_VEBERNzFbcffu3rZr-8ks5qlE7jgaJpZM4JvwRK
.

Far out man :)

I think the 1.2 release should be "groovy". :)

genuine
gnarly
gripping
gallant
glad

Let's get a survey ready for the community for the 1.2 timeframe.
On Tue, Aug 30, 2016 at 9:42 AM Vijay Pai<EMAIL_ADDRESS>wrote:

genuine
gnarly
gripping
gallant
glad
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/pull/7912#issuecomment-243500331, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJpudbiNIV19JmQvXHUbwNW38oDzh6V7ks5qlFw2gaJpZM4JvwRK
.

I'll throw 'gregarious' and 'ginormous' and 'generous' to the mix.


Error to serve when we have space in project directory (Windows)

Nothing I can do with your info.

OS : Windows 10
Steps to reproduce :

download source code
add space in project directory : d:/sources/vue-cli-plugin-vite-main\vue-cli-plugin-vite-main\e x a m p l e s\my-spa-js-app
/> yarn
/> yarn vite

`node:events:355
throw er; // Unhandled 'error' event
^
Error: spawn vite ENOENT
at Process.ChildProcess._handle.onexit (node:internal/child_process:282:19)
at onErrorNT (node:internal/child_process:480:16)
at processTicksAndRejections (node:internal/process/task_queues:81:21)
Emitted 'error' event on ChildProcess instance at:
at Process.ChildProcess._handle.onexit (node:internal/child_process:288:12)
at onErrorNT (node:internal/child_process:480:16)
at processTicksAndRejections (node:internal/process/task_queues:81:21) {
errno: -4058,
code: 'ENOENT',
syscall: 'spawn vite',
path: 'vite',
spawnargs: [
'--config',
'D:\sources\vue-cli-plugin-vite-main\vue-cli-plugin-vite-main\config\index.ts'
]
}`

sorry wont fix. you should remove space.

???
In my case i can't remove space.
paths with spaces are common in windows.
Regards

@deka dont have windows. will figure it out when possible.

https://stackoverflow.com/questions/21356372/node-child-process-spawn-not-working-with-spaces-in-path-on-windows
maybe you can try

https://www.npmjs.com/package/cross-spawn
maybe you can use it in bin/vite

Can you give me feedback, I will release new version if it works.

Hi,
cross-spawn works
In bin/vite :
const { spawn } = require('spawn')
const { spawn } = require('cross-spawn')
Regards

@deka try rc.2

hi, I'm try using this repo commit:c5bd5efa9d9567e87465faac4684f7ecf0b4aa12 under windows11.
The project name content '-' cause 'Error spawn vite ENOENT '. finally it worked. However, when I click the button, it stopped.
PS D:\code\tem\example> yarn vite
yarn run v1.22.10
$ set USE_VITE=true && node ./bin/vite
running: vite --config D:\code\tem\example\node_modules\vue-cli-plugin-vite\config\index.ts

  vite v2.5.1 dev server running at:

  > Local: http://localhost:9527/
  > Network: use `--host` to expose

  ready in 3355ms.

Loading Vetur in workspace path: D:\code\tem\example

node:internal/event_target:641
  process.nextTick(() => { throw err; });
                           ^
Error: Unhandled method window/showMessageRequest
    at handleResponse (D:\code\tem\example\node_modules\vscode-jsonrpc\lib\common\connection.js:477:48)
    at processMessageQueue (D:\code\tem\example\node_modules\vscode-jsonrpc\lib\common\connection.js:292:17)
    at Immediate.<anonymous> (D:\code\tem\example\node_modules\vscode-jsonrpc\lib\common\connection.js:276:13)
    at processImmediate (node:internal/timers:464:21)
Emitted 'error' event on Worker instance at:
    at Worker.[kOnErrorMessage] (node:internal/worker:297:10)
    at Worker.[kOnMessage] (node:internal/worker:308:37)
    at MessagePort.<anonymous> (node:internal/worker:211:57)
    at MessagePort.[nodejs.internal.kHybridDispatch] (node:internal/event_target:461:20)
    at MessagePort.exports.emitMessage (node:internal/per_context/messageport:23:28) {
  code: -32601,
  data: undefined
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.


Add blocklist for xiaomi

I'm seeing the following hosts in my pi hole that is used by xiaomi phones to track activity.
It's been blocked by nextdns here here (https://github.com/nextdns/metadata/blob/master/privacy/native/xiaomi). I noticed that the format of this file is not the standard <IP_ADDRESS> HOSTNAME, so I'm not sure how to get this integrated in the host list directly (short of copying the file over to this repository).
Incidently nextdns has a blocklist with other IoT devices as well (https://github.com/nextdns/metadata/tree/master/privacy/native) that could be integrated here.
Could you let me know how to proceed here, I'm happy to submit a PR if you point me to a previous PR that I can use as a starting point.

Thanks @ketan for this.
I suppose we could make a case for all its sibling folders too.


Support Hub Event Service

Hub Connector already supports the definition of Event processing rules in both directions:

ADAMOS Hub (AMQP) -> C8Y Measurement / Event / Alarm
C8Y Events -> ADAMOS Hub Event Service

but there is currently no execution logic for the latter. This needs to be implemented here for the two supported Hub events:

stack light update
state update

Implemented but validation errors.

Validation errors gone with correct format.


Adding enc file

Decided to close the last 2 PRs cause I think the encryption keys weren't being store correctly. This PR merges the master branch with ebi-improvements and adds the enc file.

Not sure why coveralls is not updating but this is ready to go!


Possible bug report about fluid simulation

Dear Keita Nakamura,
I greatly appreciate your Julia library "Tesserae.jl."
And I have run some of the listed tests, but encountered errors during the fluid simulation test.
The related files are attached for your review.
My julia version is 1.11.1.
System information: macos, ubuntu and windows11 (All give error information and erupt.)
If you could check them, it would be very helpful for other users of your library.
Additionally, I would like to learn more about implicit formulation cases.
Could you please provide additional example cases?
Thank you for your time and assistance.
file.zip

Thank you for the report. However, I was unable to reproduce the errors. Could you please confirm the following?

Is the simulation file exactly the same as the one used in the original fluid simulation tutorial?
Do you still encounter the errors when using a smaller time step, such as Δt = 1.0e-3?

For the implicit formulation, please open a separate issue.

Sorry to tell you, code still erupt at approximately 50% progress when Δt = 1.0e-3.
Could you send your running script of this example here?
My running file is copied from tutorial.
I want to check whether they are the same.

https://github.com/user-attachments/assets/c71110c2-6d6b-49a7-99df-b1e2439193f4
https://github.com/user-attachments/assets/c607f791-82cc-4cce-9412-6b9eabb96af5
I have uploaded videos on both Windows and Linux.
Linux displays error messages, whereas Windows does not.
The progress bar reaches 100% rapidly, but no actual processing takes place.

When viewed in ParaView, only the first three seconds of the animation play; the rest is blank.

I reproduced the error on CI and am now checking the details.

I found some information that might be helpful.
First, changing the base parameters of the input doesn't alter the result.
However, the parameters used in Newmark's method appear to differ from those recommended by Wikipedia.
My understanding of integration algorithms is limited, and there could also be other issues at play.

Great!


Inconsistent paths to rtcsync.sh

First, thanks for your scripts to fix the RTC with read-only!
I noticed a couple errors with the path to rtcsync.sh, however.  In embedtools.sh, the path is /usr/bin/rtcsync.sh.
But, in hwclock-set.sh, the path is /bin/rtcsync.sh on lines 44  and 47.

Oh! Excellent catch! It should probably all be in /bin as a system utility
I'd imagine.
I probably was running it manually for testing and didn't notice!
On Sat, Feb 23, 2019, 3:18 PM Chris Bartley<EMAIL_ADDRESS>wrote:

First, thanks for your scripts to fix the RTC with read-only!
I noticed a couple errors with the path to rtcsync.sh, however. In
embedtools.sh, the path is /usr/bin/rtcsync.sh
https://github.com/EternityForest/scripts/blob/master/embedtools/embedtools.sh#L464
.
But, in hwclock-set.sh, the path is /bin/rtcsync.sh on lines 44
https://github.com/EternityForest/scripts/blob/master/embedtools/hwclock-set.sh#L44
and 47
https://github.com/EternityForest/scripts/blob/master/embedtools/hwclock-set.sh#L47
.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/EternityForest/scripts/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAuRH5qMCWERqrjgxXG0iCCIGYesbPgOks5vQcxJgaJpZM4bON72
.

Ah, one more, in rtcsync.service: https://github.com/EternityForest/scripts/blob/master/embedtools/rtcsync.service#L6

I haven't fully tested it yet, but it should be fixed along with a few other things, like being more consistent about the ramdisk drift file, and using -w to set the time instead of assuming that -systz does it like some tutorials seem to think you should.


Using Atom inside WSL opens a blank file and System 32 as project folder in Atom for Windows

Prerequisites

[x ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
When using Atom to open a WSL file, Atom for Windows opens the file as blank and the System 32 folder is added as a Project Folder instead of the current WSL folder.
I apologize if I am using the wrong terminology for this. Please ask for any clarifications and I'll be happy to provide them.
Steps to Reproduce

Open Bash on Ubuntu on Windows (or any other bash shell)
Type atom [your file]

Expected behavior: [What you expect to happen]
Atom opens file with its contents and adds the current WSL folder as the project folder.
Actual behavior: [What actually happens]
Atom opens the file but it is blank and the System 32 folder is added as the project folder.
The file path in Atom says "C:\WINDOWS\system32[name of file] which is wrong.
Reproduces how often: [What percentage of the time does it reproduce?]
Constant.
Versions
You can get this information from copy and pasting the output of atom --version and apm --version from the command line. Also, please include the OS and what version of the OS you're running.
This occurred on latest stable Atom for Windows version and latest beta (1.23.0-beta1 x64)
OS is Windows 10 Pro with latest updates.
Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.
This seems to have started after updating Ubuntu last night.

I guess I misunderstood what was possible in WSL. This explains what can and cannot be done with editing / opening WSL files from inside WSL using Windows programs - https://blogs.msdn.microsoft.com/commandline/2016/11/17/do-not-change-linux-files-using-windows-apps-and-tools/
Is this information still correct with the Windows 10 Fall Creators Update?

Yeah I poked around a bit and found I can work with files under /mnt/c/User/username, otherwise I see the same error.  Issues like https://github.com/Microsoft/WSL/issues/2004 seem to confirm this?

Is this information still correct with the Windows 10 Fall Creators Update?

The WSL community links might point to good places to ask that question.

I get this issue:
Atom    : 1.39.1
Electron: 3.1.10
Chrome  : 66.0.3359.181
Node    : 10.2.0
Git Bash on WIndows
cd into working directory and command "atom filenm" will open the file in the directory but it will be empty (ie erases all content in file)


Support for Chrome's Tab Groups

APIs

https://developer.chrome.com/docs/extensions/reference/tabs/

https://developer.chrome.com/docs/extensions/reference/tabs/#method-group

Adds one or more tabs to a specified group, or if no group is specified, adds the given tabs to a newly created group.

https://developer.chrome.com/docs/extensions/reference/tabs/#method-ungroup

Removes one or more tabs from their respective groups. If any groups become empty, they are deleted.

https://developer.chrome.com/docs/extensions/reference/tabs/#method-query

Gets all tabs that have the specified properties, or all tabs if no properties are specified.

https://developer.chrome.com/docs/extensions/reference/tabs/#method-update

Modifies the properties of a tab. Properties that are not specified in updateProperties are not modified.

https://developer.chrome.com/docs/extensions/reference/tabs/#method-highlight

Highlights the given tabs and focuses on the first of group. Will appear to do nothing if the specified tab is currently active.

https://developer.chrome.com/docs/extensions/reference/permissions/

Use the chrome.permissions API to request declared optional permissions at run time rather than install time, so users understand why the permissions are needed and grant only those that are necessary.

Dev APIs

These APIs are only available in the Chrome Dev channel

https://developer.chrome.com/docs/extensions/reference/tabGroups/

Use the chrome.tabGroups API to interact with the browser's tab grouping system. You can use this API to modify and rearrange tab groups in the browser. To group and ungroup tabs, or to query what tabs are in groups, use the chrome.tabs API.

https://developer.chrome.com/docs/extensions/reference/tabGroups/#method-get

Retrieves details about the specified group.

https://developer.chrome.com/docs/extensions/reference/tabGroups/#method-move

Moves the group and all its tabs within its window, or to a new window.

https://developer.chrome.com/docs/extensions/reference/tabGroups/#method-query

Gets all groups that have the specified properties, or all groups if no properties are specified.

https://developer.chrome.com/docs/extensions/reference/tabGroups/#method-update

Modifies the properties of a group. Properties that are not specified in updateProperties are not modified.

Is this abandoned?

@jeanlucaslima It's not abandoned.. but it keeps getting pushed down my priority list.


Refactor synchronizers

This is a refactoring-only PR with changes to Synchronizer, CollectionSynchronizer, and DocSynchronizer classes. See the commit messages for a summary of the changes.

Current dependencies on/for this PR:

main

PR #26

PR #27

PR #28

PR #29

PR #30   👈

This comment was auto-generated by Graphite.

included in v0.0.50


What is the Torch version？

my torch.version is 1.2.0 and have problem about torch.load() .
What version do you use?
RuntimeError: ./face_paint_512_v2.pt is a zip archive (did you mean to use torch.jit.load()?)

I'm using 1.10.0. I guess you need torch >= 1.7.1

I'm using 1.10.0. I guess you need torch >= 1.7.1

think u for ur reply


NIFI-5652: Fixed LogMessage when logging level is disabled

I couldn't write a unit test as I can't change the log level in between unit tests. I reproduced the issue and verified it is no longer present with a live NiFi instance running with this patch.
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

LGTM +1, merging. Thanks @mattyb149!


nixpkg bump leads to regression

Comparing from 14799fea671749dd303f439bb27526f61d3cf344 to 2c79284755360718391121b57701938142be2e8e:
In terms of gas, 1 tests regressed and the mean change is +6.1%.
In terms of size, no changes are observed in 5 tests.
Originally posted by @github-actions[bot] in https://github.com/dfinity/motoko/issues/4114#issuecomment-1630343128

Maybe because there is a new test?

Maybe the cost model has changed...


Ice error message for invalid transaction ID prints junk

Running against master
ice ERROR: 2019/05/31 12:44:31 discard message from (relay ... related ...), invalid TransactionID ؛������+�

@ernado mind checking this out when you get a chance? thanks!


Only apply roles from first master node to fix regression

The following regression happend where master 2 completed before 01. No need to apply the same clusterroles multiple times. Only do it from master 1.
TASK [kubernetes-apps/cluster_roles : Apply workaround to allow all nodes with cert O=system:nodes to register] *************************
Sunday 18 March 2018  16:09:13 +0100 (0:00:01.164)       0:17:40.526 **********
ok: [odn1-kube-cluster01-master02] => {"changed": false, "msg": "success: clusterrolebinding \"kubespray:system:node\" created"}
fatal: [odn1-kube-cluster01-master01]: FAILED! => {"changed": false, "msg": "error running kubectl (/opt/bin/kubectl apply --force --filename=/etc/kubernetes/node-crb.yml) command (rc=1), out='', err='Error from server (AlreadyExists): error when creating \"/etc/kubernetes/node-crb.yml\": clusterrolebindings.rbac.authorization.k8s.io \"kubespray:system:node\" already exists\n'"}
ok: [odn1-kube-cluster01-master03] => {"changed": false, "msg": "success: clusterrolebinding \"kubespray:system:node\" configured"}

@kongslund Could you verify this?

Good catch. I will try it out.

ci check this


feat: switch projections & allow advanced projections

This PR brings the usage of projections, as described in #702.
Out-of-the-box, EPSG:4326 is supported in addition to the standard EPSG:3857. Other projections can be used by calling the helper-function registerProjection first, including a name and proj4 definition string. These can be obtained by via services like epsg.io, this must be handled by the parent application.
Switching the projection is possible by setting the projection-attribute. A new view is created when doing so, listener functions are moved over to the new view. It is important that set listeners retrieve the view via evt.target, instead of this.
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I have added a test related to this feature/fix
[x] For new features: I have created a story showcasing this new feature
[x] All checks have passed
[x] The PR title follows conventional commit style and reflects the type of change (major/minor/patch, breaking)
[x] The PR title describes the change within this element (each merged PR equals one entry in the element's changelog!)

@RobertOrthofer Thank you very much for this new feature! :rocket:
I have an implementation detail question - what will happen now if the old "center" of current view can not be projected into the new projection? Will it not throw an openlayers exception?
Example: When using a polar stereographic projection centered on pole and decide to switch my map to EPSG:3857 which has projection bounds geographically around 85 degrees so my original center will be unprojectable. Would it be a good idea to in that case set center to new projection center ?

that is a great question, do you have an EPSG-code to test this out? I have intentionally left out setting the extent of the view to the extent of the projection for now, as this might limit some very edge cases

@RobertOrthofer Let's try this out.
For example the North Polar stereographic can be used:
proj4 def:
{
  name: 'EPSG:3411',
  def: '+proj=stere +lat_0=90 +lat_ts=70 +lon_0=-45 +x_0=0 +y_0=0 +a=6378273 +b=6356889.449 +units=m +no_defs +type=crs',
  extent: [-3314763.31, -3314763.31, 3314763.31, 3314763.31],
}

I tried this with the registerProjectionFromCode-helper and it looks like it always manages to reproject the center. Obviously, if the center is way off the projections bounds, the map becomes quite distorted, but usable...
I think for now this would be the responsibility of the application to move the map into the area of interest beforehand, as limiting the extent might not always be the desired behaviour

Perfect! Thanks for checking!


Why does items allow non arrays?

See test here: https://github.com/json-schema-org/JSON-Schema-Test-Suite/blob/5fb3d9f1a1c4136f544fbd0029942ea559732f8e/tests/draft4/items.json#L18
It seems like this makes items useless in practice. If other types are allowed, they should be explicitly declared via oneOf.

I find it a strange design choice too, but it's the way all the validation keywords work. You need to use "type": "list" in addition to items to get what you want.

Yeah, it's in effect an Any. Thanks for clarifying!


Add Natural Language Processing course in Hindi

What does this PR do?
Add course
For resources
Description
Tutorial for Natural Language Processing.
Why is this valuable (or not)?
It covers all concepts about Natural Language Processing.
How do we know it's really free?
Its available on youtube.
For book lists, is it a book? For course lists, is it a course? etc.
It is a Youtube Tutorial.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

it's disrespectful to reviewers to abandon a PR with comments. use #8551

Okay Sir.


Update package.json

On the auth-routes example the start script sets the node environment with NODE_ENV, which is not a recognised command on Windows. The work around is to use the cross-env package or to have two separate scripts, one for Windows and one for Unix based systems. I figured using the cross-env package is much more friendly and will save confusion, tested on both Windows and Mac.

Current coverage is 71.09% (diff: 100%)

Merging #65 into master will not change coverage

@@             master        #65   diff @@
==========================================
  Files            10         10
  Lines           391        391
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            278        278
  Misses          113        113
  Partials          0          0

Powered by Codecov. Last update 0f81a13...71b6b99

Thank you @antcook
Will be added to the next release 👍


Super slow on .NET 5 WebAsm

I'm building a web app for export acb for d4dj in one key with blazor wasm, but it's super slow, that takes lots of mins to encode a 2min audio file.
Even the WaveReader.Read() takes about 20 sec!
here's my code:
        async Task<(byte[], int)> EncodeHCA(byte[] audio)
        {
            var reader = new WaveReader();

            var writer = new HcaWriter
            {
                Configuration = new HcaConfiguration
                {
                    Bitrate = 128 * 1024,
                    Quality = CriHcaQuality.Highest,
                    TrimFile = false,
                    EncryptionKey = new CriHcaKey(0x59f449354d063308)
                }
            };

            var audioData = await Task.Run<AudioData>(() => reader.Read(audio));
            var format = audioData.GetFormat<Pcm16Format>();

            byte[] output = null;

            using (var ms = new MemoryStream())
            {
                await Task.Run(() => writer.WriteToStream(audioData, ms));
                await ms.FlushAsync();
                output = ms.ToArray();
            }

            return (output, (int)(format.SampleCount / (float)format.SampleRate * 1000));
        }

Or maybe it's a issue from dotnet5 idk

I'm not sure why that would happen. Is there any sort of profiling available for .NET wasm?

I'm not sure why that would happen. Is there any sort of profiling available for .NET wasm?

I found a topic about this
https://remibou.github.io/Profiling-in-Blazor/

What does it tell you when you profile the slow code?

Problem solved! after upgrading to .NET 7 and enabling AOT, it is lighting fast


Figure out how to support "usable auth" for gcloud-node

See https://github.com/google/google-api-nodejs-client/pull/354
Ideally we would somehow integrate this code. Maybe it means a separate google-authkit-node repository that we depend on ?
/cc @stephenplusplus @ryanseys

Ideally there would be 1 module that all of auth gets incorporated into. Right now, it's all mixed across gtoken, google-service-account, among other libraries and implementations. I like the way that googleapis currently accomplishes at least the accepting of generic auth objects:
var google  = require('googleapis');
var OAuth2 = google.auth.OAuth2Client;
var JWT = google.auth.JWT;
var myAuthClient = new OAuth2(/** ...  */);
myAuthClient = new JWT(/* ... */); // or use a JWT client
myAuthClient = 'SOME API KEY'; // or even a plain API key.

var drive = google.drive({ version: 'v2', auth: myAuthClient });
drive.files.insert(options, callback);

So I think it's a matter of removing that auth logic and objects (plus the changes in https://github.com/google/google-api-nodejs-client/pull/354) into a giant epic auth module.

There's a repo in the works here for nodejs auth: https://github.com/google/google-auth-library-nodejs
@jgeewax can you get @stephenplusplus access to this?

I'm in. Thanks!

Just to be clear, what I mean by usable auth here is "if I don't want to think about it in code, I don't have to think about it".
Right now, we have to explicitly define the credentials and whatnot somewhere. If that ever happens, we should use those settings. End of story.
If those aren't set anywhere, we should evaluate things in the following order:

If I have environment variables for anything credential related, use that. break;
If I'm in App Engine, use that set of credentials and defaults. break;
If I'm in Compute Engine, use that set of credentials and defaults. break;

This means that the "default case" (var gcloud = require('gcloud');) which previously meant "this still needs to be configured" now means "we'll try the list of stuff, and it might be configured depending on the the environment".
In code this means the following should work in GAE, GCE, or if I have the right environment variables set:
var gcloud = require('gcloud');
dataset = gcloud.datastore.dataset();
query = dataset.createQuery('Person');
dataset.runQuery(query, function(...) { ... });

You can still always explicitly override those things later:
// Continued from above, with the same gcloud that *was* configured properly via env
var dataset = gcloud.datastore.dataset({
  projectId: 'myProject',
  keyFilename: '/path/to/keyfile.json'
});
// Dataset uses another set of explicit credentials.

This may or may not merit a helper method gcloud.isConfigured() which returns true if and only if the instance of gcloud is ready to send requests.

This is a good plan, it's just a matter of nailing down the environment variables to look for in all of those environments and any API endpoints/authentication procedures that may changed based on the environment.

Here's how googleapis may be changed to support this: https://github.com/google/google-api-nodejs-client/pull/374
Adding here for reference when we want to consider using the same library.

Just FYI, hopefully we can rely on https://github.com/google/google-auth-library-nodejs (npm install google-auth-library).
Do the methods in that library do the thing we want ?

If https://github.com/google/google-auth-library-nodejs has gaps, please add an issue or send a PR.  We will respond promptly.

The new library https://github.com/google/google-auth-library-nodejs is defintely the home for this. The default credentials should by fully functional now. Please have Veneer take a dependency on this library. It should allow you to have a default behavior that uses the ADC under the covers and no credentials as input, and also have way of initializing the service that takes the type returned by this method as an optional input.

https://github.com/google/google-api-nodejs-client just shipped 2.0.0 with these changes!


GraphQLQuery name attribute is ignored

Hello all,
I have the following configuration:
    @Bean
    public GraphQL graphQL(Queries queries,
                           Orchestrator orchestrator) {
        GraphQLSchema schema = new GraphQLSchemaGenerator()
            .withDefaults()
            .withResolverBuilders(
                new AnnotatedResolverBuilder()
            )
            .withOperationsFromSingleton(orchestrator)
            .withOperationsFromSingleton(queries)
            .withValueMapperFactory(new JacksonValueMapperFactory())
            .generate();
        return GraphQL.newGraphQL(schema).build();
    }

And the following bean:
@Getter
@Setter
@Accessors(chain = true)
public class TestDocument {
    @Field("custom_id")
    @GraphQLQuery(name = "custom_id")
    @GraphQLInputField(name = "custom_id")
    private String customId;
}

The actual root queries are:
@Component
public class Orchestrator {
    @Autowired
    private TestRepository testRepo;

    @GraphQLQuery(name = "persist")
    public Optional<TestDocument> persist(@GraphQLArgument(name = "tested") TestDocument tested) {
        testRepo.save(new ValueWrapper().setTest(tested));
        return Optional.of(tested);
    }
}

And
@Component
public class Queries {
    @Autowired
    private TestRepository testRepository;

    @GraphQLQuery(name = "testTest")
    public List<TestDocument> getAll(){
        return testRepository.findAll().stream().map(ValueWrapper::getTest).collect(Collectors.toList());
    }
}

Interestingly the "custom_id" field only seems to be present in the Input* part of the schema. The generated type has "customId", i.e. I have a graphql query of the following format:
{
  persist(tested: {
    custom_id: "20"
  }) {
    customId
  }
}

SPQR doesn't deal with private members, so annotations on them are ignored. If it works for input fields, it's by accident and not by design. This means it's not Lombok friendly, and if you want to customize the mappings via annotations, you'll have to add getters/setters explicitly.
I tried making it work more seamlessly with Lombok but ended up discarding everything because it required very ugly code I didn't want to have to maintain...
I'll probably try again with a different approach, but it's unlikely to happen prior to 1.0.

 This is equivalent to your example:
GraphQLSchema schema = new GraphQLSchemaGenerator()
            .withOperationsFromSingletons(orchestrator, queries)
            .generate();


train mnist

[] Close main session!
[] DiskImageData: stop threads and close session!
ｈｏｗ to solve it?
@LynnHo

@chl916185 Which command did you use? Can you show more information?

Epoch: ( 99) (    4/   10)
Epoch: ( 99) (    5/   10)
Epoch: ( 99) (    6/   10)
Epoch: ( 99) (    7/   10)
Epoch: ( 99) (    8/   10)
Epoch: ( 99) (    9/   10)
Epoch: ( 99) (   10/   10)
Model saved in file: ./checkpoints/cartoon_wgan_gp/Epoch_(99)_(10of10).ckpt
[] Close main session!
[] DiskImageData: stop threads and close session!
(tensorflow17) chl@chl-System-Produ
@LynnHo

@chl916185  It just means the training is finished.


Add tests to use Chinese and space character in source name

🚀 Feature Request
Description [REQUIRED]

Add tests to use Chinese and space character in source name

it will be for the next release (v3.1)


pull request for e2e test at 11/06/2016 18:26:08

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Implement management.cattle.io schema just Go Automation Framework

Is your feature request related to a problem? Please describe.
Because of a change made to the native eks struct, pr: https://github.com/rancher/eks-operator/pull/59, it is no longer possible to have empty values for tags, loggingTypes, subnets, etc. because of the behavior of golang json marshaling.
Describe the solution you'd like
The solution is for the go automation framework to have it's own schema, where we can change those fields back to pointers.

PR v2.7: https://github.com/rancher/rancher/pull/39034
PR v2.6: https://github.com/rancher/rancher/pull/39035


feat(linter): implement typescript-eslint/ban-ts-comment

related to #503, marked as nursery(though all tests pass) since we use rust regex crate to parse the regex, may have compatibility problems with ECMA regex.

The regex seems pretty simple to implement as a tiny lexer.
I have this from my internal code, which may or may not be correct:
In
impl Trivia {
    /// For TypeScript comment directive `@ts-ignore` / `@ts-expect-error` / `ts-nocheck` / `ts-check`,
    /// Find and return the matching span witin the comment
    #[must_use]
    pub fn find_ts_comment_directive(&self, source_text: &str) -> Option<Span> {
        let prefix = "@ts-";
        let range = self.range();
        let raw = &source_text[range];
        // For multi-line comments, get the last line
        let mut lines = raw.lines();
        let line = lines.next_back()?;
        let index = line.find(prefix)?;
        if !line[..index].chars().all(|c| {
            c.is_whitespace()
                || match self.kind() {
                    // allow `//////@ts-` for SingleLineComment
                    TriviaKind::SingleLineComment => c == '/',
                    // allow `/*****@ts-` for MultiLineComment
                    TriviaKind::MultiLineComment => c == '*' || c == '/',
                }
        }) {
            return None;
        }
        let multi_len = lines.map(|line| line.len() + 1).sum::<usize>();
        let start = index + prefix.len();
        for directive in ["expect-error", "ignore", "nocheck", "check"] {
            if line.get(start..start + directive.len()) == Some(directive) {
                let start = multi_len + index;
                let end = start + prefix.len() + directive.len();
                return Some(start..end);
            }
        }
        None
    }
}

Would you like to try and implement the regex as a tiny lexer? I can also merge this and leave the refactoring to the next person.

Looks pretty cool! I think I want to get this PR merged and maybe give it a try in another PR.


MultiSignature Accounts Error

After signing transaction from accounts, receive next error:
16: mandatory-script-verify-flag-failed (Signature must be zero for failed CHECK(MULTI)SIG operation)

I don't have enough information to help you

Please reopen if you have more information to give me


support collection creation from java enumerables

It would be great to have support for creating vavr collections from instances of java.util.Enumerable

Codecov Report

Merging #2562 into master will decrease coverage by 0.18%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #2562      +/-   ##
============================================
- Coverage     92.82%   92.64%   -0.19%
  Complexity     5336     5336
============================================
  Files            89       89
  Lines         12742    12767      +25
  Branches       1609     1614       +5
============================================
  Hits          11828    11828
- Misses          727      752      +25
  Partials        187      187

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/io/vavr/collection/HashSet.java
93.77% <0%> (-1.99%)
140 <0> (ø)

src/main/java/io/vavr/collection/Array.java
98.16% <0%> (-1.15%)
243 <0> (ø)

src/main/java/io/vavr/collection/Queue.java
94.92% <0%> (-1.54%)
202 <0> (ø)

src/main/java/io/vavr/collection/List.java
98.51% <0%> (-0.93%)
265 <0> (ø)

src/main/java/io/vavr/collection/Vector.java
97.88% <0%> (-1.51%)
215 <0> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5398c38...59d31d3. Read the comment docs.

Hi & thank you for your PR.
Creating an issue would have been the better choice in the case of an API addition.
I think creating a collection based on an enumeration isn't a common use-case.
My current efforts go in direction of having ofAll(Iterable) as only API in the ofAll family.
Therefore I will close this ticket - but I also want to give you an alternative at hand: please take a look at Guava's forEnumeration(Enumeration enumeration) (maybe in conjunction with the method reference ::iterator). It will save us a lot of additional factory methods!
Thanks
- Daniel

Hi Daniel,
I was thinking of java API calls like
https://docs.oracle.com/javase/7/docs/api/java/net/NetworkInterface.html#getInetAddresses()
which unfortunately return an enumerable.
But you are right - wrapping it inside a utility method to convert the enumerable to an iterable is straightforward and keeps vavr slim.
Thanks for giving us vavr - I enjoy it daily and it makes living with java in companies that did not change to kotlin or scala bearable again :-)

@pertl Thank you for your kind words, Peter! 🙏

[P]lease take a look at Guava's forEnumeration(Enumeration enumeration) (maybe in conjunction with the method reference ::iterator). It will save us a lot of additional factory methods!

The Servlet API is similarly dated, with HttpServletRequest returning a number of Enumerations, and I am curious on what your suggestion entails.

Can you explain in more depth?
Is this a new central forEnumeration method?
Should we leverage Guava?
Something else.


Refactor default required credential fields in authentication mixin

Description
Container provider manager does not implement the function required_credential_fields, it relays on a wrong implementation in authentication_mixin. This PR implements the required_credential_fields in the container provider and remove the bad implementation in authentication_mixin
Screenshot
Before

After

Issue
https://github.com/ManageIQ/manageiq/issues/10689

@miq-bot add_label providers/containers
@simon3z @zeari please review

@miq-bot assign yaacov

@rubenvp8510 please review this- to see that doesn't cause a regression in the hawkular provider auth. thank you

I verified this on a hawkular provider, there is no regression. this looks good to me.

I like how this turned out (cleaner code). :+1:
@miq-bot assign chessbyte

@miq-bot add_label bug
This is a bug as it prevents a legitimate API call {"action": "refresh"} from completing successfully.

@blomquisg can we merge this?

@blomquisg ping

@yaacov looks like container_manager_mixin is overriding one of the methods of authentication_mixin.  Can you include a spec to ensure that a container_manager gets the correct implementation of required_credential_fields?
I just want to make sure that the inheritance chain never gets messed up there in the future.

@blomquisg Thanks, add two specs to check the required_credential_fields, one for container_manager and one for miq_region.
in https://github.com/ManageIQ/manageiq/commit/5dd67462ae5f1f7dcad82bea02b2c09fc2dbd461 a new authentication type "system_api" is introduced.
I re-factored it's required_credential_fields because I think it has the same problem as bearer, and this PR will fix it too.
@carbonin please review.

Thanks, @yaacov
The region changes look good and the behavior looks to be the same for everything except hawkular. If that's been cleared then this looks good to me :+1:

the behavior looks to be the same for everything except hawkular.

@carbonin Thanks, I've added a spec for in spec/models/miq_region_spec.rb , if it's the right behaviour of miq_region than it's ok :-)

Can you include a spec to ensure that a container_manager gets the correct implementation of required_credential_fields?

@blomquisg It already has an extensive spec for the regular cases [1]  e.g. [:userid] , the added specs in this PR are for the the irregular cases of container_manger [2] and miq_region ( @carbonin ) [3]
[1] https://github.com/ManageIQ/manageiq/blob/master/spec/models/mixins/authentication_mixin_spec.rb#L59
[2] https://github.com/ManageIQ/manageiq/pull/10690/files#diff-5fa47523d0443aa7bedb8f5b6a19ef41L23
[3] https://github.com/ManageIQ/manageiq/pull/10690/files#diff-c55d10da4f0a175082d58a2a06b39aedL188


Users want to associate tokens to their account

Problem
There is request to add the token associate functionality to the explorer.
As a Hedera account owner, I want to associate a token using one of the supported wallets.
As a Hedera account owner, I want to dissociate a token using one of the supported wallets.
This can be a MetaMask, Blade, or Hashpack wallet.
Solution
Add support for token associations.
Alternatives
No response

A proposed implementation is currently deployed on the DevOps staging server.

Once a wallet (any wallet) a connected, a button ASSOCIATE TOKEN or DISSOCIATE TOKEN is displayed:

When connected to MetaMask, the confirmation dialog allows to choose whether to also import (watch) the token:

New HIP is in progress

This has been addressed by PR #898


Add citation for Q2 plugin!

Just copied the bibtex from Google Scholar, so feel free to change that if it's incorrect.

Thank you for remembering this!


AudioReader.stop_threads

There're two issues with this method:

The definition is incorrect, it lacks a self argument, it should be:    def stop_threads(self):

The implementation is problematic, since Python thread object doesn't have a stop() method, there's no official way to stop a thread in Python.

Seems to me we should remove this method all together...

Agreed!
We should just remove it.

Yep! I am recieving this message after training.
"Exception in thread Thread-25:
Traceback (most recent call last):
File "/usr/lib/python2.7/threading.py", line 801, in __bootstrap_inner
self.run()
File "/usr/lib/python2.7/threading.py", line 754, in run
self.__target(*self.__args, **self.__kwargs)
File "/home/r2d2/projects/tensorflow-wavenet/wavenet/audio_reader.py", line 87, in thread_main
self.stop_threads()
TypeError: stop_threads() takes no arguments (1 given)"


Broken link

The link "Get started with Mbed OS" in "Usage" section is broken.
(https://os.mbed.com/docs/latest/tutorials/your-first-program.html)

ARM Internal Ref: MBOTRIAGE-1593

Thanks for the catch! On it.


FileShareQuotaAutoIncrease - Will not deploy on US Gov Cloud

What happened? Provide a clear and concise description of the bug, including deployment details.
Appears to be an API mismatch error and will not pass validation prior to deployment.
{"code":"InvalidDeployment","message":"Support for symbolic name resources requires a minimum API version of '2020-09-01', but API version '2020-06-01' was supplied."}
Please provide the correlation id associated with your error or bug.
n/a
What was the expected outcome?
Deployment
Relevant log output
{"code":"InvalidDeployment","message":"Support for symbolic name resources requires a minimum API version of '2020-09-01', but API version '2020-06-01' was supplied."}

@JCoreMS thanks for your feedback, we are reviewing it.

@JCoreMS we just pushed a fix for the portal UI deployment initiator, please try again.


Uncaught ReferenceError: M is not defined [1.0.0-alpha3]

I have included only
<script src="<?php echo get_stylesheet_directory_uri() . '/js/anime.min.js';?>"></script>
<script src="<?php echo get_stylesheet_directory_uri() . '/js/cash.js';?>"></script>
<script src="<?php echo get_stylesheet_directory_uri() . '/js/collapsible.js';?>"></script>

In my pages, as I don't want all the components of Materialize for this website.
Notes in anime.min.js say that M was removed, however, I get the error message
Uncaught ReferenceError: M is not defined
    at anime.min.js:11
    at anime.min.js:11
collapsible.js:267 Uncaught ReferenceError: anime is not defined
    at collapsible.js:267

Is this my own misunderstanding, or an error?

I know this was closed.  But I'm using 1.0.0-beta and seeing the same error after I added Sidenav to my page.
The only difference is I imported the full js via webpack.

oh, why this nice framework miss the great feature - importing selected modules .. ?


Layouts (Stage 2): Print inferred layout more consistently on locally-abstract types

Fix a bug where layout would be printed the first time the user queried an abstract type, but only the second time that the user queried a locally-abstract type. This is an omission in flambda-backend, so I'm posting a corresponding PR there: https://github.com/ocaml-flambda/flambda-backend/pull/2115. We need not merge this merlin-jst PR; we can just wait until we pick up the flambda-backend change in the normal merge process.

Old approach that we're no longer taking (we never show layouts when the user asks for the type of a value anymore):

~Previously, when you asked for the type of a value (the second time), merlin wouldn't print a type declaration if the type was abstract. E.g. asking for the type of 4 will only ever print int, not type int. But this means that there isn't a sensible places to hang a layout annotation.~
~Now, merlin always prints a type declaration for Tconstrs when you ask for the type of a value the second time.~
~This is a bit weird for locally-abstract types, where we print type a : immediate. We could print a : immediate instead, but I think we should be consistent. That is, we should either:~

~print int : immediate and a : value~
~print type int : immediate and type a : value~

~I prefer the second because it's more uniform with how we handle non-abstract Tconstrs.~

Closing as ocaml-flambda/flambda-backend#2115 was merged.


Unparse Map plan to SQL string

Is your feature request related to a problem or challenge?
It's similar to https://github.com/apache/datafusion/issues/13477. We use the scalar function to handle the Map type but we should unparse it back to the right SQL.
Map Literal
The SQL for creating a MAP:
select MAP {'a': 1, 'b': 2}

The plan:
  Plan: Projection: map(make_array(Utf8("a"), Utf8("b")), make_array(Int64(1), Int64(2)))
EmptyRelation

Unparse the plan:
SELECT map(make_array('a', 'b'), make_array(1, 2))

Access a MAP
The SQL for accessing a MAP:
select map_col['a'] from (select MAP {'a': 1, 'b': 2} as map_col)

The plan:
Projection: get_field(map_col, Utf8("a"))
  Projection: map(make_array(Utf8("a"), Utf8("b")), make_array(Int64(1), Int64(2))) AS map_col
    EmptyRelation

Unparse the plan
SELECT get_field(map_col, 'a') FROM (SELECT map(make_array('a', 'b'), make_array(1, 2)) AS map_col)

Describe the solution you'd like
The SQL should be able to roundtrip for SQL-Plan-SQL.

The map with make_array function should be unparsed to ast::Expr::Map
The get_fields for MAP should be unparsed to ast::Expr::MapAccess

Describe alternatives you've considered
No response
Additional context
No response

I implemented it for map in #13532. I'm unsure about get_field because I'm not able to tell if it's a map column from the Expr

I implemented it for map in #13532. I'm unsure about get_field because I'm not able to tell if it's a map column from the Expr

I see. I think we can pass the plan schema when unparsing the get_field. If it's a map, we can unparse it to MapAccess with MapAccessSyntax::Bracket.


[$500] Web, Desktop - Message actions - Limited action items displayed on reply in thread hover

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.11-6
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Open the app
Open any report
Send any message
Hover and click on reply in thread on message sent in step 3
Keep the hover on reply in thread message, observe that for sometime, it displays correct action item before changing to only 2 items
Go back to parent report
Again hover and click on reply in thread on same message as used in step 3 and 4
Observe that now it displays all the action items

Expected Result:
App should display all available action items on reply in thread on any message even on first time click on reply in thread and hover on the message
Actual Result:
App displays limited action items on hover on reply in thread message after first time click on reply in thread button on new message
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/fb2af1e4-a0cc-4fa9-bc29-6d31821b710d
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01471e600d6f5d0940
Upwork Job ID:<PHONE_NUMBER>131348992
Last Price Increase: 2023-12-13

Yes, this can be closed. @kevinksullivan


Controller disconnect after running a macro

Hello!
On the webapp (windows), always after finishing a macro the controller disconnects and I need to press the "Recreate Controller" button in order to run a new macro.
This is the error I receive on the terminal:
<IP_ADDRESS> - - [09/Dec/2021 15:36:49] "POST /socket.io/?EIO=4&transport=polling&t=NsVrmS1&sid=dVb7FWinR3o8ydzuAACI HTTP/1.1" 200 219 0.001366 post request handler error Traceback (most recent call last):   File "/usr/local/lib/python3.8/dist-packages/engineio/server.py", line 431, in handle_request     socket.handle_post_request(environ)   File "/usr/local/lib/python3.8/dist-packages/engineio/socket.py", line 124, in handle_post_request     p = payload.Payload(encoded_payload=body)   File "/usr/local/lib/python3.8/dist-packages/engineio/payload.py", line 13, in __init__     self.decode(encoded_payload)   File "/usr/local/lib/python3.8/dist-packages/engineio/payload.py", line 44, in decode     raise ValueError('Too many packets in payload') ValueError: Too many packets in payload
Is there a way I can prevent it to disconnect or a way to autmatically reconnect the controller?

It seems to disconnect here too, but seems like there is no error:
Create Controller
<IP_ADDRESS> - - [17/Dec/2021 20:50:56] "GET /socket.io/?EIO=4&transport=polling&t=NtAANWw HTTP/1.1" 200 278 0.004319
<IP_ADDRESS> - - [17/Dec/2021 20:50:56] "GET /socket.io/?EIO=4&transport=polling&t=NtAAHvX HTTP/1.1" 200 278 0.004234<IP_ADDRESS> - - [17/Dec/2021 20:50:56] "POST /socket.io/?EIO=4&transport=polling&t=NtAAOGB&sid=wfBH8qzXEb814a3bAAAX HTTP/1.1" 200 219 0.003689
(1400) accepted ('<IP_ADDRESS>', 54947)
<IP_ADDRESS> - - [17/Dec/2021 20:50:56] "GET /socket.io/?EIO=4&transport=polling&t=NtAAOGD&sid=wfBH8qzXEb814a3bAAAX HTTP/1.1" 200 213 0.000311
<IP_ADDRESS> - - [17/Dec/2021 20:50:56] "GET /socket.io/?EIO=4&transport=polling&t=NtAAOGU&sid=wfBH8qzXEb814a3bAAAX HTTP/1.1" 200 181 0.000180
<IP_ADDRESS> - - [17/Dec/2021 20:50:56] "POST /socket.io/?EIO=4&transport=polling&t=NtAAOGT&sid=wfBH8qzXEb814a3bAAAX HTTP/1.1" 200 219 0.014735
Disconnected
<IP_ADDRESS> - - [17/Dec/2021 20:50:56] "GET /socket.io/?EIO=4&transport=websocket&sid=YzMa6J338Twl50JmAAAV HTTP/1.1" 200 0 252.563556

Until this gets actually fixed, you could try something like this.
let statusIndicator = document.getElementById('status-indicator-text');
function watchForDisconnect() {
  let intervalId = window.setInterval(() => {
    if(statusIndicator.innerText !== 'CONNECTED') {
      restartController();
      watchForReconnect();
      window.clearInterval(intervalId);
    }, 100);
}
function watchForReconnect() {
  let intervalId = window.setInterval(() => {
    if(statusIndicator.innerText === 'CONNECTED') {
      watchForDisconnect();
      window.clearInterval(intervalId);
    }
  }, 100);
}

Or you can try what I'm currently doing.
let statusIndicator = document.getElementById('status-indicator-text');
function restartAndRunMacro() {
	restartController();
	let intervalId = window.setInterval(() => {
		if(statusIndicator.innerText === 'CONNECTED') {
			window.clearInterval(intervalId);
			sendMacro();
		}
	}, 100)
}

Less Techy People:
If you don't know how to run those, just F12 and go to the console tab and paste the code. Also, don't run code random people on the internet give you unless you know what the code does. I'm not going to explain this code to you. If you don't understand it, you shouldn't run it, even if I tell you exactly what it does.


Add ability to specify docker image in KafkaContainer.

This code allows to use private docker registry for testing purposes.

Hi @irasikhin,
What do you think if instead we move confluentinc/cp-kafka to TestcontainersConfiguration, so that it can be configred with properties?
I really don't want to expose a public "any image" constructor because our KafkaContainer implementation expects image to be Confluent-like and that will be confusing if somebody will try to use some other image.
But I get the point about private registries, and because of them we moved our list of images we use to the TestcontainersConfiguration class

ok, done

@irasikhin I would still keep the ability to specify a version :)

@irasikhin perfect, thank you!

@bsideup thank you too =)


Unable to enter use both underline and strikeThrough formats

1.Click on Text Editor (editor view).
2.Enter some text
3.Select the underline format
4.Enter some text
5.select stricken format
6.Enter some text
In above scenario, Underline automatically disabled.

@swayam007 I have added this to my to do list, will hopefully have a fix pushed soon.

how to resolve this bug?

How can I solve this problem at last


Is this only for coaches?

I'd like to write a frontend to track my progress for myself, but I'm not a coach/teacher.

I have not looked into all the details, but I am sure that Khan Academy is calling the graphql endpoint to get your data. So you will just have to monitor and implement the graphql calls as detailed in https://github.com/jb-1980/khan-graphql/blob/master/CONTRIBUTING.md.
Note that you can create the frontend to display the data, but you will have to create a backend node service to pull the data. This script won't work in the browser.
Here is a codesandbox to give you an idea: https://codesandbox.io/s/basic-khan-graphql-server-cjvp6?file=/src/index.js

Yup, I can whip up a Node server. Was just checking if the endpoints/resolvers/etc would be restricted to Authorization: Bearer SOME_TOKEN_GENERATED_FOR_COACHES_ONLY.
Thank you!


add configuration for allowed_requrest_origins to readme

I am getting Request origin not allowed: http://localhost:3000 errors since #85 was merged.
This adds example config to allow requests to be made that pass this protection.
NOTE: I tried putting this into the config/environments/*.rb files but it caused issues with the server.logger being nil. I think that we need a actioncable/railtie to manage configuring the server at the correct point of the application load process. Would there be interest in a railtie or should that wait until actionable is being merged into rails.

I'm 👍 on adding a Railtie. Then we should just allow localhost in development :grin:

@lsylvester for now, what's a good work around for this developing locally? tried all sorts of CORS things, but not having any luck.

@dam13n have you tried what @lsylvester has added to the README? :grin:

haha, my bad! i forgot i was reading a PR and not an issue. thanks @kaspth

@dam13n No problem 👍

Continued in #93.


Arduino, only one partition scheme

Variants are added by other customers, if you want more options, please use the ESP32 S3 Dev Module, as described in the README


scribbleable mask

Description of the new feature / enhancement
to be used in case of screen presentations, where the presenter would "scribble on the screen" as if a glass apparatus were in front of the screen
Scenario when this would be used?
presentations or for self study
Supporting information
it would be as if the computer screen were covered with a transparent screen allowing the user to write with a digital pen, providing elements for a better understanding of the people who are watching the presenter's explanation.
This functionality would bring a new dimension to real-time application presentations, where the screens are interactive and do not allow markings, for example, a presentation of a SAP software screen with real data presented in the presenter's explanation

check out ZoomIt.  https://learn.microsoft.com/en-us/sysinternals/downloads/zoomit


Zsh installation instructions without Oh My Zsh

Proposal
Please add installation instructions for Zsh without using  Oh My Zsh
Use-cases
People using Zsh without  Oh My Zsh
Attempted Solutions
Pasting shellfirm.plugin.zsh to .zshrc

Hey @igorkulman, thanks for this issue!
Can you elaborate a bit how you manage your plugins in  Zsh?

Just source x.zsh, all the "plugins" allow for such installation, here is an example https://github.com/zsh-users/zsh-autosuggestions/blob/master/INSTALL.md#manual-git-clone

+1, personally this is what I use:
[ -f $HOME/repos/oth/shellfirm/shell-plugins/shellfirm.plugin.zsh ] && source $HOME/repos/oth/shellfirm/shell-plugins/shellfirm.plugin.zsh

Hey @igorkulman
I'll upload a detailed doc + docker image later this week, but it the meantime you load shellfirm adding the following commands in your ~/.zshrc file
exec-shellfirm () {
  if [[ "${BUFFER}" == *"shellfirm pre-command"* ]]; then
        return
    fi
    shellfirm pre-command --command "${BUFFER}"
  zle .accept-line
}
zle -N accept-line exec-shellfirm

let me know if it helped

Hey @igorkulman,
I have added instructor here: https://github.com/kaplanelad/shellfirm#select-your-shell
Also, prepare a docker image
https://github.com/kaplanelad/shellfirm/tree/main/docs/docker/zsh

Thanks


view STL files

Description
feature request: view, see, and track changes (visually) with .stl and other 3d files. (like github does)
Thanks!
I find this an amazing feature that github has and it would be even more amazing if gitea could implement similar functionality.

Maybe you can do it via third-party tools and add some config on app.ini

If anyone has had success with third-party tools let me know. I would gladly try it out but I have no idea where to even start.
Thanks.

At first, are there any command line tools or scripts could convert the files to HTML files?

How do you run gitea ? on windows, linux, ... ? Do you use the docker image ?

I have a script that can render .scad and .stl file but it will need openscad that may not be easily installable depending of your installation https://gist.github.com/sapk/ce1c246dd39a5f434554b31d0ca8ecdb

@sapk I am running in a freebsd jail installed with ports.
I will take a look at that script but it seems like a static render...right?
github does this: https://imgur.com/a/eZxCIHI which is amazing. It also highlights the parts of the STLs when you look at the commit diffs...

We could use something like this: https://github.com/kaitlynhova/Three.js-STL-Viewer

@lafriks there is various JS lib mostly all not maintained. The more advanced I found is https://github.com/kovacsv/Online3DViewer I feel like we should not support it but allow the use of direct HTML (no sanitize) via external render if requested by option (and add libs under public).

Would it be better to still in general sanitise but allow an external renderer to add some standard page sections?

at Github they use Three.js for the STL viewer.
We could indeed use https://github.com/kovacsv/Online3DViewer it seems to be the more active open-source STL online viewer.
The process for the diff is explained here: https://github.blog/2013-09-17-3d-file-diffs/ Not a 3D expert, but it doesn't look that complex no? (but CPU/GPU consuming for the gitea server).

gitea added this feature in documentation. i integrated madeleine.js (three.js wrapper) into our environent succesfully. i had to adjust some things to make it render with complete width on smarthpones and computers (tested with firefox and chrome)

https://gitea.fablabchemnitz.de/MarioVoigt/Madeleine.js-for-Gitea
happy for more fixes to make it clean.
the docs for integration are provided here: https://docs.gitea.io/en-us/customizing-gitea/

FWIW an alternative STL previewer based rendering directly with three.js has been implemented in https://codeberg.org/Codeberg/build-deploy-gitea/pulls/31 -- we believe controls are a bit more intuitive (especially on mobile, also no gimbal lock in rotation; trackball-style control). Implementation is a single JS file, but threejs is added as dependency.
Try and visit: https://codeberg.org/Codeberg/three.js/src/branch/dev/examples/models/stl/binary/colored.stl

Hey,
i documented it for myself some weeks ago the following (hopefully correct):
cd /opt/gitea/custom/templates/
mkdir base/
cd base/
vim /opt/gitea/custom/templates/base/footer.tmpl

<script>
function lS(src){
  return new Promise(function(resolve, reject) {
    let s = document.createElement('script')
    s.src = src
    s.addEventListener('load', () => {
      resolve()
    })
    document.body.appendChild(s)
  });
}

if($('.view-raw>a[href$=".stl" i]').length){
  $('body').append('<link href="/Madeleine.js/src/css/Madeleine.css" rel="stylesheet">');
  Promise.all([lS("/Madeleine.js/src/lib/stats.js"),lS("/Madeleine.js/src/lib/detector.js"), lS("/Madeleine.js/src/lib/three.min.js"), lS("/Madeleine.js/src/Madeleine.js")]).then(function() {
    $('.view-raw').attr('id', 'view-raw').attr('style', 'padding: 0;margin-bottom: -10px;');
    new Madeleine({
      target: 'view-raw',
      data: $('.view-raw>a[href$=".stl" i]').attr('href'),
      path: '/Madeleine.js/src'
    });
    $('.view-raw>a[href$=".stl"]').remove()
  });
}
</script>

chown git:git footer.tmpl
chmod 770 footer.tmpl

cd /opt/gitea/custom/
mkdir public/
cd public/
git clone https://github.com/JinJunho/Madeleine.js.git Madeleine.js/
chown -R git:git /opt/gitea/custom/public

service gitea restart

Check your browser's console output for possible css/js/whatever errors :-)

I will close this issue as it can be setup by many ways and at least one is in docs.
Feel free to comment or open an new issue for more specific case or come on the discord to discuss it.


Server initialization failed

Starting Microsoft Python language server.
[Info  - 3:33:04 PM] Created Microsoft.PythonTools.Interpreter.Ast.AstPythonInterpreter instance from Microsoft.PythonTools.Interpreter.Ast.AstPythonInterpreterFactory
[Error - 3:33:05 PM] Server initialization failed.
Message: Object reference not set to an instance of an object.
Code: -32000
[object Object]

I have the same issue. What do you need to see the context? Maybe debug info or anything?
I'm using Code 1.26.1, python 2.7
Interesting detail: if I open new editor instance (Ctrl + Shift + N) and open folder with few simple python files, select interpreter Python 3.5 - it works.
It doesn't work only in my current huge Django project written in python 2.

Probably it's related to issue in Python Language server, not to the intellicode itself:
https://github.com/Microsoft/vscode-python/issues/2017

@levin-ilya, we think this is  https://github.com/Microsoft/vscode-python/issues/2017 per @dpetukhov 's comment above. Can you resolve if you agree?

Yup - I guess I’ll try installing the developer version...or just wait for the next release. Thanks!


@POST with @Body doesn't work using @RetrofitClient with WebClient

Describe the bug
Please provide details of the problem, including the version of Spring Cloud that you
are using.
Hi guys, @OlgaMaciaszek
I am trying to use @RetrofitClient with WebClient using the @Body annotation for the body, but it is not working.
I am trying with the following Retrofit Client:
@RetrofitClient(name = "HelloServer", url = "http://localhost:7111", configuration = HelloClientConfiguration.class)
public interface HelloClient {

    @GET("/")
    Mono<String> hello();

    @Headers({"Content-Type: application/json"})
    @POST("/card")
    Mono<Card> create(@Body Card card);
}

I have the following questions:
I would like to know if you have implemented the CallAdapterFactory or ConverterFactory so that the @Body works serializing the object to be set as the request body? We are using the version 0.4.0-M1 of spring-cloud-square-retrofit and the serializing of the Body is not working.
We would like to know if you know about this and how We can do to solve this problem: maybe adding a specific configuration or  waiting for the next version of spring-cloud-square?
I am looking forward to hearing from you!
Important: If you need help regarding to this issue, please let me know how I can help you.
Best Regards!
Sample
If possible, please provide a test case or sample application that reproduces
the problem. This makes it much easier for us to diagnose the problem and to verify that
we have fixed it.

Hi, any update about this issue? we are facing the same error.

Hi, @dante-ricalde @jarpz it works with our tests and we cannot reproduce the issue: https://github.com/spring-cloud-incubator/spring-cloud-square/blob/05b1c9bbcb89ffc689d7ae04cb994d9ce0765cf7/spring-cloud-square-retrofit/src/test/java/org/springframework/cloud/square/retrofit/RetrofitClientUrlTests.java#L114-L122. I can verify it further, but I would need you to provide a minimal, complete, verifiable example that reproduces the issue.

Hi @OlgaMaciaszek I opened a PR https://github.com/spring-cloud-incubator/spring-cloud-square/pull/28
With a test with WebClient as HTTP client where payloads (RequestBody) are not sent.
I can see in your test that you are using OkHttpClient.
Did you check with spring webclient?
Regards.

Thanks, will take a look at this.

The bug is there. Will fix it.


Merge origin/master as of today

Merge origin/master into pg_upgrade to keep it current.
New attempt at creating this PR, the previous attempt opened it against origin/master after outsmarting me in the GUI..

You're a collaborator, feel free to push.

Yeah, mainly opened the PR for visibility, will push later.


fix_optax_wrapper

The optax wrapper was not handling value_and_grad properly anymore due to the check of dtype in value (precisely at value=jnp.asarray(jnp.inf, value.dtype) where value was a tuple when value_and_grad=True) .
Here is a MWE. The fix is simple, I can add a test and review all similar calls of dtype of value if you want. (I needed this fix for a personal project but I can workaround).
import jax
from jax import numpy as jnp
from jaxopt import OptaxSolver
import optax

def fun(x):
  return 0.5*x**2
val_and_grad_fun = jax.value_and_grad(fun)
solver = OptaxSolver(val_and_grad_fun, opt=optax.sgd(learning_rate=0.1), value_and_grad=True)
solver.run(jnp.asarray(1.))
# AttributeError: 'tuple' object has no attribute 'dtype'

Thanks. A non-regression test would indeed be great to ensure that we don't reintroduce the bug silently in the future.


Add WhatsApp Messaging Functionality

Purpose
To add the capability of sending WhatsApp messages
Approach
This PR extends the Ballerina Twilio Connector's capabilities by enabling to send WhatsApp messages using predefined message templates.

@miyurud This commit messages in this pr does not adhere to our guidelines. Refer https://github.com/ballerina-platform/ballerina-lang/blob/master/CONTRIBUTING.md for details.
We follow https://chris.beams.io/posts/git-commit/ for commit messages and PR topics.
 Refer https://github.com/ballerina-platform/ballerina-lang/blob/master/CONTRIBUTING.md for details.
We follow https://chris.beams.io/posts/git-commit/ for commit messages and PR topics.
 Then we can get rid of these incorrect commit messages.

@miyurud Shall we do the git squash since this is a new addition? Then we can get rid of these incorrect commit messages.

Based on the offline discussion I had with @wggihan we will go ahead without using git squash.


softcoded menus

What feature do you want to get added on the base engine?
the title says it all!
To test your sight, and reliability, please select the option of what should NOT be requested.
the correct option of what should not be requested, plus i know i failed this test because this is the wrong answer, the correct answer is supposed to be Stage Editor, 6K+ Support, and winning icons.

maybe i will do that for my pr

You people just love clowning on yourselves huh


pass package_name when looking up versioned model node during parsing

resolves #
docs dbt-labs/docs.getdbt.com/#

Problem

Solution

Checklist

[ ] I have read the contributing guide and understand what's expected of me
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

closing in favor of: https://github.com/dbt-labs/dbt-core/pull/8488


Segfault with IPV6 6rd enabled

This is a strange problem; apologies that the step to reproduces aren't super straightforward:
It appears that if I'm behind a router (Cisco EA3500) and leave the Internet Settings / IPV6 / IPV6 6rd to "Automatic", all request to the datastore will result in a segfault

The sensor.model:createOne method (hangs at ds.save/datastore.save):
 createOne: o => {
    return sensorEntity.new(o).then(e => ds.save(e))
  },

The sensor.model:createOne test:
  test('sensor.model:createOne', t => {
    return sensorModel.createOne(entityPayload)
      .then(d => {
        const id = extract(d, 'id')
        t.equal(id.length, 16, 'id has 16 digits')
        t.equal(typeof parseInt(id), 'number', 'id parse to number')
        entitiesIds.push(id)
      })
      .then(() => t.end())
  })

I'm trying to understand why there is no error displayed and why it only seems to affect the datastore (so far). Maybe someone has an idea on how to replicate this environment in a Docker image or something?
Environment details

OS: 4.14.4-1-ARCH
Node.js version: v8.9.1
npm version: 5.5.1
yarn version: 1.3.2
@google-cloud/datastore version:

Steps to reproduce

Enable 6RD on a router
Try to perform a datastore operation

This issue was moved to grpc/grpc-node#137


add default breadcrump template for twitter bootstrap

Do you like to have a PR for support of bootstrap template so that users are able to simple enable it via config?
See e.g. http://getbootstrap.com/components/#breadcrumbs
# app/config/config.yml
thormeier_breadcrumb:
    template: 'breadcrumbs_bootstrap.html.twig'

{# vendor/thormeier/breadcrumb-bundle/Resources/views/breadcrumbs_bootstrap.html.twig #}
{%- if breadcrumbs|length -%}

        {%- for crumb in breadcrumbs -%}
            {%- set crumbText = crumb.label|replace('%%', '%')|trans(crumb.labelParameters) -%}
            {%- if crumb.route == app.request.get('_route') -%}

                    {{- crumbText -}}

            {%- else -%}
                {%- set crumbPath = path(crumb.route, crumb.routeParameters) -%}

                    {{ crumbText }}

            {%- endif -%}
        {%- endfor -%}

{% endif -%}

Another option would be to provide default implementations for schema markup.
I can open one or more issues for Microdata, RDFa and JSON-LD if you mind.
See http://schema.org/breadcrumb and http://schema.org/breadcrumbList for implementation details.

I think this would be a great addition to this bundle, since bootstrap was added as a form template since SF2.6 already. If you'd like to contribute, feel free to open issues and PRs.

Added such a template and usage exmaple in README in https://github.com/thormeier/breadcrumb-bundle/tree/2.2.0


Add coverage.eDate and coverage.sDate to the list of “can’t be used or edited”

Mark wants to try the following on the test instance of the Edit system:

Basically, we keep one qualifier for date (digitized) from being editable in the edit interface here - https://github.com/unt-libraries/pyuntl/blob/master/pyuntl/form_logic.py#L523
I want to try to add coverage.eDate and coverage.sDate to the list of “can’t be used or edited”
Because we are trying to remove those values from records with a preference for coverage.date and an EDTF range instead

Here is what records with a date.digitized value look like in the system - https://edit.digital.library.unt.edu/?fq=dc_date.digitized_count%3A1
If this works correctly, a similar thing will happen for the coverage.eDate and coverage.sDate records.

Here are some records with an eDate - https://edit.digital.library.unt.edu/?fq=dc_coverage.eDate_count%3A1
Here are some records with a sDate - https://edit.digital.library.unt.edu/?fq=dc_coverage.sDate_count%3A1


Fixed inverted index seal: Skip to seal the columns which had no inve…

If the configuration for inverted index is enabled, all the columns have to seal there inverted index. But not every column has inverted index. So, we need to check this in the seal() method.

I think this is fixed, please reopen if not


Support for user_vars.user_key

Right now my auth user_id is the users email, we all know that email addresses can change and I don't want to update all of the relevant tables when this happens, I'd rather use the key value.
Is it possible to expose the the key value from auth as a user_var?

Reviewing the Security wiki page, it shows using a trigger to change the foreign key with the user id on the Authors table. Is this the best practice? Do I create triggers for all my tables to ensure that only the logged in user can view them?

After further working through the Security wiki page, and putting together an API, this request no longer makes sense.


Feature Request: only trait - opposite if disabled

Description
Using swift-testing with a mid October toolchain, Xcode 15.0.1 and Sonoma, I do not see any test “diamonds” in Xcode.
Thus I cannot start a specific test, and run only that test. Annoying!
The disabled trait exist, and I think we should add the reverse - an only trait.
Jest has this
What should happen if one accidentally declares more than one only? Compilation error would be good!
Expected behavior
To be able to run a specific test.
Actual behavior
Have to run all test now.
Steps to reproduce
No response
swift-testing version/commit hash
f8e8961b48d53d3272dacd7431257ed87260b3a1
Swift & OS version (output of swift --version && uname -a)
swift-driver version: 1.87.1 Apple Swift version 5.9 (swiftlang-<IP_ADDRESS>.108 clang-15<IP_ADDRESS>)
Target: arm64-apple-macosx14.0
Darwin Alexanders-MacBook-Pro.local 23.1.0 Darwin Kernel Version 23.1.0: Tue Oct  3 22:40:26 PDT 2023; root:xnu-10002.41.8~15/RELEASE_ARM64_T6020 arm64

Regarding the underlying issue of missing "diamond" button in Xcode 15, seems more users have experienced this: https://forums.swift.org/t/missing-test-diamonds-in-xcode/62543/11
But the only feature is still good! Since it allows running single test from CLI or other IDEs than Xcode.

Traits affect specific tests and, in the case of test suites, any tests they contain. It is intentional that they do not have the power to affect other tests, because that would make computing test plans intractably complex.
During macro expansion, the macro plugin cannot "see" symbols beyond the one being expanded upon, so it is not possible to detect that two tests both have a .only trait. Additionally, type information is not available during macro expansion, so it is trivally easy to accidentally escape the diagnostic logic (e.g. by using a trait declared as let myOnly = OnlyTrait.only or similar, which the macro plugin wouldn't know represented the same trait type.)
Technical constraints aside, the general utility of a .only trait seems limited. If you are writing many tests and only want to run a single one, that's something that (I think!) would be typically modelled through tooling, not in the source of the test suite itself. If and when we're able to augment swift test --filter to understand swift-testing, I think we'd suggest you use it to run individual tests. Does that sound reasonable?


Provision a MacOS box for running unit and e2e tests

Is your feature request related to a problem? Please describe.
IIUC, the current actions workflow only runs on Ubuntu. Certain jobs such as the e2e job may behave differently between MacOS and Linux and this could lead to us missing breaking changes.
Describe the solution you'd like
Provision a box running MacOS that we can run unit and e2e tests on. Hopefully it wouldn't be too much more costly since it doesn't need to run all actions. Certain actions like lint and format as they shouldn't differ between systems.

I think that the default macos runner should be good enough to test training on MPS / METAL. Let's give it a try.

Looks like GHA does not support MPS. https://github.com/actions/runner-images/issues/9254 suggests it's a limitation in Apple's virtualization:

it is not a GitHub restriction, Apple does not support nested virt on M1 CPU gen entirely

ggml_metal_graph_compute_block_invoke: error: unsupported op 'RMS_NORM'
GGML_ASSERT: /private/var/folders/3m/p59k4qdj0f17st0gn2cmj3640000gn/T/pip-install-abmja2gi/llama-cpp-python_94634b10a2fc48d9acb183bde5c3d2c4/vendor/llama.cpp/ggml-metal.m:828: !"unsupported op"

I think this is still worth having, though it may be a while longer before we can implement

We use EC2 for e2e tests. Can we set up one for a mac2-m2.metal instance?

closing in favor of #1638


Revert "Improve RequestAsync type"

Reverts JohnnyMorganz/luau-lsp#121

Looks like this wasn't the issue, it was #118


[FEAT] Copy and Paste Elements

A user should be able to select an element either on the canvas or in a layers panel, and copy the element, then paste onto a canvas or into a selected parent layer (in the layers panel)

Noting this was requested by a user on a call today.


"fcose" layout not in the list of possible layouts.

Description
Cytospace.js has recently announced about a new type of CoSE family, called fcose. I would like to have fcose in the list of dash-cytospace layouts.

@henriksergoyan Great news, it'd be a great addition to Dash! If you would like to contribute that new feature by starting a PR, we'd be happy to review it :)
See this PR for a reference: https://github.com/plotly/dash-cytoscape/pull/88

The updates above indicate that fcose was added to the library. But I can't access it in my program. Can anyone explain how this works? I'm not too familiar with the development process so apologies in advance if I'm wrong in assuming that the fix above, after merging, should allow me to access fcose


Add Dependabot for automated updates

This PR adds the config file .dependabot.yml to the repository to enable automated dependency updates using Dependabot.
Dependabot is a widely used tool that automates the process of keeping the dependencies up-to-date, and regularly checks for updates to the project's dependencies to ensure that it's always using the latest and most secure versions. This reduces the risk of security vulnerabilities and outdated dependencies.
Dependabot will automatically check for updates to the following and make a pull request if theres a new version available:

Github Actions: Check for updates to the action's configurations (daily).
NPM dependencies: Check for updates to the packages (daily).
Docker images: Check for updates to the Docker images (weekly).

Hope you will find this useful going forward.
Tom

I know Dependabot is GitHub owned, but Renovate is far superior in this area and also open source. 🤷

You are correct @m-torin, Renovate is another popular useful tool for accomplishing the same goal.
I think ultimately its slightly down to personal choice, but either would be an excellent addition to this project for keeping project dependencies up to date, which is important for security, performance, and compatibility reasons.
However, it might now in hindsight be possibly better to go with Renovate as its more configurable if you feel the project might get more complex in future. Here’s a very quick Hands On Tutorial on setting it up which requires you to configure it @mckaywrigley.
Obviously feel free to close this PR if you do go down the Renovate route.
Thanks
Tom


Encodian March 21 Release

Please check the following conditions for your PR.

[ ] apiDefinition.swagger.json is validated using paconn validate command.
[ ] apiProperties.json has a valid brand color. Invalid brand colors are #007ee5 and #ffffff.

@Encodian Please remove other connector's files from your PR to get approved and merged. You can create a new fork from dev branch, and add your connector's latest files to create a new PR. Please note that your next update will be blocked if you have not open sourced your connector. Thanks


Standardize name conventions for qibo-client

There are some discrepancies between the qibo-client naming and this package, e.g. device vs platform.

Yeah, also the AWS client (not in main yet) uses the device keyword, we might change platform to device and make it a standard.


Updates related to removal of UWP, Xamarin, NET6-Win support in 4.61.0

Fixes #395

this needs an update
https://learn.microsoft.com/en-us/entra/msal/dotnet/acquiring-tokens/desktop-mobile/wam#enabling-wam
remove this If your application targets UWP or net-windows (version-dependent Target Framework Moniker for Windows), WAM is included in the MSAL.NET package.

@gladjohn Yes, I specifically left it to not confuse folks on older versions. I added a comment that it's applicable to previous versions.

If your application targets UWP or net-windows (version-dependent Target Framework Moniker for Windows), WAM is included in the MSAL.NET package (applicable to versions 4.60.3 and below).


Allow Symfony 5

Added Symfony 5 as accepted version.
Worked for me on a 5.1 project.

Thank you @damienalexandre

Released https://github.com/maks-rafalko/tactician-domain-events-bundle/releases/tag/0.5.6

Thanks!


Feature request: ability to select output

Right now Morgan works with process.stdout.write, but these outputs are not displayed in Google Cloud functions logs, only with console.log; therefore, it would be a great addition to select which output mechanism to use, and let process.stdout.write to be the default.

Just pass a different stream you would like the logs written to by using the stream option:
https://github.com/expressjs/morgan/blob/master/README.md#stream


Remove heating =  None

Adds heating=None for all devices, was not the intention.
Causes no issues in Plugwise-beta.

Just reverting?

Yes, version reverted.