Feature request: Support secret creation for github actions

Hi friends,
it would be super cool to be able to set secrets for actions through terraform while creating a repository. ;)
ty,
g. (happy action user)

Yes please

github hasn't made api for secrets available yet, we have to wait for now

Hi. Any updates on this - is the Secrets API still private...?

+1

It looks like it was merged https://github.com/google/go-github/pull/1402#issuecomment-583939009 and released in the v29.0.3 tag.

I think we should also wait for #342 before we start implementing this one.

What do we think about usage? Should action secrets be a new resource - github_repository_action_secret and be used as follows?
resource "github_repository_action_secret" "foo" {
    repository = "${github_repository.repo.name}"
    name       = "FOO"
    value      = "secret"
}

Is there a way to provide ciphertexts in terraform and supply the key via an environment variable?
I can imagine it's problematic to store plaintext github secrets in terraform files ... which might be located on github again.

@hazcod I think one option would be that resource accepts two conflicting arguments - secret and encrypted_secret, The secret could be used to pass plain text value which would be encrypted by terraform using GitHubs public key. The encrypted_secret would accept already encrypted value and leave the encryption the user.
Off topic: In order to not store secrets in terraform files I use an approach where I store secrets somewhere else, and then use a data provider to get the secret values from there. In that case I only need to reference the value from data source.

@radeksimko @jcudit @paultyng what are your thoughts on this?

Note for the implementation: how to know if a secret has been changed and need to be updated by terraform ?
The secret API does not offer a way to read the content of a secret (normal) but return the created_at and updated_at date.
Should we store at least the updated_at date in terraform state and when we refresh terraform state (before a plan for instance) we get the secret and compare its update_at date with the one in the state.
A difference in those dates would indicate the secret was updated since the last time terraform interact with. What do you think?

Another option is that terraform does not detect and repair "drift" on this resource in case the data is updated or deleted outside of Terraform. I think it might also be possible to not store secrets in state, but instead store a hash value of the secret, in case current value hash does not match hash stored in state an update should be performed.

A lurker (who wants this feature) chiming in...
I suggest not trying to get too clever regarding secrets in state. I often treat the Terraform Vault Provider as a guide for the official-ish Hashicorp way of doing things in Terraform.

Interacting with Vault from Terraform causes any secrets that you read and write to be persisted in both Terraform's state file and in any generated plan files. For any Terraform module that reads or writes Vault secrets, these files should be treated as sensitive and protected accordingly.

(https://www.terraform.io/docs/providers/vault/index.html)
Directly on the label, they disclaim any cleverness to protect secrets.
I think Terraform core offers enough ways to protect secrets. The combination of using remote state and an encryption-enabled backend seems sufficient. See https://www.terraform.io/docs/state/sensitive-data.html.

Indeed @StephenWithPH , state is already sensitive so we do not have to be too clever using hash in state (instead of plaintext values).

Another option is that terraform does not detect and repair "drift" on this resource in case the data is updated or deleted outside of Terraform.

This is also fine if it is clearly mentioned in the doc of the resource.
In case of doubts, you can always taint the doubtful resource in your terraform and recreate it. Terraform is your source of truth so manuals changes outside of it should be committed in terraform if they are meant to be permanent.

Nice to see that this worked!
imho ... just get it done ... KISS

Hi, I'd like to take on this as we have a requirement for it in our organisation.
I think that we'd need to merge in https://github.com/terraform-providers/terraform-provider-github/pull/342 before work can be undertaken here, however, since v29 of go-github contains the calls we would need to utilise.
If it's ok, I'll start work on this in a branch of master merged with https://github.com/terraform-providers/terraform-provider-github/pull/342 and we can go from there?

@benj-fletch if you need any help, feel free to ask me.

@martinssipenko Thanks!
I think from reading the comments I'll keep it relatively simple and allow the user of the resource to protect their secrets as they see fit (where possible). I need to see what is available to configure on the API and work on that basis.

@benj-fletch my only ask on this one is to try to keep things forward compatible with these related enhancements.  I'm 👍 on designing for overall Actions support rather than just the secret functionality.

@jcudit Sure thing, I will try my best to do so!

Any idea when this will get merged? Would be very handy feature

I believe that this ticket can be closed now, following the merge of #362 ?


Example app is not working when server becomes unreachable

Steps to reproduce

Run cd ./server && docker-compose up -d && npm i && npm run start to start the server
In other terminal tab, from the root of the repo folder, run npm i && npm run start to start the client in browser
Wait for the browser window to open and for the app to fully load
Try to perform some mutations (create/modify/delete)
Stop the server
Try to perform some mutations now

Actual behaviour
Content does not change and the changed content is not saved in offline cache
Errors in the console:
core.js:1673 ERROR Error: Uncaught (in promise): Error: Network error: Failed to fetch
Error: Network error: Failed to fetch
    at new ApolloError (ApolloError.js:37)
    at Object.error (QueryManager.js:203)
    at notifySubscription (Observable.js:134)
    at onNotify (Observable.js:165)
    at SubscriptionObserver.error (Observable.js:224)
    at httpLink.js:166
    at ZoneDelegate.push../node_modules/zone.js/dist/zone.js.ZoneDelegate.invoke (zone.js:388)
    at Object.onInvoke (core.js:3824)
    at ZoneDelegate.push../node_modules/zone.js/dist/zone.js.ZoneDelegate.invoke (zone.js:387)
    at Zone.push../node_modules/zone.js/dist/zone.js.Zone.run (zone.js:138)
    at resolvePromise (zone.js:814)
    at resolvePromise (zone.js:771)
    at zone.js:873
    at ZoneDelegate.push../node_modules/zone.js/dist/zone.js.ZoneDelegate.invokeTask (zone.js:421)
    at Object.onInvokeTask (core.js:3815)
    at ZoneDelegate.push../node_modules/zone.js/dist/zone.js.ZoneDelegate.invokeTask (zone.js:420)
    at Zone.push../node_modules/zone.js/dist/zone.js.Zone.runTask (zone.js:188)
    at drainMicroTaskQueue (zone.js:595)

Expected behaviour
Changed content should be saved in the cache when the server is not available

@aliok @darahayes @danielpassos @wei-lee @StephenCoady @laurafitzgerald
Network errors (and also runtime server errors) when offline are fully handled by users and skip our
offline queue. Any error that happens inflight needs to be handled directly in UI.
Being online means that this errors can be displayed directly on the UI component and dialog itself within app in realtime. Currently our sample application logs network issues into the console.
There are couple approaches approaches that we can take to resolve this issue:

Interpret that as sample application issue and add dialog informing that error happened when saving data to the server and user should repeat action.

(leaning towards this option)

Stick to handling offline only cases and let developers themselves define what offline means (it can also mean sever is down). However If developer will want to define being offline as not being able to ping server then things will land into queue. However this aproach NetworkStatus interface will need to periodically ping server which is not desired for any production application so -100 for my side.

Add separate link to SDK that will threat network errors (server unreachable only - non 4xx or 5xx errors)
and pop them into the offline queue despite being offline.
This will require some refactoring and introduce couple corner cases

Add RetryLink to be available in the chain so failed requests will be retried with fallback option.
We tried that in our spike repo, however we do not have in current SDK.

Resolved in JS-SDK but we have not enabled that in showcase by default.


Am i missing something, Seems juggernaut client should be able to publish as well?

I am looking at using juggernaut/socket.io for some eventing chat and other items, I must be missing it but why doesn't the juggernaut.js javascript client have a publish method. I see the write method but that doesn't get a message to a redis channel via node.js.
Thanks,
Leif

Does the custom event suppose to be published to subscribers? I tried it to notify some datas but didn't work. I can see a log message below but the server doesn't publish it.
10 Nov 15:07:25 - Received: {"type":"event","data":"some data"}

Nope, custom event goes to subscribers. Clients can't publish, only subscribe for security reasons.

Thanks for the answer. Then what is the purpose of the custom event? I looked into the source code for an hour but no luck.

They're custom 'server events' - they fire events server side.


[ci] Add a build step in GitHub Actions

The build step only happens in the manual release flow.
When I added the checks.yml workflow, I neglected to add a build step to follow up.

Contributor Guide

🎨 Wireframes and designs for Layer5 site in Figma (open invite)

@nebula-aac if granted, can i work on this particular issue?

@nebula-aac I have given a PR #159 regarding this issue, kindly review this


etl job how to disable kerberos

{  "wh_etl_job_name": "HIVE_DATASET_METADATA_ETL", "ref_id": 10001, "cron_expr": "* * * * * ?", "properties": { 	"hive.metastore.jdbc.url": "jdbc:mysql://localhost/hive", "hive.metastore.jdbc.driver": "com.mysql.jdbc.Driver", "hive.metastore.username": "root", "hive.metastore.password": "ttt", "hive.schema_json_file": "/home/schema.json",  "hive.schema_csv_file":"/home/schema.csv", "hive.field_metadata":"/home/field.csv", "hive.hdfs_map_csv_file":"/home/hdfs.csv", "hdfs.namenode.ipc.uri":"hdfs://localhost:8020", "kerberos.auth":"false" },  "timeout": null, "next_run": null, "comments": "hive vm metadata etl" }
I use the above  json to add a etl job . but when the job  run ,  it's also to use  kerberos.principal.
why?
self.schema_url_helper = SchemaUrlHelper.SchemaUrlHelper(hdfs_namenode_ipc_uri, kerberos_auth, kerberos_principal, keytab_file)
the kerberos_auth is false.  why it's also use kerberos?

@lonely7345, even with kerberos_auth set to false, we still have to define the other two kerberos variables as the empty string to avoid Kerberos errors:
"kerberos.principal": "",
"kerberos.keytab.file": "",
Perhaps, that might help you as a workaroud.    Agree with your issue, however, that with it set to false, then no other variables should be required by the WhereHows code...


AV options are not sticky

Everytime I join a new ad hoc twilio video chat room my microphone/speaker/audio settings revert to the default settings. This is bad because the default doesn't work for me.

I'm going to lump this in with #3 as an issue that will require significant development in the Twilio end of things, and unfortunately won't be done in time for ICFP.

That's unfortunate. This happens also whenever I refresh the window too. And it threw me for a loop whenever it happens because I wasn't aware that my audio wasn't working.

I don't recall that this issue occurred at ICSE. Maybe my default audio settings just worked there? Not sure why they weren't working now.

It is unfortunate that we do not have more developers. This behavior hasn't changed since PLDI. I would suggest changing your OS' default audio settings, so that when Clowdr uses your OS defaults they will be what you hope they are.

I will add this to the troubleshooting part of the Clowdr user manual.

I will close this and update #3 to include persisting the preferred input devices


#svelte-announcer triggers CSP violation

Describe the bug
When creating a new Sveltekit project from the demo template and configuring CSP rules that disallows inline styles you get a CSP violation on the #svelte-announcer element which is using inline styles without a nonce.
Reproduction
https://github.com/andersekdahl/svelte-csp-repro

Clone the repo
Run npm run build && npm run preview
Open the site and you'll see these errors attached as screenshots

The only thing changed from the demo project is this commit:
https://github.com/andersekdahl/svelte-csp-repro/commit/53ebe42cb1114a696e3bb49b6c67daf6c7bd8d72
Logs
No response
System Info
System:
    OS: Windows 11 10.0.22631
    CPU: (16) x64 11th Gen Intel(R) Core(TM) i9-11950H @ 2.60GHz
    Memory: 6.04 GB / 31.73 GB
  Binaries:
    Node: 20.11.1 - C:\Program Files\nodejs\node.EXE
    npm: 10.2.4 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Chromium (122.0.2365.92)
    Internet Explorer: 11.0.22621.1
  npmPackages:
    @sveltejs/adapter-auto: ^3.0.0 => 3.1.1
    @sveltejs/kit: ^2.0.0 => 2.5.4
    @sveltejs/vite-plugin-svelte: ^3.0.0 => 3.0.2
    svelte: ^5.0.0-next.1 => 5.0.0-next.80
    vite: ^5.0.3 => 5.1.6

Severity
serious, but I can work around it
Additional Information
No response

@andersekdahl how have you worked around it?

Note on workaround suggested by @BernhardPosselt: Adding a hash as your workaround will only work if your CSP policy allows "unsafe-hashes".
Unfortunately this doesn't work for one of my clients, and I had to develop this workaround instead but allowed us to use Svelte and avoid the CSP violations.
Inside script portion of +layout.sevlte (in runes mode):
`import { browser } from "$app/environment";
// intercept innerHTML invocation and remove style before div is added to dom
$effect(() => {
if (browser) {
const originalInnerHTML = Object.getOwnPropertyDescriptor(Element.prototype, 'innerHTML');
Object.defineProperty(Element.prototype, 'innerHTML', {
  set(value: string) {
    if (value.includes('id="svelte-announcer"')) {
      const safeValue = value.replace(/style=".*?"/i, '');
      originalInnerHTML?.set?.call(this, safeValue);
    }
  }
});

}
});
// add styles back in non-CSP violating way
$effect(() => {
if (document) {
const observer = new MutationObserver((mutations) => {
for (const mutation of mutations) {
for (const node of mutation.addedNodes) {
if (node instanceof HTMLDivElement && node.id === 'svelte-announcer') {
node.style.position = 'absolute';
node.style.left = '0';
node.style.top = '0';
node.style.clip = 'rect(0 0 0 0)';
node.style.clipPath = 'inset(50%)';
node.style.overflow = 'hidden';
node.style.whiteSpace = 'nowrap';
node.style.width = '1px';
node.style.height = '1px';
}
}
}
});
observer.observe(document, {
  childList: true,
  subtree: true
});

}
});`

I am also encountering this issue. I cannot use the 'unsafe-hashes' workaround since my application is security-conscious, and @kevinpeckham's workaround honestly feels too hacky.
What I ended up doing is applying the workaround from #12661 (with a slight tweak to support Svelte 5, which deprecated <svelte:component />) . From what I've read, the announcer is only used to announce route navigations, and since my application has only a single route, it should be fine to remove the announcer:
// svelte.config.js

preprocess: [
    {
        name: 'strip-announcer',
        markup: ({ content: code }) => {
            code = code.replace(/<div id="svelte-announcer" [\s\S]*?<\/div>/, '{null}');
            return { code };
        }
    }
]


Develop a naming convention for patterns

Naming patterns after colors isn't sustainable

Patterns should be named a single emoji

Can we close this? Pretty sure it will never happen

it's not a problem, please close.


to-do list for 4.1.0

These were on #146 but none of them are critical to the new version actually working.
So I will release and move them to the to-dos for a next minor version

[ ] add contributing.md #30
[ ] add vignettes #91
[ ] (finish) adding typing #104
[ ] make dvc a dev dependency and make "data for tests" into a dvc dataset #147
[ ] add nox session to download test data #71

These are either done or backlog and we're past version 4.1, so I'm closing this


swaylock sans IFD

Sorry, I'm not going to look into this myself. Just want to get notified if/when :sweat_smile:.

This is noted in the FAQ https://github.com/catppuccin/nix#-faq under the IFD question.

Closed in favor of https://github.com/catppuccin/nix/issues/392


chore(nestjs-typegoose): update "repository" field to be properly formatted (on npmjs)

This PR updates the package.json for nestjs-typegoose to be properly formatted, this will have the effect that npmjs will properly format the url now:

Screenshots
current nestjs-typegoose:

typegoose:

@hasezoey - Thanks for that!
Scott


Make block bogon rule display consistent - RELENG_2_2

The block private networks and anti-lockout rules display in the firewall rules display with the schedule column blank. But the block bogon networks has an asterisk. That looks odd. I noticed it just now when looking at firewall rules in 2.3.
Might as well fix it here in RELENG_2_2.

Pull request on master is https://github.com/pfsense/pfsense/pull/2084


Fix Overflow in some Cellphones due to ViewPort

Fix wrong bottom spacing due to vh not including hidding nav bar from browser

Thanks for these fixes! 😊


Auth_SendCode only working for existed account

Hello,
I am using Auth_SendCode to use for Auth_SignUp, and this does not send any SMS to the new phone number.
But it works for existed account which is used for Auth_SignIn (Sending by Telegram App).
Please help me out of this stuck, thanks a lot
This is My code
`
public virtual async void NewClient()
{
try
{
string subPath = "ImagesPath"; // Your code goes here
bool exists = System.IO.Directory.Exists(_path);
if (!exists)
System.IO.Directory.CreateDirectory(_path);
                SqliteC sql = new SqliteC();
               string phoneno =  sql.GetMainPhoneNo("SELECT phone_number from account limit 1" );
                if (phoneno != "")
                {
                    try
                    {
                        _PhoneNo = phoneno;
                        _session_file = phoneno;
                    }
                    catch (ApplicationException e)
                    {
                        System.Windows.Forms.MessageBox.Show("There is an error, please help to login again");
                        AddUserDialog testDialog = new AddUserDialog();
                        testDialog.ShowDialog();
                    }
                }
                _ApiId = Convert.ToInt32(ConfigurationManager.AppSettings["ApiId"]);
                _ApiHash = ConfigurationManager.AppSettings["ApiHash"];
                _client = new WTelegram.Client(Config);
                await _client.ConnectAsync();

            }
            catch (IOException e)
            {
                System.Windows.Forms.MessageBox.Show("Session was using by another process. try Restarting..");
            }
            catch (ApplicationException e)
            {
                System.Windows.Forms.MessageBox.Show("Session was using by another process. try Restarting..");
            }
        }

static string Config(string what)
{
if (what == "api_id") return Convert.ToString(_ApiId);
if (what == "api_hash") return _ApiHash;
if (what == "phone_number") return _PhoneNo != null ? _PhoneNo : "";
if (what == "verification_code") return _verification !=null ? _verification:"";
if (what == "password") return _code != null ? _code : "";
if (what == "server_address") return "<IP_ADDRESS>:443";
if (what == "session_pathname") return _path + _session_file;
if (what == "allow_flashcall") return "0";
if (what == "first_name") return _firstname;
if (what == "last_name") return _lastname;
return null;
}
public virtual async Task<Auth_SentCode> Auth_SendCode(string phone_number)
{
_session_file = phone_number;
_client.Dispose();
NewClient();
var res = await _client.Auth_SendCode(phone_number, _ApiId, _ApiHash, null);
return res;
}
`

https://github.com/wiz0u/WTelegramClient/blob/master/FAQ.md#tlsharp

(and please don't create new Github accounts just for posting new issues, you can still use the previous one)

(and please don't create new Github accounts just for posting new issues, you can still use the previous one)

Thanks for letting me know. The reason this account been created is that I just noticed not good to use a personal account and I should have a working account

(And having multiple accounts isn't one of your business)


Make it possible to change library profile from GUI options

Currently it has 2 sub menu's

Basic Options
Advanced Options

Would be nice to also add Library options, which gives you insights in currently configured manufacter / model and allows to change.

Implemented with #2555


Install envoy.exe into ProgramData/envoy/bin

Title: Install envoy.exe into ProgramData/envoy/bin
Description:
Right now, in the envoy-windows image, envoy.exe installs into Program Files/envoy/envoy.exe
While this does match historic install directories, choosing this makes tooling more difficult as for example make struggles with spaces.
It would be easier to install into ProgramData/envoy/bin/envoy.exe similar to the Chocolatey package manager: ProgramData/chocolately/bin/choko.exe.
Doing so wouldn't be completely unconventional, in other words, and we already write files to this directory, ex. ProgramData/envoy/envoy.yaml

I am not opposed to this. I actually do agree that having spaces on the path is a bit annoying. It is a breaking change though.
@wrowe thoughts?

here's the link from chocolatey explaining their choice https://docs.chocolatey.org/en-us/faqs#why-doesnt-a-package-install-software-to-program-files

I'd like to pick up this issue as a way of getting started contributing to the repo. Do people have any reservations about moving forward with this change?
cc @wrowe @mattklein123 @codefromthecrypt
Who else can I tag to gather opinions?


Feature Request:  support custom wallets

3 words is not a sufficient "feature request" : "support custom wallets".
We put as much effort into understanding this as you put in to elaborating.


Custom labels do not work in file search

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

VS Code Version: 1.88.0
OS Version: MacOS 14.4.1

Steps to Reproduce:

Create some index.html files in multiple sub folders
Configure customLabels (see below)
Search for index.html in file search

settings.json:
  "workbench.editor.customLabels.patterns": {
    "**/index.html": "${filename}.${extname} (${dirname})"
  },

As you hopefully can see in the video, the custom labels are used in the editor tabs and in the open editors switch menu but not in the file search menu:

It should probably match the real filename and also the custom label.
Another issue is with the diff editor, when you open the changes of a file it displays the original filename:

Note that the pattern has wildcards at the end too etc: **/conanfile.json*": "conan - ${dirname(0)}
I guess the "(working tree)" or similar could be added to the list of potential ${variables}, or just always append it at the end.

Could open a separate issue, but seemed related and discussable here

@circlesmiler could you add "and diff editors" to your title please? It is sadly not working for diff editors (both working tree and index).

@benibenj is this a fix that you need @hediet and I to adopt somewhere?

@andreamah We are still deciding on where to show the custom labels and where not. For the time being we are showing it everywhere we render the current open editors. Depending on the communities feedback/wishes we will expand adoption to all other resource labels such as search. In that case I will provide a very simple way to adopt it.


Update comment in pressure.js

of -> or

Thanks @marcusmolchany 👍

thank you @yamartino this library is dope


New functions in MPhys wrapper

Purpose
Adding the following functions to the MPhys wrapper

addLocalSectionDV
getLocalIndex
addRefAxis
writeRefAxis
writeSurfaceSTL

Expected time until merged
2 days
Type of change

[ ] Bugfix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (non-backwards-compatible fix or feature)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Documentation update
[ ] Maintenance update
[ ] Other (please describe)

Testing
soon
Checklist

[ ] I have run flake8 and black to make sure the Python code adheres to PEP-8 and is consistently formatted
[ ] I have formatted the Fortran code with fprettify or C/C++ code with clang-format as applicable
[ ] I have run unit and regression tests which pass locally with my changes
[ ] I have added new tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation

Lucas is testing this with his case, don't merge until we double check that it's good there.

I added other functions I've been needing in MPhys to this PR and updated the info. Lucas said local shape variables have been working for him so I think everything is good to go.

This PR is still ready for review and now has docstrings, by popular demand.

From today's maintenance meeting, we decided to remove methods from the MPhys wrapper that do not add input / outputs for OpenMDAO (with the exception of adding a reference axis). Instead, we should add a method that just returns the DVGeo object so the user can directly interface with the underlying object methods.


⚠ GitLab Server has degraded performance

In dab875e, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 7057 ms

Resolved: GitLab Server performance has improved in 44f2a87.


[Feature Request]: Facebook oauth provider

Package
lucia-auth
Describe the problem
Facebook is probably the second most used oauth provider behind Google, but it's not implemented yet.
Describe the proposed solution
It would be nice to have a Facebook oauth provider.
Alternatives considered
No response
Additional information
No response

Done! Added in @lucia-auth/oauth 0.6.0 with #385


Renamed IT test to Test. There's no need for integration tests. Relat…

…ed to issue #250

Codecov Report

Merging #270 (d88371d) into master (18d5dfc) will increase coverage by 0.36%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #270      +/-   ##
============================================
+ Coverage     94.37%   94.73%   +0.36%
- Complexity     1883     1892       +9
============================================
  Files           195      195
  Lines          3819     3819
  Branches        391      391
============================================
+ Hits           3604     3618      +14
+ Misses          115      102      -13
+ Partials        100       99       -1

Impacted Files
Coverage Δ

.../java/net/datafaker/service/FakeValuesService.java
88.73% <100.00%> (+0.60%)
:arrow_up:

...ker/idnumbers/pt/br/IdNumberGeneratorPtBrUtil.java
92.59% <0.00%> (-3.71%)
:arrow_down:

src/main/java/net/datafaker/Internet.java
90.41% <0.00%> (+1.36%)
:arrow_up:

src/main/java/net/datafaker/Address.java
97.14% <0.00%> (+2.85%)
:arrow_up:

src/main/java/net/datafaker/Commerce.java
100.00% <0.00%> (+4.00%)
:arrow_up:

src/main/java/net/datafaker/IdNumber.java
100.00% <0.00%> (+6.66%)
:arrow_up:

src/main/java/net/datafaker/Lorem.java
97.91% <0.00%> (+8.33%)
:arrow_up:

...ava/net/datafaker/HitchhikersGuideToTheGalaxy.java
100.00% <0.00%> (+11.11%)
:arrow_up:

src/main/java/net/datafaker/OscarMovie.java
100.00% <0.00%> (+16.66%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


fix: ScaledObject docs have all the anchors

Provide a description of what has been changed
Checklist

[ ] Commits are signed with Developer Certificate of Origin (DCO)

Fixes #

Adding the titles to the lateral menu would be nice:

But at least now they have anchors that we can link for sharing info

@tomkerkhove , do yuu know if we can include them in the menu too?


Russian Peasant Algorithm in java

Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Description
(Write your answer here.)
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)

Why is two code file pushed in a single PR? Please make separate PRs for separate Issues being worked on.

how to make separate pr now...
On Wed, 24 Feb 2021, 10:36 am Harsh Mishra<EMAIL_ADDRESS>wrote:

Why is two code file pushed in a single PR? Please make separate PRs for
separate Issues being worked on.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TesseractCoding/NeoAlgo/pull/2163#issuecomment-784766212,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL7HKZHLLVQPJ4XMWN5Y4L3TASCO3ANCNFSM4YB3DNHA
.

how to make separate pr now...
…
On Wed, 24 Feb 2021, 10:36 am Harsh Mishra, @.***> wrote: Why is two code file pushed in a single PR? Please make separate PRs for separate Issues being worked on. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2163 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AL7HKZHLLVQPJ4XMWN5Y4L3TASCO3ANCNFSM4YB3DNHA .

Put your other Code into a separate branch and make a PR from there

I have deleted the cpp file from this pr and for the cpp file there is already a PR open by me...

Resolve the Merge Conflicts.

I am not able to do.. can I make new PR and submit again?
On Fri, 26 Feb 2021, 6:38 pm Harsh Mishra<EMAIL_ADDRESS>wrote:

Resolve the Merge Conflicts.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TesseractCoding/NeoAlgo/pull/2163#issuecomment-786637735,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL7HKZFBCLXOPF2ZOL5WZGLTA6MM7ANCNFSM4YB3DNHA
.


Is YARP production-ready?

My team has been eagerly awaiting for a supported/production-ready(ish?) version of YARP for perhaps a year now 😅.
I searched around but couldn't find any information on this. Is YARP officially supported? Do you consider it production-ready?
If the answer to both is "no", please share your personal experiences if you are using it for production scenarios!
Thanks!

We've been running YARP in production for quite some time now as well and it has been pretty rock-solid so far. Upgrading between previews has been pretty painless as well, except for that one time where we also had to change our configuration due to some breaking changes there. Currently running on the latest preview (including the recently released security patch). We aren't particularly high volume though, although we have peak times of around 2000 RPM.
So to summarise we haven't regretted making the choice to run YARP in production at all.

This issue is really interesting.
Can you also share hardware consumption details ? (number of cores, average CPU, and RAM consumption ?)

We run it on k8s on AWS, and a few of the downstream services are rough on keeping connections open driving up CPU a bit. Probably more tuning to be done, bit more but right now we have:
limits:
  cpu: 500m
  memory: 256Mi

Memory usage is pretty flat at 45% to 60% usage. CPU is volatile from 10% at lowest traffic and 85-95% at peak (until scaling finishes starting a new one).

Then with autoscaling setup, we drop to our min of 3 pods at lowest traffic (in theory this could drop to 1, we just don't allow that), and 6 or 7 at peak.


run build.py

not quite sure where build.py places the binaries here 🤔

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

oh only just noticed #12


API upload of new table in data frame does not check for key column presence

If a JSON object with a data frame without a key-column is uploaded, the operation succeeds.

Fixed in release B.4.1


Replace server.py with something a little more .NET

This static page server is used to make sure wasm files get the right content type and to enable XmlHttpRequests for the assemblies.
Python servers are fine for Linux and Mac, but don't work by default on Windows.
Let's replace it with a nice .NET script instead.
An F# script would be perfect, but I worry if that is too intimidating for C# devs.
Alternatively, maybe a dotnet tool or run configuration or some other magic could be used.

C# devs love F# ;)

Great idea @praeclarum with Forms!
The server.py example is for python 2, so if anyone out there is trying with python 3:
import http.server
import socketserver

PORT = 8000

class Handler(http.server.SimpleHTTPRequestHandler):
    pass

Handler.extensions_map['.wasm'] = 'application/wasm'

httpd = socketserver.TCPServer(("", PORT), Handler)

print("serving at port", PORT)
httpd.serve_forever()

Maybe this can be used instead https://github.com/natemcmaster/dotnet-serve

@hoetz I saw that this morning! Yeah I will definitely check it out.

I've tried it out and dotnet-serve works well! Need to update the guides...

Updated the wiki

You still need to update 7. in the wiki :)

Thanks!

Hi guys,
Just to give ever another option if don't want to update .NET Core to its preview or have already installed Python 3.x like me :-)
Simply use "xsp4" bin from Mono's distribution to run a local server from CLI. In macOS, you can already type this if have Xamarin installed. In Windows, simply point to mono-project.org and download it.
Thanks!
-Marcos


Change the memory policy

Hi,
In Arm Compute Library, it is possible to select different memory policies.
For instance, this can be done using the following line of code:
arm_compute::MEMInfo::set_policy(arm_compute::MemoryPolicy::MINIMIZE);

How can we change the memory policy in ARM NN?

ArmNN doesn't touch the MEMInfo API (leaves it all at defaults) so you can just call set_policy in your program before creating the Arm NN Runtime.


Duplicate files under generated build flavours

I have similar issue as explained in issue #31
Files are generated for annotated classes under both release/ and debug/ build folders when running ./gradlew assemble on my project. This results in redeclarations of files, as the same files are then found from both paths.
This can be mitigated by explicitly pointing to either debug or release outputs in the build.gradle.kts of the shared module:
kotlin.srcDir("${buildDir.absolutePath}/generated/source/kaptKotlin/")
to
kotlin.srcDir("${buildDir.absolutePath}/generated/source/kaptKotlin/debug/")
Is there a way to go about this without hardcoding the path to the generated outputs of a specific flavour? I've not yet found a way.

I'm not aware of any other way yet  but I will look into that.

Related issue: https://youtrack.jetbrains.com/issue/KT-50980

This will no longer be relevant in ksp version, which I'm planning to release soon.

Fixed in 0.11.1 with ksp


PSGO-596 - TO SPLIT

Temporary: This PR is divided into smaller parts.

@michaljurecko could we close this PR?

@Matovidlo no, there is the last commit Provisioning WIP


[tool_crash] IndexError: RangeError (index): Index out of range: index should be less than 47959178: 47966680

Command
flutter run --flavor appCenter

Steps to Reproduce

...
...
...

Logs
IndexError: RangeError (index): Index out of range: index should be less than 47959178: 47966680
#0      _Uint8ArrayView.[] (dart:typed_data-patch/typed_data_patch.dart:4073:7)
#1      InputStream.readUint32 (package:archive/src/util/input_stream.dart:229:22)
#2      new ZipDirectory.read (package:archive/src/zip/zip_directory.dart:49:34)
#3      ZipDecoder.decodeBuffer (package:archive/src/zip_decoder.dart:19:30)
#4      ZipDecoder.decodeBytes (package:archive/src/zip_decoder.dart:14:12)
#5      _WindowsUtils.unzip (package:flutter_tools/src/base/os.dart:403:42)
#6      ArtifactUpdater._downloadArchive (package:flutter_tools/src/cache.dart:969:18)
<asynchronous suspension>
#7      EngineCachedArtifact.updateInner (package:flutter_tools/src/cache.dart:775:7)
<asynchronous suspension>
#8      CachedArtifact.update (package:flutter_tools/src/cache.dart:677:5)
<asynchronous suspension>
#9      Cache.updateAll (package:flutter_tools/src/cache.dart:546:9)
<asynchronous suspension>
#10     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1144:7)
<asynchronous suspension>
#11     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1043:27)
<asynchronous suspension>
#12     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#13     CommandRunner.runCommand (package:args/command_runner.dart:196:13)
<asynchronous suspension>
#14     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:284:9)
<asynchronous suspension>
#15     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)

[✓] Flutter (Channel stable, 2.2.1, on Microsoft Windows [Version 10.0.19042.928], locale pt-CV)
    • Flutter version 2.2.1 at C:\src\flutter
    • Framework revision 02c026b03c (13 days ago), 2021-05-27 12:24:44 -0700
    • Engine revision 0fdb562ac8
    • Dart version 2.13.1

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\tania.santos\AppData\Local\Android\Sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = C:\Users\tania.santos\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[✓] Chrome - develop for the web
    • Chrome at C:\Users\tania.santos\AppData\Local\Google\Chrome\Application\chrome.exe

[✗] Visual Studio - develop for Windows
    ✗ Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[✓] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)

[✓] VS Code (version 1.56.2)
    • VS Code at C:\Users\tania.santos\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.17.0

[✓] Connected device (4 available)
    • sdk gphone x86 arm (mobile) • emulator-5554 • android-x86    • Android 11 (API 30) (emulator)
    • Windows (desktop)           • windows       • windows-x64    • Microsoft Windows [Version 10.0.19042.928]
    • Chrome (web)                • chrome        • web-javascript • Google Chrome 73.0.3683.103
    • Edge (web)                  • edge          • web-javascript • Microsoft Edge 91.0.864.41

! Doctor found issues in 2 categories.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: 8874f21e79d7ec66d0457c7ab338348e31b17f1d
Plugins
flutter_appauth-0.9.2+6
flutter_appcenter_bundle-3.3.1+2
flutter_native_image-0.0.5+3
flutter_plugin_android_lifecycle-2.0.2
flutter_secure_storage-3.3.5
image_cropper-1.3.1
image_picker-0.7.5+3
image_picker_for_web-2.0.0
path_provider-2.0.1
path_provider_linux-2.0.0
path_provider_macos-2.0.0
path_provider_windows-2.0.1
shared_preferences-2.0.5
shared_preferences_linux-2.0.0
shared_preferences_macos-2.0.0
shared_preferences_web-2.0.0
shared_preferences_windows-2.0.0

@TaniaDosSantos
Have you setup environment variables properly ?
Can you try to reboot or flutter clean and then flutter run to see if it helps ?

Hi!
Yes, I've done all the steps that you mention above an still have the same issue.

There's similar open issue, but the stacktrace is different from above, https://github.com/flutter/flutter/issues/76029. Need to confirm if this can be closed in favor of the the linked one.
Also, there were below occurences of the same issue but they got closed due to inactivity:
https://github.com/flutter/flutter/issues/77482
https://github.com/flutter/flutter/issues/65325

In flutter web 2.10.5
I facing this issue "Index out of range:no indices are valid: 0"
and I am using item count and accessing the correct index.

@TaniaDosSantos  @ps6067966
Is there a way for you to confirm if this issue still replicates using latest stable version of framework ?


Json Message sent to IoTHub is not valid

It seems that when constructing a json message with several monitoredItems, the outputted json is invalid. It miss the square brackets (first and last characters) and commas between the monitoredItems. Json message was valid with IoTHubMessaging.cs before commit https://github.com/Azure/iot-edge-opc-publisher/commit/194f30c8b81d3c16f32a8aa3be93e937aae5278f#diff-51136702e85626f290d9396364799eda so probably a regression was introduced with the monitoredItemQueue. Thanks

@banchio: yes, this is too bad. i have a fix in the works with more options to configure the telemetry which is ingested. pls check out the preview in https://github.com/hansgschossmann/iot-edge-opc-publisher
the fix will ingest always an array even there is only one single message and the data is consumable by JSON deserializers right away.

this is fixed with 60c558f50da63bf48d78225cbbdeee4abd61eff5


UX: sending coins to yourself (sender wallet == receiver wallet) should be noted in the "Approve Transaction" page

Motivation
I think users should be notified that they are trying to run a transaction whose receiver address is the same as the wallet's address they're running the transaction from.
Usage example
I tried sending ETH to myself and the "Approve Transaction" page looks like this:

Possible implementations
I've created a quick mockup:

Another solution might be to show a warning dialog when the user clicks "Approve".

I'll try to do this
I think this solution is a little better
add text tip NOTE:the sender and receiver address are same

After looking for the relevant code.
I plan to add a component to display an tip message that needs to be displayed here if the status of the transaction is a failure.


Can update a user's username to contain spaces

I'm submitting a ...

[X] bug report
[ ] new feature / enhancement request
[ ] improvement request (usability, performance, tech debt, etc.)
[ ] other

Traffic Control components affected ...

[ ] CDN in a Box
[ ] Documentation
[ ] Grove
[ ] Traffic Control Client
[ ] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[x] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] unknown

Current behavior:

When making a PUT request to /user/current with the following request body:
{
    "user": {
        "username": "space   test"
    }
}

The response returned is:
HTTP/1.1 200 OK
{
    "alerts": [
        {
            "text": "User profile was successfully updated",
            "level": "success"
        }
    ],
    "response": {
        "username": "space    test",
         .
         .
    }
}

Expected / new behavior:

Should not be allowed to update a username that contains spaces.
Should return:
HTTP/1.1 400 Bad Request
{
    "alerts": [
        {
            "text": "Couldn't parse request: username: cannot contain spaces",
            "level": "error"
        }
    ]
}

Minimal reproduction of the problem with instructions:

Anything else:

trying to think if a username with spaces would cause problems. is it weird? yes. does it cause real problems? not sure.

I think you can also create a user having a username with spaces, though the TP form doesn't like it. In any case I think changing this would break compatibility with the Perl; that is, if memory serves. I can check in a day or two to be sure.

Yeah, the Perl would let you do that and you can also create users with spaces in their username. I think this is so far "works as designed".

@ocket8888 - maybe rather than closing this, it should be reopened and the "fix" is to make TP and the TO API consistent. Meaning that TP should allow spaces as well. Thoughts?

Yeah, that sounds appropriate to me.

@ocket8888 @mitchell852 - I've included a fix for this issue in my PR #6895
Could you review it?


Truncate time to milliseconds

If we run a spine-based project under the Java 11 platform we randomly may encounter the issue when Time.currentTime() returns an invalid timestamp.
The issue is that Time uses the SystemTimeProvider by default, which rely on the fact that Instant.now() returns time in millisecond precision, that was true for Java 8. The SystemTimeProvider emulates nanoseconds in order to split in time events that happened in the same millisecond, but if we have time with increased precision it leads us to the situation when the resulting amount of nanoseconds with emulation is bigger than allowed. This happens because starting from Java 9 Instant.now() may return time with increased precision if the underlying clock supports it. This issue also described in #553.
In this PR I made the SystemTimeProvider explicitly truncate time to the millisecond precision.

@armiol PTAL.

@armiol PTAL again

@armiol PTAL again.


Input file is be corrupted, when it contains leading spaces

Problem: Leading spaces are removed from the input file.
Example:
For day 5 the example input is:
    [D]
[N] [C]
[Z] [M] [P]
 1   2   3

move 1 from 2 to 1
move 3 from 1 to 3
move 2 from 2 to 1
move 1 from 1 to 2

The leading spaces in line 1 are removed, even though they are significant (as '[D]' is part of the second pile, not the first.
I suggest to not alter the input file in any way.
Elves can be pretty specific with whitespace… I guess, that's what happens, when you live so far in the North. 😃

Nevermind, it's already fixed with 221ca47ae1fbc94ca0f75f791833f7d39c6e6940.


The onChange event not fire on time change (when included with date)

The onChange event only seems to fire when selecting a date from the picker or changing the date manually but not when the user changes the input Time. This makes it impossible to do any sort of validation on user input.
The current workaround is to set readOnly to true so that it forces users to only use the picker, but it would be ideal to also let them type into the box.
how can I detect the time manually changes?
"react-datepicker": "^0.60.2",
"react": "^15.6.1",

I have same problem. As temporal fix you can provide onSelect property that
duplicate of onChange.
    if (!isSameDay(this.props.selected, changedDate) || this.props.allowSameDay) {
      if (changedDate !== null) {
        if (this.props.selected) {
          changedDate = setTime(newDate(changedDate), {
            hour: getHour(this.props.selected),
            minute: getMinute(this.props.selected),
            second: getSecond(this.props.selected)
          }) // <------- bug, i need to receive time from new date instead of old selected date
        }
        this.setState({
          preSelection: changedDate
        })
      }
      this.props.onChange(changedDate, event) // <--------- fires only if day changed
    }

    this.props.onSelect(changedDate, event) // <--------- fires always

A workaround is to parse the value with onChangeRaw and call onChange manually.
<DatePicker dateFormat="YYYY-MM-DD HH:mm"
            onChange="this.handleInputChange"
            onChangeRaw="this.handleInputChangeRaw" />

handleInputChangeRaw = e => {
  const date = moment( e.target.value, "YYYY-MM-DD HH:mm" );

  if( date.isValid() ) {
    this.handleInputChange( date );
  }
}

<DatePicker dateFormat="YYYY-MM-DD HH:mm" onChange={(e) => this.handleInputChange(e)}             onChangeRaw={(e) => this.handleInputChangeRaw(e)} />

The issue still hasn't been resolved :-\

Can we not get this fixed? This is seriously a core piece of functionality. I'm surprised it was overlooked.

Yes this issue is still present on Android phones why can't this be fixed?

This issue is a blocker. Cannot use a component that breaks basic, standard browser behavior.

any updates?

issue still present ....

still no solution...

This is still an open issue. Best solution would be to handle onSelect={(event) => event.target.value}.


Image doesn't crop when it is selected fully

I selected a image with its full size (width,height) from the gallery and that picture taken from the camera .
It sets the image in black color in the image view

what?

Same problem here in 4.4.2 android API please check it.

I need more info

you can see the screen shot for problem in this link.
https://www.dropbox.com/sh/jdga47oxg9nw54l/AAA4u5q5qOw1MICXK4BwE-GWa?dl=0
regard.

404 on the link

i have updated the link please check again.

Thanks,
Can you reproduce it in the sample app?

I resolved issues with custom circle image view and advice not to use hdodenhof circle image view because this causes the issue when i crop a high resolution image from camera , it sets black in the image view

I think what you see is Android max allowed texture size limitation (example question)
I would advise limiting the size of the cropped image, I don't think you need 3000x3000 avatars...
See: https://github.com/ArthurHub/Android-Image-Cropper/wiki/FAQ#how-can-i-limit-the-size-of-the-cropped-image


Update renovatebot/github-action action to v34.84.1

This PR contains the following updates:

Package
Type
Update
Change

renovatebot/github-action
action
minor
v34.82.2 -> v34.84.1

Release Notes

renovatebot/github-action
v34.84.1
Compare Source
See the the changelog for changes in all releases.
34.84.1 (2023-01-06)
Bug Fixes

deps: update renovate/renovate docker tag to v34.84.1 (7bf255e)

v34.83.1
Compare Source
See the the changelog for changes in all releases.
34.83.1 (2023-01-05)
Bug Fixes

deps: update renovate/renovate docker tag to v34.83.1 (34618f3)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Branch automerge failure
This PR was configured for branch automerge. However, this is not possible, so it has been raised as a PR instead.


Treat the Identity Field as a Duplicated Field

This PR addresses a performance issue where querying over the identity field does not hit the identity column.
BEFORE:
select _ from _ as d where (d.data @> :arg0)
where arg0 = '{"Id":"00ee1302-7109-42e0-bd87-926123931c34"}'
AFTER
select _ from _ as d where (d.id = :arg0)
where arg0 = '00ee1302-7109-42e0-bd87-926123931c34'

 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.:x: Spotlight-Zachary sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Add ConfigurationObjectFactory project

This adds a project, RockLib.Configuration.ObjectFactory, as a competitor of the Microsoft.Extensions.Configuration.Binder package. Where the Microsoft package has Get<T>() and Get(Type type) extension methods, this package offers Create<T>() and Create(Type type) extension methods.
There are two advantages of this package over Microsoft.Extensions.Configuration.Binder:

It can create objects that do not declare a public parameterless constructor. Microsoft's binder only works with types that declare a public, parameterless constructor. This package only requires that constructors be public.
It can create instances of objects with polymorphic members. For example, if a class has a property that is abstract, the user's configuration needs to supply the concrete type to use at runtime that implements the abstract type. Similarly, if a property is non-abstract and not sealed, a derived type can be specified.

Note that the Microsoft.Extensions.Configuration.Binder package provides a few other extension methods, but I find little value in them, so I did not implement them.

With changes from feedback, looks good to me for alpha01.  I'll be adding some changes after this merge as well.


DONT DOWNLOAD THIS! ITS VIRUS

After downloading this and opening few times it gives error and after all that my internet will start getting down and bad
I HAVE NOT DOWNLOADED AND OPENED ANY OTHER FILES FROM GITHUB ONLY THIS!
SO its stressing your net. DO NOT DOWNLOAD

it's not a virus , code isn't been updated for 2-3 years you can join the server check the source code yourself


Overhaul the UI

Today, it's a very simple buffer to show session information and some other minor details.
To revamp and extend the UI, I'd like to pull in the extendable UI that mason.nvim built out that supports tabs, collapsible lists, etc.
Steps needed are

Map out what views we'd support
Pull in core source code - it's licensed under Apache 2, so just need to check what that does (if anything) with our dual license and how we properly attribute credit
Migrate existing DistantSessionInfo and other commands to open specific views in the UI

Thoughts in terms of views to support

List connections available in the manager so we can reconnect to one (@fabiomcosta this would make it easier to select different servers per our discussion) and show existing connection details
Open files for the current neovim session (not connection)
Active processes for the current neovim session (not connection because that cannot be retrieved)

I'm not sure if that UI makes sense for switching between servers/projects, I was thinking about using telescope for that.
The benefit of the telescope ui is that you can quickly filter options, switch between and possibly show a preview of what that config looks like. This demo represents what that would look like: https://github.com/nosduco/remote-sshfs.nvim/blob/main/demo.gif
What about using this UI for debugging info only?
Most of the other info you listed seems to match that criteria:

Session info
Open files for the current neovim session (not connection)
Active processes for the current neovim session (not connection because that cannot be retrieved)
We could also show open connections, but not necessarily provide ways to interact with those connections (that would happen through the telescope list UI)

@fabiomcosta this plugin doesn't require any other plugin for core functionality, so I want to avoid depending on telescope for something crucial. On the other hand, we discussed this as a "nice to have", so I have no problem with us adding another extension to telescope alongside the searching feature.
I'm assuming the gif you shared uses telescope? And the servers available probably come from the ssh config. So where do we want to pull in the server information? Have it be in the neovim config? Some arbitrary config file? Or were you thinking of reading the ssh config to support multiple servers for DistantLaunch?

The UI features that mason has that I wanted to port over also have a search feature, so wondering if that would be enough. I know the gif you showed had more info about the connection, of course.

I'm assuming the gif you shared uses telescope?

It doesn't look like telescope on the gif... but the code seems to use it
https://github.com/nosduco/remote-sshfs.nvim/blob/main/lua/telescope/_extensions/remote-sshfs.lua

So where do we want to pull in the server information? Have it be in the neovim config? Some arbitrary config file? Or were you thinking of reading the ssh config to support multiple servers for DistantLaunch?

I was thinking about having that on the neovim config.
DistantLaunch already uses that, so I guess it would be about maybe adding some extra fields there.
I still don't have all figured out, especially in terms of general ssh/server functionality vs internal use cases like ondemands.
For example, for ondemands we might not want to provide a fixed address, it would be about the "type" (www, etc.) and we'd always pick the first one of that type or launch a new one, or maybe let the user select the ones available in case there are more than one up.

IMO this mason UI is not meant for quick actions, it's more like "let's open mason and see what is available here", instead of "let's open mason and connect to my X server".
Do you know what I mean? haha

Branch now has a shell of a UI, but also has a lot of refactorings within the plugin itself. So planning to merge soon after an additional suite of tests are written.

@fabiomcosta check out the new UI in progress. I also show that neovim is using a distant manager running separately from neovim itself, which means it can be shared across neovim instances and you can close down neovim without losing the connection.
https://github.com/chipsenkbeil/distant.nvim/assets/2481802/dd594b49-a55d-4d14-87bc-309c79ca3d2a

Awesome!
I like it to gather an overview of what is going on ATM, what is the "global state" with a nice view, but I'm unsure about the need to allow users to change the connection there, it would be nice to provide a dedicated UI for that so they can quickly change between connection.
I built that with telescope, but we could use any UI, as long as ppl can quickly change between connections.

@fabiomcosta would you be able to provide me with a little more clarity on what you'd like to (or need to) see to give you a global state overview? What information is useful to know?
I'm planning to merge in my branch soon, so if I don't get to it beforehand, you or I could add it. The UI code is actually fairly easy to work with like a neovim version of React.

Hey Chip, maybe I didn’t express myself right, I  don’t see anything missing at this point, it is already showing the global state on a nice UI.
I guess I’ll have to play with it and if I notice any info missing I can add.
On the separate connection selector that I mentioned, I gave it more thought and your current solution might end up being good enough, changing the connection is not something that people will be doing frequently.
#chipit :P

Lol, nice one. 😄
I just re-worked buffer names because I realized that working with more than one server at once within the same neovim instance would cause problems because the buffer name could be the same. If you visited /home/fabio on two machines, the buffers would collide.
That's fixed now! Works pretty well and distant v0.20.0-alpha.5 is finally published, so I'll be merging this branch soon. You'll need to update to that version when I do. Just got to update all of the tests.

@fabiomcosta another request for feedback. I now have the plugin parse your settings for servers, and make those available for you to launch. Thoughts?
https://github.com/chipsenkbeil/distant.nvim/assets/2481802/df17dea5-568f-417c-a69f-170690601789

Great addition!
Will be super useful for the general user, just not for ODs, but that will
be addressed separately anyway
On Sun, May 14, 2023 at 7:51 PM Chip Senkbeil @.***>
wrote:

@fabiomcosta https://github.com/fabiomcosta another request for
feedback. I now have the plugin parse your settings for servers, and make
those available for you to launch. Thoughts?
https://github.com/chipsenkbeil/distant.nvim/assets/2481802/df17dea5-568f-417c-a69f-170690601789
—
Reply to this email directly, view it on GitHub
https://github.com/chipsenkbeil/distant.nvim/issues/89#issuecomment-1547032864,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAG64X3SG5DHCMEPTCSOBDXGFVXJANCNFSM6AAAAAAWU4GZZQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Fabio Miranda Costa (he/him/his)
twitter: @fabiomiranda https://twitter.com/fabiomiranda
github: fabiomcosta https://github.com/fabiomcosta/
linkedin: fabiomirandacosta
https://www.linkedin.com/in/fabiomirandacosta/

Closing this out as the UI is in the upcoming v0.3 branch.


Update Rubygems to 2.6.13+ on Chef 12 branch

Description
Rubygems 3.6.12 has security issues.  Chef 13 was updated to Rubygems 2.6.13 PR #6365 and I would like this also back ported to Chef 12 branch.
Chef Version
Chef 12.21+
Platform Version
CentOS
Replication Case
n/a
Client Output
n/a

sorry, just noticed this is a dupe of https://github.com/chef/chef/pull/6366.  I'll close this one out


[5.2] Raise exception occurred job event before releasing the job

My pull request for the JobExceptionOccurred event got merged about 6 days ago (#12659) the only problem is that the event should be fired before releasing the job so people can append data to the job or could potentially release the job manually in an event listener (by listening to the JobExceptionOccurred event) based on certain conditions as an example.
Releasing the job in an event listener wouldn't cause any problem because of the following part of code that checks if the job is deleted before releasing it.
if (! $job->isDeleted()) {
    $job->release($delay);
}

Surely this is a breaking change?

No it's not, it's just better to have it before for the reasons above.

On Mar 15, 2016, at 8:10 PM, Graham Campbell<EMAIL_ADDRESS>wrote:
Surely this is a breaking change?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

What if an event handler throws an exception? The job will now get lost forever.

I guess we need a try finally to fix that.

Ok I'll do it, nice catch

On Mar 15, 2016, at 8:20 PM, Graham Campbell<EMAIL_ADDRESS>wrote:
I guess we need a try finally to fix that.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

@GrahamCampbell ping

We still lose the first exception in an error occurs in the event handlers though.

I guess that was always an issue though.

Right

On Mar 16, 2016, at 5:04 AM, Graham Campbell<EMAIL_ADDRESS>wrote:
I guess that was always an issue though.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

@GrahamCampbell Is there anything else I need to fix/work on to merge this pull request?

This change is still wrong. What I meant was add another try finally block, because now you're calling that code when there wasn't even an exception.

@GrahamCampbell This seems to be working! Now the question is: should we stack the exceptions or only throw one of them.
<?php

try {
    try {
        throw new \Exception('First');
    } catch (\Exception $e) {
        try {
            throw new \Exception('Second');
        } finally {
            // ...

            throw $e;
        }
    }
} catch (\Exception $e) {
    do {
        print $e->getMessage() . PHP_EOL;
    } while ($e = $e->getPrevious());
}

// First
// Second

<?php

try {
    try {
        throw new \Exception('First');
    } catch (\Exception $e) {
        try {
            throw new \Exception('Second');
        } finally {
            // ...
        }

        throw $e;
    }
} catch (\Exception $e) {
    do {
        print $e->getMessage() . PHP_EOL;
    } while ($e = $e->getPrevious());
}

// Second

        } finally {
            // ...

            throw $e;
        }

No.  PHP will just discard anything thrown from there.

The two examples above worked for me though ... I was just wondering which one you wanted so I guess it's ok like this right?

@GrahamCampbell ping

@GrahamCampbell ping

@GrahamCampbell anything?

It not seems like there is no way to ever know what exception was thrown from the event handler, but need @GrahamCampbell to provide feedback.

Yes, this looks fine.


Syntax colorization/indentation takes a long time to apply for new code in large files

Something @ahejlsberg and I noticed in a recent meeting - if you open up checker.ts in TypeScript today, it quicky colorizes the area of the file you're viewing; however, if you start adding a function declaration towards the middle or bottom of the file, colorization takes a noticeable bit of time unless you try scrolling. Scrolling seems to get a bunch of content "fixed", and new text only gets the right colorization on scroll.
Here's a video of some of this in action.
https://user-images.githubusercontent.com/972891/145652457-73d9d302-9db8-467e-85d3-153194fa6823.mp4
Eventually, the file starts to always get accurate colorization and smart-indentation, but it took at least over 30 seconds from what we remembered seeing.
This occurs in VS Code's latest (1.63 I believe) along with Insiders, both on Windows.
Closest similar issue I could find was https://github.com/microsoft/vscode/issues/138822

When I saw the video, I immediately thought that this must be a recent regression. But in my attempt to track down the first bad commit, I have tried with our releases from the past 6 months, and then jumped all the way back to 1.40.2 from October 2019 and I always get the current behavior.
Basically, changing the text buffer never ran a viewport tokenization, which is quite an oversight, but not a recent regression.

@alexdima You're right, if start VS Code on a large file (like checker.ts) and edit towards the end of the file, colorization doesn't happen for the first few seconds or so. But with 1.63 it takes more like a minute before the white text gets colorized. I just verified by reverting to 1.61. There's definitely a big change in behavior.

@ahejlsberg You're right, tokenization is now running for 2ms, and then yielding for 14ms, making it roughly 8 times slower:

I think this bad yielding behavior is caused by https://github.com/microsoft/vscode/pull/137646 , where we moved away from setImmediate and to requestIdleCallback.
fyi @jrieken @bpasero

I've created https://github.com/microsoft/vscode/issues/139072 to track that typing does not trigger viewport tokenization and I suggest to use this issue to track that background tokenization takes a lot longer now.

Steps to verify:

open checker.ts
have minimap enabled so you can see when background tokenization reaches the bottom
scroll to the bottom and then do a window reload
tokenization should reach the bottom in ~10-15s
(you can optionally take a JS CPU trace and check that there aren't a lot of idle gaps in there)

Verified. Took ~11 seconds on my machine.


Snap doesn't support vim-plug

I copied your example config (https://gist.github.com/camspiers/686395ab3bda4a0d00684d72acc24c23) to a snap.lua and then called it in my init. I get the following error:
E5113: Error while calling lua chunk: /home/myuser/blabla/snap.lua:1: loop or previous error loading module 'snap'
I installed snap with Plug 'camspiers/snap'.

I suspect this means snap doesn't currently support being installed by vim plug. Genuine issue here, thanks!

I had no issues installing with Vim Plug. I did find that the key bindings did not seem to work as given in the examples

I had the same issue with nvim-treesiter, it's not vim-plug or this plugin, but you have to structure your init file in a specific way.
Here is how it works for me with vim-plug:
call plug#begin(expand('~/.local/share/nvim/plugged'))
  Plug 'camspiers/snap'
call plug#end()

lua <<EOF
local snap = require'snap'
snap.register.map({'n'}, {'<LEADER>f'}, function ()
  snap.run {
    producer = snap.get'consumer.fzf'(snap.get'producer.ripgrep.file'),
    select = snap.get'select.file'.select,
    multiselect = snap.get'select.file'.multiselect,
    views = {snap.get'preview.file'}
  }
end)
EOF

Hello Cam :)
Fantastic work!

I did not try to use it with fzy though, that seemed like it might not work... but maybe it does, i'd be curious to see that.

I had no issues installing with Vim Plug. I did find that the key bindings did not seem to work as given in the examples. This configuration works for me (in vim config directory as lua/snap/init.lua):
local snap = require 'snap'

local fzf = snap.get 'consumer.fzf'
local limit = snap.get 'consumer.limit'
local producer_file = snap.get 'producer.ripgrep.file'
local producer_vimgrep = snap.get 'producer.ripgrep.vimgrep'
local producer_buffer = snap.get 'producer.vim.buffer'
local producer_oldfile = snap.get 'producer.vim.oldfile'
local select_file = snap.get 'select.file'
local select_vimgrep = snap.get 'select.vimgrep'
local preview_file = snap.get 'preview.file'
local preview_vimgrep = snap.get 'preview.vimgrep'

local s = {}

s.files = function()
    snap.run({
        prompt = 'Files',
        producer = fzf(producer_file),
        select = select_file.select,
        multiselect = select_file.multiselect,
        views = {preview_file}
    })
end

s.grep = function()
    snap.run({
        prompt = 'Grep',
        producer = limit(10000, producer_vimgrep),
        select = select_vimgrep.select,
        multiselect = select_vimgrep.multiselect,
        views = {preview_vimgrep}
    })
end

s.buffers = function()
    snap.run({
        prompt = 'Buffers',
        producer = fzf(producer_buffer),
        select = select_file.select,
        multiselect = select_file.multiselect,
        views = {preview_file}
    })
end

s.oldfiles = function()
    snap.run({
        prompt = 'Oldfiles',
        producer = fzf(producer_oldfile),
        select = select_file.select,
        multiselect = select_file.multiselect,
        views = {preview_file}
    })
end

return s

Then, I made bindings in my init.vim:
nnoremap <leader>ff <cmd>lua require'snap'.files()<cr>
nnoremap <leader>fg <cmd>lua require'snap'.grep()<cr>
nnoremap <leader>fb <cmd>lua require'snap'.buffers()<cr>
nnoremap <leader>fo <cmd>lua require'snap'.oldfiles()<cr>

I did just as you did and I still can't get it to work.

I have tested and I was able to get snap installed with vim-plug, so this might be a different issue than general support. Hey @fabiomcosta, hope you are doing well!

I got it to work by swapping in fzf for fzy in https://gist.github.com/camspiers/686395ab3bda4a0d00684d72acc24c23
I'd installed fzy with homebrew but snap apparently requires the lua port which I didn't investigate how to install.
It's working super nicely!


Relax Minimum Python to 3.8

Thanks for building this plugin! I would like to have WikiLinks formatted properly, but I need to support older versions of Python (^3.8).
Could you please consider making a new release that supports ^3.8?
For now, my workaround has been to copy over the logic: https://github.com/KyleKing/mdformat-mkdocs/commit/5f686f1f3d461318673045b30ace574adedddb61

Thanks for reaching out!
Should be solved with #7 . The new version is 0.2.0

Thanks!


feature: myProfile is updated

Added functionality for populating values in the fields(except for profile picture)
Added change password button

ChangePassword button will get visible when user clicks on 'Edit' button.
ChnagePassword button is hidden on page load

can not edit any field and can not see the password change button

for languages / cities / locations take a look at the search bar and the registration form, where there is a new dropdown method and json files linked to it containing a list of these options

for location, we are not using the post code anywhere in the app, so maybe better to use
this.result.location = this.result.location[1]

-new dropdown method implemented for languages/Skills
-changepassword button get visible when user clicks on edit button

@deepa-thomas thank you, looks much better now! do you also plan to add the save profile functionality? and delete profile?

@deepa-thomas thank you, looks much better now! do you also plan to add the save profile functionality? and delete profile?

Yes I will do

do you want to do it in this PR or a separate one?

do you want to do it in this PR or a separate one?

I will do separate PR for update and delete

@deepa-thomas the delete profile function could look something like this
    async deleteProfile() {
      try {
        if (confirm("Are you sure you want to delete your profile?")) {
          const userId = this.user.userId;

          const res = await fetch(
            `http://localhost:5000/users/delete/${userId}`,
            {
              method: "DELETE",
            }
          );
          console.log(res);
          alert("Profile deleted");
        }
      } catch (error) {
        console.log(error);
      }
    },
    ```

@deepa-thomas the delete profile function could look something like this
    async deleteProfile() {
      try {
        if (confirm("Are you sure you want to delete your profile?")) {
          if (confirm("Please confirm again to delete your profile.")) {
            const userId = this.user.userId;

            const res = await fetch(
              `http://localhost:5000/users/delete/${userId}`,
              {
                method: "DELETE",
              }
            );
            console.log(res);
            alert("Profile deleted");
          }
        }
      } catch (error) {
        console.log(error);
      }
    },

Thanks Daniel for the code. I will do it

hi @deepa-thomas ! Great work, thank you! I have one thing to say – a user is supposed to see his/her profile picture in the profile, could you try to add code for that?

Yes Ekaterina. I am working on it

loading profile picture
-icon to change the profile picture is hidden on page load
-icon get visible when user clicks on edit button for updating the profile pic

@deepa-thomas can this be reviewed now?


Optimize entries_size in LogBatch

Signed-off-by: Fullstop000<EMAIL_ADDRESS>

Close due to duplicated with #46


add doro token

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:AXUjVTpm2SWr2fjnQpJbLzy6dSGXPMTjPjWn4LxHjrKC
Token Name:doro
Token Symbol:DR
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

Refer this https://github.com/solana-labs/token-list/pull/5803 i can help

asset too large: assets/mainnet/AXUjVTpm2SWr2fjnQpJbLzy6dSGXPMTjPjWn4LxHjrKC/logo.png is 783 KiB (must be less than 200 KiB)


Simplification problem at low zoom

Hi I have a pretty big  json with more than 1Gb of datas and a lot of polygons inside.
Here is the view of the input SHP file (I use a geojson of it):

Here are some tests:
tippecanoe \
-zg \
-o rpg2018-9.mbtiles \
--drop-densest-as-needed \
CODE_GROUP_1.json

As you can see the curve are too simplify in the center polygon

tippecanoe \
-z14 \
-o rpg2018-11.mbtiles \
--drop-densest-as-needed \
--simplify-only-low-zooms \
--no-feature-limit \
--no-tile-size-limit \
CODE_GROUP_1.json

Quite same result with --simplify-only-low-zooms

tippecanoe \
-zg \
-o rpg2018-10.mbtiles \
--drop-densest-as-needed \
--no-line-simplification \
--no-feature-limit \
--no-tile-size-limit \
CODE_GROUP_1.json

With --no-line-simplification I've got too much aberrations

How can I get a result similar to my geojson input?
Thanks !


Documentation for semantic validation rules

Semantic validation is awesome, but I think it would be useful to have documentation on what validations swagger-tools performs. I think that would be useful for:

Helping people decide to use Swagger in the first place. OpenStack is considering a transition from WADL to Swagger, and seeing what kinds of validation are built into the tools would be useful.
Helping API designers/documenters write better docs. The rules would be a good reference document for them.
Feature requests / pull requests. Knowing what semantic validation is already done makes it easier to see opportunities for new or improved rules.
Portability. I'm working on a Ruby parser for Swagger. Right now I suggest people use it alongside swagger-tools since you've got the most complete validator, but it would be nice to have a pure Ruby solution that validates most, if not all, of the same rules.

A few examples of how it could be documented:

Rubocop's rules mostly correspond to the Ruby style guide. That guide shows examples of mistakes ("bad") along with suggested fixes (ranked "ok", "good" and "better").
Foodcritic lists rules on their site, also with examples of "wrong" and "correct" ways to do things.

This is an excellent idea.  I am currently working on #8 so I'll be scanning the code.  I bet I can go ahead and do that at the same time.

Awesome.
I could only find markdown documentation for swagger-tools, but I saw that the code is also documented via comments. Is there a site where I could see what the rendered documentation looks like, or a command I can run to build it locally? I'm curious if you think you'll be able to create documentation straight from the source or if it needs to be written and maintained as markdown.

I do not have any annotated sources but I could create some.  I think documenting this via Markdown would be ideal.  As for building/running locally, do this:
# Install Gulp
npm install -g gulp

# Clone This Project
git clone https://github.com/apigee-127/swagger-tools.git

# Change Directory
cd swagger-tools

# Install Dependencies
npm install -d

# Run tests
gulp test-node

My plan is to have a document up soon, by end of week, that explains the validation I'm doing.  At that point, you should have what you need.

Here is the first pass: https://github.com/apigee-127/swagger-tools/blob/master/docs/Swagger_Validation.md
I have created issue #143 to keep track of the changes in validation for the existing validations so if you disagree with some of the validations in any way, please update that ticket.


Add initial tests for logging

Codecov Report

Merging #3 (9951696) into main (a75a9d6) will increase coverage by 0.86%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main       #3      +/-   ##
==========================================
+ Coverage   92.81%   93.67%   +0.86%
==========================================
  Files           6        7       +1
  Lines         167      174       +7
==========================================
+ Hits          155      163       +8
+ Misses         12       11       -1

Flag
Coverage Δ

unittests
93.67% <100.00%> (+0.86%)
:arrow_up:

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

Impacted Files
Coverage Δ

openpipe/test/test_logging.py
100.00% <100.00%> (ø)

openpipe/logging.py
89.28% <0.00%> (+3.57%)
:arrow_up:

Continue to review full report at Codecov.

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


Framework 13 AMD Support on Linux

I've noticed that trying this tool on Linux runs into the issue where /dev/cros_ec doesn't seem to exist on the latest kernel. I wanted to know if there was any blockers or action items that would enable this program's functionality to work on Linux
Failure Code
Self-Test
  SMBIOS Platform:     Framework13Amd
  SMBIOS is_framework: true
  Dump EC memory region
Failed to open /dev/cros_ec. Because: Os { code: 2, kind: NotFound, message: "No such file or directory" }
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', framework_lib/src/chromium_ec/cros_ec.rs:70:34
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

You can use --driver portio to directly talk to the EC without a kernel driver. But it won't work with secureboot. See #16
But kernel driver fixes are already out there and coming soon.

Is it intended that it's trying to talk to the CrOS EC when on an AMD board? I'm not 100% sure of the exact differences, but I'm pretty sure the EC on CrOS and non-CrOS devices serve some different functions.

But kernel driver fixes are already out there and coming soon

Is there a mailing list thread or link one could use to track these fixes, to see what kernel version they might end up in?

@sserenade maybe this one<EMAIL_ADDRESS>

@OctopusET I just tested the patch and the kernel still doesn't provide /dev/cros_ec with it applied (thanks for taking a look though!)

Seems like this is the patchset- I just applied it on 6.6.2 and framework_tool now functions correctly!

You can use --driver portio to directly talk to the EC without a kernel driver. But it won't work with secureboot. See #16 But kernel driver fixes are already out there and coming soon.

This works for me. Thank you!

Hello @prurigro do you if the patchset you have shared has landed upstream yet in 6.7 ? I don't think it has but I haven't checked really thoroughly

@AkechiShiro it's not in mainline 6.8-rc5, so it's probably not in 6.7.

Yeah, I'm still patching it in to each 6.7.x release

Why not making portio the default?

Why not making portio the default?

portio does not work when secure boot is enabled https://github.com/FrameworkComputer/framework-system/pull/16

The patch series has been merged to the chrome platform tree for-next branch.
I guess it will then be in kernel 6.10

portio does not work when secure boot is enabled #16

You can temporarily lift kernel lockdown using https://github.com/blastrock/unlockdown while waiting for 6.10

Seems like this is the patchset- I just applied it on 6.6.2 and framework_tool now functions correctly!

@prurigro , if you don't mind me asking, how are you doing this? I've never done that before, but I'd love to get this tool running w/o disabling Secure Boot or Kernel Lockdown. (till I get access to the 6.10 kernel)

@elreydetoda Hey, so in Archlinux what I do is grab the package for the most recent kernel and add the patches to the build script before running it (ie: patch -Np1 < ../1.patch for each of the four patches). I'll drop them in an archive here so you don't have to dig them out of the mailing list.
Let me know if you need more detail in one direction or another- I didn't want to go too in depth considering how different an experience it would be depending on the distro you're using.
framework-system-amd-support.tar.gz

I just tested the 6.10.2 release without any patches and framework_tool works with secure boot enabled. I believe this issue can be closed :)


The '-a' option should open vim directly in INSERT mode

This can be accomplished using the +startinsert option.

Closed with 50b2b0.


DigitalOcean DNS - function "service_name" not defined

When spinning up DigitalOcean DNS, the following error occurs;
Error (template: template:21: function "service_name" not defined)
Service never starts. Have tried with different DO accounts, and multiple different API tokens on the accounts.

Useful
Info

Versions
Rancher v1.6.0-rc5 Cattle: v0.179.5 UI: v1.6.1

Access
github

Orchestration
Cattle

Route
stack.code

Just tried same with AWS Route53 - same error.
Issue with the template file?

@dominicporteous do you have something defined in DNS Name Template? looks like service_name is a missing  parameter that isnt being passed in.

@dominicporteous when you run a RC, you get a custom catalog branch that doesn't necessary have all the up to date templates that will work. with this specific issue, there are {{ in the rancher-compose.yml files for the catalog templates using external-dns.
I've merged in a PR into our community-catalog that should fix digitalocean-dns for you, but to get a working route53 one, you will need to have your Rancher catalog pointed to the master branch instead of the custom v1.6.0-rc5 branch.
Please try again as this should be fixed.

Thanks a bunch!


[Marketplace] need to ignore space before and after search phrase

EPAM AI DIAL chat version
0.20.0
What happened?
Now the search results are different of to type "anth" or " anth " -> bug
Expected: " anth " should be treated like "anth"
Be sure to test the search results if the space is at the end or in the middle of the search phrase
Confidential information

[X] I confirm that do not share any confidential information

Verified on staging env successfully


Restore RDS instance from a MySQL backup for DBInstance in RDS Group

What problem are you facing?

How could Crossplane help solve your problem?
implement Restore RDS instance from a MySQL backup for DBInstance in RDS Group as followup for https://github.com/crossplane/provider-aws/pull/770

@haarchri now that restoring from a snapshot has been implemented for RDS Instance would it make sense to go ahead and do this for DBInstance and DBCluster?

@jbhennin didn you have time to Open a PR ? ;) I can review

Working on setting up time.  This should be an almost cut and paste.  Likely can get to it later this week or next.


Ensure Stackdriver Logging is set to Enabled on Kubernetes Engine Clusters

Possible check to target the following resource

[ ] google_container_cluster

closing, tracking in spreadsheet


Extract common part of if

Extract common part of if condition.
Description
Extract common part of if condition.
Motivation and Context
Extract common part of if condition. Is is code style fix.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)

👍

Thank you for your contribution! 🎉


wip: Drop two errors back down to typed: false

Motivation

Test plan

See included automated tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_LZKTAbcq53H7vB
→ https://go/builds/bui_LZKTXM6xcWmqi8


General overheads from MLIR are contributing significantly to compile time

Part of https://github.com/iree-org/iree/issues/11994
When considering running an entire compilation pipeline on a large program, some overhead costs from MLIR add enough time over enough instances to show up as significant in traces ("death by a thousand [paper]cuts").
Operator Verification
One example of this is operator verification, which we can optionally disable in iree-compile with the flag --verify=false. (See also: https://mlir.llvm.org/docs/DefiningDialects/Operations/#custom-verifier-code)
I measured one program taking 2m9s with verification and 2m3s without. At the start of compilation (Input and ABI phases), we haven't yet subdivided the program into individual dispatches/executables, so verification naturally runs on the entire input program with every pass. This causes even no-op passes to take on the order of 25ms. Some passes like Convert1X1FilterConv2DToMatmul even spent 80% of their runtime in verifiers (130ms/170ms):

Most of that file looks to be coming from linalg ops with affine maps:

General mlir:: functions
Here are some statistics breakdowns for mlir symbols when sampling:

Idea for a thousand-papercuts problem: use Pass::initialize more for frozen pattern lists and dynamic pipelines. Today most of our passes create the pattern lists and pipelines in runOnOperation() meaning that if they run for example on every function in every executable variant (thousands+) we're doing a lot of that work multiple times in the critical path. Instead we could change to providing any of the information required to construct those resources as options/args and do it once in initialize. If we do that throughout the stack for example in codegen we'd have one initializer per unique ExecutableTargetAttr instead of one per executable/function/etc being translated. To make iree-opt tests and such easier to work with we could make it optional by having a Pass shim that has a getCachedPatterns/getCachedPipeline that ran at initialization time if possible and otherwise on demand.


🎈 8.0.0

🎈 Release
Preparation for 8.0.0 stable release.
Comparison to v7: https://github.com/ignitionrobotics/ign-msgs/compare/ign-msgs7...ign-msgs8
https://github.com/ignition-tooling/release-tools/issues/459
Checklist

[x] Asked team if this is a good time for a release
[x] There are no changes to be ported from the previous major version
[x] No PRs targeted at this major version are close to getting in
[x] Bumped minor for new features, patch for bug fixes
[x] Updated changelog
[x] Updated migration guide (as needed)
[x] Link to PR updating dependency versions in appropriate repository in ignition-release (as needed): https://github.com/ignition-release/ign-msgs8-release/pull/1

Note to maintainers: Remember to use Squash-Merge

there's an issue with the DSL for the abichecker job. Our branch rules currently require ignition_msgs-abichecker-any_to_any-ubuntu_auto-amd64 to pass for any ign-msgsX branch, but I think that job is not enabled until we add the new major version number to ignition.dsl in release-tools.

https://github.com/ignition-tooling/release-tools/pull/523

@osrf-jenkins run tests please


Better example/ docs

So…I'm kinda new to pam and I'm not entirely sure how some of this is supposed to work. When creating an Authenticator it takes a service name. Does this have to be one of these? https://docs.oracle.com/cd/E19683-01/816-4883/pam-34/index.html
I'm currently running into the problem that authenticate() fails and I'm not sure why (and not sure how to debug it)

You can use any service name you like:
fn authenticate(user: &str, password: &str) {
    let mut auth = Authenticator::new("pam-auth-test").unwrap();
    auth.set_credentials(user, password);
    println!("{:?}", auth.authenticate());
}

For debugging, look for messages in the system log, eg using journalctl -f on a system using systemd.

Thanks for the feedback. Looking into the system log, I'm a wee bit confused what's going on
Jul 02 20:48:20 rayya lockchain-server[10702]: pam_tally(login:auth): Error opening /var/log/faillog for update
Jul 02 20:48:20 rayya lockchain-server[10702]: pam_tally(login:auth): Error opening /var/log/faillog for read
Jul 02 20:48:20 rayya unix_chkpwd[10704]: check pass; user unknown
Jul 02 20:48:20 rayya unix_chkpwd[10705]: check pass; user unknown
Jul 02 20:48:20 rayya unix_chkpwd[10705]: password check failed for user (testkookie)
Jul 02 20:48:20 rayya lockchain-server[10702]: pam_unix(login:auth): authentication failure; logname= uid=1000 euid=1000 tty= ruser= rhost=  user=testkookie
Jul 02 20:48:22 rayya lockchain-server[10702]: pam_tally(login:setcred): Error opening /var/log/faillog for update
Jul 02 20:48:22 rayya lockchain-server[10702]: pam_tally(login:setcred): Error opening /var/log/faillog for update

But doing su testkookie with the same password works :thinking:

Just a guess, but are you trying to authenticate a user different from the one running the program? This does not work with local users by default, as it needs access to /etc/shadow, the file which contains the passwords of users. Only root can normally read this file.
When you are using remote authentication like ldap in an active directory or nis, it generally works without root access. Also users can authenticate themselves.

Hmm, oh okay. I thought I could use pam to open a session as a different user and then do stuff with it.
Ultimately some user will be running the software but I want to authenticate users by their system accounts and only really make sure they are member of the right group. I guess pam isn't what I want then? 😅

@spacekookie I got exactly the same issue do you have a solution for this?


CircleCI: Cache public Docker images

Instead of always hitting the public Docker registry, run a mirror locally
that uses S3 in us-east-1 as its backing store.
This should hopefully reduce the number of build failures we get due to
image pulls failing against the public Docker registry. us-east-1 is the
same region that CircleCI workers are in.

@rculbertson

:+1:


I can't run make posix_sitl_default gazebo

hi,everybody.i want to do the gazebo simulation inferfacing to ROS,what i do is follow the page https://dev.px4.io/simulation-ros-interface.html ,but when i do the command
no_sim make posix_sitl_default gazebo
the console output

the OS is ubuntu x64 14.04
ROS rev is indigo
gazebo rev is 7
so,please help me to slove the problem,thank you

You're missing protobuf.
Fix :
sudo apt-get install libprotobuf-dev.


Adding workflow status

Adding workflow status

Codecov Report

Merging #22 (abc288c) into main (3f118ec) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #22   +/-   ##
=======================================
  Coverage   85.82%   85.82%
=======================================
  Files           4        4
  Lines         649      649
  Branches      159      159
=======================================
  Hits          557      557
  Misses         75       75
  Partials       17       17

Continue to review full report at Codecov.

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