tree-sitter: unquote latest version in update script

Motivation for this change
Otherwise if a tag was found it would try to fetch `refs/tag/"v1.2.0".
fetching list of grammars
checking the tree-sitter repo list against the grammars we know
writing files to /etc/nixos/nixpkgs/pkgs/development/tools/parsing/tree-sitter/grammars
Fetching latest release ("v1.2.1") of tree-sitter-agda …
Initialized empty Git repository in /tmp/git-checkout-tmp-RU9DUpjM/tree-sitter-agda/.git/
fatal: couldn't find remote ref refs/tags/"v1.2.1"
From https://github.com/tree-sitter/tree-sitter-agda
 * branch            HEAD       -> FETCH_HEAD
fatal: Not a valid object name
Unrecognized git object type:
Unable to checkout refs/tags/"v1.2.1" from https://github.com/tree-sitter/tree-sitter-agda.

Could probably be part of the latestGIthubRelease function, but I had no luck with adding the same pipe after the jq call that extracts .name.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

12 packages built:

tree-sitter-grammars.tree-sitter-clojure
tree-sitter-grammars.tree-sitter-cpp
tree-sitter-grammars.tree-sitter-cuda
tree-sitter-grammars.tree-sitter-dockerfile
tree-sitter-grammars.tree-sitter-elixir
tree-sitter-grammars.tree-sitter-fennel
tree-sitter-grammars.tree-sitter-java
tree-sitter-grammars.tree-sitter-latex
tree-sitter-grammars.tree-sitter-norg
tree-sitter-grammars.tree-sitter-perl
tree-sitter-grammars.tree-sitter-vim
tree-sitter-grammars.tree-sitter-zig


Add tests to check undo not resetting tools

Closes #69
This currently adds some tests checking for tools changing when using the unfixed undo. As far as I'm concerned we could remove them when we are ready with the other parts of this PR. They currently exist to check that the tests actually tests something that could be a problem. The same applies to the changes in the M2Backend class (except for the assignment of currentTool in saveStateInCurrentUserAction).

I have changed the undo tests to check whether undoing an action always

results in the image it was before applying that action, and
preserves the whole state of the context.

This means that tests testing unfixed undo not working have been removed because there is no point in testing something that is expecting to not work. If it works now, we're happy, regardless of what was before. I also removed the comparisons with fixed images when undoing since we want to test that undo restores the state prior to the user action and not that undo resets it to some magic state that is defined by the user actions that are tested elsewhere.
We can now decide to add more tests for undo by simply adding a test method calling execute: with no further need of setup.


Reported Challenge #37540 - India - Add Surface to Highway

Challenge: #37540 - India - Add Surface to Highway
Reported by: ZeLonewolf
All motorways are paved.  This challenge is not necessary.  Please see:
https://community.openstreetmap.org/t/maproulette-challenge-add-surface-to-highways-in-california/8200

Thanks for the feedback. We'll make challenge not discoverable. This will be slightly delayed as most of my colleagues are offline as they work in different timezones; it's already the weekend for them. Once this is completed I'll replay back to this ticket. To prevent this from recurring we'll be seeking community input prior to formally launching a new type of challenge.

The challenge is no longer discoverable.

@mvexel I think it can be closed then?


Doctrine Extensions wip-v2.4.0

Sorry to be bothering ...
I have used this bundle, but needed to use version wip2.4.0.
Does anyone could help me?
Thanks,
Paulo

@stof  the stable version is released (v2.4.0)

@maciej-pyszynski the stable version is unrelated to the experimental branch (which would be 3.0 according to semver)

Closing this issue. I won't support the experimental 3.0 branch of the extensions for now, as development of it has staled.


Using yara-x in a Rust library and handling lifetime specifiers of Scanner

Hello,
Related to #139 , but maybe not the same use case:
I'm trying to create a library that uses the yara-x crate. The library should initialize the rules internally, and create a struct that holds an instance of the yara-x Scanner.
Reducted code in scanner.rs:
use yara_x::{Rules, Scanner as YaraScanner};
use anyhow::Result;

struct MyScanner<'a> {
    scanner: YaraScanner<'a>,
}

impl<'a> MyScanner<'a> {
    pub fn new(rules: &'a Rules) -> Self {
        let scanner = YaraScanner::new(&rules);
        MyScanner { scanner }
    }

    pub fn scan(data: String) -> Result<String> {
        // Some implementation
    }
}}

Reducted code of lib.rs:
pub struct MyLib<'a> {
    scanner: MyScanner<'a>,
}

impl<'a> MyLib<'a> {
    pub fn new() -> Result<Self> {
        let rules: Rules = load_rules()?;
        let scanner = MyScanner::new(&rules);
        Ok(MyLib { scanner })
    }

    pub fn scan(&self, data: String) -> Result<String> {
        self.scanner.scan(data)
    }
}

load_rules is compiling and loading the rules from a resource file.
I tried countless variations of this code, but I always reach the obstacle of the lifetime specifier on Scanner and get an error of "rules does not live long enough".
I couldn't find a way to wrap YaraScanner in an object that outlives it and holds a Rules object safely.
I cannot create the rules in main.rs because I intend to export this as a library, and I don't want the user to load the Yara rules herself.
The only solution Claude Sonnet and I found where to Box::leak this memory or statically load it, so it will live until the program exits. I want to avoid it to support in the future getting string rules as arguments to MyLib::new, so I'm confined to the lifetime of a MyLib instance.
Please let me know how you think it can be solved, because currently, I think only changing Scanner to take ownership of the rules can solve this.

I believe you can achieve what you want with a bit of unsafe code:
/// Wraps a yara_x::Rules, but preventing it from moving around in memory.
struct PinnedRules{
    rules: yara_x::Rules,
    _pin: PhantomPinned,
}

struct MyScanner<'a> {
    scanner: yara_x::Scanner<'a>,
    // This allows MyScanner to own the yara_x::Rules and pass a reference to the
    // scanner. The use of `Pin` guarantees that the rules won't be moved.
    _rules: Pin<Box<PinnedRules>>,
}

impl<'a> MyScanner<'a> {
    pub fn new(rules: yara_x::Rules) -> Self {
        let pinned_rules = Box::pin(PinnedRules{rules, _pin: PhantomPinned});
        let rules_ptr = std::ptr::from_ref(&pinned_rules.rules);
        let rules_ref = unsafe { rules_ptr.as_ref().unwrap() };
        let scanner = yara_x::Scanner::new(rules_ref);

        Self { scanner, _rules: pinned_rules }
    }

    pub fn scan(&mut self, data: String) -> Result<String> {
        todo!()
    }
}

I haven't tested it thoroughly, so it may contain bugs.

Thank you, I tested this change in my code, all the tests passed, and nothing panics.
Even though it works, I think this solution is suboptimal - I need to test it more thoroughly, and I'll deep-dive into std::pin docs to make sure this unsafe code won't crash in the future, won't memory-leak, and there isn't any race in the destructor of MyScanner that may cause invalid memory access.
May I still suggest handling this issue in the yara-x library sometime in the future - to avoid forcing the library user to write unsafe code, or to introduce advanced Rust concepts.

@xrl1 then only thing you have to do is that you need to put Rules within your scanner struct so that the Rust compiler knows its lifetime doesn't expire before the struct is dropped.
pub struct MyScanner<'s> {
    rules: yara_x::Rules,
    scanner: Option<yara_x::Scanner<'s>>,
}

@qjerome that doesn't work because yara_x::Scanner needs a reference to the rules in MyScanner.rules, you can create a scanner that receives that reference, but you can't move it into MyScanner.scanner.

My bad, I thought it would !

Let me add a non null contribution this time:
impl<'s> Deref for MyScanner<'s> {
    type Target = yara_x::Scanner<'s>;
    fn deref(&self) -> &Self::Target {
        &self.scanner
    }
}

impl<'s> DerefMut for MyScanner<'s> {
    fn deref_mut(&mut self) -> &mut Self::Target {
        &mut self.scanner
    }
}

Should allow you to use your MyScanner as a yara_x::Scanner


Better isolation of sandboxed environments

This PR prevents users from overwriting tables of another mod by accident.
Deep copying is avoided inside LuaSandbox::Initialize on purpose, as that function just creates base templated environment for each Sandbox instance. Sandbox instance makes deep copy during construction.
Deep copy does not take into account duplicates, which should not really happen in our use case I believe. Did not found anything which would break cause of wrong copy.
We do not deep copy keys, so keys which are tables will not get properly copied (we do this only inside DeepCopySolObject function), but again, should not matter for our use case. Can be added relatively easily if required, have previous code saved which had this function in.
Before this, something like this could be done, affecting other mods in the process:
ImGui.Begin = function(name) spdlog.info("PWND!") end

Didnt test the snippet, may be wrong, but you get the picture...
Cleaned-up version of #522

Also adds Double inside meta visitor of Converter ( @psiberx )

Alongside deep copy this PR adds extra protection for usertypes (Vector4, Enum, etc.) and userdatas (main Game instance) that are shared between mod environments. This prevents adding new properties, overriding existing properties, and manipulating metatables.
An example of what was possible before PR:
getmetatable(Game).__index = function()
    print('pwnd')
    return function() end
end

Game.GetPlayer() -- pwnd
Game.GetStatsSystem() -- pwnd

So, does this PR breaks the only way for mods to talk to each other?

@bonanov no?
GetMod is still there and functions in exactly the same way.
This was done to prevent accidentaly screwing environment for other mods. If you need to communicste with some mod, you should have used GetMod, which was and still is there.

@WSSDude420 oh, i was not aware of GetMod. Thank you!


Sync reject road and half-pipes

This syncs reject road, specifically focusing on Maple Treeway's half-pipes.

Didn't realize this had a merge conflict


Addendum to 32e9ecf

rsync doesn't consider /usr/share/pkgconfig if not also /usr/share is specified.

Thanks.


Error: Receive is called after Client destroy, or simultaneously from different threads

I want to create a multi-threaded system that uses TDLib.
I wrote a Runnable class where I create a tdlib client and set an updates handler.
Each thread has its own TelegramClient and its own updates handler. Each class manages its own telegram account.
If I run only 1 thread, all works good, but if I run 2 or more threads, I got this error after the second TelegramClient was created:
[ 0][t 0][1612181768.710582017][Client.cpp:279] Receive is called after Client destroy, or simultaneously from different threads
I should use 1 global updates handler for all clients?

I am quite sure this issue is tied with this one: https://github.com/tdlib/td/issues/1242

You can't receive this error if you called td_json_client_receive for different TDLib instances.
You can find description of JSON interface in  https://github.com/tdlib/td/blob/312e3b740afbc9dbc8b8a539d4fb0d510b614779/td/telegram/td_json_client.h#L24 and description of new JSON interface in https://github.com/tdlib/td/blob/312e3b740afbc9dbc8b8a539d4fb0d510b614779/td/telegram/td_json_client.h#L105.
If you are using new interface, then all updates are received by the same thread and needs to be dispatched by it.

My hack of tdlib.go for DestroyInstance()

My hack of tdlib.go for DestroyInstance()
var IsClosed = false

func NewClient(config Config) *Client {
  // ...
	go func() {
		for !IsClosed {
      // ...
    }
  }()
  // ...
}

My hack of tdlib.go for DestroyInstance()
var IsClosed = false

func NewClient(config Config) *Client {
  // ...
	go func() {
		for !IsClosed {
      // ...
    }
  }()
  // ...
}

usage:
  tdlib.IsClosed = true
  time.Sleep(1 * time.Second)
  client.DestroyInstance()


Experimental: better type inference for ivars/cvars

This is just an experimental PR to prove that this is possible. The question is... would we want this?
This PR is inspired by https://github.com/crystal-lang/crystal/issues/11805
In that issue, the OP asks: why the compiler can't infer this?
class Thing
  DELAY = 10.milliseconds

  @timer_countdown = DELAY # can't infer the type of instance variable '@timer_countdown' of Thing
end

After all, this is relatively simple to solve:

Note that @timer_countdown is assigned a constant value
That constant value is assigned 10.milliseconds
10 is obviously an Int32
There's a method milliseconds defined on Int32 that returns Time::Span
So the type of @timer_countdown is guessed to be Time::Span!

So initially I thought:

okay, if the receiver of a method is a literal, we can do it, assuming the method has no arguments
actually, we can still do it if the argument types can be inferred. In a case like @x = 1 + 2, the compiler can easily see that this is going to be Int32#+(Int32) which is typed to be Int32

This is what this PR does.
But then I kept thinking...

Why not just try to infer the type of the call receiver and the arguments, and then lookup a call if we have all of those?

What this enables is all of this:
class Foo
  # Okay, this is what OP was proposing...
  @x = 10.milliseconds
end

class Foo
  # This is a nested call, (2 * 3) + 4, but everything can be easily inferred
  @x = 2 * 3 + 4
end

class Foo
  # The call receiver is inferred to be Array(Int32), and Array#reverse says it returns Array(T), so the compiler can easily infer this
  @x = [1, 2, 3].reverse
end

The above isn't very useful, but... what about this then?
class Foo
  def initialize(array : Array(Int32))
    # `array` is correctly guessed to be Array(Int32), and, like before, reverse has a type annotation, so...
    @reversed = array.reverse
  end
end

I think the above is actually a bit more useful. For a human it's pretty obvious that reverse is going to be returning an array of the same type, so why bother them with adding a type annotation?
This also works...
class Foo
  def initialize(string : String)
    # All of these can be easily inferred without having to look at method bodies,
   # because they all have a return type
    @value = string.upcase.reverse + " world!"
  end
end

p Foo.new("olleh")

The only problem with doing this is... it maybe becomes a bit harder to know when you don't need to put a type annotation and when you do. Right now it's a lot of rules:
The type of an instance variable, if not declared explicitly with
`@value : Type`, is inferred from assignments to it across
the whole program.

The assignments must look like this:

  1. `@value = 1` (or other literals), inferred to the literal's type
  2. `@value = Type.new`, type is inferred to be Type
  3. `@value = Type.method`, where `method` has a return type
     annotation, type is inferred from it
  4. `@value = arg`, with 'arg' being a method argument with a
     type restriction 'Type', type is inferred to be Type
  5. `@value = arg`, with 'arg' being a method argument with a
     default value, type is inferred using rules 1, 2 and 3 from it
  6. `@value = uninitialized Type`, type is inferred to be Type
  7. `@value = LibSome.func`, and `LibSome` is a `lib`, type
     is inferred from that fun.
  8. `LibSome.func(out @value)`, and `LibSome` is a `lib`, type
     is inferred from that fun argument.

Other assignments have no effect on its type.

So adding more rules will make the error really long and hard to understand!
But I'm thinking we could actually change this too. I don't think there's a need to list all these rules in the error message. The error message could be something like this:

Sorry, I can't easily infer the type of instance variable @value.
Could you add a type annotation?
Hint: the compiler might be able to infer the type of some expressions
without an explicit type annotation, but in some case you must put
a type annotation. Refer to the manual for an explanation of when
the compiler can infer these: [link to manual]

So the user would just go ahead and put a type annotation, or read the manual to understand things better. I'm also sure nobody right now reads the existing long message or remembers all those rules, it's probably always "I don't put a type annotation unless the compiler tells me to do so." So we would be reducing the amount of these occurrences, for more developer happiness, at minimal cost!
What do you think?

Thinking a bit more about this... when the compiler can't infer something, if you know it infers things based on return types, it might encourage users to add return types, which is always a good thing in my opinion.

Also note that the current diff to make all of the above work is minimal, so it's not like we are adding a lot of complexity to the compiler. All the "guess" rules are already there, we just never though of applying them to a call's receiver and arguments.

I love it. This seems to be a great improvement for DX.
My question would be: Why wouldn't we want this? I can't see any reason against this. Unless maybe it would put a lot of stress on the compiler, which I wouldn't expect it because those types need to be calculated later anyways even if the variables are annotated.

I also don't think the performance will get worse with this change. I'll try it later with the compiler itself.

Why wouldn't we want this? I can't see any reason against this.

One reason might be that it makes the code harder to understand at a glance. If you see @x : Int32 or @x = 1 it's clear what the type of @x is, but if it's a more complex expression then the type is harder to guess, specially if you don't know the methods being called. That said, this is already the case when you call class methods, so it's probably fine.

Actually, there's another reason why we might not want to do this for all types. If we have a type like IO and we deduce the type from its methods, we can see it has read_char : Char? so we could infer that. Now someone inherits from IO and defines read_char and either forgets to add a type annotation or returns something different. Now the code that used read_char to infer the type is broken.
I'm not sure we should do this... maybe only for primitive types which can't be inherited. But then it might be just simpler to leave things as they are right now.

Is that a real problem? I mean, if you redefine stuff things might break, independently of this particular feature. Does it make much of a difference if it's in an instance var or in an expression within a method?

@j8r No because instance variables don't show up in API docs. I don't know about IDEs. If it works now, it will work too with this change. This just adds more type inference rules.

Now the code that used read_char to infer the type is broken.

Broken how? Broken as in the resulting code doing the wrong thing, or broken as in does not compile anymore?
If the former, then this seems like a bad idea. If the latter, that would perhaps be fine? How bad it is would depend on how good the error message would be. Either way, if it turns out to be a big problem perhaps any overriding the return type of methods with explicit return type could be prohibited. That is, kinda similar to how you get an error if you try to monkey patch if methods are private or public or not.

@beta-ziliani Sure, there's no rush to merge this at all!
This PR is ready. The feature is experimental, but the PR is ready to merge :-)

Actually, I meant 1.4 🤦  OK then, I'll add it to the queue for review.

Oh, actually... re-reading the original comment, I think what's missing is defining what message we want to show users when we can't infer the type of an instance var.
I mentioned something like this:
Sorry, I can't easily infer the type of instance variable https://github.com/value.
Could you add a type annotation?

Hint: the compiler might be able to infer the type of some expressions
without an explicit type annotation, but in some case you must put
a type annotation. Refer to the manual for an explanation of when
the compiler can infer these: [link to manual]

Once we define that message, I can update the PR and it'll be ready to merge.

Start from there and we can polish it in the review (if needed be). Thanks! 🙇

Ooooh... 1.4 :-)
Also, we could maybe merge it as-is and then improve the error message. Right now the message tells you some ways you can assign to an instance variable without using an explicit type, but I think the rules are many more, so it's fine if we add more rules in this PR without explicitly mentioning them in that error message.

Okay, I'll change it :-)

If we can ask one more thing from you, can you add a companion PR to the relevant place in the documentation? It can certainly be after this is approved; the important point is now that you have the heuristic fresh in your head make sure the doc is and stays in sync.

I updated the PR.
For a code like this:
class Foo
  def initialize(@x)
  end
end

Foo.new(1)

you now get this error message:
Showing last frame. Use --error-trace for full trace.

In foo.cr:2:18

 2 | def initialize(@x)
                    ^-
Error: can't infer the type of instance variable '@x' of Foo

Could you add a type annotation like this

    class Foo
      @x : Type
    end

replacing `Type` with the expected type of `@x`?

I don't think we need to update the docs.
The reference for this is here: https://crystal-lang.org/reference/1.3/syntax_and_semantics/type_inference.html
There are 7 rules explained, and another section called "Other rules", which says things like "For example", so right now it clearly doesn't list all rules. And it's a bit impossible to list all rules, they are just too many.

I reported a bug that is caused by this change: https://github.com/crystal-lang/crystal/issues/11961

Thank you @asterite! It's working wonders! The following now compiles without an explicit @post : Post :heart_eyes:
class Post
  def self.find(id) : Post
    # ...
  end
end

class PostsController
  def show
    @post = Post.find(123)
  end
end

class PostsController (16 bytes)
    @post : (Post | Nil) (8 bytes)


Error in production-like environment: "Network.Socket.bind: permission denied"

Hi,
i'm trying to run multiple nodes in multiple VM's. This is the error i'm getting in the node logs:
[*] Starting Constellation node
2017 Aug-31 09:39:39715614 [INFO] Log level is LevelWarn
constellation-node: Network.Socket.bind: permission denied (Operation not permitted)
[*] geth --verbosity 6 --datadir qdata --bootnodes enode://081af0eec4688aaa14aaf847e9aae78c9fedcbedd16c6e3cff928625cab6a83103a514565dbfed04544b4bc7fb7e8a88234f6b426c387f1b5c46089107d268a9@[<IP_ADDRESS>]:10000 --networkid 87234 --rpc --rpcaddr <IP_ADDRESS> --rpcapi admin,db,eth,debug,miner,net,shh,txpool,personal,web3,quorum --rpcport 22800 --port 38000  #voterPattern# --minblocktime 2 --maxblocktime 5 --nat extip:<IP_ADDRESS>

Any idea what the problem is ?

Are you running from Windows host? If the files are shared from windows inside, linux, you would get permission issues.


crudely created a solution that isn't tested yet

replicating this section of code, crudely implemented a fix for App crashes when phone call is picked up#34

Will try testing it

@WovenCoast,Great work!, looks good to merge, once ready post a comment saying ready to merge

thanks, it's ready to merge


Disconnect frequently

Describe the bug
All sensors related to the Renault integration disconnect frequently. And display unavailable while being disconnected. Please see on the screenshots:

I do have the latest version v3.0.2 (no update available in HACS). I tried reloading the integration. What is suprising is that with the former way (writing in the yaml file), it still works fine:

Logs :
Logger: homeassistant.loader
Source: loader.py:465
First occurred: 4:40:53 PM (1 occurrences)
Last logged: 4:40:53 PM
You are using a custom integration for renault which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant.

Logger: custom_components.renault.renault_vehicle
Source: helpers/update_coordinator.py:166
Integration: Renault (documentation, issues)
First occurred: 4:42:40 PM (2 occurrences)
Last logged: 4:47:52 PM

    Error fetching VF1xxx hvac_status data: This endpoint has been disabled: ('err.func.403', 'Operation not supported Operation not supported for this can (C1A)')
    Error fetching VF1xxx battery data: Error communicating with API: ('err.tech.wired.kamereon-proxy', 'Failed to forward request to remote service.')

Logger: homeassistant.config_entries
Source: custom_components/renault/renault_vehicle.py:33
First occurred: 4:50:23 PM (1 occurrences)
Last logged: 4:50:23 PM
Error setting up entry Zoe for renault

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 236, in async_setup
    result = await component.async_setup_entry(hass, self)  # type: ignore
  File "/config/custom_components/renault/__init__.py", line 34, in async_setup_entry
    await renault_hub.set_account_id(config_entry.data[CONF_KAMEREON_ACCOUNT_ID])
  File "/config/custom_components/renault/renault_hub.py", line 47, in set_account_id
    vehicle = RenaultVehicleProxy(
  File "/config/custom_components/renault/renault_vehicle.py", line 33, in __init__
    "identifiers": {(DOMAIN, details.vin)},
AttributeError: 'NoneType' object has no attribute 'vin'

I modified the VIN number for privacy concerns.

@flouck which version of the component are you using? There was an issue in a previous version with too many calls overloading the Renault servers.
I also suggest that you turn on more advanced logging using these settings:
logger:
  default: error
  logs:
    renault_api: debug
    custom_components.renault: debug

Also, I have just found a comment @Jastroger on https://github.com/hacf-fr/hassRenaultZE/issues/84#issuecomment-743779964, at about the same time as your issues, that indicates there were issues on the Renault servers around that time.
Can you please confirm if it is still an issue for you?

I confirm same problem 4 days ago (Failed to forward request to remote service ). « My Renault » app was KO too. So Renault side!

Indeed it works now. Thank you!


Master

the status rejected is missing in Spaceship::Tunes::AppStatus.get_from_string

@KrauseFx oops

I have fixed the conflicts but now I have weird messages

@bolom i think its easier to freshly start a PR from current master and avoiding too much diff (like whitespace changes and so on)


GetType() will fail if trying to get a type that is not entirely defined in one assembly

Assembly.GetType() works fine for types that are defined in that assembly.
But consider a type such as List<Node>. This is a generic type made from a generic type definition (List<>) and a generic type parameter (Node), both of which are from different assembly.
Which means that calling GetType("System.Collections.Generic.List1[Godot.Node]")onList<>'s assembly will return null, and calling it on Node's assembly will also return null`, since the resulting type is not fully from either assembly, but rather composed of types from both of the assemblies.
The easiest solution to this problem would be to avoid relying on GetType(), and instead retrieve an IEnumerable<Type> and find the first match by comparing type names.
This solution may be significantly slower since it would likely have to search through hundreds of types. Caching the results might help with this issue, but it may not be needed as (de)serialization is only expected to take place during loading and saving.

Upon further thinking, it seems that not even comparing type names will work. A type such as List<Node> is still composed of two different types from two different assemblies, and as such, no assembly will contain the List<Node> type.
There is, unfortunately, no "proper" solution to this. Trying to account for all generic type combinations with all types in all assemblies is simply madness.
The only way to avoid this problem is to pass in the type argument when (de)serializing.


Fix Issue #22

Removed data_missing column

Needs to be removed from the Table Definition SQL as well.

Fixed also in SQL statement

I've updated the pull Request to include the split PNG/TIFF


Improve Service Section: Buttons Not Active (Book Now, Get a Quote, Contact Us)

I'd like to work on improving the Service Section, specifically where the buttons are currently not active. The following buttons require updates:
###need to changes
Book Now
Get a Quote
Contact Us
###screenshots

Label Request:
Additionally, I would like to request the following labels for the issue:
level label (appropriate for the difficulty)
Hacktoberfest-accepted label
gssoc-ext

I have completed the work related to service section on the improve-cab-service branch.

All checks have passed successfully.

There are no conflicts with the base branch.

The workflow is awaiting approval.

Could you please review and approve the workflow, and merge the PR

https://github.com/07sumit1002/CabRental/pull/202

screenshots


Sd.cpp.gui.wx uses only 57% of CPU threads

First of all, i want to thank you for adding ability to choose amount of cpu cores used for image generation. Unfortunately, this option doesn't work as it supposed to. While compiled cli version of sd.cpp uses 99% of cpu threads, your gui uses only 57% (I used AMD Adrenaline overlay to monitor cpu usage). Could it be fixed somehow, please?

Wich CPU have you?

Wich CPU have you?
Bytheway, it's a wrong idea to use all threads on your CPU. I read it somewhere earlier, only good idea to use only the cores and not the threads.

My laptop's cpu is Ryzen 7 4700u, 8 cores, 8 threads. When i run cli version of sd.cpp, it always uses all 8 threads (99% cpu usage in AMD Adrenaline overlay) and generates images faster than sd.cpp.gui.wx.

Can you post your config file's content please?
You can find here: C:\Users<username>\AppData\Roaming\sd.ui.config.ini

Can you post your config file's content please?

keep_model_in_memory=1
save_all_image=1
n_threads=8
[paths]
lora=C:\Users\John\AppData\Roaming\sd.ui\sd_ui_data\lora
model=C:\Users\John\Downloads
vae=C:\Users\John\AppData\Roaming\sd.ui\sd_ui_data\vae
embedding=C:\Users\John\AppData\Roaming\sd.ui\sd_ui_data\embedding
presets=C:\Users\John\AppData\Roaming\sd.ui\sd_ui_data\presets
taesd=C:\Users\John\AppData\Roaming\sd.ui\sd_ui_data\taesd

Can you please rewrite the "Number of the CPU cores" option in the "Settings -> Diffusion tab"? Eg. to 2, and start a job?
Please check out what's happening then. You can stop the diffusion if you close the main window, thats will abort the process. For testing please only diffuse with small resolution with minimal sampling steps.
And please try-out what's happening, when you set the same setting to 16.
Whats parameters are you use with sd.exe?

Whats parameters are you use with sd.exe?

sd.exe -m LCM_Dreamshaper_v7-f16.gguf -t 8 --steps 4 --height 512 --width 512 --seed -1 -p "a room" -o room4.png --cfg-scale 1.0 --sampling-method lcm

Can you please rewrite the "Number of the CPU cores" option in the "Settings -> Diffusion tab"? Eg. to 2, and start a job?

And please try-out what's happening, when you set the same setting to 16.

In both cases CPU usage is 57%. It takes 151 s to make a 512x512 image. When CPU usage is 99% (cli sd.exe), it takes only 91 s.


refactor!: update revocation

This PR updates the revocation package before v1.1.0 stable release. It makes the package more extensible.

LGTM


Quality

Hey there, sorry to bother you with this. I was just wondering, is scrcpy suited to watching video playback from a mobile device or is it not really for that?
I tried to play a stream through a Live Sports app on my phone, so I can view it on my laptop screen using scrcpy.
The stream is set to HD, though my device resolution is 720x1280. When watching back, it's not as smooth as it should be, but worse than that, the quality is a bit poor. It doesn't really look 720p to me.
I've got a screenshot to help show this.
https://i.postimg.cc/g07DLr9f/720.png
I have tried increasing the bitrate using scrcpy -b20M and tried changing the renderer to openGL. Could the issue just be on my end, or is my use case not what scrcpy is for?
Thanks!
Environment

OS: [Windows 10]
scrcpy version: [v1.8]
installation method: [Windows release]
device model: G8441
Android version: [Android 8]

is scrcpy suited to watching video playback from a mobile device or is it not really for that?

Yes, it can. In practice, it's not often useful, since it does not forward sound though :)

I tried to play a stream through a Live Sports app on my phone, so I can view it on my laptop screen using scrcpy.
The stream is set to HD, though my device resolution is 720x1280. When watching back, it's not as smooth as it should be, but worse than that, the quality is a bit poor. It doesn't really look 720p to me.
I've got a screenshot to help show this.

What is the quality if you just record (without streaming)?
scrcpy -b20M --record file.mp4

If the resulting quality is good, then the streaming quality is the culprit (try to adjust the settings of your streaming software).

is scrcpy suited to watching video playback from a mobile device or is it not really for that?

Yes, it can. In practice, it's not often useful, since it does not forward sound though :)

I tried to play a stream through a Live Sports app on my phone, so I can view it on my laptop screen using scrcpy.
The stream is set to HD, though my device resolution is 720x1280. When watching back, it's not as smooth as it should be, but worse than that, the quality is a bit poor. It doesn't really look 720p to me.
I've got a screenshot to help show this.

What is the quality if you just record (without streaming)?
scrcpy -b20M --record file.mp4

If the resulting quality is good, then the streaming quality is the culprit (try to adjust the settings of your streaming software).

Thanks for this response, very helpful. I did some further tests, I played a Youtube video and that seemed absolutely fine, so I went back to the app. I tried the other player they have which I don't usually use, let it buffer, and it actually seemed to be fine too. I wasn't streaming anything live this time, so I'll have to test it again when sports are on, but it might be that the key is the player on the phone.
One thing I did notice however, is that the recorded playback file is slightly smoother than how it appears when I'm using it in real time. Sorry, I'm not good at explaining it, but in the recorded file there's a smoothness to the action, whereas in the playback, it's slightly jittery. It's very slight, but I do notice it.

One thing I did notice however, is that the recorded playback file is smoother than how it appears when I'm using it in real time.

You mean when opening scrcpy only (without streaming)? Is it over USB?

Think of it like an FPS drop, it kind of looks like that. It's very slight, but I do notice it.

Is there any FPS drop? Enable FPS logging by pressing Alt+i in the scrcpy window, it will print framerate every second in the terminal. Are there lines with skipped frames?

You mean when opening scrcpy only (without streaming)? Is it over USB?
Sorry, I'm a bit unsure what you mean by without streaming? You mean streaming the video?
And yes I do use USB-C

Is there any FPS drop? Enable FPS logging by pressing Alt+i in the scrcpy window, it will print framerate every second in the terminal. Are there lines with skipped frames?
There are a few, yeah. It seems pretty consistent though.
https://i.postimg.cc/9FgHbBYq/frames.png

I tried to take a video to show this but I don't think the screen recording on my laptop is good enough to really show what I mean.

Sorry, I'm a bit unsure what you mean by without streaming?

Sorry, I confused things with other posts (I thought you were streaming the device capture with OBS or something).

One thing I did notice however, is that the recorded playback file is smoother than how it appears when I'm using it in real time. Sorry, I'm not good at explaining it, but in the recorded file there's a smoothness to the action, whereas in the playback, it's slightly jittery. Think of it like an FPS drop, it kind of looks like that. It's very slight, but I do notice it. The recorded file is OK, but when it's being streamed in real time, it isn't

If it's not only when a frame is skipped as printed in the logs (I think it is not), then here is the explanation.
On the device side, scrcpy captures the frames with their timestamp (the date at which they are captured).
On the client side, to minimize latency, scrcpy always displays immediately the last frame it received (and decoded), regardless of the timestamps (it works because the capture is real time, if it was from a pre-recorded file, it could not work, the file would be decoded and played as fast as possible).
In practice, we can assume that the delay between the capture on the device and the display on screen is sufficiently stable for the jitter to be barely noticeable (especially over USB). But there is still a small jitter: the encoding/decoding time may vary depending on frame complexity, the transfer varies depending on encoded frame size (in bytes), etc.
But to compensate the jitter (and respect the delay between frames), it would require to delay frames so that they all match the largest (estimated) possible delay, which would make the latency unacceptable.
However, on record, the scrcpy client takes care of writing the timestamps captured on the device (transmitted by the server), so when played with your video player (like VLC), the delay between frames is correct.

Thanks for the detailed response. I think I see what you are saying.
I don't really know if it is an FPS drop, and it is slight but because I'm watching a fast paced sport, it's more noticeable than it otherwise would be I think. It's just slightly jittery. When I playback the recording, I do so in VLC, but obviously when scrcpy is active, it's viewed on the window client.
I actually wouldn't mind increased latency if it meant smoother playback, but I do understand for most this is far less desirable.

Could you test running scrcpy for the same devixe on Linux?
Unfortunately I can't as I don't have a Linux device.

Unfortunately I can't as I don't have a Linux device.

Any computer is a Linux device :trollface:
You could run a LiveCD/LiveUSB (Ubuntu for example) on your computer.

Unfortunately I can't as I don't have a Linux device.

Any computer is a Linux device :trollface:
You could run a LiveCD/LiveUSB (Ubuntu for example) on your computer.

Perhaps one day, unfortunately it's not something I want to mess with right now

OK, so it's currently not an InterestingInterest for you :troll:

in the recorded file there's a smoothness to the action, whereas in the playback, it's slightly jittery

Please check #2464.

in the recorded file there's a smoothness to the action, whereas in the playback, it's slightly jittery

Please check #2464.

Wow, thanks. Will this be available in scrcpy v1.19?

Yes. But you can try it now by building the dev branch.

I added a binary at the end of the post on #2464.
I'd appreciate feedbacks if you could test :)

I added a binary at the end of the post on #2464.
I'd appreciate feedbacks if you could test :)

Hey there, thanks for this! I did a quick test just now, it's not conclusive but I tried
scrcpy --display-buffer=50
And I'm not completely sure it fixed the issue. I'd have to do the recording thing again to test it thoroughly, but it still seemed a bit jittery. It could be because I'm comparing it to my phone which is a smaller screen so it looks smoother there
scrcpy --v4l2-buffer=500.
I couldn't test this unfortunately since I don't have a Linux device

it still seemed a bit jittery

OK, so that was probably not the cause in your case (which is also suggested by the "skipped frames" you get).
You could try with a bigger buffering delay, but I think it will not fix the problem for you:
scrcpy --display-buffer=500


Add click events for TimeTableScreen

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#463  👈 (View in Graphite)
#462
#461
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Ensure that angular is loaded for commonjs

This stops some edge case loading scenarios where window.angular is not available.
RE: #35

@whitneyit what effect does this have on users that are loading this through <script> tags in the browser?

Answered my own question :-)  Thanks @whitneyit !

:)


Set AggressiveOptimization on all SepParser*.Parse

Update 5950X benchmarks incl. .NET 8 RC2
Regressions are probably within expected. Actual assembly code appears similar.
Switch due to issues with dynamic PGO in .NET 8 can cause significant regressions ~35% slower under some circumstances.

Codecov Report
All modified lines are covered by tests :white_check_mark:

Files
Coverage Δ

...p/Internals/SepParserAvx2PackCmpOrMoveMaskTzcnt.cs
100.00% <ø> (ø)

...Internals/SepParserAvx512PackCmpOrMoveMaskTzcnt.cs
0.00% <ø> (-100.00%)
:arrow_down:

src/Sep/Internals/SepParserIndexOfAny.cs
100.00% <ø> (ø)

...p/Internals/SepParserSse2PackCmpOrMoveMaskTzcnt.cs
100.00% <ø> (ø)

...p/Internals/SepParserVector128NrwCmpExtMsbTzcnt.cs
100.00% <ø> (ø)

...p/Internals/SepParserVector256NrwCmpExtMsbTzcnt.cs
100.00% <ø> (ø)

...p/Internals/SepParserVector512NrwCmpExtMsbTzcnt.cs
100.00% <ø> (ø)

...ep/Internals/SepParserVector64NrwCmpExtMsbTzcnt.cs
100.00% <ø> (ø)

... and 3 files with indirect coverage changes
:loudspeaker: Thoughts on this report? Let us know!.


Windows scripts not updated after updating

C:\Users\<user>\AppData\Local\atom\bin doesn't seem to be updated after an update has been installed. Though atom added this directory to the PATH variable so atom is available from the windows console.
So then there is a update and this is showing up in the console:
> atom .
The system cannot find the path specified.

So it happens that atom updated to version 1.0.6, but when I look at the files in C:\Users\<user>\AppData\Local\atom\bin I can see that these binary executable files all refer to app-1.0.4. By having this issue multiple times, I know I can fix this by rewriting this to app-1.0.6, but it would be very convenient if the updater all does this on instead of wasting time on figuring out why its not working.

Hmm I've done some research, though I don't know how to test if this function gets executed while the update is being executed.
https://github.com/atom/atom/blob/efb2ea544f64371fcbd8d149e06b223c3ea4ab56/src/browser/squirrel-update.coffee#L135

This happens to me occasionally as well.  Update to 1.0.7 worked perfectly, but I don't believe anything got updated during 1.0.8.  When I checked during 1.0.9 (due to the 'system cannot find the path specified' errors), this is what I found:

Both apm and apm.cmd were pointing towards 1.0.7
atom was empty (:exclamation:)
atom.cmd was pointing towards 1.0.9

Hmm.. it would be nice to collect more information about that folder from other users (if they want)... it could be used as a test case. Or are there better ways to expose this problem?

Here's some errors in my SquirrelSetup.log, not sure if they're entirely relevant though:
2015-08-29 11:05:40> UpdateInfo: Couldn't get release notes for:atom-1.0.9-delta.nupkg: System.Exception: Invalid 'ReleaseNotes' value in nuspec file at 'C:\Users\username\AppData\Local\atom\packages\atom-1.0.9-delta.nupkg'
   at Squirrel.ReleaseEntry.GetReleaseNotes(String packageDirectory)
   at Squirrel.UpdateInfo.<FetchReleaseNotes>b__2(ReleaseEntry x)
2015-08-29 11:05:41> Program: Starting Squirrel Updater: --update https://atom.io/api/updates
2015-08-29 11:05:41> Program: Starting update, downloading from https://atom.io/api/updates
2015-08-29 11:05:41> Program: About to update to: C:\Users\username\AppData\Local\atom
2015-08-29 11:05:41> CheckForUpdateImpl: Downloading RELEASES file from https://atom.io/api/updates
2015-08-29 11:05:41> FileDownloader: Downloading url: https://atom.io/api/updates/RELEASES
2015-08-29 11:05:42> FileDownloader: Downloading file: https://atom.io/api/updates/atom-1.0.9-delta.nupkg
2015-08-29 11:06:11> Program: Failed to apply updates, falling back to full updates: System.IO.PathTooLongException: The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters.
   at System.IO.PathHelper.GetFullPathName()
   at System.IO.Path.NormalizePath(String path, Boolean fullCheck, Int32 maxPathLength, Boolean expandShortPaths)
   at System.IO.Path.GetFullPathInternal(String path)
   at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractEntry(ZipEntry entry)
   at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(Stream inputStream, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime, Boolean isStreamOwner)
   at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(String zipFileName, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime)
   at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(String zipFileName, String targetDirectory, String fileFilter)
   at Squirrel.DeltaPackageBuilder.ApplyDeltaPackage(ReleasePackage basePackage, ReleasePackage deltaPackage, String outputFile)
   at Squirrel.UpdateManager.ApplyReleasesImpl.<>c__DisplayClassaf.<createFullPackagesFromDeltas>b__ab()
   at System.Threading.Tasks.Task`1.InnerInvoke()
   at System.Threading.Tasks.Task.Execute()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.UpdateManager.ApplyReleasesImpl.<createFullPackagesFromDeltas>d__b1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.UpdateManager.ApplyReleasesImpl.<ApplyReleases>d__70.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.UpdateManager.<ApplyReleases>d__a.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
   at Squirrel.Update.Program.<Update>d__41.MoveNext()
2015-08-29 11:06:11> CheckForUpdateImpl: Downloading RELEASES file from https://atom.io/api/updates
2015-08-29 11:06:11> FileDownloader: Downloading url: https://atom.io/api/updates/RELEASES
2015-08-29 11:06:13> FileDownloader: Downloading file: https://atom.io/api/updates/atom-1.0.9-full.nupkg
2015-08-29 11:06:30> ApplyReleasesImpl: Writing files to app directory: C:\Users\username\AppData\Local\atom\app-1.0.9
2015-08-29 11:07:20> ApplyReleasesImpl: Squirrel Enabled Apps: [C:\Users\username\AppData\Local\atom\app-1.0.9\atom.exe]
2015-08-29 11:07:38> ApplyReleasesImpl: Couldn't run Squirrel hook, continuing: C:\Users\username\AppData\Local\atom\app-1.0.9\atom.exe: System.OperationCanceledException: The operation was canceled.
   at System.Threading.CancellationToken.ThrowOperationCanceledException()
   at Squirrel.Utility.<>c__DisplayClassf.<InvokeProcessAsync>b__e()
   at System.Threading.Tasks.Task.InnerInvoke()
   at System.Threading.Tasks.Task.Execute()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.Utility.<InvokeProcessAsync>d__11.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.UpdateManager.ApplyReleasesImpl.<>c__DisplayClassc8.<<invokePostInstall>b__bc>d__ca.MoveNext()

Uh, don't say that windows is complaining about long paths...

I found mine SquirrelSetup.log btw. There is lots of data so I might spend time studying it.

If there is a suggestion to make though.. I really don't like that every file compressed or decompressed (from I assume azar) is being logged. Clutters the logs very easily.

/cc @paulcbetts

Tracking this under https://github.com/atom/atom/issues/11450 as there are a few that boil down to 'bin' not being installed or updated.


Email workflows

verify address email after registration
verify address email after email change
resend verification email
confirm password change email

🧪 Found ~1 bug~ ✅


Some problems with the latest update

To fix:

[ ] The repeat button don't have anymore a good active effect (now is the same as the hover effect)
[ ] The media buttons don't have a good contrast with the rest of colors
[ ] Invertion filter instead of own colors for each theme

The repeat button don't have anymore a good active effect (now is the same as the hover effect)
Do you want to give it a try to fix this bug?
https://github.com/nt1m/HTML5AudioPlayer/blob/gh-pages/css/controls.css#L43

The media buttons don't have a good contrast with the rest of colors
Ah, yes, didn't test in Chrome.

Invertion filter instead of own colors for each theme
I already use an inversion filter.

Do you want to give it a try to fix this bug?
https://github.com/nt1m/HTML5AudioPlayer/blob/gh-pages/css/controls.css#L43
I don't know how fix it, because I don't understand so much about .svg graphics.

Ah, yes, didn't test in Chrome.
I'll test it in Firefox and try to make it work like in Chrome.

I already use an inversion filter.
As I expect you can see, I'm listing problems in this issue, and I'm not suggesting. I already know you used a invertion filter, and is about it I'm talking.

I need to talk you in Hangouts.

Yes, it works very different in Firefox.
Chrome:

Firefox:


[TPU, keras preprocessing layer] Some Op must be a compile-time constant.

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): y
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): google colab
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): google colab
TensorFlow version (use command below): 2.7
Python version: google colab
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: TPU issue
GPU model and memory: TPU issue

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
Hi!
TPU error raises especially with Kears preprocessing layers.
I've tried to connect two models, augmentation model that contains preprocessing layer and segmentation model.
def new_concatenated_model(
    image_input_hw,
    mask_input_hw,
    class_n
):
    seg_model = create_segmentation_model(class_n)
    aug_model = create_augmentation_model(
        image_input_hw, mask_input_hw, class_n)

    image_input_shape = list(image_input_hw) + [3]

    @auto_tpu(device=CURRENT_DEVICE) # decorator `auto_tpu` is just context manager.
    def create():
        im = seg_model.input
        model = AugConcatedSegModel(
            inputs=im,
            outputs=seg_model(im),
            augmentation_model=aug_model,
            name='seg_model_train_with_aug'
        )
        return model

    model = create()
    return model

train_step() function code was mainly came from tensorflow official tutorial document.
class AugConcatedSegModel(tf.keras.Model):
    def __init__(
        self,
        inputs=None,
        outputs=None,
        augmentation_model=None,
        **kwargs
    ):
        super().__init__(inputs=inputs, outputs=outputs, **kwargs)
        self.augmentation_model = augmentation_model

    def train_step(self, data):
        im, ma = data
        im, ma = self.augmentation_model((im, ma))

        with tf.GradientTape() as tape:
            ma_pred = self(im, training=True)  # Forward pass
            # Compute the loss value
            # (the loss function is configured in `compile()`)
            loss = self.compiled_loss(ma, ma_pred, regularization_losses=self.losses)

        # Compute gradients
        trainable_vars = self.trainable_variables
        gradients = tape.gradient(loss, trainable_vars)
        # Update weights
        self.optimizer.apply_gradients(zip(gradients, trainable_vars))
        # Update metrics (includes the metric that tracks the loss)
        self.compiled_metrics.update_state(ma, ma_pred)
        # Return a dict mapping metric names to current value
        return {m.name: m.result() for m in self.metrics}

Describe the expected behavior
Expected to train successfully without error.
same code were tested on:

[x] CPU : No errors
[x] GPU : No errors
[x] TPU : Error

You could reproduce this error very fast
https://colab.research.google.com/drive/1LhHj1FrkZE9QnFhY-NOO8mn7aiXhZgNh?usp=sharing
Runtime - Run all.

When I changed augmentation model to just plain Conv2D layers, the error disappeared.

Contributing

Do you want to contribute a PR? (yes/no):
Briefly describe your candidate solution(if contributing):

Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
https://colab.research.google.com/drive/1LhHj1FrkZE9QnFhY-NOO8mn7aiXhZgNh?usp=sharing
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.
InvalidArgumentError: 9 root error(s) found.
  (0) INVALID_ARGUMENT: {{function_node __inference_train_function_692915}} Input 0 to node `sequential_augmentation_model/sequential_augmentation_layers/random_flip/stateless_random_flip_left_right/stateless_random_uniform/StatelessRandomUniformV2` with op StatelessRandomUniformV2 must be a compile-time constant.

XLA compilation requires that operator arguments that represent shapes or dimensions be evaluated to concrete values at compile time. This error means that a shape or dimension argument could not be evaluated at compile time, usually because the value of the argument depends on a parameter to the computation, on a variable, or on a stateful operation such as a random number generator.

	 [[{{node sequential_augmentation_model/sequential_augmentation_layers/random_flip/stateless_random_flip_left_right/stateless_random_uniform/StatelessRandomUniformV2}}]]
	 [[TPUReplicate/_compile/_1646634736830564460/_4]]
  (1) INVALID_ARGUMENT: {{function_node __inference_train_function_692915}} Input 0 to node `sequential_augmentation_model/sequential_augmentation_layers/random_flip/stateless_random_flip_left_right/stateless_random_uniform/StatelessRandomUniformV2` with op StatelessRandomUniformV2 must be a compile-time constant.

XLA compilation requires that operator arguments that represent shapes or dimensions be evaluated to concrete values at compile time. This error means that a shape or dimension argument could not be evaluated at compile time, usually because the value of the argument depends on a parameter to the computation, on a variable, or on a stateful operation such as a random number generator.

	 [[{{node sequential_augmentation_model/sequential_augmentation_layers/random_flip/stateless_random_flip_left_right/stateless_random_uniform/StatelessRandomUniformV2}}]]
	 [[TPUReplicate/_compile/_1646634736830564460/_4]]
	 [[tpu_compile_succeeded_assert/_5094882425795608634/_5/_47]]
  (2) INVALID_ARGUMENT: {{function_node __inference_train_function_692915}} Input 0 to node `sequential_augmentation_model/sequential_augmentation_layers/random_flip/stateless_random_flip_left_right/stateless_random_uniform/StatelessRandomUniformV2` with op StatelessRandomUniformV2 must be a compile-time constant.

XLA compilation requires that operator arguments that represent shapes or dimensions be evaluated to concrete values at compile time. This error means that a shape or dimension argument could not be evaluated at compile time, usually because the value of the argument depends on a parameter to the computation, on a variable, or on a stateful operation such as a random number generator.

	 [[{{node sequential_augmentation_model/sequential_augmentation_layers/random_flip/stateless_random_flip_left_right/stateless_random_uniform/StatelessRandomUniformV2}}]]
	 [[TPUReplicate/_compile/_1646634736830564460/_4]]
	 [[tpu_compile_succeeded_assert/_5094882425795608634/_5/_159]]
  (3) INVALID_ARGUMENT: {{function_node __inference_train_function_692915}} Input 0 to node `sequential_augmentation_model/sequential_a ... [truncated]

@ProtossDragoon
In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thanks!

Hi @sushreebarsa, there is COLAB notebook instead of code snippet. You could reproduce this issue without any other codes. Just change COLAB runtime to TPU device, and run all cells. Thanks.
Core code below.
def auto_tpu(device='cpu'):
    """Automatically open context manager
    If your colab environment is on 'tpu'
    """
    def decorator(fn):
        def wrapper(*args, **kwargs):
            s = time.time()
            if device == 'tpu':
                with TRAINING_PARALLEL_STRATEGY.scope():
                    ret = fn(*args, **kwargs)
            else:
                ret = fn(*args, **kwargs)
            e = time.time()
            print(f'device: {repr(device)}, time elapse: {e-s:.3} second(s)')
            return ret
        return wrapper
    return decorator

@auto_tpu(device=CURRENT_DEVICE)
def create_augmentation_model(
    image_input_hw,
    mask_input_hw,
    class_n:int
):
    _default_channel_n = 3

    # runtime augmentation pipeline
    seq = tf.keras.Sequential(
        [
            tf.keras.layers.RandomFlip("horizontal"),
            tf.keras.layers.RandomRotation(0.02),
        ],
        name='sequential_augmentation_layers'
    )

image_input_shape = list(image_input_hw) + [_default_channel_n]
mask_input_shape = list(image_input_hw) + [class_n]
x_im = tf.keras.Input(shape=image_input_shape)
x_ma = tf.keras.Input(shape=mask_input_shape)
return tf.keras.Model(
    inputs=[x_im, x_ma],
    outputs=[seq(x_im), seq(x_ma)],
    name='sequential_augmentation_model'
    )

aug_model = create_augmentation_model(
    image_input_hw,
    mask_input_hw,
    class_n
)

class AugConcatedSegModel(tf.keras.Model):
    def __init__(
        self,
        inputs=None,
        outputs=None,
        augmentation_model=None,
        **kwargs
    ):
        super().__init__(inputs=inputs, outputs=outputs, **kwargs)
        self.augmentation_model = augmentation_model

def train_step(self, data):
    im, ma = data
    im, ma = self.augmentation_model((im, ma))

    with tf.GradientTape() as tape:
        ma_pred = self(im, training=True)  # Forward pass
        # Compute the loss value
        # (the loss function is configured in `compile()`)
        loss = self.compiled_loss(ma, ma_pred, regularization_losses=self.losses)

    # Compute gradients
    trainable_vars = self.trainable_variables
    gradients = tape.gradient(loss, trainable_vars)
    # Update weights
    self.optimizer.apply_gradients(zip(gradients, trainable_vars))
    # Update metrics (includes the metric that tracks the loss)
    self.compiled_metrics.update_state(ma, ma_pred)
    # Return a dict mapping metric names to current value
    return {m.name: m.result() for m in self.metrics}

def new_concatenated_model(
    image_input_hw,
    mask_input_hw,
    class_n
):
    seg_model = create_segmentation_model(class_n)
    aug_model = create_augmentation_model(
        image_input_hw, mask_input_hw, class_n)

    _default_channel_n = 3
    image_input_shape = list(image_input_hw) + [_default_channel_n]

    @auto_tpu(device=CURRENT_DEVICE)
    def create():
        im = seg_model.input
        model = AugConcatedSegModel(
            inputs=im,
            outputs=seg_model(im),
            augmentation_model=aug_model,
            name='seg_model_train_with_aug'
        )
        return model

    model = create()
    return model

new_seg_model = new_concatenated_model(
    image_input_hw,
    mask_input_hw,
    class_n
)

@auto_tpu(device=CURRENT_DEVICE)
def run(model):
    model.compile('adam', get_loss(class_n), get_metrics())
    model.fit(tf_dataset)

run(new_seg_model)

@ProtossDragoon We could see that the TPU error raises especially with Kears preprocessing layers.
Please post this issue on keras-team/keras repo.
To know more see;
https://discuss.tensorflow.org/t/keras-project-moved-to-new-repository-in-https-github-com-keras-team-keras/1999
Thank you!

@ProtossDragoon We could see that you have mentioned this issue on   keras-team/keras repo. .Could you please move this ticket to closed status as we will track the issue there ? Thanks!

The problem should be fixed now.


.NET 5.0 not .NETFramework 5.0

Describe the bug
When viewing dependencies on the web site for a package that targets .NET 5.0 the UI reads '.NETFramework 5.0' which I believe is incorrect.
To Reproduce
Publish a package that targets .NET 5.0
Expected behavior
Should display ".NET 5.0" instead of ".NETFramework 5.0"
Screenshots

Great catch, @f2calv! Essentially the problem here is that NuGet.org is using an older version of the framework definitions prior to the introduction of .NET 5.0.
We'll need to update the newest version of these definitions and reprocess packages that have .NET 5.0 dependencies.
/cc @zkat @nkolev92, is there a version of NuGet.Frameworks that we should be using for stable .NET 5.0 framework parsing?
Note to the server team: I think feed2catalog and perhaps even catalog2registration need to upgrade as well because they interact with dependency group metadata as well.
/cc @loic-sharma, who recently updated the package versions used on the server side. We may already have the version we need and just need to deploy it then reflow the packages.
My guess is other package feeds will run into similar problems. It may be worth an announcement about the introduction of the TFM and how it interacts with NuGet client and server implementations.

@joelverhagen
5.8 early previews have all the changes necessary.
Anything above 5.8.0 P2 should work, but prefer the latest.

Note to server team/whoever makes the fix:
NuGetFramwork.DotNetFrameworkName is designed explicitly to be compatible with System.Runtime.Versioning.FrameworkName. As per the .NET 5 TFM spec, this is not the recommended string for display.  Please use NuGetFramework.ToString() to get the recommended display name.

Here are several variants and how they behave on nuget.org today.
net5.0
Example package with net5.0 in the dependency group:
https://www.nuget.org/packages/Microsoft.AspNetCore.Authorization/5.0.0-rc.1.20451.17
      <group targetFramework="net5.0">
        <dependency id="Microsoft.AspNetCore.Metadata" version="5.0.0-rc.1.20451.17" exclude="Build,Analyzers" />
        <dependency id="Microsoft.Extensions.Logging.Abstractions" version="5.0.0-rc.1.20451.14" exclude="Build,Analyzers" />
        <dependency id="Microsoft.Extensions.Options" version="5.0.0-rc.1.20451.14" exclude="Build,Analyzers" />
      </group>

Currently shows as:

.NETCoreApp5.0
Example package with .NETCoreApp5.0 in the dependency group:
https://www.nuget.org/packages/Microsoft.Extensions.Options/5.0.0-rc.1.20451.14
      <group targetFramework=".NETCoreApp5.0">
        <dependency id="Microsoft.Extensions.DependencyInjection" version="5.0.0-rc.1.20451.14" />
        <dependency id="Microsoft.Extensions.DependencyInjection.Abstractions" version="5.0.0-rc.1.20451.14" />
        <dependency id="Microsoft.Extensions.Logging.Abstractions" version="5.0.0-rc.1.20451.14" />
        <dependency id="Microsoft.Extensions.Options" version="5.0.0-rc.1.20451.14" />
      </group>

Currently shows as:

net5.0-windows7.0
Example package with net5.0-windows7.0 in the dependency group:
https://www.nuget.org/packages/ReaLTaiizor/3.7.8
      <group targetFramework="net5.0-windows7.0">
        <dependency id="System.Configuration.ConfigurationManager" version="5.0.0-rc.1.20451.14" exclude="Build,Analyzers" />
        <dependency id="System.Drawing.Common" version="5.0.0-rc.1.20451.14" exclude="Build,Analyzers" />
        <dependency id="System.Resources.Extensions" version="5.0.0-rc.1.20451.14" exclude="Build,Analyzers" />
      </group>

Currently shows as:

I will loop with PMs to find the desired behavior (especially in light of platform)

Note, those same packages as displayed by the VS PM UI:
https://www.nuget.org/packages/Microsoft.AspNetCore.Authorization/5.0.0-rc.1.20451.17

https://www.nuget.org/packages/Microsoft.Extensions.Options/5.0.0-rc.1.20451.14

https://www.nuget.org/packages/ReaLTaiizor/3.7.8

See also: TFMs in the UI from the .NET 5 TFM spec. It's aimed at VS, but if gallery wants a similar experience to VS, it might be relevant.

Related: https://github.com/NuGet/NuGetGallery/issues/8268

After the latest deployment of NuGetGallery where we updated our version of NuGet.Frameworks, all .NET 5.0 frameworks are showing as ".NETCoreApp 5.0". The platform information is completely invisible, per https://github.com/NuGet/NuGetGallery/issues/8268. We are waiting on word from PM on what the desired UI should be.

The fix is now in PROD. Thanks for reporting this, @f2calv, and for the details everyone!


Fix CI/CD pipeline - Don't trigger subsequent job if dependent job(s) fail

Type

[x] Bug fix
[ ] New feature
[ ] Improvement to existing feature

Related JIRA Issue(s)
https://www.ebi.ac.uk/panda/jira/browse/ENSWBSITES-619
Importance
Jobs in CI/CD pipeline gets triggered even if the dependent job(s) have failed/not triggered.
Description
CI/CD pileline in Ensembl client has three stages

test_build_static
build_docker_images
deploy

The job in deploy stage should not proceed if build_docker_images is not triggered or if it fails.
This leads to trigger the deployment even if the docker images are not present.
Views affected
Other effects

[ ] I have checked any requirements for Google Analytics
[ ] I have created any required tests
[ ] The PR has as its final commit a WASM/JS update commit if any Rust files were updated.

Possible complications
None


put easyquant into a package

Easy quant is now into a package
Create a virtual environment with python >3.6 (I tested with 3.8)
Build a installable wheel: python setup.py bdist_wheel
Install it: pip install dist easy_quant*.whl
There will be three commands available in the command line:
easy_quant
easy_plot_reads
easy_quant_requantify
Python dependencies are already managed in the package. Next step would be to automate the publication of the package in Python.

Will do some testing before merging


How to pass dependencies between modules?

I see that this library took as a basis code from the RxSwift book. To me, this approach also seemed very laconic and clean. I see the main difference between your approach and the book. In the book, the authors propose to transfer the object of the viewModel in the enum case, you just pass the necessary data (string, number, etc.). The only option I see. add one more associated property to case. What do you think?

You can totally do that. We try to not force users to choose a specific option.
Sent with GitHawk


Request for Intestinal organoid dataset(scEU-seq)

Hi there,
I noticed that CellRank2 already supports analysis of metabolic labeling data and provides example data, but the dataset does not provide a download link, please update the download link for the intestinal organoid dataset to datasets.py
Thank you very much!

cc'ing @WeilerP

@xfchen0912, we will not expose the data via a corresponding function in cellrank. As mentioned in the reproducibility README, you can simply download the corresponding files from FigShare here.

I think the correct download link would be:

Raw data: https://figshare.com/ndownloader/files/41674299
Processed data: https://figshare.com/ndownloader/files/41674206


Create style.css file in the assets/stylesheet folder

Create style.css whith code from https://github.com/volodymyr-kushnir/volodymyrkushnir.com/blob/develop/assets/stylesheets/base.css

You don`t add code from https://github.com/volodymyr-kushnir/volodymyrkushnir.com/blob/develop/assets/stylesheets/base.css !!!!!((((


[HTML] Support the tsserver resolving files relative to the html file

Re: https://github.com/microsoft/vscode/issues/26338
I'm not sure it completes every feature requested in #26338 so I'm not marking this PR as fixing it, it might, but I don't have enough tests to verify it - but it does solve the one I really wanted: resolving JS Doc import types inside the script tags.

This PR switches just the TSServer to support reaching into the file-system in HTML-language-features giving TypeScript the chance to handle types across file boundaries.
Auto-complete for imports

Grabbing globals from @types

Hi. Unfortunately, the Microsoft team does not want to address this issue intentionally, so merging is unlikely. They ignore it for 4 years. Please make an extension to vscode. And thank you for working to solve this problem. :-)

That's not a very helpful comment, the vscode team's project has a massive scope and the issues for this are both backlog'd and still open. This is generally a good indicator that they would accept a small  and tested 3rd party PR for those items. Though, FWIW, I am also a Microsoft employee and I've definitely ignored many GitHub issues far far longer.

Thanks for helping out, @orta ! Yes file watching is needed and access to the unsaved text documents. -> the html server needs to subscribe for open JS documents as well.

OK, I've got this green and seems to work exactly how I expect overall.
Right now, it uses mtime on app js/ts files to determine if it needs to be re-grabbed, which is potentially little resource wasteful but also can handle all of the different possible files which people use in JS (think packagers like webpack which allow for importing a .css files into a JS file)
That said, I'd like to give it one more look to  see what expanding using the watcher for html files  looks like. So this is good enough if you want to try locally, but I'd like to at least give that one more run before being considered fully ready for review.

I think making this turn itself off elegantly (and basically providing the same experience there is today) in the cases where we don't have sync access to a filesystem probably makes the most sense

Is there a recommended system-level way to detect we're in a browser and then I can disable the TSServer?

You could look at the schema of the workspace folder(s) and if it is not file, don't use fs. That solution also helps VS Code opened on virtual folders

@orta Great PR! I really hope this one is being merged sooner or later. Until then: Have you released any extension based on this? I'd really like to have this instead of the internal one.

This is pretty close now,  it also checks for a local folder in the workspace and only allows runs in that case.  It can now be  bundled for web and doesn't have runtime issues:

Just gotta get it green and make sure I've not broken the local version in the process

Wow, thank you so much already!

@orta Is there anything, one could help with?

Nah, I've just not had a chance to get back to the PR, you're welcome to rebase and get the tests green - that's all there is to do atm. I just need to find the time and inclination to wrap it up

@orta Ah cool, thanks for the quick response. Let me see if I can manage it. This feature is very helpful for tooling that is focusing on "back to the roots" workflows/architectures (see Island Arch https://jasonformat.com/islands-architecture/) or web components and such; making the use of <script> more and more popular again; also given that we can now work with most of the modern APIs and language features at a 95% browser compatibility range and thus, folks are returning to Vanilla JS/TS more often. I'm working on an open source SSG solution which is also promoting <script> for simple use-cases. The developer experience in VS Code is def. dependent on this feature. I think some Snowpack / Astro.build folks would also be happy to have it (see https://docs.astro.build/core-concepts/component-hydration/ -- last section)

Yeah, I originally started this PR because I wanted to use this in my snowpack/vite apps

The CI fail is unrelated, this ins rebased and I added a test for the vfs - I'm a little surprised that I don't see the changes when I run the extension in dev mode, and I can't attach to the LSP. So I'm not sure if I just have a borked local vscode (which could be very possible) or it only works in tests and not reality :D
So I'll try look on another computer to validate

Hey @orta, very nice that you've spent some time on this again so quickly :) It would def. take me a while to get at speed with this. I've just cloned, compiled, ran the tests etc. :) It was a liiitle bit of a bumpy road for me as I'm dealing with the vscode codebase for the first time :D So, yeah, I have a bit of experience with the tooling etc. and just delved into the GitHub actions command to pave my way thru the jungle of what I need to get everything up and running locally in dev mode ;)
Finally, I've got it working after an hour or so :))
Here are my current test results:
Verified to actually work in reality, yay! 💯

Collides with this branch of reality :D
Auto-complete on (local) file paths

Auto-complete on import

In case you'd like me to test more you, I'm very open to do so if you'd like to give me some exact instructions.
Thanks alot!
Aron

Yeah, cool, that is basically the same experience as without this PR, thanks.
So, I've definitely broke it in file:// projects when making sure vfs ones don't get the behavior 👍🏻

@orta I think I've identified the issue - the fileExists: currentWorkspace && ts.sys.fileExists check is disabling the behavior in all cases, not just vfs. It could probably be fixed by doing the check within the function body rather than before the function is assigned. I'd be happy to make a change to get this PR over the finish line, if you'd like. Also, as you pointed out, the tests don't catch this and need to be updated. (They're also not testing that the functionality is disabled for vfs correctly).

Heads up, I've opened an MR that should address this same issue: https://github.com/microsoft/vscode/pull/193590

Is this still relevant?


Support for merging mmdb or csv files？

Is it supported to merge two mmdb or csv files that record different information?
For example, one is an mmdb or csv file that records country information, and the other is a file that records ASN. Is it possible to merge these two files into one that contains both ASN and country information, just like the three free databases you provide officially?
If it is possible to implement, can you give me an example of a specific command? If it is not possible to implement temporarily, is it possible to implement it in the future?

@photkey that's tough to do, not because of the different data per-se, but because the data is indexed by start/end IP ranges and you'd have to solve the problem of joining, aggregating, flattening, etc. the two different sets of start/end IP ranges.
This is something we do when producing these MMDB/CSV files in data pipelines, and that's already a very tough and costly process (can take hours to produce even small files).


[feature request] "state fliped" attribute

It would be nice to see in events how long something has been in a "not OK" state, so we can know how long something has been broken.

Closing due to inactivity. Please feel free to create a new issue if this matter is not resolved. #monitoringlove


Improvements

Had fun working with this Image Gallery and Azure, do you know when you might add the search and page numbering?
Thank you

Hello, thanks for the interest in this project. Right now all I can say is "Soon". I do not have a time frame when the features can be implemented. Currently working on another project and once it's finished I can return to this one.

I know it's been nearly 3 years if there's still interest, but pagination and searching by title features have been implemented as of today - February 11th, 2021. Just as an FYI.


Command "new" can't find some subreddits

This error occurs when there are no posts from the given subreddit

The bug has just been fixed.


docker-compose.yml: Introduce volumes

Add to docker-compose Version 2
Introduce volumes

LGTM, merged


[BUG] [Storage] [DataMovement] checkpoint files are not deleted when transfer completes

Library name and version
Azure.Storage.DataMovement.Blobs 12.0.0-beta.6
Describe the bug
When checkpointing is enabled for a transfer from local files to blob storage, the checkpoint files are not deleted once the transfer completes.
This means they keep accululating.
I've just deleted over 22 million files from .azstoragedml (about 9 GiB) created over several weeks of sustained transfers.
Expected behavior
I expect the checkpoint files to be deleted once a transfer has completed.  I can see perhaps for a failed transfer, you'd want to keep them in case the transfer should be re-attempted.  But for a successful transfer, the checkpoint files should go.  Otherwise they just keep accumulating.
Actual behavior
Checkpoint files stay
Reproduction Steps
Run a transfer from local disk to blob storage.
Start with an empty .azstoragedml directory.
Once transfer has completed, check contents of .azstoragedml directory
Notice there are still files there.  Bug.  There should be none.
Environment
Windows Server 2019
.NET Framework 4.7.1
Visual Studio 2022 (17.11.5)

NB: this turns into a performance problem, because once there are 22 millions files in a single directory, creating a new file as part of a new transfer takes some time.  For us, just deleting the files moved us from about 1.5 Gbps to 4 Gbps.

There's code which looks like it knows how to delete an entry:
TryRemoveStoredTransferAsync()
But this is never called from anything other than test code!


Scores between the mentions

I am trying to get the scores which are shown on the neuralcoref viz but don't seem to be able to do so. I tried to call doc._.coref_scores but it mentions that the extension has not been set. Has anyone been able to get the scores?

I'm having the same issue right now. Some coreferences are not resolved because the corefs scores are too low.
It would be great to set a threshold to force the coref resolution.
I saw doc._.coref_scores from 3.0.0 but i cannot use it since it's not released as a spacy package yet.
Do you have any news about this feature ?

Looked a little more into this issue. Looks like the code to get the coreference scores is there, but the models have to be retrained. So one of the solutions is to pull the latest version of master and then follow the instructions to retrain the models. Doing so, will enable the extension doc._.coref_scores.

The latest release (4.0) now outputs coreference scores.


Deployed solutions UX questions/improvements

the vdc name is already known why do we show it in the table if all are running on the same vdc?
what is "release" in the header, what is  "deployment" ?
why do we show urls for the pods
why all of them aren't grouped together?
why there's a delete button on pods? what the point? this page should be readonly.

If just for all of the pods, maybe we should rename it to be deployed pods?

Verified
commit d0eea8f1482459bd507ba0bfcca889be09023f6a


drivers: nxp_s32_netc: configure to grant SI permissions

Configure to grant SI permissions to allow to set MAC, update hash filter table and promiscuous multicast.
Fixes #66198

@congnguyenhuu please rebase

@congnguyenhuu please rebase
I rebased


Enums does not support literal typing with computed values

TypeScript Version:  3.2.0-dev.201xxxxx

Search Terms:

enum flag
enum computed literal typing

Code
enum AnimalFlags {
    None           = 0,
    HasClaws       = 1 << 0,
    CanFly         = 1 << 1,
    EatsFish       = 1 << 2,
    Endangered     = 1 << 3,

    EndangeredFlyingClawedFishEating = HasClaws | CanFly | EatsFish | Endangered,
}

enum RegularEnum {
    None,
    HasPowers,
}

const a = AnimalFlags.EatsFish;
const b: AnimalFlags.EndangeredFlyingClawedFishEating = AnimalFlags.EndangeredFlyingClawedFishEating;
const c = RegularEnum.HasPowers;

Expected behavior:
a Should have type of AnimalFlags.EatsFish
b Should not throw. Should allow the assignment of AnimalFlags.EndangeredFlyingClawedFishEating as type.
I would expect a computed Enum to behave as a regular enum, in this case, c has type RegularEnum.HasPowers
Actual behavior:
a is typed as AnimalFlags.
const b: AnimalFlags.EndangeredFlyingClawedFishEating is throwing Enum type 'AnimalFlags' has members with initializers that are not literals.
b should be allowed to be typed as AnimalFlags.EndangeredFlyingClawedFishEating.
Playground Link:
Link
Related Issues:
I think is the same that one reported last year #18393. It was closed as a Design Limitation, but since there has been some big changes in Typescript maybe this would be worth to be reviewed again.

This is still a design limitation. For mostly historical reasons we have two fundamentally different kinds of enums in the language, numeric and literal. Numeric enums are just like the number type, except that distinct numeric enums are not assignable to each other. Literal enums are quite different: Each enum member has a distinct unit type, and the enum type itself is a union of the enum member types. As described in #9407, we distinguish between the two based on how the members are declared:

When each member of an enum type has either an automatically assigned value, an initializer that specifies a numeric literal, or an initializer that specifies a single identifier naming another enum member, that enum type is considered a union (a.k.a. literal) enum type.

We chose to do it this way such that we didn't have to introduce an explicit modifier to choose between the two kinds. This still holds. The only way I can see us allowing computed constant expressions for literal enum members is to introduce a modifier, e.g. literal enum E { ... }. But I'm not crazy about that idea.

Is there any chance of revisiting this? This would allow for composition of enums. My use case is this, as part of an integration with the HubSpot CRM where booleans are passed as "yes"/"no":
// either
export enum HubspotBool {
  True = 'yes',
  False = 'no',
}
// or
export const HUBSPOT_TRUE = 'yes';
export const HUBSPOT_FALSE = 'no';

// doesn't matter which

export enum CustomerIdentityCheckStatus {
  Completed = HubspotBool.True, // fails
  NotCompleted = HUBSPOT_FALSE, // fails
  Failed = 'failed',
}

I know it's not super necessary but it's the sort of thing I would expect to "just work", where as currently it gives me an unexpected error which leads me here.

In the typescript documents, it says each enum member has a value associated with it which can be either constant or computed.
https://www.typescriptlang.org/docs/handbook/enums.html
enum FileAccess {
// constant members
None,
Read = 1 << 1,
Write = 1 << 2,
ReadWrite = Read | Write,
// computed member
G = "123".length,
}
But the computed doesn't work as the documents described, and throws an error.
TS2535: Enum type has members with initializers that are not literals.

In the typescript documents, it says each enum member has a value associated with it which can be either constant or computed. https://www.typescriptlang.org/docs/handbook/enums.html
enum FileAccess { // constant members None, Read = 1 << 1, Write = 1 << 2, ReadWrite = Read | Write, // computed member G = "123".length, }
But the computed doesn't work as the documents described, and throws an error in Vue3 SFC. TS2535: Enum type has members with initializers that are not literals.

It's ok that you can define the numeric enum "FileAccess" which has both computed and constant members. However, if you want to use the enum member as a type, the members of the numeric enum must be initialized by literal.
const num = 1;
// it‘s ok
enum UserResponse {
  // constant member
  No = 0,
  // computed member
  Yes = num,
  // constant member
  NotSure = 1 + 1
}

// throws an error  : Enum type has members with initializers that are not literals
type aType = UserResponse.Yes;
// throws an error  : Enum type has members with initializers that are not literals
type bType = UserResponse.NotSure;

// it‘s ok
enum UserResponse_1 {
  // constant member initialized by literal
  No = 0,
  // constant member initialized by literal
  Yes = 1,
   // constant member initialized by literal
  NotSure = 2,
}

// it's ok
type aType_1 = UserResponse_1.Yes;
type bType_1 = UserResponse_1.NotSure;


Using Sticky State for multiple tabs/category

Would it be possible to use sticky state for multiple tabs ?
Let's assume we have 50 tabs in our website, so do we need to create 50 states for each tabs? Is there any easier way to combine those state, and is it a good approach for this situation?

Yes, you need one state and one named ui-view for each sticky state.  Quite a few people have been asking about having the same state open multiple times.  This is not supported, and I do not think it can be easily achieved.  However, if each of the 50 tabs are for different states in your app, then yes you can have 50 sticky states... you must have one named ui-view for each sticky state.


No license found

Hello,
I've noticed there's no license for your project. It may be worth adding one to clarify whether people can use it or not.

It's open source. I just added MIT licence so you are free to use/modify/etc :)


chore: release 1.2.298

:robot: I have created a release *beep* *boop*
1.2.298 (2024-03-20)
Bug Fixes

nelm: WERF_DIR ignored for chart path (5acfaa3)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/werf/werf/releases/tag/v1.2.298 :sunflower:


userinfo() return data but "please login first" ?

{
"status": "error",
"msg": "Please login first",
"sub": "",
"name": "",
"data": "Please login firs\r\nt",
"data2": null
}
I use https://github.com/zitadel/oidc to do that

I solve this problem

@Rebellioncry how did you resolve this issue?


Adds test for sigint during input execution (e.g. timeouts)

TODO: fix async functions outputting a bunch of unactionable traceback spam when getting sigint:ed (commented out assertion for now)

@prbot approve

👍 This looks good!


Users can create, read, update, and delete data for a pizza in the database

As a user, I can create and customize my own unique pizza

As a user, I can retrieve data about my pizzas or other pizzas created through the app

completed   by @gsaaad

completed by @gsaaad


Adding license header to dummy login module

License headers were missing in the relevant files. This change adds them.

Codecov Report

Merging #1307 into master will increase coverage by 0.67%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1307      +/-   ##
==========================================
+ Coverage   46.68%   47.35%   +0.67%
==========================================
  Files         182      182
  Lines        3725     3725
  Branches      772      772
==========================================
+ Hits         1739     1764      +25
+ Misses       1986     1961      -25

Impacted Files
Coverage Δ

...mponents/datasets/containers/dataset-compliance.ts
41% <0%> (+1%)
:arrow_up:

wherehows-web/app/services/notifications.ts
56.86% <0%> (+23.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 c0fed1d...96d01ea. Read the comment docs.

Coverage increased (+5.1%) to 27.736% when pulling 96d01eaea7fca897913cc9eb00c284355e080a06 on jywadhwani:master into c0fed1dd03cef2267c637d361fd7d5690b70a229 on linkedin:master.


Make the classname applied configurable

It would be nice if you could override easily what class gets applied.

it will be even nicer if it can return some values. Right now i have to change the source code for that.

@eiclkun this seems to be very different than original issue isn't it ?

you are right, shall i create another feature request for that?

Rather than a feature request, it would be even better to create a PR. This project doesn't have any active core members. So while I do what I can to maintain it, if you want new features you'll have to step in.

Happy Saturday!  Here is a PR for the custom ui-scrollpoint-class feature, as well as the feature of a ui-scrollpoint-action callback function.

Released as 1.2.1


Add quickstart to Readme

Signed-off-by: John Lago<EMAIL_ADDRESS>

You have to do the change in axiom (opensource/devbox) and it'll get published here.
If you do the PR in the devbox repo it'll get clobbered when we republish from axiom.


Enhance attribute table in the ANCM config reference

Is there a way to set the startupTimeLimit to unlimited using 0, -1, or do we just set it to some large int?

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

ID: fe69b3a9-896f-e189-0c2c-f87ff735df9c
Version Independent ID: 97c8549b-66ad-0a23-1971-784fb923042a
Content: ASP.NET Core Module configuration reference
Content Source: aspnetcore/host-and-deploy/aspnet-core-module.md
Product: aspnet-core
GitHub Login: @guardrex
Microsoft Alias: riande

Do you really want it to hang forever? I'd set it to the largest int you can stand.

3600 (1 hour) is the max value ...
https://github.com/aspnet/IISIntegration/blob/e1f1705d11d97560c574dbe68a0b560441470f58/src/AspNetCoreModuleV2/AspNetCore/aspnetcore_schema_v2.xml#L17
It looks like a value of 0 (zero) is not infinite timeout ...
https://github.com/aspnet/IISIntegration/blob/26964b268740150ee4aa77e6b3ddf695cbb4aaaa/src/AspNetCoreModuleV2/OutOfProcessRequestHandler/serverprocess.cpp#L561
@pakrym can confirm.

@Rick-Anderson I'd like to increase the priority of this but leave it on the backlog for a closer look later. For example, our table doesn't have mix/max values or define if/when zero is zero or zero is infinite.

@pkzOR Thanks for opening this issue. I'm going to change the title to broaden this issue out a bit.

It appears to me that if you set it higher than 3600, it defaults back to 120. Is that true?

Yes. You can see that in action by manually changing the value in web.config (put some super-high value in there ... I put in 5000 for this example) and look at the configuration in the IIS Manager Config Editor ...

If you change it to a value within the range 0-3600 and refresh the config values (navigate away and back from the aspNetCore element in the UI), you'll see it pick up and display the valid value you set. I'll set it to startupTimeLimit="300" and refresh. It now shows ...

thanks... seems like if it's set too large, then it would default to the largest allowed.  Either way, thanks for looking at it.

I guess in some meeting back in the ancient times of IIS development that they decided not to write the validation logic that way. I can wildly speculate that their reasoning may have gone along the following lines: If the admin/dev had a typo in the value (way too high or too small), it's safer not to assume the limit (the lowest or highest allowed value) and just go with the default (usually a modest value). From the 'typo error' perspective, I can see why they would have set it up the way that they did.
If you think it should be the logic for validation, you could open it as a suggestion at https://windowsserver.uservoice.com/forums/310252-iis-and-web-server-role ... the engineers and managers DO read and respond to those.


Minimum Target Version

I wanted to upgrade to version 3.0, but the minimum version for the package was set higher... unfortunately I am still supporting several Windows 10 Mobile users, lots of them dealing with older Windows 10 versions.
Is there any specific reason for the new minimum version? The only real change beneath bug fixes was Rx4, and that one should be able to run down to build uwp 10240. Do I have to upgrade my minimum supported version just for Zeroconf now? (I know, old version are not supported anymore, but I was the good samaritan so far also caring for the minorities still using old Windows versions, even having several switches to handle features not available in older windows version.😋)

There was also a fix involving the network stack, and I decided not to try to do the fix for both network implementations. That's primarily why it was dropped.
After October 9th, 16299 is the lowest version of Win10 that Microsoft will support, so if it's not now, it's in four months. 2.9 is still around and Rx3 works with Rx4 apps that use the compatibility package.

Not completly true, 1709 is the lowest version in 4 months, but on Mobile this is not API version 16299 but 15254... support lasting until end of 2019.
I have still significant users on old versions, so I won't increase the minimum version in the near future... not sure if I really stick to 2.9 then or handcraft myself some 3.0.

Unlikely minimum version will change (to lower). So okay to close this one. ;)


Don't reset temporary XRef-entries during saving (PR 16392 follow-up)

Please note: I'm not aware of any bugs caused by this, however that might be more luck than anything else.
In PR #16392 the incrementalUpdate function, and all of its various helpers, were made asynchronous. However the call-site in src/core/worker.js wasn't updated, which means that we currently reset temporary XRef-entries while saving is ongoing.

/botio test


[WIP] Reporter with RuboCop-style JSON output

Hopefully resolves #62, but it needs further testing and some specs are still missing.

Thanks for answering my questions quickly, @skateman.
Looks good, but would love to see some tests of course.
I would also like to see the files passed directly to the Report class, as it makes sense that the report would contain the list of files that were scanned. That would save you from having to modify the interface of the Runner constructor, or anything in the CLI class really. The reporter could then access the files directly from the Report.
Thanks for your work on this so far!

That's a good idea, I'm on it!

@sds I've fixed it and added some tests too, is it enough?

Merged in 7d04364. Thanks!

Thanks!


_uvc Keyword not working with non-text fields

Hi Norm,
As discussed on the forum am having trouble getting the _uvc keyword to work with non-text fields such as
Name
Email
Address
People out there might also like to use
Phone
Link
though personally the first set would be more than fine.
Now I'm a non-coder so potentially I just need to input the parameters in a certain way. Or maybe it was only designed to work with text fields - either way I think it would be greatly enhanced to just work with these other fields with out any special form of expression
Use case is probably obvious but we would use it to uniquely identify clients on our system and prevent duplicates there. Very difficult to achieve this natively
Also, I have successfully used the _uvc keyword for text fields so am confident its not my set up - it draws in a blank value whenever the field has sub-fields and otherwise works
Thanks as always Norm - I have subscribed to the changelog as you suggested

Hi Michael,
I've upgraded the _uvc keyword to support Name, Address, Phone and Link field types.
Email is not required because Knack automatically enforces unique emails by default.
You can test it out now in KTL v0.13.10.
Enjoy,
Norm