[Solved] How to change/specify Solidity linter [solc] compiler version in Visual Studio Code?

Hi may I know how to change my Solidity linter compiler version in Visual Studio Code(vscode)? OR Visual studio code how to specify solidity compiler version?
The only Solidity related extensions I have installed are solidity 0.0.38 by Juan Blanco, and Solidity Extended 3.0.2 by beaugunderson.
I have applied the User Settings in VS Code as the following but still get error:
compileUsingRemoteVersion" : "v0.4.22+commit.4cb486ee.Emscripten.clang",
"solidity.solhintRules": {
"extends": "default",
"rules": {
"avoid-throw": false,
"avoid-suicide": "error",
"avoid-sha3": "warn",
"indent": ["warn", 2]
}
},
"solidity.soliumRules": {
"no-inline-assembly": 5,
"imports-on-top": 0,
"variable-declarations": 0,
"indentation": ["error",2],
"quotes": ["error","double"]
},
Now I've also install Solidity globally in my Linux: $ sudo npm install -g solc Then I've got this from the terminal: /home/userXYZ/.npm-global/bin/solcjs -> /home/userXYZ/.npm-global/lib/node_modules/solc/solcjs +<EMAIL_ADDRESS>Then changed User Setting: "solidity.compileUsingRemoteVersion" : "latest"
Then re-started VS Code, still I am getting this error... Please help. Thank you

[Solved] Thank you Juan Blanco for his advice. So the main issue is that the forked extension(Solidity Extended by beaugunderson) was highjacking this one(solidity by Juan Blanco) hence I had the old version of the compiler 0.4.17 but not the latest version 0.4.23.

I have the same issue and using solidity by Juan Blanco 0.0.45 (No Solidity extended).

I don't believe this bug is solved -- I'm using this VSCode extension and need to downgrade the Solidity compiler, but setting the "solidity.compileUsingRemoteVersion" option to "v0.4.11" or "v0.4.11+commit.68ef581" didn't change the current compiler in that pragma error mouseover.  Is there another option I need to supply for the alternate compiler version to take over?

@john-osullivan Hi John, I used to have the same problem. But then I realize the solc version error was not given by Juan's extension but rather another vs-code extension called "Solidity Extended" that I installed by chance.
Just by deactivating the latter, And only use Juan's solve my problem. ( my guess is that the rule of which solc is being used has overlap or being overwritten in the "extended" extension)

Thanks for the tip, @AlexXiong97 .  I'm not using any other extensions related to Solidity, but that's a good tip -- I'll do more exploring to see if the setting is getting overwritten elsewhere.

I tried npm i solc and I tried setting "solidity.compileUsingRemoteVersion": "latest" but I'm still seeing the error. I'm only using this extension + solidity-solhint. Please help?

@cocacrave  having solidity solhint installed will cause problems as both can use solhint, that might be the issue.

@juanfranblanco that was it. thanks!

Removing Solidity Extended extension solved it for me.

removing other extensions or change the solc version in terminal didn't help me

The Same Problem

I have done npm install -i<EMAIL_ADDRESS>and it fixed the issue

After doing this. Restart your VSCode.

If you right click you can select a different compiler version as part of your global settings (or workspace). You can also install as per @jchittoda a local version of solc in the same folder that will take preference.

I had the same problem。After I remove the "Solidity Extended"and restart vs code, it works normally.

Source file requires different compiler version (current compiler is 0.8.6+commit.11564f7e.Emscripten.clang) — note that nightly builds are considered to be strictly less than the released version
I had the same issue, After I remove the "Solidity Extended" and restart vs code, it works normally.

If you right click you can select a different compiler version as part of your global settings (or workspace). You can also install as per @jchittoda a local version of solc in the same folder that will take preference.

This solved the issue for me.

In VSCode:

CTRL + SHIFT + P (Windows)
Type soliditychangecompilerlocal --> Change Default Workspace Compiler ...  [PRESS ENTER]
Select compiler [PRESS ENTER]:

.vscode/settings.json is created:
{
    "solidity.compileUsingRemoteVersion": "v0.8.7+commit.e28d00a7"
}

This is a setting of "solidity" package


Fix ppo

Fixes https://github.com/facebookresearch/rl/issues/92

Thanks for pointing and solving this @shagunsodhani!
I should integrate tests of the training scripts in the CI but it is not yet clear to me how that could be done efficiently, and I don't test them all every time I commit something...
I'll prioritize that!

Closing as per #95


sc.pp.filter_genes() gives error on copied data

Hi,
I'm running sc.pp.filter_genes(adata_test, min_cells=50) and getting the error below.
This is run on a copied anndata object, and I haven't been able to reproduce is on e.g., sc.datasets.pbmc3k. I wonder if you have any idea what might be going wrong here though, so I can start debugging a bit smarter.
All I want to do is filter out a few genes before using diffxpy. Oh, and my versions are:

scanpy==1.4.3+116.g0075c62 anndata==0.6.22.post1+29.g16615d7 umap==0.3.9 numpy==1.16.2 scipy==1.3.0 pandas==0.24.1 scikit-learn==0.20.2 statsmodels==0.10.0rc2 python-igraph==0.7.1 louvain==0.6.1

That should be fairly up to date.
filtered out 592 genes that are detectedin less than 50 cells
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-16-9a4043f3121d> in <module>
      4
      5 # Filter out genes to reduce multiple testing burden
----> 6 sc.pp.filter_genes(adata_test, min_cells=50)
      7 print(f'Testing {adata_test.n_vars} genes...')
      8 print("")

~/new_scanpy/scanpy/scanpy/preprocessing/_simple.py in filter_genes(data, min_counts, min_cells, max_counts, max_cells, inplace, copy)
    218         else:
    219             adata.var['n_cells'] = number
--> 220         adata._inplace_subset_var(gene_subset)
    221         return adata if copy else None
    222

~/new_anndata/anndata/anndata/core/anndata.py in _inplace_subset_var(self, index)
   1379         Same as ``adata = adata[:, index]``, but inplace.
   1380         """
-> 1381         adata_subset = self[:, index].copy()
   1382         self._init_as_actual(adata_subset, dtype=self._X.dtype)
   1383

~/new_anndata/anndata/anndata/core/anndata.py in __getitem__(self, index)
   1248         """Returns a sliced view of the object."""
   1249         oidx, vidx = self._normalize_indices(index)
-> 1250         return AnnData(self, oidx=oidx, vidx=vidx, asview=True)
   1251
   1252     def _remove_unused_categories(self, df_full, df_sub, uns):

~/new_anndata/anndata/anndata/core/anndata.py in __init__(self, X, obs, var, uns, obsm, varm, layers, raw, dtype, shape, filename, filemode, asview, oidx, vidx)
    602             if not isinstance(X, AnnData):
    603                 raise ValueError('`X` has to be an AnnData object.')
--> 604             self._init_as_view(X, oidx, vidx)
    605         else:
    606             self._init_as_actual(

~/new_anndata/anndata/anndata/core/anndata.py in _init_as_view(self, adata_ref, oidx, vidx)
    664         if adata_ref._raw is not None:
    665             # slicing along variables axis is ignored
--> 666             self._raw = adata_ref.raw[oidx]
    667         else:
    668             self._raw = None

~/new_anndata/anndata/anndata/core/anndata.py in __getitem__(self, index)
    360         new = Raw(self._adata, X=X, var=var)
    361         if self._varm is not None:
--> 362             new._varm = self._varm._view(self, (vidx,))
    363         return new
    364

AttributeError: 'numpy.ndarray' object has no attribute '_view'

I've been playing around with this a bit, and it seems it has to do with raw being set. I can recreate the same error with:
adata_test = adata[[d != 'Patient1' for d in adata.obs['Donor']]].copy()
but if I set adata.raw = None it all magically works. E.g.,
test = adata.copy()
test.raw = None
adata_test = test[[d != 'Patient1' for d in test.obs['Donor']]].copy()

Does slicing somehow not work when raw is set? Again, I haven't been able to reproduce with sc.datasets.pbmc68k(). No idea what makes my dataset so different...

I see what's happening with the first error, you've somehow got a numpy array for adata.raw.varm where it should be an anndata.core.alignedmapping.AxisArray. Any idea how that would have happened? Is it possible this anndata was saved via pickle?
An aside on the code, I'm not sure it actually makes sense to subset .raw.varm when the whole object is being subset. The var dimension of .raw should be disconnected from the var dimension of adata.

It is indeed a numpy.ndarray. I saved some diffxpy objects via pickle, but never the adata object. That was saved via anndata.write() and loaded via sc.read().
I actually don't care about adata.raw in the particular analysis I'm doing at all. So it works for me to just delete it, but this seems like a more general problem. I would agree .raw could stay unsliced when slicing the main object.

@ivirshup I now have another case where I load a previously saved adata object and adata.varm is a dict. At the moment, every time I slice an object I have to do:
adata.layers['raw'] = adata.raw.X
adata.raw = None

[slice anndata object]

adata_raw = adata.copy()
adata_raw.X = adata.layers['raw']
adata.raw = adata_raw

del adata_raw
del adata.layers['raw']

Is there a quicker way to cast adata.raw.varm to anndata.core.alignedmapping.AxisArrays?

Yeah, this was raised in #192, and I've got a fix for it in #167. My sense is this issue would be separate from having an ndarray in .raw.varm. These are different files, yeah?
Right now, I think I'd just have to recommend not using .raw.varm if possible until I merge #167. Does scanpy have any functions which use .raw.varm?
If needed we could do a bug fix release to fix this, since it's an easy fix. I ended up going with adding it to my pull request because it was much easier to test with some testing infrastructure I've been adding (tests added in be1fc820a6b55e4d).

These are different files. They are the same dataset though. One file is after I saved an analysis where I thought I had fixed the adata.raw.varm issue by reinstantiating raw as described above. It was then loaded as dict as you describe in #192.
I think scanpy stores PCs in adata.varm after sc.pp.pca(). Then instantiating raw by adata.raw = adata transfers that to adata.raw.varm. I have PCs stored there and never put anything in varm actively. Annoyingly you can't set adata.raw.varm = None either (except via a new adata object).
How far is #167 from being merged? For now I guess I can work around.

I think you could do adata.raw._varm = None as a work around for now.
#167 feels close, baring anything horrible coming up in review.

Ah, cool. So I can set the attribute by referencing raw._varm rather than raw.varm?
Looking forward to the rework! Feels like views and IO playing well together will be a major upgrade.

Fixed in https://github.com/scverse/anndata/pull/167 I assume


fix: Upgrade code detection

I replaced the system used for code detection with a different system that uses code-specific regex patterns/expressions.
The previous system was prone to false positives.

Thank you, can you share some of your tests here? It would be nice to paste a couple of examples in your PR description that demonstrate:

code that will be detected, which wasn't previously detected
english that won't be detected, that was previously detected incorrectly

Things that was previously falsely detected as code, but were correctly detected now:
`Amber ( Inventory Personnel )
Search function for PRODUCT  [(search product you need) and (search Expired product)]
Replace or ament BALANCE QUANTITY, QUANTITY  (for Old Product)
Delete expired product ( delete only quantity and balance quantity )
[Quantity must be 0 for expiry products, 0 to represent delete]
Exit ( exit inventory )
This is the notes I've made`
PA seems prone to false positives, for example: Lines ending with (semi)colons Text in parenthesies (be careful with them)
(wow) (no wonder there are so many false positives)
`My process for this question:
-16+8+4+1 = -3 (Exponent)
Next I moved the decimal point 3 places to the right
Answer = 0101.010 = 5.25 in denary
The correct answer is: 0.08203125 (However, I do not understand how to get to this answer)
`
How to optimize the following Boolean expression using a map: X'Z' + YZ' + XYZ
`i will do it!!
(i am lying)`
Hi, so i obtained the values: 27 13 34 85 59 91 50 38 as my step 1, in which 34 is sorted. However, i only have 2 values 27 and 13. How do i perform another middle of 3?
Theorem 4.2: All recursive algorithms must have a base case.   Proof: See Theorem 4.2.
1: searches for all occurrences of 1 (1) 2: searches for all occurrences of 2 (2) 2: searches for all occurrences of 2 (2) 4: searches for all occurrences of 4 (2) 4: searches for all occurrences of 4 (2) 2 + 2 + 2 + 2 + 1 = 9
`would you break it like:
$2(3^k) + 3^k - 6 > 4k + 4$`
`so from what i'm getting is:
learn the math (discrete, stats & probability, algebra, calc (for fun))`
what if you do this instead: Inside of recv you read the new frame. If you're not processing a frame you start transforming it async without awaiting (and set the boolean to true). Then return the last processed frame. The job of transform_frame is to set last_frame inside of itself (so it doesn't return anything). Or, if there is some kind of similar thing to .then as in JS, do that instead
if you want to develop native windows apps then i guess the following might be a decent option clang/llvm as compiler (works just like gcc) clangd extension for c language support in vscode ninja for building (replaces make) cmake + cmake extension in vscode for c project management

Thanks again for your effort here! Unfortunately we've decided to simply delete the feature altogether, because it is generally just annoying and not helpful. This PR would have fixed a lot false-positives where non-code was detected as code, but it does not meaningfully improve the correct-positives where code is detected as code. Doing so is a difficult challenge, and is better solved by language-specific rules in the ./code_detection/ directory.


View / Mutable View support in rblas integration

Only supporting c-order array views so far (f-order support was dropped from this PR). The concern with f-order is that the rblas library that we integrate with doesn't check the ordering for each used matrix (only one of them), so it's easy to make an error if matrices are mixed.

looking forward to the immutable blas views on arrays which are already continuous !
was about to open an issue asking for this as it will solve some issues for me.

There are some challenges, and that may mean breaking changes down the line.
cblas API takes a rowmajor/colmajor option, but it is not per matrix but must be the same for them all. We can't enforce that without re-wrapping the blas operations.
blas does not support overlapping inputs! This is not easy to enforce in the type system. I.e. A times A  would be overlapping if A is the same array view.

i see !

blas does not support overlapping inputs!
too bad. i wasn't aware.

This is not easy to enforce in the type system. I.e. A times A would be overlapping if A is the same array view.

while not necessarily elegant the current mutable-only blas views enforce this as mutable borrows are exclusive.
maybe a reason to keep it that way in a documented way ?

I am not sure where I read about overlapping read-only inputs. It does seem a bit strict when you think over it. That said, we can easily create an input that next to overlaps with the output (if the input = even rows of a matrix, output = odd rows of a matrix). Those are disjoint in current ndarray and have disjoint elements, but some blas implementation may consider them overlapping.
Deserves experimentation!
@snd What do you think about allowing this interface in the PR? It would allow both row major and column major arrays to be used without copying. However, the user must be careful not to try to multiply (for example) matrices with mismatching order.

i apologize for the delay. i've got a lot going on at the moment.

I am not sure where I read about overlapping read-only inputs.

i'd be interested to know.

It does seem a bit strict when you think over it.

i agree.
i did some research. my current understanding is that it's ok for readonly/input arguments to alias one another. it is not allowed (undefined behaviour) for a readonly/input argument to alias a modifyable/output argument.
that would be ideal !
the exclusivity of mutable borrows enforces that an input argument can't also be an output argument.

What do you think about allowing this interface in the PR?

i'd say: go for it ! but add some tests to verify this.

However, the user must be careful not to try to multiply (for example) matrices with mismatching order.

is there a reason why you can't add assertions for that ?
if not i'd add some assertions with helpful error messages.
instead of having to be careful i'd prefer it blowing asap with a helpful error message rather than getting some confusing blas error or (even worse) no error at all.
what do you think of modeling row-major and col-major via the type system ?
then it should be possible to verify matching order at compile time via the type system.
just an idea.

cblas API takes a rowmajor/colmajor option, but it is not per matrix but must be the same for them all.

am i correct in assuming that it must be the same for all matrixes involved in a single call to BLAS or must it be the same for all matrixes globally ?
here are my sources:
GSL which builds on BLAS:

The library assumes that arrays, vectors and matrices passed as modifiable arguments are not aliased and do not overlap with each other
If the arguments will not be modified (for example, if a function prototype declares them as const arguments) then overlapping or aliased memory regions can be safely used.

http://www.csse.uwa.edu.au/programming/gsl-1.0/gsl-ref_2.html
LAPACK which builds on BLAS:

Unless specified otherwise, only input arguments (that is, scalars passed by values and arrays specified with the const qualifier) may be legally aliased on a call to C interface to LAPACK.

http://www.netlib.org/lapack/lapacke.html#_aliasing_of_arguments
gonum which builds on BLAS:

It is forbidden for the modified data region of the receiver to overlap the used data area of the input arguments.

https://godoc.org/github.com/gonum/matrix/cmat128#hdr-Element_Aliasing

Thank you for looking into this.
cblas functions take one layout (ordering) parameter per operation. For example for cblas_dgemm (matrix multiplication) it regards the two input matrices and the output matrix of that operation.
This interface was to interoperate with rblas specifically, and that's why I can't enforce that the inputs have the same ordering: rblas' operations don't enforce it.
In the long run we may want to add our own high level blas-based operations.

If you're interested, a tricky question remains, imagine a 4 x 4 matrix in row major order. The columns of the matrix are technically both F-order and C-order since they are one dimensional (but with a leading dimension of 4).
Anyway, adjacent columns do not overlap, but their memory interleaves very closely. Is this compatible with blas? :smile:

View support merged, without f-order support yet (since that remains to be clarified).

Follow up issues:

#78
#79


fix: Removed extra header and fixed typos

Description
Little readme fixes
Related Issues
I also made this PR, but no issues: https://github.com/firebase/flutterfire/pull/10247
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (melos run analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

I'm building authentication for my app and finding typos in the docs as I go.


Add Onboarding to the app

Add onboarding to the app
Add request of notification permissions for iOS
Add VOST logo as an asset
NOTE: The texts are still as Strings since we don't have the translations to english

Pushed changes

@miguelpruivo if you don't mind since you have the OK I'll merge this!


Update Prism requirement to v0.28

Motivation
Update our Prism requirement to v0.28 or higher. Version v0.28 includes the new ASCIISource with performance optimization for sources that do not contain multibyte characters, which fixes the performance regression we introduced in semantic highlighting.
Implementation

Updated Prism and RBI
Re-generated gem RBIs
Fixed errors related to the restructuring of constant paths

Before, each part of a constant path was also a constant read. After this version, only the parent parts of the constant path may be constant reads, the main name of the constant is embedded inside the constant path node.
# Before
# This would be 3 constant read nodes to compose the constant path

# After
# This is now 2 constant read nodes and the name `C` is a part of the overall constant path
A::B::C

Automated Tests
Fixed a few tests.

The upgrade ended up uncovering some other Windows errors and I was trying to get CI to pass. If you prefer, I can split that into a separate PR, but the changes are necessary.

Ok, yeah I'd say we should split them since there's quite a lot of changes. I'll limit my review to the Ruby part.

Split the other part, which needs to be merged first https://github.com/Shopify/ruby-lsp/pull/2033.


Milestone/Project 4 | junction table for visits 🎫

🚩I have made the following changes to complete Vet Clinic - Milestone/Project 4
Here is a summary of what has been done
Type of files changed/added

In this new Branch, I have ✏MODIFIED✏ the succeeding *📂 folders and/or **📄 files to complete this milestone's requirements.
*📂 root/

**📄 README.md ✏

I have checked done the [x] add "join table" for visits checkbox.

**📄 data.sql ✏

I have inserted data into the vets, specializations and species tables.

**📄 queries.sql ✏

New queries have been made to the animals, owners and species tables.

**📄 schema.sql ✏

I have added new tables to the schema:

vets

Junction Tables:

specializations
visits

📸 Screenshots of queries made to the database

SQL Query Collection- Click to expand!

Thank you for taking the time to review this PR ⭐

If you require additional information or have any questions, please contact me on Slack as Arturo (Arthur) Emanuel Guerra Iturres. I'll be happy to help you out. 🎯

Hi @elisha2kyakpo1! how are you?,
Thank you for the quick approval of my PR. I was just wondering if there were any specific comments or feedback you had, as the endorsement was very fast. I'd appreciate any guidance you can provide to improve my code in the future 🤠.


Move to linux dev env

Moving to new linux workstation and making any needed changes to work,

in version 0.6.3


fix: tweak spring config for animation

This is configuration used by stack navigator. Feels very similar to what reddit app has.

@osdnk can you check if this feels "natural" on iOS?


fix: grab values from accept_channel message using stash

Some fields in the accept_channel message and the Block function were set to hardcoded values specific to a certain implementation. Using stash here sends back the values required by an implementation. This makes Lnprototest more versatile and allows it to support different implementations.

Sure! I will add a description in this PR as well.


[SPARK-40124][SQL][TEST][3.2] Update TPCDS v1.4 q32 for Plan Stability tests

What changes were proposed in this pull request?
This is port of SPARK-40124 to Spark 3.2. Fix query 32 for TPCDS v1.4
Why are the changes needed?
Current q32.sql seems to be wrong. It is just selection 1.
Reference for query template: https://github.com/databricks/tpcds-kit/blob/eff5de2c30337b71cc0dc1976147742d2c65d378/query_templates/query32.tpl#L41
Does this PR introduce any user-facing change?
No
How was this patch tested?
Test change only

cc: @kazuyukitanimura @dongjoon-hyun

Can one of the admins verify this patch?

Merged to branch-3.2.


css, charts and maps does not work when integrated with the react in dev mode

css, charts, and maps do not work when integrated with the react in dev mode

Hey @21pranav, that's because the JavaScript included is not made for React-based applications. If you wish to use Bootstrap in combination with React, we recommend the React Bootstrap library.
Closing this as this has nothing to do with the AdminKit package. We might launch a React-based version in the future.


feat: Allow the user to sign up with OAuth

Allow the user to sign up when he tries to use OAuth to login but there's no account linked to the OAuth
Deployed at: https://casdoor.leviatan.cn/login/oauth/authorize?client_id=00c5f3410de2ebf737e4&response_type=code&redirect_uri=https://www.leviatan.cn&scope=read&state=casdoor
I closed #102 by an incorrect operation, I fixed problems at this PR.

Reuesd signup page
I don't know why, but my email and SMS provider config doesn't work, so I closed the Verification Code check only on this test server, you can enter any character in the email code and phone code to pass the verification.
Reformat by IDE

Signed-off-by: WindSpiritSR<EMAIL_ADDRESS>

@Kininaru plz review.

@WindSpiritSR

@hsluoyz Deployed at: https://casdoor.leviatan.cn/login/oauth/authorize?client_id=b2f633c65937f2b478f8&response_type=code&redirect_uri=https://www.leviatan.cn/&scope=read&state=casdoor

@WindSpiritSR redirect uri seems not right


obs-studio: 28.1.2 -> 29.0.0

Changelog: https://github.com/obsproject/obs-studio/releases/tag/29.0.0
Description of changes

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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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 209687 run on x86_64-linux 1

1 package marked as broken and skipped:

obs-studio-plugins.obs-hyperion

12 packages built:

obs-studio
obs-studio-plugins.droidcam-obs
obs-studio-plugins.input-overlay
obs-studio-plugins.looking-glass-obs
obs-studio-plugins.obs-gstreamer
obs-studio-plugins.obs-move-transition
obs-studio-plugins.obs-multi-rtmp
obs-studio-plugins.obs-nvfbc
obs-studio-plugins.obs-pipewire-audio-capture
obs-studio-plugins.obs-source-record
obs-studio-plugins.obs-vkcapture
obs-studio-plugins.wlrobs

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/807


Improve ProviderConfigRed

In the documentation, it is mentioned that additional fields supported by the upstream Terraform provider are supported.
I think that having the JSON structure to the connection secret is quite limiting. When I deploy Keycloak via the bitnami helm chart, for example, I need to provide a secret with the password. Having this in json format poses some challenges as to extracting and processing that data to format it nicely
I see that the Terraform provider also supports environment variables. That would be much better UX in allowing me to pick and choose keys from multiple sources (configmaps/secrets).
Is this supported?

Make an example please.
One thing you have to keep in mind is, that a crossplane provider potentially  can configure multiple keycloak instances.
So a provider config is used for exactly one instance of keycloak. Just passing environment variables to the provider will not be good enough

Sure thing!
I didn't mean injecting env vars into the provider deployment itself. I was talking more about restructuring the provider config credential secret into individual keys as opposed to a single json value, and I was referencing the names of the env vars to be used as keys as a possibility.
Currently, it is defined like this:
---
apiVersion: keycloak.crossplane.io/v1beta1
kind: ProviderConfig
metadata:
  name: keycloak-provider-config
spec:
  credentials:
    source: Secret
    secretRef:
      name: keycloak-credentials
      key: credentials
      namespace: crossplane-system
---
apiVersion: v1
kind: Secret
metadata:
  name: keycloak-credentials
  namespace: crossplane-system
  labels:
    type: provider-credentials
type: Opaque
stringData:
  credentials: |
    {
      "client_id":"admin-cli",
      "username": "admin",
      "password": "admin",
      "url": "https://keycloak.example.com",
      "base_path": "/auth",
      "realm": "master"
    }

However what I was proposing is the ability to define it either like this:
---
apiVersion: v1
kind: Secret
metadata:
  name: keycloak-credentials
  namespace: crossplane-system
  labels:
    type: provider-credentials
type: Opaque
stringData:
      client_id: "admin-cli"
      username: "admin"
      password: "admin"
      url: "https://keycloak.example.com"
      base_path: "/auth"
      realm: "master"

Or like this:
---
apiVersion: v1
kind: Secret
metadata:
  name: keycloak-credentials
  namespace: crossplane-system
  labels:
    type: provider-credentials
type: Opaque
stringData:
      KEYCLOAK_CLIENT_ID: "admin-cli"
      KEYCLOAK_USER: "admin"
      KEYCLOAK_PASSWORD: "admin"
      KEYCLOAK_URL: "https://keycloak.example.com"
      KEYCLOAK_BASE_PATH: "/auth"
      KEYCLOAK_REALM: "master

And then rearrange this data into the required format at runtime.
Additionally, sourcing these pieces of information from more than one source could be helpful. For example, having a configMap with the url, client id, realm name and base path and then a secret just with the password or client secret.


⚠ Bofrost.fr has degraded performance

In 8f39937, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 911 ms

Resolved: Bofrost.fr performance has improved in 051ebdd.


bug: Uninstalling a(ny) plugin will reinstall the plugin on next load.

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.11.0-dev-17+g48c2ad5d2d
Operating system/version
Win10 WSL2 ArchLinux.
Describe the bug
Uninstallaing packages (with 'x' in :Lazy) will mark them as 'Not installed':

but on next nvim startup, they are reinstalled:

Steps To Reproduce

Uninstall any plugin via :Lazy
Restart Neovim
Plugins reinstall themselves.

Expected Behavior
Uninstalling should remove them permanently.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  "folke/LazyVim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

That's not how you uninstall plugins. You have to remove the spec from your configuration if you installed it yourself or disable the LazyExtra if you installed via LazyExtras or disable the plugins as per the docs if it's a core LazyVim plugin.


use an environment variable to select port in launch.json

Similar to https://github.com/microsoft/vscode-java-debug/issues/962 I would to like use an environment variable DEBUG_PORT to set the port on which the python debugger is listening.
Now it does not seem possible because the port variable in launch.json needs to be a number. If I try setting an env variable, as in the code below, I get the following error:
}
"name": "Py listen",
"type": "python",
"request": "attach",
"listen": {
"host": "localhost",
"port": "${env:DEBUG_PORT}"
}

@int19h debugpy supports string for port numbers right?

Hi there, any news about this? Thank you

Hi, is it possible to know if this is currently supported? Thank you for your time

We don't support it currently, but it would be a simple change. In fact, I think debugpy might as well support strings for all scalar properties, to enable substitutions everywhere.

If you point me to the right part of the code, I could submit a PR. Best

Awesome! We have a central place where type checking & conversions are performed on JSON being parsed; the code that does the actual parsing then calls into that while specifying the types it expects to receive, and various other constraints (e.g. optional or not).
I think it would be the best to tweak the type checker for numbers to allow (convertible) string values - and automatically convert them to the corresponding number before returning. This way we also get support for e.g. 64-bit integers. The relevant code is here:
https://github.com/microsoft/debugpy/blob/eaadb6c681d5375dae26093b93d239c019f0d2bc/src/debugpy/common/json.py#L106-L112
It just needs a branch for when validation without conversion fails, and one of the items in classinfo is a subtype of numbers.Number. This should cover your specific case, but it would also be nice to do the same for bool, on the oft chance that somebody might need to specify them via env vars also.


Format to one transformation per line.

All the solutions except for one follow the pattern of applying one transformation per line. This reformats the one outlier function and makes it easier to parse.

@michaelhodel do you mind approving this pull request?
Thanks!
Matt

thanks for spotting!


Conflit with symfony/console version from symfony/flex

Hi,
Since this package requires symfony/orm 2.5.11 which requires symfony/console ~2.5|~3.0 this package seems not compatible with the new symfony/flex that requires symfony/console 4.0.
Or maybe I'm doing something wrong.
Thanks !

I see this also, in the master branch of Doctrine the dependency is already set to "symfony/console": "~3.0|~4.0", so I guess it's a matter of time (2.6.0?) that this will be updated.
I did make a PR for allowing console 4.0 as well, but I'm not sure if it's ok for them. You can follow progress here: https://github.com/doctrine/doctrine2/pull/6785

Show your support here doctrine/doctrine2#6733. Closing this issue as there is unfortunately nothing we can do on our side.


High-pitched noise in the background when using old GPUs

Previously discussed here: https://github.com/yl4579/StyleTTS2/issues/1#issuecomment-1746083183
The model produces some high-pitched noise in the background when I use my old GPU for inference (NVIDIA Quadro P5000, Driver Version: 515.105.01, CUDA Version: 11.7)
Audio examples:

Wav from the demo, the voice is very clear - https://voca.ro/1cJK3PM67Flj
Same text generated with the inference notebook, with noticeable high-pitched noise in the background - https://voca.ro/1elSCh8VuT79

I solved this problem by switching to CPU device, so this issue is just for reference, as asked by the author.
Thank you for your work!

I'm gonna pin this in case someone else has similar problems. I don't know how to deal with this because I can't reproduce this problem with the oldest GPU I have access to now (GTX 1060).

The same problem happens to me, and my GPU is A100. and using the CPU to infer cannot help me, the noise is still there by using the CPU.

@ruby11dog could you please share the details to reproduce this problem? It seems it’s not related to the version of GPUs then?

@ruby11dog could you please share the details to reproduce this problem? It seems it’s not related to the version of GPUs then?

The noise will appear when inferencing with your pretrained model "epoch_2nd_00100.pth". But the noise seems to fade away with model i trained myself along with epoch increase in second stage.
Here is my related python package version:  torch:2.1.0

So weird, I tried in Colab (T4, V100 and A100) without specifying any version of the libraries it works perfectly fine: https://colab.research.google.com/drive/1k5OqSp8a-x-27xlaWr2kZh_9F9aBh39K
I'm really wondering what is the reason behind this problem. It doesn't seem like it's just the GPU version though.

I have just ran into the same problem after training the model on the cloud using 4xA40 GPU's. I did inference both locally (RTX 3060 + CPU) and via the cloud computer (4xA40 + CPU). Doing inference on the cloud works fine without any background pitches. However, running it locally produces the background pitches (also when running the models on CPU, both on the cloud and locally).
After doing some investigation, it seems that the sampler is the culprit. Since it produces random output and my test is running either on the cloud or locally, it's not possible to set a seed producing similar outputs within the sampler.
A quick solution for testing the bug was to obtain the output of the sampler in the cloud and copy this output to my local pc. This output is then used at the local pc for further inference. Hereafter, the background pitch was gone and the sound was exactly produced as it should.
s_pred = sampler(noise,
              embedding=bert_dur[0].unsqueeze(0), num_steps=diffusion_steps,
              embedding_scale=embedding_scale).squeeze(0)

# Save torch tensor on the cloud
# torch.save(s_pred, 'sampled_tensor.pt')

# Obtaint the tensor via rsync and load it locally
# s_pred = torch.load('sampled_tensor.pt')

At the moment, I don't have time to look into the sampler, but I think closer inspection of the sampler could lead to fixing this bug.

After more investigation, I have solved the problem for myself. The problem for me was the sigma_data in the config. In the default config, it is set to 0.2. During training, this variable changes and is written to a new config file, which is stored in your Models folder. Doing inference with the default config file, I obtained the high pitch. Using the config file which is obtained during training, the pitch was gone and the sound was good. So this is the solution that works for me.
Btw, @yl4579 Thanks you for your great work, it's really awesome that you produced this and made your code open source!

As per my investigation the Problem is not with GPUs , its because of Diffusion Steps  used in inferencing, reducing the diffusion steps removed the screeching  noises completely.


Fix Windows installation

Checklist

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

I was trying to bring up log4cxx-feedstock for Windows build, and ran into the incomplete Windows installation problem. So I decided to attempt to continue #4 work but switch to use CMake way for Windows build.

Added CMakeLists.txt for apr-iconv since it currently doesn't have CMake support.
Dropped Visual Studio 2008 build for Windows because expat for msvc2018 conda-forge packages only comes with debug library (expatd.dll), and it doesn't play well with find_package(expat).
Added more file verification checks.
Added myself as co-maintainer.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Feature/design fixes

Refactored some of the front-end code.
Needs to be tested and code-reviewed before deployed to production.

Looks good, double checked the code and tested visually every page, nothing seems broken expect for the api doc link in the task view page which appear in front of the side-bar

fixed now


When some component's DOM element transitions from "empty" to "some content", it goes to the parent element instead

Something strange is happening when a DOM element inside a component transitions from "empty" to "some content": that "some content" is appended to the parent instead of being appended to the element.
Here's a simple CodePen: https://codepen.io/rdromao/pen/MWWPrGz
I think this line:
https://github.com/cferdinandi/reef/blob/ce0d8cc52c69a6badeacafc7188e6c06eeb64fd9/src/js/reef/reef.js#L404
... should be:
domMap[index].node.appendChild(fragment)
Right? 🤔

@rdromao I experienced the same bug. Thank you for the provided fix, it works for me ;-)

No PR ?

Fixed with v4.1.6. Thanks!


MapReduce

[x] Slides:
[x] Handbook:
[x] Integrated into handbook:

MapReduce contains Spark, Hadoop and Twister2 sections.
https://github.com/cloudmesh/book/blob/master/section/mapreduce/spark-lectures.tex
https://github.com/cloudmesh/book/blob/master/section/mapreduce/hadoop-lectures.tex
https://github.com/cloudmesh/book/blob/master/section/mapreduce/hadoop-installation.tex
https://github.com/cloudmesh/book/blob/master/section/mapreduce/twister-installation.tex
https://github.com/cloudmesh/book/blob/master/section/mapreduce/spark-streaming.tex
These are the links containing slides and material bound to the book. These are already in the book.


undefined method `this'  on Ruby 2.3.1

How to reproduce:

Install scss_lint on Ruby 2.3.1 from RVM
Create file.scss:

   img {
     max-width: 100%;
     vertical-align: middle;
   }

Lint it
  $ scss-lint file.scss

What happens: error is thrown
/Users/ain/.rvm/rubies/ruby-2.3.1/lib/ruby/2.3.0/rubygems/specification.rb:2158:in `method_missing': undefined method `this' for #<Gem::Specification:0x3fc5f5888e34 scss_lint-0.49.0> (NoMethodError)
	from /Users/ain/.rvm/rubies/ruby-2.3.1/lib/ruby/2.3.0/rubygems/specification.rb:1057:in `find_active_stub_by_path'
	from /Users/ain/.rvm/rubies/ruby-2.3.1/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:64:in `require'
	from /Users/ain/.rvm/gems/ruby-2.3.1/gems/scss_lint-0.49.0/bin/scss-lint:3:in `<top (required)>'
	from /Users/ain/.rvm/gems/ruby-2.3.1/bin/scss-lint:23:in `load'
	from /Users/ain/.rvm/gems/ruby-2.3.1/bin/scss-lint:23:in `<main>'
	from /Users/ain/.rvm/gems/ruby-2.3.1/bin/ruby_executable_hooks:15:in `eval'
	from /Users/ain/.rvm/gems/ruby-2.3.1/bin/ruby_executable_hooks:15:in `<main>'

What should happen: no such error should be thrown.
UPDATE: exact same process works fine on rubies 2.1.5, 2.2.4 and 2.2.5.

Hey @ain,
This problem is due to a bug introduced in RubyGems and is unrelated to SCSS-Lint (you can find this exact issue on other projects, e.g. HAML-Lint, Overcommit, etc).
You can find out more at https://github.com/rubygems/rubygems/issues/1420.
Hope that helps!


Add CLI capabilities to origami.

Add some cli commands to origami-lib, something like

origami init -> From a template create the basic demo file.
origami push -> scp the current directory(the demo with origami.env in project root) to the server, where the demo needs to be deployed.

Seems like a nice functionality to have but can be pushed to later once the basic package is ready. To justify making a CLI tool, we need to have more commands that are required.


SDL_HINT_CUDA_DEVICE + CUDA_VISIBLE_DEVICES = crash

The textbook way to isolate CARLA server runs to specific GPUs is to leverage docker, but that isn't always available in the HPC world, where singularity reigns.  Sometimes, we must go bare-metal, and sometimes we are dealing with job schedulers like Spectrum LSF that 'isolate' GPUs between users/jobs simply by setting the CUDA_VISIBLE_DEVICES environment variable.  Down the rabbit hole one goes with the elusive SDL environment variables that seemingly help us attain the goal of specifying which GPU CARLA will run on in the absence of Docker... but what happens if one mixes SDL_HINT_CUDA_DEVICE and CUDA_VISIBLE_DEVICES as is the case when the job scheduler sets CUDA_VISIBLE_DEVICES?  Well, the answer surprised even me:
$ nvidia-smi -L
GPU 0: GeForce GTX TITAN Black (UUID: GPU-cc978f74-7059-63b8-43d8-5ec24b6bbfb5)
GPU 1: GeForce GTX TITAN Black (UUID: GPU-98f7197f-08b2-c6b9-70dd-bf4b7bc8021d)
GPU 2: GeForce GTX TITAN Black (UUID: GPU-04a3b36e-e843-fad0-703d-222271a9068d)
GPU 3: GeForce GTX TITAN Black (UUID: GPU-baf87b79-6f28-84a1-b0ca-f324a4ae0352)
$ printenv | grep CUDA | wc -l
0
$ printenv | grep SDL | wc -l
0
# success
$ SDL_VIDEODRIVER=offscreen CARLA_0.9.11/CarlaUE4.sh -opengl
# success
$ SDL_VIDEODRIVER=offscreen SDL_HINT_CUDA_DEVICE=0 CARLA_0.9.11/CarlaUE4.sh -opengl
# success
$ SDL_VIDEODRIVER=offscreen SDL_HINT_CUDA_DEVICE=0 CUDA_VISIBLE_DEVICES=0 CARLA_0.9.11/CarlaUE4.sh -opengl
# crashes with no error, expect it to run on gpu 1
$ SDL_VIDEODRIVER=offscreen SDL_HINT_CUDA_DEVICE=1 CUDA_VISIBLE_DEVICES=1 CARLA_0.9.11/CarlaUE4.sh -opengl
# success
$ SDL_VIDEODRIVER=offscreen SDL_HINT_CUDA_DEVICE=1 CARLA_0.9.11/CarlaUE4.sh -opengl
# success
$ SDL_VIDEODRIVER=offscreen CUDA_VISIBLE_DEVICES=1 CARLA_0.9.11/CarlaUE4.sh -opengl

This is likely an Epic SDL fork thing that impacts ALL of UnrealEngine and not just CARLA, but we CARLA users appear to be the primary audience that would do something like headless rendering of UE generated imagery directly to files on a distant server without an x-server...
Why does it crash when both SDL_HINT_CUDA_DEVICE and CUDA_VISIBLE_DEVICES are set to the same, non-zero value, but functions as expected when one or the other is set?
I seem to recall in the earlier days of CARLA 0.9.8, only SDL_HINT_CUDA_DEVICE would work.  Regardless, if other users run into this problem, then a nice env -u CUDA_VISIBLE_DEVICES SDL_VIDEODRIVER=offscreen SDL_HINT_CUDA_DEVICE=GPU_INDEX_HERE CARLA_0.9.11/CarlaUE4.sh -opengl will clear it, where GPU_INDEX_HERE is the GPU number you wish to run CARLA on...

Tried this on a different system running a different linux distribution and different version of the nvidia proprietary driver and it works fine in opengl, going to write this off as a driver bug or system configuration issue.

Tried this on a different system running a different linux distribution and different version of the nvidia proprietary driver and it works fine in opengl, going to write this off as a driver bug or system configuration issue.


Allow "Add Reference" to .NET Core class library that uses .NET Framework from a traditional class library

First of all, I would like to say that I love how .NET Core projects can automatically manage package dependencies. Brilliant.
Hence I began developing projects using ASP.NET MVC Core, albeit still using the full .NET 4.5 instead of .NET Core due to various reasons. So my class libraries project.json often ended up like this:
{
  "version": "1.0.0-*",

  "dependencies": {
  },

  "frameworks": {
    "net45": {
    }
  }
}

Which is great and works beautifully in MVC Core land. However, as soon as someone need Add Reference to my class libraries without being packaged into NuGet packages (actually I haven't tried packaging those classes into NuGet packages, but I don't see why they wouldn't work), this happens:
---------------------------
Microsoft Visual Studio
---------------------------
A reference to 'My.Class.Library' could not be added. An assembly must have a 'dll' or 'exe' extension in order to be referenced.
---------------------------
OK
---------------------------

I realized that this scenario is the opposite of the ".NET Core project needs to reference the .NET 4.5 DLLs", but is it possible to have this feature for convenience?

@ryanelian can you please open a new issue on aspnet/tooling and close this one? That repo will be a better place for this issue. Thanks!

Actually, now that I look at the issue, I believe this should be mitigated with moving to csproj. Will close this. Please reopen if you feel this was in error.

I believe this should be mitigated with moving to csproj

@blackdwarf Is there a task I can follow for this?


chore(package): update mocha to version 3.5.0

Codecov Report

Merging #147 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #147   +/-   ##
======================================
  Coverage    97.5%   97.5%
======================================
  Files           1       1
  Lines         120     120
======================================
  Hits          117     117
  Misses          3       3

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 976386e...6204b7b. Read the comment docs.


Split Bill - Moving through the Split participants is done by jumping when using the arrows

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Preconditions:

You must be logged in
At least 1 group with 4 or more members must be created
Steps:
Open https://staging.new.expensify.com/
Choose any group with 4 or more members
Click on the + button in the compose box.
Click on "Split bill" in the action menu
Enter any amount of currency
Click on "Next" button
Click on any participant from the "Split with" list
Use the arrows to move through the list of users and apply selected/deselected with "Enter"

Expected Result:
Moving through the "Split bill" participants with the help of arrows should be smooth and with a clear logic for the user
Actual Result:
When a user in the "Split bill" members menu clicks on any user from the "Split with" list and tries to use the arrows to navigate through the users using the selected/deselected(Enter) function, there are random jumps through the users when you press Enter
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/2e4cf351-8fd9-45d5-9806-f9c4445d2ef4
Expensify/Expensify Issue URL:
Issue reported by: Applause- Internal Team
Slack conversation:
View all open jobs on GitHub

Thanks @dukenv0307 , I'll dig in more soon, not a big priority since it might be on hold pending

https://github.com/Expensify/App/issues/12668
or the issue that's held on
https://github.com/Expensify/App/issues/11795

@cristipaval , do you think this should be put on hold pending the below, it looks like you're leading that.

https://github.com/Expensify/App/issues/11795

@mallenexpensify the thing is that I am not sure this is an issue. See this Slack thread for more info.

I'm going to close this since it has to do with arrow keys and we haven't decided to optimize the app for keyboard nav.  Also, as @cristipaval pointed out, Mills comment from 7 months ago was keep punting 🏈


Consider adding basic CORS support directly into the framework.

When dealing with APIs, supporting CORS is a common use case. It may make sense to include basic CORS functionality that allows access from any origin to the entire API.
Such a feature might be implemented along the lines of the following middleware component (albeit integrated directly into falcon.API):
# NOTE: Not fully tested

class CORSComponent(object):
    def process_response(self, req, resp, resource, req_succeeded):
        resp.set_header('Access-Control-Allow-Origin', '*')

        if (req_succeeded
            and req.method == 'OPTIONS'
            and req.get_header('Access-Control-Request-Method')
        ):
            # NOTE(kgriffs): This is a CORS preflight request. Patch the
            #   response accordingly.

            allow = resp.get_header('Allow')
            resp.delete_header('Allow')

            allow_headers = req.get_header(
                'Access-Control-Request-Headers',
                default='*'
            )

            resp.set_headers((
                ('Access-Control-Allow-Methods', allow),
                ('Access-Control-Allow-Headers', allow_headers),
                ('Access-Control-Max-Age', '86400'),  # 24 hours
            ))

I'll give this a shot

I don't think this method makes sense; you'd have to already have an on_options method that generates a non-405 response. I think we might have used a mixin or middleware for this on a project; let me see if I can dig up the code.

Ok, yeah, we used a mixin like the following:
class PreflightOptions(object):

    def on_options(self, req, res, *args, **kwargs):
        res.status = falcon.HTTP_OK

But the CORS middleware worked on process_resource, so it got routed to a valid view.
Then we check the origin domain against a white list (ALLOWED_ORIGINS) and/or a regex (ALLOWED_ORIGINS_REGEX) defined elsewhere (we use a standard app/config.py file).
def is_allowed_origin(origin):
    url = urlparse(origin)
    if url.netloc.split(':')[0] in ALLOWED_ORIGINS:
        return True
    else:
        for pattern in ALLOWED_ORIGINS_REGEX:
            if re.match(pattern, origin):
                return True
    return False

Will this still be added in falcon 2.0?

@Mumrau yes, that is the plan. Which could happen even quicker if someone wants to lend a hand and contribute :slightly_smiling_face:

I can't right now but I might give it a shot if nobody's doing it before
few weeks.
On Fri, Aug 24, 2018, 17:52 Vytautas Liuolia<EMAIL_ADDRESS>wrote:

@Mumrau https://github.com/Mumrau yes, that is the plan. Which could
happen even quicker if someone wants to lend a hand and contribute 🙂
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/falconry/falcon/issues/1194#issuecomment-415801238,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOHrWm8sWtC1MUyKReIvUzC-2uFw35RLks5uUCEugaJpZM4RT6d2
.

Is it just me, or does this not work with 1.4.1?
Traceback (most recent call last):   File "main.py", line 12, in <module>     middleware=[CORSComponent,]   File "REDACTED/venv/lib/python3.6/site-packages/falcon/api.py", line 156, in __init__     middleware, independent_middleware=independent_middleware)   File "REDACTED/venv/lib/python3.6/site-packages/falcon/api_helpers.py", line 78, in prepare_middleware     'process_response')   File "REDACTED/venv/lib/python3.6/site-packages/falcon/util/misc.py", line 245, in get_bound_method     if six.get_method_self(method) is None: AttributeError: 'function' object has no attribute '__self__'


ERC-1271 support and testing a partner wallet app.

Adds ERC-1271 support and tests for open sourced coinbase smart wallet.
closes #608

Added integration tests for signature verification, owner revocation, time travel logic.
Please give another review

Adding here that running our test suite locally breaks;  added foundry and its bundled chain tool, Anvil to the dependency list.  For anyone that hits the
failures:
    erc1271_verifier::tests::test_coinbase_smart_wallet

you need to install foundry with:
# get the installer
curl -L https://foundry.paradigm.xyz | bash

# `source` your shell environment. e.g
source ~/.zshenv

# install foundry  <- this will also install anvil as well as some other chain tools.
foundryup

then run the tests again, and they should pass.

@tuddman Thanks! I wonder can we easily add foundry in the dev/up to automate this? I tried the scripts here but didn't work.


New version: DuckStudio.GitHubLabelsManager version 1.6

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Is there a linked Issue? - Resolves #195955

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

=== manifests\d\DuckStudio\GitHubLabelsManager\1.6 ===
Missing Properties value based on version v1.4:
Switches

See https://github.com/microsoft/winget-pkgs/pull/195394#issuecomment-2508954783

Automatic Validation ended with:

No errors to post.

(Automated response - build 899.)

"IDES"...? again?

=== manifests\d\DuckStudio\GitHubLabelsManager\1.6 ===
Missing Properties value based on version v1.4:
Switches

See https://github.com/microsoft/winget-pkgs/pull/195394#issuecomment-2508954783


v.3.1 select2 customize results

In v. 2.3.3 I used to customize the results with a template like this in "navigation_autocomplete/autocomplete.html":

  {% for finded in results %}

     {{ finded.name }}
    {{ finded.code  |default:" - " }}

  {% empty %}

 Nothing found

   {% endfor %}

Now I understand that you use select2, but looking in select2.js I cannot understand where data are rendered.
I get only the "text" that is returned by the unicode method of the searched model.
What am I missing to customize display of data?

Navigation autocompletes where actually not a feature from
django-autocomplete-light, but a feature (the purpose of) autocomplete.js
from jquery-autocomplete-light.
Django-autocomplete-light just presented this feature in the doc, but
autocomplete.js has now been extracted in its own repository:
jquery-autocomplete-light.
You can still use jquery-autocomplete-light's autocomplete.js to power your
navigation autocompletes - that's what I'd do of course :D - but that
doesn't even require installing django-autocomplete-light, just
autocomplete.js.
Does this clear any doubt ?


build getting error: ERROR in Ng2SearchPipeModule is not an NgModule

Hi,
I added the ng2-search-filter to my project. But on building process i getting this message:
ERROR in Ng2SearchPipeModule is not an NgModule

node version: v6.9.5
angular CLI: 1.0.0-beta.31
Some example how i used:
import { Ng2SearchPipeModule } from 'ng2-search-filter';

@NgModule({
  imports: [
    ...,
    Ng2SearchPipeModule
  ],
  ...

Someone got this error before?
Thanks

Hey ,
No one experienced this issue ,But there are some issues with angular-cli and this thread might help you https://github.com/angular/angular-cli/issues/3426 . Hope this helps.

Ok, i can't solve the problem.
I didn't have enough time, so i imported the pipe code to my project and it's worked.
It's not the right way, but i needed to to that.
Thanks @solodynamo for your attention.

I'm experiencing the same issue with the latest update.

me too

This issue has been fixed in latest release i.e version: 0.3.1.

Just run the ng serve command in terminal window, the error will not appear again.


Consider support for clean/delete of output dir

It would be nice if the makefile had an (optional?) call to a "clean" before running ndoc. Currently my command is:
rm -r output/doc && bin/ndoc ./path_to_files

This is because, without rm, an error is emitted on the directory: EEXIST, File exists

First of all cleaning up the output dir is not ndoc's task. Assume you have correct docs on your system (not under revision control system) and you run ndoc against faulty sources. In this case you will loose your data. Or you could have different docs there for example. Anyway, documentation building is made by scripts normally (Makefile, Rakefile, whatever), so it's as simple as put this into Makefile:
CURR_HEAD       := $(firstword $(shell git show-ref --hash HEAD | cut --bytes=-6) master)
GITHUB_NAME := nodeca/ndoc
SRC_URL_FMT := https://github.com/${GITHUB_NAME}/blob/${CURR_HEAD}/{file}\#L{line}

doc:
        @if test ! `which ndoc` ; then \
                echo "You need 'ndoc' installed in order to generate docs." >&2 ; \
                echo "  $ npm install -g ndoc" >&2 ; \
                exit 128 ; \
                fi
        rm -rf ./doc
        ndoc --output ./doc --linkFormat "${SRC_URL_FMT}" ./lib

Embedding clean functionality will break *nix principle & add danger to loose useful data.
Use makefile for batch tasks.


Incomplete link highlighting

I'm not sure why the problem occurs.
Line 897 (260 characters): (Incomplete highlighting - 74 chars blue, missing 7 chars)
Line 898 (232 characters): (Incomplete highlighting - 69 chars blue, missing 10 chars)
Line 904 (232 characters): (Complete highlighting - 79 chars blue)
Line 909 (255 characters): (Complete highlighting - 79 chars blue)
Line 912 (238 characters): (Not highlighted at all - Absolute path exists)

Any solution?

What is your setting?


Support multi-module Maven projects

In a lot of (open source and enterprise) projects it is very common to create Maven multi-module projects. One can for example have a module for only domain classes, utils or interfaces meant for public consumption.
Dependencies (code dependencies, plugins etc) can be attached to any module, optionally overriding dependencies higher up in the hierarchy (often referred to as the build reactor in Maven terminology).
In order for dependabot to be useful to a lot of Java shops, this functionality must be implemented.
See this project for a sample.

I've updated this issue with a better description :-)

Thanks for creating this @evenh. Here's my plan for implementing - am I missing anything?
File fetching

Dependabot fetches the root pom.xml (i.e., the one that is in the directory specified by the user, which by default is the root directory of the git repo)
Dependabot reads the <modules> from that pom.xml and looks in each of those directories for a pom.xml

If any directory specified as a module doesn't have a pom.xml Dependabot errors

Dependabot reads each of the module pm.xml files looking for more modules, and recursively downloads them, too.

File parsing

Dependabot parses each of the downloaded files as it normally would

Question: Will Dependabot ever encounter the same dependency being required twice, with incompatible versions? Is that allowed? As I understand it only a single version of a dependency is used by Maven.

Update checking / file updating

Happen as usual

Your proposal looks sane to me. Regarding your question, yes - that's a possibility (unfortunately)

Cool. FYI, the PR tracking this is https://github.com/dependabot/dependabot-core/pull/354. Should have it done tomorrow. 🎉

https://github.com/greysteil/maven-multimodule-sample/pulls
🔥 🔥 🔥
I'll blog about this and release it on Monday. @evenh - if you have any repos you'd like to try it on before then I'd be much obliged. There are bound to be a few bugs hiding in there.
Thanks so much for your advice and encouragement on this one!

That's just awesome @greysteil - good (and quick) work 😀 I've just updated https://github.com/evenh/maven-multimodule-sample/commit/4035b8862f1911b982be583242ce6ca89e4257b3 and was expecting two pull requests with 1 file each or one pull request with 2 files after pushing that last commit.

Ah, Dependabot won't create that because it remembers that it's previously created a PR to 24.1-jre. I can make it "forget" it from console though - one moment...

And in other news, I've just added support for custom repositories 🎉


Link hash is broken, i.e. , it points to nothing . . .

in the documentation for
https://expressjs.com/en/4x/api.html#app.listen
It states:
This method is identical to Node’s http.Server.listen()
However the
#http_server_listen_port_hostname_backlog_callback
in the link
https://nodejs.org/api/http.html#http_server_listen_port_hostname_backlog_callback
does not appear to exist so the link just takes you to the top of the page.
More importantly can someone verify what the method is identical to?
Thanks for the help!

Ah, looks like they rearranged some things in their docs. Our docs needs to be updated to link to https://nodejs.org/api/http.html#http_server_listen

PR #930 was merged.


Error while training

Get this error every time I want to train, about 30 minutes into training.
RuntimeError: DDP expects same model across all ranks, but Rank 2 has 128 params, while rank 0 has inconsistent 0 params.
I can't find anything about this error online really. Any ideas?

Sorry for replying in a closed issue.
How do you solve the problem?


Add cross-report filtering and interactivity

Addition: a feature to let dashboard builders specify charts that are connected, e.g. filtering a report by clicking/selecting parts of the another visualization:

Click on a table cell
Click on Nodes/relationships in a graph
Click on a bars in a bar chart

This click action should set a parameter $neodash_xyz, just like a parameter selection. The parameter can then be used in other reports.

Close on Extensions


re-register teeracle enclave every 24h

currently, our enclave gets unregistered after a while and exchange rate updates fail:
https://integritee.subscan.io/extrinsic/299802-1
We need to ensure the teeracle service restarts every 24h. Can probably be done with service config?

I solved this with a systemd parameter:
[Unit]
Description=Integritee TEERacle
After=network.target
StartLimitIntervalSec=0

[Service]
Type=simple
Restart=always
RuntimeMaxSec=86400
RestartSec=10
User=ubuntu
WorkingDirectory=/home/ubuntu/worker/bin
ExecStart=/home/ubuntu/worker/bin/integritee-service \
        --node-url wss://api.solo.integritee.io \
        --node-port 443 \
        --worker-rpc-port 2000 \
        --ws-external \
        run \
        --interval 1m \
        --worker-url wss://tee02.integritee.network

[Install]
WantedBy=multi-user.target

let's see if this works as expected


Stop the pulling the Redis service - Free tier from Upstash, exceeded limit 10000 usage in a single day.

From upstash -redis:
ERR max daily request limit exceeded. Limit: 10000, Usage: 10000. See https://docs.upstash.com/overall/databasetypes for details

This is not relevant to the admin system. If the issue persists, please open an issue in our core repo.


Upgrading from image based on Debian

Hi,
I'm currently using postgres:14.1-bullseye, do you think that switching to Apline could cause problems? Is this use case contemplated?
Thanks

At the moment I wouldn't risk it.
Or, at least I'd make a backup before trying it, then check carefully if it went ok.
There are reports of collation problems after upgrading, and the common thread seem to be people switching from non-Alpine based builds to our (current) Alpine-only one.
It shouldn't technically be all that hard for us to make non-Alpine based images available too, but it's just not something we've looked at yet.

In my test lab I faced a similar thing with an Ubuntu-based source PG (the base postgres:15 image)
Interestingly enough, I ran the Alpine-based upgrade image and then reverted back to the stock-Ubuntu one and the collation warnings went away. All the DB-based apps I had (5 or 6 of them) are working normally, no data loss (haven't seen any deviation so far).
This is a lab setup, so I'm not saying it is prod-ready, but it may be worth testing if you're just playing with it like me.

Cool!  That might be the right approach for people to use then. :smile:

@wuast94 That might be worth your trying out too. :smile:

I am just a lowly user who has no rights to be messing with postgres but here we are...
Trying to upgrade from pg14 (Debian 14.10-1 docker image) results in the following:

Checking cluster versions                                     ok
Checking database user is the install user                    ok
Checking database connection settings                         ok
Checking for prepared transactions                            ok
Checking for system-defined composite types in user tables    ok
Checking for reg* data types in user tables                   ok
Checking for contrib/isn with bigint-passing mismatch         ok
Checking for incompatible "aclitem" data type in user tables  ok
Creating dump of global objects                               ok
Creating dump of database schemas
failure
Consult the last few lines of "/var/lib/postgresql/data/new/pg_upgrade_output.d/20231206T014727.753/log/pg_upgrade_dump_339654.log" for
the probable cause of the failure.
Failure, exiting
The files belonging to this database system will be owned by user "postgres".
This user must also own the server process.
the relevant log reports:
command: "/usr/local/bin/pg_dump" --host /var/lib/postgresql/data --port 50432 --username postgres --schema-only --quote-all-identifiers --binary-upgrade --format=custom  --file="/var/lib/postgresql/data/new/pg_upgrade_output.d/20231206T013912.915/dump/pg_upgrade_dump_339654.custom" 'dbname=immich' >> "/var/lib/postgresql/data/new/pg_upgrade_output.d/20231206T013912.915/log/pg_upgrade_dump_339654.log" 2>&1
pg_dump: error: query failed: ERROR:  could not access file "$libdir/cube": No such file or directory
pg_dump: detail: Query was: EXECUTE getDomainConstraints('1572422')
I understand this container is meant for alpine-based upgrades but figured I would chime in

@Davo1624 Oops, I didn't see this comment until just now.  Did you get this figured out?

@Davo1624 Oops, I didn't see this comment until just now. Did you get this figured out?

Unfortunately no I did not, wasn't sure if accommodating alipne-based images was within the scope of your project but I will take all the help I can get!

No worries.  Looking at the error log there, it's complaining about a missing cube file.
That's probably the cube extension for PostgreSQL:
https://www.postgresql.org/docs/current/cube.html
At present the pg_autoupgrade image doesn't include any of the optional PostgreSQL extensions (I didn't think of it until prompted recently).
So we'll probably need to add support for them to get it working.
Alternatively, is there some way to temporarily remove the extension from your PostgreSQL install while you do the upgrade?
That might fix the problem too, but only if it's the only extension you're using.

Thank you for the very detailed explanation!!
I am running the default postgres:14 image with no extensions so I guess the cube extension is baked in. I will do some internet sleuthing to see if there is a way to get around that and I really really appreciate the info!

Interesting.  I wonder why the pg_dump command is trying to use that cube file then?
Hmmm, is it feasible for you to do a pg_dump of your database (with the working database), then grep the resulting file for cube to see if anything interesting shows up?

So looking at the specific database that was erroring during the backup process it was immich, using pg_dump then grep `cube results in the following:
-- Name: cube; Type: EXTENSION; Schema: -; Owner: -
CREATE EXTENSION IF NOT EXISTS cube WITH SCHEMA public;
-- Name: EXTENSION cube; Type: COMMENT; Schema: -; Owner:
COMMENT ON EXTENSION cube IS 'data type for multidimensional cubes';
MZ.09.10701343  Distrito de Mocubela
679932  Curcubeu        26.14405        44.76261        RO      30      132226  2013-04-21
305145  Kuzucubelen     34.43765        36.83996        TR      32              2013-09-08

So that one specific database is using the cube extension.
I am a bit out of my depth here so any suggestions are welcome and thanks again for bearing with me.

No worries, that just means we're on the right track.  I'll try and look at the extensions side of this in the next few days, as adding them to the Docker container here is probably all that's needed.  Hopefully. :sweat_smile:

Sounds great!

So much for "next few days". :frowning: I've only just started looking at this particular issue, and it looks like it'll be a simple fix.
It turns out that where the build script is running make install (example here), it should instead be running make install-world.
With that change made, the common PostgreSQL extensions (including cube) are included in the image.  So you'd likely be able to do the upgrade with this Docker container without hitting issues.
I'm testing a few other minor adjustments to the build process too today.  Assuming nothing goes badly wrong, then new Docker images that include the extensions should be available later on today.  Hopefully. :smile:

@Davo1624 The pgautoupgrade docker images now include the common PostgreSQL extensions (as above), so if you're up for testing things out again then it should all work ok.  In theory. :smile:

Many thanks, will give it a whirl and report back!

@Davo1624 How'd you go with this, did it work? :smile:

I also recently upgraded from a Debian-based Postgres v15 to pgautoupgrade v16, which is Alpine-based. As @rick-giorgi mentioned, I got some collation warnings (which did not go away), but I was also able to just use the "regular" Debian-based Postgres v16 image again.
As @justinclift , I don't think it should be too much effort to offer Debian-based images, just requires some tweaks to the build pipeline and automated testing.

@andyundso Yeah, similar thought.
Any ideas for a suitable naming scheme for the Debian based ones?  Maybe just appending -debian to the end or something?

I would suggest -bookworm to align with Postgres (they also offer a bullseye-based image, which I think we do not need).

source ‘../foo.sh’

works well but it always relative to the script that invokes it, not relative to the file itself. Maybe not a problem here, reasonable to assume point of invocation is always the same.
There’s a common hack to calculate full path, it is not good looking.

Um... no. :smile:
Ruby is firmly on my list of languages to not use unless there's literally no other choice (or close to), after having been exposed to it briefly about a decade ago.
Nothing so much wrong with the language syntax or anything (back then), but when my employer at the time (Red Hat) discovered long running Ruby processes randomly segfaulted, they had one of our senior guys (Jim Meyering I think) investigate.  Turned out to be due to umm... (this is from long ago memory) Ruby leaving dangling pointers or something in functions when they terminated, rather than clearing those pointers.  So the Ruby garbage collection process would hit these dangling pointers and segfault.
The senior guy wrote a patch to fix it by correctly clearing the pointers, and proposed it upstream.  But the Ruby Community decided they'd rather keep the random segfaults than have the ~2% speed impact from the patch (!!!!!).
That was when Ruby went onto my permanent "not worth bothering with" list, and I've not seen any reason to change my opinion.  Meanwhile, I tend to prefer Go for most things these days. :grin:

Yeah, the "source" approach for including common code might be the go.
We'll probably find out if it's that easy once someone spends some implementation time experimenting with the pieces. :smile:

very interesting anecdote! would be very interested in what that patch was but probably almost impossible to find since ruby wasn't on github at the time i think.

Hmmm, if the Ruby developers have a mailing list + matching archive, that might do it?

We might even be able to use the same pgautoupgrade script piece in both (via include file maybe?) which would be pretty optimal. 😄

maybe I do not understand the issue, but I would assume we can just copy the same docker-entrypoint.sh into the potential Debian container? or what would you need to source?
I could see erb helping with templating a Dockerfile for both Debian and Alpine, but as for now I would suggest to maintain two different Dockerfile, one for Alpine and one for Debian. They will be quite similar in terms of structure, but we still have to install other tools on Debian than Alpine.

I would assume we can just copy the same docker-entrypoint.sh into the potential Debian container

The docker entrypoint script we're using at the moment was based upon the official Docker PG Alpine one (copied from their GitHub repo).
I haven't checked if they're using the same entrypoint script in bother their Debian and Alpine containers.  If they are, then you're right it should be the same.
If they're not though, then we probably can't use our entrypoint one as-is.

So I took the most recent version of the entrypoint script from here for Alpine and here for bookworm and ran them through a text compare. There is one change (left is Alpine, right is bookworm):

maybe we could use this gosu tool for Alpine as well. It is mentioned in the README of gosu that su-exec is a rewrite of gosu in C that is available to Alpine. But I think a couple megabytes more does not hurt our Alpine container for consistency across the Debian and Alpine images.

so Gosu is available in Alpine Edge, but not 3.19. But we could use the Docker build from the maintainer to copy out the binary.
not sure if this is my favorite solution. potentially we could also implement a switch in our Docker entryscript to determine if we are running under Alpine or Debian to invoke the correct tool.

implement a switch in our Docker entryscript to determine if we are running under Alpine or Debian to invoke the correct tool

that sounds like the best to me

@andyundso Sounds like a reasonable idea.  Maybe try it out and we can put put some kind of EXPERIMENTAL Debian build! notice on the front page README with info about it?
And probably also ask for people to try it out (etc).  There are a few open Issues in the repo with people that would probably give it a go.
We could also ask on that super-long thread about upgrading, on the official Docker PG image too. :smile:

@xojoc @rick-giorgi @Davo1624 If anyone's up for testing a new Debian based pgautoupgrade image, then @andyundso has officially added ones for PostgreSQL 15 and 16:

15-bookworm
16-bookworm

Anyone up for giving it a go? :smile:

Hi @justinclift! I tested upgrading from 14 -> 16 using 16-bookworm. Tested extensively in our dev environments and then deployed the image to over 20 managed PostgreSQL databases running on K8S. It worked perfectly, pos upgrade had to execute some manual cleanup to get the engine to execute queries faster than before:
REINDEX DATABASE database;
VACUUM FULL VERBOSE;
ANALYSE VERBOSE;

Thanks all for this fantastic project!! I am happy to contribute to future development 🚀

Awesome, thanks heaps @josecsotomorales! :smile:
Yeah, I'd really like for us to figure out solid post-upgrade steps to be automatically run after each upgrade.  Re-indexing is definitely on that list.  I'm just concerned about the amount of time it can take with larger data sets.
Though I suppose we should start simple (ie just do it) and then figure out more intelligent options over time if they do turn out to be needed.

maybee something we could turn on per env variable but defaulting to false?

Yeah, I think we'll ultimately need to have some kind of yes/no/(etc) variables to control the automatic re-indexing.
Creation of new indexes isn't needed on every upgrade.  However, it is needed for some specific upgrades, including (sometimes) for upgrades to a new point release when there was a bug in the prior one's indexing code.
To get things exactly right (rather than just do "always reindex" or similar), we'd need to have a good idea of the exact PostgreSQL version (including point release version) being upgraded from.
Which is a problem as PostgreSQL only records the major version in the PG_VERSION file.

We could (for pgautoupgrade images) store the exact PostgreSQL version (including point release) in an extra data file in the database directory.  Then we could use that when upgrading from one pgautoupgrade image to another for deciding whether to reindex.
If the file isn't present when upgrading (eg coming from a non-pgautoupgrade image), then we could just blindly always to do the reindexing.
Something along these lines would probably be the most accurate approach, at least to only reindex when it's truly needed.


Query Filter | Setup Issue

Original Question from deliotomaz

Hi,
I am using a Generic Repository but Global Filters are not working.
In my repository I have a property:
protected DbSet DbSet { get { return Context.Set(); } }
I am setting the Global Filter on Context constructor:
QueryFilterManager.Filter("IEntidadeConta", q => q.Where(x => x.CodigoConta == dadosUsuarioResolver.CodigoContaCorrente)); QueryFilterManager.Filter("IExclusaoLogica", q => q.Where(x => x.Ativo)); QueryFilterManager.InitilizeGlobalFilter(this);
The filter is not applied.
An important thing: I didn't create DBSet properties on my DbContext. If I create them, the filters are applied.
Can you help me ?

Asked in issue #47

Which version of Entity Framework are you using?
Query Filter doesn't work the same way in EF Core and EF6.
For EF Core, filter currently only work for DbSet specified in the DbContext.
For EF6, it shoulds work.
Let me know which version you are using.
Best Regards,
Jonathan

Hi Jonathan, Thnks for quick response.
I am using EF6.
Note that I get this issue when I don't have my DbSet<> declared on my DbContext and load data using DbContext.Set
If I declare all DbSets and them Get data using DbContext.Set it works.

Hello @deliotomaz ,
Unfortunately, this is a limitation to our library currently. We only check type which is beyond a DbContext.
This weekend, we will try to add a new option to allow you to register some type which is not part of the DbContext
Example
QueryFilterManager.RegisterType(typeof(MyCustomerType);

I will let you know more very soon.
Best Regards,
Jonathan

Hi Jonathan,
Ok No problem.
Thanks for you help. This is will be very useful.
Best Regards,
Delio

Hello @deliotomaz ,
The code for registering type has been completed.
Unfortunately, we didn't found where EF was cached all types, so you will need to explicitly register them.
The version should be ready for Monday, I will provide you more information tomorrow.
Best Regards,
Jonathan

Hello @deliotomaz ,
Sorry for the delay,
We have added the method RegisterType as previously said, can you try it to see if it's now working?
Let me know if it's successfully fixed.
Best Regards,
Jonathan

Hi Jonathan,
No problem with delay.
Thanks!!!!!
I will try it this weekend and feedback you.
Thanks for your help
Delio

Hi Jonathan,
I am registering all types in my Context constructor:
` var types = typeof(EntidadeDominioBase).Assembly.GetTypes()
.Where(type => type.IsSubclassOf(typeof(EntidadeDominioBase)) && type != typeof(TrackedEntidadeBase));
        QueryFilterManager.RegisterType(types.ToArray());

        QueryFilterManager.Filter<IEntidadeConta>(q => q.Where(x => x.CodigoConta == dadosUsuarioResolver.CodigoContaCorrente));
        QueryFilterManager.Filter<IExclusaoLogica>(q => q.Where(x => x.Ativo));
         QueryFilterManager.InitilizeGlobalFilter(this);`

When I try to use any DbSet I get an exception : "The entity type is not part of the model for the current context."
Any ideia if I am doing wrong ?
Best
Delio

Hello @deliotomaz ,
You are right, we have found the bug.
The fix will be released soon. We are currently trying to release but it seems we have some internet/TFS problem.

Hello @deliotomaz ,
The issue has been fixed in v1.4.14
Btw, you need to register type you will use in your DbSet. You don't have to register interface and base class if they are not part of your model.
QueryFilterManager.RegisterType(typeof(EntitySimple));
//QueryFilterManager.RegisterType(typeof (IEntitySimple)); // Will works but not required

ctx.Set<EntitySimple>()...

Hi,
I will try it and feedback you about the result.
Thanks again,
Best

Hi,
It works !!!!!
Thanks Jonathan.
Good job.
Best

Closing comment: Fix confirmed


Graphics freeze after waking from sleep

Godot version
4.0.beta1
System information
Manjaro Linux (KDE X11)
Issue description
If Godot 4 is open when I close my laptop to put it to sleep, when I open it again and wake it up Godot freezes. It seems to just be the renderer that freezes, because shortcuts like quit, open project manager, etc. still work.
Steps to reproduce

Open a project in Godot 4
Put device to sleep
Wake it up
The UI does not update anymore, but shortcuts work

I've been able to reproduce this every time.
Minimal reproduction project
No response

Duplicate of https://github.com/godotengine/godot/issues/52738.


get user selected date

hi. im kinda new in php and javascript coding.
so this is more of a question than an issue,
after user select date and is shown in the input area.
how do i get that value when i submit the form?
i hope my english is clear enough.
thanks

you just get the date from the input field like you would any other input field when the form is posted.
if you're trying to get the date via javascript, I suggest reading the docs.

To be honest, this needs to be documented properly I believe. It's not clear from the docs page that .date() is a function/property of what & how it can be accessed. I had to use $el.data("DateTimePicker").date() in order to get it.

Totally agree with @v1p -- although it is documented on the main page, there is absolutely zero mention of the fact that one has to use the data attribute to access any of the functions on the actual functions page. Providing example usage of each function on the function page with the proper $('#elem').data('DateTimePicker').functionName(...) syntax would've made the quest to get this working way easier.

To be clear @Eonasdan, my comment acknowledged that the alert you posted is available on the main page, but not on the actual Functions page. It should exist on both pages (rather than forcing one to hunt around on multiple pages just to figure out how to use a function).

Yeah even I never bothered to goto mainpage & went directly to functions page, as that was the area of concern for me.
I might have skipped the callout note on the mainpage, as it has no business there IMO, that the mainpage is primarily a showcase of what the plugin can do.
This is an amazing plugin as far as datetime picker goes. Quite easy to use once you know the way around, and I was able to customize it a bit to work with Foundation 5, well at least the basics of it.


levenshtein check

hey what do you think about taking out the tld from the check and checking it against just the name
Proposing
var holisticMetric = levenshtein(strDomain, "myetherwalletcom");
to be
var holisticMetric = levenshtein(strDomain, "myetherwallet");
This way, we can check for the distance of the name without the tld

I like the suggestion - will integrate in the next release - thanks!


Failing inflow_outflow.test

The test inflow_outflow.test seems to fail sometimes when running via gitlab.  The issues was first observed in regression testing on PR #223.  It is not clear what is causing the failure, and so far it is not reproducible in a local build on the same machine (even though in theory everything is the same between the local build and the automated CI configuration).
The log from the failed test is pasted below:
1..5
ok 1 [inlet-outlet with argon minimal] check for input file inputs/argonMinimal.inflow_outflow.ini
PASS: inflow_outflow.test 1 [inlet-outlet with argon minimal] check for input file inputs/argonMinimal.inflow_outflow.ini
ok 2 [inlet-outlet with argon minimal] create a mesh file
PASS: inflow_outflow.test 2 [inlet-outlet with argon minimal] create a mesh file
ok 3 [inlet-outlet with argon minimal] create a restart file with a prescribed initial condition
PASS: inflow_outflow.test 3 [inlet-outlet with argon minimal] create a restart file with a prescribed initial condition
not ok 4 [inlet-outlet with argon minimal] run tps with input -> inputs/argonMinimal.inflow_outflow.ini
FAIL: inflow_outflow.test 4 [inlet-outlet with argon minimal] run tps with input -> inputs/argonMinimal.inflow_outflow.ini
# (in test file inflow_outflow.test, line 38)
#   `../src/tps -run $RUNFILE' failed with status 134
#
# ------------------------------------
#   _______ _____   _____
#  |__   __|  __ \ / ____|
#     | |  | |__) | (___
#     | |  |  ___/ \___ \
#     | |  | |     ____) |
#     |_|  |_|    |_____/
#
# TPS Version:  1.1 (dev)
# Git Version:  044f0f5
# MFEM Version: MFEM v4.4.1 (development)
# ------------------------------------
#
# Options used:
#    --no-version
#    --runFile inputs/argonMinimal.inflow_outflow.ini
#    --no-debug
#    --no-visualization
#
# Caching input file -> inputs/argonMinimal.inflow_outflow.ini
#
# ---------------------------------
# MFEM Device configuration:
# Device configuration: hip,cpu
# Memory configuration: host-std,hip
# ---------------------------------
#
#
# TPS is using non-GPU-aware MPI.
# config.transportModel = constant
# name, input index, mixture index: E, 0, 1
# name, input index, mixture index: Ar, 1, 2
# name, input index, mixture index: Ar.+1, 2, 0
#
# Inlet mass fraction of background species will not be used.
#
# Inlet mass fraction of electron will not be used.
# Total # of mesh elements = 90
# number of elements on rank 0 = 90
# min elements/partition       = 90
# max elements/partition       = 90
# restart: 1
# HDF5 restart files mode: read
# Restarting from iteration = 0
# --> time = 0.000000e+00
# --> dt   = 1.000000e-05
# Restarting averages with 0
#  samplesReading in solutiond data from restart...
# --> Reading h5 path = /solution/density
# --> Reading h5 path = /solution/rho-u
# --> Reading h5 path = /solution/rho-v
# --> Reading h5 path = /solution/rho-E
# --> Reading h5 path = /solution/rho-Y_Ar.+1
# Reading in solutiond data from restart...
# --> Reading h5 path = /meanSolution/meanDens
# --> Reading h5 path = /meanSolution/mean-u
# --> Reading h5 path = /meanSolution/mean-v
# --> Reading h5 path = /meanSolution/mean-p
# --> Reading h5 path = /meanSolution/mean-YAr.+1
# Reading in solutiond data from restart...
# lt-tps: equation_of_state.cpp:592: double PerfectMixture::computeAmbipolarElectronNumberDensity(const double *) const: Assertion `n_e >= 0.0' failed.
# /var/tmp/oliver33/bats-run-249865-iVtpDl/bats.249896.src: line 37: 250100 Aborted                 ../src/tps -run $RUNFILE
not ok 5 [inlet-outlet with argon minimal] verify tps output with analytic solution -> inflow_outflow_output.ref.h5
FAIL: inflow_outflow.test 5 [inlet-outlet with argon minimal] verify tps output with analytic solution -> inflow_outflow_output.ref.h5
# (in test file inflow_outflow.test, line 48)
#   `h5diff --relative=$TOL $SOLN_FILE $REF_FILE /solution/rho-Y_Ar.+1' failed
# dataset: </solution/rho-Y_Ar.+1> and </solution/rho-Y_Ar.+1>
# 90 differences found
ERROR: inflow_outflow.test - exited with status 1

It indicates a failing assert on the electron number density.

In addition to not being able to reproduce locally, the tests passed when run after PR #223 was merged.  So... very confusing.


(loader: BTM and Model) passing limit and offset into groupedLimit

Closes #64

places limit and offset into groupedLimit field of findOptions
reverts changes in #55

Does sequelize support offset in groupedLimit? As far as i recall it doesn't

Also please add a test covering this.

@mickhansen, my apologies for the late response.
I don't believe it does; I was just experimenting and forgot to remove it. I'll do so, then update this weekend.


Upgrade dependencies

closes #30

Let's also update the Scala version in .travis.yml, otherwise LGTM!


Add Sophia ML example

Add Sophia ML example.
This is associated with https://github.com/microsoft/monaco-languages/pull/67

Thank you! :heart:


Change the movable contents to special list, and make view include Contents

New list implementation was required to make += and such work. Not sure if SetVar would have helped? Actually didn't test if a reference to the list still updates the object on reference implementation, but it feels like it should.
Also made view(...) include contents just like range(...) - these probably should share the implementation code, other than a boolean. They are pretty much the same, just with view culling.
Oh, this fixes:

TGUI of worn stuff (via view(), mostly)
Drones and probably borgs getting UI for RPD
Fire extinguishers dropping under your mob when you put them into the cabinet, while also being referenced by the cabinet, leading to fun teleport shenanigans

Though, worrding that it's shitcode to have public ChildEnumerator and public ChildCount exposed by the DreamObjectMovable. Maybe I could just give the MovableContentsList its Transform component?

Oh yeah, remembered another thing
I have no damn clue as to the stability of the iteration keys and trying to get items "by index" where it's just another iterator again and uhhh... yeah. It didn't seem to explode at least? Like, taking off stuff and moving things around in hands worked fine. Didn't check it with backpacks too much though.


PBC validation webhook for activeBundle

What would you like to be added:
I would like to have a validation added during upgrade packages.
Why is this needed:
When I upgrade the activeBundle of the controller using:
eksctl anywhere upgrade packages --bundle-version v1-22-34 where v1-22-34 doesn't exist in the cluster, there is no validation that leads to a failure on the cli. I see that this failure is in the logs but we should ideally inform the client of the failure. I have observed this issue using kubectl as well.

Done


Enable off-site item requesting

To improve the end-to-end procedure, including allowing readers to request off-site, materials directly from the website. This will provide foundational capability that should support more complex inventory management and circulation policies in the future.
Link to Notion: https://www.notion.so/wellcometrust/Enable-off-site-item-requesting-f6ab9f0903d84623a6da6d168017ec51?pvs=4

Link https://github.com/wellcomecollection/catalogue-pipeline/pull/2669


API parsing

Awesome tool!  I haven't had a chance to dig into the code but was using it to quickly fill out some details on the different techniques and I noticed that when it is pulling description or mitigation from the API it is also pulling in the reference included in those descriptions.  Here is an example of what is being stored for T1002. You can see there is addition of the references in the mitigation section.
https://attack.mitre.org/wiki/Technique/T1002
: Identify unnecessary system utilities, third-party tools, or potentially malicious software that may be used to compress files, and audit and/or block them by using whitelistingBeechey 2010 tools, like AppLocker,Windows Commands JPCERTNSA MS AppLocker or Software Restriction PoliciesCorio 2008 where appropriate.TechNet Applocker vs SRP
Thought I'd share in case you are able to look at the sooner then me.  Again, also PowerShell usage, and thank you!

hey @g1ldedm1n1on  ! Thank you for the observation. I actually noticed that since day 1 and I forgot to get back to it. Something that I remember doing at the beginning that I planned on testing later was to look into the references property values as shown in the figure below. For example, T1002 has all the "extra" strings showing up in the description text in the Reference property. I believe all I have to do is replace those values in a "foreach" loop on every description from "Technique", "Group" & "Software. I havent got the time to get to it yet. I will take a look at it soon. Thank you for the feedback :)

@Cyb3rWard0g is right on the find replace loop. Each tag in the Reference list uniquely identifies a reference on the site. As a note, if you wanted to make the references more obvious you could switch the underlying API queries to return the raw text by adding #-ia to each property you want to be raw e.g. ?Has technical description#-ia. This will show citations in like this [[CiteRef::<reference key>]] rather than just <reference key>.

Thank you very much @dm-mitre for the input. That is very helpful. I will try to add the #-ia to each property and test it. :) Thank you

Hey @g1ldedm1n1on ! I just got back from vacation and was able to add @dm-mitre recommendation to the script (#-ia) in description, analytic details and mitigation properties. It works as expected and looks cleaner. I like to keep the references in there but with [[CiteRef::]] syntax. I updated the example csvs and README in the repo. Please whenever you have a chance, test it and let me know if this is helpful :) . After testing your example (T1002), I now get the following:

You can see the references being displayed with the right syntax :) . Thank you for the feedback . I hope you have a great weekend! Also, once again thank you @dm-mitre for sharing the #-ia information. it made things so much easier.

@g1ldedm1n1on , I am closing this issue since it is working fine with the latest update applied to the script. Let me know if you need me to open it back up for more testing or additional features related to the same topic.

Thanks, I must of missed the notification on github of this.  I will give it a test but looks good from what you displayed. Thanks for making the modifications!


Feat. request: add editorconfig support

It would be nice, for a future release to add editorconfig support.
Editorconfig spec.

I agree, and they have a C-library with nothing special, i.e. easy to use directly by Crystal.

Fixed in 44caf7dce53ab0c30a073c26ec7a40495ee99249.