Request money - JOD currency adds extra '0' at the end of any amount

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

Action Performed:
step 1: click on + FAB
step 2: click on request money
step 3: choose 'JOD' currency
step 4: select a user and request money
step 5: observe amount when request page opens
Expected Result:
there will be only two digit after . in the money amount
Actual Result:
there are three digit after . in the money amount
Workaround:
Unknown
Platforms:

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

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: v1.3.66-3
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/115492554/48b9d95c-d7a6-41bd-bd04-be52117bada9
https://github.com/Expensify/App/assets/115492554/add92c8b-3a14-4b1e-90a5-6c6168313071
Expensify/Expensify Issue URL:
Issue reported by: @lidiyakelay
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1693334339339819
View all open jobs on GitHub

I don't think this is a bug. Subunit of JOD is 0.001

Yep, agreed. I confirmed JOD has a sub-unit of .001, so I don't think this is a bug. Closing!


JS Interop Change

JS functions are now called with the prefix of "window".

Can you please provide the error you get?
I just tested it with 0.5.1 and it works for me.

Sure enough. I just rolled it back and it's working now. Odd. If memory serves, when I attempted to call SetItem it threw a JS error saying that Cloudcrate_AspNetCore_Blazor_Browser_Storage wasn't found. Maybe I had something cached.


fix lombok version in build.gradle

had some problems to get lombok running with intellij 2020.3 due to the version mismatch.

Same as https://github.com/meilisearch/meilisearch-java/pull/92#issuecomment-748859128 😇

bors merge


TRPO Agent not working for multi discrete action space

I am trying to implement TRPO agent for a custom environment with MultiDiscrete Action Space. But I am getting "NotImplementedError". The code and system are fine because same code is working for a Discrete Action Space.
from gym import spaces, Env
from sb3_contrib import TRPO
import numpy as np

class TestEnv(Env):

    def __init__(self):

        self.action_space = spaces.MultiDiscrete([2,2])
        # self.action_space = spaces.Discrete(4)
        self.observation_space = spaces.Box(low=np.array([0,0]), high=np.array([100,100]))

    def reset(self):
        return np.array([10,10]).astype(np.float32)

    def step(self, action):
        print(action)

        return np.array([10+action[0],10+action[1]]).astype(np.float32), -1, False, {}
        # return np.array([10+action//2,10+action%2]).astype(np.float32), -1, False, {}

from stable_baselines3.common.env_checker import check_env

train_env = TestEnv()
check_env(train_env)

model = TRPO('MlpPolicy', train_env,n_steps=5, batch_size=5)
model.learn(2)

Output Error
Traceback (most recent call last):
  File "/Users/akashgarg/Desktop/PycharmProjects/rlbaselinee_practice/Burger-Dog-AI/enjoy.py", line 27, in <module>
    model = TRPO('MlpPolicy', train_env,n_steps=5, batch_size=5)
  File "/Users/akashgarg/Desktop/PycharmProjects/rlbaselinee_practice/lib/python3.7/site-packages/sb3_contrib/trpo/trpo.py", line 426, in learn
    reset_num_timesteps=reset_num_timesteps,
  File "/Users/akashgarg/Desktop/PycharmProjects/rlbaselinee_practice/lib/python3.7/site-packages/stable_baselines3/common/on_policy_algorithm.py", line 267, in learn
    self.train()
  File "/Users/akashgarg/Desktop/PycharmProjects/rlbaselinee_practice/lib/python3.7/site-packages/sb3_contrib/trpo/trpo.py", line 282, in train
    kl_div = kl_divergence(distribution.distribution, old_distribution.distribution).mean()
  File "/Users/akashgarg/Desktop/PycharmProjects/rlbaselinee_practice/lib/python3.7/site-packages/torch/distributions/kl.py", line 169, in kl_divergence
    raise NotImplementedError
NotImplementedError

System Info
OS: Darwin-21.5.0-x86_64-i386-64bit Darwin Kernel Version 21.5.0: Tue Apr 26 21:08:22 PDT 2022; root:xnu-8020.121.3~4/RELEASE_X86_64
Python: 3.7.0
Stable-Baselines3: 1.5.1a7
PyTorch: 1.11.0
GPU Enabled: False
Numpy: 1.21.6
Gym: 0.21.0

Nice catch! Looking at how SB3 fixes this it uses this function to do kl_div, which has explicit check for "MultiDiscrete" spaces. Changing TRPO code to use this kl_div function instead of torch one will probably fix the issue. A PR to do so would be gladly accepted :)


Rename devenv up to devenv processes up

To be more consistent between commands like devenv container processes.

👍

that's now included, devenv up exists for convenience.


【Feature】auto save canvas

@vicky469 the canvas drawings is saved to Browser Local Storage. What are you exactly referring to here ?


User listing is paginated & filterable by name

Can page through by 100 users.
Can enter the beginning of a name and see a shorter list of users. Can clear the filter.

https://firebase.google.com/docs/firestore/solutions/search

I think I know where this would go but adding the blocked on design tag


Release Google.Cloud.Retail.V2 version 1.0.0-beta01

Changes in this release:
Initial release, for customers in the early access programme.

(The previous release failed due to the smoke test being incorrect.)

Created release for Google.Cloud.Retail.V2-1.0.0-beta01


add offline time related fields to eicoop

Just an FYI, update the FAQ section as you are now displaying Offline column


Getting query failure metrics as NaN always.

I am using the druid version of 0.12.

hi @msaha1994, I'd need a couple of info:

Are you using the last upstream version? If so, what config file are you using?
There is a debug option, can you try to use it and see if any error pops up?

I am not using the latest upstream.

Please do test the last upstream if possible :)
You need to instruct Druid to send metrics to the ip:port that the exporter is listening on, please check https://github.com/wikimedia/operations-software-druid_exporter#how-does-it-work.

I have taken the upstream pull. I am using the below command to run this.
sudo nohup .venv/bin/druid_exporter /home/ubuntu/operations-software-druid_exporter/conf/example_druid_0_12_3.json
My druid common.runtime.properties are-
druid.monitoring.monitors=["io.druid.java.util.metrics.SysMonitor","io.druid.java.util.metrics.JvmMonitor","io.druid.java.util.metrics.JvmCpuMonitor","io.druid.java.util.metrics.JvmThreadsMonitor","io.druid.server.metrics.EventReceiverFirehoseMonitor","io.druid.server.metrics.QueryCountStatsMonitor"]
druid.emitter=http
druid.monitoring.emissionPeriod=PT1m
druid.emitter.logging.logLevel=info
druid.emitter.http.recipientBaseUrl=http://<IP_ADDRESS>:8000/

Still not getting the data for query druid_broker_query_countt.
Do i need to chnage in collector.py?

Have you tried to query the brokers some times to see if datapoints are generated?

should i try curl http://localhost:8000/query/time/?

Nope what I mean is querying the druid broker itself, your example seems to point to the prometheus exporter. If you don't query the Druid broker no data point will be emitted and sent to the prometheus exporter, that will have no data to visualize.

My druid's recipient base URL port and operations-software-druid_exporter port both are the same. Will it be a problem?

Mmmm it is a bit strange, how could two different daemons (druid broker and prometheus exporter) bind to the same port on the same host? Are you sure?

So if I will set recipientBaseurl=http://<IP_ADDRESS>:8000/ and running exporter as  sudo nohup .venv/bin/druid_exporter --listen <IP_ADDRESS>:8000&
It should work??

Ah sorry I misread the above: yes please set recipientBaseurl in the Druid configuration to the address of the prometheus exporter. Then you have to query your Broker on port 8082 and see if datapoints appear (if it is not 8082 then the port that you configured for the broker).

I should try this curl http://<IP_ADDRESS>:8082/druid/broker/query/time right?

@msaha1994 you should try to query the Druid brokers as you would do to simulate any Druid workload, and  then see if any datapoint is visualized. The flow is:
user-query -> Druid -> datapoint-via-http -> prometheus exporter  <- query for datapoints


security/crowdsec: bump version 1.0.3; acquire packet filter logs.

The filter logs are used by a new scenario in the opnsense collections to detect port scans.

Merged, thanks!


Add send form tutorial

Dear @MintoD,
As far as I know, PocketMine School has been left unmaintained for over 6 months as of writing this. If you want your edit to be live as soon as possible, consider submitting a pull request to KygekTeam's forked version of PocketMine School, which is currently maintained actively by KygekTeam. We will review your pull request as soon as possible.
Thanks for your attention.
-- KygekDev

UPDATE
Dear @MintoD,
PocketMine School is now alive again! We will merge this pull request right after #16 has been merged.
-- KygekDev

This PR will be closed because I've made such changes in https://github.com/PocketMine-School/Pocketmine-School/commit/ac2520c4a36ef352bde97a7a93d148182be6f5fa


Fill Mode Gives an Error

scrollView.parallaxHeader.mode = MXParallaxHeaderModeFill -> this line gives me the following error:
Use of unresolved identifier 'MXParallaxHeaderModeFill'
but i can cmd+click on the MXParallaxHeaderModeFill and still get the page where it's defined. am i doing something wrong everything else is working but i can't work without this line because the header image is squashed when i run. Please help. Danke!

nvm. got it. .Fill is the right one...  amature problems :)

No worries :+1:


Properly parse unicode const chars

Ensure that the characters are unicode escaped
prior to passing them to ord, otherwise things
might fail.

This seems to be failing on two of the four versions of python that CI system has. I'll check it out and fix those issues.

Fun, I learned something new about both my dev environment and python -- didn't even notice you had a utility function for string escaping until this error appeared in Travis.
@d0c-s4vage This pull request should be all set and ready to merge. Let me know if it doesn't fit the styling or something.

w00t!! Awesome! Thanks for the pull request @strazzere! Haha didn't know
pull requests were run through travis-ci :^) TIL and unit tests FTW!
Python 2 and Python 3 support was initially a huge pain to setup, not too
bad now though with six and the utils. I should probably add some
developing/contributing notes somewhere...
I'll check this out later this morning, should be good to merge.
On Wed, Aug 19, 2015, 3:27 AM Tim Strazzere<EMAIL_ADDRESS>wrote:

Fun, I learned something new about both my dev environment and python --
didn't even notice you had a utility function for string escaping until
this error appeared in Travis.
@d0c-s4vage https://github.com/d0c-s4vage This pull request should be
all set and ready to merge. Let me know if it doesn't fit the styling or
something.
—
Reply to this email directly or view it on GitHub
https://github.com/d0c-s4vage/pfp/pull/5#issuecomment-132491438.


Feature Request - Using colors other than Primary

Are we able to use colors other than Primary in components such as UButton by passing them via app.config.ts like this...
A good solution would be is to let us pass an object of colors shades (50 to 950) to the palette in app.config.ts, and create the css variables for them (--color-secondary-50.. ), safelisting the patterns automatically similar to how it's currently done for primary. Obviously adding it to the list of valid colors, so color prop validator accepts it.
This color shades object format is the same as what is in tailwindcss/colors. I have added that below just to show the relationship. I understand I could simply do secondary: 'teal' ;-)
For eg:
import twColors from "tailwindcss/colors";

export default defineAppConfig({
  ui: {
    gray: 'cool',
    primary: {
	    50: '#f6f0ed',
	    100: '#efddd3',
	    200: '#e7cab8',
	    300: '#deb79f',
	    400: '#d5a486',
	    500: '#ca926d',
	    600: '#bf8055',
	    700: '#af754e',
	    800: '#9f6b47',
	    900: '#8f6040',
	    950: '#805639',
	    DEFAULT: '#714c32',
	  },
	secondary: twColors.teal,
    tertiary: {
	    50: '#ecebeb',
	    100: '#cdc3c0',
	    200: '#af9d97',
	    300: '#917870',
	    400: '#73544a',
	    500: '#553328',
	    600: '#381404',
	    700: '#331304',
	    800: '#2e1103',
	    900: '#2a1003',
	    950: '#250e03',
	    DEFAULT: '#210c02',
	  },
  },
})

Currently, I'm able to achieve secondary & tertiary in UButton by adding the

css variables myself (via a custom tw plugin) --color-secondary-50.. to 950,
extend the theme, the above color objects, and then
adding them to the safelist (like below), but obviously just adding bg & text to the safelist is not enough.

// I'm adding css variables for primary, secondary & tertiary (--color-primary-50 etc) using a custom tailwind plugin. So, passing these here just sets the default to null for primary, so there is no flash of change in primary color

// app.config.ts
export default defineAppConfig({
  ui: {
    gray: 'neutral',
    primary: 'primary',
    secondary: 'secondary',
    tertiary: 'tertiary',
  },
})

// tailwind.config.ts

  // ...
  safelist: [
    {
      // primary is already safelisted by nuxtlabs/ui
      pattern: /(bg|text)-(secondary|tertiary)-(50|100|200|600|700|800|900|950)/,
    },
    {
      pattern: /(bg|text)-(secondary|tertiary)-(50|100|200|600|700|800|900|950)/,
      variants: ['dark'],
    },
  ],

Alternatively, when adding custom colors in app.config.ts, the below two could be done

the user could be responsible to create the css variables for those custom colors (--color-secondary-50... to 950) & extend the theme with those custom colors. THIS IS CUMBERSOME of course :-)
apply the safe safelisting rules (https://github.com/nuxtlabs/ui/blob/dev/src/module.ts#L123) of primary to those custom colors such as secondary & tertiary
add to the valid list of colors so that validator of color prop don't throw warning

Thank you!

Have you checked the color prop? https://ui.nuxtlabs.com/elements/button#style

Ohh yeah, but can it accept custom colors other than primary, for eg, secondary, tertiary etc?

Why not just define your other colors like this?
import type { Config } from 'tailwindcss'
import colors from 'tailwindcss/colors'
export default <Partial<Config>> {
  theme: {
    extend: {
      colors: {
        secondary: colors.teal
      }
    }
  }
}

Thanks for the suggestion. Do you mean, once I extend my theme like that, I can do this?

Exactly 🙂

Thanks. This feature request is to use such custom colors that are defined with rgb/hex values instead of tailwind colors.
import type { Config } from 'tailwindcss'
import colors from 'tailwindcss/colors'

export default {
  theme: {
    extend: {
      colors: {
        secondary: {
          50: '#ebecec',
          100: '#c2c7c9',
          200: '#99a3a7',
          300: '#738187',
          400: '#4e6067',
          500: '#2a424a',
          600: '#04252e',
          700: '#05222a',
          800: '#051f26',
          900: '#051c23',
          950: '#04191f',
          DEFAULT: '#02161b',
        },
      },
    },
  },
}

This will also work, what is your issue here?

Ahh.... I get it.. I did this in my app.config.ts and it probably broke the above right behaviour.
// Shouldn't do if extending theme with custom palette like secondary, tertiary etc.
export default defineAppConfig({
  ui: {
    gray: 'neutral',
    secondary: 'secondary',
  },
})

Removed it and it works fine now.. thxs!

To be honest, I still don't understand how nuxt/ui theming is configured.
If I add my colors on tailwind.config.ts I can use them with text-*-600 for example, but then, if I say <UButton color="danger" /> it does not work and defaults to primary. But if I say UButton color="red" it works.
import twColors from 'tailwindcss/colors'

const colors = {
  'mountain-meadow': {
    '50': '#ebfef6',
    '100': '#d0fbe6',
    '200': '#a4f6d2',
    '300': '#6aebbc',
    '400': '#2ed9a0',
    '500': '#0abd87',
    '600': '#009b70',
    '700': '#007c5d',
    '800': '#03624a',
    '900': '#03513e',
    '950': '#012d24',
    DEFAULT: '#0abd87',
  },
  emerald: {
    '50': '#ebfef4',
    '100': '#d0fbe3',
    '200': '#a4f6cc',
    '300': '#6aebb1',
    '400': '#2ed992',
    '500': '#0abd78',
    '600': '#009b63',
    '700': '#007c52',
    '800': '#036242',
    '900': '#035138',
    '950': '#012d21',
    DEFAULT: '#009b63',
  },
} as const

const semanticColors = {
  primary: colors['mountain-meadow'],
  accent: colors.emerald,
  danger: twColors.red,
  info: twColors.indigo,
  success: twColors.green,
  warn: twColors.yellow,
} as const

export { colors, semanticColors }

const config: Partial<DeepWritable<Config>> = {
  attributify: false,
  content: [
    './components/**/*.{vue,js}',
    './layouts/**/*.vue',
    './pages/**/*.vue',
    './partials/**/*.vue',
    './app.vue',
  ],
  darkMode: 'class',
  plugins: [
    require('@headlessui/tailwindcss')({ prefix: 'ui' }),
    require('@tailwindcss/aspect-ratio'),
    require('@tailwindcss/container-queries'),
    require('@tailwindcss/forms'),
    require('@tailwindcss/typography'),
  ],
  shortcuts: {
    'dark-img': 'hidden dark:block',
    'light-img': 'block dark:hidden',
  },
  theme: {
    container,
    fontFamily,
    screens,
    extend: {
      colors: {
        ...colors,
        ...semanticColors,
      },
      maxWidth: {
        '8xl': '90rem',
      },
    },
  },
}

So it seems that Tailwind recognizes the extension, as expected, but nuxt/ui does not.
Side note: At the same time, nuxt/ui does not allow us to manually define a primary on tailwind.config.ts due to clashes, according to the docs. Shouldn't nuxt/ui actually expect us to defien a primary, secondary and whatever we want and then use the variables defined within our tailwind.config.ts to do the theming? Instead of asking us to put stuff in tailwind.config.ts but also in app.config.ts?
If I put in both, I have code duplication. If I put in tailwind.config.ts, nuxt/ui does not recognize the colors (even with safelisting), if I add them to app.config.ts only, tailwind does not recognize the color.

@FranciscoKloganB Not sure to understand what doesn't work for you. Would you mind opening a new issue with a StackBlitz reproduction so I can help you? From what I see, the <UButton color="danger" /> should work!


Add Property parameter to assertions to be able to chain them in useful way

To simply assert multiple properties of object I would like to be able to this:

See #19


TestTaskRunTimeout is flakey : fix timeouts

Expected Behavior
TestTaskRunTimeout should pass all the time :100:
Actual Behavior
TestTaskRunTimeout is failing a lot. I'll skip it so that
Steps to Reproduce the Problem

Open a PR
See it fail :hankey:
Run /retest and :pray:

I'm working on it in https://github.com/tektoncd/pipeline/pull/691 but it's at a point where it makes a lot of build fail, and I'm really thinking of marking the test as skip and track the re-enablement of it here
cc @bobcatfish
/kind bug
/assign

@vdemeester just let me know if you want a break from the investigation for a while, I'd be happy to take it on for a while :dancer:

/assign @bobcatfish
The more the merrier (especially as I'm running out of time this week)

tl;dr: The timeout handle and the reconciler disagree about when a Run has timed out.
Okay I have a theory about what's going on. In https://github.com/tektoncd/pipeline/pull/771 I ran the timeout test 10 times (and p.s. this seems to reproduce the issue locally as well), and in https://tekton-releases.appspot.com/build/tekton-prow/pr-logs/pull/tektoncd_pipeline/771/pull-tekton-pipeline-integration-tests/1118981945099816962/ we have 2/10 of those timing out.
I also added a lot of (terrible) logs so I could see what was going on and I noticed some interesting things:

The failed Runs were considered timed out in the timeout handler (i.e. case <-time.After(timeout) occurred) for all 10 Runs (and one of them twice haha)
But for run-giraffe-0 and run-giraffe-6, the subsequent reconcile did NOT consider them timed out

For example for run-giraffe-0, this is from the timeout handler:
I0418 21:17:16.127] {"level":"info","logger":"controller.taskrun-controller","caller":"reconciler/timeout_handler.go:203","msg":"TIMEOUT timed out for run-giraffe-0","knative.dev/controller":"taskrun-controller"}
I0418 21:17:16.127] {"level":"info","logger":"controller.taskrun-controller","caller":"reconciler/timeout_handler.go:200","msg":"TIMEOUT finished for run-giraffe-0","knative.dev/controller":"taskrun-controller"}

This would then trigger the reconcile loop, and the logs from that look like:
I0418 21:17:16.128] {"level":"info","logger":"controller.taskrun-controller","caller":"taskrun/taskrun.go:248","msg":"TIMEOUT about to check if timeout for run-giraffe-0","knative.dev/controller":"taskrun-controller"}
I0418 21:17:16.128] {"level":"info","logger":"controller.taskrun-controller","caller":"taskrun/taskrun.go:531","msg":"TIMEOUT checking timeout for run-giraffe-0, starttime 2019-04-18 21:07:13 +0000 UTC","knative.dev/controller":"taskrun-controller"}
I0418 21:17:16.128] {"level":"info","logger":"controller.taskrun-controller","caller":"taskrun/taskrun.go:537","msg":"TIMEOUT check if nil run-giraffe-0","knative.dev/controller":"taskrun-controller"}
I0418 21:17:16.128] {"level":"info","logger":"controller.taskrun-controller","caller":"taskrun/taskrun.go:541","msg":"TIMEOUT not nil run-giraffe-0 timeout 30s runtime 29.98450907s","knative.dev/controller":"taskrun-controller"}

As far as the reconcile loop is concerned, the timeout is 30s but the runtime is 29.98450907s - right on the edge!
And the same thing for run-giraffe-6:
I0418 21:17:16.033] {"level":"info","logger":"controller.taskrun-controller","caller":"taskrun/taskrun.go:541","msg":"TIMEOUT not nil run-giraffe-6 timeout 30s runtime 29.843298838s","knative.dev/controller":"taskrun-controller"}

So ∴ the theory is that the timeout handler and the reconcile loop are slightly out of sync in their logic to determine when a timeout has occurred, i.e. <-time.After(timeout) is happening, but the reonciler's logic is slightly different.
Anyway I don't know what the fix is haha - at a glance they SEEM to be doing the same thing. But the key imo is in making sure that exactly the same logic is being applied in both places OR in moving the logic to handle the timeout case into the timeout handler itself (instead of triggering an entire reconcile).
Whew! (maybe :crossed_fingers::crossed_fingers: :crossed_fingers: )

We need to:

[ ] Figure out a fix for this (haha)
[ ] Add more logs to track how timeouts are being applied (for any future debugging)

Can see similar stuff happening in https://storage.googleapis.com/tekton-prow/pr-logs/pull/tektoncd_pipeline/771/pull-tekton-pipeline-integration-tests/1118995157169999873/build-log.txt as well (with "improved" logs):
I0418 22:09:34.153] {"level":"info","logger":"controller.taskrun-controller","caller":"controller/controller.go:288","msg":"Reconcile succeeded. Time taken: 47.002Âµs.","knative.dev/controller":"taskrun-controller","knative.dev/key":"arendelle-vs2m2/run-giraffe-0"}
I0418 22:09:34.153] {"level":"info","logger":"controller.taskrun-controller","caller":"reconciler/timeout_handler.go:203","msg":"TIMEOUT HANDLER timed out for run-giraffe-5","knative.dev/controller":"taskrun-controller"}
I0418 22:09:34.153] {"level":"info","logger":"controller.taskrun-controller","caller":"reconciler/timeout_handler.go:200","msg":"TIMEOUT HANDLER finished for run-giraffe-5","knative.dev/controller":"taskrun-controller"}
I0418 22:09:34.154] {"level":"info","logger":"controller.taskrun-controller","caller":"taskrun/taskrun.go:248","msg":"TIMEOUT RECONCILER about to check if timeout for run-giraffe-5","knative.dev/controller":"taskrun-controller"}
I0418 22:09:34.154] {"level":"info","logger":"controller.taskrun-controller","caller":"taskrun/taskrun.go:531","msg":"TIMEOUT RECONCILER checking timeout for run-giraffe-5, starttime 2019-04-18 21:59:31 +0000 UTC","knative.dev/controller":"taskrun-controller"}
I0418 22:09:34.154] {"level":"info","logger":"controller.taskrun-controller","caller":"taskrun/taskrun.go:537","msg":"TIMEOUT RECONCILER check if nil run-giraffe-5","knative.dev/controller":"taskrun-controller"}
I0418 22:09:34.154] {"level":"info","logger":"controller.taskrun-controller","caller":"taskrun/taskrun.go:541","msg":"TIMEOUT RECONCILER not nil run-giraffe-5 timeout 30s runtime 29.943341324s","knative.dev/controller":"taskrun-controller"}

timeout 30s runtime 29.943341324s DUN DUN DUUUUN


Fills up lambda disk space

First of, this repo worked fanatastic, thank you! I ran it overnight on a function that runs on a cron trigger and had some disk space issues on the function. I disabled the logging in webdriver but doesn’t seem to have been the culprit. Curious if you’ve seen this issue before I dive in deeper, thanks!

I used it in several scenarios and never found a disk space issue. The layer uses the /tmp folder in the AWS lambda and it's limited by design to 512MB. Maybe this is the culprit of the problem.
I have to update the layer libs to a newer version of Chrome. I will add the option to use a configurable temporary folder (in a shared folder, for example). Not sure if it may help, but it sounds like an option. And of course, please feel free to send a PR if you want; I will review and accept it!


Pom profile changes

Switch tensorflow-core-platform so it's only generated when the deploying profile is set. Lets the profiles default through so the platform has the correct values when pulled in without any profiles activated.

As I pointed out before, this isn't going to work with other tools than Maven...

It worked when we tested it with gradle earlier today as it defaults in the pom and platform pulls in all the dependencies.

As I pointed out before, this isn't going to work with other tools than Maven...

The new strategy here @saudet is that we'll build tensorflow-core-platform only when releasing TFJava as part of the CI/CD. The local build will skip it, hence removing the need of playing with platform properties like we were doing originally.

Yeah, sure, feel free to experiment, but do test with other tools is all I can recommend. In the end, I think you'll find the the way I'm doing it in the presets is the least worst


GEODE-2825: Lucene query waits for defined index to be created

If an index is in a defined state but not yet created, the
query will now wait until the index is created or no longer
defined.  Instead of throwing an exception and possibly
getting a stack overflow

Review request list:
@upthewaterspout @nabarunnag @boglesby @ladyVader @gesterzhou

Sure, I can re-add the one second sleep.


[WebToolsE2E][Aspire] When running DatabaseMigrations solution, the migration project shows no longer running in dashboard.

INSTALL STEPS

Clean machine: Win11 x64 23h2 ENU
Install VS 17.12 Preview 4
Install Docker Desktop
Install azd [1.10.3]

REPRO STEPS

Clone aspire-samples repo and open DatabaseMigrations.sln
F5 and go to dashboard

ACTUAL
The migration project shows no longer running

NOTE

This issue also repro on aspire-samples 9.0.0

EXPECTED
Migrations are running like any other project

The migrations project in that sample is a console app that runs to completion so this is expected. It runs migrations and then exits.


Run hooks in parallel

Add support for running hooks in parallel.
This has been on the todo list for a while, and while I'm not satisfied with this implementation I think it helps push us forward.
I went with bare threads instead of Celluloid because Celluloid supports only Ruby 2.0+ and I think it was pretty heavyweight for what we're trying to do here.
The biggest challenge here is dealing with interrupts, which is why you see rescue Interrupt in a number of places. Since signals aren't yet supported for multi-threaded applications (see the documentation for Thread.handle_interrupt), we need to handle a lot of this explicitly.
The output has changed slightly: since we don't run hooks serially, we have to wait until they finish before displaying anything (or else their output is interleaved). I looked into ways of updating the output after the fact (by manipulating the terminal cursor) but it became clear that would be quite the undertaking, so I considered it out of scope.
I need to do some more testing, but putting this up for now so others can have a look and comment.

Pushed some additional commits along with some documentation. If you get a chance to read the documentation let me know if it makes sense.
Hitting a raise condition in JRuby but I'm pretty sure I know the reason, and I'm glad it was caught in tests because it'll force me to implement things more rigorously.

The concurrency global setting doesn't seem to be explicitly documented outside of default.yml, though it is described in the documentation of the processors hook setting. Was that intentional?

Regarding the global concurrency option: I've added additional documentation in the README.
The tests are now all passing on Linux. Still waiting on the Appveyor builds but the first few have passed.
Once all those tests have run I'll be merging this and cutting a prerelease gem so people can try it out and report issues.

Looks like the Appveyor builds passed as well :+1:

\o/


Cannot run fend from git bash on Windows

$ cargo run
     Running `target\debug\fend.exe`
I/O error: operation failed to complete synchronously
error: process didn't exit successfully: `target\debug\fend.exe` (exit code: 0xc0000409, STATUS_STACK_BUFFER_OVERRUN)

Tried to reproduce on v1.2.2, it's now broken in a different way: the prompt >  doesn't render at all, and Ctrl-C/Ctrl-D doesn't seem to work as expected. At least it no longer seems to crash.


v7.0.1

Coverage remained the same at 100.0% when pulling bc9a7ff9d7027e4d325c05d018e19ad3b01888f6 on v7.0.1 into 49df7037a439e34502a0a0cfde4a7ec53e1943d8 on master.

Coverage remained the same at 100.0% when pulling bc9a7ff9d7027e4d325c05d018e19ad3b01888f6 on v7.0.1 into 49df7037a439e34502a0a0cfde4a7ec53e1943d8 on master.


New Course selector: Archived courses not visible when filtering

The dropdown introduced in #541 doesn't display archived courses, which makes sense in every case except when filtering by course (on activations.do). I can see the use case for being able to see archived courses in that situation

Seems reasonable

Looks good now


[DSFI] Introduce the DSFI (Scalar Function Implementations) to AFF

Rendered grid UI with some faked JSON data


error in cluster_sorting

Hi,
With a new nanopore dataset i have been running with an error in the cluster_sorting step. Attached is the log for reference.
2024-10-31T111904.682386.snakemake.log
I have checked the input files. The L2COA212S_A_cdhit.fasta  L2COA212S_A_cdhit.fasta.clstr. I traced back and the consensus.fasta looks fine. The one that is empty is the rep_consensus.fasta. Everything else including the original fastq.gz files look normal and with a decent number of reads.
Do you have any clue where the problem might be coming from?
Thank you very much in advance

Hi @dmgr90,
I looked into your issue, and it seems that the empty rep_consensus.fasta file is likely the cause. This file is important because it indirectly affects the cluster_sorting rule, as it contributes to the data used in the clustering process.
You might try adjusting some parameters in the configuration file, especially the quality, length, and clustering filters. These settings control which reads are included in the consensus generation and could help ensure rep_consensus.fasta is populated.
Also, check the previous pipeline steps and logs to confirm they ran without errors. After adjusting the parameters, try re-running the affected steps.
To make sure the pipeline is working correctly overall, you might also consider running it with test data to verify that everything functions as expected.
Hope this helps! Let me know if you have any other questions.
Best regards,
dustin


Add new applications

Adds new applications:
Angry IP Scanner - Network scanner
Calibre - E-book manager
HeidiSQL - Database management tool
Linphone - VoIP software and SIP client
Nextcloud Desktop - Official Nextcloud client application
OpenRGB - RGB lighting control, independent from manufacturer software
ownCloud Desktop - Official ownCloud client application

can you add Floorp Browser while ur at it
its really good and open source
https://github.com/Floorp-Projects/Floorp


CORE-11971 Convert Façade Kotlin APIs to Java

To be re-targeted to grammar branch later, after rudiments will become inactive.

General comment, don't forget to use the @NotNull and @Nullable annotations where appropriate.


CRA WebServer isn't rendering getUserMedia API for iOS Safari properly.

I'm trying to access the camera on iOS through implementing the getUserMedia API in my React App. My code works fine when I navigate run it in StackBlitz on iOS Safari. However when I navigate to mylocalIP:3000 on iOS Safari, the component doesn't prompt for access to the webcam. See my component here or below:
const Video: React.FC = () => {
useEffect(() => {
start()
})
const constraints = {
    audio: false,
    video: true
};

async function start() {
    const stream = await navigator.mediaDevices.getUserMedia(constraints)
    const video = document.getElementById('video') as HTMLVideoElement;
    video.setAttribute("playsinline", "true");
    video.setAttribute("controls", "true");
    video.srcObject = stream;
}

return (
    <video id='video' autoPlay playsInline></video>
)

}
export default Video

getUserMedia is only defined in a secure context, among other constraints. Is your development server using https?

@heyimalex, @CG-Stack kindly replicated my original issue #8229 verbatim albeit without the formatting...
The steps to fix this are simple- in package.json change your start script to be:
"start": "HTTPS=true react-scripts start"


Inject optional CatalogApi in auth-backend createRouter function.

Hey, I just made a Pull Request!
Adds on optional CatalogApi parameter to createRouter function in auth-backend plugin to let users inject their CatalogApi if needed.
Closes #14186
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] All your commits have a Signed-off-by line in the message. (more info)

Ah alright, one more thing that I forgot to mention. See the top and bottom of the CI output:
*************************************************************************************
* You have uncommitted changes to the public API of a package.                      *
* To solve this, run `yarn build:api-reports` and commit all api-report.md changes. *
*************************************************************************************

This is how we keep track of and can inspect the public surfaces of everything that we build. Thanks for your patience.

Oh no. You are unlucky today :( We accidentally snuck in an error in a markdown file that led to linting problems. Rebasing on latest fresh master will have that file fixed


Minor typo fixes

Er, give me a few more minutes. Deadline is???

Sorry, I thought you were done so I just fixed a conflict in your branch (very minor).

I'd ideally like to submit before 9 am because I have back-to-back stuff beginning at 9 am and I promised it "first thing" today. :)

Finished! Pull when you'd like.


feat(cms-base): remove internal imports

Description
closes: #354
tl;dr: this change allows to overwrite also the internal components with a name starting with Sw from cms-base package.
Type of change

ToDo's

Screenshots (if applicable)

Additional context

@kstala please have a look if the change does not break the way how cms elements and blocks are rendered 🙌🏼

I checked all cms elements and apart from the well-known issue with  CmsBlockImageGalleryBig, everything looks correct

Will we close that without merging?

Will we close that without merging?

it is still being reviewed, then the PR will close #354 and unblock a customer's project.


Chain is stuck on block 1880422

Running NEO 2.7.1
This explorer is also stuck on the same block:
http://antcha.in

Duplicate of #100


Explicitly specify container version for all CI jobs

Description
Since the master branch currently depends on littlefs being compiled without LFS_THREADSAFE, the lastest Docker containers (which are compiled with that macro) can't be used. Also, this PR should check if the tests now run correctly.
Fixes #281

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.89%. Comparing base (8d3f1a8) to head (e30c052).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #285      +/-   ##
==========================================
+ Coverage   96.54%   96.89%   +0.35%
==========================================
  Files          14       16       +2
  Lines         434      483      +49
==========================================
+ Hits          419      468      +49
  Misses         15       15

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update Pages “BestTermiteInspectionPage”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
83ed3bba5ce9cb467c299ca5d94270a863bed56f

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/655aa5e712f0d10008512d45


add restore backup instructions

Hi
I'm sure I can figure them out but I think would be beneficial to add restore steps in the readme
so people can jump in this git project do an quick test BACKUP/Restore and give it a thumbs up
Cheers 👍

Great idea, let's add that to the README. 👍
If anyone manages a PR, I'll be more than happy to review & merge.

is there an example of this in the meantime?

There is now a detailed example of restoring from S3.


⚡ Login

Feature: allow the user to login to CMD-online

Wasn't @juliettegroot supposed to work on the login? 🤔 If you would rather do it, since you made the create account feature already etc, maybe she could work on "changing profile/account details"?

Yeah, I thought it made sense to do both as I already have experience with Argon2 and route authentication


Fix segfault with -O0-compiled kernels

kernels_[blitType] yields a null function pointer in KernelBlitManager::initHeap due to KernelBlitManager::createProgram not initializing all the kernels because of broken layout invariants.
KernBlitManager::NumBlitKernels has two possible return values: BlitTotal, and BlitLinearTotal which are sentinels. These sentinels are used in KernelBlitManager::createProgram to initialize the kernels_ array. It correctly initializes [0, NumBlitKernels()), but InitHeap is > NumBlitKernels, so the InitHeap kernel is not loaded.
Thus, when images are disabled, and a kernel has an hidden_heap_v1 entry, the InitHeap blitkernel is not loaded, and kernelBlitManager::initHeap subsequently gets a null pointer. The kernel always has such hidden_heap_v1 descriptor entry at -O0, but I believe it's possible (unconfirmed) for this situation to occur in other circumstances.
The fix is simply to ensure the InitHeap enumeration has a numeric value less than BlitLinearTotal.
n.b. This bug does not exist on the 5.7 release, but it looks like that's by chance.

cc'ing: @gandryey @chrispaquot @saleelk

Hi @ldrumm, can you check if this is still an issue in the latest ROCm release?


run the Discord bot in Modal

at first, I thought this was not possible because bots needed to be "always on" to respond to events, which didn't jive with Modal's serverless style, so I put it on EC2, but Modal's web endpoints seem to serve as webhooks, which should work
this would simplify the automation a lot!
but even if we can do everything on Modal, it's nice to have trad cloud infrastructure in the repo, even if only as an option among others.

Yes, we can -- if we're willing to switch to a pure webhook style, which means no emoji reacts.


⚠ ClamAV has degraded performance

In 42ae905, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 75 ms

Resolved: ClamAV performance has improved in 8fb26d1.


Sterilizer for FASTQ files

This tool takes an existing FASTQ file and generates a similarly structured one, but with all identifiable information removed. Most alphanumeric characters are replaced with random substitutes, limited to what is necessary to keep the output from this sufficient to satisfy future input validators.

Just as a future comment, one particular issue for our purposes is that the munged output is inter-file inconsistent (meaning which groups-of-4 are selected for output). The next PR I'm going to do is provide a seed value option to address this.


aesv8-armx.pl: Avoid buffer overrread in AES-XTS decryption

Original author: Nevine Ebeid (Amazon)
Fixes: CVE-2023-1255
The buffer overread happens on decrypts of 4 mod 5 sizes. Unless the memory just after the buffer is unmapped this is harmless.

This pull request is ready to merge


Craftlog loses free float when prof window scaled down

If you use moveAny addon and scale window down e.g. to 85% then crat log loses its option to free float. it sticks to some random coords when trying to move, scaling back prof window to scale 100% fixes the problem.

probably an issue with moveAny but can take a look anyway

I have downgrade CraftSim to version 19.6.0 it still work well with MoveAny addon.
this problem appear when using afterward version.


Import filename typos on FSMultipartUpload.h

FSUploadoptions.h is with small case 'o' instead of capital 'O'.

^^ and this throws a warning in XCode 8.3
would be nice to have this fixed sometime
thanks and keep up the good work!

@jackyheart, @schmidan Thanks for reporting this issue! It has been addressed in 0.3.8 👍


How to support WKWebView in iOS8?

How to support WKWebView in iOS8?

I met the same problem about how to support WebP in WKWebView, since NSURLProtocol was not work in WKWebView.


Custom properties for collections

Hi,
I'm in the process of integrating an external api service as a collection of collections, with the magic of Fluidity for interface stuff. I need to add some custom properties to the collection, so it knows the parameters of the request it needs to send (in this case an id of a "channel").
I've managed to hack a working solution together, where i use the alias of the collection to send the id to the custom repository.
How would you go about this? I'd be happy to contribute some code, so far I've just made the collection available to the repository, so I could access the alias, but I was thinking along the lines of adding a dictionary (string key, object value), that you can search through from the repository.

If I'm understanding you correctly, you want to link an entity from one collection to another entity in another collection?
If so, Fluidity does come with an Entity Picker property editor so you could configure a property editor to pick your parent entity. If this is a 1 to 1 relation it should be pretty straight forward. If it's many to many, you'd have to implement a custom repository to handle saving the value to a match table.
The thing with Fluidity, whenever you need something custom like this is to think in Property Editors. Because Fluidity uses property editors for the editing values, if one doesn't exist, you should be able to create a custom one the same way you would for umbraco and use it within Fluidity.
So, if my thought is correct, you may be able to use the Entity Picker, but if not, you could either create your own property editor to solve this, or maybe look and see what other property editors might help. For example, the nuPicker package allows you to create SQL data sources for a variety of different types of pickers, so you could use that and configure it with SQL statements accessing your DB tables.
Hope this helps
Matt

I'm trying to implement a custom repository, that doesn't use SQL, but another C#-package which interfaces with an external api we're using for podcasts.
So what I've done is implemented a custom repository, that just uses the external API in the different methods. But that repository needs a channel id, to request episodes from that specific channel.

All I've done to get that working is changing the _alias property in FluidityTreeItemConfig.cs to a public property, and then I'm always sending the collection into the Repository, instead of only the defaultrepository.

Ok, first off, you shouldn't need to make _alias public, there should be a SetAlias method you can call.
But I think I get what you are doing now, although I hadn't thought of using fluidity like this. It isn't intended that collections would be created dynamically. But it's interesting it kinda works :)
In theory this should be ok as long as the list of podcasts doesn't change, as the config is only generated at app startup, so new podcasts wouldn't get picked up till the next app recycle.
Fluidity doesn't yet have the concept of collections in collections (there is actually an open issue about this, see #41 and feel free to add your feedback). The suggested approach for now would be to manage the podcasts and episodes seperately and then maybe use a data views builder to created filtered lists of episodes (probably not ideal, but it's what we go at the moment).
Then, because podcasts are also now a collection, from within the episode editor you can create a fluidity entity picker property editor to pick a parent podcast and this gives you your parent linkage.

Discussed this issue outside of GitHub and found an adequate workaround. The ultimate problem is that of child collection which we already have an issue for #41 so I'm closing this issue down.


Summary next to To be Budgeted is misaligned

Name of Defective Feature
Summary of funds next to To be Budgeted
System Settings (Required)
ToolKit Version: 1.1.0
Operating System (Mac/Windows/Etc): Windows 7 Enterprise
Browser (Chrome/Firefox/Safari): Chrome
Browser Version: 9.0.3071.115 (Official Build) (64-bit)
Other Extensions: Adblock
Export of Toolkit Settings (Bottom Left of Settings Page)
[{"key":"AccountsDisplayDensity","value":"1"},{"key":"AccountsEmphasizedOutflows","value":false},{"key":"AccountsStripedRows","value":true},{"key":"AdditionalColumns","value":true},{"key":"AdjustableColumnWidths","value":true},{"key":"AutoCloseReconcile","value":false},{"key":"ChangeEnterBehavior","value":true},{"key":"CheckNumbers","value":false},{"key":"ColourBlindMode","value":false},{"key":"CustomFlagNames","value":false},{"key":"DisplayTargetGoalAmount","value":"0"},{"key":"GoalWarningColor","value":false},{"key":"HideAgeOfMoney","value":false},{"key":"HideReferralBanner","value":false},{"key":"PrintingImprovements","value":true},{"key":"RemovePositiveHighlight","value":false},{"key":"RowHeight","value":"1"},{"key":"RunningBalance","value":true},{"key":"ShowCategoryBalance","value":false},{"key":"SquareNegativeMode","value":false},{"key":"StealingFromFuture","value":true},{"key":"TargetBalanceWarning","value":false},{"key":"accountTransactionSearch","value":true},{"key":"accountsClearSelection","value":false},{"key":"accountsDisplayDensity","value":"1"},{"key":"accountsEmphasizedOutflows","value":true},{"key":"accountsRowHeight","value":"1"},{"key":"accountsSelectedTotal","value":true},{"key":"accountsStripedRows","value":true},{"key":"activityTransactionLink","value":true},{"key":"betterScrollbars","value":"0"},{"key":"budgetBalanceToZero","value":true},{"key":"budgetProgressBars","value":"0"},{"key":"budgetQuickSwitch","value":false},{"key":"budgetRowsHeight","value":"1"},{"key":"calendarFirstDay","value":"0"},{"key":"categoryActivityPopupWidth","value":"1"},{"key":"changeEnterBehavior","value":true},{"key":"checkCreditBalances","value":true},{"key":"checkNumbers","value":false},{"key":"closeReconcileWindow","value":false},{"key":"collapseExpandBudgetGroups","value":false},{"key":"collapseSideMenu","value":false},{"key":"colourBlindMode","value":false},{"key":"compactIncomeVsExpense","value":true},{"key":"currentMonthIndicator","value":true},{"key":"daysOfBuffering","value":true},{"key":"daysOfBufferingHistoryLookup","value":"6"},{"key":"editButtonPosition","value":"2"},{"key":"enableRetroCalculator","value":true},{"key":"enlargeCategoriesDropdown","value":true},{"key":"enterInRegisterNow","value":true},{"key":"exportTransactions","value":false},{"key":"goalIndicator","value":false},{"key":"goalIndicatorWarningColor","value":false},{"key":"googleFontsSelector","value":"0"},{"key":"hideAOM","value":false},{"key":"hideAccountBalancesType","value":"0"},{"key":"hideHelp","value":false},{"key":"highlightNegativesNegative","value":true},{"key":"importNotification","value":"1"},{"key":"incomeFromLastMonth","value":"0"},{"key":"l10n","value":"0"},{"key":"largerClickableIcons","value":true},{"key":"monthlyNotesPopupWidth","value":"0"},{"key":"navDisplayDensity","value":"0"},{"key":"pacing","value":"0"},{"key":"popupCalculator","value":false},{"key":"printingImprovements","value":true},{"key":"privacyMode","value":"0"},{"key":"reconciledTextColor","value":"0"},{"key":"removePositiveHighlight","value":false},{"key":"removeZeroCategories","value":true},{"key":"reports","value":true},{"key":"resizeInspector","value":false},{"key":"rightClickToEdit","value":true},{"key":"runningBalance","value":true},{"key":"seamlessBudgetHeader","value":false},{"key":"showIntercom","value":true},{"key":"spareChange","value":false},{"key":"splitKeyboardShortcut","value":false},{"key":"splitTransactionAutoAdjust","value":true},{"key":"squareNegativeMode","value":false},{"key":"stealingFromNextMonth","value":true},{"key":"swapClearedFlagged","value":false},{"key":"toBeBudgetedWarning","value":false},{"key":"toggleSplits","value":false},{"key":"toggleTransactionFilters","value":"1"},{"key":"warnOnQuickBudget","value":true}]

Steps to Reproduce the Behavior
Please write down the steps necessary to reproduce the behavior you're seeing. Please be as thorough as possible.

Look at top of July budget screen.

Expected Behavior
Summary of Funds from Jul, Overspent in Jun, Budgeted in Jul, Budgeted in Future is aligned with the numerical values corresponding to each line.
Actual Behavior
The numbers are not aligned with the words. I've tried turning the Stealing From Future Alert on and off and it doesn't seem to make a difference. Minor issue but still an eyesore.

Any chance this applies to you? From the latest 7 August 2017 notes;
If you use the Grammarly extension with Chrome, your going to love this one: you’re budget totals should be back in alignment.

Yes it does - and it is indeed now back in alignment. Cheers!

Cool. Maybe they should have used Grammarly to catch their improper use of "your" and "you're". :-)

Oh snap!! "you're budget" is especially painful to read.


Spaces in directory name

I couldn't get the script to accept spaces in the directory name.

Thanks for reporting! You should be fine with the latest version.
Can you close this issue if all good?

I checked it myself, all good. Closing the issue. Thanks!


Second attempt for #844

This time should be better
vagrant environment needed a specific config entry for rabbitmq class in order to work properly.
Pull Request Checklist

Description

Related Issue

Fixes # .
Motivation and Context

How Has This Been Tested?

General

[ ] Update README.md with any necessary configuration snippets

[ ] New parameters are documented

[ ] New parameters have tests

[ ] Tests pass - bundle exec rake validate lint spec

Thanks!
Released in v2.41.0


Added handling for cases where the remoteUrl is more complex

for example we're using artifactory https://x.y.z/api/npm/npm-repo
because we're hosting jfrog's artifactory behind nginx

Not ready yet - sorry

Ok - ready now :-)

@CL0Sey, @mixu Could someone take a look at this please?

Is this still needed? Sorry for the delay.
I can probably have a look at testing this potentially.

We have made a private branch and are using that via our own npm repo, and
it's necessary if you want to d/l from your own repo (with nested paths).
On 5 December 2016 at 10:54, Chris Close<EMAIL_ADDRESS>wrote:

Is this still needed? Sorry for the delay.
I can probably have a look at testing this potentially.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/mixu/npm_lazy/pull/66#issuecomment-264824107, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFBK4-Rhc84P-ynpwMvYI0OGhYeBMFjeks5rE-1ogaJpZM4I1cTV
.

nvm - we're using yarn now :-)


Heroku DATABASE_URL

We cannot use pg native functionality. We've tried changing this in the module but aren't having any luck. We're able to get the DATABASE_URL read and show this through console.log in pg db-migrate module, but SSL is always undefined and off. None of the other answers here are of use, all look to lead to commits that are changed or code that has been modified and as such are deprecated :( Please help!
database.json: have tried many iterations of this
{   "default": "pg",   "pg": {      "ENV": "DATABASE_URL",     "addIfNotExists": {       "ssl": true     },     "overwrite": {       "native": true     }   } }
error:
[ERROR] error: no pg_hba.conf entry for host "<IP_ADDRESS>", user "user", database "database", SSL off     at Connection.parseE (/Users/mmowris/Workspace/csa-portal/node_modules/db-migrate-pg/node_modules/pg/lib/connection.js:539:11)     at Connection.parseMessage (/Users/mmowris/Workspace/csa-portal/node_modules/db-migrate-pg/node_modules/pg/lib/connection.js:366:17)     at Socket.<anonymous> (/Users/mmowris/Workspace/csa-portal/node_modules/db-migrate-pg/node_modules/pg/lib/connection.js:105:22)     at emitOne (events.js:115:13)     at Socket.emit (events.js:210:7)     at addChunk (_stream_readable.js:252:12)     at readableAddChunk (_stream_readable.js:239:11)     at Socket.Readable.push (_stream_readable.js:197:10)     at TCP.onread (net.js:589:20) mmowris-ltm2:csa-portal mmowris$ sub

Ahh I guess you're coming from https://github.com/db-migrate/node-db-migrate/issues/349, yes this is not there yet.
Let me have a look, maybe I can just quickly push this out.

Ok, you're lucky wasn't that a big deal.
Release under

<EMAIL_ADDRESS>

@wzrdtales thank you so much for the quick response! very helpful module!

This does not seems to work even with the current version. The content of overwrite and addIfNotExists does not seem to be returned by loadObject fn here https://github.com/db-migrate/node-db-migrate/blob/ac850c33b87e54a71cc306d6fdbcb16740b29354/lib/config.js#L154

Inorder to overwrite or modify DATABASE_URL then you need to write it like this
{
    "default": "pg",
    "pg": {
        "url":{"ENV": "DATABASE_URL"},
        "addIfNotExists": {
            "ssl": true
        },
        "overwrite": {
            "native": true
        }
    }
}


[ISSUE-github 3570]Make TSEncoding.getTsEncoding(byte encoding) from O(n) to O(1);

Description
Make TSEncoding.getTsEncoding(byte encoding) from O(n) to O(1);
Content1 ...
Make enum filter fast by HashMap.
Content2 ...
Content3 ...

This PR has:

[ ] been self-reviewed.

[ ] concurrent read
[ ] concurrent write
[ ] concurrent read and write

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods.
[ ] added or updated version, license, or notice information
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious
for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold
for code coverage.
[ ] added integration tests.
[ ] been tested in a test IoTDB cluster.

Key changed/added classes (or packages if there are too many classes) in this PR

Coverage decreased (-0.02%) to 68.158% when pulling 2e6689baf5cc721ed5dfadfaa19b7ce7f9dca8e8 on limeng32:yonyou_limengag into 5e020b5880a97c8530b39986b504c5f28fcbcd33 on apache:master.

I don't think the complexity of the implement you changed is O(n).
See https://blog.csdn.net/weixin_28750379/article/details/114595592
Can you please share your reason?

I learned. I found out in this scene switch is really faster by benchmark test.


Not fully compatible with PostCSS v8 API

Over in https://github.com/csstools/postcss-nesting/issues/70#issuecomment-819565539, I mentioned my issues with the then-in-progress v8 code hadn't yet completed all the steps in the PostCSS plugin migration guide. It looks like that step still hasn't been done; the postcss-8-nesting.js file is still using Once.
It looks like this project needs more updates to be fully PostCSS 8 compliant.
"Step 3: Take the most out of the new API" of the PostCSS plugin migration guide says:

PostCSS 8 does a single CSS tree scan. Multiple plugins can leverage the same scan for better performance.
To use the single scan you need to remove root.walk* calls and move the code to Declaration(), Rule(), AtRule() or Comment() methods in a plugin’s object:
module.exports = {
    postcssPlugin: 'postcss-dark-theme-class',
-   Once (root) {
-     root.walkAtRules(atRule => {
-       // Slow
-     })
-   }
+   AtRule (atRule) {
+     // Faster
+   }
  }
  module.exports.postcss = true

postcss-nesting still has:
const postcssNesting = () => {
	return {
		postcssPlugin: 'postcss-nesting',
		Once(root) {
			walk(root);
		}
	}
}

I'm experiencing a bug on v8.0.1 where CSS added from a previous plugin (like postcss-mixins) in the postcss config isn't processed by the later-defined postcss-nesting and I suspect that this might be the reason.
Steps to reproduce: Configure PostCSS with postcss-mixins before postcss-nesting. Then make a simple mixin with postcss-mixins that adds a nested rule. The postcss-nesting plugin will fail to convert the @nest into valid contemporary CSS.
@define-mixin hoverExample {
  @nest &:hover, &:focus {
    background: grey;
  }
}

.example {
  @mixin hoverExample;
}

Expected results:
.example:hover,
.example:focus {
  background: grey;
}

Actual results:
.example {
  @nest &:focus, &:focus {
    background: grey;
  }
}

+1. This breaks:
@define-mixin example {
  /* ... */

  &::-webkit-scrollbar  {
    height: 4px;
    width: 4px;
  }
}

(Workaround: use a class instead of a mixin)


Outlook Professional Plus 2016 new add-in issue

Hello,
I'm using Office Professional Plus 2016, on windows 10. Is there a way to add a new add-in via URL in Outlook?
I know this is possible in desktop app of the Outlook Office 365. But I need to do this in Office Professional Plus.
Thanks!
Malika

To add more info, I'm connected to Exchange 2010 server of our company network. Is that make any different how we access add-ins?

@mpasharpsl Thanks for asking about this. Exchange 2010 doesn't support add-ins unfortunately. You need to be on Exchange 2013 or later for add-ins support. Note that Exchange server support can also be more limited than what the desktop, web, or mobile client may otherwise support.
Supported clients
I'll go ahead and close this issue but if you have any further questions, let us know and we can reopen it.
Thanks.


Display a dialog box instead of showing message on text area, when client/server end the chat in private chat

added dialog box instead of showing message on text area.
FIXED


Adding pre-release license terms

Is there a reason for .doc instead of .md?

Currently talking to LCA about the file format/contents before I merge it

LGTM - squash then merge


Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
I was experimenting with a routing in scope of custom transitions.
I added a ModalBottomSheetRoute to the list of the routes.
One thing that I would like to improve is to have an underlying bunch of rotes to have some animation. Currently they just appear after slide transition.
Here is a video
https://github.com/flutter/flutter/assets/6721286/dcf5c977-c9db-4829-8d6c-efbeb9faef64
and code

import 'package:flutter/material.dart';
import 'package:go_router/go_router.dart';

void main() => runApp(App());

class DialogPage<T> extends Page<T> {
  final Widget child;

  const DialogPage({required this.child, super.key});

  @override
  Route<T> createRoute(BuildContext context) => ModalBottomSheetRoute<T>(
        settings: this,
        builder: (context) => Dialog(
          child: child,
        ),
        isScrollControlled: false,
      );
}

/// The main app.
class App extends StatelessWidget {
  /// Creates an [App].
  App({super.key});

  /// The title of the app.
  static const String title = 'GoRouter Example: Custom Transitions';

  @override
  Widget build(BuildContext context) => MaterialApp.router(
        routerConfig: _router,
        title: title,
      );

  final GoRouter _router = GoRouter(
    initialLocation: '/fade',
    routes: <GoRoute>[
      GoRoute(
        path: '/',
        builder: (context, state) => Scaffold(
          appBar: AppBar(),
          body: Column(
            children: [
              TextButton(
                child: Text('Open popup'),
                onPressed: () {
                  context.go('/slide');
                },
              ),
              TextButton(
                child: Text('Home'),
                onPressed: () {
                  context.go('/fade');
                },
              ),
            ],
          ),
        ),
        routes: [
          GoRoute(
            path: 'slide',
            pageBuilder: (BuildContext context, GoRouterState state) {
              return DialogPage(
                child: Align(
                    child: Container(
                        width: 200, height: 200, color: Colors.green)),
              );
            },
          ),
        ],
      ),
      GoRoute(
        path: '/fade',
        pageBuilder: (BuildContext context, GoRouterState state) =>
            CustomTransitionPage<void>(
          key: state.pageKey,
          child: const ExampleTransitionsScreen(
            kind: 'fade',
            color: Colors.red,
          ),
          transitionsBuilder: (BuildContext context,
                  Animation<double> animation,
                  Animation<double> secondaryAnimation,
                  Widget child) =>
              FadeTransition(opacity: animation, child: child),
        ),
      ),
    ],
  );
}

/// An Example transitions screen.
class ExampleTransitionsScreen extends StatelessWidget {
  /// Creates an [ExampleTransitionsScreen].
  const ExampleTransitionsScreen({
    required this.color,
    required this.kind,
    super.key,
  });

  /// The available transition kinds.
  static final List<String> kinds = <String>[
    'slide',
  ];

  /// The color of the container.
  final Color color;

  /// The transition kind.
  final String kind;

  @override
  Widget build(BuildContext context) => Scaffold(
        appBar: AppBar(title: Text('${App.title}: $kind')),
        body: Container(
          color: color,
          child: Center(
            child: Column(
              mainAxisAlignment: MainAxisAlignment.center,
              children: <Widget>[
                for (final String kind in kinds)
                  Padding(
                    padding: const EdgeInsets.all(8),
                    child: ElevatedButton(
                      onPressed: () => context.go('/$kind'),
                      child: Text('$kind transition'),
                    ),
                  ),
                Padding(
                  padding: const EdgeInsets.all(8),
                  child: ElevatedButton(
                    onPressed: () => context.go('/'),
                    child: Text('Root'),
                  ),
                )
              ],
            ),
          ),
        ),
      );
}

Proposal
A possibility to have some kind of transition for underlying routes or any other thing.
Is there an ability to animate underlying routes in the same way as the top most?

Can you please be more specific on the example respectively? Do you mean the animation of pages under the dialog?

Thanks for your explanation. I see the issue now. It seems the underlying route is ignoring transition animation. You can see my below example which has CustomTransitionPage for the underlying route but it doesn't work in this case when pushing to a PopupRoute (ModalBottomSheetRoute).

Sample code (CustomTransitionPage for underlying route doesn't work)

import 'package:flutter/material.dart';
import 'package:flutter/scheduler.dart';
import 'package:go_router/go_router.dart';

void main() {
	// timeDilation = 1.0;
	runApp(App());
}

class DialogPage<T> extends Page<T> {
  final Widget child;

  const DialogPage({required this.child, super.key});

  @override
  Route<T> createRoute(BuildContext context) => ModalBottomSheetRoute<T>(
        settings: this,
        builder: (context) => Dialog(
          child: child,
        ),
        isScrollControlled: false,
      );
}

/// The main app.
class App extends StatelessWidget {
  /// Creates an [App].
  App({super.key});

  /// The title of the app.
  static const String title = 'GoRouter Example: Custom Transitions';

  @override
  Widget build(BuildContext context) => MaterialApp.router(
        routerConfig: _router,
        title: title,
      );

  final GoRouter _router = GoRouter(
    initialLocation: '/fade',
    routes: <GoRoute>[
      GoRoute(
        path: '/',
        pageBuilder: (context, state) => FadeTransitionPage(child: const BeneathWidget(), key: state.pageKey,),
				// builder: (context, state) => const BeneathWidget(),
        routes: [
          GoRoute(
            path: 'slide',
            pageBuilder: (BuildContext context, GoRouterState state) {
              return DialogPage(
                child: Align(
                    child: Container(
                        width: 200, height: 200, color: Colors.green)),
              );
            },
          ),
        ],
      ),
      GoRoute(
        path: '/fade',
        pageBuilder: (BuildContext context, GoRouterState state) =>
            CustomTransitionPage<void>(
          key: state.pageKey,
          child: const ExampleTransitionsScreen(
            kind: 'fade',
            color: Colors.red,
          ),
          transitionsBuilder: (BuildContext context,
                  Animation<double> animation,
                  Animation<double> secondaryAnimation,
                  Widget child) =>
              FadeTransition(opacity: animation, child: child),
        ),
      ),
    ],
  );
}

class BeneathWidget extends StatefulWidget {
  const BeneathWidget({super.key});

  @override
  State<BeneathWidget> createState() => _BeneathWidgetState();
}

class _BeneathWidgetState extends State<BeneathWidget> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
          appBar: AppBar(),
          body: Column(
            children: [
              TextButton(
                child: Text('Open popup'),
                onPressed: () {
                  context.go('/slide');
                },
              ),
              TextButton(
                child: Text('Home'),
                onPressed: () {
                  context.go('/fade');
                },
              ),
            ],
          ),
        );
  }
}

/// A page that fades in an out.
class FadeTransitionPage extends CustomTransitionPage<void> {
  /// Creates a [FadeTransitionPage].
  FadeTransitionPage({
    required LocalKey key,
    required Widget child,
  }) : super(
      key: key,
      transitionsBuilder: (BuildContext context,
          Animation<double> animation,
          Animation<double> secondaryAnimation,
          Widget child) =>
          FadeTransition(
            opacity: animation.drive(_curveTween),
            child: child,
          ),
      child: child);

  static final CurveTween _curveTween = CurveTween(curve: Curves.easeIn);
}

/// An Example transitions screen.
class ExampleTransitionsScreen extends StatelessWidget {
  /// Creates an [ExampleTransitionsScreen].
  const ExampleTransitionsScreen({
    required this.color,
    required this.kind,
    super.key,
  });

  /// The available transition kinds.
  static final List<String> kinds = <String>[
    'slide',
  ];

  /// The color of the container.
  final Color color;

  /// The transition kind.
  final String kind;

  @override
  Widget build(BuildContext context) => Scaffold(
        appBar: AppBar(title: Text('${App.title}: $kind')),
        body: Container(
          color: color,
          child: Center(
            child: Column(
              mainAxisAlignment: MainAxisAlignment.center,
              children: <Widget>[
                for (final String kind in kinds)
                  Padding(
                    padding: const EdgeInsets.all(8),
                    child: ElevatedButton(
                      onPressed: () => context.go('/$kind'),
                      child: Text('$kind transition'),
                    ),
                  ),
                Padding(
                  padding: const EdgeInsets.all(8),
                  child: ElevatedButton(
                    onPressed: () => context.go('/'),
                    child: Text('Root'),
                  ),
                )
              ],
            ),
          ),
        ),
      );
}

it is possible but it may be hard to come up with a usable API. This will need to leverage the https://api.flutter.dev/flutter/widgets/Navigator/transitionDelegate.html
by default all non-top-most route are remove/add directly without animation, we can provide a delegate to push/pop the route underneath instead.

Hi @huycozy Sure. Let me give you more verbose explanation.
At the beginning of the video when I press on "slide transition" button the app routes me from /fade to /slide route.
The the pop up slides from the bottom as expected and then the underlying route(with "open popup" and "home" buttons) just appears behind.
When you have full screen routes this is not an issue - the routes beneath the topmost route does not need the animation.
In case if the topmost route does not fully cover the stack - underlying routes just appear from "nowhere".

You can use something like this for convert dialog to page

class ModalBottomSheetPage extends Page<void> {
final Widget child;

const ModalBottomSheetPage({
  required this.child,
  super.key,
});

@override
Route<void> createRoute(BuildContext context) => ModalBottomSheetRoute<void>(
      builder: (BuildContext context) => child,
      settings: this,
      isScrollControlled: false,
    );
}

In @huycozy 's case, if you go from '/slide' to '/fade'.
you should have the following in the transistion delegate
newPageRouteHistory = ['/fade']
locationToExitingPageRoute = {
null: '/',
'/': '/slide',
}
In this case, you can call markForPop for both '/' and '/slide' and markForPush for '/fade'

@chunhtai cool! I was having some hip cups with the delegate, but it works.
So I guess the current workaround for this issue would be passing transitionDelegate from GoRouter down to Navigator. Make a clone of DefaultTransitionDelegate and replace markForComplete with markForPush. You could check the passing-down work here
However, the issue #139471 , that I just filed, at first seemed to be related to this one, but it turned out different, as it couldn't be solved with the workaround solution above.

@ThangVuNguyenViet you mentioned you had a workaround, do you have any MRE of this? I'd like to integrate this in my project as well.


ARROW-7680: [C++] Fix dataset.factory(...) with Windows paths

cc @jorisvandenbossche @fsaintjacques

AppVeyor build: https://ci.appveyor.com/project/pitrou/arrow/builds/31429670

@pitrou this seems to have broken relative paths -> https://issues.apache.org/jira/browse/ARROW-8136


Vacant land

From Howard:

In the 2nd House, there is a district GA-01 and a district GA-1.  I believe that the GA-1 is simply vacant land that is part of no congressional district. I would label it more like GA-A and make the text. "This area was not part of any congressional district" to distinguish from vacant seats.  My experience with the older Voteview is that these "holes" recur on the frontier well into the nineteenth century.

Oddly, I can't seem to get any results in Georgia before the 15th Congress. It primarily had an at large district before then, but it did have a fixed district in the 1st and 2nd. But Howard's command suggests to me that we do have the data. Can someone give me a fixed address to check to replicate this?

That GA-1 is I believe a "-1"!  I think that were at this time some indian lands that were not considered (at least by by Martis) to be part of any CD.  I think GA also had some oddly named districts (like "Militia District One").  I will take a look to see what going on.   We might want @aaronrudkin to put in some logic for "-1" districts ("Indian land" or some such).   I will take a look at how often this happens.

I did some digging:  It turns that there are -1 areas in a number of states.  Here is a complete list:
AL-1    18
AL-1    19
AL-1    20
AL-1    21
AL-1    22
GA-1     1
GA-1     2
GA-1     3
GA-1     4
GA-1     5
GA-1     6
GA-1     7
GA-1     8
GA-1     9
GA-1    10
GA-1    11
GA-1    12
GA-1    13
GA-1    14
GA-1    15
GA-1    16
GA-1    17
GA-1    18
GA-1    19
GA-1    20
GA-1    21
GA-1    22
IN-1    17
IN-1    18
IN-1    19
IN-1    20
IN-1    21
IN-1    22
NC-1     1
OH-1     7
SC-1    13
SC-1    14
TN-1     8
TN-1     9
TN-1    10
TN-1    11
TN-1    12
TN-1    13
TN-1    14
TN-1    15
TN-1    16
TN-1    17
TX-1    28
TX-1    29
TX-1    30
TX-1    31
I will double check that this is all indian land and then we decide what to display on rollover.

The doc files for the CD data collection, I looks like the -1 code means "unincorporated area."  I will try to confirm exact wording in Martis and also see if there is some extra detail available.  But for now, we can probably go with "Unincorporated land" on roll over.

This is implemented in /static/js/voteCharts.js. The code for detecting unincorporated land is in function checkUnincorporated(), and the label is on line 438 if anyone wants to tweak either.


Add installers for bash completion

Would be nice to be able to type now al<tab> and have the shell auto-complete to now alias.

I would take a look at https://www.npmjs.com/package/tabtab to see if it's good 👌

Haven't evaluated it but it looks nice and feature complete.


Error when using this.quill.clipboard.convert(XXX) in version >=1.5

I have this code which sets the content of my editor to the content saved last time
const contents = this.quill.clipboard.convert(this.content);
this.quill.setContents(contents);

this.content is a string that contains html content such as
test

With this, no problem but as soon as this.content contains a table in it (so
test
<div class="ql-table-wrapper" ...") i got this error:
TypeError: Cannot read properties of undefined (reading 'attributes')     at index.js:4009:16     at clipboard.js:274:18     at Array.reduce (<anonymous>)     at clipboard.js:273:60     at Array.reduce (<anonymous>)     at traverse (clipboard.js:267:46)     at clipboard.js:268:27     at Array.reduce (<anonymous>)     at traverse (clipboard.js:267:46)     at clipboard.js:268:27
Version used:

quill: 2.0.3
quill-table-up: 1.6 (same with 1.5.2 but works perfectly in 1.4)
parchment: 3.0.0

quill.clipboard.convert need a object like { html?: string, text?: string}, not a string. here is the convert type define
convert({ html, text }: {
    html?: string;
    text?: string;
}, formats?: Record<string, unknown>): Delta;

you should use like this. and I test it, it works on 1.6.0. here is demo
const contents = quill1.clipboard.convert({
  html: content, // content is html string
});
quill.setContents(contents);

You're right and I sent the wrong code, I indeed have this.quill.clipboard.convert({html : this.content});


Pub Sub subscription creation

Issue summary
Registering pub sub webhook doesn't publish to pub sub.
Expected behavior
Webhook should be published to pub sub
Actual behavior
The registration is successful but webhook is not sent to pub sub
This is my code for registering the webhook
const ordersPaidWebhookResponse = await Shopify.Webhooks.Registry.register({           shop,           accessToken,           path: "pubsub://<PROJECT_ID>:order-paid-on-shopify",           topic: "ORDERS_PAID",           webhookHandler: async (topic, shop, body) => {             console.log("Orders paid sent to pub sub");           }         });
Maybe i'm doing something wrong?
Steps to reproduce the problem

Reduced test case
The best way to get your bug fixed is to provide a reduced test case.

Checklist

[✅ ] I have described this issue in a way that is actionable (if possible)

It's hard to tell what the problem could be from this however it might be a missing scope or that the permission hasn't been granted in GCP.
Please take a look at this video for how to enable Pub/Sub in your app. It quickly walks you through getting it working end-to-end.
https://www.youtube.com/watch?v=m9MQyRWnMdU

Thanks for the video link. I'll check it out and give an update.

@mikedasilva your video is heaven sent. The issue was that i didn't update the API_VERSION to July21 and set the deliveryMethod to DeliveryMethod.PubSub. All other setup were correct. Thanks a lot.


Unable to learn BGP - UnboundLocalError: local variable 'session_state' referenced before assignment

genie learn bgp --testbed-file testbeds/testbed.yml
Traceback (most recent call last):
  File "src/genie/cli/commands/learn.py", line 365, in genie.cli.commands.learn.LearnCommand._retrieve_ops
  File "/Users/edgleyl/venv/pyats/lib/python3.9/site-packages/genie/libs/ops/bgp/nxos/bgp.py", line 1110, in learn
    self.make()
  File "/Users/edgleyl/venv/pyats/lib/python3.9/site-packages/genie/ops/base/base.py", line 103, in make
    self.maker.make(*args, **kwargs)
  File "src/genie/ops/base/maker.py", line 315, in genie.ops.base.maker.Maker.make
  File "src/genie/ops/base/maker.py", line 427, in genie.ops.base.maker.Maker._call_parser
  File "src/genie/metaparser/_metaparser.py", line 308, in genie.metaparser._metaparser.MetaParser.parse
  File "/Users/edgleyl/venv/pyats/lib/python3.9/site-packages/genie/libs/parser/nxos/show_bgp.py", line 2971, in cli
    [address_family]['session_state'] = session_state.lower()
UnboundLocalError: local variable 'session_state' referenced before assignment

Testbed
devices:
  switch001:
    connections:
      cli:
        ip: <IP_ADDRESS>
        protocol: ssh
    credentials:
      default:
        password: '%ENC'
        username: admin
      enable:
        password: '%ENC'
    os: nxos
    type: nxos

Can also share snippets of config/output if necessary.

NXOS: version 9.3(5)

I have tested the same learn bgp command on NXOS: version 7.0(3)I7(3) and it works.
Also tested pyats==22.3 on nxos 9.3(5), which gives the same error. So, whatever is causing this is the difference in NXOS outputs across versions.

Seems like a workaround/solution is to instansiate the session_state variable, as below
            # For address family: VPNv4 Unicast
            m = p31.match(line)
            if m:
                session_state = ''
                if 'address_family' not in  parsed_dict['neighbor'][neighbor_id]:
                    parsed_dict['neighbor'][neighbor_id]['address_family'] = {}
                address_family = str(m.groupdict()['af']).lower()

                if address_family not in parsed_dict['neighbor'][neighbor_id]\
                    ['address_family']:
                    parsed_dict['neighbor'][neighbor_id]['address_family']\
                        [address_family] = {}

                parsed_dict['neighbor'][neighbor_id]['address_family']\
                    [address_family]['session_state'] = session_state.lower()
                if 'state_reason' in parsed_dict['neighbor'][neighbor_id]:
                    parsed_dict['neighbor'][neighbor_id]['address_family']\
                    [address_family]['state_reason'] = \
                        parsed_dict['neighbor'][neighbor_id]['state_reason']
                continue

Hi .
Thanks for reaching out piestack support team, I will be looking at your ticket and get back to you .

Hi ,
Please could you share the  complete testbed.yaml file , and also please update your genie package and try it again .
pip install --upgrade genie
make sure that the credentials are correctly defined for the device
credentials:
  default:
    username: admin
    password: 'your_password_here'
  enable:
    password: 'your_enable_password_here'

Hi ,
Please can you share the user output and show command to reproduce the issue .

Hi ,
Please can you share the user output and show command to reproduce the issue .

Hi ,
Please can you share the user output and show command to reproduce the issue .

Company policy won't allow me to paste the configuration on public GitHub. Happy to raise a TAC, if you can direct me what category to raise it under.

Hi ,
Please can you mail the devices output and show command on cisco mail<EMAIL_ADDRESS>to reproduce the issue .

Hi ,
Please can you share the devices output and show command on external mail group .


Array start from 1

when array start with 1 ex comm: [1]=> object, mustache not work

Hi @arminkardovic,
I'm curious as to why would want to such an array. Could you provide a more in-depth example / scenario and what you would have expected from mustache?
Array with holes are strange beasts...

http://speakingjs.com/es5/ch18.html#array_holes
http://www.2ality.com/2013/07/array-iteration-holes.html

If in array first key is 1 it wont interate but if is 0 mustache iterate normaly ... i ask if mustache iterate anyway if start with key 0 or with key 100 for ex.
array(1=> Object, 2=> Object) wont iterate
array(0=> Object, 1=> Object) mustache iterate
can mustache iterate in bouth way

Hi @arminkardovic, iterating an array that doesn't start with key 0 works for me. Can you submit a sample script that reproduces the issue?

I personally tried with the following one:
var Mustache = require('mustache')

var view = {
    arr: [ ,, 'a' ,,, 'b' ,,, 'c']
}

var output = Mustache.render("{{#arr}}{{.}}{{/arr}}", view)

console.log(output)

[object Object][object Object]a[object Object][object Object]b[object Object][object Object]c

@arminkardovic Are you using mustache.php by any chance?

@bobthecow The way he writes the arrays.. it could be that.

Yes i use it in php and array return from function is array(1=>Object, 2=>Object) now i fix that in array(0=>Object, 1=>Object)  and mustache works perfectly but i don't know why mustache wont iterate anyway when i get it with ajax json

You're asking a question on Mustache.js ;)
https://github.com/bobthecow/mustache.php/wiki/Mustache-Tags#lists-vs-hashes

Yes :D

Yes but when i get object comm: array (1 => array('name' => 'Some Name') , 2=> array('name' => 'Some Name'))
and tempalte
{{#comm}} {{#.}} {{name}} {{/.}}{{/comm}} - this give me just one [Object object] but when I redeclare key in array comm: array (0=> array('name' => 'Some Name') , 1=> array('name' => 'Some Name'))
everithing work fine :D

The link I posted is your answer. Basically, unlike most other languages, in PHP the "array" type is used for both lists and associative arrays. So Mustache.php needs a way to tell whether a given array should be treated as a list, or as a rendering context. The way it tells is it looks for consecutive integer indexes, starting with 0. If the indexes go 0, 1, 2, 3... it's treated as a list. If the indexes are anything else, it's treated as an associative array (like passing in an object or something).
This repo is for mustache.js. You're having trouble using Mustache.php. They're two different projects, with different developers. That means this is not an appropriate place to continue this conversation :)

Good catch @bobthecow!


Could not find command "list".

Just pip installed and got this unexpected output:
❯ vimgolf list
Could not find command "list".
Deprecation warning: Thor exit with status 0 on errors. To keep this behavior, you must define `exit_on_failure?` in `VimGolf::CLI`
You can silence deprecations warning by setting the environment variable THOR_SILENCE_DEPRECATION.

Some specs:
vimgolf==0.2.1
Python 3.8.6
ruby 2.6.3p62

I've updated vimgolf so that vimgolf show will work on challenges that are currently reporting We're sorry, but something went wrong on vimgolf.com.
# Install the latest vimgolf
pip3 install --upgrade --no-cache-dir vimgolf

After updating to vimgolf==0.2.2, vimgolf show 55b18bbea9c2c30d04000001 now works.


Add playground config validation script and CI

closes #1737
ref : https://github.com/calyxir/calyx/issues/1737#issuecomment-1986905609
Sorry it took a bit of time to get this up, got busy in other things...
This adds a node script which will validate that

All passes listed in the passes.json are valid (cross checking with calyx pass-help o/p)
all example files that are in config.json are actually present

Currently parked the CI in rust.yaml, as I figured it would be better to run on each push, can move to other yaml as per suggestions.
Also instead of checking with master, the files are checked at local level, so that addition of new file or removal of existing file is correctly detected. I can also add a step to post a comment on PR if the validation fails, but might be overkill.
Once suggested changes are added, will remove the non existent passes and example files so the CI passes. Example of failing CI https://github.com/YJDoc2/calyx/actions/runs/8331868876/job/22799692081
@sampsyo please take a look, thanks :)

Hey also, if similar to last time, you are not getting the approve and run button for PR , and checks are pending, it might be because the CI are on push only, and thus run in my fork, but not on PR here. Most other PRs I checked  are from this repo to this repo, so their CI might be working properly. When I filter in actions for event pull_request only fud2 CI is there, others do not have any runs, so this might  be the cause for pending checks.

Awesome, thanks @YJDoc2!

Hey, So I'll remove the missing stuff / passes so the CI will pass ; I'm disabling auto merge so that you can suggest if you want a specific pass / example added, and it won't just get merge automatically.

Not entirely sure why the previous commits failed, but the latest main merge is succeeding CI. Given that the demo data validation CI is passing (which is added in this PR) , and I haven't actually changed any core code, only the example config data in web demo ; can the previous failures be ignored and this merged? @rachitnigam

Yeah, it was a dockerfile failure.


Implement msgpack for sequances of numbers

@mmoskal,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Cherry-pick: OpenRPC Validator: Not always returning method result schema (#679)

What
What does this PR add or remove?
Why
Why are these changes needed?
How
How do these changes achieve the goal?
Test
How has this been tested? How can a reviewer test it?
Checklist

[ ] I have self-reviewed this PR
[ ] I have added tests that prove the feature works or the fix is effective

Cancelling in favor of mass release branch merge.


Add Dynamic Port Allocation option

This was done in https://github.com/mineiros-io/terraform-google-cloud-router/pull/20


Avoid logging duplicate deprecation warnings multiple times

Description
This PR adds logic to avoid logging duplicate deprecation messages. It resolves the issue #1108 and all other deprecation message instances. (Coming from https://github.com/opensearch-project/OpenSearch/pull/1537#pullrequestreview-805281149). There are also test changes since a lot of tests verify the deprecation warnings.
Issues Resolved
Closes #1108
Check List

[X] New functionality includes testing.

[X] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

start gradle check

❌   Gradle Check failure 931bb72 Log 1502
Reports 1502

Execution failed for task ':distribution:bwc:bugfix:buildBwcLinuxTar'.
> Building 1.2.1 didn't generate expected file /var/CITOOL/workflow/OpenSearch_CI/PR_Checks/Gradle_Check/search/distribution/bwc/bugfix/build/bwc/checkout-1.2/distribution/archives/linux-tar/build/distributions/opensearch-min-1.2.1-SNAPSHOT-linux-x64.tar.gz

This should resolve after PR #1731 is merged.

Nice job @VachaShah and thanks for a thorough code review @andrross!


Depends feature for images

Hi, I wish to contribute a new feature to dobi, which can help us use it in development more, but first I need your approval and maybe advice.
Feature consists of two parts:
depends-on: ["path1", "path2"...] property on image object which will fetch the files from host, concatenate them and calculate their hash, this hash will got into depends.sha/depends.short_sha env variables.
Using {depends.sha} in tags will tag the image accoding to the calculated hash.
In combination those features allow us to create a dev container which is tied a certain number of files like package.json or requirements.txt and never rebuild it on developer machines, but instead pre-build on CI.
Probably the naming is not very thought out, but I hope you can help me with it.
Thank you for this great little gem, looking forward to hearing from you.

Hello! Thank you for your interest in this project.
I do not have much time to dedicate to dobi these days. I still use it myself almost daily, but I haven't needed to add any features for months. I may be able to look at some PRs, but I can't promise any time frames. You might get better mileage by working on your own fork.
My initial very quick impression of those features:

being able to automatically calculate a hash from files sounds cool, but doesn't seem like it fits with depends-on. I wonder if that could be done as a job:capture() task.
once captured you would be able to use the env var as a tag in the image.

So you might be able to do this already without any code changes, but I have no tried it.


Show commonly crafted items on front page

The current sorting of objects on the initial page is not very useful. It would be better to show a list of commonly crafted items at the top.
Ideally we would track the number of visits per object and sort it based on frequency, but this isn't possible with static pages. Alternatively the list could be manually curated. The player can search for anything else they need.
Update: Another option is to track how many times the object is used to make other objects. Find a balance between usefulness and complexity and sort the list based on this.

Yeah not sure what a good sort for this would be. One possibility is recently added items so people could keep up to date with game changes. That's basically what it used to be. I added in a basic "sort objects by their likelihood to spawn", but didn't do anything further.
I'd like to avoid manual curation, if Jason will be adding 100 items per week as his plan was. One possibility would be a utility function like what you were doing for the tech tree: Food adds utility, number of things that can be made with it adds utility, and so on. Then we could just tweak the utility function as necessary.

What do you think of the current sort order based on complexity?
I was thinking we could use Google Analytics to determine the most commonly accessed objects when doing processing.

Feels like it would be a little complex to manage credentials for Analytics etc, and needs an extra step of actually recording when users go to items (haven't got that set up yet, it just records everyone as going to the root url).
I think complexity itself is a bit of a hint, but perhaps things like food value and heat value should be added as ranking things higher. Specifically, we want to rank higher the things that are valuable in gameplay terms, as those are what people should be coming to the site to find out how to make. Stuff like walls is trickier since I don't think there is a straightforward way to distinguish them as being "gameplay relevant".


not support " = []"

Input:
List list = [];
Output:
formatter stop work，unless replace "[]" with "new()"
Expected behavior:
can format use "= [];"

Collection expressions are supported, you must be on an old version of csharpier. Note that csharpier is versioned separately from the ide extensions and updating the extension does not actually update csharpier.

I'm using dotnet 8.0.1 and cshapier 1.6.0
same error
string[] test = ["1", "2"];
Failed to compile so was not formatted.\n  (40,29): error CS1525: Invalid expression term '['\n"
If I use this string[] test = new string[] { "1", "2" };, there is no problem.

I use this command dotnet csharpier --version to get the version.
It is 0.22.1.
I have installed vs code extension cshapier 1.6.0.
I see the latest version is 0.28 at gitlab.
How can I update the version to 0.28?

Finally, I know my problem.
I use wrong version at .config/dotnet-tools.json
{
  "version": 1,
  "isRoot": true,
  "tools": {
    "CSharpier": {
      "version": "0.28.0",
      "commands": [
        "dotnet-csharpier"
      ]
    }
  }
}

I guess someone is like me to follow this link to setup husky
https://csharpier.com/docs/Pre-commit
Then, we use wrong version 0.22.1 and it is a very old version.
Maybe better to use X.XX.X and remind others to input latest version.

I updated the doc to remove the hardcoded version number.


Brute force and pasword spray

we are redirected to security.microsoft.com when try run any of atack.
On this page i cant find this atacks.

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

ID: 9fbdcbd5-7bd2-6b55-b72c-cbedfdea776c
Version Independent ID: 1e6198f8-6ef1-6227-c9e8-f31944a5d1db
Content: Attack Simulator in the Security & Compliance Center - Office 365
Content Source: microsoft-365/security/office-365-security/attack-simulator.md
Product: m365-security
Technology: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

@KONIKPK, clarification please.
The topic in question is for the old v1 Attack Simulator, which is retired and can't be used anymore.
The newer Attack Simulation Training is available in the Microsoft 365 Defender portal (security.microsoft.com) at https://security.microsoft.com/attacksimulator. There are 4-6 topics about this (for example https://docs.microsoft.com/microsoft-365/security/office-365-security/attack-simulation-training-get-started.
So are you saying that help links or admin center links in other places are directing you to the wrong place? If so, where are they

OK where can i find Brute force password and Password spray attack ?

@chrisda

@KONIKPK, Attack simulation training is different than the original Attack Simulator. The topic in question describes the available attack techniques:

Credential harvest
Malware attachment
Link in attachment
Link to malware
Drive-by URL

If you think something is missing, your best course of action is to open a support case. All we do here is update documentation.

@KONIKPK Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


[Request] Levels documentation clarification

Hi!
I'm trying out Marzipano, however, i got lost somewhere amidst level settings. It is quite clear that different levels represent different resolution tiling, however, it is not clear how particular level gets selected. All the demos on marzipano.net are immediately requesting level 3, however, if i try to reproduce the code, my Marzipano instance tries to fetch level 0, which is absent on the website (even though fallbackOnly is set to true for the first level, and faceSize = 4096 is used in view limiter); also it is not clear when and how Marzipano decides to switch levels. May ask you to document this behavior?

The documentation on level selection is indeed lacking. Note that this is partly on purpose (we'd like users of the library not to rely too much on the details, since they might change in the future), but I agree that it should be better. Here's a rough overview.
As a general rule, Marzipano selects the smallest level that provides adequate resolution for the current view (the meaning of "adequate" is best understood by reading this comment) and loads tiles for that level only; there is no progressive loading of successively larger levels.
Zooming in or out, or changing the window size, might cause a new level to be selected. While the new tiles are loading, tiles from other levels that were loaded previously may be used to fill in gaps (this is called "fallback" in the source code). Levels marked fallbackOnly are only used as fallback, i.e., a larger level will eventually be loaded in their place.
Tours generated by the Marzipano Tool, as well as most of the demos, pin the first level (pinning means tiles are fetched automatically and kept in memory, so that they can serve as a fallback of last resort). This is controlled by the pinFirstLevel option passed to Viewer.createScene, and is probably the reason why you're seeing requests for level 0 tiles, even though level 3 is the appropriate one for your screen size.

@tjgq Thank you!

I will revisit this once #99 is fixed.


Improve default settings formatting when folding is disabled

VSCode Version: insiders
OS Version: os x

Steps to Reproduce:

set "editor.folding": false,
Open settings file -> notice how it is weirdly indented

Either force that file to always have folding no matter the setting or do not indent my settings when folding disabled.

This indentation in settings is not new and was there from before. Agreed that when user disabled folding, we should take an alternate approach for displaying settings properly.

I think this does not apply anymore.