What are the differences between cold_start_filter and filter_continuation ?

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

TensorFlow version: Master
Doc Link:
https://www.tensorflow.org/api_docs/python/tf/contrib/timeseries/saved_model_utils/cold_start_filter

Describe the documentation issue
It is not clear what are the differences between  cold_start_filter and filter_continuation. And in what condition they should be used?
And for "Filtering refers to updating model state based on new observations." , is the model state == model ? why one should update it  ?

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device

@StevenLOL

Both cold_start_filter and filter_continuation update model state based on new observations.
cold_start_filter starts from the model's default or uninformed state. filter_continuation applies filtering subsequent to cold_start_filter.
Model state is not the same as model, the former is being updated with new  observations in features and normally contains previous and current cell information depending on how you build the model.
The reason behind filtering is you need to update model state based on new observation in time series.


Transplant Algorithm

Hello @Tirent-user please assign this to me

I am interested in task.
Please assign it to me.


Fix db transaction race

We still have two places related to cluster update and download image.
but it's less urgent so it will be fixed in a different PR
This is the fix:
if err := tx.Model(&host).Update("checked_in_at", host.CheckedInAt).Error; err != nil {

subsystem and test-infra passed.
will run test-infra multiple times.


[Bug] There are errors when compile model with q3f16_1 quantization.

🐛 Bug
comment:
python3 -m mlc_llm.build --target cuda --quantization q3f16_1 --model ./dist/models/llama-13b

error:

No, I still have this problem.

I believe this issue has been fixed by @vinx13. Please upgrade to latest TVM Unity nightly pip package. Feel free to create a new issue if it persists!


[Kernel] Remove usages of deltaFile from Delta-Kernel

Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[x] Kernel
[ ] Other (fill in here)

Description
FileNames.deltaFile is deprecated and will be removed in future versions of Delta-Spark.
How was this patch tested?
Existing UTs
Does this PR introduce any user-facing changes?
No

@sumeet-db I believe this file is used to generate golden files for multiple different consumers (e.g. standalone) and not just Kernel. You might want to rename the PR

@dhruvarya-db @sumeet-db what does this mean? Kernel has its own version of the FileName.deltaFile. Should that be removed when Kernel starts supporting managed commit table reads and/or writes?

Hi @vkorukanti. Yes, the Kernel version of FileName.deltaFile will need to be updated once it has Managed Commit support. FileName.deltaFile should be deprecated because the way file names are discovered/generated is different in ManagedCommits. This particular change is just a refactor to indicate that using FileName.deltaFile can be unsafe to use in most scenarios.


Add Ukrainian support

Can you add Ukrainian language support please?

Ukrainian seems to be something we don't support at the moment. For now, I'll try to add it as much as possible. Don't expect too much from the translation that is implemented as it is almost never perfect since it comes from community created dictionaries. If you can recommend a complete dictionary for Ukrainian it would be nice to take a look at it (note that in this package for now the languages are stored in a string, or maps. This will change soon to only support maps as it avoids wasting time converting the string into a map).

Hi, thanks for your reply.
Here you can find the latest Ukrainian dictionaries:
dict_uk
dictionaries


[Extensibility Bug]: Provide a possibility to set the Telemetry Logger Interface explicit instead of relying on a event subscriber on publisher

Describe the issue
I'd like to have the possibility to explicit the the "Telemetry Logger" Interface, when using the Telemetry or FeatureTelemetry Codeunits.
Currently we need a subscriber in any of our apps (with identical publisher).
And since we don't know which app is installed we might subscribe multiple times to this event publisher, which results in an additional telemetry event, that I want to avoid.
Expected behavior
it should be possible to explicit set the interface via a Setter Method on the Public Facade Codeunits.
or in other words:
it should be possible to set an TelemetryLogger Interface to skip the following lines
        TelemetryLoggers.SetCurrentPublisher(Publisher);
        TelemetryLoggers.OnRegisterTelemetryLogger();

        if TelemetryLoggers.GetTelemetryLogger(TelemetryLogger) then

https://github.com/microsoft/BCApps/blob/636c354bc6d2b27ba6527fd193ac494af7f8ebfa/src/System Application/App/Telemetry/src/Logging/TelemetryImpl.Codeunit.al#L41C1-L41C1
Steps to reproduce
Create two apps with the same publisher and subscribe to the the TelemetryLogger Event.
Additional context
I already talked to some people of microsoft at Directions EMEA 2023 and they suggested to open an issue for this topic to discuss here fruther with @IhorHandziuk who created this module.

Hi @pri-kise, I think adding the ability to provide the telemetry logger implementation on a variable instance is not the best approach, as it will add more lines of code and potentially more confusion to something that is suppoused to be very siple to use.
Normally we recommend that partners extract the implementation of the telemetry logger in a separate app, and then add it as a dependency to the existing apps (see this Viva Engage post).
If this is not a viable option for some reason, we can also just remove the warning about having multiple telemetry logger implementations altogether (and assume that the extension developer is aware that a random imlementation will be picked).

I will discuss with my other collegaues internally and will close this issue until we have evaluated the best approach.

Why can't more than one interface be saved in the Telemetry Logger codeunit? (List of [interface] would be nice ;) )
And if there is an implementation for the current app, this will be used if not the one for the publisher. The register method might need an overload to specify whether the logger should be used for all apps of the publisher (default) or only for the current app.
In case I have misunderstood the task of this feature, I would be happy if someone could explain again how the feature should be used in the context of an ISV with multiple apps. What assumptions have been made?
Same Applicationsights endpoint for all apps? If so, are there any advantages that I am overlooking?
With separate endpoints I see the advantage that the limits etc. for the Application Insights workspace can be maintained individually for each app (with transformation rules it quickly becomes very complex).

Hi @JohnnyUndercover, what you are describing indeed sounds like a good improvement to the telemetry module. Unfortunately, it's currently impossible to define a list of interfaces (the code wouldn't compile). We could define a number of individual variables to sort-of support this, but that's an antipattern we would like to avoid, especially if the gain is not that big.

@pri-kise, @JohnnyUndercover and @IhorHandziuk, where are we on this issue? Which changes (which actually can be done) are requested to be made? And can we approve those changes? This issue isn't fully fleshed out, it seems.

@IhorHandziuk @JesperSchulz Would it be possible to just add the additional parameter AppId to the Register method?
For me it would be ok if the most generic Registration will be used, but if i only register loggers with an AppId are registered for a publisher, the one with the matching AppId should be used.
For example:
When LogMessage is called from App2 with AppId2
OnRegisterTelemetryLogger

TelemetryLoggersImpl.Register(TelemetryLogger1, Publisher1, NullGuid);
TelemetryLoggersImpl.Register(TelemetryLogger2, Publisher1, AppId1);
TelemetryLoggersImpl.Register(TelemetryLogger3, Publisher1, AppId2);

In this case the TelemetryLogger1 with the NullGuid would win, because this is more or less what it does today.
When LogMessage is called from App2 with AppId2
OnRegisterTelemetryLogger

TelemetryLoggersImpl.Register(TelemetryLogger1, Publisher1, AppId1);
TelemetryLoggersImpl.Register(TelemetryLogger2, Publisher1, AppId2);

TelemetryLoggersImpl.Register(TelemetryLogger3, Publisher1, AppId3);

In this case the TelemetryLogger2 with the AppId2 should be used.

Hi @JohnnyUndercover , @pri-kise , as explained above to support something like this we need to be able to first populate a collection of interfaces / codeunits, and then select the most appropriate interface / codeunit from that collection.
This is currently not possible. If / when it will become possible, we would definitely revisit this.

Do we really need the collection of interfaces?
If i read it right, the event OnRegisterTelemetryLogger is called every time a message is logged (please correct me if i'am wrong) and therefore we only need a logic to store the most explicit impementation.


Use api.generateAddress to create account. Get account information and report error rippled-1.7.2

Please help me
server start（Start a New Genesis Ledger in Stand-Alone Mode）
rippled -a --start --conf=/etc/opt/ripple/rippled.cfg
jsApi
import { RippleAPI } from 'ripple-lib'
const api = new RippleAPI({
  server: 'ws://<IP_ADDRESS>:6005', // My rippled server
  //server: 'wss://s1.ripple.com' // Public rippled server
})
api
  .connect()
  .then(async () => {
    /* begin custom code ------------------------------------ */
    let addressObj = api.generateAddress({
      test: true,
    })
    let account= api.getAccountInfo(addressObj.classicAddress)
    return account
  })
  .then((info) => {
    console.log(info)
    console.log('getAccountInfo done')
    /* end custom code -------------------------------------- */
  })
  .then(() => {
    return api.disconnect()
  })
  .then(() => {
    console.log('done and disconnected.')
  })
  .catch(console.error)

error:
message:'Account not found.'
`{"name":"RippledError","data":{"account":"r4Lbjg2SbsZDH1mdr4WcSc6reZSuYE9NVz","error":"actNotFound","error_code":19,"error_message":"Account not found.","id":1,"ledger_hash":"F5A8382FEB857BDDE8B71A391B229B3D94193E32FC1B5BA108568C24650ECB25","ledger_index":2,"request":{"account":"r4Lbjg2SbsZDH1mdr4WcSc6reZSuYE9NVz","command":"account_info","id":1,"ledger_index":"validated"},"status":"error","type":"response","validated":true}}`

Does rippled.cfg need to add any configuration？

generateAddress creates a new address, but an account object for that address does not exist on the ledger. To create the account, fund it by sending it a payment. The payment amount must be greater than or equal to the base account reserve. On Mainnet, the base account reserve is currently 20 XRP. On a stand-alone ledger, the amount may be different.
When creating a new ledger in standalone mode, the initial funds are in the genesis address, as documented here: https://xrpl.org/start-a-new-genesis-ledger-in-stand-alone-mode.html
You can send XRP from that genesis address to your newly-generated address in order to fund and activate the account.

thanks @intelliot, is there a ledger_accept equivalent in RippleAPI?

I sent the XRP, but it was never verified
There is no access on method：api.connection.on("transaction")
sever:ws://<IP_ADDRESS>:6005

const RippleAPI = require('ripple-lib').RippleAPI;

let address = "rHb9CJAWyB4rj91VRWn96DkukG4bwdtyTh"
let secret = "snoPBrXtMeMyMHUVTgbuqAfg1SUTb"

const api = new RippleAPI({server: 'ws://<IP_ADDRESS>:6005'})
api.connect()
api.on('connected', async () => {

  while (true) {
    try {
      let ac=await api.request("account_info", {account: address, ledger_index: "validated"})
      console.log(ac)
      break
    } catch(e) {
      await new Promise(resolve => setTimeout(resolve, 1000))
    }
  }

  // Prepare transaction -------------------------------------------------------
  const preparedTx = await api.prepareTransaction({
    "TransactionType": "Payment",
    "Account": address,
    "Amount": api.xrpToDrops("22"), // Same as "Amount": "22000000"
    "Destination": "r351srbb3RP8wxRLXHLy8m4zNJeAvQX9us"
  }, {
    // Expire this transaction if it doesn't execute within ~5 minutes:
    "maxLedgerVersionOffset": 75
  })
  const maxLedgerVersion = preparedTx.instructions.maxLedgerVersion
  console.log("Prepared transaction instructions:", preparedTx.txJSON)
  console.log("Transaction cost:", preparedTx.instructions.fee, "XRP")
  console.log("Transaction expires after ledger:", maxLedgerVersion)

  // Sign prepared instructions ------------------------------------------------
  const signed = api.sign(preparedTx.txJSON, secret)
  const txID = signed.id
  const tx_blob = signed.signedTransaction
  console.log("Identifying hash:", txID)
  console.log("Signed blob:", tx_blob)

  // Submit signed blob --------------------------------------------------------
  // The earliest ledger a transaction could appear in is the first ledger
  // after the one that's already validated at the time it's *first* submitted.
  const earliestLedgerVersion = (await api.getLedgerVersion()) + 1
  const result = await api.submit(tx_blob)
  console.log("Tentative result code:", result.resultCode)
  console.log("Tentative result message:", result.resultMessage)

  // Wait for validation -------------------------------------------------------
  let has_final_status = false
  api.request("subscribe", {accounts: [address]})
  api.connection.on("transaction", (event) => {
    if (event.transaction.hash == txID) {
      console.log("Transaction has executed!", event)
      has_final_status = true
    }
  })
  api.on('ledger', ledger => {
    if (ledger.ledgerVersion > maxLedgerVersion && !has_final_status) {
      console.log("Ledger version", ledger.ledgerVersion, "was validated.")
      console.log("If the transaction hasn't succeeded by now, it's expired")
      has_final_status = true
    }
  })

  // There are other ways to do this, but they're more complicated.
  // See https://xrpl.org/reliable-transaction-submission.html for details.
  while (!has_final_status) {
    await new Promise(resolve => setTimeout(resolve, 1000))
  }

  // Check transaction results -------------------------------------------------
  try {
    const tx = await api.getTransaction(txID, {
        minLedgerVersion: earliestLedgerVersion})
    console.log("Transaction result:", tx.outcome.result)
    console.log("Balance changes:", JSON.stringify(tx.outcome.balanceChanges))
  } catch(error) {
    console.log("Couldn't get transaction outcome:", error)
  }

})

log
Transaction cost: 0.000012 XRP
src/sendXrp.js:33
Transaction expires after ledger: 77
src/sendXrp.js:34
Identifying hash: D5C22448C169A74F9396147EAA209DF604422C6E10A4A585B1A3B0379D4D4156
src/sendXrp.js:40
Signed blob:<PHONE_NUMBER>0000002400000009201B0000004D6140000000014FB18068400000000000000C73210330E7FC9D56BB25D6893BA3F317AE5BCF33B3291BD63DB32654A313222F7FD0207447304502210096AEF0515024368E019E2FC0D50DE59DE9603E4634F2EB44B830BE84A0F9B82D02201A9ABFFBCA2F7FF09A69FC57CD4BC76B8EFC83ADAA69C0316C719D2AD56CA10C8114B5F762798A53D543A014CAF8B297CFF8F2F937E8831454B3B3A5EE00E45F502037ADDCCF4126CFEDBFAB
src/sendXrp.js:41
Tentative result code: tesSUCCESS
src/sendXrp.js:48
Tentative result message: The transaction was applied. Only final in a validated ledger.
src/sendXrp.js:49

generateAddress creates a new address, but an account object for that address does not exist on the ledger. To create the account, fund it by sending it a payment. The payment amount must be greater than or equal to the base account reserve. On Mainnet, the base account reserve is currently 20 XRP. On a stand-alone ledger, the amount may be different.
When creating a new ledger in standalone mode, the initial funds are in the genesis address, as documented here: https://xrpl.org/start-a-new-genesis-ledger-in-stand-alone-mode.html
You can send XRP from that genesis address to your newly-generated address in order to fund and activate the account.

dd

@pwang200 Sorry, I just saw this. Yes, you can call ledger_accept with RippleAPI using the request method:
return api.request('ledger_accept').then(response => {
  /* Do something useful with response */
  console.log(JSON.stringify(response, null, 2))
}).catch(console.error);

Note: ledger_accept is as admin request, so you'll need to be connected to rippled with an admin connection.
Does that help?

@pwang200对不起，我刚看到这个。是的，您可以使用以下request方法使用 RippleAPI 调用 ledger_accept ：
return api.request('ledger_accept').then(response => {
  /* Do something useful with response */
  console.log(JSON.stringify(response, null, 2))
}).catch(console.error);

注意： ledger_accept 作为管理员请求，因此您需要使用管理员连接连接到涟漪。
这有帮助吗？

``

@pwang200 Sorry, I just saw this. Yes, you can call ledger_accept with RippleAPI using the request method:
return api.request('ledger_accept').then(response => {
  /* Do something useful with response */
  console.log(JSON.stringify(response, null, 2))
}).catch(console.error);

Note: ledger_accept is as admin request, so you'll need to be connected to rippled with an admin connection.
Does that help?

thank you


RewardCriterion mask

Hi Ruotian,
Thanks for your code!
I'd like to ask: For the RewardCriterion, in this line:
mask = to_contiguous(torch.cat([mask.new(mask.size(0), 1).fill_(1), mask[:, :-1]], 1)).view(-1)
Why are you concatenating the mask[:, :-1] with 1s at the beginning mask.size(0), 1).fill_(1)? Since the mask comes from original seq (which will be multiplied with the logprobs of the same shape as seq), why are you changing it? Perhaps you start with 1s since there is always a word at the begining, but then why are you ignoring the last row in the mask?

I found the answer in this issue: #102
Sorry for not searching!


⚠ Aither has degraded performance

In 5f6dccf, Aither ($AIH) experienced degraded performance:

HTTP code: 200
Response time: 4185 ms

Resolved: Aither performance has improved in 885ce49.


Can't capture AsyncCommand exception

Hi Stephen,
I'm trying to use AsyncCommand, this is my example code:
      public AsyncCommand TestCommand
      {
          get
          {
              if (_testCommand == null)
              {
                  _testCommand = new AsyncCommand(
                      async (x) =>
                      {
                          await SaveAsync();
                      });
              }
              return _testCommand;
          }
      }
      private async Task SaveAsync()
      {
          await Task.Delay(1);
          throw new NotImplementedException();
      }

The exception in SaveAsync isn't captured by AsyncCommand.
What is wrong with my code?
Thank you
Giampaolo

I solved in this way:
public override async Task ExecuteAsync(object parameter) { 	//Execution = NotifyTask.Create(_executeAsync(parameter)); 	Execution = NotifyTask.Create( new Task(()=>_executeAsync(parameter)));         ...
Is this right?

Waiting for your reply, I temporarily solved this way:
public override async Task ExecuteAsync(object parameter)
{
	//Execution = NotifyTask.Create(_executeAsync(parameter));
	Execution = NotifyTask.Create( new Task(()=>_executeAsync(parameter)));

An older version of my AsyncCommand did capture exceptions; however, this version does not. This is by design. Other implementations of ICommand do not capture exceptions, so this change brings IAsyncCommand/AsyncCommand more in line with mainstream MVVM.
The best way to think about AsyncCommand is like an asynchronous event handler. The behavior of throwing from AsyncCommand should be (roughly) the same as throwing from an async event handler.


fix(runtime-core): inherit attrs inconsistent (#2549）

Close #2549
Determine  if child component is effected by root comments before invoke getChildRoot when in development environment.

Thanks for the PR, but this is an incomplete fix. The more accurate fix is by detecting such cases at compile time. See 3bc2914


Updated date

fixed the timestamp to when I last updated the docs

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/ide/editorconfig-code-style-settings-reference.md
:white_check_mark:Succeeded

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


Nice names getting removed

Per @ccelio
Given simple Firrtl like the following:
circuit Mod :
  module Mod :
    input in : UInt<32>[4]
    input idx : UInt<2>
    output out : UInt<32>
    node named = in[idx]
    out <= named

We lose the nice name "named" in favor of a generated name:
module Mod(
  input  [31:0] in_0,
  input  [31:0] in_1,
  input  [31:0] in_2,
  input  [31:0] in_3,
  input  [1:0] idx,
  output [31:0] out
);
  wire [31:0] _GEN_0;
  wire [31:0] _GEN_1;
  wire [31:0] _GEN_2;
  wire [31:0] _GEN_3;
  assign out = _GEN_0;
  assign _GEN_0 = _GEN_3;
  assign _GEN_1 = 2'h1 == idx ? in_1 : in_0;
  assign _GEN_2 = 2'h2 == idx ? in_2 : _GEN_1;
  assign _GEN_3 = 2'h3 == idx ? in_3 : _GEN_2;
endmodule

It looks like it's being removed in DCE. I think this could easily be solved via a heuristic that names that start in _ lose to names that don't. We could also introduce a "temp name" annotation that's inserted whenever a temp name is inserted and those always lose out to non-temp names.

Actually it's ConstantProp that is changing the out <= named connection to out <= _GEN_0, which then causes named to get DCEd. Same argument as above can be applied here.

With recent improvements to Constant Propagation, this problem is greatly exacerbated since intentionally named wires will now be removed as well as nodes. I believe that solving it in ConstProp should be relatively straightforward given that it already has backpropagation mechanisms. CSE is not so simple but could be refactored in a similar way.
Part of solving this problem is coming up with simple, predictable heuristics for how to choose names. The one I will start with is straightforward, always prefer something that doesn't begin with _, but I imagine there may be other desirable rules like prefer Wire to Node, etc.

Just to add to the discussion, the problem is exacerbated even if the rule "keep named names over T or _GEN names" is applied.
Real world example is unpacking the fields from an AsyncResetReg (admittedly a messy hack we shouldn't have to keep around, but to be illustrative):
val FooWrData = Wire (new FooBundle())
val FooReg = Module(new AsyncResetReg(width = 32))
FooReg.io.d := FooWrData.asUInt()
val FooReadData = new FooBundle().fromUInt(FooReg.io.q)
...
In this example the name that is preserved even if you select "real" names over T or GEN may be  FooReg_io_q[31:0]. The bundle field unpacked names are lost. It's hard to make a rule about which are "more useful" or better to keep. It's sad to think there may be no way for a user to specify intermediate wires, as this seems to undo a lot of goodness that happened recently to make the Chisel-generated code human readable.


Explicit allow tls 1.1 and 1.2

Codecov Report

Merging #30 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #30      +/-   ##
==========================================
+ Coverage   23.51%   23.57%   +0.06%
==========================================
  Files          80       80
  Lines        1229     1230       +1
  Branches       95       95
==========================================
+ Hits          289      290       +1
  Misses        881      881
  Partials       59       59

Impacted Files
Coverage Δ

.../GitLabApiClient/Internal/Http/GitLabHttpFacade.cs
44% <100%> (+2.33%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks!


Half of features listed made

Hey I didn't make a pull request but I did make a github repo of the code updated. Here is what has been done to it:

Chat bug fix
Pause button not always showing
UI positioning change
Hold crouch lock
Separate sprint button
Strafe forward left/right added
Custom icons

(It's in my profile)

Thanks for creating an issue. I will work on directly implementing:

[ ] Your updated UI
[ ] F3 button
[ ] Chat bug

And I will be rewriting and implementing:

[ ] Crouch lock (Which I think I might just change it to a toggle)
[ ] Sprint button (Which I’ll also probably make into a toggle)
[ ] Strafe (Which will activate based on touching and dragging either left or right like PE)

My only issue with your suggestions is not displaying the exit button when pointerlock isn’t called. To the best of my knowledge, the exit key is the only way to get out of chest, furnace and crafting table GUI. Once I can directly get the game state, this will not be an issue.
Thanks!

My bad didn't look into the other block gui's to see if they had exits!

You’re completely fine, I had the same thing originally. I’m currently working on cleaning up the current iteration of the mobile script, and then I’ll start adding your suggestions.
Out of curiosity, do you know if the mobile controls normally have separate icons for when they are pressed?

On newer versions when they are pressed they have a pressed icon look (different colors and sometimes design). Got my code to be updated with yours also.

Yeah I have no idea what to do with crafting. Until I can figure out how to directly inject JavaScript into the client, there’s not much we can really do

Next update is going to be updating the UI. I’m about halfway through redrawing the icons to match the current new PE design.

Okay, you could also just use the PE new textures. You can just download the base resource pack and find it in the files.
https://github.com/Mojang/bedrock-samples/releases


fix: ignore tsconfig setting files

Astro should not be reading the tsconf settings file.  This causes issues  such as strict: true causing ts errors like:
JSX element implicitly has type 'any' because no interface 'JSX.IntrinsicElements' exists.
RE: https://discord.com/channels/830184174198718474/845451724738265138/895704312544653353
Closes #91.
Changes

Stops using the tsconfig.json as part of the default configuration for the Astro language tools

Testing

Tested manually be using strict: true in the tsconfig.json file before changes.

The Visual Studio Code editor highlighted all the JSX code with the  JSX element implicitly has type 'any' because no interface 'JSX.IntrinsicElements' exists. error

After changes, Code did not produce the error.
Also manually tested creating constants, using Astro props etc to ensure TypeScript still worked properly in .astro files.

Docs

No doc changes are required.

cc: @matthewp

Looks great! Context here is that Astro files have caused people with different tsconfig settings quite a bit of pain because those config settings are meant to apply to .ts/.tsx files and by applying them to .astro files it causes false-positive errors.
@justinnoel can you add a changeset by running yarn changeset and going through the prompts? Thanks!

Hm, not reading some of what people are doing in discord makes me think maybe we should still be reading their config: https://discord.com/channels/830184174198718474/871832088263933973/905904428538789888
Can you try removing your change and instead overriding strict? Would doing that fix your issues?

Hm, not reading some of what people are doing in discord makes me think maybe we should still be reading their config: https://discord.com/channels/830184174198718474/871832088263933973/905904428538789888
Can you try removing your change and instead overriding strict? Would doing that fix your issues?

Looking at this now. Definitely seems like we need to honor tsconfig and this fix is too nuclear.

Pretty sure I found a different solution to this whole JSX problem.
I'm going to park this PR for now and submit a different one. If the new one is the right solution, I'll come back and close this one.

I'm closing this now because I've found a way to resolve the issues with strict: true implementation.
I've setup a sample repo that converts the standard Astro Blog template to use strict: true:
https://github.com/justinnoel/astro-blog-strict


There is not Topic class in BinTray

Hi.
When I installed this broker from BinTray, I saw that there is not the Topic class  in the lib folder (io.moquette.spi.impl.subscriptions.Topic).
please correct it.
best regards

The Topic class is a new entry in master 0.10 that's not yet on BinTray. As
you can see BinTray has tge latest stable version the 0.9 so there is
nothing to correct,once 0.10 is stable I'll release on BinTray
Il giorno 03/giu/2017 15:25, "BigGitWorld"<EMAIL_ADDRESS>ha
scritto:

Hi.
When I installed this broker from BinTray, I saw that there is not the
Topic class in the lib folder (io.moquette.spi.impl.subscriptions.Topic).
please correct it.
best regards
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/andsel/moquette/issues/326, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA_Y0f-RZjyAvj0cauc2Mq3fHqNgDArJks5sAV60gaJpZM4NvFW_
.

thank you very much @andsel


Can't open

Can't open the file

This project is not released, finished, or even functioning. I have not published the functional code. I'm not sure what you were expecting - but whatever it was, it's not available to you, or anyone else.
Your issue is not helpful or capable of being solved, even in the case that it was published against a functioning repository. How exactly do you expect be to help you with exactly four words describing your issue?

This project is not released, finished, or even functioning. I have not published the functional code. I'm not sure what you were expecting - but whatever it was, it's not available to you, or anyone else.
Your issue is not helpful or capable of being solved, even in the case that it was published against a functioning repository. How exactly do you expect be to help you with exactly four words describing your issue?

I saw a PUBLIC repository about a topic I was looking for, believing it would work, but there is NO information that the code is incomplete and non-functional. I tried to open it and all that appears is a blank window with nothing else.
I created an issue exactly to report that it doesn't work.
More information is missing.
But I appreciate the information.
I've already resolved it with another repository. But even so, thanks.

but there is NO information that the code is incomplete and non-functional

I otherwise would be pointing to the non-existence of any relevant code in the entire project (only 1 .cs file) on the master branch.


Use context as dict object only

Removed Context import and changed to using context as dict object only in utils.py as new Django ver only supports context as a dict object

Thanks a lot!

When will this fix be released in PyPi? Without it it makes my forms unusable. I'll start looking for alternatives to django-forms-builder if there is this package is no longer maintained.


missing introduction to GitHub

I failed to provide a proper introduction to GIT and GITHUB in class that allows students to check out the course material and in particular the tutorial notebooks.

Noted in ASANA


Node not responding with pokt-calculator

Hey, im trying to connect my node to the pokt-caculator  but it's not working... I have setup a network with the pokt-calculator and my pocket node, I tested it to verify if they are seeing each other and it worked but when I run the docker-compose and try to test it via web the node height output is 0.
bellow ill leave my code of the docker-compose that i'm using
`version: "3"
services:
monitoring-service:
container_name: pokt-calculator-api
ports:
- 8081:8081
build:
context: ./monitoring-service
environment:
LISTEN: <IP_ADDRESS>:7878
DBPATH: /db
# Update the url below to use your node's service url,
# or get a free endpoint at https://www.portal.pokt.network/#1
POCKET_URL: http://node1:8081/v1
volumes:
- ./.pokt-calculator-db:/db
ui:
container_name: pokt-calculator-ui
build:
context: ./ui
environment:
# Update the url below
RPC_URL: http://node1:8081
ports:
- 4444:80
volumes:
disk-db:
networks:
default:
external: true
name: node-network`

Hey @kaelabbott - sorry for the very late reply.  A little clarification:
RPC_URL should point to a url that your browser can access (i.e. a public url).  For example if you run this on your server at mynode.com, the "monitoring-service" will be listening at mynode.com:7878 so your RPC_URL should be https://mynode.com:7878
This is the url the react app will make requests to.  You can verify the correct url by checking in your browser, eg https://mynode.com:7878/height
To clarify further: on pokt.tools this RPC url is set to https://api.pokt.tools which can be verified by visiting https://api.pokt.tools/height
Hope that helps!


Bug: Changing the API and regenerating client-go does not add changes to code repo.

When I modify the api and try to generate client-go, my client-go is generated and placed in my go src directory (ie /home/ec2-user/go/1.20.2/src/sigs.k8s.io/jobset/client-go/).
I think that it should go to client-go folder in the repo.  Currently if I modify the API I would have to copy /home/ec2-user/go/1.20.2/src/sigs.k8s.io/jobset/client-go/ and rewrite the folder in client-go.
This is not ideal.
Any ideas on what is going wrong?

/kind bug

/assign

When originally developing this I found that generate-groups.sh script called by update-codegen.sh accepts an "output package" parameter, which I specified as sigs.k8s.io/jobset/client-go, and treats each thing between the / chars as a directory relative to $GOROOT/src. So in my case this writes to ~/go/src/sigs.k8s.io/jobset/client-go, and since my jobset repo is located at ~/go/src/sigs.k8s.io/jobset, this is the correct location.
However, I can reproduce this issue by cloning jobset in my /tmp directory and running make, it still writes the client-go to the same location relative to my Go root, not relative to my current working directory. I don't see an obvious way to change this in the script, as it only seems to accept an "output package" parameter, not a filesystem location of to create output files relative to.
I think maybe at the end of the script we just need to move the output client-go directory to the location relative to the current working directory.

I think copying is fine.  Ideally I’d like to get the this running in a container so I think we would copy the build out of co Rainer anyway.


[ENH}: Nextjs top loader

Pull Request Title
Thank you for contributing to our project! Please provide a summary of your changes and reference any related issues below.
Issue Number: Fixes # (issue number)
🛠 Type of Change

[ ] 🐛 Bug fix
[ ] ✨ New feature
[ ] 🚨 Breaking change
[ ] 📚 Documentation update

✅ Checklist
To ensure a smooth review process, please check off each item as you complete it:

[ ] Code Style: My code adheres to the project’s style guidelines.
[ ] Self-Review: I have reviewed my own code and made improvements.
[ ] Comments: I’ve added comments to explain complex or non-obvious parts of the code.
[ ] Documentation: I’ve updated the documentation to reflect my changes.
[ ] Warnings: My changes introduce no new warnings.
[ ] Tests: I’ve added tests to verify that my changes work as expected.
[ ] Passes Locally: All new and existing unit tests pass on my local machine.

We appreciate your contributions! If you have any questions or need further assistance, feel free to reach out.

Hey @Rudra-Sankha-Sinhamahapatra done!

Add Screenshots or Video links for bits you change in the UI

@cb7chaitanya ne change is we get a smaller loader when we move from route to another or when a request is being sent, other is when we scroll down we a scroll to top button

Any  further chnages required?

@Rudra-Sankha-Sinhamahapatra, @cb7chaitanya  i fixed one lint error of Image in this too, UseEffect one looking into that, kindly suggest further changes if required?

Vai maybe I have solved the eslint warning of img tag
My pr has been merged kindly please sync your branch and then see if is reproducing any issues or not
Thanks for your contribution

And it will be good if you test prettier and eslint locally and provide screenshots and videos it will be good
Ensure the prettier must override the eslint configurations with no eslint errors or warnings

@vanshavenger

@Rudra-Sankha-Sinhamahapatra  @cb7chaitanya  Done Guys! :)))

Please provide screenshots that your pr doesn't create any linting warnings

@vanshavenger bhai also give those pR that what kind of ui change you made

@Rudra-Sankha-Sinhamahapatra bhai Alreeady mentioned it in the comments and attached screenshot for scroll to the top too.


feat: add event emission to __execute__

Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?
We don't emit an event for a transaction's execution inside __execute__, which is done in case of KKRT-0.
Resolves: #619
What is the new behavior?

We emit event inside __execute__

Does this introduce a breaking change?

[ ] Yes
[x] No

This change is

crates/contracts/src/eoa.cairo line 22 at r1 (raw file):
Previously, enitrat (Mathieu) wrote…
This needs to be imported to fire events

I do not understand this comment, are u asking whether this needs to be imported? or are u saying this needs to be here, but in that case it is here.

crates/contracts/src/eoa.cairo line 56 at r1 (raw file):
Previously, enitrat (Mathieu) wrote…
Rustdoc style documentatoin

Done.

crates/contracts/src/eoa.cairo line 22 at r1 (raw file):
Previously, enitrat (Mathieu) wrote…
I edited on github but apparently it is not reflected here - It was a question lol. Is this required?

Oh, lol!
So, scarb auto-imported this, but I did a built without it, and it seems it builds just fine, so we can remove.

crates/contracts/src/eoa.cairo line 22 at r1 (raw file):
Previously, enitrat (Mathieu) wrote…
ok - actually let's keep it, it doesn't hurt and there is a risk that when we move to a more restricted prelude, we lose some auto-imported elements

Okay, lemme bring this back!


[gettext] Add feature tools

Fixes #13518.

@Neumann-A I noticed that libtools didn't generated, what's wrong?

On my mac the follwing tools get generated: autopoint gettext msgattrib msgcat msgcmp msgcomm msgconv msgen msgexec msgfilter msgfmt msggrep msginit msgmerge msgunfmt msguniq ngettext recode-sr-latin xgettext gettextize envsubst
Is there something missing?

@JackBoosY libtools is not part of gettext? autopoint is. (which is just a script)
autopoint.in and gettextize.in needs to be changed to have:
# Set variables
# - gettext_datadir     directory where the data files are stored.
prefix="$(dirname "$0")/../../../"
datarootdir="${prefix}/share/gettext"

to be relocatable. Otherwise they hardcode the absolute path


Fixes #98: Redesigned and scrollable AppDetails page

For:
Issue #98: If description is long or while in landscape mode, the bottom part of AppDetails is cropped and inaccessible
Changes:
(edited for 2f329eb)

New AppDetails page layout

Screenshots and details area now vertically scrollable

Contact button now in the overflow menu

Updated support libraries

@d4rken yeah I was thinking that too, should I change it back to expanded by default?
While the design docs has an example where it’s okay to have both the toolbar and the title underneath, i think it looks a bit strange
Also, I’ve always found the existence of close buttons itself strange as well, given that every android phone has a far more accessible back button to use instead. Considered removing the toolbar entirely and placing the options and stuff under the description
Another option is to squeeze the app icon, name, etc into the toolbar but that’s problematic for longer names

@d4rken yeah I was thinking that too, should I change it back to expanded by default?

Up to you.

While the design docs has an example where it’s okay to have both the toolbar and the title underneath, i think it looks a bit strange

I agree, the expanded view on your 3rd screenshot looks weird.

Also, I’ve always found the existence of close buttons itself strange as well, given that every android phone has a far more accessible back button to use instead. Considered removing the toolbar entirely and placing the options and stuff under the description

I think it's complex. For starters the back button is not always located in the same position or rather same distance. Think tablets. Reaching it may be more cumbersome than accessing the toolbar. There are also different things happening in some cases depending on whether the toolbar "back" and the backbutton is used (lookup backstacknavigation).
The "X" as icon might not be appropriate, the action we are doing is changing screens, "navigating", the "X" is more of a discard/delete icon (at least on Android).

Another option is to squeeze the app icon, name, etc into the toolbar but that’s problematic for longer names

Just thinking out loud:

Moving the toolbar actions below the blue header frame, at the beginning of the of the white frame.
Move the title into the toolbar, if it's longer than the full width of the toolbar, tough luck
Putting the app icon as collapsing header background

Maybe we could also look at Google Play. I wonder if we could pull the header graphic from Google Play somehow... Some more graphics would give us more to work with design wise...

@d4rken what do you think about 0a4735b? Updated the PR with new screenshots

Looked weird when I tried it because of how close to the FAB the buttons end up. I don't think many people actually attempt to contact developers, so perhaps the overflow menu is a better home for it, and the toolbar ends up looking neater.

1st and 3rd looks good, middle not due to cutoff. What would look cool here would be fade out the icon and allow the header to collapse to a 1 toolbarheight. The FAB could detach and move to the bottom right of the screen...
Or maybe the toolbar collapses to 1 tlb height and the icon + app name appear in the toolbar.
Ellipsizing a too long title and limiting it to 1 line is okay in my opinion when we just have 1 tlb height of space.
I think the title should have gravity START, not CENTER.

I tried this, but with the icon inconsistencies and the fact that icons typically aren't designed to be displayed at the size needed to fill the header, it didn't look too great. The header with the category, tags and icon does collapse now though, which I demonstrate in one of the screenshots

Okay.

@d4rken what do you think about 0a4735b? Updated the PR with new screenshots

Code looks good to me.

middle not due to cutoff. What would look cool here would be fade out the icon and allow the header to collapse to a 1 toolbarheight

That’s what happening in the middle screenshot, it’s using the parallax collapse so when you scroll up you only end up seeing the toolbar at the very top (3rd screenshot). The snap flag could force the animation to snap into place and avoid the cropping effect

I think the title should have gravity START, not CENTER.

I centered it to match the centering of the other header elements

Or maybe the toolbar collapses to 1 tlb height and the icon + app name appear in the toolbar.

This could be cool, I’ll give it a try

Oh that looks fancy! 👌

Will open a new issue for animating the app icon, for now I think this looks nice enough:

Animation has the header elements fade:

Thoughts?

Looks awesome 👌 👌 👌

When a lot of revisions happen from opening a PR to the end (ie the design changing significantly), is it best practice to keep the original post updated with the latest commit or post the changes in comments below?

I'd say comments below, i.e. chronological, just like commits appear in the feed.
That way someone following the PR can also understand the changes chronologically.
You made some nice screenshots, you could make a PR against README.md and add them. They look way more fancy then what we currently have 😄

Yeah that makes sense, will stick to that in the future
Screenshots added in #139


Listeners don't show up in server

Hi!
What could be the reason that M5AtomMatrix listeners don't show up in TallyArbiter server?
I have set correct server IP and port in listeners, server and listeners are connected using Mikrotik router with preconfigured settings.
Thank you for your help in advance!

Cancel, I was using 2.0.16 version with newer listener image.
All fine!


File saving failure

Error Code
Error: Too Many Requests

Error: Too Many Requests
    at Request.<anonymous> (https://homebrewery.naturalcrit.com/homebrew/bundle.js:78451:21)
    at Emitter.emit (https://homebrewery.naturalcrit.com/homebrew/bundle.js:1954:20)
    at XMLHttpRequest.<anonymous> (https://homebrewery.naturalcrit.com/homebrew/bundle.js:78745:10)
{
  "status": 429,
  "method": "PUT",
  "url": "/api/update/HTXpcSuLd9n3"
}

### Your description of what happened:

While I was writing on my homebrew file, the save stopped working, I tried to restart but the problem still persists.

Closing in favor of #3770


Upgrade to Bootstrap 4

From @thiennn on April 12, 2018 2:46
Bootstrap 4 has been released for a while. It would be nice if Identity UI support it by default
Copied from original issue: aspnet/Identity#1751

This would be wider than just identity, so I'm going to move it to the templating repo.

Dup of https://github.com/aspnet/templating/issues/278. We plan to do this in 2.2.0.


docs: Update style guide for user documentation.

Automated message from Dropbox CLA bot
@synicalsyntax, it looks like you've already signed the Dropbox CLA. Thanks!

Also @timabbott , I'm not sure if it's just me but the [Your Account](/#settings/your-account) link isn't working for me.

Merged, thanks @synicalsyntax!
The "Your Account" doesn't work since it's intended for the settings redesign branch which we're running on chat.zulip.org but hasn't been merged into master yet.
The test failure seems to be a nondeterministic issue; I can't reproduce it running that same suite locally...


LDE & Drop Roller & Sub Cost for Splattershot Jr. & Uncommon Ability of Inkline

LDE should start with 1 AP at a score of 50, not 49.
Drop Roller when activated should give the player 30 AP of SSU, RSU, IRU, not 10AP (not completely sure, but several jp wikis say it's 30).
Sub costs for Splattershot Jr. are not accurate. The formula should be sub.cost / 1.1 which equals to sub.cost * 0.909.
Uncommon Ability of Inkline is not showing correctly, it appears to be the one of the priorly selected brand.

Thanks for the bug report! Plan to work on fixing these this week.


looking for rtmp standard

I'm study the protocol,  but  I only find rtsp protocol,  may I ask where I can find the RTMP protocol

you can download the rtmp specification here: https://www.adobe.com/devnet/rtmp.html

you can download the rtmp specification here: https://www.adobe.com/devnet/rtmp.html

@a1q123456  Thanks

@a1q123456  Thanks


Multiple attachments in Mailable emails results in duplicate files

Laravel Version: 5.5.19
PHP Version: 7.1.10
Database Driver & Version: Mysql 5.7.20

Description:
In my controller I collect the file attachments into an array like this:
 foreach($request->file('attachment') as $attachment) {
                    $attachments[] = [
                                        'file' => $attachment->getRealPath(),
                                        'options' => [
                                            'mime' => $attachment->getClientMimeType(),
                                            'as'    => $attachment->getClientOriginalName()
                                        ],
                                    ];
                }
Mail::to('test@test.com')->send(new ResearchCreate($requestDetails,$attachments));

Then in my Mailable I attach the files like this:
$message = $this->subject('New research request')->markdown('app.emails.research.create');

            foreach($this->attachments as $attachment) {
                $message->attach($attachment['file'],$attachment['options']);
            }
            return $message;

Steps To Reproduce:
Use the above controller and Mailable and try sending.

Can you explain what happens when you run the code? What output you are getting versus what you expected to happen?

If I attach 1 file lets call it File1.txt and submit it. In the email received I have 2 files both are File1.txt.
If I attach 2 files File1.txt and File2.txt I get 4 files attached to the email. 2 files of File1.txt and 2 files of File2.txt.
I have dd($this->attachments) in my Mailable build() method and it shows the correct amount of files in the array. Just seems like something in the Mailable class that is added them a 2nd time

I have create a new laravel project with the code used to reproduce this issue.
NOTE: setup your .env file to use Mailtrap.io to capture the emails.
https://github.com/rlitten/laravel-mailable-test

Your example repo has only one commit, it's hard for us to differ between the original Laravel source and the code changes you've done.

Sorry, I have updated the repo to reflect the changes now

$attachements is used in mailables to send attachements that's why you have multiple instances, use Mailable::attach() or change the property name in your mailable constructor.


BUG: Can't use configurations from command line.

Maybe PEBCAK, but this fails for me:
$ eslint -c node_modules/eslint-config-defaults/configurations/walmart/es5-node.js *.js

with:
YAMLException: Cannot read config file: /Users/rye/scm/fmd/formidable-react-component-boilerplate/node_modules/eslint-config-defaults/configurations/walmart/es5-node.js
Error: JS-YAML: end of the stream or a document separator is expected at line 1, column 13:
    "use strict";

I ran into this error too, unfortunately. What is going wrong?

This is because the configurations are defined as js files, whereas eslint expects yaml.  @baer, are the constituent configs composable using just the YAML syntax and extends: ... ?

@divmain @baer -- Oh yeah. So, JSON works too, and I think we have easily convertable to JSON JS at this point (now that lodash is gone). Let's convert to that!

Yea the ESLint file reader is kinda YAML happy. If it can't figure something out about a file it just assumes it's YAML. Between that and the fact the JSON has no comments I think YAML all the way down might be a good call even though it's going to be messy. This is why I went with JS in the first place but it seems like ESLint is not really that happy with my choices :/
It's a tradeoff between
  "no-unused-vars": [2, {"vars": "local", "args": "after-used"}]

  # disallow declaration of variables that are not used in the code
  "no-unused-vars":
    - 2
    -
      "vars": "local"
      "args": "after-used"

Since YAML is a superset of JSON, you can also do
  no-unused-vars: [2, {"vars": "local", "args": "after-used"}] # disallow declaration of variables that are not used in the code

or
  no-unused-vars: # disallow declaration of variables that are not used in the code
    [2, {"vars": "local", "args": "after-used"}]

if you care about line length.
I find this to be much more readable than the YAML-arrays-on-new-lines syntax.

Which files have to be converted to YAML? All .js files?

@pascal08 -- Anything you want to use as a CLI configuration.
@baer @divmain -- Playing around with http://nodeca.github.io/js-yaml/ we don't actually have to do that much refactoring. Here are some YAML-compliant examples:
https://github.com/walmartlabs/eslint-config-defaults/blob/master/configurations/walmart/es5-node.js
{
  "extends": [
    "./es5.js",
    "../../rules/eslint/node/walmart.js"
  ],
  "rules": {
    "strict": [2, "global"]
  }
}

https://github.com/walmartlabs/eslint-config-defaults/blob/master/rules/eslint/strict/walmart.js
{
  "rules": {
    # require that all functions are run in strict mode
    "strict": [2, "never"]
  }
}

Basically:

Remove module.exports and use strict
Convert // to #

... or we can go "full YAML", but even there, I'd recommend something like:
rules:
  # require that all functions are run in strict mode
  strict: [2, "never"]

Which is (1) keep rule arrays / scalars as straight json, (2) keep comments above the rules.


Are you planning to make embedded transitions?

Is your feature request related to a problem? Please describe.
This is more of a question about whether there will be a built-in implementation of transitions.
Describe the solution you'd like
I would like to see a built-in ability to create transitions. As an example: https://routify.dev/docs/transitions
Describe alternatives you've considered
Write your own implementation?
How important is this feature to you?
Strongly. If you migrate a project from Routify, you will need transitions, because they make the project look more dynamic.

@FlymeDllVa It'd be super useful if you could fill out the provided template.

I don't know that there's a huge amount of value in SvelteKit providing anything beyond what Svelte provides, so I'm going to close this. If there's something that's not possible with Svelte by itself (that is somehow fixable within SvelteKit, rather than requiring a change to how transitions work in Svelte) then it would be helpful to have an issue with those specifics — thanks


2x2 Mosaic: update parameter selection frontend to allow users to select reference groups for all variables

Related to https://github.com/VEuPathDB/EdaDataService/issues/130
Final parameter selection UI can be found here: https://www.figma.com/file/Su1eNm3Bby85CfrnX42L7q/2x2-table-UI?node-id=44%3A431&t=ZCxlWaPOOG1AbgKr-0

I keep running into this bug while trying to do this:

This is currently only testable on a dev site due to the backend changes.

Also, I noticed that value selection is only available (enabled) after both X and Y variables have been selected. It left me wondering for a second why the X value selection was not available after I selected the X var.
I initially implemented this along the lines of your thinking, but then I interpreted the mockups to enable those buttons only after table vars are selected.

@jernestmyers - I spoke to the team and we feel that the Quadrant A values should become active immediately after their corresponding x- or y-axis variable has been selected. Sorry this is different from the mock-ups! And thank you for making the update.

I've merged changes to get the disable/enable behavior to work as requested. The QA site's backend is having trouble getting up-to-date with the client code, but in the meantime things can be viewed/tested on my dev site.

Looks great, thanks Jeremy!


Input 6.2.1 beta: input_controller_object depth has been changed (expected 16001, got 0)

Hi. I am new to Gamemaker so I am starting off with Hero's trail template and I wanted to add the Input library to it.
This is not necessarily an Input library bug, but I would still love to understand what is going on so I can avoid it during further game development.
I get this error after I enter the first room:
...
PS5 controller recognized here...
...
Input: Hotswapping on mouse button press
Input: Setting player 0 profile to "keyboard_and_mouse"
ERROR!!! :: ############################################################################################
ERROR in
action number 1
of Create Event
for object obj_pillar_fallen:
Input 6.2.1 beta:
input_controller_object depth has been changed (expected 16001, got 0)
Please ensure that input_controller_object is never manually created and depth is not manually set
at gml_Script___input_error (line 14) -         show_error("Input " + __INPUT_VERSION + ":\n" + _string + "\n ", false);
############################################################################################
gml_Script___input_error (line 14)
gml_Script_anon___input_initialize_gml_GlobalScript___input_initialize_4072___input_initialize_gml_GlobalScript___input_initialize (line 207) -                 __input_error("input_controller_object depth has been changed (expected ", __INPUT_CONTROLLER_OBJECT_DEPTH, ", got ", input_controller_object.depth ,")\nPlease ensure that input_controller_object is never manually created and depth is not
However, when I create an empty room with some tiles and obj_player, I don't get this error.
Can it be related to the depth spacing, perhaps? No idea.
Thanks!

Peter,
The error tells you exactly what you need to know, there's no way I can think to better explain the error to you than the message you have pasted here. If reading the error message given the situation you presented is not adequate information for you to solve the problem, perhaps you are overextending yourself by using the library. Consider carefully if you might be better served by learning without the library, and perhaps coming back to it in the future if you determine why specifically you need it, equipped with a bit more general know-how that will allow you successfully problem solve when you encounter errors. If you are determined to use the library anyway, please follow up with what steps you have tried to avoid or fix the error you are encountering, given the content of the error
message.

Hi, I found the script that changes the depth of all instances, so I excluded input_controller_obj from that change. You are right, the information was there, I just need to gather more experience. Have a great day!

Glad to hear it! depth = -y is a common enough pattern that we can standardize on that perhaps, since the controller object's position is otherwise insignificant.


Enable memory monitoring in CS

Enable memory monitoring in CS (see go/monitoring-enablement-in-hardened-image-1-pager).
This PR includes changes to:

Unmask node-problem-detector.service for hardened image.
Override the default config file system-stats-monitor.json to collect memory/bytes_used metrics only for CS.
Configure preload.sh to allow hardened/debug images have this change.
Add a library systemctl to use dbus API to start node-problem-detector.service.
Introduce tee.launch_policy.monitoring_memory_allow to launch policy and tee-monitoring-memory-enable to launchSpec.
Add a library nodeproblemdetector which provides methods to override node-problem-detector configurations. This library will not be used for now, but will be used in the future if operators want to opt-in more metrics monitoring options: cpu, disk, etc.
Add image tests to check if memory monitoring is enabled.

@jkl73 Added three another config files in addition to system-stats-monitor.json to turn off system-log-monitor and custom-plugin-monitor for node-problem-detector. These two daemon processes were intended to monitor/report kernel logs, docker logs and boot process, but we decided to turn them off.


⚠ ClamAV has degraded performance

In ca12928, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 111 ms

Resolved: ClamAV performance has improved in 4ffbc8d.


(GH-100) Fix Choco package source folder copy criteria

Changes the copy path to copy from the files folder to the root of the destination.
Fixes #100

@pauby thanks for fixing this!


android small device UI

Inheritance modal and CTA

Send screen  alignment

Resolved,

Verified this issue on keeper app v 1.1.8(274)


Add job for building docker image of Che Devfile Registry

This PR add job for building docker image of Che Devfile Registry.
https://github.com/eclipse/che-devfile-registry
It's similar to Plugin Registry and cici_build.sh script is the same (except target dockerhub repository name and used Dockerfile)

Can one of the admins verify this patch?

@jmelis @maorfr  PTAL
it looks good, but quay repos must be created for these images.

[test]

@maorfr how we can move forward with this PR?

[test]


client.subscriptions.create() Outdated

Only required params seems to be sourceType

However

The documentation and error message could be clearer. While sourceType is the only parameter that is actually required for all create calls, one of the subscription detail params is also required:
export interface SubscriptionRequest {
	sourceType: SubscriptionSource
	device?: DeviceSubscriptionDetail
	capability?: CapabilitySubscriptionDetail
	mode?: ModeSubscriptionDetail
	deviceLifecycle?: DeviceLifecycleDetail
	deviceHealth?: DeviceHealthDetail
	securityArmState?: SecurityArmStateDetail
	hubHealth?: HubHealthDetail
	sceneLifecycle?: SceneLifecycleDetail
}


StemGNN[<Library component: Model|Core|etc...>]

What happened + What you expected to happen
Following the example for StemGNN [copy paste or type]  :

does not specify how to import StemGNN [ I did From neuralforecast.models import StemmGNN
when running the example : error StemGNN does not support futr_exog_list.
when I type StemGNN ?  it shows that it does support it.
I expected it to run smoothly as the other models do.
NOTE: the summary table of all the models suggests StemGNN does not support futr.. [IE: They are not checked, ]

Versions / Dependencies
neuralforecast 1.7.3
I do not get this issue with other models from neuralforecast.models import
Reproduction script
The code is from neuralforecast StemGNN .
Issue Severity
High: It blocks me from completing my task.

Thank you for the quick response.Should it support these ?I was able to run the AirPassanger dataset on StemGNN without any exogenous list.The result were poor.
Sent from Yahoo Mail for iPhone
On Tuesday, July 30, 2024, 9:02 AM, Olivier Sprangers @.***> wrote:
Hi,
The example for StemGNN is indeed flawed, apologies.

You're right, the example should include the following line:
from neuralforecast.models import StemGNN
StemGNN does not support exogenous variables
See 2.
See 2.

Hope this helps, let me know.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Thank you.Ive been modeling Bitcoin daily prices and have some success using LSTM with an exponential moving average and a simple moving average as a historical Exog lists .MSE ~ 3000. Better than most that I’ve tried but not good enough yet.
Sent from Yahoo Mail for iPhone
On Tuesday, July 30, 2024, 10:25 AM, Olivier Sprangers @.***> wrote:
Thank you for the quick response.Should it support these ?I was able to run the AirPassanger dataset on StemGNN without any exogenous list.The result were poor.
No, it shouldn't support exogenous. And StemGNN is not the strongest model, and a bit hard to get good results from, so I can understand that the results are not that good. I'd advise you to start with TSMixerxor MLPMultivariate, if you want to use a multivariate with exogenous.
And a univariate model may perform better than a multivariate model, so just because the problem requires to forecast multiple (possibly correlated) timeseries, it doesn't mean a multivariate method is the best choice.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Design: User on-boarding experience

User Story
As a non-admin user, I want to configure/setup my settings to start monitoring/managing my AWS accounts that I have visibility into so that I can understand at a high level where my spend is located.
Impacts

API: maybe
UI: definitely
Docs: definitely

Role

User that is not in an administrative role who might not have the ability to add an AWS account, but can view cost from multiples

Assumptions

An admin has already added AWS accounts to HCCM to view data
A user has permission to login to HCCM and view cost appropriate cost information

UI Details
Acceptance Criteria

[ ] Review with Mgmt UXD (includes Insights Platform) stakeholders
[ ] Review with HCCM stakeholders
[ ] Post designs to invision

Will close out this issue as we are not going to address this for the POC.  If a user needs to be added to the app, it will be handled internally and w/o a UI for the customer owner to deal with.  In future, we will rely on the Insights Platform or RH SSO access to grant a user permission to use the app.


Bugfix nextscene

see fc609ea for the main fix. other commits are just cleanup
NextScene should add handlers for all sockets
Previously, only the player who had clicked NextScene was able to take
actions like "Give Clue", because the Server had only setup listeners
for that one socket. It was possible to workaround this by having other
players refresh the page. This change makes that unnecessary.

Nice find!


[Snyk] Security upgrade jekyll from 4.2.0 to 4.2.1

Snyk has created this PR to fix 2 vulnerabilities in the rubygems dependencies of this project.
Snyk changed the following file(s):

Gemfile
Gemfile.lock

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Uncontrolled Resource Consumption ('Resource Exhaustion') SNYK-RUBY-REXML-7577227
  738

Denial of Service (DoS) SNYK-RUBY-REXML-7577228
  631

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Uncontrolled Resource Consumption ('Resource Exhaustion')
[//]: # 'snyk:metadata:{"customTemplate":{"variablesUsed":[],"fieldsUsed":[]},"dependencies":[{"name":"jekyll","from":"4.2.0","to":"4.2.1"}],"env":"prod","issuesToFix":[{"exploit_maturity":"Proof of Concept","id":"SNYK-RUBY-REXML-7577227","priority_score":738,"priority_score_factors":[{"type":"exploit","label":"Proof of Concept","score":107},{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"6.9","score":345},{"type":"scoreVersion","label":"v1","score":1}],"severity":"medium","title":"Uncontrolled Resource Consumption ('Resource Exhaustion')"},{"exploit_maturity":"No Known Exploit","id":"SNYK-RUBY-REXML-7577228","priority_score":631,"priority_score_factors":[{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"6.9","score":345},{"type":"scoreVersion","label":"v1","score":1}],"severity":"medium","title":"Denial of Service (DoS)"},{"exploit_maturity":"Proof of Concept","id":"SNYK-RUBY-REXML-7577227","priority_score":738,"priority_score_factors":[{"type":"exploit","label":"Proof of Concept","score":107},{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"6.9","score":345},{"type":"scoreVersion","label":"v1","score":1}],"severity":"medium","title":"Uncontrolled Resource Consumption ('Resource Exhaustion')"},{"exploit_maturity":"No Known Exploit","id":"SNYK-RUBY-REXML-7577228","priority_score":631,"priority_score_factors":[{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"6.9","score":345},{"type":"scoreVersion","label":"v1","score":1}],"severity":"medium","title":"Denial of Service (DoS)"}],"prId":"eaa1f818-672e-49d7-a0c2-f7bb01496bc2","prPublicId":"eaa1f818-672e-49d7-a0c2-f7bb01496bc2","packageManager":"rubygems","priorityScoreList":[738,631],"projectPublicId":"93fa4dd8-b5c5-4bfa-b882-35106bcfd8c7","projectUrl":"https://app.snyk.io/org/scanapi/project/93fa4dd8-b5c5-4bfa-b882-35106bcfd8c7?utm_source=github&utm_medium=referral&page=fix-pr","prType":"fix","templateFieldSources":{"branchName":"default","commitMessage":"default","description":"default","title":"default"},"templateVariants":["updated-fix-title","priorityScore"],"type":"auto","upgrade":["SNYK-RUBY-REXML-7577227","SNYK-RUBY-REXML-7577228"],"vulns":["SNYK-RUBY-REXML-7577227","SNYK-RUBY-REXML-7577228"],"patch":[],"isBreakingChange":false,"remediationStrategy":"vuln"}'

@dependabot close


Update head.ejs

To make favicon available for blog post page. Currently, only top page show favicon

@phuongnq nice catch 👍


[AF] Function state Enable/Disable switch is not working.

Enable/Disable switch is not working. Below is the behaviors:
a. Disabled function could still be triggered.
b. Disable a function, then all other functions' "Function state" switch will turn to "Disabled".
c. After behavior b, F5 refresh page, verify no function is in "Disabled" state.

ea97813eb3246c7d8c06e4de014a0dfa5b8c3e39


install

Problem encountered on https://dotnet.microsoft.com/en-us/learn/dotnet/hello-world-tutorial/install
Operating System: windows
C:\Windows\System32>dotnet --info
Host (useful for support):
Version: 5.0.17
Commit:  6a98414363
.NET SDKs installed:
No SDKs were found.
.NET runtimes installed:
Microsoft.NETCore.App 5.0.17 [C:\Program Files (x86)\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 5.0.17 [C:\Program Files (x86)\dotnet\shared\Microsoft.WindowsDesktop.App]
To install additional .NET runtimes or SDKs:
https://aka.ms/dotnet-download
If I run the following command I get this error
C:\Windows\System32>dotnet new console -o MyApp -f net6.0
Could not execute because the application was not found or a compatible .NET SDK is not installed.
Possible reasons for this include:

You intended to execute a .NET program:
The application 'new' does not exist.
You intended to execute a .NET SDK command:
It was not possible to find any installed .NET SDKs.
Install a .NET SDK from:
https://aka.ms/dotnet-download

C:\Windows\System32>

Hi @patgio5!
dotnet --info is showing that there are no SDKs installed. If you're sure you installed, it could be an issue with PATH.
Check the output of the where.exe dotnet command and if your system is x64 and the x86 path returns first, please follow the steps at https://github.com/dotnet/core/issues/5962#issuecomment-780084535 to fix the issue.
Let us know how that goes.

Thanks Maira. With your indications I believe I have solved the
problem. Unfortunately
I'm a bit "rusty" and with WIN11 I still can't move well in his programming
world. Best regards
Patrizio
Il giorno ven 4 nov 2022 alle ore 23:43 Maira Wenzel <
@.***> ha scritto:

Hi @patgio5 https://github.com/patgio5!
dotnet --info is showing that there are no SDKs installed. If you're sure
you installed, it could be an issue with PATH.
Check the output of the where.exe dotnet command and if your system is
x64 and the x86 path returns first, please follow the steps at #5962
(comment)
https://github.com/dotnet/core/issues/5962#issuecomment-780084535 to
fix the issue.
Let us know how that goes.
—
Reply to this email directly, view it on GitHub
https://github.com/dotnet/core/issues/7927#issuecomment-1304316320, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A4ALCHTNR3K7GBPH66UVMD3WGWGSLANCNFSM6AAAAAARXSGHBE
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'm glad to hear the problem is solved. Good luck with your adventures with the programming world. If you run into any issues, file a new issue and we'll try to help.


Reduce record time in Stocks drawer animation by 55%

Now we use a ForcedLayer in the navigator overlay to cache the recording for
each entry in the overlay. This mechanism just caches the display list, not the
underlying pixels.
Also, remove the "dispose" notification in the Layer tree because it was
disposing layer too eagerly. We don't actually need this notification for
anything other than eagerly freeing some C++ memory.

LGTM, but file a bug to track the dispose thing, since we added that for perf originally.

We didn't really measure the perf impact, so it's a big speculative.  I filed https://github.com/flutter/flutter/issues/475 about tracking stats for ForceLayer.


atwho-inserted tag removed with backspace

I would like to style the at who-inserted tag but it gets removed when you press backspace.
The use case is when you insert multiple username and want to separate them with commas.
Example: @user1, @user2 and @user
When you select a value from the dropdown it gets inserted and there is a whitespace at the end. To add a comma you need to delete the whitespace with backspace. But doing that removes the atwho-inserted tag.
I can't use a comma as a suffix because it's only there sometimes.
It doesn't work with Chrome, Safari, IE11.
It works with Firefox.

It's confused. But the afterMatchFailed callback might help you, here was an example: new hashtags
That demo show you how to create a new tags which not in the list.

OK, thanks. I'll check it out.

close for now.

It works, except when the name has "." in it. Here's an example http://jsbin.com/nuhiza/2/ , type "#jac" to insert "Jac.ob" . Then press backspace twice to add a comma.

The matcher did not match . , but it will match now for it seems a common case.
But if there is some others char that matcher wouldn't match, this problem will appear again; and then you have to override the matcher to match that.
Then you can do this hack to solve your problem:
        afterMatchFailed: function(at, el) {
          if (at == '#') {
            text = el.text().trim();
            // dirty hack:
            $("#editable").atwho({at: at, suffix: ","});
            this.insert("<b>" + text.substr(0,text.length-1) + "</b>");
            $("#editable").atwho({at: at, suffix: null});
            return false;
          }
        }

I replaced the suffix of a space with a zero width space and that seemed to help a lot. Opinions?
at: '@',
suffix: '\u200B',
spaceSelectsMatch: true,


owasp-protection-global-unsafe rule is failing to detect previously recognised cases

I'm not sure what's going on here but in the example appspec, where security has been removed from the DELETE /pets/{petId} endpoint, the owasp-protection-global-unsafe doesn't pick it up any more:
openapi: 3.0.1
info:
  title: Security disabled for specific endpoint
  version: "0.1"
servers:
  - url: https://example.com
paths:
  /pets:
    post:
      summary: "create a pet"
      responses:
        "200":
          description: some descriptive text
  /pets/{petId}:
    parameters:
      - name: petId
        in: path
        required: true
        schema:
          type: integer
    get:
      summary: "get a pet"
      responses:
        "200":
          description: some descriptive text
    delete:
      summary: "delete a pet"
      responses:
        "200":
          description: some descriptive text
      security: []
components:
  securitySchemes:
    jwt:
      type: http
      scheme: bearer
      bearerFormat: JWT
security:
  - jwt: []

Relevant part of the OpenAPI spec, under the Operation Object see the description for the security field:

To remove a top-level security declaration, an empty array can be used.

This was detected in earlier versions of vacuum.
I'm guessing there's something funky in the logic here:
https://github.com/daveshanley/vacuum/blob/0e30ba4eacac7b5694e3c65cc23f79a5a5394b7a/functions/owasp/check_security.go#L104-L105
When I've played around with it it starts erroneously reporting that security is also missing from the POST /pets endpoint too though.

I think it boils down to the zero value of opValue.Security being [] in both the cases where no security property is present and when a security: [] is present.
https://github.com/pb33f/doctor/blob/3644e80c85901727e8082eea4aa5e4dac2acf25b/model/high/v3/operation.go#L20C15-L20C44

Bingo, if security is [] then the for loop here iterates zero times, so o.Security is never assigned to and remains nil:
        if operation.Security != nil {
                for i, security := range operation.Security {
                        security := security
                        s := &drBase.SecurityRequirement{}
                        s.Parent = o
                        s.IsIndexed = true
                        s.Index = i
                        wg.Go(func() { s.Walk(ctx, security) })
                        o.Security = append(o.Security, s)
                }
        }

Quick fix:
        if operation.Security != nil {
		o.Security = []*drBase.SecurityRequirement{}
                for i, security := range operation.Security {
                        security := security
                        s := &drBase.SecurityRequirement{}
                        s.Parent = o
                        s.IsIndexed = true
                        s.Index = i
                        wg.Go(func() { s.Walk(ctx, security) })
                        o.Security = append(o.Security, s)
                }
        }

Would you mind submitting a PR? I would be most grateful.

I re-wrote all those rules pretty quickly, I am sure there are a few other gaps.

No worries, I'm on a lil holibob until Monday so it'll be a few days.
Otherwise if anyone else wants some easy pickings in the meantime feel
free, I'd appreciate it!
On Thu, 25 Jan 2024, 14:59 quobix, @.***> wrote:

I re-wrote all those rules pretty quickly, I am sure there are a few other
gaps.
—
Reply to this email directly, view it on GitHub
https://github.com/daveshanley/vacuum/issues/431#issuecomment-1910381384,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE3FYHLTDMIBHOITPYZTFCLYQJXN3AVCNFSM6AAAAABCKKVTHKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJQGM4DCMZYGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

@daveshanley see PRs above


json: enable JSON schema versioning

In a future commit I'm going to introduce a new JSON schema ("version 2"), so set the ground-work for that here.  There are capabilities that are unable to be expressed in the current format, such as prioritization of two groups both have Blink settings but at different rates, but we want to retain backwards compatibility to any existing configs.
Change-Id: Ib4cb5761de1f80567a775cb15e8da921f539037e

Can one of the admins verify this patch?

add to approvelist


Add Number of Host Target connections to Get-Pfa2Host cmdlet

It would be helpful during certain maintenance windows for us to get the number of target connections of all the hosts that the Pure array reports. The Get-Pfa2Host cmdlet already returns a PortConnectivity but only tells you the overall status such as 'redundant', 'unused', etc:
(Get-Pfa2Host -Array $pure112 -Name PASQL444B).PortConnectivity

Details     Status
-------     ------
Unused Port unhealthy

Could another item be added for an actual number count of target connections for the host? You can see the count in the purity UI under Health -> Connections, but it would be very helpful to get with this cmdlet

@lukeroberts great suggestion. I am currently planning the next version of the SDK and will put this on the list for investigation.


problem with get_mut_type(data, type = "dbs", predefined_dbs_mbs = FALSE)

i am running the function get_mut_type(data, type = "dbs", predefined_dbs_mbs = FALSE) on a GenomicRanges object which has the form

But i am getting apparently wrong dbs context

i am not understanding if it is a problem of the input object or a mistake of the function.
Thank you very much for the help.

@FreekManders

It seems you're getting both DBS and MBS in your results. That shouldn't happen. Does this also happen when you run the examples in the get_mut_type documentation?

Hi,
yes, if i run it as in the example it works fine. But my gRanges object seems to be in the same form (the one that i attached), so i was wondering if there is some error there that i did't notice

Is it possible that your GRanges object already contains some predefined MBS or DBS? You could check this by setting predefined_dbs_mbs to TRUE.

i tried and it worked fine. probably that was problem, thank you!


Additional API for weewx weather station.

This is a follow up from #70 of adding weewx support to this plugin.  I'm not really sure where to go but need some help finishing this up.  I forked this repo edited one of the apis (weather underground) to read the json that weewx is capable to produce.  The edited api is here, the changes I made are that the apikey is utilized for the sitename/path and location for the file name so if your website is called weewx the would be the apikey and the location would be the file name (without extension) so for example this should call http://weewx/homebridge.json and scan the json file.  I also included a sample of the json file that i formatted that matches homebridge and included a few more fields that weewx can use that match up to the plugin.  The JSON sample response is here.  If I need to reformat the response that's not difficult just not sure how to get/test the edited api and ideally get it added.  Thanks!

Could you release 3.3.0 so that we can benefit from the newly added weewx support?

Fixed in v3.2.7


Exclude metrics being sent to a sink

Hello,
This is more a feature request than it is an issue.
I was wondering if it would be easy to implement a feature that would allow the sink to be configured which metrics should be sent to the sink.
We have an actual use case for this where the global cost of sending metrics to our sink is quite high, and certain metrics which are costing us money are not actively used.
Therefore, it would be nice if this can be configured on a sink level.
If someone can point me in the right direction, we could also have a try at implementing this ourselves.
Thanks in advance.

I believe certain sinks have that option currently, but there's no global option.  I'd be willing to accept a patch for that, however.  What do you need to get started?

My use case involves the Librato sink specifically.
I was going through the code and one option would be to create a configurable option with an exclusion list of metrics which is taken into account in the following 2 files:
https://github.com/kubernetes/heapster/blob/master/metrics/sinks/librato/librato.go
https://github.com/kubernetes/heapster/blob/master/common/librato/librato.go
Would that be the correct way ? Or can you show me which sink is currently using this ?

It looks like there's already something similar in the --ignore_label flag.  It's not quite what you want, but it's similar.  That's probably a good starting point.


Misc. Text Changes in GB

A list of miscellaneous text changes within the GB is available in the following doc:
https://docs.google.com/document/d/10XCIjcDQZXAPxavjq7mDXmph8WZupCgfa2P2jVw6Vc8/edit?usp=sharing

Hi @payten ,
These look great. Sorry, one more quick text change:
For the Extra Credit cell popover message, can we change the text read as follows:
Score is over item's maximum point value and is providing extra credit.
Thanks!

Looks good!


Expose record_accessor internal instance @keys variable. Fix #1806

Currently, record_accessor does not expose internal @keys instance.
We want to use this variable in fluent-plugin-anonymizer.

Thx!


Add operator colouring

Added some colour for operators.
While this may just be a personal preference, typically I've found it's easy to "lose" operators in a sea of white, unmarked variable names, since they're rarely near objects that have some special meaning in a language. In operator heavy languages like C, I find it irritating when I have to stop my natural reading flow to mentally parse out the individual symbols.
As I've said, this is just my personal opinion. If you don't think it's an issue, you can obviously feel free to close the pull request.
Also, there wasn't really any neutral colours I could use for operators, so I made it "brown", which is really just a darker and less saturated orange. If you think there'd be a better colour, please suggest one.
An image:

@Lixquid Sorry, for not merging earlier.. I was a bit hesitant adding another color and changing operators for all languages.
Maybe it could be a neutral @gray (a bit darker than the normal text) for all languages. And like in this example https://github.com/atom/one-dark-syntax/blob/271653fd2c367831f0eda9200a5110df77a484c8/styles/languages/cs.less for some specific languages more colorful.

Sorry about not replying for a while, I've been pretty busy.
I can understand the hesitation about not adding another colour; if I remember correctly, wasn't Base16 the underlying palette?
Anyway, I've got some comparison screenshots for grey and brown operator colouring:

In my personal opinion, I still prefer the brown, as I don't think the light grey really "pops" enough to be easily identifiable. (I couldn't make the grey any darker without it being the same colour as a comment.)

1+
Wanted to created a similar PR...

Gonna close this PR again. For now, styling operators should be done on a per language basis. Some languages like C, JavaScript, Python already got support.
@Lixquid But thanks for raising the issue.

Alright. While I don't agree with your opinion, I respect your decision.
For anyone else that want similar functionality, you can add this snippet to your styles.less:
atom-text-editor::shadow .keyword.operator {
	color: #a86;
}

@Lixquid How do you like using @cyan for all operators, see https://github.com/atom/one-dark-syntax/pull/61#issuecomment-179586989? That would be more subtle.

From an objective standpoint, I think it's stepping a little close to "library calls", which already get assigned a teal color.
I couldn't really find a colour that wouldn't fight with existing colours at the typical colour intensity, plus I thought adding some intensity contrast helped pick out the arguments, which is hopefully the intended goal. For comparison, I attempted to approximate the cyan in the screenshot you posted:

I'm not sure if it's only me that feels this way, but with operators looking similar to library calls, it feels as if the operators are getting "lost" in the colours. Again, this is only a personal opinion, and if you or anyone else has a differing opinion about how visually noisy it is, well, that's fine too.
From a personal standpoint, I've been using brown operator colouring for roughly a year, so it's very likely I'll unfortunately probably be biased forevermore.


Added implementation of IPCP prefetcher

This PR adds the IPCP prefetcher described in this  paper.
NOTE: This contribution is made by @AbhishekUoR and @xmlizhao
On branch ipcp
Changes to be committed:
modified:   src/mem/cache/prefetch/Prefetcher.py
modified:   src/mem/cache/prefetch/SConscript
new file:   src/mem/cache/prefetch/ipcp.cc
new file:   src/mem/cache/prefetch/ipcp.hh
Change-Id: Ie52a9602719773248aa17e9bc3d98203d9b1e23b

@ivanaamit: Is there a problem with this PR? How long should we expect it to be reviewed?

Hi @Setu-Gupta, I noticed you pushed some changes last week. Is this PR ready for review, or are you still working on it? Thanks.

Hi @ivanaamit
There are some aesthetic changes related to code formatting which are still remaining. I'll probabyly push them this week and let you know when the PR is ready for review. Right now I'm working on PR #1441

@Setu-Gupta Hello, I've noticed that you've pushed a few commits after your last message. Could you give an update on what the status of this PR is? Thanks!

Hi Erin
Most of the changes that I did were related just to code formatting. Unfortunately I haven't been able to find the time to test the code after the changes and so I'm not fully confident that the code behaves exactly like it used to before the changes. Once I verify this, then I'll open this PR for reviews.


Extension issue

Issue Type: Bug
Extension Name: LiveServer
Extension Version: 5.6.1
OS Version: Windows_NT x64 10.0.19041
VSCode version: 1.50.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:

you did'nt pasted the data

second time i posted

second time i posted

sorry i closed it by mistake
You can contact me by email -<EMAIL_ADDRESS>


Direct link to identifiers.org resolution

In the annotation table, it would be nice to be directly able to open a link to the given annotation if it's a valid identifiers.org key:value pair.

Fixed in #410


Add endpoint for the models associated to a user

Endpoint that returns a list of all the models a user has read access to.

Addressed in #434


add banner to readme

What did you fix? (provide a description or issue closes statement)
added banner to readme.md
Considerations

Does this contain a new dependency? [No]
Does this introduce opinionated data formatting or manual data entry? [No]
Does this pr include updated data files in a separate commit that can be reverted for a clean code-only pr? [No]
Have I run the linter? [No]
Is is a bug fix, feature request, or enhancement? [Enhancement]

you'll need to submit a PR to WFCD/banner first

also, typically just do either badge or banner, not both :D

The usage guide on the banner repo is a little misleading there:

To use the banner, please submit a pull request to this repository adding an entry to PROJECTS.md, providing proof that you are using the below snippet in your readme appropriately.

I can't proof that I am using the banner in my readme without putting it in there first 😅😅😅

Ok, sure, either way... You only need one or the other, but both banner and badge


SWS-101 Add Coveralls

Depends on #27

If we want coveralls we would need to signup and enable for this repository

Thanks @josejulio!

did we agree adding Coveralls? what JIRA card is this?

@josejulio

If we want coveralls we would need to signup and enable for this repository

Isn't coveralls which generated the covering table at the end of the build logs?
If not, maybe it is sufficient?

did we agree adding Coveralls? what JIRA card is this?
SWS-101

Isn't coveralls which generated the covering table at the end of the build logs?
If not, maybe it is sufficient?

jest did generate the covering table (command npm test -- --coverage), but if we want our "badge", someone with enough perimssions needs to signup to http://coveralls.io/ and enable for this repo.

@josejulio looks like this needs rebasing

@josejulio looks like this needs rebasing

Rebased

Looks like we still need someone with privs to enable for this repo and add badges
@pilhuhn is that you?

Closing/Opening PR to run Travis again

Travis error seems to be unrelated

@pilhuhn enabled this repo on coveralls (thanks!)
The badge can be seen here:
https://github.com/josejulio/swsui/blob/957b39099a0d275f1f924f6f14c59ab986dd8c3d/README.adoc
and our first coverage report is here:
https://coveralls.io/github/swift-sunshine/swsui?branch=master
Currently coverage is "Unknown" because is the coverage for "master", so once we merge it should show something


Tweak UI: Change some parts of UI for mention-unsubscribed warning.

This PR is intended to solve left-out parts of issue #4224.
This is how it looks
https://user-images.githubusercontent.com/53913514/114275882-ddd1d480-9a41-11eb-8014-39e7b531a3c9.mov

@chrisbobbe This PR is ready for a re-review. Kindly have a look 🙂


Fix handling of empty resources

Conform to XDS spec in
https://www.envoyproxy.io/docs/envoy/latest/api-docs/xds_protocol#how-the-client-specifies-what-resources-to-return.
This has not been found as a bug in normal Envoy interaction, but it
breaks istioctl wait and istioctl ps today. What happens is Envoy
will send an RDS request with empty resources. This can happen at
startup or at any point in time (removing all VS for gateway, for
example). When this happens, today, we just keep sending it an empty
list of resources, which it will accept but never ACK.
This changes to actually drop the subscription, so we never even send
these empty resources.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
2c6eab305bf9cea7af5f9fad84e3c8a258699fe2
link
/test lint_istio

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

LGTM. Minor comments. Also can you please add a test case in this https://github.com/istio/istio/blob/0d52f644f484c7a8ce433f3867b88f8160f4a4ab/pilot/pkg/xds/discovery_test.go#L327 ?

Will this cause panic when no subscribed resources, the w passed into pushXds can be nil

As far as I can tell, no.
In a request, if we delete it then we will not call pushXds. For a push ,we only look at things in WatchedResources, which would never contain nil, so its also fine.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
573b5ae9dd5c7093e02b442cecd93b89932767d3
link
/test integ-distroless-k8s-tests_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
573b5ae9dd5c7093e02b442cecd93b89932767d3
link
/test integ-distroless-k8s-tests_istio

integ-telemetry-k8s-tests_istio
573b5ae9dd5c7093e02b442cecd93b89932767d3
link
/test integ-telemetry-k8s-tests_istio

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

/retest

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio
573b5ae9dd5c7093e02b442cecd93b89932767d3
link
/test integ-telemetry-k8s-tests_istio

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

/retest

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio
573b5ae9dd5c7093e02b442cecd93b89932767d3
link
/test integ-telemetry-k8s-tests_istio

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

/test integ-telemetry-k8s-tests_istio

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio
573b5ae9dd5c7093e02b442cecd93b89932767d3
link
/test integ-telemetry-k8s-tests_istio

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

/test integ-telemetry-k8s-tests_istio


Questions on Scenario 3

Could you explain how to simulate an outage on primary server (for Scenario#3), so that:

It can trigger logic app and delete failover group and create another failover group between the remaining two regions with the same name.
What kind of a trigger would bring back the failover connection to primary, when primary is online again?

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

ID: 9d8edf47-e8df-673a-d43b-78e91196221d
Version Independent ID: dba47e18-8949-32e0-8425-18d45250fc9d
Content: Design globally available services - Azure SQL Database
Content Source: articles/sql-database/sql-database-designing-cloud-solutions-for-disaster-recovery.md
Service: sql-database
Sub-service: development
GitHub Login: @anosov1960
Microsoft Alias: sashan

@chennuchaitanya Thank you for the feedback.
This is being assigned to the content author to evaluate and update as appropriate.

#reassign: @MashaMSFT
pls work with @Sait-MSFT to suggest the trigger options.

#reassign: @Sait-MSFT

#reassign: BustosMSFT

Hi @chennuchaitanya  to perform a disaster recovery drill with failover groups, you can disable the web application or virtual machine connected to the database or failover the database to cause application connectivity failures. Here you'll find more details.
2) When the end-point monitoring has the sever back as online you can trigger the process to switch the primary server in the failover group to the original region and the following steps described in the Scenario 3.

#please-close