[RFC] Relay IR Text Format

Keyword Proposal
The final keywords are:

def for global function definitions
fn for local function definitions/anonymous functions
if and else for if-else expressions
let for let bindings
true and false for booleans
Int, UInt, Float, Bool, Tensor for type constructors
Int8, Int16, Int32, Int64, UInt8, UInt16, UInt32, UInt64, Float16, Float32, Float64, Bool for base types

Loose Ends
There are some elements of the text format that didn't make it into this PR. See #1935 for details.

Please comment on syntax design choices for #1781 here!
Notable Syntax Examples
Functions
Named, Typed
def @foo(%x: int64, %y: int64) => int64 {
    %x + %y
}

Named, Inferred
def @foo(%x, %y) => {
    %x + %y
}

Anonymous, Typed
fn (%x: int64, %y: int64) => int64 {
    %x + %y
}

Anonymous, Inferred
fn (%x, %y) => {
    %x + %y
}

Let Expressions and Mutation
Immutable
let %x = 2;
let %x = 3;
let %y = 4;
%x + %y

Mutable
let mut %x = 2;
%x = 3;
%x^

Side Effects Only
The following are equivalent
print("hello");
1

let _ = print("hello");
1

Tuples
()
(0,)
(0, 1)
(0, 1).0

Types
Base Types
Designed to look like NumPy types.
int8, int16, int32, int64
uint8, uint16, uint32, uint64
float16, float32, float64
bool
int(1,2)
uint(1,2)
float(1,2)
bool(1)

Shape Types
s
(n + 1, 2 * n, 4)
s.0

Tensors
Tensor[bt, s]
Tensor[float32, (1, 2, 4)]

def @add(t1: Tensor[bt, s], t2: Tensor[bt, s]) => Tensor[bt, s] {
    ...
}

Function Types
Function types currently cannot be explicitly created, but might be useful for debugging type errors.

Can we move most of the discussion to this issue instead? and post RFC here

cc @dmlc/tvm-team

cross posting.
We will need a spec on how nested expressions can be printed,
specifically, instead of
add(%x, add(%y, %z))

We prefer
%1 = add(%y, %z)
%2 = add(%x, %1)

Note that add will become neural network layers such as conv2d etc.

I don't think this concern is within the scope of this issue/PR. IMO the pretty printer should print the AST exactly as it's presented. I think it's too restrictive to force function arguments to be values at parse time, and thus we must allow for nested expressions. We could possibly have a compiler option that forces certain compiler passes like ANF and CSE before printing or even use the pretty printer to reformat code into "graph" style.

This is an issue because this is how user usually constructs a program via DSL, and we need to make sure when we print, they are print out in a well-defined compact fashion which is readable by the parser(without running any transformation pass)

Here's a proposal for block syntax per @yuruofeifei's suggestion. It will have the same semantics as  ReasonML's for local scope, which is
{
  let %x = 1;
  %x
};
...

desugars to
let _ =
  let %x = 1;
  %x;
...

cc @junrushao1994 @dmlc/tvm-team @jroesch @MarisaKirisame @yuruofeifei @grwlf please comment on the choices of keywords in the text format.

I think one general question we want to ask when deciding keyword is consistency, and whether we want to enforce some of these

Consistency with python style
e.g. true/false  vs ```True/False````
Consistency with numpy base type convention(and the printing output of tvm types)

e,g. float32, bool vs Float32, Bool

Consistent with common choices of existing languages

e.g. use of -> vs =>

Rust: fn(args) -> return_type
Swift func(args)-> return_type

Neat.
BTW, it is necessary to use Int, UInt, Float?

How to express common constant with type information
Rust has

let float1 = 1f32;
let double = 1.0;
let also_double = 1f64;
// same for {u32, u64, u128, ...}

necessary to use Int, UInt, Float

It will definitely be confusing unless there's a well defined spec for how the compiler chooses to use these annotations. I can see some value of this when compiling one model to different architectures with varied word sizes (e.g., native vs wasm).

tuple

This is already a major point of confusion for novice python devs. Following the convention of tuple = comma will at least keep the confusion consistent.

print Array as axis=(1,2)

Semantically, it's the same thing since the type is fixed at this point. If anything, square brackets might improve readability (e.g., x(y(z, axis=(1, 2)), axis=(3,)) vs x(y(z, axis=[1, 2]), axis=[3]))

The idea was to have all terms (e.g. variables, functions, literals) use lowercase (this is currently not enforced) and all types use uppercase (this is currently enforced by the parser). This is similar to Python type annotations and makes it clear whether something is a term or a type.
This may become especially important when we have generic shapes like (n, 1, m), because it should hopefully discourage users from writing things like (float32, 1, m).
Moreover, in Python type annotations are lowercase for builtins like int and str, but (by convention) uppercase for classes. I think this is a meaningless distinction for Relay and leads to more problems than it solves. It also can falsely equate functions and types. E.g. str in python is both a type and a function that returns a str.

@tqchen @nhynes Since we don't support tensor or list literals, it might make sense to allow both () and [] in many places. On a related note, currently type constructors use [] instead of () like function calls. This was to be consistent with Python conventions and also because things like Tensor[(1, 1, 1), float32] are easier to read than Tensor((1, 1, 1), float32).

The use of (1,) instead of (1) is mostly a technical one that could be relaxed in the future. The problem is that (1) can be parsed both as 1 with parentheses and as a tuple with a single element.

For functions:
I think using fn for function types is a good idea and an easy change to make.
The reason for using -> over => is both for consistency with Python annotations and also because lines like let %_: () -> Int64 = fn () -> Int64 { 0 }; () are easier to read than let %_: () => Int64 = fn () => Int64 { 0 }; () since => visually interferes with =.

I support @nhynes's constant literal annotation suggestion.

@tqchen it feels like we should be able to avoid embedding type information in the attributes and instead use the type system.

@jroesch I agree that in the longer term maybe explicitly annotate the output type is a good alternative. However, given that explicit ouput-type annotation is not supported for now and keep the type information in the attributes provides the numpy compatible API, it could be a good way to go for now.

To follow up with consistency issues, the constant proposal also needs some thoughts.
1.0f32 makes a lot of sense for rust because rust already uses f32 for float32. It may make less sense given the way we choose name types. An alternative could be float32(1.0)
Notably, we only need to print out common types in a friendly text format, that means we likely only have to deal with int32, float/double, bool and rest can be dealt with explicit cast

This idea just come out of my head:
can we pin a version number to every relay source/serialized code? It help to switch from one version to another version.

@tqchen I think type abbreviations like f32 make sense in Relay as well. C, C++, Java, and Python all have type suffixes for numeric literals. We could also consider forcing all literals to be type-annotated. If we decide to do that I think forcing people to write suffixes makes more sense than forcing people to write wrapping functions.

I think using Python conventions for booleans and dtypes is probably the way to go.

I'm closing this issue since the text format PR has been merged. #1935 contains the missing pieces that didn't make it into #1781 and further discussion about the text format should be moved there.


[WIP] do not omit image references that are not filtered on

fixes https://github.com/docker/docker/issues/29901
fixes https://github.com/docker/docker/issues/29203
when listing images with docker images, and filtering
the list by image:tag, the digest was not set on images.
This change always sets the digest on images (if present),
so that it will be shown in the results, even when filtering.
Similarly, when filtering by digest, the image's tag
was not shown, resulting in a different presentation,
depending on the filter that was set.
Before this patch:
docker images --digests alpine:latest
REPOSITORY          TAG                 DIGEST              IMAGE ID            CREATED             SIZE
alpine              latest              <none>              88e169ea8f46        8 days ago          3.98 MB

docker images --digests alpine@sha256:dfbd4a3a8ebca874ebd2474f044a0b33600d4523d03b0df76e5c5986cb02d7e8
REPOSITORY          TAG                 DIGEST                                                                    IMAGE ID            CREATED             SIZE
alpine              <none>              sha256:dfbd4a3a8ebca874ebd2474f044a0b33600d4523d03b0df76e5c5986cb02d7e8   88e169ea8f46        8 days ago          3.98 MB

After this patch:
docker images --digests alpine:latest
REPOSITORY          TAG                 DIGEST                                                                    IMAGE ID            CREATED             SIZE
alpine              latest              sha256:dfbd4a3a8ebca874ebd2474f044a0b33600d4523d03b0df76e5c5986cb02d7e8   88e169ea8f46        8 days ago          3.98 MB

docker images --digests alpine@sha256:dfbd4a3a8ebca874ebd2474f044a0b33600d4523d03b0df76e5c5986cb02d7e8
REPOSITORY          TAG                 DIGEST                                                                    IMAGE ID            CREATED             SIZE
alpine              latest              sha256:dfbd4a3a8ebca874ebd2474f044a0b33600d4523d03b0df76e5c5986cb02d7e8   88e169ea8f46        8 days ago          3.98 MB

/cc @vdemeester @dmcgowan @tonistiigi

oh, I see what's happening; working on a fix

Changed to "WIP". Haven't had time this weekend to complete my changes

The updated logic looks correct to me. I wonder if we still need to do filter on the cli side to filter out non-requested tags though.
For example
$ docker images --digests registry:2.1
REPOSITORY          TAG                 DIGEST                                                                    IMAGE ID            CREATED             SIZE
registry            2.1                 sha256:e641943a78a8f634c16ad69f5c9d779f470b147865c2121d89c52ea0da6fc1bd   52bb991b482e        14 months ago       220 MB
$ docker images --digests registry:2.2
REPOSITORY          TAG                 DIGEST                                                                    IMAGE ID            CREATED             SIZE
registry            2.2                 sha256:30adb707d1b4d2ad694c38da3ea1e7d303fdbdce2538ab0372afe6f1523ae3c8   ad379b517aa6        11 months ago       225 MB
registry            2.2                 sha256:a9128f456da044c4132c41973b065d2e2c46b04682c65f23dbb5d0c51071d595   ad379b517aa6        11 months ago       225 MB
registry            2.2.1               sha256:30adb707d1b4d2ad694c38da3ea1e7d303fdbdce2538ab0372afe6f1523ae3c8   ad379b517aa6        11 months ago       225 MB
registry            2.2.1               sha256:a9128f456da044c4132c41973b065d2e2c46b04682c65f23dbb5d0c51071d595   ad379b517aa6        11 months ago       225 MB

This might be related to the integration test failures. I think in that case returning 2 entries for the different digests is correct but don't see the point in showing the extra tags on the cli.

Yes, we need it for situations where (e.g.) we match on digest (which results in the tag being omitted), and there were some other cases.

@thaJeztah still WIP?

Needs a rebase and CI is failing

ping @thaJeztah @tonistiigi 👼 🙏

@thaJeztah Thanks for working on this!
Looks like this PR is inactive so I'm going to close it for now.


Winning icons pls

What feature do you want to get added on the base engine?
bru
To test your sight, and reliability, please select the option of what should NOT be requested.
Proper credit that was forgotten

my bad missclick


imgtool: ed25519: add method to return raw bytes

Add a method to return the 32 bytes Public Key in
Raw Encoding and Public Raw format.
The existing get_public_bytes returns DER format which is 44 bytes long.
This enables the clients to use the API instead of
doing a
    string_stream = (
        key._get_public()
        .public_bytes(
            encoding=serialization.Encoding.Raw, format=serialization.PublicFormat.Raw
        )
        .hex()
    )

Signed-off-by: Naveen Kaje<EMAIL_ADDRESS>

I don't think the use case is enough for adding this very specific API. At least you would have to define it in the base class, add some tests, and implement this for all key classes. For your use case why can't you just use get_public_bytes()[-32:] instead?

Yes, get_public_bytes[-32:] is sufficient. Thanks for the tip! Closing this.


Support node 7

_normalizeConnectArgs is named _normalizeArgs in node >7. Use whichever is available.

Any thoughts on this @mscdex? Maybe a small version bump is in place as well, forgot about that


problem while exporting to tar file

Done exporting layer 'ABC_Policy Security'.
Hi Eden!
I tried changed new device and install new python's environment. But it still show these problems. Can you expain these problem and any suggestions to fix it.
Thanks.
Traceback (most recent call last):
File "D:\ExportImportPolicyPackage-master\import_export_package.py", line 65, in
export_package(client, args)
File "D:\ExportImportPolicyPackage-master\exporting\export_package.py", line 54, in export_package
File "D:\ExportImportPolicyPackage-master\utils.py", line 289, in write_data
writer.writerows(res)
File "C:\Users\AppData\Local\Programs\Python39\Python39\lib\encodings\cp1252.py", line 19, in encode
return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode character '\u1ec3' in position 87: character maps to
PS C:\Users\Desktop\ExportImportPolicyPackage-master (1)\ExportImportPolicyPackage-master>

and could you sent me your python version and environment you are using. I think the problem still from environment.
Thanks Eden

Hi @quabank
From version 5.0 the tool is developed using Python language version 3.7
Also, you should set the file encoding to use UTF-8

Hi @quabank From version 5.0 the tool is developed using Python language version 3.7 Also, you should set the file encoding to use UTF-8

Thanks for reply. My python version: 3.9 . I set the file encoding and its worked. So i import this policy and there some problems after done importing object.
Thanks
Traceback (most recent call last):
File "c:\Users\Desktop\ExportImportPolicyPackage-master\ExportImportPolicyPackage-master - Copy\import_export_package.py", line 67, in
import_package(client, args)
File "c:\Users\Desktop\ExportImportPolicyPackage-master\ExportImportPolicyPackage-master - Copy\importing\import_package.py", line 56, in import_package
layers_to_attach = import_objects(args.file, client, {}, package, None, args)
File "c:\Users\Desktop\ExportImportPolicyPackage-master\ExportImportPolicyPackage-master - Copy\importing\import_objects.py", line 115, in import_objects
os.remove(general_object_file.name)
FileNotFoundError: [WinError 2] The system cannot find the file specified: '03____add-host__2022_07_11_13_40.json'

@chkp-edenbr may you check this


elixir: update default to 1.18

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 368356

x86_64-linux

:fast_forward: 2 packages blacklisted:

nixos-install-tools
tests.nixos-functions.nixos-test

:white_check_mark: 7 packages built:

elixir
elixir-ls
lexical
mix2nix
next-ls
protoc-gen-elixir
sonic-pi


开启 getPopupContainer 渲染到父节点上，侧边栏的menu菜单折叠情况下会显示异常

Subject of the feature
Describe your issue here.
Problem
If the feature requests relates to a problem, please describe the problem you are trying to solve here.
Expected behaviour
What should happen? Please describe the desired behaviour.
Alternatives
What are the alternative solutions? Please describe what else you have considered?

<ConfigProvider
:getPopupContainer="getPopupContainer"
:locale="getAntdLocale"
:theme="themeConfig"

<AppProvider>
  <RouterView />
</AppProvider>

function getPopupContainer(trigger) {
return trigger ? trigger.parentNode : document.body;
}


rabbitmq-server: 3.8.3 -> 3.8.4

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/rabbitmq/rabbitmq-server/releases.
meta.description for rabbitmq-server is: "An implementation of the AMQP messaging protocol"
meta.homepage for rabbitmq-server is: "https://www.rabbitmq.com/"
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.8.4 with grep in /nix/store/i25kykgzlj5x7s3g1ns34bgw4jhq9raf-rabbitmq-server-3.8.4

directory tree listing: https://gist.github.com/83c9a0e1adc9fc29ec947776ea9052df

du listing: https://gist.github.com/547786aac8152737b3d0f66406e3477f

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
rabbitmq-server

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/i25kykgzlj5x7s3g1ns34bgw4jhq9raf-rabbitmq-server-3.8.4 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A rabbitmq-server https://github.com/r-ryantm/nixpkgs/archive/9bbc34eb06f63c588bf525fe29484047b1945370.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/i25kykgzlj5x7s3g1ns34bgw4jhq9raf-rabbitmq-server-3.8.4
ls -la /nix/store/i25kykgzlj5x7s3g1ns34bgw4jhq9raf-rabbitmq-server-3.8.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:
- rabbitmq-server

Maintainer pings
cc @Profpatsch for testing.

@GrahamcOfBorg build rabbitmq-server rabbitmq-server.tests

Good bot


Please add virtual host to nuxt.js

Please add virtual host to nuxt.js for dev and start
Is there a way I can use this feature now? Can you help?

@Rasool-deldar could you explain more about what you mean?

@Rasool-deldar could you explain more about what you mean?

This means using http://local.me/ instead of http: // localhost: 3000 /

I personally think this is out of scope for Nuxt itself. You can move it behind NGINX or a similar proxy to achieve this without much extra work 😋


Kernel_size, pool_size should be positive integers

Kernel_size and pool_size need to be positive integer tuples. If not, a ValueError will be raised.

@googlebot I signed it!

@googlebot I signed it!

It is a PR trying to solve this issue: [https://github.com/tensorflow/tensorflow/issues/51936](Crash Issue).

Summary: Checking if kernel_size and pool_size are positive integers in convolutional layers by creating a util function.

@fchollet @qlzh727 Would you please provide your comments on this PR?


CFD dataset and load_cfd

Data loader and utility functions for  the CFD data from http://www.nobuyuki-umetani.com/publication/mlcfd_data.zip

The tests are broken - we had also tried to get open3d to work in the CI @kovachki but weren't able to.

I downloaded the zip file from the provided URL. The directory structure does not match the CFD dataset code. From inside mlcfd_data, what is the data_dir supposed to be?


Union command-line peers with persisted peers

If #2135 is merged, we will have the paradoxical situation that the command-line peer list is ignored in favour of a persisted one.  This PR changes the default behaviour to take the union of the persisted list and the command-line list, with a --replace option to state you want exactly that new list.

Having second thoughts about this...

Superseded by #2186/#2187 => closing.


issue #3232 - add resourceType IN where clause to whole-system search subquery

example from test:
runQueryTest(Resource.class, "_type", "Basic,EvidenceVariable,ServiceRequest")`

before:
      SELECT LR0.RESOURCE_TYPE_ID, LR0.LOGICAL_RESOURCE_ID
        FROM LOGICAL_RESOURCES AS LR0
       WHERE LR0.IS_DELETED = 'N'
         AND LR0.RESOURCE_TYPE_ID IN (8,52,126)
         AND EXISTS (
      SELECT 1
        FROM LOGICAL_RESOURCES AS LR1
       WHERE LR1.LOGICAL_RESOURCE_ID = LR0.LOGICAL_RESOURCE_ID)
    ORDER BY LR0.LOGICAL_RESOURCE_ID
 FETCH FIRST 10 ROWS ONLY

after:
      SELECT LR0.RESOURCE_TYPE_ID, LR0.LOGICAL_RESOURCE_ID
        FROM LOGICAL_RESOURCES AS LR0
       WHERE LR0.IS_DELETED = 'N'
         AND LR0.RESOURCE_TYPE_ID IN (8,52,126)
         AND EXISTS (
      SELECT 1
        FROM LOGICAL_RESOURCES AS LR1
       WHERE LR1.LOGICAL_RESOURCE_ID = LR0.LOGICAL_RESOURCE_ID
         AND LR0.RESOURCE_TYPE_ID IN (8,52,126))
    ORDER BY LR0.LOGICAL_RESOURCE_ID
 FETCH FIRST 10 ROWS ONLY

Signed-off-by: Lee Surprenant<EMAIL_ADDRESS>

Just one minor thing. I think the inner filter should be written against LR1, not LR0 just to make it clear:
SELECT LR0.RESOURCE_TYPE_ID, LR0.LOGICAL_RESOURCE_ID
        FROM LOGICAL_RESOURCES AS LR0
       WHERE LR0.IS_DELETED = 'N'
         AND LR0.RESOURCE_TYPE_ID IN (8,52,126)
         AND EXISTS (
      SELECT 1
        FROM LOGICAL_RESOURCES AS LR1
       WHERE LR1.LOGICAL_RESOURCE_ID = LR0.LOGICAL_RESOURCE_ID
         AND LR1.RESOURCE_TYPE_ID IN (8,52,126))
    ORDER BY LR0.LOGICAL_RESOURCE_ID
 FETCH FIRST 10 ROWS ONLY

Also, I just thought of another variant which might be worth a look:
SELECT LR0.RESOURCE_TYPE_ID, LR0.LOGICAL_RESOURCE_ID
        FROM LOGICAL_RESOURCES AS LR0
       WHERE LR0.IS_DELETED = 'N'
         AND LR0.RESOURCE_TYPE_ID IN (8,52,126)
         AND EXISTS (
      SELECT 1
        FROM LOGICAL_RESOURCES AS LR1
       WHERE LR1.RESOURCE_TYPE_ID = LR0.RESOURCE_TYPE_ID
         AND LR1.LOGICAL_ID = LR0.LOGICAL_ID
         AND LR1.RESOURCE_TYPE_ID IN (8,52,126))
    ORDER BY LR0.RESOURCE_TYPE_ID, LR0.LOGICAL_ID
 FETCH FIRST 10 ROWS ONLY

I'm not sure if the ORDER BY LOGICAL_RESOURCE_ID is required other than to ensure consistency. As long as we only need it to be ordered and not ordered specifically by LOGICAL_RESOURCE_ID then the above could work because it should follow the index we have on resource_type_id, logical_id.

It does look like PostgreSQL optimizes an index scan with an ANY clause. On a table with 3M rows:
explain analyze
SELECT lr0.logical_resource_id
  FROM fhirdata.logical_resources AS lr0
 WHERE resource_type_id IN (97,19,28)
ORDER BY resource_type_id, logical_id
FETCH FIRST 10 ROWS ONLY;
                                                                           QUERY PLAN
----------------------------------------------------------------------------------------------------------------------------------------------------------------
 Limit  (cost=0.56..3.13 rows=10 width=61) (actual time=0.047..0.064 rows=10 loops=1)
   ->  Index Scan using unq_logical_resources on logical_resources lr0  (cost=0.56..383900.40 rows=1490049 width=61) (actual time=0.045..0.060 rows=10 loops=1)
         Index Cond: (resource_type_id = ANY ('{97,19,28}'::integer[]))
 Planning Time: 0.256 ms
 Execution Time: 0.112 ms
(5 rows)

Note that there's no sort required, because the index is traversed in the correct order (even though the ANY list is not in order). I'm a little surprised that the estimate doesn't include the stop limit (10 rows).

I spoke with robin offline.  Although LR1.RESOURCE_TYPE_ID IN (8,52,126)) seems like a simple change from LR0.RESOURCE_TYPE_ID IN (8,52,126)), it would actually be a bit more involved because of how the query is being constructed.  Since it should be functionally equivalent, I'd prefer to leave it as-is for now.
We can save the additional query variation for future experimentation as well, but first
we should run through the full query performance regression suite and see if there's any surprises with the current changeset.


Merge pull request #4 from calpa/master

Merge pull request #37 from MCLofton/import-extensions

Deploy preview for calpa ready!
Built with commit 43ead4a3c22ea906e71c71e776a533a4bae47198
https://deploy-preview-39--calpa.netlify.com


datepicker - Invalid date manually input is correctly detected in chrome but not in Firefox or IE.

for example entering date '50/12/2014' is detected as an invalid date in chrome as it should, but not in ie or firefox. Sample input tag shown below:
input type="text" class="form-control" datepicker-popup="MM/dd/yyyy" ng-model="ngModel" is-open="opened" close-text="Close" show-weeks="false" placeholder=""  ng-required="true"

Do you use the datepicker as a forms control? Did you check how the form field looks like by printing it as json? (My date object is: {{ myform.mydate | json }}).
Please do so and observe the model value. Probably it was parsed as "Monday, February 12, 2018" (exactly like here http://angular-ui.github.io/bootstrap/#/datepicker).
I just commented on other issue (https://github.com/angular-ui/bootstrap/issues/3104) so just check it at the evening - I'll share my approach how to handle this problem.

plunkr showing the problem please.

http://plnkr.co/edit/yNuSZPiQYFn0Zy2COPdA

PR created: https://github.com/angular-ui/bootstrap/pull/3288
Please review.

I found also something interesting today. Maybe it's the same place to search the problem.
Using the newest version (0.13.0)
If you select a date format other than yyyy-mm-dd and your date input numbers aren't zero padded, the date is parsed wrong in Chrome. In InternetExplorer and Firefox it's not parsing at all, doesn't matter if with padding zeros or not.
Example: http://plnkr.co/edit/9TzjocE0P8F9gD7OwPoH?p=preview
Select date format dd.mm.yyyy and take as example today:
Chrome
input -> 8.5.2015
expected -> 2015-05-08
output -> 2015-08-05
input -> 08.05.2015
output -> 2015-05-08 as expected
Firefox
input -> 8.5.2015
error -> "Datepicker directive: "ng-model" value must be a Date object, a number of milliseconds since 01.01.1970 or a string representing an RFC2822 or ISO 8601 date."
IE
input -> 8.5.2015
no output or error


More extensive XMPP Features

Just discovered this repo last night, I suspect this is how people like FNPL and FortniteTracker are making bot accounts. I used to be making a node.js program Konsole with Jaren until he got a c&d from Epic. We did pretty extensive xmpp stuff such as faking being in game, forcing people to ready up, infinite emotes, etc. Obviously forcing ready up won't work on a bot (theoretically) However I think some things like showing status as in the Epic launcher vs in Fortnite and being able to set if it's in game or not would be nice additions.
Here's what it looks like in the Launcher vs in Fortnite:

"In game" status has been implemented in<EMAIL_ADDRESS>and<EMAIL_ADDRESS>🎉


clarify HTTP/2 stream as transport with link

Since HTTP/2 is referenced later, and we target it, I'm calling it out in the transport section, but being specific to it being HTTP/2 streams that we target, as most HTTP/2 clients don't expose stream APIs.

@tmontgomery you okay with this wording change, or can you recommend a better way?
This comes from feedback about how we mention HTTP/2 in the later table, but not in the earlier sentence about what we've targeted and tested.


Support Metabase actions

Describe the bug
Clickhouse driver for Metabase currently does not support Metabase Actions

I would love for Metabase Actions to be supported
Steps to reproduce

Create new dashboard
Edit -> Add action button -> Pick an action -> Choose any model -> Create new action
Put any text into the Action text field
Click Save -> Enter name -> Save

Expected behaviour
Actions can be created
Error log
clickhouse Database 2 "Clickhouse" does not support actions.
Configuration
Environment

metabase-clickhouse-driver version: 1.1.2
metabase-clickhouse-driver configuration: default
Metabase version: 0.46.4
OS: Linux

We should implement this when the actions API is fully stabilized. Currently, the feature is not fully stable and unlikely to work if we just enable it for CH; as I understood, even in the Metabase core integrations, it is implemented only in MySQL/Postgres/H2.


Messages ruby dtos serialization and deserialization (fixes #1605)

This PR addresses the feedback I gave in my review of https://github.com/cucumber/common/pull/1605
All serialization/deserialization logic is implemented in generated classes. The message superclass with reflection logic is gone - I don't think it would have been possible to implement this properly with reflection unless we load the JSON schema at runtime to get the types for various fields (we don't want that - it would be complicated).
I've also updated the NdjsonToMessageEnumerator to emit proper Envelope objects and updated the Ruby demo formatter accordingly.

When I added support for serialization/deserialization of Array fields, I discovered a bug in the schema for TestStep, so I fixed that too (all Array fields should be required so that they default to an empty array).
This had some ripple-effect to other parts of the monorepo (compatibility-kit, fake-cucumber, react)

@aslakhellesoy do you want to keep it opened for further discussion?

do you want to keep it opened for further discussion?

No, I think it's good to merge now

#1605 has been merged
It will be difficult to merge both PR
My question was more "do we keep it opened or do we close it" 😅

Ah my bad, I was mixing up the two issues. Closing this one.


Feature request: Better test story

I was trying out fresh, moving from a simple Oak implementation where I had the whole development story done, testing, deployment, etc..
I love the design of fresh, except when you get to the main.ts file, and how it manages the server life-cycle.
The thing I was looking for is a model like Oak, that can be started on a port, or called directly with handle in tests.
Overall, I went looking through fresh code, and was debating on whether to emulate the Oak model. Then stepping away for a bit, I'd want to know why not just use Oak as the server to begin with?

You know, I ended do this...
my-project/main.ts
/// <reference no-default-lib="true" />
/// <reference lib="dom" />
/// <reference lib="dom.asynciterable" />
/// <reference lib="deno.ns" />
/// <reference lib="deno.unstable" />

import init from './utils/init.ts';

const app = await init();
await app.start();

my-project/utils/init.ts
import { InnerRenderFunction, RenderContext } from '$fresh/server.ts';
import manifest from '../fresh.gen.ts';

import { config, setup } from '@twind';
import { virtualSheet } from 'twind/sheets';
import Application from './Application.ts';

const sheet = virtualSheet();
sheet.reset();
setup({ ...config, sheet });

function render(ctx: RenderContext, render: InnerRenderFunction) {
	const snapshot = ctx.state.get('twind') as unknown[] | null;
	sheet.reset(snapshot || undefined);
	render();
	ctx.styles.splice(0, ctx.styles.length, ...(sheet).target);
	const newSnapshot = sheet.reset();
	ctx.state.set('twind', newSnapshot);
}

export default async function init() {
	return Application.init(manifest, { render });
}

my-project/utils/Application.ts
import { FreshOptions, Manifest, ServerContext, StartOptions } from '$fresh/server.ts';
import { Handler, serve } from<EMAIL_ADDRESS>
class Application {
	constructor(public handle: Handler) {}
	static async init(routes: Manifest, opts: FreshOptions = {}) {
		const ctx = await ServerContext.fromManifest(routes, opts);
		return new this(ctx.handler());
	}
	start(opts?: StartOptions) {
		return serve(this.handle, opts);
	}
}

export default Application;

With the above in place, I can now test like this:
/// <reference no-default-lib="true" />
/// <reference lib="dom" />
/// <reference lib="dom.asynciterable" />
/// <reference lib="deno.ns" />
/// <reference lib="deno.unstable" />

import { assert, assertEquals } from<EMAIL_ADDRESS>import init from '../utils/init.ts';

const shouldMock = Deno.env.get('MOCK_APP') !== 'true';
const fetcher = shouldMock ? (await init()).fetch : fetch;

Deno.test(`get '/'`, async () => {
	const req = new Request('http://localhost:8000/', { method: 'GET' });
	const res = await fetcher(req);
	assert(res?.ok);

	const text = await res.text();
	assert(text.includes('Welcome to `fresh`'), text);
});

I would love if new projects included a test that fetches the homepage and makes an assertion against it.
Which would require the framework to provide an out of the box way to make requests to against the app server in unit tests.


Top Home-Section is now at the bottom of homepage

Dear all,
I just did some basic updates on my Hugo website with Academic and from now on, my "about" section which is supposed to be the first one to be shown is at the bottom of the page...
If you want to see by yourself : http://www.eurecom.fr/~dugelay/
In my index.html it is still the firt one to be called
I'm using Hugo v0.41. Even with "-v", I see no errors.
Do you any tips to give me on this ?
Thanks a lot in advance

Thank you so much @gcushen
In home/about.md I change weight=0 to weight=1 and everything is back to normal !
Thank you again !


For loop not supported by Naga

Expected Behaviour
Disabling spirt should not fix runtime errors.
Example & Steps To Reproduce
Compiling the following:
#[spirv(fragment)]
pub fn fragment(
    #[spirv(storage_buffer, descriptor_set = 0, binding = 0)] quads: &[InstancedQuad],
    #[spirv(descriptor_set = 1, binding = 0)] surface: &Image2d,
    #[spirv(descriptor_set = 1, binding = 1)] sampler: &Sampler,
    #[spirv(push_constant)] constants: &ShaderConstants,
    #[spirv(flat)] instance_index: i32,
    #[spirv(frag_coord)] surface_position: Vec4,
    out_color: &mut Vec4,
) {
    let quad = quads[instance_index as usize];

    if quad.blur != 0 {
        for y in -quad.blur..(quad.blur + 1) {
            for x in -quad.blur..(quad.blur + 1) {
                let offset = vec2(x as f32, y as f32);
                let sample_pos = (surface_position.xy() + offset) / constants.surface_size;
                let sample = surface.sample_by_lod(*sampler, sample_pos, 0.);
                *out_color += sample / (quad.blur * quad.blur) as f32;
            }
        }

        *out_color = *out_color * quad.color;
    } else {
        *out_color = quad.color;
    }
}

Causes a panic at runtime in the naga crate:
thread 'main' panicked at 'assertion failed: prev.is_none()', C:\Users\KaySimmons\.cargo\git\checkouts\naga-dbb2b19faed4
9210\f59668c\src\front\spv\mod.rs:3151:25

Adding the --no-spirt compile arg fixes this issue.
This issue upstream seems relevant: https://github.com/gfx-rs/naga/issues/1900
System Info

Rust: rustc 1.68.0-nightly (5ce39f42b 2023-01-20)
OS: Windows 11
GPU: Intel(R) UHD Graphics
SPIR-V: SPIRV-Tools v2023.2 v2022.4-135-g44d72a9b

Rewriting the for loop as a loop with break, or while loop doesn't fix the issue. Similarly doing a single iteration without a loop works fine, so the loop itself seems to be the issue when using spirt

See these links for more details:

https://github.com/gfx-rs/naga/issues/1977
https://github.com/EmbarkStudios/spirt/issues/9
https://github.com/gfx-rs/naga/pull/2290

Assuming you're using Naga 0.11, this may work:
[patch.crates-io]
naga = { git = "https://github.com/EmbarkStudios/naga", branch = "spv-in-break-if-v0.11.0" }

This is 100% a Naga bug (they're not implementing a legal form of SPIR-V control-flow) and will be fixed independently of Rust-GPU.

Sweet! Thank you for the details.


Week 7 Review Ticket - Byron Lu

Total Score /4.5      Graded By: Tanay Rayavarapu
Create Task
Test Corrections

[ ] GH Indi Page, Review Ticket, Crossover graded, Study Plan

Monday - Create study plan
Tuesday - FInal MCQ 3
Wednesday - Test Corrections for MCQ
Thursday - Finish Create Task and Practice improv
Friday - Improv csp skit

[ ] Create Task and Create Task Crossover Review comments (completed by Thu)
[ ] Quiz3 MCQ results and test corrections
[ ] Participation: .20 for classroom presence and participation each day
[ ] Quality of work: grader should be looking at effort toward learning and understanding vs perfunctory bullet chasing (0, .25, .5)

Total Score: 4.25/4.5

[ ] GH Indi Page, Review Ticket, Crossover graded, Study Plan

[ ] Create Task and Create Task Crossover Review comments (completed by Thu)

[ ] Quiz3 MCQ results and test corrections

Individual Plans with Study Plan (1/1):

Looks promising and maps out the work for the entire week, including the improv skit, which was important for Friday's work; however, your study plan could be more descriptive such as "reflecting on the mcq and what improvements I can make"

Finals Quiz 3 Score: 1/1

Corrections: Your corrections are good and reflect your level of understanding and improvement for the upcoming AP Exam. Your responses provide a reason why your original answer was a mistake and why your new answer is correct.

Create Task + Review (Score: 0.75/1)

I cannot find the review ticket from a crossover grader, but your create task looks like you developed it well and put a lot of effort into the written responses. I do not know how you got your create task to be reviewed, but next time document it on your github pages to earn the points.

Participation: 1/1
Showed up to class everyday of the week, participated in the mcq's and was dilligently working on study during class. I noticed that you also took part in the improv/skit.
Quality of Work: 0.5/0.5
Good quality of work, however you were missing some stuff. I like the depth of your test corrections and how you can reflect on your mistakes so that you do not make the same mistake again.


DeprecationWarning: Calling an asynchronous function without callback is deprecated.

Getting this when running uploadFile
DeprecationWarning: Calling an asynchronous function without callback is deprecated.
Maybe related to #145 (not the mime part but the graceful-fs part)
Node version: 7.6
Lib version: 4.4.0

Any updates on this? Getting this same thing.

@fishcharlie not to my knowledge. Have you tried to use https://nodejs.org/api/cli.html#cli_throw_deprecation to spot the issue?

The issue seems resolved, I'll close. Feel free to open if you still have it.


added provider block for dynamodb table

Issue #, if available:
Adding provider block in the dynamoDB configuration. Tried deploying this solution where my CLI defaulted to another region other than us-west-2 where i had a SCP and the deployment failed.
Description of changes:
Change provider block to awsoregon.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

All good! Thanks!


Created Share Feature

Created a share feature for individual locations. Allows for email sharing and a copyable link

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

12 of 29 (41.38%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.0008%) to 41.355%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

client/src/pages/LocationPage/LeftColumnDetails.tsx
2
7
28.57%

client/src/pages/LocationPage/SharePopup.tsx
10
22
45.45%

Files with Coverage Reduction
New Missed Lines
%

client/src/pages/LocationPage/LeftColumnDetails.tsx
1
54.55%

Totals

Change from base Build<PHONE_NUMBER>:
0.0008%

Covered Lines:
325

Relevant Lines:
610

💛  - Coveralls


ArgumentListWrapping reports false positive since 0.41.0

I have the following code snippet:
class MyClass {

    private fun initCoilOkHttp() {
        Coil.setImageLoader(
            ImageLoader.Builder(this)
                .crossfade(true)
                .okHttpClient(
                    okHttpClient.newBuilder()
                        .cache(CoilUtils.createDefaultCache(this))
                        .build()
                )
                .build()
        )
    }
}

Expected Behavior
Ktlint should not report any issues
Observed Behavior
Since Ktlint 0.41.0, the ArgumentListWrappingRule reports the following:
Unexpected indentation (expected 16, actual 20)
Unexpected indentation (expected 12, actual 16)

It means it tells this would be correct:
class MyClass {

    private fun initCoilOkHttp() {
        Coil.setImageLoader(
            ImageLoader.Builder(this)
                .crossfade(true)
                .okHttpClient(
                okHttpClient.newBuilder()
                    .cache(CoilUtils.createDefaultCache(this))
                    .build()
            )
                .build()
        )
    }
}

Steps to Reproduce
Run ktlint codesnippet.kt --android --experimental
Your Environment

Version of ktlint used: 0.41.0
Name and version (or code for custom task) of integration used (Gradle plugin, Maven plugin, command line, custom Gradle task): command line
Operating System and version: macOS 11.2.3

I have solved issue's like this in PR https://github.com/pinterest/ktlint/pull/1052 which is still waiting to be merged.

Thanks! However i've just built your branch and my case above still produced the false positives.

Sorry for miscommunication. The PR mentioned above is the first PR of a range of 5 PR's. Originally I submitted one big PR resolving many identation issues. On request of maintainers I have split it into multiple smaller PR's. The other PR's can only be submitted once the first is merged.

Looks like it was introduced in this commit: https://github.com/pinterest/ktlint/commit/3591e61cd08bf7f55e342c533eb488facb231579

Messed with this issue again on 0.42.1
fun onSendMessageClicked() {
    flow { chat.sendMessage(state.messageInput) }
        .onEach { state ->
            updateState(
                messageIsSending = state.isLoading, // <-- ArgumentListWrapping
                messageInput = "", // <-- ArgumentListWrapping
            ) // <-- ArgumentListWrapping
        }
        .launchIn(coroutineScope)
}

@osipxd You commented a closed issue. Also it is not clear what the problem or your expectation is. Maybe it is better to open a new issue?


Fixes Fetch.RequestInit.make defaults

Hopefully fixes #44. Waiting for verification

Hoping for the best...


Try source of apt-add-repository to determine appropriate parameters

Only use --no-updateparameter for apt-add-repository if it comes from software-properties-common
hopefully a working and viable solution rather than checking and parsing parameters
works on Ubuntu Mate (with #366 in place)
need to test on

[x] Mint
[ ] Pop
[x] Debian
closes #370

@flexiondotorg on Debian the list actions list "ppa" applications (with blank publication info and so if you try and install a ppa computer says no which is probably for the best in most cases ) . Would it be better not to list apps that debian users cannot deb-get install ? (Or perhaps hide them behind a --show-unsupportedswitch ? )


Can not installed LibSndFile package

Input:
>Pkg.add("LibSndFile")
return the error message:
Resolving package versions... Unsatisfiable requirements detected for package FixedPointNumbers [53c48c17]:  FixedPointNumbers [53c48c17] log:  ├─possible versions are: [0.5.0-0.5.3, 0.6.0-0.6.1, 0.7.0-0.7.1, 0.8.0-0.8.4] or uninstalled  ├─restricted by compatibility requirements with MLDatasets [eb30cadb] to versions: [0.5.0-0.5.3, 0.6.0-0.6.1, 0.7.0-0.7.1, 0.8.0-0.8.4]  │ └─MLDatasets [eb30cadb] log:......................
===================================
BackGround Info:
Julia Version 1.5.1
Commit 697e782ab8 (2020-08-25 20:08 UTC)
Platform Info:
OS: Windows (x86_64-w64-mingw32)
CPU: AMD Ryzen 5 3600X 6-Core Processor
WORD_SIZE: 64
LIBM: libopenlibm
LLVM: libLLVM-9.0.1 (ORCJIT, znver2)
Environment:
JULIA.EXECUTABLEPATH = C:\Users\USER\AppData\Local\Julia-1.3.1\bin\ & set JULIA_NUM_THREADS=12 & julia.exe
JULIA_NUM_THREADS = 12

FYI: Your issue is likely caused by SampledSignals.jl requiring an old version of FixedPointNumbers
-->https://github.com/JuliaAudio/SampledSignals.jl/issues/64#issuecomment-830925450


Missing methods that start async operations

Methods such as:

Gtk.FileDialog.Save
Gtk.FileDialog.Open
Gtk.FileDialog.SelectFolder
Gtk.UriLauncher.Launch
Gtk.ColorDialog.ChooseRgba

are missing. Although finishing methods exist (like Gtk.FileDialog.SaveFinish etc).
Gtk.FileDialog, Gtk.UriLauncher and Gtk.ColorDialog were added in GTK 4.10 and so weren't present in gir.core 0.3.0 at all.

The build log contains e.g. Did not generate method 'FileDialog.Open': GirModel.AnyType: GirModel.Type: GirLoader.Output.Callback: Async scope not yet implemented so there are some cases not yet handled in the generator

Related to #893.


Crew hierarchy endpoint not working

Hi there,
I have provlem with getting data about my crew.
I have such curl command:
curl -v -XGET -H "Content-type: application/json" 'http://pl.socialclub.rockstargames.com/crewapi/[CREW_NAME]/hierarchy/10/false?_=141977749113' -L --insecure
Curl is still trying to connect to the IP address (trying <IP_ADDRESS>) and then I got connection timed out error.
In my browser link http://pl.socialclub.rockstargames.com/crewapi/[CREW_NAME]/hierarchy/10/false?_=141977749113 is working correctly, it returns me asked data.
Why is that?
Has something changed in the API structure?
Thanks, best regards

Rockstar implemented a bunch of mean HTTP header checks to avoid requests sent by non-browsers like cURL. You need to sent exactly the same headers as your browser does, see this comment on issue #11: https://github.com/gta5-map/Social-Club-API-cheat-sheet/issues/1#issuecomment-153733540


fix small doc typo

Fixes Or Enhances # .
Make sure that you've checked the boxes below before you submit PR:

[x] Tests exist or have been written that cover this particular change.

Change Details:

fixed small typo on Length docs

@go-playground/admins

Thanks @ellisonleao


Fix become/ansible_become precedence

Fix issue #2969.

ci check this

I don't know why the CI is reported as failed, when I click it looks like it succeded ...
Anyway, problem resolved, the PR is not needed anymore.


ci: use charmcraft test rather than actions-operator

Early draft that converts this charm to use charmcraft test, rather than actions-operator.
This means that one can, locally:
# Clone the repo
❯ git clone -b craft-test https://github.com/jnsgruk/zinc-k8s-operator
❯ cd zinc-k8s-operator

# List available tests
❯ charmcraft test --list lxd
lxd:ubuntu-24.04:tests/spread/deploy:juju_4_0
lxd:ubuntu-24.04:tests/spread/deploy:juju_3_5
lxd:ubuntu-24.04:tests/spread/ingress-path:juju_3_5
lxd:ubuntu-24.04:tests/spread/ingress-path:juju_4_0
lxd:ubuntu-24.04:tests/spread/ingress:juju_4_0
lxd:ubuntu-24.04:tests/spread/ingress:juju_3_5
lxd:ubuntu-24.04:tests/spread/observability-relations:juju_3_5
lxd:ubuntu-24.04:tests/spread/observability-relations:juju_4_0

# Run the test suites, each in their own VM
❯ charmcraft test -v lxd

# Run a specific test suite in a LXD VM
❯ charmcraft test -v lxd:ubuntu-24.04:tests/spread/deploy:juju_3_5

Which will spin up a local LXD VM for each of the test suites listed, and run them. Note that there are multiple variants defined (juju_4_0, juju_3_5), which are all defined in the top-level spread.yaml using the JUJU_CHANNEL/3_5 and JUJU_CHANNEL/4_0 notation. Adding another variant to each would be as simple as adding a line such as JUJU_CHANNEL/3_6: 3.6/beta.
In CI, we do something different, and use spread to just run the tests directly on the CI runner with charmcraft test local:, which uses an ad-hoc local backend which causes spread to ssh into the system it's running on! In this case, I've chosen to generate this matrix dynamically using charmcraft list local, then mangle that into a matrix we can use with Github natively -- the other way to do this is to give spread some credentials to a cloud and just run charmcraft test openstack, for example, and it'll go and fan out the number of VMs required.
This WIP also includes the addition of a continue-on-error for the Juju 4.0 testing. I think this is important for us to incorporate going forward: centrally introducing non-blocking test runs for new versions of Juju, such that teams get a heads up if an RC is going to break them, and the Juju team can get more signal about how their upcoming builds might affect the charm teams.
I think this gets us closer to having the same experience locally, and in CI, and should make iteration time faster for our developers. There is the option to not destroy the provisioned test vms in between runs on the local machine with:
charmcraft.spread -reuse lxd

Worth noting that the setup of the machine here is done by a bash script and some associated common functions in common.sh, but in the long run that script could be:

Centrally hosted and re-used between teams (better, but not ideal)
Converted into a charm-test-setup snap that we can all use, and encapsulates the logic of setting up the machines with some parameters, e.g.:

charm-test-setup --juju-channel 3.4/stable --microk8s-channel 1.31-strict/stable ...

(Probably not the end experience we desire, but hopefully makes my point!)
Feedback welcome, we'll be iterating on this experience in the coming weeks/months!

(4.0 tests fail, because python-libjuju doesn't yet support Juju 4.0! This was expected!)


Scott Wlaschin's AsyncResult missing functionality

Do you intend to implement some of the functions Scott Wlaschin has in its AsyncResult/Result modules, e.g.
AsyncResult.catch
AsyncResult.ofAsync
AsyncResult.ofSuccess
AsyncResult.ofError
AsyncResult.ofResult?
https://github.com/swlaschin/DomainModelingMadeFunctional/blob/master/src/OrderTakingEvolved/Result.fs

If you're using the Computation Expression for asyncResult most of these are built in.  If you're trying to compose solely with functions, these are missing. This would make for a great first PR 😃 .

We would also want similar functions added to TaskResult and JobResult.

@TheAngryByrd, following our recent Slack discussion about error handling, I'd love to work on this and give something back to the community. However, this would be my first F# PR ever and it might take a while, before I am addressing this in my project.
Any advice how to contribute? Which toolchain / tools should I use? I am using Visual Studio 2019 with Nuget and don't know anything about Package, FAKE, yet.
Also, I didn't know Expecto even existed until 5 minutes ago. :laughing:

Good point, we don't have prerequisites on the README.  I'll have to add those.  Since we build for netstandard2.0;net461 you'll want at least dotnet core 2.1 and at least full framework 4.6.1 installed.  Additionally since we transpile to javascript with Fable, you'll want to have node LTS installed.
Also it looks like we need to make a build.cmd for this project too.
As for right now you can do
dotnet tool install fake-cli -g
fake run build.fsx target RunFableTests

Report back any errors so we can fix up this experience when contributing from windows.

Is anyone actively working on this? If not might be a fun weekend project for me

No, not right now. Unfortunately, I've never managed to give it a try due to other projects. Please, go ahead.

Sounds good. I got tied up last weekend but I should be able to get started soon.  As I understand it what we want to do here is add some static helpers like:

Closign via #124


Add NSG exceptional rules for the K8s master SSH service

This PR addresses the issue #45. To test, set environment variable export MS_CORP=1 before provisioning the resources.
There's a web job scanning for the NSG in MS subscriptions, and apply restrictions on the access to the MS corp network. As a result, if we deploy the resources to the MS subscriptions, they cannot access each other so the CI/CD cannot proceed. (There's only a few exceptional cases like port 443).
We cannot add NSG rules to allow access from a fixed IP address set. We can get the service IP for those pre-allocated services, like the Jenkins master, but we cannot get the public IP for the Jenkins slaves provisioned by the Kubernetes plugin, because they may not have public IP at all.
As a result, an exceptional rule with Internet source to the port 22 was added. The web job deletes the wildcard rule if the port is fixed to 22, so we have our rule defined as 21-23.
It's working as I tested, however, it is not an ideal solution.

@ZhijunZhao Could you help to review this? I don't have permission to update the reviewers.

@ArieShout Working on it.

close #45


Loading TextureMapObject from TMX file

Hey
I ran into a problem while developing. In my tmx maps, I use texture objects. During parsing the map, I noticed that these objects were only loaded as RectangleObjects. As I was reviewing the loader, I noticed that TextureMapObjects, were never created.
I edited the BaseTmxMapLoader, so that these Objects are created as well. They'll be added to the map the same way as the other MapObjects are.
I'm not sure if somebody else already fixed this (searched, but didn't found anything) or if I completely missed a point somewhere, so I'd love if somebody could review this :)

Could you clean up that commit history please?

Will do

Change looks OK otherwise.

Ok, thanks. I'll need a minute or two to figure out how to do that in git ;)

Puhh, git is quite something to get your head around :) Though I start to see why everyone likes it.
I think all my changes are now in a single commit, though I'm not quite sure.

Could you also please add this new feature to the CHANGES file?

Also, where is the TextureMapObject.java file? Did you forget to add it to the PR? This wouldn't compile without it.

I'll add them later. Actually, I had some problems getting the test to run. I develop mostly in Netbeans.
Is there a reason why the test are not in a maven project? If not, would you mind me adding it?

Its a large undertaking to get that working across all IDEs for something that would only be used by only a handful of people, so this is probably something that would only happen with a PR;  as long as it didn't disrupt the build, I wouldn't be against it.
Painless way, use eclipse for LibGDX development, even if it isn't your go to IDE.

Once the requested changes have been addressed, I'd be happy to merge this.

@siondream I'll make the changes this evening :)

@siondream I made the changes you wanted

TextureMapObject is already part of Libgdx, I'm not quite sure how I should add it again. I originally assume that Libgdx would load them without problem, discovering that it didn't. It's actually my reason to make this pull request.
About the impact on Animated Tiles: When loading a TextureObject, it will load a TextureRegion once, meaning it will appear like a static tile. It doesn't break anything, but some people might wonder why their animated tile isn't animated. Maybe an additional object (AnimatedTextureMapObject) could solve the problem, as we talked above. But maybe someone should make another pull request for that?

Sorry to be that guy, but squash commits please!

No reason to be sorry, I'm still getting the hang of git :)
Hope it's better like this.

 Thanks, this looks good!


Full featured example app

It would be nice to see full featured example app.
With working orders, checkout, cart etc.
I think it is most important feature and it would attract more contributors to QOR SDK

@alexmironof thank you for your suggestion, our main focus is still extract common things from EC and config it into qor-example, hopefully we could have a more complete example app soon.
thank you ;)

@jinzhu Thanks for the response! we wil try to make some more PR's with common e-commerce functionality like:

categories
cart
orders
cash on delivery payment method
fixed rate currier delivery method

cool, thank you!
let me know if I could help anything.

@jinzhu Thanks for you support! I'll publish some updates on categories tomorrow to firestudios/qor-example


fix(semantic-conventions): update deprecated messages for some old constants

This updates the @deprecated ... message for some of the old constants
where they referred to constants that now no longer exist, because they
where themselves deprecated or removed.
This is a partial fix for https://github.com/open-telemetry/opentelemetry-js/issues/5025

Note to self: Here is the script I cobbled to do this update: https://gist.github.com/trentm/a51c3271d17c184f1a34b64ba1170e41


Adafruit_STMPE610::readRegister8 calls global SPI object vice _spi member

-Arduino board:  ESP32 HUZZAH  Feather
-Using VSC Platformio
-Create a new STMPE610 object with (int,SPIClass*) constructor, passing an HSPI configured SPIClass, then call being(). The VPSI bus will then get a spurious endTransaction call.
In STMPE610.cpp, Adafruit_STMPE610::readRegister8, lines 284-286 reflect the following.
    if (_CLK == -1)
      SPI.endTransaction();
  }

This is likely transparent when using the default SPI port configured in SPI.h. Replacing line 285 with the below enabled using this library on other ports.
_spi->endTransaction();

thanks, please submit a PR, we'll check it out!


improving tests and new rate

Changing the calculations to make sure that the rates that are applied are the same kind of the event, and not other.
Changed the UsageType search to find only rates of the defined category (i.e.: 'Vm' or 'Container' or 'Service')
Added a new function to ShowbackPool to make sure that we can call rate with only data and not with an event from the database.
Updated the tests to cope with this.
Changed event generator due to some problems with the event refresh when the factory was used out of the box.

Coverage decreased (-1.0%) to 97.172% when pulling 37735a290b43ba9ab12a65d46e8b09259ec5d9b8 on sergio-ocon:price_plan_refactor into f581cb6e5c7f37e3bd2bcb18e9cbda6bb94645e1 on miq-consumption:master.

Coverage decreased (-1.0%) to 97.172% when pulling 37735a290b43ba9ab12a65d46e8b09259ec5d9b8 on sergio-ocon:price_plan_refactor into f581cb6e5c7f37e3bd2bcb18e9cbda6bb94645e1 on miq-consumption:master.

Coverage increased (+0.005%) to 98.153% when pulling f260d4469978fc8b0d29325ac88bdc061c064a04 on sergio-ocon:price_plan_refactor into f581cb6e5c7f37e3bd2bcb18e9cbda6bb94645e1 on miq-consumption:master.

@lpichler you are welcome to review the PR 😄

Coverage increased (+0.005%) to 98.153% when pulling 2b872b6b31cfdbf507c44941219cbaa71e9c2126 on sergio-ocon:price_plan_refactor into f581cb6e5c7f37e3bd2bcb18e9cbda6bb94645e1 on miq-consumption:master.

Coverage increased (+0.005%) to 98.153% when pulling 2b872b6b31cfdbf507c44941219cbaa71e9c2126 on sergio-ocon:price_plan_refactor into f581cb6e5c7f37e3bd2bcb18e9cbda6bb94645e1 on miq-consumption:master.


.Net: KernelPlugin is not working AzureOpenAIChatCompletionWithDataConfig

KernelPlugin is not working AzureOpenAIChatCompletionWithDataConfig
I have an application which uses semantic kernel with plugins.
Requirement
Requirement is application gets the data using own data with Azure OpenAI at the same time works with Plugins
Working code
following piece of code is working fine with plugins
builder.Services.AddKernel().AddAzureOpenAIChatCompletion(
             "gpt432k",
             "https://xxx.openai.azure.com/",
             "xxxx"
             );
 builder.Services.AddSingleton<KernelPlugin>(sp => KernelPluginFactory.CreateFromType<WeatherPlugin>(serviceProvider: sp));

Not Working code
the following piece of code not working with plugins
var config = new AzureOpenAIChatCompletionWithDataConfig
{
    CompletionApiKey = "xxx",
    CompletionApiVersion = "2023-08-01-preview",
    CompletionEndpoint = "https://xxx.openai.azure.com",
    CompletionModelId = "gpt432k",
    DataSourceApiKey = "xxxx",
    DataSourceEndpoint = "https://xxx.search.windows.net",
    DataSourceIndex = "xx-index"
};
builder.Services.AddKernel().AddAzureOpenAIChatCompletion(
             config:config
             );
builder.Services.AddSingleton<KernelPlugin>(sp => KernelPluginFactory.CreateFromType<WeatherPlugin>(serviceProvider: sp));

Controller Code (Api)
Following is the code in the controller
 var result = await this._chatCompletionService.GetChatMessageContentAsync
                (
                chatHistory: history,
                executionSettings: openAIPromptExecutionSettings,
                kernel: this._kernel
                );

Summary
When I use AzureOpenAIChatCompletionWithDataConfig with AddAzureOpenAIChatCompletion it doesn't work with plugins only get the data with now data source.

@markwallace-microsoft  @matthewbolanos  any update on the issue?

Requirement is application gets the data using own data with Azure OpenAI at the same time works with Plugins

This is not supported by Azure today:
https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/openai/Azure.AI.OpenAI/README.md#use-chat-functions
"NOTE: The concurrent use of Chat Functions and Azure Chat Extensions on a single request is not yet supported. Supplying both will result in the Chat Functions information being ignored and the operation behaving as if only the Azure Chat Extensions were provided. To address this limitation, consider separating the evaluation of Chat Functions and Azure Chat Extensions across multiple requests in your solution design."

@rupeshtech Here is a new recommended way how to use chat completion with data functionality:
https://github.com/microsoft/semantic-kernel/blob/main/dotnet/samples/Concepts/ChatCompletion/AzureOpenAIWithData_ChatCompletion.cs
As mentioned in previous comment, using chat completion with data together with function calling is not supported by Azure .NET SDK today. In case you have further questions, feel free to re-open this issue or create a new one. Thanks a lot!


Fix warning: comparison of integer expressions of different signedness

json.c:1224:26: warning: comparison of integer expressions of different signedness: ‘int’ and ‘size_t’ {aka ‘long unsigned int’} [-Wsign-compare]
 1224 |         if(n >= 0  &&  n < capacity)
      |                        ~~^~~~~~~~~~
json.c:1231:14: warning: comparison of integer expressions of different signedness: ‘int’ and ‘size_t’ {aka ‘long unsigned int’} [-Wsign-compare]
 1231 |         if(n >= capacity)
      |            ~~^~~~~~~~~~~

Merged, thanks.


Remove regex replacements from import-typescript

Rather than modifying TS's source to try and remove require/module/debugger/etc, we can instead define those variables at the top and leave the source unmodified (which TS can handle), and then remove debugger at esbuild time.
This approach is a lot more flexible for TypeScript. We don't have to think about which random bits of code happen to be regex replaced here, and it could even mean that our package could be shipped minified or optimized in some way, if we end up doing that. Mainly, this means we don't have to add a whole slew of new special cases for the module-ified TS; this approach works for all modern versions of TS.
Supercedes #3352.
This includes #3344.

This PR is now running on TypeScript-Make-Monaco-Builds and is working.

The reason we removed require calls from the file was because webpack and other bundlers were picking them up and inlining nodejs shims in the resulting bundle.

FWIW this shouldn't actually end up mattering for TypeScript internally; we detect that it's not a node system and never actually try and use those shims, if they weren't present. Of course, it's not so great that the bundle size would increase and/or warnings would appear.

Thanks!


Additions to RON level format

[ ] pools
[ ] space size

potentially make spaces growable instead

[x] component storage types

this is now instead implemented using the DefaultStorage trait. No need to complicate the level format with this.

The hope is to have entire Spaces representable in RON format so they can be created with just e.g. Space::from_ron_file(path) and no additional boilerplate.

Obsolete after #38


Add directory crawler populator to yield STAC Collections and Items + CLI utility

Changes

Add request session keyword to all request-related functions and populator methods to allow sharing a common set
of settings (auth, SSL verify, cert) across requests toward the STAC Catalog.
Add DirectoryLoader that allows populating a STAC Catalog with Collections and Items loaded from a crawled directory
hierarchy that contains collection.json files and other .json/.geojson items.
Add a generic CLI stac-populator that can be called to run populator implementations directly
using command stac-populator run <implementation> [impl-args].
Remove hardcoded verify=False to requests calls.
If needed for testing purposes, users should use a custom requests.sessions.Session with verify=False passed to
the populator, or alternatively, employ the CLI argument --no-verify that will accomplish the same behavior.

 The rm could be added before the git clone to make sure the error does not happen, but I prefer to leave it up to the user to avoid removing something they did not intend to wipe.

and if I go into the docker directory and do docker compose up, I'm getting errors like:
(stac) david@it-282:~/src/stac-populator/docker$ docker compose up
[+] Running 2/1
 ✔ Network docker_default            Created                                                                                0.1s
 ✔ Volume "docker_stac-db"           Created                                                                                0.0s
 ⠋ Container stac-populator-test-db  Creating                                                                                0.0s
Error response from daemon: Conflict. The container name "/stac-populator-test-db" is already in use by container "f701152d5d735273fc26e499b75a2bf00de839e08d18ef8f3a12c85efb5b7fda". You have to remove (or rename) that container to be able to reuse that name

I tried to remove that container, but then I get the same error with another hash.

Does adding --rm within the same command that does docker compose up work?

I think I'd prefer something like this (not sure it's actually working):
PYESSV-ARCHIVE-DIR = ~/.esdoc/pyessv-archive

$(PYESSV-ARCHIVE-DIR)%:
	    @echo [ -d $@ ] || git clone "https://github.com/ES-DOC/pyessv-archive" $@
		cd $@ && git pull
``
Not sure I understand where to put the --rm.

PYESSV-ARCHIVE-DIR = ~/.esdoc/pyessv-archive
$(PYESSV-ARCHIVE-DIR)%:
@echo [ -d $@ ] || git clone "https://github.com/ES-DOC/pyessv-archive" $@
cd $@ && git pull

I think something similar could work.

Not sure I understand where to put the --rm.

My bad. I confused with docker run commands. Maybe try docker compose down --rmi then retry make docker-start.


OCP 4.11 install fixes

Fix the issue detected with OCP4.11 related in https://github.com/rcarrata/devsecops-demo/issues/58


Add custom marshaler and unmarshaler to parser and token

hi,
I have added custom marshaler / unmarshaler to parser and token in order to make it more flexible.
My use case: I'm using protobuf generated data type as the claims and protobuf object requires protobuf json encoder / decoder to handle their special cases. (Such as timestamp etc..). So in this pr, I can set the newly added ClaimMarshaler & ClaimUnmarshaler to a custom function to correctly encode / decode object.
I think it may be a general use case, please take a look.

This was a great suggestion. I updated it to take advantage of the new options structure for 4.0. Landed in 4.0 branch.


Replace debugLogger with a better performing solution for internal logging

Debug logger was removed due to performance issues in this PR: https://github.com/rollbar/rollbar-php/pull/406
A more elegant and better-performing solution needs to be implemented.

There is no need for now for this functionality. I removed the remains of verbosity for now. A better implementation might be introduced in the future if there is a need.


[Code review: Contract Bugs]: Recommended use of modifier

Describe
https://github.com/tokamak-network/gem-nft-contract/blob/b31e833a38901bee6287b869d34c523bfe13dea2/src/L1/L1WrappedStakedTONFactory.sol#L24-L26
Impact

Modifier is usually used to check whether the requirements are met before executing the function.
When the same require statement is used multiple times, using a modifier is more intuitive and helps with code management.

Recommendation
modifier nonZeroAddress(address _address) {
        require(_address != address(0), "zeroAddress")
        _;
}

function createWSTONToken(
        address _layer2Address,
        address _depositManager,
        address _seigManager,
        string memory _name,
        string memory _symbol
    ) external onlyOwner
       nonZeroAddress(_layer2Address)
       nonZeroAddress(_layer2Address)
       nonZeroAddress(_layer2Address)
       returns(address)
    {

modification passed : Link

I confirmed, thank you.


DDO-2211 Explicitly create/delete namespaces as part of BEE lifecycle

This PR updates thelma bee delete to delete a BEE's environment namespace before we delete BEE from state.
There's a chance that someone or some CI process could sync the BEE's argo apps and accidentally repopulate the namespace between when the namespace is deleted and the BEE is removed from state. So, I've also updated thelma bee create to create the namespace for the BEE, with the goal of removing the "auto-create namespace" option from BEE argo apps once this PR is merged.
P.S. The PR also includes a random Vault client library upgrade related to some debugging Eno and I were doing earlier. I figured I'd merge it anyway.

Also, to confirm, I believe this looks like this isn't added as an option but as a just part of default behavior (which is fine, just want to make sure i'm reading that right)

Tested manually, seems to work fine.


Transcode of BBB fails with "Invalid Duration"

Request:
{
    "url"<EMAIL_ADDRESS>    "callback_url": "https://webhook.site/cd72124f-33a2-47e0-aacb-562008b91cc1",
    "output_locations": [
        {

                                "type": "object_store",
                                "url"<EMAIL_ADDRESS>
                                "outputs": {
                                        "source_segments": true
                                }
                        }
    ]
}

Source manifest:
segmented-manifest.txt
[2022-10-28 12:15:22] livepeer (387302) INFO: E1028 12:15:22.147193  387325 mediaserver.go:745] http push error processing segment url=http://<IP_ADDRESS>:8935/live/egeedeff/0.ts manifestID=egeedeff err="invalid duration 596.427"

Trace:

go-livepeer parsing of duration
catalyst-api setting the duration

Because of err="invalid duration 596.427" we see mediaDurationMillis was 596427 when passed to transcodeSegment(). It is 1000x more than normal.

Weirdly, the source manfiest we're working from starts with
#EXTM3U
#EXT-X-VERSION:3
#EXT-X-PLAYLIST-TYPE:VOD
#EXT-X-TARGETDURATION:5
#EXT-X-MEDIA-SEQUENCE:0
#EXTINF:10.4160000000,
0.ts


Incorrect reversal of "noël"

esrever.reverse("noël") returns "l̈eon". It should return "lëon".
(From https://mortoray.com/2013/11/27/the-string-type-is-broken/)

I cannot confirm this issue. https://mothereff.in/reverse-string#noël also shows the correct result. Care to elaborate?

@mathiasbynens Sorry. I may not know How To Javascript™. 😆 When I use node to run it from the command line as you show, I get the correct result.
What I tried before was to go to your source in Chrome (Version 51.0.2704.103 (64-bit)), copy-paste the code into the console so that esrever would be defined, and try it from there. In that case, it looks wrong.

You probably have better things to do, but if you feel like taking time to tell me why that doesn't/shouldn't work, I'd appreciate it.
Either way, sorry to bother you with an invalid bug report.

That’s the uncompiled source, not the actual source code. The <%= %> give it away. src/esrever.js is used as a template to generate esrever.js at build time.

I could probably make that clearer by renaming src/esrever.js to something like src/esrever.js.tpl but then again I don’t wanna lose syntax highlighting…

Ah, thanks for clarifying! BTW - after posting this I came across some of your articles on Unicode support in Javascript. Excellent work! 👍


Use device appearance when Force Dark Mode option is turned off

Related Issue
Fixes #373
Checklist:

[x] I have read the contributor's guide.
[x] I linked an issue in the previous section
[x] I have commented on the linked issue
[ ] I was assigned the linked issue (not required)
[x] I have tested the change to the best of my ability against the sandbox or a local build.

Optional items:

[ ] My change adds new text and requires a change to translations.
[ ] My change requires a change to the documentation.
[ ] I have submitted a PR to the documentation repo.
[ ] I was not able to test... (explain below, e.g. you did not have permissions to test a specific feature)
[ ] This change depends O-FISH Realm repository changes (explain below)
[ ] This change depends O-FISH Web repository changes (explain below)

Optional: Add any explanations here

UIUserInterfaceStyle is set to Automatic
Updating of SceneDelegate is removed from UserSettings
SceneDelegate is subscribed on UserSettings property changes instead
preferredColorScheme calls in views are removed as redundant

 Review PRs


python310Packages.pydevccu: 0.1.5 -> 0.1.6

Diff: https://github.com/danielperna84/pydevccu/compare/refs/tags/0.1.5...0.1.6
Changelog: https://github.com/danielperna84/pydevccu/releases/tag/0.1.6
Description of changes

Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
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

[ ] Fits CONTRIBUTING.md.

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

4 packages built:

python310Packages.pydevccu
python310Packages.pydevccu.dist
python311Packages.pydevccu
python311Packages.pydevccu.dist


metrics: Do not return when we cannot find a _stats map

In Tetragon, we have maps ending with _stats that contain metadata about the main maps. An example is that we have execve_map and execve_map_stats that contain errors and other information related to the execve_map map.
Now when we fail to open the _stats map, we just stop the loop and we can possibly miss to check for other maps. This patch changes that and tries for the next map.
Additionally, we do not try to collect stats for maps that already end up with _stats as this would result in _stats_stats map names.
FIXES: https://github.com/cilium/tetragon/issues/1775

Thanks @tpapagian , maybe some log warn only once if the final loadPinnedMap() fails? or maybe doesn't make that much sense ;-)

Thanks! Yes, it makes sense. If we see any of these warning we have to fix that. I added a warning there.

BTW for issue #1775 there is still that (3) rename map, should be moved to separate issue? so it is not lost.

OK, I will post a comment to that issue that (1, 2) are fixed by this PR. So I will not close that.


Calculate bottom padding for nav bar

What 🕵 🔍

What does the pull request solve?
What exactly was added or modified?

How to test 🥼 🔬

A brief description of the steps required for proper testing

Screenshots 📸 📱

Before
After

References 📝 🔗

source name

Codecov Report

Merging #38 (812d6a1) into main (e17b94c) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #38      +/-   ##
==========================================
+ Coverage   57.42%   57.45%   +0.02%
==========================================
  Files          46       46
  Lines        1515     1516       +1
==========================================
+ Hits          870      871       +1
  Misses        645      645

Impacted Files
Coverage Δ

lib/src/widget/nav_bar/widget/nav_bar.dart
98.48% <100.00%> (+0.02%)
:arrow_up:

Continue to review full report at Codecov.

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


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #143 (79ba44b) into main (2974dfe) will increase coverage by 0.38%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #143      +/-   ##
==========================================
+ Coverage   83.33%   83.72%   +0.38%
==========================================
  Files           2        2
  Lines         126      129       +3
  Branches       31       32       +1
==========================================
+ Hits          105      108       +3
  Misses         21       21

Impacted Files
Coverage Δ

example/src/main.ts
88.46% <100.00%> (ø)

src/index.ts
82.52% <100.00%> (+0.52%)
:arrow_up:

Continue to review full report at Codecov.

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


Pathfinder Official Sheet  - Russian Tranlsation

Changes / Comments
It's almost full translation of Pathfinder official (aka Pathinder by Roll20).
I'll complement it from time to time
I hope to see it in Roll20 in my games ;) !

Excellent. Thanks for taking the time. :) I'll have to check it out in Russian.

It's still in english in my game

it's still in english :018.12.2018, 07:12, "Cassie Levett" notifications@github.com:Merged #4503 into master.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.

They changes were only merged. They are not live on the site for about ~5 hours.


Remove deprecated legacy_custom_rules kwarg for iris.fileformats.rules.Loader

The legacy_custom_rules kwarg for iris.fileformats.rules.Loader class was deprecated in Iris 1.9
https://github.com/lbdreyer/iris/blob/015cee9877df87d64e84146975feb9eefea84591/lib/iris/fileformats/rules.py#L872

Closed by #2778


EVENTS, SENDER is an optional parameter

sender parameter always exists
CLASS lcl_event_handler DEFINITION.

  PUBLIC SECTION.
    METHODS:
        on_toolbar
            FOR EVENT toolbar OF cl_gui_alv_grid
                IMPORTING e_object e_interactive sender,

abaplint currently gives error
Variable "SENDER" contains unknown: handler parameter not found "SENDER" (unknown_types) [E]

reproduced as
INTERFACE bar.
  EVENTS moo.
ENDINTERFACE.

CLASS clas DEFINITION.
  PUBLIC SECTION.
    METHODS on_moo FOR EVENT moo OF bar
      IMPORTING sender.
ENDCLASS.
CLASS clas IMPLEMENTATION.
  METHOD on_moo.
  ENDMETHOD.
ENDCLASS.


Does not work with svelte 3.0

This does not work with svelte 3.0.
I plan on submitting a PR soon, but basically the compile function needs to be changed to the new svelte 3.0 compile api
In process version of my updates in case anyone can't wait.
compile(args) {
    if (!this.opts.name) this.opts.name = capitalize(sanitize(args.path))

    return svelte.preprocess(args.data, {}).then(result => {
      let {js, css, ast} = svelte.compile(
        result.toString(),
        Object.assign({ filename: args.path }, this.opts)
      )

      if (this.opts.extractCSS && css) {
        this.cssLookup.set(args.path, {
          code: css,
          map: cssMap,
          path: args.path
        })
      }

      return {
        data: js.code,
        map: js.map
      }
    })
  }

Hi, I would be happy for a PR.


Bug fixes for NonSpecific Search using variable modifications and protein parsimony

Smash the beetles, kuh, kuh, smash em.

Issues remaining:

[ ] Glycines are sometimes lost (same mass as carbamidomethylation)?
[ ] Ghost oxidations on N terminus (incorrect index assignment)
[ ] Oxidations sometimes not present in final sequence near C terminus

Are these issues specific to the non-specific search?

I haven't checked, but they should be.


Blog app - controllers

In this project, I created all controllers and views for my Blog app. As I won't be processing any data yet, I added simple placeholders for my views with plain HTML.
For each URL I created:
    A route.
    An action in the correct controller.
    A view file.

View files should include only placeholders written in plain HTML, e.g:
    Here is a list of posts for a given user

Make sure that you added code to handle only the 4 required URLs.


feat: scale Juju controllers according to anvil cluster size

Add controllers to machines missing them once the cluster size is at least 3, increasing the number of controllers for every odd machine added

Should we add information in the README the explains that juju controllers will grow with the cluster, rather than leaving it transparent to the user?

I just finished another round of review. Apart from the comments, I think that we should also add a scale down process. That can be handled on the same step with extra logic or the already implemented step should be renamed to something like ScaleUpJujuStep and a second one should be introduced: ScaleDownJujuStep. The scale down will just be remove-unit controller/<unit> and has to run when nodes are removed from the anvil cluster.

juju remove-unit controller/<unit> is not supported--it errors with "ERROR removing unit controller/2 failed: removing units from the controller application not supported." AFAIK you can only remove controller machines, which would also force us to remove the other applications on those machines

This is blocked by: https://bugs.launchpad.net/juju/+bug/2073986


Where have usage samples (real-world) disappeared?

I integrated older version (v1.1) of DDMathParser into my Mac App. It was wonderful. Easy to build and use. I cloned, dropped in a few sources, added DDMathParser git repo as a git submodule to my repo, and built it directly into my App.
At that time, there were bunch of great little sample project targets to show you things like - how to bind text fields against an evaluator, how to get errors and display them, how to continuously re-parse math text and display the outcome etc.  and of course - it all was Obj-C.
Now not only DDMathParser became Swift (which regrettably I don't master yet) and a standalone framework, but all the "how to integrate into your app" samples completely disappeared. Of course I can decipher the unit-tests for sample usage (math-wise) and learn independently how to integrate swift framework into my older Obj-C App project - but this is both time consuming and prone to unoptimized integration (my use of DDMathParser is quite heavy - millions of calculations in near real time).
I DO want to replace my old DDMathParser (especially after reading all those issues that have been solved since then) but I am not fluent with Obj-C to Swift integration, and the DDMathPArser also changed several aspects of its behavior
Can the author (or other, more experienced users) add some sample code to the repo, of usage in real-world applications, for the benefit of such as myself?

There's a new Demo app on the swift4 branch that shows how to tap in to each stage of parsing and evaluation. Please check that out and let me know if you have questions. :)


enhance grafana with env vars and bind datasources and dashboards

grafana:
   container_name: grafana
   environment:
     GF_SECURITY_ADMIN_USER: admin
     GF_SECURITY_ADMIN_PASSWORD: admin
     GF_INSTALL_PLUGINS: natel-discrete-panel
     GF_PATHS_PROVISIONING: /etc/grafana/provisioning
   ports:
     - '3000:3000'
   image: grafana/grafana:7.0.3
   volumes:
      - $PWD/datasource.yaml:/etc/grafana/provisioning/datasources/datasource.yaml:ro
      - $PWD/dashboards.yaml:/etc/grafana/provisioning/dashboards/dashboards.yaml:ro
      - $PWD/dashboards:/var/tmp/dashboards

was done a while ago


model.layers is empty after upgrading to 2.5.0

System information

custom CNN model
Linux Ubuntu 20.04
TensorFlow installed from pip
TensorFlow version: v2.5.0-rc3-213-ga4dfb8d1a71 2.5.0
Python version: 3.8.5
CUDA/cuDNN version: 11.1
GPU model and memory: Quadro K5200

I upgraded Tensorflow  to 2.5.0 through pip so I could get support from Cuda 11.1.
After this upgrade, when I tried to access keras.optimizers.Adam, it wasn't there anymore, so I switched to tf.keras as it's recommended I see, but doing so I then realized that model.layers is now an empty list after building my model:
    self.model = keras.models.Model(inputs=[input_img], outputs=[outputs])
    self.model.compile(optimizer=keras.optimizers.Adam(),
                       loss="binary_crossentropy",
                       metrics=["accuracy"])
    print(self.model.layers) -> returns empty list

Tried updating tensorflow to a different computer, same issue with model.layers but I can still access keras.optimizers.Adam (this one has a slightly different setup as I'm using miniconda).

@Earl7Fx3 Did you train the model? Can you please share a simple standalone code to reproduce the issue? Thanks!

Thanks for your reply!
I actually trained the model earlier, and had issues whiles trying to save it... Debugging it, I ended up figuring that model.layers was empty.
Here is a simple standalone code:
import keras
import keras.layers as kl

def cv2d(input_tensor, n_filters, kernel_size=3, batchnorm=True):
    # first layer
    cv = kl.convolutional.Conv2D(filters=n_filters,
                                 kernel_size=(kernel_size, kernel_size),
                                 kernel_initializer="he_normal",
                                 padding="same")(input_tensor)
    if batchnorm:
        cv = kl.BatchNormalization()(cv)
    cv = kl.Activation("relu")(cv)
    # second layer
    cv = kl.convolutional.Conv2D(filters=n_filters,
                                 kernel_size=(kernel_size, kernel_size),
                                 kernel_initializer="he_normal",
                                 padding="same")(cv)
    if batchnorm:
        cv = kl.BatchNormalization()(cv)
    cv = kl.Activation("relu")(cv)
    return cv

def run():
    params = {'img_size': (256, 256, 1),
                  'n_filters': 16,
                  'batchnorm': True,
                  'dropout': 0.5}
    img_size = params['img_size']
    n_filters = params['n_filters']
    batchnorm = params['batchnorm']
    dropout = params['dropout']
    input_img = kl.Input(img_size, name='img')
    c1 = cv2d(input_img, n_filters=n_filters*1,
                                kernel_size=3, batchnorm=batchnorm)
    p1 = kl.pooling.MaxPooling2D((2, 2))(c1)
    p1 = kl.Dropout(dropout*0.5)(p1)

    c2 = cv2d(p1, n_filters=n_filters*2, kernel_size=3,
                                batchnorm=batchnorm)

    outputs = kl.convolutional.Conv2D(1, (1, 1), activation='sigmoid')(c2)
    model = keras.models.Model(inputs=[input_img], outputs=[outputs])
    model.compile(optimizer=keras.optimizers.Adam(),
                  loss="binary_crossentropy",
                  metrics=["accuracy"])
    print(model.layers)

which oututs:
[<keras.engine.input_layer.InputLayer object at 0x7f2e96abb1c0>, <keras.layers.convolutional.Conv2D object at 0x7f2f2c176310>, <keras.layers.normalization_v2.BatchNormalization object at 0x7f2ec1d75e50>, <keras.layers.core.Activation object at 0x7f2e9c3009d0>, <keras.layers.convolutional.Conv2D object at 0x7f2e9c34ae20>, <keras.layers.normalization_v2.BatchNormalization object at 0x7f2e9c070370>, <keras.layers.core.Activation object at 0x7f2ec1349a60>, <keras.layers.pooling.MaxPooling2D object at 0x7f2ec1bf9dc0>, <keras.layers.core.Dropout object at 0x7f2ec1bf9820>, <keras.layers.convolutional.Conv2D object at 0x7f2e96fa45e0>, <keras.layers.normalization_v2.BatchNormalization object at 0x7f2e96fa4a00>, <keras.layers.core.Activation object at 0x7f2e9c304e50>, <keras.layers.convolutional.Conv2D object at 0x7f2e96aa55b0>, <keras.layers.normalization_v2.BatchNormalization object at 0x7f2e96aa5d30>, <keras.layers.core.Activation object at 0x7f2e9c4ff5b0>, <keras.layers.convolutional.Conv2D object at 0x7f2e9c4ff430>]

Replacing "import keras" by "import tensorflow.keras as keras" outputs an empty list.

@Earl7Fx3 I updated couple of lines and imported keras and keras.layers  from tensorflow. With those modiications, i don't see any difference in the output from keras or tensorflow.keras. As tf.keras. has no convolutional attribute,  I updated Conv2D layer in your code.  Please check the gist here.
from tensorflow import keras
from tensorflow.keras import layers as kl

Please verify once and close the issue if this was resolved for you. Thanks!

I just checked, it now works with your modifications.
Thanks a lot for going through this!


pythonPackages.rpyc: enable tests

Motivation for this change
~https://nvd.nist.gov/vuln/detail/CVE-2019-16328~
This was originally a security bump of 4.1.1 -> 4.1.2, but that got superseded. Now this is just about the enabling of the tests that I did at the same time....
Noticed that the tests weren't actually running or present in the pypi tarball, so switched to the github source and fixed up their execution.
Things done

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

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

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

Notify maintainers
cc @

@GrahamcOfBorg build python37Packages.rpyc python38Packages.rpyc

the failure for python38 needs to be handled upstream
builder for '/nix/store/93x9y6yjv4r3alcwdl5maz43admsx9q7-python3.8-rpyc-4.1.2.drv' failed with exit code 1; last 10 log lines:
      codeobj = _import_codetup(codetup)
    File "/nix/store/c1lv2w2fnmjdcnbl1mdvp2vhfvqrjsg4-python3.8-rpyc-4.1.2/lib/python3.8/site-packages/rpyc/utils/teleportation.py", line 125, in _import_codetup
      return CodeType(argcnt, kwargcnt, nloc, stk, flg, codestr, tuple(consts2), names, varnames, filename, name,
  TypeError: an integer is required (got type bytes)

  ----------------------------------------------------------------------
  Ran 57 tests in 31.582s

  FAILED (SKIP=6, errors=3)
cannot build derivation '/nix/store/w87lwcd2knpja4xxvsg27fld8bqclw4x-env.drv': 2 dependencies couldn't be built


Fix deprecation warning when minimum deployment version is iOS 13+

Fixes #8161

Thanks! Can you update the CHANGELOG as well?

@morganchen12 done, thanks!


Copy apimodel_merger.go from aks-engine.

This removes cmd/generate.go's dependency on code from github.com/Azure/aks-engine/pkg/engine/transform. Our end goal is to remove all aks-engine code dependencies.

/lgtm


Analyzer const cast fails for Never?.

Example:
  const v = null as Never?;

The analyzer reports a compile-time error for this:

line 3 • Evaluation of this constant expression throws an exception.

Doesn't say which exception, and it shouldn't since null is a valid value of any nullable type.
(Doesn't check type for nullability?)

I'm guessing that Never? means that it will either return null or never return. There are likely other places in the analyzer / linter where Never? isn't being handled correctly. For example, the code
Never? f() => null;

results in the lint Unnecessary use of the type 'Null'. Try using 'void' instead.. This seems like a poor message, at the very least.
@scheglov @pq

I'm inclined to agree that at least prefer_void_to_null is overreporting.
Issue filed: https://github.com/dart-lang/linter/issues/4867

We've got this one as well: https://github.com/dart-lang/linter/issues/4758.

I'm guessing that Never? means that it will either return null or never return.

Exactly, just like Null.
The type Never? is NORM-equivalent to Null, which means that it should behave exactly the same in most contexts (likely all contexts for this type, other types can have a few exceptions).
It's definitely true that Never? <: Null and Null <: Never?, and their Type objects are equal, and identical if const:
typedef typeof<T> = T;
void main() { const t1 = Null; const t2 = typeof<Never?>; assert(identical(t1, t2)); }


Add Pennylane to sponsors

$100 a month
[...]
We will publish your logo in the README of the project, within the "Sponsors" section.

We've been sponsoring this repo for a couple of months now. Here's our logo.

Thanks!


New embedding models not recognized

The library fails to recognize the encoding for the new embedding models: text-embedding-3-large and text-embedding-3-small
See https://platform.openai.com/docs/guides/embeddings/embedding-models

Hello @thomasdc !
Thanks for raising an issue.
As far as i understand, embeddings algorithms are caclulated on OpenAI side. This algorithm (most likely some neural network) is not publicly available

According to the mentioned docs it uses cl100k_base.

GptEncoding.GetEncodingForModel("text-embedding-ada-002"); was working fine, but throws for text-embedding-3-large

https://github.com/dmitry-brazhenko/SharpToken/pulls

Thanks for your feddback. I just introduced this change. Should work in the newer version (1.2.15) once it is available

Ok thanks, can you add small as well?

@thomasdc  could you share the full name please? I will add it

See my first comment.

Sorry, I missed it.
https://github.com/dmitry-brazhenko/SharpToken/pull/29
it should become available in v.1.2.16
Please let me know if smth does not work


Add paragraph about session context and group functionalities (rebased onto develop)

This is the same as gh-1282 but rebased onto develop.

Related to the group support improvements in OMERO 5.1.4, this commit mentions
the existence of a context associated to a session and the usage of group
parameter/key value to perform read/write operations in a group different than
the context one.

--rebased-from #1282

Staged at http://www.openmicroscopy.org/site/support/omero5.1-staging/developers/Matlab.html#working-in-a-different-group.

As this is a rebase and an improvement, good to merge as-is. :+1:


ORA-1436 Disallow self delegation

Halborn found an attack where someone could steal stake by reputers delegating to themselves. Prevent this attack by disallowing self delegation.

Just waiting to hear back from Halborn their thoughts about this solution before merge. lgtm though


Content is different from actual page

There's no "Configuration" under "Settings" and the options are mostly different. Maybe needs updating.

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

ID: d4f40a37-fa0f-6d54-57ad-8d58c6c3531b
Version Independent ID: 3b2178f9-f6a8-8771-f7f5-7a9952b45b2b
Content: Configure function app settings in Azure Functions
Content Source: articles/azure-functions/functions-how-to-use-azure-function-app-settings.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@JeffreyBai
Thanks for your feedback! We will investigate and update as appropriate.

I'm not sure what you are seeing, but the portal still looks this way for me:

Since I can't reproduce this, #please-close. If you can provide more detailed info or a repro, we can reopen this issue.


GLIBC 2.29

Hello
We are trying to use latest rustic binary however it seems to require GLIBC 2.29 but almalinux have 2.28
NAME="AlmaLinux"
VERSION="8.7 (Stone Smilodon)"

If we install 2.29 it require to update/install many other stuff.
Which version of rustic binary is compatible with GLIBC 2.28
In reference to : https://community.cyberpanel.net/t/cyberpanel-backups-v2-proposal/40370
thanks

The precompiled binaries are compiled using github latest ubuntu linux. If your glibc library version doesn't match, either

compile rustic yourself, using build.sh  (after installing the rust toolchain, e.g. using https://rustup.rs/)
Use the musl version of the binary which includes libc and is basically statically linked.

ok musl version works, thanks alot.