Input type of :text containing # does not appear in edit form

We're trying to build a markdown editing field and when we add a heading using # the text area appears empty after saving then going back to edit it.
We've noticed that there is no explicit documentation for text areas (ie input :field_name, type: :text). Is this supported?
Everything else seems to work just fine, we get a textarea and can pass through a rows attribute but editing is empty when there is a hash at the start of the first line.
Note that the hash does not cause this issue if there is anything other than whitespace before it.
Reproduction Steps

Create a field with type text like so:

input :field_name, type: :text

Add some content to the field with a hash on the first line like:

# Content that will disappear

Save the model
Edit the model and notice that the textarea is now empty

/cc @joshprice

We were able to workaround our issue by always adding a newline to the front of the trimmed text field before saving like so:
  def changeset(struct, params \\ %{}) do
    params =
      params
      |> workaround_broken_markdown_in_exadmin("article")

    struct
    |> cast(params, [:article])
    |> validate_required([:article])
  end

  # https://github.com/smpallen99/ex_admin/issues/274
  # A leading newline seems to workaround the issue
  defp workaround_broken_markdown_in_exadmin(params, field) do
    Map.update(params, field, "", &("\n" <> String.trim(&1)))
  end


[BEAM-14157] Don't request work on a closed windmill GetWorkStream

requestObserver.onNext() which is called by send should not be called after requestObserver.onCompleted() is called.
requestObserver.onCompleted() is called when the stream is closed.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @scwhittle

Run Java PreCommit

LGTM, thanks!


Add self pairs to NeighborList calculator

I've added the option to return the self terms for a general neighborlist, as can be helpful from time to time. Coding done on the advice of Guillaume, so please no pitchforks on my first foray into Rust.

:books: Documentation preview :books:: https://rascaline--175.org.readthedocs.build/en/175/

Looks good! Could you add a new test at the bottom of the file with self_pairs: true?

As soon as I figure out how, absolutely!

I would like the calculator not to crash if we only have the self-term (and no other neighbours). Is this possible?

On 11 Apr 2023, at 03:59, Guillaume Fraux @.***> wrote:
@Luthaf commented on this pull request.
Looks good overall, I have one final question
In rascaline/src/calculators/neighbor_list.rs https://github.com/Luthaf/rascaline/pull/175#discussion_r1162508188:

@@ -58,12 +60,12 @@ impl CalculatorBase for NeighborList {
}

 fn keys(&self, systems: &mut [Box<dyn System>]) -> Result<Labels, Error> {

   assert!(self.cutoff > 0.0 && self.cutoff.is_finite());

   assert!(self.cutoff >= 0.0 && self.cutoff.is_finite());

Why this change? If we really need a cutoff of 0, we should treat it separately, since most implementation assume a strictly positive cutoff.
—
Reply to this email directly, view it on GitHub https://github.com/Luthaf/rascaline/pull/175#pullrequestreview-1378763087, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALKVP3QYXEWVIAKRFLPJSO3XAUMQPANCNFSM6AAAAAAWVX46TU.
You are receiving this because you authored the thread.


fix: lots of stuff

Please describe the changes this PR makes and why it should be merged:
This PR makes several major fixes/changes:
Status

[x] Code changes have been tested against prettier, or there are no code changes
[x] I know how to update typings and have done so, or typings don't need updating

Semantic versioning classification:

[x] This PR changes the codebase

[x] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
[ ] This PR changes the internal workings with no modifications to the external API (bug fixes, performance improvements)

[ ] This PR only includes non-code changes, like changes to documentation, README, etc.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

14 of 217 (6.45%)  changed or added relevant lines in 7 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-1.7%) to 34.498%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/compiler/plugin.ts
0
2
0.0%

packages/compiler/react/jsx-element-visitor.ts
4
23
17.39%

packages/million/template.ts
1
25
4.0%

packages/compiler/react/transform.ts
2
160
1.25%

Files with Coverage Reduction
New Missed Lines
%

packages/compiler/react/jsx-element-visitor.ts
4
32.04%

Totals

Change from base Build<PHONE_NUMBER>:
-1.7%

Covered Lines:
1335

Relevant Lines:
3979

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

25 of 257 (9.73%)  changed or added relevant lines in 7 files are covered.
40 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-1.7%) to 34.498%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/compiler/plugin.ts
0
2
0.0%

packages/million/block.ts
11
15
73.33%

packages/compiler/react/jsx-element-visitor.ts
5
24
20.83%

packages/million/template.ts
1
26
3.85%

packages/compiler/react/transform.ts
2
184
1.09%

Files with Coverage Reduction
New Missed Lines
%

packages/compiler/react/transform.ts
2
2.53%

packages/compiler/react/jsx-element-visitor.ts
4
32.04%

packages/million/template.ts
4
49.65%

packages/compiler/react/call-expression-visitor.ts
11
53.95%

packages/million/block.ts
19
72.51%

Totals

Change from base Build<PHONE_NUMBER>:
-1.7%

Covered Lines:
1335

Relevant Lines:
3979

💛  - Coveralls


Generate a more secure first-time admin password

Instead of hard-coding a password like NotSecure123!!, generate a random password for the admin user when seeding the database.

I have two options to fix this issue.

4 based on https://xkcd.com/221/
Powershell ([char[]]([char]33..[char]95) + ([char[]]([char]97..[char]126)) + 0..9 | sort {Get-Random})[0..8] -join ''based on https://blogs.technet.microsoft.com/undocumentedfeatures/2016/09/20/powershell-random-password-generator/


Quagga.decoder is not a function

Im trying to implement the function on this page: https://serratus.github.io/quaggaJS/v1.0.0-beta.1/examples/scan-to-input/.
I used to code you showed on this page. But when I click the button the console shows: Quagga.decoder is not a function.
Any Idea what went wrong?

Hello! Im the same problem. How do you fix it?

Hello I have the same problem. Any ideas? Using chrome 57

if I use script from example https://serratus.github.io/quaggaJS/v1.0.0-beta.1/examples/js/quagga.js I get Quagga.decoder(...).locator(...).fromImage is not a function
When I use script from bower installation I get  Quagga.decoder is not a function

I finally got this working after viewing the source on the page and discovering that the source they are using to actually render the page is different than the source they display at the bottom of the page.
If you use this version of quaggajs:
https://serratus.github.io/quaggaJS/v1.0.0-beta.1/examples/js/quagga.js
And this version of the example code (notice 'fromImage' displayed on the pages "sample code" is replaced by 'fromSource'):
https://serratus.github.io/quaggaJS/v1.0.0-beta.1/examples/file-input/index.js
it works.  I haven't compared the versions to figure out what actual build this is that actually works but hopefully this helps if someone else searches for this issue.


Checking upgradation of Firestore emulator

Checking upgradation of Firestore emulator
Checking upgradation of Firestore emulator.txt

Please sign the CLA if you'd like us to consider this PR.


Can not prefetch actor

>> qs = Action.objects.prefetch_related('actor', 'target', 'action_object')
>> qs.first().actor
None
>> qs.first().target
<Chor: The Clothworkers Consort of Leeds>

works for me, actor might have been deleted


android after choosing account nothing happend

hello
it works goodly on local emulator
but when i generate signed apk and after installing on device. after choosing account , nothing will happen .
and after clicking button again . account chooser page show again .
is it related to type of firebase app creating or SHA1 key or something like this ?
i dont think i relate to codes , because its working on local in correctly.
or maybe for release or debuging key ?
thanks
_signIn = async () => {
    try {
        await GoogleSignin.hasPlayServices();
        const userInfo = await GoogleSignin.signIn();
        this._signed(userInfo);
        console.log(userInfo)
    } catch (error) {
        if (error.code === statusCodes.SIGN_IN_CANCELLED) {
            // user cancelled the login flow
        } else if (error.code === statusCodes.IN_PROGRESS) {
            // operation (f.e. sign in) is in progress already
        } else if (error.code === statusCodes.PLAY_SERVICES_NOT_AVAILABLE) {
            // play services not available or outdated
        } else {
            // some other error happened
        }
    }
};

    GoogleSignin.configure({
        scopes: ['https://www.googleapis.com/auth/drive.readonly'],
        webClientId: '*****1000813-re8abgd******pbcss7****923******apps.googleusercontent.com',
        loginHint: '',
    });

+1
Same for me. In my case, it works on debug and signed APK. but there's no response after choosing account on the release version on google play

Same problem. The modal after pressing on Sign in with Google would not show. IOs is working fine.

Make sure you add SHA certificate fingerprints in Firebase console for this app. All of them. If it works in debug, then you missed SHA fingerprints for release, and vice versa.

make sure sha1 matches your app in Google Developer Console.

any solution yet? my sh1 is matching my playstore app signin. i also added  in firebase SHA certificate fingerprints my clientId is also matching.

I stashed all my changes for android only and run react-native link again, then i run my android app, and it worked fine.

This issue happened to me today.
But after updating from ^1.0.2 to ^1.2.1 issue was fixed.
It doesn't looks like issue with fingerprint.

closing as stale. if you have this problem, see https://github.com/react-native-community/react-native-google-signin/issues/593#issuecomment-461691910
or open a new issue


Ng test misses files

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.
1.4.1
Repro steps.
Not changing any configs or other dependencies, migrated from 1.3.2 up to 1.4.1. As a result an entire directory of code along with app.component.spec.ts were no longer included when running ng test. Reverted back to 1.3.2 and all tests ran again. It would seem something changed in the .spec.ts file detection? The app is large/private so I cannot provide a link or plunkr.
The log given by the failure.
No failures, but only 97 tests were ran/detected when there should have been 483.
Desired functionality.
detection of .spec.ts files back
Mention any other details that might be useful.

My project structure remained the same as when you run ng new [name]
With 1.3.2 the cli finds all tests, when I update to 1.4.5 it finds none. My test.ts file has remained the same and i haven't changed any config files, but something with 1.4.x seems to be affect the regex to find .spec files? My current work around is to revert to 1.3.2

So after doing a side-by-side comparison of a fresh app generated by the 1.4.5 cli... the difference is in the tsconfig.json files.
I had to go revisit the contents of tsconfig.json and tsconfig.spec.json in order for all my .spec files to get picked up again when i ran ng test

Seems like it's sorted then 👍


Navigation between pages

User Story
AS A user who has reached the Collective Foundation website
I WANT TO be able navigate
SO THAT I can view all of the pages in the website
Acceptance Criteria
GIVEN I am on the main website/homepage page
WHEN I view the top of the page
THEN I would see the header (logo on the left, pages in the right, social icons on the right)
GIVEN I am on the main website/homepage page
WHEN I click a linked page
THEN it takes me to the page I have clicked

3

Social icons can be simple links to the the social website (like http://www.facebook.com)
DO NOT REIMPLEMENT THE API WHICH RETURNS ALL PAGES. USE THE SAME API AS THE ONE IMPLEMENTED IN THIS TICKET

Blocked by: https://github.com/users/khmdagal/projects/6/views/1?pane=issue&itemId=22061575

@KhesiweDube position issue is fixed, no navigation for now as team agreed not to move any DB here

Moving to closed because this ticket is now combined and covered in: #38


Add delay to prevent submit too fast

If test/submit too fast, the program will hang. Add some delay would help.

Good catch, dude!

Should be fixed, please LMK if any problem.

It works!


The names and specificity of the profiles

Comment by @costateixeira
this name seems suspect ”BeReferralPrescriptionNursingPreparationAdministrationMedicationChronicalPsychiatricPatient”
o	Besides, this profile isn’t really doing anything yet, except making a fixed value where we should have a valueset(?)

@costateixeira The way these very detailed profiles are set up, is by limiting the values  of the .orderDetail to a valueset, and in some cases by adding extra MS fields or making some fields mandatory.
If we can achieve the same result in another way, we can limit the number of profiles...

"Preparation Administration Medication Chronical Psychiatric Patient" is a valueset of the referral prescription
please close this issue

@Mcobbaert Could you please refer exactly to the name of the valueSet and the place in the document (I suppose you mean the document in #61 )?

WG: we close this issue, as it has been dealt with since.


Held keys are not "released" when window loses focus

If I hold a key like say W, and then Alt+Tab to a different window, the W key is still held down while the game is out of focus. If I come back, my character continues to move forward even if I'm no longer holding W. I have to tap the W key to get back to a normal state.

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Yeah it's a known issue, we will fix it eventually. Right now we are just figuring out what the game is about. :)
Duplicate of #28


ROX-25556: Definitions for api service for collector runtime config

Description

Protobuf definitions for the API service for collector runtime configuration. The service is intended to provide the following functionality, though nothing is implemented here.

Get the current configuration
Replace the current configuration with a new configuration
Change a portion of the configuration
Delete one rule from the configuration
A dry run option which reports the statuses for clusters and namespaces for all features

User-facing documentation

[x] CHANGELOG is updated OR update is not needed
[x] documentation PR is created and is linked above OR is not needed

Testing and quality

[ ] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing

- [ ] added unit tests
- [ ] added e2e tests
- [ ] added regression tests
- [ ] added compatibility tests
- [ ] modified existing tests
How I validated my change

Deployed ACS locally. Checked the API reference in the UI.
The following endpoints had the following sample request
POST /v1/collector_runtime_config
PATCH /v1/collector_runtime_config
POST /v1/collector_runtime_config/dryrun
or sample response
GET /v1/collector_runtime_config
{
  "collectorRuntimeConfig": {
    "runtimeFeatureConfig": [
      {
        "feature": "PROCESSES",
        "defaultStatus": {
          "processStatus": {
            "enabled": true
          },
          "networkConnectionStatus": {
            "enabled": true,
            "aggregateExternal": true
          },
          "networkEndpointStatus": {
            "enabled": true,
            "processesListeningOnPort": true
          }
        },
        "rules": [
          {
            "id": "string",
            "configResourceCollection": {
              "resourceCollectionName": "string",
              "resourceCollection": {
                "id": "string",
                "name": "string",
                "description": "string",
                "createdAt": "2019-08-24T14:15:22Z",
                "lastUpdated": "2019-08-24T14:15:22Z",
                "createdBy": {
                  "id": "string",
                  "name": "string"
                },
                "updatedBy": {
                  "id": "string",
                  "name": "string"
                },
                "resourceSelectors": [
                  {
                    "rules": []
                  }
                ],
                "embeddedCollections": [
                  {}
                ]
              }
            },
            "status": {
              "processStatus": {
                "enabled": true
              },
              "networkConnectionStatus": {
                "enabled": true,
                "aggregateExternal": true
              },
              "networkEndpointStatus": {
                "enabled": true,
                "processesListeningOnPort": true
              }
            }
          }
        ]
      }
    ],
    "resourceCollections": [
      {
        "id": "string",
        "name": "string",
        "description": "string",
        "createdAt": "2019-08-24T14:15:22Z",
        "lastUpdated": "2019-08-24T14:15:22Z",
        "createdBy": {
          "id": "string",
          "name": "string"
        },
        "updatedBy": {
          "id": "string",
          "name": "string"
        },
        "resourceSelectors": [
          {
            "rules": [
              {
                "fieldName": "string",
                "operator": "OR",
                "values": [
                  {}
                ]
              }
            ]
          }
        ],
        "embeddedCollections": [
          {
            "id": "string"
          }
        ]
      }
    ]
  }
}

The following is a sample response for
POST /v1/collector_runtime_config/dryrun
{
  "statusesForFeatures": [
    {
      "runtimeFeature": "PROCESSES",
      "statusWithResources": [
        {
          "status": {
            "processStatus": {
              "enabled": true
            },
            "networkConnectionStatus": {
              "enabled": true,
              "aggregateExternal": true
            },
            "networkEndpointStatus": {
              "enabled": true,
              "processesListeningOnPort": true
            }
          },
          "resources": [
            {
              "cluster": "string",
              "namespace": "string"
            }
          ]
        }
      ]
    }
  ]
}

There is also the following endpoint for deleting rules.
DELETE /v1/collector_runtime_config/id/{id}

/test ocp-4-12-scanner-v4-tests
/test ocp-4-12-qa-e2e-tests

/test ocp-4-12-scanner-v4-tests


Unable to stream recorded videos (.ts)

Hello again. Sorry to be a pain...
My understanding is, that it should be possible to access the past recordings via rtmp:///file/mp4:
When I try to do that, I get the following error (redirected node stdout to a file):
2017-11-24 18:06:19.511 [rtmp:client=5e1l3xJE] requested stream file/mp4:2017-11-24_14-41-26.ts
2017-11-24 18:06:19.512 [rtmp:client=5e1l3xJE] error: stream not found: file/mp4:2017-11-24_14-41-26.ts
The same happens if I change node-rtsp-rtmp-server/config.js like this:
/* RTSP/RTMP configurations /
liveApplicationName: 'live',
recordedApplicationName: 'file',
/ recordedDir: 'file', */
recordedDir: '/home/pi/picam-streamer/rec/archive',
Where should the standard "file" folder be? Inside /home/pi/node-rtsp-rtmp-server/file ?
I tried with an mp4 file, too, Same error, stream not found.
If the streaming of recorded files is not supported, would you have any suggestion how to achieve that?
Thanks,
Christian

My understanding is, that it should be possible to access the past recordings via rtmp:///file/mp4:

Sorry, serving .ts files is not supported yet. Only .mp4 files are supported.

Where should the standard "file" folder be? Inside /home/pi/node-rtsp-rtmp-server/file ?

Yes. You should set recordedDir to the file directory in this case.

If the streaming of recorded files is not supported, would you have any suggestion how to achieve that?

You can convert .ts file to .mp4 file after a recording has completed.


Design time bug

Identified bug whereby controls do not appear on the form, instead they appear on the area where the timer, background worker, dialog controls appear. Did try gacutil with no success. See screenshot attached for visual detail.

Did have a deep look at the source code for the toolkit, but couldn't find anything glaring at me.

I had the same problem, check if you have more than one version of the same
assembly registered in the GAC; my suggestion is to deregister everything
krypton related then register the appropriate dlls
Il giorno mar 17 gen 2017 alle 18:41 Wagnerp<EMAIL_ADDRESS>ha
scritto:

Identified bug whereby controls do not appear on the form, instead they
appears on the area where the timer, background worker, dialog controls
appear. Did try gacutil with no success. See screenshot attached for visual
detail.
[image: krypton bug]
https://cloud.githubusercontent.com/assets/949607/22032384/275a67f2-dcdc-11e6-8624-35d85dea7907.PNG
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ComponentFactory/Krypton/issues/18, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AA5eLOHQ9FDHpQvl3Q4C9iQgn1w3CKr5ks5rTP1WgaJpZM4Ll88l
.

Hi
For design time support you need to register the *.design.dll (and reboot computer believe me !). Sometimes you need to make another project to see changes.

Hi @mbagattini and @Cocotteseb
Many thanks for your input, I did try registering the assemblies using GAC and rebooting my computer, but this didn't solve the issue for me. Is there a way I can clear the cache to try again?

What do you see when, from elevated developer command prompt, you run
gacutil -l componentfactory.krypton.design?
On Wed, Jan 18, 2017 at 9:07 AM, Wagnerp<EMAIL_ADDRESS>wrote:

Hi @mbagattini https://github.com/mbagattini and @Cocotteseb
https://github.com/Cocotteseb
Many thanks for your input, I did try registering the assemblies using GAC
and rebooting my computer, but this didn't solve the issue for me. Is there
a way I can clear the cache to try again?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ComponentFactory/Krypton/issues/18#issuecomment-273409472,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA5eLLQ-zg2QX3OkO1Wf2O2BJZy4xMXsks5rTcg6gaJpZM4Ll88l
.

Hi @mbagattini
It did work, but it needed a full shutdown not a reboot. I left it overnight, powered it back up this morning and it now works. Not quite sure of the logic of that!

Glad to hear that. Maybe it needed some rest :)


python3Packages.pyvesync: 1.4.1 -> 1.4.2

Motivation for this change
Update to latest upstream release 1.4.2
Change log: https://github.com/webdjoe/pyvesync/releases/tag/1.4.2
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 146851 run on x86_64-linux 1

3 packages built:

home-assistant
python38Packages.pyvesync
python39Packages.pyvesync


Implement the "Lock screen" user-journey

Description
Implement the locking screen according to the user-journey.
Acceptance Criteria

[ ] Implement the locking screen according to the design
[x] When the locker changes its state to locked or unlocked, react accordingly
[x] Any access to the IDM wallet while locked should be denied and the lock screen should be displayed
[ ] Any rendered content when locked should not contain any information that is stored encrypted

R&D

[ ] Think of a way to restore the previous screen exactly with the state we had, e.g.: when editing the profile

Background
Left and Right padding on the input or container
Temporary logo morph

@dominguesgm can you link the PR here?

Sorry it took a while, my bad.
https://github.com/ipfs-shipyard/nomios-web/pull/3


clarifying organic.layer vs litter

I recently clarified the definition of organic.layer and added the variable litter to describe a subset of organic.layer (see variables.csv). The definition for organic.layer aligns with that used by the IPCC (https://github.com/forc-db/IPCC-EFDB-integration/issues/19). This does not fundamentally change the definition, but provides more clarity for an often-nebulous variable.
There may be some organic.layer records that would be better defined as litter, so these warrant review.

To provide better definition of the previously existing variable organic.layer, which has a nebulous definition that reflects the varied definitions adopted by original studies, we added two clearly defined variables: litter (relatively undecomposed plant material/ OL horizon), and O.horizon (entire O-horizon, including litter (OL)).
When possible, we will want to re-classify organic.layer as O.horizon or litter.


reuse existing materials

test how useful it is

there's a new material for every connection

if we just have a mat for the connections and leave the rest as is, it's good enough

also they're id's, stop calling them handles.. and use .id method instead of .into
-- amy from the future

oh here you just have to have a ConnectionsMaterial in resource and use that for all the arrows, circles and holes should stay the same way they are, they need independent mats (the checking would be ridiculous)


1.0.0 installs broken molecule

Version 1.0.0 somehow installs a version of molecule that is broken and won't run at all. It simply exits silently with return code of "1".

Git bisect tells me that this error was introduced with commit 533f1cb78ec7fe579404ac4bc19efbbe039db9c1. Beginning investigation into that, now.

Nope. That's just the first commit, with the change to subprocess.run, that allows my test code to identify that there IS an error. There's something else going on around here.

Hi @greg-hellings,
I hit a similar issue where molecule exits with rc=1. I nailed it to rich 9.5.0 and commit https://github.com/willmcgugan/rich/commit/12b268d223d2f4eed77f5d88ef875be4dfdc2170.
enrich 1.2.3 is not compatible with this change.

https://github.com/ansible-community/molecule/issues/3010

With your help I've added #55 to hopefully address this issue. Once enrich is updated, we can pin a minimum version onto that.


Add an option to enable CORS in TensorBoard

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below):  b'unknown' 1.1.0
Bazel version (if compiling from source): N/A
CUDA/cuDNN version: CUDA 8, cuDNN 5.1
GPU model and memory: 1080ti 11GB
Exact command to reproduce: N/A

Describe the problem
The DeepDream notebook example in TensorFlow provides a snippet that uses https://tensorboard.appspot.com to embed s inside a Jupyter notebook. Changing this snippet to point to a local instance of TensorBoard fails with a CORS error

Having the option to enable CORS when starting TensorBoard, or enabling it by default, would make it really easy to use the snippet to visualize directly in Jupyter using a local instance of TensorBoard.
Source code / logs
Here's the snippet in its whole, with modified URL to point to http://localhost:6006 instead of https://tensorboard.appspot.com
# TensorFlow Graph visualizer code
import numpy as np
from IPython.display import clear_output, Image, display, HTML

def strip_consts(graph_def, max_const_size=32):
    """Strip large constant values from graph_def."""
    strip_def = tf.GraphDef()
    for n0 in graph_def.node:
        n = strip_def.node.add()
        n.MergeFrom(n0)
        if n.op == 'Const':
            tensor = n.attr['value'].tensor
            size = len(tensor.tensor_content)
            if size > max_const_size:
                tensor.tensor_content = "<stripped %d bytes>"%size
    return strip_def

def show_graph(graph_def, max_const_size=32):
    """Visualize TensorFlow graph."""
    if hasattr(graph_def, 'as_graph_def'):
        graph_def = graph_def.as_graph_def()
    strip_def = strip_consts(graph_def, max_const_size=max_const_size)
    code = """

        <link rel="import" href="http://localhost:6006/tf-graph-basic.build.html" onload=load()>

          <tf-graph-basic id="{id}"></tf-graph-basic>

    """.format(data=repr(str(strip_def)), id='graph'+str(np.random.rand()))

    iframe = """

    """.format(code.replace('"', '"'))
    display(HTML(iframe))

I would recommend working around this with nginx.
server {
  listen 80 default_server;
  listen [::]:80 default_server ipv6only=on;

  location / {
    proxy_http_version 1.1;
    proxy_pass http://<IP_ADDRESS>:6006;
    add_header 'Access-Control-Allow-Origin' '$cors_origin';
    add_header 'Access-Control-Allow-Methods' 'GET, POST, OPTIONS';
    add_header 'Access-Control-Allow-Headers' 'DNT,X-CustomHeader,Keep-Alive,User-Agent,X-Requested-With,If-Modified-Since,Cache-Control,Content-Type,Custom-Header-1';
  }
}

@dandelionmane Should we add an --insecure_cors flag? Or a --header=Foo:bar flag where the user repeats all that, and it gets applied to all responses?

Since the NGINX workaround seems sufficient, I'm going to close this one out.
Please note that we're planning to productionize TensorBoard. Things like CORS will be part of our configuration scripts. Please follow https://github.com/tensorflow/tensorboard/issues/92 for more information.
With that said, I'm going to close this one out. If anyone feel strongly that header customization or CORS should be part of TensorBoard core, let me know.

@Inouyasha
I was struggled to the same problem and this worked!! Thank you so much.


Debuging FERC1 rate base table

Overview
Closes #3403 and working on #3309.
What problem does this address?
What did you change?
Testing
How did you make sure this worked? How can a reviewer verify this?
# To-do list
- [ ] Ensure docs build, unit & integration tests, and test coverage pass locally with `make pytest-coverage` (otherwise the merge queue may reject your PR)
- [ ] For significant ETL changes, ensure the full ETL runs locally
- [ ] For major data coverage & analysis changes, [run data validation tests](https://catalystcoop-pudl.readthedocs.io/en/latest/dev/testing.html#data-validation)
- [ ] If updating analyses or data processing functions: make sure to update or write data validation tests
- [ ] Update the [release notes](../docs/release_notes.rst): reference the PR and related issues.
- [ ] Review the PR yourself and call out any questions or issues you have

lol wrong branch!....


Make the font prettier

All lowercase might look better.
We load the font in as a sprite sheet.  Each character is 8 pixels by 8 pixels.
You can edit this in any tool that you want, but the characters must fit in their 8x8 quads.

I would like to try this.

That would be great!
also, numbers would be good if you're feeling ambitious

@fernandadagostin  on second thought, All upper case may be better because you won't have to deal with things like the bottom of the letter y dropping below the bottom of the line

I was thinking to do Uppercase, Lowercase, and numbers. So you can have options. Also, I am putting the letters inside of 8x8 in the correct position to be shown, so I don't how that would work for you, but my thought is if you can get an 8x8 pice from this big image it should be good to show.
Note: I just finished uppercase, I will work on the lowercase and number now.

Ok, Sweet
thanks!

#55


Error: Couldn't get any items

Any clue on what this error message means?
Error: Couldn't get any items
    at module.exports.ret (/usr/local/lib/node_modules/pageres-cli/node_modules/viewport-list/index.js:25:9)
    at memoized (/usr/local/lib/node_modules/pageres-cli/node_modules/mem/index.js:32:18)
    at /usr/local/lib/node_modules/pageres-cli/node_modules/pageres/dist/index.js:203:29
    at next (native)
    at step (/usr/local/lib/node_modules/pageres-cli/node_modules/babel-runtime/helpers/asyncToGenerator.js:17:30)
    at /usr/local/lib/node_modules/pageres-cli/node_modules/babel-runtime/helpers/asyncToGenerator.js:35:14
    at new Promise (/usr/local/lib/node_modules/pageres-cli/node_modules/core-js/library/modules/es6.promise.js:191:7)
    at /usr/local/lib/node_modules/pageres-cli/node_modules/babel-runtime/helpers/asyncToGenerator.js:14:12
    at Pageres.viewport (/usr/local/lib/node_modules/pageres-cli/node_modules/pageres/dist/index.js:211:5)
    at /usr/local/lib/node_modules/pageres-cli/node_modules/pageres/dist/index.js:165:19

What was the command you ran?

It was just (not correct domain, as it happened to a few of them)
pageres google.com

You need to provide a size. See https://github.com/sindresorhus/pageres-cli#usage. Although it's weird that this error shows up since it shouldn't try fetching viewports without keywords.

Strange thing is that it works for every other domain, except for a few, should I add a size to every domain and try again?

Can you give me an example of a domain that doesn't work?

I have a list of a lot domains, I'm not sure which domains throw the error. I'll run it with a size and get back to you.

Turns out, you don't https://github.com/sindresorhus/pageres-cli/blob/9c38bd79272e14653ab9b17230c98d02db850814/cli.js#L99-L101.

Hmm, back to square one.


Add compatibleVersion and version badges

Further reading: https://docs.sketchpacks.com/developers/publishing/badges.html

Sorry for the late merge, I was trying out another badge that I had created but it didn't quite work out. Cheers!


Disable Test_KubeSecretController in racetest

See https://github.com/istio/istio/issues/15610
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS [howardjohn]
pilot/OWNERS [howardjohn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio_auth_sds_e2e-master
08faf6c7e598fa292ef47d4f586785e025391729
link
/test istio_auth_sds_e2e-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio_auth_sds_e2e-master

/retest

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, kyessenov
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS [howardjohn,kyessenov]
pilot/OWNERS [howardjohn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
08faf6c7e598fa292ef47d4f586785e025391729
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm

@howardjohn: you cannot LGTM your own PR.

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Oops.. I meant to type retest
/retest

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
08faf6c7e598fa292ef47d4f586785e025391729
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
08faf6c7e598fa292ef47d4f586785e025391729
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
08faf6c7e598fa292ef47d4f586785e025391729
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Remove broken links in BasicTypes.md

Removes broken links in BasicTypes.md.

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

Poke re: the CLA

Rebased

Thanks!


Json dump use to compilation errors

Code
#include <iostream>
#include <nlohmann/json.hpp>

using json = nlohmann::json;

int main(){
   json data;

   data["name"] = "halloweeks";
   data["email"] =<EMAIL_ADDRESS>
   std::cout << data.dump(4) << std::endl;
   return 0;
}

Compilation Error
$ g++ a.cpp -o $HOME/main                   /data/data/com.termux/files/usr/bin/arm-linux-androideabi-ld: /data/data/com.termux/files/usr/tmp/a-5e656c.o: in function long std::__ndk1::__libcpp_atomic_refcount_decrement<long>(long&)':                              a.cpp:(.text._ZNSt6__ndk134__libcpp_atomic_refcount_decrementIlEET_RS1_[_ZNSt6__ndk134__libcpp_atomic_refcount_decrementIlEET_RS1_]+0x28): undefined reference to __atomic_fetch_add_4'                                    /data/data/com.termux/files/usr/bin/arm-linux-androideabi-ld: /data/data/com.termux/files/usr/tmp/a-5e656c.o: in function long std::__ndk1::__libcpp_atomic_refcount_increment<long>(long&)': a.cpp:(.text._ZNSt6__ndk134__libcpp_atomic_refcount_incrementIlEET_RS1_[_ZNSt6__ndk134__libcpp_atomic_refcount_incrementIlEET_RS1_]+0x28): undefined reference to __atomic_fetch_add_4'
clang-9: error: linker command failed with exit code 1 (use -v to see invocation)
$

In order to asses your issue, we need the following information:

What is the issue you have?

Please describe the steps to reproduce the issue. Can you provide a small but working code example?

What is the expected behavior?

And what is the actual behavior instead?

Which compiler and operating system are you using? Is it a supported compiler?

Did you use a released version of the library or the version from the develop branch?

If you experience a compilation error: can you compile and run the unit tests?

Closed due to inactivity. Will reopen if needed.


Conditionally render model registry modals

Towards RHOAIENG-12117
Description
Update usage of modals in the model registry section to render them only when they would be shown rather than having them be rendered but hidden.
How Has This Been Tested?
Run thru the UI and check that then modals are shown only when they should be shown.

From the Model Registry page

from a model, use the kebab and select Archive model
verify the Archive model? modal is shown

From the Model Registry page

Click on a model
from the model details page, open a kebab menu for a version and select Deploy
verify the Deploy model modal is shown

From the Model Registry page

Click on a model
from the model details page, open a kebab menu for a version and select Archive model version
verify the Archive version? modal is shown

From the Model Registry page

Click on a model
from the model details page, click on a model version
from the model version page, select the Action menu then Deploy
verify the Deploy model modal is shown

From the Model Registry page

Click on a model
from the model details page, click on a model version
from the model version page, select the Action menu then Archive model
verify the Archive version? modal is shown
feel free to note the inconsistencies

From the Model Registry page

Click the kebab next to the Register model button, select View archived models
from the archived models page, click a kebab on one of the archived models and select Restore version
verify the Restore version modal is shown

From the Model Registry page

Click the kebab next to the Register model button, select View archived models
from the archived models page, click a version
on one of the archived model detail page and select Restore model
verify the Restore model modal is shown

From the Model Serving page (titled Deployed models)

Click the Deploy model button
verify the Deploy model modal is shown

From the Model Serving page (titled Deployed models)

Open a kebab on a deployed model and select Edit
verify the Edit model modal is shown

Test Impact
Current tests already test these modals.
Request review criteria:
Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

/lgtm
/approve


[App]: Logging to the same table doesn't update the data / inconsistent & weird behavior

Current Behavior
I would expect that
table = wandb.Table(columns=["x", "x ** 2", "x ** 3"])

for i in it.count():
    table.add_data(i, i ** 2, i ** 3)
    wandb.log({"table": table})
    print(i)
    time.sleep(2)

Makes it so the data in the table is continuously updated on the website. It's not.
It displays the first row only.
Yet, if I do
run = wandb.Api().runs("julesgm/raw_test")[0] # The correct run
print(run.summary)
rich.print(run.summary["table"])

it shows that there should be 27 rows. Yet, also,
artifact = wandb.Api().artifact(f"raw_test/run-{run.id}-table:latest")
artifact.download()
print(artifact.get("table").data)

only shows a single row of data.
What gives?
Expected Behavior
I would expect that all the rows that I continuously update to the same table get displayed in the app (and are present in the artifact)
Steps To Reproduce
No response
Screenshots

Environment
OS:
Client is on
NAME="Ubuntu"
VERSION="20.04.5 LTS (Focal Fossa)"
Browsers:
Chrome
Version:
wandb 0.15.10, using pip
python 3.9.15
Additional Context
No response

So it seems like setting
table._artifact_target = None

on the modified table, and logging it again, works.

WandB Internal User commented:
JulesGM commented:
So it seems like setting
table._artifact_target = None

on the modified table, and logging it again, works.

No that was wrong, it requires a new table

mistake on my part. Being able to upgrade the same table would be much better.

Hey @JulesGM  - You’re right, this is because we only support logging a table once per run to the same key (as doing multiple calls to wandb.log will overwrite previous steps. I saw you made a feature request here, and will respond on that shortly as well.
The above code chunk you sent should be modified to:
table = wandb.Table(columns=["x", "x ** 2", "x ** 3"])

for i in it.count():
    table.add_data(i, i ** 2, i ** 3)
    print(i)
    time.sleep(2)

wandb.log({"table": table})

in order to log the full table - since we don't support logging a table to the same key multiple times in a run. Please let me know if you have any follow up questions about this.

Additionally, if you'd like to see multiple versions of a table logged during a single run, you can include the following weave expression in your runs workspace: runs.history.concat["my_table"] - please let me know if you have any questions about this.

Hello, I really need a continuously updating table.

With LLMs being this big, everyone will need a structured way to visualize text data.

That is continually being updated, with eg new text generated by the model.

@JulesGM - Will be sure to update you if progress is made on the feature request!


nanoplots in validation tables?

gt version 0.10.0 adds a new feature called "nanoplots" https://gt.rstudio.com/reference/cols_nanoplot.html
Since the validation tables are built with gt in the format_report_gt() function, there's a possibility of adding these to some of the validations.  Just an idea and something to potentially play with at a future date.

This is a nice find! Reminds me of the Tufte sparkline concept.


refactor: extract data and behaviour of layout parameters

Description
Extract data and behaviour to a dictionary with layout parameters.
Motivation and Context
Prepare further TaskLayout.* refactorings.
How has this been tested?
Unit tests + new unit tests added.
Types of changes
Internal changes:

[x] Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)

Checklist

[x] My code follows the code style of this project and passes yarn run lint.
[x] My change has adequate Unit Test coverage.

Terms

[x] My contribution follow this project's contributing guide
[x] I agree to follow this project's Code of Conduct

@claremacrae this is what I have been able to do so far, however I really struggle to find a meaningful name for AppleSauce & appleSauceDictionary... This is not a joke... =)

@claremacrae this is what I have been able to do so far, however I really struggle to find a meaningful name for AppleSauce & appleSauceDictionary... This is not a joke... =)

Ah - I made some comments but didn't see this...

@claremacrae I've merged the latest main (3039b26c5330c14fea5b6b99ca5ce243e545aab6) to my branch solving the conflicts and ensured that there is no change in the tests results. Ready for review =)

I'm sorry I noticed one small change to do and pushed one tiny commit. No more edit from my side.

Sorry, one more thing... If you haven't done so already, please run the smoke tests, then I will merge....

Sorry, one more thing... If you haven't done so already, please run the smoke tests, then I will merge....

Just ran the smoke tests on the latest version. All good.


Feature request: Except layer collector

I found out I often repeat some regexp in multiple just to exclude classes already in another layer (like everything from namespace X is general dependency except X\Gui which is gui dependency). It would be nice to be able to reference another layer as layer general dependency is everything matching exceptnof it is in X\Gui. But I can see there could be some problems here like circular dependencies between layers. Or maybe just have some way to solve class in multiple layers? Like assign layer a priority.

Sorry there was some glitch and I posted it twice


After apply the multiple solution I am unable to configure https://localhost

I have did multiple process to configure my flutter web from http://localhost to https://localhost.
Like I have created certificate as below process
openssl genrsa -out localhost.key 2048 to create private  key file
Generate certificate signing request (CSR)
openssl req -new -key localhost.key -out localhost.csr -subj "/CN=localhost"
Generate self-signed certificate using CSR and key
openssl x509 -req -days 365 -in localhost.csr -signkey localhost.key -out localhost.crt
I have added these .key and .csr file with creating a folder called cert and added the above created certificate.
then I have try with flutter run --web-launch-url=https://localhost:8444  no luck getting error as "https://localhost:8444" is not a valid HTTP URL.
I have created "serve.json" file as set-up like
{
"request_handlers": [
{
"url_pattern": "/",
"delegate": {
"require_trusted_origin": false,
"delegate_name": "flutter-tools/web-server",
"secure": true,
"web-port": 8444,
"key_path": "/Users/jaykumar/Desktop/code/flutter_toolKit/enx_flutter_plugin_new/example/cert/localhost.key",
"certificate_path": "/Users/jaykumar/Desktop/code/flutter_toolKit/enx_flutter_plugin_new/example/cert/localhost.crt"
}
}
]
} and run the command  - as flutter run -d web-server --web-port 8444, it is still open the http://locallost:8444
Then I have tried wile creating "webdev.yaml" file with configuration
serve:
hostname: localhost
port: 8444
use_https: true
certificate_path: cert/localhost.crt
key_path: cert/localhost.key
https_port: 8444
and active the webdev with command  flutter pub global activate webdev
and run the command to open the chrome with https "webdev serve"
still it id going on
[INFO] Checking for updates since last build completed, took 681ms
[INFO] Serving web on http://<IP_ADDRESS>:8080
[INFO] Running build completed, took 84ms
[INFO] Caching finalized dependency graph completed, took 149ms
[INFO] Succeeded after 237ms with 0 outputs (0 actions)
[INFO] ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------
And no luck till day end. I need to open my localhost with https as secure service.
Any mistake I did or any lead will much much appreciate please.

Dear Friends,
I need support please, As I have been struggling for a couple of days.
Please suggest the best way to configure the same.
I need to setup the secure service, please.

@jaykumarvcx
Your issue / question is better suited for support channels like StackOverflow, so please ask the same there.
Closing from here.


Problem when running playbook - Name or service not known

ERROR! Attempted to execute "/usr/lib/python2.7/site-packages/awx/plugins/inventory/awxrest.py" as inventory script: Inventory script (/usr/lib/python2.7/site-packages/awx/plugins/inventory/awxrest.py) had an execution error: HTTPConnectionPool(host='awxweb', port=8052): Max retries exceeded with url: /api/v1/inventories/2/script/?hostvars=1 (Caused by NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection object at 0x2220e10>: Failed to establish a new connection: [Errno -2] Name or service not known',))

This error comes when i try and run a template/playbook.
If i create a alias, for the awx_web link, on the awx_task composer, i get this error instead:

fatal: [XXX.dk]: UNREACHABLE! => {"changed": false, "msg": "Failed to connect to the host via ssh: No user exists for uid 1000\r\n", "unreachable": true}

So i added the "user: root" to the awx_task composer, and it ran with success!
My awx_task composer thingy now looks like this (marked with bold what i added/changed):

awx_task:
image: "geerlingguy/awx_task:latest"
links:
- rabbitmq
- memcached
- awx_web:awxweb
- postgres
hostname: awx
user: root
container_name: awx_task
environment:
SECRET_KEY: aabbcc
DATABASE_USER: awx
DATABASE_PASSWORD: awxpass
DATABASE_NAME: awx
DATABASE_PORT: 5432
DATABASE_HOST: postgres
RABBITMQ_USER: guest
RABBITMQ_PASSWORD: guest
RABBITMQ_HOST: rabbitmq
RABBITMQ_PORT: 5672
RABBITMQ_VHOST: awx
MEMCACHED_HOST: memcached
MEMCACHED_PORT: 11211

I could create a pullrequest, but i want to know if anyone else is seeing this error, and if this fix helped.

Confirming that for any Playbook I'm seeing the same behaviour:
ERROR! Attempted to execute "/usr/lib/python2.7/site-packages/awx/plugins/inventory/awxrest.py" as inventory script: Inventory script (/usr/lib/python2.7/site-packages/awx/plugins/inventory/awxrest.py) had an execution error: HTTPConnectionPool(host='awxweb', port=8052): Max retries exceeded with url: /api/v1/inventories/2/script/?hostvars=1 (Caused by NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection object at 0x1c69e10>: Failed to establish a new connection: [Errno -2] Name or service not known',))
3
None

Looks like user: root was already added: https://github.com/geerlingguy/awx-container/blob/master/docker-compose.yml#L51 but, I'll give the - awx_web:awxweb a shot

Yep! That fixes it!

@jaredledvina / @MartinTerp - Do you think I should change the hostname internally for the awx container to awx_web too then? If so, I can generate a new image and you can test. I vaguely remember running into this same issue with one of my own playbooks in a test AWX environment on a plane earlier today.

And yes, I added the user: root in https://github.com/geerlingguy/awx-container/issues/7

I think so yes, just to align all the names.
i didt add "user: root" to awx_task in the first place, since i didt see any issues at that time.

@geerlingguy - That makes sense to me, happy to give the image a shot on my personal setup as well.

Yep, I'll get this fixed in a few minutes I hope (but it will take a bit to rebuild the image and push it...). I can confirm the current images are giving:
ERROR! Attempted to execute "/usr/lib/python2.7/site-packages/awx/plugins/inventory/awxrest.py" as inventory script: Inventory script (/usr/lib/python2.7/site-packages/awx/plugins/inventory/awxrest.py) had an execution error: HTTPConnectionPool(host='awxweb', port=8052): Max retries exceeded with url: /api/v1/inventories/2/script/?hostvars=1 (Caused by NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection object at 0x2a4fd90>: Failed to establish a new connection: [Errno -2] Name or service not known',))

Actually, it looks like the settings.py file that ships with AWX has INTERNAL_API_URL = 'http://awxweb:8052', so it would be annoying to try to switch everything to awx_web. Therefore, it may be as easy as just setting the links correctly inside the docker-compose.yml file. Testing that now.

Fixed via above commit.


Patch 4

Fixed bugs and polish.

As a simple example, with your original code (same as the book), you called folders.clear() in Message::remove_from_Folders(). Test it with the following main():
int main()
{
Message m1("Message1");
Folder f1;
m1.save(f1);
m1.print_debug(); // Output: Message1
f1.print_debug(); // Output: Message1

m1 = m1;
m1.print_debug(); // Output: Message1
f1.print_debug(); // Output nothing!

return 0;

}
You'll see what I mean.

@clarelsy  I see. Thx a lot.

Could you please merge #451 too if you think it is okay?
#451 is the header file for #452... I should have put #451 and #452 in the same patch but not sure how to do that.

@clarelsy np.


timing issue with summernote 0.7.3

I don't know why this doesn't affect 0.7.2, but it doesn't.
A-S code inside this.active:
     var updateNgModel = function() {
        var newValue = element.summernote('code');
        if (element.summernote('isEmpty')) { newValue = ''; }
        if (ngModel && ngModel.$viewValue !== newValue) {
          $timeout(function() {   //<<<<<<<<<Timeout 2
            ngModel.$setViewValue(newValue);
          }, 0);
        }
      };
     callbacks.onChange = function(contents) {
        $timeout(function() {   //<<<<<<<Timeout1
          if (element.summernote('isEmpty')) { contents = ''; }
          updateNgModel();
        }, 0);
        $scope.change({contents:contents, editable: $scope.editable});
      };

In this order (this is on init):

Model set to Value1
onChange is called with Value1, Timeout1 calls updateNgModel, which makes Timeout2.
Model is set to Value2
onChange is called again with Value2, which Triggers Timeout1, which calls updateNgModel, but does not create another Timeout 2.
Timeout2 is now run, overwriting the up-to-date Value2, with the old Value1, on the Model.

Expected: Value2
Got: Value1
Fix: Would be for Timeout2 to check to see if 'updateNgModel' has been called again since it was created, and if so, do nothing.

Could you check it in v0.8?


#2034 ScreenShoot folder for TestNG

Proposed changes
#2034 Define ScreenShoot folder for TestNG
Checklist

[x] Checkstyle and unit tests are passed locally with my changes by running gradlew check chrome_headless firefox_headless command
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

@vlad-velichko May I ask why do you want to set the folder? What  are you going to do with that folder?
I am asking because probably we can suggest a better/simpler solution.

@asolntsev By default selenide saves fails screenshots into one folder.
But ScreenShoot creates a separate folder per class.
I need to have all screenshots in one folder.
To save all to the default folder I use path "."

Yes, but what are you going to do next? These files have random names. What will you do with the bunch of random files in that folder?

getLastScreenshot() -> attach to my 3rd-party custom report

getLastScreenshot() returns File instance. You can do whatever you what with it. Why do you need a custom folder? Am I missing smth?


Provide public REST API to create new cluster

This could be very helpful when provisioning new cluster. I have been looking at code and doc but do not find any generic api that we can call from mesos.

Maybe take a look at Confluents Rest API, http://docs.confluent.io/2.0.1/kafka-rest/docs/index.html

They have good set of APIs for kafka like topic, partition, broker,
consumers etc. but I don't think there's anything about kafka-manager
cluster.
On Mon, Apr 18, 2016 at 7:06 AM, elvarb<EMAIL_ADDRESS>wrote:

Maybe take a look at Confluents Rest API,
http://docs.confluent.io/2.0.1/kafka-rest/docs/index.html
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/yahoo/kafka-manager/issues/226#issuecomment-211330696

Should be trivial to add. Will take it up.

Should be trivial to add. Will take it up.

Thanks @patelh

I would welcome being able to save a cluster configuration files so restart of the kafka-manager does not wipe out all the cluster definitions.
Thanks much,
Heji


feat: enable custom securityContext

Hello,
This PR enables the customization of security contexts.
The current value is used as default to keep the same behavior.
Kr,
Fred

lgtm, thx


[Bug] Map show blue color when update option and remove some item

Version
5.2.2
Link to minimal reproduction
No response
Steps To Reproduce
1 use vue-charts
2 set map option with dataset
3 updat dataset and items is less than before
4 the removed item's area show color like it's value is 0, not the same as normal item style
Current Behavior
removed itemstill show color and seems it's value is 0, but i already remove it from dataset
Expected Behavior
like other normal items without set
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
1 set values only has value

2 set another values set with more items, and change back to current, those new missing items still show color

https://codesandbox.io/s/misty-cloud-d4su0?file=/src/index.js
click changevalue button will found this bug
if i do not use geo and only change data in series, it's ok

Seems to be a bug. Currently you can fill the missing data with '-'. In your example:

var data2 = [
  { name: "中西区", value: 20057.34 },
  { name: "湾仔", value: 15477.48 },
  { name: "东区", value: 31686.1 },
  { name: "南区", value: 6992.6 },
  { name: "油尖旺", value: 44045.49 },
  { name: "深水埗", value: 40689.64 },
  { name: "九龙城", value: 37659.78 },
  { name: "黄大仙", value: 45180.97 },
  { name: "观塘", value: '-'},
  { name: "葵青", value: '-'},
  { name: "荃湾", value: '-'},
  { name: "屯门", value: '-'},
  { name: "元朗", value: '-'},
  { name: "北区", value: '-'},
  { name: "大埔", value: '-'},
  { name: "沙田", value: '-'},
  { name: "西贡", value: '-'},
  { name: "离岛", value: '-' }
];

I have deleted geo and geoindex in series to avoid this bug, only use series to set all options now


use react-native-vector-icons 4.0 version

Does it necessary? As I see change logs of react-native-vector-icons, MateriaIcons icons isn't updated, so maybe we don't have sense to update this package...

Why wouldn't we? There's no reason not to update the dependency as far as I can see.

@NewOldMax  I used react-naitve 0.4.2 with react-native-material-ui when I build I got RCTLogLevel error from react-native-vector-icons and I found other people have the same issue from react-native-vector-icons'issues. And the solution seems just upgrade vector-icons to 4.0 and it works.
FYI: https://github.com/oblador/react-native-vector-icons/issues/375

For me RN 0.42 and react-native-vector-icons works without errors. Weird...

our app is having react-native-vector-icons v4.0. Since this lib pins to the exact version "3.0", react-native-vector-icons is included twice for us!
Ideally react-native-vector-icons should just be a peerDependency of this project.

related: https://github.com/xotahal/react-native-material-ui/pull/90

I approved #90. I'll close this one since is not actually ...


Deterministic serialization of chat demo app state

When taking a snapshot of the application state, the serialization must be performed deterministically,
such that all nodes can agree on a snapshot. Currently, a protobuf message is used to serialize the chat demo app state.
The protocol buffer implementation, however, does not guarantee deterministic serialization.
While in practice (in the concrete case of the chat demo app) the state snapshot is always the same, technically this is not guaranteed.
Fix this by providing a deterministic implementation of the Snapshot() method of the chat demo app logic:
https://github.com/filecoin-project/mir/blob/946689a112983e6384d6c697c01de00d0f433d14/samples/chat-demo/app.go#L75

This has already been solved in the main branch.


Watering Can

Water level is not increased when interacting with water.

Not a bug, just not implemented? We were waiting for accurate map interaction, it is being completed by Team 8 this sprint I believe.


syscall: reading os.Stdin does not treat ^Z as EOF on Windows

1. call io.Copy(os.Stdout, os.Stdin)
2. Type CTRL-Z on console

What is the expected output?

  Interrupt stdin

What do you see instead?

  Seeing ^Z character

Which compiler are you using (5g, 6g, 8g, gccgo)?

  8g

Which operating system are you using?

  Windows 7

Which version are you using?  (run 'go version')

  go version devel +2f13cc5d7043 Tue Aug 27 14:53:57 2013 +1000 windows/386

Please provide any additional information below.

  https://groups.google.com/forum/#!topic/golang-dev/xnjP0VlL2nw

https://go-review.googlesource.com/#/c/4310/

+1 for fixing this. Looks like the only way to terminate input from stdin in Go 1.4.2 is to be prepared for it.

Maybe make that optional - ability to choose what combination can be used to terminate the stream.


feat(markdown): add youtube video embeds (VAC-442)

Description
Page help desperately wants for video capability, so now we can add YouTube embedded videos natively like so:
![youtu.be/videoID]
What's included?

Native iframe embeds added for youtube share links
Ability to embed playlists as well as single videos

General Tests for Every PR

[x] npm run serve:prod still works.
[x] npm run tslint passes.
[x] npm run stylelint passes.
[x] npm test passes and code coverage is not lower.
[x] npm run build:lib still works.

Screenshots or link to StackBlitz/Plunker

Question: Would it be better to have a more generic in order to support potentially more:
![video url]
And then we could potentially support other sources (Vimeo, custom MP4 locations, etc)


publishing apps using just

Can you add a recipe for snap to publish easily. Like in case of flatpak, we use just publish-flatpak.

Are you asking if it's possible to script a recipe to do that or asking the developer to add that as a specific feature?


Fix enabling dev ui via env

Description
Enabling /ui via env is broken now. There are three different envs in different parts of the code that should do the trick and enable the ui.
OLS_ENABLE_UI=false
ENABLE_DEBUG_UI=false
ENABLE_DEV_UI=true

but just ENABLE_DEV_UI is actually used for configuration.
We should fix this regardless of config refactoring PR.
Type of change

[X] Bug fix

Checklist before requesting a review

[X] I have performed a self-review of my code.
[X] PR has passed all pre-merge test jobs.

/lgtm

/lgtm
/approve


add "pachctl mount" as one of the ways to export data from Pachyderm

Provide a link to the page with an issue:
https://docs.pachyderm.com/latest/how-tos/export-data-out-pachyderm/
Describe what is wrong:
It doesn't mention pachctl mount as an option
Describe the suggested changes:

Create a documentation page on pachclt mount and link to it from here.

Anything else we need to know?
We should give practical advices on the drivers needed to be installed for various environments.
Environment:

Kubernetes version (use kubectl version):
Pachyderm CLI and pachd server version (use pachctl version):
Cloud provider (e.g. aws, azure, gke) or local deployment (e.g. minikube vs dockerized k8s):
OS:
Other:

If we do this, we should slightly discourage it's use. pachctl mount uses fuse, which is full of sharp edges.

Absolutely, but it is an option...

Depends on https://github.com/pachyderm/pachyderm/issues/4558


Enable dartanalyzer for shared .packages tests.

Due to limitations in how dartanalyzer handles enumerations of files that require multiple analysis contexts (https://github.com/dart-lang/analyzer_cli/issues/53), we need to skip a number of shared tests.  Needless to say, as soon as the required rework is done we should re-enable.

Stale.


Fix Typos in Common Errors Page

Fix Typos in Common Errors Page
🚀 Goal
We want to ensure our website's documentation don't have any mistakes.
🤔 Background
This issue focuses specifically on fixing typos across our docs. Your contributions here make a difference!
🤝 How You Can Contribute
We've broken it down into specific typos for you to address. Every contribution counts!
🔑 Tasks and Acceptance Criteria
❗You must only assign yourself one task at a time to give everyone a chance to participate.❗
You may assign yourself your next task after your current task is reviewed & accepted.

Fix Typos in Common Errors page

[ ] Fix "functon" to be "function"

Make sure all tests are passing.

🌟 Resources

Common Errors Page
Docs, Developer Website

Getting Started

Comment ".take" on this issue to get assigned
Fork the repository and create a new branch for this task
Follow the tasks outlined above
Submit a pull request with your changes
Respond to any feedback during the review process

Questions?
If you have any questions or need clarification, please comment on this issue or join our Discord community.
Happy coding! 🎉

.talk

.take

.take


Live view widgets for LiberTEM results in Jupyter notebooks etc

Right now, live updating results in a Jupyter notebook have to be coded by hand using the internal asynchronous API.
In the future, it would be nice to be able to automate that and get a live updating view of a running UDF with a few simple commands, for example for live data processing.
Possible libraries to use:

https://github.com/bqplot/bqplot
https://github.com/glue-viz/bqplot-image-gl

Thx @jacobjma for the pointers!

In addition to these libraries, we can also still support something based on matplotlib, just with a slower update-rate. I tried bqplot - one of the downsides currently is the effort spent when installing. I somehow managed to mess up my environment, and also it seems that bqplot needs to be not only installed in the environment where the kernel is running, but also in the jupyter environment. This can be harder on some systems, for example jupyterhub installations on some servers, or even on HPC infrastructure like the JSC, where it probably won't be considered.
Maybe I was also missing something else - it's just that without installing bqplot into the jupyter environment, it threw JavaScript errors which I couldn't reliably track down to any specific root cause.


Overlapped markers

Hey,
I have a situation where marked items overlaps each other,
Is there any supported visualisation that handles this case?
I came across this: http://jawj.github.io/OverlappingMarkerSpiderfier/demo.html
Thanks,
Asaf.

https://tomchentw.github.io/react-google-maps/#basics/marker-clusterer

Did anyone get OverlappingMarkerSpiderfier working with React Google Maps and possibilyt the marker clusterer?

Feel free to create a wrapper component around OverlappingMarkerSpiderfier. PR is welcomed

I did get the OverlappingMarkerSpiderfier to work, however I had to get some hooks into this library before it was possible. More specifically I needed to be able to access the map and marker objects after they had been initialized. See my 3 commits to accomplish this here: https://github.com/Bazze/react-google-maps/tree/v4.7.2-oms-edits
Then I do the following in my app to get it working:
onMapInitialized(map) {
	this.oms = new OverlappingMarkerSpiderfier(map);

	this.oms.addListener('click', (marker, event) => {
		let m = _.find(this.state.markers, {_markerObject: marker});

		m.showInfo = true;
		this.setState(this.state);
	});

	this.initMarkers();
}

onMarkerInitialized(marker, gmMarker) {
	marker._markerObject = gmMarker;

	this.setState({ markers: this.state.markers });

	this.oms.addMarker(gmMarker);
}

Not very pretty, but it works. I'd definitely love a more "official" solution for this :)

Feel free to submit a PR. Close for now
We're also looking for maintainers. Involve in #266 to help strengthen our community!


Volunteeer bio & photo for Crystal

You can attach a photo here, and comment with bio, or open a PR.

@crystalrmcgowan


Slides configuration: appear on click

Description
It would be nice if certain elements (e.g. a section of markdown code, an image, etc) could appear on click.
Suggested solution
With RISE presentations, this is done by giving the cell metadata fragment instead of slide.
With marimo presentations, I expect that it would be a Layout function which would wrap around a ui element to flag it as a revealable fragment?
Alternative
No response
Additional context
No response

I assume accordion is too basic and not flexible enough to accomplish this. We also have mo.ui.run_button() that could be used to lazily render some UI, but might require more wiring than you are looking for.
What exactly are you clicking and what exactly are you trying to make appear?

Hi, thanks for the reply, sorry I missed it earlier.
I suppose rather than on-click (as it would be in powerpoint), I mean on hitting whatever button you consider to be "next" for the presentation, so for marimo I guess that would usually be the right arrow key, or if there was a different button like space (or clicking on the slide) for performing the next within-slide actions, that would also be fine I think. Basically just some way of revealing an element via a click or keyboard press without any UI element (like an arrow or a click here or a button) being visible on the slide before it's revealed.
That would allow things like showing fragments of text/images as you're speaking about them. I think that either run_button or accordian would look pretty clunky unless it was just one think you wanted to reveal. It also doesn't work if you need the audience to be surprised by the new element (like if you're making a joke).


Devanagari Vedic Extensions $1CE9, $1CEA, $1CEB, $1CEC cannot attach or be positioned with mark glyphs

Bug2 in OPENTYPE Devanagari scripts “dev” or “dev2”
Using Microsoft Word 2010, 2016, 2019, Office 365 on Windows 7 and Windows 10
Font Sanskrit2020.otf https://sourceforge.net/projects/advaita-sharada-font/files/Devanagari/
Devanagari Unicode Block $0900 to $097F
Vedic Extensions Unicode Block $1CD0 to $1CFF
a) glyphs $1CE9, $1CEA, $1CEB, $1CEC cannot attached, anchored or precisely positioned for any combination using GPOS / GSUB.
b) The geometric shape $25CC keeps getting displayed.

अं अँ कं कँ हं अः॑ कं॑ ह॑ with glyphs $0900 to $0939 these can be accurately positioned and the geometric shape $25CC is never in the way.
bug2 in OpenType Devanagari script.docx

bug2 in OpenType Devanagari script.pdf

Thank you for the feedback, can you please share the sample text string shown in your picture.


Feat/backup claduie

closes #897

why the kustomization.yaml changes?

Otherwise the secrets or config maps wouldn't be backed up and when restoring it would fail
Everything thats part of claudie should have the label so the backup will be complete

Perfect, thanks.
My point is, if the labels are missing from somewhere, we should add them to the manifests.
But yes, for configmaps and secrets, the only way to have those labels is via kustomize options, so your solution seems optimal


Bldy ci

add bldy back to the build matrix and into our hearts.

Does it still work with #!/bin/sh? I don't see any obvious bash-only syntax, and that would increase portability.

@raylai nice catch thanks.

@raylai turns out line 15 does. It really doesn't matter since that ci script isn't used anywhere but travis so portability really doesn't matter.


Document cancellation safety

Closes #2492.
This is a minor changing removing one log message and clarifying future cancellation of make_request in the code.

bors r+


update for exclude path

Add a little feature, we can exclude some folder or subfolder by passing an array.

Can you explain what you've done here?


docs: Add kwatch to showcase

Motivation
Hi, I'd like to add kwatch to the showcase
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Check if kwatch is added to the showcase
Related PRs
None

@facebook-github-bot signed


Support extra request headers for the terraform provider

Preflight Checklist

[X] I could not find a solution in the existing issues, docs, nor discussions
[X] I have joined the ZITADEL chat

Describe your problem
We are deploying Zitadel behind GCP IAP and only expose the login ui and oidc endpoints to the internet. So to be able to make request to Zitadel management API, we need to authenticate with IAP. This can be done by adding a JWT token in the Proxy-Authorization header.
Describe your ideal solution
The Zitadel terraform provider configuration supports a map field extra_headers. All key-values in this field are added as HTTP headers to every request to Zitadel. By doing so, one could create the required JWT to authentique with IAP using data "google_service_account_jwt" and use this value to set Proxy-Authorization header.
This is done by this Keycloak terraform provider
Version
No response
Environment
Self-hosted
Additional Context
No response

Thank you for sharing your idea.
If there is a significant demand from customers/community, we will carefully consider implementing the feature.
Currently, the issue will be added to our product backlog to collect feedback.


Get report in a variable

Another nice feature is the ability to get the report in a file or in a variable in order to send the report to the browser or as an attachment in an email.

Sorry, I shouldn't have merged this. I'm still trying to wake up haha.
It looks good but I would prefer the code to be in a separate method.
Basically, remove the control coupling, save_file = false.


Don't count the crypto stream as incoming on the server

We should fix this in the streams map.

I don't think this is what GQUIC is doing.

Maybe, then I think their behavior is wrong. Anyway, it shouldn't cause any issues if we fix this, and I think the streams map is the right place to do this (as opposed to in the config).

Not more issues than we currently have at least. The one stream should fall within the 10 streams / 10% tolerance anyway, so that won't hurt.
Can't wait to get the MAX_STREAM_ID frame.


begin work on serverside filtering for crosssearch. connection on fil…

…ter-change initiated

#486 implemented

As mentioned in #376 this is not possible to implement at the moment. Closing. Can be opened later.


python3Packages.simber: 0.2.2 -> 0.2.3

Motivation for this change
Version bump for a small logging library
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 nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 129051 at 8345f39d run on x86_64-linux 1

3 packages built successfully:

python38Packages.simber
python39Packages.simber
ytmdl


how to detect  continuous underline by tesseract 4?

This feature is not supported. #251.


SOR #61  Guardian of the Whills

Also included a bug fix for Omega - appears it was ignoring aspect penalty on any card so long as a clone wasn't played

Closing this as we need new logic to hand a card becoming a new unit after being played and resetting its state

Re-opened as the copy-id logic was added, and tests were updated


Remove more interaction without hands

About the PR

Fixes mice spamming "split stack" everywhere
Mice can no longer cremate people
Mice can no longer unfold things
Mice can no longer change transfer amount for fluids, with the added bonus of being able to drink drinks without right clicking

Why the verb have a variable "hands", and not this is built into a "CanInteract"?

not every interaction should be disallowed if you don't have hands. the hands component is passed into the interaction event as a lot of events may want to check if it exists or do stuff with the active hands

Honestly, this "fix" should be temporary. In the future, we should have more refined checks that disallow certain interactions even if you have hands, to allow for brain damage preventing some interactions, etc.


Proxy issues on Firefox

Firefox doesn't seem to be sending the header required by Coproxy, so proxy-needing images don't embed correctly.  There's an example on page 6 of the Dev Cocreate.

It seems that Firefox won't send Origin headers for images.  I've modified the proxy configuration to no longer require Origin attributes.  This might result in some throttling (because the proxy can't tell it's coming from Cocreate) but better than failing.


Faster CI runners

Previously Embark used paid runners to get faster builds. As part of the transition we went to the slower Github action runners. We should look into getting credits and/or sponsors for faster CI if it is a problem.
This is what they used to have:
          - os: ubuntu-20.04-16core
            target: x86_64-unknown-linux-gnu
          - os: windows-2022-16core
            target: x86_64-pc-windows-msvc
          - os: macOS-latest-xl
            target: x86_64-apple-darwin
          - os: ubuntu-20.04-16core
            target: aarch64-linux-android
            host: x86_64-unknown-linux-gnu

I ran the CI locally on Friday for the Ubuntu targets, and my initial thoughts:

dependencies seem to be duplicating.

sudo act --verbose

shows me the "installing depenendency" steps running 4 times for different tests. I think we need to fix CI to share the same ubuntu container to stop duplication. I'm looking into this.

we should switch out Embark's CI runners for something leaner since this is its own project now.


Issue with sending to M365 groups, Distribution groups or Security groups feature

Hi team,
Got quite a problematic issue which needs solving. Seems to be happening to a lot of people I have spoken to and I have tested it myself and can confirm there is a bug of some sort. Essentially, even though we have granted the correct permissions (with admin consent) we cannot use the option to send a message to the O365 groups. It shows the same "Permissions needed. Ask your IT admin to enable this option for you" warning message that is normally visible if you haven't yet granted admin permissions.
I know people with CC installed on their demo tenants and it was working fine and randomly this issue came to them and won't go away. Customers I have deployed to recently also have this issue.
To preface this before you ask: yes I have permitted admin consent and I have followed the guide and added the correct permissions, screenshot below of my permissions. And yes, I can confirm this is in the graph app, not the author or the bot app. For me, I can send 'messages to general chats', 'in chat to members of a team' and 'in chat to everyone'. It is just that o365 groups and distribution lists feature which does not work. The screenshots below are the latest v5 version of Company Communicator,.

This seems to be happening with the Cristiano fork build also (latest build)- Just deployed it with a customer and I saw they had this issue.
I remember reading an issue thread (cannot find it) where someone suggested they had this issue with the certificate deployment but when they deployed via Powershell they did not have this issue. I have tried to test this claim myself but the Powershell script will not deploy correctly for me -- I will raise a separate issue for that. I will go away and try Powershell some more and let you know of any findings but for now this needs to get looked at.

Hi @SammiRasha
Thank you for raising the query and providing detailed information on this issue and the steps you've taken to make sure everything is correct. I can see you've not included GroupMember permissions as mentioned in the Wiki and rather added Group permissions. Can you please add this permission as this is needed to read the distribution groups of the organization? Screenshot of the correct permissions below for reference,

Hi @Porkai-Pandian,  so the reason why I put group.read.all is due to the screenshots in the deployment guide, they are outdated and have been taken from the v4 guide. These need to be updated - can confirm that when I changed them to groupmember.read.all this fixed it for me.
However - I have a colleague who also has this issue. They have the correct permissions (photos posted below). He is on v5 also

Seems like V4 also has this issue. Checked the permissions on a colleagues tenant -- they had the group.read.all (which was what they were meant to put according to the V4 guide) but still didn't have permission. I tried now to add the groupmember.read.all permissions also, and it still wouldn't work

Hi @SammiRasha , Could you please add the below permission as well.

Also after adding the permission it might take up to 48 hrs. to reflect. Request you to please wait for 48hrs and then try login to MS  Teams web version and check if the 4th option is enabled or not.

@v-royavinash thanks for your reply.
Will try this with both the v4 and v5 versions and update the issue tomorrow

@v-royavinash Tried this and it still did not work, issue is still there

Found the error -- in v4 in the app service there is the author secret, user secret (which makes sense as those are the two app registrations generated during deployment) but there is also an Azure Ad secret.
It turns out that the clientID for the AzureAd, in the app service, is the same as the author ID but the secret was not. To fix this, simply replace the author secret into the AzureAD secret to match, save and restart the app service.


I'm running on runpod but getting this error: ModuleNotFoundError: No module named 'tkinter'

Getting this error when typing: python3 ./kohya_gui.py
ERROR: ModuleNotFoundError: No module named 'tkinter'

Same... =\

Solved by doing:

source venv/bin/activate might be optional
apt-get install python3.10-tk don't need to precise .10 if you only have one version of python3

I have a similar problem here, but in this case the message is
ModuleNotFoundError: No module named '_tkinter'
with the underscore symbol.

They this tutorial from SECourses:
https://github.com/FurkanGozukara/Stable-Diffusion/blob/main/Tutorials/How-To-Install-Kohya-LoRA-Web-UI-On-RunPod.md
Working on runpod. Should be the same on any other Linux system

Thanks @moon47usaco, but it doesn't work for me.

Did fastapi install without errors inside the venv?

@moon47usaco All the installation process runs flawlessly, except for this line
apt-get install python3.10-tk
I've even downgraded my version of Python from 3.11.x to 3.10.12 (and 3.10.7 before), but I can't find a way to install that package on Debian.

Sorry can not help with Debian. I know nothing about it. Try the discord from the tutorial he is really helpful.

No worries, dude!


Daemonit uses Ansistrano

Hi,
Daemonit is very proud to be an ansistrano user :)
Cheers,
Daemonit CEO

Thank you so much for using ansistrano! ❤


Refactor logging package

Refactor logging package to make log statement simpler. Current format logging.Logger.Info() doesn't look good. We can embed Logrus into pkg/log to import and use it as log.Info()

Fixed with https://github.com/infracloudio/botkube/pull/285


Myaccount custom profile attributes without display order defined are listed at the top

Is your suggestion related to an experience ? Please describe.
In myaccount user profile, for custom added attributes which do not have a display order defined, the new attributes are listed above the already existing attributes.

Describe the improvement
Ideally, newly added attributes should be displayed at the bottom unless a display order is specifically defined.

Fized with https://github.com/wso2/identity-apps/pull/3934


Localized cost models table

Localized cost models table and some small test updates

Codecov Report

Merging #2171 (4bdd1a4) into react-intl_changeover (28ad717) will decrease coverage by 0.74%.
The diff coverage is 36.84%.

:exclamation: Current head 4bdd1a4 differs from pull request most recent head 7189530. Consider uploading reports for the commit 7189530 to get more accurate results

@@                    Coverage Diff                    @@
##           react-intl_changeover    #2171      +/-   ##
=========================================================
- Coverage                  69.73%   68.99%   -0.75%
=========================================================
  Files                        407      407
  Lines                       7428     7430       +2
  Branches                    1590     1590
=========================================================
- Hits                        5180     5126      -54
- Misses                      2213     2269      +56
  Partials                      35       35

Impacted Files
Coverage Δ

src/locales/messages.ts
100.00% <ø> (ø)

.../pages/costModels/createCostModelWizard/review.tsx
38.88% <ø> (ø)

src/pages/costModels/costModelsDetails/table.tsx
51.51% <36.84%> (-38.81%)
:arrow_down:

...es/costModels/costModelsDetails/emptyStateBase.tsx
50.00% <0.00%> (-50.00%)
:arrow_down:

...pages/costModels/costModelsDetails/utils/table.tsx
32.50% <0.00%> (-42.50%)
:arrow_down:

src/store/costModels/selectors.ts
57.89% <0.00%> (-23.69%)
:arrow_down:

...ages/costModels/costModelsDetails/noCostModels.tsx
80.00% <0.00%> (-20.00%)
:arrow_down:

...pages/costModels/costModelsDetails/utils/query.tsx
30.76% <0.00%> (-7.70%)
: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 28ad717...7189530. Read the comment docs.


3.2.3-stable_win64 Won't start

3.2.3-stable_win64:
OS/device including version:
Windows 10, version 20H2
OpenGL ES 3.0 Renderer: GeForce GTX 1080 Ti/PCIe/SSE2
Issue description:
Godot starts to open with the terminal window that outputs the Godot version and renderer information.  It never gets past that to the phase where Godot would normally actually start-up.
Steps to reproduce:
Try to start Godot
Minimal reproduction project:
No project needed for this.

Can you provide more information about your setup?
Right now there is nothing we can do with that amount of information.
Consider adding more information about your hardware, also have you updated drivers recently or anything?
Is there anything unique about your setup?
Have any other Godot versions worked for you?

@clayjohn I'm closing this out.  It appears this is related to the Windows 20H2 update.  It had recently installed, but not rebooted following the install.  I tried running 3.2.2 (which I knew was working) and that was no longer working either.  I rebooted to complete the windows update and now everything is working correctly.

Thanks for following up!
Have fun!


check version satisfiability

flow versions of packages should be satisfiable, in the sense that we should not allow bogus ranges whose lower bound and upper bound point in the wrong directions

Argh, old commits got included, will try again


[Label] pointing label mau posicionado

https://jsfiddle.net/WallaceMielniczki/ca0rovs3/30/

Hi @WallaceMielniczki, it works with .inline.field variation, and intentionally so—the framework cannot make assumptions about the width of a label in such scenarios. The following is a standard reply.
Thank you for posting, but although it’s a valid usage question, we’ve limited GitHub Issues to bug reports and feature requests, keeping the board more manageable for maintainers; see the contributing guidelines for more information on what kind of posts should find themselves into the GitHub Issues board.
To get answers or feedback that might allow you to repost or reopen this issue, please use one of our other support resources:

Gitter chatroom
Official forums
StackOverflow

desculpa,  achei que fosse um problema no semantic.


Dynamic scroll-parent

Added option to select scroll-parent element (not only window)

should be closed


choose datetime object for simulation.from_imod5_data times argument

a review comment on the PR for issue #1068  says:
"I realize we are diverting from the public API. There are a lot of different types of objects in Python to describe time & calendars, which are all still commonly used. See this scheme https://stackoverflow.com/a/21916253
Modflow6Simulation.create_time_discretization accepts a lot of these, and internally converts these to np.datetime64. We use the function imod.util.time.to_datetime_internal() for this. I don't think np.datetime64 has the required year attribute for that. Quickest way would be using the line from to_datetime_internal: year = time.astype("datetime64[Y]").astype(int) + 1970.
It is probably worth a separate issue to align the from_imod5_implementation with create_time_discreatization."

It turned out that we need a larger effort to allign the datetime format throughout imod-python. If we convert all datetimes to np.datetime64 or cftime, we run into issues when using a function like expand_repetitions. It  only supports (python) datetime objects (because it makes use of a .year property not available on all datetime types).


Add tech-w-tej

Related issue

What changes does this PR do?
Screenshots (If any)
Questions

Hey, I am closing this PR as it's open for a while. Feel free to raise a new PR. if you need any help do let's know in the discord server
__
  Summary: :hammer_and_wrench: This code needs a fix


Change local variable to class variable in Rail example in README

Change local variable to class variable to make the variable appear in view for Rails example in README

Thanks !

Thats an instance variable. Class variable has @@.


Unable to uninstall pytree

I was trying to implement the SAC model from ray package and then I got the error

The same error still existed after using "pip install tree". So I have to follow the instruction from an answer in ray. When I attempted to uninstall tree, it took more than three hours and was still incomplete..

Hi, this repo is related to the Python package Tree, not tree. My package ist just a small package for drawing fractal trees.

OK Thank you for replying


Consultant webpage - Open parties not displaying under 'attend'

Expected behavior
Open parties should be displayed under the 'attend' tab under my parties.
Actual behavior
Open parties are not being displayed under the 'attend' tab. Example below. Consultant ID #505 has a few open parties for August and her webpage shows none.

Steps to reproduce behavior
Create open parties in myHQ.
View consultant webpage in website/big commerce.

This might be an API issue.

Tested ConsultantID#505, and I'm getting 1 result from the API

Fix associated with #235 has also fixed this issue.

8-10 SH -  Fixed.


IDEA: Settings menu

Once the game gets more developed, I have an idea of a couple items that could go onto the "settings" menu (and would be saved per-user).

Option to change message text - options would be "classic"  (IE messages copied straight from stars!) and "short" (abbreviated ones suitable for display on smaller screens or for people that don't like reading fluff)
Dark Mode/Light Mode toggle (important!)
Various accessibility settings (high contrast, colorblind mode, etc etc)
Screen layout settings (computer only) - mimics small/medium/large screen options in Stars! e-
(this should get the "enhancement" label but I don't think I have permission to add labels to PRs)

Come to think about it, this would also be a solid way to get testing for "opt-in" beta features still in development or needing tweaking.


cannot appear in a

Hello,
Since Vue 2.0 we must use <router-link> tag to redirect in the HTML templates, but how can we do now in
 tags ?
When I'm building with webpack it tells me:
✘ 1 problem (1 error, 0 warnings)

Errors:
 1  http://eslint.org/docs/rules/no-undef

 ERROR in ./src/pages/offer/CampaignsPage.vue
                     </router-link>
                        <router-link :to="{ name: 'offer-edit', params: { id: offer.id }}">
                        ^
                             {{offer.name}}

  Tag <router-link> cannot appear inside
 due to HTML content restrictions. It will be hoisted out of
 by the browser.

Is there a solution ? Or should i pass through a @click event ?

Where does this error come from? It's not a Vue or Vue-router error as far as I can tell. I have not encountered it so far.
Are you having some custom HTML linting for .vue files going on?
This statement is also wrong: since Vue converts the template into a render function long before the browser could ever see it and complain about it, this is a non-issue with .vue files.
Please provide a reproduction repo or something.

You are using an outdated version of vue-loader. Upgrade to 9.x.


Using filepath.Join() instead of strings with slashes

fixes: https://github.com/prometheus/tsdb/issues/426
Using filepath.Join() instead of strings containing forward slash path delimiters (needed for non-*nix OSes), as suggested by @krasi-georgiev
I hope it can be useful.
Thanks & best regards,

Ben

Thanks, appreciated!