(dev) Settings menu no longer works

I wanted to check if things worked in spanish now, but I can't :(.

On dev.pantsu.cat ?

No the latest build, running locally.

Seems to work fine for me:

Git seems to have been messing with me, after hard resetting it works fine.

The issue is back for me, I'm going to leave this open in case someone else is experiencing this. Broke this time while I was messing with .css files which makes no sense whatsoever.

After some more testing, this seems to happen at completely random after building+starting up the server. I'm going to leave this issue open in case someone else experiences this, but I'm starting to strongly suspect this has something to do with my setup.


Add Read Only Group Setting

@microsoft-github-policy-service agree

Thanks so much for your contribution, @drokliss , really appreciate it!


:ALEInfoToClipboard E354: Invalid register name: '+'

Information
VIM version
:version
VIM - Vi IMproved 8.1 (2018 May 18, compiled Mar 29 2019 20:08:59)
Included patches: 1-1073
Compiled by Arch Linux

Operating System:
ArchLinux 64 bits, up to date (rolling dist)
What went wrong

Something went wrong in specifically this place, and I also searched through both open and closed issues for the same problem before reporting a bug here.
Reproducing the bug

I did:

vim
:ALEInfoToClipboard

This happened:
[No Name]                                                     0,0-1          All
Error detected while processing function ale#debugging#InfoToClipboard:
line    5:
E354: Invalid register name: '+'
Press ENTER or type command to continue

:ALEInfo

:ALEInfoToFile worked:
 Current Filetype:
Available Linters: []
  Enabled Linters: []
 Suggested Fixers:
  'remove_trailing_lines' - Remove all blank lines at the end of a file.
  'trim_whitespace' - Remove all trailing whitespace characters at the end of every line.
 Linter Variables:

 Global Variables:

let g:ale_cache_executable_check_failures = v:null
let g:ale_change_sign_column_color = v:null
let g:ale_command_wrapper = v:null
let g:ale_completion_delay = v:null
let g:ale_completion_enabled = 0
let g:ale_completion_max_suggestions = v:null
let g:ale_echo_cursor = 1
let g:ale_echo_msg_error_str = 'Error'
let g:ale_echo_msg_format = v:null
let g:ale_echo_msg_info_str = 'Info'
let g:ale_echo_msg_warning_str = 'Warning'
let g:ale_enabled = 1
let g:ale_fix_on_save = 0
let g:ale_fixers = {}
let g:ale_history_enabled = 1
let g:ale_history_log_output = 1
let g:ale_keep_list_window_open = v:null
let g:ale_lint_delay = 200
let g:ale_lint_on_enter = 1
let g:ale_lint_on_filetype_changed = 1
let g:ale_lint_on_insert_leave = 0
let g:ale_lint_on_save = 1
let g:ale_lint_on_text_changed = 'always'
let g:ale_linter_aliases = {}
let g:ale_linters = {'rust': ['rls', 'rustfmt', 'cargo']}
let g:ale_linters_explicit = 0
let g:ale_list_vertical = v:null
let g:ale_list_window_size = v:null
let g:ale_loclist_msg_format = v:null
let g:ale_lsp_root = {}
let g:ale_max_buffer_history_size = 20
let g:ale_max_signs = v:null
let g:ale_maximum_file_size = v:null
let g:ale_open_list = v:null
let g:ale_pattern_options = v:null
let g:ale_pattern_options_enabled = v:null
let g:ale_set_balloons = 0
let g:ale_set_highlights = 1
let g:ale_set_loclist = 1
let g:ale_set_quickfix = 0
let g:ale_set_signs = 1
let g:ale_sign_column_always = v:null
let g:ale_sign_error = v:null
let g:ale_sign_info = v:null
let g:ale_sign_offset = v:null
let g:ale_sign_style_error = v:null
let g:ale_sign_style_warning = v:null
let g:ale_sign_warning = v:null
let g:ale_statusline_format = v:null
let g:ale_type_map = v:null
let g:ale_use_global_executables = v:null
let g:ale_virtualtext_cursor = 0
let g:ale_warn_about_trailing_blank_lines = 1
let g:ale_warn_about_trailing_whitespace = 1
  Command History:

commit 495bce32e957dcf45d6d6acf183599e18c92b09d (HEAD -> master, origin/master, origin/HEAD)

What's the output of :echo has('clipboard')?

:echo has('clipboard')

0

Replacing vim with gvim in ArchLinux yields 1
$ sudo pacman -S gvim
[sudo] password for user:
resolving dependencies...
looking for conflicting packages...
:: gvim and vim are in conflict (vim-minimal). Remove vim? [y/N] y

Package (2)  Old Version  New Version  Net Change  Download Size

vim          8.1.1073-1                 -3.45 MiB
extra/ gvim               8.1.1073-1     3.75 MiB       1.54 MiB

Total Download Size:   1.54 MiB
Total Installed Size:  3.75 MiB
Net Upgrade Size:      0.30 MiB

:: Proceed with installation? [Y/n]

I guess clipboard(register +) is the same as Ctrl+V in Firefox/mousepad etc., as opposed to key 'P' inside vim(register *) for example.

The difference between vim and gvim (./configure-wise) is:
-      --with-x=no \
-      --disable-gui \
+      --with-x=yes \
+      --enable-gui=gtk3 \

That makes sense. You need to install Vim with clipboard support enabled to use that comment. I'll update the command to explain that when you use it.


Does not honour "API alias" value as "-IssuingTemplate" variable

Environment
Full Operating System:
VenafiPS version: 6.7
PowerShell version: 7.4.5 (MacOS)
TLSPDC version (if applicable): N/A

Steps to reproduce

Add an issuing template and change the API alias value in TLSPC where "Issuing Template" is not equal to "API Alias"

Expected behavior

I would expect VenafiPS to work like vcert CLI does and honour the "API Alias" value
ie:

Actual behaviour

new-VaasCertificate  -Application Demo -IssuingTemplate zt -CommonName venafipsnew67.venafi.lab -PassThru

Exception: Issuing template zt does not exist

Using Alias value fails

Screenshots
Using Template name works

Looks like #314 fixed that issue. Thanks for the quick turn around


add Google Chrome 32 bit download link

Google Chrome has stopped supporting the 32 bit version for linux system
the installation files of cel it was therefore deleted from their server
Here backup link to the most recent version 48.0.2564.116-1.i386

Those links and torrent file are not belong here. Source code is checkable and verifiable, while links to external artifact storages are not. I can't vouch that links, nor torrent file, so I won't include them. Sorry.

By the way, Canonical announced some time ago that they had a permission from Adobe to redistribute PepperFlash alongside with NPAPI Flash. They are doing it in a single package: https://launchpad.net/ubuntu/+source/adobe-flashplugin.
PepperFlash version there is <IP_ADDRESS>, was taken from unstable channel of Chrome, apparently. But I don't know whenever they will continue to distribute 32-bit version. Current stable version is <IP_ADDRESS>, so version in that package is older than current, but is (somewhat) newer that <IP_ADDRESS>6 from Chrome 48.0.2564.116 from your link.

https://launchpad.net/ubuntu/+source/adobe-flashplugin now have a newer version, <IP_ADDRESS>, for both i386 and amd64.
So it's no longer feasible to use that last Chrome version, since newer PepperFlash version for i386 is available. Looks like Canonical is going to update that package, although with delays.


FluentFTP.GnuTLS: Crash when multiple threads use GnuTlsStream

Refer to #1272
Posted on behalf of @rob-hendrickx-c4t
Using it in parallel still leads to following exception though (on Dispose it seems)
Exit code is -1073741819 (Fatal error. System.AccessViolationException: Attempted to read or write protected memory. This is often an indication that other memory is corrupt.
   at FluentFTP.GnuTLS.Core.GnuTls.GnuTlsCertificateFreeCredentials(IntPtr)
   at FluentFTP.GnuTLS.Core.CertificateCredentials.Dispose()
   at FluentFTP.GnuTLS.GnuTlsInternalStream.Dispose()
   at FluentFTP.GnuTLS.GnuTlsStream.Dispose()
   at FluentFTP.FtpSocketStream.Dispose(Boolean)
   at FluentFTP.FtpSocketStream.Close()
   at System.IO.Stream.Dispose()
   at FluentFTP.Client.BaseClient.BaseFtpClient.Dispose()
// rest omitted

FYI, my parallellisation test just looks something like this:
[Test]
public async Task Then_we_should_be_able_to_connect_and_list_files_in_parallel()
      {
          var factory = services.GetRequiredService<ICreateCasFtpClients>();

          await Task.WhenAll(Enumerable.Repeat(1, NumberOfAttempts).Select(async _ =>
          {
              using (var client = factory.Create(new FtpCredentials
              {
                  Host = "xxx",
                  User = "xxx",
                  Password = "xxx",
                  Protocol = FtpProtocol.Ftps,
                  ValidateCertificate = false,
                  UseGnuTlsForFtps = true,
                  GnuTlsDisableAlpn = true
              }))
              {

                await client.ConnectAsync();

                var result = await client.ListDirectoryAsync("/download");
                result.Should().NotBeNull();
              }
          }));
      }

Posted on behalf of @rob-hendrickx-c4t
Note that, as you can see in above snippet, every client is only used by 1 thread. But as you mentioned in the original issue, the fact that this stuff is shared statically probably is the root cause of it. So until that part is fixed, I assume the implementation is not safe for concurrent code.
Do you happen to have a rough estimate on how long it could take before that is fixed (I fully understand it's not something that will be fixed straight away necessarily, and want to repeat that I'm amazed how fast the earlier fixes were handled)

Ok, I have looked into this.
Firstly, the entire inner structure of FluentFTP.GnuTLS is actually synchronous, as we state in the Wiki. I am overjoyed to see that it actually works well when internally "awaited" on via a AsyncFtpClient(...).
So using FluentFTP.GnuTLS in an inherently async environment is not detrimental in itself. Users just need to beware of possible thread lockouts on operations that have timeouts.
GnuTLS does support async internally, it is just so much more involved (many more interop callbacks to marshal), it didn't seem worth a huge effort. Maybe someone will sit down and start coding, given the stuff that is already there as a starting point?
Why do I mention this? Well, it was the first worry when I saw your async multi thread setup.
I duplicated it and have identified a number of places where it currently fails.
For example: I needed to serialize access to the log queue mechanism. There is only one global log queue.
But one of the other "places" is a big issue:
The base library wants an initial global init to setup its internal structures. After that it can be used in a thread safe fashion, for as long as you like. It then would like a global de-init to be able to dispose of its inner structures in a clean up.
Implementing this for a multi threaded use in an FTP client would mean exposing a global init (and deinit) for the user to call before he begins the actual use of the library as in creating ftp clients , using them and disposing them - de-init when he knows he is finished with the library.
These calls have no equivalent in an SslStream environment. FluentFTP (as the caller of FluentFTP.GnuTLS) has no knowledge of when to deinit the library.
Currently this deinit is done when the control connection is terminated.
This whole Init/Deinit is sort of unclean and a hassle.
I have experimented just now with calling the global init on first use and completely neglecting to de-init - thereby I have successfully run 50 simultaneous threads doing a connect-disconnect with no errors.
That's a really good sign - it tells me there might be only a little bit left to tweak and bingo, problem solved.
Because when adding a GetListing call, it went "Boom" again, so this init/de-init hack would not yet be a viable (albeit "unclean") solution.
I ran into snags in the session resume logic and disposing of the credential data areas, so the whole thing would need some more work and debugging.
I really can't give you an estimate.
Linux, ASYNC (Phew, it works) and multi-threaded operations was sort of beyond the scope of the initial attempt to go beyond just SslStream as an option. And now look how far we have come.
I will spend some time with this to give you a better estimate, but currently I feel that there won't be a solution this week.

Ok, made a small breakthrough on this.
If you accept calling a finalizer after the "everything" is finished, I think I have made the necessary changes to get thread-safety.
I will work on this tomorrow and I would like to produce a branch for you (instead of a Nuget), that you could test in your environment,
I have got the Create_Client_Connect-GetListing-Disconnect_Dispose_Client litany running here with 50 enums_tasks in parallel and no hitch. Wonderful intertwined logging... Yippeee!
The only question remains: Probably will introduce a config option to decide wether the lib will dispose  and de-init on termination of the control connection or to defer until the finalizer is invoked by the user. This also means, code is no longer totally the same between SslStream and GnuTlsStream - by the difference of one line of additional code to call the finalizer.
Might also need to introduce a thread-id in the logging format.
Please standby, out of the office tomorrow morning.

@FanDjango Thanks for getting back so quickly.
I don't have a problem with doing something like that, I just wonder what you mean exactly with "everything is finished". Maybe to give some context for the majority of our usage of FluentFTP:
We have some function apps that on one hand upload files from time to time (triggered by a user action), and on the other hand periodically list & download files from FTP (roughly every 30 seconds). Since there's constant activity, do we even need to call the finalizer in that case? Or should it still be used after every "batch of operations" is handled?
Generally speaking, why & when does this need to happen, for which reason exactly?
For the time being, I'll see if I can temporarily use locking to prevent the issue from happening in our case (but making it more sequential again)

You've definitely got a point there - no need to finalize if you're ongoing.
I've another 400km, be home in about 2-3 hours depending on traffic, dinner and then let's see about making a test version for you...
I think we'll have this working by Friday.

@rob-hendrickx-c4t
Ok, after some testing I actually feel quite confident to release this as 1.0.14. Have a go at it and see if it looks good for you.
Note the new config parameter to use:
		/// <summary>
		/// Normally, the GnuTLS library is de-initialised after dispose of the control
		/// connection. If you plan to use the GnuTLS library functions in a multithreaded
		/// environment, you need to disable this behaviour by setting this to false. The
		/// library will NOT be de-initialised.
		/// </summary>
		public bool DeInitGnuTls { get; set; } = true;

Hi @FanDjango
I can confirm everything now works as expected, much appreciated!! Feel free to close the issue :)

Ok, closed. Thanks for the exemplary feedback and issue documentation, also for the timely tests and cooperation. It was a pleasure taking this input and improving the library.


resizing from center doesn't work for groups/multiselection

steps:

create to elements and group them OR select them all without grouping
resize the group while holding Option/Alt to resize from center
doesn't resize from center

@dwelle Would love to work on this one. Just to confirm, The expected behavior is that the origin of resize for that group should be center, but only in case the user is holding Option/Alt key right?  Something like this?
https://user-images.githubusercontent.com/54989142/134804551-4f21b2e8-47f0-41d2-a2fe-8a662b468986.mov
Or Is there something else I'm missing? Let me know if I should create a PR? 🙂

That's correct!

That's correct!

Awesome! will raise a PR 🙂


Updated WeatherApp solution look & feel

Added JQuery mobile library
Removed JQuery NuGet package
Enclosed weather.js logic in an IIFE block
Updated the markup to improve the look & feel
Added error handling

Hi @jrebagliatti, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


make the cache in Valley scroll

it currently gets sampled 25 times per sample so this one should hopefully help quite a bit


Document native git options

I was attempting to make an empty commit with git commit --allow-empty, and after digging through the source, I was able to pass in {'allow-empty': true} to the commit function to get this to work.
the Repo#status docs make a vague reference to this capability, but it's not clear that this would work on any function.
Possible to include this in the docs?

Sure. Could you send a pull request?


Expose target service name in middleware

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
I'm writing a logger plugin for Traefik, it will log information like request duration, path, status code, ...
The goal is to use this middleware as a unified place to capture logs from Traefik, as it's being used as an API Gateway.
One of the things I would like to log is the 'service' the request is meant to be for.
The only place where I found this information is when I logged the next request handler.
os.Stdout WriteString(fmt.Sprintf"%+v\n", next))
gives me
&{router:whoamiadocker service:whoami-traefik-logger next:0x4000743f0}
Currently I use this hacky way to extract the value nonetheless.
// New instantiates and returns the required components used to handle an HTTP request.
func New(_ context.Context, next http.Handler, config *Config, name string) (http.Handler, error) {
	service := reflect.ValueOf(next).Elem().FieldByName("service")
....

Surely, there must be a better way to do this?
I couldn't find anything in the plugin guide and basically scoured exisiting plugins in the hope to find a way to tackle this.

I want to run Traefik in a microservice architecture, all request will flow through Traefik.
Instead of having all microservices implement their own logger middleware, I'd like to remove that responsibility from them and move it to a Traefik plugin.
The plugin would parse usefull information from the request, obfuscate headers where needed and log the result so it can be picked up by our logging setup.
On key part of usefull information would be the service the request would get routed too.
Does that clarify it enough?

Hi @mmatur
Any further comment?
Thanks

Hi @WDaan,
Why are not using the current access log filtering available in Traefik?
You can use json format and you will have a bunch of field available and the one you are looking for is ServiceName

Hi @mmatur, thanks for your reply 😄
TLDR; it's great but just not flexible enough. Traefik v3 breaks the above mentioned 'service'-hack

Can't make fields 'top level'

x-amzn-trace-id => I want this to be a top level field, because downstream services include this in their log messages and during debugging you can easily filter on this field
user_id,org_id => Same reason, currently I extract those from the JWT (I can probably set them as headers in the forward-auth middleware, but still cannot get them top level)

Custom log level => I set a log level based on status code or some other factor, so I can dashboard/report on that
Custom log format => our Fluentbit setup drops messages that do not adhere to certain rules. So I'd either have to make an exception for Traefik (which would break some dashboards) or massage/reformat it on Fluentbit side to support our format
No servicename remapping => I remap the service name from namespace-my-service-80@kubernetes to my-service, so I can see all log messages (app + requests) with the same filter e.g. service=my-service
No Path/Query Param extraction => api/organizations/392dcb9d-d855-4447-97bc-3255eab8e37a/?test=1234 gets remapped to api/organizations/:org as path and the params get logged as a separate JSON top level fields params and query. We use this for debugging and dashboarding/analytics use-cases

It was great to be able to use a Traefik middleware to generate this, instead of relying on all downstream dev teams to implement the standard properly in various languages. The middleware removes that pain point altogether.
Similar issues for metrics; the current metrics do not include a 'cleaned' path label (or any path label). We use this label so we can split our SLO's based on paths we expect to be fast/slow. Or on other labels that can be set from within the middleware. We currently export metrics ourselves because the default provided ones are not extensive enough and other labels can only be added from headers. On of the labels we set is the remapped service name.
Now with Traefik v3, this setup is broken because the 'service' field is no longer exists on the http.Handler
I tried to extract the LogData from the context like the accesslog-middleware does it but was unable get anything.
func GetLogData(req *http.Request) *LogData {
	if ld, ok := req.Context().Value(DataTableKey).(*LogData); ok {
		return ld
	}
	return nil
}

Is there any other way of getting access to the service name from within a middleware?
Thanks for your input 😄


Can't reproduce precision on pascal_voc

I'm sorry that I tried 3 times, but I can't get 74.9 map which is listed in your table.
I use command
python trainval_net.py --dataset pascal_voc --net res101 --bs 4 --nw 4 --lr 4e-3 --lr_decay_step 8 --epochs 10 --cuda
I barely changed the code and got map of 72.4,72.9,73.5 based on caffe model as pretrain.
I want to know how can I get 74.9 map for resn101?

Hi, @kakaxi314, did you use the pretrained model pointed by the link in the readme?

@jwyang Of course, I downloaded the pretrained model from you link. I want to know whether my command is wrong or anything in code I need to change?

@kakaxi314 can you reproduce the performance using vgg16?

@jwyang Hi，I trained VGG16 by using command
python trainval_net.py --dataset pascal_voc --net vgg16 --bs 1 --nw 4 --lr 1e-3 --lr_decay_step 5 --epochs 7 --cuda
I got 69.1 map, not 70.2 as your list.
I want to know is it normal?  Otherwise, is the command not right?

@kakaxi314 Hi, which epoch did you evaluate? I think the mAP is a bit lower if you set batch size =1

@kakaxi314 i am training the model on my side, will let you know when it finishes

@jwyang I almost evaluated models of all the epoches , but none of them gets similar mAP as yours.  Meanwhile, by using your trained model, I can get same mAP, which means the test_net.py works well. Thanks for your concern and wait for your reply.

Would you be so kind to tell me where could I download the author's trained-model as you mentioned above? I want to use the trained-model to test on other datasets. Thank you very much.

@kakaxi314 Hi, I trained vgg16 on my side with the same command as yours, and one epoch 6, I got:
AP for aeroplane = 0.7055
AP for bicycle = 0.7761
AP for bird = 0.6638
AP for boat = 0.5457
AP for bottle = 0.5221
AP for bus = 0.8086
AP for car = 0.8448
AP for cat = 0.8411
AP for chair = 0.5025
AP for cow = 0.7819
AP for diningtable = 0.6539
AP for dog = 0.7754
AP for horse = 0.8294
AP for motorbike = 0.7267
AP for person = 0.7726
AP for pottedplant = 0.4223
AP for sheep = 0.7188
AP for sofa = 0.6555
AP for train = 0.7526
AP for tvmonitor = 0.7277
Mean AP = 0.7014
the mAP is 70.1 on my side. Everything looks fine.

I retrained several times, but 69.2 is the best accuracy I can get.
Last time, in  one epoch 6, I got:
AP for aeroplane = 0.7398
AP for bicycle = 0.7872
AP for bird = 0.6665
AP for boat = 0.5396
AP for bottle = 0.4810
AP for bus = 0.7644
AP for car = 0.8442
AP for cat = 0.8459
AP for chair = 0.4151
AP for cow = 0.7680
AP for diningtable = 0.6091
AP for dog = 0.8030
AP for horse = 0.8347
AP for motorbike = 0.7601
AP for person = 0.7652
AP for pottedplant = 0.3716
AP for sheep = 0.6554
AP for sofa = 0.6419
AP for train = 0.7360
AP for tvmonitor = 0.6605
Mean AP = 0.6845
I can't find  the reason for this performance gap.
Do you use Pytorch 0.2.0?
If so, I will try to change my pytorch 0.3.1 to pytorch 0.2.0.

@kakaxi314 yes, I used 0.2.0, I guess it is the initialization that might affect the performance.


Feature request: a PSBT stack (or at least a finalizer)

What's missing from https://github.com/apoelstra/rust-miniscript/blob/master/src/psbt/mod.rs?

Probably just a finalizer.

Addressed in #119


Added name length cap for loadbalancer name

This change truncates the loadbalancer name to 63 characters max and ensures there are no trailing dots, dashes, or underscores.
/fixes #35

/retest

can we try this approach https://github.com/kubernetes-sigs/cloud-provider-kind/issues/35#issuecomment-2023397728 ?

[ ] we create a name prefix+hash (cap 63)
[ ] we add labels to the containers to refer to them by label instead of name

CI is failing because of the imports, just rebase and squash and get this merged 👏

/lgtm
/approve
Thanks


Support IE9

Hello,
This is a great project and I really like the work you did there, really clean and impressive.
I wonder if you guys plan to support IE9 ?
Thank you.

Hi,
First of all thank for your support !
Unfortunately, we're not going to support IE9 as we use flexbox in many components.
Flexbox is only supported by IE10+


Data.api should allow subkey access via fragment ids on the resource

The data.api should interpret a resource string like the path of a URL.  The fragment following the "#" should be interpreted as a '/' separated path to a key in the entity.  E.g.
/foo/1 -> {
  bar: {
     baz: 0
  }
}

SET /foo/1#bar/baz  to 5
/foo/1 -> {
  bar: {
     baz: 5
  }
}

SET /foo/1#bar  to 5
/foo/1 -> {
  bar: 5
}

Watches always apply to the top level entity, not subkeys.
Subkey modification should merge the permissions of the subkey into the entity permissions.
Subkey retrieval uses the entity permissions.

Dusting off the cobwebs on this, I can see some benefits and some concerns:
Brainstorming off of a mock example:

action
path
set entity
get response.entity

get
/foo/1
--
{bar: 2}

set
/foo/1#bar
5
--

get
/foo/1
--
{bar: 5}

get
/foo/1#bar
--
no

client.data().get("/foo/1").then(function(res){
    console.log(res.entity); // <------------------- { bar : 2}
    return client.data().set("/foo/1#bar",{entity: 5});
}).then(function(res){
    return client.data().get("/foo/1");
}).then(function(res) {
   console.log(res.entity); // <-------------------- { bar : 5}
   return client.data().get("/foo/1#bar");
}).then(function(res){
    console.log(res.entity); // <-------------------- 5
});

1.) Should all fragmenting be done bus-side?
I imagine handling this would be trivial with the functionality of routes added in v0.3.
2.) How should accessing a non-existent fragment function?
* A set will create the property of the node.
* A get will respond with an undefined entity?
3.) What actions would fragment access be usable with?

set : modifies given property of the node but does not impact other properties.
get: returns only the given property of the node, filters out unneeded information.
watch: behave like pre v0.3 watch capabilities (only oldValue,newValue no collection/children data) and return updates on the fragment rather than the resource as a whole.
delete: sets property to undefined.
list: should list apply? should obtaining a list of properties be allowed? normal list functionality gathers all resource paths below the given path, this would result a cluttered response if the node object depth is large.
bulkGet: see concern of list, it would return the values for the resource paths instead.
4.) Does this deter from the simplicity of pathing resources?

Using URI pathing makes resource relations fairly trivial, if a fragment of a resource is updated gathered constantly, but other fragments of the resource are not, does extracting that fragment into its  own sub-resource gain/loose any benefits? Extracting the fragment into it's own resource can add the benefit(or detriment) of more control over permissions.


[plow-technologies/onping2.0#523] Remove explicit constraints.

Gets one of its dependencies hS3 from hackage at the moment. We will have one in stackage shortly that is uses Stackage.


[Distribute Cores] Do not assume all subviews are 'distributing'

One role of the distribute-cores-and-objectfifo pass is to private L2 memory allocations.
That is, to replace IR like
  %alloc_3 = memref.alloc() : memref<2x2x8x8x4x4xi32, 2 : i32>
  ...
  %subview = memref.subview %alloc_3[%arg2, %arg3, 0, 0, 0, 0] [1, 1, 8, 8, 4, 4]
                      [1, 1, 1, 1, 1, 1] : memref<2x2x8x8x4x4xi32, 2 : i32> to
                      memref<1x1x8x8x4x4xi32, strided<[2048, 1024, 128, 16, 4, 1], offset: ?>, 2 : i32>

with
  %[[ALLOC_3:.*]] = memref.alloc() : memref<1x1x8x8x4x4xi32, 2 : i32>

Before this PR, the pass assumed all memrefs on allocs where of this form. But this is not always the case. This PR tries to determine if the memref is indeed of this form, otherwise it does nothing.
This is still not completely robust, we either do a proper privatization pass or get the root cause of the needing this in the first place: https://github.com/nod-ai/iree-amd-aie/issues/448
This PR also does a refactor of the code.

This is still not completely robust, we either do a proper privatization pass or get the root cause of the needing this in the first place: #448

+1 for we need to make this more robust. This shared buffers on L1 complicate the logic quite a bit.
The root cause would the scf.forall ops operating in parallel to compute a different slice of the output tensor. It doesn't seem that easy to handle that earlier, before we unroll.

I haven't explored where would be easy/best to run this as a standalone pass, but it would ideally be it's own pass. Future work!


[full-ci][tests-only]Bump ocis commit id edge

This PR bumps latest commit id from ocis
Part of https://github.com/owncloud/QA/issues/807

@PrajwolAmatya somehow GitHub is reporting conflicts - so that needs to be sorted out.

In this PR https://github.com/cs3org/reva/pull/3849, drone env api test url, branch and commit id is changed to aduffeck's ocis repo. Is that intentional change or forgot to revert back to owncloud repo.

In this PR #3849, drone env api test url, branch and commit id is changed to aduffeck's ocis repo. Is that intentional change or forgot to revert back to owncloud repo.

I think, we can revert to ocis repo after reva bump in ocis.

I will rebase and put in the latest commit id etc.
Give me a few minutes.

Rebase done - let's see what CI says.


Add support for checkpoint frequency option. Also rename RegisterObse…

…rverAsync(IChangeFeedObserverFactory) to RegisterObserverFactoryAsync.

@shireeshkthota

@mkolt,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Add PipelineExtensions for IRequest and INotification

PipelineExtensions.cs allows a pipeline style of programming rather than nested style. (This is not related to PipelineBehaviors.) This style is possible with F#'s Pipe Forward operator. And, there is discussion in adding Pipe Forward operators to C# in csharplang.
Instead of nesting:
Mediator.Send(
    new Request()
    {
        Value = "value",
    },
    token);

PipelineExtension:
new Request()
    {
        Value = "value",
    }
    .Send(Mediator, token);

Pipe Forward (proposed CSharp feature):
new Request()
    {
        Value = "value",
    }
    |> Mediator(@, token);

The discussions have suggested using @ or <| (what F# uses).
Unit Tests included.

Very interesting! Does this have any use if C# doesn't have the pipe forwarding operator? Or would this be included to work better in F#?
I don't like adding speculative features is why I ask.

Closing for now, until C# gets a pipe forward operator.


Flex APIs / NM Placement

Added "profile" parameter to flexdown API
Added "constraint" parameter to both flexup/flexdown APIs. Currently supports:

only 1 constraint
only LIKE operator
hostname field
Mesos slave attribute field

Improved the parameter validation of both flexup/down (Continuation of work done by @klucar)
Changed the behavior of flexdown to pick NMs in "pending", "staging" states ahead of those in "active"
Updated the API doc.

I've addressed most of the review comments (dynamic slave attributes is for future). I've also fixed/cleaned up parts of flex down code to address some edge cases. This is stable AFAIK.
Unless there are any comments/concerns, this PR is ready for merge.

Merged this PR to https://github.com/apache/incubator-myriad/commits/master.
Start commit: https://github.com/apache/incubator-myriad/commit/afa9ac904e11d72c544f6cd2499cf724dc62ab35
End Commit: https://github.com/apache/incubator-myriad/commit/e1841949196f62e861ac0a7e6f39a77d3f55fcdd
Also merging this into mesos/myriad to allow @yufeldman rebase her JHS work.


x86_64-unknown-linux-musl/TabNine is not executable

I am facing below issue with latest version of plugin.
.local/share/nvim/site/pack/packer/start/cmp-tabnine/binaries/4.4.1/x86_64-unknown-linux-musl/TabNine' is not executable
stack traceback:
        [C]: in function 'jobstart'
        ...pack/packer/start/cmp-tabnine/lua/cmp_tabnine/source.lua:234: in function '_on_exit'
        ...pack/packer/start/cmp-tabnine/lua/cmp_tabnine/source.lua:135: in function 'new'
        ...e/pack/packer/start/cmp-tabnine/lua/cmp_tabnine/init.lua:8: in function <...e/pack/packer/start/cmp-tabnine/lua/cmp_tabnine/init.lua:7>

Solved after reinstalling plugin

I am facing the same problem when I use goneovim. and it was not solved after reinstall plugin.

I'll need more info than that to help you.

I'm sorry for the mistake. I use mac, not linux, and  It's not plugin's problem.  goneovim will get an error arch x86_64 when it run the command of vim.fn.system('uname -m') on m1 Mac .


Store additional results for removed and corrected events

BACKEND PULL REQUEST
Related Issue

Fixes #3831

Changes Proposed

For test removals, create a new Result object that links the testOrder to the new removed testEvent. This is required for correctly printing and displaying results to patients.
For test corrections, instead of overwriting the Result with the corrected test event, create a new one.

Additional Information

This PR blocks #3493
This whole Results flow is just begging for a refactor, but it's going to have to wait for another day

Testing

How should reviewers verify this PR?

Checklist for Primary Reviewer

[ ] Any large-scale changes have been deployed to test, dev, or pentest and smoke tested
[ ] Any content updates (user-facing error messages, etc) have been approved by content team
[ ] Any changes that might generate questions in the support inbox have been flagged to the support team
[ ] GraphQL schema changes are backward compatible with older version of the front-end
[ ] Changes comply with the SimpleReport Style Guide
[ ] Changes with security implications have been approved by a security engineer (changes to  authentication, encryption, handling of PII, etc.)
[ ] Any dependencies introduced have been vetted and discussed

Thanks for working on this, I still see some weirdness when handling corrections Currently the old result keeps getting updated as you update the correction order and then a new copy result is created as you submit a new one, this means the old test event will end up with a result that has the updated result, which is incorrect. Happy to pair on this and show whats going on, since its hard to describe over text.

Well this ended up being quite a puzzle, but several pairing sessions later it should be working!
Previously, the Result for corrected events was being created when the correction was submitted, so in the interim editQueueItem stages the old Result object was being altered instead of a new one.
The new change looks for a "pending" result (one with a TestOrder/disease but not an associated TestEvent) and updates that on the editQueueItem mutation. In practice for corrections, there won't be a pending Result on the edit, so a new one will be created. This also has the benefit of cleaning up our logic for non-corrections, since the pending/not pending Result flow is the same regardless of correction status.
Reviewers - let me know if you have questions about any of this, I'm happy to pair!

@emyl3 realized I haven't CC'd you on any backend PRs in a while - let me know if you have questions about any of this!

@zdeveloper thank you for finding the issue with removing a corrected test event - that should be all fixed now.

I hereby give my blessing for this to be merged next week when I'm not around, if it has approvals and makes life easier for the rest of the team working on multiplex!


chore(lib): remove 0x prefix stripping

since it is now handled in ethabi

error fourbytes api related


Added utilities to expose algorithms in docs

fixes #163

Please check if the PR fulfills these requirements

[x] This PR addresses an already opened issue (for bug fixes / features)
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)
[x] HISTORY.rst has been updated
[x] bumpversion (minor / major / patch) has been called
[ ] Tags have been pushed

What kind of change does this PR introduce? (Bug fix, feature, docs update, etc.)
Adds an Utilities documentation page exposing some of the methods and algorithms used in indices

Does this PR introduce a breaking change? (Has there been an API change?)
No.

Other information:
No.

When merging, please put fixes #{issue number} in your comment to auto-close the issue that your PR addresses. Thanks!

Pull Request Test Coverage Report for Build 855

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.208%

Totals

Change from base Build 852:
0.0%

Covered Lines:
1278

Relevant Lines:
1386

💛  - Coveralls


Create many objects

With Machinist I could do
Object.make(5)

And it would return an array with 5 objects. What would be the equivalent in Fabrication?

Currently you will need to do it the old fashioned way.
5.times { Fabricate(:object) }

However, I got a pull request last week adding an optional machinist compatibility layer to fabrication. I am currently trying to get the specs running green with all three ORMs. Right now it only supports the basic make and make!, but we can add support for handling an object count as well.

Thanks!


Handling for BigInteger values added

I've added handing for BigInteger values comes from Influx response deserialization
#66

Any updates on this ? Its a blocking issue for us


Responsive Design Issue - Ensuring Proper Functionality on Tablets, Phones, and Desktop

Currently, the project lacks proper responsiveness across different devices, including tablets, phones, and desktops. It is crucial to address this issue to enhance user experience and ensure consistent functionality across all platforms.

The objective of this issue is to implement responsive design principles and make necessary adjustments to the existing codebase to achieve optimal performance on various screen sizes. This includes:
Reviewing the current layout and identifying elements that do not adapt well to different devices.
Making adjustments to CSS styles, media queries, and layout structures to create a responsive design that adapts fluidly to different screen sizes.
Testing the responsiveness of the website/application on multiple devices and resolutions to ensure proper functionality and visual presentation.
Addressing any specific issues or bugs that may arise during the implementation of responsive design.
This issue is open for discussion, and we welcome suggestions and collaboration from the community to ensure the best outcome. Let's work together to create a responsive and user-centric project!
Feel free to ask any questions or provide additional information related to this issue
Let's make our project responsive and accessible to all users!

Please assign this task of responsiveness to me. I am well versed with html , css and javascript.

Currently, the navbar is also not responsive, it squeezes and does not look pleasant. So I would like to fix this issue and .
Sample:-

https://github.com/thestarsahil/Counsellor/assets/105661636/2279c7b5-bf08-4247-b2ce-e9218297c5aa

fix other components which are not responsive.

I can make this responsive please assign this to me under gssoc'23


sane defaults for experiment caffe isolation

Fixes issue "run caffe without core isolation by default" - makes default results more sensible (and with align with presented results)
Summary of changes:

change LLC isolation for caffe to false by default

Testing done:

yes

run integration tests

run integration tests

run integration tests


Test code coverage report files (as well as XML, HTML and JUnit)

This pull request derives from #340. So needs to be handled after that.

This pull request adds an end-to-end test for code coverage.
How to run the test
Just follow README.
If you have Saxon-PE or EE, coverage-tutorial.xspec is executed with code coverage enabled. The resultant code coverage report HTML is compared with the expected result.
If you don't have Saxon-PE+, the test is skipped automatically. (Travis, AppVeyor)
How it works
In the same way as #326.

@cirulls
Merged this improvement of test.
With Saxon-PE+ or oXygen, the test runs like this:
Buildfile: ...\test\end-to-end\ant\run-e2e-tests\build.xml
...
xslt.supports.coverage=true
...
Testing coverage-tutorial.xspec [tc]
passed: 1 / pending: 0 / failed: 0 / total: 1
Comparing
   Actual: .../test/end-to-end/cases/actual__/stylesheet/coverage-tutorial-result.xml
 Expected: .../test/end-to-end/cases/expected/stylesheet/coverage-tutorial-result.xml
OK
Comparing
   Actual: .../test/end-to-end/cases/actual__/stylesheet/coverage-tutorial-result.html
 Expected: .../test/end-to-end/cases/expected/stylesheet/coverage-tutorial-result.html
OK
Comparing
   Actual: .../test/end-to-end/cases/actual__/stylesheet/coverage-tutorial-coverage.html
 Expected: .../test/end-to-end/cases/expected/stylesheet/coverage-tutorial-coverage.html
OK
Comparing
   Actual: .../test/end-to-end/cases/actual__/stylesheet/coverage-tutorial-junit.xml
 Expected: .../test/end-to-end/cases/expected/stylesheet/coverage-tutorial-junit.xml
OK

Now we have a fairly good range of test coverage. run-bats.*, run-xspec-tests-ant.* and run-e2e-tests.* cover Schematron, XQuery and XSLT including code coverage and schema-aware both on *nix and Windows.

Good stuff! Thanks a lot for improving the test suite!


Add metrics info

 {
        "id" : "geoip_6346ed52-2773-4b0a-8783-1b0c794fd0e5",
        "cache_size" : 1000,
        "events" : {
          "duration_in_millis" : 67,
          "in" : 1,
          "out" : 1
        },
        "matches" : 1,
        "name" : "geoip"

The test failling is similar to https://github.com/elastic/logstash/issues/5518
I have to do more check.

What I know so far:

We should always metric available, by default the lazy set will use a NullMetric
No threads are involved in the code.

Cleaning the *.lock and running with the latest released core allow me to reproduce the error locally.

Ok, I understand why this is happening this is actually a bug in core which will require a new snapshot release. Its because the NullMetric in the plugin.rb base class is not correctly setup when no pipeline is involved.

This should be fixed, but it probably require a new snapshot release of the core for the test to pass?


Add a default host and port for the microgrid client

What's needed?
A default host and port to connect to when connecting to the microgrid API.
Proposed solution
The SDK used to provide a default host and port to connect to:
DEFAULT_NITROGEN_HOST = "[::1]"
DEFAULT_NITROGEN_PORT = 61060

We could also provide a default host and port but the MicrogridGrpcClient takes a grpc.aio.Channel directly, so all the connections parameters should be set outside of the client.
We could provide just bare constants that the user could use when creating the channel, but the usefulness of that might be reduced.
Another option would be to add constructors to create the GRPC channel in the client, but then the client should decide on other parameters of the channel, like if it is secure or insecure, for which it should probably need to take extra parameters, also limiting its usefulness.
Use cases
No response
Alternatives and workarounds
No response
Additional context
No response

I will close this in favor of:

#221

We could still provide some localhost default if there is no config, but that should be discussed in the other issue, as it depends on the context.


[vue] Imports requires to omit .vue extension

When running vue-loader with ts-loader without this plugin, I have to use .vue extension to perform imports. But when enabling this plugin, I have to omit them for build to pass.
See this comparison => Both branches builds with no error.
It would be better embrace default import behavior, but I'm not sure how the underlying webpack configuration is involved ...

The extensions setting in Webpack configuration should be used to determine whether to add or omit .vue.


ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #16.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #16

@conda-forge-admin restart CI

@conda-forge-admin rerender


[Feature] Attention sinks kvcache on mistral

Also added sinks support on llm_chat.cc. Haven't finished SLM sinks support, but it should be ready very soon (had some work to merge for quite some time on the former flow, so decided to finish on this one first).
Follow-up PRs to make attention sinks default to other models.
Note: Mistral with attention sinks can withstand long contexts with small sliding windows as low as 1024 (tested 768 and it can provide acceptable results too) and only 4 sinks. Despite maintaining good performance, one thing that I noticed is that sometimes quantized models start repeating emojis and paragraphs. However, this can be softened by slightly increasing the repetition_penalty (around 1.2 was enough to get much better results)
cc @tqchen

SLM mistral sinks ready, same behavior as before


python310Packages.agate: Enable more tests

Description of changes
Disabled test has been fixed upstream https://github.com/wireservice/agate-excel/commit/9965cd4a252d488acb89e89113953a450cbe8aa4
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)
[x] 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.11 Release Notes (or backporting 22.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

@ofborg build python39Packages.agate


bugfix inspectorscripts.ts pytorch

Hi there,
I'm sorry. I slacked a bit today. There were some typos and I didn't test properly. This is tested.
David

Thanks :+1:

Thanks :+1:


extend and add new snippets

Ok, everything addressed I think :) Thanks!

Looks great now! Thanks!


Fixed doubling binary frames in size

Caused by the fact that a slice was allocated for the read data of size n,and then the data was appended. However, append(...) does not overwrite the contents of the slice, it makes it grow larger. Thus, the resulting slice is n * 2.

You're right...  Can you change to make([]byte, 0, n) to make it explicitly pre-allocated?

Done :) Amended the existing commit to avoid cluttering the history.

Thank you.


Convert composite build snippets to samples

Right now the content is the same as in the original readme files.
Fixes #3079

@big-guy I've adjusted the samples' sources. The samples plugin expects each sample to have a root project (with a settings file). Now each sample declares the composite build in the root directory.

merged to release


Autodoc

This PR contains fixed configurations for autodoc documentation, and updates all docstrings to a common style compatible with sphinx. It branches off the workflow_improvements, so this should be merged first.

We might want to add a workflow for automatic generation and publishing the docs e.g. on an guthub.io page.


No Option to browse and attach Private Key in Google IdP configurations in Console

Describe the issue:
When configuring Google IdP we need to attach a Private Key file in the provisioning configurations. But there is no option to browse and attach a Private key file.

How to reproduce:

Log in to the Console and click on Develop tab
Click on "Identity Providers"
Click on "New Identity Provider" and select "Google"
Fill the General settings and click Next

Expected behavior:
There should be a button any method to browse and attach the private key file
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: 5.12.0-M1
OS: Mac
Database: MySQL-5.7
Browser: Safari-14v

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Add GitHub actions

Per Lauren: also shows build failing, and trying to view on Travis has a 404...see if you can move this one to Github Actions

@somexpert This is ready for review.

This looks good to me! :thumbsup:


[NETBEANS-2109] NetBeans Updated gif

updated gif for better gradient

Looks good to me.

Awesome, merging.


Switch entropy logic

This commit builds upon #13, and aims to follow the strategy described in https://github.com/GillesVandewiele/Wordle-Bot/issues/9#issuecomment-1193172701 for the computation of entropy.

I would need to test the software to see if I did not break anything with my changes.


Unnecessary assets packaged with JAR

The assets/ folder includes GitHub README images, which aren't used in the actual mod and increase JAR download size from <1mb to 19+. This isn't particularly significant for PCs with a good internet connection, but on mobile devices with data caps and low storage space, it makes a noticeable difference. Proposed solution: Move the files into a different folder.
(If you're wondering why I'm here, it's because GitHub suggested me this repo and I was slightly bored)

Moved the README images from assets to a demos folder.
Dang it's weird seeing Anuken on a random repo like this.


Show difference histogram when Map of changes is triggered

I think it would be better to add a "Generate difference image" button or even "generate difference image histogram". It would be a better UX than just throwing windows to the user


fix recording rules to work with multiple HostedClusters

What this PR does / why we need it:
The Prometheus recording rules do not work with more than one HostedCluster.  This PR fixes them.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[x] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/test e2e-aws-metrics

/test e2e-aws-metrics
/test e2e-aws

metrics are good :+1:
/hold cancel


Expose champion index

Could be a dump of all the info or possibly search by name or ID.  Any extra data like multipliers or would also be great.

champion data is now available on IStaticDataApi in both C# and NPM client SDK packages (still TBD in python)

added to python lib


(GH-184) Enable install of remote templates

Prior to this commit the only way to install a PCT template was to have
already downloaded the tar.gz locally.
This commit updates the Install function to handle a templatePkg
specified as a valid URL, downloading the file to a temporary folder
and then handing the file off to the remainder of the function to
process like any local file.
This commit also implements the new downloadTemplate function, which
handles downloading and placing the template archive file, returning the
path to it.
TODO: unit/acceptance tests

Okay, so status: I have things working locally and in tests except on not-Windows, which is a little baffling. Took me much longer than I want to admit to get my head around how mocks work and how we're using them but I think I'm there. Out of time to investigate today, will pick it up next week.


Instructions to enable in Rider

Hello,
I've just installed your plugin in Rider (Thanks for developing this plugin!), but can't seem to get it working with the *.md files in rider. I tried uninstalling the bundled markdown editor but then the md files are just displayed as text...
It would be good if you can add the instructions to your readme
Regards
Brett

I'm on rider 2021.2 and 1.8 of markdown. But after restarting I can see the editor tab. Actually it might have been there before but I didn't notice it.
Thanks again for your plugin. Looks good!


feature:  云梯安全组添加白名单功能 (closed #1760)

PR
CheckList（PR Assigners）
确认已完成以下操作：

[ ] 已将分支与当前合入分支同步
[ ] 已在关联 Issue 中补充「功能自测」部分内容
[ ] 已更新关联 Issue 的状态 backlog -> todo -> for test -> tested（需通过测试同学测试） -> done
[ ] 将 PR 添加上 pr/reviewable

CheckList（PR Reviewers）
确认已完成以下操作：

[ ] 检查关联 Issue 「功能自测」部分内容
[ ] 必要时打回 pr/reviewable，要求重新修改
[ ] 及时 Approve

Codecov Report

Merging #1762 (64c8dce) into v2.4.1-dev (8d03634) will decrease coverage by 39.55%.
Report is 1 commits behind head on v2.4.1-dev.
The diff coverage is 36.58%.

@@               Coverage Diff               @@
##           v2.4.1-dev    #1762       +/-   ##
===============================================
- Coverage       73.94%   34.40%   -39.55%
===============================================
  Files             396      297       -99
  Lines           26706    21171     -5535
===============================================
- Hits            19748     7283    -12465
- Misses           6958    13888     +6930

Files Changed
Coverage Δ

...mponents/collections/agent_new/configure_policy.py
13.95% <0.00%> (-71.77%)
:arrow_down:

apps/backend/utils/dataclass.py
23.80% <23.80%> (ø)

apps/node_man/handlers/security_group.py
35.00% <35.84%> (-53.47%)
:arrow_down:

common/api/__init__.py
91.30% <100.00%> (-8.70%)
:arrow_down:

config/default.py
82.86% <100.00%> (+0.30%)
:arrow_up:

... and 262 files with indirect coverage changes


em-synchrony/activerecord should not require mysql2

I am using the EM-compatable PostgreSQL driver, not MySQL.

William, we're not actually requiring mysql2, the AR driver simply adds a new driver type. Unless I'm misinterpreting something here...

The statement require 'em-synchrony/activerecord' raises RuntimeError: Missing EM-Synchrony dependency: gem install mysql2 when the mysql2 gem is not installed.

Thanks—I appreciate the fix.


fix: text align at center

resolved #359

maybe it will be better to set the centering individually, right?

maybe it will be better to set the centering individually, right?

We can have an optional prop to set it, but the default should be centered.


Hide cursor from feedlist and articlelist

Do we really need the cursor (that red rectangle) in the main window (not command line)? Maybe, it worth to hide it, or add an option?

I agree, that looks bad.
I actually wonder where the red is coming from—it's certainly not the default in Newsbeuter itself, and the theme you're using (solarized, right?) only uses red to highlight URLs in articles.

Red color is intended by my urxvt's config. I only wanted to raise the question at all, regardless of the cursor's color.

#379 and #384 implemented this change for Newsbeuter, but Podbeuter still displays the cursor. This, too, shall be fixed.


Add xcode 14 support

necessary changes for Xcode 14 support.

lgtm, thanks for the bump!
These changes seem like could also fix #537 by just using the right version of Xcode?

@ob I think so. I will look into that once I merge this. I'm doing some local testing for Xcode 14 to double check the bazel piece.

The merge failed the checks. I will look into it


[bug] when using --context=, the context strings are not provided with --json=stream

Hi there,
First of all thank you for this amazing tool!
I am integrating a neovim plugin, https://github.com/MagicDuck/grug-far.nvim with ast-grep and I've noticed that the streamed json (from --json=stream) does not contain context strings (see more below).
⏯ Playground Link
not applicable
💻 Code
does not matter
Pattern:
does not matter
🙁 Actual behavior
running the following command:
sg run '--pattern=$A()' --context=2 ./tests/base/test_ui.lua

I get the match with context as normal:
tests/base/test_ui.lua
4│-- TODO (sbadragan): add tests for sg
5│---@type NeovimChild
6│local child = MiniTest.new_child_neovim()
7│
8│local T = MiniTest.new_set({

when adding the --json=stream flag though:
sg run '--pattern=$A()' --context=2 --json=stream ./tests/base/test_ui.lua | jq

I don't see any context strings to use:
{
  "text": "MiniTest.new_child_neovim()",
  "range": {
    "byteOffset": {
      "start": 161,
      "end": 188
    },
    "start": {
      "line": 5,
      "column": 14
    },
    "end": {
      "line": 5,
      "column": 41
    }
  },
  "file": "tests/base/test_ui.lua",
  "lines": "local child = MiniTest.new_child_neovim()",
  "language": "Lua",
  "metaVariables": {
    "single": {
      "A": {
        "text": "MiniTest.new_child_neovim",
        "range": {
          "byteOffset": {
            "start": 161,
            "end": 186
          },
          "start": {
            "line": 5,
            "column": 14
          },
          "end": {
            "line": 5,
            "column": 39
          }
        }
      }
    },
    "multi": {},
    "transformed": {}
  }
}

🙂 Expected behavior
Would expect context before and after strings to be in the streamed json
Neovim Plugin in action if you are curious
This functionality has not been merged yet at the time of posting this issue.

Hi @MagicDuck, thanks for building grug-far.nvim with ast-grep.

does not contain context strings

I can put context string in the lines field. Does that meet your requirements?
I noticed you need to highlight the match string. One way to do that is:

read lines and split it into string array
find the matching lines by removing context
highlight the match by calculate the column

btw, I would be more than happy if you would add grug-far.nvim in this page? https://ast-grep.github.io/guide/tools/editors.html#neovim
Source is https://github.com/ast-grep/ast-grep.github.io/edit/main/website/guide/tools/editors.md here

@HerringtonDarkholme thanks for the quick response! 😄
I would say ideally the context string would go into 2 new json fields, something like context_lines_after and context_lines_before.
The problem with putting it in lines is that I would have to parse the context arguments (-A, -B, C and their other variants) specified by the user in the Flags input in order to figure out those numbers, so I know how much to take off of lines.
Alternatively, the number of context lines can be provided as part of json, I guess. That would work too...

btw, I would be more than happy if you would add grug-far.nvim in this page? https://ast-grep.github.io/guide/tools/editors.html#neovim
will do in a bit when I merge this functionality!

btw, don't mean to sound needy, but a --glob=<pat> flag same as ripgrep would also help in the future 😄
Atm I am running rg --files --glob=<pat> to get a filtered down list of files which I then to pass on to sg in chunks.
It's definitely not the biggest issue since I have the workaround, but would be good to remove the need for rg there at some point.
see: https://github.com/ast-grep/ast-grep/issues/1062#issuecomment-2266388041
Thank you for you support! ❤

Created PR for adding it to integrations page 😄

@MagicDuck implementation wise, ast-grep's DisplayContext has only leading/trailing, leading means the line content before the match and trailing means the content after match.
leading + match + trailing
e.g. suppose the match is `log`
console.log(abc)

leading: console.
match: log
trailing: (abc)

I'm shipping an alternative way to make it easier for plugin authors to separate lines.

That works for me! Thank you so much! 😄

Was a bit tricky to implement cause off by one errors in lua are unaviodable when it indexes everything starting with 1, but it worked 🥳
Thank you for your support!


[FR] Quote new line separation in editing vs reading modes

Is your feature request related to a problem? Please describe.
I am not sure if this is a design decision or not; when in reading mode vs editing mode, multi-line quotes are separated by a border in editing mode but I would expect them to display like they do in reading mode with no separation of the quote.
Raw markdown:
> This is a quote.  This could be a much longer sentence if I really wanted it to be so here is some more info that I could possible put in this.  I hope this is long enough because I am tired of typing.
> New line of a quote

> New quote, separated by blank line

Editing mode:

Reading mode:

Describe the solution you'd like
Editing mode would look like reading mode.
Describe alternatives you've considered
I was looking at trying to change this using a CSS snippet but I was having issues trying to do so.
Additional context
Screen shots above.

Hi @mbentley,
I'm aware of this and will likely look for an alternative design that I can keep consistent across modes. At the moment there is no way for me to target the start and end of a blockquote individually in editing modes. So the start, in-between and the end all have to share the same code. which results in the unwanted top and bottom borders when using several lines. Will likely resort to possibly adding a subtle background colour and only doing the side borders. Will have to think about this one 😅

P.S. I've had a similar issue with Inline code but I've managed to get some help with resolving that and it will be in the next update. In that case there was a way to differentiate the parts that make it up so the bordered design can stay.
How that looks currently:

Solution 1:

Solution 2:

Hi @mbentley,
This is now addressed in Prism 3.2.1. I've basically reverted the design to what the default theme uses with an addition of a different background colour to help differentiate it from other content. I've had too many difficulties doing anything else while making it consistent across all editor modes.


Update audience-demo

Update deps and improve readme for  audience-demo.

When running tests locally, I'm having issues with file watchers. Seems like tests shouldn't need file watchers, and non-test runs (ex: npm start) work fine. I can't see a way to disable file watchers for tests though, so I'll see if CI can run them.


Check EAD item size when encoding

Check whether the EAD item actually fits alongside the message in the buffer.

Solved in #168


[SEDONA-676] Add ST_Perimeter

Did you read the Contributor Guide?

Yes, I have read the Contributor Rules and Contributor Development Guide

Is this PR related to a JIRA ticket?

Yes, the URL of the associated JIRA ticket is https://issues.apache.org/jira/browse/SEDONA-676. The PR name follows the format [SEDONA-XXX] my subject.

What changes were proposed in this PR?

Add ST_Perimeter

How was this patch tested?

Add unit tests

Did this PR include necessary documentation updates?

Yes, I have updated the documentation.

@furqaankhan please fix the snowflake failure

@jiayuasu Snowflake test passed


[stable/datadog] Add IPC_LOCK to system-probe container capabilities

Is this a new chart
No.
What this PR does / why we need it:
Our internal testing shows that we need this additional capability to make system-probe container to run.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[ ] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test

/lgtm

/ok-to-test

/approved

/lgtm


Render data from addresses

Note: This will require actually following the README instructions to have the API running locally (Running and Importing Data specifically).
This renames "intendedUse" to "landUse" and also "results" to "summary".  It also updates the actions to read from the API and render a very simple version of the results.

Here's a screenshot:


Adding a link to the away message

I noticed that on papercup's landing page, there is a link in the away message.

I tried doing that on my own away message, just using html, and it did not come through. Is there a way to add a link to the away message?

hey @johncadengo! you can do this with markdown, e.g. [demo](https://app.papercups.io/demo) (https://github.com/papercups-io/papercups.io/blob/5c8494f4a6edae5b61165462120c6a1d764447ca/pages/index.tsx#L488)

Oh, thank you! That wasn't clear to me earlier, but great solution. 👍


python312Packages.bc-python-hcl2: refactor

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

6 packages built:

checkov
checkov.dist
python311Packages.bc-python-hcl2
python311Packages.bc-python-hcl2.dist
python312Packages.bc-python-hcl2
python312Packages.bc-python-hcl2.dist


add to_imperative method on each collection

Merging soon if no comments.

Looks good to me. Array of Value's is a good idea. +1.


[A11y Compliance] SfAlert - Alert background and font color don't have enough contrastivity

SfAlert - alert background and the text color don't have enough contrastivity.

New colors are ready to be applied now :)

@filrak Still not good enough. I just tested the contrast ratio of blue and red, one is 1.28 and one is 2.18 respectively (too low compared to 4.5 as max)

Ok so we're waiting for another Svieta's try :)

Now it's ready ^^


[GSSoC'23] Guess Number Game Ready

Issue Number - 693
Title - Guess The Number
Closes #693

@Shreyaar12 @admindebu @panwar8279 please review this pr. I have re-created the pr


n dimensions

Hi
Is it possible to works with more than 3 dimension?

It is, but not with the functions provided here. You'll need to do more maths.


Remove HDCP from sepolicy

HDCP is depent on ia-hwc. since ia-hwc is not used. Has to remove HDCP to avoid compiling error
Tracked-On: OAM-105293
Signed-off-by: Shaofeng Tang<EMAIL_ADDRESS>

It is verified in engineering build1153
Please help to review @YuanjunHuang @feijiang1 @renchenglei

Hi @ceiba1985
Please help to review


Listing of Return Types

Just wondered where is a list of ALL ActionResult Return Types? Just think it might be an idea as the title of this doc makes it sound like you list them all here but instead presented with many words and confusing examples, just a simple one line example showing usage and a list of them would be great

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

ID: fa03501b-5e71-4c18-0269-39babbf9d93c
Version Independent ID: b51bc104-a6e2-b21f-8362-45cee22e8c22
Content: Controller action return types in ASP.NET Core Web API
Content Source: aspnetcore/web-api/action-return-types.md
Product: aspnet-core
Technology: aspnetcore-webapi
GitHub Login: @scottaddie
Microsoft Alias: scaddie

Hit More ... under Derived here :point_right: https://docs.microsoft.com/dotnet/api/microsoft.aspnetcore.mvc.iactionresult
Since they're subject to change, I wonder if the link to the API Browser is enough? ... but perhaps with an explicit instruction on how to get to them there.

Similarly having a clear list of all the controller helper methods like OK, BadRequest, NotFound...etc would be very helpful as this seems particularly hard to find...


Add crypto/des section to FIPS user guide

Noticed that our 1.22 release notes point at this doc for this API, but it wasn't documented yet.
Didn't add Implementation sections in the initial PR. I know where to dig around and find the info, but I think @qmuntal could do it a lot faster with familiarity. 😄

I think @qmuntal could do it a lot faster with familiarity. 😄

Done!


Added MUI alerts to replace browser alerts ⚠

Use MUI Alerts 📐

I Pritam Kundu have worked for #51

Changes Made

Resolved merge conflicts for #55

Used Material UI to add Snackbar Component

Added Snackbar to repalce browser alerts

Login In
Sign Up
Image Upload Error
Sign In Error
Firebase Error

Replaced all alerts and error messages and with Snackbar Notifications

Screenshots:

Logging In

Logging Out

Error Messages

Creating Post

Thanks Pritam, reviewing it right away

there's already a new post button , we don't need a double

Should I remove the top purple New Post Button

Alright I identified the glitch. I accidentally deleted this chunk of code while merging. This contained the dialogBox.

I am restoring the above chunk. The below is the result 👇. I didn't notice any other glitch. If you find any, please inform.

Bro , with these changes , even after uploading is complete modal stays open on screen, state change issue

okay I will start fresh


[Design] Apply discussed design changes for consistency with other languages

Summary
Inconsistencies with other languages were found, and renames must be done to address them. Some items still need discussion.
Updates

[ ] In FormRecognizerClient.RecognizeCustomForms, parameters formFileUri and formFileStream should be renamed to formUri and form, respectively.
[ ] In TrainingDocumentInfo, rename DocumentName to Name.
[ ] In TrainingFileFilter, rename IncludeSubFolders to IncludeSubfolders.

Open discussions

[ ] In CopyAuthorization, members AccessToken, ResourceId and ResourceRegion are internal, but in Java they're public. Find out what's the desired approach here.
[ ] In CustomFormModelStatus, in other languages the member Training is called Creating. Find out why this decision was made and rename it, if necessary.
[x] In FieldValue, consider renaming AsInt64 to AsLong to match Java's API. Edit: we'll keep it as AsInt64 since it follows pre-existing .NET BCL nomenclature.
[ ] In FieldValueType, consider renaming Integer to Long to match Java's API.
[x] Service-returned floating point numbers have 64-bit precision in Java, while we have only 32-bit precision in .NET. Find out why there's a difference and whether we need to change this or not. Edit: service team confirmed that 32-bit is enough. No changes needed.
[x] In FormRecognizerError, find out why we decided to call the error code ErrorCode, as other languages name it Code. We're likely to keep it as it is. Edit: we renamed it to ErrorCode for consistency with the Text Analytics library in .NET. We'll keep it like this in Form Recognizer.

FormRecognizerError, find out why we decided to call the error code ErrorCode

The reason why TA went with ErrorCode is also because of RequestFailedException.


Bot fails to generate DB info when added to server offline

If the bot is invited to a server while offline, it does not generate the DB information required for that server. Any commands executed once the bot is online will generate a large number of errors. In order to get the bot working on that server, kicking and re-inviting the bot while it's online is required.
Generating server info is currently dependent on the server joined event. I think it would be a good idea to make the bot more robust by having any commands that require database access to first check if the database info has been generated (and generate it if nonexistent).

https://github.com/squirrelo/MovieNightBot/commit/1dbbda70ac4b423b112bfea5454617e11f7acc05


Add method for subsetting

This PR sets up the column api's subsetting methods.
The PR does the following:

Exports the tech.v3.dataset.column/select method to the column API;
Adds a slice method to accompany select;
Updates the column_exploration doc with some stuff about these methods.

@genmeblog I'm merging this so that I can keep going, but you can still comment on the changeset here, and I can circle back if necessary.


New package: AMDGPU v0.1.0

Registering package: AMDGPU
Repository: https://github.com/JuliaGPU/AMDGPU.jl
Created by: @jpsamaroo
Version: v0.1.0
Commit: 900f4fd21010fddf34fc20d8afe754e3e6d2f49d
Reviewed by: @jpsamaroo
Reference: https://github.com/JuliaGPU/AMDGPU.jl/commit/900f4fd21010fddf34fc20d8afe754e3e6d2f49d#commitcomment-40404564

[noblock]
Name is similar in spirit to CUDA.jl, and the purpose should be easily identifiable by AMD GPU users.


Accessing repo with PyGithub

How too access this repo through the API?

This seems to have done the trick:
>>> from github import Github
>>> g = Github()
>>> repo = g.get_repo("ukBaz/webhook-test")
>>> repo.get_issue(number=2)
Issue(title="Accessing repo with PyGithub", number=2)


WebAssembly changes, Fx 79

Story
As a web developer, I want to know about the latest WASM enhancements, so that I can run code written in multiple languages on the web.
Acceptance criteria
The following bugs are dev-doc-complete, BCD is updated, an entry to Firefox release notes is added, experimental features page is updated, the content has been reviewed as needed.

[x] Bulk-memory-operations: Ship it

[x] Reference types: Ship it

[x] WebAssembly threads, see also https://www.chromestatus.com/features/5724132452859904 for an explanation and see https://bugzilla.mozilla.org/show_bug.cgi?id=1648685 that enabled this in Fx79

@chrisdavidmills Same story as last time: I reckon this needs some mentioning in the Wasm docs, but not extensive docs as we don't provide such detailed level of WebAssembly's text format on MDN. However, it should be mentioned in release notes and hacks post.
Please estimate and let me know if you could take this.

About a point I reckon.

Hrm, I've added rel notes for these: https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/79#WebAssembly, but I'm not really sure what to write about them , so I've asked Luke for some help.

Bulk memory operations:
Small description added to https://developer.mozilla.org/en-US/docs/WebAssembly/Understanding_the_text_format#Bulk_memory_operations

Reference types:

Rel note added: https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/79#WebAssembly
Brief description added to https://wiki.developer.mozilla.org/en-US/docs/WebAssembly/Understanding_the_text_format#Reference_types

Threads:

Rel note added: https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/79#WebAssembly
Brief description added to https://developer.mozilla.org/en-US/docs/WebAssembly/Understanding_the_text_format#WebAssembly_threads

Well done! :100:


fix(event_list): 11841 - fix autoselect fired when event is already selected

If url was applied with event-id not presented in event list but availible through api call - warning title is presented for 5 seconds and autoselect is not fired

I also added error message for 404 event requests as there was no other way for user to know that event cannot be reached

i've also changed "event not found in the list" toast color from error to warning state as it's not stopping user from using app properly


Validation issues

There seems to be an issue with the css for .invalid-feedback. Based on my testing, .invalid-feedback classes seem to appear under inputs that are valid when .was-validated is set on the form. See the jsfiddle here: https://jsfiddle.net/5o1y05vq/2/
What (I think) should happen is that .invalid-feedback should only display: block when the input element is invalid. Otherwise you get the following behavior:

Duplicate of #23372 and fixed in v4-dev.


Fix pylint directives

Short description

This PR relocates the pylint directives so they are only applied for the target part.
Proposed changes

Move # pylint: disable=...... from the top of the function to each target part.

Side effects

should be none
There may be lines which I overlooked. Feel free to directly commit onto this branch 😸

Resolved issues

Fixes: #3140

Pull Request Review Guidelines

@JoeyStk

Most disables are for unsued-argument. AFAIK there is no way to circumvent this. So right now I'm wondering if this rule really helps us enough to not disable it globally. But maybe it does...

I'm actually for disabling unused-argument globally. Let's do a poll in the team channel 👀

I think some of the disables should be refactored and then removed. How do we want to go about this? Create issues for it or just hope that in the future every time we see a disable we remember to think about whether or not we can refactor? :)

Yes, I saw chances of refatoring but wanted to go less-invasive in thie PR, as the goal of issue is putting disables into better positions and refactoring is always prone to bugs (and therefore reviewed indivisually focused). I'll create a generall issue for this thema.

I'd like to suggest not to put any more effort on refactoring/improvement as we are going to remove pylint (#3234 ) but merge this PR as there will be still time until ruff starts to patrol our repository.


proposal(core): support onTrigger

This is just a proposal. I just wrote a little to show my intention.
In my plan, it is used to customize the hit or match of troubleshooting attributes.In Env mode, let users know how the set matching rules work in the generator
Do you think it is necessary to implement this function? If not, I'll give it up

Thanks for working on that. I got your point and I think I have a better solution for this, keep turned :)


Creature Plugin Export Failure

Godot version:
3.2.1
OS/device including version:
Windows 10
Issue description:
Hello, when trying to export out a scene with the Creature plugin ( https://github.com/kestrelm/Creature_Godot/tree/master/creaturegodot3 ), we get an error saying "Unknown class: 'CreatureGodot'"
This plugin and code works as per normal in the editor, but fails on export. What should we do to resolve this issue?
Steps to reproduce:
Get the creature plugin and load one of the scenes here: https://github.com/kestrelm/Creature_Godot/tree/master/creaturegodot3/Godot3Demo
Minimal reproduction project:

Since this is a C++ module, you need to recompile all the export templates you plan to use as well.

Hello!
I'm using Godot 3.2.1 Stable.
This is the error I got when exporting to HTML5:
**SCRIPT ERROR**: Parse Error: Unknown class: "CreatureGodot" index.js:7:35827 At: res://morphTest.gdc:1:GDScript::load_byte_code() - Parse Error: Unknown class: "CreatureGodot" index.js:7:35827 **ERROR**: Method failed. Returning: ERR_PARSE_ERROR index.js:7:35827 At: modules/gdscript/gdscript.cpp:778:load_byte_code() - Method failed. Returning: ERR_PARSE_ERROR index.js:7:35827 **ERROR**: Cannot load byte code from file 'res://morphTest.gdc'. index.js:7:35827 At: modules/gdscript/gdscript.cpp:2240:load() - Condition "err != OK" is true. Returned: RES() index.js:7:35827 **ERROR**: Failed loading resource: res://morphTest.gdc. index.js:7:35827 At: core/io/resource_loader.cpp:278:_load() - Condition "found" is true. Returned: RES() index.js:7:35827 **SCRIPT ERROR**: Parse Error: Unknown class: "CreatureGodot" index.js:7:35827 At: res://morphTest.gdc:1:GDScript::load_byte_code() - Parse Error: Unknown class: "CreatureGodot" index.js:7:35827 **ERROR**: Method failed. Returning: ERR_PARSE_ERROR index.js:7:35827 At: modules/gdscript/gdscript.cpp:778:load_byte_code() - Method failed. Returning: ERR_PARSE_ERROR index.js:7:35827 **ERROR**: Cannot load byte code from file 'res://morphTest.gdc'. index.js:7:35827
This is the error I got when exporting to desktop .exe:
http://prntscr.com/sa300u
Thanks for the help :+1:

@MelimeliDev Same question as @Calinou asked: did you compile 3.2.1-stable export templates with the Creature plugin? If not, there's no Creature code in the Javascript template and then it won't work indeed.

@akien-mga Thanks for the tips! @MelimeliDev Can you please try what they suggested. Thanks.

User compiled the templates and it worked. Thank you.


Calling SyncUser.current  with multiple users exception vs. checking logged in status

Goals
Want to check to see if there is currently a user logged in but there are already several users logged in
Our documentation asserts that checking for a logged in user would be done by checking if SynUser.current != nil   in the case of multiple active profiles this causes an exception.
Expected Results
SyncUser.current returns non- Nil for any condition where the are logged in users/profiles
Actual Results
Crash:
2017-05-22 14:35:33.557 RealmBingo[33458:4855586] *** Terminating app due to uncaught exception 'RLMException', reason: '+currentUser cannot be called if more that one valid, logged-in user exists.'

Steps to Reproduce

Have multiple users logged in
Call SyncUser.current

Code Sample
N/a
Version of Realm and Tooling
Realm framework version: RealmSwift 2.7.0
Realm Object Server version: 1.4.1
Xcode version: 8.2.3
iOS/OSX version: 10.12.5
Dependency manager + version: ?

This is documented behavior:

The currentUser method will retrieve the current user, the last user who logged in and whose credentials have not expired. The method returns nil if no current user exists. An exception will be thrown if more than one logged-in user exists.

I feel this is correct as it might be misleading/undefined if there are more than one logged in users. You should use allUsers if your application supports such a scenario.

Hi, you can check your active users session this way (Swift code):
for user in SyncUser.all {
    debugPrint("user: \(user.key) - \(user.value)")
    user.value.logOut()
}

You will se something like this:
"user: a9c2de623e408422a3073b3276d54517 - <RLMSyncUser: 0x174247b90>"
"user: 0a12e0b10e217f228d4f313679f68c4d - <RLMSyncUser: 0x174246300>"

To call SyncUser.current without error you have to quit all other users session. For example:
for user in SyncUser.all {
    debugPrint("user: \(user.key) - \(user.value)")
    user.value.logOut()
}


implement -f flag

Implements -f flag. didn't do longopt because there are two (--file= and --makefile=)and i don't know how clap handles that.

Two things:

-f can actually be used multiple times (for example make -f file1 -f file2) to load rules from two Makefiles. I'm okay if we don't want to implement that right now, just something to keep in mind.
The return was outside of the if because not loading a Makefile is not an error by itself.

Thank you for contributing!

ah okay

I think it's okay to not support multiple files for now as we don't even have one file working


Automator: update<EMAIL_ADDRESS>dependency in<EMAIL_ADDRESS>

Generated by Automator - 2023-07-13T06:19:40+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.18
116836622a98f48f4fb149e277650e2af164959e
link
unknown
/test unit-tests-arm64

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

/retest


remove one read from aar_columns

looks like this code set aar_columns to an empty array, then just passed
the empty array along.  Lets just pass an empty array along
/cc @Fryguy @gmcculloug

Nothing but goodness here . . . thanks!


dev-cmd/bump-formula-pr: replace partial version in URL

For now, restrict to partial version excluding the last part, i.e. version with major.minor.patch will result in attempting to replace any occurrences of "/#{major}.#{minor}/" or "/v#{major}.#{minor}/".
This should take care of common situations like GNOME, KDE, and SourceForge URLs. It should also help with URLs like util-linux.
It may result in incorrect replacements if URL has partial version based on another versioning scheme like the API version or if there are some equivalent numbers in URL that happen to match

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Could go stricter or looser in restrictions. Not sure if any incorrect replacements yet.
This won't handle formulae like:

mysql - https://cdn.mysql.com/Downloads/MySQL-8.3/mysql-boost-8.3.0.tar.gz
rtf2latex2e - https://downloads.sourceforge.net/project/rtf2latex2e/rtf2latex2e-unix/2-2/rtf2latex2e-2-2-3.tar.gz

If we do hit incorrect replacements, we could test out multiple options for url. In case of mirror, we would still need to guess as these don't always exist at time of bump.

cc @Bo98 as related to https://github.com/Homebrew/homebrew-core/pull/166305#issuecomment-2001647217


MIT licence file added.

Hello Andreas,
I would like to add the MIT license file. We want to use the library in a project for a big company, where it is neccessary that the license is self contained.
Thank you!

Hello Andreas,
many thanks!


hcsr04: fix a bug, closes issue #207

@Propanu,
it's here!

Thanks a lot for fixing this!!! Cleaned & merged in 7963e6112027bfa9cb4dd89ab9a2c9041551ea38.


Networked scene object never enabled on client when testing between unity and build

Describe the bug
GameObjects in a scene with attached network identities will never be enabled for the client if one instance of the game is started from the unity editor.
This is true for both cases:

Host - started via Unity editor | Client started from build .exe
Host - started from build .exe | Client started via Unity editor

Judging by the logging, it seems that ClientScene.OnSpawnSceneObject is getting a NetworkMessage with a wrong sceneID and is therefore unable to enable it.
The correct scene ID would be 1 but the Message contains the sceneID<PHONE_NUMBER> (this number is constant between multiple tests).

Could not find scene object with sceneid:1431655766

Spawn scene object not found for<PHONE_NUMBER> SpawnableObjects.Count=1

Spawnable: SceneId=1 name=FakePlayer

Repro project
It is possible to reproduce this behaviour with the included basic example.
To Reproduce
Steps to reproduce the behavior:

Open the Mirror basic example in Unity
Add a instance of the Player Prefab to the scene
Build the project
Start the build. exe as a LAN Host
Play the project inside the unity editor (on the same machine)
Connect as LAN Client to localhost

Expected behavior
The added player prefab instance is enabled after the client-connect and the additional textbox is visible on both instances (Host & Client).
Screenshots

Diagonal text belongs to the added "FakePlayer" GameObject which is part of the scene and has a NetworkIdenty-component attached.
It is only visible for the host, and disabled for the client.
Desktop:

OS: Windows 10 x64 (1809)
Build target: Windows x86_64
Unity version: 2018.2.20f1
Mirror branch:ver 1.4 from the Asset Store

Additional context
While debugging the issue (host and client in seperate runs, while the debugged instance was always started from the unity editor) i noticed that the host seems to send a messge with the correct sceneID but the client always seems to receive<PHONE_NUMBER>. All other values seem to be correct.
Note: I did not try to debug two instances that were started from inside unity on two sepperate machines.
This issue does not occur if host & client are both started from the build.exe

on first glance it doesn't look like a real sceneid. something was messed up somewhere before that.
can you try to remove your project's library/temp/obj folders and reimport again?
maybe it still uses an old weaved DLL somewhere

The sceneid is maxint / scene count. Do the numbers of scenes in the editor differ from the build? I had that issue recently.

The sceneid should mean you have 3 scenes. It's Max uint. Not int. Sorry.

Deleting library,temp,obj and a Reimprt  did nothing.
The scene count on the other had seems to be the problem.
I only had the basic example scene loaded in the editor while the build settings contained the other two Mirror example scenes in addtion (even though they were disabled).
If i remove all scenes but the one that is loaded in the editor it starts to work.
I tried to get it to run for the whole day yesterday... bit it never occured to me that this could be the problem.
Is this an expected behaviour ? I did not have this issue while using UNet.

It was changed about a month ago to fix an issue with addative scene loading. I'll look for the commit. May need reviewed for all use cases. Dunno

The way it behaves in this particular case feels a bit odd at least.
But still a big thank you from me!
At least now i know what i have to do to be able to test & debug in the unity editor.

I agree. When I run in editor I have to remember to manage the scene count. Overall the scenes and spawning via sceneid and netid has been more trouble than anything else.

Same issue occured on me

this is fixed now. thanks for the detailed bug report.
please let us know if there are any more issues.


Refactor security.

In gitlab by @A-Lawrence on Feb 26, 2016, 23:26
For this to work:

Add fields to mship_account [password, password_set_at, password_expire_at]
Active secondary passwords need to be migrated over to the mship_account
Drop mship_security
Drop mship_account_security
Remove security relationship
Create a password_lifetime value on roles (default=0)

In gitlab by @A-Lawrence on Feb 27, 2016, 18:27
Title changed from Remove historic secondary passwords. to Refactor security.


New label appears at position of old label

Click the Destroy button to remove the first label
Click the Label 2 button to add a new label
Zoom in towards the middle of the globe.  Eventually the new label will appear around the spot the old label was, even though the new label has no position

This is specific to labels, it does not happen with billboards.  It also only happens when using the CLAMP_TO_GROUND height reference
var viewer = new Cesium.Viewer('cesiumContainer');
var scene = viewer.scene;

var labelCollection = scene.primitives.add(new Cesium.LabelCollection({
    scene: scene
}));

var label = labelCollection.add({
    heightReference: Cesium.HeightReference.CLAMP_TO_GROUND,
    text: 'label 1',
    position:  new Cesium.Cartesian3(-748060.358557192, -4909764.442609902, 3988488.2144659692)
});

Sandcastle.addToolbarButton('Destroy', function() {
    labelCollection.remove(label);
});

Sandcastle.addToolbarButton('Label 2', function() {
    labelCollection.add({
        heightReference: Cesium.HeightReference.CLAMP_TO_GROUND,
        text: 'label 2'
    });
});

This sounds similar to a billboard bug I fixed in #5339.  My guess is that label caching isn't clearing the clamped state.


[scr annotations] Merge trunk, update baseline check

I just merged the latest trunk, which incidentally means that we updated oak-store-document as well :-)
mvn clean install -DskipTests works, I expect tests will fail for now as they failed before the merge.

Thanks Robert. I merged your PR, didn't check it in detail though.


Fix attached database instances hydration

Fixes the retrieval of the attached database instances

Looks great @fabriziosestito To test is in a real environment, simply check in the SAP systems list, if any of the systems has 2 HANA entries

@arbulu89 I tested against a copy of the demo db and it works fine!


fix indent

Proposed changes:

a tiny little indent mistake broke the story reading when using an exclusion percentage

Status (please check what you already did):

[x] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog

please cherry pick onto 0.12.x as well


Prevent early publication of unofficial results

This was discussed in an inference meeting, and appeared to be broadly considered legal. However, it doesn't seem healthy for MLPerf: it allows for gaming of publication (e.g. a submitter, knowing the official numbers, could publish day-zero updated unofficial numbers), and erases some of the the advantage that official submitters should reasonably have over unofficial ones.

@petermattson @TheKanter  please review. @christ1ne for inclusion in the next WG agenda.

Not commenting on the substance:

This is something that would impact many groups, e.g., training, mobile, etc. beyond mobile. So we should probably figure out a good way to handle it.
Other companies have inquired because they would like to use unofficial results for marketing during the review period.

Commenting on the substance:
2. Is this enforceable? E.g., if a test is unchanged, a non-member could simply claim an unofficial result. I could see it being enforceable/honored by members.
3. I agree there is a 'race to disclose unofficial results' that seems contrary to the objective of review

With regard to enforcement, my understanding is that there is an option for a trademark-based nastygram to non-members.

added to the agenda. We can form an inference WG view and share with other WGs.

Most benchmark WGs OK with this.
Some folks suggest limiting to submitters only for enforceability reasons.

Why do those folks feel it's less enforceable against non-submitters than any other abuse of the MLPerf trademark?

How about this:

For two weeks after the publication of official results of an MLPerf submission round, no unofficial results can be presented in comparison to the official results.

Current text:
Unofficial results for a benchmark in a given version of an MLPerf suite may not be publicly disclosed prior to the publication of the official results for that version.
Updated text:
Submitters are not allowed to publish any results for a given version before its official publication date. Non-submitters are not allowed to publish any unofficial results until two weeks after the official publication date for that version.

@christ1ne text changed as requested - please merge.


winpython.winpython <IP_ADDRESS>

Microsoft Reviewers: Open in CodeFlow

This app must be installed with admin privilege

https://github.com/microsoft/winget-cli/issues/970

https://github.com/microsoft/winget-pkgs/pull/13076#issuecomment-927016286


Fix bulk select func

Description
Fix bulk select function that had the old selector
Fixes # (issue)
Type of change
What is it?

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I run npm run lint:js:fix to check that my code is properly formatted

Codecov Report

Merging #381 (882dbfc) into master (7d3a246) will not change coverage.
The diff coverage is 0.00%.

:exclamation: Current head 882dbfc differs from pull request most recent head 8ae1a9a. Consider uploading reports for the commit 8ae1a9a to get more accurate results

@@           Coverage Diff           @@
##           master     #381   +/-   ##
=======================================
  Coverage   34.30%   34.30%
=======================================
  Files         118      118
  Lines        2586     2586
  Branches      749      749
=======================================
  Hits          887      887
  Misses       1557     1557
  Partials      142      142

Impacted Files
Coverage Δ

src/components/general-table/GeneralTable.js
65.98% <0.00%> (ø)

Continue to review full report at Codecov.

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


Redis should support password authentication

Support deploying Redis secured via a password

Deployment of Redis secured via a password
Configure Quay to communicate with Redis using a password

added support as per #81