Crashing during export on 1.0.2

I ran the extension on 1.0.2 unpacked in developer mode, and cannot export my balances without the extension crashing. I also get an error from mint.com until I close and re-open chrome saying mint.com is unavailable. Wonder if mint is rate limiting? I can usually get to about 40 accounts before it crashes.
I have 205 total accounts

Copying the errors I saw here from chrome developer mode:
Error 1
Access to fetch at 'https://mint.intuit.com/pfm/v1/user' from origin 'chrome-extension://jekpaapjbcmkbilkckepljakpndlpepp' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
Context
src/pages/popup/index.html
Stack Trace
src/pages/popup/index.html:0 (anonymous function)
Nothing to see here, move along.
Error 2:
Unchecked runtime.lastError: A listener indicated an asynchronous response by returning true, but the message channel closed before a response was received
Context
src/pages/popup/index.html
Stack Trace
src/pages/popup/index.html:0 (anonymous function)
Nothing to see here, move along.

Thanks for reporting this. Can you test if this error still happens with https://github.com/monarchmoney/mint-export-extension/releases/tag/1.0.3?

issue persists on 1.0.3. Got to about 40 accounts before the extension crashed. reloading the page shows me:

It seems that the issue may be related to rate limiting, as you mentioned. Could you try changing the value of MINT_RATE_LIMIT_DELAY_MS locally and test to see if it resolves the problem?

I'm actually not sure how to do that - I'm just savvy enough to download and post to GitHub, but im not sure where or how to change this value. If you can direct me to what file it's in I can try and edit it? Do I need to edit the value and then manually rebuild the package myself?

1.0.5 looks to have fixed the issue, I was able to download balance history for 52 accounts.

Yeah 1.0.5 also just fixed it for me - noticed the rate was much slower too. Only other item (maybe this is already a feature request) is that while it reported some accounts could not download, I have no way of knowing which ones these were. I presume these were probably duplicate or hidden accounts with no data to download, but it would be great if the app could report out a list of what accounts these were so I could confirm.

I found the failed downloads were directly tied to having the "Hide Accounts" set to something other than "Show Everywhere". Changed my accounts to "Show Everywhere" and no more failures!

I'm still having this problem, I have 210 accounts.

I'm still having this problem, I have 210 accounts.

Did you try switching them to "show everywhere" in the "hide accounts" tab?

I'm still having this problem, I have 210 accounts.

Did you try switching them to "show everywhere" in the "hide accounts" tab?

Yep, all of my accounts are set to "show everywhere".  Just tried it again, this time it crashed after about 40 accounts.

mschmidty4

Just to clarify.  My issue is the original issue listed in this report, where you get the "Oops, we ran into an error" webpage, and can't log back into Mint for 10 minutes.  Not the "xx accounts failed to download" error.


remove BusinessUnitListener from configuration

it causes fail of console app after application installation and moreover did not exist

Thank you for your pull request.
It looks like this may be your first contribution to an Oro, Inc. open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at http://www.orocrm.com/contributor-license-agreement/
If you've already signed the CLA, it's possible we don't have your GitHub username or you're using a different email address. GitHub uses the email address you set in your local Git configuration to associate commits with your GitHub account. Please sign the CLA again using the correct GitHub username and email address or see this help article on setting the email on your git commits.
Once you've signed the CLA, please allow for some time for the submission to be processed.

Hi @andreyserdjuk
I have merged this PR to the master branch. Thank you!


Update finance_transaction_purchase_order.sql

This update is being submitted on behalf of Joanne Leary. It corrects the PO line acquisition method and PO order type data field extractions from the JSON data array in the finance_transaction_purchase_order derived table for Metadb. It also adds the acquisition method name to the table. The finance_transaction_purchase_order is already included in runlist.txt.

This pull request closes Issue #847.

Irina,
Thanks for reviewing, and for catching that error. I have just updated "Should not change once create" to "Should not change once created" in this PR.


Error w/ Upload Name Fuse.py

I'm able to use fuse.py to fuse my LoRA--this command works and creates a new folder with weights, tokenizer, config, etc.  E.g.:
python fuse.py --model /Users/williammarcellino/mlx-examples/lora/mlx_model --save-path /Users/williammarcellino/mlx-examples/lora/Fused_Models --adapter-file /Users/williammarcellino/mlx-examples/lora/adapters_GrkRom_1640Iter.npz
However when I add --upload-name, e.g.:
python fuse.py --model /Users/williammarcellino/mlx-examples/lora/mlx_model --save-path /Users/williammarcellino/mlx-examples/lora/Fused_Models --adapter-file /Users/williammarcellino/mlx-examples/lora/adapters_GrkRom_1640Iter.npz/--upload-name mistral-7b-v0.1-GreeceRome-v0.1
I get the error "fuse.py: error: unrecognized arguments: mistral-7b-v0.1-GreeceRome-v0.1"
I was assuming that for [--upload-name UPLOAD_NAME] you replaced UPLOAD_NAME with whatever you wanted to name the new model, but maybe that's wrong?

I think you're just missing a space in your command e.g. 1640Iter.npz/--upload-name should be 1640Iter.npz/ --upload-name.


Create Modal Component

Overview
Create a reusable <Modal/> component that we can use throughout our apps. We may need additional design support on this -- I didn't see a specific Modal in the Figma.
Deliverables

[ ] <Modal/> component added to the haus-components package
[ ] Story added for the component

Details

Figma component can be found here: https://www.figma.com/file/R1a1bwODnzCHHHw0EJALAU/DAOhaus-v3-Pattern-Library-(In-Progress)?node-id=895%3A91491

Thanks!

@7helles Any info around the background overlay for the modal?


Revert "[flutter_tools]: reland move engine unpack to dart"

Reverts flutter/flutter#52543
TBR @christopherfujino
This broke https://cirrus-ci.com/task/5947451991392256 , might have missed last time due to the merge error

lgtm


KDE packages

Chris, please add these essential pacman packages to make KDE more complete...
karchive
kauth
kcompletion
kdialog
kgamma5
khotkeys
kwalletmanager
kwayland-integration
kwrited

I recognize the benefit in installing a couple of these, for sure. I'll create a PR to get a few added.
But, I'm seeing several of these installed on a fresh installation using the latest test branch:

karchive
kauth
kcompletion

Nice. I didn't see these ones in any of the package lists

I didn't see them either so they must be installed as a dependency of something.

Ah ha, nice


Fix broken sticky header in details view

Resolved / Related Issues
Items resolved / related issues by this PR.

Related #4643
Closes #5360 (scrollbar now hidden until hovered over or horizontal scrolling is invoked)

Details of Changes
Add details of changes here.

Fix issue where the header wouldn't always show when scrolling vertically
Update header buttons to use winui 2.6 styling
Update column sizes when the view area is resized

Validation
How did you test these changes?

[x] Built and ran the app
[ ] Tested the changes for accessibility

Screenshots (optional)

@winston-de It looks like the header has an alignment issue in this PR.

Thanks, this should be fixed now.

@winston-de Now it looks like there is a little too much space for the icon column.

@winston-de Now it looks like there is a little too much space for the icon column.

Can you send a screenshot please? Seemingly it works on my machine :tm:.


Download Only Mode

Please include a feature to just download only and not preview files within the GDIndex. Sometimes, videos are not played due to H265 and it's better to just download those rather than going to the preview page.

Hey,
There is download button on bottom right corner, or you can just remove ?a=view from link!
Thanks


feat: Rework deployment logging to have section headers

Description of changes:
For the log output during a deployment write section headers along with descriptions that explain what is happening at this stage of deployment.
Here is what the headers look like using the CLI.

For the VS toolkit there is a new message sent through the SignalR hub that the toolkit can use to get notified when new sections are started. The toolkit can either add the section headers in the output window or create category based UI control.
Notes for PR

Renamed the logging methods on IOrchestratorInteractiveService and the SignalR client IDeploymentCommunicationHub to make the names consisted. I have discussed this with the IDE team so and they are fine adapting to the change when the adopt the new version.
On SignalR client removed the ReceiveLogMessageLineAction callback. This also meant the toolkit was displaying debug messages and it didn't make sense to add the new header names in this callback. Toolkit will update to add callback to the Info and Error messages.
Fix up the deploy to existing beanstalk integration test to make sure the testing stack is deployed to us-west-2 since the integration test that look for the existing environment was failing on my machine which didn't have us-west-2 as the default region.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Current behavior will stream the headers as they come in so in terms of UI, the toolkit will add new sections adhoc which wouldn't give the user a sense of what the overall process is going to be like. This doesn't have to be part of this PR, but it's worth thinking of a way to outline major sections like Configuration, Deployment Bundle, Deployment (things that we know will happen for sure regardless of recipe) and the rest of the recipe dependent headers will be streamed in.

Yes, Chris and I have discussed this. But since our system is so dynamic we don't currently have a way in our system to precompute all of the stages. That can be something we possibly do in the future.


No paste toolbar button (or copy) ?

There doesn't appear to be a paste or copy toolbar button.
The playground doesn't have one either.
https://xdsoft.net/jodit/play.html
Seems like an obvious missing feature?

@xdan any suggestions about how to implement paste, I found editor.execCommand('copy') however paste doesn't work, not sure why.
Seems odd to have cut and not copy and paste by default.
Thanks for this repo.

I can't see mention of Paste in the PR ??

@JMWC2019 In the new version added copy and paste buttons


Version 1.8

Hello @nicolaslopezj ,
I was intending to ask this question a while back, but as the updates for 1.8 are undergo would it be possible to add grouping to the collections. For example a user should be able to pass an array through the register process of an orion collection. Possibly separate this through tabs like how the dictionary does, or through an accordion. Items that are not assigned to a group will simply render like they typically would, or can be assigned to a default group of some sort. This would allow users that have large schemas the ability to organize the form a little better. vs. having an extremely large form of data.
Something like this :
Articles = new orion.collection('articles', {
    singularName: 'Article',
    pluralName: 'Articles',
    title: 'Articles',
    link: {
        title: 'Articles'
    },
    tabular: {
        columns: [{
                data: 'title',
                title: 'Article Title'
            }, {
                data: 'published',
                title: 'Published'
            },
            orion.attributeColumn('image', 'image', 'Image'),
            orion.attributeColumn('froala', 'content', 'Article Preview'),
            orion.attributeColumn('createdBy', 'createdBy', 'Created By'),
            orion.attributeColumn('createdAt', 'createdAt', 'Created At')

        ]
    },
	groups: [
		{name: 'admin', label: 'Admin', fields: ['published', 'slug']},
		{name: 'content', label: 'Content', fields: ['content', 'image']}
	]
});


shouldRespond handler for twitter interactions

Right now twitter responds to every message in every thread, which is annoying some people and unnecessary. We should have the shouldRespond handler so Spartan and others check if they should respond before forcing a response. We do this in the other clients, so it should be easy.

@dreaminglucid @alextitonis the code is right but it just kind of sucks in implementation a bit... always responds. I'd like to have it like posts instead, maybe? Or have the option of replying OR liking, and indicate in the prompt that it should reply, like or like and reply.


Put tags/categories to Blog pages

What would you like to be added:
Tag or/and categorize Blog pages so that users can easily search.
Why is this needed:
Currently, https://pipecd.dev/blog/ contains some kinds of articles (e.g. announcements, best practices)
Users cannot easily find the pages they're looking for.

[FYI] We can use Hugo's feature for them.
for example:
---
date: 2023-02-07
title: "PipeCD best practice 02 - control plane on ECS"
...
categories: ["best-practice"]
tags: [ "control-plane", "AWS", "config" ]
---

Categories and Tags will be shown like this:


Avalonia MapControl Hangs App if no Internet Connection

Mapsui Version
4.1.0
Mapsui Platform
Avalonia 11.1
Device
Windows 10 Pro
Describe the bug
If no internet connection is available to load map tiles, the MapControl will fail to initialize properly and essentially hangs the application.
To Reproduce
TBD
Expected behavior
This situation should fail gracefully and the MapControl should just be non-functional or not show with a blank map layer. It can retry behind the scenes but should never cause an application hang.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
This is worked-around on the app side for now by pinging for an internet connection before instantiating the MapControl. This is inherently unreliable though and is best handled within Mapsui or Brutile itself.

I don't see this if I start the Avalonia desktop sample on Windows without internet. If can still use the app, switch samples, and if I open for instance the Atlas sample I can see all the items and can pan and zoom.
Could it be related to some of the layers you use? Can you selectively turn layers off the see if the problem remains?

@pauldendulk Thanks for taking a look at this.
It was a good idea to check the layers. The offending layer is the Bing maps tile layer:
            var tileLayer = new Mapsui.Tiling.Layers.TileLayer(
                BruTile.Predefined.KnownTileSources.Create(
                    BruTile.Predefined.KnownTileSource.BingRoads,
                    ApiKeys.BingMapsKey));
            map.Layers.Add(tileLayer);

When this is commented out the application doesn't hang. I should also add it appears it is caught in a polling loop trying to download tiles. Sometimes it's possible to click away from the control so the application hasn't completely hanged/blocked the UIThread. It is just doing something that takes all resources.
Here is the full init code is below code:
            var map = new Mapsui.Map()
            {
                BackColor = new Mapsui.Styles.Color(0xFF, 0xFF, 0xFF, 0xFF)
            };
            map.Navigator.Limiter = new ViewportLimiterKeepWithinExtent();

            // Add the map tiles layer (actual mapping data)
            var tileLayer = new Mapsui.Tiling.Layers.TileLayer(
                BruTile.Predefined.KnownTileSources.Create(
                    BruTile.Predefined.KnownTileSource.BingRoads,
                    ApiKeys.BingMapsKey));
            map.Layers.Add(tileLayer);

            // Add the pins layer
            this.pins.Clear();
            this.pinsLayer = new WritableLayer
            {
                Name = "Pins Layer",
                Style = new VectorStyle
                {
                    Fill = new Brush(new Mapsui.Styles.Color(0xF4, 0x64, 0x11, 0xFF)),
                    Outline = new Pen(Mapsui.Styles.Color.White, 1)
                }
            };
            map.Layers.Add(this.pinsLayer);

I can not reproduce the problem so far. There is probably a real issue underneath but I need to reproduce it in order to fix it. Please help me to reproduce it.
It created a minimal sample based on the code you pasted above. Please check it out to see if you can reproduce it. If not, what else could play a role?
https://github.com/pauldendulk/ReproduceMapsuiIssueWithNoInternet
In order to reproduce it I just turn off my wifi. That may not be the circumstance under which this happens. Perhaps it only occurs where the status of internet is still unclear (silently hanging). How does it behave in your app when you just turn off wifi?

This issue might be related: https://github.com/Mapsui/Mapsui/issues/1889

@robloo I created a minimal sample trying to reproduce it but failed. Could you look into the difference with your situation?

Thanks for taking a deeper look at this. I think I'll try to run using Mapsui code directly in the app being tested. That will let me step through and debug better to see where the problem might be. I'll try to do this within the next week or so.

Things that may help:

It could be related to caching. In this case you would need quite some data, so longer period of browsing the map.
The TileLayer has ab ITileCache. You can provide your own implementation for that.
The TileLayer has constructor parameter to set the size of the cache, minTiles and maxTiles, you might want to make it smaller, but not too small.
It could be related to threading and thread pool starvation.


(maint) Bump to Clojure 1.7.0

Clojure 1.7.0 was just released, this commit bumps this to this version.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1571/

Those unit tests are taking forever today on travisci :-(.


Add starship helper command

Description of the Change:
This command helps execute .xbb files from the command line, as well as
to automatically reconfigure the api parameters and write them to a
config file, if needed.
Benefits:
Users who do not wish to interact with the Python shell can still execute remote commands.
Possible Drawbacks:
Related GitHub Issues:

Just left 1-2 suggestions.
Does it make sense to have tests for a shell script? (my opinion is yes, but it depends on whether testing via pytest would be straightforward or needlessly artificial)

Yes it does. This script should probably be rewritten as a python module and added to setup.py via entry_points. Technically, the functionality it is doing does not have to be exclusively in a shell script. At that point, testing would be much easier. It would probably need a bit of a rewrite before it can be tested properly.


Check buildRun.Status.BuildSpec at the correct places

Resolves #206

/assign zhangtbj

/lgtm
/approved

/approved

/approve


[Feature] Add Constant providers

Issue to track the Implemention of Constant providers.
We should have one constant provider for number, string and date, that return the specified and mandatory value.

As @bhagenbourger you added more features than expected for the string constant (pure constant parameter, list or weighted list), I'll update this ticket to express those details and new expectations for the other types.


LG webOS Smart TV component doesn't work: pylgtv package missing

Home Assistant release with the issue:

0.77.3
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

Raspberry Pi Zero W, virtual environment install
Component/platform:

https://www.home-assistant.io/components/media_player.webostv/
Description of problem:
the platform webostv doesn't start because it misses the pylgtv package
Sep 29 09:47:36 raspberrypi hass[11226]: 2018-09-29 09:47:36 ERROR (Thread-5) [homeassistant.util.package] Unable to install package pylgtv==0.1.7: Exception:
Sep 29 09:47:36 raspberrypi hass[11226]: Traceback (most recent call last):
Sep 29 09:47:36 raspberrypi hass[11226]:   File "/usr/lib/python3.5/shutil.py", line 544, in move
Sep 29 09:47:36 raspberrypi hass[11226]:     os.rename(src, real_dst)
Sep 29 09:47:36 raspberrypi hass[11226]: PermissionError: [Errno 13] Permission denied: '/srv/homeassistant/lib/python3.5/site-packages/pylgtv-0.1.9.dist-info/INSTALLER' -> '/tmp/pip-9lrsjmlf-uninstall/srv/homeassistant/lib/python3.5/site-packages/pylgtv-0.1.9.dist-info/INSTALLER'
Sep 29 09:47:36 raspberrypi hass[11226]: During handling of the above exception, another exception occurred:
Sep 29 09:47:36 raspberrypi hass[11226]: Traceback (most recent call last):
Sep 29 09:47:36 raspberrypi hass[11226]:   File "/srv/homeassistant/lib/python3.5/site-packages/pip/basecommand.py", line 215, in main
Sep 29 09:47:36 raspberrypi hass[11226]:     status = self.run(options, args)
Sep 29 09:47:36 raspberrypi hass[11226]:   File "/srv/homeassistant/lib/python3.5/site-packages/pip/commands/install.py", line 360, in run
Sep 29 09:47:36 raspberrypi hass[11226]:     prefix=options.prefix_path,
Sep 29 09:47:36 raspberrypi hass[11226]:   File "/srv/homeassistant/lib/python3.5/site-packages/pip/req/req_set.py", line 778, in install
Sep 29 09:47:36 raspberrypi hass[11226]:     requirement.uninstall(auto_confirm=True)
Sep 29 09:47:36 raspberrypi hass[11226]:   File "/srv/homeassistant/lib/python3.5/site-packages/pip/req/req_install.py", line 754, in uninstall
Sep 29 09:47:36 raspberrypi hass[11226]:     paths_to_remove.remove(auto_confirm)
Sep 29 09:47:36 raspberrypi hass[11226]:   File "/srv/homeassistant/lib/python3.5/site-packages/pip/req/req_uninstall.py", line 115, in remove
Sep 29 09:47:36 raspberrypi hass[11226]:     renames(path, new_path)
Sep 29 09:47:36 raspberrypi hass[11226]:   File "/srv/homeassistant/lib/python3.5/site-packages/pip/utils/__init__.py", line 267, in renames
Sep 29 09:47:36 raspberrypi hass[11226]:     shutil.move(old, new)
Sep 29 09:47:36 raspberrypi hass[11226]:   File "/usr/lib/python3.5/shutil.py", line 559, in move
Sep 29 09:47:36 raspberrypi hass[11226]:     os.unlink(src)
Sep 29 09:47:36 raspberrypi hass[11226]: PermissionError: [Errno 13] Permission denied: '/srv/homeassistant/lib/python3.5/site-packages/pylgtv-0.1.9.dist-info/INSTALLER'
Sep 29 09:47:36 raspberrypi hass[11226]: 2018-09-29 09:47:36 ERROR (MainThread) [homeassistant.requirements] Not initializing notify.webostv because could not install requirement pylgtv==0.1.7
Sep 29 09:47:36 raspberrypi hass[11226]: 2018-09-29 09:47:36 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform notify.webostv: Could not install all requirements.

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
notify:
  - platform: webostv
    host: <IP_ADDRESS>
    name: livingroom_tv

Traceback (if applicable):

Additional information:
I tried to install the pylgtv package (as I had to do with the RPi.GPIO package for a similar problem) with following commands:
sudo su -s /bin/bash homeassistant
source /srv/homeassistant/bin/activate
pip3 install pylgtv

and also
pip install pylgtv
and I get these results:
Requirement already satisfied: pylgtv in /srv/homeassistant/lib/python3.5/site-packages
Requirement already satisfied: websockets in /srv/homeassistant/lib/python3.5/site-packages (from pylgtv)
Requirement already satisfied: asyncio in /srv/homeassistant/lib/python3.5/site-packages (from pylgtv)

after a reboot the problem persists.

I solved in this way:
sudo su -s /bin/bash homeassistant source /srv/homeassistant/bin/activate pip3 uninstall pylgtv pip3 install  https://files.pythonhosted.org/packages/82/87/11c9028609388f3197233872c3b61b97dc8aa62de49c5131d47bb00716f2/pylgtv-0.1.7.tar.gz
So, the problem is that Home Assistant automatically install pylgtv 0.1.9, while the webostv component requires pylgtv 0.1.7.
I think this should be taken into account as a bug.


Update lazy-initial-state_en.md

In this README file, we provide an explanation of the code's output and describe what you can expect to see in the output section.

Commit Message:
Extended Description:
In this pull request, I have updated the README.md file to include a detailed explanation of the code's output and provide an improved description for the extended section. The changes include corrected spelling and improved clarity for the benefit of readers. This enhances the documentation and makes it easier for others to understand the code and its output.
If this pull request is related to an issue, reference it


Improved UI

The UI is currently laggy and shitty, it needs improvement.

Font
Color
Feel and look
Responsiveness
General design
More features

@DeveloperAspire can i pick up on this task?

@Nagendra88688 Hi! Sure you can take on it.
Do you have a design in mind?

@DeveloperAspire Yes, I'm thinking of using better fonts can make cards look more interactive. And the profile photo of each card should also be interactive on hovering on it.

@Nagendra88688 Sounds like a plan, Chief! Carry on! Make your PR and I will drop feedback. Thanks for picking this up!

okay, thank you :)

hey @DeveloperAspire
I've created a pull request with some changes, please review and let me know.
link of pull request ->   https://github.com/DeveloperAspire/developers-fact-card/pull/29

Any follow up?
@DeveloperAspire

@Nagendra88688 Thank you for this Chief! I will definitely go through it and let you know. Sorry for the delay. Work has been crazy.

No problem @DeveloperAspire , please let me know your feedback :)


Make setBit static for compiler optimistations

Made setBit static so the compiler can inline and or remove the function entirely if it decides to do so.

Merged!


foo(bar): Flunk customized commitlint bot ruleset

Obsolete, have working Github Actions but that flunks bad commit messages.


The plist images

These images do not represent the current state.

@cyberjetx can you point to which pages you noticed this issue? Thanks.

Sure and sorry,  I thought the link from the page carried over when you clicked it.
https://docs.microsoft.com/en-us/xamarin/ios/deploy-test/provisioning/entitlements?tabs=windows
Specifically:

and

Thanks, we'll take a look.

Updated today (finally).


Add groupByStreak - group values in a list by testing consecutive/adjasent values if they keep the steak going

I couldn't find anything like this in the docs, and implementation using simple re-combination of existing functions was not apparent either.
As a side note, R.groupWith is almost that, except it does not pass adjacent items in the list, but rather tests against first item in the group against each next item. "fixing" groupWith changes semantic, so I propose a new function: groupByStreak
Especially useful to group records with dates into ranges.
Here is my implementation:
const streak = (sameStreak) => {
  let s = 0
  let valueSet = false
  let lastVal = null
  return (v) => {
    if(!valueSet) {
      valueSet = true
      lastVal = v
      s = 0
    } else if(!sameStreak(lastVal,v)) {
      lastVal = v
      s += 1
    } else {
      lastVal = v
    }
    return s
  }
}
const groupByStreak = fn => R.compose(
  R.map(R.map(R.nth(1))),
  R.values,
  R.groupBy(R.nth(0)),
  R.converge(R.zip,[R.map(streak(fn)), R.identity])
)

const isConsecutive = (a, b) => a + 1 == b
groupByStreak(isConsecutive)([0,1,2,3,1,2,0,4,5,6,9]) // [[0, 1, 2, 3], [1, 2], [0], [4, 5, 6], [9]]

Would be happy to submit PR if the owners approve.

Damn, I had thought that it did work that way!
Of course they're equivalent if your comparison function is transitive, which is all I've ever tried, I suppose.
Although we might want to bikeshed on the name, I'm very much in favor of this function.  Others?  @benperez?

I think we should fix R.groupWith to work this way.
As far as the name, I think it should stay R.groupWith. I've never actually thought of using that function, (or haskell's equivalent groupBy) with anything other than equality, although as @kmamykin's example indicates, you could certainly think of it producing "streaks" instead of just "lists of equal elements". However, I think people looking for this function will find it more easily under a name containing the word "group".

The only reason I hesitate is that the current implementation is equivalent to the Haskell one; if we're going to port over the name, does it make sense to port over the behavior?
But this API is the one I find more intuitive.

The only reason I hesitate is that the current implementation is equivalent to the Haskell one

There's something ironic about @CrossEye arguing to keep something because of its similarity to Haskell 😛. That is a good point though, I missed that the Haskell version works that way.
For simple equality cases, the transitive property of equality should mean that this function works the same way a = b AND b = c -> a = c. I'm pretty sure the Haskell function was written with this use-case in mind, so they probably implemented it that way since it's simpler (although I don't think it would make a difference in the performance of the haskell version, or our version for that matter).
I do like that you can use this version of the function for more than just equality though, so I would be in favor of changing it.

There's something ironic about @CrossEye arguing to keep something because of its similarity to Haskell :smile:. That is a good point though, I missed that the Haskell version works that way.

That's funny, as I think of Haskell as providing one of our main inspirations, with Ramda serving dual roles of making available to JS folks some of the sorts of things normally found in languages like Haskell, and giving people coming from languages like Haskell some chance to use JS with less friction.  Whatever reputation I have for ... let's say ... obstinance has to do with the fact that I am often pushing the "for Javascript developers" part of our mantra.  I think Ramda needs to work with the grain of the language.

I am all for changing this function.  My misgivings are minor.  I really doubt that anyone is counting on the current behavior of the function with a non-transitive comparison function.  And the new behavior is significantly nicer.

@CrossEye, I only jest. I definitely appreciate the effort and point of view you bring to the table.
@kmamykin this PR is definitely yours if you want it, otherwise I'd be happy to make the change.

@benperez: Oh, I did understand.  I was answering something else around the same time where I was supporting doing something a bit differently from how it would be done in more strongly typed languages, and the juxtaposition amused me, and made me want to justify myself.
Yes, @kmamykin, do you want to create a PR?

I will take a stub at it...


Use dlcdn instead of downloads.apache.org

Worth a try?
I've checked and older versions are there too its not just the latest version like it used to be.
(I've checked and the IPs are different to download so it is something different
I think this should be more reliable as its a mirror network instead of an archive? https://infra.apache.org/release-publishing.html
relates to https://github.com/jenkins-infra/github-reusable-workflows/issues/40

CC @jtnord @olamy


[BUG] "Request Rearming" broken since <IP_ADDRESS>923

"Request rearming" stopped working in DCS <IP_ADDRESS>923 (the August 9th patch)
To Reproduce

Load into a cold and dark jet (A-10C II is where I've observed this, in case it matters)
Say "request rearming"

Expected behavior
Before the patch, this would bring up the refuel/rearm dialog. Now I still here the pilot voice saying "request rearming" and eventually the crew chief says "rearming complete" but no munitions change.

Thanks for the report this is a Known issue, unfortunately I have not been able to resolve it yet. It has something to do with the new cargo system added to the dialog window.


Key/Value Serialization

I recently began using the equinox serialization library to store model checkpoints. My current understanding of that code is that it writes all the non-static pytree fields in a specific order to a file.
While this works, I have found that this approach can be a bit troublesome when code is changing. For example, I often have multiple versions of the code that are basically identical except a different set of fields are marked as static to prevent them from being trained. In this case it's difficult to load checkpoints from other versions of the code because the traversal of the tree is different.
Another very similar issue is that this essentially requires you to construct a pytree of the same structure as the checkpoint before loading it. Often I only want to load portions of a checkpoint, for example when the model has multiple interchangeable components.
More generally, I feel like it would be nice to have the serialized fields stored in a more 'inspectable way' such as a key/value store. This way if you need to load old checkpoints you at least have a clue what the fields are and how to load them without having to dig through your git history and try to resurrect the specific pytree that serialized that checkpoint.
Any thoughts?

I'd generally recommend against using static_field if possible. Generally speaking it's better to either (a) filter these fields out via eqx.partition and eqx.combine (see this example), or (b) just apply lax.stop_gradient to them inside your model. (In any case, I don't think there's any  static_field-using way around this particular point, as static fields are by design completely invisible to all JAX operations.)
For loading just portions of a checkpoint, you could try saving pieces in separate files, or try something like this example to store hyperparameters alongside parameters.
In terms of being more inspectable -- JAX has recently added the new tree_map_with_path API. We could use this as part of the serialisation routine, and use that to save key-value pairs. We could also think of doing something a lot more advanced, and use something like tensorstore.KvStore. I'd be happy to take a PR on something like this.

Out of interest, you recommend not using static_field. Is that due to its usage incurring a performance hit or something else?

It shouldn't be a performance hit. It's just because:

it's a bit less flexible than leaving it as a normal field: e.g. you can't replace it with eqx.tree_at;
it's easily misused: e.g. every now and again someone asks why eqx.nn.MLP.activation isn't static -- but if it was and we used a learnt activation function (such as eqx.nn.PReLU), then its weights would not be learnt.


Update SQLServerStatement.java

fixed #1899  When executing the stored procedure, force the return of true in the dodone method of the nextresult class to allow the message parsing to continue.

Associated issue #1899 marked as resolved, PR is not needed. Closing.


Fix the errors of SAFARI's Plugin_Start_Standalone()

Fixed the errors of SAFARI plugin.
By the way, I have "KnownExtensions.plist" on my Mac, but I don't know what this file is used for.

Thanks for fixing this.


How to hide 'ALL' and 'NONE' buttons if not required?

I mean I can do through CSS but are there any params which i can disable or something to hide these buttons

There's no parameter to diable/hide the buttons. Either hide through CSS or extend the component and omit including the buttons in the html template.

Hi, @czeckd . I know this is an old issue, but i was wondering if you could give me some guidance to hide the buttons through CSS.  This is what I've tried, without success.
My css:
button.btn-block {
display: none !important;
}
I did this, trying to overwrite the following, hence the '!important':
button.btn-block {
display: block;
width: 100%;
margin-bottom: 8px;
}
Am I missing something?

In your app's base styles:
div.dual-list div.listbox button[name=addBtn],
div.dual-list div.listbox button[name=removeBtn],
div.dual-list div.listbox div.button-bar {
  display: none;
}

Sorry for the delay on my answer. It took me a while to figure out what you meant by "app's base styles". I ended up creating a file called "hide.scss" which contains the lines you provided. I referenced it from file2.scss file like this:
@import "core/dual-list-box/hide";
Finally, file2.scss needs to be included in "styles" within .angular-cli.json file


Type Error Handler/SlashCommands.js

Having this issue when hosting on replit

Should be reffering to this line

Sir I Fixed This Issue


All requests except for the big GET req for Destination Viewer done. Db schema and seed file adjusted

 Destination Viewer Backend progress


PLFM-5337: Added worker to compute the possible view model

This PR needs a new queue, the stack builder PR is https://github.com/Sage-Bionetworks/Synapse-Stack-Builder/pull/335


Update footer and navbar to match PennyLane website

Context:
The PennyLane website is updating the styling of its footer and navbar. Since the PennyLane Sphinx Theme inherits the structure and style of the Xanadu Sphinx Theme, the updated layout, configuration, etc. are also reflected here.
Description of the Change:
Updated the footer and navbar configuration and styles to match the new PennyLane website. The changes to the theme configuration are documented in the diff of the README.rst.
Navbar

Footer

Mobile Navbar

Mobile Footer

Benefits:

The navbar and footer on the PennyLane Docs pages will be consistent with the main PennyLane website.

Possible Drawbacks:

Xanadu OSS projects which do not fall under the PennyLane umbrella will adopt the appearance of the footer and navbar on the PennyLane website if they update their dependency on the Xanadu Sphinx Theme.
Some accessibility features were compromised in the adoption of the new footer and navbar styles. These will be reinstated (and improved!) in a future patch release.

Related GitHub Issues:
None.

Preview: https://xanadu-sphinx-theme--40.org.readthedocs.build/en/40/

Thanks for the review, @Lorraine-at-Xanadu!

Learn > Glossary: Link is broken and should go to https://pennylane.ai/qml/glossary

I assume this is meant for the PennyLane Sphinx Theme PR: https://github.com/PennyLaneAI/pennylane-sphinx-theme/pull/29.

When I hover on 'Learn' (for example) and then hover on 'Documentation' there's a delay when the 'Learn' menu closes causing overlap.

I've updated the navbar so that the dropdown menus instantly close when a user stops hovering over it.

Thanks for the review, @LucasSilbernagel!

The accessibility issues are really painful

I agree the accessibility of the new navbar and footer leave much to be desired. We'll tackle it soon!

I've updated the navbar so that the dropdown menus instantly close when a user stops hovering over it.

@Mandrenkov @Lorraine-at-Xanadu this is really annoying for users with accessibility requirements and defeats the purpose of the accessible example we followed ☹

@LucasSilbernagel, after a discussion with @Anakhom and @AndrewGardhouse offline, we decided to reinstate the one-second delay but ensure that only one dropdown from the navbar is displayed at any given time.


Install scripts into scripts/.venv virtual environment. Add sh shell …

…support for azd hooks when running posix/non-windows OS
Purpose

Avoid installing python packages globally + drops requirement to have PowerShell in non-windows machines.
Does this introduce a breaking change?

[ ] Yes
[ ] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

We decided to cover this in this PR: https://github.com/Azure-Samples/azure-search-openai-demo/pull/16


[Raindrop.io] Fallback for websites that don't have a favion.

Extension – Raindrop.io
Author: @lardissone
Describe the feature and the current behavior/state.
Currently, bookmarks without favicon, have an empty space. It looks like they could not be loaded or something. Maybe consider adding a default icon (such as the Raindrop icon itself).

@roelvangils good idea! will work on this!


Help!!!! zenbase-2.1.6 is defect, get NO data, but ERRORS!!!!

Hello Fluffy9,
I tested everything with the original code, even with different browsers, but I always got errors!
But I already found a solution!
########

Here is my test1 (original zenbase-2.1.6 / debug=true) link:
https://jsfiddle.net/dvhs15wp/1/

Test1 errors:
a) list File '' does not have root radix!
b) Error: Expected parameter 'json' to be type 'object', was type 'string', value '{"!":{"":1}}' at throwValidationError (https://skynet-js.hns.siasky.net/4.0-beta/index.js:15552:11) at Object.validateObject
c) Uncaught SyntaxError: redeclaration of var Gun
########
2. Here my Test2 (Original zenbase-2.1.6 / debug=true / without the gun.put command!) Link:
https://jsfiddle.net/dvhs15wp/2/
Test2 errors:
a) list File '' does not have root radix!
b) Error: Expected parameter 'json' to be type 'object', was type 'string', value '{"!":{"":1}}' at throwValidationError (https://skynet-js.hns.siasky.net/4.0-beta/index.js:15552:11) at Object.validateObject
c) Uncaught SyntaxError: redeclaration of var Gun
########
!!!Warning!!! If you change the secret, there will be four 404 GET_Errors on the first run, but all further runs are then OK!!!

Here is my Test3 (my patched zenbase-2.1.6-patch1b / debug=true) link:
https://jsfiddle.net/dvhs15wp/3/

No errors, everything OK and data!
###########
4. Here my Test4 (my patched zenbase-2.1.6-patch1b / debug=true / without the gun.put command!) Link:
https://jsfiddle.net/dvhs15wp/4/
No errors, everything OK and data!
!!!Very important for testing Zenbase!!!
To test if everything is OK, you have to comment out the "gun.put" command and then see if the data can also be loaded without first executing the "gun.put" command!!!
Here is the link to my new zenbase-2.1.6-patch1b :
https://4001kpf6rhpvb1694id2alaoa0kua26bkdspj68jejgelahsdankfno.siasky.net/zenbase/index_2_1_6_patch1b.js
Have fun testing!
With friendly greetings
parajbs

Hello Fluffy9,
Yesterday I installed "SkynetLabs/gundb-relay(branch "skynet-nodejs")" on my dev server and fixed errors while doing so
I have now created a super working new version of "zenbase-2.1.8_2-beta-plus" that works both in the browser and in nodejs!
But I think there is still a problem with the gun in the relay!
I haven't published the version "zenbase-2.1.8_2-beta-plus" on github yet because I also use "SkynetLabs/nodejs-skynet" and there in the "src/client.js" the line:
  getJSON: browserClient.db.getJSON.bind(browserClient),

added so that "getJSON" also works in nodejs!
I'm still waiting for the line to be included in the original repo!
Although I also have a version that works without the correction of "SkynetLabs/nodejs-skynet", but the version is extremely big, and double functions for browser and nodejs!
"SkynetLabs/nodejs-skynet" without correction:
// SkyDB
this.db = {
setDataLink: browserClient.db.setDataLink.bind(browserClient),
};
"SkynetLabs/nodejs-skynet" with correction:
// SkyDB
this.db = {
setDataLink: browserClient.db.setDataLink.bind(browserClient),
getJSON: browserClient.db.getJSON.bind(browserClient),
};
If you want to help with the problems then I'll try to upload both versions of my dev server to my githab repo today or tomorrow!
You can contact the developers of "SkynetLabs/nodejs-skynet" to include this line:
getJSON: browserClient.db.getJSON.bind(browserClient),
see above!!!
"db.getJSON" can then also be used in the nodejs version!
Danger! IMPORTANT! "setJSON" is not meant because this doesn't work, only "getJSON"! This is why "getJSON" is compatible with skydb data!

How about a cool WebUI?
And as an option encryption and decryption of the data by Gun or by skynet (there is no such thing for skydb at the moment!) or
other secure encryption!
Or user DB, via skynet-mysky or Gun SEC?


Added ABAP CDS

Description
Hi! This PR adds support for the ABAP CDS language, which is used for defining what is basically database views on steroids. It goes hand in hand with the ABAP language, which is already recognized.
The language is not widely used outside of the SAP ecosystem, but even the public github contains a number of repositories now. Many more can be found on github enterprise servers.
Checklist:

[x] I am adding a new language.

[x] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3Aasddls+define+view
https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3Aasdcls+define+role
https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3Aasddlxs+annotate+view

[x] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):
https://github.com/hardyp/AbapToTheFuture03/tree/master/src/zmonsters_p02_transactional/zmonsters_c07_database
Sample license(s): MIT

[x] I have included a syntax highlighting grammar: here
[ ] I have updated the heuristics to distinguish my language from others using the same extension. - I believe the extensions are obscure enough to not conflict with any other languages. @lildude can you help me test this?

Notes

I made the PR by looking at other pull requests without actually getting the project to run. Hopefully everything is in order.

The highlighting grammar may look like it has an incompatible license, but it is actually licensed separately from the code snippets in the repository. The relevant part of the license is here.

To simplify legal matters, I would like to note that though I am a SAP employee, I am making this request as my own person in my spare time and not on behalf of SAP.

@pchaigno I've now added the grammar correctly mostly thanks to @filak-sap. The build detected a license which needs reviewing. As I mentioned originally, this should be okay as the grammar is actually under the unlicense.
As for the popularity, that is for you to decide. Some supporting arguments:

The language is used, but a lot of the usage may be on enterprise github instances, which linguist also affects.
The developers at SAP are sometimes rather slow to adapt to modern development practices, which is something we're trying to change. By adding the highlighting, you actually increase people's incentive to version their code there, since it becomes easier to review.
Another number to consider: the vscode extension was released this May and has about 1800 downloads

Another number to consider: the vscode extension was released this May and has about 1800 downloads

I still think we should drop the last two extensions, at least for now. However, if you think in-the-wild usage for these extensions will grow in the near future, we can add them to #4219.

Alright, let's do it this way. Hopefully the numbers will rise over time.

@pchaigno everything should be in order now.

Please check the Travis CI build failures.

Ah, the errors are a bit higher up. I thought the license was the only remaining issue as it is the last thing in red. See why I want highlighting everywhere? :)

[ ]   1) Failure:

1638TestGrammars#test_readme_file_is_in_sync [/home/travis/build/github/linguist/test/test_grammars.rb:42]:
1639Grammar list is out-of-date. Run script/list-grammars.
1640--- expected
1641+++ actual
1642@@ -1,6 +1,5 @@
1643 "- 1C Enterprise: xDrivenDevelopment/atom-language-1c-bsl
1644 - ABAP: pvl/abap.tmbundle
1645-- ABAP CDS: SAP-samples/vscode-abap-cds
1646 - ABNF: sanssecours/ABNF.tmbundle
1647 - AGS Script: textmate/c.tmbundle
1648 - AMPL: ampl/sublime-ampl
1649
1650

[ ] 1651
2) Failure:

1652TestColorProximity#test_color_proximity [/home/travis/build/github/linguist/test/test_color_proximity.rb:21]:
1653The following 2 languages have failing color thresholds. Please modify the hex color.
1654- Go (#00ADD8) is too close to ["229fc5", "00ADD8"]
1655- Perl (#0298c3) is too close to ["229fc5", "0298c3"]
1656
1657

[ ]  3) Failure:

1658TestLanguage#test_all_languages_have_grammars [/home/travis/build/github/linguist/test/test_language.rb:367]:
1659The following languages' scopes are not listed in grammars.yml. Please add grammars for all new languages.
1660If no grammar exists for a language, mark the language with tm_scope: none in lib/linguist/languages.yml.
1661ABAP CDS source.abap cds
1662
1663

[ ]  4) Error:

1664TestLanguage#test_all_languages_have_a_valid_id:
1665NoMethodError: undefined method <' for nil:NilClass 1666    /home/travis/build/github/linguist/test/test_language.rb:389:in block in test_all_languages_have_a_valid_id'
1667    /home/travis/build/github/linguist/test/test_language.rb:389:in select' 1668    /home/travis/build/github/linguist/test/test_language.rb:389:in test_all_languages_have_a_valid_id'
1669
1670

[ ] 5) Failure:

1671TestLanguage#test_all_languages_have_a_language_id_set [/home/travis/build/github/linguist/test/test_language.rb:384]:
1672The following languages do not have a language_id listed in languages.yml. Please run script/update-ids as per the contribution guidelines.
1673ABAP CDS

This includes other languages. Is this a problem with this PR?

Nop, it does not include other languages. Only the ABAP CDS line is marked with a minus sign, the other lines have their usual dash ;)
4 and 5 are the same problem detected by two different tests.

@lildude I knew the sample license would be incompatible, which is why I discussed it with our licensing and I was allowed to license the grammar differnetly.
If you look in the license, you will see that it is split:
The following license applies to the contents of the folder 'snippets':
The following license applies to the contents of the folder 'syntaxes':
Code snippets are not part of the grammar. Or do you mean you require the repository to have a valid license, even though you are not using the incompatible part?
Another possible solution:

I copy the grammar and only the grammar to a different repository, which the Unlicense allows me to do
I submit that repository to linguist instead

This duplicates the grammar which I wanted to avoid, but I don't forsee many changes to it, so it would be okay I guess.

Code snippets are not part of the grammar. Or do you mean you require the repository to have a valid license, even though you are not using the incompatible part?

The problem is we ship the repo as a whole as part of Linguist. This means we are distributing the licensed code to peeps outside of GitHub and out of our control under a permissive license (MIT). This is expressly forbidden by the snippets license.

Another possible solution:

I copy the grammar and only the grammar to a different repository, which the Unlicense allows me to do
I submit that repository to linguist instead

This duplicates the grammar which I wanted to avoid, but I don't forsee many changes to it, so it would be okay I guess.

Due to the restrictive nature of the license on the snippets, this is the only real option short of finding another source which doesn't have a restrictive license on any of its code.
If you're going to create your own repo, keep the same directory structure and remove all of the restricted code. Keeping the structure will ensure Linguist can detect and use the grammar without jumping through hoops.

I will close this and submit a new clean PR with add_grammar executed on the updated repository.


Url path should be decode before ROUTE_FIND

if my url is image has special character,  the image in static path will load fail
lualib/protocol/http.lua
PATH = UrlDecode(PATH)
local cls, typ = ROUTE_FIND(METHOD, PATH)

I have modified and updated, the difference is that such modifications are only for static routes.
For this opportunity, I added a hard-coded utf-8 for the content-type of the static resource.


Problem: Enduro sometimes can't find the "bundle-activity"

Describe the bug
Some Enduro workflow executions fail because the worker is unable to find the "bundle-activity".
To Reproduce
Steps to reproduce the behavior:

Start a transfer as usual

Expected behavior
The workflow execution should find the "bundle-activity" on every run.
Error details
{
  "message": "unable to find activityType=bundle-activity. Supported types: [clean-up-activity, check-sip-structure, metadata-validation, start-transfer-activity, poll-transfer-activity, ZipActivity, internalSessionCreationActivity, internalSessionCompletionActivity, UploadTransferActivity, extract-package, sip-creation, download-activity, allowed-file-formats]",
  "source": "GoSDK",
  "stackTrace": "",
  "encodedAttributes": null,
  "cause": null,
  "applicationFailureInfo": {
    "type": "ActivityNotRegisteredError",
    "nonRetryable": false,
    "details": null
  }
}

Additional context

This happens intermittently for me in the Tilt dev environment. I haven't figured out why it happens sometimes but not others.
I can't see any reason that the bundle activity would not be registered with the worker.  The activity registration code looks the same as for other activities.
It occurs most often after I first start Tilt.  If I restart all the Tilt pods, it seems to resolve the problem - sometimes.

:confused:

Full event log

@sevein @jraddaoui do you have any thoughts on this problem? I'm stumped. :confused:

Very strange. I suggest to upgrade Temporal, I think we're still using 1.20.x.

@jraddaoui I think your secret generator PR fixed the problem! :) I'll give it a few days to see if it re-occurs, and if not I'll close this issue.

This happened again today, so neither the secret generator or upgrading Temporal have fixed the problem. :(

I just encountered a new error that may be related to this one. I was testing with commit d4fd85dd339d523f8aaca37583967cbddff19e11 and got the following error in the Temporal UI:
{
  "message": "unable to decode the activity function input payload with error: payload item 0: unable to decode: json: cannot unmarshal object into Go value of type string for function name: download-activity",
  "source": "GoSDK",
  "stackTrace": "",
  "encodedAttributes": null,
  "cause": {
    "message": "payload item 0: unable to decode: json: cannot unmarshal object into Go value of type string",
    "source": "GoSDK",
    "stackTrace": "",
    "encodedAttributes": null,
    "cause": {
      "message": "unable to decode: json: cannot unmarshal object into Go value of type string",
      "source": "GoSDK",
      "stackTrace": "",
      "encodedAttributes": null,
      "cause": {
        "message": "unable to decode",
        "source": "GoSDK",
        "stackTrace": "",
        "encodedAttributes": null,
        "cause": null,
        "applicationFailureInfo": {
          "type": "",
          "nonRetryable": false,
          "details": null
        }
      },
      "applicationFailureInfo": {
        "type": "wrapError",
        "nonRetryable": false,
        "details": null
      }
    },
    "applicationFailureInfo": {
      "type": "wrapError",
      "nonRetryable": false,
      "details": null
    }
  },
  "applicationFailureInfo": {
    "type": "wrapError",
    "nonRetryable": false,
    "details": null
  }
}

Interestingly I changed the output of the download activity from a string to a struct in commit 4f0db3d304ea8c8594861313a7e5f1f8a963d949, and I've restarted the "enduro", "enduro-internal", and "enduro-am" Tilt containers since adding that commit.
I'm starting to wonder if Tilt is sometimes loading old, cached Go binaries instead of the most recent ones, or if old workers aren't always shut down when the container is restarted. :thinking:

Looks like the app version reported in Tilt on the last restart is correct for the three enduro containers.
enduro:
2023-12-07T23:54:13.694Z	V(0)	enduro	enduro/main.go:87	Starting...	{"version": "0.1.0-td4fd85dd3", "pid": 1}

enduro-internal:
2023-12-07T23:54:13.692Z	V(0)	enduro	enduro/main.go:87	Starting...	{"version": "0.1.0-td4fd85dd3", "pid": 1}

enduro-am
2023-12-07T23:54:13.701Z	V(0)	enduro-am-worker	enduro-am-worker/main.go:68	Starting...	{"version": "0.1.0-td4fd85dd3", "pid": 1}

The enduro container shows a "download-workflow" execution (which failed):
2023-12-07T23:54:44.322Z	V(0)	enduro.temporal-client	log/with_logger.go:69	ExecuteActivity	{"level": "debug", "Namespace": "default", "TaskQueue": "global", "WorkerID": "1@enduro-7574989cbf-fbsr4@", "WorkflowType": "processing-workflow", "WorkflowID": "processing-workflow-150b833a-1528-4709-95fe-5a6741ae5298", "RunID": "50f855c8-a5f2-4712-b519-bedddc7c17e3", "Attempt": 1, "ActivityID": "14", "ActivityType": "download-activity"}

But enduro-am doesn't show any activity.  On a successful run the download activity logs an "Executing DownloadActivity" message (with parameters) in the Tilt enduro-am console.

Tried restarting containers individually:

Restarted enduro-am, same error. Watcher workflow ran on "enduro-internal", processing workflow ran on "enduro"
Restarted enduro, same error. Watcher workflow ran on "enduro-internal", processing workflow ran on "enduro"
Restarted enduro-internal, SUCCESSFUL run.  Watcher workflow ran on "enduro", processing workflow ran on "enduro"

@sevein no, I haven't had this problem in a while.  I'll close the ticket.


fix branch validation for converged and dedicated stacks

fixes branch validation by properly detecting dedicated/converged stacks
tests API functionality
repaired network connectivity and inter-connectivity testing
breaks functionality into paramaterized includes where possible
most importantly, i think it works

Old PR: https://github.com/blueboxgroup/ursula/pull/1789

Closing in favor of #1885


Ground Truth Files For BRATS 2015 Test Data

Hi
Dear Mr.  Kamnitsas
Thanks for your great work. Now I studied your high quality papers "DeepMedic for Brain Tumor Segmentation" and "Efficient Multi-Scale 3D CNN with Fully Connected CRF for Accurate Brain Lesion Segmentation". I used your software available at https://github.com/Kamnitsask/deepmedic. In this regard, we want to test your model on the public benchmarks BRATS2015, I downloaded BRATS2015 from https://www.smir.ch/. The training data was complete but the test data does not have ground truth files. Since you have achieved the state-of-the-art result in this competition, So do you have  the ground truth files for BRATS 2015 test data?  Please, can you get these files? Or do you know how we can access these files ?
,Very truly yours
Zeinab Salkhorde

Hello,
Thank you for your kind words and your interest in our work.
This is not however an issue relevant to this codebase. I cannot be providing data, nor is this part of the deepmedic project i am afraid. Information regarding data should be found in the corresponding papers, and requests about them should best be addressed to the relevant organizers. Please help me maintain this project by keeping the issues' section for issues on the codebase and its functionality.
The above being said, ground truth data of the "test" subset are not to be released by the organizers. Use the online evaluation platform instead.
Kind regards,
Kostas


Login and Sign-up

How can I login and Sign-up into portal, Can you please let me know?

this application is supported by 3 microservices . you need to deploy authorization server microservice and sent request to it to login or sign up

Actually I'm using asp.net core webpi microservice for this. But unable to
login and signup, I tried every possible way.
On Fri, May 13, 2022, 4:31 PM Jijo A J @.***> wrote:

this application is supported by 3 microservices . you need to deploy
authorization server microservice and sent request to it to login or sign up
—
Reply to this email directly, view it on GitHub
https://github.com/Jijoaj/Return-order-management_Angular-Application/issues/2#issuecomment-1125931558,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AINEM3B7RXY5MNPMF7VNUYTVJYY7VANCNFSM5V2JYQ3Q
.
You are receiving this because you authored the thread.Message ID:
<Jijoaj/Return-order-management_Angular-Application/issues/2/1125931558@
github.com>

Hi Aamir , am using https://github.com/Jijoaj/Return-order-management_Authorization-server
which is a springboot application with Oauth2 framework and jwt
so if you are trying to test  asp.net core webpi microservice, create a microservice for authentication which returns a Token on successful authentication

Or else if you want to remove authorization part . edit src/app/app-route.routing.ts
remove canActivate : [Authguard] from line 19 and 21


Github Login Error

Describe the bug
When clicking on Github login, and being logged into Github already, I'm redirected to /users/auth/github/callback?code={auth_code} and it's a 500 error.
To Reproduce
Steps to reproduce the behavior:

Clear browser cache
Open a browser in its version of "Incognito" mode for Chrome. (I've tried Firefox and Chrome)
Go to www.codewars.com and click on the Github Login link
Being incognito, this should redirect to the Github login page.  Login.
After a successful login, get redirected back to www.codewars.com/users/auth/github/callback?code={auth_code} and see the 500 error message displayed in the screenshot below.  Message:

"authenticable_salt returned nil for the User model. In order to use rememberable, you must ensure a password is always set or have a remember_token column in your model or implement your own rememberable_value in the model with custom logic."
Expected Behavior
To be successfully logged in to codewars after authenticating with Github.
Screenshots

Contexts
Please complete the following information:

Codewars username (if account related): nwayve
OS: Windows 10
Browser: Chrome v71.0.3578.98
Browser: Firefox Quantum v64.0

Sorry for the inconvenience. And thanks so much for all the information.
I’ll look into this today.


User Authentication

user signin done

Ready to merge @sristy17 🚀

one more thing that i wana ask form u because i am new to open source .

why i am seeing my progress still 0/4 [1 waiting ]
as u said u merged the pull request
will i get seperate pull request for signin too...???

will take 1 week to verify from hacktoberfest admins.

okay thankx and i am glad that i started my first contribution from your repo


IDK Some Errors

What is this ?

well the code in the 1.0.6 is not the code in the repository.... hmmm...
pushed a 1.0.7 and checked... the new code is in it

THX


Migration from @objc dynamic var to @Persisted annotation

How frequently does the bug occur?
Always
Description
I am trying to migrate my current Realm objects which have variables marked as @objc dynamic var to @Persisted annotation. If the object has all variables as @objc dynamic var, then everything is working correctly, I do not need to bump my Realm version. However, if the object has RealmProperty types inside, then I get the error that migration failed as those variables which have type of RealmProperty were deleted. Example:
public class UserAccessControlGroup: Object {
    @objc public dynamic var id = 0
    @objc public dynamic var name: String!

    public override static func primaryKey() -> String? {
        return "id"
    }
}

changed to
public class UserAccessControlGroup: Object {
    @Persisted(primaryKey: true) var id = 0
    @Persisted var name: String!
}

works fine, I do not need to bump my Realm schema version.
But if we have this example:
public class UserAccessControlGroup: Object {
    @objc public dynamic var id = 0
    @objc public dynamic var name: String!

    public var extensions = RealmProperty<Int?>()

    public override static func primaryKey() -> String? {
        return "id"
    }
}

changed to
public class UserAccessControlGroup: Object {
    @Persisted(primaryKey: true) var id = 0
    @Persisted var name: String!

    public var extensions = RealmProperty<Int?>()
}

I get the following error:
Domain=io.realm Code=10 "Migration is required due to the following errors:
- Property 'UserAccessControlGroup.extensions' has been removed." UserInfo={Error Name=SchemaMismatch, NSLocalizedDescription=Migration is required due to the following errors:
- Property 'UserAccessControlGroup.extensions' has been removed., Error Code=10}

I would like to understand - why it behaves like that?
Stacktrace & log output
Domain=io.realm Code=10 "Migration is required due to the following errors:
- Property 'UserAccessControlGroup.extensions' has been removed." UserInfo={Error Name=SchemaMismatch, NSLocalizedDescription=Migration is required due to the following errors:
- Property 'UserAccessControlGroup.extensions' has been removed., Error Code=10}

Can you reproduce the bug?
Always
Reproduction Steps
Try to change the Realm object properties @objc dynamic var into @Persisted var in the objects which also contain RealmProperty variables.
Version
10.54.1
What Atlas Services are you using?
Local Database only
Are you using encryption?
Yes
Platform OS and version(s)
18.1
Build environment
Xcode version: 16.1
Dependency manager and version: ...

Ok, I have noticed the following issue. Closing this one.


[eas-cli] Add --output flag to build command

Publis local build plugin before merging
Checklist

[ ] I've added an entry to CHANGELOG.md if necessary. You can comment this pull request with /changelog-entry [breaking-change|new-feature|bug-fix|chore] [message] and CHANGELOG.md will be updated automatically.

Why
Add --output option for local builds
How
Test Plan
run local build

/changelog-entry new-feature Add --output flag to the build command


Bug Report: Duplicate Labels in Command Palette

Dear abapGit Team,
I wanted to bring to your attention a bug that I've encountered in the Command Palette. When assigning labels to multiple repositories, I've noticed that the Command Palette displays duplicate entries for those labels. In my case, I've observed duplicate entries for labels such as "FM", "abapGit", and "abap2xlsx".

After investigating the issue, I've identified a solution that can prevent these duplicates from appearing in the Command Palette. By implementing a check to ensure that only items with unique titles are included in the items array, we can effectively resolve this issue.
To implement this fix, I've added the following lines of code:
// Check if the title already exists before adding to items array
var titlesAdded = {};
items = items.filter(function(item) {
  if (!titlesAdded[item.title]) {
    titlesAdded[item.title] = true;
    return true;
  }
  return false;
});

This is the result after applying the fix.

To expedite the resolution of this issue, I've already created a pull request with the proposed fix (#6821)
Please let me know if you need any further information or assistance with addressing this bug. Thank you for your attention to this matter.
Best regards,
Francisco Milan.

Sorry for some critique. But for me, it again looks like a patch instead of semantics - https://github.com/abapGit/abapGit/pull/6808#issuecomment-1937794150
The right solution is to mark labels in table raws as unlisted. Instead of deduplication, which does not understand the context.
Just imho.

Closed by #6821


Possible sync table timestamp guessing bug

Fixes #3229

@rafatower CR please

Frontend tests were OK :+1: (details)

:+1:


Fix unified build errors in raspberry pi2 of linux-armv4

I have two errors in unified builiding on raspberry pi2 of  linux-armv4 with using a different directory from source files, which is indicated by Configure.
The one is an error to find an header file in crypto directory and it was fixed to add include path of crypto in build.info.
CC="gcc" /usr/bin/perl ../openssl/crypto/aes/asm/aes-armv4.pl linux32 crypto/aes/aes-armv4.S
gcc -DDSO_DLFCN -DHAVE_DLFCN_H -DOPENSSL_THREADS -DOPENSSL_BN_ASM_MONT -DOPENSSL_BN_ASM_GF2m -DSHA1_ASM -DSHA256_ASM -DSHA512_ASM -DAES_ASM -DBSAES_ASM -DGHASH_ASM -DECP_NISTZ256_ASM -DPOLY1305_ASM -DOPENSSLDIR="\"/usr/local/ssl\"" -DENGINESDIR="\"/usr/local/lib/engines\""  -pthread -Wall -O3 -march=armv7-a -Wa,--noexecstack -Iinclude  -I../openssl  -I../openssl/crypto/include  -I../openssl/include  -Icrypto/include -c -o crypto/aes/aes-armv4.o crypto/aes/aes-armv4.S
crypto/aes/aes-armv4.S:35:23: fatal error: arm_arch.h: No such file or directory
 # include "arm_arch.h"
                       ^
compilation terminated.
Makefile:623: recipe for target 'crypto/aes/aes-armv4.o' failed
make: *** [crypto/aes/aes-armv4.o] Error 1

The other error comes from generating asm files with a perl script as
gcc -DDSO_DLFCN -DHAVE_DLFCN_H -DOPENSSL_THREADS -DOPENSSL_BN_ASM_MONT -DOPENSSL_BN_ASM_GF2m -DSHA1_ASM -DSHA256_ASM -DSHA512_ASM -DAES_ASM -DBSAES_ASM -DGHASH_ASM -DECP_NISTZ256_ASM -DPOLY1305_ASM -DOPENSSLDIR="\"/usr/local/ssl\"" -DENGINESDIR="\"/usr/local/lib/engines\""  -pthread -Wall -O3 -march=armv7-a -Wa,--noexecstack -Iinclude  -I../openssl  -I../openssl/crypto/include  -I../openssl/include  -Icrypto/include -c -o crypto/aes/aes-armv4.o crypto/aes/aes-armv4.S
gcc: error: crypto/aes/aes-armv4.S: No such file or directory
gcc: fatal error: no input files
compilation terminated.
Makefile:623: recipe for target 'crypto/aes/aes-armv4.o' failed
make: *** [crypto/aes/aes-armv4.o] Error 4
ohtsu@raspberrypi:~/github/_openssl-build_linux-armv4$

This is because the output variable of
CC="gcc" /usr/bin/perl ../openssl/crypto/aes/asm/aes-armv4.pl linux32 crypto/aes/aes-armv4.S

could not been parsed correctly as it has directory path so that all asm files are generated to stdout.
This patch fixes every related regular expressions in asm perl scripts for all platforms but it was only tested on linux-armv4.
I'm not sure that it would still work even on Windows.

@levitte @dot-asm what do you guys think?

It looks fine at first sight, but it's really @dot-asm who know the perlasm pl files, so I defer to him

As for suggested modification for .pl modules. It probably would be more appropriate to loosen it up even further. I mean right now it's programmed to recognize file.ext and suggestion is to recognize /path/file.ext. But what if tomorrow we decide to use relative ../path/file.ext? Or some day use scripts on system that doesn't use / as path delimiter? It seems that it would be sufficient to simply omit ^, i.e. look for anything that ends with file.ext. I was going to look into it, and even from angle of unifying argument treatment in all .pl modules...

I agree that removing ^ is better for flexibility. I've just updated the PR. No unified build errors are found on my raspberry pi2. Please take an another look.

Move to #848.


clower, ashpan

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[a,c]

/trunk merge


Add new experimental rule to remove trailing comma's from argument and values lists (#709)

Given code below:
            val list1 = listOf("a", "b",)
            val list2 = listOf(
                "a",
                "b", // The comma before the comment should be removed without removing the comment itself
            )

            data class Foo1(
               val bar: Int, // The comma before the comment should be removed without removing the comment itself
            )
            data class Foo2(val bar: Int,)

this should be corrected as:
            val list1 = listOf("a", "b")
            val list2 = listOf(
                "a",
                "b" // The comma before the comment should be removed without removing the comment itself
            )

            data class Foo1(
               val bar: Int // The comma before the comment should be removed without removing the comment itself
            )
            data class Foo2(val bar: Int)

Closes #709

Tnx for the link to the minimal list of types to be supported. I was not aware that trailing comma's were possible at other places as well. I have added all of them except for COLLECTION_LITERAL_EXPRESSION. I can not create a code example in which this type occurs.

Also, I think the rule should work both ways - depending on what is set in editorconfig either allow or disallow trailing commas

Isn't the option to disable rules meant for this? Or would you to like this rule to be disbaled by default and that user have to opt-in on the rule?

Tnx for the link to the minimal list of types to be supported. I was not aware that trailing comma's were possible at other places as well. I have added all of them except for COLLECTION_LITERAL_EXPRESSION. I can not create a code example in which this type occurs.

For COLLECTION_LITERAL_EXPRESSION the only case (that I know of) is when the collection literal is put as an argument for an annotation class, for example:
annotation class Annotation(val params: IntArray)

// usage
@Annotation([1, 2,])
val property: Int = 0

Would be nice if we can support this as well.

Also, I think the rule should work both ways - depending on what is set in editorconfig either allow or disallow trailing commas

Isn't the option to disable rules meant for this? Or would you to like this rule to be disbaled by default and that user have to opt-in on the rule?

What I meant is we should decide how the rule behaves based on the intellij editorconfig properties
ij_kotlin_allow_trailing_comma = true
ij_kotlin_allow_trailing_comma_on_call_site = true

If those are set to true, the rule should enforce trailing commas and throw a violation if they are not present where they should be. If those are set to false, the rule should disallow trailing commas and throw a violation if they are present where they shouldn't be. So it should work similar to final-newline rule in this regard.

I solved the COLLECTION_LITERAL_EXPRESSION based on your example. Also the rule is now configured based on the IntelliJ IDEA properties.
First I have tried to define a new property analog to insert_final_newline in class PropertyType of ec4j :
    public static final PropertyType<Boolean> insert_final_newline = new LowerCasingPropertyType<>( //
            "insert_final_newline", //
            "set to true to ensure file ends with a newline when saving and false to ensure it doesn't.", //
            PropertyValueParser.BOOLEAN_VALUE_PARSER, //
            BOOLEAN_POSSIBLE_VALUES //
    );

I could not get it working due to some Kotlin to Java interoperability problem. So I have implemented a new parser for the Kotlin Boolean as shown below:
        private var KOTLIN_BOOLEAN_VALUE_PARSER: PropertyValueParser<Boolean> =
            PropertyValueParser { name, value ->
                when {
                    value == null -> PropertyType.PropertyValue.invalid(
                        null,
                        "Property '$name' expects a boolean; found: null"
                    )
                    value.equals("true", ignoreCase = true) -> PropertyType.PropertyValue.valid(value, true)
                    value.equals("false", ignoreCase = true) -> PropertyType.PropertyValue.valid(value, false)
                    else -> PropertyType.PropertyValue.invalid(
                        value,
                        "Property '$name' expects a boolean. The parsed '$value' is not a boolean."
                    )
                }
            }

Of course, class above can also be moved to UsesEditorConfigProperties or another new class so that it can be reused for other boolean properties.

I could not get it working due to some Kotlin to Java interoperability problem.

Just curious - what was the exact problem? :thinking:

I could not get it working due to some Kotlin to Java interoperability problem.

Just curious - what was the exact problem? 🤔

Java definition in PropertyType class of ec4j
    public static final PropertyType<Boolean> insert_final_newline = new LowerCasingPropertyType<>( //
            "insert_final_newline", //
            "set to true to ensure file ends with a newline when saving and false to ensure it doesn't.", //
            PropertyValueParser.BOOLEAN_VALUE_PARSER, //
            BOOLEAN_POSSIBLE_VALUES //
    );

Copy this to my class resulted, after automatic kotlin conversion, in:
        val insert_final_newline: PropertyType<Boolean> = LowerCasingPropertyType<Any>( //
            "insert_final_newline",  //
            "set to true to ensure file ends with a newline when saving and false to ensure it doesn't.",  //
            PropertyValueParser.BOOLEAN_VALUE_PARSER,  //
            PropertyType.BOOLEAN_POSSIBLE_VALUES //
        )

The LowerCasingPropertyType<Any> can not be resolved. Error displayed is None of the following functions can be called with the arguments supplied..
Looking at the definition of LowerCasingPropertyType<Any>:
    public static class LowerCasingPropertyType<T> extends PropertyType<T> {

        public LowerCasingPropertyType(String name, String description, PropertyValueParser<T> parser,
                Set<String> possibleValues) {
            super(name, description, parser, possibleValues);
        }

        public LowerCasingPropertyType(String name, String description, PropertyValueParser<T> parser,
                String... possibleValues) {
            super(name, description, parser, possibleValues);
        }

        @Override
        public String normalizeIfNeeded(String value) {
            return value == null ? null : value.toLowerCase(Locale.US);
        }

    }

Kotlin can not distinct the method signatures. Apparently it can not distinct between Set<String> possibleValues and String... possibleValues. I wouldn't know how to resolve this. But if you do know, I would like to know how I could have found out that myself.

Hey @paul-dingemans ,
I think the automatic Kotlin conversion produced invalid code. Specifically, if we compare the following Java:
PropertyType<Boolean> insert_final_newline = new LowerCasingPropertyType<>(

With the following Kotlin:
insert_final_newline: PropertyType<Boolean> = LowerCasingPropertyType<Any>(

Notice that on the right-hand side of the Java version, the generic type is specified with an empty <>. In Java, this tells the compiler to automatically determine the generic type by looking at the left-hand side of the assignment. In this case, that generic type is Boolean, and so the right-hand side ends up being an instance of LowerCasingPropertyType<Boolean>.
However, in the Kotlin conversion, a generic type of Any is used and an instance of LowerCasingPropertyType<Any> cannot by assigned to an identifier of type PropertyType<Boolean>. I'm still a newbie when it comes to Kotlin, but it looks like we can get Kotlin to automatically determine the generic type on the right-hand side by omitting the <> completely. In other words, we should be able to do the following:
insert_final_newline: PropertyType<Boolean> = LowerCasingPropertyType(

Hi @ScottHamper,
Tnx for trying to help out on this. I tried your suggestions bu they didn't help to fix the problem. But you inspired me to take an extra glance on it. Now it is solved by defining the property as follows:
        private val allowTrailingCommaPropertyType: PropertyType<Boolean> = PropertyType.LowerCasingPropertyType(
            "ij_kotlin_allow_trailing_comma",
            "ij_kotlin_allow_trailing_comma description",
            PropertyValueParser.BOOLEAN_VALUE_PARSER,
            mutableSetOf("true", "false")
        )

Crucial part in this was to define the allowed values as a set instead of an array.

@paul-dingemans Thank you for taking time to implement this. I'm also interested in this rule. @romtsn Are there any plans on how to go forward with this PR?

@shashachu Is there any chance that this PR will get merged?

Sorry for holding this off for so long @paul-dingemans , I finalized the rule by making it work bi-directional (as mentioned in https://github.com/pinterest/ktlint/pull/1032#issuecomment-751380534), so it also can add missing trailing commas, if configured like that. Thanks for prior art 👍


Manually add .js extension to relative imports

Closes #1454
Description
We are using ES modules in the cli workspace and currently, most if not all of our imports in the cli workspace don't include the file extension, which is a requirement in ES modules. Since tsc doesn't add .js extension during compilation, there is a module not found error when we try to run node dist/index.js.
This PR:

manually adds .js extensions to all imports
adds an eslint-plugin-import package to provide linting error suggestions when imported file extension is not provided (ignores the test dir)

I've tested running the cli executable after these implementations and the module not found error has been resolved.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Deployment

[ ] Should publish npm package
[ ] Should publish Docker image

In which situation were node dist/index.js being called?
I haven't been experiencing module not found error in my use case when calling commands through yarn cli {COMMAND}. It'd be nice to remove error but I'm not sure if it is a necessary update if it adds more code with extension specification and bit more configuration while I can't see the error in use cases. I'd be grateful if you can elaborate a bit more why this update is necessary.

In which situation were node dist/index.js being called?
I haven't been experiencing module not found error in my use case when calling commands through yarn cli {COMMAND}. It'd be nice to remove error but I'm not sure if it is a necessary update if it adds more code with extension specification and bit more configuration while I can't see the error in use cases. I'd be grateful if you can elaborate a bit more why this update is necessary.

In which situation were node dist/index.js being called?
I haven't been experiencing module not found error in my use case when calling commands through yarn cli {COMMAND}. It'd be nice to remove error but I'm not sure if it is a necessary update if it adds more code with extension specification and bit more configuration while I can't see the error in use cases. I'd be grateful if you can elaborate a bit more why this update is necessary.

thanks for the feedback Nick. in cli/package.json we have
"bin": {
    "cli": "dist/index.js"
  }

we also explain in the documentation that users can call the cli command globally with npx @bisonai/orakl-cli. When you call the cli with yarn, you'll notice that we are using the --import=specifier-resolution-node/register flag which solves this module not found error. However in the bin executable that flag is not passed and it's not that simple to pass flags in the executable (couldn't find an easy workaround). Therefore when you call the cli command using the executable (through npx @bisonai/orakl-cli), you get module not found error. So we either need to update the documentation to remove executable usage or implement a fix so that the executable can be called globally. I explained this in more detail in the tagged issue.


Automatically run postBuild scripts to halt on error

By default, bash scripts won't stop execution if they hit an error, they just keep on going. This could create a lot of confusing behavior for people in Binder, since it'd look like their bash script worked, but really nothing happened.
How can we ensure that a postBuild file that doesn't run properly also halts the build of an image? I think if we use set -e, it'll only exit the bash script on error, but then repo2docker will just continue with the next step...
cc @yuvipanda

Another option would be to encourage people to use a more friendly language (like python) for executing postBuild scripts by altering what comes after the !#. E.g. we could tell people how to execute postBuild with Python (!#/usr/bin/env python3), R, whatever.

so......I may have discovered that there wasn't actually an error happening, but that instead I was looping through an empty list which I was interpreting as an error........
anyway, it looks like if you have an error in postBuild (even without -e) then it will print an error message in red in the log output, but will still continue. Is that desired behavior?

Can you run the postBuild script by hand outside of repo2docker? If yes, check what the exit code is (run echo $? right after running the postBuild script in your terminal). If the exit code is 0 then docker will continue to run because the script signalled "all good, I ran to completion!".
I don't think we should start trying to guess from log messages whether or not the script might have not run properly. The exit code is the way to communicate that IMHO.

Whether or not the script stops running is up to the script. If you want a bash script to stop you need to put set -e in it.
The only thing we can do as the "executor" of the script is check the exit code (which should be != 0 if the script does not run to completion).

@choldgraf is your script:
#!/bin/bash
gasdgasdg

or is it
#!/bin/bash
set -e
gasdgasdg

If it's the first one, it is expected that failure won't stop the build, since bash doesn't actually consider it an error. One solution there is to use a non-terrible language. If the following doesn't fail the build we have a problem:
#!/usr/bin/env python3
asgasgdsag

I just ran this gist:
choldgraf/c7846cfa96bcfe8d186dff07da102ef0
with and without the set -e and in both cases this error happened:

so I can't reproduce this issue and I guess we should just close it :-/

Closing as the last comment suggests it may not be a bug any more.


Pickling Error MultithreadSLIM

Ciao, il multithreadSLIM mi da un errore quando viene chiamata la .fit :
PicklingError: Can't pickle <type 'instancemethod'>: attribute lookup __builtin__.instancemethod failed
Il codice che sto tentando di eseguire e' :
recommender = MultiThreadSLIM(l1_penalty=0.1,l2_penalty=0.01)

train_size = int(data.shape[0] * 0.8)
# split data according to the shuffled index and the holdout size
train_split = data[:train_size]
test_split = data[train_size:]

recommender.fit(train_split)

Dove data e' una matrice in formato csc.
Qualcuno sa la causa di questo errore?
Grazie :)

Hi @umbertoDifa , what version of python are you currently using? Have you made any changes to  MultiThreadSLIM  class?

Hi @mquad , I'm using python 2.7, no I have not made any changes.
By the way, I attach the code and the dataset to replicate the error.
mySLIM.zip

As written in the Install instructions in the README.md, only python3.5 or above are supported. Please setup python3.5 and try again.

Thanks, that was the problem.

You are welcome 😄  I updated the Readme.md to make this point clear for everybody.


Allow to configure the branch name

I have configured the version updater action on my project and it works perfectly. Thanks for your project.
But as it is executed every day and with a new branch name each time, if I don't check my projects every day, I end up with several pull requests for the same patch.
On my workflow:
on:
  schedule:
    - cron: '0 0 * * *'

On main.py :
new_branch_name = f"gh-actions-update-{int(time.time())}"

I think it will be good to have a fix branch name (at least as an option), So that the pull request can be updated each time (with a force push).
Best regards,
Jean-Marie

Hi @saadmk11,
As said in my comments of #40, I have tested the test branch on my project and it is working well.
Looks good to me, I am waiting for you to merge it to the main branch.
Best regards and happy new year.

@jmlemetayer Thanks for the reminder. I still did not get a chance to test it out. I will try to release this sometime next week. Meanwhile you can use the pr-branch-config branch if this a urgent requirement for your workflow. :)


GitHub: Invite members

Who should be invited to Catena-X ng
GitHub User:
johanneskueffner
Leon-Gr
Vouching person:
@evegufy
@jjeroch

Hey @evegufy! Invites have been sent. Once accepted, the maintainer can add them to a team.


[MGX-1183] Disable AssetRegistry::register_asset on call filter level

looks like BaseFilter does not apply to sudo calls, so we would have to disable it in orml_tokens repository which seems like too much effort for sudo only call


not able to create private hosted route53 zone and a record set

HI,
I am trying to create a private hosted route53 zone and a record set. manifest below for reference:
Setup private DNS for cm.test.com
route53_zone { 'Internal - cm.test.com':
zone => 'cm.test.com',
vpc_id => vpc-6107a005,
ensure => present,
}
private
route53_a_record { 'Internal - devops-haproxy-1a1.cm.test.com.':
a_record => 'devops-haproxy-1a1.cm.test.com',
ensure => present,
zone   => 'cm.test.com',
vpc_id => vpc-6107a005,
ttl    => 3000,
values => ['<IP_ADDRESS>'],
}
getting following error:
[ec2-user@puppetmaster manifests]$ sudo puppet apply route53.pp --test --modulepath /etc/puppetlabs/code/environments/devopstest/modules
2015-11-17 03:46:43.042875 WARN  puppetlabs.facter - locale environment variables were bad; continuing with LANG=C
Error: Could not parse for environment devopstest: Syntax error at 'internal' at /etc/puppetlabs/code/environments/devopstest/manifests/route53.pp:9:21 on node puppetmaster
could you assist on this.
Thanks,
Ram.

@ram2valar I'm afraid at the moment the route53 resources don't support being associated with a VPC as you have in your examples here. Zone also doesn't have a zone property (although you can use name here for that purpose).
I'll leave this open to track evetually adding support for VPC to Route53.

Related Issue #174 and pull request #157


fix: Right sidebar breaking when end of the page is reached

Fixed this issue: #1358
Summary
The right SideNav overlaps the footer when you scroll all the way down.
Changelog
I have a new css file for the sidenav and I added a margin-bottom property in order to fix this issue.
A test plan
Go to the virtualizedlist page and scoll all the way down.
This is my user agent: "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.108 Safari/537.36"
http://localhost:3000/react-native/docs/virtualizedlist

Summary
I have fixed the alex lint issue on STYLEGUIDE.md, I modified the line number 66.
Changelog
I have added backtick character before and after the word he.
A test plan
Run: yarn test

Old Linter Warning
https://circleci.com/gh/facebook/react-native-website/8045?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

@davizp Can you rebase your PR and leave only website/static/css/sidenav.css changes? (commit 354d41abee379dd320922e07536c990c938c3b50)
I think this will help with merging your fix into master.

@davizp Just saw it's your first PR here. Sorry for not providing link which should help you with this process in my first comment - https://akrabat.com/the-beginners-guide-to-rebasing-your-pr/
I have additionally ask you to drop 7fc78bd5c1422b162f843ac30e02a20c0d2513a9 changes which can be done using git reset --hard HEAD~1 command after successful rebase but before final force push.

Fixed in https://github.com/facebook/react-native-website/pull/1506. Sorry I didn't see this PR earlier.


Particle effects

There is an issue in some instances where particle effects have visible planes around them and inside those planes the effects from reshade aren't working, a good example of it is in M2 raid

Yeah this is cause for some reason ff14 masks the particles, the only way to fix it is to disable ui masking in AcerolaFX_End but that'll apply shaders to the whole ui if that bothers you.


[Feature Request]: In order to save tokens, the message sent to LLM only retains the prototype of the skill function and removes the import and function body etc

Is your feature request related to a problem? Please describe.
No response
Describe the solution you'd like
No response
Additional context
No response

New feature development is currently focusing on v0.4 release.


ament_virtualenv: 0.0.5-1 in 'crystal/distribution.yaml' [bloom]

Increasing version of package(s) in repository ament_virtualenv to 0.0.5-1:

upstream repository: https://github.com/esol-community/ament_virtualenv.git
release repository: https://github.com/esol-community/ament_virtualenv-release.git
distro file: crystal/distribution.yaml
bloom version: 0.9.0
previous version for package: 0.0.4-1

ament_cmake_virtualenv
* fixed various bugs identified by CI

ament_virtualenv
* fixed various bugs identified by CI

Crystal is EOL as of last month we can't accept new PR for it: https://index.ros.org/doc/ros2/Releases/
https://discourse.ros.org/t/preparing-for-final-crystal-sync-and-patch-release-2019-12-12/11792/3

Created #23486 to update our index.

Oh, I didn't know. Thanks!


!patch not working

the bot doesn't reply to uploaded mail files. It doesn't error out or smth, but just doesn't do anything, as seen below:

Yeah, it doesn't work for me either

This appears to be a regression in JDA caused by Java 9. I’ll be recompiling it and upgrading dependencies as needed soon, in the meantime please use Seriel or KcrPL’s patchers.

Just saying, JDA is not intended to use Java 9.
El 08/11/2017 9:09 a.m., "Spotlight"<EMAIL_ADDRESS>escribió:

This appears to be a regression in JDA caused by Java 9. I’ll be
recompiling it and upgrading dependencies as needed soon, in the meantime
please use Seriel or KcrPL’s patchers.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/RiiConnect24/RC24-Bot/issues/53#issuecomment-342846385,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI2CdYRDPsfQmqMpVVjmRiiMbwvg6B_xks5s0cQzgaJpZM4QV8E7
.

I may have to figure out how to downgrade then. Thanks for letting me know.

Fixed in 7de84445dacf511bb6510751bb0f3f9b9207e5ef because I was too lazy to downgrade.


Copy slice along arbitrary axis

Added this to allow data-parallelization when the first dimension is not batch size.

I think you can use mx.nd.crop for this

i need to copy into pre allocated ndarrays and avoid making extra copies of memory. for data par, the targets are all pre allocated.

crop(ou=target) should do it

hmm, this should do the trick. I missed this one. Thanks!


combine latex and markdown highlighting

Using inline latex is a pretty common pattern in markdown documents destined for conversion with pandoc.
For example, one might include the command \textsc{small caps text} in a pandoc markdown doc because markdown has no native syntax for small caps.
However at present I can't work out how to get both markdown and latex syntax highlighting active at the same time. It would also be nice if the document tree (for ctrl-R) recognised both markdown headings and \part{} \chapter{} and \section{} commands.

closing due to age

That would be a really usefull feature since one of software which make write markdown is pandoc. And pandoc allows raw LaTex commands.
Could you at least add syntax and color features for two things:

math equations which can take two forms : $ inline equation $ or $$ multiline equation $$
bibliography key reference such as [@my_bib_key]

Thank you !

@hadim can you open a new issue to discuss? thanks