The current Tags and the hierarchy may be not aligned.

This is related to the PR https://github.com/BrickSchema/Brick/pull/65
I created a script to test if the tags inference is correct or not. For example, if there is an instance associated with Temperature and Sensor, a reasoner should be able to infer that this is a Temperature_Sensor, Sensor, and Point, but nothing else. Temperature_Sensor should not an instance of Temperature, for example. This is an important characteristic of Brick 1.1.0 feature as we argue that we formally validated all the tags and classes.
To do that, I instantiate entities (named as class_name + _0) and associate them with the Tags defined as restrictions for the classes. Then, I run owlrl reasoner and query what classes are inferred for the instances. A naive query to get all the superclasses are used to extract "true parents', which are compared to the inferred parents.
Here's my current result:

Inference result: https://drive.google.com/open?id=1lrBiCQS3O8RnoGaaVI8Spzbq3_wgDEGs
Test result: https://drive.google.com/open?id=18Wi-eYnBM51DCs4u793CK8RbLpCnfddZ

Inference result is a ttl file and the test result is JSON file comparing incorrect inferences.
Here's an example inside Test result:
    "https://brickschema.org/schema/1.1.0/Brick#Run_Enable_Status_0": {
      "inferred_parents": [
        "https://brickschema.org/schema/1.1.0/Brick#Run_Status",
        "https://brickschema.org/schema/1.1.0/Brick#Status",
        "https://brickschema.org/schema/1.1.0/Brick#Enable_Status",
        "https://brickschema.org/schema/1.1.0/Brick#Start_Stop_Status",
        "https://brickschema.org/schema/1.1.0/Brick#Point",
        "https://brickschema.org/schema/1.1.0/Brick#Class",
        "https://brickschema.org/schema/1.1.0/Brick#Run_Enable_Status"
      ],
      "true_parents": [
        "https://brickschema.org/schema/1.1.0/Brick#Status",
        "https://brickschema.org/schema/1.1.0/Brick#Enable_Status",
        "https://brickschema.org/schema/1.1.0/Brick#Point",
        "https://brickschema.org/schema/1.1.0/Brick#Class",
        "https://brickschema.org/schema/1.1.0/Brick#Run_Enable_Status"
      ]
    },

Start_Stop_Status and Run_Status are incorrectly inferred in this case (false positive). There are two possibilities (analyzing Run_Status as an example):

The restrictions of the class are not correctly defined. Either there should be more Tags or less Tags.
The hierarchy is incorrect. There should be an rdfs:subClassOfrelationship between Run_Enable_Status and Run_Status.

Actually, there could be a worse scenario: distinguishing two classes with Tags is not possible.
Currently, there are 177 classes like this. In any case, this test result should be investigated further and I would like to learn others' thoughts.

Distinguishing two classes with tags is absolutely possible, but it requires making sure the tags are coherently applied.
If you look at the definition of Run_Status, it is defined as being equivalent to two tags (https://github.com/BrickSchema/Brick/blob/master/status.py#L229), and thus it inherits the super class Start_Stop_Status. This should be better articulated so that each subtree of tags is unique, but it is far from a fundamental issue with the structure. We could probably remedy some of the subclass relationships; the existing hierarchy is carried forward from Brick v1 I believe, which explains the inconsistencies.
Its too bad that the inference/test results are private google drive documents. Can you put them up as Github gists instead so they are easier to work with?
Also, it would be good if we can isolate the specific classes and tags that are causing a mismatch between the expected parent classes and the inferred parent classes

I didn't use Gist because the TTL file was too large. The browser hangs. The JSON file turns out to be fine so I edited the link above. (https://gist.github.com/jbkoh/438a58d2cfb250654f2b6e15431a8a55)
I understand the structure itself has no problem with the inconsistency but the human authors are, and the testing framework should help them.
The V1 hierarchy did not assume tags subsumptions and that's why it was fine (of course there should have been obvious bugs anyway.)
I think we have to manually go through the 177 classes before we release the version.


Foreign Key constraint error: No parent table defined for foreign key

Describe the bug
When trying to create a foreign key constraint on a relation using the parent and parent_columns constraint attributes, I am getting an error at run time: No parent table defined for foreign key which is followed by a trace.
Steps To Reproduce
I have configured the model's schema like so:
  - name: example__child_table
    constraints:
      - type: primary_key
        name: pk_example__child_table
        columns: ["id"]
      - type: not_null
        columns: ["id", "name", "parent_id"]
      - type: foreign_key
        name: fk_example__child_table_1
        columns: ["parent_id"]
        parent: parent_table
        parent_columns: ["id"]
    columns:
      - name: id
      - name: name
      - name: parent_id

When running the project this results in the error message described above.
Expected behavior
The foreign key sql should be generated, and assume all else is good (e.g., the parent_table has a primary key) the foreign key should be created.
Screenshots and log output
If applicable, add screenshots or log output to help explain your problem.
System information
The output of dbt --version:
Running with dbt=1.8.6
Registered adapter: databricks=1.8.5

The operating system you're using:
Running as a dbt task in a Databricks workflow.
Compute: Single node: Standard_DS3_v2 · DBR: 14.3.x-photon-scala2.12
Additional context
The traditional expression method of creating a foreign key works as documented, but I would like to avoid using it because it has a hard coded schema name. The parent + parent_columns method assumes that the parent table is in the same schema as the child, which for me is true.

Will need to investigate why this isn't working...looking at the macro def, it looks as though it should.  Do you mind sharing the dbt.log with me via email<EMAIL_ADDRESS>

Ah, found the issue.  Thanks for reporting.

@PtrckGdwn are there any earlier versions where this does work?  I'm trying to narrow down whether a.) this never worked or b.) it used to work but dbt Labs changed how constraint config is validated.

@benc-db I don't know if there are any earlier versions that worked. I suspect the issue may be due to the introduction of some support for a similar method of adding fk constraints that was added to dbt core recently (though apparently will not be operational until v1.9 - see: https://github.com/dbt-labs/dbt-core/issues/8062#issuecomment-2316231542).
One thing to note though about the dbt core change is that I believe it will add support for ref('my_model') against the the 'to' attribute. This would be a awesome thing to have flow through to dbt-databricks, because at present FKs are not part of the dag and you need to use explicit depends_on statements in models to ensure they build in the correct order.


Propose to add @obecny as the CODEOWNERS.

Which problem is this PR solving?

@obecny has showed strong interest and commitment in the OpenTelemetry Web SDKs and Plugins. Currently, he is the main contributor for Web SDKs. We really appreciate his work! Thank you for all the contributions.

Notable PRs #334, #461, #477, #433, #455, #447, #435, #438.

Involved in SIG meetings.

Codecov Report

Merging #505 into master will increase coverage by 4.87%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #505      +/-   ##
==========================================
+ Coverage    91.6%   96.48%   +4.87%
==========================================
  Files         124      112      -12
  Lines        5864     5156     -708
  Branches      551      478      -73
==========================================
- Hits         5372     4975     -397
+ Misses        492      181     -311

Impacted Files
Coverage Δ

...ckages/opentelemetry-core/src/common/NoopLogger.ts
33.33% <0%> (-16.67%)
:arrow_down:

...ges/opentelemetry-tracing/src/NoopSpanProcessor.ts
66.66% <0%> (-8.34%)
:arrow_down:

...metry-core/src/trace/instrumentation/BasePlugin.ts
80.55% <0%> (-5.56%)
:arrow_down:

...core/src/context/propagation/BinaryTraceContext.ts
97.5% <0%> (-0.84%)
:arrow_down:

...opentelemetry-base/test/resources/resource.test.ts
100% <0%> (ø)
:arrow_up:

packages/opentelemetry-tracing/src/config.ts
100% <0%> (ø)
:arrow_up:

packages/opentelemetry-tracing/src/utility.ts
100% <0%> (ø)
:arrow_up:

...core/src/context/propagation/NoopHttpTextFormat.ts
100% <0%> (ø)
:arrow_up:

...telemetry-plugin-grpc/test/utils/assertionUtils.ts
100% <0%> (ø)
:arrow_up:

packages/opentelemetry-tracing/test/Span.test.ts
100% <0%> (ø)
:arrow_up:

... and 57 more

You probably need to add them to the list of approvers and in the approver team

Sure, done.


Pre-commit passes a list of files to lint but csvlint can only accept one

Expected Behaviour
When running csvlint in a pre-commit hook pre-commit will pass a list of files to it as seperate args. csvlint should take the list and generate a response for all. cfn-lint has an example of this support https://github.com/aws-cloudformation/cfn-lint#basic-usage
Current Behaviour (for problems)
csvlint errors:
ERROR: "csvlint validate" was called with arguments ["file1.csv", "file2.csv"]
Usage: "csvlint myfile.csv OR csvlint http://example.com/myfile.csv"
Steps to Reproduce (for problems)
Use a pre-commit config like:
---
default_language_version:
  ruby: 2.7.8
repos:
- repo: https://github.com/Data-Liberation-Front/csvlint.rb
  rev: v1.3.0
  hooks:
    - id: csvlint
      name: lint files/*.csv
      files: 'file.*\.csv'
      pass_filenames: true

against files like:
file1.csv
file2.csv
file3.csv

Current workaround is to use xargs to run csvlint against each file:
    - id: csvlint
      name: lint files/*.csv
      files: 'file.*\.csv'
      entry: bash -xc 'echo "$@" | xargs -n 1 -I filename csvlint filename --schema=schema/csv' --

Had a look at the usage, we sure this is expected behavior we'd want here @jamesmelville-gds?
Seems like the expected usage here is one linting pass per CSV.
I imagine it'd be easy to get CSV lint to parse an array of inputs here and go through them in batch...
but then wondering if that breaks the model, are linting responses now to be grouped around an offending file, or clumped into "you've got these problems in that lump". 🤔
Not sure what @Floppy thinks, but perhaps us using bash-fu to enumerate all our CSVs and lint each separately is OK?


New API Counts Orphan Transactions as Part of Balance

Could you provide more info? eg. do you know what keys add the orphan balance?

Fixed as of https://github.com/zcoinofficial/zcoin/commit/937d3bba31799a78553a0a801e70b0492768fe23


增加oracle,mssql,pgsql的Tables方法

增加oracle,mssql,pgsql的Tables方法

增加oracle,mssql,pgsql的Tables方法

Codecov Report

Merging #529 into master will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #529      +/-   ##
==========================================
- Coverage   71.52%   71.44%   -0.08%
==========================================
  Files         297      297
  Lines       25385    25415      +30
==========================================
+ Hits        18156    18158       +2
- Misses       6258     6287      +29
+ Partials      971      970       -1

Impacted Files
Coverage Δ

os/gcache/gcache_mem_cache.go
97.39% <0.00%> (+1.04%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #529 into master will decrease coverage by 2.56%.
The diff coverage is 69.83%.

@@            Coverage Diff             @@
##           master     #529      +/-   ##
==========================================
- Coverage      74%   71.44%   -2.57%
==========================================
  Files         247      297      +50
  Lines       21235    25415    +4180
==========================================
+ Hits        15716    18158    +2442
- Misses       4791     6287    +1496
- Partials      728      970     +242

Impacted Files
Coverage Δ

encoding/gparser/gparser_api_encoding.go
28.57% <ø> (-60.32%)
:arrow_down:

errors/gerror/gerror_error.go
67.53% <ø> (-6.76%)
:arrow_down:

database/gredis/gredis_config.go
36.36% <ø> (+1.98%)
:arrow_up:

container/gmap/gmap_tree_map.go
100% <ø> (ø)
:arrow_up:

encoding/gbinary/gbinary_le.go
82.84% <ø> (ø)
:arrow_up:

encoding/gjson/gjson_api_new_load.go
78.4% <ø> (ø)
:arrow_up:

encoding/gparser/gparser_api_new_load.go
44.44% <ø> (+6.94%)
:arrow_up:

database/gredis/gredis.go
86.36% <ø> (+2.43%)
:arrow_up:

encoding/gbase64/gbase64.go
51.92% <ø> (-48.08%)
:arrow_down:

encoding/gjson/gjson_implements.go
81.81% <ø> (ø)

... and 247 more

Continue to review full report at Codecov.

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


Changing the final layers of an sm.UNet

Hi people,
you created a wonderful repository!
Could you please help me apply this modification? I just want to have a Unet, with ResNet34 architecture, pre-trained with ImageNet and where the final layer of the classifcation is splitted into 3 instead of just 1 layer.
Thank you in advance.

I know you're looking to do multi-task learning, but regarding multiple outputs and multiple loss functions here's a tutorial for constructing a network.

Thank you very much. I will have a look at it!


Make canvas_minimal example rerender on refresh

I'm not sure if this actually works, because the canvas_minimal example doesn't run on my computer.

This breaks on macOS because winit doesn't send an initial Refresh event there.

Hmm, I can't find anything that looks like a "window shown" event. I think I'll move the rendering code again, and then use boolean to indicate that the screen needs to be redrawn.

@bors-servo: r+

:pushpin: Commit 263dc0a has been approved by pcwalton

:hourglass: Testing commit 263dc0ab15cbf2230622c7388fec57aee034dd6c with merge 953a1dab607546ea5fa04d5358f1fd17fd12ca9a...

:sunny: Test successful - checks-travis
Approved by: pcwalton
Pushing 953a1dab607546ea5fa04d5358f1fd17fd12ca9a to master...


Fix output type for LinearLayer

LinearLayer previously used only the input vector to determine the output type, but it should use both the input vector and the type of the weight vector.

This closes #69

as a general rule, look at the combination of all types involved and use that to compute the output type. That's what promote_type is for.


Local variable 'r' referenced before assignment

Getting this error for all invoice calls to mint deployed with defaults, replicated from minibits and cashu.me

Hi @Kodylow is this bug still present? If so, could you say which Lightning backend this uses and get the logs with DEBUG=TRUE and LOG_LEVEL=TRACE set?

This was using lnbits deployed on replit, tore it down a while ago I can try it again later today to replicate and see if it's still there


Cannot Create keybinding to restart a specific Task

VSCode Version: 1.17.0-insider
OS Version: Windows 10

Steps to Reproduce:

Add these keybindings

[
	{"key":"f6","command":"workbench.action.tasks.runTask", "args":"run"},
	{"key":"ctrl+f6","command":"workbench.action.tasks.restartTask", "args":"run"}
]

Type f6
My run task begins
Type Ctrl + f6
The task selection screen pops up

Reproduces without extensions: Yes
It seems like he arg behavior in runTask code https://github.com/Microsoft/vscode/blob/c8eb003402ce5d6e0529a4da459b863ac5610af9/src/vs/workbench/parts/tasks/electron-browser/task.contribution.ts#L1975
Needs to be replicated in runRestartTaskCommand
https://github.com/Microsoft/vscode/blob/c8eb003402ce5d6e0529a4da459b863ac5610af9/src/vs/workbench/parts/tasks/electron-browser/task.contribution.ts#L2163
The same issue also exists with runTerminateCommand

Is this issue available to work on? It would be my first time contributing, so I may need some guidance.

I have a question about what the terminate command should do. Right now if the user is in the terminal, it asks for a task to terminate, otherwise it terminates all tasks.
But that's different from what restartTask command does right now, when in the terminal, it ask for a task to restart, but otherwise it restarts the first task.
I have changed the restart behaviour to restart the task passed as argument when found, or ask for one to restart, whether we are in the terminal or not. Should the terminate command do the same?

@vemoo thanks for looking into this. If the tasks a not executed in the terminal then they are executed in the old output runner which can only execute one task at a time. This is why the code is different for the non terminal case.
To answer your question: if the task can't be found you should show a picker to run it. If in non terminal no task name is required since there can only be one task.

I have implemented the fixes and it wasn't necesary to check whether the task system is the old one or not. Because the getActiveTasks method is implemented for both, and for the ProcessTaskSystem it returns one or no tasks.
I've also found and fixed a couple of bugs in processTaskSystem.ts when terminating the task.
Should I create a pull request?

@vemoo yes please.

To verify create a key binding like
{
    "key": "ctrl+shift+w",
    "command": "workbench.action.tasks.restartTask",
    "args": {
        "type": "npm",
        "script": "compile"
    }
}

When using the above example ensure that the package.json contains a compile script (can be a simple echo)

start the compile task
press the keybinding from above
ensure the task restarts.


UseXmlDocumentation is ignored with Schema.Create

It appears when using Schema.Create that the option UseXmlDocumentation is not respected, using SchemaBuilder.New however it is.
An example test shows the documentation being created:
[Fact]
public void CreateObjectTypeWithXmlDocumentation_IgnoreXmlDocs_SchemaCreate()
{
    // arrange
    // act
    ISchema schema = Schema.Create(c =>
    {
        c.RegisterQueryType<QueryWithDocumentation>();
        c.Options.UseXmlDocumentation = false;
    });

    // assert
    schema.ToString().MatchSnapshot();
}

gives the following snapshot
schema {
  query: QueryWithDocumentation
}

"Query Documentation."
type QueryWithDocumentation {
  "Foo Documentation-"
  foo("Bar Documentation." bar: String): String
}

"The `String` scalar type represents textual data, represented as UTF-8 character sequences. The String type is most often used by GraphQL to represent free-form human-readable text."
scalar String

This sounds like a bug :)


Kotlin compiler plugin to optimize composables similar to R8

Things to do:

Replace calls to isTraceInProgress() with false
Remove calls to sourceInformation, sourceInformationMarkerStart, and sourceInformationMarkerEnd

These functions are the source of a lot of bloat in the JS file and they are not needed in production.

Filed https://issuetracker.google.com/issues/306428078 upstream since there seems to be some support for this being a first-party toggle of the Compose compiler itself.

Realizing that even with toggles this won't fully eliminate these calls since we don't control the flag value in libraries (such as the Compose runtime itself, where they're enabled). So still need to build a solution that also applies somehow to libraries, not just the code in the compilation unit.

We can do this during linking with a regular compiler plugin because linking sees all the IR, not just the compilation unit of the current module.
https://kotlinlang.slack.com/archives/C7L3JB43G/p1698130170671859?thread_ts=1698116255.957259&cid=C7L3JB43G

Might be worth stripping all calls functions in Preconditions.kt since the nullability for our code is already enforced at compile time and those functions are called very frequently. This is already possible via compiler flag for the JVM and will eventually be supported for JS.


Icon not showing ( expo 39.0.2 )

🐛 Bug Report
Icon are not showing both on expo web ( using react native web ) or on device test ( with expo )
To Reproduce
Create clean project with expo / typescript blank.
UI Kitten and Eva version

Package
Version

@eva-design/eva
^5.0.0

@ui-kitten/components
^5.0.0

expo
~39.0.2

react-native-svg
12.0.1

I have also linked with IconRegister
import React from 'react';
import { ApplicationProvider, Layout, Text , IconRegistry} from '@ui-kitten/components';
import * as eva from '@eva-design/eva';
import AppNavigator from './navigation.js'
import { EvaIconsPack } from '@ui-kitten/eva-icons';

export default () => (
  <>
    <IconRegistry icons={EvaIconsPack} />
    <ApplicationProvider {...eva} theme={eva.dark}>
      <AppNavigator />
    </ApplicationProvider>
  </>
);

Closed because the error was on my code


Add viewport meta tag

Fixes #10
This commit adds a viewport tag to the  portion of the website.
According to my research, setting the initial-scale = 0.86 & the minimum-scale=0.86 will supress the zoom feature that phones have and scale it up to make it viewable on the device.
Let me know what you think @jrobind
Cool project by the way!

@jrobind Just changed it. Check it out 😃


cmd/govim: balloon quickly disappear in terminal but not gvim

Reported as https://github.com/vim/vim/issues/4273

cc @hmarr

For people following along here, please also note the plan to ultimately move away from balloons in https://github.com/myitcv/govim/issues/283

Noting here for search purposes, that some people refer to the balloon that pops up as a popup.


Look different when using game tick add instead of animationgetframe

They should look the same
https://jsfiddle.net/w2hf1856/
https://jsfiddle.net/w2hf1856/1/

PIXI tickers don't run in milliseconds or seconds, but a target frame delta (delta of 1 = 60 frames per second). The code that Emitters use when connecting to PIXI tickers is delta / PIXI.settings.TARGET_FPMS / 1000 to get a value in seconds.


aluco, blisters

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[a,b]

/trunk merge


Update Slider version to the latest

This pull request updates the Slider version to 4.3.0.
This is to ensure we are using the latest and test against issues introduced potentially with or along with new architecture shipped.

The build for Android does not pass due to know issue that is currently fixed, but not yet released.
This will be fixed by updating to 4.3.1 once it's available.


CLI: Make namespace a command option instead of global option

Author: Ruslan Aksenov
Summary of the feature being proposed
Move namespace option from global options to command options
before:
tctl --namespace test1 workflow run --input "{}"

after
tctl workflow run --namespace test1 --input "{}"

What value does this feature bring to Temporal?
Simplifies the usage of command line by having options under the same command and help of that command. As a user i won't have to find how to specify the namespace as well since i will immediately see --namespace option with the rest of the options.
Makes semantics of the commands a bit nicer by positioning options after the command, not before
Are you willing to implement this feature yourself?
Yes

merging several CLI UX proposals into a single https://github.com/temporalio/proposals/issues/11


remove unnecessary bibliographic info on summary confirmation page.

fixed in commit https://github.com/birkin/easyrequest_project/commit/2d45f2519103794169b2475542e81b3fe8f275a9.


Official 0.3.0 release with TensorFlow 2.4.1 support

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
Hi,
We are migrating our library to use this native Java library for TensorFlow 2.x (thank you so much for the work you have done here so far!): https://github.com/JohnSnowLabs/spark-nlp/pull/2324
We have some models from TF Hub which require SentencePiece (Universal Sentence Encoder for instance). I have successfully managed to migrate them to use tensorflow-text via loadLibrary. Unfortunately, tensorflow-text doesn't support Windows below TensorFlow 2.4.1.
I was wondering since the master is now using TensorFlow 2.4.1 with the auto-build for 0.3.0-SNAPSHOT, is there an ETA on when it will be released officially and move SNAPSHOT to 0.4.0?
System information

TensorFlow version (you are using): 2.3.1
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state.
Current 0.2.0 released on Maven is compatible with TensorFlow 2.3.1, however, tensorflow-text starts supporting Windows from 2.4.1: https://github.com/tensorflow/text/releases/tag/v2.4.1
The issue with SentencePiece and tensorflow-text has been mentioned here a few times. I have successfully managed to load tensorflow-text for macOS and Linux, but I am very interested to see if the Windows support works in both tensorflow-text and TensorFlow Java.
Will this change the current api? How?
No
Who will benefit with this feature?
Any model on TF Hub that requires SentencePiece/tensorflow-text will be compatible with Windows machines.
Any Other info.

Hi @maziyarpanahi ,
The code base for 0.3.0, that includes support for TF 2.4.1, is indeed ready (or almost ready) to be released. But we are having issues with our build system right now (mostly hosted on GitHub Actions), preventing us to produce the artifacts for that release.
We are actively working to solve definitely this issue, since the stability of our build platforms has always been one of our greatest challenge. We'll keep you in touch, thank you for your patience!
CC\ @joanafilipa

Hi @karllessard
Thanks for your quick response, I appreciate it.
I'll watch this issue for any new developments and wish you guys the best of luck.

Yes please please please make a release, we're really waiting for it since November (https://github.com/tensorflow/java/pull/138)

I attempted to use the 0.3.0 version on top of 0.2.0 project and there are a lot of API changes. The readme in Master does not refer to this. Is there a migration guide?

Yes please please please make a release, we're really waiting for it since November (#138)

Well, good news! We did good progress to fix our various build issues and should be able to make a release soon, I'll provide more details later, probably today.

I attempted to use the 0.3.0 version on top of 0.2.0 project and there are a lot of API changes. The readme in Master does not refer to this. Is there a migration guide?

The only breaking change that I'm aware of is that Tensor class is no longer parameterized and now Tensor<TFloat32> can be simply replaced by TFloat32, as an example. It does make sense though to guide our users to understand what this change means and how to migrate easily between both versions, I'll make sure to write such instructions before releasing, thank you @nebulorum

So after discussing with the SIG, I’m planning to make a release of 0.3.0 from the actual snapshot by March 15th.
Only detail is that the Windows -mkl-gpu platform will be missing for that release since TF 2.4.1 does not seem to support that configuration natively. All other platforms will be available.

@karllessard BTW, maybe we can now build for the same list of compute capabilities as the binaries on PyPI?
https://github.com/tensorflow/java/blob/master/tensorflow-core/tensorflow-core-api/build.sh#L23

We can certainly give it a try, can you send me the list of capabilities or even better, make a PR for it so we'll see if the build passes? I'll apply the CI Build label to it

See #200.  I think the official is 3.5, 5.0, 6.0, 7.0, 7.5, 8.0 and higher than 8.0, not sure how to specify 8.0+.  I was never able to find an exact build command anywhere.  There was also a request to include 3.7 for K80, but that seems like it was supported by the official releases so maybe 3.5 covers it?

I've just added the same capabilities than the official TF release, which includes all of the above. We don't build anymore for Cuda 10.1 though, @saudet is that right/ok?

Probably, I'm not aware of the core team ever supporting more that a single version of CUDA.

TensorFlow Java 0.3.0 has been officially released today, enjoy! ;)


ListView:  using both linkPropertyName & selectionMode

Category
[ ] Enhancement
[ ] Bug
[ x ] Question
Version
"@pnp/spfx-controls-react": "2.0.0",
Expected / Desired Behavior / Question
I want to use selectionMode option and still be able to use linkPropertyName to open file.
Observed Behavior
I have my listView set up with selectionMode=Single
And I have linkPropertyName set so that I see a link (in my example I just put it on the Modified column).
What happens is if I click on what looks like the link, it will open my pnp panel... this part of my selection function.
Is there a reference on how I can check to see what field I'm looking at and then determine if it's a link and open the file instead?
Thanks!

I was able to cobble this code up for the "selection" function but I'm wondering if there is a simpler or more elegant solution I'm missing:
In this case I have to chase down the target props and search for an href to see if it's a link.
It works but it's always takes a lot of digging  into the event props to find a way to test it.
let isLink = e.srcElement && e.srcElement.href && e.srcElement.href.length > 0 ? true : false;
    private _onShowPanel = (item): void => {

        let e: any = event;
        console.log('_onShowPanel: e',e);
        console.log('_onShowPanel item clicked:',item);

        // This is the place where I am testing to see if a child node has an href...
        let isLink = e.srcElement && e.srcElement.href && e.srcElement.href.length > 0 ? true : false;

        if ( isLink === true ) {
            window.open(e.srcElement.href, '_blank');

        } else {
              //Code to set state and open panel here
        }
     }

Hi @mikezimm!
If you want to handle the click I would recommend to override render method for the link column.
In that case you can provide your onClick handler.
In that handler you can open your file and additionally preventDefault and stopPropagation to avoid selection.
I would also ask you to create a separate enhancement issue to add onClick property for the link column type.


[OR-1415] change the predeploy contract name legacy_erc20_ETH -> legacy_ecr20_native_token

deploy legacy_erc20_native_token (I think it will be needed to our dapp developers)
change the name of contracts
update names in tokamak/core-utils (@0x6e616d sdk is required to update,, 🥰 )

I checked it works. Thanks!


Added elevations & fixed SIDs

Added elevations for all airports, excluding American airports.
SIDs fixed in Abu Dhabi and Manchester.
A few other fixes also included, such as removal of some duplicate waypoints.

Merged via 2f25b2d.


Why i using the res101 to train the faster-rcnn  on my own dataset so slow?

I  have tested the voc2007 using the Resnet101  pre-trained on 07+12 set, for a single image, i need nearly 10S for detection, is this reasonable？ (I finally get the 78 map)
And when i use res101 to train my own data, (i only use 150 images as the training set)  speed is 38S/iter. Is this reasonable?

are u using cpu version of tf?

i install the tensorflow(version1.2) for gpu. And when i train my network,i use the command 'nvidia-smi' ,and i find the GPU actually works.  I do not change any other codes except the classes in the pascal_voc.py and iters in the train_faster_rcnn.sh and also test_faster_rcnn for my own dataset.


Database errors in metrics

Now the convention only has success metrics
https://github.com/open-telemetry/semantic-conventions/blob/main/docs/database/database-metrics.md
How about adding a label about the error or new metric?
The error can occur both when opening a connection and when executing a request (command)

Now that we have generic error.type attribute, we should be able to address it.

Does error.type make sense for any of the existing metrics? It seems like no. I do think error.type makes sense for the client operation duration metric proposed in #512.

Yes, that's what I need


Fixed out of order logging definition

If Cython can't be imported, the attempt to log will fail because the log object hasn't been defined yet.

Good catch @greggg230 - thx!


This is fantastic! Do you accept donations?

Hello!
You just happened to save me a lot of time! I can't find any way to donate on the web site. Is that possible?
Teodor

Sorry, I'm not ready to accept donation.
There are some reasons, and an important point is that I'm living Japan. The personally transfer is prohibited by law, and I have no idea the correct way to recieve donation from user.
In additon, Marp is already towarding to Marp Next managed by @marp-team. Marp is supported by some OSS contributors, not only me. So donation to me is unfair for community.

Thanks for your reply. Regardless, I really appreciate the effort you and the other Marp contributors put in. I think you've found some kind of sweet spot for making simple, beautilful presentations. Keep up the good work!
Marp Next looks interesting, and seems like a good next step.

UPDATE: I have stood in a line of GitHub Sponsors to accept your donation. It would support payouts in Japan according to a blog article. Please wait I get the sponsor!

UPDATE: Started Patreon! GitHub Sponsors think that would take a long time to accept small maintainer like me.

Hey!
I signed up for Patreon so I could support you a little. Since I was just interested in a one-time donation, I unsubscribed afterwards. Hope it helps!
Best regards, Teodor


Core: remove global style references

Clean up global style references and any style APIs that would cause breaking changes after 3.0

[x] remove all component margins
[x] remove global styles for 3.0 (typography/layout)
[x] remove use of global css props internally with core components
[x] remove global prefix convention on host scoped styles

Margins
Ensure all core components have no specified :host margins. Margins on a host element make assumptions of the layout that is external to their responsibility. Using a design token system designers and developers should be able to layout components consistently and with specified intent and constraint. The design token values are still a wip but we want to remove margins before 3.0 to prevent future visual breaking changes.
Core will need to handle design tokens and variables slightly different than clr-ui/angular due to shadow dom and component composition but we should align the space values for the token to stay consistent.
Related issues/context:
https://github.com/vmware/clarity/issues/181
https://github.com/vmware/clarity/issues/4116
https://github.com/vmware/clarity/issues/3303
Layout & Typography
Both the layout/grid utilities and typography also suffer from inconsistent layout margins (not typographic whitespace) and spacing. For 3.0 we will omit these from the core global styles until they can be addressed. Also defining non global explicit utils will allow possible migration strategies with scoped utils.
Global CSS Properties
Core Components should not reference global CSS custom properties as this creates a secondary way to theme components and creates twice the API surface area. Our clr-ui dark theme follows a similar pattern and only changes the local component scoped variables leaving global values unchanged. For core components to share values such as spacing and color they internally should use private sass variables and not global CSS Custom Properties.
Global Prefixing
Public API surface for component styling should not use the global prefixing convention. This is necessary for micro frontends where a component may need to be aliased due to app embedding. This prevents version collisions and dependencies form having to share the same global values.
cwc-badge {
  --color: red;
  --background: blue;
}

Teams can prefix their own core components to share and embed with other products without the risk of a host app breaking their product specific styles.
<style>
  cwc-badge {
   --color: green;
  }
</style>
<cwc-badge></cwc-badge> <!-- only host application badges are changed -->

<product-cwc-badge></product-cwc-badge> <!-- embedded product styles are preserved -->

In very rare cases a naming collision can occur with a local host var and global theming var breaking on the component (see example).
// global scope
:root {
  --primary-color: blue;
}

// component scope (:host)
button {
  // naming colision, due to global and local names
  // var cant resolve to global or local but refers to
  // its own redeclartion defaulting to initial.
  --primary-color: var(--primary-color);
  background: var(--primary-color);
}

Fixed with https://github.com/vmware/clarity/pull/4243


Unit Tests are broken

When I clone the repo fresh and try to run the unit tests, Async_CompressWithGzip_returnGzippedVersion and CompressWithGzip_returnGzippedVersion both fail.
Message:
Assert.Equal() Failure
↓ (pos 26)
Expected: ···SE64####H4sIAAAAAAAAE3y9W5Mcx7E0+FfG9PwVrO+XfRsBQwkSARKCCNnuf···
Actual:   ···SE64####H4sIAAAAAAAACoS92ZJk13Et+CtlfL4Bi3noNw6gKIqDJEiidfd9O···
↑ (pos 26)

No worries, thanks!


OKHttp propagate underlying cause.

When OkHttp body emission is interrupted the underlying cause is reported as suppressed exception.
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1614181&view=ms.vss-test-web.build-test-results-tab&runId=32168706&resultId=100494&paneView=debug
This has broken several tests in storage that expect com.azure.core.exception.UnexpectedLengthException
But were given:
Expected exception of type 'com.azure.core.exception.UnexpectedLengthException', but got 'reactor.core.Exceptions$ReactiveException'
	at org.spockframework.lang.SpecInternals.checkExceptionThrown(SpecInternals.java:81)
	at org.spockframework.lang.SpecInternals.thrownImpl(SpecInternals.java:68)
	at com.azure.storage.blob.specialized.BlockBlobAPITest.Upload illegal argument(BlockBlobAPITest.groovy:955)
Caused by: reactor.core.Exceptions$ReactiveException: java.io.IOException: canceled due to com.azure.core.exception.UnexpectedLengthException: Request body emitted 7 bytes, more than the expected 6 bytes.
	at reactor.core.Exceptions.propagate(Exceptions.java:392)

Additionally, not unwrapping the failure caused retry policies to keep retrying (they saw IOException)

/azp run java - core - tests

API change check
API changes are not detected in this pull request.

/check-enforcer override


difference with rust-ffmpeg?

hey i'm curious what is the difference with rust-ffmpeg:
https://github.com/zmwangx/rust-ffmpeg
? I'm using rust-ffmpeg atm in https://github.com/louis030195/screen-pipe but might be open to change because of the download feature etc. quite pain to package ffmpeg for deployment
what do you think are pros and cons?

Hey Louis,
The biggest difference is that rust-ffmpeg statically links against FFmpeg's source code library and libav directly. That means a lot of FFI, extern "C" stuff and whatnot. Compiling your program becomes a lot more involved, because you need an entire C toolchain in addition to what's required by Rust itself. I tried going down this route on Windows, and it was such a headache that it was one of the main motivations behind this crate.
Under the hood, ffmpeg-sidecar does everything through the FFmpeg CLI binary, the same way you might do it from the command line. You can think of it as just a wrapper around Rust std::Command and a bunch of helper functions for processing I/O. That means you can just ship with standard FFmpeg binaries if you want (or even have your users download them at runtime from an official source). As a bonus, since you aren't linking with FFmpeg's licensed source code, you can avoid a lot of the licensing restrictions that apply to the GPL-licensed parts of FFmpeg.

How does the performance compare? Given that data is transferred via stdin/stdout.

How does the performance compare? Given that data is transferred via stdin/stdout.

Good question, the short answer is that since IO is not the bottleneck, performance is not impacted by using stdout to transfer data.
The slowest part of the pipeline will be video encoding/decoding in 99% of cases; just a very very computationally expensive task. Stdio channels are built to handle data coming through at filesystem read/write speeds, and I've never encountered a situation where I can decode video faster than stdout can pipe it.
The biggest speed boost comes from decoding video on the GPU, and the FFmpeg CLI makes that extremely easy: just slap a -hwaccel auto on a command.
I do think it would be suitable for desktop streaming because in that use case both network bandwidth and video encoding speed would be a limiting factor before IO performance would.

cool thanks @nathanbabcock
i noticed someone forked ffmpeg-sidecar and use IPC for performance, do you recommend this?
https://github.com/CapSoftware/Cap/blob/63719073805f1fd040343ae27366272a29918078/apps/desktop/src-tauri/src/utils.rs#L47

Very interesting, I imagine it works very well!

How does the performance compare? Given that data is transferred via stdin/stdout.

Good question, the short answer is that since IO is not the bottleneck, performance is not impacted by using stdout to transfer data.
The slowest part of the pipeline will be video encoding/decoding in 99% of cases; just a very very computationally expensive task. Stdio channels are built to handle data coming through at filesystem read/write speeds, and I've never encountered a situation where I can decode video faster than stdout can pipe it.
The biggest speed boost comes from decoding video on the GPU, and the FFmpeg CLI makes that extremely easy: just slap a -hwaccel auto on a command.
I do think it would be suitable for desktop streaming because in that use case both network bandwidth and video encoding speed would be a limiting factor before IO performance would.

Gotcha, this is very helpful.
The examples that I see only seem to have the ability to give the input as file path? Any way for us to implement streams where we can continuously send RGBA data and receive H.264 frames (and vice versa).

/// To take input from stdin, use the value `-` or `pipe:0`.
pub fn input<S: AsRef<str>>(&mut self, path_or_url: S) -> &mut Self {
}

Just looked at the code, dropping it here in case anyone needs it as well.


Update Wireshark to 2.5.1

Bump version, sha, and appcast: checkpoint

After making all changes to the cask:

[X] brew cask audit --download {{cask_file}} is error-free.
[X] brew cask style --fix {{cask_file}} reports no offenses.
[X] The commit message includes the cask’s name and version.

2.5.1 is a development release
https://www.wireshark.org/#download


Ensure bundling retains order of edges

Turned out to be a trivial fix, simply sorting by the index after performing the merge of the node and edge dataframes. Retaining this ordering is important if we want to use the edges for anything other than datashading.

Looks good to me...


Archery Categories Expanded

Updated to adjust the archery category structure and add some additional archery categories that are necessary.
At Editions.dev I spoke to someone about Shopify working with some merchants in niche industries to help expand the categories and was hoping to make contact with someone there to discuss the archery niche since we are the largest merchant in the industry. Please let me know if that is possible.

I have signed the CLA!

/generate_dist

/generate_dist

@sayscraig First, thank you for your incredible contributions. We're currently working on integrating these suggestions to ensure consistency across our taxonomy. Please bear with us as we implement these changes in upcoming updates! In the meantime, do you have any links we could take a look at to support these changes? I'd also be more than happy to schedule a chat if you have a few minutes to spare 😄

@ricardotejedorsanz I would love to chat, that would be fantastic. What is the best way to connect?
Here are a couple links as well.
Filters show archery styles: https://lancasterarchery.com/collections/arrows
Menu on the website will also show the breakdown of categories into the categories I have added like mechanical/fixed blade broadheads, expanded Bow & Crossbow accessories categories, and all the others.

Hi @sayscraig, thanks again for chatting with me the other day!
Suggesting a few changes following our discussion:

New attributes for Archery target types instead of categories
ID changes for consistency within our tree
Created a Broadhead type attribute instead of categories to avoid going as deep as L8 in the tree (hope this is alright!)

Please let me know if you have any comments, and bear with us while we work on making these changes available in our next taxonomy release.

fyi @elsom25 @tylerrowsell @KComrade53 placeholder commit with a few tweaks, pending changes to the ./integrations files for the next release. Also pending new attribute descriptions + translations.

I have signed the CLA!

@ricardotejedorsanz those updates sounds great to me. I agree with the broadhead type as well, that was a good call.

@sayscraig at this point I'd say its on us — selfishly I want to use this to test a few "tools" we're adding to the repo as well to see if they help with these kinds of things.
It will certainly be addressed before our next version cutoff

@elsom25 Hi just checking in since I assume we are getting close to the release of these changes. Hoping to confirm that the plan is still for this PR to get merged and released soon?

Moving to https://github.com/Shopify/product-taxonomy/pull/391 to override merging issue.


Looser sem-ver for dependencies?

I noticed that the dependencies in the package.json are declared with straight version numbers. For example: "request": "2.81.0" instead of "request": "^2.81.0" or "request": "~2.81.0". I am not sure what the purpose is for that, but it is the only dependency in my tree that uses a different version of request and I just wonder if it may be a good idea to change that notation?

I came here just to say this. Apart from this<EMAIL_ADDRESS>seems vulnerable as it requires outdated hawk/hoek versions (https://nodesecurity.io/advisories/566)

@igorklopov when will you be able to change it? (it has been 11 days since this issue was opened and it's still a security vulnerability)

Actually, this is an issue for me as I use pkg -> pkg-fetch: 2.5.6 -> request: 2.85.0 -> hawk: 6.0.2 -> cryptiles: 3.1.2. Now GitHub is alerting me to this package vulnerability. Currently I'm ignoring it, although I want to fix this issue, as it's not a huge deal at the moment and I would like to get this resolved.
@igorklopov is it possible this could be taken care of?

Done! Closing

This issue has almost one year... Finally 🎉


poor performance on pandas groupby

Small portions of data causing performance issues on machines with 16GB+ of RAM.
https://mageai.slack.com/archives/C03HTTWFEKE/p1690561253799809?thread_ts=1690558529.045459&cid=C03HTTWFEKE

Data is a little bit larger than I initially remembered.

deployment:

aws
ec2 t2.xlarge 16GB memory
ami: ami-053b0d53c279acc90 (ubuntu-jammy-22.04-amd64-server-20230516)

execution type:

local python

Debugging I started going line by line through code and it was consistently this line that caused the kernal to crash:

most_recent_notes = notes_df.loc[notes_df.groupby('vanid')['datecreated'].idxmax()]

notes_df = 2.1 million rows x 15 columns

have you tried setting observed=True in the groupby method? seems like it defaults to False which has caused memory issues for other pandas users.
relevant issue: https://github.com/pandas-dev/pandas/issues/43999

I did not try observed=True.  I refactored the code to use duckdb which works well.

i'm going to close this issue. i can reopen it if other users experience the same issue


Grabbing node names for collapse_node in 1.0.0-alpha.21

Hello!
I am trying to implement Phylotree into a Vue app. I've run into trouble trying to utilize the collapse_node() function to try to programmatically collapse nodes (if this is even possible currently) on the tree to make it more manageable for the user (the tree has 1500+ leaves). I cannot seem to get the correct input for the function. I've ran the get_nodes() and it returns an object but I can't seem to find any identifiers for the nodes to use as input.
I am not that familiar with javascript, so this may be something simple I am missing so my apologies.
I based my implementation off the examples given in #129.
`    generateTree () {
  var tree = new phylotree(cydrasilTree)

  let height = 15000
  let width = 1600
  let svg = document.createElement('div')
  svg.setAttribute('id', 'tree_display')
  document.body.appendChild(svg)

  var renderedTree = tree.render('#tree_display', {
    height: height,
    'show-scale': false,
    compression: 0.5,
    width: width,
    'left-right-spacing': 'fit-to-size',
    'top-bottom-spacing': 'fit-to-size',
  })`

Thank you for your help!
Daniel

Dear @droush,
I have some example code for doing this in https://observablehq.com/@spond/sd-pirc-sequence-qa where you collapse everything except the clades that have "tagged" nodes.
The relevant code fragment (which is "hackish") is the following. You can traverse the tree and "tag" the nodes with n.collapsed.
Best,
Sergei
t.traverse_and_compute ((n) => {
      if (!d3.layout.phylotree.is_leafnode (n)) {
         if ("collapsed" in n) {
            delete n["collapsed"];
         }
         n.tagged_view = false;
      }
   });

   t.traverse_and_compute ((n) => {
      if (highlight_color (n)) {
        n.tagged_view = true;
      }
      if (n.tagged_view && n.parent) {
        n.parent.tagged_view = true;
      }
   });

   t.traverse_and_compute ((n) => {
      if (!d3.layout.phylotree.is_leafnode (n) && !n.tagged_view && n.parent && n.parent.tagged_view) {
          //console.log (n);
          n.collapsed = true;
      }
   });

   t.placenodes().layout();


Work-In-Progress environment_owner memory leak patch

This is an ugly patch to fix #17767
I could really use some feedback.

I won't have time to work on this, I'm closing this down.


feat: slim check

An absolute minimal port of Lean 3's slim_check, not featuring stuff like subtypes, existential quantifiers etc. But the basic infrastructure to set all of this up and some basic instances already as well.
The PhantomFunction was introduced to avoid noncomputable instances since SizeOf is not computable but we need to store it as a field in SampleableExt so we work around this by defining a PhantomFunction which is guaranteed to be eliminated by the compiler during code generation. I do however believe that this is not the right file for it to live in?

The PhantomFunction was introduced to avoid noncomputable instances since SizeOf is not computable but we need to store it as a field in SampleableExt so we work around this by defining a PhantomFunction which is guaranteed to be eliminated by the compiler during code generation. I do however believe that this is not the right file for it to live in?

One option (that I would suggest) is to use WellFoundedRelation instead.
Otherwise such a type should indeed go in a different file.  I'd port the erased type from mathlib.  Then PhantomFunction α β is just Erased (α → β).

Thanks!
bors r+

This is due to the {}  changes in class declarations, I'll make a fix.

@gebner rebased and fixed, should compile now!.

bors r+


PHOENIX-5662 The integration tests in phoenix-hive are broken

Made BaseHivePhoenixStoreIT inherit BaseHBaseManagedTimeIT
Removed Category annotations from the IT tests
Marked HivePhoenixStoreIT.testTimestampPredicate() as Ignore because this test always fails

After this fixes, we can run the integration tests successfully by running mvn verify.

It looks like the QA failed with the following error and it's unrelated to the fix:
gpg: no default secret key: No secret key
gpg: signing failed: No secret key


Request - Ability to trigger keypresses from command line interface

I don't think this is currently possible, but it would be really handy to be able to trigger key presses from the command line interface.
In my particular use case I use the app Alfred along with their companion app Alfred Remote. They have iTunes controls built into the app which is nice, but what I REALLY want to do it be able to emulate media key presses. Right now if I'm using another app to play music (e.g. Spotify or Rdio) their iTunes integration doesn't work.
Is this a realistic idea? I'm curious if anyone else would find this useful or if I'm just totally off base and there's a simpler solution.
Example
Just as an example, I would imagine the interface would be something like:
$ karabiner trigger "MUSIC_PLAY"
# or, following the convention of private.xml
$ karabiner trigger "ConsumerKeyCode::MUSIC_PLAY"

Or maybe more explicit:
$ karabiner keyDown 0x3
$ karabiner keyUp 0x3
# or, as a convenience method that combines the two
$ karabiner keyPress 0x3

I've added extra command line tool post-hid-event at v10.6.8.
Please upgrade to the latest beta version.
How to upgrade to the latest beta version:
https://pqrs.org/osx/karabiner/document.html#check-for-updates
Usage:
$ /Applications/Karabiner.app/Contents/Library/utilities/bin/post-hid-event NX_KEYTYPE_NEXT
$ /Applications/Karabiner.app/Contents/Library/utilities/bin/post-hid-event NX_KEYTYPE_PREVIOUS

Show list of keycodes
$ /Applications/Karabiner.app/Contents/Library/utilities/bin/post-hid-event

Note:
If you want to send normal keycode event from command line, use JXA.
https://github.com/dtinth/JXA-Cookbook/wiki/System-Events

I just updated to the latest beta (v10.6.11) via the "Check for update including beta version" button, but I don't see the post-hid-event in /Applications/Karabiner.app/Contents/Library/utilities/bin/. I do see your commit that should add the utility but for some reason it's not there.
I can try completely uninstalling and reinstalling Karabiner.

Excuse me.
post-hid-event was not included in recent beta versions.
I've fixed this issue at v10.6.12.
Please upgrade again.

:beers: Awesome! Works like a charm. Thank you so much for the quick turn around!


Cookie "dialog" is overlapped by "Kong Inc"

https://docs.konghq.com/hub/
Summary
The string "Kong Inc." overlaps the cookie warning.
Steps To Reproduce

Visit https://docs.konghq.com/hub/ (in an incognito window, to force the cookie "dialog" to appear in the bottom right)
Scroll up and down
Notice that "Kong Inc." is "above" the cookie dialog

Additional Details
Appears in both Chrome and Firefox on Mac OSX

https://github.com/Kong/docs.konghq.com/pull/862


Cannot Load Win PE Import Table

Describe the bug
Ghidra cannot load the import table if the size value in optional header is too big.
But Windows Operating System do not use the size value in optional header to decide the size of the import table.
The import table is null-terminated, OS will stop loading when it find an all 0 import directory.
To Reproduce
Steps to reproduce the behavior:

Choose a Win PE executable file.
Open the exe file using a hex editor (ImHex for example).
Change the value of import table size in optional header to a very large number (0xFF FF FF FF for example).
Save the change and close the file.
Open the Ghidra and create a new project.
Import the exe file into the Ghidra.
Click on imported file in Ghidra and Click 'Yes' when Ghidra ask to analyze it.
Wait.
Click "Imports" in "Symbol Tree" and you will see nothing.
If you check the IAT and CALL instructions which call those import functions, you would not see the function names.

Expected behavior
"Imports" in "Symbol Tree" should shows imported libraries and functions.
IAT should shows the libraries and functions name.
When calling import function, CALL instructions should be followed by the libraries and functions name.
Screenshots
This is what Ghidra should be:

This is what it actually looks:

The only difference is the value of import table size in optional header.
And this program can run just fine.

Attachments
None
Environment (please complete the following information):

OS: Microsoft Windows 10
Java Version: openjdk 22.0.1
Ghidra Version: 11.0.3
Ghidra Origin: official GitHub distro

Additional context
None

Do you know if this is a unique feature of just the Import Table Data Directory, or do other Data Directories ignore the size too?


GraphicsOverlay MAUI samples: layout changes

Description
Ensures MAUI samples in GraphicsOverlay category can be resized on phone, tablet, and PC. Also accounts for device orientation changes by utilizing TwoPaneView. Opted for v.next as to take advantage of fixes brought in from .NET 8.
Type of change

Bug fix

Platforms tested on

[x] MAUI WinUI
[x] MAUI Android
[x] MAUI iOS
[x] MAUI MacCatalyst

Checklist

[x] Self-review of changes
[x] All changes work as expected on all affected platforms
[x] There are no warnings related to changes
[x] Code is commented and follows .NET conventions and standards
[x] Codemaid and XAML styler extensions have been run on every changed file
[x] No unrelated changes have been made to any other code or project files
[ ] Screenshots are correct size and display in description tab (800 x 600 on Windows, MAUI mobile platforms use the MAUI Windows screenshot)

Planning on getting screenshots taken after we've agreed on UI.

cc: @cgutierrez-gis

Closing for now - attempting to fix our sample geoview, border, and grid styles to allow for device orientation changes on iOS.


Fix navigation bar

How can I fix the navigation bar. Just want to scroll content

Can you please give me more details about your problem? A screenshot would be helpful.

I'm going to close this issue, please re-open when more details are provided.


fix: revert cross-org workflow update

This reverts commit 8312b5c56010b06e867cbd92c5983d2c4be61310.

Changes

:bug: revert workflow update

Workflows are failing with error:
This reverts that commit until permissions can be sorted out.
/kind bug
Fixes #928

/lgtm


make install crashes

Hi!
Wasted two days trying to get 'make install' work - to no avail... Installed n, switched down to node 10.14.2, when running 'make install' always get the same error:

Please, advice - I am out of ideas. The beauty of the Webiny UI is greatly negated by the rough installation experience.
Thanks!

Thank you for your report, i'll check that.

Webiny change a lot since i built this skeleton and actualy with the latest version, it's not possible to use it in local without AWS : https://community.webiny.com/t/deprecated-local-installation-template/179/2
The best approche is to use : https://github.com/localstack/localstack in order to simulate aws in local. I check if i can use that.


Volume mount failing but secret being retrieved

Currently implementing a test of Azure KV FlexVolumes with AAD PodId in an AKS cluster but experiencing some strange behaviour.
AAD PodId K8's components deployed using helm https://github.com/Azure/aad-pod-identity/tree/master/charts/aad-pod-identity & https://raw.githubusercontent.com/Azure/kubernetes-keyvault-flexvol/master/deployment/kv-flexvol-installer.yaml . Azure components (User Managed ID & access controls) deployed with Terraform. All of the Azure components seem to be fine.
A test pod with a volume mount and a test secret have been deployed using:
apiVersion: v1
kind: Pod
metadata:
  labels:
    app: nginx-flex-kv-podid
    aadpodidbinding: aadpodid
  name: nginx-flex-kv-podid
spec:
  containers:
  - name: nginx-flex-kv-podid
    image: nginx
    volumeMounts:
    - name: test
      mountPath: /kv
      readOnly: true
  volumes:
  - name: test
    flexVolume:
      driver: "azure/kv"
      options:
        usepodidentity: "true"
        keyvaultname: "sandbox-kv"
        keyvaultobjectnames: "password"
        keyvaultobjecttypes: "secret"
        keyvaultobjectversions: "bda5b53cd0184fa59cbe683bf6e9e2e9"
        resourcegroup: "sandbox"
        subscriptionid: "XXXXXX-XXXX-XXXX-XXXX-XXXXXXXX"
        tenantid: "XXXXXX-XXXX-XXXX-XXXX-XXXXXXXX"

Things get interesting now.
mic log records the following:

I0206 16:40:00.119882       1 event.go:218] Event(v1.ObjectReference{Kind:"AzureIdentityBinding", Namespace:"default", Name:"azure-identity-binding", UID:"9327f0b1-2a2c-11e9-a16b-e619d6e0ff11", APIVersion:"aadpodidentity.k8s.io/v1", ResourceVersion:"12473313", FieldPath:""}): type: 'Normal' reason: 'binding applied' Binding azure-identity-binding applied on node aks-agentpool-11854310-0 for pod nginx-flex-kv-podid-default-azure-identity

nmi pod logging a 400, Identity not found:

time="2019-02-06T16:37:45Z" level=error msg="failed to get service principal token for pod:default/nginx-flex-kv-podid, adal: Refresh request failed. Status Code = '400'. Response body: {"error":"invalid_request","error_description":"Identity not found"}" req.method=GET req.path=/host/token/ req.remote=<IP_ADDRESS>

Expecting to see logs to the effect of 10 & 12 detailed here regarding PodId https://github.com/Azure/aad-pod-identity/tree/master/charts/aad-pod-identity . Save this one for a bit later because it would appear from these logs that any lookup using PodId will fail. It doesn't.
The log related to the failed volume mount is:

Warning  FailedMount  59s (x4 over 1m)  kubelet, aks-agentpool-11854310-3  MountVolume.SetUp failed for volume "test" : mount command failed, status: Failure, reason: /etc/kubernetes/volumeplugins/azure~kv/azurekeyvault-flexvolume failed, failed to get key vault

Yet kubectl exec -it nginx-flex-kv-podid cat /kv/password successfully returns the correct value of the secret referenced. This suggests the AAD PodId is working fine and that all the permissions set to allow the AKS Service Principle to use the User Managed ID to authenticate to and read from KeyVault are also ok.
Interesting conundrum. Grateful for any input that may help explain this behaviour.

Yet kubectl exec -it nginx-flex-kv-podid cat /kv/password successfully returns the correct value of the secret referenced.

@stevenicholl If you see this working, then definitely the volume has mounted and the AzureAssignedIdentity has been created.
The entire AzureAssignedIdentity creation process gets triggered when we create the volume at pod creation time. All these components (find identity, assign identity, retrieve token for key vault) will take time to complete. If volume mount fails, the kubelet will retry. Hence you are seeing errors in the log but looks like eventually all the dependent components got created successfully.

Thank you @ritazh
On one hand it is good to get some reassurance that what I was seeing can be explained.
On the other hand, based on the assumption everything has worked because I can retrieve a secret, grateful for any suggestion to prevent the failed mount log against the pod. When describing the pod, the failed mount log would lead anyone to believe the process had failed even though we now know it didn't. Perhaps some way to write to further message of success when it eventually mounts? I'm keen to avoid having error messages that incorrectly describe the state of our pods in production.

@stevenicholl Can you please share the output of kubectl describe pod <name of the pod with the mounted secret> then kubectl get events. Please redact any sensitive information.

Hi @ritazh
Below is the output for the kubectl describe pod <name of the pod with the mounted secret>
Events:
  Type     Reason       Age   From                               Message
  ----     ------       ----  ----                               -------
  Normal   Scheduled    23s   default-scheduler                  Successfully assigned flexvoltest/nginx-flex-kv-podid to aks-agentpool-11854310-5
  Warning  FailedMount  22s   kubelet, aks-agentpool-11854310-5  MountVolume.SetUp failed for volume "test" : mount command failed, status: Failure, reason: /etc/kubernetes/volumeplugins/azure~kv/azurekeyvault-flexvolume failed, failed to get key vault
  Normal   Pulling      16s   kubelet, aks-agentpool-11854310-5  pulling image "nginx"
  Normal   Pulled       15s   kubelet, aks-agentpool-11854310-5  Successfully pulled image "nginx"
  Normal   Created      14s   kubelet, aks-agentpool-11854310-5  Created container
  Normal   Started      14s   kubelet, aks-agentpool-11854310-5  Started container

On running kubectl get events against the namespace I get: (note this was executed a number of minutes after above so the timings won't perfectly match):
4m52s       Normal    Scheduled     Pod       Successfully assigned flexvoltest/nginx-flex-kv-podid to aks-agentpool-11854310-5
4m51s       Warning   FailedMount   Pod       MountVolume.SetUp failed for volume "test" : mount command failed, status: Failure, reason: /etc/kubernetes/volumeplugins/azure~kv/azurekeyvault-flexvolume failed, failed to get key vault
4m45s       Normal    Pulling       Pod       pulling image "nginx"
4m44s       Normal    Pulled        Pod       Successfully pulled image "nginx"
4m43s       Normal    Created       Pod       Created container
4m43s       Normal    Started       Pod       Started container

In the cluster events I get a corresponding azure-identity-binding applying at exactly the time the container started:
4m43s       Normal    binding applied           AzureIdentityBinding   Binding azure-identity-binding applied on node aks-agentpool-11854310-5 for pod nginx-flex-kv-podid-flexvoltest-azure-identity

If I have understood correctly, this is what you were expecting to see and it confirms the behaviour described ie. it eventually successfully mounts? I do still think the K8's events are a bit misleading in that, whilst we know the secret mounted correctly, it still logs a failure.

@stevenicholl yes this is the expected behavior following the pod's creation lifecycle. Whenever a volume mount attempt has failed, the kubelet will retry mounting the failed volume and the failed event will be recorded. These events help us debug if any of the volume mounts failed. They are the history of pod, not current state of the pod. If any of the volume mounts failed, then the pod will not be created, its STATUS will be  ContainerCreating. As you can see from your pod events, eventually as soon as the volume is mounted successfully, the container is created, then started. The pod's STATUS is Running.

@ritazh , thank you for confirmation. We will proceed on the basis that, if the pods start, the volumes have mounted. If they didn't mount correctly, the pod creation process will not have finished and will be stuck in a ContainerCreating state.

@stevenicholl That's correct. Closing this issue for now, but feel free to reopen or file new issues if you have additional questions.

Thanks @ritazh , The only other thing I'd be interested to know if it is understood why the mounts initially fail? This only occurs for us when it is a KeyVault FlexVolume, no issues when mounting secrets to pods from Kubernetes secrets.


Add mallopt

First time contributor. Thanks for the excellent contributing guide.
I'm not quite sure I put the method in the right place. The GNULib docs say its not on Android 7.1, but there are mentions of it in some android headers.

So from the Travis failures it's evident this doesn't exist in musl. Is there a place for glibc-specific APIs? Or does that place it beyond the scope of this crate?

@bors: r+
Looks great to me, thanks!

:pushpin: Commit a9590b55905c5f07c415c23e85a79d2259354482 has been approved by alexcrichton

:hourglass: Testing commit a9590b55905c5f07c415c23e85a79d2259354482 with merge b9a0e23e84d2ae5aa580838831206da64d6473cf...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing b9a0e23e84d2ae5aa580838831206da64d6473cf to master...


Projects without a .spec.displayName are not displayed in the UI

Setup

Rancher version: v2.8-head
Rancher UI Extensions:
Browser type & version:

Describe the bug

Projects without a .spec.displayName can be created via kubectl, but once created, they are not shown in the UI.
To Reproduce

Create a new Project via kubectl without setting .spec.displayName (using yaml below):

apiVersion: management.cattle.io/v3
kind: Project
metadata:
  name: das-unsichtbar-projekt
  namespace: default
spec:

Result

Project can be listed via kubectl get project -A but it is not visible through Rancher Manger UI.

Expected Result

The UI to display the created project, in this case falling back to the actual project name (.metadata.name).

Screenshots

Additional context

The newly added .clusterName validation should ensure that is the case going forward.

I validated this ticket on v2.8-6cd32b985f0b39ef2eed363458f86a352c67a42b-head, the below test cases were executed and passed. I am closing the ticke.
Test cases:

Create a project with Display Name and check for the project display on UI (V1 end point)
Create a project without Display Name and check for the project display on UI (V1 end point)
Create a project with duplicate Display Name and check for the project display on UI (V1 end point)
Create a project with Display Name and check for the project display on UI (V3 end point)
Create a project without Display Name and check for the project display on UI (V3 end point)


Is the source code based on danmakufu ph3.5 nf4?

Hello,gmestanley,Is the source code based on danmakufu ph3.5 nf4? Why I can't run games in the DNH-V after I trying to build from  source?Am I have some wrongs on build?
bin_th_dnh-v_test1.zip

Now I have some wrongs about that account,so I must change to other account now.

Hello, sorry for the late answer, I've been busy.
Do you mean that you literally can't execute games on DNH-V or that it didn't compile? I ask because I can't see any issue with the (main) code and I'm pretty sure it compiles fine on my computer.
The latter depends on what exactly did you build: did you compile the whole solution or the DnhExecutor project? That one is the core part of DNH, the executable, which is what you need and that, again, has no issue in its code.
If it's the former, please provide what exactly is happening, like if there's a message box with an error appearing after running a game or the engine itself, if the engine doesn't run or if it crashes after running a game.


Add support for gap, column-gap, and row-gap

Only supports point values - percents don't work, and shouldn't be exposed as part of the API (similar to the current situation with borders).
I've added tests, and they all work.
I'm not 100% sure I've handled the bindings all correctly. I did run enums.py - but this created white a lot of extra changes too.
Let me know if there's anything I've missed.
Fixes #812

Great work 👏 I have been trying a similar approach but got stuck with setting child's cross-axis offset. Glad to see this solution. I've added a PR for a fix. I'll be testing it more as I get time.

This looks good! Can you create a PR on this branch with yoga changes, I'll update the React native bindings to match with this version. It'll make it easier to test stuff and get feedback on this PR. AFAIKS React native has Yoga here. Would be cool to figure out if there's an automated way to get a specific Yoga version in React native.

I’m away from my laptop until the new year now. If you want I can give you write access to my fork if that makes things easier!

Cool. no worries. I'll figure it out. Happy holidays! 🥳

You too! Feel free to ping me if you need me to sanity check anything or are stuck with anything!

We need this 🙏🏻

Any update on this?

Any update on this?

+1!

Someone approve it please.

any update?

For real though, I get notified for every comment. I don't have any updates.

Common Facebook devs, its been more than 6 months and you guys just can't review & click that approve button

Still waiting for approvement

Hey folks! I want to apologize for the lack of engagement on this issue.
Yoga has been without a maintainer inside of Meta. PRs like this show that the community is still finding important issues. I'm going to import this PR, and try to find a reviewer.
There is a spiritual successor to Yoga, that has been successfully adopted by other teams at Meta. We have been looking at whether it can help provide a path to enable new capabilities  and support for layout in react-native.

Thanks @NickGerleman

There is a spiritual successor to Yoga, that has been successfully adopted by other teams at Meta.

Cool! would be awesome to learn more about this and If there are any plans to open source the implementation :D

@NickGerleman Thanks so much for taking this on! If you have any questions, me and @intergalacticspacehighway will be here to help

I left a comment in react-native-community/discussions-and-proposals, that we've decided not to go forward with flexlayout yet in react-native. This reopens the possibility to add new capabilities to Yoga, without the same concern of giving something to the community to then take it back.
The core functionality looks to be well tested from the fixtures added. I am not an expert in the internals of Yoga, but I am seeing everything that I would expect to. The space for the gap is accounted for in line-breaking, and the accumulated gaps limit the available line-length, before sizing flexible children, so items are sized correctly as to accommodate the gap. Then the gap is used for spacing during main axis and cross-axis justification.
There are a couple of more superficial issues. E.g. formatting, missing methods on the node to set/get gap in Java/C# (where the generated tests use that), or calling a function YGFlexDirectionToString instead of YGGapToString. I'm working to fix those up at the moment, but we should be able to merge this soon 🙂.

Good news! All of the changes from this PR should now be merged.
The change should be synced to React Native already, and https://github.com/facebook/react-native/pull/34360 tracks the PR exposing the capability.
I am planning to release this change along with the other recently merged commits in a new version of Yoga. This will be published to Maven Central and Cocoapods. The latter needs a little bit of infra work before this happens.
There was a delay here that I don't this is acceptable. I think we should be better positioned now to accept Yoga changes going forward.


Update n-series-driver-setup.md

Our HPC team is working to enable trusted launch on GPU SKUs and found CUDA and GRID drivers for Linux are not getting installed with Secure Boot enabled. After consulting with NVIDIA, we need to update our official doc on this. Thanks!

@vikancha-MSFT
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@vikancha-MSFT approved the PR. It shows I am not authorized to merge it to main branch. Can someone with permission help to merge it? Thanks!

I just noticed that you updated the master repo azure-docs. You need to update the azure-docs-pr version of the file which will then automatically merge with the master repo for production rollout.
From: Li Tan @.>
Sent: Thursday, April 27, 2023 9:10 AM
To: MicrosoftDocs/azure-docs @.>
Cc: Vijay Kanchanahalli @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Update n-series-driver-setup.md (PR #108757)
@vikancha-MSFThttps://github.com/vikancha-MSFT approved the PR. It shows I am not authorized to merge it to main branch. Can someone with permission help to merge it? Thanks!

Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/108757#issuecomment-1525970926, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASMORNO2XSZ6G75BQLRZKBLXDKK37ANCNFSM6AAAAAAXLUNLOI.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Create File For Gitam University, India.

Gitam University is Located In The City Of Visakhapatnam, Andhra Pradesh, India at Zip Code-530045.

@ArunVenkata Please provide us with an URL of a page at the official university website or another proof (.pdf or a screenshot are OK) showing that the university recognizes the domain which you are submitting as an official email domain for the students.

@ArunVenkata I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the abovementioned issue. Thank you for the understanding!


swiping to reset to parent tab instead going to the child tabs

Is It possible to swipe only to the parent page even when we select a child pages on the parent page?

Are you talking about the Swipe To Go Back feature on iOS?

hi ihadeed no
I pretty much have 2 tabs HomePage and AccountPage.
AccountPage has a lot of subpages - ChangePasswordPage, ChangePhonePage
I want the following scenario:
We are at AccountPage and go to ChangePasswordPage
ChangePasswordPage swipeleft goes to HomePage,
HomePage swiperight should go to AccountPage and NOT ChangePasswordPage.
The only way I can think of is replacing the  onTabSelect since changing pages just changes .  Here's what I have so far
tabs.html
<super-tabs scrollTabs (tabSelect)="onTabSelect($event)">
<super-tab [root]="HomePage" title="">
<super-tab [root]="AccountPage" title="" id='account'>

tabs.ts
ngOnInit() {
this.accountPageHtml = this.el.nativeElement.querySelector('#account');
console.log(this.accountPageHtml);
}
onTabSelect(ev: any) {
console.log('Tab selected', 'Index: ' + ev.index, 'Unique ID: ' + ev.id);
if(ev.id != 'account') {
// I need to replace the dom <super-tab [root]="AccountPage" title="" id='account'> with this.accountPageHtml
}
}


Add version: JetBrains.Kotlin.Compiler version 1.5.21

Pull request has been created with Komac v1.9.1 :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/116204&drop=dogfoodAlpha

@wingetbot waivers Add Validation-Executable-Error


v12 Webpack Context.Keys().Map No Longer Works

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[X] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?

Yes, the previous version in which this bug was not present was: 11.2.14
Description

11.2.14 the context module API for Webpack functioned properly, now any attempt to filter or map context.keys() fails/is ignored.

https://webpack.js.org/guides/dependency-management/#context-module-api
below official Webpack example to filter context keys no longer works.
const cache = {};

function importAll(r) {
  r.keys().forEach((key) => (cache[key] = r(key)));
}

importAll(require.context('../components/', true, /\.js$/));
// At build-time cache will be populated with all required modules.

🔬 Minimal Reproduction
filter or map keys in test.ts and notice all found spec.ts files from the regex and their respective modules will attempt to build
🔥 Exception or Error
Following exception from /node_modules/@ngtools/webpack/src/ivy/loader.js:60:26 is generated regardless of map or filter attempts in above webpack example.
else {
                // File is not part of the compilation
                const message = `${this.resourcePath} is missing from the TypeScript compilation. ` +
                    `Please make sure it is in your tsconfig via the 'files' or 'include' property.`;
                callback(new Error(message));
            }

🌍 Your Environment

❯ ng version

     _                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 12.2.5
Node: 14.16.0
Package Manager: npm 6.14.11
OS: linux x64

Angular: 12.2.6
... animations, cdk, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... platform-server, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1202.5
@angular-devkit/build-angular   12.2.5
@angular-devkit/core            12.2.5
@angular-devkit/schematics      12.2.5
@angular/cli                    12.2.5
@schematics/angular             12.2.5
rxjs                            6.5.5
typescript                      4.3.5
webpack                         5.52.1

Thanks for opening the issue.
However,  Webpack specific features are not supported. Using Webpack specific features can break without a warning as Webpack is considered an implementation detail.

Hi Alan,
Thank you for the quick response.
Should something like this be included in the list of breaking changes for Angular 12?

Possibly it is documented and I missed it?
any help pointing me towards documentation on this piece would be much appreciated, to discuss internally with teams.
or any information to help understand why the change to angular 12 breaks this piece.

We would like to upgrade our main repo to v12 and later versions, but, this may be a blocker for us and pin us to only v11.
We use this feature heavily to manage what spec files run based on the -c passed into ng test

The CLI creates the test.ts file with the following via ng new
// And load the modules.
context.keys().map(context);

https://github.com/angular/angular-cli/blob/20fd33f6d4ce6cef1feb508a0221222e83a85630/packages/schematics/angular/application/files/src/test.ts.template#L27

To anyone who lands here experiencing the same issue, the problem was our use of excludes in the tscofing files...

The build process needs to resolve and build any modules found via require.context('./', true, /\.spec\.ts$/);
If your tsconfig is set to excludes files found by this regex, it now throws the above exception prior to getting to the keys().map


Whatsapp Setup Workflow (with proxy) does not work in Iran

Before reporting a bug, please see our FAQ in FAQ.md!
Description
I am given a proxy, but when setting up whatsapp, the QR code fails to load (see the image) presumably because of censorship in Iran. Moreover, the alternative of linking with a cellphone also doesn't succeed. The app errors with "Connect to Wifi", which is basically the app timing out on a URL.

Tested on Windows Server client with WhatsappDesktop_2.2423.10.0_neutral.....Msixbundle

Hi! The proxy support is currently only for Android and iOS and we do not support WhatsApp desktop. Please feel free to create a separate issue if you are having trouble with Android or iOS with proxies.


[CT-3519] [Feature] Clickzetta DB adapter for dbt

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
Hello everyone, I am an engineer from Yunqi China. Yunqi Lakehouse is a fully managed data management and analysis service with a lake warehouse architecture. The Yunqi Lakehouse uses a single engine to meet the needs of data warehouse construction, data lake exploration and analysis, offline and real-time data processing and analysis, business reports and other scenarios. It meets the scalability requirements of enterprises at different stages through the separation of storage and computing, Serverless elastic computing power, and intelligent optimization features. , cost and performance needs. Built-in data integration, development and operation and maintenance, data asset management and other ready-to-use services greatly simplify data development and management work and accelerate enterprise data insights and value realization.
We now have many customers in automobile companies, securities, retail and other fields. In the next step, we will also accelerate our pace of going out of China. In our self-developed data transfer project, dbt is deeply integrated. At present, we have developed and integrated the dbt-clickzetta adapter ourselves, and verified the availability and stability of our adapter in the production environment. In my opinion dbt is a powerful and efficient library. If the community can accept and merge the dbt-clickzetta adapter, it will be a special honor for us.
If possible, I will follow the guidelines of the official documentation and submit a pull request for dbt-clickzetta adapter.
Our official website link is here: https://www.singdata.com/
Thank you all again and look forward to community responses.
Describe alternatives you've considered
No response
Who will this benefit?
All clickzetta customers can directly use dbt-clickzetta to do many interesting things with the data they store in clickzetta db
Are you interested in contributing this feature?
Yes.  I have the codes of  dbt-clickzetta already. Review is required by community
Anything else?
No response

Awesome @idling11 !
Here's the main steps you'll want to follow:

Follow the guide here
Once the adapter is ready, you can open a pull request for docs.getdbt.com (this repo) by following the docs here.

Closing issue as instructions for new adapters has been provided. If you need anything further, please open an issue or PR with the required information. Thank you!


Client auto update

Underlying schema has changed. Check changes and approve accordingly. This will not create any release, simply assimilate changes on the schema.

@raguiar9080 we may have a bug, there's this empty string enum member defined https://github.com/lune-climate/lune-docs/blob/0f4841b919424bdd23b2e6b38d686bfafc8670f9/docs/src/.vuepress/public/openapi.yml#L2894= but it's missing from this PR

I'll have a look at that, I think it'll require some special treatment

I'll have a look at that, I think it'll require some special treatment

Let me know if you need help. I'm assuming he's ignoring the empty string since he probably can't convert it to a valid enum value 🤔

What's happening here?

What's happening here?

The base client generator does not deal with empty strings in an enum. We've added them as a repo enum, so we need to handle them.


Redraw without breaking 'incsearch' in search command line

Problem
:redraw clears highlight of 'incsearch'.
This is inconvenient when using ddc.vim in search command line.
Solution
The fix is from wilder.nvim.
https://github.com/gelguy/wilder.nvim/blob/f50d79a3f9ef07be3c0c8a0f49faaeea803a8c0c/autoload/wilder/renderer.vim#L10-L15

OK.


Fix heroku build failure

Coverage remained the same at 83.902% when pulling 326c882aa6f394a6d1e6d071deba11320531a4a0 on fix-heroku into 698b280ec52abb2f9f38b1b7d1de61d9e1d9e0ae on master.


Document Android 14 behavior changes for foreground services

https://developer.android.com/about/versions/14/changes/fgs-types-required#permission-for-fgs-type
Types are now required for Android 14, and their associated permissions need to be declared.

Indeed - related is the new full screen intent item

notifee.displayNotification cause crash on Android 14 when passing prop android: {asForegroundService : true}

this is possibliiy solution

For anyone with this issue while we wait for the docs to be updated, you need to first add this to your manifest as pointed out above
<service android:name="app.notifee.core.ForegroundService" android:foregroundServiceType="location|camera|microphone" />
Then add a further permission based on the foregroundServiceType chosen, for dataSync I had to add <uses-permission android:name="android.permission.FOREGROUND_SERVICE_DATA_SYNC" />
More info here:
https://developer.android.com/about/versions/14/changes/fgs-types-required

From the docs

If the foreground service needs new permissions after you launch it, you should call startForeground() again and add the new service types. For example, suppose a fitness app runs a running-tracker service that always needs location information, but might or might not need media permissions. You would need to declare both location and mediaPlayback in the manifest. If a user starts a run and just wants their location tracked, your app should call startForeground() and pass just the location service type. Then, if the user wants to start playing audio, call startForeground() again and pass location|mediaPlayback.

We would actually need support to set foreground service types dynamically,
for example, sometimes location permission may have not been granted and we would still need to start the service
I notifee.displayNotification should support specifying the type "location|camera|microphone"

Indeed - related is the new full screen intent item

do we need this permission for notifee even? Play console only allows it for alarm apps (which ours clearly isn't but it schedules local notifications that need to be delivered with high timely accuracy)

AndroidManifest.xml
<manifest ...>
	<uses-permission android:name="android.permission.FOREGROUND_SERVICE_DATA_SYNC"/>

	<application
	...
		<service
        	android:name="app.notifee.core.ForegroundService"
        	android:foregroundServiceType="dataSync"
         	/>
	</application>
</manifest>


command "just flash" failed for "hello-world"  project

run  "just flash", failed , show the messages:
cannot find trait AllocRef in module core::alloc
--> src/lib.rs:36:1
|
36 | / heap! {
37 | |     /// A heap allocator generated from the Drone.toml.
38 | |     heap => pub Heap;
39 | | }
| |_^ not found in core::alloc

The fix is already on the master branch. It required some backward incompatible changes, so it will be a part of the next release 0.14 (presumably the end of March). Meanwhile an older Rust version can be used as a workaround, e.g. nightly-2021-01-19.

Compiled success！
with run "drone new --toolchain nightly-2020-11-25 --device stm32f407 --flash-size 128K --ram-size 20K hello-world",
then, "cd hello-world" , run "just deps","just flash"


Fix channel count upscaling

This changes channel upscaling so all input samples are spread out across the output channels, instead of repeating the last sample over and over. This approach should yield more accurate results. Fixes #558.

It seems like there is no standard order for 5.1 and 7.1 surround sound instead it differs across file formats and can even be configurable. For example see channel assignment for flac. That is gonna make correct channel conversion quite challenging. As it might need work on the decoders to unify how they provide channels. Symphonia does at least provide a consistent order it seems: symphonia docs.
It makes perfect sense to turn mono to stereo by assigning both speakers the same sample. Scaling stereo to 5.1 and 7.1 should probably be done by recognizing which of the output channels is the left and which the right channel and sending the rest zero's/nothing.
Now back to this PR and stereo to 5.1/7.1 up-scaling :)
From this cpal PR it seems cpal uses the WAVEFORMATEXTENSIBLE order (see winapi docs) just like symphonia does. That means you should be able to figure out which is the front left and front right channel for any device. Then we can have rodio send stereo to only those channels.
Does that sound good to you?

So it can be safely assumed that the first 18 channels will always be in the WAVEFORMATEXTENSIBLE order? Like in this table here?
So the way it should be is...

2 to 4: [1, 2, 3, 4] -> [1, 2, 0, 0, 3, 4, 0, 0]
2 to 6: [1, 2, 3, 4] -> [1, 2, 0, 0, 1, 2, 3, 4, 0, 0, 3, 4]
2 to 8: [1, 2, 3, 4] -> [1, 2, 0, 0, 1, 2, 1, 2, 3, 4, 0, 0, 3, 4, 3, 4]
4 to 6: [1, 2, 3, 4, 5, 6, 7, 8] -> [1, 2, 3, 4, 1, 2, 5, 6, 7, 8, 5, 6]
4 to 8: [1, 2, 3, 4, 5, 6, 7, 8] -> [1, 2, 3, 4, 1, 2, 1, 2, 5, 6, 7, 8, 5, 6, 5, 6]
6 to 8: [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12] -> [1, 2, 3, 4, 5, 6, 1, 2, 7, 8, 9, 10, 11, 12, 1, 2]

For channels from 9 to 18 it'd get more complicated because they don't alternate left/right anymore, but the idea would be the same. Match left with left, right with right, center with center.
This kinda begs the question though. As you said, when going from mono to stereo it makes sense to duplicate the sample, but should that be done as well when going higher? maybe for 2 to 8 for example, this makes more sense? [1, 2, 0, 0, 0, 0, 0, 0, 3, 4, 0, 0, 0, 0, 0, 0]
Let me know what you think.

So it can be safely assumed that the first 18 channels will always be in the WAVEFORMATEXTENSIBLE order? Like in this table here?

As far as I can deduce from the cpal source thats correct. I would say anything beyond 8 channels it too nice for rodio right now (though that's not my call to make).

Match left with left, right with right, center with center.

That would ensure audio to most speaker. Though that is not what (a quick google shows) is recommended. The tldr being:

While you can attempt to stream music in surround sound, this will often just make the audio louder without improving clarity, and often results in sound distortion.

I think we can safely generalize that to: do not use more channels then the source is offers as that will decrease clarity. With an exception for going from mono to stereo (it is quite annoying to have sound only left or right).

This kinda begs the question though. As you said, when going from mono to stereo it makes sense to duplicate the sample, but should that be done as well when going higher? maybe for 2 to 8 for example, this makes more sense? [1, 2, 0, 0, 0, 0, 0, 0, 3, 4, 0, 0, 0, 0, 0, 0]

So yes given the source above to me that makes more sense. Just for clarity that would correspond to this table:

7.1 speaker position
interleaved stereo sample numb
non interleaved stereo sample

SPEAKER_FRONT_LEFT
1
sample 1 left channel

SPEAKER_FRONT_RIGHT
2
sample 1 right channel

SPEAKER_FRONT_CENTER
0
silent

SPEAKER_LOW_FREQUENCY
0
silent

SPEAKER_BACK_LEFT
0
silent

SPEAKER_BACK_RIGHT
0
silent

SPEAKER_FRONT_LEFT_OF_CENTER
0
silent

SPEAKER_FRONT_RIGHT_OF_CENTER
0
silent

SPEAKER_FRONT_LEFT
3
sample 2 left channel

SPEAKER_FRONT_RIGHT
4
sample 2 right channel

SPEAKER_FRONT_CENTER
0
silent

SPEAKER_LOW_FREQUENCY
0
silent

SPEAKER_BACK_LEFT
0
silent

SPEAKER_BACK_RIGHT
0
silent

SPEAKER_FRONT_LEFT_OF_CENTER
0
silent

SPEAKER_FRONT_RIGHT_OF_CENTER
0
silent

oh and while your at would you mind doing a tiny refactor :)
At line 74 (right now) I think this:
if self.next_output_sample_pos == self.to {
    self.next_output_sample_pos -= self.to;

would be clearer like this
if self.next_output_sample_pos == self.to {
    self.next_output_sample_pos = 0;

and still work right? Would you agree?

thanks for seeing a fix here :) Greetings from the veloren team

Since this is a non trivial PR and touches an essential part of rodio I've asked @est32 to take a look. As soon as thats done we are ready to merge! :tada:
Thanks a lot for adding proper multi channel support @ideka to rodio! As you've seen its already appreciated by the community!