TODO

Meta

[ ] Documentation

[x] README
[x] Examples
[ ] RDoc/YARD

[x] Tests 😭

[x] Helpers
[x] Utility Classes
[x] Middleware
[x] Integration

[x] Feature comparisons (e.g. cf. this)
[ ] Performance and memory analyses
[ ] Swagger/schema.org/?? catalog/discovery integration

[x] Support HEAD requests for method discovery and CORS

Features

[x] Pass-through JSONAPI::Serializer options

[x] Class level, root and defaults (e.g. set :jsonapi_serializer_options, :meta => {})

[x] Inheritable?

[x] In return results from actions (e.g. Post[id], :include=>:author)
[x] Punch through to relationships

[ ] JSON:API states that the server must return 400 if the client passes unhandled/able query parameters. What's a nice DSL to express this?
[x] Logging

[x] Configurable progname
[x] Add logging where appropriate
[x] Audit halt/fail/raise codes and messages

[ ] More configurables in general!

[ ] Strictness level
[x] Error messages

[x] Control which relationships get passed through (or don't) on create/update

[x] Enhance graft/merge to take a :passthru option that can be :create, :update, or both?

[x] Handle exceptions that should be 409
[x] Handle exceptions that should be 422
[x] Handle exceptions that should be 404

Un-features

[x] There has to be a better way to delegate requests to relationship routes and bypass the after filters on the response than this

Some middleware that intercepts failed requests with a certain status code and delegates to an object/class wrapped in env?
Sinatra::Namespace? To the rescue!

[x] Keeping e.g. action_roles, action_conflicts, and relationships on the registering app's settings (class) is kind of janky. Can we do a better job of hiding this storage from the registering app? It's still on settings but now at least it's tucked into a Config object.
[x] Defer extending Sinatra DSL, parameterize ACTIONS into a method? It's just clumsy and weird. Well, that's a little better.

[x] Don't load unnecessary relationship helps (e.g. has-many actions on has-one routes)

[x] It's possible that requiring show for update, destroy, and relationships is somewhat misguided. I can see a situation where a user has access to e.g. delete something without being able to view it. Maybe we need to abstract out a "finder" helper (or unrestricted action helper) that is used by show, update, etc.?

Known Bugs

[x] Relationship dispatch on POST and PATCH is completely untested and probably buggy

[x] Null foreign keys and order of operations issue

[x] Can't have more than one action helper of the same name per resource block :confused:
[x] Version dependencies in gemspec need to be relaxed. Can we make this extension compatible with Sinatra 1.x?

Good enough for v1!


[App Service] sample web app deployed automatically from GitHub

Thank you for your contribution and feedback! Help us review faster by providing the following information:
This issue is a: (mark with an x)

[ ] bug report
[x] request for new sample
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Steps to reproduce:
Errors and log messages:
Expected behavior:
Provide sample about how to use source control as deployment option for web app.
OS and Go versions:
Further info:

see #216 and #215


security admin role for policy assignment?

[Enter feedback here]
Based on my test, the Security Admin role doesn't allow you to assign policy; rather you will need to have the User Access Admin instead.

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

ID: 98967f78-19ff-360f-db07-ad9900f59105
Version Independent ID: ba435b7b-7cc9-11de-ad2a-20a837221957
Content: User roles and permissions - Microsoft Defender for Cloud
Content Source: articles/defender-for-cloud/permissions.md
Service: defender-for-cloud
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

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

#label:"backlog-item-created"

#reassign:dcurwin

@billtr007
Thank you for bringing this to our attention.
I've delegated this to content author @dcurwin, who will review it and offer their insightful opinions.

Work Item opened:  https://dev.azure.com/msft-skilling/Content/_workitems/edit/185238

#reassign:alizabernstein

billtr007 The security admin role, as outlined in the article, appears to provide adequate permissions for policy assignment. You can ask about your issue in the Azure policy Q&A

#please-close


chown init-container should not render without secret

Insecurity warning

[!CAUTION]
This would only apply to insecure setups as every secure setup should at least have some secret somewhere…

Problem
When investigating #177 I discovered, that even if it is not needed for logical reasons, the chown container always runs.
https://github.com/zitadel/zitadel-charts/blob/bcfdd4f8306de34cb7869f0d7204fdb97a9d54f3/charts/zitadel/templates/deployment.yaml#L158
That means, even when no secret is present to be chown-ed, the container still tries to change its ownership (of the, I assume, empty volume).
It bothers me aesthetically but in our prototyping, which we did for #177 it basically blocked us. We have to currently delete the init conatiner after every deployment.
Proposals
Fix #177 to reduce impact
Fix #177 and the problem is not so grave anymore
Render only if secrets present
Render the container only into the chart if secrets are present, maybe like so:
{{ $hasInitContainer = if or (.Values.zitadel.secretConfig .Values.zitadel.configSecretName. Values.zitadel.dbSslCaCrt .Values.zitadel.dbSslCaCrtSecret .Values.zitadel.dbSslUserCrtSecret) }}

and then omit the whole container if false.

Alright, we can no longer wait. I will fork the chart and come back with a PR if we ever make it work 🤣

@unique-dominik do you already have something that I could take over to finish?

Hi @eliobischof
Yes I do, but its a workaround for now since I just disable the container if its not used 😢
      initContainers:
        {{- $isConfigPresent := and
          .Values.zitadel.secretConfig
          .Values.zitadel.configSecretName
          .Values.zitadel.dbSslCaCrt
          .Values.zitadel.dbSslCaCrtSecret
          .Values.zitadel.dbSslUserCrtSecret
        }}
        {{- if $isConfigPresent }}
        - args:
          - "{{ include "zitadel.joincpcommands" (dict "commands" (list
              (include "zitadel.makecpcommand" (dict "value" .Values.zitadel.secretConfig "path" "/zitadel-secrets-yaml/" ))
              (include "zitadel.makecpcommand" (dict "value" .Values.zitadel.configSecretName "path" "/zitadel-secret-config-yaml/" ))
              (include "zitadel.makecpcommand" (dict "value" (or .Values.zitadel.dbSslCaCrt .Values.zitadel.dbSslCaCrtSecret) "path" "/db-ssl-ca-crt/" ))
              (include "zitadel.makecpcommand" (dict "value" .Values.zitadel.dbSslUserCrtSecret "path" "/db-ssl-user-crt/" ))
              )) }} chown -R 1000:1000 /chowned-secrets/ && find /chowned-secrets/ -type f -exec chmod 400 -- {} + "
          command:
          - sh
          - -c
          image: "{{ .Values.chownImage.repository }}:{{ .Values.chownImage.tag }}"
          imagePullPolicy: {{ .Values.chownImage.pullPolicy }}
          name: chown
          volumeMounts:
          - name: chowned-secrets
            mountPath: /chowned-secrets
          {{- if .Values.zitadel.secretConfig }}
          - name: zitadel-secrets-yaml
            mountPath: /zitadel-secrets-yaml
          {{- end }}
          {{- if .Values.zitadel.configSecretName }}
          - name: zitadel-secret-config-yaml
            mountPath: /zitadel-secret-config-yaml
          {{- end }}
          {{- if (or .Values.zitadel.dbSslCaCrt .Values.zitadel.dbSslCaCrtSecret) }}
          - name: db-ssl-ca-crt
            mountPath: /db-ssl-ca-crt
          {{- end }}
          {{- if .Values.zitadel.dbSslUserCrtSecret }}
          - name: db-ssl-user-crt
            mountPath: /db-ssl-user-crt
          {{- end }}
          securityContext:
            runAsNonRoot: false
            runAsUser: 0 # ! FIXME this is what currently breaks
        {{- end }}

Hi @eliobischof
Yes I do, but its a workaround for now since I just disable the container if its not used 😢
      initContainers:
        {{- $isConfigPresent := and
          .Values.zitadel.secretConfig
          .Values.zitadel.configSecretName
          .Values.zitadel.dbSslCaCrt
          .Values.zitadel.dbSslCaCrtSecret
          .Values.zitadel.dbSslUserCrtSecret
        }}
        {{- if $isConfigPresent }}
        - args:
          - "{{ include "zitadel.joincpcommands" (dict "commands" (list
              (include "zitadel.makecpcommand" (dict "value" .Values.zitadel.secretConfig "path" "/zitadel-secrets-yaml/" ))
              (include "zitadel.makecpcommand" (dict "value" .Values.zitadel.configSecretName "path" "/zitadel-secret-config-yaml/" ))
              (include "zitadel.makecpcommand" (dict "value" (or .Values.zitadel.dbSslCaCrt .Values.zitadel.dbSslCaCrtSecret) "path" "/db-ssl-ca-crt/" ))
              (include "zitadel.makecpcommand" (dict "value" .Values.zitadel.dbSslUserCrtSecret "path" "/db-ssl-user-crt/" ))
              )) }} chown -R 1000:1000 /chowned-secrets/ && find /chowned-secrets/ -type f -exec chmod 400 -- {} + "
          command:
          - sh
          - -c
          image: "{{ .Values.chownImage.repository }}:{{ .Values.chownImage.tag }}"
          imagePullPolicy: {{ .Values.chownImage.pullPolicy }}
          name: chown
          volumeMounts:
          - name: chowned-secrets
            mountPath: /chowned-secrets
          {{- if .Values.zitadel.secretConfig }}
          - name: zitadel-secrets-yaml
            mountPath: /zitadel-secrets-yaml
          {{- end }}
          {{- if .Values.zitadel.configSecretName }}
          - name: zitadel-secret-config-yaml
            mountPath: /zitadel-secret-config-yaml
          {{- end }}
          {{- if (or .Values.zitadel.dbSslCaCrt .Values.zitadel.dbSslCaCrtSecret) }}
          - name: db-ssl-ca-crt
            mountPath: /db-ssl-ca-crt
          {{- end }}
          {{- if .Values.zitadel.dbSslUserCrtSecret }}
          - name: db-ssl-user-crt
            mountPath: /db-ssl-user-crt
          {{- end }}
          securityContext:
            runAsNonRoot: false
            runAsUser: 0 # ! FIXME this is what currently breaks
        {{- end }}

This doesn't look unreasonable to me 🙂 I think just not rendering the init container makes sense if it's not needed. I'll have a look at the PR, thanks 🙏

👍
I try to come up with PR's (that you did not do yet) after paternity leave 👶
The filesystem PR (if the container is needed) is more relevant I'd say as only then you get the whole impact of the rendered container.


how to get ndt localization record if i am not in us

i want to run ndt_localization_module on my local computer, and i follow this file /ApolloAuto/apollo/tree/master/docs/howto)/how_to_run_NDT_localization_module_on_your_local_computer.md
but i can't get the localization data

The URL in this file has been taken offline and the download of this data is no longer available.


Add link to infaction message in infraction log

A message link to where an infraction was issued could provide better context regarding an infraction. This feature has been suggested multiple times in #mod-meta and #mod-tools.
If the infraction was issued in a modmail category, the infraction log field for this feature will instead display "Infraction invoked in ModMail channel" or something similar.  In future, the modmail log link for the thead can be displayed.
This feature will also require PR to the site repo since a new field for the jump url has to be added to the infraction model.
(Zig has approved this issue in a private channel).

Just to be clear, this is how the "infraction log" of a user currently look like:


Return 0 on success in getvcp/setvcp

The getVcp and setVcp methods return the boolean results of the underlying API calls, which causes the program to exit 1 on success and 0 on failure. Typically 0 indicates success, so here's a small change to return the appropriate exit codes.
Due to the encoding of the .cpp file, git seems to think it's binary, so unfortunately the diff doesn't appear correctly. I've reproduced the diff below for easier review:
diff --git a/winddcutil/winddcutil.cpp b/winddcutil/winddcutil.cpp
index 6042067..9bc33ec 100644
--- a/winddcutil/winddcutil.cpp
+++ b/winddcutil/winddcutil.cpp
@@ -121,14 +121,14 @@ int getVcp(std::vector<std::string> args) {
        bool success = GetVCPFeatureAndVCPFeatureReply(physicalMonitorHandle, vcpCode, NULL, &currentValue, NULL);
        if (!success) {
                std::cerr << "Failed to get the vcp code value" << std::endl;
-               return success;
+               return 1;^M
        }

        std::stringstream ss;
        ss << std::hex << currentValue;
        std::cout << "VCP " << args[1] << " " << ss.str() << std::endl;

-       return success;
+       return 0;^M
 }

 int setVcp(std::vector<std::string> args) {
@@ -156,9 +156,11 @@ int setVcp(std::vector<std::string> args) {
        }

        bool success = SetVCPFeature(physicalMonitors[displayId].hPhysicalMonitor, vcpCode, newValue);
-       if (!success)
+       if (!success) {^M
                std::cerr << "Failed to set vcp feature" << std::endl;
-       return success;
+               return 1;^M
+       }^M
+       return 0;^M
 }

 std::unordered_map<std::string, std::function<int(std::vector<std::string>)>> commands

Looks good, thanks for the improvement!

I rebased against your latest commit 961d857ece2faba1b59ba3faa412f0f45ae51fe0 to resolve the merge conflict.


fix: Support for TS 4.5

Have you checked that after this merge pact-gen-ts can be used  in apps with older versions of typescript?

Have you checked that after this merge pact-gen-ts can be used in apps with older versions of typescript?

No, it won't work with TS 4.2. Ts-morph v13 supports TS 4.5, and as this project is built on ts-morph, we have to follow their support of TypeScript. If someone uses older version of TS, he should stay on older version of pact-gen-ts.

For me, looks nice.
But what do you think about this crazy idea:
Install different versions of ts-morph using npm/yarn aliases:
https://stackoverflow.com/a/56495651/8850253
and then use a specific version of tsMorph depending on the installed version of TypeScript (in the user project).
It would look something like that:
import * as tsMorph12 from 'tsMorph12';
import * as tsMorph13 from 'tsMorph13';

export const tsMorph = typescriptVersion >= 4.5 ?
        {...tsMorph13, getComment: (jsDocTag) => jsDocTag.getCommentText()}
        : {...tsMorph12, getComment: (jsDocTag) => jsDocTag.getComment()};

Probably, it is the most stupid idea which I have made up, but maybe ? :P

Should we add annotation about TypeScript support in the README.md file ?

For me, looks nice.
But what do you think about this crazy idea: Install different versions of ts-morph using npm/yarn aliases: https://stackoverflow.com/a/56495651/8850253 and then use a specific version of tsMorph depending on the installed version of TypeScript (in the user project). It would look something like that:
import * as tsMorph12 from 'tsMorph12';
import * as tsMorph13 from 'tsMorph13';

export const tsMorph = typescriptVersion >= 4.5 ?
        {...tsMorph13, getComment: (jsDocTag) => jsDocTag.getCommentText()}
        : {...tsMorph12, getComment: (jsDocTag) => jsDocTag.getComment()};

Probably, it is the most stupid idea which I have made up, but maybe ? :P

This only works on the project they are used on, if we want to use them to provide support for projects using pact-gen-ts, we can't really rely on npm aliases

@crpc Regarding the "crazy idea" :) - ts-morph maintainers had the same concept implemented for typescript. Looking at the comments here: https://github.com/dsherret/ts-morph/issues/144 and here https://github.com/dsherret/ts-morph/issues/1231 - it was a maintenance nightmare. So for now I guess we have to align with typescript releases and breaking changes.


angucomplete-alt on modal windows

Good morning, sorry for my orrible english.
I'm trying angucomplete-alt and it seems really a good tool.
I'm writing an application with angular where I have a page with a table with a list of data and clicking the "edit" button opens a modal window where I can edit the field.
thanks to the example 9 can not upload in angucomplete the figure for the first array. But charging other arrays (rows in the table) as the initial object I always find the first array and not the updated one. I should initialize angucomplete whenever it changes the array, but I do not know how.
what 's the best solution to solve my problem?

You should be able to update the array.
http://plnkr.co/edit/i3iQcg?p=preview

If you need me to dig deeper, just open again.


A Disable option for the shortcut key?

neocotic, it would be great if the Shortcut field had a "Disable" checkbox at its right.
I have a lot of Option+Shift+Letter shortcuts  to bookmarks and these conflict with some shortcuts in Template.
I think it makes sense for a lot of users to just use shortcuts for the most common Template actions and use the toolbar drop-down menu for the rest.
Mmm, now that I think of it (doh!), can I just remove the letter from the Shortcut field and the shortcut will be disabled?  Does your code expect this?
Thanks for a great extension.

Yes it does ;)
All that happens when you press a hotkey while holding the modifier (e.g. Option+Shift) is that it tries to find an enabled template with that key assigned to it. If no template can be found, or that template is disabled, it simply does nothing.
If you feel it would help others I can add a note on this to the help popup for the Shortcut field?
I tried to use a modifier that was less popular but it appears Option+Shift is more commonly used than I thought. Are any of the predefined templates (e.g. URL, Anchor) using common shortcuts?

If you feel it would help others I can add a note on this to the help popup for the Shortcut field?

Yes, definitely.

Are any of the predefined templates (e.g. URL, Anchor) using common shortcuts?

No, not that I'm aware of.  In my case, I use Option shortcuts on the Mac to trigger Keyboard Maestro macros and stuff like that.

Keep the suggestions etc. coming though as I love to hear user feedback.

I will.  :)

Very little feedback makes it way back to me.

I could be wrong, but I think that has to do more with popularity -- Template needs to be more well-known among "power users".  Maybe you could write an article about it for Lifehacker and other such blogs. Probably just a few of the most well-known blogs would be enough to start the ball rolling and the most effective use of your time.

Just thought I'd add this:  regarding popularity, the forthcoming "Copy selection in Markdown" feature is probably what will make power users really rave about Template.

I'm not too bothered about popularity as I originally made this extension (then called URL Copy) to simply copy the URL system clipboard without formatting and it just evolved from there as I experimented. Template has been a great chance to develop my JavaScript skills and now, CoffeeScript, as well as create useful universal libraries and frameworks for users
When I first released the extension as Template with most of the features you know now it was stuck around 200 users. When Google redesigned the Web Store this boomed to 1.5k and steadily grew to almost 2k. After almost a year of evolution I finally thought it was ready for a full release and put out version 1.0.0 which saw my user base fall to 700 users. I guessed it would fall as I added a new permission (geographic location) but that was worse than I thought. Fortunately, it's slowly beginning to grow again and is now sitting at around 830 users.
Well, that's the history of Template ;) Anyway, I've put a lot of time in to Template and obviously I'd like to see it do well. That said; I don't really have time to generate publicity for it and I'm not really one to toot my own horn.
Now to get looking at markdown ;)

Closing this off as I've added an explanatory note to the help popup of the shortcut field for templates. This will be included in the next release.

neocotic, yeah, I get you...
...Well, good tools eventually get the attention they deserve, especially among the more technically minded. ...And I'm sure Template will get the attention it deserves! :)

Now to get looking at markdown ;)

Yeah, that will be an awesome feature to have...


[Question] Authenticate Using Windows Auth

[Question]
I am running my node process as a logged in authenticated Windows users. Can I get an example of what my connection string config should look like? I am also not running on the default port of 1433. It is a single instance installed on the default instance of MSSQLSERVER.
Sorry to ask if this has already been answered prior.
Thanks...

Here's my config. This is with v3 - I wasn't able to get it going with v4, although I didn't try very hard. You need to use the msnodesqlv8 driver (which confusingly has the github project name node-sqlserver-v8).
I'm not sure if this is unique to my set up, but in my case the server name and user name must include the name of the machine followed by double backslash then the actual value.
Hope that helps, it took me a while to get it going!
const connConfig = {
  server: 'MSSQLServerName', // (in my case it's in the form 'DESKTOP-X1234Z\\DEVSERVER')
  user: 'username', // (in my case it's in the form 'DESKTOP-X1234Z\\DevUser')
  password: 'password',
  database: 'databaseName',
  driver: 'msnodesqlv8',
  stream: true,
  options: {
    trustedConnection: true
  }
};```

Sorry I have not responded earlier...I have a question about that...you have to send in the PWD...however, I am running my node service as a Windows Service using a domain login...I should be able to use my kerb stack and not have to type in the PWD...does something like what I described work?

I have no idea, I am not very familiar with windows systems and just managed to fumble my way to a working solution for this particular set up. You may be able to derive your password from an environment variable rather than coding it.

That is fair. Thank you for the help


Improve cross-borrowing ergonomics

Many Rust users say that the ergonomics for borrowing / deref / reborrowing is poor, especially compared to how magically dot (a.f) will behave (with respect to how it auto-derefs and borrows a).
This issue is meant to act as a place to collect the various proposals that have been put forward to resolve this.  I guess it might be thought of as a meta-issue, except that I think it will mostly be collecting references to RFC PR's, rather than other issues.
(My current plan is to try to put links to all the appropriate PR's into this description itself; people can feel free to refer back to this issue, but I'm not planning on annotating each PR with a link, especially since the list of references inserted by github can get somewhat polluted...  I will also try to link to meeting minutes if I can find them.)
Proposals:

#226
#241
#248

Relevant rust-lang/rust issues:

rust-lang/rust/issues/13988

Relevant Meeting Snippets

2014/05/20 removal of cross-borrowing
2013/11/19 autoderef
2013/07/30 == and autoderef

Potentially relevant discuss links (pre-RFCs, etc):

http://discuss.rust-lang.org/t/semi-explicit-coercion-control-with/520
http://discuss.rust-lang.org/t/pre-rfc-add-a-coerce-trait-to-get-rid-of-the-as-slice-calls/415/4

@pnkfelix Do we feel that this issue has any value at this point? Should we close?


[CARBONDATA-3000] Provide C++ interface for writing carbon data in CSDK

[CARBONDATA-3000] Provide C++ interface for writing carbon data in CSDK
1.suport string, short, int, long, double, float, array, boolean data type
2.provide builder, build, write, close interface
This PR base on https://github.com/apache/carbondata/pull/2792 and https://github.com/apache/carbondata/pull/2816, and cherry its commits to this PR. this PR will remove its commits after it's merged
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?
add C++interface
[ ] Any backward compatibility impacted?
No
[ ] Document update required?
Yes
[ ] Testing done
add some test case
[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
https://issues.apache.org/jira/browse/CARBONDATA-2951

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/879/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9144/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/881/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9146/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1079/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1064/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1278/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9330/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9342/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1290/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1084/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1086/

LGTM

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1091/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1303/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9355/

retest this please

LGTM

@KanakaKumar CI pass, please check it again.

LGTM


[SPARK-33634][SQL][TEST] use Analyzer in PlanResolutionSuite

What changes were proposed in this pull request?

Instead of using several analyzer rules, this PR uses the actual analyzer to run tests in PlanResolutionSuite.
Why are the changes needed?

Make the test suite to match reality.
Does this PR introduce any user-facing change?

no
How was this patch tested?

test-only

cc @rdblue @HyukjinKwon

Test build #132060 has started for PR 30574 at commit cff33a9.

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

Test build #132060 has finished for PR 30574 at commit cff33a9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

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


Automated tests of docker images

Run iojs and npm through their paces before considering them ready to deploy.

Would this be helpful: official-images test suite?  We are working on having more tests for more images.  If it can work for you, feel free to add tests and help us improve the tool for everyone :smiley:.

@yosifkit probably! I'll take a pass at it soon.

An automated build on Docker Hub would give us real time build errors.

Hi hi, one of the wercker guys here. It's true that we don't support building dockerfiles at the moment, though there are plenty of places that will do that part (we're still debating supporting it considering how easy it is to get done and how non-standardized the commands people tend to use in them are).
The two common use cases we see are 1) people using that built dockerfile image as a base on wercker for running their tests and 2) people building and testing their code using wercker and then deploying to the hub/etc using a pristine base image (you can build/test using one image and then drop that output into small image without build deps and push that).
Happy enough to help out if there are more questions :)

Thanks @termie! @Starefossen I was considering creating a Dockerfile that variably inherits from each of the images we would like to test, and that file have the unit tests in it. Docker Hub may not be a good candidate for that since it requires shipping before testing.
I was considering setting up a build server on D.O. that handles it, but it would be a bit before I can get to that point.

Did you have any thoughts on which tests to run @wblankenship? My first thought was to run the io.js core test suite – but this actually crashes v8 at some point along with a bunch of other test failures!
This was my Dockerfile setup:
FROM iojs:1.6.3

RUN curl -L https://github.com/iojs/io.js/tarball/v1.6.3 | tar -xz
RUN mv iojs-io.js* /usr/src/app
RUN ln /usr/local/bin/iojs /usr/src/app/out/Release/iojs

WORKDIR /usr/src/app

CMD python tools/test.py

I want to run the io.js test suite and https://github.com/nodesource/distributions/pull/12#issuecomment-56770431 at a minimum.

the io.js tests are pretty intensive,  I get about 28 failures a bunch of which are based on the test being run a little differently than when io.js runs them normally (which is to say there is supposed to be a bit more stuff in the Release dir than just iojs).
Anyway, I wanted to see how the new wercker stack handles it and in general poke at your use case:
https://app.wercker.com/#build/551f13156dda5c9817000207
You can see the wercker.yml I used for the build (basically just a copy of your dockerfile and the test @Starefossen described above) in the setup environment step: https://app.wercker.com/#buildstep/551f13156dda5c9817000208
or in my branch https://github.com/termie/docker-iojs/blob/wercker/wercker.yml, I tried to annotate it to explain what things were doing.

(on the plus side, v8 doesn't fail :) )

@termie: Do you mean building v8 or testing it? Testing v8 on musl would be interesting.

@jbergstroem I think @termie v8 comment was directed towards my comment that I managed to get a V8_Fatal error when running the io.js test suite inside of the official Docker image for io.js.

@Starefossen ah, ok.

Here is my current CI setup, its a sandbox for the iojs/docker-iojs project (it is really rough around the edges):
<IP_ADDRESS>:8080/job/test/11
It is hooked up to the repo: https://github.com/wblankenship/test/pull/5

Congratulations @termie and the wrecker group: http://blog.wercker.com/2015/04/02/Introducing-our-docker-stack.html
:smile:

Wow! Very nice indeed @termie 😄👏

I've been a blocker on this for a while. Unassigning myself incase someone else wants to take a pass at it.
So far, I've created a tool for automated testing of Docker images: https://github.com/wblankenship/dante
And put together some smoke tests for the Docker image, which follows the work of rvagg: https://github.com/wblankenship/docker-iojs/blob/tests/tests/iojs/Dockerfile
The tests currently do not run the Node.js project's test suite, which may be a good idea.
We have an open pull request for the build group to provision us a Jenkins server that we can wire this workflow up to: https://github.com/nodejs/build/issues/168
The Jenkins slave should trigger the test on a new PR to this repo. The test is run by simply calling dante in the base of the project directory. If the tests pass, dante will exit with a status 0, else it will exit with a status of 1. dante creates tagged images for each build for historical reference. This will chew up HDD space over time so we will need some sort of garbage collection as well.


FontAwesome 5 compatibility

Added compatibility for FontAwesome 5 by creating a seperate element for the icon and nesting it inside the button element instead of putting the classes on the button element. This makes sure that the button and all its properties stays intact when FontAwesome 5 changes elements to <svg> for their icons.
FontAwesome 4 works with this change as well. There are no visual differences except that the icon has shifted 1px up, probably because of some centering by FontAwesome.
Fixes #664

Any chance this could be added in?

Is there anyone who wants to co-maintain this package? I've pinged the original authors to see if they need help but my expertise is in other languages so would appreciate backup.

I think that is necessary to include Font Awesome 5 for next versions of SimpleMDE. 👍


Custom portals

Possible uses for custom interworld links (that don't present themselves as, e.g., linking books).

A long walk from one Cavern neighborhood to another
Magic teleporters unrelated to the standard portal theme
"Death" that drops you into an afterlife world instead of /panic
A scene change that needs to be from a global instance to a private instance

Note that some of these cases bypass the standard "examine portal, details in right-hand panel" view. They might be simple links. They might be code-triggered.
Therefore, we have to be careful.

Such links should always be uncopyable. (Or if they are copyable, they'll shed their custom nature and become normal portals).
Unless the customization is associated with the destination loc? Then any link to that loc, from your booklet or wherever, has the custom nature.
The onenter hook (when linking in) must exclude permission to link out. Avoid infinite loops.


Doc: fix broken formatting in markdown #headers

Insert a space in markdown headers between the #'s and the header text.
(It appears that the space is required since a github update a year ago
https://github.com/blog/2100-github-pages-now-faster-and-simpler-with-jekyll-3-0)

Notification: we do not merge into master, so please create PRs against current dev branch which is v0.12.2.x (I fixed this PR manually, no need to recreate)

oops. thanks
I'd created this PR entirely on GitHub - something I hadn't tried before.
It's pretty convenient for doc updates like this - you can create branch,
do the edits and open the PR all from the browser.
On Thu, May 4, 2017 at 1:18 PM, UdjinM6<EMAIL_ADDRESS>wrote:

Notification: we do not merge into master, so please create PRs against
current dev branch which is v0.12.2.x (I fixed this PR manually, no need to
recreate)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dashpay/dash/pull/1462#issuecomment-299251469, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADBpbmowj39YvoAstC0ISkPYb5kzd0x9ks5r2gh0gaJpZM4NPi_Y
.


Create API-Testing-Guidelines.md

What type of PR is this?
Add one of the following kinds:

documentation

What this PR does / why we need it:
Adds API-testing-guidelines doc
Which issue(s) this PR fixes:
Fixes #94
Special notes for reviewers:
Changelog input
 release-note

Additional documentation
This section can be blank.
docs

@bigludo7 @jlurien @mdomale: Kindly review if all your points have been covered in the doc.

@shilpa-padgaonkar Document looks fine to me.

@patrice-conil : made the fix as you suggested. I simply used qod as example because simswap example is still a PR

Hello @shilpa-padgaonkar
First, thanks for the effort :)
Like my colleague @patrice-conil  I think referring a sample is very useful.
I have an issue with QoD file because it not reflect the rules (several when/then, name). I've applied rules here: https://github.com/camaraproject/DeviceLocation/pull/119#pullrequestreview-1818257946 - Could you please take a look on this proposal and check if it aligned with your view?
We should have an exemple fully aligned with these rules i guess and use it as base.

@thanks Ludovic for your quick feedback.  I simply added qod as an example for the location (where should the feature file reside).
But you are right, a link to a sample which is based on these guidelines would be a very good add. I have looked at your feature file and it LGTM. I will also request @mdomale to briefly take a look and provide quick feedback on the file.
BTW, should we add your example link already, or would you want to do this after your PR gets merged (I am fine both ways)?

@bigludo7  Except minor comments , mentioned feature file in PR https://github.com/camaraproject/DeviceLocation/pull/119#pullrequestreview-1818257946
LGTM   @shilpa-padgaonkar

Thanks a lot @shilpa-padgaonkar and @mdomale  for your review.
Regarding:

BTW, should we add your example link already, or would you want to do this after your PR gets merged (I am fine both ways)?

I suggest we wait from Device location project feedback but I will inform then that current proposal is "OK" from our perpsective. Adding @jlurien  in the loop.

@jlurien : Thanks for your comments. I will fix the incorrect link with the new one provided by @mdomale and add it to the doc.
As you have given the example of qod api to support the case of using multiple feature files (feature in your context is an API operation or resource), I would like to ask for some additional feedback (especially from the qod codeowners and/or maintainers) to get more view points on this topic.
Tagging here the codeowners: @hdamker @RandyLevensalor @eric-murray

@shilpa-padgaonkar I tend to agree with @jlurien that we should flex the following sentence:

One feature file per API is advisable so that all scenarios can be covered corresponding to the API & the corresponding resource.

Indeed, as I'm involved in Device Status for example, we have there one API (one yaml) that feature 3 distincts capabilities:

Get on the fly if the mobile is in roaming situation
Get on the fly if the mobile is reacheable (connectivity)
Subscription to get notification when mobile change roaming/connectivity state

(the granularity of this API is perhaps challengeable but that's another topic)
I assume that it will be in this case simpler to have 3 dedicated features --> I will trigger the discussion in this project.
Let's see the feedback we have from QoD & Device Status... Perhaps we can put in like this: Granularity of the feature file should be at the project level but it is recommended to group in one file all scenario for one given API fonction. If a CAMARA API features several independent fonctions it is recommended to provide several files.

@bigludo7 : Thanks Ludovic for driving this in devicestatus. :)
Now awaiting feedback from devicestatus and qod

I agree with @bigludo7 and Device Status is a very good example. "Feature" is a very abstract concept, but the common advice is to not mix unrelated functionality in the same file. It is more easy to work with several small files dedicated to test an specific part of an application or API, in our case, that having only a  huge file with hundreds of unrelated scenarios.
Cucumber and other frameworks accept a folder as input and are prepared to look for and execute several *.feature files in one run. So I'm in favor of relaxing the guidelines and let each subproject decide on how many "features" can be identified for some specific API.

@jlurien and @bigludo7 : Could you also kindly propose a text in the meanwhile which would reflect this more relaxed guideline, as opposed to "One feature file per API is advisable so that all scenarios can be covered corresponding to the API".
As we need to agree and merge this document for the upcoming release, I would request you to already prep for this text and once we get this similar feedback as suggested above from devicestatus and qod, we can add your text and proceed with approving and merging this doc.

@jlurien and @bigludo7 : Could you also kindly propose a text in the meanwhile which would reflect this more relaxed guideline, as opposed to "One feature file per API is advisable so that all scenarios can be covered corresponding to the API".
As we need to agree and merge this document for the upcoming release, I would request you to already prep for this text and once we get this similar feedback as suggested above from devicestatus and qod, we can add your text and proceed with approving and merging this doc.

I've just provided a sentence as a review comment ;)

@bigludo7 : Thanks. Your suggestions are now added to the doc :)

Requesting @jlurien to kindly review and approve

Thanks all for the contributions, review and approval. @rartych: Could you kindly check that there are no more open points ignored as a part of the review and if not, proceed to merge the PR?

@RubenBG7 @jordonezlucena - as @jlurien approved the PR, could you approve it as codewoners?


[TokenContract Review] Refactor ExecuteUpdateMarketing

Severity:
Informational
Description:
The code block checking emty string can be refactored as follows:
fn handle_empty_string(value: String) -> Option<String> {
   if value.trim().is_empty() {
       None
   } else {
       Some(value)
   }
}

marketing_info.project = project.and_then(handle_empty_string);
marketing_info.description = description.and_then(handle_empty_string);
marketing_info.marketing = marketing.and_then(|m| deps.api.addr_validate(&m).ok());

File Name:
contracts/token-contracts/cw-hub-bnusd/src/contract.rs Line No. 419

Not contracts/token-contracts/cw-hub-bnusd/src/contract.rs  but the built on cw_20 base library.


BREAKING CHANGE: Add generic structure type, use context/getObject

As described in https://github.com/open-feature/java-sdk/issues/50, I think that expected a T back from object flag resolvers is not sustainable. I believe it pushes too much of a burden on providers to instantiate a new T (whatever it may be) and map the attributes received from the backend into it.
Instead, I've added a more generic Structure (language used in the OpenFeature spec) with methods similar to the existing EvaluationContext. I've also made the EvaluationContext inherit from this to reduce duplication (the EvaluationContext is also referred to as a "structure" in the spec.
After this change, providers will create a new Structure and add the corresponding fields from the json/xml/protobuf they are concerned with.
In summary:

Adds Structure type which represents JSON data
EvaluationContext inherits from this
Consolidated addIntegerAttribute, addStringAttribute, etc, in EvaluationContext to a single overloaded add method.
Removed ability to get multiple attributes of a single type from EvaluationContext.

FYI @rgrassian-split

FYI @rgrassian-split. I would like to get your thoughts on this. How would this change impact the provider you're working on?

@beeme1mr Hey! This is a fantastic change, I was actually running into some issues with the T implementation so I'm happy to see this. It shouldn't be too crazy to change our provider to support this.

@rgrassian-split it would be good if you could put some feedback in https://github.com/open-feature/java-sdk/issues/50

@toddbaert Just did, that is actually the bigger deal for us that I very much support switching it from the current approach

Codecov Report

Merging #51 (7db48b8) into main (465d113) will decrease coverage by 0.93%.
The diff coverage is 80.48%.

@@             Coverage Diff              @@
##               main      #51      +/-   ##
============================================
- Coverage     93.01%   92.07%   -0.94%
+ Complexity      124      118       -6
============================================
  Files            17       18       +1
  Lines           272      265       -7
  Branches         12       10       -2
============================================
- Hits            253      244       -9
- Misses           11       13       +2
  Partials          8        8

Flag
Coverage Δ

unittests
92.07% <80.48%> (-0.94%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...c/main/java/dev/openfeature/javasdk/Structure.java
77.77% <77.77%> (ø)

...ava/dev/openfeature/javasdk/EvaluationContext.java
77.77% <100.00%> (-5.83%)
:arrow_down:

...ava/dev/openfeature/javasdk/OpenFeatureClient.java
97.53% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@justinabrahms I think I've cleaned up a lot of the whitespace changes. A few are possibly improvements (missing newlines in some places, etc). I've left those.
I also added array support in Structure.
I'm going to test this locally in my provider and see if it makes things nicer. Let me know what you think.

Playing around with the API. It seems like because we broke out Structure, the return this is returning Structure objects, not EvaluationContext objects. I'm not aware of a way to have it return EvaluationContext objects except to have EC proxy those methods to an internal representation of a Structure. Thoughts?

Playing around with the API. It seems like because we broke out Structure, the return this is returning Structure objects, not EvaluationContext objects. I'm not aware of a way to have it return EvaluationContext objects except to have EC proxy those methods to an internal representation of a Structure. Thoughts?

I noticed that too... it's sort of a shame, but I think we'd have to proxy them as well... maybe there's a clever solution but I can't think of one immediately. I'll look around.

@justinabrahms I'd like to play with this a bit more locally before we merge... I'll let you know if I think any changes are needed. I think this should be sufficient though.

@justinabrahms  I made some more changes. Our abstraction was leaky because any node in a JSON structure can be a JSON structure, an array or a primitive. That wasn't reflected the implementation.
What I've done in this commit, is introduce a Value wrapper class that houses one of: Structure, List<Value>, Boolean, etc (representations of all the possible JSON node values). The actual value can be retrieved like value.asBoolean() or value.asStructure(). I believe any JSON structure can be represented this way.
This is similar to how java-grpc represents their structure (which is very similar to JSON) and something like a simplified version of LDValue.
I've maintained most of the previous work. EvaluationContext still encapsulates a Structure, etc.

I think I need to play around with constructing a few of these objects to figure out what the API feels like.

OK @justinabrahms, I'll leave it up to you to merge it then (or message with more feedback). @kinyoklion has also approved. I think we'll need to do something similar in dotnet cc @benjiro


test: replacing tmpdir with tmp_path in tests_pytorch/cli.py

What does this PR do?
Part of https://github.com/Lightning-AI/pytorch-lightning/issues/16188#issuecomment-1691189400
change tmpdir to tmp_path fixture in tests_pytorch/cli.py

Before submitting

Was this discussed/agreed via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
Did you make sure to update the documentation with your changes? (if necessary)
Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
Did you list all the breaking changes introduced by this pull request?
Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

Reviewer checklist

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

📚 Documentation preview 📚: https://pytorch-lightning--19632.org.readthedocs.build/en/19632/

Yeah this makes more sense and will be easy to traceback failures quickly, I'll go ahead and open PRs for each commit🤞

cool, lets keep this open as draft and each new PR refer to this one so we can keep track at one place or refer to the original issue, up to you :)

Yeah this makes more sense and will be easy to traceback failures quickly, I'll go ahead and open PRs for each commit🤞

cool, lets keep this open as draft and each new PR refer to this one so we can keep track at one place or refer to the original issue, up to you :)

Just branched every commit to its own branch from my fork going to open PRs


fix: copy libs to public/lib folder

Fixes #869.
Copies the assets/libs to the docker container.

Thanks!


jfrog rt build-promote fails when multiple builds with same name and number have been uploaded

Artifactory returns 400 Bad Response to jfrog-cli if two builds have been uploaded with the same name and number.
jfrog rt bpr my_build_name 1.2.3 target-repo
[Info] Promoting build...
[Error] Artifactory response: 400 Bad Request
{
  "errors": [
    {
      "status": 400,
      "message": "Unable to find artifacts of build 'my_build_name' #1.2.3: aborting promotion."
    }
  ]
}

Multiple builds of the same name and number can get into Artifactory via steps described in https://github.com/jfrog/teamcity-artifactory-plugin/issues/35
The Artifactory rest API only returns the most recent buildinfo object, not both, which means it cannot be used as a workaround.

I find this very annoying as well. the build should either be unique on name/number or when you publish to an existing name/number it should add/replace artifacts

Thanks for reporting this issue @daveneeley and @djgilcrease.
We opened RTFACT-19143 on Artifactory's JIRA project to track this request.

If jfrog-cli creates the duplicate buildinfos, as outlined in the linked gist, then there is no failure but only the most recent buildinfo will be promoted.
https://gist.github.com/daveneeley/18a72abe2e9a4c5de9d256b1115d5cdb

@daveneeley and @djgilcrease,
Thanks for sharing your input and the information.
Since we now have the above JIRA issue open, and if a behavior change will be made, it'll probably be on the REST endpoint (and not in JFrog CLI), I'm closing this issue for now.
Feel free to add more comments here.


handle displaying accounts not created by wallet in authz details

fixes issue #87

@justinbarry can your team check this PR I think it should either be merged or closed as it's pretty stale.

Closing as stale.


How to create a torrent?

how to cteate a .torrent file from the client？

use MetadataBuilder class
simple example:
byte[] torrentMetadataBytes = new MetadataBuilder()
            .addDataSource(new ByteArrayInputStream(new byte[]{1,2,3}), "singleFile.txt", true)
            .setTracker("http://localhost:12346")
            .buildBinary();

For adding local files you can use addFile method instead of addDataSource

Thank you very much for your answer，but i have confused about the other client how to download the shared files. When i create the .torrent file with the MetadataBuilder class, and which client be the seeder frist?

I create a .torrent file and use client download it,but the tracker allways get the request event NONE.
how can i fix it?
2019-07-22 14:18:18,026 [Dispatcher-4             ] WARN : RequestEvent event: NONE
2019-07-22 14:18:18,026 [Dispatcher-4             ] WARN : peer!!

Looks like you created a separate issue for this question. Ask me there


95% test coverage

Jacoco
Codecov?

Enforcing through CI could be part or follow-up of this

@deepankarb why did you choose jacoco over codecov in your PR?
@pld do you have any particular recommendations? Thanks!

We user coveralls and we're pretty happy with it.
E.g. https://coveralls.io/github/OpenSRP/opensrp-client-core?branch=master
@rkodev not sure if you have any recs

On Mar 28, 2021, at 16:30, Jing Tang @.***> wrote:

@deepankarb why did you choose jacoco over codecov in your PR?
@pld do you have any particular recommendations? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@deepankarb why did you choose jacoco over codecov in your PR?
@pld do you have any particular recommendations? Thanks!

Jacoco generates the code coverage reports both offline and in CI runs.
Codecov will use the report to provide running coverage status and various other useful features.
HAPI FHIR are using codecovhttps://github.com/hapifhir/hapi-fhir#test-coverage.

#502 Adds code coverage metrics. Keeping this issue open to track progress to > 95% coverage.


Honour pixel ratio and allow degrading quality for performance

Add olcs.OLCesium.setResolutionScale to allow improving performance at the cost of quality.
Automatically use device pixel ratio to configure the Webgl 3D globe.

LGTM

Not yet ready, there is an issue with view synchronization. I am looking at it.

Should be fixed now.


Interface identification over json not possible

Hi @danielperna84.
currently we use this code to identify the interface a device (name) belongs to:
https://github.com/danielperna84/hahomematic/blob/0ac8ef8efbca579984689dc4cf4bffdd088dd206/hahomematic/client.py#L429-L448
Result of Interface.listInterfaces is always:
[{'name': 'CUxD', 'port': 8701, 'info': 'CUxD'},
{'name': 'BidCos-RF', 'port': 32001, 'info': 'BidCos-RF'},
{'name': 'VirtualDevices', 'port': 39292, 'info': 'Virtual Devices'},
{'name': 'HmIP-RF', 'port': 32010, 'info': 'HmIP-RF'}]
This only works, if the configured port matches a port of the list (+30000,+40000, -10000 or other additions doesn't matter here).
It possible to configure (e.g. in Docker) other ports than the default one: maybe use port 12345 for hmip. In such a case name resolution doesn't work.
I haven't found any other way to identify the interface, except with a different convention that might not work either.
My solution: Get rid of the interface for name resolution.
The current assumption is that addresses are unique for an interface.
My assumption is that addresses are unique for a ccu (bigger scopes don't matter here).
Did I miss something?

It's been a while since I have implemented this. But AFAIK I did it this way for performance reasons. With pyhomematic we're getting all the names on every startup, which would take too long when looping over all interfaces for every device. However with hahomematic the names can be cached, so this only would be a performance hog once. I would be fine with that.

I'd stays with the single Device.listAllDetail call, but the resulting names dict will be {address, name} and not {interface, {address, name}}.

Sounds ok to me. 👍

Thanks  a lot.


Add other permissions for heuristic-based recommendations

Add other read-only permissions we'll probably want in the near future for heuristic-based cost/performance recommendations

@dvanaken just a note when uploading the new template file, we need to make sure the object has the right permissions. We need read access for AWS authenticated users. Let me know if you want another set of eyes when doing this.
Also sorry if you already knew this!


Rename how-to guides to developer guides

There was an inconsistency in the naming, as it said "HOW-TO GUIDES" in the paragraph whereas they are actually called "DEVELOPER GUIDES" in the sidebar.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Thanks! 🤗


Added new version of SetNewRootAndResetAsync based on PageModel. Also…

… as ExtensionMethod of IBasePageModel and removed currentpage parameter from it as it is not used

Nice 👍 Thanks.


Tectonic terraform apply - ssh handshake failed, etcd fail

Issue Report Template
Tectonic Version
1.7.9
Environment
preboot env (DHCP, DNS. iPXE)
vms:

faiserver (matchbox, tectonic installer)
master nodes (core1,core2,core3)
worker nodes (worker1,worker2,worker3)
What hardware/cloud provider/hypervisor is being used with Tectonic?
vmWare ESXi 5.5

Expected Behavior
Starting Tectonic cluster
Actual Behavior
Terraform failed
Reproduction Steps

Preparing to run tectonic:
a) make sure ssh-agent is running*: $eval $(ssh-agent -s)
b) check your currentlly attached keys (none by default) $ssh-add -L
c) add your ssh private key that coresponds to the ssh-rsa key you will be using in the installer $ssh-add ~/.ssh/id_rsa
d) remove entries from  ~/.ssh/known_hosts

[root@faiserver linux]# vim ~/.ssh/known_hosts
[root@faiserver linux]# ssh-add -L
Could not open a connection to your authentication agent.
[root@faiserver linux]# eval $(ssh-agent -s)
Agent pid 14003
[root@faiserver linux]# ssh-add -L
The agent has no identities.
[root@faiserver linux]# ssh-add ~/.ssh/id_rsa
Identity added: /root/.ssh/id_rsa (rsa w/o comment)
[root@faiserver linux]# ssh-add -L
ssh-rsa blablabla rsa w/o comment

Run tectonic installer
[root@faiserver linux]# ./installer -address <IP_ADDRESS>:4444 --open-browser=false

Apply Terraform config and start up vm nodes

CoreOS installed on nodes (rebooted several times). After that i can login to any node w/o password:
[root@faiserver ~]# ssh core@core1
Last login: Thu Nov 16 07:43:56 UTC 2017 from <IP_ADDRESS> on pts/0
Container Linux by CoreOS stable (1520.8.0)
Update Strategy: No Reboots

But Terraform still applying..
After one hour Terraform failed:
...
null_resource.etcd_secrets.1: Still creating... (59m50s elapsed)
null_resource.etcd_secrets.0: Still creating... (59m50s elapsed)
null_resource.etcd_secrets.2: Still creating... (1h0m0s elapsed)
null_resource.etcd_secrets.1: Still creating... (1h0m0s elapsed)
null_resource.etcd_secrets.0: Still creating... (1h0m0s elapsed)
Error applying plan:
3 error(s) occurred:

null_resource.etcd_secrets[1]: 1 error(s) occurred:

ssh: handshake failed: ssh: unable to authenticate, attempted methods [none], no supported methods remain

null_resource.etcd_secrets[0]: 1 error(s) occurred:

ssh: handshake failed: ssh: unable to authenticate, attempted methods [none], no supported methods remain

null_resource.etcd_secrets[2]: 1 error(s) occurred:

ssh: handshake failed: ssh: unable to authenticate, attempted methods [none], no supported methods remain

Terraform does not automatically rollback in the face of errors.
Instead, your Terraform state file has been partially updated with
any resources that successfully completed. Please address the error
above and apply again to incrementally change your infrastructure.
And on master nodes we see:
core@core1 ~ $ journalctl -xe
-- Subject: Unit etcd-member.service has finished shutting down
-- Defined-By: systemd
-- Support: https://lists.freedesktop.org/mailman/listinfo/systemd-devel
-- Unit etcd-member.service has finished shutting down.
Nov 16 07:43:57 core1.mydomain systemd[1]: Starting etcd (System Application Container)...
-- Subject: Unit etcd-member.service has begun start-up
-- Defined-By: systemd
-- Support: https://lists.freedesktop.org/mailman/listinfo/systemd-devel
-- Unit etcd-member.service has begun starting up.
Nov 16 07:43:57 core1.mydomain rkt[936]: rm: unable to resolve UUID from file: open /var/lib/coreos/etcd-member-wrapper.uuid: no such file or directory
Nov 16 07:43:57 core1.mydomain rkt[936]: rm: failed to remove one or more pods
Nov 16 07:43:57 core1.mydomain etcd-wrapper[943]: ++ id -u etcd
Nov 16 07:43:57 core1.mydomain etcd-wrapper[943]: + exec /usr/bin/rkt run --volume etcd-ssl,kind=host,source=/etc/ssl/etcd --mount volume=etcd-ssl,target=/etc/ssl/etcd --tr
Nov 16 07:43:57 core1.mydomain kubelet-wrapper[772]: Downloading ACI:  3.44 MB/245 MB
Nov 16 07:43:58 core1.mydomain etcd-wrapper[943]: run: stat of host path /etc/ssl/etcd: stat /etc/ssl/etcd: no such file or directory
Nov 16 07:43:58 core1.mydomain systemd[1]: etcd-member.service: Main process exited, code=exited, status=254/n/a
Nov 16 07:43:58 core1.mydomain systemd[1]: Failed to start etcd (System Application Container).
-- Subject: Unit etcd-member.service has failed
-- Defined-By: systemd
-- Support: https://lists.freedesktop.org/mailman/listinfo/systemd-devel
-- Unit etcd-member.service has failed.
-- The result is failed.
Nov 16 07:43:58 core1.mydomain systemd[1]: etcd-member.service: Unit entered failed state.
Nov 16 07:43:58 core1.mydomain systemd[1]: etcd-member.service: Failed with result 'exit-code'.
Nov 16 07:43:58 core1.mydomain kubelet-wrapper[772]: Downloading ACI:  7.27 MB/245 MB
Nov 16 07:43:59 core1.mydomain kubelet-wrapper[772]: Downloading ACI:  10.9 MB/245 MB
Nov 16 07:43:59 core1.mydomain systemd[1]: Started OpenSSH per-connection server daemon (<IP_ADDRESS>:55689).
-- Subject: Unit sshd@6-<IP_ADDRESS>:22-<IP_ADDRESS>:55689.service has finished start-up
-- Defined-By: systemd
-- Support: https://lists.freedesktop.org/mailman/listinfo/systemd-devel
-- Unit sshd@6-<IP_ADDRESS>:22-<IP_ADDRESS>:55689.service has finished starting up.
-- The start-up result is done.
Nov 16 07:43:59 core1.mydomain sshd[960]: Connection closed by <IP_ADDRESS> port 55689 [preauth]
Nov 16 07:44:00 core1.mydomain kubelet-wrapper[772]: Downloading ACI:  13.4 MB/245 MB
Nov 16 07:44:01 core1.mydomain kubelet-wrapper[772]: Downloading ACI:  15.2 MB/245 MB
Nov 16 07:44:02 core1.mydomain kubelet-wrapper[772]: Downloading ACI:  17.2 MB/245 MB
Nov 16 07:44:02 core1.mydomain systemd[1]: Started OpenSSH per-connection server daemon (<IP_ADDRESS>:55691).
-- Subject: Unit sshd@7-<IP_ADDRESS>:22-<IP_ADDRESS>:55691.service has finished start-up
-- Defined-By: systemd
-- Support: https://lists.freedesktop.org/mailman/listinfo/systemd-devel
-- Unit sshd@7-<IP_ADDRESS>:22-<IP_ADDRESS>:55691.service has finished starting up.
-- The start-up result is done.

Desired Feature
Other Information

I generally see this occur when there is a failure to match a pxe booted node to a matchbox profile/group. Look at the logs of the matchbox.service or matchbox container. Does matchbox see any requests? If no, you need to figure out if network or firewall settings are blocking the request. If yes, there should be a reason listed why the request did not match to a profile. Although this doesn't seem to be the case here it is still worth looking into.
Please provide the full log for the etcd-member.service:
journalctl -u etcd-member.service > etcd-member.log

Solved!
There is been a problem with issued certificates from Matchbox (wrong hostname)


Add the ability to throw an error to python debugger

This patch makes it possible to throw an error from the python debugger client using the throw command.
JerryScript-DCO-1.0-Signed-off-by: Daniel Balla<EMAIL_ADDRESS>

@zherczeg PR Updated, thanks for the review!

@zherczeg I have updated the PR, thank you for the suggestions!

@zherczeg Sorry for the late update, thank you for the review.

Please rebase. The debugger tests are failing at the moment.

@zherczeg I've rebased the PR .


Revert "Merge main into rhcos"

Reverts coreos/coreos-layering-examples#15

I meant to merge into rhcos, not main :sob:
@jmarrero was definitely onto "something is fishy here" and I was like "naah it's all good!" and I was definitely wrong :smile:

I meant to merge into rhcos, not main sob
@jmarrero was definitely onto "something is fishy here" and I was like "naah it's all good!" and I was definitely wrong smile

I should have looked at the branches closely


Weird Movie Card Returns

When you run a search, sometimes a few "undefined' cards come back - example "Horror" + "King" - returned two undefined cards.

This is likely due to a movie that is in TMDb but not in OMDb - so we get a movie when we do the initial search in TMDb but when we hand it off to OMDb to get the title and summary it doesn't find a match. We'll want to check if OMDb has a result in the getMovieInfo() function then only send to displayMovie() if there is a result, else don't do anything.


Cypress Testing v12.8.1 - Event Analytics fully working

Description
[Describe what this change achieves]
Fixed all test in first module
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

All 31 test passing reliably.


Update scripts to python3

Since python2 reached EOL, all scripts should be updated to use python3 syntax.
Changes made:

Initial conversion of all python scripts using 2to3 -w scriptname.py
Executed all scripts multiple times to find possible errors that weren't caught by the conversion scripts
most of the changes are related to braces in print statements

Seems good to merge!


crash game after the end of the song "Unhandled Microsoft C ++ Exception"

crashinfo.txt
/-----------------------------------------------------------------/
Etterna crash report (build ,  @ )
Crash reason: Unhandled Microsoft C++ Exception
Thread: Main thread
GraphicsWindow.cpp:63<PHONE_NUMBER>1203B4, 256, 00000041, 001e0001
GraphicsWindow.cpp:63<PHONE_NUMBER>1203B4, 257, 00000041, c01e0001
GraphicsWindow.cpp:63<PHONE_NUMBER>1203B4, 257, 00000053, c01f0001
ScreenGameplay.cpp:2048 HandleScreenMessage(SM_DoNextScreen)
StageStats.cpp:671 Uploading score with replaydata.
Thread: Decode thread
RageSoundDriver_Generic_Software.cpp:235 Processing the sound while buffers are available.
RageSoundDriver_Generic_Software.cpp:235 Processing the sound while buffers are available.
RageSoundDriver_Generic_Software.cpp:235 Processing the sound while buffers are available.
RageSoundDriver_Generic_Software.cpp:235 Processing the sound while buffers are available.
RageSoundDriver_Generic_Software.cpp:235 Processing the sound while buffers are available.
Thread: Mixer thread
Thread: Music thread
Thread: DirectInput thread
InputHandler_DirectInput.cpp:891
InputHandler_DirectInput.cpp:867
InputHandler_DirectInput.cpp:885
InputHandler_DirectInput.cpp:891
InputHandler_DirectInput.cpp:867
Thread: Streaming sound buffering
debug resource file 'C:\Games\Etterna 0.66.0\Program\Etterna.vdi': CreateFile failed (Не удается найти указанный файл.).
Static log:
Etterna
Compiled  @  (build )
Log starting 2019-06-15 13:16:36
Verbosity: 1
Windows 6.1 (Win7) build 7601 [Service Pack 1]
Attempting to connect to: multi.etternaonline.com, Port: 8765
Server Version: 0
Video renderers: 'd3d, opengl'
WARNING: RageDisplay_D3D::TryVideoMode( 0, 1366, 768, 32, 0, 0 )
D3D Fullscreen 1366x768 32 color 32 texture 60Hz NoVsync SmoothLines
WARNING: Font import recursion detected
WARNING: /Themes/Til Death/Fonts/_open sans semibold 24px.ini
WARNING: /Themes/_fallback/Fonts/Common default.ini
WARNING: /Themes/Til Death/Fonts/_16px fonts.ini
WARNING: Current font: /Themes/Til Death/Fonts/_open sans semibold 24px.ini
WARNING: Font /Themes/Til Death/Fonts/_16px fonts.ini has no characters
Players joined: P1
Top Screen: ScreenGameplay
Current renderer: Direct3D
Partial log:
01:08.684: Attempted to load a non-existing mod '2.3xMusic' for the Player. Ignoring.
01:08.684: Attempted to load a non-existing mod '2.3xMusic' for the Player. Ignoring.
01:08.759: Attempted to load a non-existing mod 'savescore' for the Player. Ignoring.
01:08.759: Attempted to load a non-existing mod 'savescore' for the Player. Ignoring.
01:08.759: Attempted to load a non-existing mod 'savescore' for the Player. Ignoring.
01:08.759: Attempted to load a non-existing mod 'savescore' for the Player. Ignoring.
01:08.760: ScreenOptions::~ScreenOptions()
01:08.769: Unloading replaydata.
02:08.128: bAllReallyFailed = 0 bIsLastSong = 1, m_gave_up = 0, m_skipped_song = 0
02:13.132: saving stats and high scores
-- End of report
//
Log
//
00:00.016: Etterna
00:00.016: Compiled  @  (build )
00:00.016: Log starting 2019-06-15 13:16:36
00:00.016: 	Verbosity: 1
00:00.016:
00:00.016: Reading 'Data/Defaults.ini' failed: No such file or directory
00:00.018: Reading 'Data/Static.ini' failed: No such file or directory
00:00.018: Reading 'Data/Static.ini' failed: No such file or directory
00:00.383: Windows 6.1 (Win7) build 7601 [Service Pack 1]
00:00.399: Reading 'Themes/Til Death/Languages/ru.ini' failed: No such file or directory
00:00.410: Reading 'Themes/_fallback/Languages/ru.ini' failed: No such file or directory
00:00.427: Reading 'Themes/_fallback/ThemeInfo.ini' failed: No such file or directory
00:00.435: Reading 'Themes/Til Death/ThemeInfo.ini' failed: No such file or directory
00:01.990: Loaded 1 songs from "Songs/Helblinde"
00:02.618: Profile::LoadAllFromDir( /Save/LocalProfiles/00000000/ )
00:02.618: Error opening /Save/LocalProfiles/00000000/Etterna.xml: No such file or directory
00:02.737: Attempting to connect to: multi.etternaonline.com, Port: 8765
00:02.737: Could not create ettp connection because: Connection creation attempt failed
00:04.171: Connected to ett server: multi.etternaonline.com
00:04.172: Server Version: 0
00:04.172: C:\projects\etterna\src\RageUtil\File\RageFileDriverDirect.cpp:167: remove 'C:/Games/Etterna 0.66.0/Cache/Downloads/Helblinde.zip'
00:04.196: C:\projects\etterna\src\RageUtil\File\RageFileDriverDirect.cpp:167: remove 'C:/Games/Etterna 0.66.0/Cache/Downloads/new.Cirque du Lykan.zip.new'
00:04.224: Video renderers: 'd3d, opengl'
00:04.225: RageDisplay_D3D::RageDisplay_D3D()
00:04.258: Driver: aticfx64.dll
00:04.258: Description: AMD Radeon HD 6520G
00:04.258: Max texture size: 16384
00:04.258: Alpha in palette: no
00:04.258:
00:04.258: This display adaptor supports the following modes:
00:04.269:   640x480 60Hz, format 22
00:04.269:   800x480 60Hz, format 22
00:04.269:   800x600 60Hz, format 22
00:04.269:   1024x600 60Hz, format 22
00:04.269:   1024x768 60Hz, format 22
00:04.269:   1280x720 60Hz, format 22
00:04.269:   1280x768 60Hz, format 22
00:04.269:   1360x768 60Hz, format 22
00:04.269:   1366x768 60Hz, format 22
/////////////////////////////////////////
00:04.269: WARNING: RageDisplay_D3D::TryVideoMode( 0, 1366, 768, 32, 0, 0 )
/////////////////////////////////////////
00:04.269: Testing format: display 20, back buffer 20, windowed 0...
00:04.269: Testing format: display 22, back buffer 22, windowed 0...
00:04.269: This will work.
00:04.282: Testing format: display 20, back buffer 20, windowed 0...
00:04.282: Testing format: display 22, back buffer 22, windowed 0...
00:04.282: This will work.
00:04.282: Present Parameters: 1366, 768, 22, 1, 8, 1, 1180596, 0, 1, 80, 0, 0, -2147483648
00:04.716: D3D Fullscreen 1366x768 32 color 32 texture 60Hz NoVsync SmoothLines
/////////////////////////////////////////
00:11.059: WARNING: Font import recursion detected
00:11.059: WARNING: /Themes/Til Death/Fonts/_open sans semibold 24px.ini
00:11.059: WARNING: /Themes/_fallback/Fonts/Common default.ini
00:11.059: WARNING: /Themes/Til Death/Fonts/_16px fonts.ini
00:11.059: WARNING: Current font: /Themes/Til Death/Fonts/_open sans semibold 24px.ini
/////////////////////////////////////////
/////////////////////////////////////////
00:11.059: WARNING: Font /Themes/Til Death/Fonts/_16px fonts.ini has no characters
/////////////////////////////////////////

Without further details other than the crash reason, which tells us that it may be related to Lua, we cannot determine any issue or fix here.
Please provide more information in the future and follow the bug reporting format.


C&W attack on inceptionv3

I want to generate adversarial examples with C&WL2 on the inceptionv3, but it appeers the error as follows:
[INFO 2019-06-06 21:09:06,138 cleverhans] Constructing new graph for attack CarliniWagnerL2
Traceback (most recent call last):
File "cw/cw_inceptionv3.py", line 163, in
tf.app.run()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 125, in run
_sys.exit(main(argv))
File "cw/cw_inceptionv3.py", line 139, in main
x_adv = cwl2.generate_np(x_input,**cw_params)
File "/home/ra/cleverhans/cleverhans/attacks/attack.py", line 185, in generate_np
self.construct_graph(fixed, feedable, x_val, hash_key)
File "/home/ra/cleverhans/cleverhans/attacks/attack.py", line 157, in construct_graph
x_adv = self.generate(x, **new_kwargs)
File "/home/ra/cleverhans/cleverhans/attacks/carlini_wagner_l2.py", line 70, in generate
labels, nb_classes = self.get_or_guess_labels(x, kwargs)
File "/home/ra/cleverhans/cleverhans/attacks/attack.py", line 277, in get_or_guess_labels
preds = self.model.get_probs(x)
File "/home/ra/cleverhans/cleverhans/model.py", line 84, in get_probs
d = self.fprop(x, **kwargs)
File "/home/ra/cleverhans/cleverhans/model.py", line 108, in fprop
raise NotImplementedError('fprop not implemented.')
NotImplementedError: fprop not implemented.
and the the executable file is as follows:
"""My changed full source code."""
from future import absolute_import
from future import division
from future import print_function
import os
import warnings
warnings.filterwarnings('ignore')
import sys
sys.path.append('/home/ra/cleverhans/')
from io import BytesIO
import numpy as np
import tensorflow as tf
from tensorflow.contrib.slim.nets import inception
from PIL import Image
from scipy.misc import imsave,imread
from cleverhans.model import Model
from cleverhans.attacks import CarliniWagnerL2
slim = tf.contrib.slim
def load_images(input_dir, batch_shape):
"""Read png images from input directory in batches.
Args:
input_dir: input directory
batch_shape: shape of minibatch array, i.e. [batch_size, height, width, 3]
Yields:
filenames: list file names without path of each image
Lenght of this list could be less than batch_size, in this case only
first few images of the result are elements of the minibatch.
images: array with all images from this batch
"""
images = np.zeros(batch_shape)
filenames = []
idx = 0
batch_size = batch_shape[0]
for filepath in tf.gfile.Glob(os.path.join(input_dir, '*.png')):
print(filepath)
--- change open method . This change has no impact for code ---
with open(filepath,"rb") as f:
  image = np.array(Image.open(f).convert('RGB')).astype(np.float) / 255.0
# Images for inception classifier are normalized to be in [-1, 1] interval.
images[idx, :, :, :] = image * 2.0 - 1.0
filenames.append(os.path.basename(filepath))
idx += 1
if idx == batch_size:
  yield filenames, images
  filenames = []
  images = np.zeros(batch_shape)
  idx = 0

if idx > 0:
yield filenames, images
def save_images(images, filenames, output_dir):
"""Saves images to the output directory.
Args:
images: array with minibatch of images
filenames: list of filenames without path
If number of file names in this list less than number of images in
the minibatch then only first len(filenames) images will be saved.
output_dir: directory where to save images
"""
for i, filename in enumerate(filenames):
# Images for inception classifier are normalized to be in [-1, 1] interval,
# so rescale them back to [0, 1].
with tf.gfile.Open(os.path.join(output_dir, filename), 'w') as f:
img = (((images[i, :, :, :] + 1.0) * 0.5) * 255.0).astype(np.uint8)
Image.fromarray(img).save(f, format='PNG')
---change Model from object---
class InceptionModel(Model):
"""Model class for CleverHans library."""
def init(self, nb_classes):
super(InceptionModel, self).init(nb_classes=nb_classes,
needs_dummy_fprop=True)
self.built = False
def call(self, x_input, return_logits=False):
"""Constructs model and return probabilities for given input."""
reuse = True if self.built else None
with slim.arg_scope(inception.inception_v3_arg_scope()):
# Inception preprocessing uses [-1, 1]-scaled input.
x_input = x_input * 2.0 - 1.0
_, end_points = inception.inception_v3(
x_input, num_classes=self.nb_classes, is_training=False,
reuse=reuse)
self.built = True
self.logits = end_points['Logits']
# Strip off the extra reshape op at the output
self.probs = end_points['Predictions'].op.inputs[0]
if return_logits:
  return self.logits
else:
  return self.probs

def get_logits(self, x_input):
return self(x_input, return_logits=True)
def get_probs(self, x_input):
return self(x_input)
def main(_):
"""Run the sample attack"""
Images for inception classifier are normalized to be in [-1, 1] interval,
eps is a difference between pixels so it should be in [0, 2] interval.
Renormalizing epsilon from [0, 255] to [0, 2].
tensorflow_master = ""
checkpoint_path   = "/home/ra/cleverhans/cw/inception_v3.ckpt"
input_dir         = "/home/ra/cleverhans/cw/tmp"
output_dir        = "/home/ra/cleverhans/cw"
image_width       = 299
image_height      = 299
batch_size        = 50
nb_classes=1001
cw_params = {'batch_size': 1,
'confidence': 10,
'learning_rate': 0.1,
'binary_search_steps': 5,
'max_iterations': 1000,
'abort_early': True,
'initial_const': 0.01,
'clip_min': 0,
'clip_max': 1}
batch_shape = [batch_size, image_height, image_width, 3]
tf.logging.set_verbosity(tf.logging.INFO)
with tf.Graph().as_default():
# Prepare graph
x_input = tf.placeholder(tf.float32, shape=batch_shape)
model = InceptionModel(nb_classes)
ses = tf.Session()
cwl2 = CarliniWagnerL2(model,ses)
x_adv = cwl2.generate_np(x_input,**cw_params)
sess.run(tf.global_variables_initializer())
save_images(x_adv, filenames, output_dir)
# Run computation
saver = tf.train.Saver(slim.get_model_variables())

adv_images = sess.run(x_adv, feed_dict={x_input: images})
save_images(adv_images, filenames, output_dir)
session_creator = tf.train.ChiefSessionCreator(
    scaffold=tf.train.Scaffold(saver=saver),
    checkpoint_filename_with_path=checkpoint_path,
    master=tensorflow_master)
with tf.train.MonitoredSession(session_creator=session_creator) as sess:
  for filenames, images in load_images(input_dir, batch_shape):
      adv_images = sess.run(x_adv, feed_dict={x_input: images})
      save_images(adv_images, filenames, output_dir)

if name == 'main':
tf.app.run()

The error specifies that your model object is missing an fprop method, which probably explains why the code is failing. To see an example of an implementation of a model class, see: https://github.com/tensorflow/cleverhans/blob/f1cf6f06b6f0f68334f7fc5b862b01e439dcdc37/cleverhans/model_zoo/all_convolutional.py#L11

The error specifies that your model object is missing an fprop method, which probably explains why the code is failing. To see an example of an implementation of a model class, see:
https://github.com/tensorflow/cleverhans/blob/f1cf6f06b6f0f68334f7fc5b862b01e439dcdc37/cleverhans/model_zoo/all_convolutional.py#L11

Thank you very much，I have solved it.


Getting parameters from history of spot prices

Hello!
Is there any way to make this model sutable for not simulation of N-prices path with random process, but rather Historical prices?
Thanks!

Hello,
If this is a theoretical question about the rough bergomi model (rather than the specific use of the model in this repo), then the answer is probably yes, but you need to be clearer.
If you are asking whether this actual repo can be used in the context of historical rather than risk-neutral problems in quant finance, then the answer is no, not without some work.
Ryan
Get Outlook for iOShttps://aka.ms/o0ukef

From: hlbkin @.>
Sent: Sunday, December 5, 2021 8:16:56 PM
To: ryanmccrickerd/rough_bergomi @.>
Cc: Subscribed @.***>
Subject: [ryanmccrickerd/rough_bergomi] Getting parameters from history of spot prices (Issue #1)
Hello!
Is there any way to make this model sutable for not simulation of N-prices path with random process, but rather Historical prices?
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/ryanmccrickerd/rough_bergomi/issues/1, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA25733CRHRLLSSICWQRDBLUPPCDRANCNFSM5JNFBLVA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Understood thanks!
The question is more generic, rather than specifically this model - what is the best book/subject to start expoling how to fit generic process to time series historical prices to a given complex random process (like described in this package for example)?

well yes that is a very general question. i am going to assume that by
"complex random process" you mean something like fractional process. in
which case https://arxiv.org/pdf/1610.00332.pdf is a good paper. this
calibrates and compares many such fractional models. if you are new to the
topic, though, i would start by simply trying to calibrate fractional
brownian motion (fBm) to a time series by maximum likelihood estimation.
this amounts to estimating the parameters of multivariate gaussian noise.
see e.g. https://en.wikipedia.org/wiki/Fractional_Brownian_motion and
https://stats.stackexchange.com/questions/351549/maximum-likelihood-estimators-multivariate-gaussian
for a basic intro
On Mon, 6 Dec 2021 at 09:13, hlbkin @.***> wrote:

Understood thanks!
The question is more generic, rather than specifically this model - what
is the best book/subject to start expoling how to fit generic process to
time series historical prices to a given complex random process (like
described in this package for example)?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ryanmccrickerd/rough_bergomi/issues/1#issuecomment-986581327,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2573YOH2M33IVYOCMM2D3UPR5D3ANCNFSM5JNFBLVA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Tochd only processing either CUE/BIN or ISO, not both

When running tochd in a folder containing BIN/CUEs and ISOs, it only processes one format, but not both.
I'm not sure under which conditions one format gets chosen over the other, I have folders where it ignores ISOs and folders where it ignores CUE/BIN files.
Is this expected behavior?
An sample test case, just showing the general problem, not an exhaustive test:
butnut@PAD /mnt/demo % ls -AhgG games
total 4.3G
-rwxrwxrwx 1 587M Dec 24  1996 BIN_PS2game.bin
-rwxrwxrwx 1   74 Sep 20 12:49 BIN_PS2game.cue
-rwxrwxrwx 1 2.8G Dec 24  1996 ISO_PS2game.iso
-rwxrwxrwx 1 978M Sep 20 10:15 ZIPPED_ISO_PS2game.zip

butnut@PAD /mnt/demo % tochd -X -s games
Files in queue: 2
Job 1     Skipped:      /mnt/demo/games/BIN_PS2game.cue
Job 2     Skipped:      /mnt/demo/games/ZIPPED_ISO_PS2game.zip
Started: 0
Skipped: 2
Failed: 0
Completed: 0
Elapsed time: 0:00:00

butnut@PAD /mnt/demo % mv games/*.bin games/*.cue hidefiles

butnut@PAD /mnt/demo % ls -AhgG games
total 3.7G
-rwxrwxrwx 1 2.8G Dec 24  1996 ISO_PS2game.iso
-rwxrwxrwx 1 978M Sep 20 10:15 ZIPPED_ISO_PS2game.zip

butnut@PAD /mnt/demo % tochd -X -s games
Files in queue: 2
Job 1     Skipped:      /mnt/demo/games/ISO_PS2game.iso
Job 2     Skipped:      /mnt/demo/games/ZIPPED_ISO_PS2game.zip
Started: 0
Skipped: 2
Failed: 0
Completed: 0
Elapsed time: 0:00:00

System information:
butnut@PAD /mnt/demo % tochd --version
tochd v0.13
butnut@PAD /mnt/demo % uname -a
Linux PAD <IP_ADDRESS>-microsoft-standard-WSL2 #1 SMP Fri Mar 29 23:14:13 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
butnut@PAD /mnt/demo % lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 22.04.5 LTS
Release:        22.04
Codename:       jammy

In the example BIN_PS2game.bin, BIN_PS2game.cue and ISO_PS2game.iso do NOT come from ZIPPED_ISO_PS2game.zip.
These are all 3 different games, one is a ISO in a ZIP, one is in ISO format and one has a CUE and a BIN file, as you can see by looking at the filesize of the ls command.
I'm aware of the workaround you speak off from reading the readme, but it seems like it leads to unexpected side effects like I demonstrated above.
Ideally the workaround should only look at the files and determine if they are even related to each other. Currently it just assumes they are related.

I actually agree with you. I actually wanted to do this properly, but could not get it to work correctly when writing the script.
The best case would be to actually read the .cue files and analyzed the paths (which can be either Windows or POSIX/Linux paths BTW) and are formatted in different ways, because there are different .cue formats (and .gdi). And it needs to work with the temporary files too. Because I never needed this, I went with this (lazy) workaround and crossed the fingers.
So for now there is no quick fix for this unfortunately. I will look into it again, if this can be resolved. But it needs a bit more rewrite and can take time. It's definitely something I would like to see solved myself.


Introduce a CMake build control file

Using CMake (or some other "makefile maker" style build system) eases comparisons among different compilers, and more easily incorporates third party libraries.

Great, thank you!


Compile error after updating to version 2.5.0

After updating to version 2.5.0 I have the following errors:
D:/flutter2.10/.pub-cache/hosted/pub.dartlang.org/persian_datetime_picker-2.5.0/lib/src/ptime_picker.dart:813:48: Error: Property 'systemFonts' cannot be accessed on 'PaintingBinding?' because it is potentially null.
 - 'PaintingBinding' is from 'package:flutter/src/painting/binding.dart' ('/D:/flutter2.10/packages/flutter/lib/src/painting/binding.dart').
Try accessing using ?. instead.
  }) : super(repaint: PaintingBinding.instance.systemFonts);
                                               ^^^^^^^^^^^
/D:/flutter2.10/.pub-cache/hosted/pub.dartlang.org/persian_datetime_picker-2.5.0/lib/src/cupertino/cupertino_date_picker.dart:668:30: Error: Property 'systemFonts' cannot be accessed on 'PaintingBinding?' because it is potentially null.
 - 'PaintingBinding' is from 'package:flutter/src/painting/binding.dart' ('/D:/flutter2.10/packages/flutter/lib/src/painting/binding.dart').
Try accessing using ?. instead.
    PaintingBinding.instance.systemFonts.addListener(_handleSystemFontsChange);
                             ^^^^^^^^^^^
/D:/flutter2.10/.pub-cache/hosted/pub.dartlang.org/persian_datetime_picker-2.5.0/lib/src/cupertino/cupertino_date_picker.dart:681:30: Error: Property 'systemFonts' cannot be accessed on 'PaintingBinding?' because it is potentially null.
 - 'PaintingBinding' is from 'package:flutter/src/painting/binding.dart' ('/D:/flutter2.10/packages/flutter/lib/src/painting/binding.dart').
Try accessing using ?. instead.
    PaintingBinding.instance.systemFonts
                             ^^^^^^^^^^^
/D:/flutter2.10/.pub-cache/hosted/pub.dartlang.org/persian_datetime_picker-2.5.0/lib/src/cupertino/cupertino_date_picker.dart:1226:30: Error: Property 'systemFonts' cannot be accessed on 'PaintingBinding?' because it is potentially null.
 - 'PaintingBinding' is from 'package:flutter/src/painting/binding.dart' ('/D:/flutter2.10/packages/flutter/lib/src/painting/binding.dart').
Try accessing using ?. instead.
    PaintingBinding.instance.systemFonts.addListener(_handleSystemFontsChange);
                             ^^^^^^^^^^^
/D:/flutter2.10/.pub-cache/hosted/pub.dartlang.org/persian_datetime_picker-2.5.0/lib/src/cupertino/cupertino_date_picker.dart:1245:30: Error: Property 'systemFonts' cannot be accessed on 'PaintingBinding?' because it is potentially null.
 - 'PaintingBinding' is from 'package:flutter/src/painting/binding.dart' ('/D:/flutter2.10/packages/flutter/lib/src/painting/binding.dart').
Try accessing using ?. instead.
    PaintingBinding.instance.systemFonts
                             ^^^^^^^^^^^
/D:/flutter2.10/.pub-cache/hosted/pub.dartlang.org/persian_datetime_picker-2.5.0/lib/src/cupertino/cupertino_date_picker.dart:1590:31: Error: Method 'addPostFrameCallback' cannot be called on 'SchedulerBinding?' because it is potentially null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('/D:/flutter2.10/packages/flutter/lib/src/scheduler/binding.dart').
Try calling using ?. instead.
    SchedulerBinding.instance.addPostFrameCallback((Duration timestamp) {
                              ^^^^^^^^^^^^^^^^^^^^
/D:/flutter2.10/.pub-cache/hosted/pub.dartlang.org/persian_datetime_picker-2.5.0/lib/src/cupertino/cupertino_date_picker.dart:1800:30: Error: Property 'systemFonts' cannot be accessed on 'PaintingBinding?' because it is potentially null.
 - 'PaintingBinding' is from 'package:flutter/src/painting/binding.dart' ('/D:/flutter2.10/packages/flutter/lib/src/painting/binding.dart').
Try accessing using ?. instead.
    PaintingBinding.instance.systemFonts.addListener(_handleSystemFontsChange);
                             ^^^^^^^^^^^
/D:/flutter2.10/.pub-cache/hosted/pub.dartlang.org/persian_datetime_picker-2.5.0/lib/src/cupertino/cupertino_date_picker.dart:1816:30: Error: Property 'systemFonts' cannot be accessed on 'PaintingBinding?' because it is potentially null.
 - 'PaintingBinding' is from 'package:flutter/src/painting/binding.dart' ('/D:/flutter2.10/packages/flutter/lib/src/painting/binding.dart').
Try accessing using ?. instead.
    PaintingBinding.instance.systemFonts
                             ^^^^^^^^^^^
/D:/flutter2.10/.pub-cache/hosted/pub.dartlang.org/persian_datetime_picker-2.5.0/lib/src/cupertino/cupertino_date_picker.dart:2063:31: Error: Method 'addPostFrameCallback' cannot be called on 'SchedulerBinding?' because it is potentially null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('/D:/flutter2.10/packages/flutter/lib/src/scheduler/binding.dart').
Try calling using ?. instead.
    SchedulerBinding.instance.addPostFrameCallback((Duration timestamp) {
                              ^^^^^^^^^^^^^^^^^^^^

Here is the flutter doctor summary:
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 2.10.5, on Microsoft Windows [Version 10.0.19044.1706], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
[√] Chrome - develop for the web
[X] Visual Studio - develop for Windows
    X Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components
[√] Android Studio (version 2021.1)
[√] IntelliJ IDEA Ultimate Edition (version 2021.3)
[√] Connected device (4 available)
[√] HTTP Host Availability

I have this problem

version 2.5.0 is for Flutter 3
for flutter 2.8 and below you should use persian_datetime_picker 2.4.0


RUM-6093:Add TimeBank in session replay recorder for dynamic optimisation

What does this PR do?
Add TimeBank in session replay recorder for dynamic optimisation.
The main idea is to have a time balance in this TimeBank, every execution of session replay recording will consume this balance, and the balance will be recharged during the time passing until a maximum balance is reached.
Before every execution of recording, if the balance is negative, the execution will skipped until the balance is recharged to positive.
Motivation
RUM-6093
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report
Attention: Patch coverage is 82.14286% with 5 lines in your changes missing coverage. Please review.

Project coverage is 70.01%. Comparing base (679de59) to head (705fa22).

Files with missing lines
Patch %
Lines

...droid/sessionreplay/internal/recorder/Debouncer.kt
66.67%
4 Missing and 1 partial :warning:

Additional details and impacted files
@@                         Coverage Diff                         @@
##           feature/sr-dynamic-optimisation    #2247      +/-   ##
===================================================================
- Coverage                            70.12%   70.01%   -0.11%
===================================================================
  Files                                  727      728       +1
  Lines                                27062    27085      +23
  Branches                              4560     4562       +2
===================================================================
- Hits                                 18975    18962      -13
- Misses                                6827     6842      +15
- Partials                              1260     1281      +21

Files with missing lines
Coverage Δ

...ssionreplay/internal/recorder/RecordingTimeBank.kt
100.00% <100.00%> (ø)

...nternal/recorder/listener/WindowsOnDrawListener.kt
97.22% <100.00%> (ø)

...droid/sessionreplay/internal/recorder/Debouncer.kt
81.48% <66.67%> (-18.52%)
:arrow_down:

... and 25 files with indirect coverage changes


2023-05-10 updated VScode, now powershell changed to orange color with error

Version: 1.78.1 (user setup)
Commit: 6a995c4f4cc2ced6e3237749973982e751cb0bf9
Date: 2023-05-04T09:48:08.683Z
Electron: 22.5.1
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19044
Sandboxed: No

Steps to Reproduce:

It's telling you the issue here:


Goodbyedpi suddenly not working. I can't enter a lot websites...

Operating system / операционная система
Windows 10  21H2
Running as service / Запуск программы как сервис
I installed it as a service / Установил как сервис Windows
Describe the issue / Опишите проблему
Goodbyedpi suddenly not working. I can't enter a lot websites... It was good until 2 day ago. Now it's complately broken.
Please help me .
Additional information / Дополнительная информация
No response

https://ntc.party/c/community-software/goodbyedpi


Fix a few missing entries and a broken link in the Table of Contents.

I noticed a few entries were missing in the TOC, so I did a quick scan to pull out the link targets:
grep "#### <a name" 3.0.md | grep -v "#####" | sed -e "s/^.*\"\(.*\)\".*$/\1/"
That gave me a list of objects which I used to update the TOC. I also noticed that the linkObject anchor was "#linkObject", which seemed to break the TOC link in the current doc.

The # fixes #956 (I just some minutes ago opened #966 for this, with the same change, not having seen this PR).
I checked the new links in the TOC, all work.
:+1:


Shift+Click then Copy does not preserve newlines

Environment
Windows build number: Microsoft Windows [Version 10.0.18362.657]
Windows Terminal version (if applicable): Version: 0.9.433.0

Steps to reproduce

Select some text
Shift + Click to select more text across multiple terminal lines (this is the important step, clicking and dragging to select does NOT produce the same effect).
Copy (if you have copy on select enabled this is done automatically)
Paste into any text editor

Expected behavior
If this was the text that was selected:
C:\Users\nicholao>
C:\Users\nicholao>
C:\Users\nicholao>
C:\Users\nicholao>
C:\Users\nicholao>

this should be pasted:
C:\Users\nicholao>
C:\Users\nicholao>
C:\Users\nicholao>
C:\Users\nicholao>
C:\Users\nicholao>

Actual behavior
Width of terminal determines how much white space is between each line, and there are no newlines present in the pasted content:
C:\Users\nicholao>                                                                   C:\Users\nicholao>                                                                   C:\Users\nicholao>                                                                   C:\Users\nicholao>                                                                   C:\Users\nicholao>

This is likely because..
https://github.com/microsoft/terminal/blob/2f60cf0e9138f0d788f0a7bf189ee4116a701a4d/src/cascadia/TerminalControl/TermControl.cpp#L980
https://github.com/microsoft/terminal/blob/2f60cf0e9138f0d788f0a7bf189ee4116a701a4d/src/cascadia/TerminalControl/TermControl.cpp#L988
"shift" is being held down during the pointer release, which triggers "no newlines" mode. hm.

For anyone else that comes across this issue right now, there is a work around: release Shift before releasing your mouse button. As long as you can consciously remember to do things in that order, you can preserve newlines.


Refactor the OCM library

Epic: Refactor the OCM library
Description
To get the OCM project ready for the next phase of adoption, which targets both, commercial players and other opensource projects, the OCM library should be refactored. This is essential to keep the current momentum, but also to keep the project alive over a longer time and to quickly attract more people to contribute to the project.
The goals for refactoring are:

Define public API (aka v2)
Create a graphical overview of the API to be able to better discuss the new API
Try to return concrete types, not interfaces
Ensure the code and its structure are easy accessible to and understandable by newcomers
Each function and package has documentation and examples
Do not force technology choices on users through dependencies (e.g. gingko in env package)
Define/capture areas where extensibility is necessary
Respect the original design goals

A topic that needs to be decided before going on is if the OCM CLI as reference implementation of the OCM library should stay in the ocm repository or if it should be moved to an own, dedicated GitHub repository.
Steps for the refactoring

Definition of public API aka v2 under github.com/open-component-model/ocm/v2
Rewire existing v1 functionality to v2
(Move CLI to dedicated repository github.com/open-component-model/ocmcli) <-- decision pending
Refactor CLI so that it only consumes v2
Gradually replace v1 wirings with new v2 implementation

Existing issues from OCM simplification workshop related to the OCM lib (to be checked if still valid)

[ ] open-component-model/ocm-project#120
[ ] open-component-model/ocm-project#114
[x] open-component-model/ocm-project#106
[ ] open-component-model/ocm-project#102
[x] open-component-model/ocm-project#100
[ ] open-component-model/ocm-project#44

remove EPIC and just keep distinct issues in backlog.


fix: alter sql columns to long text #7463

Merge lyft-release-sp8 to lyft-develop

Codecov Report

Merging #33 into lyft-develop will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           lyft-develop      #33   +/-   ##
=============================================
  Coverage         64.95%   64.95%
=============================================
  Files               433      433
  Lines             21357    21357
  Branches           2352     2352
=============================================
  Hits              13873    13873
  Misses             7356     7356
  Partials            128      128

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 dfdf09b...69e5f21. Read the comment docs.