version `GLIBC_2.32' not found

Describe the bug
It seems that Supabase CLI no longer supports older versions of Debian. I'm using Deepin OS based on Debian 10 and after updating the CLI, I get the following errors:
supabase: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by supabase)
supabase: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by supabase)

To Reproduce
Steps to reproduce the behavior:

Use latest version
Error

Expected behavior
Could be awesome if we still can support "old" OS.
Screenshots
/
Desktop (please complete the following information):

OS: Deepin (Debian 10)
Browser (if applicable): /
Version of CLI 1.77.6
Version of supabase-js (if applicable): /
Version of Node.js (if applicable): 16.20.1

Additional context
/

Thank you for raising this issue. I've enabled static linking in cli v1.77.8 so it doesn't depend on glibc at runtime.

Encountered this error message in Bitbucket pipeline. Updated download script to
npm install --location=global<EMAIL_ADDRESS>
and error still exists. Can any assistance be offerred @sweatybridge? Thanks

Could you share with the full error logs from bitbucket here? Including the command you used to run and npx supabase -v

No worries.
pipeline config:
image: node:16.16.0
pipelines:
  branches:
    development:
      - step:
          name: Apply DB migrations in development
          script:
            - export SUPABASE_ACCESS_TOKEN=$SUPABASE_DEVELOPMENT_ACCESS_TOKEN
            - export SUPABASE_DB_PASSWORD=$SUPABASE_DEVELOPMENT_DB_PASSWORD
            - npm install --location=global<EMAIL_ADDRESS>            - npx supabase -v
            - npx supabase link --project-ref $SUPABASE_DEVELOPMENT_PROJECT_ID
            - npx supabase db push

err:
npx supabase -v
npm WARN exec The following package was not found and will be installed: supabase
supabase: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by supabase)
supabase: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by supabase)

It's likely that npm install gave up. Could you try without --location flag?
            - export SUPABASE_ACCESS_TOKEN=$SUPABASE_DEVELOPMENT_ACCESS_TOKEN
            - export SUPABASE_DB_PASSWORD=$SUPABASE_DEVELOPMENT_DB_PASSWORD
            - npm install<EMAIL_ADDRESS>            - npx supabase -v
            - npx supabase link --project-ref $SUPABASE_DEVELOPMENT_PROJECT_ID
            - npx supabase db push

Experiencing the same issue.
Dev Environment
Node version: 18.16.1
OS: Debian 11 bullseye
Error
$ npm install --location=global<EMAIL_ADDRESS>
added 23 packages in 9s

4 packages are looking for funding
  run `npm fund` for details

$ npx supabase -v
supabase: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by supabase)
supabase: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by supabase)

Removing the location flag worked, thanks for your help @sweatybridge. Any ideas why that would have caused it to fail?

Global installation via npm/yarn isn't supported atm because the binary cannot be correctly downloaded to node_modules via postinstall script. If you need to install globally, I'd suggest using one of the system package managers, such as homebrew or scoop, since they are designed for this purpose.
We added support for local installation via npm because it allows you to pin the version of CLI in package.json. This ensures consistent setup between local machine and CI pipelines. Perhaps this is closer to what you want?
npm i --save-dev<EMAIL_ADDRESS>
After committing changes to package.json, update CI pipeline like below
            - npm ci
            - npx supabase -v


RevitConverter: Material quantities add length of elements

Related to issue #1302 and PR #1314
#1314 implements the material_quantities to the revit connector. material_quantities contains the volume and area of each Material in an element.
Sometimes it is alos essential to have the length of an element divided by Material Thus, I would like to add another parameter to material quantities.  Besides area and volume it should also have a length.
Please let me know if you see any problem with that?
Cheers,
Kilian

I don't see issues with it, please go ahead :)

@connorivy @AlanRynne @ks-cph  what is left to do to close #1976

@fraguada The ball is in my court. I haven't been working on Revit at all this sprint which is why this hasn't been closed yet. I just made my last structural PR that I'm going to make this sprint so I'll work on this now.


[VS Code] Starting collaboration session: Invalid root scheme: vsls, expected 'file'

<!--
For Visual Studio problems/feedback, please use the Report

@jojokaran72 Closing as duplicate of #1672 .  Let's continue the conversation there. Can you provide some more information about your scenario? Was this blocking you from starting a collaboration session?


Fix tuning.py (python3 / mac)

Fix 'expected a bytes-like object, str found' error message.
Maybe specific to my environment, but seems legit.

Fix 'expected a bytes-like object, str found' error message.
Maybe specific to my environment, but seems legit.

I think this pull request should be merged. I faced the similar problem.

I had the same problem. Please keep your projects manintained and apply this patch.

Thanks for your help! @GillesDebunne


Parsing error: Unexpected token await/*

What version of standard?
11.0.1
What operating system, Node.js, and npm version?
Linux x86, Node v10.8.0, npm v6.2.0
What did you expect to happen?
Continue without parsing error
What actually happened?
When linting following code an parser error occured
async function* fromLinesToNumberedLines(lines) { // Parsing error: Unexpected token *
  let counter = 1;
  for await (const line of lines) { // Parsing error: Unexpected token await
    yield `${counter}: ${line}`;
    counter++;
  }
}

https://zaiste.net/nodejs_10_asynchronous_iteration_async_generators/

Works with babel-eslint parser.


Close bug reports that don't use the template automatically

We have received bug reports in the last time that do not use the bug report template and do not include all the required information. I would like to help people that have issues installing flare-vm. But issues that do not provide all the information we need to be able to help only make us waste time, as we need to ask for the missing information and wait for it until we can help.
I purpose that we use GH actions to identify this kind of issues and close them automatically with a comment that links to the template to use.

Maybe it is enough if we:

migrate to issue forms
Add a form for features ideas and discussions
Set the setting blank_issues_enabled to false to not allow to use blank issues


Conflict with google_mlkit_face_detection

I am getting following error when trying to run mobile_scanner.
[!] CocoaPods could not find compatible versions for pod "GoogleMLKit/MLKitCore":
  In Podfile:
    google_mlkit_face_detection (from `.symlinks/plugins/google_mlkit_face_detection/ios`) was resolved to 0.5.0, which depends on
      GoogleMLKit/FaceDetection (~> 3.2.0) was resolved to 3.2.0, which depends on
        GoogleMLKit/MLKitCore (= 3.2.0)

    mobile_scanner (from `.symlinks/plugins/mobile_scanner/ios`) was resolved to 3.2.0, which depends on
      GoogleMLKit/BarcodeScanning (~> 4.0.0) was resolved to 4.0.0, which depends on
        GoogleMLKit/MLKitCore (= 4.0.0)

My pubspec.yaml has following dependencies:
  cupertino_icons: ^1.0.2
  firebase_auth: 4.3.0
  firebase_core: 2.8.0
  google_fonts: ^4.0.1
  form_field_validator: ^1.1.0
  lottie: ^2.2.0
  image_picker: ^0.8.6+1
  path_provider: ^2.0.12
  image_cropper: ^3.0.1
  flutter_image_compress: ^1.1.3
  provider: ^6.0.5
  http: ^0.13.5
  animate_do: ^3.0.2
  pin_code_fields: ^7.4.0
  google_mlkit_face_detection: ^0.5.0
  google_mlkit_text_recognition: 0.5.0
  camera: ^0.10.3+2
  image: ^4.0.12
  dropdown_search: ^5.0.3
  percent_indicator: ^4.2.2
  geocoding: ^2.0.5
  location: ^4.4.0
  google_maps_flutter:
  flutter_polyline_points: ^1.0.0
  google_maps_webservice: ^0.0.19
  flutter_google_places: ^0.3.0
  basic_utils: 5.5.3
  time_range: ^1.0.3
  jwt_decoder: ^2.0.1
  sliding_up_panel: ^2.0.0+1
  flutter_background_service: ^2.4.6
  flutter_background_service_ios: ^2.4.0
  flutter_background_service_android: ^3.0.3
  shared_preferences: ^2.0.20
  barcode_widget: ^2.0.3
  encrypt: ^5.0.1
  socket_io_client: ^2.0.1
  geolocator: ^9.0.2
  intl: ^0.17.0
  mobile_scanner: ^3.2.0

for me
Conflict with google_mlkit_text_recognition

Same problem with google_mlkit_text_recognition for me too.


Cannot access VNC server

I have latest versions selenoid + selenoid-ui installed on the same virtualbox (tried both Centos 7.1.1 and Ubuntu 16.10) with all default settings as specified in the current documentation. However when I try to view VNC session from selenoid-ui, I am getting "Disconnected" message and selenoid container log contains the following lines:
[VNC_ENABLED] [43823227-4cdc-4743-ba8c-965af5da4f09]
[VNC_ERROR] [dial tcp <IP_ADDRESS>:5900: getsockopt: connection refused]
It looks as though selenoid container cannot access browser container's port 5900. Should not browser containers expose port 5900 or perhaps run in "net=host" mode? Or should there be any additional system configuration steps?

Did you set enableVNC=true capability in tests?

Sure, otherwise there would be no "VNC_ENABLED" and no available VNC sessions in Selenoid UI.

Summon @aandryashin.

@nebehr got it. This is because you are using container without VNC server inside. By default there is no VNC server. We are currently working to provide VNC containers for all supported browsers. Please try whether it works with these images. Just update images in browsers.json and reload Selenoid.

Hmm, this is embarrassing. It works now. :)

@nebehr we decided to not include VNC by default because Selenoid is considered to be used in big cluster and VNC is mainly local debug feature. Also VNC makes images bigger. Here is the issue about adding VNC: https://github.com/aerokube/selenoid-images/issues/3


Electron support

http://stackoverflow.com/a/31725712/8418

Wasn't suppose to go here.. sorry :/
(unless we could keep it)

Uhm, did this ever get merged? Is there an official way to do this?


Doesn't update multiple repos in the same deb822 file (Ubuntu 24.04)

The fast-apt-mirror.sh find --apply feature doesn't update the repo URI if there are multiple repositories in the same deb822-formated file, as is the case by default in Ubuntu 24.04. It appears to only update the first one.
Default Ubuntu 22.04 /etc/apt/sources.list.d/ubuntu.sources file (relevant lines):

Types: deb
URIs: http://archive.ubuntu.com/ubuntu
Suites: noble noble-updates noble-backports
Components: main universe restricted multiverse
Signed-By: /usr/share/keyrings/ubuntu-archive-keyring.gpg

Types: deb
URIs: http://security.ubuntu.com/ubuntu
Suites: noble-security
Components: main universe restricted multiverse
Signed-By: /usr/share/keyrings/ubuntu-archive-keyring.gpg

Ubuntu 22.04 /etc/apt/sources.list.d/ubuntu.sources file after running fast-apt-mirror.sh find --apply (relevant lines):

Types: deb
URIs: http://fast.mirror.example.com/ubuntu
Suites: noble noble-updates noble-backports
Components: main universe restricted multiverse
Signed-By: /usr/share/keyrings/ubuntu-archive-keyring.gpg

Types: deb
URIs: http://security.ubuntu.com/ubuntu
Suites: noble-security
Components: main universe restricted multiverse
Signed-By: /usr/share/keyrings/ubuntu-archive-keyring.gpg

Note that only the URI of the first repository is changed even though both are "main" repos.
The desired behavior would be that both URIs are changed.

Types: deb
URIs: http://fast.mirror.example.com/ubuntu
Suites: noble noble-updates noble-backports
Components: main universe restricted multiverse
Signed-By: /usr/share/keyrings/ubuntu-archive-keyring.gpg

Types: deb
URIs: http://fast.mirror.example.com/ubuntu
Suites: noble-security
Components: main universe restricted multiverse
Signed-By: /usr/share/keyrings/ubuntu-archive-keyring.gpg

I doubt it would be advisable to point to the security repo to a mirror of the main repo.

I guess that is the recommendation: https://wiki.ubuntu.com/SecurityTeam/FAQ#What_repositories_and_pockets_should_I_use_to_make_sure_my_systems_are_up_to_date.3F
Would it be possible for this to be a non-default override option? Then users who choose could change the security repo to a mirror.

Hm, I think this gets complicated. currently we simply extract the main URL and after running the tests do a dumb search/replace of that exact URL in the config file. If you can come up with a resilient approach how to change the URL of another repo to the main URL for all OSes and repo config types you are welcome to provide a PR.


Remove references to docassemble.LanguagePack

docassemble.LanguagePack has been archived; you can update any links to point to docassemble.AssemblyLine instead.

Thanks for the heads up!


How can I exempt stretches of text from being processed as Markdown?

Is there anyway to turn off the parser for stretches of code? Like:
This is some Markdown.

# markdownoff
This is some crazy HTML construct that I don't want Markdown to touch or even know about.
# markdownon

This is some more Markdown.

Clearly that doesn't work, but that's basically what I'm looking for.

Markdown inside html blocks will be copied verbatim.
For example
*foo*

*bar*

foo

*bar*

That's what I thought, but I don't think it's what I'm seeing. I'll re-test. If I can't duplicate, I'll close.

I did some testing, and this depends on there being no blank lines inside the HTML blocks. If you have this:
*foo*

*bar*

That will general HTML inside the div because of the blank line after the opening tag.
In my situation, due to some pre-processing that generates the Markdown, I can't guarantee there won't be any blank lines.
Do I have any options to toggle Markdown processing in this situation?

You could wrap the content in a  yourself? Then it's a known quantity.
S.
From: Deane Barker<EMAIL_ADDRESS>Sent: 02 July 2019 11:00
To: lunet-io/markdig<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [lunet-io/markdig] How can I exempt stretches of text from being processed as Markdown? (#350)
I did some testing, and this depends on there being no blank lines inside the HTML blocks. If you have this:
foo

bar

That will general HTML inside the div because of the blank line after the opening tag.
In my situation, due to some pre-processing that generates the Markdown, I can't guarantee there won't be any blank lines.
Do I have any options to toggle Markdown processing in this situation?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/lunet-io/markdig/issues/350?email_source=notifications&email_token=AAPQMIWLD7PVFNMYFVLCZP3P5MRKBA5CNFSM4H4VX5U2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZAX4BA#issuecomment-507608580 , or mute the thread https://github.com/notifications/unsubscribe-auth/AAPQMIULT6LQ6GRKQ3MP3ALP5MRKBANCNFSM4H4VX5UQ .  https://github.com/notifications/beacon/AAPQMIV74AMVNM6MLQXTSXTP5MRKBA5CNFSM4H4VX5U2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZAX4BA.gif

There might be child tags, also with blank lines. Worst case I could try to strip all newlines in a post-post-processing step, but I feel like this would be a handy addition to the library in general. There are other situations where this might come up.

It requires a new kind raw custom container. PR welcome. If someone tackle this, I would like to see a proposal of the syntax, including a check on other major Markdown libraries (including Pandoc) in case we could reuse/agree on a similar syntax

You could also use fenced code blocks for this, I think that would be the least intrusive language-wise.
If you don't want to see  and language html in the result, you can replace those blocks when post-processing the document.
For example using "mdignore" as the info:
foreach (var codeBlock in document.Descendants<FencedCodeBlock>())
{
    if (codeBlock.Info == "mdignore")
    {
        var parent = codeBlock.Parent;
        int childIndex = parent.IndexOf(codeBlock);
        parent.Remove(codeBlock);
        parent.Insert(childIndex, new HtmlBlock(null)
        {
            Lines = codeBlock.Lines
        });
        // HtmlBlock is used as it's a simple non-abstract LeafBlock
    }
}

If you want to extract the actual content from the block (for example if you want to use a ParagraphBlock instead):
foreach (var codeBlock in document.Descendants<FencedCodeBlock>())
{
    if (codeBlock.Info == "mdignore")
    {
        var parent = codeBlock.Parent;
        int childIndex = parent.IndexOf(codeBlock);
        parent.Remove(codeBlock);

        int startIndex = markdown.IndexOf('\n', codeBlock.Span.Start) + 1;
        int endIndex = markdown.LastIndexOf('\n', codeBlock.Span.End) - 1;
        if (markdown[endIndex] == '\r') endIndex--;

        var replacement = new ParagraphBlock
        {
            Inline = new ContainerInline()
        };
        replacement.Inline.AppendChild(new LiteralInline()
        {
            Content = new StringSlice(markdown, startIndex, endIndex)
        });

        parent.Insert(childIndex, replacement);
    }
}

Side-note:
Even for a simple example like this it would be useful to have some more "document api" like ReplaceBy for Blocks. I can do PR for that.

Here's what I found in AsciiDoc: Literal blocks
Some Markdown...
...................................
Stuff you don't want processed as Markdown...
...................................
Some more Markdown...

Elsewhere it says that block fencing is:

normally a series of four or more repeated characters

So, I think it would open and close with four dots.

Here is Textile's version: notextile
<notextile>
Don't touch this!
</notextile>

The inline version is double equals:
Don't ==**bold**== this.

Some Markdown...
...................................
Stuff you don't want processed as Markdown...
...................................
Some more Markdown...

That looks to me like a similar approach to using code blocks.

Explain to me the syntax. I see the mdignore stuff above, to go in the "info", but I'm a little fuzzy on how that syntax might actually work/look.

The simplest syntax is the indented code block where you indent a block with 4 spaces.
I personally prefer the fenced code blocks, for those you enclose text with three (or more) backticks or tildes and you can also specify the "info" aka language. This is what you use to get syntax highlighting on GitHub for example.
This will enclose the inner text in

 tags, so that's what I meant by replacing those elements from the document. Then markdown like this would not emit extra tags but simply embed the text as it was in the source.

So...?
```mdignore
Don't touch this

Yes just three backticks at start and end
```mdignore
Copied verbatim
```

Okay, putting together your stuff from above, I get this, which does work.
Markdown
This is some *Markdown*.
```mdignore

      This is my content in an ASIDE tag
      with all sorts of indents and line breaks and stuff.

```
This is some more *Markdown*.

Code (note: I've never gone past the Markdown.ToHtml() static method before, so this might suck...)
var document = Markdown.Parse(markdown);
foreach (var codeBlock in document.Descendants<FencedCodeBlock>())
{
  if (codeBlock.Info == "mdignore")
  {
    var parent = codeBlock.Parent;
    int childIndex = parent.IndexOf(codeBlock);
    parent.Remove(codeBlock);
    parent.Insert(childIndex, new HtmlBlock(null)
    {
      Lines = codeBlock.Lines
    });
  }
}

var writer = new StringWriter();
var renderer = new HtmlRenderer(writer);
renderer.Render(document);
writer.Flush();
var html = writer.ToString();

Result

This is some Markdown.

      This is my content in an ASIDE tag
      with all sorts of indents and line breaks and stuff.

This is some more Markdown.

Does that code look fine?  If so (and even if not), I have a solution and we can probably resolve.
(I do think this might be handy to build into the core @xoofx, even if just like @MihaZupan has suggested above.)

The code looks fine, if you find yourself using a custom pipeline don't forget to also call pipeline.Setup(renderer); before rendering.

Just implemented this in my production project. It works beautifully -- solves so many problems and eliminates so many workarounds.
Thank you for all the help on this, and I hope this issue resolution helps others down the road.


Final Scrum Report

What did you work on this past iteration (2 weeks)?
Added nutritional info to the UI, fixed bugs in the test suite, and worked extensively on the readme and the project demo.
What were any major issues/challenges you ran into?
Timing the demo under 7 minutes. Practiced it a couple of times and were good to go as a team.
What do you plan to work on for this coming iteration (2 weeks)?
Make the UI more responsive. Implement text to speech functionality.

What did you work on this past iteration (2 weeks)?
Debugged final version, worked on readme and demo preparation.
What were any major issues/challenges you ran into?
No major issues this iteration, mainly makes sure no bugs were present for demo version
What do you plan to work on for this coming iteration (2 weeks)?
Implement front page for recipes and look at similarity search to reduce LLM costs

What did you work on this past iteration (2 weeks)?
Final touches to the UI. Preparing the presentation.
What were any major issues/challenges you ran into?
Ensuring the demo didn't exceed time limits.
What would you work on if you had more time?
Making the UI more aesthetic. Implementing an interface for voice controlled recipe generation.

What did you work on this past iteration (2 weeks)?
Final bug fixes and implementation for the demo. fixed image upload in backend. Added multiple ingredient views. Added share page. reworked recipe saving.
What were any major issues/challenges you ran into?
None
What would you work on if you had more time?
Making the app more social hy adding a recipe discover page. Adding more detail to profiles and adding a comment section in every recipe for tips.


weeds

I planted grass in preparation for my next crop. I did not weed it, so there are large weeds. I'm currently rolling the field but it is not removing weeds. I chose to roll instead of mulch because in the instructions it appears that is the route we should be taking if possible. The large weeds are not being removed by the roller and are not removed by the seeder. Am I doing this correctly? I assumed the weeds would have been removed in this process since one of the draws to doing this properly is no weeds after seeding.

directly across the street in the field on the left, you can see weeds there too. it's currently cultivated and has large weeds. if i go seed over there, the large weeds disappear.

CA does not prevent weed removal, if anything, it additionally removes weeds in some situations. Can you try a new save game on the same map with no mods and check if you get the same issue? It almost looks like the map has a problem here


Error: '/opt/hostedtoolcache/node/15.8.0/x64/bin/npx'

I am trying to use the action, however it keeps failing with the error:
Run<EMAIL_ADDRESS>  with:
    github-token: ***
    jest-command: npx jest --coverage
    only-changes: true
    delete-previous: true

Error: ExecCommand error: Error: The process '/opt/hostedtoolcache/node/12.20.1/x64/bin/npx' failed with exit code 1
Error: The process '/opt/hostedtoolcache/node/12.20.1/x64/bin/npx' failed with exit code 1

I thought this might be due to the node version but it also fails on versions up to Node 15.x.  Are you able to advise where I am going wrong?
unit_tests_latest:
    name: Unit Tests - Latest
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - uses: actions/setup-node@v1
        with:
          node-version: "12.x"
      - name: Restore cached dependencies
        uses: actions/cache@v2
        with:
          key: node-modules-${{ hashFiles('yarn.lock') }}
          path: |
            ~/.cache/yarn
            **/node_modules
      - name: Install dependencies
        run: yarn install --immutable
      - name: Test using Jest
        run: yarn run test:ci
        env:
          CI: true
      - name: Add Jest coverage to PR
        uses<EMAIL_ADDRESS>        id: jest_coverage
        with:
          github-token: ${{ secrets.GITHUB_TOKEN }}

@charklewis after executing the yarn run test:ci in your local environment the coverage files are created correctly?
Try to open the lcov.info file in an editor ./coverage/lcov.info.
Normally this error happens when no coverage is generated.

Sure does! However the path is different... is this the problem?
tests/jest_coverage/lcov.info

I was able to fix the issue by updating the coverage output path 👍🏼 I notice it deletes the prior comment (to avoid having a million comments on the PR) which is great, however is it possible to have this edit the comment instead?

@charklewis Wow, this can be awesome!.
Currently, it's not an available feature (edit comment) but if you want to propose a PR 😉

Sure thing! If I get a chance I'll have a look creating a PR for that feature 🙂


feat(lsp): add optional filter function

This PR provides a optional filter to give user the ability to filter the results of the LSP calls before viewing.
This is helpful in situations the user doesn't want to list some references. Here is an example usage in Go to filter out all the items in mock files:
local check = "_mock.go"
local filter = function(items)
  local ret = {}
  for _, item in ipairs(items) do
    if item.filename:sub(-#check) ~= check then
      table.insert(ret, item)
    end
    return ret
  end
end
require("fzf-lua").lsp_implementations({
  jump_to_single_result = true,
  filter = filter,
})

@arsham, could this be solved by file_ignore_patterns?
https://github.com/ibhagwan/fzf-lua/wiki#file-ignore-patterns

@arsham, could this be solved by file_ignore_patterns?
https://github.com/ibhagwan/fzf-lua/wiki#file-ignore-patterns

If we can change the settings as per usage basis then yes. But I'm afraid I couldn't come up with a setup that wouldn't have a side effect in other scenarios. For example we want to be able to browse to the mock files with file selector. I'm not sure how you would go about filtering out any methods containing the word test.
I'm happy to drop this PR if there is an alternative solution.
Cheers!

Merged, ty @arsham!


HubSpot CRM (Indpendent Publisher) - Fix scopes

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[ ] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[ ] I attest that the connector works and I verified by deploying and testing all the operations.
[ ] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[ ] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[ ] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[ ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hello @troystaylor,
We have seen that you submitted the PR, thank you very much for correcting it.
We have noticed that you have submitted the PR, and we appreciate your effort in correcting it. However, we observed that you added scopes for only one action. It is essential to add scopes for all the actions to avoid unnecessary back and forth.
Therefore, we suggest creating an OAuth app using your account locally, testing it thoroughly, and then submitting the PR. This will ensure all necessary scopes are included and streamline the process.
Thanks for your patience and understanding.

@vmanoharas Please provide an update on why this PR is still open.
CC: @jopanchal @ShefaaliP

Additionally, this PR was submitted when there was an issue in the pipeline.  Please resubmit your PR/commit so that it can be triggered again and undergo validation? Thank you for your understanding. We apologize any inconvenience caused.

Hi @troystaylor,
/crm/v3/objects/contacts/search action has been removed. Can you please confirm that no existing flows or users will be affected by this change?


updated key features and table of contents in read.md file

Pull Request
Title

Description

Related Issues

Changes Made
Updated  table of content section and features and explain solution proposed  more elaborately.
Checklist

[ ] I have tested the changes locally
[ ] Documentation has been updated (if necessary)
[ ] Changes are backward-compatible

Screenshots (if applicable)

Additional Notes

I have updated the readme file successfully


[Suggestion] New tile property: Glass.

Glass is a new tile property that allows lasers(and possibly light bridges too) through the block, but doesn't let anything else through. I see no reason why this shouldn't be added, seeing as we already have grates.

I have two ideas for this, which would simplify matters a lot:

A new tile property for each filter: ignores bridges, ignores lasers, ignores funnels, ignores portals shot, etc., or;
A new trigger-like entity - or tile-like entity - that acts as a filter, allowing us to manually determine which beams go through (as well as potentially other properties).

I like the trigger-like filter entity, I think it could have lots of uses. Perhaps add a new option in the "tools" tab that lets you layer the tile properties over the existing tiles, which overwrites the tile's default properties.

Took the liberty to adapt the issue to the new proposal


Pressed card button unselected if StyleOptions set in Minimizable bot sample

Version
4.13.0
Describe the bug
In the minimizable bot sample if you select a card button, minimize and open the bot again the selected button is unselected if StyleOptions is set.
Without StyleOptions the buttons do remain selected even after minimizing and reopening the bot.
Steps to reproduce

Add the below StyleOptions to the minimizable bot sample

Select a card button e.g. 'Show Video Card', it returns the card and the button remains selected.
Minimize the bot and reopen it to see the button has been unselected.

Screenshots
Without StyleOptions

With StyleOptions

Expected behavior
The buttons that were selected should still show as selected when reopened even if we add StyleOptions.

@stevkan Could you please take a look at this one?

@ryub3n, please try wrapping styleOptions in useMemo() and passing that in as property into createStyleSet(). This is what appeared to work for me. Your code should look something like the below:
MinimizableWebChat.js
const styleOptions = useMemo(
  () => {
    return {
      primaryFont: 'Arial',
      timestampColor: '#1b1a19',
      hideUploadButton: true,
      botAvatarInitials: null,
      userAvatarImage: null,
      userAvatarInitial: null,
      botAvatarImage: null,
    }
  }, []
)

const styleSet = useMemo(
  () =>
    createStyleSet({
      backgroundColor: 'Transparent',
      ...styleOptions
    }),
  [styleOptions]
);

[ ... ]

<WebChat
  className="react-web-chat"
  onFetchToken={handleFetchToken}
  store={store}
  styleSet={styleSet}
  token={token}
/>

WebChat.js - styleSet is passed in to the WebChat function component
<ReactWebChat
  className={`${className || ''} web-chat`}
  directLine={directLine}
  store={store}
  styleSet={styleSet}
/>

@stevkan - This works but there's another problem. hideUploadButton isn't passed to StyleSet (Ref) I guess because the upload button is showing up even though I've set it to true in StyleOptions.

You are right. I missed that small detail. It's a bit of an oddity, but passing styleOptions into both styleSet as well as an individual Web Chat property appears to work.
MinimizableWebChat.js
const styleSet = useMemo(
  () =>
    createStyleSet({
      backgroundColor: 'Transparent',
      ...styleOptions
    }),
  [styleOptions]
);

[ ... ]

<WebChat
  className="react-web-chat"
  onFetchToken={handleFetchToken}
  store={store}
  styleSet={styleSet}
  styleOptions={styleOptions}
  token={token}
/>

WebChat.js - styleSet and styleOptions are read from the WebChat function component and passed into the ReactWebChat component.
<ReactWebChat
  className={`${className || ''} web-chat`}
  directLine={directLine}
  store={store}
  styleSet={styleSet}
  styleOption={styleOptions}
/>

I tried that but then the original issue i.e. the pressed button one still exists.

If you are able to do so, could you zip your project up into a file and post it here (or you can email it to me, if that makes you more comfortable)? Just remove any secrets/keys listed anywhere.

My bad, it's working now. I missed the empty dependencies array for the styleOptions useMemo 😅.

Closing as resolved


Call SIMD variants for transforms when possible.

Closing and re-opening this PR, looks like there is a problem with github.


ElasticSearch crashes when running rule

Hello,
Elasticsearch keeps crashing whenever I start up Elastalert. I have a feeling it has something to do with one of the rules I made. This is the rule:
name: Metricbeat CPU Spike Rule
type: any

#es_host: localhost
#es_port: 9200

index: metricbeat-*

buffer_time:
  hours: 1

metric_agg_key: system.cpu.user.pct
metric_agg_type: avg
query_key: beat.hostname
doc_type: metricsets

bucket_interval:
  minutes: 5

sync_bucket_interval: true
#allow_buffer_time_overlap: true
#use_run_every_query_size: true

min_threshold: 0.01
max_threshold: 0.1

filter:
- term:
    metricset.name: cpu

# (Required)
# The alert is use when a match is found
alert:
- "debug"
- "slack"

slack:

Any help is appreciated. I left out the slack part.

Please give the output of the crash. You should get a traceback or message.

@Qmando Yes, the actual service crashes. I always have to manually start it up again.

Hmm, you'll definitely need to check the Elasticsearch logs, they should have some useful information. I've managed to crash Elasticsearch a few times when we ask it to perform an aggregation it can't fit into memory, no idea if that's the issue here. You can use --es_debug_trace trace.log to get the raw queries being made to elasicsearch, that might help you debug.

@TheAnimal12345  i have same issues. How do you fixed it ?

@Qmando  When i run elastalert rule, the elasticsearch service was dead (failed). My version ELK is 6.2.2. Please help me fix it. Thanks you!

Did you check the Elasticsearch logs?

@dungkma What worked for me was deleting the ElastAlert index and creating a new one. Mine was somehow corrupt. The same thing happened with Kibana when I was using it. Kibana would not load and I tried everything to fix it and no luck. Finally deleted the Kibana index and created a new one and everything started running smoothly.

@Qmando  Elasticsearch logs:
[2018-08-22T00:01:04,228][DEBUG][o.e.a.s.TransportSearchAction] [Elastic-01] [elastalert_status][1], node[77Jo8GGyQWS0X9yHq9QjRA], [P], s[STARTED], a[id=-IbQ5EiJTVm4-kWP8rq-sQ]: Failed to execute [SearchRequest{searchType=QUERY_THEN_FETCH, indices=[elastalert_status], indicesOptions=IndicesOptions[id=38, ignore_unavailable=false, allow_no_indices=true, expand_wildcards_open=true, expand_wildcards_closed=false, allow_aliases_to_multiple_indices=true, forbid_closed_indices=true, ignore_aliases=false], types=[elastalert], routing='null', preference='null', requestCache=null, scroll=null, maxConcurrentShardRequests=20, batchedReduceSize=512, preFilterShardSize=128, source={"size":10000,"query":{"query_string":{"query":"aggregate_id:-0HKXGUBpdqEbiOevJ70","fields":[],"type":"best_fields","default_operator":"or","max_determinized_states":10000,"enable_position_increments":true,"fuzziness":"AUTO","fuzzy_prefix_length":0,"fuzzy_max_expansions":50,"phrase_slop":0,"escape":false,"auto_generate_synonyms_phrase_query":true,"fuzzy_transpositions":true,"boost":1.0}},"sort":[{"@timestamp":{"order":"asc"}}]}}]


chore: explain ordering for the jest setup better

The setup showed now can be used in any component depending on next/link etc. By first mocking it away all further dependencies will then actually use that mock. I hope the jest example is now more clear. If not decline it, no worries :)
https://github.com/scottrippey/next-router-mock/blob/325757ac1fce4f48fa324748d8a36ccf2736b84b/README.md#usage-with-jest

Closing for now, I'm revisiting the README soon and will make it clearer!


updated days of risk calculation

This PR resolves Issue #2670 "Reduce "14" to "10" where it corresponds to the days of the risk calculation" by updating mentions of the number of days for risk calculation in the FAQ to include the number of days used since version 2.20.
The following FAQ entries have been updated:
English FAQ:
What do I have to consider when switching to a new device?
Why is voluntary self-quarantine recommended?
I received the message that I had a risk encounter. The next day it was gone again. Shouldn't this be deleted after 14 days?
German FAQ:
Was muss ich beim Wechsel auf ein neues Gerät beachten?

 Am nächsten Tag war sie wieder weg. Sollte das nicht erst nach 14 Tagen gelöscht werden?

Just made the suggested changes. Please let me know if there are any issues.

Many thanks to @amouzayanni @Ein-Tim and @MikeMcC399 ! @amouzayanni Congrats to your first PR in this project!


Add toggle option between available graph and candlestick chart

https://github.com/drorata/stocks-playground/commit/8422c375870264b2b19967259f7277b8f2f12668


Improve exec env

I pulled this, rebased, fixed the failing test locally and pushed directly to refactor, so closing this PR (because it is effectively merged).


WIP: Noto Nastaliq next

This is a collection of fixes to go into the next release of NNU. I'll keep adding to this as I go through the issue list, and when we've fixed all that we can, I'll add a commit to change the version.

Fix missing letters in googlefonts/noto-fonts#2240
Encode 06A0 and 06CF, fixes googlefonts/noto-fonts#2244
Fix dot positioning of 0768. Fixes googlefonts/noto-fonts#2238

LGTM, nice work. I clicked on the review button and I didn't realize it will change from the draft (feel free to convert it back to draft, if this is a work in progress). Please let me know when it's ready.


Making UserData docs more clear about base64

This commit updates the UserData docs to be more clear that you should not base64 encode the value when providing the parameter. The text is now bold to make it more pronounced.
Also added a minor typo fix for the default AutoPopulatedParam description.
Closes #585
@jamesls @kyleknap

:shipit:


BLEHIDPeripheral setBondStore

Hi,
how can I correctly set the BondStore of a BLEHIDPeripheral?
If I call bleHIDPeripheral.setBondStore(...) the BLEPeripheral class method gets called which doesn't update the _bleBondStore private field. Calling bleHIDPeripheral.setup() then overrides my previous call...
Thanks

@llemtt why do you need to override the BondStore?

My current project runs on a BLE Nano and requires a different flash page for the BondStore because offset=0 overwrites the bootloader!
I managed this modifying the library source code, definitely not the right way to do it.
Besides that, if I understood correctly, setBondStore(...) purpose is to let the developer supply even a completely different store location like for instance an external eeprom.

Oh got it, why don't we just change the default offset on the BLE Nano to be non-zero?
What do you think? Do you have time to submit a pull request?

@llemtt ping ...


Add phonenumberlib for enhanced number validation #14

can be tested on: https://defikarteformapp.z1.web.core.windows.net/

can be tested on: https://defikarteformapp.z1.web.core.windows.net/

Reviewed and successfully tested..


Consider using ideas from shtub or even merging the projects

So we already have shtub for mocking commands.. it works with the same $PATH trickery under the hood.
Things shtub has that shmock is missing IMHO:

verifications - mock frameworks are not very useful if there is no way to assert which calls were made
DSL-flavored set up for mocking and verifications
thread safety for verifications (calls are serialized)
optional delays when calling commands (useful to test for concurrency issues)
partial input bound definition with the at_least_with_arguments styled matchers
stacked answers, e.G. first return "foo" then "bar" then exit with 42

The idea of shmock was not to replace shtub, but to have a shell command mocker that is much simpler than shtub. Mainly because I couldn't even get shtub's "hello world" example to run in reasonable time.
The optional delays would be easy to implement. Even a
delay=(1.0, 3.5)
which chooses a random delay between 1.0 and 3.5 seconds for each run would be simple. And useful.
For the DSL, do you mean syntax like shtub's
when.calling('cmd').at_least_with_arguments('foo', '--verbose=1')
or something else?
The other ideas you mention would require to hold state, which makes things a lot more complicated. I'd like to avoid that and use shtub for such sophisticated setups.

Well yes, I also think there is value in a tool that can be set up with "just a dict". You're right that shtub requires some additional setup.
The DSL I was referring to is in fact the when.calling() and verify.called() part. Clearly it's a matter of taste but I like the type "safety" due to using actual functions/methods.

Delays are dangerous though - I don't think it was a good idea to add them to shtub. We fixed the concurrency bugs with theoretical analysis anyway and they just add up the the test execution time.

Partial input would be great though, it doesn't require state and can improve test resilience.
If I change johnny -b --good to  johnny -b --good -v a test with a partial bound will not fail but a strict one will.
FTR I think it's great to use shtub as an argument to keep shmock simple.


Fix master branch check in Travis-CI config.

The Hub Framework is being phased out at Spotify, and therefore we will not be maintaining it further.


Documentation for tile-drawing event

The new tile-drawing functionality isn't documented. I've been giving it a go via guesswork, looking at the source, and using a debugger, and so far I've got:
sd.addHandler('tile-drawing', function(viewer){
        ctx = viewer.rendered;

        if(ctx){
            var width=viewer.tile.size.x ;
            var height=viewer.tile.size.y ;
            width*=2 ; // ?? tile size is not necessarily canvas size??
            height*=2 ; // ?? tile size is not necessarily canvas size??

            if(width>0 && height>0){
                var imgd = ctx.getImageData(0, 0, width, height);

               // Do some Simple image processing

                // Draw the ImageData at the given (x,y) coordinates.
                ctx.putImageData(imgd, 0, 0);
            }

        }
    }

});

I have 2 issues:
i) Surely there is a better way of setting the width/height of the canvas than multiplying the tile size by 2 (certainly if you just set it to the tile size you don't process the whole tile canvas .. presumably due to tile scaling).
ii) While it works fine at low zoom, it is really slow when you zoom in. Surely the same number of tiles should be rendered (approximately) whatever the zoom?
I presume this is mainly down to me not having any documentation, so thanks in advvance for the help!
Derek

I have been developing a filtering plugin the last few days. (I hope to be able to publish it soon)
I discovered that you shouldn't flag the tile as already processed but the context.
The reason being that OSD will discard some rendered context to free up memory.
When the rendered context of a tile get re-created, you will need to re-perform whatever operation you are doing on it.
So, your code should look like:
osd.addHandler('tile-drawing', function(event){
   var context = event.rendered;
   if (context._alreadyProcessed) {
      return;
   }

   // do stuff
   context._alreadyProcessed = true;
}

@avandecreme Good to get that clarification!


Transfer technical documentation from README.md to official docs (single-source-of-truth simplification)

Hi,
A couple of tasks which can be achieved now that the official Bartholomew documentation is live (public).

Reduce the amount of pre-existing technical documentation in Bartholomew's README.md file.
Transfer technical content like commands, instructions on how to install/use and so forth, to the official docs site [1].

This will provide a single-source-of-truth approach for us and reduce effort (only update documentation in 1 place, as the software evolves). Thanks @vdice for the feedback/comment @ https://github.com/fermyon/bartholomew/issues/77#issuecomment-1198422643
I have assigned myself buy please anyone feel free to jump in with any ideas/comments/feedback i.e. thanks @karthik2804 for pointing out the binary release point the other day @ https://github.com/fermyon/bartholomew/pull/93#discussion_r931863229 that comment was useful and thought provoking.

https://github.com/fermyon/bartholomew/pull/110 closes this.


Testing in Swift - Invalid Entity Description

Hello!
I have some problems testing my program using MMRecord in Swift. Here is the error message I am having:
[MMRecord ERROR]-- Invalid Entity Description. This could be because this record class is not used in your managed object model, or because your persistent store coordinator or managed object model are not defined properly. An entity description is required for creating records..
And here is the test file:
override func setUp() {
    super.setUp()
    GSServer.registerAFHTTPSessionManager(SessionManager.sharedInstance);
    GSRecord.registerServerClass(GSServer)
    MMRecord.setLoggingLevel(MMRecordLoggingLevel.All)
}

override func tearDown() {
    super.tearDown()
}

func testLoadNewsfeed() {
    var expectation = self.expectationWithDescription("Newsfeed loaded")

    var managedObjectContext = setUpInMemoryManagedObjectContext()
    Post.startPagedRequestWithURN("feed", data: nil, context: managedObjectContext, domain: self, resultBlock: { (record, pagemanager, requestNextPage) -> Void in
        expectation.fulfill()
    }) { (error) -> Void in

    }

 self.waitForExpectationsWithTimeout(5, handler: nil)

}

And the in memory managed object context
 func setUpInMemoryManagedObjectContext() -> NSManagedObjectContext {
let managedObjectModel = NSManagedObjectModel.mergedModelFromBundles([NSBundle.mainBundle()])!

let persistentStoreCoordinator = NSPersistentStoreCoordinator(managedObjectModel: managedObjectModel)
persistentStoreCoordinator.addPersistentStoreWithType(NSInMemoryStoreType, configuration: nil, URL: nil, options: nil, error: nil)

let managedObjectContext = NSManagedObjectContext()
managedObjectContext.persistentStoreCoordinator = persistentStoreCoordinator

return managedObjectContext
}

I tracked down the error to the (NSEntityDescription *)MMRecord_entityForClass:(Class)managedObjectClass which is called for initial entity (Post in this case) in + (NSArray*)recordsFromResponseObject:(id)responseObject and returns nil.
Now this happens only in Testing case, everything works as expected in the production environment.
I found that the modules might be the problem as written here: http://www.jessesquires.com/swift-coredata-and-testing/ but adding prefix to the class name didn't help.
Any idea what might be the problem and how to fix it?
The MMRecord version is 1.4.3. and swift version 1.1.
Thanks

Hi there,
Thank you for taking the time to write up this very detailed issue, and I'm sorry we haven't given it more investigation.
At this point we're not using MMRecord in any new Swift projects. So, while some testing has been done to just smoke test the library on Swift, this type of detail isn't something I have experienced personally. I wish I had a better answer, but I think the article you linked actually hits the nail on the head near the end. MMRecord is a very Objective-C style of dynamic interpretation of what is going on from an object population perspective. It may just turn out not to be the Swift way of doing things. I'm sure we'd do things differently in a Swift version of MMRecord, but right now that's not something we have planned.
Thanks,

Conrad


Add app views

In this milestone,

Created a new branch

Implemented app view

Updated existing code

1

2

3

4

Hey there, thank you for reviewing my code. It seems you didn’t leave
anything behind to ask for a change. Should you check one more time? There
might be more potential issues. I don’t understand why you guys pick on
something that has already been reviewed.
Now i just have redactor all of it. 😐
On Sun, Sep 3, 2023, 2:09 AM Munsa Mibenge @.***> wrote:

@.**** requested changes on this pull request.
Hi @MoyasiGinko https://github.com/MoyasiGinko,
Good job so far!
There are some issues that you still need to work on to go to the next
project but you are almost there!
Highlights
Achieved A good UI Design ✔
Correct Git Flow ✔
Descriptive ReadMe File ✔
Required Changes ♻
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop
the approval of this PR. However, I strongly recommend you to take them
into account as they can make your code better.
You can also consider:

x
y
z

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In app/views/users/index.html.erb
https://github.com/MoyasiGinko/Rails-Blog/pull/7#discussion_r1313957501:

 <% @users.each do |user| %>

<%= user.name %>

<img src="<%= user.photo %>" alt="User Image">

       <img src="<%= user.photo %>" alt="User Image">

<%= user.name %>

Joined: <%= user.created_at.strftime('%B %d, %Y') %>

<% end %>

Good job creating the all users page, however, the requirements
state that you have to implement the design in the given wireframes. Kindly
ensure your design matches the given wireframe in terms of layout and page
elements. Refactor your design to display the number of posts and not the
date joined

Given frame
[image: image]
https://user-images.githubusercontent.com/81932478/265224771-7ef1ec3e-b8e8-4ee3-9755-7aa1b09303b7.png
Your design
[image: image]
https://user-images.githubusercontent.com/81932478/265224779-51449860-99be-4ef1-b81e-d0c0fc398bd8.png

Also clicking on each user should take you to their show page as shown
in this wireframe below, Kindly fix this
Clicking on a user should bring you to this page
[image: image]
https://user-images.githubusercontent.com/81932478/265224857-8d30cc1f-c981-408f-8b31-8ea911976118.png
(optional) It is better practice to use partials instead of using
@objects.each. Consider it
example of a partial
# app/views/users/_user.html.erb
 #   HTML Code Here displaying
 #   user.photo, user.name and user.posts_counter

# app/views/users/index.html.erb

<%= render @users %>

In app/views/users/show.html.erb
https://github.com/MoyasiGinko/Rails-Blog/pull/7#discussion_r1313962683:

<%= @user.name %>

<%= @user.name %>'s Posts

Number of Posts: <%= @posts.count %>

Your views in app/views/users/show.html.erb should use #recent_posts
method in your models to render the user's 3 most recent posts. Now it
seems this requirement was overlooked and your method in the User.rb
model is actually returning 5 recent posts, I would advise you to fix the
method before using it here. Clicking the see all posts button should
show all of a user's posts.

Your design
[image: image]
https://user-images.githubusercontent.com/81932478/265225556-fb08d480-1a8b-43a3-9503-5fa24f28f700.png
Given design
[image: image]
https://user-images.githubusercontent.com/81932478/265225566-2d657624-85a3-40b2-bbde-2df7f136d485.png
Note that the given wireframe only returns three of the most recent posts
while your implementation returns all posts

Also kindly follow the design of the given wireframe, Your design is
good but the layout is different from the wireframes. In this design, your
design should show also the number of comments and likes for each post

In app/views/users/show.html.erb
https://github.com/MoyasiGinko/Rails-Blog/pull/7#discussion_r1313964107:

<%= @user.name %>

<%= @user.name %>'s Posts

Number of Posts: <%= @posts.count %>

Clicking on the see all posts button should take the user to the
wireframe below, kindly implement this as the link is currently not working
[image: image]
https://user-images.githubusercontent.com/81932478/265225889-920c8877-280c-4272-ac1f-784f95453b89.png

In app/views/posts/index.html.erb
https://github.com/MoyasiGinko/Rails-Blog/pull/7#discussion_r1313968694:

<%= post.comments.count %> Comments

               <% post.comments.each do |comment| %>

               <% post.recent_comments.each do |comment| %>

<%= comment.text %>

               <% end %>

The app/views/posts/index.html.erb should also have the number of
likes displayed and they should be aligned to the right with the number of
comments as in the given design.
The image should be at the top to the left instead of the user name
as well
The comment should also bear the name of the user as in the wireframe

Your design
[image: image]
https://user-images.githubusercontent.com/81932478/265226424-b604908f-5615-46f6-b7a0-03eb950e3356.png
Given frame
[image: image]
https://user-images.githubusercontent.com/81932478/265226180-d7de354f-b45d-4afe-afea-2500848a3532.png
In app/views/posts/show.html.erb
https://github.com/MoyasiGinko/Rails-Blog/pull/7#discussion_r1313970827:

<%= @user.name %>

<%= @targated_post.text %>

Number of Likes: <%= @targated_post.likes_counter %>

Number of Comments: <%= @targated_post.comments_counter %>

Again here kindly ensure your design follows the given wireframes
for the post-show page

Your design
[image: image]
https://user-images.githubusercontent.com/81932478/265226590-7eb7832d-caf9-4f72-97fe-7790a26714a5.png
Given frame
[image: image]
https://user-images.githubusercontent.com/81932478/265226608-1f5accb9-76e3-43ba-a054-135ba596b4dc.png

(Optional). It would be nice if clicking on an individual post in
the index post page redirected the user to the particular post-show page.

e.g. if clicking on a post in the wireframe below
[image: image]
https://user-images.githubusercontent.com/81932478/265226714-40c3f439-921c-4551-b5af-d28c048e5ce5.png
Redirected the user to the wireframe below
[image: image]
https://user-images.githubusercontent.com/81932478/265226736-df250267-262d-4a72-aad7-0cde6e6ca95b.png
—
Reply to this email directly, view it on GitHub
https://github.com/MoyasiGinko/Rails-Blog/pull/7#pullrequestreview-1608366553,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZYV2GBYYWTNRIMBGWBBODDXYOHAHANCNFSM6AAAAAA4IYFTR4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey there, thank you for reviewing my code. It seems you didn’t leave anything behind to ask for a change. Should you check one more time? There might be more potential issues. I don’t understand why you guys pick on something that has already been reviewed. Now i just have redactor all of it. 😐
…
On Sun, Sep 3, 2023, 2:09 AM Munsa Mibenge @.> wrote: @.* requested changes on this pull request. Hi @MoyasiGinko https://github.com/MoyasiGinko, Good job so far! There are some issues that you still need to work on to go to the next project but you are almost there! Highlights Achieved A good UI Design ✔ Correct Git Flow ✔ Descriptive ReadMe File ✔ Required Changes ♻ Check the comments under the review. Optional suggestions Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. You can also consider: - x - y - z Cheers and Happy coding!👏👏👏 Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag me in your question so I can receive the notification. Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise. ------------------------------ As described in the Code reviews limits policy https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s. ------------------------------ In app/views/users/index.html.erb <#7 (comment)>: >  <% @users.each do |user| %> - <%= user.name %> -  +  +  + <%= user.name %> + Joined: <%= user.created_at.strftime('%B %d, %Y') %> +  <% end %>  - Good job creating the all users page, however, the requirements state that you have to implement the design in the given wireframes. Kindly ensure your design matches the given wireframe in terms of layout and page elements. Refactor your design to display the number of posts and not the date joined Given frame [image: image] https://user-images.githubusercontent.com/81932478/265224771-7ef1ec3e-b8e8-4ee3-9755-7aa1b09303b7.png Your design [image: image] https://user-images.githubusercontent.com/81932478/265224779-51449860-99be-4ef1-b81e-d0c0fc398bd8.png - Also clicking on each user should take you to their show page as shown in this wireframe below, Kindly fix this Clicking on a user should bring you to this page [image: image] https://user-images.githubusercontent.com/81932478/265224857-8d30cc1f-c981-408f-8b31-8ea911976118.png - (optional) It is better practice to use partials instead of using @objects.each. Consider it example of a partial # app/views/users/_user.html.erb # HTML Code Here displaying # user.photo, user.name and user.posts_counter # app/views/users/index.html.erb <%= render @users %> ------------------------------ In app/views/users/show.html.erb <#7 (comment)>: >   - <%= @user.name %> + <%= @user.name %>'s Posts Number of Posts: <%= @posts.count %>  - Your views in app/views/users/show.html.erb should use #recent_posts method in your models to render the user's 3 most recent posts. Now it seems this requirement was overlooked and your method in the User.rb model is actually returning 5 recent posts, I would advise you to fix the method before using it here. Clicking the see all posts button should show all of a user's posts. Your design [image: image] https://user-images.githubusercontent.com/81932478/265225556-fb08d480-1a8b-43a3-9503-5fa24f28f700.png Given design [image: image] https://user-images.githubusercontent.com/81932478/265225566-2d657624-85a3-40b2-bbde-2df7f136d485.png Note that the given wireframe only returns three of the most recent posts while your implementation returns all posts - Also kindly follow the design of the given wireframe, Your design is good but the layout is different from the wireframes. In this design, your design should show also the number of comments and likes for each post ------------------------------ In app/views/users/show.html.erb <#7 (comment)>: >   - <%= @user.name %> + <%= @user.name %>'s Posts Number of Posts: <%= @posts.count %> - Clicking on the see all posts button should take the user to the wireframe below, kindly implement this as the link is currently not working [image: image] https://user-images.githubusercontent.com/81932478/265225889-920c8877-280c-4272-ac1f-784f95453b89.png ------------------------------ In app/views/posts/index.html.erb <#7 (comment)>: >  <%= post.comments.count %> Comments  - <% post.comments.each do |comment| %> + <% post.recent_comments.each do |comment| %> <%= comment.text %> <% end %>  - The app/views/posts/index.html.erb should also have the number of likes displayed and they should be aligned to the right with the number of comments as in the given design. - The image should be at the top to the left instead of the user name as well - The comment should also bear the name of the user as in the wireframe Your design [image: image] https://user-images.githubusercontent.com/81932478/265226424-b604908f-5615-46f6-b7a0-03eb950e3356.png Given frame [image: image] https://user-images.githubusercontent.com/81932478/265226180-d7de354f-b45d-4afe-afea-2500848a3532.png ------------------------------ In app/views/posts/show.html.erb <#7 (comment)>: >  <%= @user.name %> <%= @targated_post.text %> + Number of Likes: <%= @targated_post.likes_counter %> Number of Comments: <%= @targated_post.comments_counter %>  - Again here kindly ensure your design follows the given wireframes for the post-show page Your design [image: image] https://user-images.githubusercontent.com/81932478/265226590-7eb7832d-caf9-4f72-97fe-7790a26714a5.png Given frame [image: image] https://user-images.githubusercontent.com/81932478/265226608-1f5accb9-76e3-43ba-a054-135ba596b4dc.png - (Optional). It would be nice if clicking on an individual post in the index post page redirected the user to the particular post-show page. e.g. if clicking on a post in the wireframe below [image: image] https://user-images.githubusercontent.com/81932478/265226714-40c3f439-921c-4551-b5af-d28c048e5ce5.png Redirected the user to the wireframe below [image: image] https://user-images.githubusercontent.com/81932478/265226736-df250267-262d-4a72-aad7-0cde6e6ca95b.png — Reply to this email directly, view it on GitHub <#7 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZYV2GBYYWTNRIMBGWBBODDXYOHAHANCNFSM6AAAAAA4IYFTR4 . You are receiving this because you were mentioned.Message ID: @.***>

Hi @MoyasiGinko  Sorry for the frustration, if you need me to clear up anything slack me for a quick response

Hi @MoyasiGinko sorry for any frustrations, If you need any clarifications Slack me for a quicker response.


[Windows] Remove ndk-bundle

Description
ndk-bundle was explicitly marked as "Obsolete" by Google and no one should be using it anymore. The current symlink creation process breaks the Android SDK package manager. More details can be found here https://github.com/actions/virtual-environments/issues/2689#issuecomment-1172129050
Related issue:
https://github.com/actions/virtual-environments/issues/5879
Check list

[x] Related issue / work item is attached
[x] Tests are written (if applicable)
[x] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows2019, windows2022

/azp run windows2019, windows2022


Update to the newest ledger

Description
Fixes invalid ExtLedgerState serialization. Produced CBOR encoding contained a list without a length, thus it was not decodable as arbitrary CBOR.
Integration of latest cardano-ledger and cardano-base.

Update iohk-nix sources to get latest libsodium needed for cardano-crypto-praos
Account for introduction of versioned CBOR in ledger. The new interface EncCBOR/DecCBOR is very similar to the plain ToCBOR/FromCBOR, with helper functions to go between them.
Remove usage of MemoBytes for Header, since Header cannot be parameterized on era and MemoBytes now requires it.
Account for addition of Shelley translation context: FromByronTranslationContext
Account for change of parametrization of ShelleyGenesis from era to crypto
Account for switch of PParams and other types from HasFields to lenses
Account for various fields getting renamed
Account for the fact that ConwayGenesis has to also include AlonzoGenesis fields, because otherwise CanStartFromGenesis can't work. We need to figure out the whole business of TranslationContext and Genesis, it is a bit messy right now IMHO.
Update golden tests, which were needed due to:

Fixes to PParams serialization
Changes of default protocol versions in emptyPParams in order eras
Fixes to test examples
Addition of a new field with deposits to the ledger state
Fix to ExtLedgerState serialization
Changes to Conway types

Remove some arbitrary instances (such as Slotting for example), since they are now imported from ledger. Notable instances are also UTCTime and NominalDiffTime, that now are imported from quickcheck-instances. In order to preserve older semantics standalone generators for those types were retained and used in delayNextSlot property test. Also guarded that against an "infinite" loop with within.

Blocked by https://github.com/input-output-hk/cardano-ledger/pull/3282 and CHaPS releases
Checklist

Branch

[x] Commit sequence broadly makes sense
[x] Commits have useful messages
[ ] The documentation has been properly updated
[x] New tests are added if needed and existing tests are updated
[ ] Any changes affecting Consensus packages must have an entry in the appropriate changelog.d directory created using scriv. If in doubt, see the Consensus release process.
[ ] If this branch changes Network and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[x] If serialization changes, user-facing consequences (e.g. replay from genesis) are confirmed to be intentional.

Pull Request

[x] Self-reviewed the diff
[ ] Useful pull request description at least containing the following information:

What does this PR change?
Why these changes were needed?
How does this affect downstream repositories and/or end-users?
Which ticket does this PR close (if any)? If it does, is it linked?

[ ] Reviewer requested

bors r+

bors r+


Notification polling - Long duration

The section that refers to "Notification polling" states that "if a user stays on a page for a long duration, new notifications will be retrieved."
"Long duration", here, is not precise and I received a call with a customer that had a different expectation. They're creating in-app notifications through SDK and setting ttlinseconds to 60, and they could not see notifications toasting while remaining on the same page. If they refreshed or opened a different page, then it worked, so the request here is just to clarify in the documentation how long the user should be on a page for getting the notifications.

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

ID: f495f03e-28c9-0ebb-ef03-72e16b267b3a
Version Independent ID: a9dcec59-8b72-966e-fa53-f4b2e42d325e
Content: Send in-app notifications within model-driven apps - Power Apps
Content Source: powerapps-docs/developer/model-driven-apps/clientapi/send-in-app-notifications.md
Service: powerapps
Sub-service: mda-developer
GitHub Login: @adrianorth
Microsoft Alias: aorth

This needs to be changed to say that new notifications will not be received until the page navigation happens.  Setting ttlseconds to 60 is not recommended because that will delete the notification in 60 seconds and likely will not be seen by the user.  Default ttlseconds setting is 14 days so suggest a value measured in days.

Thanks for your comment, @fmarcellino. This issue has been assigned to the doc author for review.

The doc has been updated with to call out new notifications won't be available until page navigation happens.


Np 45210 unable to fetch profile picture

Dont know if this fixed the problem, but the stack now displays picture data. Added a integration test so we can capture this if it happens again


Updated ParseOrder function to read marketSymbol from product_id

Updated ParseOrder function to read marketSymbol from product_id instead of id. Updated TargetFramework in ExchangeSharp.cspoj to move nestandard2.0 before net472 to be able to support intellisense in netcore projects also.

Thanks. That works.


.vimrc error pls help

Hello,
I just installed vim and spf13 which is quite nice vim configuration. After i installed , to run type this command "vim" there is following errors:
line    1:
E492: Not an editor command: Skip to content
line    2:
E492: Not an editor command: This repository
line    3:
E492: Not an editor command: Search
line    4:
E492: Not an editor command: Pull requests
line    5:
E492: Not an editor command: Issues
line    6:
E492: Not an editor command: Gist
line    8:
E492: Not an editor command:  Watch 596
line    9:
E492: Not an editor command:   Star 9,333
line   10:
E492: Not an editor command:   Fork 2,853 spf13/spf13-vim
line   11:
E492: Not an editor command:  Code  Issues 269  Pull requests 21  Projects 0  Wiki  Pulse  Graphs
line   12:
E492: Not an editor command: Branch: 3.0 Find file Copy pathspf13-vim/.vimrc
line   13:
E172: Only one file name allowed: cd86dbd  on Feb 3
line   15:
E492: Not an editor command: 93 contributors @spf13 @johntyree @taxilian @sbennett18 @bloveridge @JoveYu @chrisnicola @lukedomanski @jcbwlkr @perfectworks @jamestomasino @jphustman @victoredwardocallaghan @mmikitka @parherman @teasp00n @j1mr10rd4n @wolfgangpfnuer @Gaelan @stephpy @ramonmaruko @rekendahl @SaswatPadhi @toejough @munkt0n @lihuazhang and others
line   16:
E492: Not an editor command: RawBlameHistory
line   17:
E492: Not an editor command: 1252 lines (1090 sloc)  49.3 KB
line 1269:
E492: Not an editor command: Contact GitHub API Training Shop Blog About
line 1270:
E492: Not an editor command: © 2016 GitHub, Inc. Terms Privacy Security Status Help
my .vimrc file has a lot of datas.
Pls help guys

@NerdKid this is not a problem with vundle. You installed spf13 incorrectly. It looks to me like you saved the github page itself rather than install the bundle. I recommend wiping out your vimrc and starting over following the spf13 docs.

@NerdKid Any updates?

Closing due to long time with no response.
@NerdKid please reply/respond if you still need help.


Fixes issue #854 - Fix SingleThread HTTP server tests

Fixes #854

I will merge this, but it might fail on the pipeline. That is why it was set to @Ignore previously but it is time to make sure we keep everything consistently running.


livekit-plugins-anthropic: Repeated stream text output after function call

Description:
When using livekit-plugins-anthropic with the voice_assistant agent, I've encountered an issue where the LLM repeats the exact same stream text output after triggering a function call.
Steps to reproduce:

Set up a project using livekit-plugins-anthropic and voice_assistant agent
Implement a scenario that triggers a function call
Observe the LLM's output before and after the function call

Expected behavior:
The LLM should continue generating new, relevant text after the function call.
Actual behavior:
The LLM repeats the exact same stream of text that was output before the function call.
Environment:

Python version: 3.12.4
livekit-plugins-anthropic: Installed from latest GitHub code
(pip install 'git+https://github.com/livekit/agents.git#subdirectory=livekit-plugins/livekit-plugins-anthropic')
Other relevant dependency versions:
livekit==0.14.0
livekit-agents==0.8.8
livekit-api==0.7.0
livekit_plugins_elevenlabs==0.7.3
livekit-plugins-deepgram==0.6.5
livekit-plugins-openai==0.8.1
livekit-plugins-silero==0.6.4
livekit-protocol==0.6.0

Additional information:

I noticed that pip version 0.1.1 doesn't work, which is why I'm using the latest GitHub code.

Thanks for the bug report, will look into this today (as well as the deployment issue)


gene_summary and gene_detailed in Gemini

To be fully compatible with Gemini, is there any way for us to include the gene_summary and gene_detailed tables in the DB output as well?
One suggestion would be porting arq5x/gemini/gemini/gene_table.py to vcf2db, while allowing GTF file input from user.
Thanks a lot.

I had planned to deprecate those tables. Do you use them? If I know the use-case, maybe we can figure out a workaround.

My existing Gemini pipeline is based on GRCh37, and I want that to be
almost 100% compatible when I port to GRCh38. As part of the provenance
requirements of my environment, gemini's ability to bundle gene annotations
with variants is essential for me.
Therefore I have drafted a branch that allows me to include gene
annotations as well using vcf2db. The codes were tested against GRCh38
annotations only. The URL to my branch can be found below:
https://github.com/a113n/vcf2db/tree/gene_table?files=1
Hope that's useful for someone else who need that feature!
On Wed, 19 Oct 2016 at 8:29 PM, Brent Pedersen - Bioinformatics <
<EMAIL_ADDRESS>wrote:

I had planned to deprecate those tables. Do you use them? If I know the
use-case, maybe we can figure out a workaround.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/quinlan-lab/vcf2db/issues/15#issuecomment-254797957,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADEGJ_lZp_YFHAPxRszYeU0rHp7mTjZEks5q1g08gaJpZM4Kah6i
.


Tutorial

Hey there,
I'm pretty new to Node.JS and had a difficult time understanding your example of how to use ReJSON.
I had to do hours of googling before I found something that worked for me. I first discovered that yield statements only work inside of generator functions. That was easy enough to find out. Of course afterwards I also read about Promises. However, I had a very difficult time finding out how to get your Promises to resolve.
I ended up finding someone who had this code that lets me pass in a generator function, and it resolves all your promises returned in your yield statements.
I'm unsure if there is a better way, but for now it is what I am doing.
reference: https://developers.google.com/web/fundamentals/getting-started/primers/promises
Wrapper code:
function spawn(generatorFunc) {
  function continuer(verb, arg) {
    var result;
    try {
      result = generator[verb](arg);
    } catch (err) {
      return Promise.reject(err);
    }
    if (result.done) {
      return result.value;
    } else {
      return Promise.resolve(result.value).then(onFulfilled, onRejected);
    }
  }
  var generator = generatorFunc();
  var onFulfilled = continuer.bind(continuer, "next");
  var onRejected = continuer.bind(continuer, "throw");
  return onFulfilled();
}

Example code:
spawn(function* example(){
    try{
        var instance = new Rejson();
        yield instance.connect();
        yield instance.set('foo', '.', {
        bar: {
            hello: 'world'
        }
        });

        value = yield instance.get('foo', '.');
    }
    catch(e){
        console.log(e);

    }
    console.log(value);
});

Thanks for the feedback! I agree that generator is a bit confusing and the language itself is moving away from it as well (with the whole async/await).
Technically, everything just returns a promise, so you can yield it or you can just do the good old promise way:
var instance = new Rejson();
instance.connect().then(function() {
  instance.set('foo', '.', {
    bar: {
      hello: 'world'
    }
  }).then(function() {
    instance.get('foo', '.').then(function(value) {
      console.log(value);
    });
  });
});

I totally agree with tylo. Thank you evan for the code above, it helped me as I was unfamiliar with the yield keyword. Maybe add a note on the README?


Can't backspace during text entry

Hi,
Just found your app and giving it a try for my daily driver. One thing I encountered is I cannot backspace if I typed something wrong. I can arrow to the left and use my delete key.

Any ideas?

Which terminal emulator are you using? I just tried in xterm, and indeed backspace didn't work. I have added another keycode for backspace, which seemed to fix the issue for xterm, can you try out the newest commit?

Bam your spot on, I'm using xterm.

Can confirm, backspace is now working in xterm, Thx!

Awesome, thanks for making me aware of the issue


Fix Fluentd probe

What this PR does / why we need it:
See screenshots and https://github.com/elastisys/compliantkubernetes-apps/pull/1542#issuecomment-1521382910
Special notes for reviewer:
Related to #1542
Add a screenshot or an example to illustrate the proposed solution:
Before

After

Further investigation needed to find out whether the probe is correct about fluentd being down. Could also be more wrong with the probe URL.
Checklist:

[ ] Added relevant notes to WIP-CHANGELOG.md
[ ] Proper commit message prefix on all commits
[ ] Updated the public facing documentation
Is this changeset backwards compatible for existing clusters? Applying:

[ ] is completely transparent, will not impact the workload in any way.
[ ] requires running a migration script.
[ ] will create noticeable cluster degradation.
E.g. logs or metrics are not being collected or Kubernetes API server
will not be responding while upgrading.
[ ] requires draining and/or replacing nodes.
[ ] will change any APIs.
E.g. removes or changes any CK8S config options or Kubernetes APIs.
[ ] will break the cluster.
I.e. full cluster migration is required.

Chart checklist (pick exactly one):

[ ] I upgraded no Chart.
[ ] I upgraded a Chart and determined that no migration steps are needed.
[ ] I upgraded a Chart and added migration steps.

Pipeline config (if applicable)
If you change some config options (e.g. add/rename variable or change the default value) you may need to update the config used by the pipeline in pipeline/config.

Further investigation needed to find out whether the probe is correct about fluentd being down.

It's down, the probe is correct. Good to have that part of the dashboard tested :)

Furthermore, is Fluent running in the service cluster at all? It seems to be in the workload cluster, so a probe on the cluster-internal name from the sc seems unlikely to work. Finally, I'm not sure if the URL is even correct, running the same probe in the wc reports DOWN as well.

Furthermore, is Fluent running in the service cluster at all? It seems to be in the workload cluster, so a probe on the cluster-internal name from the sc seems unlikely to work. Finally, I'm not sure if the URL is even correct, running the same probe in the wc reports DOWN as well.

Yes, if you have fluentd enabled in the SC cluster and at least one of audit or scLogs enabled.

Yes, if you have fluentd enabled in the SC cluster and at least one of audit or scLogs enabled.

Turns out I had (via defaults) fluentd.enabled = false...


external location: alertmanager uses external locations for internal listener

For example prometheus configured with the -alertmanager.url=http://alertmanager:9093  option.
And alertmanager has -web.external-url=https://my-external-ingress-controller.example.com/alertmanager option.
For some reason alertmanager uses /alertmanager location even for internal listeners and as a result you receive time="2016-10-06T11:20:37Z" level=error msg="Error sending alerts: bad response status 404 Not Found" alertmanager="http://alertmanager:9093" count=3 source="notifier.go:308" error message in prometheus.
External and internal configuration options should be independent.

I believe this behaviour is correct. You told the alertmanager it'd only be accessed via a prefix, and then did not access it via that prefix.

@brian-brazil that's not correct. my prometheus ingress location has my-external-ingress-controller.example.com domain name. And this location serves several services including alertmanager. And when you define external-url you define external url, not internal.

The URL provided is what all external access to the AM is via. There is no internal AM http.

I ran into this issue earlier today and wanted to document my usecase:
I have prometheus and alertmanager deployed on a k8s cluster which is publicly accessible(not behind VPM/firewall). We use an ingress controller which
a) terminates TLS and
b) optionally forces OAuth authentication for users  to access various dashboards, internal pages etc.
So far it hasn't been a problem as most applications either have some form of robust auth on their own, or allow direct access to the backend unsecured(in which case we proxy it through oauth).
For the setup to work, I need the  -web.external-url flag because I want the users to get the correct URL in their alerts, but I do not want to expose the alertmanager API directly to the public. Unfortunately, there's no way of pointing prometheus to a external URL which will force the request to go through a custom authentication proxy.
What would be a good approach to resolve this issue? I'd be glad to submit a PR for separating the API and external URL if needed.

Unfortunately, there's no way of pointing prometheus to a external URL which will force the request to go through a custom authentication proxy.

That's configurable on the Prometheus side.

Related...
If I'm using -web.external-url=https://example.com/alertmanager  to reverse proxy to Alertmanager, the /metrics endpoint is also moved to /alertmanager/metrics, which is unfortunate from my perspective. A 302 to /alertmanager/metrics would be nice.

This is still an issue. I have a prometheus that is sending alerts to an alertmanager running on the same host, but an external display of the alerts comes in via a reverse proxy, the hostname of the reverse proxy is not visible to alertmanager or prometheus but I need this url in the alerts that are generated.
ie: I need to say alertmanager is here, but alert urls need to be generated to point to here instead

As mentioned in https://github.com/prometheus/alertmanager/pull/706

Actually starting with the 0.7.0 release the Alertmanager has the -web.route-prefix flag in addition to
the -web.external-url just like Prometheus, to be able to control the prefix of the internal http handlers
and how Alertmanager is accessed.

You should be able to use the -web.route-prefix for this requirement.

Sounds like this was incorrect usage.

I have a similar issue I believe.
If I use "--web.external-url" with an actual external url on my deployment, the external URL works for accessing, but Prometheus fails to communicate with alertmanager (alertmanager-0.alertmanager and alertmanager-1.alertmanager).
If I remove the "--web.external-url", Prometheus works properly, but the link on slack alert will point to either alertmanager-0.alertmanager or alertmanager-1.alertmanager, which are not accessible outside of kubernetes cluster.
  containers:
  - name: alertmanager
    image: prom/alertmanager:v0.12.0
    args:
      - '--config.file=/etc/alertmanager/config.yml'
      - '--storage.path=/alertmanager'
      # - '--web.external-url=http://myaddress.com:31080/alertmanager'
      - '--mesh.listen-address=:6783'
      - '--mesh.peer=alertmanager-mesh'

Is your prometheus communicating within the kubernetes cluster with the alert manager?

Yes @stuartnelson3 , I have a prometheus Deployment and an Alertmanager statefulset. The communication on network level is fine, I can ping services, name resolution works. The problem is if I set the '--web.external-url=http://myaddress.com:31080/alertmanager' parameter, being myaddress.com my cluster address, 31080 my ingress controller service port and /alertmanager my ingress path. If I set those values, seems like alertmanager will only listen in this path.
In this case, If I set my targets on Prometheus to - alertmanager-1.alertmanager:9093 (alertmanager-0 is the pod name, alertmanager is the service name), the connection will be refused by alertmanager.
If I set my targets on Prometheus to - myaddress.com:31080, it will build a path myaddress.com:31080/api/v1/alerts, it doesn't let me change to myaddress.com:31080/alertmanager/api/v1/alerts (if I could do it, I think it would work, but then I'd be sending alerts to the service and not to the pods as recommended on some articles.
If I remove this option '--web.external-url=http://myaddress.com:31080/alertmanager' from my alertmanager, Prometheus and Alertmanager work as they should, but then my URLs on Slack alerts will be directing to http://alertmanager-0.alertmanager:9093 which isn't available outside of Kubernetes (users for example).

myaddress.com:31080, it will build a path myaddress.com:31080/api/v1/alerts, it doesn't let me change to myaddress.com:31080/alertmanager/api/v1/alert

You want the path_prefix option.

@brian-brazil I used path_prefix like this on my Prometheus configmap and it didn't work (nothing happened actually). I even deleted the deployment and redeployed thinking that maybe the curl -XPOST myaddress.com:31080/prometheus/-/reload wasn't working properly (and it would be a different problem), but no... still doesn't work. My prometheus target keep being myaddress.com:31080/api/v1/alerts.
Am I setting it in the wrong place? Thank you so much
alerting:
alertmanagers:
- path_prefix: alertmanager
- scheme: http
static_configs:
- targets:
- myaddress.com:31080
- alertmanager-0.alertmanager.monitoring.svc.cluster.local:9093
- alertmanager-1.alertmanager.monitoring.svc.cluster.local:9093

The path prefix goes with the existing definition, not as a new alertmanager.

@brian-brazil Sorry, but I don't know where do you mean. Tried "randomly" putting this "path_prefix: alertmanager" on my documents and they didn't work. Also I can't find documentation or samples of path_prefix anywhere. Could you please be a bit more specific?

Should the path_prefix be prefixed with a /
- path_prefix: "/alertmanager"

@muniyalr made no difference. with or without "/", the configuration is shown the same on my mydomain:31080/prometheus/config
global:
scrape_interval: 15s
scrape_timeout: 15s
evaluation_interval: 15s
external_labels:
monitor: news360-monitor
alerting:
alertmanagers:

scheme: http
path_prefix: /alertmanager
timeout: 10s
static_configs:

targets:

mydomain:31080
scheme: http
timeout: 10s
rule_files:

/etc/prometheus-rules/prom.rules

@muniyalr @brian-brazil It seems to me that most adequate way would be to have Alertmanager accessible by its host address (alertmanager-0.alertmanager) AND from its external-url.
After all, if it is external-url, it should only matter to whoever is accessing it from a different network (like users). And other services on the same network (as Prometheus) trying to access Alertmanager on :9093 should just work. As I see, the external-url option shouldn't bind the service to this address.

As discussed above, that doesn't make sense. Syntax questions are best handled on the mailing list.

@brian-brazil How does it not make sense? Questions about the same thing on mailing list are open without answers since November last year. Clearly it is not just me having this issue and documentation about the path_prefix  you mentioned is nearly inexistent.

@muniyalr Fixed my problem as you suggested with "/", but at the end.
`
alerting:
      alertmanagers:
      - scheme: http
        path_prefix: "alertmanager/"
        static_configs:
        - targets:
            - alertmanager-0.alertmanager:9093
            - alertmanager-1.alertmanager:9093

`
Now it works

changing --web.external-url also affects the internal url if it contains a path. This create more problems than it fixes.


Fixed popup_menu buttons getting activated on press instead of release

Fixes #51778
The real issue is that popup_menu buttons activate on pressing the LMB whereas in 3.x they activated on releasing it.
This is (presumably) unintended, so I modified it to work the way it did in 3.x.
The behaviour described by the comment directly below my change is still present, though I am not sure why it's a thing.

Thanks!


http_path wrongly added to list of restricted CLI parameters

Hey all,
I wanted to transfer some settings into grunt but grunt-contrib-compass is blocking me from using http_path with config. Running compass compile -c config.rb --http-path /hello works.
Could it be it had been added to the raw options by mistake?
This is what my config.rb looks like:
# Directories
public_path = "build"
sources_path = "src"
project_path = File.expand_path(File.dirname(__FILE__))

css_dir = public_path + "/css"
fonts_dir = public_path + "/fonts"
generated_images_dir = public_path + "/images"
images_dir = public_path + "/images"
javascripts_dir = public_path + "/js"
sass_dir = sources_path + "/sass"
sprite_load_path = project_path + "/" + sources_path + "/sprites"

# URL
if http_path.nil?
    base_path = "/fallback"
else
    base_path = http_path
end

http_path = base_path
http_fonts_path = base_path + "/fonts"
http_generated_images_path = base_path + "/images"
http_images_path = base_path + "/images"

# OUTPUT
output_style = :compressed
environment = :production
sass_options = { :debug_info => false }
line_comments = false

Not a mistake as it were introduced after this task was made: https://github.com/Compass/compass/commit/85d383e7c24671b39adc150212f8e44d35a8e6bc


how to trace timeline on phalcon?

Hi, i cant view nothing on timeline :(
i put in my index.php:
include APP_PATH . 'vendor/autoload.php';

$profiler = new \Fabfuel\Prophiler\Profiler();
and the end of file:
after:
echo $application->handle()->getContent();
i put:
$toolbar = new \Fabfuel\Prophiler\Toolbar($profiler);
$toolbar->addDataCollector(new \Fabfuel\Prophiler\DataCollector\Request());
echo $toolbar->render();

too:
$di = new FactoryDefault();
$di->setShared('profiler', $profiler);

i dont know where put:
 $pluginManager = new \Fabfuel\Prophiler\Plugin\Manager\Phalcon($profiler);
 $pluginManager->register();

at the end of my index.php too?
Help!, what I'm missing?

Hi,
you should register the plugin manager as soon as possible:
$profiler = new \Fabfuel\Prophiler\Profiler();
$di->set('profiler', $profiler, true);

$pluginManager = new \Fabfuel\Prophiler\Plugin\Manager\Phalcon($profiler);
$pluginManager->register();

It's also important to inject the EventsManager manually to your dispatcher, view and db services, as they don't report to the events manager by default. When you set up e.g. your dispatcher service, please do:
$dispatcher->setEventsManager($di->get('eventsManager'));

Best
Fabian

Hi @fabfuel , thanks for your answers, i changed:
$di->setShared('profiler', $profiler);

for
$di->set('profiler', $profiler, true);
but i dont know where put:
 $dispatcher->setEventsManager($di->get('eventsManager'));

Help please

who is $dispatcher ?, sorry but iam new on phalcon...... :(

help  @fabfuel :(

Here you find all the infos about the Phalcon dispatcher:
http://docs.phalconphp.com/en/latest/reference/dispatching.html
In you services.php (or what ever place you define your services) do:
$di->set('dispatcher', function() use ($di) {
    $dispatcher = new \Phalcon\Mvc\Dispatcher();
    $dispatcher->setEventsManager($di->get('eventsManager'));
    return $dispatcher;
}, true);

thanks a lot", this works!


feat: replace regex with provider function

This change will go for a more native approach on splitting on names, instead of using regex we now use a new function from azurerm to split resources.
https://www.hashicorp.com/blog/terraform-azurerm-provider-4-0-adds-provider-defined-functions

Looks good to me


Create additional info blocks

You can make your resume less official by adding more entertaining info, like your hobbies and interest, or what do you like, some interactive content or joke, easter egg, etc.

Additional block is fullfiled with info about education and languages


[BUG] SAML logout don't work

Describe the bug
We using Microsoft ADFS as SAML authentication. Login woks fine. But logout doesn't. While logout user in browser see error: "An error occurred. Contact your administrator for more information". On SAML server side we see: "MSIS7084: SAML logout request and logout response messages must be signed when using SAML HTTP Redirect or HTTP POST binding", but signature in our configuration is disabled.
To Reproduce
Steps to reproduce the behavior:

Configure OpenSearch for using SAML auth
Configure Dashboards like this:

opensearch.requestTimeout: 240000
opensearch.pingTimeout: 6000
opensearch.hosts:
- https://hostname:9200
opensearch.username: opensearch_dashboards
opensearch.password: password
opensearch.ssl.certificateAuthorities:
- "/path/to/ca"
logging.dest: "/var/log/opensearch-dashboards/opensearch-dashboards.log"
opensearch.logQueries: false
logging.verbose: false
opensearch_security.ui.basicauth.login.title: Login message
opensearch_security.auth.type:
- basicauth
- saml
opensearch_security.auth.multiple_auth_enabled: true
server.xsrf.allowlist:
- "/_opendistro/_security/saml/acs/idpinitiated"
- "/_opendistro/_security/saml/acs"
- "/_opendistro/_security/saml/logout"

Login
Logout
See error

Expected behavior
When the user clicks logout button it is redirected to login dialog without any errors.
OpenSearch Version
2.11.1
Dashboards Version
2.11.1
Plugins
Defaults.
Screenshots
No.
Host/Environment (please complete the following information):

OS: CentOS Linux release 7.9.2009
Browser and version: Brave [Version 1.62.165 Chromium: 121.0.6167.184 (Official Build) (64-bit)]

Additional context
This issue previously discussed here and have workaround don't know how correct this is but it works.

@derek-ho, link for forum thread in description, if you can propose a solution without editing the code it will be great. Waiting you on forum.

Create new topic here: https://forum.opensearch.org/t/saml-logout-dont-work/20347


make DevToolsRpc versions more useful

@swbsi if you get a chance, can you take a look at this?

@swbsi if you get a chance, can you take a look at this?

I will give it a try today or Monday.

@pmconne mind taking a pass?


main function documentation is unclear. needs an example

"accepts a function you built" isn't clear. An example is needed to demonstrate the syntax for using main.

I'll get one added!

Example added: https://github.com/OthersideAI/chronology/blob/main/README.md#example


Training time finished without completing a successful trial. Either no trial completed or the metric for all completed trials are NaN or Infinity

Training time finished without completing a successful trial. Either no trial completed or the metric for all completed trials are NaN or Infinity
at Microsoft.ML.AutoML.AutoMLExperiment.RunAsync(CancellationToken ct)
at Microsoft.ML.ModelBuilder.AutoMLService.Experiments.MultiClassificationExperiment.ExecuteAsync(IDataView trainData, IDataView validateData, CancellationToken ct) in //src/Microsoft.ML.ModelBuilder.AutoMLService/Experiments/MultiClassificationExperiment.cs:line 140
at Microsoft.ML.ModelBuilder.AutoMLEngine.StartTrainingAsync(ITrainingConfiguration config, PathConfiguration pathConfig, CancellationToken userCancellationToken) in //src/Microsoft.ML.ModelBuilder.AutoMLService/AutoMLEngineService/AutoMLEngine.cs:line 178

same Issue, I have follow up the same steps here (https://dotnet.microsoft.com/en-us/learn/ml-dotnet/get-started-tutorial/train) on my project and got the same error.


Issue with binary sampler

Hi, I encountered an issue with the binary sampler. That is, it gives the wrong result. The slice sampler does give the correct result. Thoughts?
library(nimble)

# the model: is the mean b[1] or b[2]?
modelBern <- nimbleCode(code = {

	indicator ~ dbern(0.5)
	mu <- indicator * b[1] + (1 - indicator) * b[2]
	for (i in 1:n) {

		x[i] ~ dnorm(mu, 1)

	}

})

# simulate data
set.seed(31415)
b = c(1e3, 5)
n = 1e3
x = rnorm(n, b[2], 1) # mean is b[2], indicator should be 0.
mean(x) # 5.01399

rModel <- nimbleModel(code = modelBern,
				  data = list(x = x),
				  constants = list(b = b, n = n),
				  dimensions = list(b = 2),
				  calculate = TRUE)
MCMCconf <- configureMCMC(model = rModel)
MCMCconf$printSamplers()

bMCMC <- buildMCMC(conf = MCMCconf)
cModel <- compileNimble(rModel)
mcmcObjBern <- compileNimble(bMCMC, project = cModel, resetFunctions = TRUE)

samplesBern <- runMCMC(mcmc = mcmcObjBern, niter = 1e3, nchains = 2, nburnin = 1e2, returnCodaMCMC = TRUE)
plot(samplesBern) # should be 0, but it is 1
colMeans(as.matrix(samplesBern)) # should be 0, but it is 1

# now with slice sampler

MCMCconf <- configureMCMC(model = rModel)

MCMCconf$removeSamplers("indicator")
MCMCconf$addSampler(target = 'indicator', type = 'slice')
MCMCconf$printSamplers()

bMCMC <- buildMCMC(conf = MCMCconf)
mcmcObjBern <- compileNimble(bMCMC, project = cModel, resetFunctions = TRUE)

samplesBern <- runMCMC(mcmc = mcmcObjBern, niter = 1e3, nchains = 2, nburnin = 1e2, returnCodaMCMC = TRUE)
plot(samplesBern) # should be 0 and is 0
colMeans(as.matrix(samplesBern)) # should be 0 and is 0

@vandenman thanks for providing a reproducible example.  We'll look into this soon, and get back to you.

This issue results from numerical underflow.  A proposed fix is in pull request #387.  For immediate purposes @vandenman you can install from branch fixIssue386_binary_sampler

Thanks for the quick fix! I did some tests and the branch fixIssue386_binary_sampler seems to solve my problem.

This is now on branch devel and will be released in version 0.6-5 soon (next week)


sdk(dotnet): Use same Apache 2.0 license as all other SDKs

Engine & CLI use the same Apache 2.0 license.
If you agree with this change @nabsul, please:

edit this commit message & remove this paragraph
append your Signed-off-by
commit & force push

Superseded by https://github.com/dagger/dagger/pull/5788. Closing this PR.