Can't change a cursor theme in KDE (Wayland)

KDE settings do not change the cursor theme.
Steps To Reproduce
Steps to reproduce the behavior:

Run Plasma (Wayland) session
Open the settings
Pick a cursor theme

Expected behavior
A cursor should use the cursor theme selected in the settings.
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.78, NixOS, 21.11 (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.5pre20211007_844dd90`
 - channels(root): `"nixos-21.11pre329879.c5ed8beb478"`
 - channels(suhr): `"home-manager"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Works for me. You may need to restart your session for it to apply fully.


Fetch account data from Canvas API, create and populate account table (#138)

This PR expands course_inventory/inventory.py to model, gather, and store Canvas account data, namely, account.canvas_id, account.sis_id, and account.name. It introduces a new function gathering account data that accepts a list of account IDs (pulled from associated course data) and returns a pandas.DataFrame, which is then written to CSV (optionally) and the database. The existing course.account_id field was also made a foreign key, connecting to account.canvas_id. The PR aims to resolve issue #138.

I had local DB with the latest migration changes from the master and after the I pulled in this PR and ran the process again. I see below error
course_inventory | INFO:__main__:MySQL caught up
course_inventory | INFO:__main__:Applying any new migrations
course_inventory | INFO:yoyo.migrations:Applying 0016.add_account_table
course_inventory | INFO:yoyo.migrations: - applying step 0
course_inventory | INFO:yoyo.migrations: - applying step 1
course_inventory | INFO:yoyo.migrations: - rolling back step 0
course_inventory | Traceback (most recent call last):
course_inventory |   File "/usr/local/lib/python3.7/site-packages/yoyo/migrations.py", line 313, in apply
course_inventory |     getattr(self.step, direction)(backend, force)
course_inventory |   File "/usr/local/lib/python3.7/site-packages/yoyo/migrations.py", line 413, in apply
course_inventory |     self._execute(cursor, self._apply)
course_inventory |   File "/usr/local/lib/python3.7/site-packages/yoyo/migrations.py", line 379, in _execute
course_inventory |     cursor.execute(stmt)
course_inventory |   File "/usr/local/lib/python3.7/site-packages/MySQLdb/cursors.py", line 209, in execute
course_inventory |     res = self._query(query)
course_inventory |   File "/usr/local/lib/python3.7/site-packages/MySQLdb/cursors.py", line 315, in _query
course_inventory |     db.query(q)
course_inventory |   File "/usr/local/lib/python3.7/site-packages/MySQLdb/connections.py", line 239, in query
course_inventory |     _mysql.connection.query(self, query)
course_inventory | MySQLdb._exceptions.IntegrityError: (1452, 'Cannot add or update a child row: a foreign key constraint fails (`course_inventory_local`.`#sql-1_3`, CONSTRAINT `fk_account_id` FOREIGN KEY (`account_id`) REFERENCES `account` (`canvas_id`) ON DELETE CASCADE ON UPDATE CASCADE)')

and with additional stack trace as
Traceback (most recent call last):
course_inventory |   File "run_jobs.py", line 136, in <module>
course_inventory |     db_creator_obj.migrate()
course_inventory |   File "/app/db/db_creator.py", line 61, in migrate
course_inventory |     backend.apply_migrations(backend.to_apply(migrations))
course_inventory |   File "/usr/local/lib/python3.7/site-packages/yoyo/backends.py", line 435, in apply_migrations
course_inventory |     self.apply_migrations_only(migrations, force=force)
 File "/usr/local/lib/python3.7/site-packages/MySQLdb/connections.py", line 239, in query
course_inventory |     _mysql.connection.query(self, query)
course_inventory | MySQLdb._exceptions.OperationalError: (1305, 'SAVEPOINT sp_2 does not exist')

I had local DB with the latest migration changes from the master and after the I pulled in this PR and ran the process again.

@pushyamig, can you wipe out your database (delete .data) and try this again? I think having data in the tables will cause this to fail because of the new relationship to account.

@pushyamig, can you wipe out your database (delete .data) and try this again? I think having data in the tables will cause this to fail because of the new relationship to account.

Then we need to do the same process in test and prod process once this gets merged.
isn't migrations should handle that?

@pushyamig, can you wipe out your database (delete .data) and try this again? I think having data in the tables will cause this to fail because of the new relationship to account.

Then we need to do the same process in test and prod process once this gets merged.
isn't migrations should handle that?

Yes, we need to do that when we roll out changes to the OpenShift processes. Migrations don't know what the data is in the account table (or the term table; we did this for the release, so the process is documented). We are not currently dropping records before doing migrations, because that would mean the database would be empty for an hour or more while the job finishes.

ok, after deleting the .data I can see data is populated correctly and the job ran fine
Interesting that the root account in canvas doesn't have a sis id int he UMdept table.

I think in Django it applies migrations first and then delete and load happen. I don't think we never encountered an issue for the Myla process or we stepped into those cases yet. Maybe we don't have strict Foreign key constraints for some of the tables.

If this isn't already merged by then, I'll have time to review this on Friday morning.

Thanks for the reviews and approvals! I'm merging now.


Bot: Switch from Discord.InteractivityAddon to Fergun.Interactive

Discord.InteractivityAddon by Playwo seems unsupported now, the GitHub Repo was deleted or set to private.
Fergun.Interactive is an actively maintained fork of that Package, adding more functionality and, most important, with update to Discord.Net 3.0.
I think this change is not that breaking, but just to not forget it, i'm writing it down.
https://www.nuget.org/packages/Fergun.Interactive
must happen before the bot part of #13 can happen.

4a8be96 implemented this.


LayoutSubviews should be restricted to the ContentSize and not screen size.

@tig I think this wasn't happening before.

@tig I think this wasn't happening before.

Uncheck ClearContentOnly and ClipContentOnly I have them on by default to illustrate how they work.

See the API docs for these here:
https://gui-cs.github.io/Terminal.GuiV2Docs/api/Terminal.Gui.ViewportSettings.html

What I want to get across with this post is that recalculating subview positions should only change when the ContentSize changes and not when the viewport size changes. Of course I'm referring to non-absolute Pos/Dim.

That doesn't sound right to me. 🤔
If something is centered or otherwise relative to anything but top left, its position will change if the viewport changes, no?

What I want to get across with this post is that recalculating subview positions should only change when the ContentSize changes and not when the viewport size changes. Of course I'm referring to non-absolute Pos/Dim.

Right you are.
Not sure how that happened or how I missed a unit test for it. I'll fix asap.

That doesn't sound right to me. 🤔
If something is centered or otherwise relative to anything but top left, its position will (or should, anyway) change if the viewport changes, no?

Layout is tied to the "Content Area" which is defined by View.ContentSize. A subview with X = Pos.Center() will be at the center of ContentSize.Width.
By default Viewport.Size is the same as ContentSize. In that case, you are right.
However, the whole point of ContentSize is to enable a virtual content area that is independent from the Viewport. In this scenario, we set ContentSize to 60x40 and thus X = Pos.Center() should NOT be centered in Viewport.

What I want to get across with this post is that recalculating subview positions should only change when the ContentSize changes and not when the viewport size changes. Of course I'm referring to non-absolute Pos/Dim.

Right you are.
Not sure how that happened or how I missed a unit test for it. I'll fix asap.

I see what I did. In cleaning up some old Dim.Auto code I had in LayoutSubviews I goofed. This is correct:

I'll fix this asap and add more unit tests!

By default Viewport.Size is the same as ContentSize. In that case, you are right.

Yeah, that was the thought process involved there.
The rest makes sense and sounds good to me, as well. 👍

I'll fix this asap and add more unit tests!

How about enable FineCodeCoverage again? This way we could have statistics on the APIs whose unit tests remain to be created. Of course, we have to exclude unit testing of operating system-specific drivers for now.


[CI] New Github Action to add screenshot diff report

Adds a new Github Action to automatically generate a screenshots diff report.
GOAL: To enable Transifex "Reviewers" (essentially admins for their language) to submit their own updated translation PRs and get immediate, automated feedback to see how the new translations affect various screens. This way the translators can iterate extremely quickly on improvements without a developer being a bottleneck.
If they don't like how an updated screen turns out, they can tweak the translation in Transifex, download the new messages.po from Transifex, update their PR, and immediately see the next result. Iterate, iterate, iterate. With no human dependencies.
Current draft instructions for how Transifex Reviewers can create translation PRs (assumes they are not super technical nor experienced with Github): https://gist.github.com/kdmukai/89071afb68a6a9018524c593141a0adb
If this PR is merged, those instructions will be updated at the end for how to review the screenshot changes detected by the CI run.

see example CI runs in my fork: https://github.com/kdmukai/seedsigner-translations/actions
(CI also runs right here in this PR, but there are no translation changes vs dev right now)
Sample resulting CI artifacts: ci-artifacts.zip

The included minified css is copied directly from https://github.com/picocss/pico

I think this should eventually be moved to the main repo, but that is a "high stakes" repo and there is a lot of new mechanism here that contributors will need time to get fully comfortable with. But that being said, this is a pretty crucial tool to have during this initial translations phase. And since this is a lower stakes repo, the bar is somewhat lower for enabling this automation in here first.

@dbast you are our Github Actions guru and would love your eyes and feedback on this!

Hi @kdmukai , think I am doing the commit for testing this on wrong repo, right?

Maybe now?

The PR looks pretty good: The action versions are outdated and could be updated actions/upload-artifact@v3 -> actions/upload-artifact@v4, actions/setup-python@v4 -> actions/setup-python@v5, actions/upload-artifact@v3 -> actions/upload-artifact@v3. This also the resolves the deprecation warnings shown in https://github.com/SeedSigner/seedsigner-translations/actions/runs/12551708293?pr=9

@dbast I applied the changes:

actions/upload-artifact: v3 -> v4
actions/setup-python: v4 -> v5

CI ran successfully with those changes: https://github.com/SeedSigner/seedsigner-translations/actions/runs/12562703726
No deprecation warnings in the output (just the note that ubuntu-latest is versioning up soon).

Concept ACK
I cannot wait to see this in action, it will make a huge difference to shorten the feedback loop for translators.
I think I saw an artifacts file from a fedebuyito commit, changing "borrar to supr"? but I'm unsure about when that run occured and/or if it reflects latest changes.  Will watch this pr with hope.
Have a safe end-of-year celebration fellows!  All the best for 2025.

Sorry, forgot this in my previous comment: actions/checkout@v3 can also be updated to actions/checkout@v4.
Thanks and all the best for 2025!


[UI] use createTheme instead of createMuiTheme

Current Behavior

When make run-ui-dev is run, a log appears stating Material-UI: the createMuiTheme function was renamed to createTheme.
Below is an attached screenshot.

Desired Behavior

No such log should appear
Implementation

import and use createTheme instead of createMuiTheme.
Some information regarding the log.
From above, it can be assumed that both functions are currently acceptable. However, there is an unnecessary log that appears which can be prevented.
Would you like to work on the issue?
Yes
Contributor Guides and Resources

🎨 Wireframes and designs for Meshery UI in Figma
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Hello, Is this issue open for contribution? If so can I work on it?

@sudhay23 Actually, I made the changes in a branch. I was not able to update the issue context as I was out of the station for the past 2 days. I apologize for the inconvenience caused due to this.


Chapter 5 Exercise does not work as written

On page 118-119 the exercise indicates that new folders should be created under the root and a new yaml file should be created.  Attempting to build from the root directory using the commands in the yaml fail.   There is no solution or project referenced in the build command.
Furthermore, attempting to build packtbookslibrary-api or packtlibrary-api-Test fails as well.
error NU1101: Unable to find package packtbookslibrary.Shared.Models. No packages exist with this id in source(s)

Hi @rogertucker - You may compare the file that you have created, with the samples shared under the _starter-kit folder of this repository.
Please note that the intent of the steps mentioned in the exercise was to assist you to develop get familiarity the constructs of creating build definition files using YAML. However, for completeness of the solution, you will have to try out the steps in the other subsequent chapters as well. After you are done reading through all the chapters, and have tried out the various exercises, you should be able to build and deploy the solution.
Thanks
Subhajit


Tracking speed doesn't match Pointer speed with acceleration set to 0

OS
macOS 14.1
LinearMouse
v0.9.5
Describe the bug
I was under the impression that with pointer acceleration set to 0 and pointer speed set to 1, it should match the same behavior when pointer acceleration is disabled with tracking speed set to 1. When I move the mouse across the screen in both scenarios, it tracks at a different distance on the screen for the same physical mouse movement.
For background, I want to determine what Pointer settings I should use with Linear Mouse when using Logitech GHub software to ensure that the DPI set for my Logitech mouse is the actual DPI being used.
I also noticed that when toggling "Disable pointer acceleration," the value of "Pointer acceleration" is taken from "Tracking speed," and vice-versa.
To reproduce

Go to Pointer settings.

Untoggle "Disable pointer acceleration."

Set Pointer acceleration to 0.

Set Ponter speed to 1.

Move the mouse around across the screen.

Now, toggle "Disable pointer acceleration."

Set Tracking speed to 1

Move the mouse around across the screen.

Expected behavior
The tracking speed and distance to be the same.
Anything else?
No response

Pointer speed is essentially the inverse of pointer resolution, which can be simply understood as a scaling of the movement speed of the cursor. A value of 1 isn't a special value for pointer speed, so the different movement speeds you're experiencing in the two scenarios you mentioned are to be expected.
There have been discussions before about how to configure the pointer speed to be exactly the same as on Windows. However, due to the closed-source nature of both macOS and Windows, answering this question isn't straightforward.
If you want to obtain an accurate DPI, my suggestion is to use a DPI analyzer to measure the actual DPI, and then adjust the pointer speed according to the ratio between the measured DPI and your target DPI.

Thanks for explaining the definition of the pointer speed.
How is tracking speed (when pointer acceleration toggle is disabled) different from pointer speed?
I guess I am trying to get at if I set a DPI by some third-party software, should I have the pointer acceleration toggle enabled or disabled to adjust either pointer speed or tracking speed, respectively, to achieve the desired DPI? What do you recommend?

Since you are using macOS Sonoma, I would suggest disabling pointer acceleration and adjusting the tracking speed instead of merely setting it to zero. This approach, a new feature in Sonoma, effectively and completely turns off pointer acceleration.

I'm closing this issue. Please feel free to reopen it if you have any further questions. :)

As a note for others, turning off pointer acceleration and setting the tracking speed to 1 results in the DPI matching what is specified by third-party mouse software (e.g., Logitech G Hub).


Performing 1 SCF step of a B3LYP calculation

Hi,
I want to take a density matrix, give it to the DFT.SCF driver to perform one cycle of SCF, not the whole run, and then return the Fock matrix. Shown below is the equivalent HF calculation to what I want, found in the source code:
# Skip SCF iterations. Compute only the total energy of the initial density
if mf.max_cycle <= 0:
    fock = mf.get_fock(h1e, s1e, vhf, dm)  # = h1e + vhf, no DIIS
    mo_energy, mo_coeff = mf.eig(fock, s1e)
    mo_occ = mf.get_occ(mo_energy, mo_coeff)
    return scf_conv, e_tot, mo_energy, mo_coeff, mo_occ

But I want to do this for a B3LYP calculation where I provide a geometry and a density matrix. I have performed B3LYP/3-21G* calculations via Gaussian 16 and saved all the density and Fock matrices for all steps of SCF, so I can use these to compare. For the below tests, I set my input density matrix to the dm from the 1st SCF cycle of my g16 data. So the Fock matrix I calculate from PySCF should be similar to the Fock matrix for the 2nd step of the G16 data.
This is what I currently have:
import pyscf
from pyscf import dft, scf, gto

# input_density_matrix = dm in AO basis from previous calculations via G16
# geom= Elements and geometry from G16

mol = gto.M(
    atom = geom,  # in Angstrom
    basis = '3-21g',
    symmetry = False,
    unit = 'Angstrom')

mf = dft.RKS(mol) # Perform a single step of SCF for given dm via kernel driver
mf.xc = 'b3lyp'
mf.max_cycle=0
mf.kernel(dm0=input_density_mat)
F1=mf.get_fock()

mf2 = dft.RKS(mol) # Perform a single step of SCF for given dm via scf.get_fock()
mf2.xc = 'b3lyp'
F2=mf2.get_fock(dm=input_density_mat)

mf3 = dft.RKS(mol) # Perform full SCF for given dm
mf3.xc = 'b3lyp'
mf3.conv_tol=1e-12
mf3.kernel(dm0=input_density_mat)
F3=mf3.get_fock()

When I set the input_density_matrix to the density matrix from the 1st SCF cycle of my G16 data, I hoped that I could perform one step using the above PySCF functions and get a Fock matrix that matches the Fock matrix for the 2nd SCF cycle of my G16 data. However, F1 and F2 are very different, which I don't understand. Don't they both calculate the Fock matrix for the given dm using one cycle of B3LYP? Or does get_fock() inherit from the base SCF class in a strange way?
Furthermore, F3 should match or at least be similar to the converged Fock matrix from my G16 data, but it differs by a significant margin (Unless I haven't set the tolerances right, G16 uses dm RMS tolerance of 1e-6 whereas PySCF uses total energy devation tolerance of 1e-10). I suppose there would be discrepancies between how PySCF and G16 perform the convergence calculations (algorithms used to speed up convergence and whatnot) and hence get two different, but equally viable, converged Fock matrices.
If anyone can spot something I've got wrong with the code or explain the different mechanics behind F1 and F2, I would greatly appreciate it.
Kind regards,
Adam

F1 does a Fock build with the density after one diagonalization while F2 builds Fock using your input density.
Could you please check if using b3lypg gives you the same converged Fock as Gaussian? I believe Gaussian uses a different implementation from PySCF, and b3lypg is how you trigger to use the same as Gaussian.

Furthermore, F3 should match or at least be similar to the converged Fock matrix from my G16 data, but it differs by a significant margin (Unless I haven't set the tolerances right, G16 uses dm RMS tolerance of 1e-6 whereas PySCF uses total energy devation tolerance of 1e-10). I suppose there would be discrepancies between how PySCF and G16 perform the convergence calculations (algorithms used to speed up convergence and whatnot) and hence get two different, but equally viable, converged Fock matrices.

Gaussian and PySCF have different convention in basis function order, so if you pass Gaussian's dm to PySCF directly, the calculated Fock is meaningless -- but it may be occasionally right since 3-21g is very small.

F1 does a Fock build with the density after one diagonalization while F2 builds Fock using your input density. Could you please check if using b3lypg gives you the same converged Fock as Gaussian? I believe Gaussian uses a different implementation from PySCF, and b3lypg is how you trigger to use the same as Gaussian.

Since version 2.3.0, b3lyp is the same as b3lypg. In previous versions they are different.

I am not sure I am following, however...

But I want to do this for a B3LYP calculation where I provide a geometry and a density matrix. I have performed B3LYP/3-21G* calculations via Gaussian 16 and saved all the density and Fock matrices for all steps of SCF, so I can use these to compare. For the below tests, I set my input density matrix to the dm from the 1st SCF cycle of my g16 data. So the Fock matrix I calculate from PySCF should be similar to the Fock matrix for the 2nd step of the G16 data.

you seem to be assuming that Gaussian does a pure Roothaan step, without any EDIIS+DIIS acceleration or the use of level shifts.

Do you get full agreement with Hartree-Fock?

@hebrewsnabla
Gaussian and PySCF have different convention in basis function order, so if you pass Gaussian's dm to PySCF directly, the calculated Fock is meaningless -- but it may be occasionally right since 3-21g is very small.
Not entirely sure what you mean by this, but is there a way to format the G16 density matrix to have the same convention as PySCF?
@susilehtola I figured there are loads of different options that I would need to set to make the PySCF calculation consistent with Gaussian, just not entirely sure which options those include, plus there is another disconnect as I'm not providing previous Fock matrices for any extrapolation-type algorithms that Gaussian would perform in a full SCF procedure. I'll give Hartree Fock a go and see if the two softwares are similar.

but is there a way to format the G16 density matrix to have the same convention as PySCF?

Yes. MOKIT (https://gitlab.com/jxzou/mokit) is a toolkit to do that. It provides a utility bas_fch2py to generate a PySCF input which could read MOs from Gaussian fch file (and convert it into PySCF order). There's no utility to convert dm yet, but it's easy to get dm with MOs. See documentation at https://jeanwsr.gitlab.io/mokit-doc-mdbook/chap4-5.html#453-bas_fch2py for more information.

Why do you need such 1-step fock matrix with PySCF?

For the full context of what I'm trying to do. I have neural networks that were trained on B3LYP converged density matrices to predict the converged B3LYP Fock matrix, which they do fairly well. I can give the networks an unconverged density matrix and they can also predict this Fock matrix. But diagonalising this F to get a new D and cycling this process to emulate SCF doesn't work; the Fock matrices diverge. This is obviously due to ML overfitting to converged B3LYP density/fock matrices.
To address this, we want to expand our training set with unconverged Fock matrices, this can be achieved by taking a converged F, adding Gaussian noise to the diagonals (lets call this F_noisy), diagonalising to get the new, unconverged D (lets call this D') and then performing one step of a QC calculation to get the following unconverged F (F'). This means that I can replicate a Fock matrix as many times as I want, build up a lot of (D', F') pairs for a given molecule and hence have more varied training data that includes unconverged density matrices, where I can directly control the degree of non-convergence. I will also add the D and F from all the G16 SCF steps and Harris/Huckel/Core guesses to this training data. After training, the network should have seen unconverged and converged density matrices and be better at modelling an SCF run. So thats why I need a 1-step fock matrix, to go from D' to F' via a desired choice of QC method (B3LYP in this case).
G16 is installed on my Uni computer cluster. There is a lot of overhead of going from my python scripts to G16 input files on the cluster, to performing a single iteration using G16, then reading all that information back in (from python to G16 then back to Python). I only care about the step of going from D' to F', so I want to stay pythonic and I figured PySCF can allow me to do that. I specifically want a B3LYP calculation; if I was dealing with simple RHF data, I could code the  D' to F' step explicitly in a function, but explicitly coding B3LYP doesn't look as straightforward to me.
The question I had earlier this week: What is more faff, fiddling with the python-G16-python route or figurring out PySCF. In hindsight, both are probably as time-consuming...
Back to the above advice, I tried HF density matrices:
mf = scf.hf.SCF(mol)
mf.verbose = 1
mf.max_cycle = 0 # I compared both the full SCF convergence and cycle=0
data=mf.kernel(dm0=input_density_mat)
F4=mf.get_fock()

This also gives a different Fock matrix to the one calculated via G16, however, I think @hebrewsnabla is right with the basis set conventions and thats why we see a difference. In the below screenshot are the first 9 elements of a PySCF HF Fock matrix (top) and a G16 Fock matrix (bottom). The index row=0,1 and col=0,1 blocks are consistent but row,col=2 is where they differ. For example, in the top block row 2 col 0 is -2.38914 h, whereas this has moved to row 5, col 0 on the bottom image. G16 Atomic orbital ordering [for 3-21G] is 1s, 2s, 2p-xyz, 3s, 3p-xyz. So I can see that for PySCF the convention is 1s, 2s, 3s, 2p, 3p. This didn't even cross my mind before.

I'll have a play around with MOKIT and the AO basis set ordering and see if it solves the problem. Or what is more likely is that I'll come back here when I inevitably run into another silly error I overlooked haha :)
So watch this space
Kind regards,
Adam

The question I had earlier this week: What is more faff, fiddling with the python-G16-python route or figurring out PySCF. In hindsight, both are probably as time-consuming...

What about using PySCF to get all the D and F matrices (i.e. no Gaussian used in the workflow)? That does not require any convertion or heavy coding.
If you insist on converting matrices back and forth with MOKIT, you also need a utility to do fock matrix convertion. You may need to ask the MOKIT developers to implement that for you. Feel free to open issues in the MOKIT repo.

I found the PySCF documentation on basis set ordering to confirm what I assumed in my last post. I then wrote a function that can reorder a matrix according to G16 or PySCF basis set formatting. I reordered my input_density_matrix, passed it into dft.RKS, obtained the Fock matrix (F) and then reordered it to the Gaussian basis order (F_new).
This input density matrix was from the 1st SCF step of my G16 calculation. The final Fock converged at the 12th step.
The converged PySCF F_new matrix is identical to the corresponding converged Fock matrix from G16.
# input_density_matrix = dm in AO basis from previous calculations via G16
# geom= Elements and geometry from G16

# convert dm from 1st step of SCF to PySCF basis ordering
new_dm, new_orbs, idx = convert_matrix_basis_order(input_density_matrix, input_orbitals, new_order_idx=[0,1,5,2,3,4,6,7,8])

mol = gto.M(
    atom = geom,  # in Angstrom
    basis = '3-21g',
    symmetry = False,
    unit = 'Angstrom')

mf = dft.RKS(mol)
mf.xc = 'b3lypg'
mf.max_cycle=50
E=mf2.kernel(dm0=new_dm)
F=mf2.get_fock()
print("Converged:", mf.converged)

F_new, F_orbs, idx = convert_matrix_basis_order(F, new_orbs, new_order_idx=[0,1,3,4,5,2,6,7,8]) # convert back to G16

There is a little bit of hardcoding in my function but for now it works for my specific purposes.
So the actual issue was just simple basis set ordering. I can move onto the training data generation now!
Thanks for all the help! :)

Unfortunately, basis order is just one of the problems. But for 3-21G basis set, it is probably that this is the only problem. For a general basis set, there are many factors to be considered. See my answer to a similar question.

Thanks to everyone for the help, it saved me a lot of trouble :)
I'm closing this issue.


DAMASK example

@muh-hassani Can you add the notebook from Vitesh? https://github.com/vitesh13/DAMASK_workflows/blob/master/DAMASK_WORKFLOWS.ipynb and the required example files these are available at https://github.com/vitesh13/DAMASK_workflows/tree/master/Examples In addition the notebook also requires two more python modules https://github.com/vitesh13/DAMASK_workflows/blob/master/Plot_stress_strain.py and https://github.com/vitesh13/DAMASK_workflows/blob/master/simulation_runner.py . I created a cleaned up fork of the repository here https://github.com/matbinder/DAMASK-example so either you can use this or fork it to your account to add the elastic constant example to demonstrate the combined workflow of LAMMPS and DAMASK.

Already in the image, I have included the example from Vitesh. In damask/include_notebooks.sh:
git clone https://github.com/vitesh13/DAMASK_workflows
cp -r "$HOME"/DAMASK_workflows/DAMASK_workflows.ipynb "$HOME"/

And in .travis.yml the notebook is tested via:
docker run --rm pyiron/damask /bin/bash -c 'source /opt/conda/bin/activate; i=0; for f in $(ls ~/*.ipynb); do jupyter nbconvert --ExecutePreprocessor.timeout=9999999 --to notebook --execute $f || i=$((i+1)); done; if [ $i -gt 0 ]; then exit 1; fi;'

I can change the source of this to https://github.com/matbinder/DAMASK-example. But is this sufficient? I also applied the postBuild script you had in the binder directory.

@jan-janssen can you please check damask/include_notebooks.sh . I have included the repository you mentioned, and the build and tests were successful.

In the output of the test it says:
0.82s$ docker run --rm pyiron/damask /bin/bash -c 'source /opt/conda/bin/activate; i=0; for f in $(ls ~/*.ipynb); do jupyter nbconvert --ExecutePreprocessor.timeout=9999999 --to notebook --execute $f || i=$((i+1)); done; if [ $i -gt 0 ]; then exit 1; fi;'
1671ls: cannot access '/home/pyiron/*.ipynb': No such file or directory
1672The command "docker run --rm pyiron/damask /bin/bash -c 'source /opt/conda/bin/activate; i=0; for f in $(ls ~/*.ipynb); do jupyter nbconvert --ExecutePreprocessor.timeout=9999999 --to notebook --execute $f || i=$((i+1)); done; if [ $i -gt 0 ]; then exit 1; fi;'" exited with 0.

So it does not find the notebooks for DAMASK.

Thanks for pointing this out. The problem is in the build. When the postBuild script runs, it fails and include_notebooks.sh is not even called. I paste below the error:
Cloning into 'DAMASK'...

Note: switching to '264fab44'.

You are in 'detached HEAD' state. You can look around, make experimental

changes and commit them, and you can discard any commits you make in this

state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may

do so (now or later) by using -c with the switch command. Example:

  git switch -c <new-branch-name>

Or undo this operation with:

  git switch -

Turn off this advice by setting config variable advice.detachedHead to false

HEAD is now at 264fab44 [skip ci] updated version information after successful test of v3.0.0-alpha-482-g846cbdd2e

/home/pyiron/postBuild: line 8: patch: command not found

WARNING: The directory '/home/pyiron/.cache/pip' or its parent directory is not owned or is not writable by the current user. The cache has been disabled. Check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.

Processing /home/pyiron/DAMASK/python

    ERROR: Command errored out with exit status 1:

     command: /opt/conda/bin/python3.8 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-req-build-wbulfa0a/setup.py'"'"'; __file__='"'"'/tmp/pip-req-build-wbulfa0a/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-fxqnnfmf

         cwd: /tmp/pip-req-build-wbulfa0a/

    Complete output (5 lines):

    Traceback (most recent call last):

      File "<string>", line 1, in <module>

      File "/tmp/pip-req-build-wbulfa0a/setup.py", line 5, in <module>

        with open(Path(__file__).parent/'damask/VERSION') as f:

    FileNotFoundError: [Errno 2] No such file or directory: '/tmp/pip-req-build-wbulfa0a/damask/VERSION'

    ----------------------------------------

ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

Cloning into 'DAMASK-example'

So I guess I need to install patch and try it again.


Show build profile on eas build:run

Why
I love using eas build:run to quickly install a build on an emulator/ simulator, but found I had to go back to my project page to check the build ID, because I was occasionally building non-development builds for various test cases (especially when it comes to APK files), and I could't tell which one was which.
Adding build profile to the selected list item lets you match your build with something easily at-hand in your local dev environment (your eas.json file), so it's easy to tell if you're grabbing the latest development build, or a standalone internal test version, etc.
How
Added build profile to the list of data that shows when you arrow down to a build in the list.
Also slightly compressed the not-available warning based on available information..
Test Plan

/changelog new-feature show build profile when selecting a build for eas build:run

/changelog-entry new-feature show build profile when selecting a build for eas build:run


Add search keywords field

Formats such as Finders in Whitehall and Calendars send separate indexable content to Rummager as a top level field. This should be an optional field for any document that wishes to send any keywords for Rummager to pass as indexable content into Elasticsearch.
https://trello.com/c/ut9cWxrT

@dwhenry what's the advantage of having it in details?

Closing this as we decided there is not enough documents that would make use of it at this time

Specialist documents have hidden_indexable_content, would that perform the same function as this?

The field itself, potentially. But the issue was more to do with sending the field from wherever each document is published. For example Whitehall just sends it as a top level field which the Publishing API ignores. In the end we just decided that for 3 documents it is not worth adding another top level field.


Add UI to ReadMe

Pull Request Test Coverage Report for Build 221

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 71.935%

Totals

Change from base Build 220:
0.0%

Covered Lines:
2030

Relevant Lines:
2822

💛  - Coveralls


[BUG] Editing a comment to empty crashes the website

Fixes #411
Proposed Changes

user cannot edit a comment for an empty comment

This pull request closes #411

Why not disable the "OK" button if the user hasn't typed anything? otherwise enable it when the box has more than one char


[1.10.2] Solar Generators work below other Solar Generators

Advanced Rocketry: 1.10.2-1.0.8
LibVuples: 1.10.2-0.1.14
First off, a little screenshot to show what I'm talking about.

This feels like it's an unintended behavior. Since the Solar Generator does not work under the Blocks of Quartz there is little to no reason they should work below the Solar Generator itself. It kind of feels like the Solar Generator is being treated as a Transparent block.

for some reason the block for the solar panel (which is the same block for most othermachines) was transparent, It may have been an early hack to make the observatory not render dark, but should be fixed now.


Demo?

Where can I watch a demo?

Hey @DesSolo you can run Autopsy locally,  please walk through the Installation steps
let me know if you have any further question

Hey @DesSolo, thank you for your patience.
We roll out a demo available:
http://autopsy.ep4sh.cc/
Please consider, that Autopsy  is under turbulent development and some features can work improperly.
Do not hesitate to open new issues for the project.
Cheerz,
Pasha


Queue locking strategy error in Client Socket

The queue locking strategy is incorrect for the SocketContext.cs(68-70) call to
if (datagramQueue.Count > 0)   {     packet = datagramQueue.Dequeue();
In a highly concurrent environment, the queue isn't being locked for the entire transaction and a thread can pull an item off the queue making an empty queue when the datagram dequeue is called and causing a System.InvalidOperationException.
Maybe you should try to convert this to a CAS style operation or move the storage to a tested concurrent data structure.

I changed your socketcontext to read as follows
public bool TryDequeue(out Datagram datagram)
{
  lock (queue_mutex){
    if (datagramQueue.Count > 0)
    {
      datagram = datagramQueue.Dequeue();
      return true;
    }
  }
  datagram = new Datagram();
  return false;
}

The enqueue call in the Pump() should also be updated to use the Enqueue method inside the class and not directly on the queue.
The bonus of TryDequeue is that it simplifies the calls in the SocketContext as follows
public bool Read(out Datagram packet)
{
  return datagramQueue.TryDequeue(out packet);
}


Version <IP_ADDRESS> and onwards seem to break files with binary content

I've been using SIAT in unit tests with mock files containing binary content for a while (and loving it).
Binary content used to be fine but a recent update is causing my units to fail because it seems now this content is attempting to be read as Unicode.
The following code works fine in v232, but throws exception in subsequent versions:
var binaryData = new byte[] { 0xC0 };             var fs = new MockFileSystem(new Dictionary<string, MockFileData>() { { @"c:\source\file.jpg", new MockFileData(binaryData) } } );             fs.AddDirectory(@"c:\source");             fs.AddDirectory(@"c:\dest");             fs.File.Copy(@"c:\source\file.jpg", @"c:\dest\file.jpg");
Is this expected behaviour? I can change my tests to ensure the binary content can be read as Unicode - but there's a MockFileData taking byte[], and it seems reasonable for a mock file system to be able to handle binary files. I can help knock up better tests if that would help.
Exception is:
System.Text.DecoderFallbackException
HResult=0x80070057
Message=Unable to translate bytes [C0] at index 0 from specified code page to Unicode.
Source=System.Private.CoreLib
StackTrace:
at System.Text.DecoderExceptionFallbackBuffer.Throw(Byte[] bytesUnknown, Int32 index)
at System.Text.DecoderExceptionFallbackBuffer.Fallback(Byte[] bytesUnknown, Int32 index)
at System.Text.DecoderFallbackBuffer.InternalFallback(Byte[] bytes, Byte* pBytes, Char*& chars)
at System.Text.UTF8Encoding.GetChars(Byte* bytes, Int32 byteCount, Char* chars, Int32 charCount, DecoderNLS baseDecoder)
at System.Text.DecoderNLS.GetChars(Byte[] bytes, Int32 byteIndex, Int32 byteCount, Char[] chars, Int32 charIndex, Boolean flush)
at System.Text.DecoderNLS.GetChars(Byte[] bytes, Int32 byteIndex, Int32 byteCount, Char[] chars, Int32 charIndex)
at System.IO.StreamReader.ReadBuffer()
at System.IO.StreamReader.ReadToEnd()
at System.IO.Abstractions.TestingHelpers.MockFile.ReadAllBytes(Byte[] contents, Encoding encoding)
at System.IO.Abstractions.TestingHelpers.MockFileData..ctor(MockFileData template)
at System.IO.Abstractions.TestingHelpers.MockFile.Copy(String sourceFileName, String destFileName, Boolean overwrite)

Reproduced using the following testcase:
      var binaryData = new byte[] { 0xC0 };
      var mockFile1 = new MockFileData(binaryData);
      var mockFile2 = new MockFileData(mockFile1);

It looks like the TextContents are copied in this constructor, rather than the binary contents. Looks like it was introduced with PR #354
To make sure that the issue that fixes is not reintroduced, the MockFileData constructor taking the MockFileData as parameter, should use Array.Clone on the contents, rather than copying the TextContents.

Could you please push a new release of the 2.1.0 line to nuget.org? The current latest stable <IP_ADDRESS>6 was released four days before this critical fix ;-)

Our next release will be 3.0 - I’ll try to deploy within the next days.

@jbaehr Done -> https://www.nuget.org/packages/System.IO.Abstractions/3.0.2


Focus effect on checkboxes is not aligned correctly in Safari on iOS.

Focus effect on checkboxes is not aligned correctly in Safari on iPadOS 16.6.
When pressing the checkboxes the focus effect is displayed to far to the right and to the bottom.
Go to https://material-web.dev/components/checkbox/ and open the interactive demo. Press some of the checkboxes.

Verified that this was fixed in https://github.com/material-components/material-web/commit/0e8afc01757c3f1f8a5bf904e8cb857ea88bd7bf. The catalog will update on the next prerelease.


Pick up dev/run defaults from install.mk

To avoid having to specify --enable-nouveau and --with-nouveau twice.
Once it's configured, dev/run should keep starting it. Until we run configure again to disable.

This way if nouveau is configured dev/run will default to starting it.
% dev/run --admin=adm:pass -n1
[ * ] Setup environment ... ok
[ * ] Ensure CouchDB is built ... ok
[ * ] Ensure Erlang boot script exists ... ok
[ * ] Prepare configuration files ... ok
[ * ] Start node node1 ... ok
[ * ] Booting nouveau ... ok
[ * ] Check node at http://<IP_ADDRESS>:15984/ ... ok
[ * ] Running cluster setup ... ok
[ * ] Developers cluster is set up at http://<IP_ADDRESS>:15984.
Admin username: adm
Password: pass
Time to hack! ... ^Cok


GPL stacktrace with empty design

It seems that sending an empty design thru OpenROAD flow script:
module noop(clk);
  input clk;
  wire clk;
endmodule

makes GPL error out with a stacktrace:
Stack trace:
 0# 0x0000000000A86976 in /opt/conda/envs/silicon/bin/openroad
 1# 0x00007F09253A1090 in /lib/x86_64-linux-gnu/libc.so.6
 2# gpl::NesterovBase::initFillerGCells() in /opt/conda/envs/silicon/bin/openroad
 3# gpl::NesterovBase::init() in /opt/conda/envs/silicon/bin/openroad
 4# gpl::NesterovBase::NesterovBase(gpl::NesterovBaseVars, std::shared_ptr<gpl::PlacerBase>, utl::Logger*) in /opt/conda/envs/silicon/bin/openroad
 5# gpl::Replace::initNesterovPlace() in /opt/conda/envs/silicon/bin/openroad
 6# gpl::Replace::doNesterovPlace(int) in /opt/conda/envs/silicon/bin/openroad
 7# 0x0000000000ADA84B in /opt/conda/envs/silicon/bin/openroad
 8# TclNRRunCallbacks in /opt/conda/envs/silicon/bin/../lib/libtcl8.6.so
 9# 0x00007F0925D061BF in /opt/conda/envs/silicon/bin/../lib/libtcl8.6.so
10# Tcl_EvalEx in /opt/conda/envs/silicon/bin/../lib/libtcl8.6.so
11# Tcl_Eval in /opt/conda/envs/silicon/bin/../lib/libtcl8.6.so
12# sta::sourceTclFile(char const*, bool, bool, Tcl_Interp*) in /opt/conda/envs/silicon/bin/openroad
13# ord::tclAppInit(Tcl_Interp*) in /opt/conda/envs/silicon/bin/openroad
14# Tcl_MainEx in /opt/conda/envs/silicon/bin/../lib/libtcl8.6.so
15# main in /opt/conda/envs/silicon/bin/openroad
16# __libc_start_main in /lib/x86_64-linux-gnu/libc.so.6
17# 0x0000000000A80C02 in /opt/conda/envs/silicon/bin/openroad

I realize this is a limit case, but I'm curious if that's the expected behavior.
global_place_noop_asap7_base_2022-08-17_12-07.tar.gz

crashes are never expected

I don't know how you generated the packaging of this test case but it is not useable.
Error: read_liberty.tcl, 17 cannot read file ./platforms/asap7/lib/asap7sc7p5t_AO_RVT_FF_nldm_201020.lib.
There is no platforms directory.  I tried linking one and then I get
[ERROR ORD-0007] ./results/asap7/noop/base/3_2_place_iop.odb does not exist.
Please package a useable test case using make issue.

I had to pass EXCLUDE_PLATFORM=1 otherwise the make global_place_issue` fails with:
[INFO][FLOW] Using platform directory ./platforms/asap7
[INFO-FLOW] ASU ASAP7 - version 2
Default PVT selection: BC
# Creating run-me-noop-asap7-base.sh script
# Creating vars-noop-asap7-base.sh/tcl script
# remove variables starting with a dot
# remove non portable commands
# remove QT variable
# Archiving issue to global_place_noop_asap7_base_2022-08-18_02-29.tar.gz
tar: ./platforms/asap7/*.cfg: Warning: Cannot stat: No such file or directory

should I file a different bug about that?

I had to pass EXCLUDE_PLATFORM=1 otherwise the make global_place_issue command fails with:
[INFO][FLOW] Using platform directory ./platforms/asap7
[INFO-FLOW] ASU ASAP7 - version 2
Default PVT selection: BC
# Creating run-me-noop-asap7-base.sh script
# Creating vars-noop-asap7-base.sh/tcl script
# remove variables starting with a dot
# remove non portable commands
# remove QT variable
# Archiving issue to global_place_noop_asap7_base_2022-08-18_02-29.tar.gz
tar: ./platforms/asap7/*.cfg: Warning: Cannot stat: No such file or directory

should I file a different bug about that?

Please do it in the ORFS repo so I can keep track, I will take a look as soon as I am able. Thanks.

Please do it in the ORFS repo so I can keep track, I will take a look as soon as I am able. Thanks.

https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/issues/561

@proppy
Unable to check as asap7 library got updated to 211120.lib.gz.
If still there is a problem can you attach latest reproducible.?


fix(optimizer): rewrite ?commonjs-require, since `rollup-plugin-com…

…monjs` will inject require suffix for require call
fix #1566

@underfin it seems not work again.


Now serialize effect layers in scene serializer + parse in babylon file loader

TODO: update documentation (.babylon file specification)
Do you think we have to deal with AssetContainer for effect layers ?

@deltakosh ok
Going to define .addEffectLayer and .removeEffectLayer in scene

Done,
Tested in the Babylon.js Editor (serialize + parse) for highlight layer and glow layer

Wonderful!

Please stop pinging me for review, this is not the sevan you are looking for.

Sorry about that!!
Pinging @sebavan

Look good to me :-)


Fix LGTM Potentially inconsistent state update

Codecov Report

Merging #15 (5afc955) into main (f8ca0f8) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   55.42%   55.42%
=======================================
  Files          39       39
  Lines         415      415
  Branches       58       57    -1
=======================================
  Hits          230      230
  Misses        170      170
  Partials       15       15

Impacted Files
Coverage Δ

...ice-panel/components/ServicePanel/ServicePanel.tsx
30.33% <0.00%> (ø)

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 f8ca0f8...5afc955. Read the comment docs.


Are there actually any unsupported platforms?

The documentation states that Environment.GetFolderPath can throw a PlatformNotSupportedException.  Are there actually any platforms that will trigger this exception?
If so, they ought to be listed here.

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

ID: ab867d87-5271-6b58-dbbd-f8e9d08a0f86
Version Independent ID: f795178b-62a0-d839-21ae-7f738230029f
Content: Environment.GetFolderPath Method (System)
Content Source: xml/System/Environment.xml
Product: dotnet-api
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

It seems to be based on a Windows native error code:
https://referencesource.microsoft.com/#mscorlib/system/environment.cs,1522
Not sure if we can call out which platforms can trigger that exception. @karelz who could help with this question?

@pjanotti @joperezr can you please help?
Quick scan of the code didn't reveal anything obvious that would throw PlatformNotSupportedException. Is it maybe for iOS/Android platforms?

As you saw core doesn't raise that, as @mairaw pointed this seems some possibility for mscorlib in netfx. Grepping the sources I see that it was possible for some special folder constants in NDP/unix/pal but I don't know if that was ever deployed anywhere... perhaps @JeremyKuhne knows some of the history here...

We didn't support it prior to Core 2.0 for UAP. Windows actually added features to make it possible. (The existing Win32 APIs for getting known folders aren't allowed on UAP.)

So is there anything else we can say about this exception on the docs? It's not clear to me.

So is there anything else we can say about this exception on the docs? It's not clear to me. And is it Framework only?

The desktop code you linked to is a little misleading. It isn't the OS throwing it, it was the runtime PAL for Unix. That predates this functionality showing up in .NET Core. It might have been for Silverlight, or perhaps just the Shared Source CLI. I don't know (or think) that you can actually get this exception anywhere that we've publicly exposed the method that we currently document.


TPV possibly , lagging chromosomes

Not sure about this but is lagging chromosomes a separation defect? (it sounds like a segregation defect rather than separation).
A cellular process phenotype observed in the vegetative growth phase of the life cycle in which sister chromatids do not move towards the spindle poles at the same time during mitosis, but instead one or more chromosomes remain distant from the spindle pole after the bulk of the DNA has separated. Mitotic sister chromatid segregation may stop before completing separation of chromosomes, or may eventually be completed.

fixed

edit file: 653f4b9a27d3fcabf1d06845eeb5caab64d1d108
release: 21dbe123051bc992f88e2e2f9cc898264ea4e28d


Fix a bug in HashJoinOperator

This PR fixes a bug in HashJoinOperator w.r.t. the swapping of probe/build sides in a previous PR.

Hold for adding tests to cover these buggy cases to prevent future bugs.

Tests added. The problematic code would fail on the added tests.


Blog Post: Tips first KubeCon Presentation (Pt. 2)

Part 2 about experiences from my first KubeCon presentation. Probably delay by 1w before approving.
Pre-review and collaboration with @kbarnard10 and @parispittman.
Note: needs fix for the link to part 1 after the first one has been released.
First part PR here: https://github.com/kubernetes/website/pull/10509

/retest

/lgtm
/approve
/hold cancel


dw sources

@csae8092 had a script to replace the shortened sources with the longer versions in note @type="source" @subtype="publ"; some didn't work because the script couldn't match, since there were two or more short sources in the note element. These have now been split to two or more note elements. Pls run script again to replace those with longer versions, thanks!

https://github.com/semantic-kraus/dw-data/commit/56819598852a0ff164d52724fca831bae451717b looks perfect!


Solidity watcher/runner package

Chatted with @0xOlias about the need for a generic way to watch Solidity contracts and run some redeploy script/command.
Might be nice to separate our deploy pipeline into a more generic package that has hooks, and then build the MUD-specific stuff on top of it.

+1. I've observed several folks building full-stack apps using Foundry (usually in a monorepo), and the experience has some rough edges.
Without any additional tooling, the most common workflow seems to be:

Manually reset the anvil instance (ctrl+c)
Run a forge script command to deploy the contracts, e.g. forge script script/01_Deploy.s.sol:Deploy --rpc-url http://<IP_ADDRESS>:8545 --broadcast
(Sometimes) Run another forge script command to add some test/seed data
Reload and continue development of downstream services (frontend or indexer)

Haven't given it much thought, but perhaps Foundry could support a forge dev command which:

Starts Anvil
Starts a local dev server that reloads on changes to project files (src/, script/, maybe test/)
On reload, resets Anvil and runs a specified deploy/setup script

This way, you can just save changes to Foundry project files and your downstream services should "just work".
One remaining question (important for Ponder, maybe less important for MUD) is some mechanism for the Anvil node to announce that a reload has occurred, so that downstream services know to invalidate any data that was previously fetched from this RPC URL.
See also @wagmi/cli which solves some related problems in this domain.


Site Relationship [Model]

Remove relationship from StudyDesign to Site. Make the relationship go from StudyDesign to ResearchOrganization.
See master ticket #262

incorporated changes and new model uploaded in 2.9 branch

Organizations should be optional - not mandatory -> change in UML.


datasette publish heroku fails without name provided

It fails with the following JSON traceback if the -n option isn't provided, despite the fact that the command line help says that's not needed for heroku publishes.

Traceback (most recent call last):
  File "/usr/local/bin/datasette", line 11, in <module>
    sys.exit(cli())
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/datasette/cli.py", line 265, in publish
    app_name = json.loads(create_output)["name"]
  File "/usr/local/Cellar/python/3.6.5/Frameworks/Python.framework/Versions/3.6/lib/python3.6/json/__init__.py", line 354, in loads
    return _default_decoder.decode(s)
  File "/usr/local/Cellar/python/3.6.5/Frameworks/Python.framework/Versions/3.6/lib/python3.6/json/decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/local/Cellar/python/3.6.5/Frameworks/Python.framework/Versions/3.6/lib/python3.6/json/decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Looks like this was a red herring actually, and heroku had a blip when I was testing it...


how to unsubscibe all topics

MqttTest.zip

`var mqttSubscribeOptions = mqttFactory.CreateSubscribeOptionsBuilder()
            .Build();
        //MqttClientUnsubscribeOptions mqttClientUnsubscribeOptions = new MqttClientUnsubscribeOptions();
        //mqttClientUnsubscribeOptions.TopicFilters = mqttClient.Options.
        //await mqttClient.UnsubscribeAsync(mqttClientUnsubscribeOptions);
        mqttSubscribeOptions.TopicFilters = mqttTopicFilters;

        var response = await mqttClient.SubscribeAsync(mqttSubscribeOptions, CancellationToken.None);`

when I click Subscribe several times，I received several identical messages at the same time, Can't it remove duplicate topics,or how to unsbuscribe all topics?

You can subsribe to the same topic multiple times if you want, but you shouldn't. And you can unsubscribe using UnsubscribeAsync, so what's the point?


Implement MainActivity.changeFragment()

Description
I have created a new method in MainActivity called changeFragment().
This is designed to allow fragment transitions where the parent activity is MainActivity, and therefore should eliminate the need of having one method each for fragment transitions (eg. moveToBarcodeEntry(), moveToPreferences()).
Note that a Fragment object is required as an argument, so this object will need to be created before using this method.
Related issues and discussion
N/A - minor refactor, so did not create
Checklist
Make sure you've done all the following (You can delete the checklist before submitting)

[x] Apply the AndroidStyle.xml style template to your code in Android Studio.
[x] Code is well documented
[ ] Include unit tests for new functionality [N/A - refactor]
[x] All user-visible strings are made translatable
[ ] Code passes Travis builds in your branch
[x] If you have multiple commits please combine them into one commit by squashing them.
[x] Read and understood the contribution guidelines .

Hey @ross-holloway94,
Your changes look good to me!


[Event Extension Request] Codeunit 90 - "Purch.-Post" - OnBeforeReleasePurchDoc

Dear ALAppExtensions team,
through this issue we would like to request the addition of parameter PreviewMode to event publisher OnBeforeReleasePurchDoc in Codeunit 90 "Purch.-Post" with the following signature:
[IntegrationEvent(false, false)]
local procedure OnBeforeReleasePurchDoc(var PurchHeader: Record "Purchase Header"; PreviewMode: Boolean);
begin
end;

which would be used in procedure ReleasePurchDocument as follows:
local procedure ReleasePurchDocument(var PurchHeader: Record "Purchase Header")
var
	PurchaseHeaderCopy: Record "Purchase Header";
	ReleasePurchaseDocument: Codeunit "Release Purchase Document";
	LinesWereModified: Boolean;
	PrevStatus: Enum "Purchase Document Status";
	IsHandled: Boolean;
begin
	with PurchHeader do begin
		if not (Status = Status::Open) or (Status = Status::"Pending Prepayment") then
			exit;

		PurchaseHeaderCopy := PurchHeader;
		PrevStatus := Status;

		// >>>> OLD >>>>
		// OnBeforeReleasePurchDoc(PurchHeader);
		// <<<< OLD <<<<
		// >>>> New >>>>
		OnBeforeReleasePurchDoc(PurchHeader, PreviewMode);
		// <<<< New <<<<
		LinesWereModified := ReleasePurchaseDocument.ReleasePurchaseHeader(PurchHeader, PreviewMode);
		if LinesWereModified then
			RefreshTempLines(PurchHeader, TempPurchLineGlobal);
		...

Thanks in advance!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10392.


chore: release deno-runtime 1.1.0

:robot: I have created a release beep boop
1.1.0 (2023-08-30)
Features

monorepo setup for lifecycle-operator, scheduler and runtimes (#1857) (84e243a)

Other

release deno-runtime 1.0.0 (#1975) (8df9ca4)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/mowies/lifecycle-controller/releases/tag/deno-runtime-v1.1.0 :sunflower:


[release-1.6] Update min k8 version required for Istio

This is an automated cherry-pick of #24134

Added a hold just to fix the preliminary.istio URL. This can be removed once we fix that.

@googlebot I consent


Python artifact build broken on upstream/master

Possibly caused by #10427?
https://grpc-testing.appspot.com/view/All/job/jtattermusch_build_artifacts_linux/6/testReport/(root)/build_artifacts/build_artifact_python_linux_x64_cp27_cp27m/
maybe we just need to switch the build to C99.
src/core/lib/support/env_linux.c: In function gpr_getenv:
src/core/lib/support/env_linux.c:63:3: error: for loop initial declarations are only allowed in C99 mode
   for (size_t i = 0; getenv_func == NULL && i < GPR_ARRAY_SIZE(names); i++) {
   ^
src/core/lib/support/env_linux.c:63:3: note: use option -std=c99 or -std=gnu99 to compile your code

Adding -std=c99 to here https://github.com/grpc/grpc/pull/10427/files#diff-2eeaed663bd0d25b7e608891384b7298R119 results in:
src/core/lib/support/time_posix.c:50:24: error: return type is an incomplete type
 static struct timespec timespec_from_gpr(gpr_timespec gts) {
                        ^
src/core/lib/support/time_posix.c: In function ‘timespec_from_gpr’:
src/core/lib/support/time_posix.c:51:19: error: storage size of ‘rv’ isn’t known
   struct timespec rv;
                   ^
src/core/lib/support/time_posix.c:58:3: warning: ‘return’ with a value, in function returning void [enabled by default]
   return rv;
   ^
src/core/lib/support/time_posix.c:51:19: warning: unused variable ‘rv’ [-Wunused-variable]
   struct timespec rv;
                   ^

On mac:
running build_ext
Traceback (most recent call last):
  File "setup.py", line 308, in <module>
    cmdclass=COMMAND_CLASS,
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/distutils/core.py", line 148, in setup
    dist.run_commands()
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/distutils/dist.py", line 955, in run_commands
    self.run_command(cmd)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/wheel/bdist_wheel.py", line 179, in run
    self.run_command('build')
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/distutils/cmd.py", line 313, in run_command
    self.distribution.run_command(command)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/distutils/command/build.py", line 135, in run
    self.run_command(cmd_name)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/distutils/cmd.py", line 313, in run_command
    self.distribution.run_command(command)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/setuptools/command/build_ext.py", line 49, in run
    _build_ext.run(self)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/Cython/Distutils/build_ext.py", line 164, in run
    _build_ext.build_ext.run(self)
  File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/distutils/command/build_ext.py", line 338, in run
    self.build_extensions()
  File "/private/var/folders/g4/3ppgfd8x4274zrnf_tf60_8h0000gn/T/pygrpc.4hhz0R/src/python/grpcio/commands.py", line 289, in build_extensions
    sys.stderr.write(make_err + '\n')
TypeError: can't concat bytes to str
+ rm -rf /var/folders/g4/3ppgfd8x4274zrnf_tf60_8h0000gn/T//pygrpc.4hhz0R


build in depth-k abstraction

To properly analyze the impact of depth-k abstraction, it should be in the implementation.

A function for depth-k abstraction is now available.


add environment variables for provider options

@rashmi43 Your PR is failing tests because of linting errors:
http://prow.purple-chesterfield.com/log?job=check_ibm-commonui-operator&id=1237779396035612673
You can install the following (all necessary) linting tools and run a make check locally to see if they are passing before committing.
https://github.com/IBM/ibm-commonui-operator/blob/master/docs/development.md

All tools except for mdl are running on my system.

/lgtm


Refresh Token revocation - all tokens for the associated subject are deleted instead the only one that was requested to revoke

I posted this https://github.com/IdentityServer/IdentityServer3/issues/3776 and given it is relevant to Identity Server 4 asking in this forum too.
In our scenario multiple clients (native mobile Xamarin applications) request JWT (short-living) and Refresh (long-living) tokens on the user login.
Both tokens securely stored in the application.
Tokens SubjectId is uniquely associated with the authenticated user (let's say it is userId).
Long-living Refresh token is later revoked on a user logout.
The problem: revocation deletes ALL refresh tokens that are associated with a refresh token to be revoked as oppose to deleting only the given refresh token.
This effectively logs a user off on ALL mobile devices where they are logged in.
I am realising this is a very edge use case, however would expect that revoke only should delete the given token or as alternatively offer an option to cascade as oppose always deleting ALL tokens for the related SubjectId.
Identity Server 3 code:
File: \IdentityServer3\CoreAndHosting\source\Core\Endpoints\Connect\RevocationEndpointController.cs
// revoke refresh token only if it belongs to client doing the request
private async Task RevokeRefreshTokenAsync(string handle, Client client)
{
var token = await _refreshTokens.GetAsync(handle);
if (token != null)
{
	if (token.ClientId == client.ClientId)
	{
		// ***************************
		// Issue: here we pass the subject only, so the underlying revoker RevokeAsync() will not have access to the original token that has to be deleted
		// and will futher delete all tokens associated with token.SubjectId
		// ***************************
		await _refreshTokens.RevokeAsync(token.SubjectId, token.ClientId);

Identity Server 4 code:
File: \Idsrv4\IdentityServer4-dev\src\IdentityServer4\ResponseHandling\TokenRevocationResponseGenerator.cs
// revoke refresh token only if it belongs to client doing the request
private async Task RevokeRefreshTokenAsync(string handle, Client client)
{
var token = await _refreshTokens.GetAsync(handle);
if (token != null)
{
	if (token.ClientId == client.ClientId)
	{
		// ***************************
		// Issue: here we pass the subject only, so the underlying revoker RevokeAsync() will not have access to the original token that has to be deleted
		// and will futher delete all tokens associated with token.SubjectId
		// ***************************
		await _refreshTokens.RevokeAsync(token.SubjectId, token.ClientId);
		await _tokenHandles.RevokeAsync(token.SubjectId, token.ClientId);
		await _events.RaiseTokenRevokedEventAsync(token.SubjectId, handle, Constants.TokenTypeHints.RefreshToken);
	}

Note that the standard https://tools.ietf.org/html/rfc7009 does not mandate revocation of other "siblings" refresh tokens for the giving SubjectId.
Can you please advise if the described behaviour and the implementation are intentional? If so, we can workaround.

Sorry - for the delay. Will look into it ASAP.

You are right - we are indeed removing all refresh tokens for a give subject ID / client ID combination
await RefreshTokenStore.RemoveRefreshTokensAsync(token.SubjectId, token.ClientId);
@brockallen can you remember why we did that?

Done with PR.

Thanks a log Guys, I am using your Identity Servers for the last 4 years and really appreciate your product quality, support,  prompt actions and all the rest.

Is there a possibility that 1.5.3 can be released with this fix, or is this planned to go out with 2.0.0? I'm using 1.5.2 currently and need this fix in for my app. I'm willing to make a PR to patches/1.5.2 with the same fix if that helps.

joining @chris-yoon90  request, currently we cannot upgrade to 2.0.0 and would like to have that fix in 1.5.x if possible

When will this fix be released? Which version?


feat(config): expose several typings as public apis

Summary

Expose several properties in ConfigSet

Expose JsonableValue and some json utils

Expose updateOutput function in compiler

TypeScript doesn't allow subclass to use parent class getters so we have to convert public getters to public properties, see https://github.com/microsoft/TypeScript/issues/338
Test plan

Green CI
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
N.A.

Pull Request Test Coverage Report for Build 6267

22 of 22 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 93.654%

Totals

Change from base Build 6266:
-0.04%

Covered Lines:
1018

Relevant Lines:
1047

💛  - Coveralls


Why can't the position of the first control be modified when the parent object is cont?

Why can't the position of the first control be modified when the parent object is cont?
in the test:
lv_obj_t * cont2 = lv_cont_create(lv_scr_act(), cont1);
lv_cont_set_fit(cont2, true, true);
lv_obj_t * obj2_1 = lv_label_create(cont2, NULL);
lv_obj_set_pos(obj2_1, 100, 30);
lv_label_set_text(obj2_1, "Short");

lv_obj_t * obj2_2 = lv_label_create(cont2, NULL);
lv_label_set_text(obj2_2, "A longer text");
lv_obj_set_pos(obj2_2, 180, 30);

The position of “Short”does not change?

Hi,
I updated the documentation of the container to explain it. See the "Notes" section here: https://littlevgl.com/object-types/container-lv_cont


shapefile parser option to exclude non-shapefile files

If the shapefile is a zip/folder, add an option to remove files that are not shp, prj, dbf, or shx.

in 1.0.9


How to hide a cursor?

I would like to hide a cursor if an editor is read only. Is there API for it?

Hmm, I don't think so. Only through CSS...

Any news on that? It would be nice if readonly mode actually looked like readonly mode to not confuse the people using it.

Read only is differentiated out-of-the-box by the cursor not blinking. Further customizations can be achieved via the cursorStyle editor option and CSS


Create distribution stage to ensure that -Pdistribution builds pass.

A test PR to see how much longer a -Pdistribution` build takes. Currently the distribution build is not tested, which resulted in a few issues with the build not being detected until the release process begain for 9.1.0.Alpha1

@ryanemerson What's missing here? Did CI run as expected? Latest build failed for other reasons.

@galderz Hmm I'm not sure tbh. The distribution field appears for the build, but it has an execution time of 19ms which can't be right. @slaskawi any ideas?

@ryanemerson Looks like the distribution step never ran because of when { master }. Wouldn't it be simpler to run the build step with -Pdistribution?

Yeah sure... that could work!