[MB-6392] Replace MomentJS with DayJS

Description
This PR replaces our date/time parse & format library Moment.js with Day.js (due to the Moment.js project being deprecated).
In most cases this was a 1-1 swap out since Day.js largely uses the same API as Moment.js. For certain features (such as fromNow() and parsing from non-ISO formatted date strings), Day.js relies on being extended with plugins.
I did have to make one change where we use react-day-picker, because there was a slight discrepancy between how Day.js & Moment.js parse date strings. Since the DayPickerInput is always working with a parsed Date object for its value, the easiest way to reconcile this was to provide it with a formatDate function that always formats a date from ISO (instead of a custom format).
I also deleted any date/time formatter functions that were not being used or referenced anywhere. There's 💯  some more organization that could be done around all of the various date formatting code we have, but I limited the scope to avoid making this PR bigger than it needed to be.
Because Moment was listed under Cypress dependencies, there is a corresponding PR in our Cypress docker image repo: https://github.com/transcom/circleci-docker/pull/81
We were not actually using Moment in any Cypress code, so I removed it.
Reviewer Notes
Tests passing are a good sign, but you can also go into the app and verify anywhere dates are displayed or forms that include a date picker input are still functioning as expected (verify initial date values still work, and you are able to select a new date value). Forms that include date picker inputs are:

Orders Info (Customer onboarding)
Create/Edit MTO Shipment (Customer onboarding)
Orders Detail (Office)

Setup
To run the app locally:
make server_run
make client_run

Code Review Verification Steps

[ ] If the change is risky, it has been tested in experimental before merging.
[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.
Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Warnings

:warning:

Files located in legacy directories (src/shared or src/scenes) have
been edited. Are you sure you don’t want to also relocate them to the new file structure?
View the frontend file org ADR for more information

Messages

:book:
:link: MB-6392

  Generated by :no_entry_sign: dangerJS against 335aa7184ad2246501361eb9ec4e7596d04d9464

UPDATE: After investigating the integration test failures on this branch, I've deduced that these changes to our date parse/format functions are not working on date inputs that use the SwaggerField component (this includes the Customer PPM flow and Office PPM forms). SwaggerField is a legacy component that, due to being extremely reliant on configuration and itself used in 30+ forms to generate any kind of form input, I don't feel comfortable making significant edits to. It's also difficult (and would be tedious) to pinpoint all of the places where SwaggerField is used to generate date inputs, since this logic is driven by the Swagger API schema rather than declarative UI code.
Given that we are planning to replace usage of SwaggerField with more explicit forms built using Formik and ReactUSWDS components, it doesn't seem worth trying to debug and resolve a component that could have sweeping effects in places of the app that have relatively low test coverage. Given all of that, I am recommending we put this work on hold until after SwaggerField is deprecated. MomentJS has been deprecated, but there are no immediate known issues resulting from us using it, so I don't believe there is any risk to us delaying this work.

Closing this for the meantime since the way PPM Office uses date inputs is not working with DayJS


ci.yml: improve macos package installation

All brew install steps moved into one.
Install audio device for playing sound since some of macos-12 machines have no audio device installed. Is returning false, shouldn't be the case anymore.
libtool is skipped on macos-12 due already installed.
diffutils installed for macos-14

thanks, looks good now.

Hm no wait, don't we need diffutils always being installed? If I see that correctly, it is now only installed for non-macos 12 versions?

Hm no wait, don't we need diffutils always being installed? If I see that correctly, it is now only installed for non-macos 12 versions?

All suggestions are sort out, should be fine now.

Hm, honestly I am not sure this is an improvement.
If I compare this:
if [ $(uname -r | grep ^21 | cut -c1-2) == 21 ]; then

to
 if: matrix.features == 'huge' && matrix.runner == 'macos-12'

I find the latter to be easier to understand.
Also, even so we would now have a single install packages section I find it easy to understand if we split this up like it is currently:

install packages
Install audio device for playing sound
install packages for testing

So after all, I would just keep the current logic, sorry.


Feedback for “Sui Programmable Transaction Blocks Basics”

How to send non-sui tokens. I don't see instructions

Thanks for your suggestion @hz73219. We'll try to add that to the docs.
In the meantime, I imagine you can transfer an object just like transferring a SUI coin.
Have you tried some similar code and it did not work?
const obj_address = address_here;
txb.transferObjects([obj_address], '0xSomeSuiAddress');


Cookbook - iPhone physical device - sound switches to receiver speaker after viewing 'Channel/Device Routing' screen

macOS Version(s) Used to Build
macOS 12 Monterey
Xcode Version(s)
Xcode 14
Description
Download the Cookbook from Github.
Perform the personalization steps needed to run the workspace on a physical iPhone (not iPad, not simulator).
Select the Dynamic Oscillator screen and play a sound. Observe that sound is coming from the loudspeakers, not the receiver earpiece.
Return to the main menu and select the Channel/Device Routing menu item. You don't have to press anything.
Return to the main menu and select Dynamic Oscillator screen again.
Playing a sound now comes from the receiver speaker and won't change until you kill and restart the app.
What's odd is that this issue doesn't happen if you visit the Tuner screen (which also uses the mic). It only happens after you visit the Channel/Device screen. After viewing Channel/Device screen, all sounds are routed to the receiver speaker, not just the Oscillator screen.
Crash Logs, Screenshots or Other Attachments (if applicable)
No response

It probably has something to do with setting the AudioSessiontry Settings.setSession(category: .playAndRecord,                                     with: [.mixWithOthers, .allowBluetooth, .allowBluetoothA2DP]) It needs .defaultToSpeaker

That is how it is set in the app delegate which is only called when the app is launched. That line might be able be omitted entirely since the session is already setup.

Thanks, Nick.
I can edit that here and give it a try. I’m having problems with my own app, as well, using .defaultToSpeaker.
What worked great two years ago is now killing the mic input on iPhones.
Dave.

On Oct 20, 2022, at 1:37 PM, Nick Culbertson @.***> wrote:
It probably has something to do with setting the AudioSessiontry Settings.setSession(category: .playAndRecord, with: [.mixWithOthers, .allowBluetooth, .allowBluetoothA2DP]) It needs .defaultToSpeaker
—
Reply to this email directly, view it on GitHub https://github.com/AudioKit/AudioKit/issues/2790#issuecomment-1285981051, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALXQ4DA74ZP5UVXUNI6IQ3DWEGGP3ANCNFSM6AAAAAARKNKVWU.
You are receiving this because you authored the thread.

You might need mic permissions if you haven’t set that up already.

On Oct 20, 2022, at 2:44 PM, DaveHineman @.***> wrote:
Thanks, Nick.
I can edit that here and give it a try. I’m having problems with my own app, as well, using .defaultToSpeaker.
What worked great two years ago is now killing the mic input on iPhones.
Dave.

On Oct 20, 2022, at 1:37 PM, Nick Culbertson @.***> wrote:
It probably has something to do with setting the AudioSessiontry Settings.setSession(category: .playAndRecord, with: [.mixWithOthers, .allowBluetooth, .allowBluetoothA2DP]) It needs .defaultToSpeaker
—
Reply to this email directly, view it on GitHub https://github.com/AudioKit/AudioKit/issues/2790#issuecomment-1285981051, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALXQ4DA74ZP5UVXUNI6IQ3DWEGGP3ANCNFSM6AAAAAARKNKVWU.
You are receiving this because you authored the thread.

—
Reply to this email directly, view it on GitHub https://github.com/AudioKit/AudioKit/issues/2790#issuecomment-1285987958, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGI66QGFSO3W7YXMXXPL3TWEGHIVANCNFSM6AAAAAARKNKVWU.
You are receiving this because you commented.

Mic permissions are in place.
In the Cookbook, I found two files that reference .setCategory and they both already have .defaultToSpeaker in the options.
If there are any others that I overlooked, please advise.
So, the session begins, defaulted to Speaker. The sound output routing is changing whenever the Channel/Device screen is accessed. Are you able to reproduce my issue?

Thanks, Nick, I will check that out.

After adding .defaultToSpeaker to ChannelDeviceRouting.swift, the sound output is no longer switching to the receiver speaker as I move around the menu items. The microphone is continuing to function, as well.
I tested on an iPhone XR with iOS 16.0.2 and also an iPhone7S with iOS 15.7. I don't see the problem anymore. Thanks a ton!

Sweet! Does it also work with removing that and the next line entirely since it is already being set in the app delegate?On Oct 20, 2022, at 4:26 PM, DaveHineman @.***> wrote:
After adding .defaultToSpeaker to ChannelDeviceRouting.swift, the sound output is no longer switching to the receiver speaker as I move around the menu items. The microphone is continuing to function, as well.
I tested on an iPhone XR with iOS 16.0.2 and also an iPhone7S with iOS 15.7. I don't see the problem anymore. Thanks a ton!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

No, that sounds good. It just feels odd that the AudioSession is set 3 different ways in the app.I’ll add your fix to the Cookbook soon unless you feel like doing a pull request on the Cookbook.On Oct 20, 2022, at 4:26 PM, DaveHineman @.***> wrote:
After adding .defaultToSpeaker to ChannelDeviceRouting.swift, the sound output is no longer switching to the receiver speaker as I move around the menu items. The microphone is continuing to function, as well.
I tested on an iPhone XR with iOS 16.0.2 and also an iPhone7S with iOS 15.7. I don't see the problem anymore. Thanks a ton!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

Thanks, Nick. I’m good with you adding it in to the next version.
Best,
Dave.

On Oct 20, 2022, at 5:04 PM, Nick Culbertson @.***> wrote:

No, that sounds good. It just feels odd that the AudioSession is set 3 different ways in the app.I’ll add your fix to the Cookbook soon unless you feel like doing a pull request on the Cookbook.On Oct 20, 2022, at 4:26 PM, DaveHineman @.***> wrote:

Thanks, Dave! Fixed here: https://github.com/AudioKit/Cookbook/pull/116

Thanks so much for your help, Nick.
Working through this issue with you confirmed that I should be able to get ‘defaultToSpeaker' working on my own app and that it isn’t an Apple bug. Whew!
Because of your help, I am making some changes to the Flutter audio capture plugin that I’m using and will submit those improvements to the author.
Dave.

On Oct 28, 2022, at 12:10 PM, Nick Culbertson @.***> wrote:
Thanks, Dave! Fixed here: AudioKit/Cookbook#116 https://github.com/AudioKit/Cookbook/pull/116
—
Reply to this email directly, view it on GitHub https://github.com/AudioKit/AudioKit/issues/2790#issuecomment-1295247475, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALXQ4DGJW2JN6N5ZINEZEDTWFQCJZANCNFSM6AAAAAARKNKVWU.
You are receiving this because you authored the thread.


Swapping Layouts moving dash-item

Hi
I have a problem with dynamically swapping between layouts, and have recreated the issue here
https://codesandbox.io/s/vue-responsive-dash-bvrwi
When you click the swap button it just swaps between two predefined layouts, however after you click swap twice all the boxes are moved to the bottom of the page, but they should be at the top like when it was first loaded.
I have found that the y positions are getting changed from the v-bind.sync="item" - an update:y is getting emitted for some reason and changing the layout.
This is critical to my project, hope you can find the issue.
Thanks
Peter

Further update, I just found that it is caused because the id 1 is used in both layouts. If I change the id on the second layout to 5 it seems ok, so when swapping layouts, it seems it is incorrectly  "remembering" the id from the previous layout.

Hi @peterjslater,
This error is most likely down to how vue insert's/removes components during a single cycle. As the id is used in both layouts i can see the potential for corruption. See solution below which takes advantage of the nextTick https://vuejs.org/v2/api/#mounted. Note i am clearing the layouts waiting for next tick and then changing to the new desired layouts. Thus giving vue time to remove the components (and therefore the dash items) before re-inserting them
https://codesandbox.io/s/vue-responsive-dash-3zorp


e2e dra: add support for containerd from main in kind

These instructions bring up a kind cluster with containerd 34d078e99, the latest commit from the main branch. This version of containerd should have support for CDI, but in practice it didn't work.

I have the right containerd:
$ kind export logs
Exporting logs for cluster "kind" to:
/tmp/2312019231
pohly@pohly-desktop:/nvme/gopath/src/k8s.io/kubernetes$ head /tmp/2312019231/kind-worker/containerd.log
Sep 28 08:27:39 kind-worker systemd[1]: Starting containerd container runtime...
Sep 28 08:27:39 kind-worker systemd[1]: Started containerd container runtime.
Sep 28 08:27:39 kind-worker containerd[213]: time="2022-09-28T08:27:39.923818288Z" level=info msg="starting containerd" revision=34d078e99fbdb8c28feec359634335a2c684e703 version=v1.6.0-830-g34d078e99
...

But the E2E tests fail:
$ export KUBECONFIG=/home/pohly/.kube/config
$ _output/bin/ginkgo --fail-fast -focus=DynamicResourceAllocation ./test/e2e
...
  Sep 28 11:04:55.187: container env variables
  Expected
      <string>: KUBERNETES_SERVICE_PORT=443
      KUBERNETES_PORT=tcp://<IP_ADDRESS>:443
      HOSTNAME=tester-1
      SHLVL=1
      HOME=/root
      KUBERNETES_PORT_443_TCP_ADDR=<IP_ADDRESS>
      PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
      KUBERNETES_PORT_443_TCP_PORT=443
      KUBERNETES_PORT_443_TCP_PROTO=tcp
      KUBERNETES_SERVICE_PORT_HTTPS=443
      KUBERNETES_PORT_443_TCP=tcp://<IP_ADDRESS>:443
      KUBERNETES_SERVICE_HOST=<IP_ADDRESS>
      PWD=/

  to contain substring
      <string>:
      user_a=b

  In [It] at: /nvme/gopath/src/k8s.io/kubernetes/test/e2e/dra/dra.go:672

The reason for the failure was broken support for nodes with multiple clusters in the E2E test. Should be fixed now.


Replace Google services with FLOSS alternatives

Is your feature request related to a problem? Please describe.
When trying to use your website with ad or script blockers, such as uBlock Origin, uMatrix or NoScript, the website breaks unless you allow GStatic and Google Fonts to work, which can be detrimental to users' privacy, and in recent years one of the main interests of open source software is to fight back mass surveillance.
Describe the solution you'd like
To use FLOSS alternatives that could replace G services.
Describe alternatives you've considered
Below you will see a list of options from where you could choose, I recommend you Fork Awesome (and maybe the Open Font Library, although I do not know if they work in the same way) in the best case scenario, and Font Awesome in a better case scenario.
https://alternativeto.net/software/google-web-fonts/?license=opensource
https://alternativeto.net/software/font-awesome/?license=opensource
https://fontlibrary.org/
https://forkaweso.me/Fork-Awesome/
https://fontawesome.com/
Additional context
Screenshot of how it looks: https://upload.vaa.red/3TuL4#a29e7189af55bdeb9b237c9f4686e787

Are you considering this? I know you might be busy and that you have a life, I don't want this to be implemented right now, just to know if it will happen. After all it's not that privacy respecting if it's full of Big G services.
Thanks for your hard work!

Yes, I am considering this. You have raised a good point, but moving from Google Material Icons and Google Fonts do break a bit of functionality, I need to spend a bit more time on it. But it's definitely something in the pipeline.
I'm working on my personal website right now, as well as another template for the resume, so trying to balance projects. I will keep you updated on the progress of this issue :) Thank you so much for the constant support and raising feedback through issues. It really means a lot.

Don't worry about how long does it take, as long as it is something that you think it's worth from a privacy standpoint and you support the cause.
Don't worry, when I see a good project that deserves feedback, I try to chip in!

@5a384507-18ce-417c-bb55-d4dfcc8883fe I'm happy to inform you, that after 12 days of opening the issue, this is finally resolved 🎉
Sorry that it took a while, but I hope it was worth it :) I've self-hosted everything now, the animation library, the fonts, the icons. There should be no external HTTP requests made now (except for Google Analytics, which can be blocked through using extensions and will not affect performance of the app).


bug(tabs) : ngIf changes tab order

Short description of the problem:
When binding to a specific ion-tab to make it visible or not, the tab ordering is incorrect.
What behavior are you expecting?
I want to bind a specific tab to a variable on whether it should show or not. The tab bar does hide/show the tab. However when the tab is shown, it's at the end of the tab bar instead of where it is specified in the HTML.
HTML - note the invite tab position
<ion-tabs>
    <ion-tab [root]="tab1Root" tabTitle="Invite" tabIcon="people" *ngIf="competitionRunning"></ion-tab>
    <ion-tab [root]="tab2Root" tabTitle="Info" tabIcon="information-circle"></ion-tab>
    <ion-tab [root]="tab3Root" tabTitle="Gallery" tabIcon="images"></ion-tab>
    <ion-tab [root]="tab4Root" tabTitle="Inbox " tabIcon="chatbubbles"></ion-tab>
</ion-tabs>

TS
@Page({
  templateUrl: 'build/pages/tabs/tabs.html'
})
export class TabsPage {
  tab1Root: any = InvitePage;
  tab2Root: any = InfoPage;
  tab3Root: any = GalleryPage;
  tab4Root: any = InboxPage;

  competitionRunning: boolean = true;

  constructor(private userState : SystemUserState, private appState : ApplicationInfoState){
      // find the user's university
      var university = appState.findUniversity(userState.universityId);
      this.competitionRunning = university.Competition.IsRunning;
  }
}

Screenshot:

Ionic Version: 2.x
Browser & Operating System: iOS / Android / Chrome
Run ionic info from terminal/cmd prompt:
Cordova CLI: 6.0.0
Ionic Version: 2.0.0-beta.3
Ionic CLI Version: 2.0.0-beta.19
Ionic App Lib Version: 2.0.0-beta.9
OS:
Node Version: v5.7.0

I think this is a know issue (:
It has also been reported to Angular here: angular/angular#5363

Instead of using *ngIf, we've added show to <ion-tab> to cover this scenario. It sets if the tab button is visible within the tabbar or not.
Additionally, we also added enabled.

If the tab is enabled or not. If the tab is not enabled then the tab button will still show, however, the button will appear grayed out and will not be clickable.

You can change the tab order using just CSS.
.tabs-icon-top > .tabs .tab-item:nth-of-type(1) {   order:1; }


return null why

    $key = public_path().'/cmsmrtho-firebase-adminsdk-jhhpo-ac662abbc0.json';
    $client = new \Google_Client();
    $client->setApplicationName('Play Cool Math Notifications');
    $client->setAuthConfig($key);
    $client->addScope('https://www.googleapis.com/auth/firebase.messaging');
    $client->authorize();
    $token = $client->getAccessToken();
    echo $token;

 // it return null why ?

Hi @khanhnam99
Sorry for a very late reply on this.
But from what I can see from the source of the Client class, because you have supplied a key in the setAuthConfig method, those credentials are used as default credentials.
After this, in the authorize method, the credentials are attached to the auth handler, which is returned to you in the authorize call. So, the access token is never set for the Client class.
So you can make the calls like so:
$httpClient = $client->authorize();
$response = $httpClient->get('https://www.googleapis.com/auth/firebase.messaging');

Hope this answers your question.


Failed to execute (OSMGPFileLoader)

Have been trying out the new 10.4 version (and thank you for that!!). Everything seems to run OK except in the final stages. After 43 hours (mostly spent on resolving multi-part geometries), I run into the following error:

The given key was not present in the dictionary
at System.Collections.Generic.Dictionary'2.get_Item(TKey key)
at ESRI.ArcGIS.OSM.GeoProcessing.OSMGPFileLoader.Execute(IArray paramvalues, ITrackCancel TrackCancel, IGPEnvironmentManager envMgr, IGPMessages message)
Failed to execute (OSMGPFileLoader)

Any ideas?

In what geographical extent are you working or what is the OSM file we should use for testing? What are the OSM tags you are attempting to extract as part of the load process?

Hi Thomas, I'm trying to load the full Australia dataset which is around 6.1gb. I have done it before and not had any problems. The one thing I think I did differently last time was to specify a set of tags. This time around I just selected ALL. I've commenced a run tonight using only the same set as last time and I'll see how it goes but it will take around 48 hours to run.
Thanks for your assistance!

If you do not intend to edit the data you might want to try a new tool in this release called OSM File Loader (Read Only). This tool skips some functionality that is needed for the read/write access to the OSM server. Using the new tool I was able to load Australia in a little less than 4 hrs on my desktop. Depending on the use of the machine where the tool is being executed you can make use of the parallel processing factor environment variable. To optimize further you should have the scratch workspace pointing to a folder on a different disk than the file geodatabase you use to load the data. For example, have the scratch workspace point to c:\sample and have the target feature classes at d:\OSM\Australia.gdb.

Thanks Thomas, didn't notice that option. I've just tried using it and I pretty quickly get the error "Unable to create a valid point feature class (Object reference not set to an instance of an object). i have tried with and without manually defining tags. I'm also trying to load into a dataset as I would still like to create a network dataset from the data. Any ideas?

Hmm, that looks like a bug. Can you post your parameters? Are you running 64bit or 32bit background processing?

I've tried 32 and 64 bit processing. Here's a copy of my parameters and the results:
Messages
Executing: OSMGPMultiLoader S:\SpatialData\australia-latest.osm # # # DO_NOT_DELETE_NODES DO_NOT_DELETE_OSM_FILE S:\SpatialData\SpatialDB.gdb\OSM\osm_pt S:\SpatialData\SpatialDB.gdb\OSM\osm_ln S:\SpatialData\SpatialDB.gdb\OSM\osm_ply
Start Time: Tue May 17 11:51:23 2016
Counted 31177712 nodes, 2421838 ways, and 30023 relations.
Unable to create a valid point feature class (Object reference not set to an instance of an object.).
Failed to execute (OSMGPMultiLoader).
Failed at Tue May 17 11:52:23 2016 (Elapsed Time: 59.79 seconds)

I was able to reproduce the error.
For the time being do not attempt to load the data into a feature dataset just standalone feature classes. Then as a second step move the feature classes into a feature dataset.

Thanks Thomas. I've tried as you describe and run into a new problem. Sorry! I don't know how I've complicated this so much. Had things working perfectly on 10.3 :( Error as follows:
Messages
Executing: OSMGPMultiLoader S:\SpatialData\australia-latest.osm # # # DO_NOT_DELETE_NODES DO_NOT_DELETE_OSM_FILE S:\SpatialData\OSMData.gdb\osm_pt S:\SpatialData\OSMData.gdb\osm_ln S:\SpatialData\OSMData.gdb\osm_ply
Start Time: Thu May 19 08:04:10 2016
Counted 31177712 nodes, 2421838 ways, and 30023 relations.
Loading nodes...
Done loading nodes (Elapsed Time: 0 hours 0 minutes 1 seconds).
Error HRESULT E_FAIL has been returned from a call to a COM component.
at ESRI.ArcGIS.Geoprocessing.GeoProcessorClass.IGeoProcessor2_Execute(String Name, IVariantArray ipValues, ITrackCancel pTrackCancel)
at ESRI.ArcGIS.OSM.GeoProcessing.OSMToolHelper.loadOSMNodes(List1 osmNodeFileNames, List1 nodeGDBNames, String featureClassName, String targetFeatureClass, List`1 tagsToLoad, Boolean deleteNodes, IGPMessages& toolMessages, ITrackCancel& CancelTracker)
at ESRI.ArcGIS.OSM.GeoProcessing.OSMGPMultiLoader.Execute(IArray paramvalues, ITrackCancel TrackCancel, IGPEnvironmentManager envMgr, IGPMessages message)
Failed to execute (OSMGPMultiLoader).
Failed at Thu May 19 08:07:36 2016 (Elapsed Time: 3 minutes 25 seconds)

Did you add the python directory to the PATH system environment variable?
If we have hidden this information, could you please suggest a location to more prominently feature the changes to the tool?
The current location is here: https://github.com/Esri/arcgis-osm-editor/wiki/Release-Notes

Yes, PATH variable is set as C:\Python27\ArcGISx6410.4 but I'm still receiving the error

And another bug. We have an API call that is not working as expected in a 64bit process. https://github.com/Esri/arcgis-osm-editor/blob/master/src/OSMGeoProcessing/OSMGPFactory.cs#L514
For now please use the tool in a 32bit call. Either in 32bit background processing or if you have the 64bit background processing installed, please run the tool in the foreground.

Thanks Thomas, yes changing to 32-bit did the trick. Thanks! And I'm happy to test further if it would help at all.

Hello, apologies for opening a very old issue. Having the same issues as OP, but with ArcMap 10.8. Would someone be able to help me on how to change background processing in the UI? Not sure if it''s 64bit or 32 bit. Thanks!

Here is the help on how to switch between foreground and background.
https://desktop.arcgis.com/en/arcmap/latest/analyze/executing-tools/foreground-and-background-processing.htm


Synchronous prompt on CustomScriptDialogOpening WebView2Bug

I'm developing a MAUI application in C# that implements a WebView2 and I need to change the style of the alert, prompt, and confirm dialogs. In WinUI applications everything is very modern, avoiding synchrony, but apparently only when it suits Microsoft, let me explain: If I do:
webView.CoreWebView2.Settings.AreDefaultScriptDialogsEnabled = false;
webView.CoreWebView2.ScriptDialogOpening += CustomScriptDialogOpening;

I can intercept the default JavaScript dialogs, but even if the CustomScriptDialogOpening function is async, the WebView doesn't await it.
So,
How the heck can I make a prompt if I can't await it, And there's no way to launch the user prompt window synchronously, and if I launch it asynchronously, WebView2 doesn't await it?
I don't know if this has a solution... Thank you!
Steps to reproduce the bug
Create MAUI project with windows support.
Add webview.
Set AreDefaultScriptDialogsEnabled = false
add Event on ScriptDialogOpening
Expected behavior
EventHandler has to await their added handler events.
Screenshots
No response
NuGet package version
None
Windows version
Windows Insider Build (xxxxx), Windows 11 (22H2): Build 22621, Windows 11 (21H2): Build 22000, Windows 10 (21H2): Build 19044, Windows 10 (21H1): Build 19043, Windows 10 (20H2): Build 19042, Windows 10 (1809): Build 17763
Additional context
No response

An example may not make much sense as it is a very simple problem, but I will try to explain it better:
In JavaScript, when you launch a prompt, it runs synchronously, stopping the JavaScript execution thread and returning the user's entered response. This is also applicable to confirm and alert.
If we access this PlatformView and use its "wb.CoreWebView2.ScriptDialogOpening" property, we can add an event listener that returns these prompt, confirm, and alert calls to C# code.
NOTE: To make it work and have the calls returned, you need to disable the default dialog execution with the "wb.CoreWebView2.Settings.AreDefaultScriptDialogsEnabled = false;" property.
Then, if you add a method to handle the events, you will see that we can do (I will give the example of prompt()):
wb.CoreWebView2.ScriptDialogOpening += async (CoreWebView2 sender, CoreWebView2ScriptDialogOpeningEventArgs e) => {
	if (e.Kind == CoreWebView2ScriptDialogKind.Prompt) {
		e.ResultText = await DisplayPromptAsync("title", "Question", "Cancel", "OK", "");
		e.Accept();
	}
}};

But the method, despite being async, is not awaited by the function that launches it.
If I don't make it async, I can't launch a Prompt to the user because the main thread is busy waiting for the prompt response in the WebView, so it is never processed.
This makes it practically impossible to display a custom Prompt to the user.
If you need more information, I can provide it.

The description suggests the issue is Edge/CoreWebView2 and should be moved to WebView2Feedback. @champnic can you take a look?

Hey @OMendozaG - To block the script execution until you are ready with a response, please use the Deferral mechanism of the event args, CoreWebView2Deferral deferral = e.GetDeferral();:
https://learn.microsoft.com/en-us/dotnet/api/microsoft.web.webview2.core.corewebview2.scriptdialogopening?view=webview2-dotnet-1.0.2151.40#remarks
https://learn.microsoft.com/en-us/dotnet/api/microsoft.web.webview2.core.corewebview2scriptdialogopeningeventargs.getdeferral?view=webview2-dotnet-1.0.2151.40#microsoft-web-webview2-core-corewebview2scriptdialogopeningeventargs-getdeferral
And then continue the script execution by completing the deferral, deferral.Complete();.

Closing as the behavior is expected as per @champnic's comment above.


dialect/sql/sqlgraph: support creating nodes without scanning their ids

This PR will make it possible to improve the efficiency of <T>Create.Exec in generated code.

Update:  <T>Create.Exec is implemented currently using Save. Changing Exec to not return the ID field (and the created entity), will break the behavior of hooks. i.e. A hook of type OpCreate will return an ent.Value(nil) after the mutation was called.
func(next ent.Mutator) ent.Mutator {
	return ent.MutateFunc(func(ctx context.Context, m ent.Mutation) (ent.Value, error) {
		v, err := next.Mutate(ctx, m)
		// v is nil.
		return v, err
	})
}

Unfortunately, need to abandon this PR as this breaking change is unacceptable.

cc @rotemtam, @tarrencev


Invitation Contributing to Authentic Theme Support for Debian 11

Summary
This ticket is about inviting volunteer service contributions to facilitate Authentic Theme support for the released Debian 11 Bullseye. And ask if there is anything needed we could contribute to? Documentation? Testing? What else? Both me and other contributors expressed interest in this. We're wondering is any is presently needed?
2 questions below

Below is the same message as above. But with details if you're interested in those.
Contributing
This ticket is for all contributors interested to facilitate Authentic Theme support for Debian 11. If Virtualmin team, or any other contributors need any, you are really welcome to use this ticket.
This ticket is to offer volunteer contributions to facilitate Authentic Theme support for Debian 11 Bullseye. If Virtualmin team, or any other contributors need any, you are really welcome to use this ticket.
For example, in addition to my offer of volunteer contributor time, my company Ubertus.org is both willing & happy to offer access to up to USD 4,000 yearly of advanced online & reputable testing tools. Such as BrowserStack and Sauce Labs accounts. For those not familiar with those tools, they allow you to remotely test with 3000+ remotely controlled real devices and browsers. Those accounts are limited. So Ubertus offers them first to contributors who demonstrated past public interest to contribute to Authentic Theme. No strings attached. This is a gift for those efforts. And Ubertus do not expect anything in return.
Questions
Question 1. For Contributors.
Any other volunteer contributors interested to offer testing or documentation or anything else to your liking? All are welcome to join the efforts to facilitate Authentic Theme support for Debian 11 Bullseye. I believe that both small and large contributions can facilitate such efforts. You're welcome to subscribe or add comments on this ticket. So that you're notified about future needs to facilitate any efforts.
Question 2. For Staff.
Beside testing and documentation, is there any other types of volunteer service contributions that would facilitate the efforts for Authentic Theme support for Debian 11 Bullseye?
Intention
What is my intention with this ticket?

Facilitate interested volunteer contributions. Toward Authentic Theme support for Debian 11.
Invite others to express any need for contributions for this
All of the above free from interference. In other words, other volunteers are really free to decide what is their gift, where, when, how they contribute it. I would welcome the same in return.

What is not my intention with this ticket? Nothing else than the above. I understand that Virtualmin staff do not pre-annonce date for Debian 11. I believe that this is fair and make sense. This choice was successful in the past. I know that it is a lot of work to support any new OS version. Obviously, this ticket is an invitation to express need if any are needed. Not force contributions to others. For those not interested, feel really free to decline this invitation. I also have no intention of interfered with other contributors. Speaking for myself, you are really free from interference. I believe that diversity is a strength.
Speaking for myself, one of the main benefit I see with Libre/Open Source software is that if I'm interested in a product supporting a new OS version. One of many ways to potentially reach that goal faster, is for volunteers to offer to contribute. In turn, others, from their own free will, are welcome to decide if any are needed. Both very small and very large contributions can make a difference 💚 .
Cheers,
Francois Carpentier (Francewhoa)
Related Tickets

#1570
#343
#1565

I will close this as a direct duplicate of https://github.com/virtualmin/virtualmin-gpl/issues/343

https://forum.virtualmin.com/t/virtualmin-7-and-debian-11/113031/7?u=ilia


sndio: 1.6.0 -> 1.7.0

Motivation for this change
Updates sndio to the latest version.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Fails to build on macos (10.14):
mkdir -p /nix/store/mq3gi0vq1jarrzhk9qg8pn4pvhiyjnv5-sndio-1.7.0/bin /nix/store/mq3gi0vq1jarrzhk9qg8pn4pvhiyjnv5-sndio-1.7.0/share/man/man1
  cp midicat /nix/store/mq3gi0vq1jarrzhk9qg8pn4pvhiyjnv5-sndio-1.7.0/bin
  cp midicat.1 /nix/store/mq3gi0vq1jarrzhk9qg8pn4pvhiyjnv5-sndio-1.7.0/share/man/man1
  make[1]: Leaving directory '/private/tmp/nix-build-sndio-1.7.0.drv-0/sndio-1.7.0/midicat'
  post-installation fixup
  gzipping man pages under /nix/store/mq3gi0vq1jarrzhk9qg8pn4pvhiyjnv5-sndio-1.7.0/share/man/
  strip is /nix/store/gmhd52yac3w4zk8xqsgh97makjh8gi7l-cctools-binutils-darwin-927.0.2/bin/strip
  stripping (with command strip and flags -S) in /nix/store/mq3gi0vq1jarrzhk9qg8pn4pvhiyjnv5-sndio-1.7.0/lib  /nix/store/mq3gi0vq1jarrzhk9qg8pn4pvhiyjnv5-sndio-1.7.0/bin
  patching script interpreter paths in /nix/store/mq3gi0vq1jarrzhk9qg8pn4pvhiyjnv5-sndio-1.7.0
  error: install_name_tool: can't open file: /nix/store/mq3gi0vq1jarrzhk9qg8pn4pvhiyjnv5-sndio-1.7.0/lib/libsndio.7.0.dylib (No such file or directory)

Rather than chasing the exact .dylib name in a postFixup all the time, it does seem to work if you simply add fixDarwinDylibNames to the build inputs if stdenv.isDarwin.
Also, obviously, could do with a rebase.

WFM, macos 10.14, non-nixos linux x86_64.


"Fix all auto-fixable problems" does not seem to do anything

I am trying vscode-eslint plugin in VS Code.
I took a file (typescript) that is formatted incorrectly, and I ran "Fix all auto-fixable problems", and nothing happened.
To make sure this isn't a problem with ESLint I run
node_modules/.bin/eslint src/Path/File.ts --fix
This works as expected, and the file is reformatted.

@Thorocaine auto fix is not enabled by default for TypeScript since there were cases in the past that it caused problems. To enable it do
"eslint.validate": [ { "language": "typescript", "autoFix": true } ]

Thank you, I missed that.


Remove the unique index for lemma names and add a validator instead.

Mysql isn’t able to distinguish between e.g. a and á. This wouldn’t allow to add some lemmata.

Fixed for lemmata and parts of speech.


Node.js default timeout

I've been using Koa v2 to implement an API and recently ran into an issue with long lasting HTTP requests, where they would crash after 120 seconds. Curl (which I was using to test this API) would return the error curl: (52) Empty reply from server
I also noticed that Koa skipped the whole part where the middlewares would filter and manipulate the response upstream (probably because the HTTP connection was closed).
It turns out that node's HTTP server has a default timeout of 120 seconds.
I managed to disable this timeout in Koa by creating this simple middleware:
app.use(async (ctx, next) => {
    ctx.req.setTimeout(0);
    await next();
});

I'm not sure if it is within the scope of this project to disable this default behavior, but I thought of at least sharing the knowledge as someone else might find it useful.

I'm not sure if it is within the scope of this project to disable this default behavior

I think we'll probably stick to the HTTP module defaults. Thanks for the tip though!

I agree with @PlasmaPower, this is a node issue, and chances are more people expect the timeout to exist than not. I'm closing this, but yes, thanks for the suggestion :)

Just use server.setTimeout to  change default timeout :)
https://nodejs.org/api/http.html#http_server_settimeout_msecs_callback


Eslint all files with prettier

ESLint was previously set up to do the default file glob, which only finds .js files.  With all the awesome TypeScript conversion @colebemis has been doing, lint has been covering less and less files.  This PR does the following:

adds coverage back to all files (.js, .ts, .tsx)
adds the prettier plugin into the eslint config (it was a dependency, but wasn't being used)
fixes all the related lint issues
adds yarn lint-fix which can be used to run prettier if your IDE isn't already configured to do so
updates all the eslint/prettier related dependencies.

Note: I split the base eslint fixes and the prettier fixes into two separate commits.  The base fixes probably deserve the closer look of the two.

Hmm, there must be another build output dir that I missed.  @colebemis can you think of another spot that .js files are generated?  I'd prefer not to limit it to src/... because there are lots of other one-off js files that wouldn't get picked up.


Couldn't include module

I getting this:
Couldn't include file 'includes\modules\3d2dvgui.lua' (File not found) (@addons/profile/lua/autorun/client/cl_init.lua (line 146))
I placed 3d2dvgui.lua in "lua/includes/modules" but it still can't find it. Where do i need to place 3d2dvgui.lua?

Same :/

ive placed it in lua/includes/modules/3d2dvgui.lua and it isnt included

having the same issue aswell


add(rpc): Create documentation resource for all RPCs

Motivation
Zebra now contains RPCs that are only documented in the API docs rendered at https://docs.rs/zebra-grpc/0.1.0-alpha.2/zebra_grpc/server/struct.ScannerRPC.html. These docs are Rust-specific and not suitable as general documentation for an RPC endpoint.
Zebra also implements JSON RPCs for lightwalletd: https://docs.rs/zebra-rpc/1.0.0-beta.35/zebra_rpc/methods/index.html; and JSON RPCs for mining, which are currently not rendered in the public API due to #8169. Note that the JSON RPCs are in a different crate than the gRPCs, which complicates discoverability.
zcashd renders its RPCs here: https://zcash.github.io/rpc/. Ideally, we should create a similar resource that would:

be language-agnostic,
contain the docs for all RPCs under one roof,
render the docs directly from the source code.

It would be great if we found a tool that would fulfill the requirements above, and we could integrate it into CD.

I think we can close this and do the grpc in https://github.com/ZcashFoundation/zebra/issues/8407


Missing rooftops on agility courses

Rooftops seems to be missing at several (not all) agility courses. And no, "hide rooftop" toggle doesnt help. This only seem to be a problem on Runelite.

After turning it off and rebooting the roofs are back. Thanks.


(v1.15) Bump golang.org/x/net

Clears some CVEs reported by vulnerability scanners.

/test-backport-v1.15

/test-backport-1.15


[BUG] list set item doesn't use PyList_SetItem when wraparound/boundscheck False

Describe the bug
If I disable boundscheck and wraparound, list gets switch from __Pyx_GetItemInt_List to PyList_GET_ITEM but list sets stay as __Pyx_SetItemInt.
Code to reproduce the behaviour:
# cython: language_level=3

import cython

def foo1(list bar not None):
    bar[1] = bar[2]

@cython.wraparound(False)
@cython.boundscheck(False)
def foo2(list bar not None):
    bar[1] = bar[2]

Expected behaviour
The bar[1] = bar[2] line in the first function compiles to:
  __pyx_t_1 = __Pyx_GetItemInt_List(__pyx_v_bar, 2, long, 1, __Pyx_PyInt_From_long, 1, 0, 1); if (unlikely(!__pyx_t_1)) __PYX_ERR(0, 6, __pyx_L1_error)
  __Pyx_GOTREF(__pyx_t_1);
  if (unlikely((__Pyx_SetItemInt(__pyx_v_bar, 1, __pyx_t_1, long, 1, __Pyx_PyInt_From_long, 1, 0, 1) < 0))) __PYX_ERR(0, 6, __pyx_L1_error)
  __Pyx_DECREF(__pyx_t_1); __pyx_t_1 = 0;

and in the second:
  __pyx_t_1 = PyList_GET_ITEM(__pyx_v_bar, 2);
  __Pyx_INCREF(__pyx_t_1);
  if (unlikely((__Pyx_SetItemInt(__pyx_v_bar, 1, __pyx_t_1, long, 1, __Pyx_PyInt_From_long, 1, 0, 0) < 0))) __PYX_ERR(0, 11, __pyx_L1_error)
  __Pyx_DECREF(__pyx_t_1); __pyx_t_1 = 0;

I had hoped for something more like:
  __pyx_t_1 = PyList_GET_ITEM(__pyx_v_bar, 2);
  __Pyx_INCREF(__pyx_t_1);

  __Pyx_INCREF(__pyx_t_1);
  __Pyx_GIVEREF(__pyx_t_1);
  PyList_SET_ITEM(__pyx_v_bar, 1, __pyx_t_1);

  __Pyx_DECREF(__pyx_t_1); __pyx_t_1 = 0;

(I may have messed up the reference counting in this – I'm assuming that the code generator would need to add in the INCREF/GIVEREF to make PyList_SET_ITEM work.)
OS
macOS
Python version
3.11.4
Cython version
3.0.2
Additional context
No response

Ah, no, wait.
Having attempted a low-level experiment and actually consulted the API docs, I see that this would have to be PyList_SetItem not PyList_SET_ITEM or the existing item in the list will not get DECREFed and will leak.
I can explicitly do this:
# cython: language_level=3

import cython
from cpython cimport PyList_SetItem, Py_INCREF

def foo1(list bar not None):
    bar[1] = bar[2]

@cython.wraparound(False)
@cython.boundscheck(False)
def foo2(list bar not None):
    cdef object tmp = bar[2]
    Py_INCREF(tmp)
    PyList_SetItem(bar, 1, tmp)

and the code works as I expected, but feels an order of magnitude more hacky…

I'm not convinced this is a bug.
If we look at the definition __pyx_SetItemInt
https://github.com/cython/cython/blob/4df7e67000575b42183735fabbbec5a5bdc01465/Cython/Utility/ObjectHandling.c#L477-L481
It takes is_list, wraparound and boundcheck parameters, so it's designed to account for these settings.
The intention for a lot of these functions in that they're largely inlined in place (so the wraparound/boundscheck parameters will just allow the C compiler to drop code as needed). With that said __pyx_SetItemInt_Fast isn't declared as inline.
I think the idea is that it's easier to hide the things behind macros/functions that have a single interface rather than change the code generation substantially for the different compiler settings.

With that said __pyx_SetItemInt_Fast isn't declared as inline.

Yes, it is, exactly to optimise the fast path for lists, and then gradually fall back to less and less efficient code for other data types.
I'm closing this ticket since the resulting binary code will be virtually the same for lists.

Thanks for this. Having just spent the last 15 minutes staring at the assembler output and running microbenchmarks, I can confirm 🙄. I was thrown by the fact that the get is optimised differently.
Gonna go hunt elsewhere for speed-ups…


Best policy practices for production

In config.js line 11;
// In actual production, the policy document should be generated dynamically.
Isn't it a waste that generating a policy for every single thing? Assume that thousands of devices for an IoT solution. I think it should be only a single policy which allows publish and subscribe to topics by their certificate ID's. An example from AWS IoT docs is like this:
{
    "Version": "2012-10-17",
    "Statement": [{
        "Effect": "Allow",
        "Action":["iot:Publish"],
        "Resource": ["arn:aws:iot:us-east-1:123456789012:topic/${iot:CertificateId}"]
    },
    {
        "Effect": "Allow",
        "Action": ["iot:Connect"],
        "Resource": ["*"]
    }]
}

What do you think about this, what are your recommendations about that?

The policy for each device should be dynamic. In other words, it should not be that all devices have the same security policy, so that there is enough flexibility.

For the same security policy that some devices have, you can assign them to an iot device group and then attach a policy for the group.


Download & install renderer extension when opening a VSC Notebook

For #10496

Codecov Report

Merging #12652 into master will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #12652      +/-   ##
==========================================
+ Coverage   60.04%   60.07%   +0.02%
==========================================
  Files         657      659       +2
  Lines       35792    35910     +118
  Branches     5051     5061      +10
==========================================
+ Hits        21493    21572      +79
- Misses      13238    13263      +25
- Partials     1061     1075      +14

Impacted Files
Coverage Δ

src/client/common/utils/localize.ts
96.03% <100.00%> (+0.04%)
:arrow_up:

src/client/datascience/notebook/constants.ts
100.00% <100.00%> (ø)

...c/client/datascience/notebook/rendererExtension.ts
100.00% <100.00%> (ø)

...atascience/notebook/rendererExtensionDownloader.ts
100.00% <100.00%> (ø)

...nt/datascience/notebookStorage/vscNotebookModel.ts
9.43% <0.00%> (-8.75%)
:arrow_down:

src/client/common/utils/platform.ts
64.70% <0.00%> (-5.89%)
:arrow_down:

src/client/interpreter/interpreterService.ts
65.62% <0.00%> (-0.34%)
:arrow_down:

src/client/datascience/commands/commandRegistry.ts
33.80% <0.00%> (-0.16%)
:arrow_down:

src/client/common/startPage/startPage.ts
31.09% <0.00%> (-0.16%)
:arrow_down:

src/client/logging/levels.ts
61.11% <0.00%> (ø)

... and 10 more

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 a42b026...edecd6a. Read the comment docs.


Update monitor-azure-resource.md

Grammatical fix.

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


Feature/client side encryption: internal design doc review

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@a07bbb5). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main       #1   +/-   ##
=======================================
  Coverage        ?   60.45%
=======================================
  Files           ?      234
  Lines           ?    24408
  Branches        ?        0
=======================================
  Hits            ?    14757
  Misses          ?     8642
  Partials        ?     1009

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Implement reddit processor

Still need to figure out why the endpoint isn't working.


Differentiate between microbenchmarks and macrobenchmarks

We should classify the current set of benchmarks as microbenchmarks. They don't really resemble any "real-world" workload. In addition, we should introduce a notion of macrobenchmarks, which are supposed to resemble real-world workloads. One example of a macrobenchmark is a httpserver.

@dhil @codrutiftode and I had a chat, and Codrin decided that bringing in some HTTP server benchmarks would be good goal


OpenLDAP not working after upgrade to 21.05

Describe the bug
After I converted my configuration back to the OLC format (which is a very annoying task especially when I already converted from OLC to the config file format when I switched to NixOS), OpenLDAP isn't starting anymore.
To Reproduce

systemctl start openldap.service
journalctl -u openldap.service

Jun 08 19:43:22 lech systemd[1]: Starting LDAP server...
Jun 08 19:43:22 lech openldap-pre-start[3136]: 60bfc85a str2entry: entry -1 has no dn
Jun 08 19:43:22 lech openldap-pre-start[3136]: slapadd: could not parse entry (line=886)
Jun 08 19:43:22 lech slapadd[3136]: DIGEST-MD5 common mech free
Jun 08 19:43:22 lech systemd[1]: openldap.service: Control process exited, code=exited, status=1/FAILURE
Jun 08 19:43:22 lech systemd[1]: openldap.service: Failed with result 'exit-code'.
Jun 08 19:43:22 lech systemd[1]: Failed to start LDAP server.

Expected behavior
OpenLDAP should start without any problems.
Additional context
The script openldap-pre-start is trying to load the database with a generated configuration, that includes the schemas to be included. This is configured with services.openldap.settings.attrs.children."cn-schema".includes. This configuration results in the following section of the generated ldif file:
dn: cn=schema,cn=config
cn: schema
objectClass: olcSchemaConfig

include: file:///nix/store/1355lwpqmh9mbm11sdkn0d8hk2rbs38b-openldap-2.4.58/etc/schema/core.ldif

include: file:///nix/store/1355lwpqmh9mbm11sdkn0d8hk2rbs38b-openldap-2.4.58/etc/schema/cosine.ldif

include: file:///nix/store/1355lwpqmh9mbm11sdkn0d8hk2rbs38b-openldap-2.4.58/etc/schema/inetorgperson.ldif

include: file:///etc/evolutionperson.schema

I don't think that this can work because of the empty lines within the cn=schema,cn=config section which makes it actually multiple sections.
I think the problem is in the following Nix script:
  attrsToLdif = dn: { attrs, children, includes, ... }: [''
            dn: ${dn}
            ${lib.concatStringsSep "\n" (lib.flatten (lib.mapAttrsToList valueToLdif attrs))}
          ''] ++ (map (path: "include: file://${path}\n") includes) ++ (
            lib.flatten (lib.mapAttrsToList (name: value: attrsToLdif "${name},${dn}" value) children)
          );

A newline gets added within the map (path: "include: file://${path}\n") includes as well as in the line above.
Notify maintainers
@lovek323
Metadata

system: "x86_64-linux"
host os: Linux 5.10.40, NixOS, 21.05.804.5de44c15758 (Okapi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.12
channels(root): "nixos-19.09.1019.c5aabb0d603"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

It seems that this has been solved. I tried it and LDAP is working again for me.


POTA: change MY_SIG_INFO from K-nnnnn to US-nnnnn

Not sure, if this can be done already.
I have many files, whose park numbers MY_SIG_INFO were in the old K-nnnnn format.
Now they all are US-nnnnn format.
I could easily change the data part by using sed or some other tool. But the length part needs to be +1 like:
<MY_SIG_INFO:6>K-1234 to <MY_SIG_INFO:7>US-1234
Can that be done now. I suppose I could assume all the records in the adif file would have the same <MY_SIG_INFO>.
So basically, can edit --set change only PART of field, and adjust the length properly?
Jerry WF5W

There isn't currently a way to do this purely within adif-multitool, but I'd like to think about how it could be expressed.  A programmer's first instinct might be to do something sed-like, perhaps adifmt edit -replace 'my_sig_info=/^K-(\d+)/US-\1/'.  Introducing user-supplied regular expressions is something that should be done with care (I intentionally haven't supported them yet), so I'd like to consider other ways to express this.
The simple proposal above also wouldn't be great if someone had a lot of DX POTA contacts and wanted update the park references for many countries at once; some kind of lookup table might be more useful here.  (Though I suppose you could use sed to generate a really long adifmt edit command line.)  Before enacting this, I'd like to try to think of other fields that someone would want to do this kind of replacement on to make sure the way the feature is expressed is general-purpose.  If we don't add a general-purpose feature, I suppose it could be a special flag for adifmt fix that just knows the full mapping and applies it to POTA_REF, MY_POTA_REF, SIG_INFO, and MY_SIG_INFO.
One hack that might work for you, combining adif-multitool and sed:
adifmt cat -output adx my_log.adi | \
  sed 's/<MY_SIG_INFO>K-/<MY_SIG_INFO>US-/' | \
  adifmt save my_log_updated.adi

The ADX format gets you out of the field-length problem, and the XML tag name structure (one tag per line) is probably easier to work with than -output tsv and matching the right column.


C Parser gives bogus line number in error message

I'm trying to import classic Mac OS header files into Ghidra. I added all .h files to the Parse C Source window and clicked Parse to Program. (Using Ghidra 10.1-BETA, Windows 10.)
There is some error but the message isn't very helpful since ConditionalMacros.h only has 1846 lines...
C Parser: Encountered errors during parse.
in C:\path\to\file\CIncludes\ConditionalMacros.h near line 2669
near token: ;
Last Valid Datatype: enum_131
Check around CParserPlugin.out around line: 2675

Attached is an archive of the header files in case anyone can reproduce this.
CIncludes.zip

This is a shot in the dark but the issue doesn't happen to be around line 6 of ConditionalMacros.h does it?

I think line 6 is part of a comment.
Anyone know where the CParserPlugin.out file is written to?

I think line 6 is part of a comment.
Anyone know where the CParserPlugin.out file is written to?

User home directory

This issue has been around for some time... The C-Parser got significantly better, but I'm still getting incorrect line numbers.
This is my test case for Ghidra 10.3.2: a slightly modified tinyfiledialogs.h from the unofficial Github project for the library. I've added a bunch of empty lines, removed the last comment and inserted a letter "A" at line 356.
tinyfiledialogs.txt
(Github didn't want me to upload a file with the *.h extension for some reason)
The error message:
C Parser:  Encountered errors during parse.
         in <path>\tinyfiledialogs.h near line 243
Error: Encountered " <IDENTIFIER> "A "" at line 260, column 1.
 That's too much (for my taste, at least). Not to mention the fact that the parser repeats itself about the tokens like <PRAGMA> ....

Hi,
tinyfd_arrayDialog is not in tinyfiledialogs.c anymore. It has been moved to a separate file in more_dialogs/tinyfd_moredialogs.c


Publish the package to NuGet

Feature Request 🛍

Use Case

Possible Solution

Context

Detailed Description

Closed by #7


Docs improvements

This PR addresses several typographical errors across various files in the project. The changes improve readability and maintain the professional standard of the documentation and code comments.
Justification
Typographical errors, while minor, can detract from the overall quality of the project. Correcting these errors ensures clarity and professionalism, making the project more accessible and understandable for current and future contributors.

We don't accept typo PRs


Quote text in Markdown

If we select text and hit single or double quotes, please quote the selected text, instead of replacing the selected text with a quote character.
This would make it work like in other programming languages, then.

This is by design. It is the standard behavior in our text-file type languages as automatically inserting double quotes can often be annoying while writing normal text


Made plus sign in the dashboard active

Issue that this pull request solves
Closes: #304
Proposed changes
Made the plus sign in the dashboard active
Brief description of what is fixed or changed
Added a click action to the plus sign so that after clicking on that user can add new workspace.
Types of changes
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] My changes does not break the current system and it passes all the current test cases.

Screenshots
https://user-images.githubusercontent.com/56181018/107234555-ced2a400-6a49-11eb-831e-82ee08cf0711.mp4

LGTM :+1:
Thanks for the enhancement, @Dhamodhar-DDR!!

@Dhamodhar-DDR, Is this SWOC or DWOC related?

@Dhamodhar-DDR, Is this SWOC or DWOC related?

Not related to any of those.


Add docker compose file

Update to running docker with docker compose so that it is easier to run this container with others

done


Live into Master

Docs Build status updates of commit 402481b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/install-and-config-guides/post-configuration-steps-to-optimize-your-environment.md
:bulb:Suggestion
View
Details

biztalk/core/connecting-systems.md
:white_check_mark:Succeeded
View

biztalk/core/messages-represented-as-xlangmessage.md
:white_check_mark:Succeeded
View

biztalk/core/using-business-activity-monitoring.md
:white_check_mark:Succeeded
View

biztalk/core/walkthrough-as2-receiving-edi-over-as2-with-an-asynchronous-mdn.md
:white_check_mark:Succeeded
View

biztalk/core/walkthrough-consuming-wcf-services-with-the-wcf-basichttp-adapter.md
:white_check_mark:Succeeded
View

his/install-and-config-guides/what-s-new-in-his-2013.md
:white_check_mark:Succeeded
View

biztalk/install-and-config-guides/post-configuration-steps-to-optimize-your-environment.md

[Suggestion] Invalid value for ms.author: 'mandia,niklase' is not a valid Microsoft alias. NOTE: This Suggestion will be elevated to a Warning on 9/1/2019.

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.

Docs Build status updates of commit f99f68a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/connecting-systems.md
:white_check_mark:Succeeded
View

biztalk/core/messages-represented-as-xlangmessage.md
:white_check_mark:Succeeded
View

biztalk/core/using-business-activity-monitoring.md
:white_check_mark:Succeeded
View

biztalk/core/walkthrough-as2-receiving-edi-over-as2-with-an-asynchronous-mdn.md
:white_check_mark:Succeeded
View

biztalk/core/walkthrough-consuming-wcf-services-with-the-wcf-basichttp-adapter.md
:white_check_mark:Succeeded
View

biztalk/install-and-config-guides/post-configuration-steps-to-optimize-your-environment.md
:white_check_mark:Succeeded
View

his/install-and-config-guides/what-s-new-in-his-2013.md
:white_check_mark:Succeeded
View

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.


es6-promise overwrites zone.js

I am not sure exactly which library is exactly responsible for the issue, but if one looks at this line:
https://github.com/stefanpenner/es6-promise/blob/a5838bd94a76e31ad80ac1685bcd0dbc3d8ce963/dist/es6-promise.js#L1133-L1144
One can see that they are expecting
Object.prototype.toString.call(window.Promise.resolve()) to be [object Promise], but zone.js gives [object Object]
So, I am wondering, should es6-promise (coming from another widget hosted on our page served over their CDN) look for [object Object], or should zone.js have [object Promise] for this toString()

This seems to work, though not clean or reliable:
var oldToString = Object.prototype.toString;
Object.prototype.toString = function () {
    if (this instanceof Promise) {
        return '[object Promise]';
    }
    return oldToString.call(this);
};

Maybe this could be added to zone.js so other polyfills won't overwrite?

Or perhaps Zone.js could make window.Promise a setter function, and anytime it is set, Zone.js wraps with ZoneAwarePromise? So then if any library were to set window.Promise, it could reject it, wrap it, or at least warn?  Just brainstorming here

@josephliccini , it seems duplicate with #499, I will check the behavior later.

This one seems fixed! Thanks!
There is one issue I am seeing, but it's not affecting me but may be good for you to know.
I did a test to see what would happen with core-js Promise.
My code has this now:
import 'zone.js';
import 'core-js/es6/promise';

And before bootstrap I get this error:
Uncaught TypeError: NativePromise.resolve is not a function
    at Object.setNativePromise (zone.js:634)
    at desc.set (zone.js:1028)
    at webpackJsonp.../../../../node_modules/core-js/modules/_redefine.js.module.exports (_redefine.js:19)
    at $export (_export.js:27)
    at Object.../../../../node_modules/core-js/modules/es6.promise.js (es6.promise.js:230)
    at __webpack_require__ (?mockauth=2:61)
    at Object.../../../../node_modules/core-js/es6/promise.js (promise.js:4)
    at __webpack_require__ (?mockauth=2:61)
    at Object../vendor.ts (zone.js:3026)
    at __webpack_require__ (?mockauth=2:61)

@josephliccini , thanks for the information, I will check this error.

@JiaLiPassion  thanks I am excited to try in next release :)

Our app is not working on IE11 due to this issue, using version 0.8.18.
Waiting desperately for a fixed version!

any update on this?

@goyalvarun601 , this one have been merged, please wait for the next release.

thanks @JiaLiPassion , when is the next release?

@goyalvarun601 , I am not sure, please wait for a while.

I am also interested in when this is resolved.

The same is at my side. What is the approximate date for a new release ?

for now, please update your package.json with
    "zone.js": "https://github.com/angular/zone.js.git"

next release? when's that

I am still getting the same error using "https://github.com/angular/zone.js.git".
zone.js:634 Uncaught TypeError: NativePromise.resolve is not a function
    at Object.setNativePromise (zone.js:634)

Did it solve anyone else's issue?

@goyalvarun601 , sorry,  the zone.js/dist/zone.js in this repository is not updated, please used the attached version.
zone.zip

The zip doesn't have the updated dist/zone.js

@goyalvarun601 , you are right, I used the old version, please use this one.
zone.zip

thanks @JiaLiPassion

@goyalvarun601 , sure, you can use my branch, https://github.com/JiaLiPassion/zone.js/tree/some-fix

@JiaLiPassion The branch doesn't have the updated dist/zone.js

@goyalvarun601 , please use this one, https://github.com/JiaLiPassion/zone.js/tree/issue-891

Thank you. This works.

thanks @JiaLiPassion it works also for me
=> any idea when it will be merged in zone.js to have an official new version ?

@jdespatis , it has been merged, but I am not sure when there will be a new version.

nice news @JiaLiPassion , but strangely, if I use your branch in my package.json:
"zone.js": "https://github.com/JiaLiPassion/zone.js.git#issue-891",
all is running well,
but when using zone.js master branch, the problem with NativePromise still exists
my package-lock.json indicates the very last commit on zone.js:
"zone.js": {
"version": "git+https://github.com/angular/zone.js.git#326a07fb9095e4a87ff7561cb45fe1d4917c2174"
}
Maybe I'm doing something wrong, or maybe your branch contains more than the fix committed in zone.js ?

@jdespatis , the content are the same between my branch and master, the reason master not work because dist folder is not updated in master branch, it will only be updated when new version released.

ok thanks for your feedback @JiaLiPassion !


All command line flags can go at the end of the line (fix #261).

bibliopixel demo bloom --numpy --loglevel=info with all the flags at the
end is now the officially correct way to use flags.

(I changed the documentation to say this.  I doubt anyone will run into this before we push this change to master, and if they complain, we'll push this change to master.)

They only quibble I have here is that the title of the PR should be ".. must go at the end of the line" instead of "can". lol
Otherwise +1

Well, "must" isn't strictly true, which is why I waffled.
I think we should tell people that "they must go to the end" because that
works, but in fact,
bibliopixel demo --numpy bloom --loglevel=info
also works.
What doesn't work is  bibliopixel --numpy demo bloom --loglevel=info
So the real rule is "command line flags have to come after the BiblioPixel
command".  Perhaps that would be a better commit description?
On Mon, Jul 24, 2017 at 12:54 PM, Adam Haile<EMAIL_ADDRESS>wrote:

They only quibble I have here is that the title of the PR should be "..
must go at the end of the line" instead of "can". lol
Otherwise +1
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/pull/309#issuecomment-317387657,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdsoDTYzknh4OJbMaWDw4FRAmca9Fiks5sRHfggaJpZM4OhBKc
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com

Agreed.
On Mon, Jul 24, 2017 at 7:08 AM, Tom Swirly<EMAIL_ADDRESS>wrote:

Well, "must" isn't strictly true, which is why I waffled.
I think we should tell people that "they must go to the end" because that
works, but in fact,
bibliopixel demo --numpy bloom --loglevel=info
also works.
What doesn't work is bibliopixel --numpy demo bloom --loglevel=info
So the real rule is "command line flags have to come after the BiblioPixel
command". Perhaps that would be a better commit description?
On Mon, Jul 24, 2017 at 12:54 PM, Adam Haile<EMAIL_ADDRESS>wrote:

They only quibble I have here is that the title of the PR should be "..
must go at the end of the line" instead of "can". lol
Otherwise +1
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/pull/
309#issuecomment-317387657>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAPdsoDTYzknh4OJbMaWDw4FRAmca9Fiks5sRHfggaJpZM4OhBKc>
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/pull/309#issuecomment-317390410,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6pugZ8pGIb_dL0EK9Kb6LOB6Ws9Nks5sRHsWgaJpZM4OhBKc
.


[BUG]

Cannot import C:\Users\Z0Z00FICIAL\ComfyUI_windows_portable_nvidia_cu118_or_cpu\ComfyUI_windows_portable\ComfyUI\custom_nodes\prompt-generator-comfyui module for custom nodes: Command '.\python_embeded\python.exe -s -m pip install optimum[onnxruntime-gpu]' returned non-zero exit status 1.
Operating System
Windows 11 Pro
Python Version
Python 3.10.6 (tags/v3.10.6:9c7b4bd, Aug  1 2022, 21:53:49) [MSC v.1932 64 bit (AMD64)] on win32

To append.
Cannot import C:\Users\Z0Z00FICIAL\ComfyUI_windows_portable_nvidia_cu118_or_cpu\ComfyUI_windows_portable\ComfyUI\custom_nodes\prompt-generator-comfyui module for custom nodes: No module named 'transformers.models.gpt_bigcode'

Hello, thanks for reporting. I reinstalled and tried with the new release of portable ComfyUI and it seems working. I am going to write the steps that I take below:
I tested it with Windows 11 and Python 3.10.6
prompt-generator-comfyui hash => f577714
ComfyUI portable release => which is released in the first week of October 2023

Go to Release section of the ComfyUI
Download the latest version of the ComfyUI_windows_portable_nvidia_cu118_or_cpu.7z file. direct link of the file
Extract the zip file
Get in the ComfyUI_windows_portable folder
Check if ComfyUI is running without custom nodes with running run_nvidia_gpu.bat file (It is)
Go to ComfyUI/custom_nodes folder
Clone the repository with https://github.com/alpertunga-bile/prompt-generator-comfyui.git command
Go back to ComfyUI_windows_portable folder
Run the run_nvidia_gpu.bat file
Open the basicWorkflowWithPromptGenerator.json workflow which is located in the ComfyUI/custom_nodes/promp-generator-comfyui folder
Put the text prompt generator model under the ComfyUI/models/prompt_generators folder
Refresh the ComfyUI and test if it is running

If the model is not recognized even after clicking refresh button, please restart the ComfyUI

I am sorry that I can't reproduce the problem you encounter and provide a solution for that. You may want to check which folder you start the ComfyUI because the installation script that I wrote checks if python_embeded folder is present. You have to run it from the ComfyUI_windows_portable folder to work properly.
If you encounter any problems after following these steps let me know.

To append.
Cannot import C:\Users\Z0Z00FICIAL\ComfyUI_windows_portable_nvidia_cu118_or_cpu\ComfyUI_windows_portable\ComfyUI\custom_nodes\prompt-generator-comfyui module for custom nodes: No module named 'transformers.models.gpt_bigcode'

I solved it by doing a "pip uninstall transformers" and "pip install transformers"

Hello, thank you (@rauldlnx10) for providing a solution for the problem. I assume that an issue is fixed in the new version of the transformers package.
As a note @rauldlnx10, you can use --upgrade option with pip to update a package. With this option you don't have to uninstall and install the package again. You can check this stackoverflow post. I am providing this solution because I think the problem is not about the package integrity but the package version.

For portable version users the command is
.\python_embeded\python.exe -s -m pip install --upgrade transformers
For manual installation users the command is
pip install --upgrade transformers

If the user is using virtual environment, the user has to start the virtual environment before executing the command.


metal-cli: 0.14.1 -> 0.15.0

Description of changes
https://github.com/equinix/metal-cli/releases/tag/v0.15.0

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Works on my x86 mac too.


Display tool version(commit id) for tools.

Currently there is no way to tell what version of tool is being used.
It would be good to display tool version(commit id) for tools.

@sangee2004 is this covered by the version api?

@cjellick Yes this is now presented in http://localhost:8080/api/version
{
  "chrome": "Chromium 130.0.6723.31",
  "emailDomain": "",
  "github.com/gptscript-ai/datasets": "31ce6c3c682b79c190d58d3e74793340ac7548ed",
  "github.com/gptscript-ai/workspace-provider": "b70ef6fceaca4da5699d8709e7436b22258820bc",
  "github.com/kubernetes-sigs/aws-encryption-provider": "5ea85bbe1c6811bfb16be6a3a06d4d06c0a48d7c",
  "github.com/otto8-ai/tools": "9f8787a621c988d8e215d173610558e6acae7c01",
  "otto": "v0.0.0-dev+36ab3bd4"
}


How to set initial value of ValidatedInput type text?

For the update case with a value on this.state, how can it be the default value?  Thx!

+1, i have the same issue, please help

i just found out how to do that, it's simply using model parameter in the form

When i am loading page where the values are fetched from databse the model property is set up after that, but the form is not rerendered..so the input fields is not filled with values.. some render force?

Is any way how to change model of form to refresh with new values?


Terraform

Tweaked a couple things based off of Jennie's comments on the last PR#22, but I didn't know how (or if it's even possible) to merge that into an open PR, so I just closed it, pushed the changes, and opened a new PR. If there's a better way to do that, I'd love to know!

Are the test datafiles used in this PR? They seem out of place. Maybe move to a different PR?

@mayaspivak  For this build they're used to load the BQ tables and the SQL query won't work without them. It's intended to be a temporary placeholder so there's a structure to build off of, and will be deleted and redirected to the dynamically generated tables once the pipelines are in place.

@mayaspivak  Okay I'm still learning how to github, wasn't sure how to comment directly on your change request, so hit the "re-request review" button but not sure if that's the right workflow.

Tweaked a couple things based off of Jennie's comments on the last PR#22, but I didn't know how (or if it's even possible) to merge that into an open PR, so I just closed it, pushed the changes, and opened a new PR. If there's a better way to do that, I'd love to know!

If you commit and push your changes, they should be automatically added to the PR so you can just hit 're-request review'
(found the answer from this article: https://homes.cs.washington.edu/~mernst/advice/github-pull-request.html#respond-to-reviews)

Tweaked a couple things based off of Jennie's comments on the last PR#22, but I didn't know how (or if it's even possible) to merge that into an open PR, so I just closed it, pushed the changes, and opened a new PR. If there's a better way to do that, I'd love to know!

If you commit and push your changes, they should be automatically added to the PR so you can just hit 're-request review'
(found the answer from this article: https://homes.cs.washington.edu/~mernst/advice/github-pull-request.html#respond-to-reviews)

Ah okay, that's helpful to know. And also makes WAY more sense. Also I've been using PRs incorrectly (when to use them and how to logically group changes and such) so good to know to make them more readable going forward. Thanks!


Use Python's unittest ability to detect tests

LGTM if that also handles the twisted tests .

Seems to be causing 3 of the 4 Travis build bots to stall (it seems "make -C python test" is not terminating on those bots, for no obvious reason). Works fine on 1 of the bots though, strangely.


modules/iam-assumable-role-with-oidc: Support multiple provider URLs

This allows using one role with multiple OIDC providers, for example
when you want to use a single role across multiple EKS clusters.
Deprecates provider_url in favor of provider_urls which allows
specifying more than 1 provider URL. Existing behavior is retained, and
both options can be specified.

Looks good to me, if you could rebase it on top of recent changes in the master branch.

Hah, gotta love when your own PRs conflict with each other. Rebased.

Please try to run example to confirm that it works as expected (now it does not, because rebase went wrong).

Yeah, I had to fix a few things to make it work. In particular adjusting of the conditions for_each was new.

Thanks, @chancez !
v2.18.0 has been just released.


Http client

Hi, is there an http client for chez? I haven't been able to find anything.
Sorry to open an issue but I couldn't find a better way to contact you and you seem to know more about chez than anyone.

I have the same problem, did you find a solution @shmish111?

I did not, sorry

no problem, I'll keep looking. if I find something I'll let you know 🙂


Handle case were query does not have a default property

Fixes #81

Looks great, thank you so much!

Published as v1.0.2!


Standardize NeoVim to Neovim

I realized we have some entries as NeoVim.
This MR changes them to Neovim.
Checklist:

[x] The plugin is specifically built for Neovim.
[x] The lines end with a .. This is to conform to awesome-list linting and requirements.
[x] It's not already on the list.
[x] If it's a colorscheme, it supports treesitter syntax.
[x] The title of the pull request is Add `username/repo`  when adding a new plugin.
[x] Neovim is spelled as Neovim (not nvim, NeoVim or neovim), Lua is spelled as Lua (capitalized).

I think we should respect those intros since they are from the original repos' "About" sections.

Thanks for the PR!


Add StructArray Constructors (#3879)

Which issue does this PR close?

Part of #3879
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

I've added tests, well re-used the existing tests :smile:


Add tooltip to relationships and ability to navigate to the location at either end

It would be useful to know the names and types of resources/locations at the end of a link through a tooltip. It would also be useful to have the ability to navigate to either end of the relationship and focus on it from the tooltip given that they may be geographically far apart and outside of the user's viewport.

This is a nice idea, if you have time to contribute a PR with a possible solution, we can look to add that support.

Project is being archived


RESTCONF server implement

Hi all,
We are from intel sweetcomb team. We have developed a RESTCONF server and it covers basic requirements of RESTCONF(RFC-8040) . Some detailed information:

delighted by Clixon, use Nginx as web server and together with fastCGI to respond the requests
use sysrepo as datastore
use libyang to deal with yang
use libnetconf2 for error report and other trivias
function implement is similar to netopeer, RESTCONF server will parse request and dispatch to NETCONF rpcs to process

You can view code on gerrit: https://gerrit.fd.io/r/c/sweetcomb/+/21580
We think this RESTCONF server implement is more close to Netopeer project, and ** could you please consider merging it to your repository? **
Best regards,
Hongcheng

Hi,
thank you for your contribution, we will take a look at the server and decide whether it can be merged into our repository.
Regards,
Michal

Hi,
thank you for your contribution, we will take a look at the server and decide whether it can be merged into our repository.
Regards,
Michal

Thanks!

Hi,
so I have successfully tried your implementation and, generally, we think we could cooperate. But we have a few questions.
Firstly, when I did not have the target module enabled and the error on restconf_methods.c:248 was being executed, I have encounter a crash because i on restconf_methods.c:249 is not initialized. That immediately points to immaturity of the server and lack of tests. So, my question is, will you continue your work and add features, fix bugs, and add tests?
Another question connected to the previous one, if it was merged, would you use our repository as the main one for your work or just merge some bigger changes/new releases?
Secondly, we are in the process of migrating from the old sysrepo in master (version 0.7.8 and lower) to the new one in devel (version 1.0 and higher). There is not much point in developing your RESTCONF server more while it uses the old sysrepo and you should migrate to the new one. One note for when you do, it no longer includes NACM and it was implemented directly in netopeer2-server. I have also noticed you have reused a lot of code from it so it may be a good idea to share the same code directly for both servers, which should make the maintenance easier.
Regards,
Michal

Hi Michal,
Thanks for your reply and I am glad that we could cooperate. I will answer your questions in my position first. And after sweetcomb meeting on Thursday I will give you our community's decision.

I'm sorry that the code is lack of tests indeedly. (In my developing environment maybe compiler fix this bug for me.)
I will continue to contribute on this restconf development. And there will be other colleague in Intel join the maintenance.
I think we will use netopeer2 repository as the main one. I will verify this point on Thursday.
We will migrate to new sysrepo definitely.

Best regards,
Hongcheng

Okay, so our position is basically that once you migrate to the new sysrepo and reuse netopeer2-server code where it makes sense, we will be happy to merge it. Let us know what you decide on the meeting.
Regards,
Michal

Hi Michal,
So I tried to compile netopeer2 in devel-server branch. And I met some dependence problems that I don't know which libyang branch or libnetconf2 branch it uses. Also, the sysrepo haven't release the new version 1.0 (and there's a lot of commits these days).
I think maybe I could start to merge RESTCONF server to new sysrepo after sysrepo v1.0 releases.
What do you think?
Best regards,
Hongcheng

Hi,
to compile netopeer2-server from devel-server, you need libnetconf2 devel, and sysrepo devel. You are right that the new sysrepo has not been released yet but it is finished and usable.
If you want to wait with the migration it is up to you.
Regards,
Michal

Hi
Just checking if there was any conclusion made on this topic. I am also implementing a restconf server that works with sysrepo. I would like to know if sysrepo latest release happened and @spartazhc was able to merge.
@spartazhc do you have the latest drop of your SW.
Thanks

Hi @michalvasko @spartazhc @balag1986
Could you please share if you have any update on this. I would like to give it a try.
Thanks & Regards,
Sriram.K

Hi,
I have not heard from the authors of that server since the last reply here so I have no information to share.
Regards,
Michal

Hi Michal,
Thank you for the information.
Thanks & Regards,
Sriram.K

hi,
I think @spartazhc is not implments the "RESTCONF"  which indicated by "https://tools.ietf.org/html/rfc8040   RESTCONF Protocol" .
it is like a converted RESTful API for "NETCONF" protocol.
it is even do not touch yang modules<EMAIL_ADDRESS>and "ietf-restconf-monitoring"..
Please kindly have a check for this..
Thanks.

hi,
I am sorry that I do not have further work on this any longer. And I know that is an very immature implement. Sorry for this.
Thanks

I do not know about any usable RESTCONF servers built on sysrepo so I cannot help you.


Gracefully handle required missing retire arguments

Closes #253

Add helper function in rebar3_hex_retire that gets a required param or raises
add basic tests for retire in integration suite
add post retirement route support in hex api model

Any chance of a point release with this fix in it? :-D

@oubiwann Yup, v6.11.6 released.


Change color on community when hovering

Description
In this part of the page :

According to the Figma file the color should change to purple oh hover:

Screenshots
No response
Additional information
No response

I will work on it


org1peer1, org2peer1 show "waiting for bootstrap"

Am attempting to use the latest version of your wonderful code to set up a fabric for composer on a paid IBM cluster. After upgrading this morning to the latest master, cannot join channel fails and org1peer1 and org2 peer1 both show 'waiting for bootstrap' as the only entry in the log files.

Note, after a little testing, this error only occurs if I use the --paid option on the create_all.sh script.
This always fails: ./create_all.sh --with-couchdb --paid
While this appears to work: ./create_all.sh --with-couchdb

closing this one. I had not added the --include-volumes option on delete-all and old persistent storage was messing up the start process.


feat: Used version number in deploy steps

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


SRAM2 wrong on certain stm32h7 SOC (system crashes during startup)

https://github.com/zephyrproject-rtos/zephyr/pull/54593


Resolve issue with Prisma by providing a way to pass --allow-scripts

I'm trying to get Prisma set up in my new Fresh project. I imitated this repo and it works locally, but I run into issues when deploying to Deno Deploy. More specifically, when deno task build runs, this warning shows up in my GitHub Action logs:
Warning The following packages contained npm lifecycle scripts (preinstall/install/postinstall) that were not executed:
┠─<EMAIL_ADDRESS>┠─ npm<EMAIL_ADDRESS>┠─ npm<EMAIL_ADDRESS>┃
┠─ This may cause the packages to not work correctly.
┖─ To run lifecycle scripts, use the `--allow-scripts` flag with `deno install`:
   deno install --allow-scripts=npm:prisma@6.0.0,npm:@prisma/client@6.0.0,npm<EMAIL_ADDRESS>
followed by this error:
error: Uncaught (in promise) TypeError: Module not found "file:///home/runner/work/urban-admin/urban-admin/prisma/generated/client/index.d.ts".
    at file:///home/runner/work/urban-admin/urban-admin/prisma/client.ts:8:15
  const manifest = (await import(toFileUrl(join(dir, "fresh.gen.ts")).href))
                    ^
    at async dev (https://deno.land/x/fresh@1.7.3/src/dev/dev_command.ts:38:21)
    at async file:///home/runner/work/urban-admin/urban-admin/dev.ts:8:1
Error: Process completed with exit code 1.

This makes sense. The @prisma/client NPM package has a postinstall hook that generates the Prisma client. Such hooks won't run with deno install unless you have the --allow-scripts flag. I can't tell where deno install is actually being triggered, but given the log output, it appears that the --allow-scripts flag isn't being used.
To see where deno install might be being triggered, my reasoning is as follows. The issue comes during the "Build step" step of the "Deploy" job, which runs deno task build. From deno.json we can see that deno task build executes deno run -A dev.ts build. I didn't touch dev.ts. Here's what it looks like:
#!/usr/bin/env -S deno run -A --watch=static/,routes/

import dev from "$fresh/dev.ts";
import config from "./fresh.config.ts";

import "$std/dotenv/load.ts";

await dev(import.meta.url, "./main.ts", config);

From here I'm not sure what's going on, nor how I can find deno install and add --allow-scripts. I can see that dev is being called but I can't find any docs for dev. I tried digging into main.ts and config, but couldn't really figure out what's going on there or whether there's any tweaks I can make in either of those places to resolve this issue.
To support Prisma and other NPM packages with postinstall hooks, I think Fresh should provide and/or document a way to allow such scripts to run.

Deno by default will try to install any missing dependencies when running deno run. As you discovered it won't run npm postinstall scripts automatically though because they could contain malicious code in theory. To execute these postinstall scripts you need to run deno install --allow-scripts explicitly. You can do this by putting it before the deno task build command in your GH actions workflow.
  name: Deploy
  on:
    push:
      branches: [main]
    pull_request:
      branches: main

  jobs:
    deploy:
      name: Deploy
      runs-on: ubuntu-latest

      permissions:
        id-token: write # Needed for auth with Deno Deploy
        contents: read # Needed to clone the repository

      steps:
        - name: Clone repository
          uses: actions/checkout@v4

        - name: Install Deno
          uses: denoland/setup-deno@v1
          with:
            deno-version: rc

+      - name: Install dependencies
+        run: deno install --allow-scripts=npm:prisma@6.0.0,npm:@prisma/client@6.0.0,npm<EMAIL_ADDRESS>
        - name: Build step
          run: "deno task build" # 📝 Update the build command(s) if necessary

        - name: Upload to Deno Deploy
          uses: denoland/deployctl@v1
          with:
            project: "fresh" # 📝 Update the deploy project name if necessary
            entrypoint: "./main.ts" # 📝 Update the entrypoint if necessary
            root: "."

Since Prisma is on 6.0.1 now I added:
- name: Install dependencies
  run: deno install --allow-scripts=npm:prisma@6.0.1,npm:@prisma/client@6.0.1,npm<EMAIL_ADDRESS>
before the "Build step" step. That successfully resolved the postinstall issue but lead to the following error:
error: Uncaught (in promise) Error: Cannot find module './generated/client/index.cjs'
Require stack:
- /home/runner/work/urban-admin/urban-admin/prisma/client.ts
    at Function.Module._resolveFilename (node:module:619:15)
    at Function.Module._load (node:module:497:27)
    at Module.require (node:module:681:19)
    at require (node:module:812:16)
    at file:///home/runner/work/urban-admin/urban-admin/prisma/client.ts:5:16
Error: Process completed with exit code 1.

That error is happening because in following this setup, the standard prisma generate command won't work. We need to run this custom prisma:generate command instead. So I tried updating the "Install dependencies task" and adding a "Generate Prisma client" step before the "Build step" step like so:
- name: Install dependencies
  run: deno install

- name: Generate Prisma client
  run: deno task prisma:generate

- name: Build step
  run: "deno task build"

That leads to the following error in the "Upload to Deno Deploy" step:
Error: The deployment failed: BOOT_FAILURE

Uncaught SyntaxError: Export 'Extensions' is not defined in module
    at file:///src/prisma/generated/client/runtime/library.d.ts:1338:10

I haven't made any notable changes to the "Upload to Deno Deploy" step:
- name: Upload to Deno Deploy
  uses: denoland/deployctl@v1
  with:
    project: "urban-admin"
    entrypoint: "main.ts"
    root: "."

At first when I submitted this GitHub Issue I was thinking that it might be an issue with Fresh, either 1) because there isn't a way to pass --allow-scripts or 2) because there is a way to do so but it's not documented. I now see that (1) is not an issue. I'm not sure if (2) is an issue or not; I could see arguments on both sides.
Anyway, I'm not sure if my current issue is really related to Fresh. It probably isn't. And so I think it'd be understandable to want to close this issue. On the other hand, Prisma is a popular ORM, I expect that people trying to use Prisma with Fresh will run into issues similar to the ones I'm running into, and that this conversation will help those people. For that reason I feel like it'd be worth keeping this issue open and working to resolve it, but I don't feel strongly.