Dynamically mounting a component?

Is there some example code on how to dynamically mount a component to a certain element during run-time? For example, using a button?
Thanks

A component can only be mounted through it being an element. Mounting one through a button could be done like this:

  <example m-if="mounted"></example>
  <button m-on:click="toggle">Mount</button>

Moon.component("example", {
  template: "
I'm an example component
"
});

new Moon({
  el: "#app",
  data: {
    mounted: false
  },
  methods: {
    toggle: function() {
      this.set('mounted', !this.get('mounted'));
    }
  }
});

Here is a jsFiddle demo

That defeats part of the flexibility of having templates and components, how would I be able to handle an application the can have large forms and sub apps if they are only to be seen if the user actually needs them, it would be sub optimal to try to render every possible combination of components beforehand.

A component can manage it's own state, and there is no other way for Moon to know if a component needs to be mounted.
This is exactly how you would accomplish it in Vue or React, and is how components work. You can't expect Moon to know how your application works. It is your responsibility to have the data needed to know when to render components.
If you'd like a router, check out moon-router


feat: custom Microsoft need to have input text for tts voice

#307

@davehorton do you reckon this will be merged soon? Or is there an issue still?


Clean up layout markup

The grid was thrown together very quickly. It could really do with some tidying up.
(Note for Garrett: port over CUTS improvements)

Fixed in v2.0.0


Backup key derivation uses older OWASP standard

There is an updated OWASP recommendation for PBKDF2 key derivation. We should move to it for backups.

Addressed in PR 11.

See issue 8, which tracks eventual use of Argon2 for this purpose.


[SP-5225] Backport of PDI-18073 - Bulk Loading steps returning NPE if previous steps does not have rows (8.3 Suite)

@bcostahitachivantara @RPAraujo
Part of a set of PRs:

https://github.com/pentaho/pentaho-kettle/pull/6835
https://github.com/pentaho/pdi-teradata-tpt-plugin/pull/50

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine,plugins/gpload/core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine,plugins/gpload/core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/trans/steps/orabulkloader/OraBulkLoader.java
  engine/src/test/java/org/pentaho/di/trans/steps/orabulkloader/OraBulkLoaderTest.java
  plugins/gpload/core/src/main/java/org/pentaho/di/trans/steps/gpload/GPLoad.java
  plugins/gpload/core/src/test/java/org/pentaho/di/trans/steps/gpload/GPLoadTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.job.entries.checkdbconnection.JobEntryCheckDbConnections

Branch Change: -1.3% :small_red_triangle_down:
Complexity Change: -1.5% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Branch Change: -1.2% :small_red_triangle_down:
Complexity Change: -1.2% :small_red_triangle_down:

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.1% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.step.StepMeta

Branch Change: -0.6% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Method Change: -1% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -3.9% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:
Line Change: -0.4% :small_red_triangle_down:

org.pentaho.di.trans.steps.processfiles.ProcessFilesMeta

Branch Change: -3.5% :small_red_triangle_down:
Complexity Change: -2.1% :small_red_triangle_down:
Instruction Change: -0.4% :small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Complexity Change: -1.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -1.8% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.googleanalytics.GaInputStepMeta

Branch Change: -1.9% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.gpload.GPLoadMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -0.2% :small_red_triangle_down:


Unable to map ManageIQ container port 443 to docker host public IP.

Hello,
Used the document - https://www.manageiq.org/docs/get-started/docker in downloading and running manageiq as a container.
Its a remote server without GUI in it. So map host IP to container port like below.
# docker run -d -p <host-ip>:8443:443 manageiq/manageiq:ivanchuk-1

# docker ps
CONTAINER ID        IMAGE                          COMMAND                  CREATED             STATUS                   PORTS                                         NAMES
ac64279b7199        manageiq/manageiq:ivanchuk-1   "/usr/local/bin/dumb…"   55 seconds ago      Up 52 seconds (Paused)   22/tcp, 80/tcp, <host ip>:8443->443/tcp   cocky_davinci

When I try to access the  host ip: 8443, getting unable to connect message in the browser.
Netstat shows port is listening correctly.
# netstat -aptun | grep 8443
tcp        0      0 <host-ip>:8443      <IP_ADDRESS>:*               LISTEN      15796/docker-proxy

Even stopped running firewalld, still same issue. How to fix this issue.
Thanks,
Mohan

@carbonin Can you look into this one?  cc @simaishi

@MohanrajBabu can you get a process list from inside the container? docker exec ac64279b7199 ps -leaf --forest
I was having some issues recently with httpd not starting in the container.

Closing this is there has been no response for additional information.


Fix location flag; Improved PNNxPSN sitedb apis

Fixes a bug of empty location when useSiteListAsLocation is enabled.
Added a couple siteDB apis.
I changed the return format of PNNtoPSN, it was returning a dict where the key was the PNN you were trying to map. I could not see any reason not to return a simple list with the PSNs, @alexanderrichards let me know if I'm wrong.
@ticoann welcome back from vacation ;-) Do not merge yet please.
My tests are still ongoing, but everything looks good so far. I fixed the emulator as well, hopefully no unittests will scream

test this please

Damnit! I was going to tell you it's ready to merge :-( Let me fix these unittests first and then we merge it tomorrow if everything is green

test this please

test this please

All the tests above are failing with empty input location
Invalid WMSpec: 'MultiTaskReReco': Input data has no locations "/Cosmics/ComissioningHI-PromptReco-v1/RECO
Invalid WMSpec: 'ReRecoWorkload': Input data has no locations "/MinimumBias/ComissioningHI-v1/RAW"
Invalid WMSpec: 'testParentProcessing': Input data has no locations "/SingleMu/CMSSW_6_2_0_pre4-PRE_61_V1_RelVal_mu2012A-v1/RECO"

because, AFAIR from a previous discussion, the emulator is not thread safe and thus it ends up calling the cmsweb services in jenkins. Since those data are only at *_MSS or *_Export, it cannot find input location sites. Anyways, only one test changed from success to error. All the others are from failure to error.
I'll look for other datasets with valid location, though at some point they may be not accessible as well.
Otherwise I do not have anything else to add to this PR.

Thanks Alan, patch looked good to me I will merge this. As far as unittest goes using Emulator we will fix it separately.


Support Gitalk

Provide Gitalk comment support.

Currently Gitalk has a serious issue: https://github.com/gitalk/gitalk/issues/102

Useful link: add gitment module by sslyxhz · Pull Request #36 · olOwOlo/hugo-theme-even

Now, Jane-theme support utterances comment which I think is better comment base on github issues.
https://github.com/xianmin/hugo-theme-jane/issues/74


Pretty times and dates (relative & absolute)

GitHub does an excellent job with that, shows times like this

5 seconds ago
20 hours ago
Feb 12

We should do something similar, maybe they even use a Ruby library for it.
We also have to consider the browser's user agent when formatting dates and times

Actually, let's not start work on this yet, as I think GitHub also renders that on the front-end, so it will probably be related to our front-end framework cc @nakhbari


subset

@Nivedita967  i have solved issue #26

@Mahindra-7 Nice code. Do comment your language in which you are working in issue comment section so that we don't get confused.


Documentation out of date

Domain access approval process has changed at GoDaddy. Users no longer have the approve button. HTML / Domain validation are the one ways to approve now.
Kindly update the documentation.

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

ID: fa7e21c2-d024-dda5-a468-b140422b9fae
Version Independent ID: 5475c8e6-1186-8ed1-2ad6-6eaa2cac871f
Content: Buy and Configure an SSL Certificate for your Azure App Service
Content Source: articles/app-service/web-sites-purchase-ssl-web-site.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: apurvajo;cephalin

@nmallick1 Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Docs are out of date. There is no "SSL Certificate" option anymore, it is "SSL Settings" and that doesn't provide an option to "Import Certificate" at all.


Should not clobber new node fields (like taints)

Right now the components use Update, not Patch.
See https://github.com/coreos/flannel/issues/667 for why this is a problem.
We need to switch to patch, ideally sooner rather than later.

Container Linux is deprecated. Closing this PR as stale.


Expose new new verify method signature.

Hi Auth0 team :)
recently I found that it would be quite useful to have following method signature,
public DecodedJWT verify(DecodedJWT decodedJWT)

I have use case for first, checking some stuff inside JWT token, then, in some cases I want to check signature - to not waste CPU cycles, it would be good to have verify method that accepts DecodedJWT instance.
Pull request with change is here : https://github.com/auth0/java-jwt/pull/280
Thanks !

This is useful when using JWKS but not KeyProvider. Unfortunately KeyProvider provides no means for dealing with exceptions that may occur when retrieving the key.

How about using this custom validator I just created? https://github.com/auth0/java-jwt/pull/290

From my perspective, it would be simpler to just call method that already is there :)

@rocketraman a new custom validation would not solve the case where we need to read the contents of the JWT before we can verify it. This issue makes sense because we would not have to parse the JWT a second time.

@rocketraman no, sometimes you have to decode the JWT before you can verify it. If it is signed with the RS246 algorithm and the public key is available via JWKS, it is necessary to decode the JWT to retrieve the key ID so the public key can be looked up in the providers' well-known location for publishing JWKS public keys.

@martinoconnor  Yes, but that is what the KeyProvider is for. I just used this myself to validate tokens from Microsoft Azure AD. That is acceptable because its a limited purpose hook, not a general "give me everything and let me decide what to do".

guys - @martinoconnor , @rocketraman - are you sure that someone still maintains that project ? XD

@rocketraman

This PR is not for allowing the JWT to be decoded without verification, that functionality has long existed. This PR allows a previously decoded JWT to subsequently be verified. You should probably actually look at the PR source
The KeyProvider does not currently handle exceptions very well, for example, if there are network issues while accessing the JWKS, there is not really a good way to handle it yet

@KowalczykBartek I think @lbalmaceda maintains this project

This PR is not for allowing the JWT to be decoded without verification, that functionality has long existed. This PR allows a previously decoded JWT to subsequently be verified. You should probably actually look at the PR source

Fair enough. Though note the statement in the docs: "Use it only if you trust the token or you already verified it." I'll leave it at that -- this is all mostly O/T anyway.

2. The KeyProvider does not currently handle exceptions very well, for example, if there are network issues while accessing the JWKS, there is not really a good way to handle it yet

With appropriate caching this should happen quite rarely. Why not just throw a specialized JWTVerificationException? It's likely you'll need to treat this the same way as an invalid JWT anyway, and if not you can always catch your specialized exception and do something else.

Yes, but that is what the KeyProvider is for.

I agree with @rocketraman here.
@KowalczykBartek @martinoconnor wouldn't it be better if the KeyProvider is refactored, so that it now throws an unchecked exception that the Algorithm can handle and wrap into SignatureVerificationException? Note as @rocketraman mentions above, not being able to obtain the key (for whatever reason) would mean the signature won't be able to be verified so that's a fail scenario as well. How would you expect the library to behave in such a case?

@lbalmaceda honestly I am out of discussion at the moment, it somehow changed direction :) I just wanted to have method that will do some specific stuff according to existing logic.

How would you expect the library to behave in such a case?

I expect that nothing change :)

It just verify a string token or a object token, just it. Not trust token, as you mind, verify the string token you have trusted the string token, so object token with string token has any difference.

The problem with allowing people to just decode the token without a following verification is that they might start to use the token claims as trusted data immediately after they decode it. And that's an error: No data should not be trusted at all until the token's signature is verified. You might know the risks and would try to decode it and if required, verify the signature yourself. But most of the users don't know or don't read those warnings; I rather have a library that can protect those users from misuse than having to open the API in the other direction. For the next major I plan to have something like this that allows users to check existence of claims but not obtain its value until the token is verified. Doing this today would be a breaking change since methods signature would change.
The only use case this library and the spec that it tries to implement support is using the key id claim to fetch the proper JWK on demand (RS algorithms). That's what the KeyProvider interface is here for, and I'm willing to refactor it as much as required to fix whatever mistakes you find on its implementation.

Was an architectural decision just made based on the presumption the user base being incompetent rather than an expressed use case?

`
public DecodedJWT verify(String token) throws JWTVerificationException {
    DecodedJWT jwt = JWT.decode(token);
    verifyAlgorithm(jwt, algorithm);
    algorithm.verify(jwt);
    verifyClaims(jwt, claims);
    return jwt;
}

`
just call this line DecodedJWT jwt = JWT.decode(token); by myself, why cannot? It's not influence the verify logic.
In java world, every thing is object, so verify a DecodedJWT object is a right way, not a plain string token. Even the string token verify method, the code also decode string to an DecodedJWT object first, then verify the DecodedJWT object, It just move decode method by my hand, it cannot , it's very funny！

@lbalmaceda looks like the OP got frustrated and decided to leave. Can we reopen this issue to keep the conversation going here?


dynamic_rest.patches.patch_prefetch_one_level - too many values to unpack

Django==2.0.3
DRF==3.7.7
The dynamic_rest.patches.patch_prefetch_one_level patch is incompatible.
prefetcher.get_prefetch_queryset returns ValueError: too many values to unpack (expected 5)

Сaught the same error. Try to switch version to master or reinstall dynamic-rest lib with @master branch.

Thanks

Just released 1.8.0 which adds support for Django 2.0 and DRF 3.7.


Pass auth to Active RM check

This is to fix #57
One thing I'm still not sure about is next:
https://github.com/dimon222/hadoop-yarn-api-python-client/blob/82022a1574c6134ade132736ef64d642053b13dc/yarn_api_client/hadoop_conf.py#L52-L57
Should we validate for 401 and throw exception for auth immediately or its valid scenario when we should return just False?

@kevin-bates i've applied the suggested change. Logger is not accessible from method, so it's printing to stdout for time being.

I've validated new active RM check on existing cluster with such setup and it seems to work.
One suggestion is still have in mind - response.text is showing very verbose output of page markup (basically html). Maybe it makes more sense to keep only the status code in log instead of full response body?

Yes - that sounds good.  Thank you for checking this!

Adjusted the print statement. Hopefully nothing else missed.

@lresende any chance u can confirm as well?


Get manila CR before checking the condition state of the api resource

The condition never met as the manila CR needs to be fetched.

/lgtm


API naming for LG

fixes #1879
API naming for LG

Rename 'LGFile' to "Templates"
Rename "CreateActivity" in "ActivityFactory" to "FromObject"
Rename "SimpleMultiLangGenerator.cs " to "MultiLanguageLG.cs" and move into LG library,
and remove LG library and ILanguageGenerator dependencies
Rename "EvaluateTemplate" to "Evaluate", and the original "Evaluate" is replaced by a new name "EvaluateText"

Pull Request Test Coverage Report for Build 112173

292 of 403 (72.46%)  changed or added relevant lines in 16 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.6%) to 81.751%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/botbuilder-lg/src/analyzer.ts
7
8
87.5%

libraries/botbuilder-lg/src/templateParser.ts
24
25
96.0%

libraries/botbuilder-lg/src/templates.ts
16
17
94.12%

libraries/botbuilder-lg/src/errorListener.ts
2
4
50.0%

libraries/botbuilder-lg/src/extractor.ts
0
2
0.0%

libraries/botbuilder-lg/src/staticChecker.ts
31
34
91.18%

libraries/botbuilder-lg/src/evaluator.ts
22
27
81.48%

libraries/botbuilder-lg/src/activityFactory.ts
140
146
95.89%

libraries/botbuilder-lg/src/multiLanguageLG.ts
2
37
5.41%

libraries/botbuilder-lg/src/expander.ts
34
89
38.2%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/diagnostic.ts
1
84.21%

libraries/botbuilder-lg/src/range.ts
3
30.77%

Totals

Change from base Build 111862:
-0.6%

Covered Lines:
9293

Relevant Lines:
10954

💛  - Coveralls

A quick update on this: @tomlm and @stevenic are working to drive this to closure today, so we can push out the RC candidate of the JS SDK.


json log handler

https://github.com/ServiceWeaver/weaver/blob/2b260e769cc47dd4058befa4087b24e9fb170236/internal/weaver/singleweavelet.go#L334C1-L335C1
Are there any plans to provide the option to use json log in the future? Or how to use json log instead by now(v0.22.0)?

This work has been deprioritized for now


Support for Enum Values

Enum values don't seem to be supported currently, a minimal example:
type TestType = {
  enumVar: "val1" | "val2";
};

export const Test: toZod<TestType> = z.object({
  enumVar: z.enum(["val1", "val2"]),
});

Produces the following:
Type 'ZodObject<{ enumVar: ZodEnum<["val1", "val2"]>; }, "passthrough", ZodTypeAny, { enumVar: "val1" | "val2"; }, { enumVar: "val1" | "val2"; }>' is not assignable to type 'ZodObject<{ enumVar: never; }, "passthrough", ZodTypeAny, TestType, TestType>'.
  Type '{ enumVar: z.ZodEnum<["val1", "val2"]>; }' is not assignable to type '{ enumVar: never; }'.
    Types of property 'enumVar' are incompatible.
      Type 'ZodEnum<["val1", "val2"]>' is not assignable to type 'never'.

I suspect this may be related to #4 as under the hood, ZodEnums are just ZodUnions.

Here's a PR with a version that supports enums as well: https://github.com/colinhacks/tozod/pull/27

Do we have an update on this? I am trying to use toZod with a type that has an enum prop. After toZod enum is never.

Is there an update ?


Why Context Precision and Context Recall are yielding a binary score instead a range between 0 and 1?

@shahules786 can you help out here?
@Anjalisoni99-byte which models are you using for evaluations

@jjmachan I am using GPT3.5 for evaluation

Hey @Anjalisoni99-byte Are you providing more than one chunk as context in your samples? If not, according to the formula it will be only 0 or 1.
Please close the issue if this is the case.

Hello @shahules786 We have a retriever which collects data from top 5 documents and passed as context for QA. So , yes we can say we are providing more than one chunk as context.

@Anjalisoni99-byte
Are you using any tracing tool to validate the evaluations? can you share a sample data point as a JSON file for me to inspect?

Hello @shahules786  and @jjmachan , I was wondering if we could break that context into smaller simpler sentences as a part of context precision and recall and then do the evaluation, similar to what has be done with faithfulness?
Do you think we can do that?

@sahusiddharth Sure, then you'll also have to think about an ideal way to break up the context so that it won't affect the results in a negative way. This process can be done prior to sending the chunks into ragas to evaluate.


Improve package options for modflow 6

In GitLab by @i.farias.g on Apr 8, 2021, 18:19
Right now each package created supports a limited amount of options. Some of them are easily activated via modifying directly the written text files. A brief list of the ones I've encountered (and having to change manually):

[x] - Activate storage coefficient use in storage package. Currently only supports specific yield and storage but by just adding the option it can be easily changed.
[x] - Change when the output control saves results. Currently defaults to ALL but saving only the last time step of a stress period is useful for large models.
[x] - Input time discretization so that time steps don't default to 1. Currently, by creating tdis with specific dates you can only control the length and number of stress periods but not the time steps within them. This apparently is already possible but the documentation is lacking and only shows floats as inputs to the time_discretization object.

In GitLab by @i.farias.g on Apr 8, 2021, 18:19
marked the checklist item - Activate storage coefficient use in storage package. Currently only supports specific yield and storage but by just adding the option it can be easily changed. as completed

In GitLab by @JoerivanEngelen on May 10, 2021, 13:38
created merge request !116 to address this issue

In GitLab by @JoerivanEngelen on May 10, 2021, 14:26
marked the checklist item - Activate storage coefficient use in storage package. Currently only supports specific yield and storage but by just adding the option it can be easily changed. as completed

In GitLab by @JoerivanEngelen on May 10, 2021, 19:40
marked the checklist item - Change when the output control saves results. Currently defaults to ALL but saving only the last time step of a stress period is useful for large models. as completed

In GitLab by @i.farias.g on Apr 8, 2021, 18:19
marked the checklist item - Activate storage coefficient use in storage package. Currently only supports specific yield and storage but by just adding the option it can be easily changed. as incomplete

In GitLab by @JoerivanEngelen on May 10, 2021, 19:40
marked the checklist item - Input time discretization so that time steps don't default to 1. Currently, by creating tdis with specific dates you can only control the length and number of stress periods but not the time steps within them. This apparently is already possible but the documentation is lacking and only shows floats as inputs to the time_discretization object. as completed

In GitLab by @JoerivanEngelen on May 10, 2021, 19:48
The merge request has the following changes:

I created separate packages for SpecificStorage and StorageCoefficient, concurrent with imodflow

Output Control settings can now be set to "first", "last", "all", and "frequency".

Added documentation on time_discretization. It was possible, but not documented. Reason why we do not let n_timesteps be set with the functions is because when a n_timesteps argument would be proviced, the amount of stress period is not known yet. So a user could in theory provide a DataArray with an amount of stress periods which differs from the amount of stress periods determined by time_discretization.

Example code to set n_timesteps:
simulation = imod.mf6.Modflow6Simulation("example")
simulation.time_discretization(times=["2000-01-01", "2000-01-02"])
# Set number of timesteps
simulation["time_discretization"]["n_timesteps"] = 5

In GitLab by @Huite on Jun 1, 2021, 14:41
mentioned in commit 050bb9052fddcb671083cfece4cfad85e5911e0f


Integer utilities module

A new module providing additional functionality on the standard library integer types (and more generally, any fixed-width integer type), including:

bitwise rotation
full add / sub (with carryin/carryout)
saturating arithmetic (functions, not operators)
division with rounding control
"true" mod function (flooring remainder, hard error if modulus is negative)
division by constant (precompute magic multiply and shift a la libdivide)
modulus by constant (use Lemire's method)

Is bit reverse fit on this?
The FFT algorithm required bit reversal
https://en.wikipedia.org/wiki/Bit-reversal_permutation
https://github.com/apple/swift-numerics/blob/796a0dc3f74a60f102937f61cc363644513671d1/Sources/Accelerate/bit_reverse.swift

Definitely!

Would basic number-theory functions be in scope for this module?

gcd
lcm
Bézout coefficients
Modular inverse

Yes, absolutely.


SW-4728 Do not show projects filter in inventory table views when there are no projects, refresh projects when the component loads

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

Current dependencies on/for this PR:

main

PR #2096

PR #2097

PR #2098   👈

This stack of pull requests is managed by Graphite.

Merge activity

Jan 26, 5:35 PM: @nickgraz started a stack merge that includes this pull request via Graphite.


Test mem alloc fixes

Fixes #20598

minor change in CMakeLists.txt to copy globals from libc_nano.a
tests/lib/mem_alloc updates for coverage increase
minor typos and fixes

@galak I added backport label for LTS - not sure if this is required (i am not sure of the status of newlib libc nano in LTS). you can comment.

Removed the last commit that updated the test-cases .yaml, in the wake of @stephanosio 's feedback.
Was tested by @hakehuang , see comment in #20598


Validation errors

Hello! Thank you to whoever is kind enough to read this and help me out. I've been following the guide for making a discord bot and i'm stuck at the spot where it asks me to add a Interactive Endpoint URL and i click the share button within glitch and then add the /interactions and try to save, but it says

Validation errors:
interactions_endpoint_url: The specified interactions endpoint url could not be verified.

Greetings! Typically when folks get this error, it's because they haven't updated their .env to include the right information.  Have a look here, and make sure you're following the steps in the getting started guide!
https://github.com/discord/discord-example-app?tab=readme-ov-file#get-app-credentials
It could also be a result of the tunnel you're setting up not working.  After you start ngrok are you able to reach the site by directly navigating to https://<your_tunnel_url>/interactions?
Beyond that - it's very helpful if you can provide us with information from the app itself.


JavaScript behavior with tabbable panes

Description

On v4.0.0-beta tabbable panes aren't working. It isn't possible to switch between the tabs. It works with the bootstrap.min.js from v4.0.0-alpha.6.

System

Win 10
Mozilla Firefox

Includes files

    <link href="css/bootstrap.min.css" rel="stylesheet">
    <link href="css/style.css" rel="stylesheet">
    <link href="css/font-awesome.min.css" rel="stylesheet">

Code

                  About

                  Web

                  Reputation

                  Skills

Code seems to work for me tried on FireFox and Chrome.
However if I bind on the show.bs.tab, shown.bs.tab or click event of the tab and try to load something dynamic e.g. data from db, tab is not removing the active class from the previous active so you'll see 2 tab contents the same time.

Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via JS Bin or CodePen and report back with your link, Bootstrap version, and specific browser and OS details.

Link
Codepen
Bootstrap Version
v4.0.0-beta
Browser
Firefox 54.0.1 (32-Bit)
OS Details
OS Name:	Microsoft Windows 10 Pro
Version:	10.0.15063 Build 15063
System Type:	x64-based PC
Processor:	Intel(R) Core(TM) i7-2600K CPU @ 3.40GHz, 3401 Mhz, 4 Core(s), 8 Logical Processor(s)
BIOS Version/Date:	American Megatrends Inc. 0406, 2/14/2012
Total Physical Memory:	15.9 GB

@TiBej your example don't work you're right but it's not even working on chrome.
If you add  popper.js your example works just fine for me.
See my Codepen

All right, closing this since popper.js is needed.

@svjansen sorry for my mistake, thx for your help

Alright, i checked your example @svjansen and included popper how you said. It still didn't worked.
The "real" problem wasn't, how i realized pooper. The mistake was that included the js file on the top of the document. It's nessesary to include it at the bottom.


[Security] Command Injection in pdf-officegen

Hi.
A vulnerability was found in your module. I'd like to invite you to the HackerOne submission if possible (I require your account name on HackerOne or any e-mail for that), in order to responsibly disclose the issue. If you prefer that the details of the issue is shared in a public issue, I will go ahead and disclose the details here.
kapytein

@scottoreilly.

We are fine with a public disclose here in this issue, thanks

Sure @codecounselor.
At https://github.com/SpiderStrategies/pdf-officegen/blob/master/lib/engine.js#L202, we could clearly see that the arguments are not passed as an array into child_process.exec(). This will allow command injection. It's probably better to make use of spawn or execFile in this case.
Given the fact that file name is often controlled by user input, an user would be able to inject arbitrary shell commands.
Steps To Reproduce:
const pdf = require('pdf-officegen');

const p = new pdf.Powerpoint({
    engine: 'mupdf',
})

const options = {};

p.convertFromPdf(["test.pdf | touch hello5.txt |.pdf", "test2.pdf"], options, (err, result) => {

    if(err) {
        console.log("oopsie");
        console.log(result);
    }
});

You'll notice that hello5.txt is created in the cwd.
The vulnerability occurs in more methods (for the different engines mupdf/ghostscript/inkscape) in engine.js.

Thank you @kapytein!  @codecounselor please fix this when you have a chance

Hey, do you have any ETA for a patched version? I would like request a CVE for this.

I just opened https://github.com/SpiderStrategies/pdf-officegen/pull/17 if you want to review and test your application against that version.
I need to complete a few integration tests yet and then I'll merge and publish this in a day or so.

released https://www.npmjs.com/package/pdf-officegen/v/2.0.2


Novo-elements and Angular Universal are not compatible - Angular 10

This issue is to track investigation into workarounds to make server-side rendering work after the angular 10 upgrades.  Until this is resolved v3.3.0 is the best version for use with Angular Universal

The problems appear to be brace and some of the polyfills included with dragula.

bullhorn/novo-elements#752 talks about dropping dragula


Party Robot:  value not explicit

We don't know that we need the value of the table in hexadecimal and upper.
Maybe rename the value ? Or adjust the exercise.

Thanks for logging an issue.
The instructions do refer that the value must be in hexadecimal:

The robot provides the table number in hex (uppercase) due to a misalignment of its creator and the rest of the world.

source
Maybe you missed this part of the instructions? If so, what do you think it should be done so you wouldn't miss it?

We will use #1776 to track this issue.


Failure output from search should output response body

raise Tire::Search::SearchRequestFailed, response.to_s if response.failure?
uses the response object as the message, however it should use response.body.to_s
raise Tire::Search::SearchRequestFailed, response.body.to_s if response.failure?

I'm sorry, but I don't understand the problem. Could you be more specific as to what problem you have?

The error output is wrong. It's just printing the Ruby object .to_s instead of something helpful like the response body JSON.
As it is now with response.to_s:
[REQUEST FAILED] Tire::Search::SearchRequestFailed #Tire::HTTP::Response:0x007ff42e896aa0
With response.body.to_s:
[REQUEST FAILED] Tire::Search::SearchRequestFailed {"error":"IndexMissingException[[green] missing]","status":404}

I don't think it's the case on current master and version 0.3.11, this code behaves right:
require 'rubygems'
require 'tire'

extend Tire::DSL

s = search 'articles' do
  query { string '[x' }
end

p s.results

Are you using the latest version? Does the code above print an informative statement in the exception (response code and body)?

Sorry, looks like it works on master. Upgraded and all is good.


Chore: Run make github/init

what

Run make github/init.

why

Updates GHA-related files to use latest distribution of build-harness.
Allows multiple PRs to be consolidated to a single release.

references

N/A

/test all


chore: numexpr to fix CVE-2023-39631 (2.8.4 => 2.9.0)

SUMMARY
superset 3.1.0 / 3.1.1 has a critical finding: CVE-2023-39631 because of numexpr 2.8.4
TESTING INSTRUCTIONS
unknown (sorry), this pull request is to update the numexpr to a non-faulty version depending the CVE (2.9.0 is the current latest available version based on https://github.com/pydata/numexpr)
To test "my" finding: open final image in docker scout => no finding for numexpr is there anymore

ADDITIONAL INFORMATION

[x] Has associated issue: https://github.com/apache/superset/issues/26967
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

HINT: This is my first pull request ... if I made something wrong please let me know that I can make a next one better :)

Hint: made a "manual dirty" update in the docker image to update to 2.9.0 (which for sure kills starting the docker image).
from apache/superset:3.1.1

USER root:root

RUN pip uninstall numexpr --yes
RUN pip install numexpr==2.9.0

USER superset:superset

With this update there is no critical finding of the CVE-2023-39631 in the container anymore - an update of numexp should finally fix this finding

fixed name of PR (seems I forgot the ":" after chore)

@nigzak it seems like numexpr is an indirect dependency:
$ git grep numexpr
requirements/base.txt:numexpr==2.8.4
requirements/base.txt:    #   numexpr

To ensure that CVE is addressed for all situations (and not just CI etc. which leverage the pinned/frozen dependencies) we should add the necessary bump to setup.py. This should likely be done by updating the pandas constraints given that's the package which is depending on it.

HI @dpgaspar
that is bad to me, I am not allowed to deploy this version because it has a critical CVE inside ... so I must stay to superset 3.0.x ... (this has not included this dependency)
I even don't know what others do ... but at least normally - as I know - deploying software which having critical or even high findings is not allowed - or only with "hard discussions" (more or less mostly only MAX for high or where no fix is avaiable).
This "panda" thing I did not know yet so I sadly cannot discuss anything about this ... I tought it is a "normal" package like usual in requirements for pip
As I understood as result this pull request then will be rejected and you will stay on the bad version - right?

made a pull request on panda site to update to 2.9.0 as well - lets see how they take it up :)

made a pull request on panda site to update to 2.9.0 as well - lets see how they take it up :)

I think it is acceptable/desirable to update the requirement/base.in and base.txt like your already doing (don't forget to run pip-compile-multi --no-upgrade to check everything and update the comment with references

please add numexpr>=2.9.0 to base.in

Add this ^^^ and run pip-compile-multi again, please

Done @dpgaspar
... sorry, I have read over this detail

Running CI... 🤞

@nigzak can you please rebase and fix conflicts

I am not sure if I made the rebase correct ... sorry ... this is not my daily thing to do, I tried my best to get it to work ... :/
At least I got a confirmation in the bash
...
Successfully rebased and updated refs/heads/master.

also the push with force afterwards was sucessfully.
I think it should be right? If not please let me know ... again sorry, I still try to learn how this all works ...

hey @nigzak I'm sorry but you have to rebase again or merge an updated master into your branch, this is to fix docker-build step on CI

rebase local failed (added the previous commits to my commit in addition) so I re-synced the forc with the master and commited again the changes to avoid this. Hope now it is fine?

Looks like CI is happy! Maybe @dpgaspar can give it another look before it needs another rebase 😅

synced fork again with master, executed "pip-compile-multi --no-upgrade" and pushed again to git


Axios limit

Description of changes
Summarize the changes made by this PR.
Hot fix for axios max content size, thanks to @jonluca for the code
https://github.com/chroma-core/chroma/pull/367
Test plan
Tests

I'd verify that the ts-axios openapi client is actually using this instance of the axios client, and not the global one. But welcome change :)
Would be good to add a test for this

@jonluca womp womp womp... looking more into this, it appears it does not use it. 🤦  Let's just switch to fetch.


W3C CSS validation errors

Hi, first of all thanks a lot for the plugin, I use it often
But I realised that the CSS file has more than 500 W3C validation errors.
I know that on the W3C they are very strict with the rules, but what do you think about removing all the non-standard CSS properties? Or creating another "clean" CSS file
Try to parse the jAlert.css file into here: https://jigsaw.w3.org/css-validator/#validate_by_input
The issue remains mainly with the @-webkit-keyframes all over the place in the CSS file.
How needed and relevant is that tag?
thanks

Just for the record and FYI, if you'd like to have a "clean" version without W3C validation errors. I tested with Chrome and Firefox and it works
`/==== HEADER =====/
#banner_top {
background: #ececec;
padding: 5px 0;
z-index: 1000;
position: fixed;
top: 0;
left: 0;
right: 0;
border-bottom: 1px solid;
border-bottom-color: #E9E9E9;
-webkit-box-shadow: 0 0 8px rgba(0, 0, 0, .15);
box-shadow: 0 0 8px rgba(0, 0, 0, .15);
display:table;
width:100%;
}
#main_title {
margin: 10px 0;
font-family: inherit;
font-weight: bold;
color: inherit;
line-height: 20px;
font-size: 28px;
text-align: center;
color: #606060;
}
#left_top_div {
width: 15%;
display:table-cell;
}
#header_main_title {
text-align:center;
display:table-cell;
width:70%;
}
#country_box {
width:15%;
display:table-cell;
vertical-align: middle;
}
#country_box_inline{
display:table;
margin: 0 auto;
white-space:nowrap;
}
#banner_flag{
display: table-cell;
}
#country_select,
#banner_flag_div,
#country_select_div {
display: table-cell;
vertical-align:middle;
}
#container{
width:100%;
margin: 0 auto;
}
#container_table{
display:table;
width: 100%;
margin: 0 auto;
}
#description {
text-align: justify;
text-justify: inter-word;
padding-bottom: 2%;
display:table;
margin-top: 40px;
font-size: 18px;
}
#description br {
display: none;
}
.flag {
background-image: url("https://cdn.autocosts.info/images/flags24.png");
height: 20px;
width: 24px;
float: left;
margin: 2px 5px;
}
.is_logo {
padding: 14px 0;
}
.roundCorner,
.roundCorner img {
border-radius: 10px;
}
.roundCorner {
border: 2px solid #b4b4b4;
}
form.roundCorner {
border: 0;
}
.roundCorner img {
border: 0;
display: block;
}
.hidden {
display: none;
}
.clear{
clear: both;
}`

Errors = something's Wrong, always, no matter what you may think.
Warnings = something might be wrong, if you know why, ignore, else best to look into it
You and the W3C validator are right, this property is non-standard; but we need to use it to solve a problem.
W3C validator will give this issues only because on Iphone Safari will not be index as normal !
You should not be worry about it

Thanks for the reply
But what are the consequences of removing all the @-webkit-keyframes ?
I tried in Chrome and Firefox and it works

Keyframe Animation Syntax : the Consequences is you remove the animation style syntax.
If you fallow this link : https://css-tricks.com/snippets/css/keyframe-animation-syntax/  Then you will understand what exacly is it.
Like I told you about W3C you will have warning doesn't matter what you will do, because first when you check with W3C you must prodive few option when you check it.
But if you lose the animation and you're okay with that don't worry about it.

What do you mean by animation?
Kindly look here: http://autocosts.work/UK (just press the next button in the form to check the alert box)
I use there this amended jAlert.css and I have animation, in the sense the the alert boxes "fly" and move smoothly on the screen.
Thank you


fix: add timeout when checking spi upload secret is removed

Description
Adding a timeout when checking if k8s secret used to upload data for SPI binding is deleted automatically; this will avoid any timing issue causing the test to fail improperly.
Issue ticket number and link
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/retest

/retest

/retest

/retest

/approve
/lgtm


player points show 0 when viewing "League -> Players"

The table of players shows 0 for points for all players, however clicking on player will show proper points:

After running update.py -stats_summary the points is populated.  Is there harm to running this all the time to keep display updated?

Yep I have the same issue.

This is because the summary stats haven't been calculated yet. So from the python directory run "python update.py -stats_summary"
This calculates player totals and stores them in a summary table which all of the multi-week statistics pages use.
You could probably schedule it to run as often as you'd like. Think of it as a backend process that collects permanent stats as games complete.


AliBi + Flash Attention Support

Currently our flash attention implementation doesn't support Alibi.
For that we'd need to use flash_attn_triton from: https://github.com/HazyResearch/flash-attention/blob/57ee618170e1adecbf787365cdf330c63768abd2/flash_attn/flash_attn_triton.py#L753

Passing to @dashstander to test for now since I'm slammed.


load_points tests

load_points with whitespace delimiter, comma delimiter, invalid delimiter, and not real file. Adjustments to load_points in GUI and LoadPointsDialog in dialogs.

LGTM!


Apply "EmptyLineSeparator" to the server modules.

Continuation of https://github.com/jpmorganchase/tessera/issues/699
Applies the EmptyLineSeparator to jaxrs-client and server modules.

Codecov Report

Merging #701 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #701   +/-   ##
=========================================
  Coverage     99.49%   99.49%
  Complexity     1864     1864
=========================================
  Files           285      285
  Lines          5546     5546
  Branches        278      278
=========================================
  Hits           5518     5518
  Misses            5        5
  Partials         23       23

Impacted Files
Coverage Δ
Complexity Δ

...com/quorum/tessera/jaxrs/client/ClientFactory.java
100% <ø> (ø)
8 <0> (ø)
:arrow_down:

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 e09842e...8af322f. Read the comment docs.


Thumb is above the track?

Am I the only one having this kind of issue?

Also note that the back track is gone.
Here is my code: (Footer comes from native-base)
<Footer style={{
    backgroundColor: '#7DC265',
}}>
            <Slider
                style={{ flex: 1, alignSelf: 'stretch', margin: 10, alignItems: 'baseline' }}
                minimumValue={0} maximumValue={100}
                value={volume}
                trackStyle={{
                    height: 4,
                    borderRadius: 2,
                }}
                thumbStyle={{
                    width: 15,
                    height: 15,
                    backgroundColor: 'white',
                    borderColor: '#30a935',
                    borderWidth: 2
                }}
                minimumTrackTintColor='#30a935'
                onValueChange={value => debounce(volumeChange(value), 150)}
            />
</Footer>

I'm using:
    "native-base": "^2.0.12",
    "react-native": "0.42.0",
    "react-native-elements": "^0.9.7",
    "react-native-vector-icons": "^4.0.0",

Hi,
I'm having the same issue from RN 0.42, what I did is I added "top: 29" to thumb style. Which is ugly of course...I was trying to debug it, it seems like it has something to do with the logic that applies to transformation:
https://github.com/jeanregisser/react-native-slider/blob/master/src/Slider.js#L276
But not sure yet, didn't had time to find a proper way of fixing it, just using the workaround with "top" offset.

Same issue here.

Thanks for the temp fix @asgvard that will do it for now :)

Thanks @asgvard - I did a thumbStyle={{ top: 22 }} which put it right in the centre for me

The same issue. I check the source code and found the position part is not quite correct. Do you need a pr?

@creeperyang please do send a PR if you have a fix! 👍  Thanks

I send the pr https://github.com/jeanregisser/react-native-slider/pull/65 just now. But I'm not quite sure it will solve all your problem.

same here...

I was facing the same issue, but it was solved by making this change in slider.js
transform: [                 { translateX: thumbLeft },                 { translateY: -(trackSize.height + thumbSize.height) / 2 }               ],
to
transform: [                 { translateX: thumbLeft },                 { translateY: 0 }
Would be  very helpful is someone push the solution asap.

@samridhgupta I tried your solution with no success, what led you to that answer?

Thanks @mcnamee The magic number for me was top: 26. It's not consistent but you can find the right value by trial and error.

@aurnik replying late ... you can check out my fork version of this component
https://github.com/samridhgupta/react-native-slider

Thanks a lot, that worked for me!

Hi, sorry for the noob question: how to import samridhgupta/react-native-slider instead of the default react-native-slider?
I tried npm uninstall react-native-slider and npm install https://github.com/samridhgupta/react-native-slider.git but then, when I launch react-native run-android, a red screen pops telling me it has been unable to resolve the module with:
import Slider from 'react-native-slider'

This: https://github.com/facebook/react-native/issues/4968 (I also tried all the workarounds, and node_modules/react-native-slide is there)
Is there something that I am missing?
Thank you

Hi there!
You know, I failed to install samridhgupta/react-native-slider and I had no time to deal with that, so I just created a file named "Slider.js" in my src directory and imported from there. Ugly but stone-proof :) .
You're very welcome,
Robin.

Среда, 31 мая 2017, 16:18 +03:00 от Edgar Fournival<EMAIL_ADDRESS>Hi, sorry for the noob question: how to import samridhgupta/react-native-slider instead of the default react-native-slider?
I tried npm uninstall react-native-slider and npm install  https://github.com/samridhgupta/react-native-slider.git but then, when I launch react-native run-android, a red screen pops telling me it has been unable to resolve the module with:
import Slider from 'react-native-slider'
This:  facebook/react-native#4968 (I also tried all the workarounds, and node_modules/react-native-slide is there)
Is there something that I am missing?
Thank you
—
You are receiving this because you commented.
Reply to this email directly,  view it on GitHub , or  mute the thread .

A fix for this has (finally) landed :tada: Use the latest published version (0.10.0)!
Thanks everyone.

Thank you so much both of you!

Hey this wont work form react native version 0.40 right?

As I don't know if you wanted to write "for" or "from": I can confirm this is working with RN 0.45.1.


Correct spelling of license file

FYI this misspelling causes this: https://github.com/open-vsx/publish-extensions/runs/2981924283?check_suite_focus=true

I'm on vacation, but the reason for this is because I am from England and that's how we spell licence.

Hi Liam.
I persuaded the fine people at Open-VSX to accept the British spelling - so you can ignore this PR... 💥

Thank you @chrjorgensen! 😄


[rsm-bsa] Update to 4.0.2

Describe the pull request

What does your PR fix?
Updates bsa to 4.0.2

Which triplets are supported/not supported? Have you updated the CI baseline?
Just a port update

Does your PR follow the maintainer guide?

Yes

@Ryan-rsm-McKenzie ,Thanks for your pr, have you tested the features locally?

Yes, I have tested the features locally


Keycloak Client post_logout_redirect_uri Support

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
Recent keycloak (>v19) has introduced the client parameter post_logout_redirect_uri specifying the valid URIs a client may redirect to after logout. See the logout docs.
It would be nice if pulumi gained support for this, as Keycloak by default does not allow any redirect. So in order for logout to work currently have to spam "+" into this field in every realm manually ;)
Affected area/feature

Hi @SimplicityApks - thank you for filing this issue!
While we are awaiting upstream changes that will properly add this as an attribute to the Client resource, per that same upstream discussion, you should be able to leverage the ExtraConfig option to set this in the meantime.

Closing since the upstream issue was fixed a while ago


specifying multiple properties on MSBuild task only adds the last property to the executed command

See The test branch on the example repository for an example.
Ensure you increase the buffer size of your command window to 9999, run the build:
./build.ps1
then scroll back to the start of the msbuild output:

from build task:
      MSBuild("./src/Example.sln", settings => settings
		.SetConfiguration(configuration)
		.WithProperty("firstProperty", "one")
		.WithProperty("secondProperty", "two")
		.SetVerbosity(Verbosity.Diagnostic));

Looking at the unit tests this appears to be covered by passing tests (Should_Append_Property_With_Multiple_Values_To_Process_Arguments) yet I'm not seeing this behaviour.

@Philo I'm unable to reproduce this. Are you sure that you built the right branch?

@Philo this is indeed weird issue, can't see in the code how it would occur in the code and like @patriksvensson I currently can't reproduce it.
I just cloned the repo, switched branch and ran build.ps1 and filtered log for those outputs and it's both in log and args.

Indeed, I'm inclined to think this could even be an issue with the diagnostic reporting from MSBuild itself, as I was able to get my scenario working (running MSBuild with the "Package" target for generating a web deploy zip.  This requires that MSBuild is building the web csproj file and if you try to set a PlatformTarget you get an MSBuild error that was originally assuming was related to this.
Happy to close this issue as not actually a cake issue.
NB: My log output from from MSBuild and not the cake diagnostic logging.

@Philo you're indeed right, now see that we have the same issue in our screen shots, we should file an issue to @Microsoft/msbuild if one hasn't already been made.
The property gets set and I can use it in the MSBuild scripts, so hopefully it's just an display error.

@Philo FYI and fix for this was just merged in Microsoft/msbuild#477
Kudos to you for finding it :+1:

@devlead Great job fixing this!


Add pause

Morph start button into pause/continue button after it was first pressed.

To pause all threads I can implement some Pauser class which would have a state (paused or not) and insert wait locks (while(Pauser.isPaused()) wait()) into all threads' runnables which need to get paused.
Then pause/continue button will switch Pauser's state.

Implemented with class Pauser and interface Pausable


Switch to rustcrypto/elliptic_curves 0.10.0

Codecov Report

Merging #20 (911b5e9) into main (ca7d0b2) will decrease coverage by 0.15%.
The diff coverage is 12.50%.

@@            Coverage Diff             @@
##             main      #20      +/-   ##
==========================================
- Coverage   74.08%   73.93%   -0.16%
==========================================
  Files           7        7
  Lines         490      491       +1
==========================================
  Hits          363      363
- Misses        127      128       +1

Impacted Files
Coverage Δ

src/ecc_compact/mod.rs
72.97% <12.50%> (-0.67%)
:arrow_down:

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 ca7d0b2...911b5e9. Read the comment docs.


Added command to preocess fio bank payments from terminal

Current coverage is 94.46% (diff: 100%)

Merging #220 into master will not change coverage

@@             master       #220   diff @@
==========================================
  Files            46         46
  Lines          1373       1373
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1297       1297
  Misses           76         76
  Partials          0          0

Powered by Codecov. Last update 79df0a6...8463ab0

LGTM


Hosted pinpoint as a service?

I am wondering if anyone is offering pinpoint as a hosted service? Are there other folks interested in such a service ?

Unfortunately. we don't offer PaaS. And we don't see someone to provice pinpoint service.


[CI] Squash Merge PRs

Description
We squash merge all of the prs in the repo apart from the release prs, I think we should squash then to as they are starting to through errors
Screenshots

Do you want to work on this issue?
Yes
Additional information
No response

Thank you, assigned


Different color for stdout than ie in kitchen-vagrant

Somehow standard console output is yellow like it would typically be for warnings.
In kitchen-vagrant it's also colored but green (like for a successful operation).
I don't want to be nit-picked but as colors usually also have a semantic meaning in build and test runs I currently don't know wether it is a bug in my configuration or perhaps something in kitchen-docker.
In this case I am even unsure which configuration files or other information could be relevant so let me know if you need something. I didn't even found something in the kitchen-docker source regarding colors but something seems to be different here. :wink:
IMHO it could also be the standard terminal text color so it wouldn't be any mixing when runnings in combination with specs. So either consistent with kitchen-vagrant or standard color would probably be useful.

Sorry but today everything is fine on my system so nevermind. Probably an issue on my system.

Color in Kitchen comes from which instance you are using, following a standard order.


pls add Stylus support

Hi. guys many developer already using not sass and not less, stylus better them all, pls add support, because plugin, what add stylus support  working is not fine

Please try out some of the stylus extensions. This one looks promising: https://marketplace.visualstudio.com/items?itemName=sysoev.language-stylus

They what I got to fight back and I'm getting all who violated my right
Sent from my iPhone
On Mar 1, 2017, at 5:45 PM, Matt Bierner <notifications<EMAIL_ADDRESS>wrote:
Please try out some of the stylus extensions. This one looks promising: https://marketplace.visualstudio.com/items?itemName=sysoev.language-stylus
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/vscode/issues/21670#issuecomment-283530342, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOpQEoDwEGZZiAvN1934gl47nQQGvo5Kks5rhh9NgaJpZM4MQXMq.

@mjbvz Is there any chance that vscode could add the ability for other languages to implement colorDecorators (similar to how less and scss are able to access it)?
I tried forking the repo and attempted to make a few changes to see if it would be as easy as just adding styl or stylus to here and here, but it seems to be a deeper-rooted issue than just doing that.
Stylus is a pretty popular language. I'm sure @Venegrad and I aren't the only ones who are interested in this.
(Also, the Color Highlight package, while being an alright substitute, is not what I'm looking for. I prefer the square decorators over the entire hex being highlighted).
Thanks in advance. Happy to help if you can just point me in the right direction.


Audit Log Git events and REST API GA

Summary
In https://github.com/github/roadmap-internal/issues/18 we added Git events (clone, fetch, and push) and a REST API to the GitHub Audit Log as a public beta. We will now bring this to GA.
Intended Outcome

REST API is GA for all GHEC customers.
All GHEC customers have access to the export Git events button
Git exports button in Audit Log UI can accept exporting over the full 7 days of history (up to 100MB).

How will it work?
REST API and Git events will be GA for all GHEC customers.

https://github.blog/changelog/2021-04-12-audit-log-git-events-and-rest-api-are-generally-available/


Fix mass-media console

About the PR

Fixes #23989
Technical details

Substring out-of-bounds issue.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

fix: The mass-media console can publish again

#19610 would've fixed this issue

#19610 would've fixed this issue

I'm sorry, I didn't know you were working on it. I saw that Reporters were blocked and just jumped on it.


use timelimit to kill deadlocked tests

restrict any test to 60s
don't trigger the 1h timeout for the auto-tester
get a make error message w/ a specific test on timeout failure
the timelimit program is used b/c the auto-tester already relies on it

See also https://github.com/D-Programming-Language/druntime/pull/1503

Same issue as the druntime pull, not working on linux.  Also, due to the default value of QUIET, can't be sure it's actually being used just from the logs.

Presumably the change is working, since the errors about not finding 'command' went away, but I think it'd be good to go ahead and change the default value of QUIET in this change so there's solid proof, and to match what's been done elsewhere.

I disable QUIET by default, let's see whether that works.

Auto-merge toggled on

I think there is a problem here, see e.g. https://auto-tester.puremagic.com/show-run.ghtml?projectid=1&runid=1988453&isPull=true
What I think is happens is that because we fork 8 threads to run the tests, many of them can run in parallel. If too many of the time consuming tests are running at the same time, they can reach this time limit. Perhaps the time limit can be set to 8 * 60?

I think there is a problem here, see e.g. https://auto-tester.puremagic.com/show-run.ghtml?projectid=1&runid=1988453&isPull=true

I don't think that std/experimental/allocator/building_blocks/free_list.run would ever take 1 minute to complete (it's usually 6ms). And it shouldn't starve b/c all the other tests already completed.

I misread the details, I thought the cluster of tests completing indicated they were all running together.

Another one, also on free_list.run.
https://auto-tester.puremagic.com/show-run.ghtml?projectid=1&runid=1996466&isPull=true
Might this be something to do with the test? I haven't looked at it in detail.

Might this be something to do with the test? I haven't looked at it in detail.

Most likely some busy loop in the SharedFreeList test. Might as well be an extreme contention w/ the 20 threads, but taking longer than 1 second instead of the usual 10 ms seems unlikely.


FORMS_USE_HTML5 causes a migration

Setting FORMS_USE_HTML5 = True will trigger a migration, because the editable attribute for the placeholder field in models.py depends on it.
It looks like the purpose behind the conditional editable is to easily show/hide the placeholder in the Field inline admin. Is that correct, or am I missing something?

That's right - we should change it to not use editable and find a different way to show/hide the field.

Seems like we can use ModelAdmin.get_fields() to check for USE_HTML5 there. Docs.

Or even just set fieldsets on FieldAdmin? That would be consistent with a lot of other admin classes we have.

Or maybe something like this?
field_fields = [all fields minus placeholder]
if html5:
   field_fields.insert('placeholder')

Then just assign that to FieldInlineAdmin.fields.
I like the get_fields() option because we can dynamically get all fields with a call to super, instead of manually defining each of them. Plus, we gotta start using those Django 1.7+ goodies, hahaha.

Oh yeah, that's what I meant :-)

I meant fields not fieldsets I mean.
I guess I like that better than overriding the method since that's how we've done it everywhere else, and that's the main API that Django offers.

Got it. Will get a PR ready.

Any update on this?

Sorry guys, completely forgot about this. PR submitted.


Failed to extract file.

Hello. I'm trying to extract cab files from an InstallShield installer.
Link: https://www.kodensha.jp/sstec/update/jb/jb7up.exe
First, I extracted the executable using isx.exe. I got data1.cab, data2.cab, etc.
Then, running "unshield -d output x data1.cab", I got a error message:
extracting: out/JCMAIN/jcmain.DDB
Failed to extract file 'jcmain.DDB'.Run unshield again with -D 3 for more information.
Am I doing something wrong? Please give me some suggestion. Thanks.

So what output do you get when your run with -D 3 ?

You could also try the -O parameter, it means Use old compression

Option "-D 3" outputs lots of messages flashing away quickly.
Option "-D 2" outputs "[unshield_file_save:749] Decompression failed with code -3.".
I tried extract with"-O", but the program crashed right after the line "  extracting: out/JCMAIN/jcmain.DDB".

code -3 usually means that -O should work :)
But I'll have to check these myself then.

I checked the version of I'm using is "1.4". And the latest version in Github release is "1.4.3".
Could this be the reason I'm getting errors? If it is so, where can I download the latest version?
The "download page" link in readme expired and automatically deleted.

The Windows version of unshield is a mess, sorry about that.

I will try with the latest code.

There are some fixes regarding character encoding that might be relevant when you download from .jp?

It might not be the problem of character encoding. This error file contains only ASCII characters ("JCMAIN/jcmain.DDB"), and all the other files with similar name ("JCMAIN/jcmain.DIC") extracted with no error.
Anyway, thanks for your investigation in advance :)

But if you are on Windows why not just run the installer?

The file I'm trying to extract is an update file installer. I reinstalled the OS a year ago, and some registry of installation gone with the previous OS. When I run the updator, it shows me the program is not installed. I also can't found the install disk of main program which I bought over six years ago. So I want to extract the update files and patch them manully.

Is the size of one of the cabfiles bigger than 2GB? If so that's the problem on windows. Did you try unpacking on bsd/linux?

@kpyrkosz  jb7up.exe is 129 127 321 bytes, so that should be fine

I had some problems getting the data2.cab from jb7up.exe but in the end I got these files and could extract them:
Sizes:
513208 data1.cab
26602 data1.hdr
126144698 data2.cab
MD5 sums:
d79e4d1403e01828710a142c9b142eef  data1.cab
08dfee0adeca298e1a3cfae8eb1a69d3  data1.hdr
93542b855b274fd1fa4cce412ea93674  data2.cab
Output:
unshield -d extracted x data1.cab
Cabinet: data1.cab
  extracting: extracted/CJDLL/KodenshaCjDllV7.dll
  extracting: extracted/JCMAIN/jcmain.DDB
  extracting: extracted/JCMAIN/jcmain.DIC
  extracting: extracted/_Support_0x0411_Files/_isres_0x0411.dll
  extracting: extracted/_Support_Non-SelfRegistering/isrt.dll
  extracting: extracted/_Support_Non-SelfRegistering/default.pal
  extracting: extracted/_Support_Misc_i64_Support_Files/ISBEW64.exe
  extracting: extracted/CW_LTB5IME_64/B5IMEL2.IME
  extracting: extracted/jBeijing7_64bit/JBCJT.dll
  extracting: extracted/jBeijing7_64bit/Addins/TARO/JSTRADD.exe
  extracting: extracted/jBeijing7_64bit/Addins/JB7Addins_AUSw.exe
  extracting: extracted/jBeijing7_64bit/Addins/Jb7Addins_A.exe
  extracting: extracted/jBeijing7_64bit/Addins/Jb7Addins_U.exe
  extracting: extracted/jBeijing7_64bit/Addins/TrustInstaller.dll
  extracting: extracted/jBeijing7_64bit/Addins/JBV7OA.exe
  extracting: extracted/jBeijing7_64bit/Addins/WORD_JB7.DOT
  extracting: extracted/CJDLL_64bit/KodenshaCjDllV7.dll
  extracting: extracted/_Support_Misc_x64_Support_Files/ISBEW64.exe
  extracting: extracted/ROBOWORD_SYSTEM64/Rm64.dll
  extracting: extracted/ROBOWORD_SYSTEM64/Rm64.exe
  extracting: extracted/_Support_Build_Generated_Support_Files/FontData.ini
  extracting: extracted/_Support_Build_Generated_Support_Files/DIFxData.ini
  extracting: extracted/CW_LTB5IME/B5IMEL2.IME
  extracting: extracted/JCTXHex/JCTSHEX7.dll
  extracting: extracted/CW_LTGBIME_64/ugbimel2.IME
  extracting: extracted/JBeijing7/transpdf.exe
  extracting: extracted/JBeijing7/CjDic.dll
  extracting: extracted/JBeijing7/JCT.exe
  extracting: extracted/JBeijing7/JbNavi.exe
  extracting: extracted/JBeijing7/ChnGen.dll
  extracting: extracted/JBeijing7/ExcelTMI.exe
  extracting: extracted/JBeijing7/JBCJT.dll
  extracting: extracted/JBeijing7/JBJCT.dll
  extracting: extracted/JBeijing7/JBV7_PDFmanual.pdf
  extracting: extracted/JBeijing7/JCTFILET.exe
  extracting: extracted/JBeijing7/TeamViewer - _l_b_g_o_R___p_\_R_____u_____i_____[_g_R___g___[___j_\_t_g.url
  extracting: extracted/JBeijing7/TranslateMemoryCJ.dll
  extracting: extracted/JBeijing7/TranslateMemoryCJWS2.exe
  extracting: extracted/JBeijing7/SYSTEMDIC/BunreiJc_Z.DDB
  extracting: extracted/JBeijing7/SYSTEMDIC/BunreiJc_Z.DIC
  extracting: extracted/JBeijing7/SYSTEMDIC/JCFRDIC.DDB
  extracting: extracted/JBeijing7/SYSTEMDIC/JCFRDIC.DIC
  extracting: extracted/JBeijing7/SYSTEMDIC/cwcdic.dic
  extracting: extracted/JBeijing7/SYSTEMDIC/cwcjdic.dic
  extracting: extracted/JBeijing7/SYSTEMDIC/ffc.idx
  extracting: extracted/JBeijing7/SYSTEMDIC/fmemdic.dat
  extracting: extracted/JBeijing7/SYSTEMDIC/omiai.dic
  extracting: extracted/JBeijing7/SYSTEMDIC/omiai.idx
  extracting: extracted/JBeijing7/SYSTEMDIC/omiaiidx.idx
  extracting: extracted/JBeijing7/SYSTEMDIC/ssknecbg.dat
  extracting: extracted/JBeijing7/SYSTEMDIC/ssknecbg.idx
  extracting: extracted/JBeijing7/SYSTEMDIC/zfc.idx
  extracting: extracted/JBeijing7/SYSTEMDIC/zmemdic.dat
  extracting: extracted/JBeijing7/Addins/ExcelJB7.dll
  extracting: extracted/JBeijing7/Addins/ExcelJB7_2007.dll
  extracting: extracted/JBeijing7/Addins/ExcelJB7_2007.dll.manifest
  extracting: extracted/JBeijing7/Addins/ExcelJB7_2007.vsto
  extracting: extracted/JBeijing7/Addins/ExcelJB7_2010.dll
  extracting: extracted/JBeijing7/Addins/ExcelJB7_2010.dll.manifest
  extracting: extracted/JBeijing7/Addins/ExcelJB7_2010.vsto
  extracting: extracted/JBeijing7/Addins/ExcelJB7_2013.dll
  extracting: extracted/JBeijing7/Addins/ExcelJB7_2013.dll.manifest
  extracting: extracted/JBeijing7/Addins/ExcelJB7_2013.vsto
  extracting: extracted/JBeijing7/Addins/Excel_JB7.xla
  extracting: extracted/JBeijing7/Addins/JBV7OA.exe
  extracting: extracted/JBeijing7/Addins/Jb7Addins_A.exe
  extracting: extracted/JBeijing7/Addins/Jb7Addins_U.exe
  extracting: extracted/JBeijing7/Addins/Microsoft.Office.Tools.Common.v4.0.Utilities.dll
  extracting: extracted/JBeijing7/Addins/Microsoft.Office.Tools.Common.v4.0.Utilities.xml
  extracting: extracted/JBeijing7/Addins/PP_JB7.ppt
  extracting: extracted/JBeijing7/Addins/PowerPointJB7.dll
  extracting: extracted/JBeijing7/Addins/PowerPointJB7_2007.dll
  extracting: extracted/JBeijing7/Addins/PowerPointJB7_2007.dll.manifest
  extracting: extracted/JBeijing7/Addins/PowerPointJB7_2007.vsto
  extracting: extracted/JBeijing7/Addins/PowerPointJB7_2010.dll
  extracting: extracted/JBeijing7/Addins/PowerPointJB7_2010.dll.manifest
  extracting: extracted/JBeijing7/Addins/PowerPointJB7_2010.vsto
  extracting: extracted/JBeijing7/Addins/PowerPointJB7_2013.dll
  extracting: extracted/JBeijing7/Addins/PowerPointJB7_2013.dll.manifest
  extracting: extracted/JBeijing7/Addins/PowerPointJB7_2013.vsto
  extracting: extracted/JBeijing7/Addins/TrustInstaller.dll
  extracting: extracted/JBeijing7/Addins/WORD_JB7.DOT
  extracting: extracted/JBeijing7/Addins/WordJB7.dll
  extracting: extracted/JBeijing7/Addins/WordJB7_2007.dll
  extracting: extracted/JBeijing7/Addins/WordJB7_2007.dll.manifest
  extracting: extracted/JBeijing7/Addins/WordJB7_2007.vsto
  extracting: extracted/JBeijing7/Addins/WordJB7_2010.dll
  extracting: extracted/JBeijing7/Addins/WordJB7_2010.dll.manifest
  extracting: extracted/JBeijing7/Addins/WordJB7_2010.vsto
  extracting: extracted/JBeijing7/Addins/WordJB7_2013.dll
  extracting: extracted/JBeijing7/Addins/WordJB7_2013.dll.manifest
  extracting: extracted/JBeijing7/Addins/WordJB7_2013.vsto
  extracting: extracted/JBeijing7/Addins/Microsoft.Office.Interop.Excel.dll
  extracting: extracted/JBeijing7/Addins/Microsoft.Office.Interop.PowerPoint.dll
  extracting: extracted/JBeijing7/Addins/Microsoft.Office.Interop.SmartTag.dll
  extracting: extracted/JBeijing7/Addins/Microsoft.Office.Interop.Word.dll
  extracting: extracted/JBeijing7/Addins/Microsoft.Office.Tools.Common.v9.0.dll
  extracting: extracted/JBeijing7/Addins/Microsoft.Office.Tools.v9.0.dll
  extracting: extracted/JBeijing7/Addins/Microsoft.Vbe.Interop.dll
  extracting: extracted/JBeijing7/Addins/Microsoft.VisualStudio.Tools.Applications.HostAdapter.v9.0.dll
  extracting: extracted/JBeijing7/Addins/Microsoft.VisualStudio.Tools.Applications.Hosting.v9.0.dll
  extracting: extracted/JBeijing7/Addins/Microsoft.VisualStudio.Tools.Applications.Runtime.v9.0.dll
  extracting: extracted/JBeijing7/Addins/Microsoft.VisualStudio.Tools.Applications.ServerDocument.v9.0.dll
  extracting: extracted/JBeijing7/Addins/Microsoft.VisualStudio.Tools.Office.Runtime.v9.0.dll
  extracting: extracted/JBeijing7/Addins/Office.dll
  extracting: extracted/JBeijing7/Addins/TARO/JSTRADD.exe
  extracting: extracted/CW_LT/UDICB5/CwL2B5u.ADC
  extracting: extracted/CW_LT/UDICB5/CwL2B5u.SDC
  extracting: extracted/CW_LT/UDICGB/CwL2GBu.ADC
  extracting: extracted/CW_LT/UDICGB/CwL2GBu.SDC
  extracting: extracted/CW_LT/UDic_Original/UDICB5/CwL2B5u.ADC
  extracting: extracted/CW_LT/UDic_Original/UDICB5/CwL2B5u.SDC
  extracting: extracted/CW_LT/UDic_Original/UDICB5/CwL2B5u.dic
  extracting: extracted/CW_LT/UDic_Original/UDICGB/CwL2GBu.ADC
  extracting: extracted/CW_LT/UDic_Original/UDICGB/CwL2GBu.SDC
  extracting: extracted/CW_LT/UDic_Original/UDICGB/CwL2GBu.dic
  extracting: extracted/CW_LTGBIME/ugbimel2.IME
  extracting: extracted/CW_LTSYS32/CWBuCodeLT.dll
  extracting: extracted/_Support_Misc_Support_Files/corecomp.ini
  extracting: extracted/_Support_Misc_Support_Files/dotnetinstaller.exe
  extracting: extracted/CW_LTSYS64/CWBuCodeLT.dll
  extracting: extracted/JCTXHex_64bit/JCTSHEX7.dll
  extracting: extracted/_Support_Language_Independent_OS_Independent_Files/license.rtf
  extracting: extracted/ROBOWORD_SYSTEM/RmNT.dll
  extracting: extracted/ROBOWORD_SYSTEM/RmNT.exe
  extracting: extracted/ROBOWORD_SYSTEM/TcHkMsRd.dll
  extracting: extracted/ROBOWORD_SYSTEM/TcHook32.dll
  extracting: extracted/ROBOWORD_SYSTEM/TcNls.dll
  extracting: extracted/ROBOWORD_SYSTEM/TcRmApi.dll
  extracting: extracted/_Support_0x0411_String_Tables/StringTable_0x0411.ips
 --------  -------
          130 files

Process finished with exit code 0

@kpyrkosz  Could you provide an MS Windows build of unshield main branch for @Reminiscence0 ?

Using parameter -e CP932 will extract the TeamViewer.url file with the right encoding on Linux:
extracting: extracted/JBeijing7/TeamViewer - ネット経由のパソコン遠隔操作（リモートコントロール）ソフト.url

Both builds (my fixed for windows + your master branch built with mingw) fail to extract the file with error:
Failed to extract file 'TeamViewer - âlâbâgîoùRé╠âpâ\âRâôëôèuæÇì∞üiâèâéü[âgâRâôâgâìü[âïüjâ\âtâg.url'.Run unshield again with -D 3 for more information.

Using -e CP932 prints out This version of Unshield is not built with encoding support. and exits.

Thanks a lot @kpyrkosz - very good information. Maybe this is the actual issue for @Reminiscence0?

Windows will certainly not like the backslashes in the file name and maybe not everything else either


Improve Typescript on free-programming-books-ja.md

What does this PR do?
Add Resources | Remove Resource
For resources
Description

Add TypeScript Deep Dive Japanese version

https://github.com/yohamta/typescript-book-jp/

Add clean-code-typescript Japanese version

https://github.com/MSakamaki/clean-code-typescript

Remove TypeScript quick guide

Because the contents were deleted

Why is this valuable (or not)
How do we know it's really free?
For book lists, is it a book?
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

Thanks!


Error: no such option: --subscription-id (need to use --resource-id)

[Enter feedback here]
Trying to run the command from this page:
databricks secrets create-scope --scope     --scope-backend-type AZURE_KEYVAULT --subscription-id  --dns-name
gives the error
Try 'databricks secrets create-scope -h' for help.
Error: no such option: --subscription-id
it looks like --resource-id TEXT should be used instead.
I've tried that, but I'm currently getting an error message
"message":"Authentication is temporarily unavailable. Please try again later.","error_code":"TEMPORARILY_UNAVAILABLE"
but I have a support ticket open with regards to that.
regardless, I think this page needs to be updated to indicate the use of --resource-id instead of --subscription-id

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

ID: 810a218a-80a5-2da0-b540-26347f9ef908
Version Independent ID: 41c2a12f-99b6-0faa-500b-3e4308cd00de
Content: Secret scopes - Azure Databricks
Content Source: databricks/security/secrets/secret-scopes.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@rharris-hs Thank you for bringing this to our attention. We are evaluating the feedback you have provided and will have the document updated as appropriate.

@rharris-hs Yes, you are correct. It should be replaced  --subscription-id  with --resource-id .
databricks secrets create-scope --scope azure --scope-backend-type AZURE_KEYVAULT --subscription-id “XXXXXXXXXXXXXXX” --dns-name “https://chepra.vault.azure.net/”

As per databricks cli help page, there is no parameter called --subscription-id.

databricks secrets create-scope --scope azurekv --scope-backend-type AZURE_KEYVAULT --resource-id “/subscriptions/XXXXXXXXXXXXXXXXXXXX/resourceGroups/Analytics-Resources/providers/Microsoft.KeyVault/vaults/chepra” --dns-name “https://chepra.vault.azure.net/”.
@mssaperla Could you please review further and update the document as appropriate.

This was already fixed in the reported article. Apologies for not updating the status of this issue at the time.
#please-close

Only Microsoft employees can close issues or pull requests via commenting in this repo.

#please-close


UnicodeDecodeError on installation

setup.py install gets the error:
Traceback (most recent call last):
  File "setup.py", line 107, in <module>
    install_requires=install_requires,
  File "C:\Program Files (x86)\Python36-32\lib\distutils\core.py", line 121, in setup
    dist.parse_config_files()
  File "C:\Program Files (x86)\Python36-32\lib\distutils\dist.py", line 395, in parse_config_files
    parser.read(filename)
  File "C:\Program Files (x86)\Python36-32\lib\configparser.py", line 696, in read
    self._read(fp, filename)
  File "C:\Program Files (x86)\Python36-32\lib\configparser.py", line 1012, in _read
    for lineno, line in enumerate(fp, start=1):
UnicodeDecodeError: 'cp932' codec can't decode byte 0x98 in position 87: illegal multibyte sequence

This is caused by setup.cfg's this line, LEFT SINGLE QUOTATION MARK is only in unicode characters:
    # E265: block comment should start with ‘# ‘...

Fix example:
    # E265: block comment should start with '# '...

Thanks! Should be fixed on PyPI. Can you re-check please?

It's OK. Thank you.
C:\>py -m pip install --upgrade pywinauto
Collecting pywinauto
  Downloading pywinauto-0.6.2-1.zip (505kB)
    100% |████████████████████████████████| 512kB 728kB/s
Requirement already up-to-date: six in c:\program files (x86)\python36-32\lib\site-packages (from pywinauto)
Collecting comtypes (from pywinauto)
  Using cached comtypes-1.1.3-2.zip
Installing collected packages: comtypes, pywinauto
  Found existing installation: comtypes 1.1.3
    Uninstalling comtypes-1.1.3:
      Successfully uninstalled comtypes-1.1.3
  Running setup.py install for comtypes ... done
  Running setup.py install for pywinauto ... done
Successfully installed comtypes-1.1.3 pywinauto-0.6.2


Recommended solution for local scripts that generate Kubernetes YAML

Raffo in the Tilt channel writes:

I have the following problem that I’m trying to figure out:
I have a local_resource (a shell script) that renders some kubernetes yamls according to some internal conventions.
I want to apply all of the yamls in the rendered folder
I want to use k8s_resource to apply/consider a specific resource only when some dependencies are ready (in kubernetes terms).
I’m using something like:

local_resource('render', cmd='./render.sh')
k8s_yaml(listdir("tmp/rendered", recursive=True))
k8s_resource('test', resource_deps=['foo'])

But this has some drawbacks about order of execution and dependency-tracking. You can also do this with a local() command but that also has some drawbacks (the Tiltfile can only be re-executed in one go, it's easy to accidentally hit infinite reload problems, etc.) Here are some bugs we're run into one trying to support this approach:

https://github.com/windmilleng/tilt/issues/2957
https://github.com/windmilleng/tilt/issues/1930

This seems like a common use-case. Wiring it together on your own seems hard to get right. Which makes me feel like all these bugs / feature-requests are indicative that the APIs aren't working together great
I think this all hints at a build primitive that generates other kubernetes resources, with the ability to list dependencies between them, but I'm not sure quite what it is yet - it might be simple. A way to resolve this ticket might just be a doc / working example on a recommended approach on how to do this

Thanks for summarizing this in an issue. Unfortunately, none of the workarounds work for me. Let me go into the details:

if I use .tiltignore, the tmp/rendered folder gets ignored and none of the workarounds to reload the file work.
if I use k8s_yaml(local("cat file.yaml")), this will also not register a dependency as above.

This means that at the first run of the Tiltfile everything will work as expected, but none of the subsequent runs works.
I think what I am really trying to do as a user is to model a workflow made of three steps: "render, apply, run tests". The render step, being based on a local, can only be triggered manually and not at the change of templates.
I think what is very confusing for me is that there are things that happen at the first run and things that don't. This makes it very confusing and it's hard to understand, as a user writing the Tiltfile, what is going to run when.
Currently, I am refactoring my very little Tiltfile with an indication to be run with this workflow:

tilt up
if you make any changes, quit tilt and tilt up again

Happy to talk more about solutions with you, if you think there is any and it's worth going a bit deeper on this topic.

This came up today in a conversation that @jazzdan and I were having. In particular, the user had a docker container that produced yaml, and needed a way to load it up into Tilt's engine.
I wonder if an API like this might look something like:
local(cmd, generated_files)
k8s_yaml(generated_files)

to prevent the Tiltfile from reloading when the files get generated
I've also thought about something like
local_resource(name, cmd, output_k8s_yaml=['./path/to/dir'])

such that, after the resource ran, Tilt would scan the output directory for new yaml files and create k8s resources for them. But that has much more complex ramifications the order of operations for how Tilt assembles the build.


usd_foil mode for price

The documentation says that usd_foil is a valid price mode for a card, but this isn't reflected in the prices method in card_object.py

Branch develop is now updated with this functionality. It will roll out in the next release, apologies for the huge delay.


Can you make your experimental model publicly available for reproduction?

Your work is extremely valuable!
If possible, could you make your experimental model publicly available for replication?
Thanks in advance!

Your work is extremely valuable!
If possible, could you make your experimental model publicly available for replication?
Thanks in advance!

hi，sorry that we wiil not release the full code because of copyright issues.
Thank you for your attention to our work


Second assignments

I know that I have not finished my assignments, but I would like to know if I am on the right way

Well done. I'm merging this


PP-12826 Update GatewayAccountResponse classes to records

WHAT YOU DID

Modify GatewayAccountResponse to be an interface to replace extensible class
Create new DefaultGatewayAccountResponse record to replace existing GatewayAccountResponse
Update GatewayAccountWithCredentialsResponse and GatewayAccountWithCredentialsWithInternalIdResponse to records

Closing as this overcomplicates the existing implementation


Document how to roll back

This aims to improve the documentation following a question from an user, as I couldn't see anything in this repository, nor on weave.works/docs.

You may click the big friendly merge button @marccarre :)


Add basic TypeScript type definitions

This PR adds basic ambient type declarations for Router to improve code hinting for the dynamic/proxied methods and make it easier to code with itty-router.
I'm not 100% certain how this would actually fare working in a TypeScript project, but at the very least it should help in TS-powered editors like VS Code by giving you better IntelliSense.

Well Travis builds aren't [yet] working, but this certainly can't break anything! 💯


Feature/npz/ast indices

Overview
This PR adds a few formulas useful for the detection of algal blooms. See http://www.sciencedirect.com/science/article/pii/S0034425716304928
Checklist
~- [ ] Styleguide updated, if necessary~
~- [ ] Swagger specification updated, if necessary~
~- [ ] Symlinks from new migrations present or corrected for any new migrations~
Notes
Included formulas:
Phycocyanin detection: 47.7−9.21(Red/Blue) + 29.7(NIR/Blue) − 118(NIR/Red) − 6.81(SWIR1/Red) + 41.9(SWIR2/Red) − 14.7(SWIR/NIR)
'Improved NIR with SAC': B5 - 1.03 x B6
'NIR over Red with SAC': (B5 - B6) / (B4 - B6)
Closes #1998

You might want to take a look @notthatbreezy


Plugin 'Protocol Buffer Editor' (version '2.2.0') is not compatible with IntelliJ 2021.1

I'm unable to utilise .proto autocompletion using the package you've got listed in the marketplace.
Has this package been abandoned and if so, where should I look for a package that supports protobuf?

This repo was never supported. You're looking for https://github.com/jvolkman/intellij-protobuf-editor.

Oops, did not know that. I'll close this off. Did you want me to port this over to https://github.com/jvolkman/intellij-protobuf-editor?

No need; https://github.com/jvolkman/intellij-protobuf-editor/issues/53 is already open.

Too easy. Cheers.


Create New Account should hide after clicking on "plus-circle"

There was an old issue for this that I guess got closed. I was never able to reproduce this myself though.

It used to be a bug but it's def gone now!


DOMException: Failed to construct 'WebSocket': An insecure WebSocket connection may not be initiated from a page loaded over HTTPS.

I've application which is loaded via HTTP and tries to connect to HTTP.
It's not right, but even though it shouldn't crash my application totally. But it does.

@ohjames Is there any way to prevent application crash?

Ctrl-l ctrl-c
Then paste the URL somewhere. Notice how it is https. Then regret not believing the error message ;)

You can prevent the application crash by using not using an inappropriate ws URL.

I could make it to the code detects the constructor failure and creates an observable which emits a single error, but that sounds like worse behaviour. This is why they issue the warning from the WebSocket constructor right... to tell you to use the right kind of URL. You should be using a // or https:// prefixes URL for your ws and never http.


feat: Configurable cgroup ID regex for process discovery component

PR Description
This PR adds a new regex argument to Process discovery component that captures the cgroup ID of the process and exports it in the target.
Which issue(s) this PR fixes
Fixes #1461
Notes to the Reviewer
@simonswine Please take a look at it when you have time.
PR Checklist

[ ] CHANGELOG.md updated
[x] Documentation added
[x] Tests updated
[ ] Config converters updated

@mattdurham I pushed the changes I proposed in the previous comment.
Simply put, we export cgroup path as one of the labels. In the case of cgroups v2, it is a single path as there is only one unified cgroup path for a given process. In the case of cgroups v1, there is a path for each controller and "relevant" path depends on which controllers are active. So, in order to keep the component simple, we just export all the cgroup paths delimited by | as a string.
Users will be able to use relabel component to retrieve the information they need from these paths very easily. At the same time, there is no need to add new arguments and keep the component simple as before.
Let me know what do you think.
@korniltsev Could you take a look at it and see if it what you mean?
Cheers!

@simonswine Thanks a lot for coming back to this PR. Totally understandable on why things take long time and its fine.
So, I added an example on how to use the new cgroup path meta data with relabel component. Our use case of HPC batch jobs might be too specific for generic users. So, I chose an example of discovering systemd services using cgroup path and relabel component.
Let me know if it is clear for you and what do you think of example. If you think adding a "complete" config example on how to add service_name label and then pass them to Pyroscope's ebpf component to continuously profile systemd services adds more value, I can do that too. Cheers!!

@simonswine Done!! Cheers for everyone's time in reviewing this as well.


Added SQLite in Relational DB

thank you, @Ash9311. I have merged your pr. I am sorry it took a while.

Thank you @Timonwa . No issues


Won't load

It hangs on the load screen when I click on any audio files in a shared folder. Could this be a permissions problem

Thanks for the report. Can you add new audio files in Drive and play them?


Add instance scale-in protection for consul-cluster

This change solve issue: https://github.com/hashicorp/terraform-aws-consul/issues/177
The changes are tested with an existing consul-cluster without scale-in protection. The optional scale-in protection is set to true.
The terraform plan output is here:
https://gist.github.com/mawa-jnd/476ab6680a68d6c4b1770a372cfff5aa
The change is:

backward compatible
has no downtime with default setup as changing the ASG will not automatically restart instances

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes
Have you signed the CLA already but the status is still pending? Recheck it.

Tests passed! Merging now.

https://github.com/hashicorp/terraform-aws-consul/releases/tag/v0.7.5


Add support for TCP_DEFER_ACCEPT and TCP_QUICKACK

Motivation:
When using the native transport have support for TCP_DEFER_ACCEPT or / and TCP_QUICKACK can be useful.
Modifications:

Add support for TCP_DEFER_ACCEPT and TCP_QUICKACK
Ad unit tests

Result:
TCP_DEFER_ACCEPT and TCP_QUICKACK are supported now.

@trustin @nmittler @Scottmitch PTAL

@normanmaurer - 2 comments then lgtm

Test failures? Non-test code looks good to me.

Strange works here... Let me check

Am 08.03.2016 um 03:18 schrieb Trustin Lee<EMAIL_ADDRESS>Test failures? Non-test code looks good to me.
—
Reply to this email directly or view it on GitHub.

Cherry-picked into 4.0 (5317c7b6486ce88ae95389df3a5fa6e835399dba) and 4.1 (d09547deb8d0b1de43637f206d49711535fd9e09)


how to handle error after retry

I want to log exception after retry 4 times，  Can anyone tell me, what should I do？

If you are using container-based configuration for the bus, the bus configuration should pull the ILoggerFactory from the container and use it to output log information (within the "MassTransit" log scope). If you are not using a container, you can configure the log context during configuration:
LogContext.ConfigureCurrentLogContext(loggerFactory);

MassTransit will write error log messages once the retry limit has been reached.
Your other option is to create custom middleware that can log the error.

As this isn't an issue with MassTransit, I will close this issue. You're welcome to reply here, or via our Discord server.


Separate teachers in free periods from students

instead of just sorting by firstname, now sorts by firstname and then by if theyre a teacher, so now it shows all teachers alphabetically then all students alphabetically
fixes #328

Looks good to me.

LGTM!


Updated podRoot  -> rktRoot

Nack, why?
Am 06.05.2015 09:49 schrieb "Ramit Surana"<EMAIL_ADDRESS>

You can view, comment on, or merge this pull request online at:
https://github.com/coreos/rkt/pull/871
Commit Summary

Updated podRoot -> rktRoot

File Changes

M networking/networking.go
https://github.com/coreos/rkt/pull/871/files#diff-0 (14)

Patch Links:

https://github.com/coreos/rkt/pull/871.patch
https://github.com/coreos/rkt/pull/871.diff

—
Reply to this email directly or view it on GitHub
https://github.com/coreos/rkt/pull/871.

/cc @eyakubovich

Please reopen if you'd like to revisit, thanks


Fix formatting in polyfill-library readme

The polyfill-library readme currently renders all @param lines as one long line:

This PR changes it to use list syntax so it's easier to read.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[config] removing abbreviations in config files for clarity

keypairs in config files were abbreviated (e.g. ns for network_signing_pubkey).
This doesn't make sense as our config files are in a self-describing format and this obfuscates the information they contain.
This PR fixes this.

do I need to fix some .toml configs manually here after this change?

This was done intentionally at some point to attempt to reduce the size of these I believe due to some size limit (env vars I think). We should at least hold off until next week to get some more eyes on this.
I think once we move to dynamically generated configs this should no longer be a problem, but it might already no longer be an issue since a lot of things got moved to secretsmanager.

:umbrella: The latest upstream changes (presumably 119ed015f0be04971d3e36caeb65a4c7ea057f6d) made this pull request unmergeable. Please resolve the merge conflicts.


Docs: copy/paste typo

Thanks a lot @stav! :+1: I'll fix the failed tests a bit later.

What is the cause of the failed tests?

wheel requires Python '>=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*' but the running Python is 3.3.6

Lools like it comes from https://github.com/pypa/wheel/pull/216/files. We probably just need to consider Python 3.3 as unmaintained and drop the environment from our Travis tests.


Tests are not working correctly on JVM configured in a different timezone

Hi,
The tests are not passing correctly when we launch them in a JVM set in a different timezone (Paris in my case). The default timezone must be set to be able to use arbitrary timestamps.
I can open a PullRequest to correct this if needed :
https://github.com/JulienArzul/CompactCalendarView/tree/%40tests/jvm_config_independance
(in this branch, I also set the default locale to english and correct a Timestamp that had a wrong value)

I thanks for the issue, I think that should be set as the default since it that is my default time zone
        System.out.println(tz.getDisplayName());
        System.out.println(tz.getID()); // prints Europe/London

So if you add:
        Locale.setDefault(Locale.ENGLISH);
        TimeZone.setDefault(TimeZone.getTimeZone("Europe/London"));

The code should work without modification then. Let me know if this works for you.

Yes, I've actually added this (to keep it GMT-based and not associated with a country/city) :
Locale.setDefault(Locale.ENGLISH);
TimeZone.setDefault(TimeZone.getTimeZone("GMT"));

Beware that if you do this, there is one timestamp that stops working (probably because of the change of time in London). It's the one line 93. It now should be :
//Wed, 01 Apr 2015 00:00:00 GMT
assertEquals(new Date(setTimeToMidnightAndGet(cal,<PHONE_NUMBER>000L)), underTest.getFirstDayOfCurrentMonth());

Ok gotcha, we can update that test then. If you want to send just a pull request for this change I can merge it at some point. Thanks!

Closing as code as been merged.


Simplify is_literalipaddrv6 & is_hostnamewithport (resubmit/rebase of PR #962)

rbgarga asked for PR #962 to be rebased/resubmitted.
On doing so I find that

is_ipaddrwithport() has been sorted out effectively already, so that part is redundant
is_hostnamewithport() can be simplified (we don't need to pop() or assign a new variable just to test 2nd member of the array)
is_literalipaddrv6() can be much simplified - we don't need an expensive preg_match() to test if it's a valid IPv6 wrapped in "[" ... "]"

If this is ok then the original PR can be closed as it's superseded.

Superseded by PR #2320 - can be closed