Small typo

On the step 6:
<Button x:Name="translateButon" Text="Translate" Clicked="OnTranslate" />

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

ID: 2de3e72c-8948-4dd5-e37f-9cf81e62c127
Version Independent ID: 7b7c5c7f-4102-8e91-fbe6-7d21fcc8f5be
Content: Xamarin.Forms Quickstart - Xamarin
Content Source: docs/xamarin-forms/get-started/hello-xamarin-forms/quickstart.md
Product: xamarin
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @MihailRomanov
Thanks for pointing this out. It's fixed in a PR that will be published within a few days.


Batch Update with Navigation Property does not work

Using a navigation property does not appear to work:
var foos = context.Foos.Where(f => f.DeletedAt == null);
foos.Update(f => new Foo { Sum = f.Bars.Sum(b => b.Value) });

Produces:
System.Data.SqlClient.SqlException : Incorrect syntax near 'Id'.
   at System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   at System.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at System.Data.SqlClient.SqlCommand.RunExecuteNonQueryTds(String methodName, Boolean async, Int32 timeout, Boolean asyncWrite)
   at System.Data.SqlClient.SqlCommand.InternalExecuteNonQuery(TaskCompletionSource`1 completion, Boolean sendToPipe, Int32 timeout, Boolean asyncWrite, String methodName)
   at System.Data.SqlClient.SqlCommand.ExecuteNonQuery()
   at Z.EntityFramework.Plus.BatchUpdate.Execute[T](IQueryable`1 query, Expression`1 updateFactory)
...

Using EF Core 2.1, MSSQL.

Hello @psyren89 ,
Thank you for reporting, we successfully reproduced the issue.
We will look at if that's possible to fix it for EF Core.
Best Regards,
Jonathan

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations •  Dapper Plus •  LinqToSql Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

I also have this issue. What's the status?

Hello @megumus ,
I don't think anything will be done until we completely re-write this feature.
If I remember well, that issue was caused by how EF Core generate the query. Since we don't control how the query is generated, that make it very hard on our side to support all this kind of special scenario.


Add option : use let and const instead of var

Hello.
The idea is to add an option for using const and let statements instead of var like this
Before
function timesTwo(arr) {
    var _a = arr;
    var _f = n => n * 2;
    var _r = [];

    for (var _i = 0; _i < _a.length; _i++)
        _r.push(_f(_a[_i], _i, _a));

    return _r;
}

After
function timesTwo(arr) {
    const _a = arr;
    const _f = n => n * 2;
    const _r = [];

    for (let _i = 0, len = _a.length; _i < len; _i++)
        _r.push(_f(_a[_i], _i, _a));

    return _r;
}

I don't think this is really necessary for two reasons:

let: we don't need to replace with let because babel will automatically generate a unique identifier
const: because this is a unique identifier, it will only be used by the plugin and mutated as it is supposed to.

Also it doesn't serve any documenting purposes/etc. because the generated code is not even supposed to be read anyway.


How to reproduce NOT_FOUND error with context.mirageSchema.findBy()

Hi,
I have a custom resolver (like https://github.com/miragejs/graphql?tab=readme-ov-file#example-deleting-a-person). For a minimal example, consider that the database table person is empty and the result of
const person = context.mirageSchema.db.people.find(1)

is person be null. How do I create a NOT_FOUND error? Currently, Mirage JS GraphQL will return null but no error.
I would appreciate any help to reproduce the follow query to GitHub GraphQL API

You can return or throw an error from a resolver. I don’t know exactly how GitHub is adding the "type" field to their errors but you can do something similar by using GraphQL’s built-in error class to create an error with some extra information in the response.
For example:
// From your resolver...

return new GraphQLError("Person not found", {
  extensions: {
    type: "NOT_FOUND"
  }
});

Your response will look something like this:
{
  "errors": [
    {
      "message": "Person not found",
      "locations": [
        {
          "line": 1,
          "column": 21
        }
      ],
      "path": [
        "person"
      ],
      "extensions": {
        "type": "NOT_FOUND"
      }
    }
  ],
  "data": {
    "person": null
  }
}

Thanks for the answer. Next week, I will submit a pull request to the README.


Add the ability to get source dimensions

Add the ability to query what the dimensions of a named source is. This will help troubleshoot scale issues for cams from chat.

Added in v1.7.0 and broken sometime before v1.7.2 via Bug #73


[24.1] JFR integration tests fail for Mandrel for JDK 23 - JDK events initialization

Latest Mandrel integration tests fail the JFR integration test with this warning:
[warn][jfr,system] Could not initialize JDK events. jfrTracing

... which usually indicates a larger problem, so we should investigate.
Example test output:
Error:    JFRTest.jfrOptionsSmokeTest:784->jfrOptionsSmoke:922 build-and-run.log log should not contain error or warning lines that are not whitelisted. See /home/runner/work/mandrel/mandrel/mandrel-integration-tests/testsuite/target/archived-logs/org.graalvm.tests.integration.JFRTest/jfrOptionsSmokeTest/build-and-run.log and check these offending lines:
[warn][jfr,system] Could not initialize JDK events. jfrTracing ==> expected: <true> but was: <false>
Error:    JFRTest.jfrPerfTest:185->jfrPerfTestRun:234 build-and-run.log log should not contain error or warning lines that are not whitelisted. See /home/runner/work/mandrel/mandrel/mandrel-integration-tests/testsuite/target/archived-logs/org.graalvm.tests.integration.JFRTest/jfrPerfTest/build-and-run.log and check these offending lines:
[warn][jfr,system] Could not initialize JDK events. jfrTracing

See this CI link:
https://github.com/graalvm/mandrel/actions/runs/9720067442/job/26831418361#step:11:20582

@roberttoyonaga Could you please take a look at this? Thanks!

Seems related to https://bugs.openjdk.org/browse/JDK-8331876 new in JDK 23+26.

It doesn't seem to be happening for a Mandrel build with JDK 24+3. Looks like we'd need a backport of https://github.com/oracle/graal/pull/9212

Steps to reproduce:
export JAVA_HOME=/path/to/mandrel-build-based-on-jdk23
export GRAALVM_HOME=$JAVA_HOME
export PATH=$JAVA_HOME/bin:$PATH
export FAIL_ON_PERF_REGRESSION=false
mvn clean verify -Dquarkus.version=999-SNAPSHOT -Ptestsuite -Dtest=JFRTest#jfrPerfTest

@roberttoyonaga Could you please take a look at this? Thanks!

Yup I can look at this!

No longer an issue. Fixed with  graalvm/mandrel#762.


mark all vendor intrinsics as unsafe

I kind of lost track of the conclusions of the target_feature RFC, but I think any function labeled with #[target_feature] needs to be marked as unsafe. Currently, that's not the case.

I have the conn on this.

Done!


feat: added port and network expose and removed react-scripts

This PR will add feature which will allow user to pick custom port, hostname, add featured to remove ../.../.../ like import.
closes #1230
👷🏻 Changes made

package.json
vite.config.js
frongend.md
.env.example

New changes have been applied


Hugo generates "+50" timezone in sitemap when the sitemap template specifies +10 timezone

What version of Hugo are you using (hugo version)?

$ hugo version
hugo v0.126.0-32c967551be308fbd14e5f0dfba0ff50a60e7f5e+extended linux/amd64 BuildDate=2024-05-14T13:24:11Z VendorInfo=gohugoio

Does this issue reproduce with the latest release?
Yes.
Details
To reproduce:

Download the test site: test.tar.gz. The test site contains layouts/_defaults/sitemap.xml, which is same as the embedded sitemap template, with -07:00 replaced with +10:00.
Run hugo inide the unarchived directory.
public/sitemap.xml contains this line: <lastmod>2024-05-14T19:08:54+50:00</lastmod>, a +50 timezone.

<lastmod>{{ safeHTML ( .Lastmod.Format "2006-01-02T15:04:05+10:00" ) }}</lastmod>

The above is nonsensical. If you want to include an offset in a layout string it must be 07:00.
Again, this is not a bug, but a conceptual disconnect. Please use the forum (https://discourse.gohugo.io/) for questions and troubleshooting.


Test query counts for more stream-create scenarios.

If you create a public stream in a realm with a notifications stream, then we have tests like test_multi_user_subscription that check the query counts to make sure they're not too expensive.
We should have similar tests for when...

there is no notifications stream (this is important, because we used to send PMs instead, which was super expensive)
the stream is private
the user passed in announce=False

@zulipbot claim

Yes, I am working on it !

Yup!

Yup! Will likely have something soon.

@zulipbot claim

@zulipbot claim

Hi @showell, I sent a pull request for this issue, would you mind having a look at it?


Standard way for evaluation multistep forecasting

Thank you again for the amazing work.
I would like please ask about the evaluation method:
if I have a test data set size of 65 observations,
suppose that the prediction length is 20.
So, each observation of the test dataset will generate 20 predictions = array(65,20).
Now, to calculate the evaluation metrics MAE between(predictions, trues). what is the standard way to do so in timeseries forecasting? I mean, should I: Take the first 20 predicted values and compare them with the first 20 true values. Calculate the absolute difference between each predicted value and its corresponding true value. or what? Can you please give me hints or some codes? This is what I have done:
preds = np.array(np.reshape(predictions, (65, 20))) trues = np.array(np.reshape(values, (65, 20))) preds= mm.inverse_transform(preds) trues= mm.inverse_transform(trues) preds= preds[0:,1] trues= trues[0:,1] metrics = metric(preds, trues)

Yes, I think your code is right.


Score counter

Describe your problem here.
Is there any way to get the score counter that is seen in the base game?

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

Custom Info Bar v5.1.zip
Hope this helps

I edited that one to only show the score combo and misses, here's the original one https://gamebanana.com/tools/9217

Yeah that works fine! is there any way to just change it to the score count without the misses and combo? :)

Sorry for responding so late, I'll edit it to that if I can.

Like this

?

Or do you want the 0 to be closer to the :

Or do you want the 0 to be closer to the :

Just a little closer would be fine :)

So like this
that is perfect!

yeah that is perfect!

Custom Info Bar v5.1.zip

woah, how did you even edit the script to say that?

To say what?

No problem :)


Creating an API doesn't naturally output the URL

If you follow the AWS API tutorial , in step 2 it instructs you to run npx sst dev which in turn should give you the APIGW invoke URL for the API.

It doesn't do this - here is what I got in my attempt.

I posted this issue within the Discord, and for anyone else who runs into this issue, and I was told you can return the URL in the run() block with something like (not tested this yet)
async run() {
  const api = new sst.aws.ApiGatewayV2("MyApi");
  api.route("GET /", {
    handler: "index.upload",
  });
  api.route("GET /latest", {
    handler: "index.latest",
  });
  return {
    url: api.url
  }
}

I just return this
Here is how I deploy mine ApiGatewayV2 and I get the URL with the latest

export async function setupApiGateway(myVar) {
  const api = new sst.aws.ApiGatewayV2("mydomain-com-API", {
    domain: { name: "api.mydomain.com", dns: sst.aws.dns() },
    cors: {
      allowMethods: ["POST", "OPTIONS"],
      allowHeaders: ["Authorization", "Content-Type"],
      allowOrigins: ["https://mydomain.com"],
    },
   });

  // Define API routes
<multiple routes redacted>

  api.route("OPTIONS /", {
    handler: "src/functions/options.handler",
  });

  return api;
}

Hmm this is a bug. It used to print this out.


chore: cleanup packages/open-scd and packages/plugins after split

I came up with a list of work items put into this massive PR, I hope it helps:

Added named exports for packages/open-scd
Extracted scl related foundation files and functions to packages/core
Updated imports on packages/plugins and packages/open-scd
Downgraded snowpack to allow esm transpired imports into require statements
Included root script clean:modules to remove package-lock.json and all node_modules folders
Updated @typescript-eslint/parser and @typescript-eslint/eslint-plugin to version 7.8.0 in all packages
Updated tsconfig in both packages/open-scd and packages/plugins and included new tsconfig.build.json files to customise exports and still allow typescript to run in all the code (src and test files)
Updated web-test-runner config in both packages/open-scd
Removed deprecated Wizarding and Editing mixins which were being used in tests at packages/open-scd and packages/plugins
Fixed mock-editor and mock-wizard-editor to use Editor and Wizards addons instead

The changes in the PR seem to be necessary, but it is just too bit. I'd prefer to split it up and solve the conflicts in smaller PRs.

The changes are not worth it anymore because of the latest refactorings.


  Also note that you cannot dig any deeper than what the picture shows.

The next picture shows the landscape with HFM,RVT, and rvt cache removed/disabled.  I was then able to dig a deep hole as expected.

From what I can tell I believe the problem is with the rvt bounding boxes.

Hopefully this will give you a little more info to help debug the problem.
https://1drv.ms/v/s!Aq4OgSZRh__uifxxxVip4d5NHKIXkQ?e=a5RF8s

More debug info
https://1drv.ms/u/s!Aq4OgSZRh__uifxxxVip4d5NHKIXkQ?e=hFghG7

Ah! got it.
Will try to fix this.

Video of problem.
https://www.youtube.com/watch?v=e19lXzqWt6k


Why FAQ plot average code not working?

Seems to be working fine to me. It includes average and individual ones too.


assert database could not be found?

I think your image meta data file (.meta) not upload
so sprite missing everywhere.
the unity report warning : "Rebuilding Library because the asset database could not be found!"
My unity version : unity2018.03

yes you need to set unity to non hidden meta data or else this is pretty much useless as you have nothing showing how to use this


Create liceovolta.txt

LICEO SCIENTIFICO STATALE "A. VOLTA"

@gianlucapetito Please provide us with:

the school official website URL,
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the school recognizes the domain which you are submitting as an official email domain for the students.

Hi,
in real the school have 3 official website urls: liceovolta.it,
liceovolta.edu.it and voltaweb.it
to students and teachers is assigned voltaweb.it because on that domain is
active g-suite for educational (maybe I should add voltaweb.it to the swot)
in the screenshot of our website, in the contacts, you can see that the
email address of the boss (dirigente scolastico) is voltaweb.it
(https://www.liceovolta.it/nuovo/la-scuola/contatti)
[image: Schermata 2020-03-18 alle 21.28.06.png]
Il giorno mer 18 mar 2020 alle ore 19:04 Philip Torchinsky <
<EMAIL_ADDRESS>ha scritto:

@gianlucapetito https://github.com/gianlucapetito Please provide us
with:

the school official website URL,
an URL of a page or another proof (.pdf or a screenshot are OK)
showing that the school recognizes the domain which you are submitting as
an official email domain for the students.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/7948#issuecomment-600780681, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPJLMATFJNQFTOTDJTVCPTRIEEKBANCNFSM4LOPG2JA
.

@gianlucapetito I am sorry, I a bit confused now. Which email domain you need to add now?
Shall I add all of the domains, or just one of them?
You said that all of the domains are official for the school:
liceovolta.it
liceovolta.edu.it
voltaweb.it
Do you wish to add all of them?
Thank you!

yeah, you are right, I’m sorry.
What do you need to add, to swot, to let students and teachers get free
licenses of software?
If you need emails domain, we can just add voltaweb.it
Il giorno lun 23 mar 2020 alle ore 13:03 Philip Torchinsky <
<EMAIL_ADDRESS>ha scritto:

@gianlucapetito https://github.com/gianlucapetito I am sorry, I a bit
confused now. Which email domain you need to add now?
Shall I add all of the domains, or just one of them?
You said that all of the domains are official for the school:
liceovolta.it
liceovolta.edu.it
voltaweb.it
Do you wish to add all of them?
Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/7948#issuecomment-602550293, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPJLMBYUF5AYSQPB2OKSY3RI5FXRANCNFSM4LOPG2JA
.

@gianlucapetito I am sorry for a delay with processing your request.
I already added voltaweb.it on your behalf, and close this request without merging.

thank you very much!
Il giorno lun 30 mar 2020 alle ore 18:43 Philip Torchinsky <
<EMAIL_ADDRESS>ha scritto:

@gianlucapetito https://github.com/gianlucapetito I am sorry for a
delay with processing your request.
I already added voltaweb.it on your behalf, and close this request
without merging.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/7948#issuecomment-606111396, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPJLMDU3B3NJAR2GTTSPATRKDD3FANCNFSM4LOPG2JA
.


SpriteLoader Incorrect UVs

SpriteLoader returns tex_3 when tex_1 is inputted.

Turns out they were correct all along.


Fix: python ident error

removed blank space that was causing a python ident exception.

This is causing a huge problem and needs to be approved

this is good to go, verified

done


USHIFT-94: openshift-apiserver: remove dead configuration code

The mocked command line parsing code is not needed to configure the
openshift-apiserver, since we can construct an options struct
directly.

/retest

Successfully tested in my dev env.
/lgtm


Extend support for specifying quotas with u64

I'm looking to run the governor with a rate limit of >~4GB/s ie 10GB/s which cannot be specified with the u32 values passed into Quota. I understand this might be slightly over what most people expect but we all know the story of "16 bits ought to be enough for everyone".
Looking at the code, this might not be as straightforward of a change as the title would imply.

Huh! That's a new one!
I think we can change the quota type and others to accommodate that number, but this is getting really really close to the 1-nanosecond granularity that our time duration types dictate. Are you accounting for single bytes?

We are, though we might just shim our own end to do some unit upsizing. However a large chunk of the traffic we want to be able to track are <1KiB frames at very high rates.

That makes sense! Hm, ok, doing the math with frink a bit:
> 1 second -> nanoseconds
1000000000
> 2^32
4294967296

I thiiink u32 is actually the highest we can go, since intervals can't go smaller than 1ns, in any of the clock abstractions that I have found so far... and GCRA requires non-zero intervals to keep track of that theoretical arrival time.
The more scaling-down of the measured values you can do (even by a factor of 10, better by 100 or even 1000), the more suitable the GCRA in governor will be for your use case.

Yeah makes sense, I'll try to do my own take on it at some point. It's not a super urgent thing, it's a very specialized case where we can work around it by using a slightly larger factor. Also not rate limiting is an option for some of these setups.
Just wanted to get it out there so that in some future we don't have to bill by the KB :D
Appreciate the quick responses!

Sounds great! I'm actually thinking about this now and ... I think there's a trade-off that could be made between higher granularity of timekeeping (not time measurement, mind! Clocks will still return values scaled to the next nanosecond) and duration over which you can keep time:

The main thing about a positive rate-limiting outcome is that it must move the TAT (expected arrival) timestamp back.
You can move that timestamp back by a fraction of a nanosecond.
The time measurement readings could be scaled to those fractions.

Now, that means that the "~200 years max lifetime" of a live ratelimiter object would get reduced. But then, since those live in memory only, I would still count a 20 years uptime of in-memory state a success (:
I think together with #89, a design is crystallizing in my mind around scaling these timekeeping granularities (such that larger and timekeeping types could be supported). What do you think?

Oh nice! Yes, down scaling the max lifetime would be a decent tradeoff and I guess a process restart every 20 years ain't all that bad 😄
Following that thought train, doing basically the same logic to go down in resolution to support 32bit architectures would be wonderful (we have a usecase for it, though limited usefulness).
Even a crude preset of 2-3 possible configurations would very much expand the domain governor can serve. I do foresee a bunch of feature flagging though.

Looking at how to implement this, I think const generics would be good to use for the Nanos abstraction (bubbling up to Clock and Reference traits). However, the limitations around generic type arguments make this difficult for now.
I'd say until we have at least something like https://github.com/rust-lang/rust-project-goals/pull/161/ (for trait inherent consts to be valid const generic arguments), this will be hard to implement and result in a very gross public API.


pull out filter_params and add archived/unarchived filter to reports

I've separated the two filter forms again, I think that's cleaner than doing a bunch of if @filters.options[:archived] or something like that


Fix: Limit sqlalchemy dependency to <1.4

This pull request addresses #885 until #919 is completed.
It ensures that sqlalchemy 1.4 is not pulled in as a dependency prior to adding support for it.
At present, the dependency on sqlalchemy >= 1.2 results in the latest release, 1.4, being added.
The module does not yet support sqlalchemy 1.4, resulting in new installations failing when using
local databases (#885).

fixes #885

Checklist:

[x] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[x] Run pre-commit hooks and fix any issues.
[x] Run pytest and tox, no tests failed.

We don't pin max dependencies in libraries. Projects should use a tool like pip-complie or poetry to control when they upgrade.


added a dependency so that dev_solr waits for dev_solr_initializer to…

… complete
What this PR does / why we need it:
modifies docker-compose so that dev_solr service waits for dev_solr_initializer to complete
Which issue(s) this PR closes:

Closes #179

Special notes for your reviewer:
Suggestions on how to test this:
Call ./run-env.sh  to start the dataverse container.
Go to dataverse (localhost:8000) and login as dataverseAdmin.
Try to create a dataverse.  It should create the dataverse successfully.
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
No
Is there a release notes update needed for this change?:
Additional documentation:

A number of automated checks are failing (chromatic, e2e, and component) but I'm not sure if that's expected or not. We should check with @GPortas and @MellyGray
I did get on this branch and try the run-env script. Dataverse spun up fine and I was able to create a collection with no Solr error. Moving to QA.
I'm going to slap a size of 10 on this for the combined hours spent.


Next meeting: February 6?

Can be at the Free Software Office (C/ Real de Cartuja) and then we can proceed to Meninas for beers.
With this, I think we can close issue #2

Looks good to me.  Though I may end up talking more about node, packaging and the browser if thats all right :smile_cat:?

Sure! Whatever rocks your boat. What do @m42 @fdavidcl @psicobyte @renatolrr and the rest think?

I don't know... Febrary 10 I have the most difficult exam...

Maybe after the exam period?

Every date has its own problems. I'd rather have it every month, with people coming and going, than postpone it until everyone is ready. @iblancasa , maybe you can prepare something for the first Friday in March.

@JJ For example...?

Intro to Fiware? (If it's still working) TweetMonuments? Socket.io? Whatever you want, really...

Socket.io it's a good topic :neckbeard:

I'm ok with the date :+1:

socket.io it is :-) For first Friday in March, March 6th again.

:+1:

Feb 6 is fine for me :smile:. Maybe @ncordon is also interested?

Sure! Feb 6 is also ok for me :+1:


Precise Location - GPC Translations

Task/Issue URL: https://app.asana.com/0/1157893581871903/1198047159635362
Description:
We've released several projects that only have strings in English. This PR adds translations for both of these projects:

Enable Precise Location in our Android app
[Converted to project] Create Do Not Sell (GPC)Spec

Internal references:
Software Engineering Expectations
Technical Design Template

@malmstein will this PR move all English translated copies from string-untranslated.xml to values/strings.xml?

@malmstein will this PR move all English translated copies from string-untranslated.xml to values/strings.xml?

Yeah that's the other thing, the strings from untranslated need to be deleted from there.


Improve readme and add migration guidelines

Coverage remained the same at 100.0% when pulling 02310c4ed6c6d2168f622436febd18fa0d025cbe on juanjoDiaz:imrpove_readme into 827d9f451700406c22e86cbb00ff60bc9897fca0 on zemirco:master.

Coverage remained the same at 100.0% when pulling 02310c4ed6c6d2168f622436febd18fa0d025cbe on juanjoDiaz:imrpove_readme into 827d9f451700406c22e86cbb00ff60bc9897fca0 on zemirco:master.


Feat/http encoding and host

record origin to log with option recordUrl is true.
adjust bufferTransformer arguments, like:

type bufferTransformer = (buffer, req?: IncomingMessage) => object | string;

Codecov Report

Merging #246 into master will decrease coverage by 0.11%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #246      +/-   ##
==========================================
- Coverage    80.7%   80.58%   -0.12%
==========================================
  Files         190      190
  Lines        6415     6417       +2
  Branches      870      871       +1
==========================================
- Hits         5177     5171       -6
- Misses        853      859       +6
- Partials      385      387       +2

Impacted Files
Coverage Δ

packages/hook/src/patch/HttpClient.ts
95% <ø> (ø)
:arrow_up:

...ges/hook/src/patch/shimmers/http-client/Shimmer.ts
80.32% <100%> (-1.64%)
:arrow_down:

packages/hook/src/patch/HttpServer.ts
84.7% <33.33%> (-2.05%)
:arrow_down:

packages/pandora/src/application/ScalableMaster.ts
79.68% <0%> (-3.13%)
:arrow_down:

Continue to review full report at Codecov.

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

+1


Vuu Table UI is showing raw values instead of display value

Feature Request
Description of Problem:
Server is providing UI the raw values in rows, however we would like to display it differently
i.e. if raw data is a FIX message, tag 35 value = 0, 1, 2 etc
but on UI we would like to see 0 = Heartbeat, 1 = Test Request, 2 = Resend Request etc
Potential Solutions:
When server is sending the columns, shall we have the map of possible values vs display values (per column), if this map is not defined, UI shall show the raw value

This could be a specific renderer we add or we could implement it as a special function in calculated columns stuff.
Something like =fix2english(fixField)
@heswell @ddimtirov  what do you reckon..?

It would be difficult to handle Fix value rendering out of the box. There are way too many FIX enums and the FIX standard is liberally extended by users with custom fields and values.
We could benefit from a few high quality core renderers that can be easily configured and specified on the server side, and understood by the client side.
For example, these 5 would go a long way:

numbers  (add thousand separators, fixed number of decimals, right aligned, conditional formatting for ranges - positive/negative)
strings (configurable alignment, trim to max size)
Error - used when the main renderer threw an error, or if we want to customize the view of null cells
Time and duration (accept all iso patterns and reformat as configured)
lookup - to handle cases like the fix enums. For each looked up value we should be able to configure icon and text format. The lookup renderer should also be able to handle booleans.

We should be able to configure a renderer for each column. For the cases where no column renderer is specified, we should be able to specify default renderers (at table level) for the basic types, to be used when no column renderer is found. These should have reasonable defaults.
The FIX enum use case would amount to a lookup renderer, that would be configured based on the fix session's dictionary. The lookup mappings should be sent from the server (which allows us to update them dynamically).
Calculated field would be simpler to implement in the current code, but will be more limited, put more load on the server and increase the required bandwidth.

I think we're talking about two kinds of translation here:

translation of numeric enum values to user readable labels eg 2 => 'Cancelled'
flexibility around formatting of numeric, date, string types etc

My take is that the  UI should handle most of he latter - default locale sensitive display of numbers dates, for example, but offering user opportunity to customise some of these display options via settings. We already have some of this capability. eg using the ColumnDescriptor on the client you can specify the number of decimals to display for a numeric value on a particular column, whether the numbers should align on the decimals, be zero padded etc. This is currently done only through static config, but UI to allow users to change config at runtime is a WIP.
(I do think we should expand the configuration options on the server beyond the existing types , but that is the subject for a different discussion)
The enum mapping was the main subject of this issue. I see two possibilities here. Either we handle all mapping entirely on the server and the client isn't even aware of the underlying enums or we send the lookup map to the client.
Having implemented a very rudimentary mapped lookup renderer on the UI for existing KeyTrader data, I'm inclined to think we should do all the translation on the server, unless we also see a perf advantage of only storing/trasmitting the raw numeric values ?
Bear in mind that if we do mapping on the UI, its not just the rendering of values in cells  that is affected. Filters, too, need to display the mapped values to users, but send the raw values back to the server in filter strings, so UI filtering is made more complex.
Serverside or UI, we also need to think about sort order. The user needs to see values sorted according to the string labels, so if we are storing raw values in the server, these MUST follow the same sort order as the human readable label.

Trying to put an example (for enum case) and see the pros and cons (The formatting of numeric, date, string types I think they can be quite standard / with user preference)
Server side mapping:
Data provider (provide both enum map + raw data) --> Vuu server (convert row data with the display value of that cell - and it should only be the visible viewport) --> Vuu UI (only display)
When sorting: recalculating from the column of display values
Pros: lookup function is only in Vuu Server
Cons: passing strings instead of ints to UI
UI side mapping:
Data provider (provide both enum map + raw data)  --> Vuu server (pass raw data - the visible ones & lookup function / enum map) --> Vuu UI (get display value from lookup function)
When sorting: recalculating from the column with raw data (but still need to use the lookup method to get display values for sorting)
Pros: passing raw data to UI (for enum case, only int instead of string)
Cons: both Vuu server and Vuu UI need to know the lookup function

@heswell you convinced me that the FIX enums use case is better implemented with calculated column.
Indeed, the complication of sorting and filtering are not worth it at the current level of code, and the bandwidth savings would not be all that great, considering we only transmit the viewport rows.


I have already submitted this and it has been invalidated but listen, please.

whenever i use the modpack version(the one that has the custom title screen andnot the standalone sculk horde that is supposed to be combined with other mods) i can get to the title screen, but all i can do is press singleplayer, upon which it will say "saving world" and promptly crash putting me back to the launcher. here is some evidence:

i am on the latest version of minecraft, curseforge, AND the modpack and nothing is added on as you can see here:

i am only able to play it succesfully with the standalone version aka this one:

as for the crash log here is the launcher's notification and the log itself:

crash-2024-09-04_13.50.41-client (1).txt
i submitted the log by itself, but they said there was no crash in it. I mean the title SAYS crash report. if you didnt get access to it for some reason heres a link to a google doc with it all copy and pasted directly from my notepad.
---- Minecraft Crash Report ----.pdf
just do not dismiss this again please.
Davros364
⇐{=====>

sorry i forgot to add this but i could not get a screenshot of the "saving world" screen as it crashed immediantly and it is too fast for me to screenshot it in time. i tried to capture a vidio of it happening, but im new to that kind of thing and couldnt do it myself.
just please take this seriously, k?

I will investigate this

thx so much

sassy label is hilarious

sassy label is hilarious

Im not sure if its serious or not though.


DO-NOT-MERGE: payload: Update the SSH demo payload

SSH demo payload has been released based on the CC-0.2.0 tag, let's update the image here.
Signed-off-by: Fabiano Fidêncio<EMAIL_ADDRESS>

/test
/test-tdx
/test-clh-tdx

/test
/test-tdx
/test-clh-tdx


[bug]: Install script doesn't work

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
8GB
What happened?
When I tried to run install.bat, I got this error. I also get 4 "This app can't run on your pc" errors.
edit: I also tried to run this as an administrator and with disabled av
"InvokeAI source installer..."
""
"Some of the installation steps take a long time to run. Please be patient."
"If the script appears to hang for more than 10 minutes, please interrupt with control-C and retry."
"<Press any key to start the install process>"
Press any key to continue . . .
""
"Downloading micromamba from https://github.com/cmdr2/stable-diffusion-ui/releases/download/v1.1/micromamba.exe to C:\image_generation\InvokeAI\installer_files\mamba\micromamba.exe"
A subdirectory or file C:\image_generation\InvokeAI\installer_files\mamba already exists.
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (56) Failure when receiving data from the peer
Micromamba version:
Access is denied.
Access is denied.
"Packages to install: conda"
Access is denied.
"There was a problem while installing conda using micromamba. Cannot continue."
Press any key to continue . . .

Screenshots

Additional context
No response
Contact Details
No response

I had this error, too. but somehow it installed after a few tries.

To fix edit the install.bat file.
locate the line:
call curl -L "%MICROMAMBA_DOWNLOAD_URL%" > "%MAMBA_ROOT_PREFIX%\micromamba.exe"
change it to:
call curl --ssl-no-revoke -L "%MICROMAMBA_DOWNLOAD_URL%" > "%MAMBA_ROOT_PREFIX%\micromamba.exe"
save and run. it should now work.

Yep, it seems to fix the issue, but why it's working and why it's not implemented in InvokeAI?


Update documentation for senzing-tools integration

Pull requests:

https://github.com/Senzing/initdatabase/pull/17


A question about test data

Hi, great paper and thanks for providing the code and related dataset.
I'm having difficulty reproducing your test result. May I ask how to get ground truth brain MRI test image? I didn't see reconstructed image in fastMRI multicoil brain test dataset.

Hello,
Thanks for your kind words, you may find a smaller sized dataset with fully-sampled images in tfrecords format in our previous project page: https://github.com/icon-lab/FedGIMP?tab=readme-ov-file


Fix in-memory WavPack decoding

Hi all,
I noticed that in-memory WavPack decoding wasn't working. I'm pretty sure this was because the WavpackStreamReader64 implementation didn't acknowledge the decoder->dataPos variable in the read_bytes function. So the actual WavPack library expected the file cursor to advance after calling read_bytes, but it always got data starting from byte 0.
I tried changing the read_bytes function so it copied the requested bytes starting from decoder->dataPos, and that worked, BUT when the WavPackInteral destructor was called, which calls WavpackCloseFile, for some reason I got a segfault inside the WavPack library. I started reading the WavPack docs to figure out what might be going on, and I didn't figure out what the problem was, but I noticed there is a function called WavpackOpenRawDecoder that is intended to be used to decode a WavPack file in memory.
I decided to just re-implement WavPackDecoder::LoadFromBuffer with that function. It makes WavPackDecoder.cpp a lot simpler because we can get rid of the WavpackStreamReader64 implementation (read_bytes, write_bytes, etc.) We can also get rid of the data vector and dataPos variable. The only quirk is that WavpackGetNumSamples doesn't work with WavpackRawDecoder. This is explained in the docs. It seems like you just have to parse the first "block" and get its total_samples property. Once you have that, everything else seems to work more or less the same way as it does when reading from a file. I added a private method readNextHeader to get the contents of that header. I also factored out all the common code from the two WavPackInternal constructors into one private method decode(size_t totalSamples). (I'm not sure about the name for that private method, let me know if you have a better one.)
I tried using this implementation to read each of the .wv files (loaded into memory) in test_data/ad_hoc, and they all work. I also checked that reading them as files still works.
Let me know how this looks! I'm not an expert in C or C++ so please let me know if I did anything wacky. Thanks!

Thanks for your PR @mikeperri ! I'll take a look at this over the weekend.


force_ssl is documented to be executed from the class context

Not all redirections to https were properly handled. By making this
change we manually redirect to https as a before_action as before,
but handling a redirect_to in the chain (effectively stopping the
chain and in the expected order).

This code will be removed from  here in the short term, as this will be properly handled by a reverse proxy and rails won't need to care about this at all. This is a short term solution.

Way to reproduce the issue. Start a fresh devenv, when it's up, run:
~ curl http://localhost/setup/discovery
You are being redirected.

This is not what's expected, as the first redirection should always be to https, we'd expect that redirect to be: https://localhost/setup/discovery
In any case, if we follow the proposed redirection:
~ curl http://localhost/users/sign_in
<!DOCTYPE html><title>SUSE CaaSP: Velum</title>...

I can properly see the sign in page, which is not supposed to happen at all.
With this PR:
~ curl http://localhost/setup/discovery
You are being redirected.

Following the proposed redirection:
~ curl -k https://localhost/setup/discovery
You are being redirected.

testing this right nao as well.

tested and i can confirm this works as expected, i now get redirected, properly, from http to https on all endpoints i can access in the dashboard. no more ruby errors.


Expection Server tick loop crash.

Game crashes unexpectedly without a warning. I dunno why. I killed the midnight lurker and like 10-20 seconds after it crashed. Please fix. I really enjoy the mod.
crash-2024-06-16_12.46.29-server.txt
latest.log

I couldn't seem to recreate crashing after killing the Lurker, but I think I fixed the problem based on what the crash logs said, so when I publish the next update, it should be fixed, if not just open another issue here or whatever.


[Ultima Torcia - Pagina Singola] added data-i18n values to make sheet translation

Changes / Comments
Hi. I put data-i18n values in my sheet, so i can do the translation. From what I understood, i have to set firts these values and then make the translation with Crowdin.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Nixos/qbittorrent init

Description of changes
work for adding a qbittorrent module option to nixos
what works.

basic profile location definition
qBittorrent/config/qBittorrent.conf definition with genINI ( should have a defined custom type.. )

what does not or may not work

openFirewall
extraPackages

potentially nice extras.

custom webui definition.
my first idea for this was to make packages for each webui project and add them to extraPackages to then reference their place in the RootFolder option.
a nice way to include search plugins.
they seem to accept just web links so id assume there's some reasonably nice way to supply a list of links that are downloaded to the appropriate location, from there it seems the program will maintain them on its own including updates and imports etc

I'm using this already but obviously having troubles, for reference you can see my use here
Drafting this so i can get feedback and collect thoughts on how to approach problems like including custom webuis and search plugins, any insight is appreciated.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

For custom WebUIs, users should just configure it separately and set something like serverConfig.WebUI.RootFolder = ${some-custom-webui-drv}. I don't think this module needs to add it to systemPackages or do anything special, apart from maybe documenting the option and typing it.

definitely agreeable, documenting it especially.

renamed to init-nixos-qbittorrent because i got annoyed with the / :sweat_smile:


Developers’ Code Context Models for Change Tasks

Thomas Fritz†
<EMAIL_ADDRESS>David C. Shepherd*
<EMAIL_ADDRESS>Katja Kevic†
<EMAIL_ADDRESS>Will Snipes*
<EMAIL_ADDRESS>Christoph Bräunlich†
<EMAIL_ADDRESS>†Department of Informatics *
Industrial Software Systems
University of Zurich ABB Corporate Research
Switzerland Raleigh NC USA
Dataset link Broken: https://github.com/abb-iss/study-artifacts-for-codecontext-models
Try emailing the authors

**Link To the paper: http://laser.inf.ethz.ch/2014/students/kevic.pdf
**Datasets:  open source projects.


Missing quotation marks

The x:Name attribute value in the example is missing quotation marks

@jwmsft: The PR passed the acceptance checklist. Please evaluate further and approve/complete the PR if appropriate. Thanks!

@moshegutman, thanks for submitting this fix.


Bad hyperlinks to models from API docs

Bad hyperlinks to models from API docs
Request Type
Bug
Work Environment

Question
Answer

Browser
Mozilla, Edge (yuck!)

Problem Description
When clicking the hyperlinks to be taken to the docs for the various models used in the different methods, A 404 error is thrown!
Picture of webpage displayed on 404

Steps to Reproduce

Go to the API docs
Find a function with a hyperlink to a model, say update_case. Click the link to a Case
Observe a 404 error as seen above.

Possible Solutions
(keep this section if you have suggestions on how to solve the problem. Otherwise delete it)
I am assuming it is simply looking at the docs and making updates to links? I am ignorant of how your docs are set up though.

Browsable documentation is deployed at: https://thehive-project.github.io/TheHive4py/


[Breaking Change] BreakingChangeReviewRequired label should be added but bot doesn't do that

in this pr:https://github.com/Azure/azure-rest-api-specs/pull/24296
breaking change check is failed , and in ci Swagger Breaking Change is required to review, but bot doesn't add "BreakingChangeReviewRequired" label. Instead , if I add this label manually, after running/azp run or updating changes by author, this label will be removed by bot.
I think it is a bug

Most likely this is because there was a runtime error
Adding to:

6169

This may be a dup of #7770


Notebooks for AI with Qiskit

Summary
Details and comments

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


feat: Improve performance during rendering

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.92%. Comparing base (b32ae2b) to head (c7b9a05).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #53      +/-   ##
==========================================
+ Coverage   94.91%   94.92%   +0.01%
==========================================
  Files          42       42
  Lines        2577     2583       +6
==========================================
+ Hits         2446     2452       +6
  Misses        131      131

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


MySQL port missing in config.php file?

The config.default.php contains all keys needed to reach a MySQL/MariaDB instance except the port number. Is it just missing in this file but supported? Or not supported?

There is no default in the config.php file, but it a supported value based on the part of the code that uses it: https://github.com/miniflux/miniflux/blob/e2bf7222130d0baaa8a11ebaee49c850a754ac54/app/common.php#L14
If you want to define a port simple add the following to your config.php
define('DB_PORT', 3306);

You can of courser replace the port value with anything you would like. Please note, if you are using the default port on MySQL then this is not necessary because the default connection will try to reach localhost:3306.

perfect, thanks. I close this ticket.


Unable to initialize cropper when first tab is showed

I know that this is not really an issue that has to to with the cropper but I am trying to initialize the cropper when the first bootstrap3 tab is showed and i am doing

$(document).ready(function () {
		$('a[data-toggle="tab"]:first').on('shown.bs.tab', function (e) {
			mycrop(e, $(this));
			console.log("first");
		});
...

the console.log is not reached. The initialization is done only when I click on the tab's link but I need the cropper visible when the tab is visible/active

As the first tab is shown by default, so the shown.bs.tab will not been triggered any more. You should initialize the cropper directly (outside of the shown.bs.tab event handler)


Include Julich-Brain Atlas as parcellation map

In siibra python, Julich-Brain is a parcellation and MNI152 or colin27 is a space.
In siibra explorer inside the layer box the Julich-Brain atlas does not appear directly under parcellations but can only be selected as default parcellation via the spaces.
Shouldn't Julich-Brain also appear as a parcellation?

This is a valid question. What I suggest is to add the cytoarchitectonic maps to the list of maps in the layer selector. Currently, they are not inlcueed there since they are considered the "base layer" which is always active. But it is true that this makes them less visible.

related to #1230

fixed in 2.8.0


add tests for our exceptions

@sean797 now tests break, am I using your exception wrong?

@sean797 this should pass tests now, am I using the exception as you intended to?


Creating series from 2 seqs of different lengths is possible

Hi.
Currently it is possible to create a series from two sequences (keys and data) which have different lengths. You get no exception from a constructor but when you try to Series.map some function you have a wrong index exception.
I don't know if it was done intentionally or it is a bug.

This looks like a bug, thanks for reporting it!

To reproduce
let keys = [ 1; 2; 3 ]
let data = [ 1; 2 ]
let s = Series(keys, data)

yields
val s : Series<int,int> =
1 -> 1
2 -> 2
3 -> <missing>

I don't see a use case for this (adding missing values at the end) so this should probably throw an exception.

It probably should throw, but it might be worth exploring what implications this would have (there might be performance reasons for this).
The Series constructor is used quite heavily internally - and users should typically create series using the series function or Series.ofObservations, so the constructor might be ignoring some checks (assuming it is "practically internal")

Yeah, I noticed. I added some naive test checking index and vector length, and a whole bunch of tests didn’t pass anymore. I didn’t investigate further, but it might be more reasonable to stay with the „practically internal“ assumption.

This is strange!
I assume the 2 <> 3 mismatch happens when creating a series to represent a row. The index for the series should be the index of the data frame (which has keys Sparse, Dense and Time) and the vector should be a vector created by the createRowReader function - this creates a virtual vector that reads data at a specified address in a data frame (it takes the data of the data frame as IVector<IVector>).
So. my guess is that somehow, the missing value in the row causes this vector to just have length 2. However, I do not quite see how. The TryGetRow function just passes the frame data to createRowReader and the RowReaderVector class also does not seem to contain anything that would drop the value...
(I need to update VS to actually run and debug things, so this is just a guess, but I'm happy to investigate more once my VS updates :-))

That's great. Please take a look as I was totally lost in the call stack.
BTW, do not install 15.8 yet. Install 15.7.6 maybe. I just updated to 15.8 yesterday and then SDK has changed location. During document generation, fsi was not found somehow. I'll look into it.

I spent 30 minutes looking into this and it's more subtle than I thought.
The RowReaderVector that I thought is a problem gets created correctly. The issue seems to be in some other part of "virtual Deedle" which tries to make it possible to create frames with virtualized data sources. It looks as if there was something created lazily and it kept the reference to the two-column data frame (which the test then mutates). However, everything in the test seems to be working fine, so it looks like this is just forcing evaluation of something lazy that is not needed.
So, I'm giving up too. For now, I think we could handle most situations by checking for array vector (which is the in-memory vector type) in the constructor too:
if index :? Deedle.Indices.Linear.LinearIndex<'K> &&
   vector :? Deedle.Vectors.ArrayVector.ArrayVector<'V> then
  if index.KeyCount <> vector.Length then
    invalidOp "Index and vector of a series should have the same length!"

As an aside, I think nobody actually uses the "virtual Deedle" parts, so - as much as I had a lot of fun writing that - it would not be entirely unreasonable to just refactor Deedle and drop that altogether. It would probably make our life easier...

@tpetricek I tried your virtual Deedle and the idea looks great. It's just I haven't got a use case for that yet as I am not dealing with that amount of data yet. I don't think we need to drop it at all. I'll create an issue to track how to fix it.
Let's get the core of Deedle more robust and more adopted before we look ahead for bigger cases.


Use MSAL in pipeline tasks - group 3

Task names:

AzureRmWebAppDeploymentV3
AzureRmWebAppDeploymentV4
AzureSpringCloudV0
AzureVmssDeploymentV0

Description: Update version of azure-pipelines-tasks-azure-arm-rest-v2 to ^3.213.0
Documentation changes required: N
Added unit tests: N
Attached related work item: https://dev.azure.com/mseng/AzureDevOps/_workitems/edit/2006427
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

For the failing tests you can run this command locally:
node make.js build --task TASK_NAME && node make.js test


Add Go Report Card and Apache2 licence to README.md

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


List of option from object without keys

Hi there!
I haven't find it so I guess I already have my answer. But before reformatting anything, is there any way to have the option list working from a source like the following:
       `"options": [
          {
            "": "--"
          },
          {
            "AB": "Alberta"
          },
          {
            "BC": "British Columbia"
          },
          {
            "MB": "Manitoba"
          },
          {
            "NB": "New Brunswick"
          },
          {
            "NL": "Newfoundland"
          },
          {
            "NT": "Northwest Territories"
          },
          ....
        ]`

Thanks for the help!
Heyddi

id say you would need to bind it to a computed property where you reformat it into name and value pairs (or just preprocess it before hand)
var newObj = {}
for(var key in options) {
if(options.hasOwnProperty(key)) {
var name = options[key];
newObj ['value'] = key
newObj ['name'] = name
}
}
that would convert one of them to {value: 'AB', name='Alberta'} you'd just need to loop it and build the new array.

Thanks so much for the help! That was already something like this that I worked on but I was wondering if I missed something simpler or not.
Thanks again,
Heyddi


chore(release): 1.0.0-rc.3

Purpose
Merge pre-release version in.

Restored branch release/1.x.x, will you see again for 4th release candidate, and for full version release we can switch to a new release process.

Restored branch release/1.x.x, will you see again for 4th release candidate, and for full version release we can switch to a new release process.


make sure to unsubscribe from file events

I looked through vscode's code base and It doesn't look like fileSystemWatcher.dispose() through disposables.push(fileSystemWatcher); will unsubscribe each file watch subscription.
each must be disposed individually by this.hookListeners(fileSystemWatcher, watchCreate, watchChange, watchDelete, **disposables**); like how registerRaw does.
here are relevant code in vscode
https://github.com/microsoft/vscode/blob/main/src/vs/workbench/api/common/extHostFileSystemEventService.ts#L22
https://github.com/microsoft/vscode/blob/main/src/vs/workbench/api/common/extHostFileSystemEventService.ts#L91
https://github.com/microsoft/vscode/blob/main/src/vs/base/common/event.ts#L617
https://github.com/microsoft/vscode/blob/main/src/vs/base/common/event.ts#L681
https://github.com/microsoft/vscode/blob/main/src/vs/base/common/lifecycle.ts#L356
basically, no one is calling dispose on listener.subscription which is disposable returned by fileWatcher.onXXX call.

@aeschli , can you have a look as well.

@heejaechang Did you file a bug against vscode as well?

@aeschli is it a bug in vscode? I don't see any other code that use Event class remembers all subscription made to the event and dispose them when event itself is disposed. instead, all other case, each caller that calls Event.event(...) remember returned disposable and explicitly dispose them.
so, it seems fs watcher behavior around event is just normal pattern in vscode?
you can search for Event.event usages and check how they are used in vscode.

is there something I need to do to be checked in?


Running command  python3 -m venv venv fails.

I followed the steps and when I executed the command python3 -m venv venv it failed with an error
Error: Command '['/mnt/c/WORK/projects/bulk-policy-migrator-scripts-for-account-cost-billing-consoles/venv/bin/python3', '-Im', 'ensurepip', '--upgrade', '--default-pip']' returned non-zero exit status 1.
When I tried running the command manually venv/bin/python3 -Im ensurepip --upgrade --default-pip it threw an error
bulk-policy-migrator-scripts-for-account-cost-billing-consoles/venv/bin/python3: No module named ensurepip
can some help me on how to fix this.

I see same error even with the above command.

Some microsoft strangeness going with that ensureip thing, perhaps you need to pip install it?
This is what I get:

git clone https://github.com/aws-samples/bulk-policy-migrator-scripts-for-account-cost-billing-consoles.git
Cloning into 'bulk-policy-migrator-scripts-for-account-cost-billing-consoles'...
remote: Enumerating objects: 135, done.
remote: Counting objects: 100% (135/135), done.
remote: Compressing objects: 100% (105/105), done.
remote: Total 135 (delta 78), reused 75 (delta 29), pack-reused 0
Receiving objects: 100% (135/135), 47.92 KiB | 732.00 KiB/s, done.
Resolving deltas: 100% (78/78), done.
cd bulk-policy-migrator-scripts-for-account-cost-billing-consoles
ls
CODE_OF_CONDUCT.md  CONTRIBUTING.md  LICENSE  policy_migration_scripts  README.md  requirements.txt
python3 -m venv .venv
source .venv/bin/activate
pip install -r requirements.txt
Collecting boto3
Downloading boto3-1.28.56-py3-none-any.whl (135 kB)

https://docs.python.org/3/library/ensurepip.html

Looks like it is because of this
ensurepip is disabled in Debian/Ubuntu for the system python.

running a command python3 -m venv venv --without-pip helped. On ubuntu or debian machines we need to use this command.

Up and running?

Up and running?

Yes

@niharbabu99 - Glad you have it up and running.
I spun up an ubuntu instance (20.04 LTS) which comes with Python 3.8.10 pre-installed. I ran the command: python3 -m venv venv. I got the below error:
The virtual environment was not created successfully because ensurepip is not
available.  On Debian/Ubuntu systems, you need to install the python3-venv
package using the following command.

    apt install python3.8-venv

You may need to use sudo with that command.  After installing the python3-venv
package, recreate your virtual environment.

So I had to install this package by running sudo apt install python3-venv. Once I installed this package, I re-ran the command: python3 -m venv venv and this worked.
I will update the README with this extra command specifically for ubuntu.
However I noticed one problematic issue in the screenshot you shared - You seem to be using Python 2.7, which is not the recommended version for running these scripts. As you can see in the Prerequisites, we recommend using Python 3. Please upgrade your python version, create a new venv using Python 3 and then run the scripts.
These scripts are not built to run with Python 2

@niharbabu99 - Glad you have it up and running.
I spun up an ubuntu instance (20.04 LTS) which comes with Python 3.8.10 pre-installed. I ran the command: python3 -m venv venv. I got the below error:
The virtual environment was not created successfully because ensurepip is not
available.  On Debian/Ubuntu systems, you need to install the python3-venv
package using the following command.

    apt install python3.8-venv

You may need to use sudo with that command.  After installing the python3-venv
package, recreate your virtual environment.

So I had to install this package by running sudo apt install python3-venv. Once I installed this package, I re-ran the command: python3 -m venv venv and this worked.
I will update the README with this extra command specifically for ubuntu.
However I noticed one problematic issue in the screenshot you shared - You seem to be using Python 2.7, which is not the recommended version for running these scripts. As you can see in the Prerequisites, we recommend using Python 3. Please upgrade your python version, create a new venv using Python 3 and then run the scripts.
These scripts are not built to run with Python 2

Yes, you are right, command I used is wrong. Later I realized and ran command pip3 install -r requirements.txt, post which I am able to use the scripts.


Intergrating to Matte-Anything raise model mismatch

as title

do you want to train model ? i think you must makes the strict =False if you want to load the pretrained vit-tiny model

I am using MobileSAM with Matte-Anything for inference here - https://github.com/neuromorph/image-anything
Have a look if it helps.


Replacing deprecated traffic_line with traffic_ctl

Closes #1128

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-whatchanged/64/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/195/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/156/

This looks good to me. This is one of several important steps toward bringing TC up to where it can support ATS 7.x and forward.

Closing as there was just another pull request to do the same.


Does not follow previous conventions of Restify

Using this new module:
return next(new errors.BadRequestError('400 bad request message here'));

Returns the following JSON:
{"code":"BadRequest","message":"400 bad request message here"}

Previously, using Restify:
return next(new restify.BadRequestError('400 bad request message here'));

Returned the following JSON:
{
    "message": "/foo does not exist",
    "statusCode": 404,
    "body": {
        "code": "ResourceNotFound",
        "message": "/foo does not exist"
    },
    "restCode": "ResourceNotFound"
}

I actually don't care about the body property, but statusCode and restCode are useful, and should be named the same key.

Any update on this?

Hi @nodesocket, to update the convo I suspect you may have a custom JSON formatter that is returning the full key set of the error object. The default formatter returns only the body property. Can you confirm?

Sorry for the delay responding.
Correct, using a custom JSON formatter of:
_server = restify.createServer({
        formatters: {
            'application/json': function(req, res, body, callback) {
                return callback(null, JSON.stringify(body, null, 4));
            }
        }
    });

Gotcha, that would do it. restify's default serializer used to do serialization for the error objects, that's now been offloaded to restify-errors' toJSON() method. Out of the box serialization hasn't changed for restify users. You've been seeing the full output cause you had your own serializer. If you want the full output, I'd recommend doing an body instanceof Error check, and then copying the properties that you want exposed.
Alternatively, if your errors are inheriting from restify-errors, you can specify a custom toJSON() when you call makeConstructor().

This appears to be resolved, let us know if you're still seeing issues.


How do we get page title for 3a?

Unit Anchors has a title of Test Title.
See how we actually get the page title, in a repeatable/reliable way.
Reproduce in content.

Captured in #416


Improved code to work with class @property, fixed tests.

This PR includes:

Functionality in footprints.py to work with @property.
Fixed tests.

Already repaired


Xcode 8 beta 2 error

I get an error on latest pod, on line 111 CWStatusBarNotification.m (return _preferredStatusBarStyle) it says "undeclared identifier _preferredStatusBarStyle"

Same to me, when I change to self.preferredStatusBarStyle to fix it but when will the official update will fix this issues?

i synthesised the property to fix it
`@implementation CWViewController
@synthesize preferredStatusBarStyle = _preferredStatusBarStyle;

(UIStatusBarStyle)preferredStatusBarStyle
{
return _preferredStatusBarStyle;
}`

@zhonghuiwen Did you add a PR for it?

Just need update cocoapods version. Thanks

Hi, it'd be great to get a push of this change :)

When pod gonna update?

When pod gonna update?

GM has been released today please update the pod

In the meantime you can use the following in your podfile
pod 'CWStatusBarNotification', :git => 'https://github.com/cezarywojcik/CWStatusBarNotification', :commit => '029ba79'

2.3.5 is pushed.


Windows 10 K: is already being used

I happen to have quite a few drives and in this case K: was already being used by a USB drive. Nothing happens when I try click to "open folder"

Please do a keybase log send and send us the ID.

3e0602cd06e0bc4132def21c
As a follow up I tried to switch the app_data\local\Keybase\config.json -> mountdir to a new value. After starting the app it was a bit grumpy and said I needed to repair the install.

You can change the drive letter by running keybase kbfsmount set X:

The crux of this is that it didn't seem to check that K was actually available before trying to mount it and after that things went awry. Seems like a good default would be to check and iterate to later letters until it  finds a free drive starting with K.
Looks like in the client\cmd_update.go we get:
func (v *cmdUpdateCheckInUse) Run() error { 	mountDir, err := v.G().Env.GetMountDir() 	if err != nil { 		return err 	} 	inUse := install.IsInUse(mountDir, v.G().Log) 	result := checkInUseResult{InUse: inUse} 	out, err := json.MarshalIndent(result, "", "  ") 	if err != nil { 		return err 	} 	fmt.Fprintf(os.Stdout, "%s\n", out) 	return nil }
The install.IsInUse just gives up if it's in use. (I think this is the right part of the code...??)

We call GetDriveType (https://msdn.microsoft.com/en-us/library/windows/desktop/aa364939(v=vs.85).aspx?f=255&MSPPError=-2147217396) on letters D-Z and consider it free if it returns DRIVE_UNKNOWN or DRIVE_NO_ROOT_DIR, and we also skip ones with volume name beginning with \device\cdrom. Your logs showed I-Z as free at the time, but the filesystem mount failed to find the right DLL. Driver install logs said a reboot was pending, was that ever completed?
2018-01-22T23:12:05.563532-08:00 - [DEBU kbfs keybase_service_base.go:1163] 03c Choosing mountdir K: from [I: J: K: L: M: N: O: P: Q: R: S: T: U: V: W: X: Y: Z:]
2018-01-22T23:12:05.563532-08:00 - [INFO kbfs start.go:99] 03d Got mount dir from service: -K:-
2018-01-22T23:12:05.563532-08:00 - [DEBU kbfs fs_notifications.go:89] 03e Launching notifications channel [tags:]
2018-01-22T23:12:05.563532-08:00 - [DEBU kbfs start.go:114] 03f New folder name guess: "New Folder" "New folder"
2018-01-22T23:12:05.563532-08:00 - [DEBU kbfs mount_interrupter.go:42] 040 Starting to mount the filesystem
2018-01-22T23:12:05.564033-08:00 - [ERRO kbfs mounter.go:30] 041 Failed to mount dokan filesystem (i=8): Dokan failed: code=-99 "Error loading Dokan DLL"

That is odd, I have USB drives on I, J, and K. It seems that it didn't detect the USB drives. The drives are Seagate Backup Plus Drive and a Kindle. The Kindle is what was mounted on the K Drive but it just appears as a standard USB large storage device.

The screenshot doesn't show the K drive plugged in (I removed it while poking at it)
The reboot was due to me trying to change the drive letter and it needing a reboot due to keybase trying to do a install repair. I am guessing the Dokan driver got a bit confused after that.
What is an easy way to see what GetDriveType returns so I can tell what the USB device is returning? (Happy to test / compile a golang app to test this)


Ignore virtual files

Fixes https://github.com/dotnet/aspnetcore/issues/18927.
This seemed to require editing a lot of test files to exclude the Blazor project from running other tests, if anyone on the O# side has a better way to handle that I'm open.
CC @NTaylorMullen, @ajaybhargavb

@JoeRobich I'm not 100% sure this is the correct fix, maybe you could correct my thinking (or convince me I'm already right).
Repro steps:

dotnet new blazorserver
In Index.razor add the line @DoesNotExist.
Watch the Problems pane, you should see an entry for Index.razor_virtual.cs flash briefly (occasionally it stays permanently). If you didn't see it try again, it should repro each time.

When I track this down I can find requests to _validateDocument for the Index.cshtml_virtual.cs file, which has returned a result for Index.cshtml, even though the request was for _virtual.cs.
Hopefully that all makes sense, I can go into more detail offline if you need more context or a demo.

@ryanbrandenburg I tried the step you listed and this is what I see in the latest release 1.21.12. I do see a reported problem, but it is for Index.razor.

@ryanbrandenburg I spoke too soon. I see what you are talking about.

@JoeRobich, I've updated it with your feedback but it seems I don't have permissions to merge.

Thanks @ryanbrandenburg !


Download Helper tests print log to screen

Even though:
// Prevent Logger output during tests
sandbox.stub(dh, "log");

is set in the tests, the logger still prints to screen when running tests

I recall messing around with this to properly silence the logger. winston and its dependencies accounts for a significant portion of the bundle, and it's annoying to fight through this silencing - I propose we remove Winston and implement a very basic logging class ourselves.

fixed with 19375a1a1b36860cb8d450f5c95ad989b302145b


fzf: 0.38.0 -> 0.39.0

Description of changes
Diff: https://github.com/junegunn/fzf/compare/0.38.0...0.39.0
Changelog: https://github.com/junegunn/fzf/blob/0.39.0/CHANGELOG.md
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Happy to report that the proposed changes in this PR also work on aarch64-darwin 👍


Store all the events scraped to the Firestore Database

Feature purpose
Store all the events scraped to the Firestore Database
Ideal solution
-- Create a event collection on firestore
-- Add code to start storing events

Code has been pushed blocked on the small bug, will make a PR after the bug has been fixed.


Add user guide for streams & events

Part of https://github.com/cupy/cupy/issues/4902#issuecomment-813205619.

Thanks @leofang, great work!

pfnCI, test this please.

Jenkins CI test (for commit e67aa68e437aebdcdaec1df9a850e2839990f50c, target branch master) succeeded!

Thanks, @kmaehashi. I realized I forgot to document ExternalStream. I will send a separate PR.


Token Guidelines

We should have a comprehensive documentation of our token system which explains the basic concept and the architecture we're using.
Basics
Design tokens — or tokens, for short — are design decisions, translated into data. They’re ultimately a communication tool: a shared language between design and engineering for communicating detailed information about how to build user interfaces.
Tokens consist of values needed to construct and maintain a design system, such as spacing, color, typography, object styles, animation, and more. They can represent anything that has a design definition, like a color as a RGB value, an opacity as a number, or an animation ease as Bezier coordinates.[^1]
Other resources:

Atlassian Design Tokens
Salesforce Lightning Design Tokens
Material Design Design Tokens
W3C Design Token Community Group

Architecture
Tokens are categorised into 3 groups: core, semantic and component tokens. Each group has it's specific purpose and rules. The core and semantic layer are meant to be internal to the Design System, the component layer is intended to be used by the implementation. This enables introducing changes to the core and semantic layers without breaking changes in the component layer.
Core tokens
Core tokens are the most basic tokens and represent concrete values, for example yellow = #ffcc00 or size-16 = 16px.

Don't assign semantic names to core tokens. Do yellow = #ffcc00 instead of brand-color = #ffcc00. brand-color is a semantic name and belongs in the next group
Core tokens can reference each other in composite or derived tokens: sandgray = #050400 can be reused for a transparent variant with sandgray-alpha-50 = rgba(sandgray, 0.5)
Core tokens should not be aliased. When there's a definition for sandgray = #050400 there should not be another core token with the exact same value, e.g. othergray = sandgray or othergray = #050400. This would unnecessarily increase the number of tokens
Core tokens should not be referenced or used directly in any application since they represent static values that are intended to be referenced by either semantic or component tokens

Semantic tokens
Semantic tokens represent changes in design decisions across different environments, for example the body font-size that grows larger with increasing browser size. The semantic layer can also be seen as a mapping layer between the core and the component layer. It reassigns tokens to the correct values based on the current environment, for example styles for an external post themed page on a mobile device in dark mode.

Semantic tokens should be named around design concepts like brand-color or font-size curves (TODO: better define what design concepts are)
Example of a responsive font size token definition in the semantic layer, referencing three different core tokens:

Token
Mobile
Tablet
Desktop

font-curve-base
size-16
size-18
size-20

Semantic tokens can reference core tokens as well as other semantic tokens

Semantic tokens should not be referenced or used directly in any application since they represent changing values. In any given environment, a semantic token is never guaranteed to be the finally desired value since it could be referenced and overridden by another semantic token

Tokens from the semantic layer are intended to be implemented in an automated process

Semantic tokens should be grouped by environment: modes (light/dark), themes (post/subsidiary X/subsidiary Y), devices (mobile/tablet/desktop), channels (internal/external).

It should be possible to extend or reduce semantic groups or their environments and re-map values without introducing breaking changes to the system. This is only possible if implementations use an automated process to ship semantic tokens and never directly reference semantic tokens from within components

Component tokens
On the last layer, component tokens are tied directly to components (or utilities) and their properties.

Component tokens should be named according to the property of a component, for example button-padding-left = interactive-padding-left. The naming of properties should be as platform agnostic as possible (foreground-colour instead of just colour which would be CSS specific and not self explanatory in a token context since it could mean border colour, background colour or text colour).
Every component should have it's own set of tokens, component tokens should not be shared between components (see #3356)
Component tokens should reference semantic tokens if their value is expected to change based on the environment they are used in. If the value is static, component tokens can also reference core tokens directly

[^1]: Design tokens - Spectrum: https://spectrum.adobe.com/page/design-tokens/

Started something on experience hub but didnt get to finish it yet, maybe we can combine that with the text above

Closing in favor of https://github.com/swisspost/design-system/issues/3448


More gate sizes

Today I had this idea and while I'm aware that this is most likely not on a scale for a community patch, I wanted to post it anyway so that the idea is at least noted. Imagine Firefly doing a Remastered Stronghold 1 and Crusader at some point and browse through such a forum for improvements (Hi future Firefly) or maybe some hobby-devs find it to be easier to implement than I'd think. Anyway, it'd be so exciting to have such a feature in the game especially when designing AI castles. Due to the limited options of the AIV-Editor it is rather an idea for a remastered version (yes I know how unlikely that is going to happen).
I think most players are aware that small and especially large gates aren't quite the most pretty objects in the game in terms of size and fitting to walls. It just looks ridiculous when placing that 7x7 cube onto that 2-layered wall. And unless one builds a wall of 5 layers, it will still look just bad.
So the idea is to have more gate sizes, especially non-square ones, without creating a completely new design for gates.
Remastered Solution: Make small and large gates being click'n'drag in size, where the width is fixed but the depth can be alternated. Small gates would range from 2x5 to 5x5 in size while large gates range from 3x7 to 7x7.
Community Solution: Create some hard coded gates based of the available options and add them to the gates section of the building UI.  That are 3 additional small gates and 4 additional large gates. If you were to limit the options, I'd go for 3x5; 4x5; 5x5 and 3x7; 5x7 and 7x7.
Just dreaming a bit of what could be made out of this game =)

Further to this, Id love if the large gate had a 2-3 tile path through (instead of 1 currently), and seige engines and cavalry couldnt path through the small gate.

I like this:

3 additional small gates and 4 additional large gates. If you were to limit the options, I'd go for 3x5; 4x5; 5x5 and 3x7; 5x7 and 7x7.

and this:

Further to this, Id love if the large gate had a 2-3 tile path through (instead of 1 currently)

Also I guess this:

seige engines and cavalry couldnt path through the small gate.

could cause some problems with AI's using knights and small gates and would be too restricting overall.
Upon implementing more gate sizes I just want to recommend scaling their HP. Thus the 3x5 version of the small gate should only have 3/5 the HP of the original 5x5 version. Otherwise you could just place 2 of the 3x5 gates in a row and have doubled HP in almost the same amount of tiles, basically making the original 5x5 size gate very unfavorable.


Updated applies to - exclude MI

Updated applies to from 'appliesto-ss-asdbmi-xxxx-xxx-md.md' to 'appliesto-ss-xxxx-xxxx-xxx-md.md' since Managed Instance does not support resizing SQL agent job history.
Reference article:
https://docs.microsoft.com/en-us/azure/sql-database/sql-database-managed-instance-transact-sql-information#sql-server-agent
•SQL Server Agent settings are read only. The procedure sp_set_agent_properties isn't supported in Managed Instance.
And there is no right click - properties - option when working with Managed Instance for SQL Server Agent.

@sqlazure : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thank you, @sqlazure,
We will update the article.
Thank you.

#sign-off


Fix/762

Fixes #762
To test this feature on localhost, enter the following URL:
https://localhost:3000/#/swap/:senderToken/:signerToken?serverURL=server.com
For server.com in the URL: on Goerli you can use the server https://airswap-goerli-http.herokuapp.com/.
A working example URL on Goerli is: http://localhost:3000/#/swap/native/0xb4fbf271143f4fbf7b91a5ded31805e42b2208d6?serverURL=https://airswap-goerli-http.herokuapp.com/
Some possible servers on mainnet for server.com are:
https://airswap-pmm-rfq-server-i24uhfiu3fh.alphalab.cc
https://b2c2.xyz/airswap/
https://aomcfsa7.altono.xy
https://ast.ngrok.io
A working example URL on mainnet is: http://localhost:3000/#/swap/native/0xa0b86991c6218b36c1d19d4a2e9eb0ce3606eb48?serverURL=https://aomcfsa7.altono.xyz
Note that the translations for order.selectedServer are blank except for English

I'm unable to test out anything on Goerli. No token pairs seem to exist with ETH, other than WETH (which isn't a pair)

@piersss @dmosites I'm unable to test this out correctly on Goerli. The only pair that works in ETH - WETH. ETH - USDC, or ETH - USDT doesn't work.
On Mainnet I'm able to test out ETH - USDC on the server below, but unless I click "complete swap" I'm not able to test further. I don't want to test with real funds: http://localhost:3000/#/swap/native/0xa0b86991c6218b36c1d19d4a2e9eb0ce3606eb48/?serverURL=https://ast.ngrok.io/.
This is a blocker. How should I proceed?

Please view PR #786 instead of this. That PR is a copy of this but with more updated code from the latest merged branch


"insert on hover" feature

As far as I see, inserting nested element is based on insertZone threshold currently. User has to move the mouse some distance from the left edge of the element and if this distance is above the threshold, dragged element is inserted as a child to hovered one.
We could add optional feature that allows to change (or extend) this behavior (from UX perspective): when user hovers the element, dragged one becomes its child. In my case all attempts to simulate this by changing insertZone made widget unusable (ex. when value is 0 there was no way to reorder elements).
Technically we may reuse the mechanism and create something like 'vertical insert zone'. If the mouse is close enough to the vertical center of hovered element, dragged one will become its child when dropped, otherwise elements are reordered.
See attached image for explanation.

What do you think?

I think I understand you. It is quite interesting.

Hi, I have it. You can try it in dev3 branch. Lok at that please and tell me it there is no bug. It looks pretty good.

It's perfect, thanks!

No I am going to release it as version 2.0.0. I want to keep the code as clean as possible. So I don't want to add new variables and conditions. I think this is more intuitive behavior as that before and is not necessary to support it.

But I need to check if it works right. Mainly if export functions does not produce wrong results e.g empty hintWrappers and so.

Oo there is a problem with opening closed lists. It shouldn't be opened every time.

Ok done, but I am not sure. List jumping up and down too quickly while dragging.

Now this behavior is optional. You can allow it in options via insertZonePlus param. Default is false.

:+1:


can't use same id twice in columndefintion

Griddle version
v1.1.0
Expected Behavior
I can add multiple <ColumnDefinition /> with the same id, so that i can
have multiple columns accessing the same value
Actual Behavior
Only the last columndefinition is visible
Steps to reproduce
add two columnDefinition with same id

I can't speak for the maintainers, but reusing id seems undesirable since that is how columns are uniquely identified. See https://github.com/GriddleGriddle/Griddle/issues/623 for how to reference data from other columns in a custom component.

We were not intending for this sort of behavior. It's definitely possible I'm being short-sighted but can you please explain where this would be necessary? Maybe I can be convinced otherwise :D If not maybe there's another way to solve what you're trying to do -- I'd love to help either way. Thanks!


IDE error reports nonstop when the DDEV project is stopped

Is there an existing issue for this?

[X] I have searched the existing issues

Are you sure that this bug is related to this DDEV Integration Plugin?

[X] I am sure

Enter your error report ID (If available)
No response
Describe the bug
Whenever the DDEV instance is stopped or not running, the IDE will report every so often about an error.

Error output stacktrace:
de.php_perfect.intellij.ddev.cmd.CommandFailedException: Command '/opt/homebrew/bin/ddev describe --json-output' returned non zero exit code {exitCode=1, timeout=false, cancelled=false, stdout=, stderr={"level":"fatal","msg":"\u001b[31mFailed to describe project(s): PROJECT_NAME is not a valid project name. Please enter a project name in your configuration that will allow for a valid hostname. See https://en.wikipedia.org/wiki/Hostname#Restrictions_on_valid_hostnames for valid hostname requirements\u001b[0m","time":"2023-09-25T09:42:08-04:00"}
}
	at de.php_perfect.intellij.ddev.cmd.DdevImpl.execute(DdevImpl.java:73)
	at de.php_perfect.intellij.ddev.cmd.DdevImpl.describe(DdevImpl.java:38)
	at de.php_perfect.intellij.ddev.state.DdevStateManagerImpl.checkDescription(DdevStateManagerImpl.java:155)
	at de.php_perfect.intellij.ddev.state.DdevStateManagerImpl.lambda$updateConfiguration$1(DdevStateManagerImpl.java:67)
	at de.php_perfect.intellij.ddev.state.DdevStateManagerImpl.checkChanged(DdevStateManagerImpl.java:90)
	at de.php_perfect.intellij.ddev.state.DdevStateManagerImpl.updateConfiguration(DdevStateManagerImpl.java:65)
	at de.php_perfect.intellij.ddev.state.StateWatcherImpl.lambda$startWatching$0(StateWatcherImpl.java:33)
	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539)
	at java.base/java.util.concurrent.FutureTask.runAndReset(FutureTask.java:305)
	at com.intellij.util.concurrency.SchedulingWrapper$MyScheduledFutureTask.run(SchedulingWrapper.java:275)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
	at java.base/java.util.concurrent.Executors$PrivilegedThreadFactory$1$1.run(Executors.java:702)
	at java.base/java.util.concurrent.Executors$PrivilegedThreadFactory$1$1.run(Executors.java:699)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:399)
	at java.base/java.util.concurrent.Executors$PrivilegedThreadFactory$1.run(Executors.java:699)
	at java.base/java.lang.Thread.run(Thread.java:833)

This error above also speaks to a failed pre-start hook that I have (that I failed on purpose). Either way the project is halted but the error pop up keeps appearing.
When I clicked "Report" it generated another 97 notifications.
Steps to reproduce
No response
Additional context
No response

Hello @kevinquillen,
The error says:

PROJECT_NAME is not a valid project name. Please enter a project name in your configuration that will allow for a valid hostname. See https://en.wikipedia.org/wiki/Hostname#Restrictions_on_valid_hostnames for valid hostname requirements

Could you please show your .ddev/config.yml?
It looks like you have an invalid ddev configuration, that's why the status check of the plugin is failing.

Yes, understood - but the problem is that it keeps probing the project. At least for me. We are creating a project template for projects (ex cloning GitHub to new repo). It starts with
name: PROJECT_NAME
type: drupal10
docroot: docroot
php_version: "8.1"

It errors with ddev start, because it should (we want people to change PROJECT_NAME obv to what the project will be). But the JetBrains plugin keeps throwing the error which makes it hard when developing around. As such, the notifications keep mounting. In this case I am aware of the problem, but in this scenario it isnt technically a problem.

It is the expected behavior of the plugin to monitor the status of the ddev instance. As the ddev binary throws an error (non zero exit code) this will bubble up and throw an error in the plugin as well.
But you can disable this background check for the "template project" by turning it off in the settings:

When creating a new project from this template, the first step should be to make the ddev config valid.
Does this solve your problem? What would you suggest on how to handle the error (non zero exit code) of the ddev binary?

Oh, okay, thanks. I didn't know that setting existed. That should work.


[docs]: duplicated h1 on overview pages

The fix will be to release and integrate the PR on pfetools, then set renderTitleInOverview to false

renderTitleInOverview will be available when  https://github.com/RedHat-UX/red-hat-design-system/pull/854 merges

#854 should also unset it in config

Closed in #854


Step 5 doesn't work.

The UI we see in AWS console doesn't match description
"5. Click the Attach button next to addPicture on the right of the schema editor and select PictureTable as the data source. For the request mapping template use the code in ./AppSync/Mutation.addPicture.request and for the response mapping template select Return single item from the drop-down. Press Save at the bottom."
Can't select "PictureTable" as data source. Can't set request mapping template.

Hi @MikeVaz Did you saw Save Complete after hitting save on schema window?

After repeating the step I was able to "select PictureTable as the data source" but couldn't do the next step "the response mapping template select Return single item from the drop-down."

It is not Amazon Dynamo DB for me. And previous steps didn't mention that.
Anyway trying to change it now.

What should be here?

If you did step 3 and 4 it should be correct. I would retry those steps and then it should be fine, I think the problem was that Step 4 had a problem.

I deleted the API and repeated the steps. I am stuck on step #4.
"Still on the Schema page, press Create Resources in the upper right corner.
Select the Picture type from the drop down.
Then click on Additional Indexes followed by Add Index, set the index name as owner-index with Primary key as owner and sort key as id.

This will create DynamoDB tables, some default queries/mutations and connect them to Resolvers"

It looks likes this table already exists and I can't find a way to delete it.

Hi @MikeVaz we updated awsmobile-cli, and the README has new instructions to run the app with the latest version.


Why am I getting expected string, got function?

I'm getting this issue all of a sudden when visiting a new route I just added:
Exception in template helper: Error: Match error: Expected string, got function in field ["class"]

How could this be?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/24894473-why-am-i-getting-expected-string-got-function?utm_campaign=plugin&utm_content=tracker%2F522373&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F522373&utm_medium=issues&utm_source=github).

it seems to be caused by using findOne() instead of find() as the returns of iron router data handlers. The issue went away when i changed it to a plain find() call.

The object returned from the database when using findOne, most likely had a member named 'class" which was a function. The helpers now uses the data context to populate the options which is most likely the cause of this error message.

I'm getting error too.  I'm using
{{#if isNotActivePath regex='login|password'}}
and I'm getting this error:
Exception in template helper: Error: Match error: Expected string, got undefined in field ["class"]

@zimme that is exactly the issue! ...is there another options? the 'class' property cant change for me--what are my other options to avoid this?

On vacation right now in magaluf but when i get back ill think about skipping the data context inheritance in favour of a data attribute the way dynamic template does it

I'm not sure what's the best approach to this "problem" is.

Skipping the data context inheritance and inherit from the data attribute instead.
Adding an option className which when set to true will use className instead of class, this would remove the class/className alias which is in place right now.

i have both class and className properties on my objects. So for me only the fist option would work. It does seem precarious relying on the data context for the route.

The reason I added this was to mimic the behaviour of {{pathFor ...}} and {{linkTo ...}} helpers which iron:router provides.
I think I'll just add support for the data attribute and an option to disable data context inheritance.
Sound good?

or maybe it's better to have data context inheritance as opt-in instead? but Meteor do have implicit data context inheritance by default for templates.

meteor does, but urs is more of an automated solution, which now is assuming maybe too much about people's code. it's probably best to keep it all to what's specifically provided as parameters.

I haven't forgotten about this, I've just been busy with work.
My thinking is that I'll skip the data context inheritance and inherit from data attribute just like Template.dynamic does.
As this is a breaking change I'll release a new major version when this is done, I'll be leaving support for meteorhacks:flow-router behind with this release too.

I run into this because I have a path member. Please make these kind of assumptions about the context an opt-in, or have a decent opt-out.

Hopefully I can get this done tonight, if not it should be done by the end of the week.

Would be grand!

On 18 Aug 2015, at 23:05, Simon Fridlund<EMAIL_ADDRESS>wrote:
Hopefully I can get this done tonight, if not it should be done by the end of the week.
—
Reply to this email directly or view it on GitHub https://github.com/zimme/meteor-active-route/issues/35#issuecomment-132350627.

This should be fixed with version 2.3.1. Please ping me here if it's not resolved and I'll reopen this issue.

Works... Thanks!


polish: Align filter dropdowns

Asana task: 📺▶ : 🇵🇱 | Align filter controls with Places table columns
Filter dropdowns are now aligned with the column that they are a filter for. I also cleaned up/renamed a few CSS classes so that there is more consistency.

Sorry for the late review on this! Overall it looks great, but I wonder if we should make the drop-downs go the full length of the column that they're set over, or just wide enough to fit the widest possible text on the dropdown + the (X) button. I found it a little weird when the mode dropdown extends to the left after selecting a mode. The other two dropdowns are a bit better imo since they're justified on the left. Happy to leave this for another polish task if we think it's necessary though.

Sorry for the delay. Finally getting back around to Screenplay stuff. This is a good call out. I don't think I ever really paid attention to the width before but now that you mentioned it, I can't unsee it. Went ahead and got that fixed. Widths I picked come from the designs. I also fixed a border radius issue when the clear button is visible.

Looking great! Only thing now is that the following two pieces of text seem to overflow out of the dropdown button:

Looking great! Only thing now is that the following two pieces of text seem to overflow out of the dropdown button:

Yikes! Good catch. Should be good now. Also, I noticed in the designs that Emergency takeovers has actually been shortened to just Emergency. Got that changed, too.


add dns caching

Pass through cache for A and AAAA records with ttl 30 seconds
Only captures Pods requests
TODO:

autopath
nxdomain caching
cache dns requests or request content?

oh no

oh no

/hold