Filter issue (no result at all, or weird behaviour)

Problem Description
I implemented the filter function, letting the normal filter method by default. However, on one column, the filter doesn't seem to work, and on another, the filter gives no result back.
My table is composed as the following:

One column is used as a checkbox, the value passed in it is a file's full path. This is the value contained in my database.
The one next to also uses the path, but filters it to only keep the filename.
The last one is the date when the file was added.

(there are two more columns, but I have the same problem with each of them, so it's useless to detail them)
If I do a console.log of this.state.filtered, I do obtain what I write in my filters. However, if I disable the "filterable' on the checkbox column, nothing happens if I type something in the "filename" column.
Code Snippet
const columns = [{
    Header: ' ',
    accessor: 'path',
    // filterable: false,
    Cell: row => (
        <input
            type="checkbox"
            name="file_selected[]"
            value={row.value}
        />
    )
}, {

    Header: 'File',
    accessor: 'path',

    sortMethod: (a, b) => { // functional
        let reA = /[^a-zA-Z]/g;
        let reN = /[^0-9]/g;
        let afile = a.split('/').pop();
        let bfile = b.split('/').pop();
        let aA = afile.replace(reA, "");
        let bA = bfile.replace(reA, "");

        if(aA === bA) {
            let aN = parseInt(afile.replace(reN, ""), 10);
            let bN = parseInt(bfile.replace(reN, ""), 10);

            return aN === bN ? 0 : aN > bN ? 1 : -1;
        } else {
            return aA > bA ? 1 : -1;
        }
    },

    Cell: row => {
        let filename = row.value.split('/').pop();

        return
{filename}

    }

}, {
    Header: 'Tags',
    accessor: 'tags',
    Cell: row => {
        /* @Todo: Mettre en place ce système de tag par array */
        let tags = row.value.replace(/[^a-zA-Z0-9]/g, "");

        return
{tags}

        /*{ tags.split(' ').map(function(tag, i) { // For now, renders a string
            return (
                { tag }
            )
        })}*/
    }
}, {
    Header: "Date d'ajout",
    accessor: 'created_at',
    sortMethod: (a, b) => {
        return a > b ? 1 : -1;
    },

    Cell: row => {
        let date = new Date(row.value);
        return (

{ date.getDate() + '/' + (date.getMonth()+1) + '/' + date.getFullYear() }

        )
    }
}];

// ...

render() {

        console.log(this.state.filtered);

        return (
            <form action="/download/zip">
                <ReactTable
                    data={this.state.data}
                    noDataText='Pas encore de fichier'
                    columns={columns}
                    filterable
                    filtered={this.state.filtered}
                    onFilteredChange={filtered => this.setState({filtered})}
                    defaultFilterMethod={(filter, row) =>
                        String(row[filter.id]) === filter.value}
                />
                <input type="submit" value="Télécharger" />
            </form>
        )
    }

System Information

Browser & Browser Version: Chrome - 59.0.3071.115
Node Version: 8.1.2
OS Version: Ubuntu 16.04
NPM Version: 5.3
Yarn Version: 0.27.5

@aaronschwartz Can you see anything obvious in this code example?

You may be having issues because columns have the same accessor without defining an id field. Try and set a unique id on your columns with matching accessors. Otherwise it would be helpful to re-create it in Codepen if possible.

Good find @aaronschwartz, we should add a check for that.

As far as I can tell, this should work (and has worked when I try it locally).  @Drillan767, could you please create a codepen replicating this functionality? Closing this for now for triaging. Please open when you have the pen. :)

Hello, I'm sorry I couldn't answer earlier, I was abroad and without computer, so I couldn't do much myself.
So I created a pen but failed to make it work like you did in your example (even though I added React-Table.
I also added some actual datas that I get from my own database, and tried for my part to add this 'id' you guys were talking about, but in vain, even though I check your documentation to have an idea about how to do it.
So far, even with a few ids, my issue move slightly : now if I try to write anything in any searchbox, the result is always empty (before this, nothing happened when stuff was written in the "file" column searchbox). Here is my github file so you have a better idea of what I'm talking about.
thank you guys for your help so far!

Fork this codepen and modify it. It is already preloaded with React-Table (including styles) and the simple example. That might get us closer to the problem.

Ok here is the the new codepen with id added at each column, no change so far

The filter is based on the original 'value' of the row. So if you start typing /uploads/ you'll see the filter working since the default filter is .startsWith(...). You can override the filter method for that column to change it if desired. Or you could make the accessor a function that does the split and then the value will be just the filename.

@aaronschwartz Bingo!

Wow you're right! So if I follow you, I need to override the "startsWith()" function with my own filters for every columns? Since it looks like the problem is the same for each of my columns?

You can set the global defaultFilterMethod for the entire table to .includes() if you want.

Ok I must admit I have some trouble finding out the correct way to replace the .startWith() method: I've been trying a few things but I don't really know what to replace the default filter with : I can't manage to find a way to say that the filter is equal to what is rendered in the column. For example, I'd like to be able to filter among the row.value.split('/').pop(), but I can't manage to do correctly the following
Thank you in advance

Is this what you're trying to do? I used both methods I outlined. The filename uses a custom filterMethod and the tags uses a custom accessor function.
https://codepen.io/aaronschwartz/pen/KvgXpz?editors=0010

Ok, thank you a lot, I could put this in my own code and it woked like a charm, the creation date won't work for now but I think i'll make it work through a filterMethod. Thank you both a lot!


CosmoDB pricing

Please don't just present CosmoDB as a 'premium version' of tables. It's probably hundreds of orders of magnitude more expensive. $23.10 minimum per month for a single table / collection. I'm now sad and annoyed this wasn't more obvious.
'Scalable services' should mean you can start with 'tens and hundreds', and not just that you can go from 'millions to billions' with ease..

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

ID: 83df0ac8-6349-4ea5-4698-fdf00c69c92f
Version Independent ID: 5afd32d5-5585-c495-b68d-7d68963b6b3d
Content: Introduction to Table storage - Object storage in Azure
Content Source: articles/storage/tables/table-storage-overview.md
Service: storage
GitHub Login: @SnehaGunda
Microsoft Alias: sngun

@simeyla Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@simeyla apologies for delay in getting back to you. Thanks for your feedback on the pricing, we are working on improving our Pricing model and the RUs offered. Please expect to see some announcements and doc updates in by end of this month.

#please-close

@SnehaGunda Great to hear :-) I'd love to see some kind of entry-level offering so I can play with it and see if it is a fit for us.


Ready to publish v1.5.5

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

Comparison is base (f5283f5) 100.00% compared to head (d4faf56) 100.00%.
Report is 2 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #17   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           11        11
  Lines          283       279    -4
  Branches        16        16
=========================================
- Hits           283       279    -4

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


OpenAPI and JSON Schema don't support tuple declarations

In this example, the parts property is declared as [Part]. This is incorrect syntax for an array -- it should be Part[] -- but the compiler silently ignores the problem and generates an empty schema for itmes.
Playground Link

I guess we don't have a good support for tuples in openapi3 and not sure what we should generate here. In this specific case we could treat it as an array of part with length 1 but if you lets say have [A, B] it is a bit more complex.

I thought I recalled implementing this in JSON Schema, but I don't see that it's supported (crash in #2460 aside). I think a reasonable representation of tuples in JSON Schema is using an array type with a fixed length and each element's schema defined with prefixItems. I don't know if prefixItems is reasonable for OpenAPI 3.0 though, thoughts @mikekistler?

I don't think it is an option for openapi3 but for 3.1 it does seems to be the way
https://stackoverflow.com/questions/57464633/how-to-define-a-json-array-with-concrete-item-definition-for-every-index-i-e-a

I guess I didn't realize that this was valid syntax. This represents a tuple? Where are the docs for tuple?

Looks like they are not documented under language basics and should be. They come up for various decorators which expect them, however.

Separate issues for OpenAPI. - 5
Fixing EF to not crashing (track in other issue)
Supporting tuple declaration in JSON Schema - 5

For 3.0 we'd have to make it a oneOf all the options and of the length of the tuple

I think that for 3.0 it should just be an error.

Feels a little strong to me to be an error, specially in cases where you have something like
alias Point2D = [float64, float64];

this can be acurrately represented with
type: array
items:
  type: number
  format: double
minLength: 2
maxLength: 2

My vote would be:

If all the types in the tuple are the same, represent it as above with no error
If types are different emit a warning and use oneOf

For your last bullet, just to be clear, are you saying the items schema is anyOf the tuple types? If so, I think that sounds reasonable. the oneOf above seems cool but I suspect the codegen from that wouldn't be pretty.
@mikekistler can you say more about why an error is appropriate here? I think in general we should strive to emit something for all TypeSpec unless there's a very good reason not to.

Yeah for the 2 point I would say that if you have
alias Stuff = [float64, string];

you get the following openapi3 + a warning
type: array
items:
  anyOf:
     - type: number
       format: double
     - type: string
minLength: 2
maxLength: 2

Also, just want to point out that tuples are not exactly that exotic. E.g. GeoJSON defines Position as an array of exactly length 2 or 3, where the indexes correspond to long, lat, and optionally elevation, respectively. A type like:
model Geo {
  coordinates: [float64, float64] | [float64, float64, float64]
}

would give better codegen for languages that support such types (and would be even more terse if we supported optional tuple elements, which I imagine we will one day!), over something like:
model Geo {
  @minLength(2)
  coordinates: float64[];
}

@timotheeguerin I feel like we should leave the maxLength out actually, that feels like as if we added additionalProperties: false to every schema... The tuple demands you provide at least those elements, any extra elements might be sent and likely ignored?

I guess but also if you were to look at the openapi/jsonschema for that and not see the maxLength it would look like any array and not sure you would ever regenerate a tuple seeing that

In practice I haven't seen anyone generating tuples from openAPI with length constraints, though I haven't looked super broadly. It would need to be heuristic though, the tuple type for @minLength(1000) @maxLength(2000) would be ridiculous 😂


Add files via upload

@leecarus First stage of verification passed, official domain is jiu.ac as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@leecarus Pull request merged. Thank you.


⚠ Bofrost.es has degraded performance

In 98c3cc7, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1567 ms

Resolved: Bofrost.es performance has improved in b02f748 after 10 minutes.


Conda switch to gcc7 and inherent conda-forge environment solvability

DO NOT CLOSE THIS UNTIL WE ARE FULLY BACK TO conda-forge
To monitor: issue on conda/issues: https://github.com/conda/conda/issues/8413

Solved by #1041


AR.js Slam or markerless

I have reviewed many issues or feature requests , and followed your progress on documenting and adding slam feature in your trello . but no where found any solutions for this .
Is this feature ( slam ) or markerless tracking developed and usable in AR.js ?
thank you very much for your supporting

you can use goelocation tracking AR.js
or just show the object according to your scale and position
``

Repo has been moved. Please open issue here: https://github.com/AR-js-org/AR.js/issues


Delete transient tags on newly encrypted volume

Existing process was not cleaning up transient tags. Added code to delete transient maid tags left behind on newly encrypted volume.

lgtm, but it feels like we need to get a recording of just the delete tags to insert into the extant placebo dir to cover this. also curious why the extant test wouldn't error without a recording of the delete tags.

We're in process of refactoring the test case. We'll let you know when we're ready for final review.

the easiest thing might just be dropping in a delete tags recorded response into the test directory, there's a good amount of setup involved, that needs sanitizing. ping me on gitter if you'd like to discuss additional options.

@kapilt, do you have a tool for sanitizing the placebo data we're recording?

Hey @kapilt, I think we're ready for another review here. The test case has been improved to verify that the resulting volume is encrypted, delete on termination matches the source, and the tags created during the process are removed.

Anything more to be done here?


Desktop examples use the deprecated mediapipe::GlCalculatorHelper::CreateSourceTexture

This is kind of between documentation and bug issue.
The Desktop examples use CreateSourceTexture. Using these, I get the warning like:
warning: ‘mediapipe::GlTexture mediapipe::GlCalculatorHelper::CreateSourceTexture(const mediapipe::ImageFrame&)’ is deprecated: Use 'GpuBufferWithImageFrame'
I haven't figured out how to use GpuBufferWithImageFrame, and I can't find it being used in tests anywhere (are GPU things even tested?).
I think update of examples demonstrating proper usage is warranted.

Hi @Algomorph,
Could you please provide more details about your query to help us better understand the issue?
Thank you!!

@kuaashish what more details do you need?
MP examples are themselves using deprecated MP code.

@kuaashish any update on this?


Fishing net : text layout bug

Minecraft version & Mod Version: 1.12.2-1.10.2
Single player or Server: both

https://minecraft.curseforge.com/projects/cyclic/files/2508261


Clean up repo

Removing example and unused contracts to clean up repo before audits begin
Renamed contracts to src
A branch tracking these examples has been pushed called example-contracts

Hey guys, just curious to know why did we remove the hook examples?

Hey guys, just curious to know why did we remove the hook examples?

@heypran pretty sure it had to do with getting an audit. They were temporarily moved to another branch


Getting error while tower-cli ssl 590 protocol

Unable to run tower-cli commands against latest version of ansbile-tower <IP_ADDRESS> from vagrant image https://app.vagrantup.com/ansible/boxes/tower.
Any help would be greatly appreciated.
`
(ansible2) mcgonagle@macbook:~/Dropbox/ansible_f5> sudo tower-cli user list --insecure -v
*** DETAILS: Getting records. *************************************************
GET https://localhost:10443/api/v2/users/
Params: {'page': 1}
SSL connection failed:
HTTPSConnectionPool(host='localhost', port=10443): Max retries exceeded with url: /api/v2/users/?page=1 (Caused by SSLError(SSLEOFError(8, u'EOF occurred in violation of protocol (_ssl.c:590)'),))
Error: Could not establish a secure connection. Please add the server to your certificate authority.
You can run this command without verifying SSL with the --insecure flag, or permanently disable verification by the config setting:
tower-cli config verify_ssl false
(ansible2) mcgonagle@macbook:~/Dropbox/ansible_f5> sudo tower-cli config
User options (set with tower-cli config; stored in ~/.tower_cli.cfg).
username: tom
password: redacted
host: https://localhost:10443
verify_ssl: False
description_on: True
verbose: True
format: human
color: True
use_token: False
Defaults.
certificate:
`

Are you using a custom certificate? I can't fully tell from the output above.

@AlanCoding I am using the default certificate that comes with Ansible-Tower Vagrant image.

This looks like you hit the same error as was seen in #122. I also have the gut feeling that I've seen this when I put in https while the server was actually serving http. I would compare to what URL you see in a browser. I would also probably avoid setting verify_ssl to False, and specifying a cert path at the same time. @jangsutsr, does anything other configurable here ring a bell for you?

Tried to follow the following directions to download the cert so I could specify your suggestion to use cert path instead. And got the following error:
https://superuser.com/questions/97201/how-to-save-a-remote-server-ssl-certificate-locally-as-a-file
(ansible3) mcgonagle@macbook:~> openssl s_client -connect localhost:10443 -showcert </dev/null 2>/dev/null|openssl x509 -outform PEM >mycertfile.pem
unable to load certificate
41861:error:0906D06C:PEM routines:PEM_read_bio:no start line:/BuildRoot/Library/Caches/com.apple.xbs/Sources/OpenSSL098/OpenSSL098-64.50.6/src/crypto/pem/pem_lib.c:648:Expecting: TRUSTED CERTIFICATE

Tried the following. No luck:
(ansible3) mcgonagle@macbook:~/Dropbox/Tower2> tower-cli config
/Users/mcgonagle/ansible3/lib/python2.7/site-packages/tower_cli/conf.py:57: RuntimeWarning: File /Users/mcgonagle/.tower_cli.cfg readable by group or others.
.format(fpname), RuntimeWarning)
User options (set with tower-cli config; stored in ~/.tower_cli.cfg).
username: tom
password: Ilv2xlR8!!
host: localhost:10443
verify_ssl: False
description_on: True
verbose: True
format: human
color: True
use_token: False
certificate: /Users/mcgonagle/Dropbox/Tower2/tower.cert
(ansible3) mcgonagle@macbook:~/Dropbox/Tower2> tower-cli user list -v
/Users/mcgonagle/ansible3/lib/python2.7/site-packages/tower_cli/conf.py:57: RuntimeWarning: File /Users/mcgonagle/.tower_cli.cfg readable by group or others.
.format(fpname), RuntimeWarning)
*** DETAILS: Getting records. *************************************************
GET https://localhost:10443/api/v2/users/
Params: {'page': 1}
SSL connection failed:
HTTPSConnectionPool(host='localhost', port=10443): Max retries exceeded with url: /api/v2/users/?page=1 (Caused by SSLError(SSLEOFError(8, u'EOF occurred in violation of protocol (_ssl.c:590)'),))
Suggestion: add the correct http:// or https:// prefix to the host configuration.
Error: Could not establish a secure connection. Please add the server to your certificate authority.
You can run this command without verifying SSL with the --insecure flag, or permanently disable verification by the config setting:
tower-cli config verify_ssl false

Created a new tower.cert and tower.key, restarted nginx. No luck still getting the same error.

part of this is that a default tower install will use the self sign cert that is not signed by a CA and this connection tries to verify those certs. What I have done is change the host from https to http. I did not change the verify_ssl but, it still shows up as false. What version of tower-cli are you using? I will test this on my end.
tower-cli config

# User options (set with `tower-cli config`; stored in ~/.tower_cli.cfg).
host: http://<IP_ADDRESS>
username: admin
password: password
verify_ssl: False

# Defaults.
description_on: False
verbose: False
certificate:
format: human
color: True
use_token: False

just tried it and it worked for me fine on the latest vagrant.
tower-cli version Tower CLI 3.2.0 Ansible Tower <IP_ADDRESS>
`
==== ============ =================
 tower-cli config
User options (set with tower-cli config; stored in ~/.tower_cli.cfg).
host: https://<IP_ADDRESS>
username: admin
password:
verify_ssl: True
Defaults.
description_on: False
verbose: False
certificate:
format: human
color: True
use_token: False
`

@jlozadad Thanks so much for trying this out. I am afraid I can't access my vm on <IP_ADDRESS>. I follow the directions to create a Vagrantfile with vagrant init ansible/tower, but the tower instance is unavaialble on https://<IP_ADDRESS>. The only way to access it is to specifically enable config.vm.network "forwarded_port", guest: 443, host: 10443
And then connect to https://localhost:10443. I tbelieve this is the crux of the problem. Could you share your Vagrantfile and what you are doing to be able to connect to the VM at <IP_ADDRESS>?

I used what is on the ansible download page
$ vagrant init ansible/tower $ vagrant up --provider virtualbox $ vagrant ssh
that is the IP that vagrant gives me.


Teams Functional Testing

💡 Summary
As part of Glacier release, to ensure that the tool's update is accurate, detailed functional testing has been planned. Each policy will be functionally tested to check that all pass/fail scenarios have been correctly processed by the tool using existing automated functional test plans.
This issue functionally tests each policy of Teams product of ScubaGear tool. The main objective is to test Teams policies and ensure that ScubaGear tool is providing the correct result for each possible configuration option.
Motivation and context
This functional testing ensures that ScubaGear results of pass/fail are accurate based on the latest version of the code.
Implementation notes
For each Teams policy do the following:

[ ] Configure the automated testing and run against each tenant type for which a variant test plan is present.  Run the automated test plans (e.g. regular, g5, etc.) and note the results.
[ ] For each test failure, investigate and note the cause of the failure and log any new issues needed to resolve

For details on how to use and execute the functional tests, please see #592 or reach out to the dev team via other channels.
Acceptance criteria

[ ] All existing test plan variants for the product have been successfully run
[ ] Test plan results have been reviewed and any issues resolved or reported as new issues

Teams functional testing on G5 tenant - all tests (non-complaint and complaint) passed:

Teams functional testing on G3 tenant - all tests passed:

Teams functional testing on GCCHIGH - all functional tests passed. There is an open issue #1059 to further investigate the warning.

Additional manual tests on E5, E3 tenants all passed. Closing this task as completed.


[Feature] Change password

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
由于安全问题,需要增加诸如修改密码,动态验证码(或者滑动)等 防止安全扫描
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[x] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Currently, this feature is not available, and we would be happy if you could contribute it


unit tests: basic unit tests for pmem

Adding some basic unit test for PMEM

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Interaction count starts at zero instead of one

It's a problem that {{UserInfo.InteractionCount}} starts at zero only after your first question. So if I have:
So far, you have interacted with me {{UserInfo.InteractionCount}} times.
in my answer block, after my first question "what color is the sky" it will give the answer, "blue," and repeat:
So far, you have interacted with me 0 times.
Which is obviously not correct. What can I do for this? I want to give users a running total of how many questions they have asked because I want them to limit themselves to 10.

Hi @tommcgee57  - thanks for logging the issue. I staged a fix - if you are building from source you can try it now from the fix branch, otherwise it will be in our next release.

Thanks, Robert.
Unfortunately, I’m trying to implement this through the AWS dashboard and I have absolutely no idea where these files are stored or how to get at them to make the fix. I’ll watch for the next version to come out.
Best,
Tom
From: Robert<EMAIL_ADDRESS>Sent: Thursday, December 10, 2020 8:51 PM
To: aws-samples/aws-ai-qna-bot<EMAIL_ADDRESS>Cc: Thomas A McGee<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [aws-samples/aws-ai-qna-bot] Interaction count starts at zero instead of one (#235)
Hi @tommcgee57https://nam05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftommcgee57&data=04|01|Thomas.McGee%40shu.edu|42ff1a00320c4445a57108d89d773739|51f07c2253b744dfb97ca13261d71075|1|0|637432482648780301|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=CgUkh%2BbeTu4xI9ND1W6Fboi93HJkb%2F71RFxaMzXu36s%3D&reserved=0 - thanks for logging the issue. I staged a fix - if you are building from source you can try it now from the fix branch, otherwise it will be in our next release.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Faws-samples%2Faws-ai-qna-bot%2Fissues%2F235%23issuecomment-742912354&data=04|01|Thomas.McGee%40shu.edu|42ff1a00320c4445a57108d89d773739|51f07c2253b744dfb97ca13261d71075|1|0|637432482648780301|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=SNFa5IZZayfuM87T4GPd2fDpulE6murZgpwyKA%2Bxrt8%3D&reserved=0, or unsubscribehttps://nam05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAABMXW4L7KPK2LXNX26KPGTSUF3ILANCNFSM4UVXLM2Q&data=04|01|Thomas.McGee%40shu.edu|42ff1a00320c4445a57108d89d773739|51f07c2253b744dfb97ca13261d71075|1|0|637432482648790301|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=p9CTxdpbm3qcoX3kosvYfJBZPdJEUXT0dzAUxp%2FXvrs%3D&reserved=0.
** WARNING: This email originated from outside of Seton Hall University. Do not click links or open attachments unless you recognize the sender and know the content is safe. **


[WIP] Verify codegen

What do these changes do?

Related issue number

Fixes

Closed since not needed.


add an inline view mode

Use crossterm and inline viewport
Reposition cursor on exit and fix The cursor position could not be read within a normal duration errors due to blocking reads

Duplicate of #331

That's the plan 🙂

Left a comment on https://github.com/ellie/atuin/pull/331/files#r1100358866 to determine the best way to backport https://github.com/fdehau/tui-rs/pull/552.

The diff is now so smol 😄

Awesome, thanks @conradludgate!
I'll try and take another look at finishing this PR ASAP. Feel free to to hack on it if you like :)

Looks like this is actually working great in its current state.
I guess what's left to do is to make this new behaviour optional, isn't it?

Looks like this is actually working great in its current state.
I guess what's left to do is to make this new behaviour optional, isn't it?
Also, being able to configure the number of lines would be great too.

It does! I've been using it since last night. Started to make it configurable but my parents are visiting this weekend so haven't finished 🤣

Make it optional
Configure the number of lines
Release! 🚀

I think it would also feel a bit better if the search box was at the top for inline mode, but that can wait - I'd like to make the whole UI more configurable

Added an inline_height setting which can be used to define the number of lines to use in inline mode.
When setting it to zero (the default), inline mode is disabled.
Maybe the naming could be improved.
@conradludgate @ellie WDYT?

Started to make it configurable but my parents are visiting this weekend so haven't finished

@ellie oops, read that part of your reply too fast, I didn't realize you actually started implementing the setting, sorry about that.
Feel free to scratch my version if you prefer yours.

Working on putting the input chunk at the top when inline mode is active...

We can merge as is, I like it. We can fix the UI later. I've been daily driving for a few days and it still feels pretty nice

Good for me! :)
Submitted #808 as a draft, will have to come back to it at a later time.

No worries! Thank you for your efforts. They are really appreciated


RN 0.29 android config

With the latest changes, the getPackages method has been moved to an application class, and are no longer in the main activity. As such, this call is no longer possible:
new FIRMessagingPackage(getIntent())
How do I set up react-native-fcm on android when using RN 29?

working on it

1.0.12 released, please try it out.
new FIRMessagingPackage(getIntent()) is no longer needed

UPDATE: still working on it. React native is trying to prevent modules from fetch and store activity data in constructor. So I should follow the direction and provide initial data in another way

UPDATE: new fix pushed please try

I can build now, which is great 👍
However, FCM.getToken() returns null in the promise.
Logcat output:

07-12 04:26:49.111  2772  2817 W InstanceID/Rpc: Failed to resolve REGISTER intent, falling back
07-12 04:26:49.112  2772  2817 W InstanceID/Rpc: Both Google Play Services and legacy GSF package are missing

...

07-12 04:26:59.118  2772  2845 D FirebaseInstanceId: background sync failed: MISSING_INSTANCEID_SERVICE, retry in 20s
07-12 04:26:55.891  2772  2812 D com.evollu.react.fcm.FIRMessagingModule: Firebase token: null

My setup:
App build.gradle
// Top-level build file where you can add configuration options common to all sub-projects/modules.

buildscript {
    repositories {
        jcenter()
    }
    dependencies {
        classpath 'com.android.tools.build:gradle:2.1.2'
        classpath 'com.google.gms:google-services:3.0.0'
        // NOTE: Do not place your application dependencies here; they belong
        // in the individual module build.gradle files
    }
}

allprojects {
    repositories {
        mavenLocal()
        jcenter()
        maven {
            // All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
            url "$rootDir/../node_modules/react-native/android"
        }
    }
}

Manifest
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
    package="..."
    android:versionCode="1"
    android:versionName="1.0">

    <uses-permission android:name="android.permission.INTERNET" />
    <uses-permission android:name="android.permission.SYSTEM_ALERT_WINDOW"/>
    <uses-permission android:name="android.permission.WAKE_LOCK" />
    <uses-permission android:name="android.permission.VIBRATE" />

    <uses-sdk
        android:minSdkVersion="16"
        android:targetSdkVersion="22" />

    <application
        android:name=".MainApplication"
        android:allowBackup="true"
        android:label="@string/app_name"
        android:icon="@mipmap/ic_launcher"
        android:theme="@style/AppTheme">
        <activity
            android:name=".MainActivity"
            android:label="@string/app_name"
            android:screenOrientation="portrait"
            android:configChanges="keyboard|keyboardHidden|orientation|screenSize"
            android:launchMode="singleTop" >
            <intent-filter>
                <action android:name="android.intent.action.MAIN" />
                <category android:name="android.intent.category.LAUNCHER" />
            </intent-filter>
            <intent-filter>
                <action android:name="fcm.ACTION.HELLO" />
                <category android:name="android.intent.category.DEFAULT" />
            </intent-filter>
        </activity>
        <activity android:name="com.facebook.react.devsupport.DevSettingsActivity" />
        <service android:name="com.evollu.react.fcm.MessagingService">
            <intent-filter>
                <action android:name="com.google.firebase.MESSAGING_EVENT"/>
            </intent-filter>
        </service>

        <service android:name="com.evollu.react.fcm.InstanceIdService" android:exported="false">
            <intent-filter>
                <action android:name="com.google.firebase.INSTANCE_ID_EVENT"/>
            </intent-filter>
        </service>
    </application>

</manifest>

Also added the following to MainActivity
@Override
    public void onNewIntent(Intent intent){
        setIntent(intent);
    }

and of course it is added to getPackages in MainApplication, and the downloaded and added the google-services.json file to the project.

@evollu: It still bugs with RN 0.29.
Please check below the compatibility table.

FCM
getIntent()*
RN 0.28
RN 0.29

1.0.9
:white_check_mark:
:white_check_mark:
:x:

1.0.9
:x:
:x:
:x:

1.0.13
:white_check_mark:
:x:
:x:

1.0.13
:x:
:white_check_mark:
:x:

*This column refers to the presence of getIntent() in new FIRMessagingPackage()
**:x: for RN versions means that the build failed
A screenshot when the build fails:

you still need to follow the upgrade instructions in rn 0.29 https://github.com/facebook/react-native/releases

@evollu: Oh right, I completely missed it, thank you !
Works fine (Build + Notifications) when following the RN 0.29 upgrade instructions.
As it also works for @sorenburonn we can close the issue :)


Improve pod communication

Yet, are being added pod names to /etc/hosts.conf from each pod to each pod. Since Kubernetes has a DNS feature, it must be implemented in order to improve readability and autonomy when instantiating the HDFS cluster in Kubernetes.

Fixed this issue by changing the datanodes kind: Deployment to kind: StatefulSet. This happens because a StatefulSet creates one DNS entry for each pod created by the object, instead of the Deployment.
Related commit: https://github.com/hrchlhck/k8s-bigdata/commit/93b1564f528e08f5501adbbb956763db80031c5e


Airflow 1.10.2

https://github.com/apache/airflow/blob/master/UPDATING.md#new-dag_processor_manager_log_location-config-option

Other than this section, the rest LGTM.


AzureADPreview – Privileged Role Management commands don't work in Azure China environment

Azure Environment: Azure China
AzureADPreview Version:
C:\windows\system32> Get-Module -name AzureADPreview | fl
Name              : AzureADPreview
Path              : C:\Users\wesu\OneDrive - Microsoft\Documents\WindowsPowerShell\Modules\AzureADPreview\<IP_ADDRESS>\Mic
rosoft.Open.AzureAD16.Graph.PowerShell.dll
Description       : Azure Active Directory V2 Preview Module.
This is a Public Preview release of Azure Active Directory V2 PowerShell Module.
For detailed information on how to install and run this module from the PowerShell Gallery
including prerequisites, please refer to
https://docs.microsoft.com/en-us/powershell/scripting/gallery/overview
ModuleType        : Binary
Version           : <IP_ADDRESS>
NestedModules     : {Microsoft.Open.Azure.AD.CommonLibrary, Microsoft.Open.AzureAD16.Graph.PowerShell.Custom,
Microsoft.Open.AzureAD16.Graph.Client, Microsoft.Open.AzureADBeta.Graph.PowerShell...}
ExportedFunctions :
ExportedCmdlets   : {Add-AzureADAdministrativeUnitMember, Add-AzureADApplicationOwner, Add-AzureADApplicationPolicy,
Add-AzureADDeviceRegisteredOwner...}
ExportedVariables :
ExportedAliases   :
Errorr message:
Add-AzureADMSPrivilegedResource -ProviderId AzureResources -ExternalId "/subscriptions/21edxxxxxxxx-9xxxx6-76xxxxxxxx5c37" -debug
Add-AzureADMSPrivilegedResource : Your version of Powershell is outdated, and some cmdlets may stop working. To ensure
you are able to use all your cmdlets, please update to the newest version which can be found at
https://go.microsoft.com/fwlink/?linkid=870046.
Error occurred while executing RegisterAzureADMSPrivilegedResource
Code: BadRequest
Message: Resource not found for the segment 'privilegedAccess'.
InnerError:
RequestId: 13d649e6-0d06-41c7-bf1d-ffeae2c9bf49
DateTimeStamp: Thu, 17 Jun 2021 06:08:33 GMT
HttpStatusCode: BadRequest
HttpStatusDescription: Bad Request
HttpResponseStatus: Completed
At line:1 char:1

Add-AzureADMSPrivilegedResource -ProviderId AzureResources -ExternalI ...

  + CategoryInfo          : NotSpecified: (:) [Add-AzureADMSPrivilegedResource], ApiException
  + FullyQualifiedErrorId : Microsoft.Open.MSGraphBeta.Client.ApiException,Microsoft.Open.MSGraphBeta.PowerShell.Reg
 isterAzureADMSPrivilegedResource

Note: It works well in Azure Global with the same version.
Ask: Could you help to confirm if AzureADPreview – Privileged Role Management commands are available in Azure China?

In case, MSGraph SDK can works well in Azure China.
for example, New-MgRoleManagementDirectoryRoleAssignmentScheduleRequest is using /beta/roleManagement/directory/roleAssignmentScheduleRequests which is working well in host https://microsoftgraph.chinacloudapi.cn/.
AzureADpreview is using endpoint /beta/privilegedAccess/aadRoles/roleAssignmentRequests which seems not available in host https://microsoftgraph.chinacloudapi.cn/


AMP HTML Support

Does it provide AMP email support as well ?
I want it to be used as an AMP template generator.

@Anshdesire that's on our roadmap

@adeelraza  If there is any plan and if i can contribute that would be good right ?


[FEATURE REQUEST] Remove set / add methods from ISQLServerBulkRecord

Is your feature request related to a problem? If so, please give a short summary of the problem and how the feature would resolve it
The ISQLServerBulkRecord interface is a mixture of specification and implementation details.
Describe the preferred solution
Remove methods which are not called as part of actual bulk copy process via SQLServerBulkCopy#writeToServer.
Describe alternatives you've considered
Currently it's necessary to implement methods even if they just throw a UnsupportedOperationException

Hi @jonfreedman
These exposed APIs are based on driver's implementation of SQLServerBulkCSVFileRecord and SQLServerBulkBatchInsertRecord. You are right they are not "required" for Bulk Copy operation.
You may choose to ignore these APIs but since they are already public and there will be customers overriding them in their implementations, we would not go ahead for removal. This would cause breaking change for customers implementing and overriding these APIs.

Would you accept a pull request that introduced a parent interface (e.g. ISQLServerBulkData) that didn't have this pointless methods?

Yes, that should be fine 👍


No Scrollbar in the Attachments

Describe the Bug
We are having the same issue as described in issue #3789 (https://github.com/BookStackApp/BookStack/issues/3789).
We have a lot attachments that have been added and the scrollbar is not working. Using the scroll wheel on the mouse does not work either.
I have attempted to apply the custom html, but I have not able to make it work with any chrome based browsers or Firefox.
Steps to Reproduce

Go to Page
Click Edit
Click on the "Paper Clip" button
See the list of attachments, but can not scroll down, to see the rest of the items.

Expected Behaviour

Go to Page
Click Edit
Click on the "Paper Clip" button
See the list of attachments.
Able to scroll through uploaded items.

Screenshots or Additional Context
No response
Browser Details
Google Chrome Version 108.0.5359.99 (Official Build) (64-bit) on Windows 10
Exact BookStack Version
22.11
PHP Version
PHP 8.1.2
Hosting Environment
Ubuntu Server 22.04.1 LTS on VMWare ESXi 7.3

This happed to us as well. The scenario can be reproduced at the demo page for Bookstack.  Items on the side panel couldn't be scrolled. Testing v22.11 before deployment and this so far has been a hurdle. It was working fine on v22.09.

Thanks for reporting, but I'm going to close this off as a duplicate of existing issue #2887.


Method to get local index folder

Currently the BoosterCatalog does not have method to find the local index folder, sometimes it will be useful to fetch some files from local cloned repo

as per discussion with @gastaldi we can use BoosterCatalog.copy()

Also, The index folder is an internal structure and should not be exposed externally.


Implementing Fourier Features into NERF

Hey, I think you have noticed this work https://github.com/tancik/fourier-feature-networks. Do you think it is straight forward to replace the positional encoding with this Fourier Features. I have tried and didnt see much improvement.

Yes, I am currently experimenting. I have discussed with someone here https://github.com/bmild/nerf/issues/60
What I found is different than the paper as well... I found gaussian features to be worse, then I also tried uniform sampling and found it better when using more features, but in the end sampling random numbers is not necessary at all, just use the current linspace with more points suffices...
https://github.com/kwea123/nerf_pl/blob/34bb7a9c24a6475a1cb711007a8efc4634129175/models/nerf.py#L17
To conclude I tried the followings embeddings:

sin(ax+by+cz) with a,b,c ~ N(0, var)
sin(ax+by+cz) with a,b,c ~ U(0, max)
sin(ax), sin(by), sin(cz) with a, b, c ~ N(0, var)
sin(ax), sin(by), sin(cz) with a, b, c ~ U(0, max)
sin(ax), sin(ay), sin(az) with a ~ U(0, max)
sin(ax), sin(ay), sin(az) with a fixed beforehand (the current PE)

And it turns out 6. is the best, and it gets better if I use more frequencies (current PE uses 10, I increased it to 64 and it's better with +1 point PSNR)
the moidification just looks like this, very simple...
self.freq_bands = 2**torch.linspace(0, N_freqs-1, 64)
I didn't (and probably won't) perform any further analysis like drawing the NTK, so to be honest I don't know what's wrong with the settings, but there are some points that I think might be the problem:

The experiment in the original paper is oversimplified: there is neither coarse to fine nor direction embedding. When combined with everything, the performance gain might not be that significant, or even get worse.
The scene I use is a forward-facing scene which uses NDC, so it means the scale in x,y,z is different especially for z. Therefore it might require more careful tuning on the frequencies, notably we shouldn't be using the same distribution for all x,y,z.
Hope these insights help.
What scene and parameters did you try?

So i have followed the other thread as well and I found that adding Gaussian embedding does improve the result just a little bit but we if look at the result closely then there are small artifacts. I tested on my 2 scenes (I captured using my phone). The indoor scene work best with 64 frequencies, the another indoor looks good with 32. In the paper, they said that learning this Gaussian embedding doesn't help but I think that hand-tuning like this is sub-optimal.

how if we implemented the fractional fourier features


infer too slow

it has cost me an hour to analyse my ios app,not done yet.how to make it quick

How long does it take to compile your project?
It's expected that analysing a whole project can take some time, but there may be something wrong if running Infer takes a lot longer than compiling your project. What is the command you use to run Infer and which version of Infer are you using?
If you're only interested in analysing some changes (eg introduced by a commit), have a look at the differential workflow which does not run a full analysis.

i use the this command : infer -- xcodebuild -workspace HelloWorld.xcworkspace -scheme HelloWorld
Infer version v0.8.1

How long does it take to compile your project?

about 2 or 3 mins to compile my project.my project is about 50m in app store.if my project use cocopos,will infer analysis all dependices or just my framework.

It cost me 1-2 hours to analyze my android project includes 7000+ java files.
How can I improve the speed?

Are you using the Gradle plugin for Infer? This should speed up the analysis quite a bit over the default Gradle integration, especially on a multicore machine.
Another thing you can try is instructing Infer only analyzing the files that have changed since the last analysis: infer --reactive -- gradle assembleDebug (assuming you have not blown away the infer-out directory from the last Infer run).

--changed-files-index
What is the use of this? Is it same as "infer_whitelist ":[ ] and Can you give the full command, because It is giving me error:  Unknown command-line option '--changed-files-index'.
I am just looking for an options to run infer on a particular file which should be faster than normal in android application.
Thanks.

There have been a number of improvements in infer since this issue was open so I'm closing it. @fancygeek please re-open if infer is still many times slower than compilation.
@harshikesh sorry for the delay; infer --changed-files-index file_list.txt --reactive ... will not analyse all the source files that have been captured, but instead only the ones in "file_list.txt", and the ones that they depend on.

ok，thank you!


[ENHANCEMENT] Semicolons

Semicolons currently are saved as END_CMDs in your and SEMICOLONS in my version. Also I think it would be possible to change it, so the parser has no need for them.

More in #23. It will stay SEMICOLON for now.


Fixes silent header duplication that occurs upon successful revalidation when duplicate headers are present

Inner loop's auto spot2 iterator contains a member variable, _block that references the memory address of the response headers, not the cached headers

Method called on the cached_headers object uses a memory address, not a named header, and leads to all response headers being deleted from the duplicate header forward
Once response headers are deleted, the first duplicate header is written to the cached headers as a new and additional field which leads to duplication in the cached object's headers, then the current iteration ends
The iterator of the outer loop then attempts to increment forward one step via the overridden ++ operator on spot, and due to header slot checks via MIMEField::is_live() within MIMEHdrImpl::iterator::step(), _slot is incremented to limit and iteration of the outer loop halts, because all response headers from that point forward were deleted in the prior iteration

This change moves away from the address-based approach to delete the header and instead uses the header name

Using the header name instead of relying on header alignment also fixes a secondary issue that could arise if response header ordering does not match the cached object's header ordering
Using the header name is slightly less efficient due to having to call find_header, however, this is necessary to ensure the cached headers are correctly removed before the response headers are merged into the cached object
Using a slightly less efficient approach that occurs only on successful revalidations that also contain duplicate headers should be acceptable given the tradeoff is allowing duplication if response header ordering differs from the cached object

Due to the existing logic, if a cached header is not in the response, it will remain in the cached object's headers

Thanks to Masakazu and Leif for providing the unit test and helping with the fix, respectively.

This is a serious bug introduced with v9.2.0, from #7476. The fix here restores the behavior as it was prior to this PR, using the API that ends up calling find_header(). So performance wise, it will not be any worse than versions before v9.2.0.

Cherry-picked to v9.2.x

Reverted from 9.2.x, in favor of #9534


Add UCP 3.1.9 Release

Proposed changes
Add UCP 3.1.9 Config file and Offline Bundles Patch.
Unreleased project version (optional)

Related issues (optional)

Deploy preview for docsdocker ready!
Built with commit 4e38ce37e84f2b57d048f2a74f84b14318398c06
https://deploy-preview-9087--docsdocker.netlify.com

LGTM @ollypom. Thanks.

@ry4nz  @ollypom Could you PTAL? Thanks.

@usha-mandya LTGM


Add dedicated nodes count to stats page

the count of rentable nodes + rented nodes
require: https://github.com/threefoldtech/tfgrid-sdk-go/issues/421

Verified, Devnet.
Dedicate node cards have been added to the stats page.

TC868 - Node Distribution

Verified:

Devnet
000eea2


Doesn't handle multiple admin interface declarations.

Me again.  lol
I have had an issue when I set up multiple admin interfaces in the admin_variables section:
mysql_ifaces: '<IP_ADDRESS>:6032;/var/run/proxysql/proxysql.sock'
The code looks to see if there is a .sock in the string and then adds the string to the command that is used in the load-config command.
    connection = if admin_mysql_ifaces =~ /\.sock/
                   "--socket #{admin_mysql_ifaces}"
                 else
                   host, port = admin_mysql_ifaces.split(':')
                   "--host #{host} --port #{port}"
                 end

And I get this error:
mysql --user=admin --password=admin --socket localhost:6032;/var/run/proxysql/proxysql.sock returned 2
I have fixed it with this code:
connection = if admin_mysql_ifaces =~ /.sock/
iface = admin_mysql_ifaces.split(";").select {|inf| inf =~ /.sock/}
"--socket #{iface.first}"
else
host, port = admin_mysql_ifaces.split(':')
"--host #{host} --port #{port}"
end
Thanks again for this cookbook!

Ahh yes I see, @daveallen maybe you would like to contribute a pull request with this fix :) ?

Yes, I will give you a PR.

Here it is.  https://github.com/ernestas-poskus/chef-proxysql/pull/32

Released 2.3.0 https://supermarket.chef.io/cookbooks/proxysql
Thank you

Published fix in 3.4.1 https://supermarket.chef.io/cookbooks/proxysql


added documentation on segment count under detailed queue_metrics

Segment file count added under queue metrics in PR https://github.com/rabbitmq/rabbitmq-server/pull/11245
This change will reflect that in the documentation

Thank you!


Need help

After I change the "libqcocoa.dylib", I can not open *.app(uncompress from ipa file) with my ida pro.
It always open it as a folder.
What happened to it?

Added load bundle for open dialog (The official Qt does not support this feature, but this is suitable for ida )

This is a new feature. It will be convenient when there are multiple binary file(execute, framework, dylib) in the .app to be analyzed.


Fix typo in docs

It's enough to change commit message and title

Fixed, thanks, I will remember that.


Flowcept doesn't work if root-level keys are not informed in the settings file

Requirement: Flowcept should work with the default settings in the following situations:
(1) The user specified a settings file (in the env var) that is completely empty
(2) The user specified a settings file (in the env var) that is partially filled
(3) The user did not specify any setting file. The setting file simply does not exist.

Linked issue: https://github.com/ORNL/flowcept/issues/119

Let's try to keep source code and configs as separated as we can. Let's avoid having hardcoded raw YAML or even a configs dict hardcoded in the Python source configs.py file. We can have the default settings specified in their own configs file.

Start up the database services using docker compose

So the database services (like mongodb) are required to run Flowcept?

Start up the database services using docker compose

So the database services (like mongodb) are required to run Flowcept?

Yes. The current implementation of Flowcept requires a database (currently, MongoDB) and a MQ (currently, Redis or Kafka).

The README states that MongoDB and Redis services are needed to take full advantage of flowcept. So this makes me think that MongoDB and Redis are optional. The simple example in the README doesn't require any of these services or does it?

The README states that MongoDB and Redis services are needed to take full advantage of flowcept. So this makes me think that MongoDB and Redis are optional. The simple example in the README doesn't require any of these services or does it?

This is outdated. Thanks for pointing this out. I will fix it in the README.

TL;DR:
It remained as it was, except that we are now including the settings file into the package directly, eliminating the extra step to create the settings file.
Longer explanation:
The settings YAML file will remain necessary because the adapters rely on it to determine their default values.
Flowcept works both as a package and a complete system with backend dependencies, and when adapters are enabled, it relies on additional services like MLFlow and Dask. To operate in this complex setup, the settings are needed, particularly for integration with these external systems.
In the future, we could consider splitting settings into multiple YAML files—one per adapter—for modularity, but this would require a longer implementation time.
In the end, I included sample_settings.yaml directly in the package as a resource (which was actually the intention in the original design) by modifying pyproject.toml. This allows for a simple run configuration without forcing users to install unnecessary dependencies or specify a separate YAML file through an environment variable to test the simple case.
I also modified the flowcept/init.py
The current behavior is as follows:
The settings in settings.yaml are overridden by any file specified in the environment variable FLOWCEPT_SETTINGS_PATH.
If no path is set, Flowcept will check for a file at ~/.flowcept/settings.yaml. If found, it will use this file.
If neither option is available, Flowcept will fall back to sample_settings.yaml from the package’s resources directory, which contains all default values.
This closes the issue https://github.com/ORNL/flowcept/issues/119

This PR closes this issue as well. I opened a new one for the CLI implementation.


Download hover overstays its welcome

https://images.zenhubusercontent.com/228681195/c461b356-d71d-4d57-9721-f62e682732a1/screen_recording_2021_11_17_at_4_15_22_pm.mov

To reproduce:

Click on Download
Cancel Download modal
“Download” hover remains
Refreshing the page clears the hover

Not entirely sure what could have caused this, so probably use git bisect to find the offending commit would be a good start!

This does not reproduce in production.


[Bug]: leading zeros after dot are dropped in defaults

Describe the bug
If I define
model Foo{
  bar?: float64 = 0.01 <- leading zero
}

it gets outputted as
Foo:
  type: object
  properties:
    bar:
      type: number
      format: double
      default: 0.1 <- missing zero

meanwhile if I use anything but a zero after the dot it works. See the link to the playground
Reproduction
Playground Link
Checklist

[X] Follow our Code of Conduct
[X] Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

I played around a bit more in the playground. The issue seams to be only with 0.[zeros]n. If I do 1.01 it works


trial - run rails from a test branch

the test branch has minimal bumps to the ruby version, without trying to do a full bundle install.

Codecov Report
Base: 61.26% // Head: 8.26% // Decreases project coverage by -52.99% :warning:

Coverage data is based on head (c57cda1) compared to base (a9b168a).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #1929       +/-   ##
==========================================
- Coverage   61.26%   8.26%   -53.00%
==========================================
  Files         179      93       -86
  Lines       14523   10111     -4412
==========================================
- Hits         8897     836     -8061
- Misses       4516    9220     +4704
+ Partials     1110      55     -1055

Flag
Coverage Δ

acceptance-api
?

acceptance-cli
?

unittests
8.26% <ø> (ø)

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

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

And it is good!

As a trial/test PR this will not be merged.
The fix under test here is https://github.com/epinio/example-rails/pull/6

When done simply close and abandon the PR


Team Section fix in home page

Move the #team and button to the respective positions marked. Also check if the page is responsive if not please make it

Attach a screenshot before sending a Pull Request🐱👤

I want to contribute. Please assign it to me

@sumana2001 go for it 🔥


Make Kubernetes collector production ready

Motivation
Kubernetes is the de-facto standard to orchestrate containerized applications. It is the go-to solution, no matter where your infrastructure is running on. Resoto can already collect resources in AWS, GCP, and DigitalOcean - all of them support Kubernetes. Resoto should support k8s as well
Desired Behavior
Collect resources from k8s.
Additional Context
No response

Fixed via #903


Reuse forceApprove in UniERC20 library

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+3.9%) to 66.926%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

contracts/libraries/UniERC20.sol
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
3.9%

Covered Lines:
120

Relevant Lines:
165

💛  - Coveralls


[BUG] Swiping to home is registered as scroll

Describe the bug
When trying to trigger the home gesture, fritter registers a scroll.
Screenshots
https://user-images.githubusercontent.com/50382414/137813520-d4c6e41d-71cf-4ea8-929f-283490c0efd9.mp4
Device
Xiaomi Mi 9T Pro
Android 10 (lineageos without google play services)
Fritter version 2.14.1

Please could you fill in the rest of the bug template? I'm not able to reproduce this, so I need to narrow it down, based on at least the version of Fritter and Android, and maybe the device that you're using.

@jonjomckay Do I still need to provide more info?

This should be fixed in d90cd5b263ae5466671e3441b0fabea41221548b, and released in the next version!


How can a default value be set using data attributes?

How can a default value be set using data attributes? Tried the following, but doesn't work:
data-defaultvalue
data-emptyvalue
Are these even supported?

Unfortunately due to this project being supported by volunteers we cannot provide user support at this time. Please try a site like Stack Overflow:  http://stackoverflow.com/questions/tagged/x-editable
Thank you.


New command: m365 pp dataverse table remove

Usage
m365 pp dataverse table remove
Description
Remove the specified table
Options

Option
Description

-n, --name<name>
The name of the dataverse table

-e, --environment <environment>
The name of the environment

--asAdmin
Run the command as admin for environments you do not have explicitly assigned permissions to.

--confirm
Don't prompt for confirmation

Examples
Remove the contoso-users table for the given environment with confirmation prompt
m365 pp dataverse table remove --name "contoso-users" --environment "Default-2ca3eaa5-140f-4175-8261-3272edf9f339"

Remove the contoso-users table for the given environment with confirmation prompt without confirmation as Admin
m365 pp dataverse table remove --name "contoso-users" --environment "Default-2ca3eaa5-140f-4175-8261-3272edf9f339" --confirm --asAdmin

Additional Info
You have to use a delete request to dynamicsApiUrl/api/data/v9.0//EntityDefinitions(LogicalName = 'logicaltablename'), and this results in a 204 No Content

Hi @appieschot , you can't with a delete request to dynamicsApiUrl/api/data/v9.0/logicaltablename.
You have to use a delete request to dynamicsApiUrl/api/data/v9.0//EntityDefinitions(LogicalName = 'logicaltablename'), and this results in a 204 No Content

LGTM 🚀

I would like to work on this one

@appieschot looks good to me as well 👍
@nicodecleyre all yours 🫡. you rock 🤩


Compile issue

Have I written custom code (as opposed to using a stock example script provided in MediaPipe)
Yes
OS Platform and Distribution
Android 12
MediaPipe Tasks SDK version
v0.10.10
Task name (e.g. Image classification, Gesture recognition etc.)
Pose
Programming Language and version (e.g. C++, Python, Java)
Java
Describe the actual behavior
compile failed
Describe the expected behaviour
AAR Should compile successfully
Standalone code/steps you may have used to try to get what you need
I have compiled successfully on v 0.10.9, but the same process cannot work on v 0.10.10.

Other info / Complete Logs
ERROR: /private/var/tmp/_bazel_volvo/3fb8898b4c02b503c801f09a4a3be05c/external/XNNPACK/BUILD.bazel:1515:19: Compiling src/amalgam/gen/neoni8mm.c failed: (Exit 1): clang failed: error executing command (from target @XNNPACK//:neoni8mm_prod_microkernels) external/androidndk/ndk/toolchains/llvm/prebuilt/darwin-x86_64/bin/clang '-D__ANDROID_API__=21' -isystemexternal/androidndk/ndk/sysroot/usr/include/arm-linux-androideabi -target ... (remaining 66 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
clang: error: the clang compiler does not support '-march=armv8.2-a+i8mm+fp16'
Target //mediapipe/examples/android/src/java/com/google/mediapipe/apps/build_aar_pose:mediapipe_pose_tracking failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 19.574s, Critical Path: 17.42s
INFO: 171 processes: 130 internal, 41 darwin-sandbox.
FAILED: Build did NOT complete successfully

Hello @fchen09
Would you please elaborate on the steps followed for compilation so that we can reproduce the issue at our end?
Additionally, please provide link for the documentation you are following to compile the solution.

Hi @fchen09,
We are no longer maintaining the Building App via the AAR solution. Instead, we suggest utilizing the pre-built package from Maven. You can locate the setup documentation for Android here, as well as the implementation steps for Pose Landmarker here.
Thank you!!

I met same problem.
Have you solved it?

I met same problem.
Have you solved it?

add --define=xnn_enable_arm_i8mm=false ,

I met same problem.
Have you solved it?

add --define=xnn_enable_arm_i8mm=false , eg： bazel build -c opt --fat_apk_cpu=arm64-v8a,armeabi-v7a --define=xnn_enable_arm_i8mm=false //med...

I upgraded gcc and the problem is soloved.
Thank you very much.

Hi @fchen09,
We are no longer maintaining the Building App via the AAR solution. Instead, we suggest utilizing the pre-built package from Maven. You can locate the setup documentation for Android here, as well as the implementation steps for Pose Landmarker here.
Thank you!!

According to : https://github.com/google-ai-edge/mediapipe/blob/master/docs/getting_started/android_archive_library.md
The MediaPipe Android Archive (AAR) library is a convenient way to use MediaPipe with Android Studio and Gradle. MediaPipe doesn't publish a general AAR that can be used by all projects. Instead, developers need to add a mediapipe_aar() target to generate a custom AAR file for their own projects. This is necessary in order to include specific resources such as MediaPipe calculators needed for each project.

What is the official information? Does the pre-built package contain all calculators?


Can WhereHows support cloud base source system or cloud base datalake

Hello Gurus,
As we would like to build data lake base on cloud, such as AWS S3.
Can WhereHows support datasource system on cloud?

Dear issue owner,
Thanks for your interest in WhereHows. We have recently announced DataHub which is the rebranding of WhereHows. LinkedIn improved the architecture of WhereHows and rebranded WhereHows into DataHub and replaced its metadata infrastructure in this direction. DataHub is a more advanced and improved metadata management product compared to WhereHows.
Unfortunately, we have to stop supporting WhereHows to better focus on DataHub and offer more help to DataHub users. Therefore, we will drop all issues related to WhereHows and will not accept any contribution for it. Active development for DataHub has already started on datahub branch and will continue to live in there until it's finally merged to master and project is renamed to DataHub.
Please check the datahub branch to get familar with DataHub.
Best,
DataHub team


Update perf-benchmarks pipeline to run custom data benchmarks test

Description
7825
This PR adds another stage in the Performance benchmarks pipeline to display / collect custom data results of benchmarks tests (e.g., summarySize, opSize). It is broken down into three steps:

Adding a appInsightsCustomDataTestHandler.ts to process custom benchmarks output (stored in ./customBenchmarksOutput).
Updating the configuration of test-perf-benchmarks.yml by adding an additional step.

Step 3 from the ADO ticket is already covered by the addition of test:customBenchmarks.

Relevant Links
FluidFramework-PerfTests
Performance Benchmarks
Relevant PRs
https://github.com/microsoft/FluidFramework/pull/22140
https://github.com/microsoft/FluidFramework/pull/22051

https://github.com/microsoft/FluidFramework/pull/22225
Will be adding new pipeline stage as works on tablebench custom benchmark is completed.

https://dev.azure.com/fluidframework/internal/_build/results?buildId=287663&view=results
Manual run on Performance benchmarks pipeline.

https://dev.azure.com/fluidframework/internal/_build/results?buildId=288133&view=results
New pipeline run to verify that the custom benchmark data is sent to Kusto & Azure AppInsights.

Kusto
https://dataexplorer.azure.com/clusters/kusto.aria.microsoft.com/databases/742fa5a288b045e5beab1a2b8e445a71?query=H4sIAAAAAAAAA12QzW6DMBCE73mKFSeQEpG2UhMp4pL20ktzKLcqQg5eg%2Fmxkb00perD1zhKQ7itxvPN7rhBAkvMUCpbBEiAFTrc8Gi3aNwLKu71BJQ%2Bh07UQsgcM9H0kmdCoCqkQjRSFVmHRmjTMpXj4hfOJRqEV0Yswy9U9M5cTMksBCdUedkyUwf3tpNxaAlJAoFBYeMSGbcxoaW4krVsV%2F%2Fgalw1p3vZ8Dfu8cft9mnzPDdc6XTo0Nteeku63U%2FPIVYjPKzXbsRvcu2hY8aii70MWWW1Cn1e7ulxjG5u0sSaQ%2Fchf%2FCK8M8gHVU4dOD1cD%2B4UlFwXILtW7d5mNknanB02Z3RFeZ0qWF7STh%2B5nJWa6LdTltOL7rb9wf2D3OW%2BAEAAA%3D%3D

AppInsights


Add host.docker.internal overide for docker dev

Just adding the host.docker.internal override to the docker command for the dev branch.
This got me on Linux where docker does not natively support host.docker.internal and so open webui could not connect with my local ollama. Adding the override fixes this.

Thanks!


Add an engine example demonstrating a scene depth being used in a custom shader

related to https://github.com/playcanvas/engine/pull/4244
It should demonstrate:

CameraComponent.requestSceneDepthMap
uSceneDepthMap
shader chunk used to sample the depth texture for both webgl1 and 2

maybe something from here?
https://files.unity3d.com/talks/Siggraph2011_SpecialEffectsWithDepth_WithNotes.pdf


Typings replace tsd

Just passing by, notifying that tsd is now deprecated in favour of typings.

Created a pull for this: https://github.com/b091/ts-skeleton/pull/17

Cool, thanks. I'll check it later.

:+1:  implemented! :)


pypi distfile does not include tests

The pypi distfile does not include the tests. Please add them.

What value-add is there here?  Even if they're included in the distribution they aren't executable (and setting up setup.py test requires eggs and easyinstall and sadness).
The supported workflow for running the tests is cloning this repository and using tox

@0-wiz-0 I agree with @asottile in that I don't really see the point of this. If you want to run the tests, just clone the repo and make test.

I agree that for randomuser, the workflow you suggest is fine.
However, for packagers, it's good to have some self-tests included in a tarball distributed for packaging, so that trust in a new version can easily be established.

Packagers are going to look at (and probably contribute to) the source git repo anyway.  The github url is linked on pypi and the README covers continuous integration as well as coverage and the repository contains checked in tests.  Adding complexity to our packaging for zero value add seems not the best idea.

Closing as won't fix.


add support for net, url, and mail types

@yields
@f2prateek
This PR adds out-of-the-box support for more standard types, making it possible to write code like this:
var a net.TCPAddr

json.Unmarshal([]byte(`"<IP_ADDRESS>:80"`), &a)

// a.IP   == net.IPv4(192, 168, 0, 1)
// a.Port == 80

My motivations behind this changes are:

We cannot modify the standard library types to make them implement encoding.TextMarshaler or objconv.ValueEncoder
I'd like to provide support for these types in github.com/segmentio/conf without having to write custom code in that package (we get it for free if it's in objconv)

Please take a look and let me know if anything should be changed.

LGTM.
I think this shows a a need for there to be an abstraction in this library to allow for custom adapters (particularly types from for stdlib/3rd party that prevent you from making them implement the required interfaces). Something like https://github.com/square/moshi#custom-type-adapters.
I think the simplest way would be to add 2 APIs (and similarly for encoder):
type DecoderFactory interface{
  CreateDecoder(t reflect.Type, opts DecodeFuncOpts) decodeFunc // expose DecodeFuncOpts and decodeFunc
}

and after these are registered, add a lookup step in makeDecodeFunc.
func makeDecodeFunc(t reflect.Type, opts decodeFuncOpts) decodeFunc {
  for _, f := range decoderFactories {
    d := f.CreateDecoder(t, opts)
    if d != nil {
      return d
    }
  }
  // fallback to builtin ones
}

And you'll want to cache the decoders created with their type (and maybe options) as the key to avoid iterating through the full list each time.

That's a good idea, I'm gonna merge this to get the behavior working and I'll start experimenting on what you suggested.
Thanks for the review!


'The keyWindow or contentView is missing' message before scraping

When testing the command line Example from the repo, this message gets printed in the console, before doing anything: The keyWindow or contentView is missing., then the script works as intended.
What does it means and how can we prevent this to show?

Are you using storyboards? If so, check if it helps to embed your viewcontroller in a navigation controller.

@fishfisher can you explain a little more please?

@avisual Are you seeing this printed to the log as well? For me the log went away when I embedded my root view controller in storyboard to a navigation controller. For me the keyWindow was nil when my main view controller was root.


Added image for eccv-22

👷 Deploy Preview for gladia processing.

Name
Link

🔨 Latest commit
0ccb331bdc48b0110f08dc4e933f5bb2359e0dc5

🔍 Latest deploy log
https://app.netlify.com/sites/gladia/deploys/642d946e655aed0008554ca1


fix: prevent failure using input with dash

Based on issues found on #6 but we keep the options using -.
Co-authored-by: Jacques Supcik<EMAIL_ADDRESS>

This is a good solution too.
(Merci pour le co-auteur 👍🏼)


How to use with a Vault server behind a self-signed certificate?

I have a vault server I setup following the Kelsey Hightower Vault on GKE guide which has the user create a self-signed TLS certificate for communication with the vault server. There is no option to configure this in Vaultingkube and I would love some guidance.
time="2018-07-30T04:46:54Z" level=info msg="Started with sync period every 30 seconds"
time="2018-07-30T04:47:25Z" level=fatal msg="Get https://<REDCATED>:8200/v1/sys/mounts: x509: certificate signed by unknown authority"

Ah. I ended up creating a new image and putting the certificate in the right place. Thanks!


False Positives?

This sure is a lot of detections from VT - https://www.virustotal.com/gui/file/6f9e759f8993632a23ee84513e274202fa0df1e2ced4a868c1aeb845d810e509/detection
Any explanation? Thanks....

I don't have anything to add other than what I've written here: https://github.com/alexzorin/authy/issues/12#issuecomment-734174185


Auto login when there are  Windows credentials

I understand that in order to get windows principles we need to issue a challenge (401 code) in controller and IIS server would send populated LogonUserIdentity object. I wonder if there is a way to do all that in the Owin middleware pipeline before we get to the MixedAuth middleware for processing. I think it would be more practical to automatically login user when there are Windows credentials (internal users) and present a local login page when there is none (external users).

You can do that by hooking to the application EndRequest event, Instead of using the provided RegisterMixedAuth() extension method, implement your own:
/// <summary>
/// Adds a mixed-authentication middleware support.
/// </summary>
/// <remarks>
/// This is required to enable the modification of the fake status code.
/// </remarks>
/// <param name="app">The HttpApplication instance.</param>
public static void RegisterMixedAuth(this HttpApplication app)
{
    app.EndRequest += (object sender, EventArgs e) =>
    {
        if (app.Context.Response.StatusCode == MixedAuthConstants.FakeStatusCode)
        {
            bool isInternal = myCustomCheckLogic();
            if (isInternal)
            {
                //proceed with the challenge
                app.Context.Response.StatusCode = 401;
                app.Context.Response.SubStatusCode = 2;
            }
            else
            {
                //redirect to login page.
                context.Response.StatusCode = 302;
                context.Response.RedirectLocation = "/Account/Login";
            }
        }
    }
}

Is this what you are trying to accomplish  ?

Thank you for your help.
Actually I found what I need. Here what I meant:

    [AllowAnonymous]
    public ActionResult Login(string returnUrl)
    {

        var loginInfo = AuthenticationManager.GetExternalLoginInfoAsync().Result;

        if (loginInfo == null)
        {
            return new ChallengeResult("Windows", Url.Action("Login", "Account", new { ReturnUrl = returnUrl }));
        }
        else
        {
            return View("ExternalLoginConfirmation", new ExternalLoginConfirmationViewModel { Email = loginInfo.Email, Name = loginInfo.DefaultUserName });
        }
    }

Do you think your MixedAuth would work with Identity Server v.3 ?
It is very cool product. https://github.com/IdentityServer

I haven't used it, But I think yes!
Thanks for the feedback.

Hello,
I am using the code you suggested above by creating my own hook to the EndRequest event (and registering it in Global.asax), but what else do I need to change so that when a user hits my /Account/Login page, it will automatically log them in if they have valid Windows credentials?
Thank you!

@emf411 This is different, If your target is to have auto windows login as described in the MVC5-MixedAuth solution, You could do the following:
1. Startup.Auth.cs: Change LoginPath to point to Windows Login Action
LoginPath = new PathString("/Account/WindowsLogin"),

2. AccountController.cs: Add WindowsLogin action to request the windows challenge.
[AllowAnonymous]
public ActionResult WindowsLogin(string returnUrl)
{
    return new ChallengeResult("Windows", Url.Action("ExternalLoginCallback", "Account", new { ReturnUrl = returnUrl }));
}

3. 401.html: Add 401.html error page to your app root that will redirect to the normal login page:
<!DOCTYPE html>
<html>
<head>
    <title></title>
    <!-- redirect to normal login page -->
    <meta http-equiv="refresh" content="0;URL='/Account/Login'" />
</head>
<body>
    Invalid credentials, redirecting ...
</body>
</html>

3. Web.config:  Use 401.html to respond to 401 HTTP error (in case of login failure):
<system.webServer>
    <httpErrors errorMode="Custom">
      <remove statusCode="401"/>
      <error statusCode="401" path="401.html" responseMode="File" />
    </httpErrors>
</system.webServer>

Thanks!

This is perfect, thank you so much!!  I figured out a likely inefficient work-around, your WindowsLogin function was exactly what I needed.  Code is now working as expected, Windows users in my database are automatically being logged in, and non-Windows users are getting to the Login page.
Thanks very much for the quick reply to this!

One more question around the ExternalLoginCallback method.
I deleted a Windows user account from my AspNetUsers table which is correctly causing  SignInManager.ExternalSignInAsync to return SignInStatus.Failure
I am correctly returned to /Account/Login at this point.  Locally on IIS Express, everything runs as expected, however my Windows 2008 IIS is incorrectly storing that Request.IsAuthenticated is True, and User.Identity.GetUsername() is displaying the Windows username.
Do you know how I can clear out the User.Identity model when the SignInStatus.Failure occurs, so that there is no Username displaying, and the "Log Off" link is also not displaying?
Thank you!

Request.IsAuthenticated will be True when the user has been authenticated (for example: by a cookie), while Request.LogonUserIdentity.IsAuthenticated will be True when the Windows user has been authenticated.
Clear your cookies and try again!

Thank you Mohammad!  I really appreciate your help on this.
I tried clearing cookies but I'm still having the same problem.
Maybe my scenario is slightly different.  My site will be both internal on my LAN and external to the Internet.  What I'm trying to do is this:

When the user hits my site, determine if they are internal or external (I already have this working)

If they are external, send them to /Account/Login

If they are internal, grab their Windows credentials and run the ExternalCallBack method to make sure they are in my database

If they are not in my database, return them to /Account/Login and make sure that the AuthenticationManager has signed them out so the Login page doesn't display their Windows username when I call User.Identity.GetUsername()

Step 4 seems to be where I am stuck, even if I try AuthenticationManager.SignOut() I am still seeing that the User.Identity model has not been cleared.

Seems you have a specific scenario, but all can be managed in your actions. In step 4, you may also need to sign out the external cookie  AuthenticationManager.SignOut(DefaultAuthenticationTypes.ExternalCookie);.
As for not registering those users, it's controllable in your ExternalLoginCallback (don't send them to External Login Confirmation).

Hi @MohammadYounes , thanks for your help of this question. I have one doubt regarding this.
I already have a custom error 401 page, so I cant redirect user to /Account/Login .
Is there any other alternate?
Thankyou for your time.

Hello @MrHmP
Unfortunately, I have no other alternative at the moment.
Thanks!

Hello @sylfree9999
Did you try checking the network tab to see the sequence of requests ? Also, Did you unlock the windowsAuthentication section as mentioned here ?

Hi @MohammadYounes , yes, I have unlock that in the .vs config and actually I am able to login using windows domain account. But I still could not auto login the domain user since the application went straight to login action instead of winlogin.

@sylfree9999 FormsAuthentication module should be removed from the whole app, not just MixedAuth path.


Reduce the amount of UDT packets send

Not calling send_exact twice ensure that UDT does not send out a
packet only containing the payload length before transmitting the
payload.

Nice one :). Wow, I really do need to rewrite this whole thing.


Generated model is final

Hi All,
I'd like to know why the generated model by Apollo is always final, and in case you need to edit a property on client-side, what your best approach to handle it is.
Thank you for your precious work and support.

Best approach would be to transfer to your own data models. Mutablity would open up a host of concurrency problems we do not want to think about


Fix semantic of support binary equality

What changes were proposed in this pull request?
With introduction of trim collation, what was known as supportsBinaryEquality changes, it is now split in isUtf8BinaryType and usesTrimCollation so that it has correct semantics.
Why are the changes needed?
With introduction of trim collation, what was known as supportsBinaryEquality changes, it is now split in isUtf8BinaryType and usesTrimCollation so that it has correct semantics.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Everything is covered with existing tests, no new functionality is added.
Was this patch authored or co-authored using generative AI tooling?
No.

so let me check if I understand correctly
we keep supportsBinaryEquality, which will be:

true for UTF8_BINARY
false for UTF8_BINARY_RTRIM

and we also introduce isUtf8BinaryType, which will be:

true for UTF8_BINARY
true for UTF8_BINARY_RTRIM

then, we refactor the execution-related code path branches to rely on isUtf8BinaryType, rather than supportsBinaryEquality?

Everything is correct except the last sentence. We will keep relying on supportsBinaryEquality in most cases (see for example hashing and aggregate). We will use isUtf8BinaryType when we don't want to have different behaviour for UTF8_BINARY_RTRIM and UTF8_BINARY collations.

I see, this makes sense - I'm on board with the approach
one small ask for you tho - let's hold off all unnecessary changes in this PR (for example: modifying the branching pattern in CollationSupport classes). I don't like the idea of modifying some code (that the execution flow can't reach anyways at this time, because analysis will block it) and not adding the appropriate tests
please opt for one of the following:

undo all needless & untested changes here, and only keep the minimally required scaffolding for further implementation
keep the supportsBinaryEquality -> isUtf8BinaryType changes, but also apply the proper space trimming policy, whitelist the expression and add corresponding tests

so let's just avoid half-baked code, it can be confusing both for you and the reviewers

Agree, makes sense. Let's go with 1, i.e we want have any changes without tests. Will (today) follow up with prs that will have tests and use that code path.

+1, LGTM. Merging to master.
Thank you, @jovanpavl-db and @uros-db for review.


SDK to support the base use case

SaaS SDK to support the following:

activation workflow
Subscription operations

Activation
Change quantity ( per user plan )
Change Plan
Calls to Metering service API

Unsubscribe

Updates to documentation.

@dev-kiranbanda There are some conflict files that have to be resolve to allow merging.

@dev-kiranbanda There are some conflict files that have to be resolve to allow merging.

I see that there were modifications to 5 ReadMe files in master and I pulled them into my branch and resolved the conflicts ( I did review the conflicts, took the appropriate changes, fixed lint issues and checked in ).


Add www connection support

As far as I'm aware, there cannot be more than 1 entry in a CNAME file. Plus, www.1pt.co already redirects to 1pt.co, which is preferred for SEO anyways

You can have two if the second one is just a www of the first

Hmm, but will that not just hurt the SEO by splitting it?

Shouldn't be that different
more important pr: https://github.com/paramt/memeadvisor/pulls


Set necessary DNS in dnsmasq config

This ensures that:

bootstrap's dnsmasq allocates defined IPs for masters
"external" DNS resolves these IPs to <cluster-name>-etcd-<index> name
sets etcd discovery DNS entry

@celebdor can you check this pls (for some reason I can't add you as a reviewer?)


SSHCure direct commands

I can't get NFSEN to work and I've been at it for hours.  I'm just trying to run SSHCure on my historical netflow data that I copied over to the system.  I've got the plugin installed but can't see my data in NFSEN unless I use a very specific time that matches one of the filenames.
Is there any way to just run SSHCure on my nfdump files without all the nfsen headache?

Ok, I'll keep at it with NFSEN and see where I get.  Does SSHCure rely on the port being 22?  My SSH activity is on a non-standard port.

In its current form, SSHCure assumes sshd to run on port 22, indeed. In theory, if you search and replace all references to this static/hardcoded port number, you should be good to go with sshd running on other ports too.

I've modified the code to use the alternative port and from what I can tell from the interface and syslog, everything is working correctly.  However I haven't been able to get SSHCure to display anything when I put my data in the profile.  Trying to find out when the first successful login was during the brute force on port 2202 - it's a forensics challenge.  If you have any ideas, I'd be grateful for the tips.
nfcapd.201405230000.zip

It's historical data that I placed in the live directory as if it were recorded.  I thought that by maybe creating a new profile that covered the time period, it would initiate the sshcure process to run against the existing data.
Thanks for the constraints.  I'll see if I can figure it out along with reviewing the packets per flow.  The brute force attempt is obvious in the log - it's identifying the time of compromise(s) that I can't figure out. It's a challenging one for sure and sshcure seems to be the only tool that might be able to give me the answer.

Your approach to integrate historic data in a live profile does not work; NfSen is not aware of any data placed manually in your profile. As such, the data will never be analyzed by SSHCure either.


discount range fix

there's a problem
when you give it a discount of "25"
it tells you it needs to be between 0 and 1
but when you give it a value on that range
it shows 0.25% on the chargily page
which means the range should be arount 0 to 100
Not 0 - 1

another question,
why is this repo marked as a "template"
IT shouldn't be

Looks like it was a bug.
Thanks for contributing.


Consolidate pattern in top-level test script into loop.

Every directory has one test script named directory-test.sh. So make test.sh a loop. While we're at it, add basic pass/fail counts and run every test by default.
If you want to stop running if a particular test fails, you can still do:
sh -e test.sh

@arq5x The commit https://github.com/arq5x/bedtools2/commit/dad1fcbb6250ad4a566cdbc80915e6342c06c6ec breaks make test
I think it always says that all test of all tools work OK.
The following lines don't do what they are supposed to do:

(cd $dir bash && bash test-$dir.sh) || FAILURES=$(expr $FAILURES + 1);
if [ $? -eq 0 ]; then
...

   failed="**$failed** $dir"

Hmm, it seems to work for me. I just intentionally changed a test file and it exited with 1 and said one test failed. What are you seeing when you run "make test"

In my case I didn't have samtools in my PATH, so conversions from SAM to BAM fail.
$ make test

...

Ah, I see. Yes, I am updating to just exit on first failure. Thus, the first tool to have >=1 failure with stop the testing.


timeLimit attribute on procedures appears to no longer work

Boogie allows a per-procedure timeLimit attribute:
procedure{:timeLimit 10} p1() {}
procedure{:timeLimit 20} p2() {}

This used to produce a different TIMEOUT option on each of the two SMT queries:
(set-option :print-success false)
(set-info :smt-lib-version 2.0)
...
(set-option :TIMEOUT 10000)
...
(check-sat)
...
(set-option :TIMEOUT 20000)
(reset)
(set-option :TIMEOUT 20000)
(set-option :print-success false)
(set-info :smt-lib-version 2.0)
...
(set-option :TIMEOUT 20000)
...
(check-sat)
...

I think this may have unintentionally been changed in commit 09a9f17b9a8642f779ea0e5097c0aab8ce9f370b.  After that commit, the TIMEOUT is 10000 in both queries.
I'm not sure what the best fix is; as the commit rightly points out, the old code was messy (e.g. TIMEOUT 20000 is printed three times in the SMT code shown above) and the timeLimit attribute handling was Z3-specific.  But it would be nice if there were a way to make the timeLimit attribute work again.

@bkragl : This issue suggests that Boogie used to support a time limit per implementation.  But now Boogie does not appear to do so.  For example, I tried to run Boogie on the program mentioned above:
procedure{:timeLimit 10} p1() {}
procedure{:timeLimit 20} p2() {}
When I look at the prover log file, I don't see any lines that set the timeout option.  So, it looks like the "timeLimit" attribute is being ignored.  I am wondering if this option got nuked when you did your cleanup.

I am wondering if this option got nuked when you did your cleanup.

No, this issue is much older. @Chris-Hawblitzel already pointed out commit 09a9f17b9a8642f779ea0e5097c0aab8ce9f370b, which changed this behavior. After having a brief look, I must agree that the code for setting timeouts is quite unwieldy.


FIX: avoid implicit string comparison in Sphinx 7.26

This PR mirrors executablebooks/sphinx-book-theme#791.

thanks @agoose77! (btw, for your next PR: we have a pre-commit hook for linting, etc that it looks like you don't have enabled)

Hmm, that should've been installed by default. Thanks for the heads up!

Hmm, that should've been installed by default. Thanks for the heads up!

even if the package is installed, I think you still need to run pre-commit install once from the repo root, to install the hooks into your local  pydata-sphinx-theme/.git/hooks/ folder

@drammock just following this up — do you have an eta for the next release? jupyter-book depends upon sphinx-book-theme, which in turn depends upon this theme, and we're hoping to make a 1.0 release soon!

There is one big issue I'm working on now (on mobile, will add crossref later) and I want to do a point release after that. So hopefully within a week


Setting go.mod module to v2

Changing module name in go.mod to be able to use the go-fastly dev-v2 branch code with go modules.

Note for other viewers of this PR - this is a merge into the dev-v2 branch and not master


maven.sumup.com is down

As of today (8 Sepetember 2022) , I have been unable to build our android app due to the following error:
FAILURE: Build failed with an exception.

What went wrong:
Could not determine the dependencies of task ':app:compileDebugJavaWithJavac'.

Could not resolve all task dependencies for configuration ':app:debugCompileClasspath'.
Could not resolve com.google.android.gms:play-services-location:11+.
Required by:
project :app
Failed to list versions for com.google.android.gms:play-services-location.
Unable to load Maven meta-data from https://maven.sumup.com/releases/com/google/android/gms/play-services-location/maven-metadata.xml.
Could not get resource 'https://maven.sumup.com/releases/com/google/android/gms/play-services-location/maven-metadata.xml'.
Could not GET 'https://maven.sumup.com/releases/com/google/android/gms/play-services-location/maven-metadata.xml'. Received status code 503 from server: Service Unavailable: Back-end server is at capacity

when trying to access 'https://maven.sumup.com' via browser, it comes back with the same error.

Issue still present

It did get resolved that day  (8th September 2022), but it's down again today (14 September 2022) resulting in our inability to build the android version of our driver app.:
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:processDebugResources'.

Could not resolve all files for configuration ':app:debugCompileClasspath'.
Could not resolve com.android.support:support-v4:27.+.
Required by:
project :app
Skipped due to earlier error
Could not resolve com.google.android.gms:play-services-location:11+.
Required by:
project :app
Failed to list versions for com.google.android.gms:play-services-location.
Unable to load Maven meta-data from https://maven.sumup.com/releases/com/google/android/gms/play-services-location/maven-metadata.xml.
Could not get resource 'https://maven.sumup.com/releases/com/google/android/gms/play-services-location/maven-metadata.xml'.
Could not GET 'https://maven.sumup.com/releases/com/google/android/gms/play-services-location/maven-metadata.xml'. Received status code 503 from server: Service Unavailable: Back-end server is
at capacity
Could not resolve com.android.support:support-v4:26+.
Required by:
project :app
Skipped due to earlier error
Could not resolve com.android.support:appcompat-v7:26+.
Required by:
project :app
Skipped due to earlier error
Could not resolve com.android.support:support-v4:26.+.
Required by:
project :app
Skipped due to earlier error

when trying to access 'https://maven.sumup.com/' via browser, it comes back with the HTTP 503 error.

Closing this one.


Please invite me to the Recode-Hive GitHub Community Organization

Name
Akshith ssaai
Discord Username (if applicable)
No response
Additional Context
No response

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people


[Suggestion] Share IPG files as iso

The goal from my understanding is to keep the VM out of the network. But IPG files can be added to the iTunes library, and synced directly to an iPod without any issues. (no need to copy the iPod_Controle folder, just do the sync)
My proposition is to either have the VM include all the games already (heavy option), or share the iPod IPG files as an ISO. This way, you can add it directly into UTM as a second DVD. This way, the VM stays offline, but it is still possible and easy to share the files.
Moreover, the ISO files can be mounted by either a Mac or a Windows machine, so the files will still be accessible like a zip file does.

Hello,
Syncing games with the VM is slow and also limited to only games from 5 different Apple IDs. You should read the tutorial to sync only one game per account using the VM.
The VM is not meant to distribute all games; only to auth each accounts individually into your iPod so you can copy all games manually in the next step of the tutorial.


Feat/ui custom castproxy

This is an attempt at extending ui/ui.js with an optional external castProxy class.
there are still some open ends, but to start the discussion I started this pull request.
If there are any good idea's on how this should better be implemented, I love to hear them!

I have some technical feedback around what looks like some compiler issues, but first let's discuss your goals with this change.
What are the main goals of a custom CastProxy?

I have a project where I would like to use the stock ui, with some minor modifications. One of the modifications being the Chromecast sender implementation.
The main goal for this custom CastProxy is to be able to use a different Chromecast receiver implementation, using the default Shaka build.
I now can use a custom CastProxy sender implementation, using CAF v3 api's. This works fine for me, using this additional argument for shaka.ui.Overlay. Though, I'm struggling a bit on how to make closure compiler cope with this externally defined castproxy, and make it a good general addition to the project.

Yeah, the compiler is difficult sometimes.  I'll leave you some specific technical feedback on the changes about the compiler.  Hopefully we'll be able to help you resolve that.
I'm open to this change generally, but I wonder if you'd be willing to share your CAF v3 implementation with us as well.  Does it use CAF on the sender side instead of the older Cast SDK? Does it still expect Shaka Player on the receiver side, or does it use the CAF default receiver?

I've just noticed that you're developing against the v2.5.x branch.  We normally only accept PRs against master.  Cherry-picks to a release branch are always done by the team after carefully considering the risk of a change and any backward-compatibility issues it might create.  This does not seem like something we would cherry-pick to an older branch.
If you'd like to continue this PR, please rebase against master and address the feedback you've received so far.  Thanks!

Thanks for the explanation, I'll probably handle this rebase and changes the coming week! I couldn't find any time to finish this MR due to the unexpected chain of events. My productivity plummeted, because, you know, kids ;)

No worries!  We understand, and there's no urgency from our perspective.

Soon, we are going to work on https://github.com/shaka-project/shaka-player/issues/4214 so for now I don't think we're going to add this. If you have any suggestion tell me.