Table pagination

Hello !
First, thank you very much for this library, it helps a lot !
I tried to use the autopaging feature of the table, but it seems not to work. My table overflow from the first slide and doesn't split on multiple slides.

I used this options for the table:
  const colWidths = [0.7, 1, 1, 0.7, 0.7, 0.7, 0.4, 0.4, 0.4, 0.4, 0.4, 0.4, 0.8, 0.8];
  slide.addTable(rows, {
    x: 0,
    y: 0,
    w: colWidths.reduce((a, b) => a + b, 0),
    colW: colWidths,
    rowH: 0.3,
    autoPage: true,
    lineWeight: -1.0,
  }, defaultCellType);

Do you have any idea of what reason can lead to this ?

Hi @jmurgey ,
It looks like either that data and/or the options are the cause.  I took the code, used the colWidths array for many rows along with {} for options and it worked...
Try removing some options and let me know what you find.

Code:
var colWidths = [.3,.5,.6,.6,1,.6,.5,.5,.3,.3,.3,.75,.6,.75,.3,.3,1,.5];
var tabOpts = {x:0, y:0, w: colWidths.reduce((a,b) => a + b, 0), colW: colWidths, rowH: 0.3, rowH: 0.3, autoPage: true, lineWeight: -1.0};
var slide = pptx.addNewSlide(pptx.masters.DEFAULT_SLIDE);
slide.addTable(rows, tabOpts);

Data:
var rows = [['aaaaaaaaaaaa','aaaaaaaaa','aaaaaa Aaaaaaaaaaa','aaaaaaaaaaa Aaaaaaaaaaa','aaaa aaaaaaaaaaaaaa','aaaaaaaaa aaaaaaaaa','aaaaaaaa a aaaaa','aaaaaaaaa','aaaaaaaaa aaaaaaaaaa','aaaaaaaaaa aaaa','aaa aaaa aaaaaaaaa','aaaaaaaa','aaaaaaaa Aaaaaaaaaaaaaa','aaaaaaaa Aaaaaaaaaaaaa aaaAaa','aaaa a aaaaaaaaaaa aaa aaaaaaa Aaaaaaaa','aaaaaaaaaaa aaaaaaaaa','Aaaaaaaaaaaaaaaaaa aaaaaaa','aaaa aa aaaaaaa'],
['aa','aaaaaa','aaaa','aaaaaAa','aaaaaaaa aaaaaaa aaaa Aaaaaaaa aaaaaa','aaaaaaa','aaaa','aaaaaaaaa','aa','aa','aaA','','aaaaaa','aaaaaa Aaaaaaaaaaa','aaa','aaa','aaaaaaa aaaaaaa aaa aaa aa aaa aaaa','aaaaaaaaa'],
['aa','aaaaaa','aaaa','aaaaaAa','aaaaaaaa aaaaaaa aaaa Aaaaaaaa aaaaaa','aaaaaaa','aaaa','aaaaaaaaa','aa','aa','aaA','aaaaaaaa aaaaaaaa aaa aaaaaaaa Aaaaaaaaaaaaaa','aaaaaa','aaaaaa Aaaaaaaaaaa','aaa','aaa','aaaaaaa aaaaaaa aaaa aa aaa aaaa aaaaaaaaaaa','aaaaaaaaa'],
['aa','aaaaaa','aa aaaaaaa aaa','aaaaaAa','aaaaaaaa a aaaaaaaaaaa, aaaa aaaaaa, aaa aaaa, aa aaaa, aaaaaaa aaaaaaa..','aaaaaaAa','aaaaa','aaaaaaaaa','aaa','aaa','aaa','aaaaaaaa aaaaaaaa aaa aaaaaaaa Aaaaaaaaaaaaaa','aaaaaa','aaaaaa Aaaaaaaaaaa','aaa','aaa','aaaaaaa aaaaaaa aaaa aa aaa aaaa aaaaaaaaaaa','aaaaaaaaa'],
['aa','aaaaaa','aaa','aaaaaAa','aaa aaaaaaaa a aaaaaaaa aaaa aaaaaaaa aaa aaaa aaaaaaa aaaaaaaa','aaaaaaa','aaaa','aaaaaaaaa','aa','aaa','aaa','aaaaaaaa aaa aaa','aaaaaa','aaaaaa Aaaaaaaaaaa','aaa','aaa','aaaaaaa aaaaaaa aaaa aa aaa aaaa aaaaaaaaaaa','aaaaaaaaa'],
['aa','aaaaaa','aa aaaaa aaaaaaaaaaa aa aaaaaaaa','aaaaaAa','aa aaaaaaaa aaaaaa aaaaaaaa a aaaaaaa aaaaaa aaaaaaaaaaa','aaaaaaa','aaaa','aaaaa aaaaaa','aa','aaa','aaa','aaaaaaaa aaa aaa','aaaaaa','aaaaaa Aaaaaaaaaaa','aaa','aaa','aaaaaaa aaaaaaa aaaa aa aaa aaaa aaaaaaaaaaa','aaaaaaaaa'],
['aa','aaaaaa','aa aaaaa aaaaaaaaaaa aa aaaaaaaa','aaaaaAa','aa aaaaaaaaaaa','aaaaaaa','aaaa','aaaaaaaaa','aa','aaa','aaa','aaaaaaaa aaa aaa','aaaaaa','aaaaaa Aaaaaaaaaaa','aaa','aaa','aaaaaaa aaaaaaa aaaa aa aaa aaaa aaaaaaaaaaa','aaaaaaaaa'],
['aa','aaaaaa','aaaaaAa','aaaaaaaaaaaaa','aa aaa aaaaaa aaaa aaaaa aaaaaaa','aaaaaaaaaaaa','aaaa','aaaaaaaaa','aa','aaa','aaa','aaaaaaaa aaa aaa','aaaaaa','aaaaaa Aaaaaaaaaaa','aaa','aaa','aaaaaaa aaaaaaa aaaa aa aaa aaaa aaaaaaaaaaa','aaaaaaaaa'],
['aa','aaaaaa','aaaaaAa','aaaaaaaaa','aaaa, aaaaa, aaaaa, aaaaa','aaaaaaa','aaaa','aaaaaaaaa','aa','aaa','aaa','aaaaaaaa aaa aaa','aaaaaa','aaaaaa Aaaaaaaaaaa','aaa','aaa','aaaaaaa aaaaaaa aaaa aa aaa aaaa aaaaaaaaaaa','aaaaaaaaa'],
['aa','aaaaaa','aaaaaaaaaaaaaa','aaaaaAa','aaa aaaaaaaaa aaaaaaaaaaa','aaaaaaaaaaaa','aaaa','aaaaaaaaa','aa','aaa','aaa','aaaaaaaa aaa aaa','aaaaaa','aaaaaa Aaaaaaaaaaa','aaa','aaa','aaaaaaa aaaaaaa aaaa aa aaa aaaa aaaaaaaaaaa','aaaaaaaaa'],
['aa','aaaaaa','aaaaaaaaaaaaaa','aaaaaAa','aaa aaaaaaaaaaa aaaaaaaaaaa','aaaaaaaaaaaa','aaaa','aaaaaaaaa','aa','aaa','aaa','aaaaaaaa aaa aaa','aaaaaa','aaaaaa Aaaaaaaaaaa','aaa','aaa','aaaaaaa aaaaaaa aaaa aa aaa aaaa aaaaaaaaaaa','aaaaaaaaa']];```

I am also seeing a similar issue, both when using the table options provided in the screenshot above, and when using {} options.
It seems to be paginating the data randomly, sometimes ending early and leaving most of the slide blank, and sometimes overflowing several inches past the edge of the slide.
  These tiny columns with lots of characters are tougher to calculate and tend to result in more page spanning.  The auto-paging isn't an exact science.  IT'll always be a 90/10 solution.  Try adjusting the column widths to not be so small and try adjusting the lineWeight option.
Your code did break into 3 slides for me, but i can see where those odd columns pushed it far on the first slide:


[SPARK-25154][SQL] Support NOT IN sub-queries inside nested OR conditions

What changes were proposed in this pull request?

Currently NOT IN subqueries (predicated null aware subquery) are not allowed inside OR expressions. We currently catch this condition in checkAnalysis and throw an error.
This PR enhances the subquery rewrite to support this type of queries.
Query
SELECT * FROM s1 WHERE a > 5 or b NOT IN (SELECT c FROM s2);

Optimized Plan
== Optimized Logical Plan ==
Project [a#3, b#4]
+- Filter ((a#3 > 5) || NOT exists#7)
   +- Join ExistenceJoin(exists#7), ((b#4 = c#5) || isnull((b#4 = c#5)))
      :- HiveTableRelation `default`.`s1`, org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe, [a#3, b#4]
      +- Project [c#5]
         +- HiveTableRelation `default`.`s2`, org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe, [c#5, d#6]

This is rework from #22141.
The original author of this PR is @dilipbiswal.
Closes #22141
Why are the changes needed?

For better usability.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Added new tests in SQLQueryTestSuite, RewriteSubquerySuite and SubquerySuite.
Output from DB2 as a reference:
nested-not-db2.txt

Test build #120993 has started for PR 28158 at commit 1f30d08.

Merged build finished. Test PASSed.

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

Test build #120993 has finished for PR 28158 at commit 1f30d08.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #120996 has started for PR 28158 at commit 9b54235.

Merged build finished. Test PASSed.

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

Test build #120996 has finished for PR 28158 at commit 9b54235.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #121011 has started for PR 28158 at commit 9b54235.

Merged build finished. Test PASSed.

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

Test build #121011 has finished for PR 28158 at commit 9b54235.

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

Merged build finished. Test PASSed.

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

cc: @cloud-fan @dongjoon-hyun

Test build #121056 has started for PR 28158 at commit f65e68a.

Merged build finished. Test PASSed.

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

Test build #121063 has started for PR 28158 at commit 02f788f.

Merged build finished. Test PASSed.

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

Test build #121056 has finished for PR 28158 at commit f65e68a.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Test build #121063 has finished for PR 28158 at commit 02f788f.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Merged build finished. Test FAILed.

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

retest this please

Test build #121065 has started for PR 28158 at commit 02f788f.

Merged build finished. Test PASSed.

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

Test build #121072 has started for PR 28158 at commit be9dc0a.

Merged build finished. Test PASSed.

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

Test build #121065 has finished for PR 28158 at commit 02f788f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #121072 has finished for PR 28158 at commit be9dc0a.

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

Merged build finished. Test PASSed.

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

Thanks, @cloud-fan @dilipbiswal ! Merged to master.


Check guaranteed pod is properly scheduled into a NUMA zone

In order to test workloads are properly scheduled in the right node
taking into account the NUMA Zones resources, testing environment should
fulfill some requirements:

to have at least two working nodes.
to have at least one working node with at least more than one NUMA zone

Once those requirements are cheked, we should create a Pod with such a resource requirements that it could
be scheduled into one NUMA zone of one node but also into another node using the combined NUMA zone resources.
This way we can check the scheduler uses the resources in one NUMA zone instead the other ones.

/test all

/test all

/hold

/test all

/test all

/unhold

/retest-required

/hold

/test e2e

/retest

/test ci/prow/ci-install-e2e

/test ci-install-e2e

/lgtm
/approve
let's implement the missing parts (https://github.com/openshift-kni/numaresources-operator/pull/124#pullrequestreview-880857269) with a followup PR

/unhold

/cherry-pick release-4.10


Show mainnet results first

Didn't seem to work :/

I think this might work, but it's still, after much using GHA, unclear to me whether we run the workflow file from master/merge-base in PRs, or the modified one.
Going to merge this and find out.
If this does't work, I will investigate another method.

It works :)

It does!


Added Martin Zibricky into list of codeowners

Description
Added Martin Zibricky into list of codeowners
Fixes #177
Type of change

Admin task

Testing steps
N/A

Approved


add fio benchmark for the SSD used by es

I run the rally pmc track against my new 3 node es cluster, but the throughput seems too low. I saw the official benchmark use the SSD: Samsung MZ7LN512HMJP-00000 (
https://elasticsearch-benchmarks.elastic.co/ ). I want to know if this difference is because the SSD performance. So I think it will be good if you can provide a fio benchmark.
|   All |                       Min Throughput |                  index-append |      580.62 |  docs/s |
|   All |                    Median Throughput |                  index-append |      625.59 |  docs/s |
|   All |                       Max Throughput |                  index-append |      639.55 |  docs/s |

Thanks for the suggestion @tclh123 . Rally is primarily meant to be used as a tool for benchmarking Elasticsearch and not the underlying system itself, but we'll discuss a bit more about whether something like that makes sense.
I'd like to point out that there a few more factors that can influence the performance of the benchmarks, apart from the pure SSD performance.
Most of them, have been covered in @danielmitterdorfer 's [Seven tips for Better Elasticsearch Benchmarks[(https://www.elastic.co/blog/seven-tips-for-better-elasticsearch-benchmarks) blog article so this is a highly recommended read.
In general when throughput seems low it is important to use any of the standard methodologies to evaluate where's the bottleneck. One of the most used ones is Brendan Gregg's USE method and you can find some useful checklists he's been presenting to quickly triage CPU / IO / Network bottlenecks.
For example you can see in this discuss topic that you need to be careful about the saturation levels on the network side as well, especially with multi node clusters and 1GbE interfaces.
Also many generations of SSDs get adverse performance if they haven't been trimmed; check the is your Elasticsearch TRIMmed blog article for more information.

Thanks for your idea but at this point this is not something we want to pursue. We aim to state as much as possible about our environment but we will not provide additional hardware stress tests.

@dliappis @danielmitterdorfer Thank you.
I've checked the disk performance about those 2 SSD. It seems the IOPS of the Samsung SSD is much better than the Intel SSD I used. Any suggestions?

Samsung MZ7LN512HMJP-00000

Ran. Read
97 K IOPS (4KB, QD32)
Ran. Write
88 K IOPS (4KB, QD32)
https://www.cnet.com/products/samsung-pm871a-mz7ln512hmjp-solid-state-drive-512-gb-sata-6gb-s/specs/
https://www.samsung.com/semiconductor/ssd/client-ssd/MZ7LN512HAJQ/
https://www.samsung.com/uk/memory-storage/850-pro-sata-3-2-5-inch-ssd/MZ-7KE512BW/ (can't find MZ7LN512 page)

Intel SSDSC2BB800G6 DC S3510 800GB 2.5IN

Random Read (100% Span)
67000 IOPS
Random Write (100% Span)
15300 IOPS
https://www.cnet.com/products/intel-ssd-dc-s3510-series-800gb/specs/
https://ark.intel.com/products/86199/Intel-SSD-DC-S3510-Series-800GB-2_5in-SATA-6Gbs-16nm-MLC


Adds the support for using column keys with row ids and vice versa

Coverage remained the same at 100.0% when pulling 6307a315b38624227387fbccf297c403aa03ffc6 on yuce:orm-key-id-support into f619925970a36c849d0e46c1befa6ca226a89240 on pilosa:master.

@jaffee updated


New tiles, world art, character and ui (example)

I don't understand why github remove the AppID from PhotonServerSettings every push.

Done manually


ZodSerializerDto issue with array shape

I'm having an issue serializing the following. Nestjs throws an 500 internal server error when using @ZodSerializerDto decorator but testing the Dto with validate gives the expected result (password & registerDate is stripped).
const stuffThatNeedsToBeStripped =  [
      {
        id: 'd7197e2b-ab76-4889-8989-710b3253b6a6',
        status: 'PENDING',
        creator: {
          id: 'abb6eedf-2642-4ee9-851e-3d29e1d215d5',
          username: 'user1',
          password: 'PASSWORD',
          registerDate: '2024-02-22T09:14:14.698Z',
        },
      },
    ];

const Base = z.object({
  id: z.string(),
  status: z.string(),
  creator: z.object({ id: z.string(), username: z.string() }),
});

const ArrayOfBase = z.array(Base)

const ZodDto = createZodDto(ArrayOfBase)

Reproduciton (Codesandbox)
Thanks in advance !

In the source you can see that it has special handling for responses that are arrays:
https://github.com/risen228/nestjs-zod/blob/d24982ed89fed5156d79dcc96b78079b04037e6c/src/serializer.ts#L38-L42
Basically, if what you're returning from a controller method is an array, then it'll validate each element in that array against your provided schema.
I got confused by that behavior at first as well... In my opinion, it would probably be better if that behavior would be removed from the library.

Thanks for the reply !
Damn, I had that idea that it would work that way and I thought I tested it...
Yeah just defining the schema as below works.
const Base = z.object({
  id: z.string(),
  status: z.string(),
  creator: z.object({ id: z.string(), username: z.string() }),
});

//const ArrayOfBase = z.array(Base);

const ZodDto = createZodDto(Base);

Closing the issue and thanks !

@BenLorantfy can we reopen this and actually implement how it's supposed to work?

+1 on reopening this issue. I just stumbled on it too and was super confused because my controller correctly returned an array without the serializer, but then errors saying it got an object when I re-add the serializer...


Notification sent to Facebook Message

I am using a remote computer to do the scanning, and it is hard to access/control the computer using a cell phone. It would be great to receive the information (spawn, time left, and location/direction) through FB message. I know there is a branch about using Twitter, but still have some trouble to set it up. Can anyone help? Thanks in advance.

Facebook? You can easily do it with Telegram.

@and7ey Would you mind to tell how to realize that? Thanks


Student

i would like to use this please

hi, Joseph, thank you for your contribution! you can use this code as you like, but please fork a copy to your own GitHub account and work from there, so that this code stays the same. Thanks!

hi, Joseph, thank you for your contribution! you can use this code as you like, but please fork a copy to your own GitHub account and work from there, so that this code stays the same. Thanks!

thanks a lot


function arguments outputed in decoded form and shown as a json object with names

Would be great to have function arguments  decoded via the abi information and be shown as json object in the output:
for example for function "transfer" it would be
"decoded_inputs" : {"_to":"0xfdfdf...", "_amount":"10000"}

Great idea. I'll get back to you tomorrow.

Two things:

Can you do me a favor? Can you send me a single .json object representing one function call where you've taken the current output and physically inserted the "decoded_input"? In this way, I will be sure to give you the output you want without us having to go back and forth.

This will require a change in the binary format of the saved 'slurp.' Currently, there is no way to 'clear the cache,' so before running the next version (with the decoded_inputs field) you will have to manually 'clear the cache' yourself. You do this by removing all the *.bin files in ~/.ethslurp/slurps.

for
 //function start(string gameID, uint64 blockNumber,uint8 competitionIndex, uint32 version)

it would be something like :
{
...
"decoded_input":{
 "gameID" : "catchme",
 "blockNumber" : "123000",
 "competitionIndex" : 1,
 "version" : 63
},
...
}

for this
//function submit(string gameID, uint64 seed, uint32 score, uint32 durationRoundedDown, bytes32 proofHash)

it would be something like :
{
...
"decoded_input":{
 "gameID" : "catchme",
 "seed" : "12390415930956723389",
 "score" : 12000,
 "durationRoundedDown" : 163,
 "proofHash" :"0x8b638e78c8d9160620db7c539ae2cb24b614e2ad89136437c2c0b808d9724338"
},
...
}

In the above comment I forgot to include the function name so I came up with the following format
{
...
"decoded_call":
   {
     "name": "start",
     "input":
      {
         "gameID": "catchme",
        "blockNumber": "122"
        "competitionIndex": 1,
        "version": 2
     }
  }
...
}

Moved to https://github.com/Great-Hill-Corporation/quickBlocks/issues/71


Godot 3.2.4.beta4 does not actually use seperate textures for GLTF files exported with seperate textures enabled.

Godot version: v3.2.4.beta4.official, affects master as well.

OS/device including version: Linux Mint 20 x86_64
Issue description:

When exporting a blender scene to GLTF for use in Godot 3.2.4.beta4, with the export settings shown in the screenshot above, with textures exported into a folder instead of packed into the GLTF itself, Godot is supposed to use those textures in the resulting SpatialMaterial resources after importing.
beta4 currently fails to do this, even with separate textures enabled, it always ends up packing the textures directly into the SpatialMaterial, leaving the textures in the exported folder unused.
This does not happen in 3.2.3, so I'm guessing some regression may have happened at some point.
Steps to reproduce:

Setup a blender scene in a recent version of blender, 2.8 or 2.9.
Make sure textures are used in it. (ofcourse)
When exporting to GLTF, make sure to set it so it exports textures separately. (as shown above)
After exporting to Godot, right click the resulting SpatialMaterial(s), and click "Edit Dependencies".
Notice that there are no dependencies.

cc @fire

I hope this one can get fixed before 3.2.4.stable releases, my workflow relies heavily on this working correctly, since GLTF currently works the best out of all the export options available right now for me.

Are you able to do a bisect to find the commit that change the behaviour? I have some suspects.

@fire
For what its worth, all of the 3.2.4 betas suffer from this issue, so it has to be a commit from sometime before beta 1.

https://github.com/godotengine/godot/commit/9529ce41eb4c58a395962ea3cdd61f090717fd54#diff-391b8dac176705a7434fd9ca93ac540183658308fcb6a91cc40f6f2ecd4439baR1384 is suspicious, but I'm currently busy and in the middle of something.

On an extra note, this issue affects the 4.0 master branch as well currently, but I'm guessing this will probably be resolved with the upcoming 3D asset import rework for 4.0.

Might indeed be related to #42504, I'll have a look. It would help me out if there was a minimal reproduction project that I can use right away to debug the issue.

Might indeed be related to #42504, I'll have a look. It would help me out if there was a minimal reproduction project that I can use right away to debug the issue.

@akien-mga
mrp.zip
This includes a blender file that you can use for exporting, open it in blender, export it to GLTF using the export settings shown in OP, export it to the import folder in the MRP to reproduce.

@akien-mga
mrp.zip
This includes a blender file that you can use for exporting, open it in blender, export it to GLTF using the export settings shown in OP, export it to the import folder in the MRP to reproduce.

Thanks! I exported as described with Blender 2.91.0, and it seems to work fine for me in Godot 3.2.4 beta 4 and latest 3.2 branch:

Can you still reproduce it, and which Blender version do you use?

Thanks! I exported as described with Blender 2.91.0, and it seems to work fine for me in Godot 3.2.4 beta 4 and latest 3.2 branch:

Can you still reproduce it, and which Blender version do you use?

Please read the OP carefully, I'm not talking about the material dependencies, this issue concerns the textures that are copied over, they do not get linked to the materials, instead they packed inside of them.

Please read the OP carefully, I'm not talking about the material dependencies, this issue concerns the textures that are copied over, they do not get linked to the materials, instead they packed inside of them.

Thanks, I can confirm that. The .material files are indeed 3 MB of embedded data.

Thanks, I can confirm that. The .material files are indeed 3 MB of embedded data.

Indeed this change is wrong: https://github.com/godotengine/godot/pull/42504/files#diff-391b8dac176705a7434fd9ca93ac540183658308fcb6a91cc40f6f2ecd4439baR1322-R1338
This causes loading the external images as a byte array instead of a resource if it has a proper mimeType, which is wrong.  I'll fix it.

Indeed this change is wrong: https://github.com/godotengine/godot/pull/42504/files#diff-391b8dac176705a7434fd9ca93ac540183658308fcb6a91cc40f6f2ecd4439baR1322-R1338
This causes loading the external images as a byte array instead of a resource if it has a proper mimeType, which is wrong.  I'll fix it.

@akien-mga Thanks a bunch!

@akien-mga Thanks a bunch!


[FieldGroup] Horizontal variant doesn't control for content direction.

Description
The following CSS doesn't control for content direction:
https://github.com/adobe/spectrum-css/blob/8e6fade0bfee690d8d9a0b2d94a63884eeba4d2f/components/fieldgroup/index.css#L25-L29
Seems like the fix is to move to logical properties in the source. If there is no alternative path already being pursued, I can submit a PR.
Steps to reproduce

Go to https://opensource.adobe.com/spectrum-css/fieldgroup.html
Click on content direction = RTL
Observe the collapse of margin between the field items.

Expected behavior
Has reciprocal margin based on direction.
Screenshots

Environment

Spectrum CSS version: Latest
Browser(s) and OS(s): Chrome on macOS

Also, while I'm here, it looks like the usage of margin-inline-start on items after the first will cause unexpected wrapping behavior (note the lack of start [left] edge alignment):

If this is indeed unexpected, should I use margin-inline-end on all but the last child to avoid this?

Another question:
The styles here are applied via .spectrum-FieldGroup--horizontal or (though also and if a user is mistaken) .spectrum-FieldGroup--horizontal. Should one of these be the "default" and the other be the modifier? Based on the styles applied to .spectrum-FieldGroup, .spectrum-FieldGroup--horizontal is almost the default as it is. Is there reason why the default here doesn't supply margin?

We've been slowly moving components to logical properties in React Spectrum, https://react-spectrum.adobe.com/react-spectrum/CheckboxGroup.html these changes are making their way into Spectrum-CSS.
I think I'd actually argue against putting a margin on anything and leave that to container components. Margin's affect things outside of the component, which isn't great for reusability. This is why we now have Flex/Grid components in React Spectrum, to make this kind of thing easier and more predictable, and they accept standard Spectrum sizes using the native gap property (kinda polyfilled for flex).
https://react-spectrum.adobe.com/react-spectrum/Flex.html
In other cases, such as the Checkboxes, we actually switched to padding. This way the size is still dependable.

The styles here are applied via .spectrum-FieldGroup--horizontal or (though also and if a user is mistaken) .spectrum-FieldGroup--horizontal. Should one of these be the "default" and the other be the modifier? Based on the styles applied to .spectrum-FieldGroup, .spectrum-FieldGroup--horizontal is almost the default as it is. Is there reason why the default here doesn't supply margin?

We're starting to avoid defaults, and instead are making it so you must add classes for all variants you want to apply. This makes it so we don't have to override default values in variant classes, cleaning up the CSS and making things simpler to generate.

If this is indeed unexpected, should I use margin-inline-end on all but the last child to avoid this?

That's a good solution, we should probably go that direction instead.

In other cases, such as the Checkboxes, we actually switched to padding. This way the size is still dependable.

Applying padding to a Checkbox in checkbox groups will increase its hit area, and we actually want dead space between them in a group, so we should be using margin. Unless I'm misunderstanding?

Ah, if we want dead space in there, we should probably move our checkbox group to be in a Flex. Not what we currently have. I'll make an issue on our side if that's the case.
I just tried it out though, is it actually dead space or should that "empty" space be part of the hit area for the next checkbox (in horizontal) because that's how it currently works. The order of dom elements places each sequential checkbox above the previous one when receiving interaction events.


orderOfOperations causes error UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'sort' of undefined

Describe the bug
I added orderOfOperations to my config file, and the program errors out around line 22 of orderCollectionRequests.js
To Reproduce
Steps to reproduce the behavior:

After setting up my config file and cli file (which I will attach here) I run:
portman -t variationTests --cliOptionsFile portman/portman-cli.json
The error message I've described is displayed to me
portman copy.zip

Screenshots

Update:
The "orderOfOperations" was initially focussed on sorting requests in folders, while your collection has a great number of request on "root" level.
Based on your collection, we are working on a PR to include to the sorting on root level, next to the folder sorting.

@thim81 what would be an example of sorting requests in folders? Would it be like there were multiple openapi.yaml files spread over multiple folders?

@whorst in OpenAPI you can define "tags" to group requests. On conversion from OpenAPI to Postman, you can have to option to group them in "folders".
Have a look at https://github.com/apideck-libraries/portman/tree/main/examples/cli-filtering

By default Portman groups them by tags, since we are using the "folderStrategy": "Tags" https://github.com/apideck-libraries/portman/blob/main/postman-config.default.json
Another strategy could by "Paths": "folderStrategy": "Paths"

@whorst The PR is merged and a fix will be part of the next release of Portman.

@thim81 thank you! You guys are awesome, best of luck!

@whorst We just release Portman 1.13.1, which should contain a fix for the orderOfOperations. Please report back if the issue is not resolved.


ko-KR: Caught up with recent changes.

This should be minor updates but synchronizes ko-KR materials to the most recent en-US contents.

Thanks!


Do not emit warnings on CI

What kind of change does this PR introduce?
build related change
Did you add tests for your changes?
no
Does this PR introduce a breaking change?
no
What needs to be documented once your changes are merged?
nothing

Those warnings are never useful?

Those warnings are the deprecation notices. They are not extracted from the CI and are polluting the error logs. Tobias created a card to remove them.

The idea was to solve the problems these warnings point out, not to hide them.
More like this: #8617


[Webportal] Update k8s proxy API

close #4704

Coverage remained the same at 34.34% when pulling 80a19c67fda5503594ab8782c0480fa1c45aa64d on yiyi/webportal_update_k8s_proxy_api into 41afdc7ae8f1c11faa001e46580c01b9e9a87ff3 on master.


feat: Add to_path() method to ExplicitEnvironmentSpec

Based on feedback from @baszalmstra on discord, I added a to_path method to the ExplicitEnvironmentSpec struct so that it could be dumped to file. This would be a first step in a pixi export PR to be submitted later.
This is my first rust-based PR to a project (although I've fooled around with it a bit on my own), so feedback is very welcome. For testing, ExplicitEnvironmentSpec and ExplicitEnvironmentEntry didn't implement PartialEq, so I just unrolled the content. This isn't ideal, but I didn't want to start poking around on the main struct definitions, but I can if that's preferred.

Might also be interesting to implement this (still draft) spec completely: https://github.com/conda/ceps/pull/79

@baszalmstra -- Thank you for the quick review. I'll make the code changes later today. I did have a few questions:

Might also be interesting to implement this (still draft) spec completely: conda/ceps#79

I read through the CEP and was trying to figure out what was needed to implement this completely. The only ideas that I had were arguments to the proposed to_string method like to_string(&self, include_platform: bool, comments: Option<String>). Those some arguments would be passed to the to_path method and then would be passed down to to_string. Let me know if that sounds reasonable or if you prefer a different way or signature.
The other requirements in the CEP were:

Topologically sorted url list: This seems outside of the scope of this struct as it has no information or knowledge about package inter-dependencies
Expanding paths and no environment variables: I wasn't sure about this as the Url type in rust seems to be "absolute" and will error on relative file paths when using from_path() for example.

Please let me know what in the CEP is missing in your mind and I'll try to implement it.

I hadnt really checked what needed to happen to implement the CEP. 😅 I think most of it is related to how we parse the file not so much how we write it. In that sense I think your implementation already contains everything required.
I dont think it makes sense to add a comments argument to to_string. Users can just call the to_string method and prepend whatever if they really want to. The platform is also not really needed because you could just as well set it to None in the struct itself.

@baszalmstra -- So it sounds like what you think is the best approach is:

Add to_string(). This should take the ExplicitEnvironmentSpec and return String containing:

"# platform<EMAIL_ADDRESS>
if Some(plat), else:
<EMAIL_ADDRESS>

Modify my current to_path to use to_string(). I guess here maybe we'd still want comments: Option<String> to allow custom comments to be written to the file? Or just basically write the output of to_string() to file and leave it to the user to write their own custom function if they want any prepended comments.

Indeed. And I would indeed forgo the comments for now. If people want that they can write their custom function.

@baszalmstra -- I made the requested changes. I just wanted to note that cargo clippy complains about the to_string method:
❯ cargo clippy
    Checking rattler_conda_types v0.26.1 (.../rattler/crates/rattler_conda_types)
warning: implementation of inherent method `to_string(&self) -> String` for type `explicit_environment_spec::ExplicitEnvironmentSpec`
   --> crates/rattler_conda_types/src/explicit_environment_spec.rs:156:5
    |
156 | /     pub fn to_string(&self) -> String {
157 | |         let mut s = String::new();
158 | |
159 | |         if let Some(plat) = &self.platform {
...   |
169 | |         s
170 | |     }
    | |_____^
    |
    = help: implement trait `Display` for type `explicit_environment_spec::ExplicitEnvironmentSpec` instead
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#inherent_to_string

So maybe an alternative name? Ignore the warning? Or implement Display?. Also, I'm wondering if the to_path() is worth adding since it does so little.

If you can think of a better name be my guest. I would also be fine with suppressing this particular lint.
Even though to_path does little its a nice little utlity function. 👍

The best alternatives I could think of at this point are to_spec_string() and render_spec_string(). You're right that implementing Display doesn't seem correct since what this is outputting is a rendering of the struct to the spec, rather than a general representation of the struct. If you like this alternative, I could also change to_path() -> to_spec_path()/render_spec_path().

I think to_path is still fine and to_spec_string also sounds great!

@baszalmstra I've renamed the method so everything should be in order now.


⚠ Bofrost.es has degraded performance

In 8ef9b3a, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1724 ms

Resolved: Bofrost.es performance has improved in c0af048 after 6 minutes.


Make it easier to assemble images of other sizes

Until https://github.com/googlei18n/noto-emoji/issues/9 arrives, this will make it slightly more convenient to use this Makefile to build Twemoji, which uses 72 × 72 pixel PNGs.

@googlebot: I signed it!


Excel CreateOpenXmlReader - Embedded Image => Brings in only 1 column tables

I have a series of xlsx files that are given by a third party vendor and I import the data into a database.  I use:
Excel.IExcelDataReader excel = Excel.ExcelReaderFactory.CreateOpenXmlReader(filename);
DataSet ds = reader.AsDataSet();
This works 100% of the time for xlsx files up until I started working with these files...I opened one and removed the embedded images and it read the file just fine...Is there a setting to ignore embedded images or a way to remove them In-Code? Thanks.
I'm using Excel 4.5.dll

Could you try with the 3.0 release to see if it is fixed? If not or if it still doesn't work with 3.0 could you provide a sample so we can reproduce the issue?

Just tried to repro this using ExcelDataReader 3.0 and the old 2.1 version, but both of them work as expected when testing a quick xlsx with some data, embedded images, shapes and smart art.

Then I think we should close the issue. It can always be reopened if we get a sample that we can use to reproduce the issue.

Ok, closing. @InfOracle Please let's know (with an example) if this is still an issue


I just to to say thank you. I have been looking for it since 2020

You've very much welcome. 😄
I'll try to add some additional features when I find some time.


any possibility for wildcard?

need to require all folder named tasks

i added a generic filter function that should be out in the next release, you can use that in tandem with the recurse option to achieve this behavior


EventTypes.page sends the previous pageview to segment

export const updateCurrent = (val) => ({
  type: CURRENT_SYMBOL_UPDATE,
  payload: val,
  meta: {
    analytics: {
      eventType: EventTypes.page,
      eventPayload: {
        event: CURRENT_SYMBOL_UPDATE,
        properties: {
          path: `/quote/${val}`,
          url: window.location.hostname + `/quote/${val}`
        },
      },
    },
  },
})

This was my workaround... However, not fond of it...

Hello, same problem for me... did you find a solution ? Thank you


Support binary payloads and responses, and non-UTF8 string payloads for Python functions

closes #205 and replaces #207
Description
In addition to handling binary input as in #207, this PR also handles binary output, properly encodes returned Python dicts as compliant JSON (double quotes instead of single quotes) and can handle all text encodings. All this processing is controlled by the headers according to web standards.
Motivation and Context

[x] I have raised an issue to propose this change (required)

Which issue(s) this PR fixes
Fixes #205
How Has This Been Tested?
Deployed on an internal project that takes audio files/JSON as payload and response, running on Ubuntu and macOS.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Version change (see: Impact to existing users)

Impact to existing users
UTF8 text payloads (the only ones supported so far) are passed as before.
Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

I think this looks good, @alexellis what do you think?

I haven't had a chance to look at it yet.
It may be better just to use the existing support for binary responses in the HTTP/flask templates that I added whilst building out the LF training course on serverless
https://github.com/openfaas-incubator/python-flask-template/commit/9fb55274a5fed0840122a9eb88466d4742fd901d

Quite a few changes and output from testing. It would be good to see a sample GitHub repo where you have deployed samples from each of the scenarios that are supported in the response writing code.

@jpauwels we didn't hear back from you after providing feedback, so I'm going to assume you're no longer interested in seeing this through.
For anyone else landing here, do check out the python3-flask/http templates in the store. These are in a separate repository in this organisation.

/add label: abandoned


Background does not update unless VSCode is closed and reopened

The bootstrap-window.js file that is being used to load the background is only read when VSCode is first opened. The reload window command that is used to refresh the window no longer works because of this change.
vscode.commands.executeCommand('workbench.action.reloadWindow');

Currently to see any changes users must close and reopen VSCode for the background to update.

It looks like the issue can be fixed if we switch to using the vs/workbench/workbench.desktop.main.js instead.

I know this issue, but I've been in a busy state lately and don't have time to deal with it


Experimenting with Undo redo

The idea here is to make engine not have to worry about the internals of undo-redo (set_previous_lines).
and 2nd have an abstraction that encapsulates index handling etc, out of editor
and 3rd to enable various pluggable strategies for what to track in the undo stack.
NOTE: THIS IS NOT TESTED YET!

I have not yet taken a deeper dive into the implementation parts (so I don't grok the ugly looking type signature screaming indirection)
I would list my personal priorities for the undo implementation as follows:

How easy is it for someone not intricately familiar with reedline to add or change some editing action (e.g. new completion system, helpful editing shortcut) without subtly breaking the undo machinery?
How simple is the surface of the undo machinery if I want to debug or change some behavior with it. (meaning that the implementation it self might be complicated but not magically doing weird things)
How easy is it to make changes to what is undoable based on preferences e.g. @mzanrosso implementation tracks the non-editing moves of the cursor as well, most implementations would only move the cursor on text-changing edits.
Performance on the insert-side (especially single keypress insert/delete/cursor moves)
Overall performance when pressing undo/redo
How easy it is to completely change the implementation/ Hiding all the interactions behind abstractions for that

In generally I am all for renaming Editor::reset_olds and Editor::set_previous_lines to your more descriptive names.
I am unsure if it is fully helpful if every method is responsible for the undo tracking step (see Prio 1).

Let me try to take a stab at answering all the mentioned concerns.

ugly looking type signature screaming indirection

The type signature is ugly but what part makes you say that it is an indirection?

Is it the part where I pick a name for the type signature?
or the part where it takes UndoStack<LineBuffer> and LineBuffer as argument and somehow this could have been a method on the UndoStack?
or something else?

How easy is it for someone not intricately familiar with reedline to add or change some editing action (e.g. new completion system, helpful editing shortcut) without subtly breaking the undo machinery?

I would like the callers of Editor struct not to be bothered by the internals of Editor (including how it does undo/redo). The caller can worry about resetting the whole editor but given that the undo-redo stack is an internal detail to the Editor I don't think the caller needs to worry about it.
Assuming we can achieve this (which is what I hope we can)

it should simplify the job of the caller
(on the other hand) one will have to be familiar with the undo machinery if they introduce a core edit action in the Editor itself.

Aside on edit actions: I think these are of two types

Core/Atomic/etc (Don't really know what to call them) -> these are edit actions that we offer as single method at the end of the editor.
Composed: given the core ones we can string them together to achieve more complex ones.
I don't think there should be a lot of the core edit actions.

How simple is the surface of the undo machinery if I want to debug or change some behaviour with it. (meaning that the implementation itself might be complicated but not magically doing weird things)

The approach used in this PR has two components

UndoStack - This is essentially an abstraction so that we don't have to fiddle around with index management etc in the Editor struct and
UndoStrategy - This part looks at the old undo actions (the UndoStack) and the current LineBuffer and decides whether or not to put this item on the UndoStack.

Here I would like to pull the run_edit_commands stuff into the Editor itself (not sure about it atm). With that, we can have a choice of using the Vec<EditCommand> to take a call about undo-redo without exposing the internals of the Editor.

How easy is it to make changes to what is undoable based on preferences e.g. @mzanrosso implementation tracks the non-editing moves of the cursor as well, most implementations would only move the cursor on text-changing edits.

Given that we have access to current LineBuffer and old LineBufferss (via UndoStack) when taking this call (the function adhering to the UndoStrategy type). It should be easy to achieve.

Performance on the insert-side (especially single keypress insert/delete/cursor moves)

Given that both the pieces are pluggable (UndoStack and UndoStrategy) we can tweak performance in them. If that does not suffice (which I doubt) one consideration might be that we are doing this on a single operation if we want to avoid that then we can batch multiple operations by using some sort of a producer-consumer setup (again I doubt performance would become a concern).
Also note, if we do something like this it will probably go against point 2 (that you raised).

Overall performance when pressing undo/redo

As mentioned above we can have more efficient implementors of UndoStack trait or UndoStrategy type)

How easy it is to completely change the implementation/ Hiding all the interactions behind abstractions for that

Refer to response to points 4 and 5.

In generally I am all for renaming Editor::reset_olds and Editor::set_previous_lines to your more descriptive names.
I am unsure if it is fully helpful if every method is responsible for the undo tracking step (see Prio 1 and 3).

I agree with the point about each and every method being responsible to call the insertion to undo-stack, though I disagree that the internals should be exposed and then Reedline struct has to understand the internals of Editor.
One experiment I have wanted to do is try and pull run_edit_commands inside the editor, but I am still unsure about.

Why do we need right now to change an already working functionality ?


Macro: Monomer does not move on first click and drug

Steps to Reproduce

Add monomer to canvas
Select 'Rectangle selection tool'
Drug monomer to new position

Actual behavior
Monomer does not move. It needs to click and then drug to move monomer.
Expected behavior
Monomer moves without additional click.

Tested
https://github.com/epam/ketcher/assets/82752700/245e7583-4032-4b33-a8df-3ffcf9a69a8b


Clean up consumption of defaults

It doesn't make sense to specify defaultProps for props that will always exist at a given point in the subtree.
Will rebase once #105 lands.

Rebased and fixed the tests.


Moving par back to before plot.window

In this case:
gs <- gsplot() %>%
   points(y=c(3,1,2), x=1:3, xlim=c(0,NA),ylim=c(0,NA)) %>%
   error_bar(1:3, y=c(3,1,2), y.high=c(0.5,0.25,1), y.low=0.1) %>%
   error_bar(x=1:3, y=c(3,1,2), x.low=c(.2,NA,.2), x.high=.2, col="red",lwd=3)
gs

Would we want the horizontal NA error bar to have a vertical line, or no line?

@ldecicco-USGS I would say no line for NA

:+1:


Change Add Command button clickable area

Description
Add Command button is not entirely clickable
Replication Steps
Open overlay settings -> Commands -> Add Command
Screenshots/Video

How Has This Been Tested?

[x] ran npm run ng:lint
[ ] ran npm run format
[x] ran npm run ng:test
[ ] added unit tests
[x] manually tested

this change has some lint format changes that i believe got lost in a merge.


pydantic should be less than 2

https://github.com/pylakey/aiotdlib/blob/4c1390aad0bbb85dbb552810b5c519fd0ccb9bf0/pyproject.toml#L30C9-L30C23
I wonder the last release added a check to make sure it's smaller than version2. but requirement is asking for less than v3. i found this contradicting

The package wouldn't be installed this way. But for some simple projects it's possible to downgrade pydantic to continue using aiotdlib.
Perhaps you are right and it was not a great idea) but in the next release it will be fixed (as in main branch already)


Changes in Spectral Layout

This contribution includes a faster implementation of the spectral_layout function. The implementation replaces the eigenvalue compute with a faster MKL based method. The modification provides a  speedup of ~9x in some cases.
Important - This modification is dependent on "libmkl_rt.so" library.

While I appreciate the work, and the speedup would be helpful, I am wary of a new dependency on a binary library like this -- I also note you have an environment variable set that is hardcoded to a path that may not be the install location on some machines, and won't work at all on non-Linux machines.

While I appreciate the work, and the speedup would be helpful, I am wary of a new dependency on a binary library like this -- I also note you have an environment variable set that is hardcoded to a path that may not be the install location on some machines, and won't work at all on non-Linux machines.

Hi @lmcinnes ,
We have made the required changes , the environment variable is changed to a dynamic implementation which will work on the on Linux and non-Linux machines. However , we are facing some issues while raising a PR as some pre commit tests aren't passing. Is there a way to run these tests on local environment?


Action started failing on 06.02.2023

Describe the bug
Action shivammathur/setup-php started either failing or been working for ~40 minutes before it's finished
Version

[*] v2

Runners

[*] GitHub Hosted

Operating systems
ubuntu-20.04
PHP versions
8.0
To Reproduce
name: Debug
on:
  push:
jobs:
  phpunit-tests:
    name: PHPUnit Tests (PHP ${{ matrix.php-versions }})
    runs-on: ubuntu-20.04
    container: shivammathur/node:focal-slim
    strategy:
      fail-fast: false
      matrix:
        php-versions: ['8.0']
    steps:
      - name: Checkout
        uses: actions/checkout@v2
      - name: Setup PHP, extensions and composer with shivammathur/setup-php
        uses: shivammathur/setup-php@v2
        with:
          php-version: ${{ matrix.php-versions }}
          extensions: mbstring, intl, zip, json, pdo, imagick, bcmath, redis, apcu, gd, mysql
          tools: composer:v2
        env:
          update: true

Expected behavior
Workflow works correctly
Screenshots/Logs
Output is like:
Run shivammathur/setup-php@v2
  with:
    php-version: 8.0
    extensions: mbstring, intl, zip, json, pdo, imagick, bcmath, redis, apcu, gd, mysql
    tools: composer:v2
    ini-file: production
  env:
    update: true
/usr/bin/docker exec  9703e6ad126f2b0ecd1e5a2c45208f9ee0f61024e23c1f8da7d67b26c289275b sh -c "cat /etc/*release | grep ^ID"
/usr/bin/bash /__w/_actions/shivammathur/setup-php/v2/src/scripts/run.sh

==> Setup PHP
Error: The process '/usr/bin/bash' failed with exit code 1

And if it finishes, there is an error
✗ imagick Could not install imagick on PHP 8.0.27

That causes errors on another our steps.


Generate files that are not included

gddify generate --from ../myproject --type myblock

Generates files from other blocks (empty and does not override, just ensures)
Probably it is related to set meta and the ensure promise.

3e343e6b29348a1ba5c88391feb98d6a53c34290


Applying Image preprocessing to patches before segmentation

Hi,
I found that the images I want to segment benefit from skimage.exposure.equalize_adapthist as well as percentile normalization before Cellpose.
What would be the best way to apply that to patches in the Snakemake pipeline?

Ah now I see the clip_limit option that does exactly this 🤦

No problem, it's great to see that you found it!


Can't find version compatibility matrix for KServe

/kind bug
What steps did you take and what happened:
I am trying to find compatibility matrix for KServe + K8S + KNative.
There is this compatibility matrix:
https://kserve.github.io/website/latest/admin/serverless/serverless/#recommended-version-matrix
But there is no information how to choose KServe version itself.
Please help!

You are viewing 0.13 release docs. So it is meant for kserve 0.13.

@sivanantha321 thank you!
So the minimum supported K8S version for current KServe version (0.13) is the one that specified on that matrix (1.27)?

UPD: Ah, I see:

Kubernetes 1.22 is the minimally required version and please check the following recommended Knative, Istio versions for the corresponding Kubernetes version.

Closing this issue. Please feel free to open a new one.


Error when applying the firmware.patch

Don't know if that is important or not, but while applying
workspace % git apply firmware.patch --ignore-space-change
I got the following errors

firmware.patch:281: trailing whitespace.

firmware.patch:283: trailing whitespace.
  // ZDSecMgrPermitJoining is never called. Joining via a router would always fail now since
firmware.patch:1231: trailing whitespace.
    // Set TXRX pin to 0 in RX and high impedance in idle/TX.
firmware.patch:1249: trailing whitespace.
    // Set TXRX pin to 0 in RX/TX and high impedance in idle.
firmware.patch:1450: new blank line at EOF.
+
error: patch failed: znp_CC1352P_2_LAUNCHXL_tirtos_ticlang/znp.syscfg:125
error: znp_CC1352P_2_LAUNCHXL_tirtos_ticlang/znp.syscfg: patch does not apply

Plateform: Mac Os
Dev Stack: simplelink_cc13xx_cc26xx_sdk_6_20_00_29/examples/rtos

Did you do step 5 of https://github.com/Koenkk/Z-Stack-firmware/blob/develop/coordinator/Z-Stack_3.x.0/COMPILE.md ?

I think so.
Will redo tonight

You are probably right I didn't do it.
Thanks for taking the time to respond

No problem, if I may ask, what is your goal of the firmware compilation? If you find any possible improvements please let me know.

Sure. If you recall I asked you about the APS ACK timeout value which is 1s on your firmware, and which is 6 seconds on Zigate.
I discover that I'm having issue with battery devices when doing OTA on ZNP based firmware, but nothing on the Zigate and nothing on bellows (silicon labs).
So I wanted to do a test while putting the firmware with a timeout of 6s and see how it behaves.
For instance I don't think that making retrys, route-discovery makes a lot of sense when you are just waiting for an end device doing it's regular polling window.
I understand on this other side, that for main powered device, the approach to reduce the APS ACK to 1s has a lot of benefit

Hi tried as weel. same sdk and ccs releases, all steps and still have issues:
firmware.patch:1459: new blank line at EOF.
+
error: patch failed: znp_CC1352P_2_LAUNCHXL_tirtos_ccs/znp.syscfg:4
error: znp_CC1352P_2_LAUNCHXL_tirtos_ccs/znp.syscfg: patch does not apply
warning: znp_CC26X2R1_LAUNCHXL_tirtos_ccs/cc13x2_cc26x2.cmd has type 100644, expected 100755
warning: znp_LP_CC2652RB_tirtos_ccs/cc13x2_cc26x2.cmd has type 100644, expected 100755
i turned things in all directions ... redo and redo but still the same :-)

Hi tried as weel. same sdk and ccs releases, all steps and still have issues: firmware.patch:1459: new blank line at EOF. + error: patch failed: znp_CC1352P_2_LAUNCHXL_tirtos_ccs/znp.syscfg:4 error: znp_CC1352P_2_LAUNCHXL_tirtos_ccs/znp.syscfg: patch does not apply warning: znp_CC26X2R1_LAUNCHXL_tirtos_ccs/cc13x2_cc26x2.cmd has type 100644, expected 100755 warning: znp_LP_CC2652RB_tirtos_ccs/cc13x2_cc26x2.cmd has type 100644, expected 100755
i turned things in all directions ... redo and redo but still the same :-)

I guess you are working on a Windows machine, 755 means that the directory has the default permissions -rwxr-xr-x (represented in octal notation as 0755), while 644 means owner can read/write, group/others can read only. I had the same issue and I find out that Windows break the permission sometimes so I switched to a Linux virtual machine, the warning about 100644 is gone, but I still have the error: patch failed: znp_CC1352P_2_LAUNCHXL_tirtos_ticlang/znp.syscfg:4 error, hmm...

Used code composer patch system right click project root->team->apply patch and the it worked nicely.
Thanks

On 10 Apr 2023, at 08:52, Sheng_L @.***> wrote:
Hi tried as weel. same sdk and ccs releases, all steps and still have issues: firmware.patch:1459: new blank line at EOF. + error: patch failed: znp_CC1352P_2_LAUNCHXL_tirtos_ccs/znp.syscfg:4 error: znp_CC1352P_2_LAUNCHXL_tirtos_ccs/znp.syscfg: patch does not apply warning: znp_CC26X2R1_LAUNCHXL_tirtos_ccs/cc13x2_cc26x2.cmd has type 100644, expected 100755 warning: znp_LP_CC2652RB_tirtos_ccs/cc13x2_cc26x2.cmd has type 100644, expected 100755
i turned things in all directions ... redo and redo but still the same :-)
I guess you are working on a Windows machine, 755 means that the directory has the default permissions -rwxr-xr-x (represented in octal notation as 0755), while 644 means owner can read/write, group/others can read only. I had the same issue and I find out that Windows break the permission sometimes so I switched to a Linux virtual machine, the warning about 100644 is gone, but I still have the error: patch failed: znp_CC1352P_2_LAUNCHXL_tirtos_ticlang/znp.syscfg:4 error, hmm...
—
Reply to this email directly, view it on GitHub https://github.com/Koenkk/Z-Stack-firmware/issues/396#issuecomment-1501467994, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVXYD3EYZDPIVAHY72JEZH3XAOU4HANCNFSM6AAAAAARJJIJNQ.
You are receiving this because you commented.

I have same error，Has anyone solved this problem?
Plateform: windows
Dev Stack: simplelink_cc13xx_cc26xx_sdk_7_10_00_98/examples/rtos
warning: source/ti/zstack/boards/cc13x2_cc26x2/cc13x2_cc26x2_tirtos7_ticlang.cmd has type 100644, expected 100755
warning: source/ti/zstack/boards/cc13x2x7_cc26x2x7/cc13x2x7_cc26x2x7_tirtos7_ticlang.cmd has type 100644, expected 100755
error: patch failed: workspace/znp_CC1352P_2_LAUNCHXL_tirtos7_ticlang/znp.syscfg:4
error: workspace/znp_CC1352P_2_LAUNCHXL_tirtos7_ticlang/znp.syscfg: patch does not apply

Don't know if that is important or not, but while applying
workspace % git apply firmware.patch --ignore-space-change
I got the following errors

firmware.patch:281: trailing whitespace.

firmware.patch:283: trailing whitespace.
  // ZDSecMgrPermitJoining is never called. Joining via a router would always fail now since
firmware.patch:1231: trailing whitespace.
    // Set TXRX pin to 0 in RX and high impedance in idle/TX.
firmware.patch:1249: trailing whitespace.
    // Set TXRX pin to 0 in RX/TX and high impedance in idle.
firmware.patch:1450: new blank line at EOF.
+
error: patch failed: znp_CC1352P_2_LAUNCHXL_tirtos_ticlang/znp.syscfg:125
error: znp_CC1352P_2_LAUNCHXL_tirtos_ticlang/znp.syscfg: patch does not apply

Plateform: Mac Os Dev Stack: simplelink_cc13xx_cc26xx_sdk_6_20_00_29/examples/rtos

Did you manage to solve this problem in the end? I'm facing the same issue now


cannot import name 'IECore'

Hi! I'm running python object detection demo as follows:
python object_detection_demo_ssd_async.py -m /home/toutaudio/projects/open_model_zoo/tools/downloader/Retail/object_detection/pedestrian/hypernet-rfcn/0026/dldt/FP16/person-detection-retail-0002.xml -d MYRIAD -i cam

but getting error:
Traceback (most recent call last):
  File "object_detection_demo_ssd_async.py", line 26, in <module>
    from armv7l.openvino.inference_engine import IENetwork, IECore
ImportError: cannot import name 'IECore'

Openvino is installed and setup script source /opt/intel/openvino/bin/setupvars.sh is executed .
In another application, I'm using IENetwork and IEPlugin and it just works.

@YaroslavSchubert what is the version of OpenVINO you're using? Did you take it from https://software.intel.com/en-us/openVINO-toolkit/choose-download ?
IECore is available since OpenVINO R2, but if you take IE from https://github.com/opencv/dldt - then most likely it's still R1.
Alternatively, you can try to use demos from https://github.com/opencv/open_model_zoo/tree/2019_R1/demos/object_detection_demo_ssd_async
For the question about publication of OpenVINO R2 to Github: cc @moslex @vmatveic
Hope that it helps.

Thanks, @kchechil. Yes, I'm using Openvino R1 but demos are from R2 release.
Fixed by installing Openvino 2019 R2.


Library cards' corners covered on hover

Describe the bug
Not a bug, but a small visual issue. I noticed that the library cards on the 'Home' section which enlarges on hovering gets covered at the corners. This also holds for the 'Drives' cards. Another thing I noticed is that it happens only when 'Recent Items' are enabled, otherwise the animations are not covered.
To Reproduce
Steps to reproduce the behavior:

Make sure "Recent Items' is enabled.
Hover over any card (Library or drive)
See the corners covered.

Screenshots

Desktop (please complete the following information):

OS Version: [e.g. Windows 10 20H2 19042.610]
App version: [v<IP_ADDRESS>]

Additionally the cards are covered when the window size is very low (even when 'Recent Items' are not enabled).

When the application is run on larger window size, the corners are overlapping sometimes.

@AnugrahJames This issue should be resolved in main.


How to reduce side margins of resume body?

When going to "Print" the resume in Chrome, under the margin option, select minimal.


chore(sdk): specify versions requirements_test.txt

Description
NOTE: This was originally https://github.com/wandb/wandb/pull/8653 which got mistakenly merged into a branch that wasn't main.

Adds version constraints to requirements_test.txt. While package dependency versions should be loose, requirements files should be strict. I chose to use ~= (compatible version) rather than == (exact version) so that we don't have to update these as often, but using exact versions would also work (in which case we should set up dependabot for this file).
Using explicit versions can also help debug issues in CI or development environments.
NOTE: pyte~=0.8 would match 0.9, which is wrong, since in major version 0, minor increases may cause incompatible changes. Using pyte~=0.8.0 works like pyte >= 0.8.0, pyte < 0.9.

@SocketSecurity ignore<EMAIL_ADDRESS>(since this is a requirements file for tests that run in CI, not for production)


Minimizing Communications

@e-kayrakli (Mentioning so you are aware of a new thread)
Lack of privatization has a dire impact on not only performance, but development outright. I thought you said you distributions are automatically privatized, but I suppose I misheard you, in that the field itself is not privatized and incurs a communication cost, which was responsible in causing performance to slow to a crawl ( to the point I thought I honestly had a deadlock, until I added a println showing it was working, just very slowly). It turns out as well that config variables inside of a module cannot be accessed without a communication cost either. To fix this I have to expose the internal per-locale queue because accessing each time just is not viable. Privatization is the only way to make this viable and convenient. In fact, I'm not even sure I eliminated everything.

Furthermore, I'd like to know if its possible that performance results can vary based on how active the work queue is? I got some really good results, like the kind that makes you gleeful that you've found the perfect solution... only for it to drop back down to scales-worse-than-before-even-though-there-is-no-communication levels.

NumLocales
Op/Sec

1
1689350

2
3012000

4
6033020

8
11998700

The above is phenomenal and shows the scaling that it should be doing since each locale does it's enqueue locally (Note: This is only Enqueue, so no communication needed).
After, after attempting to get 16 locales, I waited for half an hour, started at 8 again to double check, and I get results as low as 744515... this is below the baseline! It makes no sense, but I'm assuming its the fact that its busy. Is there a way to get by CPU time?
Speaking of which, have you ever tried the chplvis tool?

I have seen some unexplainable scalablity issues with network synchronization in the past. So, it is possible that it may be something internal.
I am not well-versed in chplvis. I recommend reading the extensive documentation about it.
In it's place I use CommDiagnostics (which shares the same comm callback backbone as chplvis).
I definitely recommend either of these to understand your communication issues.

As for privatization:
class Foo {
  var domField = {1..10} dmapped Block({1..10});
  var arrField: [domField] int;
}

In this case instances of Foo will not be privatized. However, domField and arrField will be privatized. Otherwise, I would be inclined to say it is a bug or unimplemented feature. It is very ulikely, though.

In this case instances of Foo will not be privatized. However, domField and arrField will be privatized.

Hence, accesses to the fields themselves, not their contents, would incur a communication cost, right? When I ran CommDiagnostics, it shows it in the same location: DistributedQueue.chpl:48.
Each time it incurred a get, and after breaking it down into multiple assignments...
From...
descriptors[getLocaleDescriptorIndex()].localQueue.enqueue(elem);

To...
var idx = getLocaleDescriptorIndex();
var descr = descriptors[idx];
var localQueue = descr.localQueue;
localQueue.enqueue(elem);

It showed that descriptors[idx] was the culprit. Just to make sure I'm following you right by "instances of Foo will not be privatized". This penalty was a major bottleneck.
As well, its making me a bit suspicious of the bad performance of the old strictly-FIFO queue, as it was full of little communications like that (I'm thinking of bringing it back)

Sorry for not replying to this...
var descr = descriptors[idx];

Here; if the descriptors is (was) a field of type block distributed array (or any privatizable) array, the communication that you are seeing is not accessing the descriptor but this. I think that was my point. But maybe this is a moot discussion..

Hm, for the test, we won't have more than one instance of it right? What if we kept a static distribution that maintains the pointer offset to each field? It would require some compiler __primitive hacking to do so but I've already done something similar in Chapel for local pointers. Just to provide a more idealized environment (say after class instance privatization is done). What say you?

Won't solve the problem. Compiler will still read the this...
Only solution would be using a module-scope variable for descriptors instead of a field of a class. It breaks all the nice software design principles, though. One thing that may help you with that is the limited information hiding support in chapel: I think you can have private module variables. (I may be mistaken and this may be limited to module functions)

To build on the above idea...
var privatizeGuard$ = sync bool;
var privatizeDom = { 1 .. numLocales };
// Can be resized... but for now monotonically increasing (never reuse existing index)
var privatizePerQueueDom = { 1 .. 1 };
var privatizeDomMap = privatizeDom dmapped Cyclic();
var privatizeDomArr = [privatizeDomMap] [privatizePerQueueDom] ClonedFields;

Then when you add a new queue, you can add to privatizePerQueueDom and assign the pointers to the fields of interest.
I agree it breaks part of Chapel's design principles, but overall the current state of Chapel interferes with progress, this would be the ideal performance of the queues (applicable to other data structures). At least then it would be given a higher priority if we got, say, 10 - 100x performance by eliminating any and all excess communication.

I think for now, let's still stick with what Chapel has to offer. Optimizations like this should be in the last stages of the development.
 Its accessible outside DSI?

Nevermind, requires DSI. I do see some interesting snippets of code inside of it though, I'll make sure to study it for other optimizations.


Extend the Bookstore with Custom Code

Tutorial URL: https://developers.sap.com/tutorials/cp-cap-java-custom-logic.html
Please specify the step you are referring to
2.2 Decrease stock when posting order
Hi Max! First off, forgive me for sharing feedback here but,  awesome10x mission so far!!! Valuable notes and pointers since the 1st tutorial (and this is 'Beginner'?!... anxious and excited for the next levels!)   :D
Anyway, I'm stuck at this step, the OrdersService.java is not saving correctly , hence failing in the next step for build/test (screenshot below).
Also, maybe related, the 'complete OrdersService.java' code seems different compared to if its built step by step from step1?  ( I see some 'cds.gen' imports)
Looking forward to completing this, thanks again!
p.s. let me know if it'll help if i upload / link to a github repo
-ch

hey @starxcx,
thanks for your feedback! Let me go through the steps once again and double-check the code pieces ;-)
Uploading it as a repo is always helpful! anyway, the OrdersService class is generated based on the services CDS file. could you trigger the build of this class with maven once again and check if the error is gone? (otherwise its indeed a problem with the corresponding import statements.
mvn clean install

by the way, some more resources where you can get information about CAP (and especially Java):
https://www.youtube.com/c/MaxStreifeneder - my livestream and replay with a focus on Java
https://www.youtube.com/user/qmacro99 - @qmacro livestream and replays with a general focus on SAP Cloud Platform, CAP (Node.js), Cloud Foundry, Workflow, OAuth, etc.
https://github.com/sapmentors/cap-community - the SAP community for CAP with a dedicated Slack Channel, etc.
I will come back with an update with regards to my code snippets I've put in the tutorial!
Cheers Max

Thanks for the quick feedback Max! I was only able to get back to this now -- I tried the mvn clean install and it worked (no more error)!
It happened again after a couple of steps at step 2.6 so I did the clean install command again, and it worked again.  I wonder why it needs a build after those specific steps...
Anyway, thanks again! Yup, I'm subscribed to both yours and DJ's channels :) Was able to catch I think it was your premiere or 2nd livestream. Planning on watching everything (finally) this lockdown =)) Also I thought it would be a good idea to go through this mission at the same time with the openSAP course ongoing now.
Cheers and stay safe!

Glad I could help! I'm in investigation with the Business Application Studio folks why that happens...
Cheers and let me know if you need any further help :-)


How to force VirtualizeSwipeableView to re-render its children?

Use case:

A (VirtualizeSwipeableView) displaying dates
B (SwipeableView) displaying months' names
the A is currently displaying 28 (with the following pre-rendered dates: [28, 1, 2, 3, 4])
the B is currently displaying February

If a user now swipes the B to point to March (which has 31 days), the A will not re-render its children, since its index (28) has not changed.
While it is generally OK, in this particular case I need to somehow let the A know that its children must be re-rendered to reflect that after the item 28 should go items 29, 30, and 31.
The question: how do I do that properly?
Your Environment

Tech
Version

react-swipeable-views
0.12.12

React
16.2.0

platform
Google Chrome 64.0.3282.119 (Official Build) (64-bit)

I join this issue as well

The parent component of A and B can handle the shared stated. This state can be used as the index prop of both A and B.

If somebody wants to put together a codesandbox example we can play with some ways to make this work. Closing until then.


Add climate-cooperation-competition

Insert URLs to the project here:
https://github.com/mila-iqia/climate-cooperation-competition
In one sentence, explain what the project is about:
AI for Global Climate Cooperation: Modeling Global Climate Negotiations, Agreements, and Long-Term Cooperation in RICE-N. ai4climatecoop.org

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

I have to say that I am really fascinated by this project.


Bump version to v0.4.2.

Bump version to v0.4.2.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


[UI] Padding issue in the select language section

The padding should be equal on both the sides of the text in the select language section.
Currently the left padding is more than the right padding.
Screenshot :

would you like to work on this issue?

@Byte-Sultan can I work on this issue?

Hi [@krao301],
Thank you so much for your interest in contributing! 😊 i'll be thrilled to have you work on the issue. If you have any questions or need clarification on any aspect of the task, feel free to ask. Don't forget to fork the repository, create a new branch for this enhancement, and follow the steps outlined in the issue description.
Looking forward to your contributions, and thanks again for being a part of our project!
Best,
[Byte Sultan]

Hi [@krao301],
Thank you so much for your interest in contributing! 😊 i'll be thrilled to have you work on the issue. If you have any questions or need clarification on any aspect of the task, feel free to ask. Don't forget to fork the repository, create a new branch for this enhancement, and follow the steps outlined in the issue description.
Looking forward to your contributions, and thanks again for being a part of our project!
Best,
[Byte Sultan]


ERROR Pl keeps stopping when I try to install an apk

Pl running using shizuku
Android 13
Coloros13
Oppo a74

Hello
I'm also facing this issue, it was fine until the latest release.
I'm attaching a Log
PI.zip

Thanks for the response, issue resolved


"AddSpanCountToRoot" field reports wrong span count due to including span annotations

Versions

Go: 1.19.3
Refinery: 1.18.0

Steps to reproduce

In the Refinery configuration, set the new "AddSpanCountToRoot" field (as of #532) to true.
Publish several traces that flow through Refinery to Honeycomb, including at least one span event on some of the trace spans.
Inspect the root span of one of these traces in a Honeycomb query.
Observe that the "meta.span_count" field is present on the root span and populated with a positive integer.
Observe that the "meta.span_count" field's count is greater than the number of spans in the trace, and is instead the sum of the trace count and the span event count.

Additional context
I reported this problem first in the "discuss-refinery" channel of the "Honeycomb Pollinators" Slack workspace.
In file types/events.go's (*types.Trace).SpanCount method, the current implementation reports the length of the "spans" field's slice:
// SpanCount gets the number of spans currently in this trace as int64
func (t *Trace) SpanCount() int64 {
	return int64(len(t.spans))
}

A slower but more accurate way to report this count is as follows:
// SpanCount reports the count of spans currently in this trace.
func (t *Trace) SpanCount() int64 {
	var count int64
	for _, s := range t.spans {
		if annoType, ok := s.data["meta.annotation_type"]; !ok || annoType != "span_event" {
			count++
		}
	}
	return count
}

Alternately, if you don't mind using the zero value returned for a key that's not present in the map:
// SpanCount reports the count of spans currently in this trace.
func (t *Trace) SpanCount() int64 {
	var count int64
	for _, s := range t.spans {
		if s.data["meta.annotation_type"] != "span_event" {
			count++
		}
	}
	return count
}

Are there other annotation types that we need to omit? Perhaps "link?"

@seh thanks for the issue and the code.
The main driver of this feature was to annotate traces with span counts in a way that Refinery's rules could evaluate them, so that someone could, for example, use a higher sample rate on "large" traces. If we exclude span events from that, it actually breaks that particular use case.
Because of this, the fix is a little more subtle than you've outlined here, and I need to do a bit of work and testing to minimize its impact and do it in a backwards-compatible way. So it's in progress, but will be a while before a fix appears.

Further discussion in this thread on Pollinators.

As noted above, our existing meta value is badly named. We call it meta.span_count but it's actually the count of all the children of the root, including spans and span events.
We should actually have several values:

meta.spanevent_count -- the number of span events on the trace
meta.spanlink_count -- the number of span links on the trace
meta.span_count -- the number of spans on the trace
meta.event_count -- the number of honeycomb events on the trace (the sum of the 3 preceding values)

In order to do this, we need to allow old configurations to continue to use meta.span_count and to opt into the new names.
There are 2 ways to do this that I've thought of:

Bump the config file version number and add individual config values for each of these fields; if version is 2, then we use the old definition, but in version 3 we interpret the new way.
Add a new, single config value called AddSpanCountsToRoot, with validation allowing only one of the two to be used, and if the plural form is used we use the new definitions. This wouldn't have individual control over all these values, but I don't think that matters very much.

Personally I prefer the second option, but I'd love to hear other thoughts.

I like the second option except for that it took me multiple re-readings to notice which plural form (Counts) you were referring to! Is there a more intention revealing name?

I am partial to the first option, wherein there would be a new struct field that offered individual options for adding any of the four counts, all defaulting to omitting them. That way, if I only care about one of the counts, I won't wind up with all four just to get that one.

We're leaning  toward the second option right now, so that we can do it in a backwards-compatible way.

To better differentiate between the new config and the existing AddSpanCountToRoot, we are thinking to use AddCountsToRoot as the new config name

Since enabling AddCountsToRoot will essentially overwrite AddSpanCountToRoot, we have decided to NOT add validation logic in the config layer to make sure only one of these two configs are set.  Instead, whenever AddCountsToRoot is set to true, Refinery will just ignore AddSpanCountToRoot.


The links in header are not working.

1 issue - Even though the UI looks good but the links in the header are not working, it goes to a 404 error page.
2 issue - I searched Spiderman, but it didn't fetched the images of him. So it may be that the api used doesn't contain the images of some superheros.
3 issue- The hover button of the search button is grayish and not going with the website.
4  issue-  As soon as the user enter the name of the superhero and press the enter button the website does not search the hero until the search button is clicked. I would like to add the enter key search functionality to the search.
Please assign this issues to me under HacktoberFest 2023 so that I can start working on it.

You can start working on these issues @PiyushK79

Issue 1 - navbar is now properly liked with about and contributors page.

Issue 1 - The navbar is now properly liked with the About and Contributors page.
You can work on issues 2,3 and 4.

Alright!!


VectorTable's behaviour against null metadata

Passing None as metadata results in the column containing the literal string null (a valid JSON).
Either allow null metadata (currently the idea is to have an empty dict at least) or forbid it (e.g. normalizing to {} when writing).

Fixed on dev


[WIP] Added get_targets_for_ base class methods

Turns on RBAC filtering by default at the base class level for all allowed_ methods_ in the CloudManager class.
https://bugzilla.redhat.com/show_bug.cgi?id=1514237
Note: Tests coming at the provider level where needed
Steps for Testing/QA [Optional]

Add tags to a user for use on a Cloud provider*
Login in as that user and Provision an Instance
Only Tagged objects should appear in the dropdowns

There are specific providers that have already overridden the Base Class methods and in some cases the overridden methods do not yet support RBAC filtering.  e.g. Amazon and the allowed_floating_ips method.

@miq-bot assign @gmcculloug
@miq-bot add_label providers, bz, blocker, wip

@miq-bot add_label bug

@miq-bot remove_label wip


OpenFisca Variables need renaming

[ ] Update in FE
[ ] update in BE

do this at the same time as #85

On hold. We'll just add this year's numbers to the old version of the rules.
And completed this story later


Nonhardcode python location in dev_tools/env_tools.py

Fixes: https://github.com/quantumlib/Cirq/issues/1270
@Ashalynd  Please review.

Hm, we should also check that it doesn't refer to Python 2 interpreter executable.

@Ashalynd sys.executable uses the PATH variable. It refers to the interpreter that runs it; so it won't refer to a Python2 interpreter unless you are using a Python2 interpreter to run env_tools.py


Example > CodeSystem ^url property is wrong.

// @Name: Local Code Systems

// Define a local code system
CodeSystem: LymphNodeCS
Id:         lymph-node-cs
Title:     "Lymph node code system"
Description: "Lymph node terms that could not be found in standard sources such as SNOMED CT."
// For information about creating a code system in the `terminology.hl7.org` namespace, see
// this documentation on the UTG process: https://confluence.hl7.org/display/VMAH/Vocabulary+Maintenance+at+HL7
// You can also create a URL local to your Implementation Guide like `http://hl7.org/fhir/uv/your-ig/CodeSystem/lymphnodes`
// The "Terminology" stream on chat.fhir.org (free registration required) is a good place to ask
// questions about how to handle this for your specific use case (direct link: https://bit.ly/terminology-fhir-chat)
* ^url =  http://terminology.hl7.org/CodeSystem/lymphnodes
// Spacing layout over three lines per term is optional, for clarity
// The definition (second text string) is optional
* #LN_AX_L1
    "Level 1 Axillary Lymph Nodes"
    "Level 1 is the bottom level, below the lower edge of the pectoralis minor muscle."
* #LN_Ax_L2
    "Level 2 Axillary Lymph Nodes"
    "Level II is lying underneath the pectoralis minor muscle."
* #LN_Ax_L3
    "Level 3 Axillary Lymph Nodes"
    "Level III is above the pectoralis minor muscle."
* #LN_Neck_IA
    "Level IA (Submental) neck node"
    "Lymph nodes within the triangular boundary of the anterior belly of the digastric muscles and the hyoid bone"
* #LN_Neck_IB
    "Level IB (Submandibular) neck node"
    "Lymph nodes within the boundaries of the anterior and posterior bellies of the digastric muscles, the stylohyoid muscle, and the body of the mandible"
// truncated (you get the idea)

Result
error Cannot find definition for Instance: http://terminology.hl7.org/CodeSystem/lymphnodes. Skipping rule.
Line: 13

Reporting an issue to the wrong repository

Thanks. I see you've posted it over at https://github.com/FSHSchool/FSHOnline-Examples/issues/25. We'll handle it from there.


When is 3.0 going to be officially released?

It has been a while since 3.0 beta was released but it is still in beta..
I want to know the timeline for the official release.

He @chuganzy I really want to get 3.0 out the door, but we need a bit more time and we can't promise a date :(

Got it. Is it due to some issues remaining or internal stuffs inside Pinterest?

@chuganzy mainly that someone needs to take the time to make sure the release is in good shape :) I.e. make sure Swift support is working correctly, Carthage support, Readme and documentation are up to date, that type of thing.

Got it. Thank you! Closing.


Update demo project

2/2 main <- #2897 <- this

Not sure what is going on, cannot recreate the problem locally. Will come back to it later today.


Execute failed: Item could not be found on GPCRDB server

The GPCRDB Structure-ligand interactions is returning not found error.
As reported in https://forum.knime.com/t/3d-e-chem-gpcrdb-item-could-not-be-found-on-gpcrdb-server/22040

The nodes are using http://gprcrdb.org which seems to operate differently.


After clicking cancel submission nothing happens.

After converting my video from .mov to .mp4 it came out a little blurry and I need to resubmit it. I've spent several hours trying to cancel but the box becomes red, it spins, and nothing happens. I restarted my computer, used 3 different browsers, used my wife's pc, also tried iphone, ipad, etc. and I cannnot get it to cancel. I have more than enough ETH in my connected wallet. Please help can you manually cancel so I can resubmit with clear mp4 video? I'm so worried I'll lose my deposit.
https://app.proofofhumanity.id/profile/0x9c2177026a9c0396EF0D209d892EecE513a3eAe3

@AnthonyWSmith we can't manually cancel it. It can only be done from the wallet you used to submit your profile. Are you using MetaMask? If you have pending transactions on your wallet make sure to clear them.

Thanks for the response! I already have confirmation on my MetaMask that it went through though?
Anthony

On May 2, 2021, at 10:31 AM, fnanni-0 @.***> wrote:

@AnthonyWSmith we can't manually cancel it. It can only be done from the wallet you used to submit your profile. Are you using MetaMask? If you have pending transactions on your wallet make sure to clear them.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@AnthonyWSmith what do you mean? I don't see any pending transaction on etherscan yet. Did MetaMask popped up with the transaction and the Reject/Confirm buttons?

Thanks for your patience as I just started using MetaMask. But yes the ETH for the proof of humanity deposit is confirmed as withdrawn from my MetaMask (I attached the screenshot in previous reply). So I’m not sure what to do at this point in order to resubmit video and not lose my deposit?
I cannot cancel submission it just spins and spins on different computers and on different browsers?
Anthony

On May 2, 2021, at 10:45 AM, fnanni-0 @.***> wrote:

@AnthonyWSmith what do you mean? I don't see any pending transaction on etherscan yet. Did MetaMask popped up with the transaction and the Reject/Confirm buttons?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

You have to cancel your submission before uploading again. It's odd that it doesn't work in so many different settings. Sometimes it takes a few seconds (or even minutes) before MetaMask pops up with the transaction request. As I said, double check in your wallet activity that you don't have pending transactions. You might also want to try reseting your MetaMask account (beware that you will need your seed phrase to log in later if you do this).

Closing this due to inactivity. Please reopen if this is reproducible.


Hide scale bar when distance > max distance.

As described in this ticket, scale bar in iOS will be hidden  when current distance  is lagger than the max distance(400 miles or 800km). We will add this logic to scale bar plugin.

Currently we have expanded the max distance, no need to hide scale bar any longer.


Android: unable to resolve content:// URL

Hi, I try to open content:// paths on Android that I receive as a share target.
With an input of content://com.android.providers.downloads.documents/document/raw%3A%2Fstorage%2Femulated%2F0%2FDownload%2F1607722645567.jpg  I get the following error:
 E/MethodChannel#flutter_absolute_path(21745): Failed to handle method call
E/MethodChannel#flutter_absolute_path(21745): java.lang.NumberFormatException: For input string: "raw:/storage/emulated/0/Download/1607722645567.jpg"
E/MethodChannel#flutter_absolute_path(21745): 	at java.lang.Long.parseLong(Long.java:594)
E/MethodChannel#flutter_absolute_path(21745): 	at java.lang.Long.valueOf(Long.java:808)
E/MethodChannel#flutter_absolute_path(21745): 	at com.kasem.flutter_absolute_path.FileDirectory.getAbsolutePath(FileDirectory.kt:47)
E/MethodChannel#flutter_absolute_path(21745): 	at com.kasem.flutter_absolute_path.FlutterAbsolutePathPlugin.onMethodCall(FlutterAbsolutePathPlugin.kt:42)
E/MethodChannel#flutter_absolute_path(21745): 	at io.flutter.plugin.common.MethodChannel$IncomingMethodCallHandler.onMessage(MethodChannel.java:233)
E/MethodChannel#flutter_absolute_path(21745): 	at io.flutter.embedding.engine.dart.DartMessenger.handleMessageFromDart(DartMessenger.java:85)
E/MethodChannel#flutter_absolute_path(21745): 	at io.flutter.embedding.engine.FlutterJNI.handlePlatformMessage(FlutterJNI.java:692)

Sorry, this actually works on a real device, but not on the emulator. Not sure if this is a valid bug - keeping it open for now.

Sorry, this actually works on a real device, but not on the emulator. Not sure if this is a valid bug - keeping it open for now.


docs: update njs build instructions

Update the Enabling njs steps to include changing branches to the latest supported version

njs has moved to GitHub

Seconding @ac000's suggestion of removing mentions of mercurial given the mailing list announcement he pointed to.

removed the mercurial steps
changed the git steps to
 $ git clone https://github.com/nginx/njs.git
 $ cd njs
 $ git checkout -b 0.8.2 0.8.2


Option to make the search bar static

I'm not really fond of the disappearing search bar, could that be made static ?

I'll add a checkbox for it in the settings modal

👍


Use livestream timestamp to show when livestream started in various areas

[ ] tiles/lists
[ ] content page
[ ] live component

This was done.


.parse() Typescript linter accepts anything (string, number...)

Hi,
I didn't find someone having my issue so I'm probably doing something wrong but... I can't figure the solution out.
Let's say I have:
const TestSchema = z.object({
  email: z.string(),
});

TestSchema.parse('hello');

The last line will not throw an error with the Typescript whereas when hovering the parse function I can see:
(method) ZodType<{ email: string; }, ZodObjectDef<{ email: ZodString; }, "strip", ZodTypeAny>, { email: string; }>.parse(data: unknown, params?: Partial<z.ParseParams> | undefined): {
    email: string;
}

Note that I use strict: true in my tsconfig.json (and at runtime, zod throws an error obviously).
A "workaround" would be to always do this:
const TestSchema = z.object({
  email: z.string(),
});
type TestSchemaType = z.infer<typeof TestSchema>;

TestSchema.parse(<TestSchemaType>'hello');

... but it's pretty verbose and annoying.
Any idea what's the issue to get the linter working normally?
Thank you,
EDIT: it seems a solution would be to use
const typedParse = <Schema extends z.ZodSchema> (
  schema: Schema, data: z.infer<Schema>
): z.infer<Schema> => schema.parse( data )

typedParse(ATest, 'hello');

(ref: https://github.com/colinhacks/zod/issues/1748#issuecomment-1364572126)
... but I don't get why this is not used by default?

Any idea what's the issue to get the linter working normally?

What do you mean by 'normally'?

but I don't get why this is not used by default?

https://github.com/colinhacks/zod/issues/1748#issuecomment-1364573095

const typedParse = <Schema extends z.ZodSchema> (
  schema: Schema, data: z.infer<Schema>
): z.infer<Schema> => schema.parse( data )

typedParse(ATest, 'hello');

Another way you can solve this problem:
type TypedParseSchema<Schema extends z.ZodSchema> = Omit<Schema, 'parse'> & {
    parse ( data: z.infer<Schema> ): z.infer<Schema>
}
const useTypedParse = <Schema extends z.ZodSchema>
    ( schema: Schema ): TypedParseSchema<Schema> => schema

const schemaWithTypedParse = useTypedParse( z.object( {
    email: z.string(),
} ) )

schemaWithTypedParse.parse( 'hello' )
//                          ^^^^^^^
// Argument of type 'string' is not assignable to parameter of type '{ email: string; }'.

Would this work for you?


[FLINK-31894][runtime] ExceptionHistory and REST API failure label integration

https://issues.apache.org/jira/browse/FLINK-31894

Add support for failure labels on JobExceptionsInfoWithHistory
Extend JobExceptionsHandler to support failure label filters
Extend JobExceptionsHandlerTest to make sure JobExceptionsInfoWithHistory responses contain labels and filters are properly applied

The biggest question I have is whether the filters are supposed to be ANDed or ORed 🤔 Right now if a single filter matches we keep the exception.

The biggest question I have is whether the filters are supposed to be ANDed or ORed 🤔 Right now if a single filter matches we keep the exception.

Changed this to ANDed filter returning Exceptions with all provided filters -- users can always get all Exception by avoiding the filter parameter.

👍 good job; let's just get rid of the separate FailureLabel class as discussed offline

Thanks much @dmvk ! Updated :)


v11: Issue with parking restrictions

From Steam user on v11 alpha:

Dying 19 Jul @ 11:47pm
Old parking restrictions (set with LABS) works fine after loading with V11A, but new ones seems to be ignored: https://steamcommunity.com/sharedfiles/filedetails/?id=1809224714

Apparently parking restrictions set via v11 are being ignored, but those set by LABS were still working fine.

I noticed that too. The signs are set, but the cars just stay where they are parked.

Just had this from another v11 user on Steam:

benzoll 15 hours ago
Parking prohibited in a street works: you have to wait for vehicles to go or eliminate them, nobody goes back to park.

I'm not sure what to do in this instance; if we just randomly moved them somewhere else, the parked cars might end up in a place that owners can not access (there is a longstanding nightmare issue with ditched/abandoned cars).

Bobbo662 5 hours ago
Just noticed the No Parking isn't work - here's a screen shot.
https://steamcommunity.com/profiles/76561198030920207/screenshot/789737583030397107


Refactor commit_message to allow custom prompts

Hello,
Thank you for open sourcing this project!
This PR allows the user to pass a custom --prompt via the command line.

FYI The newline characters and spaces in the prompt are tokenized and sent to GPT:
                        default="""Write a git commit message for the following. The message
                                starts with a one-line summary of 60 characters, followed by a
                                blank line, followed by a longer but concise description of the
                                change."""

This default prompt has a newline character and a bunch of spaces at the beginning of each line. If this is of concern to you @rtwfroody, I can submit another PR to fix it.

Good point about the tokens. I kind of assumed that any amount of whitespace is just a single token, but it sounds like that's not the case. That would be a nice thing to fix as well.

I kind of assumed that any amount of whitespace is just a single token

Using the openai/tiktoken library (which is a dependency of gpt-commit-msg I've been able to confirm that different amounts of whitespace result in different tokens, and newlines are treated differently from spaces.
I'll follow up with another PR.


Remove default resources from migrationOptions

resourceTypes can't be override from environments

@microsoft-github-policy-service agree


include previous changes and add "Getting set-up" to guide-es.md

@oscarfelipe Awesome, thank you very much!
:four_leaf_clover: :grinning: