Requires editor config to display List View properties

First, I want to say this is an amazing project - jaw droppingly good! The number of features and quality of the code and documentation is stunning. I managed to get a simple List View working in about 20 mins from install to running in Umbraco, including reading docs. So that is a testament to how easy it is to use.
However, I was left scratching my head as to why some column data wasn't being displayed. For the project I was working on I only required a read-only list view of some data stored in a table within the Umbraco DB (T-SQL).
My POCO was this:
[TableName("GradedUrls")]
public class GradedUrl
{
    [PrimaryKeyColumn]
    public int Id { get; set; }

    public DateTime Date { get; set; }

    public string Url { get; set; }

    public string Domain { get; set; }

    public decimal TotalScorePercent { get; set; }

    public string Email { get; set; }
}

and the relevant bit of config I used was this:
treeConfig.AddCollection<GradedUrl>(p => p.Id, "Graded Url", "Graded URLs", "List of graded URLs", "icon-link", "icon-link", collectionConfig =>
{
    collectionConfig.SetNameProperty(p => p.Domain);
    collectionConfig.SetViewMode(FluidityViewMode.List);
    collectionConfig.SetDateCreatedProperty(p => p.Date);
    collectionConfig.SetSortProperty(p => p.Id, SortDirection.Descending);
    collectionConfig.MakeReadOnly();

    collectionConfig.ListView(listViewConfig =>
    {
        listViewConfig.AddField(p => p.Date, fc => fc.SetHeading("Date Submitted"));
        listViewConfig.AddField(p => p.Url);
        listViewConfig.AddField(p => p.TotalScorePercent, fc => fc.SetHeading("Score %"));
        listViewConfig.AddField(p => p.Email);
    });

});

When I ran this I got a list view, but only the Domain value (the name property) was showing any data. All the other column headings were there, but each row contained no data for any of the properties in the List View. They were completely empty (yet I knew the table contained data and the field names matched the POCO).
Eventually, I tried adding an Editor Config to the collectionConfig (to match your example in docs):
collectionConfig.Editor(editorConfig => {

    editorConfig.AddTab("General", tabConfig => {
        tabConfig.AddField(p => p.Url).MakeRequired();
    });
});

As soon as I did this the values for the columns were displayed correctly. I'm not quite clear why the editorConfig is needed for this. As I say, in my case I was using a read-only view that isn't editable, so I wouldn't have expected to require an editor config to see the values of the properties in the list view.

Interesting! It shouldn't need an editor config in order to show list view fields. Is it possible you setup the listview config but forgot to compile? then when you added the editor config you did compile so it then worked?

Hah, that's just the kind of thing I would do, but in this case, "no". I can easily reproduce by removing the editorConfig, compiling, and then I get:

As soon as I add it back, compile, I get data in all those blank columns.

Ok, I think I've found the problem. In the mapping code for the entity, for some reason I'm checking if the editor has tabs, which is completely irelivant for list views. I think it's probably left over from some copy paste thing https://github.com/umco/umbraco-fluidity/blob/master/src/Fluidity/Web/Models/Mappers/FluidityEntityMapper.cs#L72
I'll get it removed and get a nightly built

Thanks for looking so quickly. But there's  no rush, honestly, it's easily worked around for me. I've just made the editor view read-only, too, which works perfectly (for my needs). Loving the possibilities for this! :)

Hey @DanDiplo a new nightly is here https://ci.appveyor.com/project/UMCO/umbraco-fluidity/build/artifacts will get pushed out in the next release

Thanks, much appreciated - really enjoyed working with this!


tensorflow-py3 docker image doesn’t work because it contains python2 code

I tried to create a dataset on DIGITS using the nvcr.io/nvidia/digits:20.01-tensorflow-py3 docker image following the instructions from https://github.com/dusty-nv/jetson-inference/blob/master/docs/segnet-dataset.md
It seems like the /opt/digits/digits/tools/create_generic_db.py still try to run python2 code

2020-02-13 21:06:02.096817: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.2
Caffe support disabled.
Reason: A valid Caffe installation was not found on your system.
2020-02-13 21:06:03 [ERROR] TypeError: 'range' object does not support item assignment
Traceback (most recent call last):
File "/opt/digits/digits/tools/create_generic_db.py", line 480, in
args['stage']
File "/opt/digits/digits/tools/create_generic_db.py", line 446, in create_generic_db
force_same_shape)
File "/opt/digits/digits/tools/create_generic_db.py", line 298, in create_db
entry_ids = extension.itemize_entries(stage)
File "/opt/digits/digits/extensions/data/imageSegmentation/data.py", line 182, in itemize_entries
feature_image_list = self.split_image_list(feature_image_list, stage)
File "/opt/digits/digits/extensions/data/imageSegmentation/data.py", line 224, in split_image_list
random.shuffle(self.random_indices)
File "/usr/lib/python3.6/random.py", line 277, in shuffle
x[i], x[j] = x[j], x[i]
TypeError: 'range' object does not support item assignment

I found a similar issue https://github.com/NVIDIA/DIGITS/issues/2208#issue-557221271


React to aspnet/Razor#164: Camel-case tag helper directives

/cc @NTaylorMullen

Note: You'll need to update the perf tests to reflect this as well.
:shipit:

Gogogogo

commit 451db6fb1621c0b67f1948085dd0bafe8092badf


Supporting "Video Player" plugin for Windows desktop.

Hello, do you plan to support desktop applications on Windows? I need a multi-platform video player including web.

Hi @AndriiReverchuk ! Desktop application support on Windows is already available. And the new 2.10 Version added more stability.
Please check Flutter Destop Support and The Announcing Flutter for Windows

I am asking about  support "Video Player" plugin for Windows desktop.

Duplicate of https://github.com/flutter/flutter/issues/37673


vagrant: add little helper script to step through ansible

optionally provide a task to start at
Signed-off-by: Michael Adam<EMAIL_ADDRESS>

This change is

LGTM


[ENH] all dashes in tags should be replaced by underscores

In tags, all dashes should be replaced by underscores.
The reason is that in a vanilla call to set_tags, one would write set_tags(tag_name=tag_value), but this will break if tag_name has a symbol in it that is not permitted in a python variable name (for instance, a dash).
For each tag, the changes need to be done:

in registry/_tags
all estimators where it is set
any other logic that accesses the tag, including: concrete estimator inits (dynamic tags), base class plumbing, and the tests.

vscode "find in files" allows to highlight all occurrences.
If someone would like to work on this, please make yourself known here and pick one tag first, to keep changes small and avoid duplication.

sir can i do my first contribution here ..can you please give me chance
(i am new in open source i need all of your support so i can motivate my self)

Hi @PuneetDabral, yes, no-one else is working on this as far as I can see so please give it a go! Let us know if you get stuck.

Hello, i am new to open source contribution. I want to contribute. Please guide me .....

go to registry/_tags and pick one with a dash, then say here which one you picked; then make a PR in which these are renamed.

ok i will clone this


[Bug] Github Copilot stuck in the same window

Summary
Github Copilot not allowing me to signin in the new github codespaces
How to reproduce

Go to 'github codespace'
Click on 'copilot extension icon'
Click on "Sign in"
Stuck in the same window

👋 !  Just to clarify, which step in the course are you stuck?  I did a quick search and didn't see where the course mentions signing-in.

It shows me once I access the codespaces

👋 @mdabir1203 I'll close this issue out as I don't believe it's an issue with the course. If you're having trouble with Codespaces or Copilot in general, please reach out to our support team.


Update home.md

Fixed the links . They now should redirect users to the right place

Hi,
We are not currently deploying the integrations page on our docs: https://docs.lavague.ai/en/latest/docs/get-started/customization/
I can delete the .md to avoid confusion, but the redirects link to an old version of the docs which is not up to date.


New implementation

Hi, I would like to thank you for such a great piece of code.
I am using your code in one of my project. However, there is a slight change. Instead of showing white colored text, I want to show clear (non-blurred) background as my text color. I mean wherever the text comes, the background should not be blurred.

Well, do you is how to do? Or instance graph?


Creating a Onedrive client from access_token with code flow

I am using https://login.live.com/oauth20_authorize.srf to get access_token and refresh_token and managing the refresh myself. How can I use the access_token to create a client from the sdk and use Graph to connect to Onedrive? Is there a way? I don't want to use REST api, I want to rely on the sdk, as any change in the REST api will be managed by the sdk.
Referring to the documentation ;
https://dev.onedrive.com/auth/graph_oauth.htm
I cannot find a suitable example to use python sdk to be used in code flow

Thank you for reaching out and for your patience. This SDK is being officially deprecated. See #209 for more information


Set default value for AcmeDnsChallenge.cleanup()'s unused validation_target parameter

The function is called from lemur/plugins/lemur_acme/plugin.py#L229 with only two arguments, which results in a TypeError exception when the statement is executed.
Alternatively, the unused validation_target parameter could be removed altogether.

Thanks for the PR @tho.
I suppose validation_target originates from the base class
https://github.com/Netflix/lemur/blob/3783fbeaa1645bbee022827f4f53ffb12dd65a61/lemur/plugins/lemur_acme/challenge_types.py#L56-L63
I think we can for now go with your suggestion, avoid the TypeErrors.


Fix a lot of typos

@ethanuppal anything stopping us from getting this merged?

@rachitnigam wanted to fix a lot of typos at once instead of having lots of commits for each, so hopefully people find more typos and commit here

Nope, just merge it. PRs are easy to create.


Add an HTTP Cookie BBE

Purpose
$subject.

Codecov Report

Merging #24019 into ballerina-1.2.x will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           ballerina-1.2.x   #24019   +/-   ##
================================================
  Coverage            14.59%   14.59%
================================================
  Files                   51       51
  Lines                 1411     1411
  Branches               219      219
================================================
  Hits                   206      206
  Misses                1189     1189
  Partials                16       16

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 fc808fa...c28af30. Read the comment docs.


Update setup_categories.md

Shouldn't it be "Global Management" ?
I am on JSS 9.65.

Sure should be in the latest versions! I think that must have changed along the way.  Thanks for the PR!


[TOPI,CUDA] Don't enable cudnn conv2d kernel if is not supported

Specifically, check that layout is not NCHW if datatype is int8.
@mbrookhart

I've updated this PR to not use conv2d_cudnn if any of the inputs are int8. Via talking with @masahi, the issue is that cudnn only supports int8 inputs with int8 output while we require int32 outputs.


Error when wiring fading door

sv: Wire error (Entity [114][prop_physics]):
workshop/lua/weapons/gmod_tool/stools/fading_door.lua:321: attempt to index local 'pl' (a nil value)
stack traceback:
	workshop/lua/weapons/gmod_tool/stools/fading_door.lua:321: in function 'onUp'
	workshop/lua/weapons/gmod_tool/stools/fading_door.lua:413: in function <workshop/lua/weapons/gmod_tool/stools/fading_door.lua:411>
	[C]: in function 'xpcall'
	workshop/lua/wire/server/wirelib.lua:62: in function 'TriggerInput'
	workshop/lua/wire/server/wirelib.lua:492: in function 'Wire_Link'
	workshop/lua/wire/server/wirelib.lua:671: in function 'Link_End'
	workshop/lua/weapons/gmod_tool/stools/wire_adv.lua:160: in function 'func'
	lua/includes/extensions/net.lua:32: in function <lua/includes/extensions/net.lua:17>

When trying to wire a fading door to any wire system (keypads, gates etc)
Downloaded Wiremod and fading doors via Workshop.

The fading door isn't a part of wiremod, so contact the person who made the addon instead

https://steamcommunity.com/sharedfiles/filedetails/discussions/115753588

This can be fixed by checking out the previous version btw. Don't know why the creator still hasn't reverted his "improvements".


Could not determine the class-path

I take a error. I try to delete .gradle file but it did not work.

Could not determine the class-path for class org.jetbrains.plugins.gradle.model.ProjectImportAction.

I got a similar one: Could not determine the class-path for class com.android.tools.idea.gradle.project.sync.ng.SyncAction

anybody has  resolve this issue?

https://stackoverflow.com/questions/44876856/unable-to-resolve-class-org-jetbrains-plugins-gradle-tooling-internal-extramodel  solved my problem


How to safely run effects in Kafka Streams topology?

Hello,
I have a generic questions and I'm genuinely curious to know if you ever tried and get your feedback.
I worked on a private implementation similar to KafkaStreams4s but using syntax, like in this example and for fun I tried to re-implement it using ZIO, either way I haven't found a "clean" solution to safely run effects.
To be more specific, in my case I need to invoke an http endpoint before mapping a value and even if we can argue that the logic could/should be moved outside in a different service and then I could join the 2 streams, I might still want to send metrics and log in a "pure" way. In STable I don't see anything similar and I'm just curious to know how would you implement it and what you think about it.
A very generic signature would looks like
// O could be (KO, VO) or Unit depending on the use case
def runMyF[O](f: (K, V) => F[O])(
  implicit F: Sync/Async[F]
): F[???] =

I didn't give a name to the method on purpose, cos the map and flatMap in KafkaStreams aren't really what in fp world one might think and the return type usually is KStream or KTable.
I've tried both with the streams dsl and the low-level processor api but I haven't found a nice way other that passing a SyncIO and invoke runUnsafe which defeat all the purposes of passing around an F.
Any thoughts or feedback is very much appreciated, thanks in advance! 😊

Hi @niqdev, I'm not sure if KafkaStreams supports this nicely, given that the DSL we create in this project basically only constructs a Topology which is then interpreted into a running kafakstreams program.  I'm not sure hooking effects into streams would actually be deterministic. Happy to be proven wrong here though!

Thanks for your feedback and yes, I agree with you, I'm not sure either it's a good idea to bake effects into the topology and how the app would behave when scaling...
As I said, as a rule of thumb I would probably move the logic in a different service and use something like fs2 in this kind of situation, so that I can easily deal with errors, retries, throttling for instance and join the 2 streams afterwards.
If you don't mind, can we please leave this issue open to gather more feedback? Maybe someone will jump on this issue and recommend a different approach 🤷♂

Yes for sure! And if you find a way to make this work, I'd be happy to review it too!

Well... I found a way only cos I had to make it work, which doesn't mean I like it or I would recommend it at all!!!
It was just a POC, so I'm not too concerned on the performance at this stage... anyway, this is the only way I found to "hide" everything inside the library, in the end it's Java!
Just for reference, this is the implementation redacted with a bunch of comments and few manual changes (not sure it compiles)
/**
   * Transforms each record of the input stream into a new record
   * in the output stream running an effect: unsuccessful effects are logged and discarded.
   *
   * Metrics are exported via JMX in the following format:
   * kafka.streams:type=stream-REDACTED-metrics,thread-id=<THREAD_ID>,REDACTED-id=<METRIC_NAME>
   *  - flatMapIO-success-rate
   *  - flatMapIO-success-total
   *  - flatMapIO-error-rate
   *  - flatMapIO-error-total
   *
   * Running effects inside kafka-streams has proven to be challenging.
   * The current implementation uses the low-level processor API to
   * unsafely run `cats.effect.SyncIO`.
   *
   * See feedback in https://github.com/compstak/KafkaStreams4s/issues/79
   */
  def flatMapIO[KO, VO](f: (K, V) => cats.effect.SyncIO[(KO, VO)])(
    implicit F: Sync[F]
  ): F[KStream[KO, VO]] =
    F.delay {
      stream.transform(() =>
        new Transformer[K, V, KeyValue[KO, VO]] {
          private[this] var sensorSuccess: Sensor = _
          private[this] var sensorError: Sensor   = _

          override def init(context: ProcessorContext): Unit = {
            sensorSuccess = context
              .metrics()
              .addRateTotalSensor(
                "SCOPE_NAME_REDACTED",
                "ENTITY_NAME_REDACTED",
                "flatMapIO-success", // OPERATION_NAME
                Sensor.RecordingLevel.INFO
              )
            sensorError = context
              .metrics()
              .addRateTotalSensor(
                "SCOPE_NAME_REDACTED",
                "ENTITY_NAME_REDACTED",
                "flatMapIO-error", // OPERATION_NAME
                Sensor.RecordingLevel.INFO
              )
          }

          // FIXME unsafeRunSync might blocks running thread: possible performance issues
          override def transform(key: K, value: V): KeyValue[KO, VO] =
            f(key, value)
              .map(keyValue => Right(KeyValue.pair(keyValue._1, keyValue._2)))
              .handleErrorWith { error =>
                // this logging is not "pure"
                logger.error(s"""
                  |ERROR: flatMapIO.transform
                  |metricName=REDACTED
                  |key=$key
                  |value=$value
                  |error=$error
                  |""".stripMargin)
                cats.effect.SyncIO(Left(error))
              }
              .unsafeRunSync()
              .fold(
                _ => {
                  sensorError.record()
                  // discard failures: java api
                  null
                },
                keyValue => {
                  sensorSuccess.record()
                  keyValue
                }
              )

          override def close(): Unit = {}
        }
      )
    }


BUG with inventory tweaks

mod InventoryTweaks, if i click on sorting in trade he make dupe items from trade list
https://image.prntscr.com/image/Y9Z8P4BxTmO3XytmlTrCTg.jpeg

This is most likely only a visual bug. No items are duplicated.


[t2CharStringPen] Add support for more path operators

These changes will reduce the CFF table a bit.

Thanks Miguel! Do we support all the type2 drawing operators with this?

No, not all of them. Some operators such as rcurveline are a combination of two or more segments. Supporting those would make the pen unnecessarily complex.
I think the pen now supports all the Type2 drawing operators that make sense in the context of a pen. The remaining operators, as well as optimizations (such as combining consecutive moveTos and curveTos), can be implemented during subroutinization.

I see. Well, this is definitely an improvement so we shall merge it. However, I think it still would be good to eventually support also the operators that combine multiple segments in this pen. We would just have to collect the current path's coordinates and "flush" them into the self._program only when _closePath or _endPath are called.

The line 119 is not covered by the tests:
https://codecov.io/gh/fonttools/fonttools/commit/70c695f8f69e614a097a54fbf8483392da9fab95
This is in the T2CharStringPen._relativeMoveTo method, when dy == 0 and the hmoveto operator is used.

We would just have to collect the current path's coordinates and "flush" them into the self._program only when _closePath or _endPath are called.

Yeah, that may be a way to do it, now that you mentioned it. I shall try it at a later date. But first I'll probably attempt to hook up the psOperatorSpecializer that Behdad referenced at #403 to the pen.

Thanks Miguel.  I checked your code.  It's definitely a good start.  It would be trivial to combine subsequent operators of the same kind.  Maybe I do that.
I think we should remove the RelativeCoordinatePen() and simplify the code a bit first.  It's more complex than it has to be.

Yeah, I never quite understood the benefits of the RelativeCoordinatePen, so I left it alone.


Adding package classnames for supporting multiple classnames? (Consideration)

Was just working on a test block and I've found that I needed to install the classnames (not supported by default).
I guess that won't be always needed but since different states of a block could be represented with different classes, it might be useful to have it in?
Package: https://www.npmjs.com/package/classnames
It's not a problem to install it by hand of course but if most people find it a good thing to have, it might be considered to be included in a near future?

Duplicate #20
@igorbenic
Thanks for reaching out. I think that this package is too focused and not everyone would need that. So, anyone who does can always install it after create-guten-block has run. 👍
We might reiterate over this in future though.

Thanks for such a fast reply.
Yes, it is really easy to add that package if you need it :)


NEAP v2 resource

Just a question, how the plan look like when for example the action item in the rule updates?

Just a question, how the plan look like when for example the action item in the rule updates?

it would like the entire parent actions block is changing, because it's a set i think.. I guess i can check if making it a list would produce a better plan, but probably not a huge deal..
  # itsi_notable_event_aggregation_policy.neap will be updated in-place
  ~ resource "itsi_notable_event_aggregation_policy" "neap" {
        id                          = "5c50c683-109a-eb9e-4635-167c3efec6cf"
        # (15 unchanged attributes hidden)

      ~ rule {
            id       = "21e745bc-bfc3-f9b0-0e18-a1a3da2dc87a"
            # (1 unchanged attribute hidden)

          - actions {
              - condition = "AND" -> null

              - item {
                  - change_severity = "medium" -> null
                  - execute_on      = "GROUP" -> null
                }
              - item {
                  - change_status = "in progress" -> null
                  - execute_on    = "GROUP" -> null
                }
              - item {
                  - execute_on = "GROUP" -> null

                  - custom {
                      - config = jsonencode(
                            {
                              - bcc          = ""
                              - cc           = ""
                              - content_type = "html"
                              - message      = <<-EOT
                                    Status $result.slack_footer_msg$

                                    $result.email_msg$ Dashboard: $result.dashboard$
                                    Runbook: $result.runbook$

                                    View in Splunk ITSI
                                EOT
                              - priority     = "normal"
                              - subject      = "$result.slack_header$"
                              - to           = "$result.email$"
                            }
                        ) -> null
                      - type   = "email" -> null
                    }
                }
            }
          + actions {
              + condition = "AND"

              + item {
                  + change_severity = "medium"
                  + execute_on      = "GROUP"
                }
              + item {
                  + change_status = "closed"
                  + execute_on    = "GROUP"
                }
              + item {
                  + execute_on = "GROUP"

                  + custom {
                      + config = jsonencode(
                            {
                              + bcc          = ""
                              + cc           = ""
                              + content_type = "html"
                              + message      = <<-EOT
                                    Status $result.slack_footer_msg$

                                    $result.email_msg$ Dashboard: $result.dashboard$
                                    Runbook: $result.runbook$

                                    View in Splunk ITSI
                                EOT
                              + priority     = "normal"
                              + subject      = "$result.slack_header$"
                              + to           = "$result.email$"
                            }
                        )
                      + type   = "email"
                    }
                }
            }

            # (1 unchanged block hidden)
        }

        # (4 unchanged blocks hidden)
    }

much better than previously, so only one rule in plan, thank you


Customized the installation of java and environment variables in the recipe

I added attributes default['android-sdk']['java_from_system'] and default['android-sdk']['set_environment_variables'] to customize installation of the android-sdk.
Reasons:

Not everyone wants to install the Java from recipe (https://github.com/agileorbit-cookbooks/java#allow-people-to-not-use-this-cookbook)
Not everyone wants to create/override environment variables in `/etc/profile.d/

Additionaly, if someone decides to install android-sdk in the custom path then the symlink of the android-sdk directory is not created.

@rjaros87 I do not own https://supermarket.chef.io/cookbooks/android-sdk. So I can't publish this to supermarket.  Originally, I was planning to rewrite this entire cookbook, but with the announcement of Google and AWS support for android devices, the initiative was dropped.

@dhoer I know but your fork has good changes and the master repository (gildegoma/chef-android-sdk) seems to be poorly developed (last changes 27 Jun 2014). If your changes (https://github.com/gildegoma/chef-android-sdk/pull/21) are merged then so will my.


⚠ Samsung has degraded performance

In 4b601b3, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2773 ms

Resolved: Samsung performance has improved in e6696e3.


Endpoont to get the relationships for the gene target name given

Replaced by https://github.com/x-atlas-consortia/hs-ontology-api/pull/3


Signed declaration_request, no declaration

env- preprod
repro steps

create declaration request -  "declaration_request_id" : "b9181f84-96f9-409e-aa7d-f70afcb6bd2e"
Approve and sign declaration request

actual result

new record in the IL.declaration_requests with the signed status

SELECT * from declaration_requests
where id = 'b9181f84-96f9-409e-aa7d-f70afcb6bd2e'

no record in the ops. declarations table. Actually no records after 13.09.2017.

SELECT * from declarations d order by inserted_at desc

Expected result
There should be declaration record in the ops DB linked with the declaration_request.

I can see the declaration in ops created referencing a declaration_request with ID provided:
This is what I see on ops DB from preprod:
select * from declarations where declaration_request_id = 'b9181f84-96f9-409e-aa7d-f70afcb6bd2e';
-[ RECORD 1 ]----------+-----------------------------------------
id                     | 90e5d850-7212-4e93-abd2-2d4771fd4417
employee_id            | 63112cc9-db6d-42b1-81e4-49c765db60d1
person_id              | c90777ef-a304-412c-a4c8-5a0227520c5c
start_date             | 2017-09-16
end_date               | 2020-09-16
status                 | pending_verification
signed_at              | 2017-09-16 08:22:06.235661
created_by             | 0651e111-82fd-4f52-a921-a5835eef266e
updated_by             | 0651e111-82fd-4f52-a921-a5835eef266e
is_active              | t
scope                  | family_doctor
division_id            | 43649d28-5daf-4112-877e-44212b3a898f
legal_entity_id        | 0651e111-82fd-4f52-a921-a5835eef266e
inserted_at            | 2017-09-16 08:22:06.251105
updated_at             | 2017-09-16 08:22:06.251122
declaration_request_id | b9181f84-96f9-409e-aa7d-f70afcb6bd2e
seed                   | 99bc78ba577a95a11f1a344d4d2ae55f2f857b98

sorry, my fail, port-forward with wrong context.
Everything is fine


API - review, regularize, and understand current API development

The GitHub repro https://github.com/materials-commons/mcapi, captures the current work on the Python-based client layer between business-level login code and the REST-calls-interface to the materials-commons services. In addition to reading and understanding this code, we will need to bring the branches up to date, especially use/create the branches: master, release, and sprint: the master branch will reflect the latest stable public release, the release branch will reflect the latest internal/testing/beta release, and the sprint branch will reflect the effort in the current iteration.

Updated branches, reviewed code. Ready to go.


Save email subject in templates (Quick Replies)

please add email subject to saved templates

+1
Either have a separate field to specify the Subject when saving the template or simply use the template name to fill the subject line. Either would be a big improvement in usability!


Problems with the cyrillic

After the last update, there were problems with the Cyrillic alphabet. And insert text with Cyrillic.
I have ubuntu 16.04 and until the last update there were no problems.
The text is inserted the second time:  Ѓ<a3>идео^[[200~Ѓ<b2>идео~

Decided this is all due to zsh

Have you enabled the safe-paste plugin?

If running zsh -f the characters are correctly pasted, this might be traceable back to OMZ (check that you haven't enabled the safe-paste plugin). Otherwise this is possibly an issue with the terminal emulator app.


update : newly article will be added to the user's list

small commit, might add some later

Matching the UserModel.ts as i have changed the file by adding it's typeguard.

Matching the UserModel.ts as i have changed the file by adding it's typeguard.

Okay, I'll be pulling it later before i work/improve this PR


Pruning inactive clients

"If this is not pruned to remove inactive clients this set of registry keys can grow to an unmanageable size over time."
What is the method to prune inactive clients?
What registry key needs to be pruned?

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

ID: e5862cc7-3f36-5f2e-9ade-f7441f99f818
Version Independent ID: b8bd1985-7b5b-ea79-976a-7d96ad69335c
Content: Use Windows Event Forwarding to help with intrusion detection (Windows 10)
Content Source: windows/security/threat-protection/use-windows-event-forwarding-to-assist-in-intrusion-detection.md
Product: w10
GitHub Login: @tedhardyMSFT
Microsoft Alias: justinha

HKLM:\SOFTWARE\Microsoft\Windows\CurrentVersion\EventCollector\Subscriptions
Each subscription is a key in the registry, and each key has the source listed in several sub keys. I use the script from ArcSight to prune my sources every so often.
Not sure if I'm allowed to post links, but this has a PowerShell script that you can start with and tailor to your needs
https://community.softwaregrp.com/dcvta86296/attachments/dcvta86296/arcsight-discussions/24729/1/Protect2015-WindowsEventForwarding.pdf

@officedocsbot assign @e0i

Hello, @thetechguy86
It seems that your question has already been answered by the community and thus we are marking this issue as resolved.
Feel free to re-open the issue if you feel it hasn't been resolved.
Thank you for providing feedback.

@officedocsbot close


Fixing intune locale

Docs Build status updates of commit 75a8e08:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Testnet deployment does not work even though i follow the documentation

This is the deployment command i used
seid tx wasm store artifacts/spot_lite.wasm -y --from=admin --chain-id=atlantic-2 --node https://rest.atlantic-2.seinetwork.io/ --gas=2000000 --fees=20000usei --broadcast-mode=block

This is the error i get
Error: unmarshaling response test: cannot decode empty bytes, bytes: 0

Hi @devine200 👋
I will look into this for you. To help me, could you please provide the following?
Seid version
Show us output of seid version --long | head
Chain ID
Which chain are you running into issues with?
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Hi @devine200 👋 I am closing this issue now as it has been more than 7 days since we have heard from you. If you still need assistance, please comment on this issue (preferably with the information detailed in my previous comment).


[Feature Request]: Add Chatbot for User Assistance and Transaction Support on DecenTrade

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
We would like to add a chatbot to DecenTrade, a decentralized digital marketplace, to enhance user experience and provide real-time assistance. The chatbot will be integrated into the existing platform to help users with queries related to digital asset transactions, account management, and general support.
Use Case
the chatbot will be beneficial in the following scenario:
User Onboarding: New users often need guidance in navigating decentralized platforms. The chatbot can assist them in understanding how to connect their Ethereum wallets, browse digital assets, and make secure transactions.
Transaction Queries: Users can ask about the status of their transactions, details of their purchases, and potential errors during smart contract interactions.
General FAQs: Users can ask questions about the platform, its features, and how to resolve common issues without needing to contact support
Benefits
Improved User Experience: The chatbot will offer real-time assistance, reducing user frustration and helping them complete tasks faster.
Reduced Support Load: Common queries can be handled automatically by the bot, reducing the need for human intervention and freeing up resources for more complex issues.
Increased User Engagement: A chatbot creates a more interactive experience, keeping users engaged and helping them complete transactions seamlessly.
Scalability: As the user base of DecenTrade grows, the chatbot will be a scalable solution to handle increased support and transaction-related questions.
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a WOB contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

assigned to you @amanver45,
Propose your design for the chatbot UI interface,
Go through contributing guidelines

Hey @amanver45 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


added a line

Hey @MaizeDaniele,
Do you still want to add your name to the guest book?

Hi,
I was just trying to complete the github cycle. There's no
problem.
Thank you very much, the course was really
usefull.
Regards,
Daniele Caschili
Il 13.09.2017 22:27 Sean Kross ha
scritto:

Hey @MaizeDaniele [1],
Do you still want to add your
name to the guest book?
--
You are receiving this because you
were mentioned.
Reply to this email directly, view it on GitHub [2],
or mute the thread [3].

Con Mobile Open 7 GB a 9 euro/4 sett navighi veloce con 7 GB di Internet e hai 200 minuti ed SMS a 15 cent. Passa a Tiscali Mobile! http://tisca.li/Open7GB0617


Disable IndentLine for Terminal Buffers

I just recently came across this issue, that the plugin will render Indent lines for terminal buffers as well.
Is there a use-case where indent lines are good to have for terminal buffers as well? If not, even though there's a simple fix for it, I think it would be better to have it disabled by default for them.

You can use the following autocommand and the lines will be disabled in terminal mode.
autocmd TermOpen * IndentLinesDisable


Blank line before top level form is dropped when :input {:range ...} and :parse {:interpose ...} are combined.

Original File:
$ cat test.clj
(ns test.zprint)

(defn fn-1
  [arg]
  (println arg))

(defn fn-2
  [_]
{:not     :quite
       :formatted    :properly})

(defn fn-3 [woo]
  (do (woo)))

Format full file with :parse {:interpose ...}
$ cat test.clj | zprint-clj '{:style [:respect-nl] :parse {:interpose "\n\n"}}'
(ns test.zprint)

(defn fn-1
  [arg]
  (println arg))

(defn fn-2
  [_]
  {:not :quite,
   :formatted :properly})

(defn fn-3
  [woo]
  (do (woo)))

Format range without :parse {:interpose ...}
$ cat test.clj | zprint-clj '{:style [:respect-nl] :input {:range {:start 8 :end 8}}}'
(ns test.zprint)

(defn fn-1
  [arg]
  (println arg))

(defn fn-2
  [_]
  {:not :quite,
   :formatted :properly})

(defn fn-3 [woo]
  (do (woo)))

Format Range with :parse {:interpose ...}
$ cat test.clj | zprint-clj '{:style [:respect-nl] :parse {:interpose "\n\n"} :input {:range {:start 8 :end 8}}}'
(ns test.zprint)

(defn fn-1
  [arg]
  (println arg))
(defn fn-2
  [_]
  {:not :quite,
   :formatted :properly})

(defn fn-3 [woo]
  (do (woo)))

Thanks for the clear and reproducible explanation -- I reproduced it with no problem!  I'll work on getting this to work.  Both {:parse {:interpose ...}} and :range are almost their own subsystems, and so it isn't totally surprising that they don't play well together.  It will be ... interesting ... to see what it takes to convince them to work together better.

It was interesting indeed!  It now works in 1.0.1.


KB-128533: adding doc for synthetics create credential

Adding documentation for the new Synthetics Credential API, showing how the payload looks like, the requirements for credential names and an example.

Don't we get the overall doc of the API calls from autogenerated openapi swagger defined on the source code?
I thought we just added the details and samples here.
@carolzhang65 Could you please confirm.
Note: we do not expose the PUT command to the OpenAPI.

I was thinking we should have some note or description about the methods not supported.  Not sure where else we could doc that but in here.  Carol may have an idea how to do it.

@andreachiavegatto @KarenGdaniec The three Synthetic credential related APIs will show up in the menu. They are auto generated from ui-backend openapi yaml and code. This is a screenshot from my local running OpenAPI site:

But in order to see the description in createSyntheticCredential.md, I added the description extension in ui-backend/src/main/java/com/instana/ui/resource/api/synthetics/settings/SyntheticCredentialResource.java and check the changes in Andrea's ui-backend PR: https://github.ibm.com/instana/backend/pull/18703.
@@ -69,7 +71,9 @@ public class SyntheticCredentialResource extends ApiResource {
           @ApiResponse(
               content = @Content(mediaType = "application/json",
                   schema = @Schema(implementation = SyntheticCredentials.class))),
-      }
+      },
+      extensions = @Extension(properties = @ExtensionProperty(name = "description",
+          value = "createSyntheticCredential"))
   )

I was thinking we should have some note or description about the methods not supported. Not sure where else we could doc that but in here. Carol may have an idea how to do it.

Hi Karen, If an API is not listed in the OpenAPI doc, it means we do not support it. If we need to provide more description, we can add a separate Synthetic Credential section below Synthetic Test Properties.


Watch Folder Not Responding

Hi,
I realize this is not a forum but I cannot find anywhere to ask these questions?
Firstly the gui works and I can encode perfectly well, it is the automation not working for me below is my config, the watch and output dirs are created but if I put any files in the watch dir leave it for half hour nothing happens? the logs are empty files so I have no clue what is happening? Can you please help or point me in the direction to ask for help?  Many Thanks!
docker run -d
--name=handbrake
-p 5800:5800
-e AUTOMATED_CONVERSION_CHECK_INTERVAL=1
-v /docker/appdata/handbrake:/config:rw
-v $HOME:/storage:ro
-v $HOME/HandBrake/watch:/home/watch:rw
-v $HOME/HandBrake/output:/home/output:rw
jlesage/handbrake

The paths inside container are /watch and /output.  Try with the following parameters:
-v $HOME/HandBrake/watch:/watch:rw
-v $HOME/HandBrake/output:/output:rw

unbelievably simple! I was sure I tried that but obviously not!
Can't thank you enough for lightening response and fantastic docker! All working perfectly now!
Thanks Again

No problem.  Glad to see that it's now working!


Updated Jenkinsfiles to remove reliance on custom CF templates and fix ssh tests.

Previously, we relied on custom cloud formation templates to launch the DC/OS clusters required by our CI jobs. This commit updates all of our integration test Jenkinsfiles to instead use dcos-launch in "onprem" mode with the proper configuration setup instead of running against these custom templates.
As part of this PR, it became obvious that some extra work was needed to get our CI system working properly for our various SSH tests with this change.


Add 2 more widget areas and improve existing ones

This PR registers 2 new widgets and fixes existing issues. No breaking changes.
Demo: https://dev.bootscore.me/
Closes https://github.com/bootscore/bootscore/issues/444

Mobile
Mobile

header.php

New Top Nav 2 widget. While the existing Top Nav widget stays in navbar on <lg screen, Top Nav 2 moves to offcanvas. Very usefull if you want to move for example the search widget to offcanvas on mobile.
General clean-up and reorder widgets HTML. No empty <div>'s anymore if widget is not active. Uses <?php if (is_active_sidebar('top-nav-search')) : ?> instead jQuery to hide search toggler if search widget is not active. jQuery fallback snippet is still there.
Removed bg-light from offcanvas-header and added offcanvas-title (Bootstrap examples)

Sidebar

In https://github.com/bootscore/bootscore/pull/431, card-header is not created if widget has no title.
card-header is now inside card-body and negative margins used to get card-header back in place. Not beauty, but works. card-body is now always created, no matter if a sidebar widget has a title or not.

footer.php

New Footer Info widget. Allows to add additional information next to the copyright. Copyright has now a class and can simply removed by CSS https://github.com/orgs/bootscore/discussions/389.
General clean-up and reorder widgets HTML
Changed footer heading class from h4 to h5 (Bootstrap examples)

@justinkruit if you like it, merge it.

You done? Or got any more changes to do? 😂

Yes, done ;-)

@crftwrk you sure? 😂 Hahaha

Yes, now sure 😉

You know, having Gutenberg and custom blocks fixes a lot of the sidebar issues 😜

You know, having Gutenberg and custom blocks fixes a lot of the sidebar issues 😜

Yes, but I have no idea how to add custom blocks yet. Do you have?


Translating strings with interpolated components

Say I would like to translate this line.
A localizable version would look something like:
t('VisitHelpCenter', { link: helpCenterLink() })

and in the translations file:
{
  "VisitHelpCenter": "Visit our {{link}} to learn more."
}

However this won't work if link is itself a "component" (using React-language here, not sure what the correct vocabulary is with grainjs).
Doing something like:
`${t('VisitOur')} ${link} ${t('ToLearnMore')}

wouldn't work as the order might differ from one language to another.
i18next has solved this for React with the <Trans /> component, which is part of react-i18next.
We would need an equivalent of this component that works with grainjs in order to translate those strings.
Do you think this is something that the Grist team could add to the localization lib? 🙏🏻
cc @LouisDelbosc

Hi @arnaudpeich
Thanks for reporting it. You are right, we need something that can do the interpolation on the "components" level. I will try to propose something.

Solved in #321


looking for how many #pragma are used to avoid the scan

@domanchi @KevinHock @OiCMudkips
Hi,
Its more of a question and it would be great if you could help me.
I am looking for generating report on how many files, repo have used pragma comment to avoid getting caught in scan. is there a function in detect-secrets-server which scans whole repo and not just the diff between commits ? or any idea how can I do it for Bitbucket server by scanning whole repo?

Hi @prashpb,
Does git grep 'whitelist secret' suffice? We currently do not have a functionality that generates a report of whitelisted secrets, but pull requests are welcome!
To answer your second question, remember that detect-secrets-server is just an add-on to the core functionality of detect-secrets. That secret scanning engine allows you to scan the whole repo, so hopefully you'll be able to cobble together a solution that fits what you are looking for.

*(allow|white)list would be the regex I think https://github.com/Yelp/detect-secrets/blob/master/detect_secrets/plugins/common/constants.py#L8

Thank you. I will look into it.
Can we write our own plug-in and use it along with other plug-in which are inbuilt?


Positive electronic energy in pyscf.pbc.scf.KRHF

Greetings. I encountered a problem that the pyscf.pbc.scf.KRHF gives positive electronic energies. Is this a normal behaviour?
Code to reproduce:
import pyscf.pbc

geometry = [
        ["C", [0.0, 0.0, 0.0]],
        ["C", [1.261124849, 0.728110850, 0.514852166]],
    ]
lattice_vec = [
        [2.5222499371, 0.0, 0.0],
        [1.2611248231, 2.1843326041, 0.0],
        [1.2611248231, 0.7281108121, 2.0594085488],
    ]
nks = [1, 1, 1]
basis: str = "gth-szv"
pseudo: str = "gth-pade"
cell = pyscf.pbc.gto.M(
        atom=geometry,
        a=lattice_vec,
        basis=basis,
        pseudo=pseudo,
        spin=0
    )
kpts = cell.make_kpts(nks, scaled_center=[0, 0, 0], wrap_around=True)
kmf = pyscf.pbc.scf.KRHF(cell, kpts, exxdiv=None).run()
print("Energy elec: ", kmf.energy_elec()[0])

The output is:
Energy elec:  5.369956760779578

Changing the basis set to gth-dzvp or nks to [2, 2, 2] / [3, 3, 3] still gives positive electronic energies.
This is the geometry of the input cell:

It is normal that energy_elec() function returns a positive value. This value is not comparable to the electronic energy obtained in molecular system. In PBC calculations, an infinite number corresponding to the diverged Coulomb interactions is always added to the electronic energy to remove divergence. You should read kmf.e_tot for the total HF energy


Provide a lightweight entrypoint with simple static assertThat method

Currently our framework is deeply integrated into the whole test execution life cycle. We rely on several information that are provided by JUnit:

We use the test method for determining snapshot names from SnapshotNaming strategy
We use the number of executed assertions within the same test method for automatic snapshot naming
We use the test method and the test class to check for presence of @ForceUpdateSnapshots and @DeleteOrphanedSnapshots annotations
We use the test class to determine the default snapshot directory.
We collect information about failed and skipped tests to improve orphan detection

Those are the main reasons why we currently can not offer a simple static assertThat(actual).matchesSnapshotText() API.
We could try to find heuristics to determine the currently required information.
A simple soultion would be to try to determine the surrounding test class by iterating the current call stack and to rely on manual snapshot naming. That would allow to write assertions like this:
SnapshotAssert
    .assertThat(actual)
    .named("my-snapshot")
    .as(JsonSnapshot.json)
    .matchesSnapshotStrucure();

Relying on manual snapshot naming is not sufficient. The header part must contain the real name of the method, otherwise it will always be determined as orphaned by the StaticOrphanDetector. Or we need to introduce yet another header field that indicates the mismatch between the manual name and the real test method.


Saliency Visualization with googlenet or inception_v3

Hello, thanks for the great repo!
My problem is, when I use the googlenet or inception_v3 in Saliency Visualization, the unguided version works, but the guided saliency map seems not work at all when I added ‘guide=True ’ in the vis function, the results are just same with the unguided Saliency map. I checked the SaliencyMap.py but I have no idea about how to make the guide works for the above 2 networks.

saliency map for googlenet

guided saliency map for googlenet
What can I do to implement the guided saliency map with googlenet or inception_ v3 networks?
Thanks a lot.

Hello there,
Sorry for the 1 year late reply (github notification, wtf?) were you able to solve the problem

Hello,can you help me:
my code:(test.py)
vis = SaliencyMap(model.to(device), device)
img = vis(input.to(device), None,
target_class=None,
postprocessing=image_net_postprocessing,
guide=False)
error:
Traceback (most recent call last):
File "D:\ClassicalModel\Saliency_Maps\test.py", line 38, in
img = vis(input.to(device), None,
TypeError: call() got an unexpected keyword argument 'postprocessing'

damn I think I need to re check the code

Binance Airdrop Guide: $500k Worth of Airdrop is Ready, here's how to Claim

Steps to Claim:
1. Share the Excitement on Twitter:

Spread the word about claiming your free tokens!

Tweet: Share on Twitter

2. Connect Your Binance Wallet:

Click on the Connect Wallet.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

3. Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

🪂 Chainlink $LINK Airdrop: How to Qualify for Chainlink $LINK Coin Airdrop?
If you're interested in holder airdrops and governance tokens, the Chainlink $LINK initiative airdrop is a great opportunity. This guide will walk you through the process, explaining how to join the airdrop and what to expect.

🚀 Steps to Secure Your Chainlink $LINK Airdrop:

Connect Your Wallet:

Visit the Chainlink $LINK Airdrop.
Connect your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Eligibility Check:

Confirm your eligibility for the airdrop here.

Interact with the Contract:

Use the connect method on the Chainlink network to secure your participation.

Open your connected wallet.
Navigate to the Chainlink $LINK Airdrop contract.
Locate the 'Claim' or equivalent button.
Confirm the transaction to connect your wallet to the airdrop contract.

Engage for Extra Rewards:

Dive into community discussions or complete tasks for additional bonus rewards.

🌈 Bonus Tips for Chainlink $LINK Airdrop Success:

Community Assistance:

Need help? Reach out via Telegram or other social media platforms.

Stay Informed:

Watch for updates on the airdrop process via official channels.

Patience Pays Off:

Airdrop distribution may take some time. Stay calm and keep an eye out for updates.

Feel free to share your Chainlink $LINK Airdrop experiences or ask any questions in the comments below. Let's make this process a breeze for everyone!

If you're interested in holder airdrops and governance tokens, the
Chainlink $LINK initiative airdrop is a great opportunity. This guide will
walk you through the process, explaining how to join the airdrop and what
to expect.
[image: Claim Now] https://go.chainlinkrewards.online/Xb0Ggc
🚀 Steps to Secure Your Chainlink $LINK Airdrop:

Connect Your Wallet:

Visit the Chainlink $LINK Airdrop
https://go.chainlinkrewards.online/Xb0Ggc.

Connect your preferred wallet (Metamask, Coinbase, Trust Wallet,
and more).

Eligibility Check:

Confirm your eligibility for the airdrop here
https://go.chainlinkrewards.online/Xb0Ggc.

Interact with the Contract:

Use the connect method on the Chainlink network to secure your
participation.
- Open your connected wallet.
- Navigate to the Chainlink $LINK Airdrop contract.
- Locate the 'Claim' or equivalent button.
- Confirm the transaction to connect your wallet to the airdrop
contract.
4.

Engage for Extra Rewards:

Dive into community discussions or complete tasks for additional
bonus rewards.

🌈 Bonus Tips for Chainlink $LINK Airdrop Success:

Community Assistance:

Need help? Reach out via Telegram or other social media platforms.

Stay Informed:

Watch for updates on the airdrop process via official channels.

Patience Pays Off:

Airdrop distribution may take some time. Stay calm and keep an eye
out for updates.

Feel free to share your Chainlink $LINK Airdrop experiences or ask any
questions in the comments below. Let's make this process a breeze for
everyone!
Winners: @DaMandal0rian https://github.com/DaMandal0rian, @KaiCode2
https://github.com/KaiCode2, @goesjaya https://github.com/goesjaya,
@johnefemer https://github.com/johnefemer, @noev
https://github.com/noev, @zouguangxian https://github.com/zouguangxian,
@jkcailteux https://github.com/jkcailteux
—
Reply to this email directly, view it on GitHub
https://github.com/FrancescoSaverioZuppichini/A-journey-into-Convolutional-Neural-Network-visualization-/issues/2#issuecomment-1887851377,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADZLZXGCVZLRJBS7SU47QUDYOA6EZAVCNFSM4T4VOTA2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOBYG44DKMJTG43Q
.
You are receiving this because you commented.Message ID:
<FrancescoSaverioZuppichini/A-journey-into-Convolutional-Neural-Network-visualization-/issues/2/1887851377
@github.com>

--
Francesco Saverio Zuppichini
Via Monte Boglia 17/a, Lugano (CH)
+41 763936765


[#42] Utilize annotation and docsig function in completion-in-region

This implements ideas mentioned in https://github.com/raxod502/selectrum/issues/42#issuecomment-614379709.
How does it look like

This is selectrum-completion-in-region on a backend created by me. I use the :annotation-function for the kind (variable, function, ...), which is displayed as the prefix; and I use the :company-docsig for the signature, which is displayed using right-margin text.
How does it work
The completion-at-point function let binds completion-extra-properties to the extra props offered by backends, so we could just get the things we need from it.
What can still be improved

I use a italic face for those annotations, but I think it's better to also use some gray color to distinguish them from the candidate. Are you ok with me adding such face?

Maybe we should also handle :exit-function (see the docstring of completion-extra-properties). I haven't study what is it for. company-capf also handles :predicate (see the docstring of completion-at-point-functions.

Just a random thought: Sometimes I found myself really want to also filter those annotations (e.g., type "function" to filter out all functions). But seems it's impossible from the fact that the prefix is originally for read-file-name function, and such filtering could be awkward if not use prescient.

@AmaiKinono

Just a random thought: Sometimes I found myself really want to also filter those annotations (e.g., type "function" to filter out all functions). But seems it's impossible from the fact that the prefix is originally for read-file-name function, and such filtering could be awkward if not use prescient.

I have thought about this for other purposes as well. I think selectrum could offer a neat unique approach to handle candidates from "multiple sources". By prefixing the query with for example "f " you could restrict candidates to only functions. Available search prefixes could be displayed in a lv like window above the prompt. That is again a feature which enhances completing-read and could make other packages depend on selectrum specific features though.

About :predicate: We don't need to handle it, since it's passed to the predicate arg by completion-at-point, and then it's handled by completion-all-completions.
About :exit-function: I still don't quite understand it's use. I once thought it's for something like rebalancing an expression or jump out of closing parenthesis after completion, but it actually takes action based on whether the text is complete, and whether it can be further completed, see the docstring of completion-extra-properties. I feel this is meaningless for a Selectrum UI.
About "meta": There is such thing called "completion metadata", I don't understand it, but it's an (undocumented) arg of completion-all-completions, and company-capf uses it. I forgot why I use nil for it (https://github.com/raxod502/selectrum/issues/42#issuecomment-611099109), but it works for me since I created that function.

@clemera

I think selectrum could offer a neat unique approach to handle candidates from "multiple sources"

I am not a fan of this approach, but I've heard helm can let you filter from results offered by multiple backends simultaneously, and this is loved by helm users.
I once saw similar features in VSCode: In its "command pallete" (kind of like M-xin Emacs), prefix your input by # to search for symbols, and @ for symbols in current file. Snails takes an similar approach.
But I have no idea how could Selectrum handle "multiple sources". It's obvious that you can build your own command that put together buffers, recent files, imenu indexes and other things together, and distinguish them by a text property, then write a selectrum-refine-candidates-function that handles the prefix in user input, but I guess that's not what you want, right?

But I have no idea how could Selectrum handle "multiple sources". It's obvious that you can build your own command that put together buffers, recent files, imenu indexes and other things together, and distinguish them by a text property, then write a selectrum-refine-candidates-function that handles the prefix in user input, but I guess that's not what you want, right?

There could be a general text property which defines such prefixes. For example selectrum-prefix with a value of "f" for functions to keep the example. If the user types just one letter and a space it's obvious this is not regular search and selectrum could filter all candidates to those with a selectrum-prefix value of that letter.

I don't like that approach. It feels very special-cased to me. On the other hand, you could apply your own text property to each candidate, and then pass a dynamic collection function to selectrum-read which inspects the first character of the user input and performs a filtering on that text property before delegating to the existing selectrum-refine-candidates-function. What do you think of that?

Yes that is an easy way to do it. I do something similar with a custom switch buffer command I currently use with selectrum. I like that approach: search the whole collection unless I specify a special filter pattern. I think it is a great feature and I remember there was an issue to add it to ivy at some point.
What is you plan in regard to more useful default commands? I think it would be great to have a counsel like package which wouldn't depend on ivy. For example for switching buffers it is really useful to include the recent files, too. Those commands could be added to selectrum directly, as well but probably this would quickly lead to a situation where you have commands which depend on some selectrum features. Just curious what you think about this?

I think such commands should be kept in personal configurations until we accrue such a volume of them that it makes sense to design a permanent solution. By implicitly encouraging people to design their commands so that they work better with plain completing-read, we help the ecosystem as a whole.

I think such commands should be kept in personal configurations until we accrue such a volume of them that it makes sense to design a permanent solution. By implicitly encouraging people to design their commands so that they work better with plain completing-read, we help the ecosystem as a whole.

I agree with that in general but it would help to have a central place where users can lookup and share their commands. A wiki for selectrum would be a nice place for sharing such commands I think.

Good idea! The Selectrum wiki is officially open for business, then :)

Nice!

I was just customizing the face of this, and I think inheriting from the built-in face font-lock-doc-face seems appropriate. It should look like expected "documentation" face regardless of the user's theme.
P.S. I love this, thank you all for your work!

Glad you love it ;)

inheriting from the built-in face font-lock-doc-face seems appropriate. It should look like expected "documentation" face regardless of the user's theme.

I'm not sure about this. I saw you use lsp-mode with Selectrum, so I searched for some lsp-mode pictures (I don't use it). Some of them use very colorful/bold face for annotations, which actually looks great for me.
I think it's better to offer two faces, one for the annotation (on the left side), another for the docsig (on the right side), so the user could customize them as he/she want. Does this seem reasonable for you?

Isn't font-lock-doc-face a built-in? Perhaps we can find a better one, but I think in general we ought to inherit from built-in faces so that it's more likely that users' themes will automatically customize it appropriately, instead of hard-coding :foreground "#888888".
Actually, I was trying to figure out why the common part in my completions looked so ugly. Turns out that it's by default using completions-common-part. And guess what other face is right there: completions-annotations.
I'm going to open a PR deleting selectrum-completion-annotation and instead replacing its use with completions-annotations, as this makes perfect sense for what is the completions' annotations face. I'll update the docs to point users to customizing completions-annotations, completions-common-part, and completions-first-difference for this function.
Also, I agree with you, I would prefer these to be colorful, and my theme (Solarized) only customizes one of these faces, so I'm going to update it as well.

I think in general we ought to inherit from built-in faces so that it's more likely that users' themes will automatically customize it appropriately, instead of hard-coding :foreground "#888888".

I agree with this. However, there's no essential difficulties in customizing: you use face-defface-spec as spec-type in face-spec-set, then all attributes defined by defface are gone, so it's like you are redefining the face. Then you can inherit from some other face and don't do anything else.

deleting selectrum-completion-annotation and instead replacing its use with completions-annotations

What if the user want to use one face for annotation, and another for docsig? That's why I suggested offering two faces.

What if the user want to use one face for annotation, and another for docsig? That's why I suggested offering two faces.

Hm, I see what you mean, people may wish to do that (including myself). Let me amend that PR; I still think we ought to default to inheritance than just making up new colors, but we could certainly provide two faces for users' to change as they see fit.


WIP: Don't update 'shared' anymore

WIP: This still waits for some small API changes related to the groups

Closing, because this is now already fixed.


Eye dropper for the color picker widget

It would be nice if the color picker widget overlay showed an eye dropper.
There doesn't seem to be any cross platform eye dropper crates, I've tried to find the simplest way (that don't involve taking a screenshot and extracting one pixel from it) for each platform:

On linux, there's a a method for that in the Screenshot portal
On web, there's the EyeDropper api although it's not very well supported, currently just on desktop chromium browsers (it should also be possible to fallback to a color input which usually includes a color picker inside it)
On windows, I found this function
On mac, there doesn't seem to be a proper way to do it, the best I can find is using this method to get a 1 by 1 screenshot

These can probably live in a separate eyedropper crate.
The main problem I can think of is that picking a color from the screen is an inherently an asynchronous operation (will often be several seconds until the user clicks a color) and is actually an async fn for linux and web, and I'm not sure how that fits into a widget in iced...

I have no idea on how this would be implemented since this indeed would have some cross platform differences. So yes there would need to be some buffer crate for this. Maybe winit has something here? but cant rely on that crate fully just in case Iced starts to support others that don't.

I am closing this as Window would need to support this and none of them that I know of does atm for these events.


[Desktop][Mobile] persist login session after refresh

env: https://staging.fightpandemics.work/
Steps:

Log in to app
Go to Profile page (See you email, account data)
Refresh page
Login screen is shown. User is logged off.

Should add "keep me logged in" feature.

@galeza following your description of the issue
is it ok if I rename the title to "persist login session after refresh" or something like that?
then we can look at a more advanced "keep me logged in" feature  later which is not only session after refresh, but can keep a logged in user later, also depending on token expiration time etc.

Hi @robinv85
I'm changing the title of this issue to "[Desktop][Mobile] persist login session after refresh"
Should I add a new issue "[Desktop][Mobile] Add advanced "keep me logged in" feature" or sth like that? Just I want to keep track of this problem?

@galeza maybe we should ask to product / designers if it should be added on the login page

@robinv85
I'll ask product/designers.

@indirabhowmick @ppagarawal
Are we going to implement an advanced "keep me logged in" feature. Some application have checkbox:  keep me logged in?
If yes, I'll create a separate issue for that.

Hi @galeza I think an advanced "keep me logged in" feature is a great idea!  I personally think for now though we should add it to our Product backlog on Notion to be discussed further. I'm not sure adding it as an issue right now is the best strategy as we're all laser-focused on launching our MVP-Critical features.

@galeza @robinv85 @ppagarawal - I like the idea of having a persist login session after a refresh. Like @ppagarawal mentioned - we can add to the product backlog and prioritize for later release. Thanks so much!

But let's eitherway keep them logged for softlaunch. For a later release we give them the option.

@galeza @manualzuru @indirabhowmick @pthoma14 This will be resolved with #732.
With regards to keeping them logged in - I set it for a week expiry for now. We can either ask them to keep them logged in (after browser close) or just do it for them. Because this isn't an app where, for example, you aren't moving funds around like a banking app, the later may be better. GitHub, as one example, does this but but if you want to do something destructive (like deleting a repo) requests your password.

@joshmorel i think this should probably be aligned with token expiration time?
And we don’t have refresh tokens configured yet either.


Silent-refresh html page not loading on token expiration

Hi Team,
I am trying to use silent renew token on token expiration but silent-refresh html page is not calling/loading when token expired.
Below are the configuration implemented. Kindly let me know if anything is incorrect on below setup.
auth,ts
Office.initialize = function () {
     var settings = {
      authority: "https://xxxx.xxxxx.com/xxxx/v1",
      client_id: "https://xxx.xxx.com/",
      redirect_uri: "https://localhost:3000/taskpane.html",
      post_logout_redirect_uri: "https://localhost:3000/logout.html",
      revokeAccessTokenOnSignout: true,
      response_type: "id_token token",
      scope: "openid read:xxxx read:xxxx",
      state: true,
      filterProtocolClaims: true,
      loadUserInfo: true,
      nonce:true,
      clearHashAfterLogin: true,
      automaticSilentRenew: true,
      accessTokenExpiringNotificationTime: 4,
      silent_redirect_uri: 'https://localhost:3000/silent-refresh.html',
      monitorsession:true,
      metadata: {
        issuer: 'https://xxx.xxx.com/xxx/v1',
        authorization_endpoint:  "https://xxx.xxx.com/xxxxx/v1/connect/authorize"

    }
    };

    var mgr = new Oidc.UserManager(settings);
    mgr.signinRedirect();

   mgr.events.addAccessTokenExpiring(function(){
    console.log("token expiring...");
});
}

silent-refresh.html
<head>
    <title>RefreshToken</title>
    <meta http-equiv="Content-Security-Policy" content="upgrade-insecure-requests">
</head>
<body>
    <script type="text/javascript" src=https://cdnjs.cloudflare.com/ajax/libs/oidc-client/1.10.0/oidc-client.js></script>
<script>
    new Oidc.UserManager().signinSilentCallback().then((user)=>
    { consolse.log("silentrenewed");}
    )
        .catch((err) => {
            console.log(err);
        });

</script>
</body>

And why is it not working? What have you done to diagnose the problem?

Yes.   I also implemented below log but it is not logging....Can you confirm about the configuration setting once?
addAccessTokenExpiring also not fired...
var mgr = new Oidc.UserManager(settings);
Logger.logger=console;
Logger.Level=DEBUG;
Apart from adding silent_redirect_uri as redirect_uri in idsrv any other things to be configured?

Hi brockallen,
Now i re-written the code below and trying to trigger silent token manually with  automaticSilentRenew: false, so that it can trigger method  this.UserManager.events.addAccessTokenExpiring on which tried to call silentsignin but this method also not triggering and could not find any logs... Any issue with below settings?

import { UserManager, WebStorageStateStore, Log } from "oidc-client";
export default class AuthSigninService {
  UserManager;
  accessToken;

  constructor() {
    this.UserManager = new UserManager({
      authority: "https://xxxx.xxxx.com/oauth2/v1",
      client_id: "https://xxx.xxxx.com/",
      redirect_uri: "https://localhost:3000/taskpane.html",
      post_logout_redirect_uri: "https://localhost:3000/logout.html",
      revokeAccessTokenOnSignout: true,
      response_type: "id_token token",
      scope: "openid profile read:xxxx read:xxxx",
      includeIdTokenInSilentRenew: true,
      filterProtocolClaims: true,
      loadUserInfo: true,
      automaticSilentRenew: false,
    accessTokenExpiringNotificationTime: 4,   // Number of seconds before the token expires to trigger the `tokenExpiring` event
      silent_redirect_uri: "https://localhost:3000/silent-refresh.html",
      monitorSession:false,
      metadata: {
        issuer: 'https://xxxx.xxxx.com/oauth2/v1',
        authorization_endpoint:  "https://xxx.xxx.com/oauth2/v1/connect/authorize"

        // end_session_endpoint: "https://localhost:44339/oauth2/v1/connect/endsession"
    }
    });
    // Logger
    Log.logger = console;
    Log.level = Log.DEBUG;

    this.UserManager.events.addUserLoaded(user => {
      this.accessToken = user.access_token;
      localStorage.setItem("access_token", user.access_token);
      localStorage.setItem("id_token", user.id_token);
      this.setUserInfo({
        accessToken: this.accessToken,
        idToken: user.id_token
      });
       this.UserManager.events.addSilentRenewError(e => {
      console.log("silent renew error", e.message);
    });

    this.UserManager.events.addAccessTokenExpiring(() =>
            {
                this.UserManager.signinSilent({scope: "openid profile read:xxxxx read:xxxxx", response_type: "id_token token"})
                    .then((user) =>
                    {
                        console.log('First Expiring' + user); // This function just set the current user
                    })
                    .catch((error: Error) =>
                    {
                        this.UserManager.getUser()
                            .then((user) =>
                            {
                              console.log('Second Expiring' + user);
                            });
                    });
            });

    this.UserManager.events.addAccessTokenExpired(() => {
      console.log("token expired");
      this.signinSilent();
    });
  }
  signinRedirect = () => {
    localStorage.setItem("redirectUri", window.location.pathname);
    this.UserManager.signinRedirect({});
  };
  signinSilent = () => {
    this.UserManager.signinSilent()
      .then(user => {
        console.log("signed in", user);
      })
      .catch(err => {
        console.log(err);
      });
  };
  signinSilentCallback = () => {
    console.log("Token renew method");
    // this.UserManager.signinSilentCallback();
    this.UserManager.signinSilentCallback().then(user => {
      console.log("Silent User", user);
    });
  };

Hi.
Somewhat i was able to invoke signinSilent  on button click with that silent-refresh.html page called but still am checking why it is not firing automatically.
Anyhow when trying to invoke signinSilent  on button click it is throwing below exception in catch block.
metadata does not contain property jwks_uri
Does it mean any configuration to be implement on IDSRV side? Can you please confirm on this in details.
I referred the specs it saying those are signingKeys.

Hi brockallen,
As per recent logs is it required jwks_uri endpoint to silet renew token?

Hi Team,
I could able to fix the issue metadata by configuring standard jwk_uri.
Now I have come across another thing that once silent token generated what will be the expiration for the same and how it will be set?

Having same issue @brockallen any fixes ?

Having same issue @brockallen any fixes ?


Improve ignoreClass

How about adding a check for "ignoreClass"?
Now, if you use, for example, the plugin "chosen" in the elements of list, then all its elements also become movable.
Locally in jquery-sortable-lists.js (on mousedown event) I fixed it like this:
target.hasClass( setting.ignoreClass )
->
( target.hasClass( setting.ignoreClass ) || target.closest('.' + setting.ignoreClass).length )

It is little complicated. Cause target can have a lot of childs and it is too expensive to check all hierarchy for class name.
PS: target.hasClass is not necessary. closest() checks also the target element.


taylored param.h90 for visualization

Hello.
It's sorta kinda a feature request, but please consider, for the 'examples' folder, to taylor the param.h90 so that it fits what's in the dns.in for each example.
It is natural, that we the new users, will want to check out the examples and then take a look in Paraview, but w/o the aforementioned paratemters matching, that's not trivial.  I enclose the dns.in I used, long with the param.h90 I made for it (in the end, it did not work).

Thanks for the feedback!
We could settle this issue swiftly by porting the visualization workflow in CaNS to here, to replace these old routines.
Leaving here the links for future reference: 1, 2.

Hello thank you very much I will test it ASAP

Hi,
#3 merged to the code the approach for visualizing field data mentioned in https://github.com/Multiphysics-Flow-Solvers/FluTAS/issues/2#issuecomment-948772713.
It's quite easy to use it (one needs python3 and numpy):

copy the file under utils/visualize_fields/gen_xdmf_easy/write_xdmf.py to the data folder
run the XDMF generator python write_xdmf.py and hit enter three times to enter the default values.
done! paraview viewfld_DNS.xmf will allow you to visualize the field.

This approach has several advantages to the old method:

The physical time is accurately preserved in the metadata (.xdf) file
Vector fields are automatically detected and lumped together
Much simpler workflow w/ no need to input by hand the computational setup
... and still one single .xdf file per time series, per geometry

More info in the revised INFO_VISU.md.
PS: to be able to visualize the data, you'd need to re-do the simulations with the new main branch.

This seems settled, so I'll go ahead and close this. But please feel free to reopen if you experience issues. Thanks, @rabarbar42 @nscapin!


PDF generation is a bit broken...

let's fix that...

the pdf module is a bit picky abotu remote URLs for images:

dynamic URLs (?param=value) break the build
static URLs are not fetches (because of security reasons?)

So, let's skip all the shield buttons for PDF generation (http://img.shields.io/github/issues/aim42/htmlsanitycheck.svg)

the pdf module is not able to find the generated plantUML diagrams.
As a workaround, all diagram names should be prefixed with {plantumldir}:
[plantuml,"{plantumldir}whitbox-hsc-level-1",png]
....
include::../plantuml/whitebox-hsc-level-1.puml[]
....


docs: docs for aa-ethers (wip)

Current dependencies on/for this PR:

main

PR #47   👈

This comment was auto-generated by Graphite.


[firebase_admob] iOS 14 App Tracking Transparency authorization

According to google's document
https://developers.google.com/admob/ios/ios14
Starting in iOS 14, IDFA will be unavailable until an app calls the App Tracking Transparency framework to present the app-tracking authorization request to the end user. If an app does not present this request, the IDFA will automatically be zeroed out which may lead to a significant loss in ad revenue.
It would be helpful if firebase_admob can add a function to request tracking.
import AppTrackingTransparency
import AdSupport
...
func requestIDFA() {
  ATTrackingManager.requestTrackingAuthorization(completionHandler: { status in
    // Tracking authorization completed. Start loading ads here.
    // loadAd()
  })
}

Hi @tonypottera24
This is resolved in https://github.com/FirebaseExtended/flutterfire/issues/3236 and merged https://github.com/FirebaseExtended/flutterfire/issues/3236#issuecomment-682079621
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


Uploading file along with other question

Are you requesting a feature, reporting a bug or ask a question?
feature
What is the current behavior?
Currently, the file upload is a type of question. However, it is generally a supporting document for an answer
What is the expected behavior?
We should allow file upload to any type of question. E.g. we can have radio button, text and checkbox type of question but these should allow file upload as well along with answering the question. This would be similar to HTML type,which can be attached to any question.
How would you reproduce the current behavior (if this is a bug)?
NA
Provide the test code and the tested page URL (if applicable)
Tested page URL:
Test code
your_code_here

Specify your

browser:
browser version:
surveyjs platform (angular or react or jquery or knockout or vue): angular
surveyjs version:

@deepakjain2502, hello! What about using any question with the file question but with the same name? May be this workaround is ok for you?

Hi,
Thanks for response,
No, that might not work, it would be confusing for user, as question numbering would be different. Also the backend data would be saved without relation and we might not use it for backend processing.

@deepakjain2502 may be it would be something like common "alternative question" feature? Just possibility to add alternative question of any type to question of any type. What do you think?

You mean a child question? thats a bit of workaround but not the right solution. As file is linked with question and not with a response per say

@deepakjain2502 not worlaround but new feature. I think it could be more common solution than just file

Not sure how are you perceiving the "alternative question", but if we look into survey, its always question answer and sometime a proof to substantiate the answer we are providing. The answer could be in the form of attachment (document) or a "data form". Data form is just a form with data which can be attached to a question

Hello, do you think anyway of doing this? Or any plan to add these features?

@deepakjain2502 Do be honest I do not think we will implement it in the near future. As I see it, you have a pretty unique scenario.
However, you may use the afterQuestionRender event or custom widget to implement this functionality.
I think we will be able to create an example on the next week, if you have a problem to do it by yourself. Just ping us in the middle next week. It will be cool if you show us UI you want to implement.
Thank you,
Andrew

Hi Andrew,
I could not understand the implementation, if you can please guide with some examples? I do not have a UI in mind, but if we have a question and on right side of question an icon to upload a file.
Thanks,
Deepak Jain | Subtlelabs Technologies Pvt. Ltd | Phone: +91-9818542953 | Email<EMAIL_ADDRESS>|  http://www.subtlelabs.com www.subtlelabs.com
logoFinal_RGB(web&video_purposes).png
From: Andrew<EMAIL_ADDRESS>Sent: 08 September 2017 13:56
To: surveyjs/surveyjs
Cc: deepakjain2502; Mention
Subject: Re: [surveyjs/surveyjs] Uploading file along with other question (#634)
@deepakjain2502 https://github.com/deepakjain2502  Do be honest I do not think we will implement it in the near future. As I see it, you have a pretty unique scenario.
However, you may use the afterQuestionRender event or custom widget to implement this functionality.
I think we will be able to create an example on the next week, if you have a problem to do it by yourself. Just ping us in the middle next week. It will be cool if you show us UI you want to implement.
Thank you,
Andrew
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/surveyjs/surveyjs/issues/634#issuecomment-328037557 , or mute the thread https://github.com/notifications/unsubscribe-auth/AFR9XdzmY9i6mfZCPsW_1A3Wrkc4DTsFks5sgPoHgaJpZM4PPeKX .Image removed by sender.

This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus

@deepakjain2502 Here is the example: https://plnkr.co/edit/xsdl0DoNRIaGGvWluRhQ?p=preview
I may add the property, showTitle (true by default). So you will be able to hide the title of the question. In you case for the file type question.
Let me know is it any close to what you have in your mind?
Thank you,
Andrew


for 'clone' views created when twin strategy=clone, schema will not be created if needed

Maybe the twin strategy feature should be restricted to allow/prevent, and the clone view creation left to the user.

Thanks @fabrice-etanchaud ! We will look into this for 1.3.1

Hi @fabrice-etanchaud! Restricting twin_strategy to only allow/prevent is definitely a viable option. Another possible option I'm investigating is manually triggering create_schema if 'clone' is specified as the strategy. As always, your feedback is always appreciated.


Introduce a data-objects package into experimental namespace

This package will export custom data-objects like our key-value pair and other future objects. Right now kvpair is simply part of this package, but we could consider breaking them up in the future.
FluidStatic is also part of this PR, but that class will probably change a bit as we iron out the API, and might move to it's own package as well.

@chumpchief maybe i should remove FluidStatic from this PR. it depends on get-tinylicious-container, which makes layercheck complicated.

@chumpchief maybe i should remove FluidStatic from this PR. it depends on get-tinylicious-container, which makes layercheck complicated.

Interesting w.r.t. get-tinylicious-container. Could you help me understand the issue with layercheck?

Interesting w.r.t. get-tinylicious-container. Could you help me understand the issue with layercheck?

layercheck isn't expecting to find tinylicious in the framework as a dependency, only a dev-dependency. So either we have to move this AND the top level export out of 'framework' or we need to omit any dependencies on server code

layercheck isn't expecting to find tinylicious in the framework as a dependency, only a dev-dependency. So either we have to move this AND the top level export out of 'framework' or we need to omit any dependencies on server code


Control barriers deadlock

Consider this test
{
x=1;
P0:r0=0;
}
 P0@cta 0,gpu 0 | P1@cta 0,gpu 0 ;
 bar.cta.sync 0 | bar.cta.sync 1 ;
 bar.cta.sync 1 | bar.cta.sync 0 ;
 ld.weak r0, x  |                ;
~exists
 (P0:r0 == 1)

The outcome should not be observable because there is a deadlock due to the circular dependency of barriers.
However, unless I am missing something, the model allows this behavior.
I think this problem can be avoided by adding the following axiom to the model (*)
let dl = (po; sync_barrier; po) & sync_barrier
empty dl as axiom-NoDeadlock

However, this is not enough to handle bar.arrive. This variant should not deadlock because arrive is non-blocking
{
x=1;
P0:r0=0;
}
 P0@cta 0,gpu 0 | P1@cta 0,gpu 0   ;
 bar.cta.sync 0 | bar.cta.arrive 1 ;
 bar.cta.sync 1 | bar.cta.sync 0   ;
 ld.weak r0, x  |                  ;
~exists
 (P0:r0 == 1)

The axioms can be fixed (assuming bar.cta.arrive is tagged as ARRIVE) as follows
let dl = ([~ARRIVE] ; po; sync_barrier; po) & sync_barrier
empty dl as axiom-NoDeadlock

Any thoughts?
(*) I think that it would be better to flag executions where this axiom is violated, rather than silently making the program have zero consistent executions. I know this is possible in .cat, but not sure if alloy allows this.

Meanwhile, although I agree for example that the first test is clearly a deadlock, I don't currently expect the memory model rules to adjudicate that.

This is why I said I would prefer to flag these executions rather than simple filtering them by the consistency axioms.
However, consider this test having the same kind of deadlock
{
x=0;
P0:r0=0
}
 P0@cta 0,gpu 0 | P1@cta 0,gpu 0 ;
 ld.weak r0, x  | bar.cta.sync 1 ;
 bar.cta.sync 0 | st.weak x, 1   ;
 bar.cta.sync 1 | bar.cta.sync 0 ;
~exists
 (P0:r0 == 1)

The current model filters out this execution due to a rf ; cause cycle.

I do think having some kind of forward progress/deadlock model eventually would be interesting. We don't yet know what the axioms should be, or even how tightly integrated it would want to be with the memory model...that's a research project by itself.
But it's something we're starting to think about internally, so stay tuned :) Is forward progress/deadlock avoidance something you are actively working on, or just something that you hit along the way while studying the memory model?

As mentioned in the last bullet point here, we can already find some liveness bugs. We inherited this from the liveness checking for CPU/PL models based on this paper.
However, it has several limitations for GPU models. One of which is that we assume some sort of progress which GPU specifications normally do not commit to. I have some ideas on how to integrate the progress models from this paper in a way which is hopefully not very intrusive to the memory model.


Support multiple results in custom queries

This adds multiple results supports to custom queries. This allows having
several tags for a single metric.

doc wise it's good to go :)


[BugFix] Fix report errors by time_slice

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes https://github.com/StarRocks/starrocks/issues/13059
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 6 / 6 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/ExpressionAnalyzer.java
6
6
100.00%
[]

Can you please explain why the same code swapped places solves the problem.
If it is a bug, please add UT coverage appropriately

[FE PR Coverage Check]
:heart_eyes: pass : 6 / 6 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/ExpressionAnalyzer.java
6
6
100.00%
[]


Check inherited members for __init__ when merging docstrings

Is your feature request related to a problem? Please describe.
Currently the Python handler only checks in declared members for an __init__ method when merging docstrings together. That means it won't show the docstring of an inherited __init__.
Describe the solution you'd like
Also check inherited members.
Describe alternatives you've considered
/
Additional context
https://github.com/mkdocstrings/griffe/issues/205

As example of this issue is provided in this comment.


[SPARK-4989][CORE] backport for branch-1.0(-jdbc) catch eventlog exception for wrong eventlog conf

JIRA is SPARK-4989

Test build #25307 has started for   PR 3971 at commit 22d2833.

This patch merges cleanly.

Test build #25307 has finished for   PR 3971 at commit 22d2833.

This patch fails some tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25307/
Test FAILed.

This is failing due to a known issue in branch-1.0. I will spend some time to investigate the failure shortly.

Jenkins, retest this please.

Test build #27033 has started for   PR 3971 at commit 22d2833.

This patch merges cleanly.

Test build #27033 has finished for   PR 3971 at commit 22d2833.

This patch fails some tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27033/
Test FAILed.

Jenkins, retest this please.

Test build #28121 has started for   PR 3971 at commit 22d2833.

This patch merges cleanly.

Test build #28121 has finished for   PR 3971 at commit 22d2833.

This patch fails some tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28121/
Test FAILed.

Heh, are we back-porting to 1.0.x anyway at this point? I think it has some known problems with tests. If not, we could just close all this out. But I'm interested in opinions on whether there would ever be a new 1.0.x release.

@srowen we should still back port to branch-1.0. The reason why these PRs haven't been merged is because the context cleaner suite hasn't been passing and no one has really looked at it yet. I closed my other UI port PR by mistake when I was deleting my github branches, but before I reopen it I would like to fix the test first (though unfortunately that is of lower priority than fixing flaky tests in master branch).

I think we should close this at this point. With 1.4 about to come out, I don't see any expectation of new releases from 1.0.x or even probably 1.1.x. It's back-ported to 1.1.x

Ok, now that 1.4 is basically released, @liyezhang556520 would you mind closing this?


file is not storing as .csv

According to the documentation, the file should store as table.snapshot.csv but it is storing as table.csv.snapshot

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

ID: 57f28e6c-a22d-484b-6599-9b17e6c011f0
Version Independent ID: 307cb997-5d54-a43c-f278-5247769e682b
Content: Configuring dataflow storage to use Azure Data Lake Gen 2 - Power BI
Content Source: powerbi-docs/transform-model/dataflows/dataflows-azure-data-lake-storage-integration.md
Service: powerbi
Sub-service: pbi-dataflows
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @tahossain42. This issue has been assigned to the doc author for review.

Thanks for your input, we'll look at that update in the article.


ThreadingViolationError

seen on api4 server:

The original error was in fact "ChunkedEncodingError". Given the next update of Addrindexrs I think it is fair to close the ticket and reopen them if this happens again without "ChunkedEncodingError".


extendr-bindgen CLI

This is just an idea without thinking much.
At last, I figured out how to R_UnwindProtect() properly. Since it requires to implement C wrapper functions, I found I need a functionality to generate C code. But, extendr can generate wrapper function code (currently, only R code) only after compiling the crate for the package, which usually happens on building the R package, so it's too late to generate C code.
After some struggle, I came up with this idea of creating CLI to parse the Rust code and generate wrapper functions. The usage is like this:
$ extendr-bindgen help
Generate C bindings and R bindings for a Rust library

Usage: extendr-bindgen.exe [COMMAND]

Commands:
  c-header  Generate C header file
  c-impl    Generate C implementation for init.c
  r-impl    Generate R wrapper functions
  help      Print this message or the help of the given subcommand(s)

Options:
  -h, --help  Print help

$ extendr-bindgen c-impl ..\R-package\src\rust\src\lib.rs

#include <stdint.h>
#include <Rinternals.h>
#include "rust/api.h"

...snip...

SEXP unextendr_to_upper_wrapper(SEXP x) {
    SEXP res = unextendr_to_upper(x);
    return handle_result(res);
}

...snip...

static const R_CallMethodDef CallEntries[] = {
    {"unextendr_to_upper", (DL_FUNC) &unextendr_to_upper_wrapper, 1},

   ...snip...

    {NULL, NULL, 0}
};

void R_init_unextendr(DllInfo *dll) {
  R_registerRoutines(dll, NULL, CallEntries, NULL, NULL);
  R_useDynamicSymbols(dll, FALSE);
}

I think there are pros and cons in this approach.
Pros:

It can be used for generating C code (my original motivation)
We can test the generated wrapper code without launching an R session or writing an R package
Users can update wrapper code without compiling their Rust code

Cons:

The package author has to install the CLI (I think the binary can be provided, so this shouldn't be a big problem)
Since it just statically analyze the Rust code, something might not work as expected. Especially, macros might be expanded by cargo expand beforehand. This might cause a lot of troubles.

This would require a lot of effort to migrate to this, so I don't think I can address this in the near future, but let me file this for sharing. Here's some more resources in case you are interested:

My blog post about R_UnwindProtect()
My example implementation

Using a CLI tool written in Rust to use extendr may make sense also for some issues like extendr/rextendr#284.

I've some scattered thoughts.

You can compile code in build.rs. The R CMD config provides you with all the compiler flags you'd need.
The cc crate takes care of most of the configuration things. I believe you can then even run bindgen on the generated code and everything works.

If you want an except, I can find one that I put together. I fear that you already know this.

I have been tinkering with an extendr-xtask if you will, i.e. a CLI for developer tasks. But I've not accumulated enough
tasks to warrant this yet.

I think an extendr-cli is a very good idea, and especially when we are in the situation that we
have to play nicely with R, makefiles, and other crap that most rust crates simply ignore.


Upgrade Terraform provider versions

Context
Currently, we are using the following Terraform provider versions:

Provider
Current version
Current version release date
Latest version
Latest version release date

digitalocean/digitalocean
2.29.0
2023-07-17
2.38.0
2024-05-03

gavinbunney/kubectl
1.7.0
2020-09-18
1.14.0
2022-03-24

hashicorp/aws
5.33.0
2024-01-18
5.48.0
2024-05-02

hashicorp/azurerm
3.97.1
2024-03-22
3.102.0
2024-05-02

hashicorp/google
4.8.0
2023-08-28
5.28.0
2024-05-06

hashicorp/helm
2.1.2
2021-04-27
2.13.1
2024-04-15

hashicorp/kubernetes
2.20.0
2023-04-20
2.29.0
2024-04-11

kreuzwerker/docker
2.16.0
2022-01-24
3.0.2
2024-03-17

mrparkers/keycloak
3.7.0
2022-02-02
4.4.0
2024-01-08

tehcyx/kind
0.4.0
2024-02-15
0.4.0
2024-02-15

Value and/or benefit
While we might not need to have the latest versions for all the providers, we probably don't want to stay too far behind either.
Anything else?
No response

It might be nice to compare dates rather than version numbers.  Maybe we make a goal/policy to try to stay up to date within the last year for example.

@Adam-D-Lewis I updated the issue with information about the release dates of both versions. I agree we should have some kind of policy in place. As you can see, there are some that are a couple years behind.
Also, https://github.com/gavinbunney/terraform-provider-kubectl has not seen any activity in the last two years.

Also, https://github.com/gavinbunney/terraform-provider-kubectl has not seen any activity in the last two years

We should filter out where that's being used and remove if we can find a better way. This might be been used by the kind provider for authentication, though it might be good to double check if this is really needed (maybe a newer version of the kind provider does not require this anymore)