Icons

I didn't want to write all the icons by hand and created some functions in app to generate all icons as code.
Now I've moved it to it's own crate: https://github.com/alexkazik/yew-bootstrap-icons
If you're interested I'll create a PR.

I think this is an worthy developer UX improvement. Our current support for icons is poor; I think just this:
https://github.com/isosphere/yew-bootstrap/blob/a8b48fb046fc7bb66afdcf4d819ab02074edcdfd/packages/yew-bootstrap/src/util/include.rs#L42
... and support for a NavBar BrandIcon:
https://github.com/isosphere/yew-bootstrap/blob/a8b48fb046fc7bb66afdcf4d819ab02074edcdfd/packages/yew-bootstrap/src/component/navbar.rs#L195-L197
Which is used in a clunky way that can't be compile-time checked:
https://github.com/isosphere/yew-bootstrap/blob/a8b48fb046fc7bb66afdcf4d819ab02074edcdfd/examples/basics/src/main.rs#L21
Your approach is better.
I see some nightly hints in your code (must_use, at least); I'd like to target the stable toolchain.
Happy to accept a PR!

Sounds good, I'll create a PR (may take a few days).
And it can be compiled with stable (all my public crates compile on stable,must_use is stable since a long time).
On the branch https://github.com/alexkazik/yew-bootstrap-icons/tree/icons all icons are shown in the rustdoc (the BI documentation adds bootstrap-icons via cdn). It's great that it's possible but should it be used? (Just check it out and run doc to see it.)
Unless you have another idea I'll place it in a icons module, and document that module (what's in my current root doc / readme).

Sounds good, I'll create a PR (may take a few days). And it can be compiled with stable (all my public crates compile on stable,must_use is stable since a long time).

Ah, my mistake maybe; must_use is still marked as nightly only here: https://doc.rust-lang.org/std/hint/fn.must_use.html and the corresponding issue (https://github.com/rust-lang/rust/issues/94745) shows stabilization as incomplete.

On the branch https://github.com/alexkazik/yew-bootstrap-icons/tree/icons all icons are shown in the rustdoc (the BI documentation adds bootstrap-icons via cdn). It's great that it's possible but should it be used? (Just check it out and run doc to see it.)

I took a look; I don't have strong feelings about this, but I don't think I personally would use it when coding. I think I'd use my IDE to click-through to the generated file.

Unless you have another idea I'll place it in a icons module, and document that module (what's in my current root doc / readme).

I like that approach; I'm working through that documentation now.
I prefer the way patternfly_yew handles the generation. The development team generates the icons, and then they include that generated code in the project rather than requiring the user modify their build chain accordingly: https://github.com/patternfly-yew/patternfly-yew/tree/main/generator/icons
This makes the user experience much smoother; if they want to self-generate, they can, but they don't have to. No futzing about in Cargo.toml and making new binaries just to use some pretty icons.

I have a very first version ready at https://github.com/alexkazik/yew-bootstrap/commits/icons.
Changing BrandType is included and a breaking change!
I think the crate needs a cargo fmt --all (once in packages and examples), if you do that I'll base my work on that otherwise I'll base on the current (had to disable features on my IDE, like removing trailing whitespace or running fmt in order to not change everything).
All icons are available all the time, the build script takes care of that.
I've only described how you can build a binary which copies the fonts to your dist directory.
I can extend basic (or another example) to include it, or better create two examples, one for each case.
But since trunk can only copy files already there or created by running a binary,
I don't know how to make this even easier, other than using the cdn.

I've never been a fan of using code formatters, as I often disagree with their choices, but I'll give cargo fmt a look. I agree that a consistent formatting of code throughout the project is ideal.

The big advantage of cargo fmt is that it is consistent.
If you don't like the default settings you can customize them a lot (https://rust-lang.github.io/rustfmt watch out for Stable: No - some are nightly only).
While I started with customize my settings I removed most of them (except that I'd like to use group_imports:One, because that's what I do anyway but it's nightly only; and the other exception is my first project which started with indent of 2 and I don't want to reformat all).
And I don't want to force you into it, it's just a suggestion because it can help a lot.


[WFCORE-4577] Upgrade WildFly Elytron to 1.10.0.CR4

https://issues.jboss.org/browse/WFCORE-4577
    Release Notes - WildFly Elytron - Version 1.10.0.CR4

        Bug

[ELY-1843] -         The Digest NonceManager creates a ScheduledThreadPoolExecutor which is never shut down

[ELY-1847] -         WildFlyElytronProvider Loading wrong class for Bearer

[ELY-1849] -         WildFlyElytronSaslDigestProvider does not register necessary password factories

        Task

[ELY-1841] -         Update the pre-recorded messages in AcmeClientSpiTest using the latest Boulder instance

[ELY-1852] -         Address Message ID Allocation

        Release

[ELY-1857] -         Release WildFly Elytron 1.10.0.CR4

retest this please

retest this please

@fjuma thanks!


Stream sometimes fails in bazel

Receiving the following errors:
INTERNAL: Received unexpected EOS on empty DATA frame from server

From my reading online, it appears this is a grpc stream timeout. I believe it's the action stream timing out. If it is the action timing out, we should make a timer that periodically sends a message reminding the clients that the state did not change on the action.

Will be fixed in: #365


USER STORY - 06( continuining) - SIGN UP AS A GRADUATE( explaining in details for the implementation of  SENDING VERIFICATION EMAIL to the GRADS)

Naming Rules
Name your PR like this: ISSUENUMBER-#14-IMPLEMENTATION OF SENDING VERIFICATION EMAIL  to the GRADS-ADIBA BELAYETE
Description
I have added a form to provide the email id & certificate number by the grads so their certificate id can be verified & a verification email will be sent to the provided email address to confirm their identity.
Related to
Make sure you include the issue number with a hash sign # so Github can link this PR to the right issue, like this:
ISSUE NUMBER # 14
Fixes #1
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have carefully reviewed my own code
[ ] I have commented on my code
[ ] I have updated any documentation

Hi, I have done the formate files with prettier. thanks.

Can you merge main back into this branch


Add folder animations to the theme

Would there be a way to implement the folder animations like in the Sublime Text Material UI theme?
Apologies for the number of issue submissions already.

That would involve changing the icons and it's something I'm not going to do until I find a way for not having blurry icons. Check #193 for some more info.

Understood. Thanks for the reply.


Rename Skip to Break

Skip is understood as skipping the current step. Whereas Break is understood as stopping the execution of the whole story. IMHO.

How about Up, Continue, Next? It should sound good in the #88 as well. Rollback(to=Next()) make more sense to me. Thoughts?

My 2 cents:
Skip pretty much confused me, indeed making me think it was skipping the current step.
Next seems ok, but in the context of rollback may seem linguistically counter-intuitive: roll-BACK-to-NEXT
Up hints to continue flow in the parent story.
I like Continue as it fits nicely in a stories context.

@casio Thanks for the feedback!
What do you think of https://github.com/dry-python/stories/issues/352#issuecomment-613730266 ?
Will it make sense to see something like return Continue({'token': 'x'})?


MODLD-564: readme + env variables update

Hi @PBobylev, Can we remove SYSTEM_USER_PASSWORD environment variable from Module descriptor? It is not needed for our application.

Hi @PBobylev, Can we remove SYSTEM_USER_PASSWORD environment variable from Module descriptor? It is not needed for our application.

Done and other env variables there were updated too.


kvo not work

here my example，when click button  change usename observe not work, is anything i miss?
class ViewController: UIViewController {

    override func viewDidLoad() {
        super.viewDidLoad()

        Defaults.observe(\.username, options: [.initial, .old, .new]) { update in
            print(update)
        }

    }

    @IBAction func click(_ sender: Any) {
        let num = Int.random(in: 0..<10)

        let username = Defaults[\.username]
        Defaults[\.username] = " abc \(num)"
    }
}

extension DefaultsKeys {
    var username: DefaultsKey<String> { .init("username", defaultValue: "default") }

}

The DefaultsDisposable instance is released, so you can't keep observing. It should be a storage property of the ViewController to prevent it from being released.


feat: Create sync.yml

This should sync up oryworks repo to the other ORY repos.
Is this all that is needed?

Hm guess it would be good to make sync it's own action then?

But what is it supposed to sync?

Repos from an template
Or what is the purpose of it in this repo?

*a template into a repo

Repos from an template
Or what is the purpose of it in this repo?

"add code of conduct and cla to oryworks" automated.
with https://github.com/ory/oryworks/pull/2/commits/f035e1a93d71f8a1a2fa536a0cc56d0bb52ee481 it should sync oryworks now to the template in meta, correct?
or do I need to change the action/workflow somewhere else?

Nope, the sync script is missing, nothing will happen. Why would you sync stuff from here to meta and not just change it in meta?

Yea that was my question ^^.
So I just have to add this here to the sync in meta, correct?

If you want to sync meta into this repo, then yes.


kkkkk hiena

LASIUDHF ÇQJHWNÇGUHA DÇFIVJH QOÇ OÇNÇOm
TOKEN

@HCley Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


There is a need to change flex-row -> flex-row-reverse

Hi,
To make right arrangement of flex items when there is a flex-row parent, it needs to convert  flex-row-reverse
as a good sample you can check this component :
https://tailwindcomponents.com/component/tailwindcss-flow-chart
This manual modification to final css created by tailwind did the job for me:
[dir="ltr"] .flex-row {
flex-direction: row;
}
[dir="ltr"] .flex-row-reverse {
flex-direction: row-reverse;
}
[dir="rtl"] .flex-row {
flex-direction: row-reverse;
}
[dir="rtl"] .flex-row-reverse {
flex-direction: row;
}
of course I made these change to all other flex-row classes for sm, md, lg, xl and 2xl
Thanks,

@Ali-Javanmardi if I recall correctly, flex-row will lay elements left to right on LTR, and right to left in RTL, am I right? If you always want your children displayed in left to right, you need to add dir="ltr" to the container, and dir="auto" to the children components.

@ranisalt You are right, I checked many other things and they are working right with current setting.
But still there is a question about above sample I mentioned in first post.
If you open that component in browser and then change page direction from LTR to RTL you will find unordered connection lines.
As I wanted to find if it is usable in RTL pages I did above changes and it worked.
But I am agree with you that it may make problem with other components.
So although it may be a rare situation to face problem like mentioned above component I think there is a value to solve it in a better way.


engine: improve ergonomics of writing a phase

[ ] have a just rule to interactively add a new phase (based on a template)
[ ] make the location of errors more precise in inline PPX

cc @maximebuyse

still relevant


Upload to Paratext

Allow the user to upload the drafted text to Paratext via its APIs.

https://registry-dev.paratext.org/developer/index.html#!/Projects/get_projects

Make UI section for 'Sync' with tab for 'Paratext'.
-- It should contain 'Username' and 'Password/Registration Code' fields.
-- It should have a 'Connect' button, which connects to the internet, if connection is available.
If the app is able to connect successfully, it should list the 'Available Projects' in the under the username/password form.
-- The user should be able to select which books/projects to import and click 'Import' button. At this point a dialogue box shows 'Warning' as title and text: 'This will overwrite any existing text in the selected books' and options of 'Continue' and 'Cancel'.
-- If the user clicks 'Continue', the app should import those books and load them in the database. It should a loading icon while doing this.

If the user tries to 'Upload' data without 'Import'ing first, then it's not possible for us to know what exact form the USX file is in, since internally we do not capture the extended markup. To overcome that we should first have a 'Enable' button on each project listing, on clicking which we automatically import the files (in their respective project folder in the user project directory) and then allow the user to continue using the sync options for that project. Also the listing of books shouldn't show any books other than 66 ones that the app can handle.

Moved over for development to the Live version of Autographa: https://github.com/friendsofagape/autographa-live/issues/2


Document PR #417 for release 0.5

#417 adds an explicit confirmation if --no-ssl is to be used.
@minrk Do you want any documentation that this will be happening in the next release [upcoming change in the security section] or leave as is until we get closer to 0.5 release?

Yes, it should definitely be covered, in the changelog and probably the security section.


Command Prompt

Description
So, I was on the files app. And, when I typed cmd, it didnt work and said Ran into a problem. Please help.
Steps To Reproduce

Go to file
Type "cmd"

Expected behavior
Fix
Files Version
v1.5
Windows Version
Windows 10 Pro
Relevant Assets
No response

Please upload the app's debug logs and confirm the app's version.

v1.5 is no longer supported, please let us know if this is still an issue in Files v2.0.34.


Optimistic mainnet deployment

Closes #29

@jpalvarezl Can you verify the contracts on sourcify? I see they are now supporting Optimistic

I am not sure how to achieve that. If you don't mind I would ping you privately to ask you how to do it :D


Implement script for style dataset classification

The script has to test for each type of classification with all possible parameters and set as output the best result.

Classificators:

[x] LR
[x] DTC
[x] KNC
[x] GNB
[x] SVC

Test from all features to one

Done doing a generic class model https://github.com/FNClassificator/FNC/blob/master/src/fake_news_detector/core/classification/ml_models.py


question: how can i download elevenlabs response locally?

My question is how can i download elevenlabs response locally?
Thank you!!

https://github.com/elevenlabs/elevenlabs-python/blob/main/API.md#save
from elevenlabs import generate, save

audio = generate(
    text="Hi there",
    voice="Arnold",
    model='eleven_multilingual_v1'
)

save(audio, 'myvoice.mp3')

Hey Thank you soo much for your prompt response!!
I will try this out right now.

I am building a chatbot in gradio using whisper and elevenlabs, so that user can talk to and listen to the chatbot:

When running it locally I was using elevenlabs Play( ) function to play the audio response and everything was working fine.
And now when I have deployed it on heroku everything is working same as it is except that the elevenlabs audio is not played.
I noticed that previously the audio was played on my system and not on the website (when running locally), that's why when i even close the website the audio was still playing and now when it is deployed on internet it is no longer working.
Searching for solution of this I created this issue so that I can save the audio in some database and later play it.....
But this may not work either because my code is on huggingface and the saved file wouldn't be saved there in the repo...
I highly require your help @flavioschneider in this regard, this issue is soo concerning to me.
I truly appreciate your efforts!!
Thank you for your help.

Hey @flavioschneider ,
Sorry to disturb you but could you please guide me to the right way or provide any resources which can potentially help me.
Thanks


Fix flaky unit test

Investigate flaky unit test.
Failure [20.009 seconds]
[AfterSuite] AfterSuite
/go/src/github.com/medik8s/self-node-remediation/pkg/peerhealth/suite_test.go:95
  Unexpected error:
      <errors.aggregate | len:1, cap:1>: [
          <*errors.errorString | 0xc00018d180>{
              s: "timeout waiting for process kube-apiserver to stop",
          },
      ]
      timeout waiting for process kube-apiserver to stop
  occurred
  /go/src/github.com/medik8s/self-node-remediation/pkg/peerhealth/suite_test.go:99

Happens sometimes in OpenshiftCI, e.g. https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/medik8s_self-node-remediation/49/pull-ci-medik8s-self-node-remediation-main-4.13-test/1612741423029817344
Happens more often in github workflow of NHC, e.g. https://github.com/medik8s/node-healthcheck-operator/actions/runs/3885493260/jobs/6629439227
Couldn't reproduce locally yet...
Solution
Cleaning up some test code and fixing small issues didn't resolve the flaky tests. In the end I found how to increase the timeout (default 20s) we are running into and I needed to disabe the graceful shutdown. See last commit.

found something which might cause this, see 2nd commit

unrelated other failure, ignoring for this PR
/test 4.12-test

/test 4.12-openshift-e2e
/override ci/prow/4.13-openshift-e2e

/retest

/test 4.12-openshift-e2e

/lgtm


Input error when delete with backspace on touch device

Bug, feature request, or proposal:
Input error when delete with backspace on touch device. The cursor is not placed at the beginning of the placeholder but  is in a forward position.
in touch device.
What is the expected behavior?
have the cursor at the beginning of the placeholder
What is the current behavior?
The cursor is in a forward position
What are the steps to reproduce?
go to email example in this page:
https://material.angular.io/components/input/overview
try to write a word like:  bohumn
delete with backspace
the cursor is not at the beginning but in a forward position
What is the use-case or motivation for changing an existing behavior?
is not correct
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Is there anything else we should know?

Seems to work fine in my Android. Any particular config?

I have the problem on iPad

I was able to reproduce on ios as well

It also occurs on the "input hint" and "prefix suffix" examples, which leads me to believe it is an issue with inputs within forms (since the "basic" example is not in a form)

The cursor problem does not affect Android, but the input underline overflowing the screen does.

The input sizing issue looks like just an issue with the docs site, not the input itself. I'm not able to reproduce the cursor issue on an iPad, but the iPad is big enough that I don't see the overflow. I wonder if the cursor issue is also related to the sizing issue.

@mmalerba https://github.com/angular/material2/pull/5839 should fix the overflow when deployed. Also this repro shows the cursor issue for me on iOS 10.3.3

@willshowell, the link to the repro is broken.

Oops, something weird about full screen plunks.
Also I think I was wrong about it being related to forms. The below gifs show that it is dependent on having an auto|never floating placeholder. For me, to get reliable results, I have to hold the delete key while removing the input, and I get the same results in Safari and Chrome on iOS.
gif 1 (plunker)

gif 2 (plunker)

Ok very interesting. I was still not able to repro this with an iPad running iOS 9, but I was with an iPhone running iOS 10, so it must be related to either the form factor or the specific version. Will do some experiments and see if I can narrow it down any further.

Ok I've narrowed this down a little further. It seems like mobile safari has some issue where the cursor gets stuck when you change the page content in response to input events. Minimal repro using native input: http://plnkr.co/edit/kzyVA8wHqC6g5uVcnsJv?p=preview
I'll see if anyone has written anything about how to work around this

Couldn't find any existing documentation about this issue, but after some experimentation I think I've found a way to fix it. Added the fix code to the plunker above and will prepare a PR to implement it for md-input-container.


Question: Difference between variable setting?

A question for @avu0ng Monday or Tuesday ... What's the difference between these variables?
https://github.com/UCLALibrary/ansible_uclalib_role_iiifimages/blob/master/templates/config.json.j2

Discussed with Anthony


[DDW-713] Remove "Decentralization countdown" tada icon and info tab

This PR removes the "Decentralization countdown" tada icon and info tab.

Testing Checklist

Slack QA thread
[ ] Test

Test Scenarios
Scenario 1 - Check "Info" tab is not visible
Given that Daedalus is synced
When I access to Staking Center
Then I can see that the "Info" tab is no longer visible

Scenario 2 - Check "tada" icon is not visible
Given that Daedalus is synced
When I check the top bar
Then I can see the "tada" icon is no longer displayed

Review Checklist
Basics

[ ] PR has been assigned and has appropriate labels (feature/bug/chore, release-x.x.x)
[ ] PR is updated to the most recent version of the target branch (and there are no conflicts)
[ ] PR has a good description that summarizes all changes
[ ] PR has default-sized Daedalus window screenshots or animated GIFs of important UI changes:

[ ] In English
[ ] In Japanese

[ ] CHANGELOG entry has been added to the top of the appropriate section (Features, Fixes, Chores) and is linked to the correct PR on GitHub
[ ] Automated tests: All acceptance and unit tests are passing (yarn test)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (yarn dev)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (yarn package / CI builds)
[ ] There are no flow errors or warnings (yarn flow:test)
[ ] There are no lint errors or warnings (yarn lint)
[ ] There are no prettier errors or warnings (yarn prettier:check)
[ ] There are no missing translations (running yarn manage:translations produces no changes)
[ ] Text changes are proofread and approved (Jane Wild / Amy Reeve)
[ ] Japanese text changes are proofread and approved (Junko Oda)
[ ] UI changes look good in all themes (Alexander Rukin)
[ ] Storybook works and no stories are broken (yarn storybook)
[ ] In case of dependency changes yarn.lock file is updated

Code Quality

[ ] Important parts of the code are properly commented and documented
[ ] Code is properly typed with flow
[ ] React components are split-up enough to avoid unnecessary re-renderings
[ ] Any code that only works in main process is neatly separated from components

Testing

[ ] New feature/change is covered by acceptance tests
[ ] New feature/change is manually tested and approved by QA team
[ ] All existing acceptance tests are still up-to-date
[ ] New feature/change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review

[ ] Merge the PR
[ ] Delete the source branch
[ ] Move the ticket to done column on the YouTrack board
[ ] Update Slack QA thread by marking it with a green checkmark

Overall clean and correct 👌👌👌
just two very small stuff that are very common for naming conventions

Thanks @danielmain. These variables are temporary and will be replaced in DDW-712, but it's important to have this one ready for the next release. Nevertheless, your suggestions were applied :)


Dynamic Extraction

Check List

[ ] Does your submission pass tests by running make test?
[ ] If you have specified a [no ci] tag, does your submission also pass tests by running make test-docker?
[ ] Have you linted your code locally with make lint before submission?

Added

Dynamic extraction features

Changed

Fixed

Removed

The Ubuntu tests failed because apt update was missing before apt install in the docker scripts. I'm not sure why it was not a problem before, but I've fixed it now.
What's remaining seems to be the test tests/conversion/zx2qc/extractStep.dof not passing. This was because the compiler implicitly cast the nullptr to false in the constructor of Extractor, so the extractor is not extracting to the focused QCir. I've swapped the param for QCir and the random flag to avoid this bad casting from happening.


Add list_meetings for activity resource

Another activity that we have in Close is meeting.
This PR adds listing for meetings.

@dmitrytrager This was released on Rubygems in v3.16.0
Thanks for the help!


Prevent accidentally creating lifecycle rule without filtering

Description
Added extra guard option to create lifecycle rule for expiration whole bucket
Motivation and Context
In our case, we created a bucket without any filter prefix, which causes expiration of about 1,5TB of data in bucket.
We had code like snippet bellow
module "s3_bucket" {
  source = "terraform-aws-modules/s3-bucket/aws"

  lifecycle_rule = [
    {
      enabled = true
       prefix = "prefix-to-cleanup/"

      expiration = {
        days = 1
      }
    }
  ]
}

There is a missing filter section wrapping prefix. Sadly terraform does not throw any error that config is inaccurate. That code created lifecycle rules for the whole bucket.
We wanted to be sure not to make the same mistake again and create some kind of guard before this situation. Without passing extra option, no expiration section will be created - that should be less pain than losing data ;)
Passing one extra option should not be a big deal - AWS Console have a similar checkbox to prevent creating rule for the whole bucket.
Breaking Changes
Conditionaly/Partialy - if someone wanted to create lifecycle to cleanup whole bucket it is needed to pass extra option.
How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

@antonbabenko could you look at this PR? What do you think?

@rysi3k not all CI checks are passed. Make sure to add the convention to the start of the PR title. You can look here or on previous PRs to see what is a valid convention.

I am not sure that this kind of enhancement will lead to safer setups. Terraform AWS modules don't provide extra guardrails because it will be very hard to maintain them and reason about them.


Bookmark Toolbar

Is there a way to show the bookmark toolbar on another line? I absolutely love this theme, but not having quick access to bookmarks is a bit of a killer.

@MudkipBanana I tried adding Bookmarks Toolbar but it was breaking Navigation Toolbar :( By the time I manage to get this done, you can either use Ctrl + B or a New Tab page to access your bookmarks.

There were few bugs due to bookmarks toolbar. Fixed now!!!!


Point to CoC which contains the contributor covenant

What was the end-user problem that led to this PR?
The generated README contains two different links to contributor covenant. Since the generated Code of Conduct contains the text of the Contributor Covenant, we can just point the generated README to the generated CoC, and explicitly refer to it as the code of conduct.
What is your fix for the problem, implemented in this PR?
The generated readme now points to the gems CoC (which contains the Contributor Covenant)
Closes #6905.

Commits squashed! Thanks so much @ajwann!
@bundlerbot r+

@bundlerbot r+


fix lazygit launching on Windows

I use this plugin on Windows and it recently broke. I noticed it was attempting to launch lazygit using bash. Looking at the code, it seems that a change went in to enforce usage of 'bash' as the Shell. This broke functionality on Windows. This PR fixes this bug with a simple check for Win32 and not changing the shell if that's the case.

Thanks for making the PR!


Menu - fixed menu has low z-index

Hi,
there is problem with fixed menu - his z-index is lower than corner label in form input.

Same problem if menu is initialized with jq :
$('.main.menu').visibility({ type: 'fixed' });

Can confirm same issue here @osmanraifgunes
Using the fixed class directly helps in this case.


[DevOps] Create new aws account for S3 as previous one is locked for Russian account

[DevOps] Create new aws account for S3 as previous one is locked for Russian account
reconfigure current s3 using
create same tree, fill folders with pictures

done


Host aliases in ~/.git/config do not work

Attempting to pull a repository URL (e.g. git@gitlab-alias:group/project.git) that uses Host alias from ~/.ssh/config fails with the following error:
ssh: Could not resolve hostname gitlab-alias: no address associated with name
fatal: Could not read from repository.

Contents of ~/.ssh/config:
Host gitlab-alias
  HostName gitlab.com
  User git
  IdentityFile /c/Users/me/.ssh/gitlab_id_rsa

I'm trying to use a separate pair of rsa keys which I specified in .ssh/config.
git-plus does always take the default keys.
Is there a way to configure which keys to use?

This package doesn't touch or choose anything related to your ssh configurations. All the commands are executed in a spawned child processes so those configurations should translate.
I'm able to use host aliases in  and everything still works. @gulench When you try to pull from the command line does it work?
@dampfhamm3r No there isn't a way because the package doesn't do anything with those settings.

@akonwi I have the same situation as @gulench, my config file is
Host github.com
	IdentityFile /c/Users/aroc/.ssh/github

Making a pull or a push doesn't work, using Git Bash under windows works. (My ssh key has passphrase).


Error loading AWS configuration for region

The importer fails when trying to import swagger.json.
I get the error:
$ ./aws-api-import.sh --create ../swagger.json
 ERROR - Could not load region from profile and no region parameter specified

after knowing for certain that my aws cli is configured:
$ aws configure
AWS Access Key ID [********************]:
AWS Secret Access Key [**********************]:
Default region name [us-east-1]:
Default output format [json]:

Note: I am able to use other functions successfully with the AWS CLI.

configure aws cli first
ᐧ
On Tue, Dec 22, 2015 at 11:18 AM, Ted Armstrong<EMAIL_ADDRESS>wrote:

The importer fails when trying to import swagger.json.
I get the error:
$ ./aws-api-import.sh --create ../swagger.json
ERROR - Could not load AWS configuration. Please run 'aws configure'
after knowing for certain that my aws cli is configured:
$ aws configure
AWS Access Key ID [********************]:
AWS Secret Access Key [**********************]:
Default region name [us-east-1]:
Default output format [json]:
—
Reply to this email directly or view it on GitHub
https://github.com/awslabs/aws-apigateway-importer/issues/130.

--
Nicolas Menciere
Co-Founder and CTO
Sport Archive Inc.

@koxon
My AWS CLI was configured first.

Hi Ted, I'm sorry you're having issues with the tool. While we investigate this, can you confirm whether passing the region in explicitly in the command works around the issue?
i.e.:
$ ./aws-api-import.sh --create ../swagger.json --region us-east-1

@lorenzoaws
Passing the region in explicitly does workaround the issue

OK great, what about passing in the profile instead? I'm guessing this will fail because this is what we do by default.
$ ./aws-api-import.sh --create ../swagger.json --profile default
If this does fail, can you try creating and passing in a new profile?

@lorenzoaws
Sorry for my delayed response, I've been on break.
Please see my attempted queries (I don't think the crashing readlink is related) :
$ ./aws-api-import.sh --update <API_ID> --deploy stage ../apigateway/swagger.json
      0 [main] readlink 38336 open_stackdumpfile: Dumping stack trace to readlink.exe.stackdump
2015-12-28 11:45:38,468 ERROR - Could not load region from profile and no region parameter specified. Please run 'aws configure' or specify region with '--region' parameter.

username@COMPUTERNAME /d/git/aws-apigateway-importer (master)
$ ./aws-api-import.sh --update <API_ID> --deploy stage ../apigateway/swagger.json  --profile default
      0 [main] readlink 47324 open_stackdumpfile: Dumping stack trace to readlink.exe.stackdump
2015-12-28 11:46:07,878 ERROR - Could not load region from profile and no region parameter specified. Please run 'aws configure' or specify region with '--region' parameter.

username@COMPUTERNAME /d/git/aws-apigateway-importer (master)
$ aws configure --profile test_user
AWS Access Key ID [None]: <Same_Access_Key_ID_as_Default>
AWS Secret Access Key [None]: <Same_Secret_as_Default>
Default region name [None]: us-east-1
Default output format [None]: json

username@COMPUTERNAME /d/git/aws-apigateway-importer (master)
$ ./aws-api-import.sh --update <API_ID> --deploy stage ../apigateway/swagger.json  --profile test_user
      0 [main] readlink 54496 open_stackdumpfile: Dumping stack trace to readlink.exe.stackdump
2015-12-28 11:48:00,607 ERROR - Could not load region from profile and no region parameter specified. Please run 'aws configure' or specify region with '--region' parameter.

Environment variable:
AWS_DEFAULT_REGION=us-east-1
Does the trick for me.
ᐧ
On Mon, Dec 28, 2015 at 2:49 PM, Ted Armstrong<EMAIL_ADDRESS>wrote:

@lorenzoaws https://github.com/lorenzoaws
Sorry for my delayed response, I've been on break.
Please see my attempted queries (I don't think the crashing readlink is
related) :
$ ./aws-api-import.sh --update <API_ID> --deploy stage ../apigateway/swagger.json
0 [main] readlink 38336 open_stackdumpfile: Dumping stack trace to readlink.exe.stackdump
2015-12-28 11:45:38,468 ERROR - Could not load region from profile and no region parameter specified. Please run 'aws configure' or specify region with '--region' parameter.
username@COMPUTERNAME /d/git/aws-apigateway-importer (master)
$ ./aws-api-import.sh --update <API_ID> --deploy stage ../apigateway/swagger.json  --profile default
0 [main] readlink 47324 open_stackdumpfile: Dumping stack trace to readlink.exe.stackdump
2015-12-28 11:46:07,878 ERROR - Could not load region from profile and no region parameter specified. Please run 'aws configure' or specify region with '--region' parameter.
username@COMPUTERNAME /d/git/aws-apigateway-importer (master)
$ aws configure --profile test_user
AWS Access Key ID [None]: <Same_Access_Key_ID_as_Default>
AWS Secret Access Key [None]: <Same_Secret_as_Default>
Default region name [None]: us-east-1
Default output format [None]: json
username@COMPUTERNAME /d/git/aws-apigateway-importer (master)
$ ./aws-api-import.sh --update <API_ID> --deploy stage ../apigateway/swagger.json  --profile test_user
0 [main] readlink 54496 open_stackdumpfile: Dumping stack trace to readlink.exe.stackdump
2015-12-28 11:48:00,607 ERROR - Could not load region from profile and no region parameter specified. Please run 'aws configure' or specify region with '--region' parameter.
—
Reply to this email directly or view it on GitHub
https://github.com/awslabs/aws-apigateway-importer/issues/130#issuecomment-167639458
.

--
Nicolas Menciere
Co-Founder and CTO
Sport Archive Inc.

@koxon. Interesting, what environment are you running in?
@Shockolate It looks like you're on a Windows machine running Cygwin? Does the environment variable trick work for you?

@lorenzoaws
Adding the environment variable does not work for me. I'm running these commands (and the AWS CLI) with the Git Bash window (with ruby and ruby dev kit)

I'm on Linux.
See:
http://docs.aws.amazon.com/cli/latest/userguide/cli-chap-getting-started.html#cli-environment
You can also use the configuration file:
http://docs.aws.amazon.com/cli/latest/userguide/cli-chap-getting-started.html#cli-config-files
ᐧ
On Mon, Dec 28, 2015 at 3:51 PM, Ted Armstrong<EMAIL_ADDRESS>wrote:

@lorenzoaws https://github.com/lorenzoaws
Adding the environment variable does not work for me. I'm running these
commands (and the AWS CLI) with the Git Bash window (with ruby and ruby dev
kit)
—
Reply to this email directly or view it on GitHub
https://github.com/awslabs/aws-apigateway-importer/issues/130#issuecomment-167653192
.

--
Nicolas Menciere
Co-Founder and CTO
Sport Archive Inc.

@Shockolate : It might be related to redline. Can you try pulling this change to see if it works for you?
https://github.com/awslabs/aws-apigateway-importer/pull/133/files

@lorenzoaws
I pulled the change, but apparent improvement.
user@COMPUTER /d/git/aws-apigateway-importer (PullRequest133)
$ ./aws-api-import.sh --update <API_ID> --deploy stage ../apigateway/swagger.json
2015-12-29 10:47:00,038 ERROR - Could not load region from profile and no region parameter specified. Please run 'aws configure' or specify region with '--region' parameter.

@Shockolate
We apologize that you cannot use the region setting from your profile. We found the issue with loading region from the different environments. We will prioritize this issue in our backlog. As workaround, please use the '--region' parameter.

@Shockolate
Please use "--profile test_user" or "--region us-east-1". Reopen if still experiencing issues

@rpgreen
specifying the region with --region does work and how I will be working around the issue.
The --profile test_user does not work.


refactor: refactor ddl manager

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
refactor ddl manager
Checklist

[ ]  I have written the necessary rustdoc comments.
[ ]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

@fengjiachun PTAL


Why is .register returning cached certificates?

I have recently discovered that .register also returns an existing certificate if not expired. This seems to be inconsistent with the rest of the API, for instance the .check method, and the example in the README, where it seems that register will force a registration (note the comment   // Register Certificate manually). Furthermore, I guess that the most common pattern is to renew certificates several days before it really gets expired, what would be the correct API to call in order to force a certificate renewal?
// Check in-memory cache of certificates for the named domain
le.check({ domains: [ 'example.com' ] }).then(function (results) {
  if (results) {
    // we already have certificates
    return;
  }

  // Register Certificate manually
  le.register({

    domains: ['example.com']                                // CHANGE TO YOUR DOMAIN (list for SANS)
  , email<EMAIL_ADDRESS>                               // CHANGE TO YOUR EMAIL
  , agreeTos: ''                                            // set to tosUrl string (or true) to pre-approve (and skip agreeToTerms)
  , rsaKeySize: 2048                                        // 2048 or higher
  , challengeType: 'http-01'                                // http-01, tls-sni-01, or dns-01

  }).then(function (results) {

    console.log('success');

  }, function (err) {

    // Note: you must either use le.middleware() with express,
    // manually use le.challenges['http-01'].get(opts, domain, key, val, done)
    // or have a webserver running and responding
    // to /.well-known/acme-challenge at `webrootPath`
    console.error('[Error]: node-letsencrypt/examples/standalone');
    console.error(err.stack);

  });

});

It will renew a few days early by default.
But you can pass renewBy and/or renewWithin to adjust this. These are documented in the https://github.com/Daplie/letsencrypt-express README, and unfortunately not here.
You can also pass duplicate:true to force duplicate registration/early renewal, which doesn't seem to be documented anywhere, except in a roundabout way in https://github.com/Daplie/letsencrypt-cli/ with some discussion of it at https://github.com/Daplie/letsencrypt-cli/issues/22
Pull requests to improve the documentation are welcome.

@insightfuls I am still a bit confused about how the api is supposed to work.
According to the example above (from the README), first a call to check is done, if we get a valid certificate then returns, otherwise it calls to .register. Now, in my code, I want to call register some time before the certificate expiration, but instead of renewing the certificate it is returning the old one. So I get that I should also send the option renewBy/renewWithin, but then, could I just skip the call to checkabove,  or is register still doing an extra call to the ACME server and getting the old certificate whereas check is more efficient by just returning a local cached one?

Yeah, the semantics of check and register are a little mixed up. However, here's what happens.
check will just check the cache and return a certificate or not.
register will:

use the same function as check uses to identify if there is a cached cert already
register and return a new cert if there is none
return a cached cert if there is one
if the cached cert is 'renewable' (due to expire within renewWithin or duplicate was specified), initiate a renewal in the background, and return a promise for the renewal in _renewing (along with the cached cert); this is undocumented but fairly important, and we probably need to clean it up somehow at some stage

Consequently:

calling register prior to the certificate becoming renewable will not initiate renewal; it will do the same as check, which is somewhat unexpected
even if you force renewal with duplicate, register will still return the cached cert, which is fairly unexpected; if the response contains _renewing, watch it if you want the new cert
calling register multiple times after a certificate becomes renewable will initiate multiple renewals in the background; rate limits can be hit by doing this; renewals that fail will probably become unhandled rejections

There is another function renew that probably does more what you want:

if the cert is not renewable, fail
if the cert is renewable, return a new one

It accepts duplicate and renewWithin just like register does to determine if a cert is renewable.
le.check({ domains: domains }).then(function(cert) {
  if (cert) {
    // we have a certificate; renew it
    return le.renew({ duplicate: true }, cert);
  }

  // register a certificate manually
  le.register({
    domains: domains
  , email<EMAIL_ADDRESS>  , agreeTos: ''
  , rsaKeySize: 2048
  , challengeType: 'http-01'
  }).then(function(cert) {
    console.log('got cert for ' + cert.domains.join(", "));
  }, function(err) {
    console.log('failed: ' + err);
  });
});

O, and I was very sloppy above. Whenever I've said "return" I mean fulfill the returned promise; likewise "fail" means reject the returned promise.

@insightfuls that was a very useful answer, thank you very much. Specifically the renewmethod is the silver bullet for my case.

FYI the official repo is at https://git.coolaj86.com/coolaj86/greenlock.js
Latest version supports Let's Encrypt v2 :)


etcd v 3.0.0 and later generate error: System error: json: cannot unmarshal object into Go value of type libcontainer.syncType.

The docker run command to start etcd in standalone mode with etcd v 3.0.0 and later generates error:
System error: json: cannot unmarshal object into Go value of type libcontainer.syncType.
Docker run command:
docker run -d -v /usr/share/ca-certificates/:/etc/ssl/certs -p 4001:4001 -p 2380:2380 -p 2379:2379 \
 --name etcd quay.io/coreos/etcd:v3.0.9 \
 -name etcd0 \
 -advertise-client-urls http://${HostIP}:2379,http://${HostIP}:4001 \
 -listen-client-urls http://<IP_ADDRESS>:2379,http://<IP_ADDRESS>:4001 \
 -initial-advertise-peer-urls http://${HostIP}:2380 \
 -listen-peer-urls http://<IP_ADDRESS>:2380 \
 -initial-cluster-token etcd-cluster-1 \
 -initial-cluster etcd0=http://${HostIP}:2380 \
 -initial-cluster-state new

etcd version 2.3.7 and earleir do not generate the same error.

@dvohra The error logging seems to come from docker not etcd, right?

@dvohra Please report to https://github.com/docker/libcontainer.


fix(docs): solve type checking errors in Markdown code samples

Fixes issues found by running deno test --doc --unstable at the root of the manual
Updates Deno.Signal examples in examples/os_signals.md to reflect changes made in 1.14.0

Notes:

I changed the type of several code blocks in runtime/workers.md to js because ts throws the following errors when running deno test --doc:

TS2345 [ERROR]: Argument of type '{ type: "module"; deno: { namespace: boolean; permissions: { env: boolean; hrtime: boolean; net: string; ffi: boolean; read: boolean; run: boolean; write: boolean; }; }; }' is not assignable to
parameter of type 'WorkerOptions'.                                                                                e: boolean; net:
  Object literal may only specify known properties, and 'deno' does not exist in type 'WorkerOptions'.             'WorkerOptions'.
    deno: {
    ^
    at file:///path/to/my/dir/manual/runtime/workers.md$197-215.ts:4:5

There appear to be two interfaces called WorkerOptions in two different .d.ts files and it's using the wrong one somehow

Examples with local paths such as ./foo.js don't work with deno test --doc so I added ignores for now
During local testing I replaced the $STD_VERSION vars with 0.107.0 to make it work. If they can also be handled in the CI pipeline then I believe deno test --doc --unstable can be added and run for every workflow

@GJZwiers thank you, I already started similar PR in https://github.com/denoland/manual/pull/32. Let me try to point you on a few things:

I changed the type of several code blocks in runtime/workers.md to js because ts throws the following errors when running deno test --doc:

This appears to be caused by not using --unstable flag - deno object in WorkerOptions is an extension provided by Deno and is considered unstable. So you might want to try running with --unstable flag and reverting back to using ts.

Examples with local paths such as ./foo.js don't work with deno test --doc so I added ignores for now

That's fine 👍

During local testing I replaced the $STD_VERSION vars with 0.107.0 to make it work. If they can also be handled in the CI pipeline then I believe deno test --doc --unstable can be added and run for every workflow

This can be solved by using an import map for testing; please refer to #32 to see how it's done.

I changed back the code blocks to ts and running that with --unstable works. Also fixed some more typechecking issues. Something that was also mentioned in the other PR is that the syntax highlighting goes away when using ts, ignore blocks, so I don't know if that's desirable to push through.
Also encountered a weird error where deno test --doc --unstable panics, which would go away by changing LF to CRLF line endings in VS Code. I will try to see if I can reproduce that and if so make a separate issue. Otherwise I managed to get a clean run of deno test --doc --unstable locally with the current changes

In any case, feel free to combine the changes here with #32 if that's convenient.


Cannot find a way around indexFile easily

Hello there.
Just updated fourseven:meteor-scss to the latest version and I have tried to adapt to the new import syntax without success.
I had this kind of architecture :
.
lib/
 | - client/
 |         |- assets/
 |                  |- core/
 |                  |      |- _base.scss # imports all the other files in core/
 |                  |      |- _*.scss # lots of other files like normalize, typography, etc...
 |                  |
 |                  |- someModule/
 |                               |- _base.scss # imports all the other files in someModule/
 |                               |- _*.scss # lots of other files like normalize, typography, etc...
app/
 | |- module_name1/
 |                |- client/
 |                         |- assets/
 |                                  |- module_name.scss # Has an @import "core/base";
packages/
|       |- package_name/
|                      |- client/
|                               |- assets/
|                                        | - package_name.scss # Has an @import "someModule/base";

With a scss.json in my root directory
{
  "includePaths": [
    "lib/client/assets"
  ]
}

It allowed a concise syntax with @import "core/base"; or @import "someModule/base";
I guess that's not coming back ?
Anyway I have tried to adapt by doing a new core local package /packages/Core/styles/ contains my core folder and someModule folder.
Here is the package.js
Package.describe({
  name: "core",
  summary: "Application core package",
  version: "0.0.1",
});

/* This defines your actual package */
Package.onUse(function (api) {
<EMAIL_ADDRESS>'wolves:bourbon', 'wolves:neat']);
  api.versionsFrom('METEOR@1.2');

  api.addFiles(/* files in folder styles/ */, 'client', {isImport: true});
});

And then doing
@import "{core}/style/core/_base.scss";   and @import "{core}/style/core/"; but they both don't seem to work. Not in packages, nor in modules.
Any light on this ?
Thanks a lot :)

And the core package is added to the .meteor/packages file?
You shouldn't even need to add it to a package, you can import from the app using '{}/folder/file.scss'
In theory this should work, we have tests for this in 3.4.0-beta1, So I would need a reproduction to figure out what's going wrong.

Would you be so kind as to share the error message?

Silly me, I had forgotten to add the new package. My apologies.
Thanks.


CAPolicy.inf doesn't use the parameter value for [InternalPolicy].Url

In Capolicy.inf on subordinate CA, the value for URL under [InternalPolicy] uses default folder name (machine name), instead of the parameter value entered.
E.g.result:
URL=http://{bucker}.s3-{region}.amazonaws.com/SubCa/cps.txt
Where bucket name and region is correcly resolved from parameters, but the folder name was not resolved from machine name parameter. "SubCa" is the default value which should be overridden by the parameter value in the stack.
(This for 2 tier CA stack, on subordinate CA)

If UseS3ForCRL for = no, then it will use the computer name for the URL regardless if a bucket name is set. This is expected behavior.


String escape

related to #49
This adds a better string escape function but also a handful of utility functions to handle utf8 strings better

all CI fail with the 11 tests that showcase known issues / TODOs
This is all good to be merged


Does the model not converge?

Could you specify which model is not converging?

PSNR found negative values

DBPN

These models are all problematic. Can you correct them?

maybe your file path is wrong

FSRCNN works fine at my end and DBPN does converge but it's just SLOW.


Pls add function: Auto save info of videos

Pls add a function:
Auto save info of all downloaded videos in one log file,
or in diff .txt file for each video.
.
Need infos: The video link & title, Uploader link & name, Video Uploaded timestamp, Video Duration,
the files I saved for the link (filetype, resolution, filesize), for example, mp4 3840x2160 1.00G, aac 128k 30.00M, srt en 25.00K

--write-info-json               Write video metadata to a .info.json file
                                (this may contain personal information)

--print-to-file [WHEN:]TEMPLATE FILE
                                Append given template to the file. The
                                values of WHEN and TEMPLATE are same as that
                                of --print. FILE uses the same syntax as the
                                output template. This option can be used
                                multiple times

Do not remove the issue template!!

[x] I understand that I will be blocked if I intentionally remove or skip any mandatory* field


Failed to open image

I'm getting the following error message trying to open an image
imgcat: Failed to open image: yorkshire_terrier_9.jpg
It doesn't appear to be a permission problem because I can do a hexdump on the file with hexdump Yorkshire_terrier_9.jpg. I doubt that the image is corrupted as it's part of a well known image dataset.
I've been able to correctly display images from another dataset but they were png's so I don't know if this problem is specific to jpgs.

Hi @zacharywhitley. Could you run file Yorkshire_terrier_9.jpg? It's difficult debug this without more info :/

Hi @zacharywhitley. Could you run file Yorkshire_terrier_9.jpg? It's difficult debug this without more info :/

I should have included that I had run that command before.
file yorkshire_terrier_9.jpg
yorkshire_terrier_9.jpg: JPEG image data, JFIF standard 1.01
If I convert the image with convert yorkshire_terrier_9.jpg yorkshire_terrier_9.png it displays correctly
imgcat yorkshire_terrier_9.png
Maybe I'm missing a library? I'll go back and check the built to make sure I didn't miss any errors or warnings
BTW, love the tool. It's been great for AI image sets where I'm often in a terminal and don't really care exactly what the image looks like but just want to get an idea that I'm looking at the right thing.

I should have included that I had run that command before.
file yorkshire_terrier_9.jpg
yorkshire_terrier_9.jpg: JPEG image data, JFIF standard 1.01
If I convert the image with convert yorkshire_terrier_9.jpg yorkshire_terrier_9.png it displays correctly
imgcat yorkshire_terrier_9.png
Maybe I'm missing a library? I'll go back and check the built to make sure I didn't miss any errors or warnings
BTW, love the tool. It's been great for AI image sets where I'm often in a terminal and don't really care exactly what the image looks like but just want to get an idea that I'm looking at the right thing.

Aaaaaaaand there it is. Sorry about that. I had missed the following error when running ./configure
checking to link with libjpeg [using jpeg_set_defaults()]... /usr/bin/ld: cannot find -ljpeg

Aaaaaaaand there it is. Sorry about that. I had missed the following error when running ./configure
checking to link with libjpeg [using jpeg_set_defaults()]... /usr/bin/ld: cannot find -ljpeg

That's okay! I don't know what system you're on, but you'll want to run either sudo apt install libjpeg-dev or brew install libjpeg

That's okay! I don't know what system you're on, but you'll want to run either sudo apt install libjpeg-dev or brew install libjpeg

Thanks, and for anyone following along in the future. It was on Amazon Linux (which I think is based on RHEL). Installing libjpeg-turbo-devel did the trick.
yum install libjpeg-turbo-devel

Thanks, and for anyone following along in the future. It was on Amazon Linux (which I think is based on RHEL). Installing libjpeg-turbo-devel did the trick.
yum install libjpeg-turbo-devel


[Backend] Fake data should be populated after putting data in "search for profile page"

Waiting on https://github.com/codefordc/cfsa-referral/issues/14

Closing, done in https://github.com/codefordc/cfsa-referral/pull/39


Unable tosave file to MediaStore

Hi, I'm trying to save file in android in Download folder.
I've read this issue #https://github.com/RonRadtke/react-native-blob-util/issues/62 , but I can't find a solution...
My code is pretty simple:
ReactNativeBlobUtil .config({     fileCache: true }) .fetch('GET', myUrl, {Cookie: authCookie}) .then(async (res) => {                      let pathnew = await ReactNativeBlobUtil.MediaCollection.createMediafile({name: attachment.name, parentFolder: '', mimeType: attachment.mimeType}, 'Download');     await ReactNativeBlobUtil.MediaCollection.writeToMediafile(pathnew, res.path()); })
The error is
TypeError: Cannot read property 'createMediafile' of undefined TypeError: Cannot read property 'createMediafile' of undefined
I have installed
react-native: "0.71.7",
react-native-blob-util: "^0.13.18"
Why MediaCollection is undefined?
By the way in IOS everithing works fine, so this is not an issue with my url or something else.

MediaStore api got inteoduced in 0.14.0. So you have to updste the lib
Why do you use such an old version of the library?

Why do you use such an old version of the library?

I don't know, it was the default version when I installed the library.
However updating did the trick! Thank you very much.


Latest Forecat values

Detailed Description
The current API takes 1.5 minutes to get one forecast. This is too slow.
SELECT *
FROM (
	SELECT DISTINCT ON (forecasts.site_uuid) forecasts.forecast_uuid, forecasts.timestamp_utc FROM forecasts
	WHERE forecasts.site_uuid IN ('04b3d67f-af48-42f7-ac0f-9d07d4dd84f6'::uuid)
	ORDER BY forecasts.site_uuid, forecasts.timestamp_utc DESC)
	AS anon_1
	JOIN forecast_values ON anon_1.forecast_uuid = forecast_values.forecast_uuid
ORDER BY anon_1.timestamp_utc, forecast_values.start_utc

Context

The UI needs to be able to get 1000's of forecasts

Possible Implementation

Create forecast_value_latest table. This table will only save the latest forecast (forecast_horizon 0) and the latest one in the future
Change the save functions to also save / update to this table
Change the read function to load from this table
This is how we do it in the nowcasting project

Add a forecast_value_forecast_hoirzon_0 table, This table will only save the latest forecast (forecast_horizon 0)

update save method to also save to this table
update read method to read from this table

Sorry if i missed anything @simlmx, i might have

Note that getting history is easy with, so we just worried about getting the forecast values from now on
SELECT DISTINCT ON (forecast_values.start_utc) * from forecasts
JOIN forecast_values ON forecasts.forecast_uuid = forecast_values.forecast_uuid
WHERE forecasts.site_uuid IN ('04b3d67f-af48-42f7-ac0f-9d07d4dd84f6'::uuid)
ORDER BY forecast_values.start_utc, forecasts.timestamp_utc desc

Note that the forecast value table has 100 million rows.
If we made a latest table it would have
1000 * 96 *7  ~ 1 million rows for 7 days of data

If you add SET random_page_cost = 1.1; before your query (the setting has not been set in the database yet, it's still pending; the SET command will set it temporarily), your initial query runs in 3ms.

If you add SET random_page_cost = 1.1; before your query (the setting has not been set in the database yet, it's still pending; the SET command will set it temporarily), your initial query runs in 3ms.

wow - amazing, ill do that right away

I'll close this for now, as I dont think we are going to do this


Bootstrap tables

When combined with bootstrap tables it destroys the table borders, leaving it behind the table elements.

@animalillo do you have a CodePen/JSfiddle demonstrating the issue?


Add headers to SIP inbound/outbound trunk API requests

Add headers to:

Inbound Trunk
Outbound Trunk

in sip_service

Added in https://github.com/livekit/python-sdks/pull/303/


Accessing monaco.languages

The readme says that the component fires editorWillMount where we can access editor.languages.
In my current use case, I wan to add support for a whole language (including multiple providers) to the monaco.languages object, so every editor instance can use these.
However, editorWillMount seems to fire for every editor instance. Initializing the language sopport in this method will result of multiple executions of said initialization.
Is there a better way for doing that? Some editor instances I have don't even use my custom language. Using some sort of flag seems to be a dirty solution for me.

Digging through the source of the monaco-editor package I found out how the services of HTML and TypeScript are getting loaded. They use monaco.languages.onLanguage:
    /**
     * An event emitted when a language is first time needed (e.g. a model has it set).
     * @event
     */
    export function onLanguage(languageId: string, callback: () => void): IDisposable;

This method can be used to initialize a language in editorWillMount . Works for me.


Add the SCSI version as a configuration option

Info
There have been a couple point solutions (#828 and #1109) to add support for SCSI-1 in the inquiry response. These solutions work for HD's and CD's, but this should expanded to all device types and should not be based upon file name extension.
Proposed solution is to add a configuration parameter, similar to the Vendor/Product/Revision options, that allows the SCSI version to be specified. This should be an option that applies to all devices, not just HD and CD.
The web interface should also be updated to allow the user to select which SCSI version.

We should look at making a generic configuration input that allows the user to specify key/value pairs as configuration options. We're getting a lot of command line arguments in scsictl.
Something like: (-a for argument, maybe... there aren't many letters left)
scsictl -c attach -i 3 -f ~/images/drive.hda -a SCSI_VERSION:1

@akuker Are there any real-world scenarios (user reports) where the current solution does not work?

I suggest to close this issue as "won't" do, based on my last comment: There are no other SCSI-1 device types than the ones already covered.

@uweseimet I agree that introducing a SCSI level selection right now isn't high priority, since the known corner cases have been covered by the special file ending code paths now. I suggest we revisit this if unhandled SCSI level corner cases are discovered.
I still think it's worth discussing quirks mode in general as @akuker suggests. We have one major pending compatibility issue with a known workaround (see https://github.com/PiSCSI/piscsi/issues/741 ) that may benefit from introducing a quirks mode option. Thoughts?

@rdmark IMHO this is not primarly about priorities. PiSCSI should not emulate SCSI-1 device types that have never existed, because (unless somebody proves the opposite) there is no computer that would expect such a device.
Regarding any kind of quirks mode you might end up with a bunch of work-arounds where nobody knows which mode to use when, and what it means to have a combination of these modes. The original RaSCSI code contained many strange work-arounds, and instead of improving compatibility they just did the opposite.
Often when you think a work-around is needed in practice this indicates that the underlying problem has not fully been analyzed. In the case of https://github.com/PiSCSI/piscsi/issues/741, why would artifically slowing down the rate at which PiSCSI executes commands be an improvement? I doubt that changing MIN_EXEC_TIME is the right solution but rather think the core of the problem has not sufficiently been analyzed. And users are still waiting for feedback on their SCSIMON logs, as far as I can tell.

Withdrawing this ticket.


CSV input plugin from local file

The ability to import a local CSV file
Advantage:
Right now the CSV plugin can only read files from an external URL; there are some sites where it is zipped and or the access is not direct; therefore it should be easier to download the files and then send it to flexget.

Please reopen in feathub


Update platform.md

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65b2f3b8f3f6e94ddf1b9703--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65b3b8b9fdec1d0e6ccf02e9--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65b3eb8481c5093a03029c60--harness-developer.netlify.app


I almost did not pick debug...

but really glad that I did 👍
Here's the long story - I was looking for a debugging library for my own work. Obviously debug was very attractive because of its popularity. However when I read the docs, there was no mention of TypeScript! It was hard to believe that such a popular dialect was simply ignored. Searched the issues, found one (https://github.com/visionmedia/debug/issues/669), which was open since Jan, 2019 - not very promising. Yes, there was a mention of @types/debug, but looking at these type definitions without any docs made it difficult to understand the usage :-(
So I moved on to my second and third choice. They both supported TypeScript and at least had a small section devoted to it in the docs. Very promising! Used them for a bit, but somethings did not feel right, e.g namespace support was an afterthought.
So I came back to debug and looked harder to understand the TypeScript definitions. Finally found this StackOverflow question that helped me move forward with debug. Now that I have started using debug, I love it!
Another thing that was puzzling me was the absence of logging levels. Only after digging into issues, did I understand the philosophy that debug is not a general purpose logger, it is simply meant for debugging, i.e. console.debug!
Now that I am a fan of debug, may I suggest to emphasize the above two points in the docs. It will save hours for people evaluating and choosing a debug library!

Debug is a JavaScript library, not typescript. Typescript is not a dialect of JavaScript, it's its own language.
First instinct should be to check DefinitelyTyped. There's nothing actionable here.


Typo in variable name in comment_notification templates

In both txt and html versions, a variable is attachmnet_url instead of attachment_url

Where exactly are those variables defined?

My apologies. I posted this to the wrong GitHub repo.


WebXR in "immersive-ar" session mode shows opaque artifacts overshadowing the background

Godot version
3.5.x
System information
Meta Quest 2
Issue description
After starting the AR session,

every pixel in the viewport to which an opaque object was rendered stays opaque. This kind of looks like a ghosting artifact, but the trail is not vanishing. As if the transparency information is not cleared between the drawing of frames.
Transparent objects get more and opaque at the position on the screen, where I look at them (as if they are 'burning in')
I used the amazing tutorial from @dsnopek at his website to build a simple AR application by replacing immersive-vr with immersive-ar and setting get_viewport.transparent_bg = true before entering AR mode (otherwise I would get a similar artifact as shown in https://github.com/godotengine/godot/pull/45548)

Further information:

I also built an example in godot 4 and it does work, no artifacts at all
There are also no artifacts visible when I use my smartphone for immersive-ar
The immersive-web webxr examples are working just fine on my Quest 2, so it does not seem to me that this is an issue with my device or browser I am using.

Steps to reproduce
The easiest way for me is to just view this example project of @dsnopek (https://bit.ly/godot-webxr-10) in the Meta Quest 2 browser.
Regarding the projects I built, I was able to reproduce the problem with the 3.5-stable and 3.5.2-stable releases.
Minimal reproduction project
Here is a minimal example using the code from the tutorial (leaving out the controllers) and adding two meshes to show the rendering problems.
webxr-ar-bug-minimal.zip

Thanks for the report!
I'm able to reproduce this on the Meta Quest 2. However, I checked with AR on my smartphone, and it's still working, so I think this is unrelated to PR #45548
Perhaps we aren't clearing the render target with the clear color when doing stereo rendering?
Needs more investigation!

An untested theory: it looks like Godot is clearing the render target before each eye as it should. But in Godot 3, we're using a shader to draw the texture from the render target from Godot onto the framebuffer provided by WebXR. I think that means that any transparent pixels in the texture won't overwrite pixels that are already on the framebuffer.
So, perhaps we need to either:

Clear the framebuffer from WebXR at the beginning of each frame. I think we could do this when commit_for_eye() is being called with either the LEFT or MONO eye.
Change the way we're doing the shader, so the resulting fragment overwrites what's there, rather than mixing it. Perhaps doing something clever with blendFunc()?

My untested theory turned out to be correct!
PR #76072 fixes this issue in my testing

Awesome! I just tested it on my Quest 2 ... thank you!

That's great! Thanks for testing the PR :-)
Let's keep this issue open until it gets merged though

PR #76072 was just merged - yay! So, closing again :-)


Add configuration for CI: Travis-CI and AppVeyor

The addition of these simple .travis.yml and appveyor.yml files here configures Travis-CI and AppVeyor to run automated tests for each push to the repository.
An explanation of CI setup and configuration is currently available here.

You can see the build results here: https://travis-ci.org/SeattleTestbed/seattlelib_v2
This includes a variety of failures, listed here as links to the build output with the relevant lines highlighted:

Running: ut_seattlelib_dylinkrepyportabilityimportmodulesymbols.py [ FAIL ]
Running: ut_seattlelib_httpretrieve_content.py              [ FAIL ]
(excluding the other errors in that class of "'dy_import_module_symbols' is not defined")
Running: ut_librepy_recvmess-reuse.r2py                     [ FAIL ]
Running: ut_librepy_socket-nonblocksend.r2py                [ FAIL ]
Running: ut_librepy_waitforconn-reuse.r2py                  [ FAIL ]
Running: ut_librepy_waitforconn-stop.r2py                   [ FAIL ]

I won't go into the same amount of detail for all the other Travis-CI runs, or the AppVeyor runs here, but most of them are similar.

Seems to be working as intended now - 3 Python versions on Linux (2.5, 2.6, 2.7), 2 on OS X (2.6, 2.7), and 2 on Windows 2012 Server RC2 (2.6, 2.7), with the Windows builds running on 32- and 64-bit architecture.

I confirm successful run of travis-CI with provided config in seattlelib_v2 master branch and test branch by pushing newly created .travis.yml to each branch respectively.  Tests ran on Linux (Python v2.7.9, v2.6.9, v2.5.6) and OS X (v2.7.5,  v2.6.9).
Results can be found on https://travis-ci.org/lukpueh/seattlelib_v2/branches

.travis.yml and .appvoyer.yml are generic for all(?) SeattleTestbed components so it would be nice to have a central SeattleTestbed/CI repo for the config files where they can be maintained and cloned from.

That (central config) is a good point, Lukas. (: Maybe they can live in SeattleTestbed/common or SeattleTestbed/buildscripts. Thoughts?

SeattleTestbed/buildscripts sounds good. SeattleTestbed/common as this would

Candidates and docs for this PR have moved (see SeattleTestbed/continuous-integration#2 and SeattleTestbed/docs#19). We'll address modifications there and also deploy the fresh CI configs from there. I'm closing this PR for the time being.
To be sure that we won't omit any changes, I locally compared the relevant files:
$ diff -u continuous-integration/appveyor.yml seattlelib_v2-awwad/appveyor.yml
# no output - files don't differ
$ diff -u continuous-integration/.travis.yml seattlelib_v2-awwad/.travis.yml
# no output - files don't differ


Component is missing template or render function.

Throw error: Component is missing template or render function.
with Vite 2.7.3 and Vue 3.2.30

@kevinmou-sh Hoe do you import and register your component?


crypto/des: error in comment

What version of Go are you using (go version)?

$ go version 1.14.4

Does this issue reproduce with the latest release?
Yes, in the master-branch.
What operating system and processor architecture are you using (go env)?

OS: Windows10 (64bit);
Arch: amd64

go env Output
$ go env
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\***\AppData\Local\go-build
set GOENV=C:\Users\***\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\***
set GOPRIVATE=
set GOPROXY=https://goproxy.cn,https://proxy.golang.org,direct
set GOROOT=C:\***
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=C:\***\pkg\tool\windows_amd64
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:***\AppData\Local\Temp\go-build153534510=/tmp/go-build -gno-record-gcc-switches

What did you do?
I found a comment error in src/crypto/des/block.go in line 130:
// block is now b1 b3 b5 b7 b0 b2 b4 b7, the permutation:

What did you expect to see?
I think it should be :
// block is now b1 b3 b5 b7 b0 b2 b4 b6, the permutation:

the last byte may should be "b6" instead of "b7"

/cc @FiloSottile


feat: allow specifying runtime class in runner spec

This allows using the runtimeClassName directive in the runner's spec.
Our use case for this is Kata Containers, which use runtimeClassName in a pod spec as an indicator that the pod should run inside a Kata container. This allows us a greater degree of pod isolation.
I tried my best to follow a similar PR https://github.com/actions-runner-controller/actions-runner-controller/pull/436. I had a bit of trouble running integration tests locally, but I was able to test this on a live cluster. Let me know if anything looks off.

Waiting for the CI workflow to pass.

Glad that error wasn't my fault it was pretty confusing 🤣
Thanks for reviewing so fast!

Yes could we get it added to the docs please, additionally there needs to be a description included for the key in the CRD @inahga

Sorry for the delay, it was US holiday this weekend. README and CRDs have been updated!
For future pioneers, to update the CRD we need to comment the field in the API definition, like we would for godoc. Then run make manifests.


stop using forked bls now that it's updated with the necessary method

With https://github.com/protolambda/go-kzg/pull/25 being merged we should be able to move off of this fork entirely, so I will close after confirming this.


Feature/update block position #129

Description
Added the ability to do an optimistic update of the position of a block. The code is now ready to send a background message to the server.
Testing instructions
Add a set up instructions describing how the reviewer should test the code

Review code
Check Travis build
Review changes to test coverage

Agile board tracking
connect to #129

Codecov Report

Merging #150 into version1 will decrease coverage by 3.92%.
The diff coverage is 51.72%.

@@             Coverage Diff              @@
##           version1     #150      +/-   ##
============================================
- Coverage     92.34%   88.41%   -3.93%
============================================
  Files            47       47
  Lines           875      941      +66
  Branches        132      142      +10
============================================
+ Hits            808      832      +24
- Misses           61       93      +32
- Partials          6       16      +10

Impacted Files
Coverage Δ

src/malcolm/reducer/attribute.reducer.js
89.47% <ø> (-0.4%)
:arrow_down:

src/layout/block/blockWidget.component.js
92.3% <0%> (-7.7%)
:arrow_down:

src/malcolm/reducer/malcolmReducer.js
87.5% <0%> (-3.81%)
:arrow_down:

src/malcolm/malcolmActionCreators.js
100% <100%> (ø)
:arrow_up:

src/malcolm/malcolm.types.js
100% <100%> (ø)
:arrow_up:

src/viewState/viewState.actions.js
63.63% <20%> (-36.37%)
:arrow_down:

src/malcolm/reducer/layout.reducer.js
76% <42.85%> (-24%)
:arrow_down:

src/layout/block/block.model.js
76.92% <50%> (-23.08%)
:arrow_down:

src/mainMalcolmView/middlePanel.container.js
80% <50%> (-20%)
:arrow_down:

src/layout/layout.component.js
65.9% <60.52%> (-34.1%)
:arrow_down:

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 e334084...344d8bc. Read the comment docs.

Reviewed code myself


[MINOR] Avoid yarn conflict + Save Metrics after waiting to get full data

Major changes:
Minor changes to note:

Change yarn command in run_beam.sh file to avoid conflict with javascript package manager
Split metric saving part from flushMetric function and call metric saving part after waiting metric flush

Tests for the changes:
Other comments:
Closes #GITHUB_PR_NUMBER

@Lemarais Congrats on your first PR!

Great job!


openjdk11: split gnome3 and gnome2 deps

Motivation for this change
Change this as a proof of concept to get rid of gnome2 deps in mid-term as requested in #39976.  I'm not very familiar with Java but I'm getting problems running applications with these deps: (#72986  and #73246).
If this is the way to go I can change other openjdk versions.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @edwtjo

Some external references:
JEP 283: Enable GTK 3 on Linux
[JavaFX] Conditional support for GTK 3 on Linux
[JavaFX] Backport GTK3 support on Linux to 8u

any opinion?

How do we test this?

what do you mean? as in nixos testing framework? No test are needed because default behavior is the same. We can start to disable gnome2 in app by app basis after this. Have sense?

I meant do you have an app I can test this with?

Wow, it's true that nobody is using openjdk11 in nixpkgs. I make an grep search and the only candidate is ghidra-bin, I'm testing it now. Also I'm building Bisq but is not yet finalized for commit.

I've test ghidra-bin without gnome2 but GUI doesnt work. I'm wondering: with only 2 leaf packages depending on openjdk11 maybe I could try to remove totally gnome2 deps while try to make ghidra work?

Do you know if the JavaFX/Swing/whatever is toolkit independent? Is the API abstract enough that apps do not have to care about which version of GTK are we using? If so removing gnome2 deps altogether would be a good idea.

Sorry but i don't know. i'm not very familiar with Java stuff. Anyways I feel confident to research and test it.

Both gtk2/gtk3 only seem to be a runtime dependency, not even a build dependency: https://git.archlinux.org/svntogit/packages.git/tree/trunk/PKGBUILD?h=packages/java-openjdk#n179
Therefor I think we could drop gtk2. It seems to be used for themes.

glib is required to read proxy settings from GSettings.
If java.net.useSystemProxies system property istrue, GSettings is checked (and GConf is checked if GSettings is not available).
https://docs.oracle.com/en/java/javase/11/docs/api/java.base/java/net/doc-files/net-properties.html
https://github.com/openjdk/jdk11u/blob/master/src/java.base/unix/native/libnet/DefaultProxySelector.c
https://github.com/openjdk/jdk11u/blob/727ddb06fd780ecba80dd31db0700a0005ffedda/src/java.base/share/classes/sun/net/spi/DefaultProxySelector.java

The main motivation for this change was to workaround a crash in Bisq software. Nowadays Bisq is working nice with vanilla openjdk11 as in latest nixpkgs commit.
So I close this PR because have big implications and require in-depth java knowledge.


Possible issue with NetCDF mesh reader

For some numbers of processors the coil.gen mesh in the mfem/data repository issues an error.  A user reported problems with 4 processors, I experienced trouble with 6 or 8 processors.
$ mpirun -np 4 joule -m coil.gen  -p coil
....
Skin depth sqrt(2.0/(wj*mj*sj)) = 0.551329
Skin depth sqrt(2.0*dt/(mj*sj)) = 0.126157

Verification failed: (stria_flag[i] == stria_master_flag[i]) is false:
 --> inconsistent vertex ordering found, shared triangle 539: (6899, 6898, 3171), local flag: 2, master flag: 1
 ... in function: virtual void mfem::ParMesh::ReorientTetMesh()
 ... in file: mesh/pmesh.cpp:2726

See discussion #2023.

@mlstowell and @Dan2997925 -- any progress on this?

@tzanio, I believe the problem is related to ParMesh::MarkTetMeshForRefinement. If we finalize the ParMesh in Joule but set refine to false the problem seems to go away.
Also, if we open the coil.gen mesh in mesh-explorer and use this to save it as an MFEM mesh then Joule will run fine on the new mesh regardless of the refine setting. After comparing the original mesh and the copy produced by mesh-explorer we find that 80 of the roughly 250,000 elements have been reordered. The first five of these 80 elements (I didn't check the rest) all share a particular property; two of the three longest edges have the same length (to machine precision). The Tetrahedron::MarkEdge member function reorders element vertices based on the three longest edge lengths. This is clearly not a coincidence.
My guess is that the marking of tetrahedra for refinement is occasionally failing in parallel because neighboring processors might make different marking/reordering decisions when the three longest edge lengths are not unique. This fits with the observation that different partitionings can often avoid this problem since the equilateral elements may not always lie on processor boundaries.
I am very confident this is the root of the problem but I don't understand the marking/reordering strategy well enough to devise a solution that will work in parallel.

The code in ParMesh::MarkTetMeshForRefinement is supposed to ensure consistent marking across different ranks -- exactly to support the case of edges with the same lengths. However, it is possible that the algorithm used there might not be correct in all cases. We can add some test code at the end of ParMesh::MarkTetMeshForRefinement to look for such cases -- just make sure the problem is really there.


Review Letter: Address Verification

This is a part of ProgramEquity/amplify-front-end#44 where after user inputs address, we use the Lob API address_verification method. The goal is to create address verification call on user input.
To do:

[x] Update the api url to createAddress instead of addressVerification as it is currently

submit () {
            this.formHasErrors = false

            Object.keys(this.form).forEach(f => {
                if (!this.form[f]) this.formHasErrors = true

                this.$refs[f].validate(true)
            })

            axios.post('https://murmuring-headland-63935.herokuapp.com/api/lob/addressVerification', this.form)
                .then((response) => {
                    console.log(response)
                    this.message = 'Address verified!'
                    }
                })
                .catch(function (error) {
                    console.log(error)
                })
        }

[x] On successful verification, store the address_id that is received from the createAddress request. This will used as the to variable in createLetter
[x] Create a letter object using the createLetter api which adds the user to the addressbook in (https://docs.lob.com/#operation/create_address) and the following information from Lob. From will be the representative address that should already be accessible.

#91


tests/quay: test login authentication

Add a login test for quay.io. Credentials for a dedicated CI user
are set up on travis.

Uhm, this worked on master but this PR is not seeing user/passwd vars.
@steveeJ it's a free "dkregistry" user on quay/dockerhub. It doesn't have any repo but we can grant read access to existing private ones.

Uhm, this worked on master but this PR is not seeing user/passwd vars.
Probably some deeply hidden travis setting.

it's a free "dkregistry" user on quay/dockerhub. It doesn't have any repo but we can grant read access to existing private ones.

I granted that user read access to steveej/cincinnati-test which will be available for the time being.

@steveeJ changed to use your pull-secrets. This covers login+tags+manifest right now. We can land it first and add more things later, or I can add a few more here, as you prefer.

This now also covers fetching the blob from a single-layer private image, PTAL.


docs: update kythe release path

This is where the file appears on my machine.  I guess it's changed with a more recent version of Bazel?

thanks!

Out of curiosity @justbuchanan, how does the static site release work?  I saw this change still isn't on kythe.io.

Typically the person who commits the change needs to deploy the changes using https://github.com/kythe/kythe/blob/master/kythe/release/appengine/default/deploy.sh


Add option to move a technology from one endpoint to another

Suggested by @DorinevanderVlies
It would be useful for users to be able to move a technology from one endpoint to another. Perhaps hidden in the "Advanced settings" section of a technology so as not to clutter the form too badly.

The motivation is that a visitor may create a scenario in the ETM which is meant to represent a real region, and then wants to move technologies from one endpoint to another to more closely match the real-life situation. Currently, they would have to delete and re-create the technology on the other endpoint, making sure to get all the attributes identical. A "move to" feature would save a lot of effort.
@grdw As I'm not too familiar with the technology editor, I'm not confident in estimating how long this would take. Do you have any idea how hard this might be?
I can imagine buffers complicating things somewhat: a user might want to move a space heating technology to a node with no space heating composite; do we not give them that option – excluding that node from the list of possible destinations – or do we automatically create a space heating composite on the destination node? Not sure... the former is probably more "correct", but less intuitive.
@ChaelKruip Do you think this would be useful?

As I'm not too familiar with the technology editor, I'm not confident in estimating how long this would take. Do you have any idea how hard this might be?

Well, the easy way would be a 4 hours? That's just moving the technology under a separate node and updating the matrix and making buffers move correctly.
What I think that users want to happen here is that when your matrix looks like:
- Endpoint 1
   EV-car (10 units)
- Endpoint 2
   EV-car (5 units)

Move EV-car from Endpoint 2 -> Endpoint 1:
- Endpoint 1
   EV-car (15 units)
- Endpoint 2

So should it be possible to merge? Or only merge when the capacities / volumes / demands are not similar?

a user might want to move a space heating technology to a node with no space heating composite

Maybe you should only have the option to move buffers and not the technologies that lie under them? Or maybe we should exclude that end-point option when an end-point doesn't have that specific buffer.

So should it be possible to merge? Or only merge when the capacities / volumes / demands are not similar?

That's a good point. Merging techs would be nice, but I guess the attributes would have to be identical.

Maybe a user should only have the option to move buffers and not the technologies that lie under them?

This also seems like a good suggestion to me, but it might be a bit drastic to move the entire buffer from one node to another (all heat demand would disappear from the source node).
@DorinevanderVlies Do you have any preference for how we would handle moving space heating and hot water technologies?

Merging techs when identical sounds like a good idea
Regarding space heating and hot water technologies. When I move my electricity demand from one node to the other (because my households are not evenly distributed) I would like (a representative portion of) my buffer to move as well.
For now (Ameland project) I will try to get around this by using .csv input for my technology table.

I like the idea of moving technologies, but with the need to merge techs, move buffers and demand, it seems that this could be a rabbit-hole of complexity and special-cases. 😬
Perhaps a better approach would be for us to set a number of units on the endpoints, and assign technologies in proportion when importing from ETModel. That is, if you have two endpoints and one has units: 2 and the other units: 1, then the first endpoint should get twice the number of technologies and demand.
This is described in #336.

assign technologies in proportion when importing from ETMode

Yes, that would be better. Couldn't we also allow for the whitelisting of certain technologies? Let's say endpoint-1 can only have Solar Panels or something along those lines.

The idea to influence the way the technologies are distributed after importing from ETM would be a great improvement as well.
This could be complementary or instead of shuffling technologies around.

Couldn't we also allow for the whitelisting of certain technologies

I agree that whitelisting  certain technologies would be nice.

assign technologies in proportion when importing from ETModel

Yes, if the combination of the above is implemented I think it will be easier to make the initial scenario represent a real situation

cc @dennisschoenmakers and @ChaelKruip

I like the following idea (courtesy of @antw): When the user creates a new LES, the current 'technology matrix interface' is not shown but the user will be looking at something like this:

Here, the user can move technologies from one end-point to another and change concurrencies.
After completing this, the next step would be to show the load page.
Later, we can make the technology blobs clickable so the user can edit its attributes. For now, clicking 'edit' will lead to the technology matrix as it currently is.


Only linearize active constraints

Codecov Report

Merging #19 into master will increase coverage by 1.29%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
+ Coverage   62.57%   63.86%   +1.29%
==========================================
  Files          48       48
  Lines        2402     2239     -163
==========================================
- Hits         1503     1430      -73
+ Misses        899      809      -90

Impacted Files
Coverage Δ

src/solver/system.jl
14.79% <0.00%> (-1.21%)
:arrow_down:

src/bounds/bound.jl
12.50% <0.00%> (-12.50%)
:arrow_down:

src/joints/rotational0.jl
6.25% <0.00%> (-10.83%)
:arrow_down:

src/joints/joint.jl
82.92% <0.00%> (-9.76%)
:arrow_down:

src/joints/translational0.jl
0.00% <0.00%> (-8.83%)
:arrow_down:

src/util/util.jl
75.00% <0.00%> (-2.78%)
:arrow_down:

src/util/custom_static.jl
47.72% <0.00%> (-2.28%)
:arrow_down:

src/solver/newton.jl
100.00% <0.00%> (ø)

src/joints/rotational.jl
100.00% <0.00%> (ø)

... and 25 more

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 6986bd6...284cee3. Read the comment docs.


Adds structured page builder content types

While fixing the merge conflicts I seem to have messed up the history, I will recreate this MR to keep it clean.


Create rule DDF99981: Each StudyElement must be linked to at least one StudyCell

Create rule defined for #144

ElementId not added to test data Excel for studyCell while it is in the model.
This involves cross class checking (since the elements are nested in studyDesign and only cross-referenced in studyCell). It is not possible to create this rule based on the current capabilities.


Stormpath sibling dependency breaks apiKey option

This is a strange one, @typerandom can you confirm this reproduction?
Create a new node app, with the following package.json:
{
  "name": "test",
  "main": "server.js",
  "dependencies": {
    "express": "~4.10.0",
    "express-stormpath": "^3.0.1",
    "stormpath": "^0.17.5"
  }
}

Install the modules, then run this server with some valid credentials and an application, and put them into the code as seen here:
var express = require('express');
var stormpath = require('express-stormpath');

var app = express();

app.use(stormpath.init(app, {
  apiKey : {
    id: 'xxx',
    secret: 'xxx'
  },
  application: {
    href: 'xxx'
  }
}));

app.on('stormpath.ready',function(){
  console.log('Stormpath Ready');
});

I get the error  Error: API key ID and secret is required.
Then I do the following:

Remove stormpath from package.json
Delete the node_modules folder
npm install

Then I run server.js again, and it works: they keys are accepted, and Stormpath initializes.

I was seeing something like this, but I tried running
npm shrinkwrap
and then it went away.  But I haven't tried restarting the service since then, not wanting to push my luck.

@robertjd I'm looking into this. It sounds very weird, like there's an old dependency (either linked) or fixed (with shrinkwrap) that is causing it... But I'm going to try myself and see if it's reproducable.

I just tried this and I wasn't able to reproduce the issue. I got the Stormpath Ready message immediately.

Did you make sure that you don't have your API keys specified elsewhere,
such as environment variables, yaml files, or properties files in your
homedir?
On Mon, Mar 7, 2016 at 4:35 PM, Robin<EMAIL_ADDRESS>wrote:

I just tried this and I wasn't able to reproduce the issue. I got the Stormpath
Ready message immediately.
—
Reply to this email directly or view it on GitHub
https://github.com/stormpath/express-stormpath/issues/375#issuecomment-193527131
.

I didn't. But I tried first running the application href as 'xxx' and that threw an error, then I ran the api key with 'xxx' and it threw an error. Then I replaced the values with real ones. And then it worked.
@robertjd If you do the steps, are you able to replicate the error each time?

@typerandom and I just dug into this, and we think it's an issue with npm 2.x - we can reproduce my problem there, but not with 3.x.
@schamp can you tell us what npm version(s) you're using?  Thanks!

I'm running on a cloud foundry container, the NPM version I have appears to be 2.14.9

I'm going to close this issue for now, we've found several instances where upgrading to NPM 3.x will resolve the problem.  I'd still like to know why our library can get into this edge case, but this NPM workaround is acceptable for now.


config: move config update interface from pd to status server

What is changed and how it works?
What's Changed:
move config update interface from pd to status server
Check List
Tests

Unit test
Integration test

/merge

/merge

/run-integration-common-test

/merge

/run-integration-copr-test

/run-cherry-picker


Question on slashing module

Hi,
As i see the slashing module is not imported to stargateclient.ts
Not sure how we can use it to query? Maybe i'm missing something
Appreciate the answer. Thanks!!

Hi, @webmaster128 , not sure my question is clear. I meant https://github.com/cosmos/cosmjs/blob/main/packages/stargate/src/stargateclient.ts doesnt import slashing module to export it. How do we use this slashing module then? thanks

Yeah, so there is no great answer here, bu maybe I can help. The StargateClient/SigningStargateClient only install the query extensions they need to power the higher level methods that are implemented there. They do not include slashing. Their query client is private and should not be used directly. If you need a QueryClient with custom extensions, you can create one on you own, like here: https://github.com/cosmos/cosmjs/blob/v0.29.0-rc.1/packages/stargate/src/modules/slashing/queries.spec.ts#L12
This is the query part. For slashing, no messages are implemented. Do they exist?


[EN] Feedback for: /web/tools/setup/?hl=en

why  DevTools search can not search html files,but when i click pretty print it can search the html file

Duplicate of #2606 (even though it was filed after, I've already answered there.)


button can‘t set icon and title.

button can‘t set icon and title,just only one,support all better.

so did i


OADP-4225: add tzdata to Dockerfile.ubi

this is already in the d.s. build and  does not impact product build


Swap image and provenance registries

@ianlewis / @laurentsimon  Here is the PR that does:

image pushed to GHCR.io
provenance pushed to docker.io with username laurentsimon (hardcoded) and ${{ secrets.E2E_DOCKER_HUB_TOKEN }}
Both registries expect the slsa-framework/<workflow-name> as the namespace/repository to exist
Doesn't support any deletion of images

Let me run it and see if it works


Standardize results

Builders should return a standard shape of what they did. It will also help with reporting. Right now it's an undocumented part of the api, but also just a dumping ground of random results. The reporting feature should help design this.

I guess this is doneish.


CGO is disabled by default

Hi!
I'm using go-sqlite3 lib to interact with SQLite DB from the application.
At the same time, I'm using the apiserver-boot script to build docker image apiserver-boot build container --image image_name. Since the go-sqlite3 is using C code, I need to set CGO_ENABLED=1, but it's hardcoded as CGO_ENABLED=0 inside the script.
Is there a way to provide CGO_ENABLED=1 from outside? Probably, do you have any suggestions on how to fix this?
Thanks!

mmm it should override the CGO option if it already exists in the env-vars.. will send a pull to make it override'ble