Why are there no GitHub releases after 2.0.0 but still updates?

this is kinda confusing, especially because the home bridge UI shows the 2.0.0 Changelog even for the smaller updates...
Maybe you could add releases for smaller updates, just like you did for pre 2.0.0 releases?
Thanks and keep up the great work!

All changes are here: https://github.com/grzegorz914/homebridge-xbox-tv/blob/master/CHANGELOG.md
Only major and minor changes are released to git and show before update in HB.  Patches are displayed also in HB but after update.

I've actually found the Changelog file and also saw the minor patches Changelog after updating.
But is there a reason you handle it this way? For me its very confusing, searching for new hb addons mostly on GitHub and finding yours with a version not matching the version whilst searching/installing it inside home bridge ui.
In my opinion releases are some sort of standard and should show the latest stable version to download (also be the same as the source code in the main branch).
If you don't want to change your style of doing this maybe you could add some sort of disclaimer in the mayor release Changelog?

The code on GitHub is alway the same as version published to NPM. Homebridge get plugin updates from NPM not from GitHub. Just Im not release patch versions on GIT, that's all.


Add NTP servers accessible in China

Motivation
Fixes #2063
Changes
Adds a bunch of NTP servers that should be accessible in China. Also adds pool.ntp.org which should work anywhere. Sources:

https://www.ntppool.org/en/
https://www.pool.ntp.org/zone/cn
https://intl.cloud.tencent.com/document/product/213/32379
https://partners-intl.aliyun.com/help/doc-detail/92704.htm

Test Plan
No tests required, no change in functionality.
Will ask community members in China to test.

@noamnelke I added servers as discussed in #2063. Note that this is not an ideal solution for two reasons:

probabilistically nodes in China will still fail occasionally. We currently make five requests and the node exits if < 3 of these succeed. If it unluckily happens to choose, e.g., the Google servers, it will still fail.
nodes outside China will end up syncing time against the China NTP servers - I tested them locally and they all work for me here, but this isn't ideal.

Two ways we might decide to improve this:

try to only list servers that work everywhere - that would mean, e.g., removing the google servers. pool.ntp.org should theoretically work anywhere.
do something smarter/more complicated, e.g., detecting the node's region (or letting the user specify it in the config) and choosing a pool of servers based on this.

You're right about the possibility of it failing even after this change. I think there's an easy solution, though, that doesn't require any code changes. The number of NTP queries, where 3 or more must succeed, which you quoted as 5 is configurable.
For a user in China, there's a 55% chance of success at randomly picking a working server. I asked Sofi to run the numbers for me and we need 18 attempts to get to a total of 99.99% success rate. It really depends on how often we run this check. The current default check interval is 30 minutes, so it's expected to fail once every ~20 days.
If we make ntp-queries=15 and increase the number of tries here to 3, the probability of failure becomes negligible (0.000000001):
https://github.com/spacemeshos/go-spacemesh/blob/ce9d5da1a17aee64458de361cbe4f85a03d1b49f/timesync/ntp.go#L193
@y0sher @lrettig you see any problems with what I suggested?

Sounds good to me.
note about pool.ntp.org. we removed it since we had weird problem, it might be related to the servers themselves or maybe something with our cloud provider (gcp), requests just came back empty even though there was no error. we couldn't reproduce as it was really rare and happened only on the gcp machine. but we couldn't write code to properly handle this since we didn't know what exactly happened and we couldn't reproduce.

@lrettig given what @y0sher wrote about pool.ntp.org, it's probably best to remove it.

you see any problems with what I suggested?

It totally makes sense and I trust your math @noamnelke.

If we make ntp-queries=15 and increase the number of tries here to 3

@noamnelke I made the first change, not sure what you meant about "increase the number of tries" as this is already what it's set to - please double check but I think we're good here.

@lrettig

already what it's set to
You're right, I misread the code (the line I referenced is the initialization, not the number of attempts) - we can just leave it at 3, as it is now.

let's not merge this for now.
@y0sher

adding so much china server won't impact the non-china miners ?

It will. I agree with @lrettig that we should make the list configurable, so users in China can use Chinese servers and Americans can use American servers.
There's probably more we can do to improve the experience here, like randomize the order of servers and when one fails put it at the bottom, so it's tried last next time. Try a few servers concurrently and stop waiting once we have enough successes, then put the successful servers at the top of the list so the fastest ones are tried first next time. If we had that, we could include both Chinese and western servers and it would just sort itself out.

Okay, once the new testnet is up let's start with a separate app/node build
for our Chinese smeshers and see if this list of NTP servers works better
for them.

FYI I created a new tag for the current testnet that includes these changes, and I've asked some of our Chinese smeshers to test it: https://github.com/spacemeshos/go-spacemesh/releases/tag/v0.1.14-ntpfix

One of our community members in China has been testing the pool of candidate NTP servers we identified. They report that on one system, they're all producing errors except nist.gov, while on another system on a different network, they all work. Not sure what to make of these results yet, I think we need to test more.

This was handled in #2221

This was handled in #2221


Scene description should be two lines

The tile has only one line with ellipses

Two lines force tile to be taller


Even better fixup to project fixups

These need to happen on the unevaluated MSBuildProject.
No idea how it ever worked before (e.g. in 876cc98 or even before that),
but it seems to work now. Hooray!
Should fix https://github.com/WildernessLabs/NetMF_MonoDevelop_VS_Mac_Addin/issues/12 but a little extra testing wouldn't hurt 😁

Awesome!!!!!


GraphQL - Error when querying for circuits with _type filter applied

Environment
https://demo.nautobot.com

Nautobot version (Docker tag too if applicable): 1.6.2

Steps to Reproduce

Execute the following GraphQL query:

query {
  circuits(_type: "transit") {
  	cid
  }
}

Expected Behavior
Nautobot should return circuits that have circuit type set to transit.

Observed Behavior
An error {'_type': ['Unknown filter field']} is returned.
{
  "errors": [
    {
      "message": "{'_type': ['Unknown filter field']}",
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],
      "path": [
        "circuits"
      ]
    }
  ],
  "data": {
    "circuits": null
  }
}

The GraphQL layer has code to correctly translate _type back to the actual filter field name of type; it looks like something at a higher layer (probably around STRICT_FILTERING) is intercepting and rejecting the filter name before this can be fixed up.


Video and audio are not in sync

While watching Youtube videos (on Youtube or in FaceBook) there is an audio delay of approximately 0.5 seconds compared to the video.

I do not experience this, what browser are you using to watch videos?

Google Chrome Version 60.0.3112.90 (Official Build) (64-bit)
On Tue, Aug 15, 2017 at 9:47 AM, Roman Kisil<EMAIL_ADDRESS>wrote:

I do not experience this, what browser are you using to watch videos?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/romankisil/eqMac2/issues/38#issuecomment-322522381,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Adhe_CiE_repHVm36Pi6wYyZzNQcbagFks5sYcuqgaJpZM4O1t70
.

Did you use a stopwatch to get the approximate delay timing?

lack of description closing

Hey everyone. Excuse me for the lack of updates. I have been really busy on the new version of eqMac. I am happy to announce an open beta release.
You can download it here: https://eqmac.app
There are a lot of known bugs that I could not fix just yet:

Driver not installing from the first try. Sometimes need to restart eqMac a couple times.
Complex Device switching logic is not perfect (for example when switching off the device that's currently playing, plugging in new device etc.)
UI takes long to load (a lot can be done to improve that)
UI dimensions are not synching properly
There's a noticeable audio delay in the playback, so not suitable for video yet (I know where the problem lies just haven't had the time to fix yet)
Sometimes the playback doesn't start, in that case please try to switch the output to the device you want and eqMac should try and build the audio pipeline again.

So as you can see there's still some work to be done but at least it's a completely modern tech stack that I am fully willing to work on: 100% Swift, User Space driver and a Web UI, instead of C++, Kernel Drivers. As I said, should work on Catalina and should not have any pesky ear drum blasting volume bugs (at least I didn't come across any).
I'm open to feedback and issue reports here: https://github.com/bitgapp/eqmac
That repo will become the main repo for eqMac and this repo will be archived eventually.
And yes it's called just eqMac, not eqMac3 :) And it will stay that way forever.

Hey everyone 👋
I will be open sourcing the new version of eqMac, therefore, I am closing this issue and repository as I will be transfering nodeful/eqMac2 repo to bitgapp/eqMac
Please bear with me for the next couple of days as I setup that new repo. The eqMac2 code will be living in the old-eqMac2 branch if anyone is interested in referencing it for whatever reason.
Please try out the new version and report bugs to the new repo.
Thanks for using eqMac.


feat: typescript port

Hi Zack,
I've changed the project to use Typescript instead of JavaScript to ensure type-safety throughout. Caught quite a few errors already. Some of other notable changes aside from the direct port is:

Removing the "Standard Preset" from options as it seems to throw errors under certain conditions.
Changed the package manager to not have package install as an option when using PNPM, since it's not supported.
Made the project ask for a project name first so we can add everything to a subdirectory.

Wow great stuff. Let me test it locally tomorrow and see if we can merge without breaking any of the new Auth stuff. Appreciate the work!

Let me test it locally tomorrow and see if we can merge without breaking any of the new Auth stuff

Honestly, I wouldn't mind rewriting Auth related stuff, mainly because this port could solve some issues I ran into while developing the auth implementation

I'd also say to have a look at this, since I'm working on this and the auth is a day or two from being done as well
https://github.com/zach-hopkins/create-t3svelte-app/issues/33

Hi @Stoffberg , great work. Just tested the latest on the CLI redo. Looks good and I like all of the changes outside of requiring a manual db-push command (any particular reason for choosing this?).
Is there anything else that needs to be patched up before pulling?


[Bug]: Cross referencing: On deleting a branch, and bringing it back in, lot of uncommitted changes show up

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I delete a branch [which has all the latest changes committed] and then bring it back from remote repo,  it shows uncommitted changes
Steps To Reproduce

Go to a git connected app and create a few branches [A,B,C] with some additional pages/queries as compared to the master branch
Do a few git operations, like merge between branches. Commit the C branch with all latest changes.
Now checkout the B branch and delete C branch.
Bring back the C branch from the remote repo.
Note there are uncommitted changes on C though they were already committed.

Public Sample App
No response
Environment
Deploy Preview
Severity
High (Blocker to building or releasing)
Issue video log
https://jam.dev/c/77230fc1-7b49-4b09-b13b-a975300abb05
Version
Cloud

git@github.com:btsgh/Bugverifications_crdp.git
The above is the repo. It is a public repo - so you should be able to access it.
aptest is the branch showing uncommitted changes
aptest_extension is the new branch created from aptest to give an idea about the commits that are coming up

This is an existing issue. it's also reproducible on release under circumstances. Let's not consider this as a blocker. A separate fix would be shipped for this.


Create a folder for Coding Files

Added the function that will ask the user if they want a folder created to put programming files into


feat: TryGetFolderFromPathAsync return null instead of throw exception

TryGetFolderFromPathAsync return null instead of throw exception when empty folder path
What does the pull request do?

What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

Fixes #11681

You can test this PR using the following package version. 11.0.999-cibuild0036327-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


Support for local hoogle

https://github.com/ndmitchell/hoogle/blob/master/docs/Install.md

https://github.com/ambiata/mafia/pull/26/files

Done.


print black also

What the program do ?
print on emore line
In what programming language it is written?

This pull request do not contains files in specific folders. Please fix it and reopen


[PA-848] Minimize obtaining px-backup ctx in tests

What this PR does
we do backup.GetAdminCtxFromSecret() wayyy to much whereas it can be done once and forgotten for most cases. Making cosmetic change to clean things up.
It also refactors some functions in portworx (backup driver) in order so that they use the provided ctx.
Which issue(s) this PR fixes
Closes #PA-848
Special notes for your reviewer

run pipeline for all tests:

Jenkins build link: https://jenkins.pwx.dev.purestorage.com/job/Users/job/Sumit/job/Custom-Pipelines/job/px-backup-on-demand-system-test-byoc/1090/
Aetos Dashboard: https://aetos.pwx.purestorage.com/resultSet/testSetID/156273

test DeleteIncrementalBackupsAndRecreateNew failed due to intermittent failure. re-ran successfully:

Jenkins build link: - https://jenkins.pwx.dev.purestorage.com/job/Users/job/Sumit/job/Custom-Pipelines/job/px-backup-on-demand-system-test-byoc/1109/
Aetos Dashboard: - https://aetos.pwx.purestorage.com/resultSet/testSetID/156951

More runs when updating from master (NamespaceLabelledBackupSharedWithDifferentAccessMode):

Jenkins build link:
Aetos Dashboard:

This is no longer relevant as the change is being addressed in other PRs


iOS EAS Build fails (Android is OK) while using gluestack-ui v2 with EXPO 51

Description
iOS EAS-BUILD fails after using gluestack-ui
CodeSandbox/Snack link
GItHub https://github.com/tkhwang/expo-gluestack-ui-ios-eas-build-error
Steps to reproduce

create expo app using create-expo-app
init gluestack-ui v2
add gluestack-ui components : VStack, HStack, Text

common libraries
    "expo": "~51.0.21",
    "react-native": "0.74.3",

gluestack-ui libraries
    "@gluestack-ui/nativewind-utils": "^1.0.21",
    "@gluestack-ui/overlay": "^0.1.15",
    "@gluestack-ui/toast": "^1.0.7",

eas-build iOS fails and Android builds well.

iOS EAS BUILD: Error
Error occurs in Run fastlane step
...
› Packaging expo-eas-client Pods/EASClient » libEASClient.a
› Executing expo-eas-client Pods/EASClient » Copy generated compatibility header
› Executing expo-updates Pods/EXUpdates » [CP-User] Generate updates resources for expo-updates

❌  error: global.css: A jest worker process (pid=6452) was terminated by another process: signal=SIGTERM, exitCode=null. Operating system logs may contain more information on why this occurred.

❌ Metro encountered an error:
global.css: A jest worker process (pid=6452) was terminated by another process: signal=SIGTERM, exitCode=null. Operating system logs may contain more information on why this occurred.
Learn more: https://docs.expo.dev/build-reference/troubleshooting
▸ ** ARCHIVE FAILED **
▸ The following build commands failed:
▸ 	PhaseScriptExecution [CP-User]\ Generate\ updates\ resources\ for\ expo-updates /Users/expo/Library/Developer/Xcode/DerivedData/tkhwangwhattodo-dmyphtzpxtrumyaygvpyoswkfeln/Build/Intermediates.noindex/ArchiveIntermediates/tkhwangwhattodo/IntermediateBuildFilesPath/Pods.build/Release-iphoneos/EXUpdates.build/Script-46EB2E0001C260.sh (in target 'EXUpdates' from project 'Pods')
▸ (1 failure)
** ARCHIVE FAILED **

Android EAS BUILD : OK
It builds well.
gluestack-ui Version
v2
Platform

[X] Expo
[ ] React Native CLI
[ ] Next
[ ] Web
[ ] Android
[X] iOS

Other Platform
No response
Additional Information
Full log attached:
Xcode build log 2d6d45c1.log

Hey @tkhwang, Thanks for reporting this issue. We will have a look.

Also running into this. I think it's something with nativewind but I haven't been able to figure out a workaround yet.

+1, I've encountered it too just now, when building for ios development build

Also facing this. It's difficult to debug since the EAS output offers no useful error message. I've tried a lot of variations on metro config, the import, and pre-building the css without luck.

OK I figured it out @tkhwang it's probably not a gluestack problem. Node is running out of memory. Add an environment variable in eas.json to increase node heap size and it will no longer crash ex: "NODE_OPTIONS": "--max-old-space-size=4096",

@evelant  Oh. I also observed with increasing heap size, the build error has gone. Thanks.
    "production": {
      "ios": {
        "image": "latest",
        "resourceClass": "m-medium"
      },
      "channel": "production",
      "env": {
        "NODE_OPTIONS": "--max-old-space-size=4096"
      }
    }


[Bug Report]  I lost the home icons on top under the location bar

What is the issue you are having?
I used to see my character icon , mail icon, and other things under the location bar, but now it is gone and I cannot get it back.
What is the expected behavior?
To get the icons back
What actually happened?
I dont know it stopped working
Suggested Solution/Workaround
No response
Errors
No response
ElvUI Changelog/Troubleshooting

[X] I have confirmed that I have read the Changelog and done the Troubleshooting Steps.
[X] I have disabled all addons except the AddOns called 'ElvUI' and 'ElvUI OptionsUI' and 'ElvUI_MerathilisUI' by using /muidebug on? This is important in order to rule out that any of your other addons are causing the issue. If you manage to pinpoint a conflict between ElvUI and/or MerathilisUI and another addon, make sure you include that information.
[X] I have verfied my issue doesn't exist in the default UI.
[X] I gave you information above on which XPack i play Classic, WotLK or Retail.

Okay, first off all:

No, you haven't.
Second:
You haven't read the Changelog.


Invalidate workspace cache

When a new workspace is added, it is not displayed immediately in list of workspaces.

A simple idea for a custom caching mechanism would be to store the last time the workspaces were indexed and simply update whenever the last modified time of the workspace directory is more recent than that. Thoughts?

Would be at least a temporary solution until a database or smarter cache invalidation (without rebuilding the whole cache) is available.

Works for me.


add in ph_num using textgrid files

This is an alternative way for generating the ph_num using textgrids for polysyllabic phoneme systems (English, Russian, etc.). If a user has used the pipelines/no_midi_preparation.ipynb and has created the textgrids already.

Great work and thanks to your contribution!
By the way, have you deal with the leading phoneme of each word? As the README says,

In singing, vowels, instead of consonants, are used to align with the beginnings of notes. For this reason, each word should start with a vowel/AP/SP, and end with leading consonant(s) of the next word (if there are any).

That is to say, we may need to know about which phonemes are consonants and which are vowels to do the word division correctly. Would you please consider adding a --consonants option to take this rule into consideration?
Also, it would be better if you would write all your additional dependencies in requirements.txt and write a brief introduction to your scripts in README.md.

Ah, I had read that vowel / consonant note but completely forgot about it while implementing this!
I'll take a look and get an improved version out, with some more notes and requirements.

I've added a simple split on phoneme option, from the phones supplied via a text file, it felt like that would offer the most flexibility should the user want to split on "y" or "AP" etc.. they could just add them into the file. Also added info into the README.md & requirements

Sorry for the late respond.
What does split_on_phones mean? Can you explain it with an example in README? Also, since different people use different dictionaries, is it really proper to add a preset split phoneme list, and which dictionary is your current split_on_phones.txt suitable for?


Healthcheck params have no defaults, produce broken config if not set

rancher/server: v1.6.8 (not new, but apparently worked in 1.1/1.2 timeframe)
The 6 health check interval, threshold, and timeout fields have no default values.. If a service is created with a health_check that doesn't specify them, the resulting config gets inter 0 on the server lines, which is invalid and makes haproxy fail to start.
The fields should have appropriate minimums and defaults so that you can't create this situation.
Steps to Reproduce:
Add a service as below, then add it as the target of a balancer:
version: '2'
services:
  foo:
    image: nginx
    scale: 1
    start_on_create: true
    health_check:
      port: 80
      strategy: none
      request_line: GET "/" "HTTP/1.0"

Results:
haproxy.cfg:
backend 90_
...
server af6f9200a038f4b9ef06e42530a1e2e093c21b14 <IP_ADDRESS>:80  check port 80 i
nter 0 rise 0 fall 0

Startup error:
time="2017-09-07T06:14:03Z" level=error msg="Failed to apply lb config on provider: error reloading  -- reloading haproxy config with the new config changes
 * Reloading haproxy haproxy\n[ALERT] 249/061403 (50) : parsing [/etc/haproxy/haproxy.cfg:50]: invalid value 0 for argument 'inter' of server

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


Fixed dialog field dynamic refresh

This is the second part of https://github.com/ManageIQ/ui-components/pull/246.  In order to test out the fix for that PR, we had to fix field dynamic refresh in SUI and also bump ui-components to take in the fix from 246.
@miq-bot add_label bug
@miq-bot add_label gaprindashvili/yes

fixes https://bugzilla.redhat.com/show_bug.cgi?id=1539871

@chalettu any chance we can get a gif of this in action?

@chalettu bz reads:

"Add to shopping cart" button remain disabled after refresh on some dialog fields in SUI.

any chance we can get a gif of this issue resolved?

Here is a gif of the issue resolved

Gaprindashvili backport details:
$ git log -1
commit 13910e7528ffbf7c6ada5055db03d936b0c8b5cd
Author: Allen Wight<EMAIL_ADDRESS>Date:   Wed Feb 7 15:55:23 2018 -0500

    Merge pull request #1380 from chalettu/dynamic_dialog_field_refresh

    Fixed dialog field dynamic refresh
    (cherry picked from commit 6bf479c03afb666daa618078ed3f769eb532cb57)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1543180


[9.x] Update AuthServiceProvider variable names and docs

A tiny bit of modernazation of AuthServiceProvider for doc-blocks and variable names.

It's not necessary imo to do this everywhere. just in the places where it makes sense.


Alter rounding behaviour of sig fig checking code after FP issues

A question relies on the value of 0.2425 to 3sf, which ought to be 0.243 but when doing the rounding it gets parsed into something like 0.2424999999997 which then rounds down and not up.
To fix this, use BigDecimals as we do when working out the number of sig figs to use; this can exactly represent all of the cases we're likely to encounter on Isaac. We can also be explicit about the rounding scheme "round half away from zero" that we want to use.
This will need additional testing, but I've added the case to the unit tests too.

(Ensure you view the diff with "Hide whitespace changes" set!)

Codecov Report

Merging #225 into master will decrease coverage by <.01%.
The diff coverage is 88.88%.

@@             Coverage Diff              @@
##             master     #225      +/-   ##
============================================
- Coverage      10.3%   10.29%   -0.01%
  Complexity      542      542
============================================
  Files           353      353
  Lines         16491    16490       -1
  Branches       2026     2026
============================================
- Hits           1699     1698       -1
  Misses        14621    14621
  Partials        171      171

Impacted Files
Coverage Δ
Complexity Δ

...c/cam/cl/dtg/isaac/quiz/IsaacNumericValidator.java
78.97% <88.88%> (-0.12%)
43 <1> (ø)

Continue to review full report at Codecov.

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

Moving entirely to arbittary precision libraries seems quite sensible, but not in this PR. So I think this is good to go!


The mutation/custom read action fails in simple data layer resource

Describe the bug
After upgrading to the ash v3 and ash_graphql v1.1, update mutation with read_action is having an issue in our simple data layer resource
actions do
  default [:read, :update]

  read :read_post do
    ...
  end

  update :update_post do
    ...
  end
end

mutations do
  update :update_post, :update_post, read_action: :read_post, identity: false
end

It always defaults to the primary action even though it's specified to use the custom read action. If the custom read action is set to primary then it works. Here is the error message.
11:29:51.300 [error] a55f43fb-2809-4315-bd89-30a1382d2636: Exception raised while resolving query.

** (Ash.Error.Framework) Framework Error

* No data provided to resource Elixir.SomePackage.Post. Perhaps you are missing a call to `Ash.DataLayer.Simple.set_data/2`?.

Another common cause of this is failing to add a data layer for a resource. You can add a data layer like so:

`use Ash.Resource, data_layer: Ash.DataLayer.Ets`

To Reproduce
I've created a test for this in my forked repo
https://github.com/jichon/ash_graphql

I forgot to mention. I've added the test to update_test.ex

Fix released in 1.1.1


Idle, Translators, Nickname issues

Hello,
As I see, if you are on idle and write even '/whois ' then your idle are cleared
As you see: 0 days 0 hours 0 mins 4 secs [signon: Sat Jun 13 23:00:50 2020]
Also, there is an issue on 'irc/handlers.go', if you have two authors on Translators then on '/info' will show just one, '\n' variable don't work on it,
      tlines := server.Languages().Translators()
      if 0 < len(tlines) {
              rb.Add(nil, server.name, RPL_INFO, nick, client.t("Translators:"))
              for _, line := range tlines {
                      rb.Add(nil, server.name, RPL_INFO, nick, "    "+strings.Replace(line, "\n", ", ", -1))
              }
              rb.Add(nil, server.name, RPL_INFO, nick, "")
      }

There is a problem on '/nick' too,
23:35 !irc.math.whitehats.net -NICK- "><script changed nickname to "><script>alert(1)</script>
You can restricted that, but not entire name:
        restrictedNicknames = []string{
                "=scene=", // used for rp commands
                "Global",  // global announcements on some networks
                // common services not implemented by us:
                "MemoServ", "BotServ", "OperServ",
                "a", "b", "c", "d", "e", "f", "g", "h", "i", "j", "k", "l", "m", "n", "o", "p", "q", "r", "s", "t", "u", "v", "w", "x", "y", "z", "\"", ">", "<", "$", "=", ";",
        }

And this affect IRCCloud too and/or others IRC Web clients,
https://whitehat.ro/dl/2020-06-13_23-49-56.png

Thanks,

Awesome, thanks for the in-depth report!
Hmm, I'm not sure whether the idle thing was an active decision to try and improve privacy or not, but will take a look and can make some commands not reset idle time if we need to.
Definitely makes sense for us to handle \n's in the TL name info, thanks for the shout there (replace \n with space, collapse multiple spaces to one?)
Hum. Yeah. <> shouldn't be allowed in nicks for sure, and given that / is used for commands it probably shouldn't be allowed as the first character in a nickname either.

Thanks for the report.
Have you reported the nickname issue to irccloud and/or other affected web clients? I'm not sure if I want to do a server-side mitigation here, but even if we do, the issue will still exist in Oragono 2.1.0 and lower.

Honestly I'd also aim towards disallowing the characters <> because they're extremely traditional nickname-surrounding-characters in clients. Seeing messages like   <<dan>> hi or <dan>> yo feels like a pretty good way to confuse users/chanops. But eh, might be worth getting client dev opinions on that.

Thanks for the report.
Have you reported the nickname issue to irccloud and/or other affected web clients? I'm not sure if I want to do a server-side mitigation here, but even if we do, the issue will still exist in Oragono 2.1.0 and lower.

Yes, they fixed.

Definitely makes sense for us to handle \n's in the TL name info, thanks for the shout there (replace \n with space, collapse multiple spaces to one?)

Yes, I know what the code do but that don't do, I am on ro-RO-lang.json, and my name are not showing up on '/info'

I see there's a problem on NickServ too,
13:44 !irc.math.whitehats.net -ACCOUNT- Client [zmeu__!~ZmEu@whitehat/zmeu] registered account [zmeu__]
13:44 !irc.math.whitehats.net -ACCOUNT- Client [zmeu__!~ZmEu@whitehat/zmeu] logged into account [zmeu__]

13:44 <zmeu__> register ZmEu pazzw0rd123
13:44 -NickServ(NickServ@localhost)- Account created
13:44 -NickServ(NickServ@localhost)- You're now logged in as zmeu__
13:44 <zmeu__> info zmeu
13:44 -NickServ(NickServ@localhost)- Account: zmeu_
13:44 -NickServ(NickServ@localhost)- Registered at: Sun, 01 Jan 2006 01:01:01 UTC
13:44 -NickServ(NickServ@localhost)- Additional grouped nick: zmeu
13:44 -NickServ(NickServ@localhost)- Additional grouped nick: z

Yeah definitely, the instructions there are just the new steps we should go through when we do fix it ehe

As I see there's an issue on NickServ too,

16:48 <ZmEu> set zmeu_ 123pazz0000rd
16:48 <*sasl> Username has been set to [zmeu_]
16:48 <*sasl> Password has been set to [123pazz0000rd]

16:48 -!- * <script>alert('xss');</script> You are now logged in as <script>alert('xss');</script>
16:50 -NickServ(NickServ@localhost)- Successfully unregistered account <script>alert('xss');</script>
16:50 -!- * zmeu_ You are now logged in as zmeu_


[GSSOC'23] Footer Section

This website also need Footer Section .
Why Footer Section is important? -:
Sure! Here are the five points in a shorter format:

Navigation and Site Map: The footer provides easy navigation and a site map, helping users find information quickly.

Contact Information: Display important contact details like phone numbers, email addresses, and social media links.

Additional Links and Resources: Showcase related links, partners, and important pages that don't fit in the main navigation.

Copyright and Legal Information: Include copyright notices, disclaimers, and terms of service to protect intellectual property and ensure compliance.

Design Consistency and Branding: Maintain a unified design with the website logo, brand colors, and a brief description in the footer.

Can you assign this issue to me under GSSOC'23.

Can you assign this issue to me under GSSOC'23.

I will


Remove debian-9 as it is now deprecated

Should fix errors like The resource 'projects/debian-cloud/global/images/family/debian-9' was not found.

The presubmit failure is caused by the known SLES 12 repo signature verification issue.


Improve verboisty on hitting domain conflict when creating web app or functions app

User: "mvn:deploy says that the function app does'nt exists and tries creating it. Then it runs into an error that the function app already exists. v 1.41.0"

already fixed by checking name availability


Document paste keyboard shortcuts

As part of implementing pasting that preserves formatting, we should document the two ways to paste into Zulip.

Paste formatted text (Ctrl/Cmd + V)
Paste as plain text (Ctrl+Shift+V / Command-Option-Shift-V on a Mac)

This documentation should be added to https://zulip.com/help/keyboard-shortcuts and the ? help menu. We can try putting into "The Basics" section, just below "New direct message".

@zulipbot claim

@zulipbot abandon

@zulipbot claim

claim

@zulipbot claim

@ArfaNada We noticed that you have not made any updates to this issue or linked PRs for 10 days. Please comment here if you are still actively working on it. Otherwise, we'd appreciate a quick @zulipbot abandon comment so that someone else can claim this issue and continue from where you left off.
If we don't hear back, you will be automatically unassigned in 4 days. Thanks!

Sorry for the delay,I am still working on it


VSTest Alias does not work when only VS 2022 Preview is installed

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched issues to ensure it has not already been reported

Cake runner
Cake .NET Tool
Cake version
2.0.0
Operating system
Windows
Operating system architecture
64-Bit
CI Server
none
What are you seeing?
VSTest errors out with the exception "Could not load executable".
One of my machines is running VS 2022-preview, one is running VS 2022-professional. Both have VS2019 build tools installed additionally.
What is expected?
Since VSTest is part of VS2022 (all editions) and VS2019 build tools I would expect VSTest to work on both machines.
This bug is partially related to #3708 and #3464 where probably VS2022 and VS2019 build tools should have been added to https://github.com/cake-build/cake/blob/a7f023d7ddfeb0dbd24e86bef89eb4f402380584/src/Cake.Common/Tools/VSTest/VSTestRunner.cs#L153
Steps to Reproduce
Remove all VS versions but [VS2022 | VS2019 build tools] and use VSTest
Use the following task:
Task("Default")
.Does(() => {
   var files = GetFiles(File("**/*.csproj").Path.FullPath);
   VSTest(files);
});

Output log
Error: System.AggregateException: One or more errors occurred. (VSTest: Could not locate executable.)
 ---> Cake.Core.CakeException: VSTest: Could not locate executable.
   at Cake.Core.Tooling.Tool`1.RunProcess(TSettings settings, ProcessArgumentBuilder arguments, ProcessSettings processSettings) in C:\projects\cake\src\Cake.Core\Tooling\Tool.cs:line 164
   at Cake.Core.Tooling.Tool`1.Run(TSettings settings, ProcessArgumentBuilder arguments, ProcessSettings processSettings, Action`1 postAction) in C:\projects\cake\src\Cake.Core\Tooling\Tool.cs:line 81
   at Cake.Core.Tooling.Tool`1.Run(TSettings settings, ProcessArgumentBuilder arguments) in C:\projects\cake\src\Cake.Core\Tooling\Tool.cs:line 60
   at Cake.Common.Tools.VSTest.VSTestRunner.Run(IEnumerable`1 assemblyPaths, VSTestSettings settings) in C:\projects\cake\src\Cake.Common\Tools\VSTest\VSTestRunner.cs:line 57
   at Cake.Common.Tools.VSTest.VSTestAliases.VSTest(ICakeContext context, IEnumerable`1 assemblyPaths, VSTestSettings settings) in C:\projects\cake\src\Cake.Common\Tools\VSTest\VSTestAliases.cs:line 120
   at Cake.Common.Tools.VSTest.VSTestAliases.VSTest(ICakeContext context, IEnumerable`1 assemblyPaths) in C:\projects\cake\src\Cake.Common\Tools\VSTest\VSTestAliases.cs:line 92
   at Submission#0.VSTest(IEnumerable`1 assemblyPaths)
   at Submission#0.<<Initialize>>b__0_2()
   at Cake.Core.CakeTaskBuilderExtensions.<>c__DisplayClass31_0.<Does>b__0(ICakeContext _) in C:\projects\cake\src\Cake.Core\CakeTaskBuilder.Execution.cs:line 26
   at Cake.Core.CakeTaskBuilderExtensions.<>c__DisplayClass34_0.<Does>b__0(ICakeContext context) in C:\projects\cake\src\Cake.Core\CakeTaskBuilder.Execution.cs:line 81
   at Cake.Core.CakeTask.Execute(ICakeContext context) in C:\projects\cake\src\Cake.Core\CakeTask.cs:line 119
   at Cake.Core.DefaultExecutionStrategy.ExecuteAsync(CakeTask task, ICakeContext context) in C:\projects\cake\src\Cake.Core\DefaultExecutionStrategy.cs:line 72
   at Cake.Core.CakeEngine.ExecuteTaskAsync(ICakeContext context, IExecutionStrategy strategy, Stopwatch stopWatch, CakeTask task, CakeReport report) in C:\projects\cake\src\Cake.Core\CakeEngine.cs:line 322
   at Cake.Core.CakeEngine.ExecuteTaskAsync(ICakeContext context, IExecutionStrategy strategy, Stopwatch stopWatch, CakeTask task, CakeReport report) in C:\projects\cake\src\Cake.Core\CakeEngine.cs:line 345
   at Cake.Core.CakeEngine.ExecuteTaskAsync(ICakeContext context, IExecutionStrategy strategy, Stopwatch stopWatch, CakeTask task, CakeReport report) in C:\projects\cake\src\Cake.Core\CakeEngine.cs:line 355
   at Cake.Core.CakeEngine.RunTask(ICakeContext context, IExecutionStrategy strategy, CakeTask task, String target, Stopwatch stopWatch, CakeReport report) in C:\projects\cake\src\Cake.Core\CakeEngine.cs:line 258
   at Cake.Core.CakeEngine.RunTargetAsync(ICakeContext context, IExecutionStrategy strategy, ExecutionSettings settings) in C:\projects\cake\src\Cake.Core\CakeEngine.cs:line 221
   at Cake.Cli.BuildScriptHost`1.RunTargetAsync(String target) in C:\projects\cake\src\Cake.Cli\Hosts\BuildScriptHost.cs:line 73
   at Cake.Core.Scripting.ScriptHost.RunTarget(String target) in C:\projects\cake\src\Cake.Core\Scripting\ScriptHost.cs:line 105
   at Submission#0.<<Initialize>>d__0.MoveNext()
--- End of stack trace from previous location ---
   at Microsoft.CodeAnalysis.Scripting.ScriptExecutionState.RunSubmissionsAsync[TResult](ImmutableArray`1 precedingExecutors, Func`2 currentExecutor, StrongBox`1 exceptionHolderOpt, Func`2 catchExceptionOpt, CancellationToken cancellationToken)
   at Microsoft.CodeAnalysis.Scripting.Script`1.RunSubmissionsAsync(ScriptExecutionState executionState, ImmutableArray`1 precedingExecutors, Func`2 currentExecutor, Func`2 catchExceptionOpt, CancellationToken cancellationToken)
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
   at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
   at Cake.Infrastructure.Scripting.RoslynScriptSession.Execute(Script script) in C:\projects\cake\src\Cake\Infrastructure\Scripting\RoslynScriptSession.cs:line 86
   at Cake.Core.Scripting.ScriptRunner.Run(IScriptHost host, FilePath scriptPath) in C:\projects\cake\src\Cake.Core\Scripting\ScriptRunner.cs:line 172
   at Cake.Features.Building.BuildFeature.RunCore(IRemainingArguments arguments, BuildFeatureSettings settings) in C:\projects\cake\src\Cake\Features\Building\BuildFeature.cs:line 99
   at Cake.Features.Building.BuildFeature.Run(IRemainingArguments arguments, BuildFeatureSettings settings) in C:\projects\cake\src\Cake\Features\Building\BuildFeature.cs:line 49
   at Cake.Commands.DefaultCommand.Execute(CommandContext context, DefaultCommandSettings settings) in C:\projects\cake\src\Cake\Commands\DefaultCommand.cs:line 73

MSTest has a very similar logic, perhaps it needs modification, too.


Add Link Call node

[x] New feature (non-breaking change which adds functionality)

Proposed changes
Adds a new 'link call' node that can be used to 'call' a link-in node and get a response.
To support this behaviour, the link-out node has an option to 'return to sender'. When in that mode, if cannot be wired to link-in nodes and will send any messages it receives that originated from a link-call node, back to the calling node.
There are some specific restrictions in  behaviour:

a link-call node can only send to one link-in node. It cannot be used to trigger multiple flows and somehow combine the results. (That could be considered in the future)
the link-call node can be configured with a timeout, default 30 seconds. If a response is not received in that time, it will call node.error() and trigger any Catch nodes in scope. Note that if the called flow then does response after the timeout, its response will be sent on as normal, but the complete node will not get triggered.
the called flow must return a message via a link-out node.

if the single message passed to the link-call node results in multiple messages being passed to the link-return (such as a Split node inside the called flow), then the Complete node will be triggered after the first of the return messages is received. Similarly, the timeout handling will be cleared when that first message comes back.

Coverage decreased (-0.09%) to 67.592% when pulling b01fd24e153c1ada17d9644d984538bfa0d4c3b2 on link-call into 36bb172f298ea214f2159415ab1abfcedb8f95ea on dev.

Decided to add timeout option to the link-call node for the initial version as it makes cleaning up internal state easier for calls that never return.

Help text updated. Now ready to merge.

looks good to start. Mentally I think of calling down to a subroutine and tend to lay out flows with link function further down the page so the icon seems at odds to me - but functionally works fine. Happy to merge

Great to see this merged.
Had a bit play & did a recursive string reverse

[{"id":"346af0519ba4be19","type":"link in","z":"b872cb4b.5a6448","name":"string reverse","links":[],"x":430,"y":420,"wires":[["bd8f78bd4dd96f52"]],"l":true},{"id":"65131b465df6cf7d","type":"inject","z":"b872cb4b.5a6448","name":"olleH","props":[{"p":"payload"},{"p":"topic","vt":"str"}],"repeat":"","crontab":"","once":false,"onceDelay":0.1,"topic":"","payload":"olleH","payloadType":"str","x":410,"y":320,"wires":[["20a876e9039cbf4a"]]},{"id":"bd8f78bd4dd96f52","type":"switch","z":"b872cb4b.5a6448","name":"length == 1","property":"payload.length","propertyType":"msg","rules":[{"t":"eq","v":"1","vt":"num"},{"t":"else"}],"checkall":"true","repair":false,"outputs":2,"x":610,"y":420,"wires":[["43ef4c342967de07"],["d06dd3480408574d"]]},{"id":"1aa2b1be9675d866","type":"link out","z":"b872cb4b.5a6448","name":"","mode":"return","links":[],"x":1675,"y":440,"wires":[]},{"id":"8ee4e912a201841f","type":"link call","z":"b872cb4b.5a6448","name":"","links":["346af0519ba4be19"],"timeout":"30","x":1200,"y":440,"wires":[["231862c8feba4aa3"]]},{"id":"261e9bc5cd4ec0b8","type":"change","z":"b872cb4b.5a6448","name":"slice 1 off end","rules":[{"t":"set","p":"payload","pt":"msg","to":"$substring(payload, 0, ($length(payload) - 1))\t\t","tot":"jsonata"}],"action":"","property":"","from":"","to":"","reg":false,"x":1000,"y":440,"wires":[["8ee4e912a201841f"]]},{"id":"20a876e9039cbf4a","type":"link call","z":"b872cb4b.5a6448","name":"","links":["346af0519ba4be19"],"timeout":"30","x":580,"y":340,"wires":[["82359688f01ff310"]]},{"id":"82359688f01ff310","type":"debug","z":"b872cb4b.5a6448","name":"","active":true,"tosidebar":true,"console":false,"tostatus":true,"complete":"payload","targetType":"msg","statusVal":"payload","statusType":"auto","x":770,"y":340,"wires":[]},{"id":"231862c8feba4aa3","type":"change","z":"b872cb4b.5a6448","name":"append","rules":[{"t":"set","p":"payload","pt":"msg","to":"/*  $substring(a[$count(a)-1], $length(a[$count(a)-1])-1, 1) & payload */\t\t/* $substring(payloadBefore, $length(payloadBefore)-1, 1) & payload */\t/* $substring(a[$count(a)-1], $length(a[$count(a)-1])-1, 1) & payload*/\tlastcharArray[$count(lastcharArray)-1] & payload","tot":"jsonata"}],"action":"","property":"","from":"","to":"","reg":false,"x":1380,"y":440,"wires":[["fa34b41b05dcaee0"]]},{"id":"fa34b41b05dcaee0","type":"function","z":"b872cb4b.5a6448","name":"pop last char","func":"msg.lastcharArray && msg.lastcharArray.pop()\nreturn msg\n\n ","outputs":1,"noerr":0,"initialize":"","finalize":"","libs":[],"x":1550,"y":440,"wires":[["1aa2b1be9675d866"]],"icon":"font-awesome/fa-copy"},{"id":"d06dd3480408574d","type":"function","z":"b872cb4b.5a6448","name":"Push last char","func":"\nmsg.lastcharArray = msg.lastcharArray || []\nmsg.lastcharArray.push(msg.payload.substr(-1))\nreturn msg\n\n","outputs":1,"noerr":0,"initialize":"","finalize":"","libs":[],"x":800,"y":440,"wires":[["261e9bc5cd4ec0b8"]],"icon":"font-awesome/fa-copy"},{"id":"43ef4c342967de07","type":"link out","z":"b872cb4b.5a6448","name":"","mode":"return","links":[],"x":735,"y":400,"wires":[]},{"id":"38408f72199ae701","type":"inject","z":"b872cb4b.5a6448","name":"eybdooG","props":[{"p":"payload"},{"p":"topic","vt":"str"}],"repeat":"","crontab":"","once":false,"onceDelay":0.1,"topic":"","payload":"eybdooG","payloadType":"str","x":400,"y":360,"wires":[["20a876e9039cbf4a"]]}]

Obviously it is of little use other than to demonstrate recursion is possible :)


About scroll event

Hello 👋
Interesting utility, I'm keen to give it a spin.
There is something I would like to share about scroll event. I've read some docs recently and I've noticed this:

You don't need to worry about the value of passive for the basic scroll event. Since it can't be canceled, event listeners can't block page rendering anyway.

from here
and another note from here

Note: The basic scroll event cannot be canceled, so it does not need to be set passive.

What are your thoughts on this matter?

Hey @plesiecki!
I think you might be right - though I cannot find any other sources about that (especially, nothing connected to the standard itself). I cannot tell, if this behaviour (not scroll event being passive by default) is true in every major browser (chromium-based ones, firefox, safari).
In light of this, I'd keep the scroll event as a part of this library.
I'm closing this issue as answered (and also because it's so old I don't expect you to answer 😄). But, if you have other arguments and/or opinion on this topic - I'm happy to reopen and discuss it here!


pluralize-renamefiles

Current dependencies on/for this PR:

main

PR #541   👈

This stack of pull requests is managed by Graphite.


A few bugs with PossibleValuesfield

Bug in GraphicsStateParameter.@LW: Its PossibleValues field says [fn:Eval(@LW>0)], but line width is non-negative and zero is meaningful. I guess it should be [fn:Eval(@LW>=0)] instead.

The same might apply to GraphicsStateParameter.@ML as well?

In Annot*.{@CA, @ca}, their PossibleValues currently is<EMAIL_ADDRESS>&& (@ca<1.0))] but should be<EMAIL_ADDRESS>&& (@ca<=1.0))], since 1.0 is default value and supported.

In Projection.@OS, its PossibleValues currently is [fn:Eval(@OS>=0)], but zero is invalid here, since the spec specifies this is a positive number.
In Projection.@PS, it should also be positive when using the number form, but PossibleValues field is empty.

Thank you for finding and reporting these!
The only one I am unsure about and you also highlight is unclear in the spec, is ML since it is defined to be "... a maximum on the ratio of the miter length to the line width. When the limit is exceeded, the join is converted from a miter to a bevel" so would ML==0 mean that a bevel is always done for non-zero width lines??  I think this is work raising/asking over in pdf-issues.
I also confirmed that all ca/CA PossibleValues are now identical.

Fixed now except for ML

For ML entry, i opened https://github.com/pdf-association/pdf-issues/issues/154 .
Personally now i'm inclined to think the correct range is [fn:Eval(@ML>=1)] (consistente with PostScript).

Waiting for resolution of https://github.com/pdf-association/pdf-issues/issues/154 before I close


Reduce package size

After installing the core package in an empty NPM project using npm install --save @cloud-carbon-footprint/core here is the corresponding node_module directory:
> ls node_modules/ | xargs -I {} du -h -d0 ./node_modules/{} | grep 'M\s' | sort -k1 -n
1.1M	./node_modules/ajv
1.3M	./node_modules/@opencensus
1.7M	./node_modules/@cloud-carbon-footprint
1.8M	./node_modules/node-forge
2.2M	./node_modules/@grpc
2.6M	./node_modules/@types
2.6M	./node_modules/underscore
2.9M	./node_modules/date-utils
3.0M	./node_modules/ramda
5.1M	./node_modules/moment
6.3M	./node_modules/google-gax
7.3M	./node_modules/protobufjs
 12M	./node_modules/@azure
 13M	./node_modules/@google-cloud
 62M	./node_modules/aws-sdk
 83M	./node_modules/googleapis

# total: 229M

I was wondering if you are ok to try to reduce it, but I totally understand it is a won't fix issue on your side.
Possible Implementation
As a first goal, it seems googleapis is only used one time here:
https://github.com/cloud-carbon-footprint/cloud-carbon-footprint/blob/68a27b70efcd19dcd19a9b92048ecf05358ec09f/packages/core/src/application/GCPCredentials.ts#L11
The package has been recently broken in smaller parts:

https://github.com/googleapis/google-api-nodejs-client/issues/806#issuecomment-790920924
https://github.com/googleapis/google-api-nodejs-client/pull/2557

If ok for you then I can have a look.

PoC PR: https://github.com/cloud-carbon-footprint/cloud-carbon-footprint/pull/276
I am able to generate an ID token by calling the method in its own script, but having someone running an e2e would be great


What is a circle?

The GeoJSON spec committee considered adding circles and there's quite a bit of discussion about them on the old list - for example, http://lists.geojson.org/pipermail/geojson-geojson.org/2011-July/000564.html
This added circle type is missing the pretty critical detail: what kind of circle is it? Is it a spherical cap? A geodesic circle? Or a circle that'll present as a circle in the web mercator projection? The three are very distinct shapes, and without a clear definition, it seems like a pretty vague addition to the otherwise-precise specification.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d0bcb412-cf2d-607b-8538-614a088f281e
Version Independent ID: 9c274da1-f658-3f34-4417-e5f93ecb1e7f
Content: Extending GeoJSON geometries in Azure Maps
Content Source: articles/azure-maps/extend-geojson.md
Service: azure-maps
GitHub Login: @sataneja
Microsoft Alias: sataneja

@tmcw Thanks for reporting! I have assigned the issue to the content author to investigate and update as appropriate.

@tmcw I noticed you closed the issue, was your issue addressed?
Thanks!


Adds changes for Colab support to the ml_ops/stage2/get-started-with-vertex-featstore notebook

Opening a PR for Community Notebooks under the notebooks/community folder:

[x] This notebook has been added to the CODEOWNERS file under the # Community Notebooks section, pointing to the author or the author's team.
[x] Passes all the required formatting and linting checks. You can locally test with these instructions.

Passed Colab and Workbench.


i don't know what's wrong

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

No information provided or steps to replicate - Closing


Fix markdown to point to proper Medium link.

Minor update to README.md to make sure the proper Medium article link is referenced.

Thanks!


Create codeql-analysis.yml

Not really sure how to test this. I guess it's low risk if it doesn't work and can just revert it?

Aye, trying to see how this compares to sonarcloud and do a like for like comparison atm


The size of tensor a (8) must match the size of tensor b (64) at non-singleton dimension

how to fix

Likely a duplicate to https://github.com/pharmapsychotic/clip-interrogator-ext/issues/33

Also this, a closed issue
https://github.com/pharmapsychotic/clip-interrogator-ext/issues/21

use the previous commit 1ecedb5 which uses clip-interrogator 0.6.0.

I can't get it to work either. I have tried both the lastest version and 1ecedb5

interesting, in "venv\Libsite-packages" delete the folders (should be 2) called "clip_interrogator" and "clip_interrogator-0.6.0" (if clip_interrogator-0.5.4 exists, the same). Although it is also true that I use a more recent version of transformers.

Which version of transfers do you use? Are you using it with 1ecedb5

Sorry i didn't see your message! currently i use transformers==4.29.2, but you can use previous versions, always higher than =4.25.1, and yes that commit (i made a fork for myself, i didn't want to unintentionally update the extension and have it break again). i had no problem. if you want to update transformers edit requirements_versions.txt (don't forget to restore it before updating AUTO1111 or whatever you are using).
you may also need to delete the folders "clip_interrogator" and "clip_interrogator-0.6.0.dist-info" in \venv\Lib\site-packages.

Sorry i didn't see your message! currently i use transformers==4.29.2, but you can use previous versions, always higher than =4.25.1, and yes that commit (i made a fork for myself, i didn't want to unintentionally update the extension and have it break again). i had no problem. if you want to update transformers edit requirements_versions.txt (don't forget to restore it before updating AUTO1111 or whatever you are using). you may also need to delete the folders "clip_interrogator" and "clip_interrogator-0.6.0.dist-info" in \venv\Lib\site-packages.

same issue with your fork...Loading CLIP model...
Loaded CLIP model and data in 3.89 seconds.
The size of tensor a (8) must match the size of tensor b (64) at non-singleton dimension 0

Error happens again as reported in issue https://github.com/pharmapsychotic/clip-interrogator-ext/issues/64.


In SzAbstractFactory change from createSz* to create*

See https://github.com/senzing-garage/sz-sdk-go/issues/194

Pull requests:

https://github.com/senzing-garage/senzing-tools/pull/257


Initial port of react-svg-text to vx.  Issue #16

:boom: Breaking Changes

Removes <TextWrap>, <TextOutline> and <TextBackground> components, which were incomplete

:rocket: Enhancements

Adds new <Text>, with the following features

Word-wrapping (when width prop is defined)
Vertical alignment (verticalAnchor prop)
Rotation (angle prop)
Scale-to-fit text (scaleToFit prop)

This is mostly just a direct port of my react-svg-text component, which I plan to deprecate in favor of @vx/text (since it can be installed standalone as well).

A few points of discussion before merging/releasing:

The tests rely on the ability to query the DOM for an element's size (using getComputedTextLength).  Since jest currently only supports jsdom, which lacks this feature (although puppeteer support might be coming soon), I've commented out all the tests so as to not break the build
I wasn't sure of the best way to demo the capabilities of the component in a concise manner (like the other tiles) so for now I just ported my Demo page as it's currently the best way to understand what the component provides.  Feel free to replace and/or remove the demo, although I find it useful when developing

Btw, I noticed I'm receiving the following error for the Fit example

Fit works fine when enabling at the top, but the standalone example is throwing this error (I'm assuming from next.js / SSR.  This might be an actual bug, but I was having trouble identifying the cause, especially if it's only during SSR.

Should we replace all/most/some of our <text> usage with <Text>, especially for things like Axis which pass tickLabelPropsObj?
I'm not sure if I like angle or rotate as the name for the prop to rotate.  Thoughts?
I was having trouble using a 2.x version of reduce-css-calc, even after they resolved this issue, although I'm getting a different error now.  This not a big issue, and we're using the same version as react-svg-text, but I was hoping to update it while making the transition.  At some point I'll try to submit another issue to work through the current issue, but it's not a make/break to get this into vx.

Sorry for the lengty PR message, I mostly wanted to get some notes down while they were fresh.  A lot of these items above we can work through after merging if you'd like as well, or I can make some changes before merging.

Wonderful! Thanks for this @techniq! This is a huge improvement and I'm excited to welcome your excellent react-svg-text work to the vx family!

I've commented out all the tests so as to not break the build

makes sense and sounds good for now.

Btw, I noticed I'm receiving the following error for the Fit example

I'll take a look after merging and see what's up.

Should we replace all/most/some of our  usage with , especially for things like Axis which pass tickLabelPropsObj?

Yeah probably where it makes sense/simplifies the API. We can do this in a later PR.

I'm not sure if I like angle or rotate as the name for the prop to rotate. Thoughts?

I'm fine with leaving it as angle right now and see if folks get caught up on it.

I was having trouble using a 2.x version of reduce-css-calc

No problem, something we can keep an eye on.


HADOOP-17614. Bump netty to the latest 4.1.61.

HADOOP-17614

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 39s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
32m 55s

trunk passed

+1 :green_heart:
compile
0m 23s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
0m 21s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
mvnsite
0m 28s

trunk passed

+1 :green_heart:
javadoc
0m 24s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
0m 24s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
shadedclient
48m 20s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 17s

the patch passed

+1 :green_heart:
compile
0m 13s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
0m 13s

the patch passed

+1 :green_heart:
compile
0m 12s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
javac
0m 12s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 15s

the patch passed

+1 :green_heart:
xml
0m  2s

The patch has no ill-formed XML file.

+1 :green_heart:
javadoc
0m 14s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
0m 14s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
shadedclient
14m 21s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
0m 19s

hadoop-project in the patch passed.

+1 :green_heart:
asflicense
0m 35s

The patch does not generate ASF License warnings.

67m  0s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2850/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2850

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell xml

uname
Linux 4d01276abd95 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 555c0a9bf2a6a0550d40aa1043fadb073e69e84f

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2850/1/testReport/

Max. process+thread count
688 (vs. ulimit of 5500)

modules
C: hadoop-project U: hadoop-project

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2850/1/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 40s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 17s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
20m 29s

trunk passed

+1 :green_heart:
compile
20m 54s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
18m  9s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
mvnsite
4m  5s

trunk passed

+1 :green_heart:
javadoc
3m 15s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
3m 41s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
shadedclient
98m 25s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 31s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 40s

the patch passed

+1 :green_heart:
compile
20m 11s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
20m 11s

the patch passed

+1 :green_heart:
compile
18m  7s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
javac
18m  7s

the patch passed

-1 :x:
blanks
0m  0s
/blanks-eol.txt
The patch has 1 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

+1 :green_heart:
mvnsite
4m  3s

the patch passed

+1 :green_heart:
xml
0m  6s

The patch has no ill-formed XML file.

+1 :green_heart:
javadoc
3m 10s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
3m 40s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
shadedclient
22m 23s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
0m 35s

hadoop-project in the patch passed.

+1 :green_heart:
unit
2m 36s

hadoop-hdfs-client in the patch passed.

-1 :x:
unit
250m 18s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
unit
1m  8s

hadoop-yarn-csi in the patch passed.

+1 :green_heart:
asflicense
1m 13s

The patch does not generate ASF License warnings.

426m  1s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.namenode.ha.TestStandbyCheckpoints

hadoop.hdfs.server.datanode.TestDirectoryScanner

hadoop.hdfs.qjournal.server.TestJournalNodeRespectsBindHostKeys

hadoop.hdfs.server.namenode.snapshot.TestNestedSnapshots

hadoop.hdfs.server.blockmanagement.TestBlockTokenWithDFSStriped

hadoop.hdfs.server.datanode.TestBlockRecovery

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2850/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2850

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell xml

uname
Linux d8a19a1a2438 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / e6f43ab93e0cf3cfd47ed84347ddeb208bc2a4d3

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2850/2/testReport/

Max. process+thread count
3278 (vs. ulimit of 5500)

modules
C: hadoop-project hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs hadoop-yarn-project/hadoop-yarn/hadoop-yarn-csi U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2850/2/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Tests failures doesn't seems related, @jojochuang you plan to test something more, or waiting for something/someone.
If not, this seems good to go

thanks I think this is good. I'll push another commit to revert the unrelated change.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m  0s

Docker mode activated.

-1 :x:
patch
0m 21s

https://github.com/apache/hadoop/pull/2850 does not apply to trunk. Rebase required? Wrong Branch? See https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hadoop/pull/2850

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2850/3/console

versions
git=2.17.1

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.


generators.toPlist doesn't do XML encoding

The generators.toPlist function generates an XML document but does not seem to encode reserved XML characters, which can cause the generated document to be malformed. See https://github.com/nix-community/home-manager/issues/1976 for real-world example.
My proposed solution is to do something like
escapeXml = replaceStrings ["\"" "'" "<" ">" "&"] [""" "'" "<" ">" "&"];
…
str = ind: x: literal ind "<string>${escapeXml x}</string>
key = ind: x: literal ind "<key>${escapeXml x}</key>";

CC @matthewbauer

not stale


remove vendoring and add build script

Good job! I would also mention this build.sh in Readme

Good point! Added it to the readme.


Program/App shortcuts like Parallels desktop

Please browse the existing issues and filter by 'enhancement' to make sure this has not already been suggested!
Add optional folders per VM when you use UTM virtual machines that has the shortcuts to all windows (win32 and UWP/Metro/Store/Modern and Windows Subsystem for Android apps), Linux, macOS, and android applications you have installed. For windows Classic, macOS before Big Sur, and Linux and legacy android apps, it creates a frame with either a color or the OS wallpaper. There should be no ugly icon on the side and no tinted name to keep it seem less. When you launch it, it opens the app in the VM. This may only be possible with a driver however, or command prompt.

Also add shortcuts to the virtual machines themselves in a folder.

This is not possible currently in an App Sandbox app. Note that Parallels’ App Store version doesn’t support this either.

FYI, I've made apps that use the utm:// uris to launch the various VMs.  I could probably make a generic app that you could use to walk through UTM VMs and create .webloc shortcuts for each one.
If there's enough interest, I'll set some of this up publicly as helper software for UTM.  So far, I'm just generating pre-configured .utm files with custom icons and screenshots for various OS versions (bring your own install iso).

@adespoton I believe the OP is asking for an alias to apps installed within the guest OS, not to launch the VM itself (which can be achieved with Apple Shortcuts).

This is not possible currently in an App Sandbox app. Note that Parallels’ App Store version doesn’t support this either.

Exclusively for the macOS version

Closing for future issue to bundle winapps


add/rename metadata fields for datasets

Describe the improvement
This overview of metadata fields for datasets is a simplified version of the overview shared earlier.
This file contains the following fields to be addressed, as well as examples and info on expected behavior (DOI-based and alternative identifier-based). Fields with "1" and in bold are priority fields.

[x] Data - Identifier 1 - V
[x] Data - Identifier type 1 - V (auto)
[ ] Data - Alternative identifier no
[ ] Data - Identifier type no
[x] Data - Abstract 1
[x] Data - Abstract language
[x] Data - Creator Name 1 - V
[x] Data - Creator Identifier
[x] Data - Creator Affiliation 1 - V (auto)
[x] Data - Title 1 - V
[x] Data - Publisher 1
[x] Data - Publication year 1 - V
[x] Data - Embargo date
[ ] Data - Date type
[x] Data - Keyword(s) 1 - V
[ ] Data - Contributor type
[ ] Data - Contributor name
[ ] Data - Contributor NameIdentifier
[ ] Data - Contributor NameIdentifierScheme
[x] Data - Format 1 - V
[x] Data - Licenses - V (from 2023 onwards)
[x] Data - Access Rights 1 - V
[x] Data - link to Project(s) 1 - V
[x] Data - link to Publication 1
[ ] Data - link between Datasets
[ ] Data - Research disciplines

"rightsList": [
{
"rights": "Creative Commons Attribution 4.0 International",
"rightsUri": "https://creativecommons.org/licenses/by/4.0/legalcode",
"schemeUri": "https://spdx.org/licenses/",
"rightsIdentifier": "cc-by-4.0",
"rightsIdentifierScheme": "SPDX"
},
{
"rights": "Open Access",
"rightsUri": "info:eu-repo/semantics/openAccess"
for licenses: only extract SPDX licenses and their rightsIdentifier
for access rights: only extract if rightsUri is an openAIRE access right (See example above and excel)

predefined list of licenses: https://github.com/spdx/license-list-data/blob/master/json/licenses.json
predefined list of formats: media types https://www.iana.org/assignments/media-types/media-types.xhtml

@nics can embargo date and data type (of the embargo date) by added for datasets, as is done for publications? This is needed to decide on which datasets to provide to FRIS at a certain point in time.

predefined list of contributors: Contact Person
Creator
Data Collector
Data Curator
Data Manager
Distributor
Editor
Other
Producer
Project Leader
Project Manager
Project Member
Related Person
Researcher
Rights Holder
Sponsor
Supervisor
Work Package Leader

Stale issue, has a mix of "done", "important now" and "important later" tasks, and tasks that are unclear whether they are important or not at all – and comments that are about different topics. Need to split this out into actionable issues once the intent and priority is clear. Will track this in #1099

[ ]  Data - link between Datasets
[ ]  Data - Research disciplines

These are two metadata fields which are still relevant to implement when time permits, but were not implemented as not set as 1st priority. For the research disciplines, we could borrow info from the project (as mentioned in https://github.com/ugent-library/biblio-backoffice/issues/81), but that is assuming that a project is linked and the scope of the project and the dataset is similar.

Thank you for the info!


Tree priorities

Will aim to review this tomorrow. Could you link the github issue?

Apologies for the belated review here. Looks overall solid, left a few small comments for future reference.


Remove test now covered by zarf init usage of config file

Removes a test that was previously needed to make sure a package in a default path was read. Now that #1039 is merged, we have this covered on init package create / deploy + it was was trampling over that file during this test.

🙏


1.6 is great but flaw with design...

So you have added and fixed all the issues so far.. It wasn't until I added it that I realized people can steal from anyone. I have a world called shops, this is where the perms say they can make the shops. any player can go here and place a buy sign with a purchase price of 1 dollar (on the fourth line) then go to any world and select anyone's chest even if it is on a grief prevention claim then tp back to the sign and steal everything from the chest for 1 dollar. Can you make it so the chest must be placed by the same person that made the sign or so that the chest must be within X amount of blocks of the sign in the same world, or even the chest must be below the wall sign?

I'll fix it right away

Thank you so much for all your work on this plugin and the quick turn around for these issues.

Here is a quickfix, I'll push version 1.8 in a few minutes after I finished with a command revamp and /shop help command that point to the wiki
CarrotShop-1.7-quickfix.jar.zip (you need to unzip it)

Should be fully fixed on 1.8: https://github.com/TheoKah/CarrotShop/releases/tag/1.8

Hmm, i loaded 1.8 and had a player place a chest inside of his claim. I made sure my perms were the same as a normal players (can't override claims or shops). I was able to select his chest and attach it to my sign. I'm not sure if i did something wrong or if it is still not working. I am using sponge vanilla api 5 build 385 and grief prevention <IP_ADDRESS>. I know they aren't the latest versions but I dont have time at the moment to update my test server.

The redstone thing now has normal priority. I was not able to link chests in claimed area with 0 perms (didn't try with grief prevention but with nation). Can you check the perms again?

Sorry I haven't gotten back to you, was working the past few days. I'm not sure what you mean. I have checked the perms and the members have carrotshop.create and basic grief prevention stuff but i dont see anything that would let them do this.

Sorry, didn't see your answer. I was talking about the protection plugin permissions

Can you try this version and tell me if it's better: https://github.com/TheoKah/CarrotShop/releases/tag/1.9

It appears that the issue is fixed with 1.9. Thank you again for all your work on this. Safe to close this one.

Glad to hear that. Sorry it took me so long to fix it.


Add Python 3 support

Installing on Python 3.6.5 gives ConfigParser error which is removed in Python 3, so we need to use either configparser2 or a back-ported configparser.
()~> python --version
Python 3.6.5
() pip install dynamodb-config-store

Collecting dynamodb-config-store
  Using cached https://files.pythonhosted.org/packages/45/b0/ee8344f8d4f6a5592a94d0240aa733a2570bdc7cc97260a71f4d220b4ea5/dynamodb-config-store-0.2.2.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/kw/k_sd90fd56v_ppm425b0rp9h0000gn/T/pip-install-f826pw4m/dynamodb-config-store/setup.py", line 4, in <module>
        from ConfigParser import SafeConfigParser
    ModuleNotFoundError: No module named 'ConfigParser'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/kw/k_sd90fd56v_ppm425b0rp9h0000gn/T/pip-install-f826pw4m/dynamodb-config-store/

Hey!
Happy to see someone using this little lib :smile: and thanks for your email :bowing_man:.
I am not actively using it in any project at the moment. I noticed that you forked the lib. If you feel like taking a stab at making it Python 3 compatible that would be :100: :taco: and :tada:. I'd be more than happy to review and merge.

Thx for the message, Here is the PR
https://github.com/sebdah/dynamodb-config-store/pull/22

@nehemiah-hivery thank you for your contribution on this. I have just now released version 0.3.0 to PyPI.
:bowing_man:


Update msal-net-token-cache-serialization.md

reword No token cache serialization as some developers confuse it with no token cache

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

#label:"aq-pr-triaged"

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

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

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

#sign-off

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


Implement unloadable plugins

Some plugins aren't meant to be dynamically loadable/unloadable. Plus, unloadable plugins should be destroyed at then end (for output destruction)

Resolved in master.


Implement Rb* calculation

Implemented in GUI branch.
Still working on more examples and elaborate the documentation.

Examples made.

Will implement a linear interpolation in the Rb* value. The difference between a constant Rb* and a dynamic one is significant. Constantly recalculating Rb* is very time consuming. Will investigate whether or not a linear interpolation makes any sence.

Linear interpolation makes no difference, so not implemented.


Add funding info

Per https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/displaying-a-sponsor-button-in-your-repository

Funding links provide a way for open source projects to receive direct financial support from their community. We don’t support the use of funding links for other purposes, such as for advertising, or supporting political, community, or charity groups. If you have questions about whether your intended use is supported, please contact GitHub Support.

As-is, I don't think it is appropriate to link to the Stanford donation page for the Eterna research fund. If we want we could reach out to support to look into it more, but I don't think it's worth it at this point. Maybe worth revisiting at some point depending on how our governance-related decisions pan out


Ensure that tests compile as-is before stripping annotations

Fixes #389.

I think that build failure uncovered a broken test suite! Fixing tomorrow :)

Just to confirm, when we manually remove the @Ignore annotations, the compileTestJava is re-run (not skipped) when executing the test task. This means if our manual annotation stripping were to ever change and "break" a class somehow (e.g. we accidentally removed the end of the import statement), the build should fail.
++find . -name '*Test.java'
+for testfile in '`find . -name "*Test.java"`'
+sed s/@Ignore// ./src/test/java/CustomSetTest.java
+mv /tmp/journey-test.sh-unignore_all_tests.txt ./src/test/java/CustomSetTest.java
+gradle test
:compileJava UP-TO-DATE
:processResources UP-TO-DATE
:classes UP-TO-DATE
:compileTestJavaNote: /home/travis/build/exercism/xjava/build/exercism/java/custom-set/src/test/java/CustomSetTest.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details. // note: not marked as up to date!
:processTestResources UP-TO-DATE
:testClasses
:test

Thanks for tackling this one, @stkent!  Thorough and inexpensive.


Uncaught error in client creation

General information

SDK version: 3.83.0
Environment: Sandbox and production
Browser and OS: Chrome 96.0.4664 Windows, Mobile Safari 15.0, IOS,

Issue description
When connection to braintree gateway fails (Both graphql and standard one) promise is in status rejected
client.js, line 126
analytics.sendEvent(promise, 'custom.client.load.initialized');
fails with unhandled rejection
Steps to replicate:
run client create script and in  dev-tools block traffic to
braintree-api.com/graphql and api.sandbox.braintreegateway.com/merchants/
stack trace:
sendAnalyticsEvent @client.js?f06b:2677
Client.initialize  @client.js?f06b:807
create  @client.js?f06b:1418
eval  @client.js?f06b:314

Sounds to me like something on your end is blocking requests to Braintree domains. Do you have a content security policy or a browser extension that may be doing this?
If not, please reach out to our support team with a staging site so we can troubleshoot it: https://developer.paypal.com/braintree/help/

Main thing is that this analytics.sendEvent(promise, 'custom.client.load.initialized'); is being called even in case of failure to connect to Braintree apis which causes unhandled error inside that i cannot process with rest of my code even if I can handle main rejection from client

This should be fixed in v3.84.0.


i cant download it because if i click to download it doesnt want to download it just sends me to code

The README on the repository has a "Download latest release" link.


[SDK repo][pipeline] upgrade autorest.python from 6.0.1 to 6.1.4

https://github.com/Azure/azure-sdk-for-python/issues/25731

/check-enforcer override


State as another class with finder & copy method on it?

Would it hurt to have a function like following, where State is a separate class and have helper functions on it. is this good practice.
var SimpleStore = function(path) {
	function State() {
		this.data = [];
		this.selected = null;
	}

	var p = State.prototype;

	p.copy = function() {
		var ns = new State();
		ns.data = this.data;
		ns.selected = this.selected;

		return ns;
	}

	p.findById: function(id) {
		return _.find(this.data, function(item) {
			return item.id === id;
		});
	}

	var store = Hoverboard({
		init: function(state) {
			GET(path, function(data) {
				store.data(data);
			});

			return new State();
		},

		data: function(state, data) {
			var ns = state.copy();
			ns.data = data;

			return ns;
		},

		select: function(state, selected) {
			var ns = state.copy();
			ns.selected = selected;

			return ns;

		}
	});

	return store;
}

@0419 I hadn't thought of that technique, but you can definitely do that.
Note that you can also put functions in your state, and if you used a plain object for your state, Hoverboard automatically copies into a new object each time.
So something like this would be basically the same, but a bit simpler:
var SimpleStore = function(path) {
    function findById(id) {
        return _.find(this.data, function(item) {
            return item.id === id;
        });
    }

    var store = Hoverboard({
        init: function(state) {
            GET(path, function(data) {
                store.data(data);
            });

            return { data: [], selected: null, findById: findById };
        },

        data: function(state, data) {
            return { data: data };
        },

        select: function(state, selected) {
            return { selected: selected };
        }
    });

    return store;
}

@0419 I just realised that Hoverboard was treating class instances as plain objects, ie. it would copy the properties over when you use a class instance as your state. I only intended that true plain objects (those created with {} or new Object) would do this. I'll fix this bug - so your original example will work better and you won't lose your instance.


Remove requirement of nesting perhaps?

Would be good if one could define types without requiring nesting inside an object:

// e.g. pass array without nesty object
const Point = Type([isNumber, isNumber])

var p = Point('bad', 4) // no Point.Point :)

// or pass a function directly
const SomeNumber = Type(isNumber)
const AlternativePoint = Type([SomeNumber, SomeNumber])
var p = AlternativePoint('bad', 4) // still no nesting

I think you're confusing types with values. If you where to define the boolean type with union-type it would look like this:
var Boolean = Type({True: [], False: []});

Here Boolean is the type and True and False are values. So what you're asking is really to define a value without it belonging to a type.
What benefits do you think this would offer? Maybe a shortcut for defining a type with only a single possible value would be better?

hm, I don't know what I'm asking anymore, nvm. :+1:

No problem!


Create missing APIs

Right now, none of the APIs (DSS C-API, COM, DDLL) support all OpenDSS classes.
The alternative is usually to use the DSSProperty API or drop down to the DSS scripting language. Some classes like LineGeometry have non-trivial properties as described in https://github.com/NREL/OpenDSSDirect.py/issues/19.
Exposing the missing classes would be ideal since it would simplify the usage, remove many type conversions, and reduce DSS scripting language usage.
Since adding this to COM would also benefit users that, e.g., run OpenDSS through Microsoft Office or other restrictive contexts that don't allow using the dss_capi DLL, it would be ideal to create a COM version of the new interface classes. This should be done as such that it is easy to integrate in the official OpenDSS repository if desired down the line, while not forcing additional workload on EPRI.
From a quick look comparing the exposed COM classes and the classes listed in DSS.Classes, the following classes are not exposed (ordered loosely based on priority, feedback is welcome),
(To do for v0.10)

[x] LineGeometry
[x] WireData
[x] LineSpacing
[x] CNData
[x] TSData
[x] Reactor

(For v0.11+)

[x] Storage (basic version added upstream)
[ ] StorageController
[ ] XfmrCode
[ ] Spectrum
[ ] Fault
[ ] ESPVLControl
[ ] ExpControl
[ ] GICLine
[ ] GICTransformer
[ ] GenDispatcher
[ ] GrowthShape
[ ] IndMach012
[ ] InvControl
[ ] PriceShape
[ ] TCC_Curve
[ ] UPFC
[ ] UPFCControl
[ ] VCCS
[ ] VSConverter

Classes that don't expose all properties (most classes actually, listing only ones with explicit requests):

[ ] PVSystem
[ ] Storage

Some classes are not essential since their usage would be rare. There are also auxiliary data classes that are not listed but might be necessary.

Initial experimental support for LineGeometries just added. I'll try to add support for WireData and LineSpacing soon.

In the original OpenDSS code, both COM Direct DLL, many properties use the parser to set properties. For example, see the Transformers API:
https://github.com/PMeira/electricdss-src/blob/e095b6e978eb27c545afb2e5f7437e355a079168/Source/DLL/ImplTransformers.pas#L69-L77
https://github.com/PMeira/electricdss-src/blob/e095b6e978eb27c545afb2e5f7437e355a079168/Source/DLL/ImplTransformers.pas#L309-L330
Since DSS C-API is based on the COM code, currently it replicates that.
With DSS Python, running %timeit transformer.kVA = val yields
6.71 µs ± 68.4 ns per loop (mean ± std. dev. of 7 runs, 100000 loops each)

LineCodes.NormAmps modifies the value direct. Running %timeit linecode.NormAmps = val:
335 ns ± 0.826 ns per loop (mean ± std. dev. of 7 runs, 1000000 loops each)

Since the API is the same here, Python/CFFI overhead is out of the picture. So using the parser to feed new values for properties runs 20x slower. Why the parser was used? I imagine that the COM interface was not added at the inception of OpenDSS, so to handle the side-effects of changing the properties without changing some of the design, the parser was used.
For many properties I'm exposing the data in electricdss-src as public instead of private, otherwise I'd need to change the code there more, or use the OpenDSS parser. I'm creating separate functions to handle that, out of the original units.
After finishing the tasks for this issue, I'll probably change at least of the other classes to match the new API code.

Out of curiosity, running the same two timeit examples with win32com (with EnsureDispatch):
%timeit transformer.kVA = val
8.14 µs ± 22.1 ns per loop (mean ± std. dev. of 7 runs, 100000 loops each)

%timeit linecode.NormAmps = val
1.67 µs ± 27.8 ns per loop (mean ± std. dev. of 7 runs, 1000000 loops each)

From this we can see that the COM overhead vs DSS Python is around 1.3 microseconds.

17c2a753578379de57d83415c870c694614fa827 exposes Loads_Set_Phases and Loads_Get_Phases. Besides the missing classes from the main body of the issue, I guess we'll end up exposing most properties that are missing from the other classes, in the long term.

Since adding this to COM would also benefit users that, e.g., run OpenDSS through Microsoft Office or other restrictive contexts that don't allow using the dss_capi DLL, it would be ideal to create a COM version of the new interface classes. This should be done as such that it is easy to integrate in the official OpenDSS repository if desired down the line, while not forcing additional workload on EPRI.

Although it would still be good to build a test (plain C-API) DLL with Delphi on Windows, I don't believe there is a need to port anything to the Delphi COM DLL anymore. Some tests showed that the approach for https://github.com/dss-extensions/dss_sharp/issues/7 is good enough and would reduce maintenance a lot. It's possible that it would even be faster for part of the methods (benchmark pending).

Final solution to this is coming soon. A new low-level API will expose most components for advanced users. This low-level API can then be used to create a higher level version which is compatible with the concepts from the COM version.

@PMeira What's the status on the addition of InvControl to the C API? I notice these issues have been closed, but the InvControl isn't marked as completed nor is it present in the C API.

@keegit The new API exposes all DSS objects, but the low-level API is not documented yet. We don't indent to add  dedicated APIs (besides what's required for compatibility) at the current level since most of the functions are to read/write DSS properties. The idea is to process the JSON output from DSS_ExtractSchema() , map it to the Obj_* (and maybe Batch.*) functions, and generate some wrapper code for the target language. So far we've done that for C++ and Python.
If you'd like to use it now, let me know if there's something we can add to make it easier.
Some working examples:

https://github.com/dss-extensions/dss_python/blob/0.14.4/tests/test_obj.py
https://github.com/dss-extensions/dss_python/blob/0.14.4/tests/test_batch.py

The main Python impl. is at https://github.com/dss-extensions/dss_python/blob/0.14.4/dss/IObj.py
It's mostly autogenerated, the docstrings are derived from the property help strings, contains most enums.
There's a slightly older C++ version in https://github.com/dss-extensions/dss_capi/blob/0.13.4/include/dss_obj.hpp
I haven't updated it for the current release yet. It's nearly the same thing, except that in Python we use "magic numbers" for the property index (to avoid the lookup at runtime since Python is kinda stupid), while in C++ we can rely on the compiler to handle things.
Currently, we don't generate separate C headers for wrapping the Obj_* functions since they'd probably wouldn't be directly useful, but maybe exporting at least the enums in a header could be useful. Most of this is generated from the output of the DSS_ExtractSchema() function. The property types and flags are a bit messy since they're directly from the current internal implementation and we don't intend to set that in stone. If plain C is useful for someone, we could generate the full wrappers, but it's kinda redundant and I'm avoiding doing it now so less experienced users don't try crazy things in a very unsafe language.
What's missing is to move part of the classic/original API to this new API to allow working with multiple objects without switching active elements/etc. Right now I'm finishing the last major refactor of the internals before further developing the API. We'll probably announce some things soon-ish on the Discussions.

All I need is a set of Storage_SetkVAr(), Storage_Set_kW(), and Storage_SetPF() API calls similar to what is there for load/generator.

@keegit, would this help?
Left it in Python so it's closer to pseudocode than plain C. Besides all the boilerplate, this should be enough for those functions.
For the other components that currently have no dedicated API at all, you would need to either use the *ActiveClass* functions to iterate through the elements (ensuring the target class is indeed active), or just grab the count once and use that to emulate the classic API.
from dss import dss
api_util = dss._api_util
lib = api_util.lib_unpatched # this is basically the set of functions as seen in the dss_capi.h and dss_capi_ctx.h
ffi = api_util.ffi
ctx = api_util.ctx
# if you're not using the ctx functions yet, you can grab the default DSS instance pointer from:
# ctx = lib.ctx_Get_Prime()

# Preparation -- grab the relevant indices

# Unfortunately we need to have an element to check the properties
dss.ClearAll()
dss.NewCircuit('empty')
dss.Text.Command = 'new storage.we_need_this_for_the_properties'

# Grab the class index
STORAGE_CLS_IDX = dss.SetActiveClass('Storage')
assert STORAGE_CLS_IDX != 0
assert dss.ActiveCircuit.Storages.Count != 0
dss.ActiveCircuit.Storages.First

# Convert the property names to lowercase to ensure future compatibility
prop_names = [p.lower() for p in dss.ActiveCircuit.ActiveDSSElement.AllPropertyNames]
STORAGE_PROPERTY_kvar = prop_names.index('kvar') + 1
STORAGE_PROPERTY_kW = prop_names.index('kw') + 1
STORAGE_PROPERTY_PF = prop_names.index('pf') + 1

def Storages_Get_kvar() -> float:
    element_idx = dss.ActiveCircuit.Storages.idx
    assert element_idx != 0
    element_ptr = lib.Obj_GetHandleByIdx(ctx, STORAGE_CLS_IDX, element_idx)
    assert element_ptr != ffi.NULL
    result = lib.Obj_GetFloat64(element_ptr, STORAGE_PROPERTY_kvar)
    dss._check_for_error()
    return result

def Storages_Set_kvar(value: float):
    element_idx = dss.ActiveCircuit.Storages.idx
    assert element_idx != 0
    element_ptr = lib.Obj_GetHandleByIdx(ctx, STORAGE_CLS_IDX, element_idx)
    assert element_ptr != ffi.NULL
    lib.Obj_SetFloat64(element_ptr, STORAGE_PROPERTY_kvar, value)
    dss._check_for_error()

# Since these are all float64 values, only the property index (and function names)
# change across these functions below

def Storages_Get_kW() -> float:
    element_idx = dss.ActiveCircuit.Storages.idx
    assert element_idx != 0
    element_ptr = lib.Obj_GetHandleByIdx(ctx, STORAGE_CLS_IDX, element_idx)
    assert element_ptr != ffi.NULL
    result = lib.Obj_GetFloat64(element_ptr, STORAGE_PROPERTY_kW)
    dss._check_for_error()
    return result

def Storages_Set_kW(value: float):
    element_idx = dss.ActiveCircuit.Storages.idx
    assert element_idx != 0
    element_ptr = lib.Obj_GetHandleByIdx(ctx, STORAGE_CLS_IDX, element_idx)
    assert element_ptr != ffi.NULL
    lib.Obj_SetFloat64(element_ptr, STORAGE_PROPERTY_kW, value)
    dss._check_for_error()

def Storages_Get_PF() -> float:
    element_idx = dss.ActiveCircuit.Storages.idx
    assert element_idx != 0
    element_ptr = lib.Obj_GetHandleByIdx(ctx, STORAGE_CLS_IDX, element_idx)
    assert element_ptr != ffi.NULL
    result = lib.Obj_GetFloat64(element_ptr, STORAGE_PROPERTY_PF)
    dss._check_for_error()
    return result

def Storages_Set_PF(value: float):
    element_idx = dss.ActiveCircuit.Storages.idx
    assert element_idx != 0
    element_ptr = lib.Obj_GetHandleByIdx(ctx, STORAGE_CLS_IDX, element_idx)
    assert element_ptr != ffi.NULL
    lib.Obj_SetFloat64(element_ptr, STORAGE_PROPERTY_PF, value)
    dss._check_for_error()

# Sample run

dss.Text.Command = 'redirect ~/electricdss-tst/Version8/Distrib/Examples/StorageTechNote/Example_9_1_Default/Storage_default.dss'
dss.ActiveCircuit.Solution.Number = 1

# For comparison, grab the implementation from IObj.py
sto_obj = dss.Obj.Storage[1]

Storages_Set_PF(0.9)
for n in range(24):
    dss.ActiveCircuit.Solution.Solve()
    for sto in dss.ActiveCircuit.Storages:
        print(f'h={dss.ActiveCircuit.Solution.dblHour:02.0f}, {sto.Name}')
        print(Storages_Get_PF(), Storages_Get_kvar(), Storages_Get_kW())
        print(sto_obj.pf, sto_obj.kvar, sto_obj.kW)
        print()


Issue in opening blog posts || Bug

Describe the bug
Error on trying to open blog posts after merging #290
To Reproduce
Steps to reproduce the behavior:

Go to home page
Click on READ MORE on any posts

Expected behavior
Should open the blog posts.
Note:

If you want to work on an issue, you should check if it has already been assigned to anyone. If the issue is free you can comment /assign to get the issue assigned to you.
If you are raising a new issue and want to work on it then also you should comment /assign under the issue to get it auto assigned.
Please refrain from adding labels to your issue/pull-request on your own. It is the job of the Project Admin and the Mentors to review your issue/pull-request and add labels accordingly.

It must be route related something, I am working on it

/assign


invite command invoked inappropriately importance/high

Opened by @jayvdb

See gitter archive of today in coala room

https://github.com/coala/corobo/issues/323 has more occurrences


Collision with groups

Arcade physics already support passing an array for collision checks, but not groups. My current solution to check a sprite against a group with sprites looks like this:
this.scene.enemyGroup.children.entries.forEach(enemy => {
      this.scene.physics.world.overlap(this, enemy, ()=>{enemy.kill()});
});

If I could pass groups it would look like this:
this.scene.physics.world.overlap(this, this.scene.enemyGroup,(me,enemy)=>{enemy.kill()});
If I'm successful with the git-stuff I will make a PR within a few minutes.

Strange, the Making your first game tutorial I wrote uses sprite vs. Group and group vs. group and it works for overlap and collide. What’s the difference between how I do it and your approach?

You add
platforms = this.physics.add.staticGroup();
And I just add a group
enemyGroup = this.add.group();
I never seen staticGroup before. Maybe that's the correct way to do it? On the other hand adding support for collision with normal groups just add two lines of code.

A Physics Group is basically just a Group with a couple helper methods. The important part is this: https://github.com/photonstorm/phaser/blob/master/src/physics/arcade/World.js#L1395-L1409 which should work on any Group, because they all set the 'isParent' property.

Ok. I digged a little bit more. Everything works fine with isParent but something goes wrong becase the group has no physics:
https://github.com/photonstorm/phaser/blob/master/src/physics/arcade/World.js#L1493
I updated my PR to target groups without physics only. IDK if that's the right approach, but with this PR a normal group will be handled just like an object containing an array of collidable objects so the collision handler could just focus on the array.

Thanks for opening this issue, and for submitting a PR to fix it. We have merged your PR into the master branch and attributed the work to you in the Change Log. If you need to tweak the code for whatever reason please submit a new PR.