Removed  as not supported in javadoc.

Check List:

Fixes # JavaDoc
Unit tests : NA
Javadoc with a code example (API only) :  NA

Ide used InteliJ to build the javadoc.
Also tried Netbeans ide but still getting errors on building the javadocs.
Removed
 as not supported in javadoc and erroring on build.
Changed < to &lt as error when building javadoc.
Changed > to &gt as error when building javadoc.
changed
 to
 as error when building javadoc.
If it is something I am doing wrong then let me know as I am only a hobby programmer.

Integrated, thanks !


Which validation dataset is used for the pre-trained models?

I wonder whether you have used the training/validation split that is included in the Vimeo-90k dataset or used the whole dataset to train the published pre-trained models?
Since the find(pattern, path) script defined here results in folders.npy containing all sequences in Vimeo-90k, it seems training is done on the whole dataset. If this is true, then what dataset is used for validation?

Thanks for comment. For simplification, we didn't have the validation phase, but yes, itbis better to use some other public datasets for validation——Ren Yanghttps://renyang-home.github.io/Doctoral studentComputer Vision Laboratory | ETH ZürichSternwartstrasse 7 | ETF D 114.2 | 8092 Zürich, Switzerland-------- 原始邮件 --------发件人： heydarshahi @.>日期： 2021年10月1日周五 11:42收件人： RenYang-home/OpenDVC @.>抄送： Subscribed @.***>主    题： [RenYang-home/OpenDVC] Which validation dataset is used for the pre-trained models? (#17)
I wonder whether you have used the training/validation split that is included in the Vimeo-90k dataset or used the whole dataset to train the published pre-trained models?
Since the find(pattern, path) script defined here results in folders.npy containing all sequences in Vimeo-90k, it seems training is done on the whole dataset. If this is true, then what dataset is used for validation?
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.


CQD-Power-BI-query-templates - "call queues" and "Auto attendants"?

Very nice idea and good Dashboards. But one questions about the microsoft teams "call queues" with "Auto attendants". I want to create a separate report with filters "second upn" and "total call count" as example for one month but all i get is the error message "index was outside the array. Is querying the "call queues" with "Auto attendants" not supported?

thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Simplify new equipment list.

This code is a bottleneck. It's possible there is a reason the current code is the way it is, but I can't see one - unless there is some object-identity issues that are being worked around. newItems is mutated in this method but then the new value is never referenced afterwards.

This code is a bottleneck.

Any basis for this assertion besides intuition and static analysis?

Any basis for this assertion besides intuition and static analysis?

Profiling while running a typical adventuring script indicates this function is taking something like 7-8% of execution time, because retainAll iterates through every element of the relevant list and checks whether the other list contains it.

Any basis for this assertion besides intuition and static analysis?

Profiling while running a typical adventuring script indicates this function is taking something like 7-8% of execution time, because retainAll iterates through every element of the relevant list and checks whether the other list contains it.

OK.  I presume you have before and after timings? :-)
I too wish I knew why it was done that way but I cannot construct a test data where the results are different.  And the existing tests covered before and after so at least things were exercised.
IntelliJ reports several Lint type things that could be addressed.  If you are running it, you might consider doing Analyze and addressing most of the issues.  If not I'll try and do that, probably for another PR.
I'll Approve but wait a while before I push just in case someone does know why things were done the way they were.

I don't see an issue with this change.
currentItems is a SortedListModel. If it weren't, after this change the list could be in a different order to before, as it would contain all the items it contained before, followed by all the ones it contained after. But because it's sorted, this isn't a problem.
The other change is to remove newItems.removeAll(currentItems);. newItems comes from equipmentLists, which is generated from scratch in updateAllModels using List<List<AdventureResult>> equipmentLists = this.populateEquipmentLists(); and never referenced after this method, so this removal seems fine too.


Add offset value, offset mode and cap value API's

Have added options to:

Set offset mode in Offset Register for frequency adjustment
Set offset value in Offset Register  as part of offset mode
Set CAP_SEL value in Control_1 Register

@maustin-redant can you rebase? Just pushed some changes from @bandogora

Dup of #6


Add kitspace.yml for RevC1

Per discussion in #193 switch to using the "multi project" kitspace yml format.

@whitequark does this look more suitable for future revisions?


Add a Rate Us Option

Hello, my name is A.A.D.Sailaja. I am a Tech Enthusiastic and i'm looking forward to contribute in open source projects. Here in this project i would like to add a rate us option it will allow the user to rate our website and also he can suggest us to improve if any. I will Change the page According to our Website below image is for Reference
Can you please assign me this issue under gssoc'24

@sailaja-adapa
There is already a feedback form there in the menu bar (top left corner). If you have any additional issues, feel free to reach out. To keep our issue tracker organized, this issue will be closed.


Bump fedora-coreos-config

Periodic testing-devel update

/override ci/prow/e2e-aws-upgrade
known upgrade disruption
/retest

/hold
[sig-storage] In-tree Volumes [Driver: hostPath] [Testpattern: Inline-volume (default fs)] volumes should store data [Suite:openshift/conformance/parallel] [Suite:k8s]
[sig-storage] In-tree Volumes [Driver: hostPathSymlink] [Testpattern: Inline-volume (default fs)] volumes should store data [Suite:openshift/conformance/parallel] [Suite:k8s]

permafailing

/retest

/test e2e-aws

/retest

Known upgrade issue
/override ci/prow/e2e-aws-upgrade
/hold cancel


Use build.merproject.org as SFOS-OBS' DNS-name

… even though build.sailfishos.org is supposed to be the "official" address, because it does not work for many months as of December 2022.

ToDo:Check, if build.sailfishos.org is still present at another place in this project.

ToDo: Check, if the string build.sailfishos.org is still present at another place in this project.

Done: It is, in qml/pages/AboutPage.qml and all translations, including the source strings (actually, just a single one).
Will fix.


dirname.absolute hangs with paths with brackets

When computing absolute on a path with brakets [ ], the function enters on a never-ending loop.
realpath.absolute "components/web-app/src/app/[lang]/__tests__/page-test.tsx"
# whoops, it hangs above and never returns
echo "$REPLY"

Scaping the brackets with \ does not work also
realpath.absolute "components/web-app/src/app/\[lang\]/__tests__/page-test.tsx"

Thank you for your awesome work!

wow! that was fast! Thanks for this @pjeby !


Offline backup of organization / all forks

System could use the organization migration API
https://developer.github.com/changes/2018-05-24-user-migration-api/
...or it could use a simple shell script to git pull all repos.

This was addressed with the github_fork_updater script.
Closed by 1b81a8bb00acdc87848881b2b0190f2a5198a07f

Forks are mainly addressed with the github_fork_updater script.
However, if there is other data associated with this account -- e.g. a wiki -- it should be handled with a general purpose records archive, e.g.
https://github.com/benbalter/github_records_archiver


How to mimic the default Lollipop's look?

as the title says. What are the configurations for this?
Also, can I make it so that if newer versions of Android arrive, it will use their default style and not the one of the library?

The defaults come very close to the default look. The only big difference
is the color, and that likely depends on your theme's color. There is a PR
for using the theme color so that will be in soon.
The timing of the animation duration is slightly off but I came as close as
I could playing it by eye. The shape of the animation should remain the
same I'd you keep the default number of steps.
On Apr 22, 2015 07:28, "AndroidDeveloperLB"<EMAIL_ADDRESS>wrote:

as the title says. What are the configurations for this?
—
Reply to this email directly or view it on GitHub
https://github.com/rahatarmanahmed/CircularProgressView/issues/9.

Can you also add support for older versions of Android, like the support library?

Since the color changes based on your theme, for now you just have to set the color to match your theme's accent. In the next release this will happen automatically, and should hopefully support the support library as well.

ok, I've decided to use a different library (this: https://github.com/fengdai/AlertDialogPro ). Sorry for all the trouble/time. And thank you.


Update ios-install.md

Added a note about Zero-touch that it cannot be configured on iOS devices that are enrolled without user affinity.

@JohanFreelancer9 Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/7756


Testing, please ignore. pbemyh

This is a bug bounty test. Please do not approve this! pbemyh

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.pbemyh tnmbm seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


insufficient connect_port() invocation

The latest import of sfizz triggered process()-time failure for some reason.
It seems that our resetPorts() in android-audio-plugin-lv2-bridge.cpp is receiving 20 from lilv_plugin_get_num_ports(lilvPlugin) which should return 23. AudioPluginServiceImpl receives 23 prepareMemory requests and prepare() correctly passes them. Only resetPorts() at process() time gets 20 instead.

It was due to an Android Studio issue about insufficient deployment for native code (we have to turn on  "Always install with package manager" option in the build/run configuration).


Karpenter label validator fails on valid labels

Version
Karpenter: v0.3.2
Kubernetes: v1.21.0
Expected Behavior
I'm trying to add labels to the Provisioner. It seems that only simple strings are supported, such as:
apiVersion: karpenter.sh/v1alpha3
kind: Provisioner
metadata:
  name: default
spec:
  cluster:
    endpoint: https://1myendpoint.eu-west-1.eks.amazonaws.com
    name: mycluster
  ttlSecondsAfterEmpty: 30
  labels:
    mylabel: hello

However, when trying to add more complex labels such as
node.k8s.aws/subnet-tag-key: hello

Karpenter fails to validate, and throws the error:
"a valid label must be an empty string or consist of alphanumeric characters, '-', '_' or '.', and must start and end with an alphanumeric character"
This is clearly a bug since this field should support any valid Kubernetes label (documented here https://kubernetes.io/docs/concepts/overview/working-with-objects/labels/#syntax-and-character-set)
Actual Behavior
Validating webhook throws a validation error
Steps to Reproduce the Problem
Attempt to apply the following:
apiVersion: karpenter.sh/v1alpha3
kind: Provisioner
metadata:
  name: default
spec:
  cluster:
    endpoint: https://1myendpoint.eu-west-1.eks.amazonaws.com
    name: mycluster
  ttlSecondsAfterEmpty: 30
  labels:
    node.k8s.aws/subnet-tag-key: hello

scratches head . Will double-check and report back!

Any luck?

Hi, sorry this took a while.
We're using CDK's EKS module, which has a cluster.add_manifest() function for adding arbitrary manifests. That function seems to (for some weird reason) fail when writing the provisioned, but it works fine when using kubectl locally, so no bug on your end. Sorry for the confusion :-)

argh, that took me a while. So: It's not the key that's the problem, its the value. My subnet's Name tag is vpc-aws-euw-1-e2/VPC/PrivateSubnet1.
When I try and add the label node.k8s.aws/subnet-name: vpc-aws-euw-1-e2/VPC/PrivateSubnet1, validation fails.

Gotcha -- this value isn't compliant with Kubneretes Labels. I'm working on a PR to address this to support strongly typed cloud provider parameters that will resolve the root cause of this: https://github.com/awslabs/karpenter/pull/648
#642

solved in v1alpha4


Data science with Python: RANSAC Algorithm#7355

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes #issue_number_here

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[#] Doc#umentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[#] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[ ] I follow Contributing Guidelines & Code of conduct of this project.
[#] I have performed a self-review of my own code or work.
[#] I have commented my code, particularly in hard-to-understand areas.
[#] My changes generates no new warnings.
[#] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

Add your issue number? @Ruchika-20

Don't close it. @Ruchika-20

okay
On Thu, Dec 2, 2021 at 11:50 PM Prathima Kadari @.***>
wrote:

Reopened #8599
https://github.com/girlscript/winter-of-contributing/pull/8599.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/pull/8599#event-5707702861,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUEIB5ZFWEHCBME7P4L24SDUO62FVANCNFSM5I32FTCQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Sorry for the inconvenience
On Fri, Dec 3, 2021 at 12:37 AM Ruchika Singh @.***>
wrote:

okay
On Thu, Dec 2, 2021 at 11:50 PM Prathima Kadari @.***>
wrote:

Reopened #8599
https://github.com/girlscript/winter-of-contributing/pull/8599.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/pull/8599#event-5707702861,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUEIB5ZFWEHCBME7P4L24SDUO62FVANCNFSM5I32FTCQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Unique currency formats are not displayed correctly

Frequency: 100%
Repro App version:  Android Valora Mainnet build v1.1.0<PHONE_NUMBER>), Test Flight build Valora app build  v1.1.0 (223)
Repro on device: Samsung Galaxy A70(9.0), iPhone 11 (14 iOS)
Repro Steps:

Launch the Application and sign in with a test account
Tap on Hamburger menu
Tap on settings button
Tap on currency button
Select CVE currency
Observe amount

Current Behavior: $18.44 is shown when user select CVe currency
Expected Behavior: 18$44 should show when user select CVe currency
Attachment: CVECurrency.jpg

Hi @tarikbellamine we have verified this issue on latest build Android internal build V1.15.2 & Testflight build 1.15.2 (65) and observed issue is still repro.
Devices: iPhone 6+ (12.4.5), Vivo V11 pro (10)


Add USER to must-gather Dockerfile

Seeing the following error in the EC (ecosystem-cert-preflight-checks):
            {
                "name": "RunAsNonRoot",
                "elapsed_time": 0,
                "description": "Checking if container runs as the root user because a container that does not specify a non-root user will fail the automatic certification, and will be subject to a manual review before the container can be approved for publication",
                "help": "Check RunAsNonRoot encountered an error. Please review the preflight.log file for more information.",
                "suggestion": "Indicate a specific USER in the dockerfile or containerfile",
                "knowledgebase_url": "https://access.redhat.com/documentation/en-us/red_hat_software_certification/2024/html-single/red_hat_openshift_software_certification_policy_guide/index#assembly-requirements-for-container-images_openshift-sw-cert-policy-introduction",
                "check_url": "https://access.redhat.com/documentation/en-us/red_hat_software_certification/2024/html-single/red_hat_openshift_software_certification_policy_guide/index#assembly-requirements-for-container-images_openshift-sw-cert-policy-introduction"
            }

Relates to https://github.com/openshift-knative/serverless-operator/pull/3018

/lgtm


[React] I need to offset the  by one pixel

I'm using the <FloatingArrow> component to render the arrow and it works great, but I would like to always offset it by one pixel so that the last white stroke lies above the floating element so that it looks like one contineous stroke. This would be a nice addition to this arrow component.
<FloatingArrow
  ref={arrowRef}
  context={floatingContext}
  width={ARROW_WIDTH}
  height={ARROW_HEIGHT}
  strokeWidth={1}
  tipRadius={1}
  className="fill-white [&>path:first-of-type]:stroke-gray-300[&>path:last-of-type]:stroke-white"
/>

I could add a my-[-1px] class to the arrow and that works for when it's placement is bottom but not at the side:

It's not zoomed but I am using the useTransitionStyles and that's almost taken line by line from the reusable popover example.
Result: https://play.tailwindcss.com/rsyjYOxDLA
My component:
const ARROW_WIDTH = 14;
const ARROW_HEIGHT = 7;

const Content = forwardRef<HTMLElement, ComponentProps<"div">>(function Content(
  { children, style, ...rest },
  propRef
) {
  const {
    context: floatingContext,
    arrowRef,
    floatingStyles,
    middlewareData,
    getFloatingProps,
    ...context
  } = usePopoverMenuContext();
  const ref = useMergeRefs([context.refs.setFloating, propRef]);

  const arrowX = middlewareData.arrow?.x ?? 0;
  const arrowY = middlewareData.arrow?.y ?? 0;
  const transformX = arrowX + ARROW_WIDTH / 2;
  const transformY = arrowY + ARROW_HEIGHT;

  const { styles: transitionStyles, isMounted } = useTransitionStyles(
    floatingContext,
    {
      duration: 100,
      initial: {
        opacity: 0,
        transform: "scale(0.95)",
      },
      common: ({ side }) => ({
        transformOrigin: {
          top: `${transformX}px calc(100% + ${ARROW_HEIGHT}px)`,
          bottom: `${transformX}px ${-ARROW_HEIGHT}px`,
          left: `calc(100% + ${ARROW_HEIGHT}px) ${transformY}px`,
          right: `${-ARROW_HEIGHT}px ${transformY}px`,
        }[side],
      }),
    }
  );

  if (!isMounted) return null;

  const floatingProps = getFloatingProps({ className: "z-10", ...rest });

  return (
    <FloatingPortal>
      <FloatingFocusManager context={floatingContext}>
        <div
          ref={ref}
          style={{
            ...floatingStyles,
            ...style,
            zIndex: style?.zIndex ?? 1400,
          }}
          {...floatingProps}
        >
          <div
            style={transitionStyles}
            aria-labelledby={context.labelId}
            aria-describedby={context.descriptionId}
          >
            {children}
            <FloatingArrow
              ref={arrowRef}
              context={floatingContext}
              width={ARROW_WIDTH}
              height={ARROW_HEIGHT}
              strokeWidth={1}
              tipRadius={1}
              className="fill-white [&>path:first-of-type]:stroke-grey-300 [&>path:last-of-type]:stroke-white my-[-1px]"
            />
          </div>
        </div>
      </FloatingFocusManager>
    </FloatingPortal>
  );
});

Strange as I can't reproduce it locally and I'm not sure how to get that Tailwind result. A CodeSandbox repro would be better.
This PR simplifes some of the path logic, does it help if you change it locally? https://github.com/floating-ui/floating-ui/pull/2423/files (you need to remove the last-of-type target.)

https://github.com/hornta/vitejs-vite-oyyhgd/tree/main
It's reproducible in either GH codespaces or stackblitz.

You're putting the border on the children of the floating element but the arrow is positioned relative to the floating element. If you put the border on the floating element instead then it's fine.

I had that at first but then I was having some weird jumping issue with the arrow. As long as the transition was going on the arrow is correctly positioned but not when it's done.

Since this has a workaround and seems to be related to browser rendering issues, I am going to close this. But PRs welcome if better solutions are found for this issue.

I'm having this issue as well.
The arrow is correctly positioned, but since my popover has a border it looks like it's 1px away from where it should be when top or bottom positioned; or 2px when left and bottom positioned.
Applying a negative margin to my child object caused the same bug described in this thread.
https://github.com/floating-ui/floating-ui/assets/741969/5e0091f4-7020-47ef-be18-12a0642de218
I believe having an offset prop in FloatingArrow would fix this issue. My idea would be to use like so:
const offsetMap: Map<Placement, number> = new Map(
  [
    ['top', 1],
    ['bottom', -1],
    ['right', 2],
    ['left', -2],
  ]
);

<FloatingArrow
   (...)
   context={context}
   offset={offsetMap.get(context.placement.split("-")[0])}
/>

For the above example, the expected dynamic computed style would be:

Placement
Computed dynamic style

top
top: calc(100% + 1px)

bottom
bottom: calc(100% - 1px)

left
left: calc(100% - 2px)

right
right: calc(100% + 2px)

Doesn't <FloatingArrow style={{ transform: 'translateY(-1px)' }} /> do that?

Ohh, I haven't even considered style because that was already being defined and I wasn't expecting it to get merged.
It worked like a charm. 👌
So silly that I went full-over-engineering and haven't thought about style merging.
Thank you, James.


Update init of import module

Need to update init file of import module to make the import functions more easily accessible.

Also need to update readme.md to work with changed import rules.

Fixed in latest commit


Add to team wizard - add phone numbers

^ Single line phone input on mobile ticketed separately. cc @keybase/y2ksquad

Heh I found a fun bug: if you delete all the phone number inputs, you can't add any more.

Ah good catch @jakob223, fixed by not allowing removing the last phone number.


Use a Mutex to make MemoryDatabase genuinely thread safe.

While the "check and raise" boolean method sometimes works, it quickly
starts raising errors and spamming the console when run under a modern
threaded server like thin or puma. Since Ruby has a reliable, genuine
mutex, use it so that instead of tossing away any conflicting
transactions, they just line up and go one at a time.

:+1:


In the Device ID Application the Device Credential should be labelled as "Device ID"

Like this?
Before (Home):

After (Home):

Before (Credential)

After (Credential)


[WPStarter 3] Empty env value not used

Describe the bug
If i define an empty value for an env var, this var is not stored/used or visible in caching. This should work, since some settings like COOKIE_DOMAIN="" in a Multisite enviornment with different domains are often used.
To Reproduce
Steps to reproduce the behavior:

Create an .env file and basic setup
Add COOKIE_DOMAIN="" or COOKIE_DOMAIN=''
Open the Page
See .env.cached.php

In SYMFONY_DOTENV_VARS the variable is listed, but below in define|putenv|$_ENV|$_SERVER the definition is missing.
Expected behavior
The COOKIE_DOMAIN should occure in the cached env file.
System (please complete the following information):

OS: Windows 10 Pro
PHP 7.2.1

Any news here?

Fixed. I added unit tests to ensure empty strings from env are retained properly.


Bug Bash

Chat Text Box overlaps with conversation

Clicking on support document returns 404.

The endpoint URI isn't obvious when azd up finishes. Maybe the postdeploy step could be earlier? (Postprovision?) Or could it be shorter, like by removing the funding notice in npmrc?

Funky alignment of headings

Response included "Followup questions: " before the citations:

Follow-up questions have no question marks:

"Show thought process" button is greyed out on previous question when I have clicked it on later question.

Doc says to check "Suggest follow-up questions" but it seems to already be default on. Not sure if the app wants to keep it on (its off for Python) or if docs should change.

Clicking on support document returns 404.

{
    "statusCode": 404,
    "error": "Not Found",
    "message": "Not Found"
}

Will be fixed with #103

"Show thought process" button is greyed out on previous question when I have clicked it on later question.

This is the expected behavior. The button is disabled when the panel is open.

Response included "Followup questions: " before the citations:

I think this is an edge case but we will try to reproduce.

Clicking on support document returns 404.

{
    "statusCode": 404,
    "error": "Not Found",
    "message": "Not Found"
}

This one is prompt-related, as it generates an extra source:  in front of the filename sometimes.
GPT35 needs careful prompting to follow format instructions correctly 😅

All items have been solved or are in progress.


Use col offset in bandit output to tag the correct node in the AST

For #12118

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Ready for review

Example screenshot of release extension and how it highlights the whitespace on the left instead of the correct AST node.

Screenshot of this PR with the same snippet highlighting the correct AST node


list index out of range

How many players? 2
What's the name of player1?Henk
What's the name of player2?Jan
26 => ♥ 6
38 => ♠ 8
08 => ♣ 8
33 => ♠ 3
02 => ♣ 2
20 => ♥ joker
00 => ♣ joker
which card (id) do you want to play? (or pass) 20
Henk played 20 => ♥ joker
31 => ♠ aas
22 => ♥ 2
36 => ♠ 6
18 => ♦ 8
17 => ♦ 7
05 => ♣ 5
37 => ♠ 7
which card (id) do you want to play? (or pass) 22
2
Traceback (most recent call last):
File "game.py", line 78, in
players[index + 1].takeCard(deck, stack, 2)
IndexError: list index out of range

elif Card.rankNames[playedCard.rank] == '2': 	players[index + 1].takeCard(deck, stack, 2)


Enhance FilesFromDisk (close #331)

Now permit suffixing root on disk with separator to enumerate only
its contents and not to create that actual file in archive.
And attempt improving Windows compatibility. I don't have Windows handy so this'll have to be CI-based feedback.
@nicola-spb could you give this a try?

@mholt I am working in Linux too :(

That's fine, I mostly want you to test the feature you requested.

Yes it's work on Ubuntu.
Tested it:
files, err := archiver.FilesFromDisk(nil, map[string]string{
	"/path/on/disk/folder/": "",  // adding files to root
})

files, err := archiver.FilesFromDisk(nil, map[string]string{
	"/path/on/disk/folder": "",  // adding files to "folder" in archive
})

Perfect, thanks for verifying it does what you need!
I'll clean up the implementation and get them working on Windows. I might also work on the dot (.) situation but how complex it is depends whether it goes in this PR.


improve write json serialization/de-serialization path

when pk is explicitly provided in the request option, there is no need in parsing json
avoid multiple rounds of json serialization/desrialization when possible in the write path.

withdrawn, still working on it


Amend mention message html output

This PR:

makes it so that when you translate a mention to "message format" html then the following rules are applied:

if it's an at-room mention, only output the raw text @room
if it's a room or user mention, output the mention as display_text
adds a tiny bit of functionality in the web app to make sure that we can test this functionality
expose the function to get the message content in html to the relevant parts of the TS code

Codecov Report
Patch coverage: 100.00% and project coverage change: +1.69 :tada:

Comparison is base (b431568) 88.38% compared to head (802492f) 90.07%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #716      +/-   ##
============================================
+ Coverage     88.38%   90.07%   +1.69%
============================================
  Files           146       81      -65
  Lines         17003    14261    -2742
  Branches        789        0     -789
============================================
- Hits          15028    12846    -2182
+ Misses         1785     1415     -370
+ Partials        190        0     -190

Flag
Coverage Δ

uitests
?

uitests-android
?

uitests-ios
?

unittests
90.07% <100.00%> (+1.98%)
:arrow_up:

unittests-android
?

unittests-ios
?

unittests-rust
90.07% <100.00%> (+0.05%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

crates/wysiwyg/src/composer_model/mentions.rs
98.52% <100.00%> (+0.09%)
:arrow_up:

crates/wysiwyg/src/dom/nodes/mention_node.rs
100.00% <100.00%> (+2.79%)
:arrow_up:

... and 66 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


feat: add CORS config option to allow for disabling cors

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other: <!-- Please describe: -->

What is the current behavior?

Issue Number: N/A
What is the new behavior?
In some cases, cors needs to be disabled to perform certain tests. For example, using <IP_ADDRESS> and localhost as different hosts.
This PR will add a new config option to optionally disable CORS.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

Can this please get merged?


Future of the Website WG

What is the plan to move forward as a WG for nodejs.org?
It's been said that we need to connect with the Node website group/maintainers. How do we do that? I imagine they have GitHub accounts; we could mention them here to get the ball rolling. What progress do we need to make and what steps do we need to take to start working with the official site?
Will we be building an entirely new strucutre, using nodejs.org's structure, or using iojs.org's structure? If so, how can we start implementing it? I'm including pushing updates to site content and the build process in that.

There are already several threads about the naming and related repos and it was discussed quite a bit in the last meeting so I'm going to close this issue as a duplicate.


add ready manifest from ImpStation

About the PR

Details can be found here, https://github.com/impstation/imp-station-14/pull/43 - Funky Station takes no credit for this.
🆑12rabbits

add: Added a ready manifest display to the pre-round lobby.

hi i just noticed this. you're gonna want to copy over this PR too: https://github.com/impstation/imp-station-14/pull/63, it fixes a pretty glaring issue where people can ready (or unready) multiple times in a row, causing the counts to get skewed

Thank you for the heads up! Appreciate the work you do 👍


fix: modified editor width(#175397)

Fix #175397

Hi @hediet, anyone else who can do the PR approval?

I´m just noticing, this is still in issue in the diff editor v2. Could you look into that?

Thank you!


Bedrock Future ?

Hello there and sorry posting that small post in issues.
I just would like to know if WP using MVC structure could be an interesting thing for Bedrock ?
I see more and more interests too around using WP with most used frameworks like Laravel or Symfony.
Because I'm making a lot of research on that topics, I naturally found Bedrock project.
Thanks for any answer.
Best Regards,
3Ky.

please don't use github issues for support, use https://discourse.roots.io/


Added bower.json and package.json

Added bower.json and package.json to make possible to install through bower and npm.

@re6exp Same story here as with your Bitters PR.
Related Refills Issue for why we don’t publish to npm and Bower.

@tysongach - that kind of makes sense to me - but if that's the case, why package them as a gem?

@anicholson Because the gem doesn’t just install all of the Refills into a project in un-editable directory, rather it comes with a CLI that allows you to quickly generate individual Refills on an as-needed basis.


Aggregated Catalog API

The aggregated catalog endpoint provides a means to view the whole catalog of offered services at Service Manager level.
Acceptance criteria:

[x] Implementation
[x] Tests

Pull Request Test Coverage Report for Build 405

40 of 42 (95.24%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 85.777%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

api/catalog/catalog_controller.go
26
28
92.86%

Totals

Change from base Build 362:
0.3%

Covered Lines:
1170

Relevant Lines:
1364

💛  - Coveralls


use application config

Hi! Thanks for the package!
When trying to run the analyze command with my own config file I found that the config is not shared throughout. It seems like the Application instance holds and manages its own configuration instance which is not necessarily the same config instance that will be imported directly from the configuration file.
In this patch I've just adjusted the relevant commands to grab the config off the application instance. Let me know if you think a bigger change makes sense to ensure all config is pulled from the app instance rather than the config file.
I also haven't added any tests, I wasn't sure what the best way to test this would be. Let me know if this needs a test to proceed with.
Thanks

Great catch, thank you!


[Bug]: The Igloo css' are included multiple times when looking at the DevTools

Contact Details
No response
What happened?
When we look at the css of all Igloo components via the DevTools, we can see that it is included multiple times. See an example with a Button:

I expect to see the css there only one time.
Component
All the components
Component Version
0.0.0
Which browsers are you seeing the problem on?
No response
Mobile Device
No response
Relevant log output
No response

I'm pretty sure it has always been like this. This is really not ideal.
What happens is that a component's css file is included from its component and each component is included inside OV.
If an Igloo component includes another component like a Button, you will see the Button css for each Igloo component that includes it.
I'm not really sure how to go about fixing this for Igloo. I know hopper will not be like this thankfully.
Maybe @alexasselin008 has an idea of how we can easily improve this.

It is the way it currently works due to the fact that each components are self contained. TagPicker has a peer dependency on the button component, therefore loading it's CSS.

Since this issue has been present for all of igloo's existence, we will tolerate it for now. The plan would be to develop Hopper asap, and migrate Officevibe to it.
We'll make sure this kind of issue is not present in the next Design system.
I've talked with Alex Martel and he agrees with this course of action

Ok! Thank you all for your answers!


remove edge template from sql projects new project dialog

As discussed, removing the Edge specific sql project template due to low usage and to avoid confusion for users. Now there's only the Azure template and Sql Server database template.
ADS:

vscode:

Should we remove any related constants/strings as well?

Should we remove any related constants/strings as well?

I was thinking of doing that in a follow up PR later on, to make it easier to add them back in case we wanted to add the template back after a couple months.


Can't use import echarts from GitHub

Hi,
I am really interested in using the latest feature of eCharts, in particular the "markArea".
First, I would really like to have it published on npm (also tagged as a beta version, it could let more people test the new stuff).
Then, I tried to import it using the GitHub last commit, but it doesn't work at all: in the index.js file it uses a lot of "import './lib/...', but there is no 'lib' folder.
Is there a way to let it work?

Hi, @gabrielenosso.
lib folders are generated by npm script before publish. So if you wan't to use the latest code with import, following steps may be helpful.

Get the latest code from https://github.com/ecomfe/echarts and https://github.com/ecomfe/zrender and put the folders under node_modules.
run npm run prepublish in echarts folder and zrender folder respectively.
import 'echart'


Enable common data loading

@xavijam FYI, adding this with a (global) feature flag.
@Kartones CR this, please (there's also a fix for a c&p typo).

:+1:

Frontend tests were OK :+1: (details)

+1


How to remove "about" and "archive" from the homepage？

hello

On the demo live, I found these on homepage. Do you know how to remove these?
when i use hugo-notepadium, it doesn't appear.

Fixed in 36ed27ef2c4d4f1cb74194086bc89cb7f3f65c0f.
If you want to hide any other page on homepage, add hidden = true in the frontmatter before the particular page.

@qdzhang it works for me, tks
but when I use git pull to update the theme, it doesn't work.
I think it should be git submodule foreach git pull origin main in your README.

git pull should be used in a submodule directory, as your first way does. When using git pull in the root directory, it will pull the main repository but not the submodule. Maybe I need to update the README, thank you for pointing out this.
And  if you have other themes or git submodules, the second way will do git pull for all git submodules.
I'll close this issue because of problem solved. You could open new issues or pull requests if you have other problems.


Rationale behind aria-hidden="true" on the navdrawer element by default?

I ran into an issue earlier today where users of my application using NVDA and Apple's VoiceOver were unable to see my application's navdrawer. After investigating, it appears this is due to the aria-hidden="true" attribute set by default on the entire navdrawer.
Looking through the JS that controls the navdrawer, I'm not finding anything that unsets this value when the navdrawer is visible. To many screen readers, this appears to make the entire navdrawer literally invisible and inaccessible.
I'm curious what the rationale behind this decision was, and if it's supposed to be fixed somewhere that I'm not seeing. If there's no purpose for it to exist there, maybe it can be removed from the main documentation?

Hi @SlvrEagle23,
The aria-hidden does get updated when the navdrawer is toggled.
In the doc site, the navdrawer has aria-hidden="true" to start with because the framework is developed mobile first, so navdrawer will be skipped if it's closed and pronounced if it's open.
However, I do see that there is a problem for desktop users with permanent navdrawers (e.g. doc site). When the site is loaded on desktop, the permanent navdrawer is visible but aria-hidden attribute is still applied.
A simple solution is to remove aria-hidden for permanent navdrawers from HTML markup.
A proper solution for now is to add some JavaScript that listens on window resize event and then add or remove aria-hidden attribute accordingly. I'll incorporate this fix in a future version.


Add link list

Looks good to me.

k, the tests seem to be having various issues, one of them being that it can't find jQuery.  Since I plan to take jQuery out of the tests (see #115  and #96 for some more info), I have a feeling some of the reasons the tests are breaking may resolve themselves so I think I'm going to merge this for now.

Here is the screenshot to go along with this component:

as a heads up, I've been going back and forth about the best way to build this.  I'm still not positive about my decision, but I wanted to get v1 out at least, so we'll take it from there.  Feedback is welcome tho, and we can always review if needed.  Feel free to file issues 😄

related to #91


Add "Structure" tag to switches, buttons, and levers

About the PR

Adds "structure" tag to switches, buttons, and levers.
Why / Balance

This allows players to use the hand labeler to put labels on these items, to indicate what the switch is used for.
Technical details

Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

tweak: Hand labelers can now apply labels to buttons, switches, and levers

Pumps also would benefit from this if they do not count already.

TIL that is what this tag is for


Add json-build

json-build is a zero-allocation JSON serializer made in c89 targetting embedded systems.

Apparently 393c940 overwrites ac71ed1, making the prior PR to this #1193 worthless (my bad)! Sorry about this, I’ll send another PR for Concord tomorrow


[Fix] WPF-BindingLeak: Bind to property of non INotifyPropertyChanged without using BindingMode.OneTime

@xxMUROxx 👍

@xxMUROxx
I have one objection: ThicknessToDoubleConverter.ConvertBack was changed to throw an exception instead of returning DependencyProperty.UnsetValue. What's the reason for that change? The documentation for IValueConverter.ConvertBack says (emphasis mine):

The data binding engine does not catch exceptions that are thrown by a user-supplied converter. Any exception that is thrown by the ConvertBack method, or any uncaught exceptions that are thrown by methods that the ConvertBack method calls, are treated as run-time errors. Handle anticipated problems by returning DependencyProperty.UnsetValue.
A return value of DependencyProperty.UnsetValue indicates that the converter produced no value and that the binding uses the FallbackValue, if available, or the default value instead.
A return value of Binding.DoNothing indicates that the binding does not transfer the value or use the FallbackValue or default value.

/cc @punker76

@thoemmi Ah, yeah, your're right for this. I'll change it.

@thoemmi thanks for the notification. It was not intended to change this piece of code.


go get instruction from README.md doesn't work

Description
Impossible to install tool via go get.
Steps to reproduce
Run go get github.com/nem-toolchain/nem-toolchain/cmd/nem.
Expected behavior
Install it successfully.
Actual behavior
# github.com/nem-toolchain/nem-toolchain/pkg/keypair
D:\Go\src\github.com\nem-toolchain\nem-toolchain\pkg\keypair\keypair.go:59:7: undefined: sha3.SumKeccak256
D:\Go\src\github.com\nem-toolchain\nem-toolchain\pkg\keypair\keypair.go:69:6: undefined: sha3.SumKeccak256

Library version
latest snapshot
Go version (e.g. go version)
go version go1.9.1 windows/amd64

OS version (e.g. uname -a)
MINGW64_NT-6.1 dulanov-PC 2.9.0(0.318/5/3) 2017-09-13 23:16 x86_64 Msys

@r8d8 I see only one possability to fix it - migrate from https://golang.org/src/crypto/crypto.go to another library or as alternative extract KECCAK256 & ed25519 and include in our code directly. I will do it after #154.

So far, I haven't found any crypto alternative.
We also can use C libs for keccak and ed25519.

https://github.com/agl/ed25519

same error
# github.com/nem-toolchain/nem-toolchain/pkg/keypair
./keypair.go:59:7: undefined: sha3.SumKeccak256
./keypair.go:69:6: undefined: sha3.SumKeccak256

I saw the pr, and it does'nt be merged for a long time........ Why?


Greater teleport causes dedicated server to hang indefinitely when teleporting to unloaded chunks

Using version 0.9.4 on forge.
When using the spell long distance it can consistently cause the server to enter a frozen state, forcing every player to time out. The only way to recover from this state is by force killing the server process.
Notes:
It seems that if the destination chunk is loaded the greater teleport will not cause a server crash, regardless of how many of the surrounding chunks are loaded as well
The built in /tp command does not seem affected.
Lowering the render and simulation distances to the very lowest seems to mitigate this issue.
Modlist if relevant. No logs or crash reports to provide as the server is unable to record anything during it's frozen state.
Apologies if this isn't super helpful, I only have one dedicated server and was unable to replicate this bug in singleplayer.

One more piece of information that I hope is useful. Remember how I said that lowering the render distance seemed to help? Well it looks like doing that causes none of the surrounding chunks to appear even after setting the render distance back to normal.

Relogging or spamming the render distance option seems to fix it though.

That looks to me like your server just doesn't have enough memory to load those chunks that quickly ... but I'm not sure how come /tp doesn't crash, then. Lemme take a look.

That should fix it but let me know if it doesn't.

That should fix it but let me know if it doesn't.

Letting you know that it doesn't. Playing TNP Limitless 5 2.30.0. Minecraft version: 1.18.2. Forge version: 40.2.0.
Seems to hang the server inconsistently, as in, sometimes it does unhang, and sometimes it hangs indefinitely.
I also tried claiming and loading the chunks via FTB Chunks, but this doesn't seem to affect the bug.


Foreground services not working sometimes in android O and MIUI os

I have some question about location tracking and background services like (background/foreground/firebasejobdispatcher/jobshedular/workmanager).
Currently, my app is targetting android 28 version. And I am testing on  Android 8.0 now. Above code is working perfectly on pre-O and post-O android version. But getting location after every 1 min.
I am creating foreground service but some devices like MI, Oneplus which stop this foreground service.
Is the correct way to get locations or service will run in the background/foreground?
jobshedular/workmanager periodic time which is not lesser than 15 min.
Have anyone working on facing the same problem and resolving then please guide me.
Thank you
Originally posted by @amitdoiphode11 in https://github.com/codepath/android_guides/issues/220#issuecomment-445515533

this is due battery optimiser from manufacture itself. u have manually allow background in setting.

In order to run my foreground service even after app the was killed, I enabled Settings -> Apps -> Permission -> Auto Start Permission for my App. This was the only thing which worked on MIUI

If you read carefully in the end of second line
requires android.permission.FOREGROUND_SERVICE
I was also facing the same error in one of my project I resolve it by adding permission in the manifest file. So just simply add foreground permission in the manifest file.
<uses-permission android:name="android.permission.FOREGROUND_SERVICE"></uses-permission>
Hope it's work for you.


[BUG] QueryParams error

Information

Version: 6.114.9
Packages: @tsed/common

I was updating tsed packages from 6.114.3 -> 6.114.9. I am now having problem with our custom implementation of pagination example
Example
// SomeController.ts
 async fetch(
    @QueryParams() filter: FindQuery,
  ): Promise<Results<User>> { }

// SomeFile.ts
export class FindQuery {
  @Property()
  pageIndex?: number;

  @Property()
  pageSize?: number;

  sortBy?: string;

  @Property()
  globalFilter?: string;
}

The code above works fine with the older package versions, but for some reason now I get the following error from this @QueryParams() filter: FindQuery piece of code:
TypeError: Cannot read properties of undefined (reading 'FindQuery')
If I change the code to @QueryParams() filter: any this, it will pass the value as expected, but I guess the AJV validation then doesn't work anymore.
Am I missing something in my code, or has there been some changes to the @QueryParams() decorator?

Hello @tuunjuuso
I'll try to fix that :)

I'm not able to reproduce. The framework have the same usecase as integration test and it works as expected.

Have you the same packages for all Ts.ED package?
Please take a time to create a reproducible example! I close the issue until, the bug isn't reproducible.
See you
Romain

The full error is this:
[2022-05-19T13:39:45.219] [ERROR] [TechnicalLog] - Controller.fetch TypeError: Cannot read properties of undefined (reading 'FindQuery')
    at Map.refToParameters (/home/tsed-project/node_modules/@tsed/schema/src/domain/JsonParameter.ts:129:36)
    at Map.build (/home/tsed-project/node_modules/@tsed/schema/src/domain/JsonParameter.ts:90:25)
    at Map.toJSON (/home/tsed-project/node_modules/@tsed/schema/src/domain/JsonParameter.ts:71:17)
    at /home/tsed-project/node_modules/@tsed/schema/src/domain/JsonOperation.ts:229:52
    at Array.forEach (<anonymous>)
    at Map.toJSON (/home/tsed-project/node_modules/@tsed/schema/src/domain/JsonOperation.ts:223:28)
    at AuthMiddleware.use (/home/tsed-project/libs/serviceutils/src/auth/middleware.ts:38:39)
    at /home/tsed-project/node_modules/@tsed/platform-params/src/builder/PlatformParams.ts:86:39
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

So, if I change the parameter like this
@QueryParams() filter: typeof FindQuery
it doesn't cause any errors anymore.
And the missing @Property() was a copy-paste error. Sorry.
I am just wondering could this be cause by the SuperTest environment? I failed to mention that these error came from tests.
Anyways, I'll try to see if I can create simple reproduceable example. There are so many libraries involved in the code overall, that I have hard time finding the root of the cause.
Thanks for the quick reply!

So, if I change the parameter like this
@QueryParams() filter: typeof FindQuery
it doesn't cause any errors anymore.

Because it's transformed to an interface by typescript. But in this case, the @tsed/json-mapper won't be able to use the interface to map your input data.

I did some more digging. Even though I didn't change the version of tsed libraries, but did some other changes to packages, it still gave that error. So my best guess at the moment is that there is something wrong with something else in the project. At least my initial error post about updating the version is irrelevant.
I'll try to give an update after finding the reason.

I found the cause of the error. It is tsed related, though not sure what is the actual reason.
So the error came from our Middleware where we had this call:
import { Context } from "@tsed/common"; // Source of Context
...
  public async use(@Context() ctx: Context): Promise<void> {
    const { request, endpoint } = ctx;
    const { query, headers } = request;

    const opData = endpoint.operation.toJSON(); // THIS LINE

I used debugger to find out what was the function that caused the error, and it was this line in tsed/packages/specs/schema/src/domain/JsonParameter.ts
Other tests which don't use our custom Pagination (customized from your example) work just fine, but all the tests which are using the before mentioned FindQuery parameter crash.
The js code from debugger looks like this before crash:
    refToParameters(parameter, options, schemasContainer) {
        const name = (0, ref_1.createRefName)(this.$schema.getName(), options);
        const schema = options.schemas[name];

With values:
{
  name: "FindQuery",
  options: {
    groups: undefined,
  },
  parameter: {
    in: "query",
    required: false,
  },
  schema: undefined,
  schemasContainer: {}, // Map(0) {size: 0}
}

So yeah, it probably crashes because it can't find [name] from options.schemas which is undefined. Why it doesn't have all the required properties I don't know.
I would like to hear your thoughts @Romakita on the issue. Is this a valid bug, or is ctx.endpoint.operation.toJSON() just misuse of the library?
I hope this helps. Thanks!


Discord Invite Invalid

Hey, I am facing an issue using Cengal/this library (SharedMemory seems to mainly come from Cengal) and I want to reach out via Discord (or email), but the link in the README does not work.

Thank you! Fixed


Remove unnecessary print statement.

Coverage decreased (-0.002%) to 97.39% when pulling 08820fa6f5dc014ea49fd10cd2178ac8f662adaa on frechette-alex:master into dff4491e173859f841255734f7a40b5c2b81f4ee on dwavesystems:master.

Coverage decreased (-0.002%) to 97.39% when pulling 08820fa6f5dc014ea49fd10cd2178ac8f662adaa on frechette-alex:master into dff4491e173859f841255734f7a40b5c2b81f4ee on dwavesystems:master.

Duplicate with https://github.com/dwavesystems/dimod/pull/68.


Add all navbar components to styleguide

Tasks

Add all the updated navbar components to the styleguide
Make sure they're ready! Silver birch and Willow Glade are updating the mobile navbars

There were so many options, so I added the once that covered the most of them, and the rest are updated in the more detailed styleguide I think?


CUPS: fix misleading comment

Motivation for this change
ServerBin directive is set not in cupsd.conf but in cups-files.conf
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @

Thanks!


don't write out empty error span tags

It wasn't a huge problem, but it was i) irritating and ii) inconsistent.
Before

After

👍  great improvement. Let's merge it so I can use this for the new income flow!


Notarizing timeout error 404 Not Found

What happened:
All my attempts at notarizing end with a timeout.
In Quill, the timeout is 15 minutes, and in Goreleaser, if you set it to 60m or more, it results in a 401 error, as described in this issue: https://github.com/anchore/quill/issues/354.
I tried setting it to 50m, but the notarizing still ends with a timeout.
Here is the job log:
quill -v submission logs 42f706eb-6523-44cf-816d-3cfee55dcd6d
[0000]  INFO quill version: 0.5.0
[0000]  INFO fetching submission logs for "42f706eb-6523-44cf-816d-3cfee55dcd6d"
unable to fetch log metadata with ID=42f706eb-6523-44cf-816d-3cfee55dcd6d: http status="404 Not Found": body="{\"errors\": [{\"id\": \"69dd08b5-92a2-4c0c-a10b-27debde866e1\", \"status\": \"404\", \"code\": \"NOT_FOUND\", \"title\": \"The specified resource does not exist\", \"detail\": \"There is no resource of type 'submissionsLog' for id '42f706eb-6523-44cf-816d-3cfee55dcd6d'\"}]}"

What you expected to happen:
Successful notarizing
How to reproduce it (as minimally and precisely as possible):
I dont know
Anything else we need to know?:
I have a new developer account, this is my first attempt to sign and notarize binaries.
Environment:
quill - 0.5.0
goreleaser - 2.4.8-pro
Linux x86_64
openSUSE Tumbleweed

It seems that it was connected with the fact that I did it for the first time.
After 12 hours everything was certified, today it is certified quickly within 2-3 minutes.
https://developer.apple.com/forums/thread/769899


[feat] refactoring of stable mem ops to avoid extra function call  and save code

A final refactor of the load/store stable memory operations that avoids a function call to read/write operations (and saves some code)

Merged in claudio/stable-mem


Almost identical exercises: binary, trinary, octal, hexadecimal

In an unrelated discussion, it came to my attention that, just looking at the .md files, these 4 exercises are almost the same:

binary
trinary
octal
hexadecimal

At least in xhaskell, the implementations for both, tests and solutions, are mostly the same, and it's boring to solve the same problem multiple times.
Are there some subtle differences among them that I'm missing?
Are they that similar by design, or is it an historical accident?
(Possibly related to #223)

Yeah, the differences are very subtle: All are converting from different bases…
But yeah, I think you are right, these exercises are similar enough to do one of these:

Deprecate 3, keep one, add another one which has a parametrized converter convert(from, to, number)
Deprecate all and have the parametrized version
Deprecate 3 and keep only one, don't add anything else

Across all tracks I have implemented this, the major difference between these four are the way how I bitshift and which input I consider valid. The logic/recursing part is the same.

I'm in favor of combining these four exercises into a single exercise. The exercise should thus focus on being able to convert from different bases, with tests for at least two or three of the current bases.

Definitely related to #223.
I would favor collapsing all of this into two exercises:

binary, which remains as it is, except that we would encourage people to solve it using the standard library or usual tools.
trinary, using alternate symbols so that people can't use a trivial conversion (e.g. in Ruby: 120.to_i(3)`.

I really would like people to have a basic understanding of how numbers actually work.

So you suggest making trinary more of a crypto-kind with a fixed key? Sorry, but I really don't like it.
It is a numeric conversion, fullstop. In ruby it is ease as input.tr("abc", "210").to_i(3), in most other languages you have similar mechanisms, in the worst case, it is a lookup in a map while folding the input.
Having something else than plain arabic digits adds obfuscation and not value into the exercise, I think.
I will write something to #223 as well later this day.

That's fair. How would you design an exercise that encourages people to actually understand the underlying numbers?

I wish I had an idea, but to be honest, as long as we don't monkeypatch #to_i away (and we can't even do that in most languages), the only way to encourage it, is to write it out LOUD and bold and italic in the README :(
But I think, that asking for the following function/method might help: string convert(string input, int from_base = 2, int to_base = 10) (pseudo typesignature, but should be clear).
This can only work by converting from the input into an intermediate integer and then back into a string having the correct base.
Most languages I am aware of, have very easy stuff to convert a string represantation of an integer with base X into base 10, but not the other way round.

I think the exercise is more educative when it fuses the two problems:

Composition / Decomposition
Representation

The composition/decomposition - certainly the main problem as noted by @NobbZ - is just about conversions between a number and a list of numbers:
compose :: Integral a => [a] -> a
decompose :: Integral a => a -> [a]

It's a good starter problem!
The representation problem involves two functions:
encode :: Integral a => a -> Something
decode :: Integral a => Something -> a

What makes it interesting, IMHO, it that, when you combine those two problems, the student is forced to see that the two lists of symbols are equivalent by same underlying number, especially if they don't use common numeric symbols:
ColorsToAlpha :: [Color] -> [Char]

If one of the representations is numeric, it could wrongly think that there is a natural representation of a number, probably decimal. So I think there is some merit in having a encoded version of the problem too.

I really like the idea of having a more generic conversion exercise as suggested by @rbasso, but as @kytrinyx stated, this family of exercises was meant to teach numeric conversion.
And for numeric conversion there is probably only that one way of intermediate representation: the languages native integer type. We do not even know for sure, how it looks like internaly, all we know is, that values of this type are printed in their decimal representation in most languages or at least string-converted.

If these problems are just about composing/decomposing in digits, so there are still three distinct types of exercise possible:

construction (from a list of numbers to a number)
deconstruction (from a number to a list of number)
conversion (from a list of numbers to a list of number)

Those types has nothing to do with string conversions, but - from what I remember - the exercises are specified to work with strings of characters.

Yeah, currently they are as a string/list of chars/array of chars/whatchamacallit.
Perhaps a list of digits could in fact make it harder to use the stdlib, since most stdlibs are working on strings. But making it a string in the right format again is just a join away…

So, instead of the current problem descriptions,

Write a program that will convert a binary number, represented as a string (e.g. '101010'), to its decimal equivalent...

What do you think about...

Write a program that will convert a sequence of numbers, in base a, in another sequence of numbers, in base b.

I believe this is in the spirit of what @ErikSchierboom proposed and compatible with the signature you provided earlier.
This way we completely avoid mixing the base conversion problem with encoding in the descriptions,  and each language can implement it freely.

@rbasso correct, that is indeed what I was proposing.

@rbasso @ErikSchierboom I'm all for this approach!

:+1:  to base a -> base b (to replace all three existing exercises?) and for discussing here.

So... what would be the new exercise's name?

So... what would be the new exercise's name?

Good question. all-your-base? Too cheeky? How about base-to-base? Or numeric-conversion?

base-conversion? Naming things is hard...

+1 for all-your-base. We have to use that!

Into it.

Definitively a :+1: for all-your-base or all-your-bases! I like the pun…
But if it has to stay more pragmatic and punfree I'd go for base-conversion.

I like the consensus around converting from an arbitrary base to another arbitrary base that seems to have formed here. I and also strongly support naming the exercise all-your-base.

So, if I'm counting right, all-your-base it is.
Now, what?
I think we need a all-your-base.yml and all-your-base.md, right? Anything else?
I have a PR ready here, just waiting for confirmation... :smile:

Yepp, we need the yml and the md, and let's also start with a json file where we define the test cases. That will let us discuss and work out all the details, and it will be easier to create the various track implementations based on that.

I've posted a new issue that is about deprecating the old issues. Once the PR for all-your-base is merged we can close this one.

Working on that!

@NobbZ @ErikSchierboom @yurrriq @Cohen-Carlisle @petertseng if you have time to review https://github.com/exercism/x-common/pull/280 that would be great!

Now that all-your-base is merged, I'm going to close this issue in favor of https://github.com/exercism/x-common/issues/279


Improve text rendering

There are a few text issues that have slipped through the cracks a bit that we could polish up.

[ ] Nested quotes are unsupported
[ ] Newline after nested quotes isn't respected
Example:>>> Line 1
Line 2

will render as|
|
| Line 1 Line 2

[ ] Superscript follows bottom baseline instead of top baseline
[ ] Nested superscript isn't supported
[ ] Inline code not supported
[ ] We can use YYTextBorder for code blocks instead of making separate items in the TextStackView for them. Should we?

It looks like DTCoreText supports superscript, but the baseline is only offset for text that is drawn using DTCoreTextLayoutFrame. We instead pass the attributed string into YYText. How can we detect superscript in YYText and display it properly?

From what I'm seeing, YYText does support superscript, but it isn't rendering it with the correct baseline. It also screws up pretty badly depending on the font. Unless we want to write our own text renderer we'll just have to settle for one level of superscript.

I force superscript by doing , and remove all sup tags. I have found that text renderers are pretty terrible at this in iOS so I think overriding it is the way to go. As for the quotes, I tried that but there is no way to do a left-side border which I feel is the standard for a quote, so that method might not work well for us

Newline after nested quotes isn't respected

Reddit is the one turning them into one blockquote, not Slide. We just parse the html

All of these should be fixed now, except nested quotes. I feel that is niche enough to not warrant re-doing quote rendering


Sample for SQLite usage on Linux or Mac

Hey,
is there any working example available with ASP.NET vNext and EF7 on Linux or Mac?
I have checked already many resources and even tried to setup it by myself but Im unable to get it to work. An example would be really nice. Im currently writing my bachelor thesis about ASP.NET vNext and its cross platform capabilities. To proof the current state I need to integrate an simple web service with some REST API's. Instead of SQL Server I wanted to use SQLite because it should be available as well.
Any feedback about this topic would be really nice!
Thanks a lot in advice
regards
Schreda

Have you installed the x86 version of libsqlite3? On Ubuntu, the command is:
apt-get install libsqlite3-0:i386

Hey thanks for you reply. Yes the package is already installed. Today I tried to setup everything on OSX 10.10 to improve my workflow... except the database and the "dun build" bug is working. I used yo to generate a new project (Web Application), dnu restore, dnx . kestrel to run it... but as soon as I'm trying to Register a new User I get an exception:
SocketException: Could not resolve host '(localdb)'

So I tried to add SQLite support and tried to adding the
option.UseSQLite('Filename=sample.sqlite;')

instead of
options.UseSqlServer(Configuration["Data:DefaultConnection:ConnectionString"]).

but Im even unable to get the intenseness method signature. So Im somehow worry about my project configuration. By the way here is my project.json file:
{
    "webroot": "wwwroot",
    "userSecretsId": "aspnet5-test-aac12b4d-cbe2-4f2d-912a-eaad4f484b99",
    "version": "1.0.0-*",

    "dependencies": {
        "EntityFramework.SqlServer": "7.0.0-beta4",
        "EntityFramework.SQLite": "7.0.0-*",
        "EntityFramework.Commands": "7.0.0-beta4",
        "Microsoft.AspNet.Mvc": "6.0.0-beta4",
        "Microsoft.AspNet.Mvc.TagHelpers": "6.0.0-beta4",
        "Microsoft.AspNet.Authentication.Cookies": "1.0.0-beta4",
        "Microsoft.AspNet.Authentication.Facebook": "1.0.0-beta4",
        "Microsoft.AspNet.Authentication.Google": "1.0.0-beta4",
        "Microsoft.AspNet.Authentication.MicrosoftAccount": "1.0.0-beta4",
        "Microsoft.AspNet.Authentication.Twitter": "1.0.0-beta4",
        "Microsoft.AspNet.Diagnostics": "1.0.0-beta4",
        "Microsoft.AspNet.Diagnostics.Entity": "7.0.0-beta4",
        "Microsoft.AspNet.Identity.EntityFramework": "3.0.0-beta4",
        "Microsoft.AspNet.Server.IIS": "1.0.0-beta4",
        "Microsoft.AspNet.Server.WebListener": "1.0.0-beta4",
        "Microsoft.AspNet.StaticFiles": "1.0.0-beta4",
        "Microsoft.AspNet.Tooling.Razor": "1.0.0-beta4",
        "Microsoft.Framework.ConfigurationModel.Json": "1.0.0-beta4",
        "Microsoft.Framework.ConfigurationModel.UserSecrets": "1.0.0-beta4",
        "Microsoft.Framework.CodeGenerators.Mvc": "1.0.0-beta4",
        "Microsoft.Framework.Logging": "1.0.0-beta4",
        "Microsoft.Framework.Logging.Console": "1.0.0-beta4",
        "Kestrel": "1.0.0-beta4",
    },

    "commands": {
        "web": "Microsoft.AspNet.Hosting --server Microsoft.AspNet.Server.WebListener --server.urls http://localhost:5000",
        "kestrel": "Microsoft.AspNet.Hosting --server Kestrel --server.urls http://localhost:5001",
        "gen": "Microsoft.Framework.CodeGeneration",
        "ef": "EntityFramework.Commands"
    },

    "frameworks": {
        "dnx451": {},
        "dnxcore50": {}
    },

    "exclude": [
    "wwwroot",
    "node_modules",
    "bower_components"
  ],
    "publishExclude": [
    "node_modules",
    "bower_components",
    "**.xproj",
    "**.user",
    "**.vspscc"
  ],
    "scripts": {
        "postrestore": ["npm install", "bower install"],
        "prepare": ["grunt default"]
    }
}

By the way i tried to follow this tutorial as well but with all the latest libraries.
http://www.dotnetthoughts.net/how-to-use-sqlite-in-asp-net-5/.
Am I missing some additional dependencies? I really don't know why I get not the UseSQLite method.
I already tried to just write the method but server is than not starting anymore ...
Thanks a lot
regards
Schreda

The first error you're getting is from ASP.NET Identity (aspnet/Identity) trying to still use SQL Server. You may want to file an issue on their repo to see what their recommendation is for running on OSX.
For the second error, you'll need to add the EntityFramework.Sqlite package to your project.json file before the UseSqlite method will be available.

FWIW, ASP.NET Identity product code does not call UseSqlServer(). It will pick up whatever the DbContext is configured with in either Startup.cs or the IdentityDbContext's OnConfiguring() method.
cc @bricelam

Thanks a lot for the answers. Im currently using the InMemory Db after I finish all the other tasks I will return to the configuration of the sqlite version.

@natemcmaster was able to get SQLite working on OSX. Closing this issue for now, but please file any new issues you run into when trying to make it work.

Hey,
Thank you for your help, I made an Sample application based on the Microsofts Mvc Example.
It contains an SQLite Database and its fully working with beta-6.0.0 bellow is the repository link maybe someone else is searching for it :dancer:
https://github.com/Schr3da/ASP.net-vnext-samples

:metal: Rock on.

Also, we recently added an article to our documentation that uses SQLite on OSX:  http://ef.readthedocs.org/en/latest/getting-started/osx.html


Corrected bug in Flores fewshot tasks and added flores task

Corrected bug: fewshot examples were using the same source and target sentence
Added task: wmt_hi2en - take fewshot examples from wmt hi-en dev set

@rbawden Good catch! This LGTM - I've just formatted the files and removed debugging statements. Just want to confirm there's nothing else you'd like to add; then we can merge this.


Added month separator for schedule view

Summary of Changes
Added Full-Width Month Separator and Custom Styling Option

Month Separator Component: A new full-width month separator has been introduced to visually indicate the transition between months in the event schedule view. The separator displays the month and year (e.g., "October 2024") and spans the entire width of the container.

The separator is styled with center alignment, padding, and the primary theme color for visibility.

Custom Styling Option: Added a new scheduleMonthSeparatorStyle prop to the Schedule component. This allows users to apply custom styles to the month separator, providing flexibility for different use cases.

Separator Integration:

The month separator is rendered above each new month’s event group in the renderFlatListItem function.
Logic ensures that the separator only appears when transitioning to a new month, based on event dates.

Storybook Updates: Updated the corresponding stories to reflect these changes and showcase the month separator functionality with customizable styles.

Added 2 extra events to the story to highlight the new feature and demonstrate how the month separator works in different contexts.

Impact

Provides a clear visual distinction between months in the event schedule, improving usability.
The scheduleMonthSeparatorStyle prop enhances flexibility by allowing customization of the separator’s appearance.

@ARedmanDodd Thanks! That change is fantastic. The code looks great to me. Just one thing, can you fix this TSC check? Best regards
https://github.com/acro5piano/react-native-big-calendar/actions/runs/11795110129/job/33080336865?pr=1115

Thanks for the feedback, I'll get that fixed up now 👍

Changes submitted:
fix: update monthSeparatorStyle type to TextStyle for color support

Replaced ViewStyle with TextStyle for monthSeparatorStyle in CalendarContainerProps to allow the color property.
Fixed TypeScript error (TS2353) related to unsupported color in ViewStyle.

This change resolves the tsc error and ensures compatibility with style properties requiring color.

@ARedmanDodd Thank you for your contribution!

Thank you for accepting it!


About retraining BD/DN model of vdsr and srcnn

Thanks for your impressive work. You mentioned that you retrained VDSR and SRCNN for BD/DN model. Could you tell me that you strictly followed the original settings and retrained the VDSR, SRCNN model? or simply use same settings as your main methods but the structure?

For SRCNN, we used the official code for training on BD and DN degradation models.
For VDSR, we used vdsr-caffe for training (reason: No official training implementation).
Except degradation models, all settings strictly followed SRCNN/VDSR paper.

Thanks for your reply! So you implement bicubic interpolation on the generated DN/BD LR images to get the LR training images for  VDSR/SRCNN?

Yes.

OK, I got it. Thanks.


run_tests.sh - Make script more portable

It's a bit hacky but I think it should be more portable.
By the way, is there any reason to call the tests with root?

Any chance to have a look at this patch?

@tuxillo many thanks:)


Make comments jsdoc compatible

Similar to what was done in #18 for lib/DataFrame.js in 9675dd35bbff8ab5bc0ede5b1755981ab1666208, but for the other API wrappers in lib/

[x] DataFrame.js
[ ] DataFrameReader.js
[ ] GroupedData.js
[ ] functions.js
[x] sqlContext.js

Row.js doesn't need to be documented (at least yet) -- it is only used internally so far.

Note that (as of<PHONE_NUMBER>80c65682af48ed0e6e76ce637d44de) jsdoc creates docs for the class SQLContext and its constructor, which isn't accessible outside of sqlContext.js. Need to figure out how to address this.

Also need to make a pass to remove the backticks in comments. This isn't markdown, they just add noise.

lib/sqlContext.js done in<PHONE_NUMBER>80c65682af48ed0e6e76ce637d44de and 81be03622047bd78b249316ab7b9d7fe8f3644d8
lib/DataFrameReader.js done in c73dbdc6ce189db3152c719d4b5c7739b138351f
lib/GroupedData.js done in bdf9d1d5830f2507ea96e4d0290397440dc0ab6e

lib/functions.js done in b30df948ca88b095f11d96c60244af9f76a01352

lib/Column.js docs done in da6c0cd515381ba589e6f906e38bf7b32f16e2e5 and db2f67eeff8610e22f0278e1527ca77e70b29730


Add a member to a class

Would be great if you could get "add a member to class" style of refactoring.

Keybinding : alt+enter

I would like to see this also mapped as alt-enter like #256.
If possible, I would like alt-enter to be the standard "fix this" command. I am not sure if Atom has a good API for this but if possible. This would be a killer feature.

I would like alt-enter to be the standard "fix this" command

Agreed.

I am not sure if Atom has a good API for this but if possible. This would be a killer feature

Can be done easily (using atom services). But not my goal here :heart:.

And when in doubt, just ask : https://github.com/Microsoft/TypeScript/issues/2788

Sweet, I will look in to this on Friday. I'm excited!

@filimon-danopoulos I'm still working on this. Just documenting my process for transparency :rose:

After my next push:

Next up will be type inference, followed by documentation and then this issue can be closed.

I think this would look way better as part of the original message. Instead of using the "pop-up" for lack of better term.
So instead of
Property X missing on Y

we would have
Property X missing on Y.
Quick fix (alt-enter): Add member to class.

What do you think about that?

Then alt-enter would immediately fix it

Also intend to (later) add stuff like "remove unused imports" which will be there in all alt-enter.

I think that might be a design flaw. When I am (and presumably other developers) typing code I don't want distractions or having to take unnecessary decisions.
In this case I want to be able to tell the editor to fix a particular error. I have already decided what I want to do and as long as there is no confusion about how to do it I would expect the editor to do it. In this case just add the member, if there are multiple choices I can be show some sort of prompt.
If I am given choices at this stage it will be disruptive. Since I am probably not interested in general refactoring functionality I should not be shown that information.
Additionally it feels like that type of command (remove imports) belongs under ctrl-shift-p in the atom workflow not under a general "fix-it" type command.
I might have misunderstood what you intended with quick-fix but this is not what I expected.

Glory:

I don't want distractions or having to take unnecessary decisions

You don't have to. Press alt+enter followed by enter and it will just work :rose: (if we have done our sorting right)

That's looking just great! Almost like ReSharper.
BTW is there any particular reason that you create the first 'something' member as public. I would probably have made it private.

is there any particular reason that you create the first 'something' member as public. I would probably have made it private

The target is ES6 js to ts code refactoring. The ES6 code this.something is assumed public.

@basarat That screencap looks quite good, I like it much better than the first draft. I wonder how you would integrate other refactor functionality into this menu without making it unclear?

other refactor functionality into this menu without making it unclear?

Refactoring can (in future) come with an optional html display string :) with embedded icons 🌹


[v2.5] KDM release: k8s july patches

Kubernetes updates:

This is a KDM release for new k8s versions:  v1.19.13 and v1.20.9.  It's not going out with any Rancher release, it's a KDM + RKE release only.

New k8s versions
Rancher versions

v1.19.13-rancher1-1
2.5.0+

v1.20.9-rancher1-1
2.5.6+

Rancher updates:
Note: Please update system images and addons in the template as and when required.

rke-tools is updated to v0.1.77

Developer changelog: - TBD
Note: Please update system images and addons in the template as and when required.

There are existing k8s versions in dev-v2.5 that haven't been released (absent in release-v2.5), so the same versions can be updated. Following versions need to be updated with their compatible rancher server versions:

update version from
to
update rancher version constraints

v1.20.8-rancher1-2
v1.20.9-rancher1-1
2.5.6+

v1.19.12-rancher1-2
v1.19.13-rancher1-1
2.5.0+

Following system images need to be updated:

Name
Kubernetes Version
Version

Update constraints for all addon versions so the new Kubernetes versions use the right template. Refer to the table below to confirm the template name.

 Confirm the versions for System Images and Addon templates are correct. Update this table if new versions get introduced. Click to view the full list.

Name
Kubernetes Version
Version
Update constraints for addon versions if required

etcd
v1.18.17
v3.4.3-rancher1

etcd
v1.19.9
v3.4.13-rancher1

etcd
v1.20.5
v3.4.14-rancher1

kube-dns
v1.18.17
v1.15.2
kubeDnsv116

kube-dns
v1.19.9, v1.20.5
v1.15.10
kubeDnsv116

core-dns
v1.18.17
v1.6.9
coreDnsv117

core-dns
v1.19.9
v1.7.0
coreDnsv117

core-dns
v1.20.5
v1.8.0
coreDnsv117

flannel
v1.18.17
v0.12.0
flannelv116

flannel
v1.19.9, v1.20.5
v0.13.0
flannelv116

calico, canal
v1.18.17
v3.13.4
canalv117PrivilegedCalico3134, calicov117Privileged

calico, canal
v1.19.9
v3.16.5
canalv3165, calicov3165

calico, canal
v1.20.5
v3.7.2
canalv3171, calicov3171

weave
v1.18.17
v2.6.4
weavev116

weave
v1.19.9
v2.7.0
weavev116

weave
v1.20.5
v2.8.1
weavev120

aci
all
<IP_ADDRESS>.1ae238a
aciv500

metrics-server
v1.18.17, v1.19.9
v0.3.6
metricsServerv18

metrics-server
v1.20.5
v0.4.1
metricsServerv120

podInfra
v1.18.17
3.1

podInfra
v1.19.9, v1.20.5
3.2

node local
v1.18.17
1.15.7
nodelocalv115

node local
v1.19.9, v1.20.5
1.15.13
nodelocalv115

rancher forks

dns-autoscaler
v1.18.17
v1.7.1

dns-autoscaler
v1.19.9, v1.20.5
v1.8.1

flannel-cni
all
v0.3.0-rancher6

ingress
v1.18.17, v1.19.9
nginx-0.35.0-rancher2
nginxIngressV11512

ingress
v1.20.5
nginx-0.43.0-rancher1
nginxIngressV11512

ingress-backend
all
1.5-rancher1

rke-tools
all
v0.1.71

windowsPodInfra
all
v0.1.6

Note milestone is set to v2.5.10 mostly for easier tracking, we still want this as separate KDM + RKE release as per the issue description.

Verified on v2.5.9
Fresh install checks - 2.5.9 - pointing to dev-v2.5

Deployed clusters with k8s version v1.19.13-rancher1-1, v1.20.9-rancher1-1 on all network providers - weave/flannel/calico/canal
Validations of Network checks on all clusters look good
Validation tests on Canal cluster look good

Upgrade checks - v2.5.9 pointing to release-v2.5

Deployed cluster with k8s v1.19.12-rancher1-1, v1.20.10-rancher1-1 version on all network providers weave/flannel/calico/canal with KDM job pointing to release-v2.5
pre upgrade checks look good
Upgrade KDM to dev-v2.5
Upgrade K8s versions to new versions - v1.19.13-rancher1-1, v1.20.9-rancher1-1
Post upgrade checks look good.

Min checks on 2.5.3

Deployed 2.5.3 pointing to dev-v2.5
Created Clusters for k8s versions -v1.19.13-rancher1-1 with all network provider
Validations of Network checks on all clusters look good

RKE checks

Deployed a k8s cluster using RKE version v1.2.10-rc0 and k8s - v1.19.12-rancher1-1, v1.20.10-rancher1-1
Basic checks on this cluster worked.


Follow a style guide and use a linter

Right now the codebase has a pretty... disparate... style. Lots of very long lines, for example.
We may want to consider picking a Rust style guide to follow (are there any ones in widespread use?) and then setting up a linter (configuring rustc?) to make sure we have a somewhat uniform style.
This is ultra low priority, however. Just something to keep in mind as the project matures.

Agreed! Linter wise I always hear about clippy


Confusing FN Icon Behaviour

I think there is some potential for confusion with the FN icon in the taskbar of the VM. I would expect clicking on the FN icon on the taskbar to start FakeNet. I now know I can run FakeNet by typing fakenet into this window, however new users could be confused by the fakenet.py usage information.

Updated behavior to launch fakenet once you click on the icon instead of displaying help


Not work

I install plugin and nothing wrong.
How use it?
Version IntelliJ IDEA 2022.1.2

https://github.com/e1fueg0/intellij-gitea-plugin/issues/6

I configure server. But how i can use this plugin?
пт, 15 июл. 2022 г. в 17:13, Roman Pedchenko @.***>:

#6 https://github.com/e1fueg0/intellij-gitea-plugin/issues/6
—
Reply to this email directly, view it on GitHub
https://github.com/e1fueg0/intellij-gitea-plugin/issues/12#issuecomment-1185396427,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2CF5DGGHCU6NGIX5GIDFPLVUE2VZANCNFSM53UUOVDA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
С уважением, Виктор Тарасов
E-mail: @.***
Моб.тел.: +7(902) 983-34-31
Гор. тел.: +7(384-2) 33-34-31
ICQ:   282348497

Have you ever used another issues trackers?
Try Tools -> Tasks & Contexts -> Open task.


update usage of rcutils_join_path()

Connects to ros2/rcutils#102

1440372 (bigger than I thought) should address the memory leak related to the use of rcutils_join_path().


[bug] Height issue with label that shows plugin author name

Description
The height of the label showing the plugin author's name in a Search Results list item is currently too limited. The text is cut-off right under the label's baseline (i.e. characters like "g", "y", etc. aren't shown entirely).
OS X Version
macOS High-Sierra 10.13
Sketch Plugin Manager Version
1.6
Screenshot

Did you notice if this was an issue before upgrading to High Sierra? I believe it may be related to the default font used by your system. I haven't updated to High Sierra yet and am not seeing the issue, so I'm wondering if it's related to the OS.

I started using this plugin manager after installing High Sierra so I'm afraid I can't confirm.


Remove "final" from most of the classes

This will allow mocking StorIO because at the moment we have to cheat and use real db via Robolectric in tests.
// Also, I'm slowly working on MockStorIOSQLite
@nikitin-da PTAL :)

Ook


Ctrl + Q keybind isn't keyboard-layout aware.

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.85.2
OS Version: Arch Linux 6.7.6

Steps to Reproduce:

Switch to a keyboard layout that doesn't map the letter "Q" to the key below "1" and "2" (in my case the xkb layout "il" that maps / to that key)
Press control and the aforementioned key

VSCode stops as if Ctrl+Q was pressed. But I expected to toggle the comment on the current line.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.87.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Updated and checked; still relevant.


Key Exchange in NIF

@SanjoDeundiak the goal is to make it possible to replace these functions with Rust equivalents
https://github.com/ockam-network/ockam/blob/develop/implementations/elixir/lib/ockam/channel/xx.ex#L116-L212

There is a possibility that a call like encode_message_2 is too long to be a NIF ... the concurrency strengths of Erlang VM come from being able to easily schedule thousands of concurrent processes. A long running NIF can totally kill that .... so we;ll have to measure once we have the above in place.
If this works, that would be win because then we won't have to maintain an elixir version of the key exchange.

To start let's set this up in the ockam_vault_software mix project itself.  We can move to a different project later
https://github.com/ockam-network/ockam/tree/develop/implementations/e/applications/ockam_vault_software

There is a possibility that a call like encode_message_2 is too long to be a NIF ... the concurrency strengths of Erlang VM come from being able to easily schedule thousands of concurrent processes. A long running NIF can totally kill that .... so we;ll have to measure once we have the above in place.
If this works, that would be win because then we won't have to maintain an elixir version of the key exchange.

Encode message should be much faster than network request, db read/write, etc. So, if those things don't harm Erlang concurrency, how encode message would?

The answer to that requires a deep dive into how the Erlang VM does scheduling of millions of processes.
Here's an old talk that delves into that https://www.youtube.com/watch?v=tBAM_N9qPno
In short, long running native functions are problematic.
Here's the warning form the docs, which provides a heuristic

a well-behaving native function is to return to its caller within 1 millisecond

http://erlang.org/doc/man/erl_nif.html#WARNING
http://erlang.org/doc/man/erl_nif.html#lengthy_work

Here is a short overview of the challenges from the tranches https://www.youtube.com/watch?v=xmUfTl33-fU#t=32m27s in just 3 minutes :)

Duplicate of #676


update

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR introduces new features for managing token components in a React application, enhancing the user interface for ERC20 and native crypto tokens, and updating documentation links for better guidance.
Detailed summary

Added a new token navigation link with name Token.
Updated documentation links in page.tsx files.
Introduced a new SVG asset for usdc.svg.
Created a new Page component for token components with descriptions and examples.
Implemented multiple token components: TokenImageBasic, TokenImageOverride, TokenNameBasic, TokenSymbolBasic, and TokenCard in token-examples.tsx.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5677  👈 (View in Graphite)
#5657
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


[Bug report] MySQL catalog: Load MySQL TIMESTAMP  data in Trino , it turns to TIMESTAMP WITH TIME ZONE data type

Version
main branch
Describe what's wrong
In MySQL data source:
create table my_tb05 (col_timestamp TIMESTAMP);

insert into my_tb05 values (CURRENT_TIMESTAMP);

select * from my_tb05;
+---------------------+
| col_timestamp       |
+---------------------+
| 2024-11-23 09:43:51 |
+---------------------+
1 row in set (0.01 sec)

Create MySQL catalog in Trino, load the data in MySQL:
call gravitino.system.create_catalog(
    'gt_mysql1',
    'jdbc-mysql',
    map(
        array['jdbc-url', 'jdbc-user', 'jdbc-password', 'jdbc-driver', 'trino.bypass.join-pushdown.strategy'],
        array['jdbc:mysql://<IP_ADDRESS>/?useSSL=false', 'trino', 'ds123', 'com.mysql.cj.jdbc.Driver', 'EAGER']
    )
);

create schema gt_mysql1.db1;

use gt_mysql1.db1;

 select * from my_tb05;
      col_timestamp
-------------------------
 2024-11-23 09:43:51 UTC

Error message and/or stacktrace
N/A
How to reproduce
refer to the description section
Additional context
No response

@diqiu50  Can you help check this issue?

I think this is the correct mapping of MySQL 'timestamp' to trino as' TIMESTAMP (n) WITH TIME ZONE '. If you want to remove TIME ZONE, you can use CAST (col_timestamp AS TIMESTAMP) to convert it


Update validator route with pagination

Description
Regarding https://github.com/iotaledger/iota-sdk/pull/1080
and https://github.com/iotaledger/iota-sdk/issues/1053
The pagination wasnt specified yet and should be properly added when the tip updates this

Closed by https://github.com/iotaledger/iota-sdk/pull/2101


Add hashing for coordinate class

Added hash algorithm for Coordinate object using XOR, bit shifts and prime multiplication

I think the hash collisions are still high. You can refer to this code

#include <bits/stdc++.h>

std::size_t getHash1(int x, int y, int z)
{
    return (std::hash<int>()(x) * 31) ^
           ((std::hash<int>()(y) * 37) << 8) ^
           ((std::hash<int>()(z) * 41) << 16);
}

std::size_t getHash2(int x, int y, int z)
{
    // Minecraft coordinate bounds
    int lower = -3e7, upper = 3e7;
    size_t base = upper - lower + 1;

    // Make x,y,z non negative
    size_t nx = x - lower, ny = y - lower, nz = z - lower;

    // Use overflow instead of modding
    size_t hash = nx * base * base + ny * base + nz;
    return hash;
}

int main()
{

    std::set<std::size_t> visited1, visited2;

    int startX = 0, endX = 1000;
    int startY = 0, endY = 20;
    int startZ = 0, endZ = 1000;

    for (int x = startX; x <= endX; x++)
    {
        for (int y = startY; y <= endY; y++)
        {
            for (int z = startZ; z <= endZ; z++)
            {
                visited1.insert(getHash1(x, y, z));
                visited2.insert(getHash2(x, y, z));
            }
        }
    }

    int expectedSize = (endX - startX + 1) * (endY - startY + 1) * (endZ - startZ + 1);
    std::cout << "getHash1 collisions: " << expectedSize - (int)visited1.size() << "\n";
    std::cout << "getHash2 collisions: " << expectedSize - (int)visited2.size() << "\n";
}

Looks good to me, lets go with that then

Would be great to get a solution with minimal collisions. Additionally, some sort of testing would be nice to make sure this compiles and runs on all systems. In any case, this is delayed until next release.

@JoshEliades are you willing to write some tests for this implementation? If not maybe @nhatdongdang can do it

I can write some tests for the implementation


Allow JS values as ZkProgram inputs

We can already pass in normal js values when witnessing provable types. This PR enables passing normal values into ZkProgram provers (both normal and recursive provers).
Also improves the supported JS values for a few important types like Signature and UIntX

Ready for review!

with #1931 and #1933 approved, this is ripe for review!


668 deposit tests

https://github.com/ConnextProject/indra/issues/668 has a checklist
Update: Only doing low hanging deposit tests in this PR so we can merge more frequently. Will amend the issue above too.

Looks good, I'll keep an eye on it & merge once tests pass


ContentPart attribute is not included in placement.info documentation

From what I can tell, the ContentPart attribute has been available for the Match element in placement.info since Orchard 1.7.2, but there is no documentation on it. I've made an update to the documentation and will submit a PR shortly.

Thanks, I try to make a note of the version that it changed so if you could work the v1.7.2 in to the wording that would be great.

Fixed with #300


New Profile Card

I would like to add a new profile card. Kindly assign it.

Hello! @abhijitmondal121 , I would like to contribute in this project. Kindly assign it to me.


Add golang github actions

This will add some CI testing linting and building.
A test needs to be fixed. Please either edit my PR or fix it in master I'll rebase upon the fix.

Please see  https://github.com/jooola/zabbixctl/runs/323253297

I've fixed the issue, could you please rebase your fork and check that action?

Done, this is now a basic CI. I'll submit another PR to setup release process.
Also I might come up with a bigger PR to restructure the project.

See https://github.com/jooola/zabbixctl/runs/323476691

Thanks!


Enhancement - dynamic center point update when diameter is selected

The center point could dynamically update as the user moves the cursor when Diameter is selected in the drop down.  This will prevent the Center Point location to display a different location than the user initially selected and could be confusing.

This issue was moved to ArcGIS/military-tools-for-arcgis-build#7


feat(server): add an option to enable/disable dynamic model loading

@Ladicle , how did you generate the values schema? I followed https://www.arthurkoziel.com/validate-helm-chart-values-with-json-schemas/

make generate-chart-schema (or make lint)

thx!


[1118] fix for division in py3

fixes second issue from #1118

Codecov Report

Merging #1120 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master    #1120   +/-   ##
=======================================
  Coverage   41.77%   41.77%
=======================================
  Files          69       69
  Lines       13249    13249
  Branches     2218     2218
=======================================
  Hits         5535     5535
  Misses       7286     7286
  Partials      428      428

Impacted Files
Coverage Δ

autobahn/websocket/protocol.py
27.74% <0%> (ø)
:arrow_up:

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 ac1dd18...550bf6e. Read the comment docs.


Update README with developer guide

For the non-docker version,

It's not mentioned anywhere that you also need to install play
it's not explained how to start it after packaging it with activator

Thanks @pliguori. You are talking about something like a developer guide? To get you started for development? Or a user getting started?
You don't need activator, sbt is perfectly fine. Activator is just sbt on drugs.

I'm talking more about a developer guide, since a user should just use the dockerized version, right? But if you want to develop and you're not familiar with how scala/play normally work, you have to spend some time to get it.
I assume just installing sbt/play may be fine, but if you don't write it, I don't know :)

@ishmeetkaur you should be able to develop the UI using the docker image and mounting the public folder from your filesystem.
docker run -it -v $(pwd)/public:/cluster-broccoli/public --entrypoint bash frosner/cluster-broccoli


Remove unneeded dependencies

awc is only used by the CLI, and cargo-llvm-cov is installed from the Makefile as a tool.

great! nice catch, thank you!

Thanks for the contribution!