improve & granularize ExcalidrawElement types

In short, this PR adds more type safety.

Add ExcalidrawLinearElement type, and make points only specific to that type. This is in expectation of followup https://github.com/excalidraw/excalidraw/pull/949 where I'll be adding another, arrow-specific property.
Rewrite Element type-specific constructors (move away from parameter list to object; move away from nested constructors when creating text elements).
Ensure element.type narrows down the element type when possible (i.e. make it into a union discriminant). For now, I haven't split rectangle/ellipse/diamond shapes into their own constructors because they all share the same props.
Don't create generic element on every pointerdown. Instead, create specific elements when needed.
Make APIs that mutate elements (mutateElement, duplicateElement, restore...) more typesafe by improving typing and introducing Mutable generic helper instead of casting to any and whatnot.
Fix return values from element constructors not being immutable (and fix mutations around the codebase).

I'm sure that the typings for each separate excalidraw element could be made more readable by either improving the types, or writing them out manually. Lemme know.

Will fix tests in a bit.

If all in favor, I'll merge this to minimize conflicts with future PRs, and so that I can work on the #949.
If anything new comes up, we can improve/fix this later.

Gonna make food, and then I'll figure out how to make rescalePoints()  assignable to mutable points in updates object :crying_cat_face:

feel free to revert the conflicting pr if it'd be easier for me to rebase on top

No no, it's fine. Just gotta find a workaround for TS :).

I'm gonna kick the can down the road and make getSizeFromPoints() return a mutable array, since we're currently using it only in places where we expect mutable arrays. That said, later we can easily cast as const when we need an immutable return value (AFAIK), but not the other way round. We can also figure out a better solution later. Right now, it doesn't reduce type safety anyway.
That said, the merge commit diff looks weird --- not sure if you've removed that much code in your PR. Will review before merge. But I've basically used your changes, and just shuffled the code to satisfy TS.

i took a peek at the merge commit, lgtm


cmd/trace: Shorten file paths in output

Export the function to shorten file paths from the terminal package so
that it can be used in the trace subcommand.

cc: @aarzilli for review.

lgtm

Merged with d4bfd25a28cab4589808471d987e28ae6c6175c6


chore(add-replica): add replica monitoring

Context
Adds monitoring for the Supabase read replica following this reference.
Note: This project has been deprecated in favour of https://github.com/supabase/supabase-grafana so we might also migrate to the latest solution which includes more metrics.

@Cali93 think we can close this since the read replica monitoring was set up in another PR with Nodar right?

Yes indeed, it was taken care of by https://github.com/WalletConnect/infra/pull/102


No tutorial page: Drive a Lamborghini With Your Keyboard

i wanted to know the insights of the arduino code. when i opened the link
Inspired by this aiticle: Drive a Lamborghini With Your Keyboard
it shows no page. can u explain it please

it's here


add support for REPAIR sql stmt

What this PR does:
Which issue(s) this PR fixes:

Fixes #658
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report
Patch coverage has no change and project coverage change: -0.50 :warning:

Comparison is base (41901cc) 36.71% compared to head (282ecee) 36.21%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #680      +/-   ##
==========================================
- Coverage   36.71%   36.21%   -0.50%
==========================================
  Files         206      214       +8
  Lines       22480    22940     +460
==========================================
+ Hits         8253     8308      +55
- Misses      13310    13713     +403
- Partials      917      919       +2

Impacted Files
Coverage Δ

pkg/executor/redirect.go
3.81% <0.00%> (ø)

pkg/runtime/ast/ast.go
69.71% <0.00%> (-0.97%)
:arrow_down:

pkg/runtime/ast/proto.go
53.33% <ø> (ø)

... and 14 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

pls fix github CI/CD bug


[DatePicker] Remove helperText default value, document it

Steps to reproduce
function App() {
  const [value, setValue] = React.useState(null);

  return (
    <LocalizationProvider dateAdapter={DateFnsUtils}>
      <DatePicker
        value={value}
        onChange={newValue => {
          setValue(newValue);
        }}
        renderInput={props => <TextField {...props} />}
      />
    </LocalizationProvider>
  );
}

https://codesandbox.io/s/condescending-hoover-m5jk7?file=/index.js:234-580
Expected behavior
Should the helperText have a value by default? It feels strange. I think that we should remove it to:

reduce duplication of information: placeholder + input mask should be good enough.
reduce the height of the input takes by default. It might not fit into people's layout.
encourage the usage of the helper text for related information regarding the field.

Actual behavior

Environment

Tech
Version

@material-ui/lab
v5.0.0-alpha.16

@oliviertassinari  What's the status of this issue, have you agreed that it shouldn't be default to include helperText? In that case, can I work on a pull request? :)

@montelius The proposed direction is:

Change the default, remove the helper text, optimize for higher density
Add a section in the documentation for implementing the current default

How does it sound?

@oliviertassinari Sounds good! I'll look into it.

Sorry to say but I've given up on this! So anyone's free to give it a try.

@montelius Thanks for the update. Is there a specific pain point you faced that made it hard?

No not really, it just took too long for me to understand how the components worked, but that's probably because I'm a new contributor!

What is the status on this? Passing helperText={null} does not do anything for me on version 5.0.0-alpha.27.

same here helperText={null} does not do anything for me on version 5.0.0-alpha.27.

any update ?

I've found a temporary workaround, it's ugly but for now you can try this :
  const deleteHint = (props: MuiTextFieldProps): MuiTextFieldProps => {
    delete props.helperText
```

and

renderInput={(startProps, endProps) => (
<>
<TextField
size="small"
variant="standard"
{...deleteHint(startProps)}
/>
            <Box sx={{ mt: 1, mb: 2, mr: 3, ml: 2.5 }}>
              <Typography
                variant="subtitle2"
                color="secondaryText"
                component="p"
              >
                au
              </Typography>
            </Box>

            <TextField
              size="small"
              variant="standard"
              {...deleteHint(endProps)}
            />
          </>
        )}

This diff should be enough:
diff --git a/packages/material-ui-lab/src/internal/pickers/hooks/useMaskedInput.tsx b/packages/material-ui-lab/src/internal/pickers/hooks/useMaskedInput.tsx
index 25fe7e9311..623150ae24 100644
--- a/packages/material-ui-lab/src/internal/pickers/hooks/useMaskedInput.tsx
+++ b/packages/material-ui-lab/src/internal/pickers/hooks/useMaskedInput.tsx
@@ -105,7 +105,6 @@ export function useMaskedInput({
     label,
     disabled,
     error: validationError,
-    helperText: formatHelperText,
     inputProps: {
       ...inputStateArgs,
       disabled,

Hi, I will try to implement this.


Unexpected behaviour when loading fragments

Hi,
pretty sure this is a bug, but it is not easy to be certain with Kconfig ;)
Lowering a value in a fragment doesn't seem to work.
Steps to reproduce:
cd Kconfiglib/examples
git remote add sebo_remote https://github.com/SebastianBoe/Kconfiglib
git fetch --all
git checkout 34327f56891e2a6583df6d03144042084fca07a8
python merge_config.py Kconfig merged conf1 && cat merged

Actual output:
sebo@mach:~/repos/kconfiglib_zephyr/examples$ python merge_config.py Kconfig merged conf1 && cat merged
conf1:2: warning: BAR (defined at Kconfig:9) set more than once. Old value: "y", new value: "n".
warning: BAR (defined at Kconfig:9) was assigned the value "n" but got the value "y" -- check dependencies
# Generated by Kconfiglib (https://github.com/ulfalizer/Kconfiglib)
# CONFIG_FOO is not set
CONFIG_BAR=y

Expected output:
sebo@mach:~/repos/kconfiglib_zephyr/examples$ python merge_config.py Kconfig merged conf1 && cat merged
conf1:2: warning: BAR (defined at Kconfig:9) set more than once. Old value: "y", new value: "n".
# Generated by Kconfiglib (https://github.com/ulfalizer/Kconfiglib)
CONFIG_FOO=y
# CONFIG_BAR is not set

Interestingly, removing CONFIG_BAR=y will give the expected behaviour, which leads me to believe one can not lower config values in fragments, which is unexpected as the warning states it is getting the new value.
PS: https://github.com/SebastianBoe/Kconfiglib/commit/34327f56891e2a6583df6d03144042084fca07a8

This is due to some special behavior for choices. It will work as expected for non-choice symbols.
Setting a choice symbol to y also sets Choice.user_selection to that symbol. Setting it to n later doesn't change Choice.user_selection. Setting some other symbol in the choice to y would update the user selection though.
The only way to get rid of the user selection is to call Choice.unset_value(). The choice then reverts back to the default selection.
This behavior makes .config loading "just work" for choices. It is pretty unintuitive for the case above though. I guess you could unset Choice.user_selection if the Choice.user_selection symbol is set to n, which might make the above example more intuitive. I'm not sure it's worthwhile though. It's a bit weird to have setting a user value unset a different user value too.
Note that "no user value" and "user value n" are different btw.

The C implementation works the same way btw, and has the same behavior there. It turns out that it's handiest to remember the choice selection separately.
Imagine if someone calls set_value("y") on a bunch of symbols in the same choice for example. In that case, what you probably want is to have the last symbol set to y be the user selection. At the same time, you don't want to have to update the user values of all the other choice symbols, as it gets messy in practice. It's also nice to have Symbol.user_value always reflect the assigned value.
There's a bunch of trickiness related to choices that isn't obvious until you try to implement them. Kconfiglib 1 handled them in a messier way, so I know the pain.

Is this inspired by some real-world example btw?

"Is this inspired by some real-world example btw?"
Yes, turning off bluetooth log's in a bluetooth example.
e.g.
CONFIG_BT_DEBUG_LOG=n
in mesh.

@SebastianBoe
Looking at subsys/bluetooth/common/Kconfig, the right way to override it might be to do CONFIG_BT_DEBUG_NONE=y. That will automatically unset CONFIG_BT_DEBUG_LOG in the final configuration as well, as bool choices have the constraint that only a single symbol can be y.

Closing for now.


Provision new redis instance for search-api-v2 in staging

Trello: https://trello.com/c/H4PW2hNC/1399-create-new-redis-instance-for-search-api-v2-and-move-search-api-v2-to-it-peri-peri-🌶

Closing as this was the wrong approach https://github.com/alphagov/govuk-helm-charts/pull/2673


Bugfix 118 rmq connection timeouts

see #118

Short-term solution so indexer is on a par with metadig (reopening closed channels) - see https://github.com/DataONEorg/dataone-indexer/issues/118#issuecomment-2249046990


gstreamer-receive example not working

In jsfiddle.net text areas are showing empty. There is nothing to copy as mentioned in the readme section

Hey @JibinNajeeb, sorry for the delay!
Sorry for the delay, can you try removing STUN from your iceServers list? If you are in a restricted network it will take 30 seconds for the jsfiddle.net entries to be populated.

@JibinNajeeb if you are still having issues please re-open! Resolving as stale for now.


Should use get "TrueTarget" for extract targets

Like this
        {             "name": "extract_target_step",             "primitives": [{                 "primitive": "d3m.primitives.data.ExtractColumnsBySemanticTypes",                 "hyperparameters":                     {                         'semantic_types': (                             'https://metadata.datadrivendiscovery.org/types/TrueTarget',),                         'use_columns': (),                         'exclude_columns': ()                     }             }],             "inputs": ["to_dataframe_step"]         },
Because some datasets have multiple suggest targets columns and not every column is used for problem.
Example:  LL0_uci_facebook_metrics, seed_56_sunspot

Will making this change to the default template steps break other datasets?

I guess not, we already had a step in controller.py that adds sementictype "TrueTarget" to the target columns.
As for my testing on LL0_uci_facebook_metrics using feature_selection template; it did not break

I am not very familiar with the evaluation process. Will our templates be evaluated remotely? If that's the case adding a semantic type outside of a primitive might be a problem.

https://github.com/usc-isi-i2/dsbox-ta2/blob/template-2018-june/python/dsbox/template/runtime.py#L503 this is the exact function that called in our system to add semantic type "TrueTarget" for a column. I guess it is fine since we do this step for every problem and dataset that matching to it and then in template level we extract the "TrueTarget".


Replace deprecated ember/load-initializers in example

Replaces a reference in the README to the deprecated ember/load-initializers with one to ember-load-initializers

@HeroicEric Thank you for your attention to detail!


Recommended configuration when using custom lint checks?

Hello!
We're using this plugin on a project that also includes custom lint checks.
Unfortunately, the plugin is flagging dependencies within the custom lint check project, even though those aren't compiled into the final application (and therefore do not need to be jetified to avoid conflicts with AndroidX dependencies).
Here's a sample app that demonstrates the issue: https://github.com/stkent/ByeByeJetifierSample
Running ./gradlew canISayByeByeJetifier -Pandroid.enableJetifier=false in the root of that repository leads to a failure.
What's the recommended configuration here? Is there a way to ignore the entire lintchecks project? Or do we need to suppress using excludedFilesFromScanning (which is a globally-scoped ignore list)?

Hi. There is no way to exclude a project. You need to add the suppress on excludedFilesFromScanning. Something like "com/stkent/lintchecks" should work.

Unfortunately excluding com/stkent/lintchecks in addition to default exclusions does not change the result, as the Android support classes within dependencies are still scanned.
I can pass the build by excluding
"com/android/support/AndroidxNameUtils.class",
"com/android/support/AndroidxName.class",
"com/android/SdkConstants.class",
"com/android/ide/common/rendering/api/ResourceNamespace.class",
"com/android/ddmlib/testrunner/AndroidTestOrchestratorRemoteAndroidTestRunner.class"

but this suppression applies to all projects (the lint check project, which I want it to; and the app project, which I don't).
Does supporting project exclusion based on project name seem like a reasonable feature request in light of this? :)

You are right, project exclusion is the only way. I am going to add support for that on next version

https://github.com/dipien/bye-bye-jetifier/issues/46

Amazing, thanks @maxirosson! Let me know if you need any help testing release candidates when you get there :)

Feel free to close this issue in favor of #46 if that makes sense for your flow :)

Thanks, I will share you the snapshot version before releasing so you verify whether it is working fine on your project

You can try the snapshot using this config:
buildscript {
	repositories {
		maven { url = "https://oss.sonatype.org/content/repositories/snapshots/" }
	}
        dependencies {
               classpath("com.dipien:bye-bye-jetifier:1.2.0-SNAPSHOT")
       }
}

apply plugin: "com.dipien.byebyejetifier"

byeByeJetifier {
	excludedProjectsFromScanning = ["lintchecks"]
}


add: sonika

Description
I have added my profile data to the  json file.

@nsonika Thank you so much for your contribution! Your efforts are greatly appreciated and will go a long way in improving our project. Please feel free to share it with others and Star the repo to help us grow even further!


Weekly Permissions sync 2024-01-01

Weekly Permissions sync 2024-01-01

Superceded by https://github.com/microsoftgraph/microsoft-graph-devx-content/pull/439


Concurrent map writes errors when running karavel render

Occasionally, running karavel render on a KCP project yields a nice fatal error: concurrent map writes. We must investigate it and solve it.

Closed by #5


syz-fuzzer: boost priority for larger triage jobs

Let's first triage those work queue items that target larger amounts of new signal.

Set up a syz-testbed experiment.

Judging by syz-testbed experiments, this change either brings no benefits to the total signal/coverage (neither with initial corpus nor without) or even shows a slightly worse performance. This is interesting.
Maybe this approach will have more success once there's progress on https://github.com/google/syzkaller/issues/1541


increase height of textinput for multiline true

No issue ? multiline={tru} not set autoheight to input


[Q] Service Management Query type ?

Hello, I am rather new to working with BMC in general. So please disregard if this is a weird question. Do you plan on adding Service Management as a Query type for the plugin anywhere in the future ? Or is that not on your roadmap.

Please see here:
https://docs.bmc.com/docs//bhd/222/service-management-query-1077398377.html

Hi @bfarrell , thanks for your answer ! This is how I suppose you'd do it in a BMC Dashboard, which uses underlying Grafana. I am asking if there is any plans to make this currently available in Grafana through the Grafana plugin. Because using the plugin v3.0 I can currently see only: "Events", "Metric", "CloudSecurity" & "Log" as a selectable Query type. Am I missing something ?

Ah, got it.  Right now the ITSM query type is not supported in the public version.  We are looking to add this support, likely in a separate public ITSM specific plugin.  However, we don’t have timelines at this point.


squid:UselessParenthesesCheck - Useless parentheses around expression…

This pull request is focused on resolving occurrences of Sonar rule squid:UselessParenthesesCheck - Useless parentheses around expressions should be removed to prevent any misunderstanding
You can find more information about the issue here: https://dev.eclipse.org/sonar/coding_rules#q=squid:UselessParenthesesCheck
Please let me know if you have any questions.
M-Ezzat

thank you, @m-ezzat!


OS X 10.10.1 current head (be4e40e90a) segfault

Hi. I've got system error twice. First was segfault when I triend to open attached project (it's pong from tutorial, made some time ago, maybe .scn is not compatible?), but project opened. Next I've tested that pong and closed game - there was second segfault.
Project: http://youkai.pl/godot/pong.tgz
Error log: http://pastebin.com/k6UCqmUb

Due to the age of the issue and lack of further comments I'll assume that it's not reproducible anymore, so archiving.


surface in App CR status set of namespaces used for the deployment

Describe the problem/challenge you have
[A description of the current challenge that you are experiencing.]
it's useful for users of App CR and PackageInstall CR to know which namespaces hold deployed resources. internally kapp ls presents a list of used namespaces, however such information would only be accessible to direct kapp CLI users. it would be nice to somehow surface this info in App CR status, e.g.
status:
  deploy:
     usedNamespaces:
     - "" # empty to represent cluster-scope
     - knative-serving
     - knative-eventing

Describe the solution you'd like
[A clear and concise description of what you want to happen. If applicable a visual representation of the UX.]
Anything else you would like to add:
[Additional information that will assist in solving the issue.]
this problem might be also generally related to letting users identify what are resources that are part of this app/packageinstall.

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

@cppforlife in discussion with @benmoss @neil-hickey we're wondering if it's worth a discussion re. solving the full case of "letting users identify what resources are part of this app/pkgi" vs the special case of just namespaces?

possibly. though, i could see how identifying all resources might be an expensive operation vs identifying namespaces at the time of the deploy might be cheap. would love to hear ideas on more generic api though.

We have the resources at deploy time, right? I think this is a good example of a case where if we were using a kapp SDK this would be easy, but instead all that data is lost inside the CLI.
status:
  deploy:
     resources:
     - {apiVersion: "v1", kind: "Namespace", name: "knative-serving"}
     - {apiVersion: "v1", kind: "ServiceAccount", name: "knative-serving", namespace: "knative-serving"}

i have a concern about size of the App CR since this list could be 100s of resources long
if we find this feature useful, it's more than likely other uses of kapp would benefit as well (ie having a mechanism to get this info from kapp CLI should be a reasonable ask)

Good point re it being too long. Yet another case where either we need a custom APIService or a CLI in order to expose data that doesn't really fit into a CRD.


Updates to Issue / PR Temps

Description
Please Describe the Bug(s) Fixed and/or the Feature(s) Added:
(1) Fixed formatting issue in Bug Report Template
(2) Added contact information for person submitting an issue request (user)
(3) Separated out two of the questions at the bottom of the PR template, under CDC Checks, that cannot be answered with a check mark + provided some information on how to fill the box in with a check mark (placing an "X" inside of it)
Checklist
Go Through Checklist Below and Place A :heavy_check_mark: if Completed
General Checks

[X] Have you ran appropriate tests (unit/integration/end-to-end) to check logic across run environments (Conda/Docker/Singularity on Scicomp/AWS/NF Tower/Local)?
For each relevant configuration:

Can the program run completely through without erroring out?
Does it produce the expected outputs, given the inputs provided?

[X] Have you conducted proper linting procedures?

Numpy formatted docstrings for functions
Comments explaining lines of code
Consistent and intuitive naming conventions for variables, functions, classes, methods, attributes, and scripts
Single empty line between class functions, two lines between non-class functions, and two lines between imports and code body
Camel case formatting for class names

[X] Have you updated existing documentation (README.md, etc.) or created new ones within docs?

CDC Checks

[X] Did you check for sensitive data, and remove any?
[No] Are additional approvals needed for this change?
[No] Are there potential vulnerabilities or licensing issues with any new dependencies introduced?
[X] If you added or modified HTML, did you check that it was 508 compliant?

@kyleoconnell changed 'ran' to 'run' in PR template


Functional tests for .NET Core CLI

Current behavior which we may want to change:

Uses first dotnet executable in path
Uses env var NUGET_PACKAGE_SOURCE to specify private feed(s)

Closing to replace with new PR which preserves history


Service Metrics Refactor

Metrics refactoring includes:

Metrics calculation no longer hardcoded into single class
Prometheus publisher/scheduler can create arbitrary metric types
Metric/endpoint/validator definitions now strongly encapsulated
Heirarchy of interfaces/abstract classes to remove boilerplate metric code
New Identity metric to demonstrate how to add new metrics
Metric creation guide in service/docs

Address #263
Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Pull Request title is properly formatted: FAI-XYZ Subject
[ ] Pull Request title contains the target branch if not targeting main: [0.3.x] FAI-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

/test trustyai-explainability-e2e


Add ability to timeout a container/script using activeDeadlineSeconds

Split this issue out from issue #498 to support a timeout feature when scheduling pods. We can support this by adding activeDeadlineSeconds to templates and carry forward this information when creating the pod.

This has been fixed in commit 133a23c


Adding option for the voyager migrations to be autoloaded

In some cases users may regenerate their migrations and have them all located in the /database/migrations folder. If this happens, it will conflict with the migrations in the tcg/voyager folder.
This new addition to the config will allow the user to prevent autoloading the migrations in the tcg/voyager folder.

That's actually not true.  Laravel enumerates migrations by filename, so its impossible to have duplicates conflict.

Here's what I've done in the past.
I would regenerate my migrations based on the current mysql database. This means that my DataTypes and DataRows table gets a new migration created in the /database/migrations folder.
This would mean that I now have 2 migrations for DataTypes and DataRows table. Adding this config will give the user the option to exclude the migrations from the tcg/voyager folder when running:
php artisan migrate

That seems like an odd use-case to me.  I personally would put the data in seeders and just overwrite those.  Leave the migrations for structure, which likely won't change from what Voyager provides (combined with any custom migrations you may add, but that shouldn't affect the data_rows/data_types tables).

I understand it may be a smaller use case than others, but I feel that more options couldn't hurt anything. This would be an additional line in the config. I could see many times where a user might need this option.
The loadMigrations method is optional via the Package developer, but in this case we could add a single line to the config to allow users to exclude them.
This is not going to change the default functionality, it will only allow for an additional configuration for the developer of the application.
Due to projects that I've migrated over from other frameworks I've needed to do a reverse migration and seed and have ran into this same issue a few times.

The migrations actually used to be added to the migrations folder but was changed via this commit: https://github.com/the-control-group/voyager/commit/e5a1464a6245925ad9c4551f77d5fa7bcc771c40#diff-1de6db76b51f87f079c5c336de8d8ecc which is totally cool, but now this will allow for more flexibility. Let me know if you don't think it should go in and we can discuss further.
Thanks :)

@fletch3555 Let me know if you can merge this in today :)
Thanks man!

Please keep in mind that we need that config-var in config/voyager-dummy.php also.

Yes, fair point.  Mind submitting a PR to get that added as well?


Testing, please ignore. (tjltzu)

This is a bug bounty test - please ignore.. Please DO NOT APPROVE THIS! (tjltzu)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.tjltzu zpsam seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add path property to Image interface.

Update interfaces of the following components: Audio, Image, Model, Video.
Thank you for your contribution to MagicScript Project.
Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You are using approved terminology

Codecov Report

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

@@           Coverage Diff           @@
##           master     #322   +/-   ##
=======================================
  Coverage   98.68%   98.68%
=======================================
  Files          49       49
  Lines         304      304
  Branches       25       25
=======================================
  Hits          300      300
  Misses          4        4

Impacted Files
Coverage Δ

src/components/audio.js
100.00% <ø> (ø)

src/components/image.js
100.00% <ø> (ø)

src/components/model.js
100.00% <ø> (ø)

src/components/video.js
100.00% <ø> (ø)

src/properties-mapping.js
100.00% <ø> (ø)

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 1fb67b4...1c0ddc1. Read the comment docs.


add BaseURL for js and css

If BaseURL=https://xxx.com/blog
In head.html, a js location is assembled as such src=https://xxx.com/blogjs/theme.min.js but it should be src=https://xxx.com/blog/js/theme.min.js
In footer.html, the js location is src="js/theme.min.js" which fails to point to blog/js/theme.min.js

Thanks :)


[Event Request] codeunit 90 "Purch.-Post" OnSaveInvoiceSpecificationOnBeforeModify

Hello,
Is it possible to add the following Event:
TempTrackingSpecification.Get(TempInvoicingSpecification."Entry No.");
                TempTrackingSpecification."Qty. to Invoice (Base)" += TempInvoicingSpecification."Qty. to Invoice (Base)";
                TempTrackingSpecification."Quantity Invoiced (Base)" += TempInvoicingSpecification."Qty. to Invoice (Base)";
                TempTrackingSpecification."Qty. to Invoice" += TempInvoicingSpecification."Qty. to Invoice";
                // DINNL <<
                OnSaveInvoiceSpecificationOnBeforeModify(TempTrackingSpecification, TempInvoicingSpecification);
                // DINNL >>
                TempTrackingSpecification.Modify();
            end;
            OnSaveInvoiceSpecificationOnAfterUpdateTempTrackingSpecification(TempTrackingSpecification, TempInvoicingSpecification);
        until TempInvoicingSpecification.Next() = 0;
        TempInvoicingSpecification.DeleteAll();
    end;

[IntegrationEvent(false, false)]
local procedure OnSaveInvoiceSpecificationOnBeforeModify(var TempTrackingSpecification: Record "Tracking Specification" temporary; TempInvoicingSpecification: Record "Tracking Specification" temporary)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


file_answer.py:23: Перенести классы в отдельные файлы 58

The puzzle 899-1d0de90e from #899 has to be resolved:
https://github.com/blablatdinov/tg-quranbot/blob/686fe4a4224e67820a84993d231830e322988ca2/src/srv/files/file_answer.py#L23-L23
The puzzle was created by Ilaletdinov Almaz on 20-Jun-24.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 899-1d0de90e has disappeared from the source code, that's why I closed this issue.


Avoid exception in changes_display_dict when model is missing

As django-auditlog v3 now stores the actual PKs for foreign models instead of their string representation, I switched my application from changes_dict to changes_display_dict so that the actual string-representation of foreign models is being shown again. While this worked in my local environment, it broke while trying out this change against a previously used testing database, specifically by raising a KeyError.
As it turns out, if a model has been previously registered with django-auditlog and made any entries, followed by then being no longer registered (e.g. if the model has been deprecated or further logging is no longer desired), any calls to changes_display_dict will fail. The reason behind this is that the method attempts to lookup the model within the registry, without checking if it actually still exists.
This lookup is only needed for respecting mapping_fields and has no other purpose. This PR extends the logic to first check if the model is contained in the registry, and only if that is the case, the actual lookup happens. If a previously existing model is being looked up instead, it will still show up properly, just without the mapped fields as they no longer exist.

Thanks @ppmathis


adding the CSG logo before the name of website in the navigation bar

Is your feature request related to a problem? Please describe.
Logos help in recognizing the website more efficiently as compared to the website name.Therfore if the user sees the logo at the top of the website also then it would provide more wholesome experience.
Describe the solution you'd like.
Therefore i would like to modify the navigation bar and place the CSG logo at the left most corner followed by the name of the website.
Describe alternatives you've considered.
I think the placing the logo on navigation bar is itself enough and no further alternative is needed.
screenshot of the change suggested is attached hereby

Add any other context or screenshots about the feature request here.
No response

The current one looks good


Fix devcontainer/Codespaces

Moves to ubuntu-jammy
Fixes #377

That broke things. Can't spin up this repo on github codespaces:
2022-11-01 08:20:15.981Z: #12 5.106 Package httpie is not available, but is referred to by another package.
2022-11-01 08:20:15.981Z: #12 5.106 This may mean that the package is missing, has been obsoleted, or
2022-11-01 08:20:15.981Z: #12 5.106 is only available from another source
2022-11-01 08:20:15.981Z: #12 5.106
2022-11-01 08:20:15.981Z: #12 5.109 E: Version '3.2.1' for 'httpie' was not found

Looks like https://github.com/githubtraining/training-manual/pull/378/commits/555cf5193fc6739445018afbf7ac3656dad94aa1 broke it by specifying the HTTPie version to appease the linter. Should be fixed now. Can you confirm?

Yeah, no version fixes things.

Commit githubtraining/training-manual@1249f63 should have fixed the problem without removing the HTTPie version. Did you try it before removing the version?

I did, computer still said no.

Hello, Thank you for updating codespaces environment. I could update my devcontainer environment to Ubuntu jammy with these updates. I hope this can help you.

Rebuilt the codespace from scratch by deleting it, works now!


Best Library EVER !!!

Thank you very much

Thank you very much!
This is the best issue I've seen 😆


Touchscreen calibration not working

@DustinWatts do you happen to have the values for the calibration array handy?
While uploading new contents for the data directory (while iterating on #57) I seem to have accidentally wiped the existing config 😢
If you have them handy I can just overwrite the existing values 👍

I forgot to add: While rebooting the device prompts for me to calibrate the screen, but it doesn't recognise, respond to, or store any boundary information, so touch doesn't work after boot is complete

I do not have that data ready for you as that is completely done by code and always lives in the SPIFFS. But you can re-calibrate whenever you want by typing cal reset (technically cal is enough) in the serial monitor (No Line Ending). This will erase the calibration file, reboot, and start the calibration over again.
Just realized, are you using the ESP32 TouchDown? If so, you do not need calibration as this is the capacitive touch screen. If you uncomment #define USECAPTOUCH works without calibration (just make sure you have the FT6236 library installed).

No problem, I though I remembered you were using the ESP32 TouchDown :)


fix: Make content script matches optional

This better supports adding content scripts via the scripting APIs while using registration: "runtime". Updated the docs to reflect this.

Released in v0.19.17

Released in v0.19.17

It didn't work out. Although the ts type allowed optional matches, an error was reported when packing

@chen99-long Fixed in https://github.com/wxt-dev/wxt/pull/1306, released in v0.19.23


WIP added component for result entries

Coverage decreased (-0.01%) to 46.789% when pulling 89d53a393e255ac85fc03db10562b698417cc316 on DAME-August-144-long-title-truncation into 3867bd72554758c1bf03469172b5ce6e144b3c46 on DAME-August-staging.


Category / Stock location selector usability

I can't say for everyone but for me new double-line selector is inferior to the previous one.
New one is just name sorted while previous was tree based which is more logical and suitable for tree structures.
At the same time it looks very nice for plain tables like suppliers or manufacturers (tiny logos are cute).
Please consider returning to the old selector in parts and stock items.

@sintech can you please share a screenshot of the particular form you're talking about?

My bad, that's what I'm talking about:

New, only three rows at once:

Old (from admin, but it looked the same on site):

I would add some space around '/' but anyway it is better than new one.

Ah I see. I've made an improvement in #1833, certainly ordering the results in a "tree" fashion makes a lot of sense.


Automatically create a ngrok internet accessable kafka broker

You might want to check this out.  https://rmoff.net/2023/11/01/using-apache-kafka-with-ngrok/

I also built one that uses ngrok with debezium and kafka.  https://github.com/alberttwong/onehouse-demos/tree/main/postgressql-debezium


CI: Replace WaitForXXX helpers by kubectl wait

It looks like we should be able to replace most of your WaitforXXX helper functions in test/helpers/kubectl.go by calls to kubectl wait --for=... -l ... --timeout=.... This was suggested in https://github.com/cilium/cilium/pull/12658#discussion_r517901918.

Not sure if we still want to tackle this given the ginkgo stack is in maintenance mode (though still in use). According to https://github.com/cilium/cilium/pull/19666, kubectl wait ... doesn't wait for Pods to be actually terminated.


Only process applicable repos for suborg setting change

The previous code did not handle suborg setting change efficiently. The PR has the optimization to only process the repos that are applicable for a suborg change and skips processing other repos.
This will reduce the number of API calls being made for suborg setting changes,
The PR also includes some cleanup and NPM package updates.

The logic  on the event handler for push will fail to do the right thing if multiple changes are done on a single commit and we may lose needed synchronizations
eg (some of the failing conditions):

settings.yml + one suborg
repo config + suborg (only repo is handled)
Multi repo configs (only first one is handled)
multiple suborgs (only first one is handled)

You know how about someone joining me in getting back at the people who are doing this to me. My wife killed herself and ever since everyone I know has bleed my dry , left me with NO ONE TO TRUST . I HAVE NO ONE.. I AM TO THE POINT OF SHOOTING MYSELF. SO THANKS FOR NOTHING. What or why do I care , I fell in love With Cheyenne.  If you find my car they have stolen give it to my son... I loved my wife.. And I love my children .. And u love you Cheyanne.  And I knew about the spy app.  But I will kill myself now , or soon.. I am sorry I was never good enough


JAMES-3777 Incremental events for Filters

Goals: capture most changes of the filtering aggregate to reduce the complexity of this aggregate.
This work (in progress) proposes an incremental update that covers:

Rules addition at the beginning of the rule set or at the end
Rules deletion
Rules modification

Actions falling outside of this scope cannot be captured in an incremental fashion and would lead to a full reset (todays behaviour). This is the case when:

Rules are being reordered.
Rules are added in the middle of existing rules.

I suspect no UI allows this. Hence this is more that likely enough.
Work remaining

[x] Plug the Incremental events into the aggregate
[ ] Serialization for this incremental event
[x] Conditionally disable incremental change via ENV and explain that it should be disable during rolling update of the upgrade.
[x] Tests with 1.000 rule changes
[ ] Rebase the subscriber JAMES-3777: subscriber first might need to load the aggregate again?

Benchmarks
The bench
Create 1.000 sequencially, one by one:
    @Test
    void test(EventStore eventStore) {
        FilteringManagement testee = instantiateFilteringManagement(eventStore);

        ImmutableList.Builder<Rule> rules = ImmutableList.builder();

        final Stopwatch gstopwatch = Stopwatch.createStarted();
        for (int i = 0; i < 1000; i++) {
            final Stopwatch stopwatch = Stopwatch.createStarted();
            rules.add( Rule.builder()
                .id(Rule.Id.of("id-subject" + i))
                .name("name")
                .action(Rule.Action.of(Rule.Action.AppendInMailboxes.withMailboxIds("mbx1")))
                .condition(Rule.Condition.of(
                    Rule.Condition.Field.SUBJECT,
                    Rule.Condition.Comparator.NOT_CONTAINS,
                    "A value to match 2"))
                .build());

            Mono.from(testee.defineRulesForUser(USERNAME, rules.build(), Optional.empty())).block();

            System.out.println("Step " + i + ": " + stopwatch.elapsed(TimeUnit.MILLISECONDS));
        }
        System.out.println("Total: " + gstopwatch.elapsed(TimeUnit.MILLISECONDS));
    }

Results: before
Failed at step 964 after 36 minutes (!!!) -> Cassandra timeout (2s)
Step 963 took 14.7s (for storing a single rule!)
Cassandra storage is very well compressed (biiiiig redundancy in the data) yet still the aggregate is big: 7MB on-disk, 210 MB uncompressed.
As such, long histories is clearly non-viable.
Results: after
Storing the 1.000 rules sequentially took roughly 1 minute with last step taking only 95 ms.
Cassandra storage is well compressed (twice less) yet still the aggregate is smaller: 26KB on-disk, 400KB uncompressed.
Event running 5000 updates the 5000+ update, the latest step is done in ~500ms.
Long histories for filter are thus more viable.
Further improvements in long histories can be unlocked via complementary approaches like event sourcing snapshoting.

Snapshots
The snapshots allows to easily improve insertions.
Before above benchmark for 5000 rules: 31 min 42 with last rue creation taking 954 ms.
After 6 min 18 with last rule creation taking 124 ms
TODO:

[ ] Split in 2 PRs
[ ] Tests for snapshots


Send Error Message is not Working

I'm currently working error handle on express. but there is problem that does not send status text well.

I set status text to TEST Code but status text is Not Found.

By using res.send you can set only the response body, not the status message.
A working answer to your problem can be found in this Stack Overflow answer. The correct way to set status message is modifying the statusMessage property of the underlying Node.js response object.

By using res.send you can set only the response body, not the status message.
A working answer to your problem can be found in this Stack Overflow answer. The correct way to set status message is modifying the statusMessage property of the underlying Node.js response object.


Update building-simple-cmdset-with-dialog-api.md

Updating the title of this section. The app is not hosted by localhost in this tutorial, so I have updated the documentation to match.
This is to fix #4577
Category

[x] Content fix
[ ] New article

Related issues:

fixes #4577

What's in this Pull Request?
Updating Title in document
Guidance

thx @bcameron1231, appreciated.


Menu Overview

HugoBot Overview - See Attached Files

HugoBot-OverView-v0.1a.zip

HugoBot Overview - See Attached Files

HugoBot - Menu Overview - Updated


README and CHANGELOG stuff

Clean up the README some.

This includes dropping the build-status shield which was reading from Travis; it'll go back in once we hook it properly into GitHub Actions.
This also includes tweaking the version shield to pull from a little JSON file which will need updating pretty much the moment we merge this.

Drop everything prior to 2.0 from releaseNotes.yml.
Make sure that everything prior to 2.0 is in CHANGELOG.tpl.
Make sure that we fix the date in releaseNotes.yml when we fix it in CHANGELOG.md, so that the next make generate doesn't shred things.
Allow make release/start to work if it doesn't really need to do anything (since the start of the release checklist invokes it).
Finally, hopefully fix the apro updater.

Forcing DCO because, uh, all of these commits have been signed off. WTF. 😂

Closing in favor of #3829, since I edited the wrong branch by mistake. Ooops.


Implement file replacing during upload with optional replacing

Implementing a feature that allows for replacing a file during upload without deleting the original file first. The replacing of the original file should be optional and controlled by a parameter in the uploadData method.

PR resolving this also should fix #192


Add API type conversion

Currently the rules specified in the configuration use explicitly version v1alpha2 of the API. This means that if in the future we introduce and support a new version of the API we will have to stop supporting v1alpha2, or else explicitly convert those types to the new version of the API. To improve that this patch uses the Kubernetes API version conversion mechanisms to deserialize the rules. That way the user will be able to specify in the configuration file the version of each rule. For example:
rules:

- apiVersion: autoheal.openshift.io/v1alpha2
  kind: HealingRule
  metadata:
    name: my-old-rule
  ...

- apiVersion: autoheal.openshift.io/v1
  kind: HealingRule
  metadata:
    name: my-new-rule
  ...

The default values for apiVersio will be autoheal.openshift.io/v1alpha2 so users don't need to explicitly indicate it.

@zgalor @yaacov @moolitayer @nimrodshn @elad661 please review.

/lgtm

/lgtm


Bytecode interpretation for Ebisp

https://bernsteinbear.com/blog/bytecode-interpreters/
the article explains bytecode interpreters for lisp and can be used to improve ebisp.

Huh, funny seeing my post in the wild. Did you ever get around to poking at it?

@tekknolagi nah, not really, sorry! This project is in deep hiatus for a while. :) Thanks for the post tho! :+1:


rename project to Kargo Render

Fixes #181
This PR renames the project to Kargo Render.
The new configuration file (previously Bookfile.yaml) is kargo-render.yaml.
New package name is render, so some exported types like RenderRequest and RenderResponse got renamed to avoid the so-called "stutter" of render.RenderRequest, etc.
There are no functional changes in this PR.
Some non-code follow-up work may be required -- corresponding changes to CodeCov, Netlify, etc.

Holding for additional LGTM from @jessesuen


Create codeql-analysis.yml

Description
New tool, Bug fixing, or Improvement?
Please include a summary of the change and which issue is fixed. Also include relevant motivation and context.
For new tools, please provide total size and installation time.

Related issue:
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

@aloviedo is it possible to add the script to the exclusions? We don't use it during the image generation, it is needed only for VM provisioning process and we do not store any passwords in it
https://github.com/actions/virtual-environments/blob/main/images/macos/provision/bootstrap-provisioner/kcpassword.py

.>

Closed in favour of https://github.com/actions/virtual-environments/pull/5411


NixOS - squash logo, add new package location.

Logo was too tall before, normalized based on ubuntu and manjaro logo.
There is additional directory where user "packages" can be located - when you you set home-manager.useUserPackages = true;
Even tho nix-store -q accepts multiple paths - if one of them doesen't exist it fails without any stdout output, so i had to wrap in a function.

There is a mistake in the code for macOS. I think it should be manager=nix-user   && tot nix-user-pkgs instead of manager=nix-user   && tot nix-store -qR nix-user-pkgs.

There is a mistake in the code for macOS. I think it should be manager=nix-user && tot nix-user-pkgs instead of manager=nix-user && tot nix-store -qR nix-user-pkgs.

You are right, thanks for noticing!


Behavioral pattern analysis

Analyze wallet behaviors on the Arbitrum ecosystem to identify patterns indicative of a) Sybil accounts or b) fraudulent activities
The analysis and EDA can include deep dives into -

transaction data - transaction history, frequency, and types of transactions
wallet balance data & changes
Token Flow Analysis - related to airdrops and more use cases where sybil/fraud has potential. Exchange Withdrawal Patterns - both cex & dex

This is not an exhaustive list and can have multiple areas depending on the initial findings and insights with the EDA here. Find out the top patterns or known use cases based on the analysis.

Thank you!  THis is excellent.  I hope you agree that issue #15 and others may address this.  We WILL seek to reward analysis of this type done.


envoy: Update envoy version to 1.25.x

Description
Envoy 1.24 will be EOL in Oct 2023, this commit is to proactively bump envoy version to 1.25.
Related build: https://github.com/cilium/proxy/actions/runs/6160387030/job/16717229803
Note
This PR contains the backport of https://github.com/cilium/cilium/pull/24684 to pickup the fix 466ab24e261c6093ba9fbacbe24b6160ca6326b9 for below issue.
2023-09-29T15:02:01.513660624Z level=warning msg="NACK received for versions after  and up to 3; waiting for a version update before sending again" subsys=xds xdsAckedVersion= xdsClientNode="host~<IP_ADDRESS>~no-id~localdomain" xdsDetail="Only unique values for domains are permitted. Duplicate entry of domain foo.bar.com in route default/cilium-ingress-default-host-rules/listener-insecure" xdsNonce=3 xdsStreamID=6 xdsTypeURL=type.googleapis.com/envoy.config.route.v3.RouteConfiguration

/test-backport-1.13

/test-backport-1.13

/test-backport-1.13

/test-backport-1.13

/test-backport-1.13

/test-backport-1.13


Error returned from daemon: Error Domain=com.apple.accounts Code=7

I installed FMPhotoPicker on my project, but I get this error when try to apply filters and app crash.

It seems some kind of conflict with SceneDelegate.swift file.

@ucelme
I couldn't reproduce this error on my side.
Can you provide the sample project?

Feel free to reopen if you still need it.


Range downloads design

After #8611 got merged there is an opportunity to implement #622.
The problems are:

The main question we should decide is whether we want range downloading scenario by default.
We need to notify the user in some way about what actually happens, whether we are downloading a fragment rather that the whole video or not (something like --no-playlist message analogy).
For DASH there is a high probability of async in resulting muxed file due to approximate audio/video fragment boundaries. Most likely we'll need to fallback to best is case of range download.
We probably need an option to enable/disable range download similar to e.g. --no-playlist.

As for me we the key question is whether this should be default or not. According to it we have the following possible outcomes:

Download fragment by default when possible and output a warning message that is only fragment is being downloaded and suggest using a --no-range-download option (name is negotiable) to download the whole video.
Download whole video by default and output a warning message suggesting using --range-download to download only the range

I'm not sure which one is better. We can also add and probably should add both options.
Any ontopic opinions and thoughts are welcome.

i will put my optinion about some of this points:

The main question we should decide is whether we want range downloading scenario by default.

i think this is different from --no-playlist(the url contain information about video and playlist in the same) while in range url there is no ambiguity(the url clearly contain information about time):
http://www.youtube.com/watch?v=BaW_jenozKc&t=1s&end=9
so it should be the default to respect the download url information.

For DASH there is a high probability of async in resulting muxed file due to approximate audio/video fragment boundaries. Most likely we'll need to fallback to best is case of range download.

from man page of ffmpeg it said that:

When transcoding and -accurate_seek is enabled (the default), this extra segment between the seek point and position will be decoded and discarded. When doing stream copy or when  -noaccurate_seek is used, it will be preserved.

or

if we pass the info dict directly to the downloader and let ffmpeg download all files it shoud sync the files(need some tests).
example:
ffmpeg -t start -ss duration -i video_url -i audio_url -c copy out.ext

or using what you've suggested:

Most likely we'll need to fallback to best is case of range download.

i think this is different from --no-playlist(the url contain information about video and playlist in the same) while in range url there is no ambiguity(the url clearly contain information about time)

Basically it not about ambiguity but about what user expects. When you grab a video URL from somewhere you have no idea whether it contains time marks or not and may expect downloading of the full video

One argument I find for not making it the default is the following: you open a link like https://www.youtube.com/watch?v=BaW_jenozKc&t=2s on a browser and I decide to watch the video from the start, the URL is not modified (by their video player) and you copy the link from the location bar. If you pass it to youtube-dl now, wouldn't you expect it to download the whole video? This is not the case for different playlist or videos, as soon as you switch to a new one the URL is updated and when you copy it it refers to the video/playlist you are currently watching.
An additional reason is that because it uses ffmpeg the download is not recoverable, since that departs from the normal behaviour of youtube-dl I think it would surprise most users. And since it's an important change I don't think we should base the decision just on the URL having some specific parameters.
Personally I don't like too much that the parameters in the URL influence the file I get in the end, so I prefer if it isn't the default behaviour. But since I don't have know what most users would expect, my position is not too strong.

And since it's an important change I don't think we should base the decision just on the URL having some specific parameters.

than may be adding some option whould be better(for example: --start-time and --end-time or --duration) and ignore the params of the url, this will make it possible to support the feature in all extractors.

than may be adding some option whould be better(for example: --start-time and --end-time or --duration) and ignore the params of the url, this will make it possible to support the feature in all extractors.

To clarify my view: actually I'm not opposed to using the parameters of the URL , I'm opposed to automatically cutting the video if they are present (I'm fine with mpv using them, but there you can seek to anywhere you want so it's not harmful).
Since this is related to #622, your proposed --start-time and --end-time are quite useful anyway. So maybe --range-download tries to use the url parameters, while --start-time and --end-time override it or are used if the URL has no parameters. I had though of something like --download-range auto to use the URL parameters and --download-range start-end (start or -end would also be allowed), but then you can't just override one of the parameters from the URL (not sure if that's feauture is that useful).

okay I was referred here.
in case of youtube videos,we can instruct user to give the start time as --start [start_time], then the end time --end [end_time]  then a proper get request may help in retriving the parsed url
eg :
`_url= 'https://www.youtube.com/watch?v=2QPbriKs_sI'
_start=0
_end=1970
_api_version=3
_url+= '&start='+str(_start)+'&end='+str(_end)+'&version='+str(_api_version)
`
The problem I still get is it till shows me the full video url

as I played with:yt-dlp -f "(bestvideo+bestaudio/best)" --downloader ffmpeg --external-downloader-args "ffmpeg_i:-ss 00:43:00.00 -to 00:44:36.00 -http_proxy http://<IP_ADDRESS>:7890" "https://www.youtube.com/watch?v=pPdp1kXBiiA&list=PLdbzRdDJq2MzVmpUQrJXzlFq3kI49DNuD&index=2" --no-playlist --proxy "http://<IP_ADDRESS>:7890"
but Term returns
[tls @ 000001fe5257ec80] Failed to read handshake response [URL] : End of file \n\n\n ERROR: ffmpeg exited with code 1
What's wrong?


Cherry pick Mark boolean comparison kernels public to active_release

Automatic cherry-pick of bb05b00

Originally appeared in https://github.com/apache/arrow-rs/pull/913: Mark boolean comparison kernels public

Codecov Report

Merging #920 (6fc2a06) into active_release (03d95e6) will decrease coverage by 0.14%.
The diff coverage is 100.00%.

:exclamation: Current head 6fc2a06 differs from pull request most recent head 72d4a9f. Consider uploading reports for the commit 72d4a9f to get more accurate results

@@                Coverage Diff                 @@
##           active_release     #920      +/-   ##
==================================================
- Coverage           82.45%   82.31%   -0.15%
==================================================
  Files                 168      168
  Lines               48175    48091      -84
==================================================
- Hits                39725    39586     -139
- Misses               8450     8505      +55

Impacted Files
Coverage Δ

arrow/src/compute/kernels/comparison.rs
92.63% <100.00%> (ø)

arrow/src/array/transform/fixed_binary.rs
75.00% <0.00%> (-3.95%)
:arrow_down:

parquet/src/schema/visitor.rs
67.10% <0.00%> (-2.64%)
:arrow_down:

arrow/src/compute/kernels/cast_utils.rs
87.50% <0.00%> (-2.50%)
:arrow_down:

parquet/src/encodings/decoding.rs
90.45% <0.00%> (-2.02%)
:arrow_down:

arrow/src/datatypes/ffi.rs
70.85% <0.00%> (-2.02%)
:arrow_down:

parquet/src/arrow/arrow_reader.rs
89.32% <0.00%> (-1.95%)
:arrow_down:

arrow/src/ffi.rs
84.69% <0.00%> (-1.89%)
:arrow_down:

arrow/src/csv/reader.rs
90.00% <0.00%> (-1.78%)
:arrow_down:

arrow/src/util/data_gen.rs
75.44% <0.00%> (-1.66%)
:arrow_down:

... and 70 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 03d95e6...72d4a9f. Read the comment docs.


Error on first attempt to sign as AH and Recommend Ready as Staff

In both cases going back to the plan and trying again works.
signature error:

recommend ready error:

@calebissharp

@calebissharp I got an error message logging in as Leslie.knope to sign originally but I think it was a connection speed thing (there was a super long delay). I logged her out, did another Draft plan and couldn’t replicate. Also moved plan through to approved using recommend ready and didn’t get an error. Closing this one.


[ADD] new parts in readme

Added 'Uploading a file that taken by params' and 'Generate a CSV file and upload' parts in readme.md

@jhunschejones

Hi @atagunay! Thanks for taking the time to put this together. I pulled your example and some of mine into https://github.com/jhunschejones/pcloud_api/pull/10. Does that capture the gist of what you're after?

Closing in favor of https://github.com/jhunschejones/pcloud_api/pull/10


Fix Grammar in Landing Page

I have noticed some grammatical errors in the landing page text.

Corrections:
experiment -> experiments
quizzes question -> quizzes
 examination ->  examinations

I would like to work on this as part of GSSoC'23.


 My finding is that if you don't do that, you may not get all the test docs in your first pass, it takes some very shot time for ES to propagate your indexing changes before it makes them searchable.

Hi there - most of this plugin functionality is now supported natively in ES: https://www.elastic.co/guide/en/elasticsearch/reference/current/dense-vector.html
Also see https://www.elastic.co/blog/text-similarity-search-with-vectors-in-elasticsearch.
See also an update to related Code Pattern on IBM Developer that illustrates a recommendation use case: IBM/elasticsearch-spark-recommender#54


Error generating client code

As of today, we are getting the following error when trying to generate client code:
* What went wrong:
A problem occurred configuring root project 'data-graph'.
> Could not resolve all files for configuration ':classpath'.
   > Could not download graphql-java-client-runtime-1.18.7.jar (com.graphql-java-generator:graphql-java-client-runtime:1.18.7)
      > Could not get resource 'https://plugins.gradle.org/m2/com/graphql-java-generator/graphql-java-client-runtime/1.18.7/graphql-java-client-runtime-1.18.7.jar'.
         > Could not GET 'https://jcenter.bintray.com/com/graphql-java-generator/graphql-java-client-runtime/1.18.7/graphql-java-client-runtime-1.18.7.jar'.
            > Read timed out
   > Could not download graphql-java-common-runtime-1.18.7.jar (com.graphql-java-generator:graphql-java-common-runtime:1.18.7)
      > Could not get resource 'https://plugins.gradle.org/m2/com/graphql-java-generator/graphql-java-common-runtime/1.18.7/graphql-java-common-runtime-1.18.7.jar'.
         > Could not GET 'https://jcenter.bintray.com/com/graphql-java-generator/graphql-java-common-runtime/1.18.7/graphql-java-common-runtime-1.18.7.jar'.
            > Read timed out
   > Could not download graphql-java-spring-mvc-1.18.7.jar (com.graphql-java-generator:graphql-java-spring-mvc:1.18.7)
      > Could not get resource 'https://plugins.gradle.org/m2/com/graphql-java-generator/graphql-java-spring-mvc/1.18.7/graphql-java-spring-mvc-1.18.7.jar'.
         > Could not GET 'https://jcenter.bintray.com/com/graphql-java-generator/graphql-java-spring-mvc/1.18.7/graphql-java-spring-mvc-1.18.7.jar'.
            > Read timed out
   > Could not download graphql-java-18.0.jar (com.graphql-java:graphql-java:18.0)
      > Could not get resource 'https://plugins.gradle.org/m2/com/graphql-java/graphql-java/18.0/graphql-java-18.0.jar'.
         > Read timed out

Nothing was changed recently from our end.
plugins {
    id 'java-library'
    id "com.graphql_java_generator.graphql-gradle-plugin" version "1.18.7"
}

implementation 'com.graphql-java-generator:graphql-java-client-runtime:1.18.7'
implementation 'com.graphql-java:graphql-java-extended-scalars:19.0'

Hello,
The url used to retrieve the maven plug-in is wrong.
Do you have mavenCentral configured, like this:
´´´
repositories {
mavenCentral()
}
´´´
Étienne

Seems like the problem was resolved somehow and we are longer experiencing this issue.


Include simple-install for *nix users

It would be nice to have an install script for *nix users similar to the one for the message_ix repository, but for users who only want to install ixmp as a stand-alone project.

solved in #16


V-78995, dev_to_RHEL7r4 branch  - NEW control! Needs logic added!

V-78995, dev_to_RHEL7r4 branch  - NEW control! Needs logic added!
https://github.com/ejaronne/inspec-profile-disa_stig-el7/blob/dev_to_RHEL7r4/controls/V-78995.rb

Commit: https://github.com/ejaronne/inspec-profile-disa_stig-el7/commit/65186c61ec151ec9c40d4314d11e41120804e78b
Runs on both systems.


[support]: SuperLU fails built-in Spack test

Name of Software
SuperLU
URL for Software
https://portal.nersc.gov/project/sparse/superlu/
Contact Details
<EMAIL_ADDRESS>HPC System
NERSC Perlmutter
Request Description
SuperLU fails the self test. Cross-Ref to the Spack Issue: https://github.com/spack/spack/issues/38051
Looks like another permissions issue with the file creation, possible similar to https://github.com/E4S-Project/e4s/issues/125
@shahzebsiddiqui , is this an issue with the permissions on the folders Spack is running the tests in that we can fix on our end?
Relevant log output
==> Installed packages
-- linux-sles15-zen3 /<EMAIL_ADDRESS>-------------------------------
tnurd2ul2lqav4gc4nj3yla5yxdkyud6<EMAIL_ADDRESS>build_system=cmake build_type=RelWithDebInfo
==> 1 installed package

epalmer@perlmutter:login23:~> $ spack load superlu
epalmer@perlmutter:login23:~> $ spack test run superlu
==> Spack test r6smlz4irtbq3uoqdbojqmd7etj6aj4d
==> Testing package superlu-5.3.0-tnurd2u
==> Error: FileNotFoundError: [Errno 2] No such file or directory: '/global/homes/e/epalmer/.spack/test/r6smlz4irtbq3uoqdbojqmd7etj6aj4d/superlu-5.3.0-tnurd2u/cache/superlu/make.inc'

/global/common/software/spackecp/perlmutter/e4s-22.11/83104/spack/var/spack/repos/builtin/packages/superlu/package.py:142, in test:
        139
        140        # Write configuration options to make.inc file
        141        make_file_inc = join_path(self.test_suite.current_test_cache_dir, self.make_hdr_file)
  >>    142        with open(make_file_inc, "w") as inc:
        143            for option in config_args:
        144                inc.write("{0}\n".format(option))
        145

See test log for details:
  /global/homes/e/epalmer/.spack/test/r6smlz4irtbq3uoqdbojqmd7etj6aj4d/superlu-5.3.0-tnurd2u-test-out.txt

======================== 1 failed, 0 passed of 1 specs =========================
==> Error: 1 test(s) in the suite failed.

epalmer@perlmutter:login23:~> $ spack test results -l r6smlz4irtbq3uoqdbojqmd7etj6aj4d
==> Results for test suite 'r6smlz4irtbq3uoqdbojqmd7etj6aj4d':
==> test specs:
==>   superlu-5.3.0-tnurd2u FAILED
==> Testing package superlu-5.3.0-tnurd2u
==> [2023-06-01-08:37:17.763814] Completed testing

======================== 1 failed, 0 passed of 1 specs =========================

epalmer@perlmutter:login23:~> $ ml -t
craype-x86-milan
libfabric/<IP_ADDRESS>
craype-network-ofi
xpmem/2.5.2-2.4_3.47__gd0f7936.shasta
PrgEnv-gnu/8.3.3
cray-dsmml/0.2.2
cray-libsci/<IP_ADDRESS>
cray-mpich/8.1.25
craype/2.7.20
gcc/11.2.0
perftools-base/23.03.0
cpe/23.03
xalt/2.10.2
Nsight-Compute/2022.1.1
Nsight-Systems/2022.2.1
cudatoolkit/11.7
craype-accel-nvidia80
gpu/1.0

Reproduce Bug
1. Login to Perlmutter
2. module load e4s/21.11
3. spack env activate gcc
4. spack load superlu
5. spack test run superlu
...

I acknowledge this issue. We will investigate and find a solution. Thanks for reporting this issue.

This issue should be resolved with changes to both spack test and specifically the superlu-dist Spack merged to upstream Spack.

Closing this issue as resolved. If you find the issue is not resolved, we will re-open.


Fixes tree shaking

What

Forces es6 modules with the babel env preset
Declared no side effects in package.json - this is mostly for migration towards webpack 4.
Removes babel-loader query, which was negating .babelrc babel-env settings.

Why

Tree shaking was broken

GIF

Thanks for the PR! The tests fail because of the modules: false I believe 🤔
I'll take a look tmrw 😄

Ok looks good to me. I'll merge when I get back home :pray:

Sick thanks @mrozbarry 🎉


feat: add Astro example shoe store

A new apps/astro app has been added. I've tried to keep it as simepl as possible, and as close to the other shoe stores as I could. This means that the markup is copy/pasted with only very few changes (removal of key attributes, renaming className to class, etc). Some of the markup is rendered conditionally based on a loading flags, and even though these will always evaluate to false for now, I decided to keep them. When we add visual editing, we'll begin to need them. But for now, the pages are rendered using SSR with no need for loading states.

I've added https://vercel.com/sanity-io/visual-editing-astro. When this PR is merged, then we can adjust that project so it points to the correct root dir.


Problem with CameraIntrinsics

Hi,
I am starting on unity and openCV. When I import your project and import the other unitypackage needed I get this error. Do you have an idea how to solve it?

Paul

Based on the error message you sent me about CameraIntrinsics not be found, I'm guessing that the error is due to HoloLensCameraStream (https://github.com/camnewnham/HoloLensCameraStream) not importing it.

Thanks for your answer!
I tried to import HoloLensCameraStream by following the tutotrial on the github of camnewnham. I did it but i still have the same error :/ Do you have an another idea ?

I tested it again in my environment and the project setup was successful.
My environment :
Windows 10
Unity 2018.4.28f1
I found that the order in which the assets are imported is important during the project setup process.
I also found out that enabling the "Enable MSBuild for Unity" option in the "MRTK Project Configurator" window in step 5 will break the project. MRTKProjectConfigurator window
Please be aware that this condition cannot be fixed later.
When redoing the setup process, start with a clean slate.
Detailed Instructions:

Download the latest release unitypackage. HoloLensWithOpenCVForUnityExample.unitypackage
Create a new project. (HoloLensWithOpenCVForUnityExample)
Change the platform to UWP in the "Build Settings" window.
Import the OpenCVForUnity.
Setup the OpenCVForUnity. (Tools > OpenCV for Unity > Set Plugin Import Settings)
Move the "OpenCVForUnity/StreamingAssets/haarcascade_frontalface_alt.xml" and "OpenCVForUnity/StreamingAssets/lbpcascade_frontalface.xml" to the "Assets/StreamingAssets/" folder.
Clone HoloLensCameraStream repository.
Copy the "HoloLensCameraStream/HoloLensVideoCaptureExample/Assets/CamStream/" folder to the "Assets/" folder.
Set the scripting backend of the plugin inspector to "Any Script Backend". (IL2CPP support)
Import the Microsoft.MixedReality.Toolkit.Unity.Foundation.2.2.0.unitypackage.
Setup the MRTKv2. (Mixed Reality ToolKit > Utilities > Configure Unity Project)
Do not enable the "Enable MSBuild for Unity" option in the "MRTK Project Configurator > Modify Configurations" tab.
Import the HoloLensWithOpenCVForUnityExample.unitypackage.
Add the "Assets/HoloLensWithOpenCVForUnityExample/*.unity" files to the "Scenes In Build" list in the "Build Settings" window.

It didn't work i don't know why :(
My environment:
Windows 10
Unity 2018.4.31
MRTK 2.2.0
The last version of OpenCV for Unity
I followed your instruction even MSbuild. I tried 5 times.
I have more errors :(

Maybe i don't use the good unitypackage or files are missing for hololens ...

project file structure:

The errors up to the fourth one are caused by the fact that the CametaStream example code contains an old Unity API, so you can either let UnityEditor fix it automatically or delete the "CamStream/Examples" folder.
The CameraIntrinsics definition is contained in "CamStream/Plugins/WSA/HoloLensCameraStream.dll".
Could you make sure that the file size of "HoloLensCameraStream.dll" is correct?
HoloLensCameraStream.dll = 30,720 bytes
In addition, Could you use VisualStudio's Object Browser to make sure that the namespace of HoloLensCameraStream contains CameraIntrinsics?

My HoloLensCameraStream.dll = 27,000 bytes
i don't have cameraIntrinsics

I think you apparently downloaded the "CameraStream" asset file from a different repository than my development environment.
The "HoloLensCameraStream.dll" file looks obviously wrong. It appears to be from the VulcanTechnologies repository, as there is no CameraIntrinsics definition.

Thank you for your help ! :)
I have another question if i want to detect two differents markers at the same time
i juste have to duplicate this part ?

Multiple markers can be detected at the same time if they belong to the same dictionary ID.
However, the HoloLensArUcoExample limits the number of AR objects that can be displayed to one. In order to display AR objects corresponding to multiple detected markers, you will need to modify the code.

Hello @Patsufr, I know this is an old thread but i'm encoutering the same issue.
What was wrong ? and how did you solved it.
Best Regards.

If m’y memoris are good you need to download the right CameraStream from here https://github.com/camnewnham/HoloLensCameraStream/archive/master.zip

I need the dll to run this Project : https://github.com/EnoxSoftware/HoloLensWithOpenCVForUnityExample
due to several problems in importing the project, I used the dll from HoloLensVideoCaptureExample, and I put it under Assets/OpenCVForUnity/Plugins/WSA in the project in the link above.
PS: I'm using OpenCVForUnity v 2.4.7.

@Patsufr I need the dll to run this Project : https://github.com/EnoxSoftware/HoloLensWithOpenCVForUnityExample
due to several problems in importing the project, I used the dll from HoloLensVideoCaptureExample, and I put it under Assets/OpenCVForUnity/Plugins/WSA in the project in the link above.
PS: I'm using OpenCVForUnity v 2.4.7.

Sorry i didn’t touch This library since a moment!
I Know i it was difficult to import correctly all package ! Try again to import the correct package lt’s the only issue i think !

Okay Thanks. @EnoxSoftware Could you suggest any solutions?

I recently set up this example from scratch for another matter and it worked fine.
I think Patsufr's advice is valid.


:robot: Triggering CI on branch 'release-next' after synching to openshift/master

/retest

/test e2e

OCF Webhook is merging this PR


Universal rendering

Hello!
I've allready asked it here, and your wrapper seems to be just working out of the box without swiper modifications.
But it calcuates the wrong sections height on the first render, just the same way. Any ideas?

As I mentioned, it becomes normal after resizing of the page. Thanks!

Interesting. Yeah, the point of the SwiperLib wrapper was to allow for SSR (iDangerous-Swiper looks for window and other DOM APIs so can't be loaded in Node without erroring).
The first thing that comes to mind would be to force iDangerous-Swiper to run its resize/update hooks on mount. The thing that's seems odd, though, is that these should run during initialization. Anyhow, the API for these methods is documented here. There are a few ways to access the swiper instance from react-dynamic-swiper documented here  -- particularly calling .swiper() on a ref to the Swiper component (from react-dynamic-swiper) will return the current iDangerous-Swiper instance. Invoking onResize() or update() (or any of the sub-update operations) may be a quick-solve for you.
It would be nice to know if any of the methods I described above work, but also why the initialization after SSR isn't calculating sizing correctly. The hope of react-dynamic-swiper is to support SSR, so I will look into this.

I'm having trouble re-creating this issue, SSR seems to be working fine for me. Could you please provide more details on the Swiper options you were using?

Hey guys, I am using this library in production. Does it support universal rendering currently?

@rahulbasu710 This library was intended to be used as such. The iDangerous Library (which depends on DOM) is wrapped as to not be loaded in a Node.js (or really, non-DOM) environment. That said, @surepe was having issues which I have not been able to re-create. As you can see, the issue here is technically on the client. Still not positive if this is because of SSR or otherwise. @rahulbasu710, Out of curiosity, have you encountered any issues thus far?

Hey Nick,
I was using it till now with the swiper init after componentDidMount or think of it as showing the Swiper after that as the data was fetched post the mounting. But now I am using API to render the swiper data universally as well.
I remember there was a problem that the swiper used to be close to each other and after init it used to load it's full functionality like the arrows etc.
I was wondering if that has changed. Also do you mind having a quick chat at Gitter sometime you are free. We can discuss and make it better if there are room for incremental changes.
But the library is awesome as always
Thanks!

Also I recently got stuck with this for some babel problem
.babelrc
{
  "presets":[
      "env", "react", "stage-0", "es2015"
  ],
   "compact" : true,
  "env": {
    "development": {
      "presets": ["env", "react", "stage-0", "es2015"],
      "plugins": ["transform-runtime", "transform-object-rest-spread"],
      "presets": ["react-hmre"]
    }
  },
  "env": {
    "build": {
      "presets": ["env", "react", "stage-0", "es2015"]
    }
  }
}

terminal error
ERROR in ./node_modules/react-dynamic-swiper/lib/index.js
Module build failed: ReferenceError: Unknown plugin "transform-object-assign" specified in "/Users/rahulbasu/Desktop/repository/beckfriends-heroku/node_modules/react-dynamic-swiper/.babelrc" at 0, attempted to resolve relative to "/Users/rahulbasu/Desktop/repository/beckfriends-heroku/node_modules/react-dynamic-swiper"
    at /Users/rahulbasu/Desktop/repository/beckfriends-heroku/node_modules/babel-core/lib/transformation/file/options/option-manager.js:180:17
    at Array.map (native)
    at Function.normalisePlugins (/Users/rahulbasu/Desktop/repository/beckfriends-heroku/node_modules/babel-core/lib/transformation/file/options/option-manager.js:158:20)
    at OptionManager.mergeOptions (/Users/rahulbasu/Desktop/repository/beckfriends-heroku/node_modules/babel-core/lib/transformation/file/options/option-manager.js:234:36)
    at OptionManager.init (/Users/rahulbasu/Desktop/repository/beckfriends-heroku/node_modules/babel-core/lib/transformation/file/options/option-manager.js:368:12)
    at File.initOptions (/Users/rahulbasu/Desktop/repository/beckfriends-heroku/node_modules/babel-core/lib/transformation/file/index.js:212:65)
    at new File (/Users/rahulbasu/Desktop/repository/beckfriends-heroku/node_modules/babel-core/lib/transformation/file/index.js:135:24)
    at Pipeline.transform (/Users/rahulbasu/Desktop/repository/beckfriends-heroku/node_modules/babel-core/lib/transformation/pipeline.js:46:16)
    at transpile (/Users/rahulbasu/Desktop/repository/beckfriends-heroku/node_modules/babel-loader/lib/index.js:50:20)
    at Object.module.exports (/Users/rahulbasu/Desktop/repository/beckfriends-heroku/node_modules/babel-loader/lib/index.js:175:20)
 @ ./src/containers/Home/index.js 1:1364-1395
 @ ./src/routes.js
 @ ./src/client.js
 @ multi ./src/client.js

Sorry man resolved this. Had to add the babel-plugin-transform... in the .babelrc and I think it works universally like your demo as of now.
Thanks man!

Awesome! Glad you're making use of react-dynamic-swiper!


Publish to npm

I see you have a package.json in the directory, but I don't think you've published the actual package because I get a 404 at https://www.npmjs.com/package/FontLoader.
Could you publish this?

Done!


Cross-Platform App using Python

MicroByte Title: Cross-Platform App using Python
Domain: Python, Usage of APIs, Kivymd, a bit of UI/UX
Scope of learning for learners: This micro byte will walk you through the basics of building an app in Python. Here I will briefly discuss various components to be used, how to fetch data from API, and process in the background to be presented to the user. All the code will be written in Python and a bit of kivy lang which is very easy to understand. I will not explain the conversion part (python file to apk) in detail and just an overview as that is a wide domain and requires a separate byte but the application build is cross-platform.

A very good example of this is PyDelhi 2019. They build an android app in Python for their annual conference!
I build a university result portal android app which fetches data of around 13k students

Activities: Some activities include:

Basic layout of the app
Binding input with output
Integrating API with I/O
How to convert it to APK

Hey @kaustubhgupta you can start working on this.
Keep in mind the learn by doing approach and the guidelines for creating the PR as you might be aware of already
All the best!


Publish as npm module

Very curious in your approach, wanted it to be directly from npm... especially given the GPL licensing.  In the mean time I'll try building locally.

Please see https://github.com/rhashimoto/wa-sqlite/issues/12
You can avoid building by fetching the buildless branch.

My apoligies


Cleanup long offset in completion protocol

Cleanup long offset and always use String offset in completion protocol
Long offset was deprecated in #5486, before 0.5.0 release.

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

Project coverage is 0.00%. Comparing base (59551e4) to head (111a68f).
Report is 1398 commits behind head on master.

:exclamation:  There is a different number of reports uploaded between BASE (59551e4) and HEAD (111a68f). Click for more details.
HEAD has 53 uploads less than BASE

Flag
BASE (59551e4)
HEAD (111a68f)

integration
7
1

integration2
3
1

temurin
12
1

java-21
7
1

skip-bytebuffers-true
3
0

skip-bytebuffers-false
7
1

unittests
5
0

unittests1
2
0

java-11
5
0

unittests2
3
0

integration1
2
0

custom-integration1
2
0

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #14557       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436        3     -2433
  Lines        133233        6   -133227
  Branches      20636        0    -20636
=============================================
- Hits          82274        0    -82274
+ Misses        44911        6    -44905
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <ø> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <ø> (ø)

java-11
?

java-21
0.00% <ø> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <ø> (-61.75%)
:arrow_down:

skip-bytebuffers-true
?

temurin
0.00% <ø> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


fix: downstream type inference for Float/Int schemas

This is necessary because of an issue with type-inference for downstream buildable libraries which emit .dts.  Without exporting these brands, TypeScript can't know what the underlying types are.  This became an issue introducing the type alias for float / int default values.  Changing that to an interface also fixes the issue, but Opaque from type-fest is a type-alias and hence cannot be upstream to an interface.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2597

Relevant Lines:
2597

💛  - Coveralls


Disable BZip2 when building libarchive

Summary: Building libarchive on a system that has bzlib installed
currently fails since CMake detects it and enables it but the
sysroot doesn't include bzlib. Since we don't rely on bzip support,
lets explicitly disable it in the CMAKE config.
Relevant Issues: N/A
Type of change: /kind bug
Test Plan: libarchive now builds successfully on my machine

@pixie-io-buildbot retest this please.


Keep uploading images in background on iOS

I'm using your plugin to upload images to a server, and I was wondering if there is a way to keep uploading images in the background on iOS, after an app is minimized?
Now the upload stops instantly when minimizing the app.

Hi, that's currently not supported by this plugin. There are other plugins that are specifically made for uploading in a background service.


clean up undefined in our logs

see log files.  eg:  POST /v1/escrow/verify undefined
Either remove the undefined parameter or get good data to appear.

Closing issue as part of project shutdown


go/format cmd/gofmt: requires 2 pass for multiline comments with spacing

Go version
Go 1.22
Output of go env in your module/workspace:
GO111MODULE='on'
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/davewalker/Library/Caches/go-build'
GOENV='/Users/davewalker/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/davewalker/src/go/pkg/mod'
GONOPROXY='github.com/Workiva'
GONOSUMDB='github.com/Workiva'
GOOS='darwin'
GOPATH='/Users/davewalker/src/go'
GOPRIVATE='github.com/Workiva'
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/opt/homebrew/opt/go/libexec'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/opt/homebrew/opt/go/libexec/pkg/tool/darwin_arm64'
GOVCS=''
GOVERSION='go1.22.0'
GCCGO='gccgo'
AR='ar'
CC='cc'
CXX='c++'
CGO_ENABLED='1'
GOMOD='/Users/davewalker/src/go/src/github.com/Workiva/fmttest/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/mv/8cw5jx652lg0hyrqjn28dctc0000gs/T/go-build2871199811=/tmp/go-build -gno-record-gcc-switches -fno-common'

What did you do?
package main

/*
    This is a multiline comment with
    things
  */
func A() {}

The file above needs to have go fmt run twice on it to be formatted properly.  Discovered via go/format package with format.Source() where read in AST comments are improperly formatted after a single pass.
What did you see happen?
After the first go fmt
package main

/*
	This is a multiline comment with
	things
*/
func A() {}

After the second go fmt
package main

/*
This is a multiline comment with
things
*/
func A() {}

What did you expect to see?
An idempotent formatter that can format source in one pass.

seems to be single pass? https://go.dev/play/p/a1kbFn5Nn0Y

I am not seeing the behavior you show on go.dev, via the command line, or through format.Source on my mac.
davewalker@WK01450:~/src/go/src/github.com/Workiva/fmttest › cat example.go
package main

/*
    This is a multiline comment with
    things
  */
func A() {}
davewalker@WK01450:~/src/go/src/github.com/Workiva/fmttest › go fmt example.go
example.go
davewalker@WK01450:~/src/go/src/github.com/Workiva/fmttest › cat example.go
package main

/*
        This is a multiline comment with
        things
*/
func A() {}
davewalker@WK01450:~/src/go/src/github.com/Workiva/fmttest › go fmt example.go
example.go
davewalker@WK01450:~/src/go/src/github.com/Workiva/fmttest › cat example.go
package main

/*
This is a multiline comment with
things
*/
func A() {}
davewalker@WK01450:~/src/go/src/github.com/Workiva/fmttest ›

See also, behavior of this issue
https://go.dev/play/p/SKkAi6di5QK


Write a Java program to find volume of a cuboid

Description
Write a Java program to find volume of a cuboid
Volume of a cuboid = Length × Width × Height
Length : 2
Width  : 3
Height : 4

Volume : 24

How to contribute

Save the solution in FindVolumeOfACuboid.java file
Add FindVolumeOfACuboid.java file in find-volume-of-a-cuboid folder

Hi , sir I have gone through the problem you mentioned and it seems that its also match my interest ,so please assign this project to me .
Your's Faithfully,
Ashank Ritwik

Hi , sir I have gone through the problem you mentioned and it seems that its also match my interest ,so please assign this project to me . Your's Faithfully, Ashank Ritwik

Hey @ashankritwik07
Thanks for showing your interest in this project 😍
You can simply comment !assign on the issues you want to contribute, you will be auto-assigned
!assign

!assign


Java Agent docs refactor

https://docs.google.com/document/d/1f761dYCNFTPJpAbLJAdmMFhwr9uy1j9gSrEmPdqnmHw/edit

We try to keep issues for things that can be worked on and the issue can be closed eventually, we have a discussion for that already, let's figure out the next steps there and create issues afterwards:
https://github.com/open-telemetry/opentelemetry.io/discussions/4853


Export types publicly

closes #86

Is this related to this pull request? https://github.com/jonathanpmartins/v-money3/pull/84


Bootstrapper to get paket.exe from NuGet.org if GitHub is down.

Bootstrapper takes paket.exe from GitHub. But today GitHub has been down. So Bootstrapper could also try NuGet. If NuGet is down, then the whole build probably fails.

The latest bootstrapper should do this
On May 6, 2015 2:02 PM, "Tuomas Hietanen"<EMAIL_ADDRESS>wrote:

Bootstrapper takes paket.exe from GitHub. But today GitHub has been down.
So Bootstrapper could also try NuGet. If NuGet is down, then the whole
build probably fails.
—
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/issues/815.

Great, that was fast!

Hi,
this is already implemented in the latest release.
Am 6. Mai 2015 14:02:03 MESZ, schrieb Tuomas Hietanen<EMAIL_ADDRESS>
Bootstrapper takes paket.exe from GitHub. But today GitHub has been
down. So Bootstrapper could also try NuGet. If NuGet is down, then the
whole build probably fails.

Reply to this email directly or view it on GitHub:
https://github.com/fsprojects/Paket/issues/815

--
Diese Nachricht wurde von meinem Android-Mobiltelefon mit K-9 Mail gesendet.


polygon in filtered ways

My question is why sometimes there are polygon geometries in the filtered ways (lines), and sometimes not (I tried different values from the custom_filter and also different places)? What are those polygon geometries?
from pyrosm import OSM, get_data
import time

my_place = 'helsinki'
fp = get_data(my_place)
osm = OSM(fp)

# 1: Using filter_type='keep', keep only 'river' and 'stream' in waterway
waterways_keep = ['river', 'stream']

my_rivers_streams = osm.get_data_by_custom_criteria(custom_filter={
    'waterway': waterways_keep},
    filter_type='keep',
    keep_nodes=False,
    keep_ways=True,
    keep_relations=True)

my_rivers_streams['geom_type'] = my_rivers_streams.geometry.geom_type
print(my_rivers_streams['geom_type'].unique())
# out: ['LineString' 'Polygon']

# 2: Using filter_type='keep', keep only 'river' in waterway
waterways_keep = ['river']

my_rivers = osm.get_data_by_custom_criteria(custom_filter={
    'waterway': waterways_keep},
    filter_type='keep',
    keep_nodes=False,
    keep_ways=True,
    keep_relations=True)

my_rivers['geom_type'] = my_rivers.geometry.geom_type
print(my_rivers['geom_type'].unique())
# out: ['LineString']

@der-karl Hi!
This relates to a perhaps a bit unintuitive definitions used by OpenStreetMap. So how OSM defines a "way" is not necessarily related to line-geometries (i.e. streets), but they are also used to construct closed geometries such as linear rings or polygons. See here for further documentation: https://wiki.openstreetmap.org/wiki/Way
Pyrosm follows this OSM stucture, and hence "way" features can contain LineString, Polygon or LinearRing geometries. The documentation seems to be missing details about this, so good that you asked 👍
If you want to filter out only LineString geometries, it can be done easily in GeoPandas:
from pyrosm import OSM, get_data

my_place = 'helsinki'
fp = get_data(my_place)
osm = OSM(fp)

waterways_keep = ['river']

my_rivers = osm.get_data_by_custom_criteria(custom_filter={
    'waterway': waterways_keep},
    filter_type='keep',
    keep_nodes=False,
    keep_ways=True,
    keep_relations=True)

my_rivers['geom_type'] = my_rivers.geometry.geom_type

# Filter line geometries (something like following should work)
line_rivers = my_rivers.loc[my_rivers["geom_type"].isin(["LineString", "MultiLineString"])]

thanks @HTenkanen for your quick feedback, yeah I expected to get only 'LineString' and 'MultiLineString'. Your method works fine, thanks.

I also have a question related to this.
In the OSM data there are walking areas (such as inside train stations and large sidewalks and public "squares" (of various shapes).  I attached an example with many of these pedestrian areas.
These are polygon ways in OSM, and they are areas where people can walk.  So, when generating the walking network, and especially when exporting it to NetworkX,, my expectation is that these polygons are simply removed.
However, in terms of accessibility, it seems it would be better to interpret them as part of the network...perhaps simply by making the polygon boundary a LinearRing or LineString.  The next issue would be, are they even connected to the surrounding network?

@bramson Thanks for pointing this out! Currently, they are not handled in any manner (I actually need to check what happens in these cases). But yes, this is something that needs to be taken into account, and in terms of walking areas, I think probably the easiest way to solve this would be to do as you described and possibly generate extra LineStrings that would allow e.g. passing through squares etc.
Could you @bramson make an issue about this so it can be included in the next iteration of pyrosm?


Int instead of string added to a document

Describe the bug
When I try to put "string" value while the string happens is a valid int for Go, the int is put in the document.
This is not expected, because, one cannot add a string (singe character) like this: '0'.
To Reproduce
echo 't:' | dasel put -r yaml -t string -v "0x12_11" -s 't' | dasel -r yaml 't.type()'
number

Since the string is provided, I have expected string on output, not int.
Same for bool:
echo 't:' | dasel put -r yaml -t string -v "true" -s 't' | dasel -r yaml 't.type()'
bool

As a bonus two panics:
echo '' | dasel put -r yaml -t string -v "0x12_11" -s 't'
panic: reflect: call of reflect.Value.Set on zero Value
echo 't:' | dasel put -r yaml -t string -v "0x12_11" -s ''
panic: reflect: reflect.Value.Set using unaddressable value
Expected behavior
Add string to a document as requested by user.
Add value to an empty document on first panic.
Do not panic on second panic.
Desktop (please complete the following information):

Windows 11
dasel version v2.3.3

Thanks for this.
The YAML parser used really dictates whether something is a number or string, but I can take a look to see if I can work around this issue.
Re the panics, potentially related to https://github.com/TomWright/dasel/issues/282

The number is is resolved in v2.3.5. I missed the bool issue on the first pass and will resolve it shortly.

This has been released under v.2.4.0.


Evaluate iPad support

Please see #170347 and #170345. They are both caused by what I believe is bug in iPad Safari.


Progress bar support

@dpankros @yogiben, I had a thought regarding progress. What if it's just a generic ability to pass in a progress template, which we then include with a this.percent context?
{{> afFieldInput name="myFile" type="cfs-file" progressTemplate=pt}}

<template name="myProgress">
  {{this.percent}}% done
</template>

Template.registerHelper("pt", function () {
  return Template.myProgress;
});

That way I have ultimate flexibility to render whatever I want as the progress bar. We could also have a default one.

Has the progressbar functionality been implemented yet?

No. It hasn't been implemented but is in development.

On Jan 16, 2015, at 5:01 PM, Habib Ayob<EMAIL_ADDRESS>wrote:
Has the progressbar functionality been implemented yet?
—
Reply to this email directly or view it on GitHub.

Any status update on the progress bar? I have a usability issue on where large files do not seem to be ending up on the server, and users have no idea that the upload did not complete.

@jalik seems to have a file progress pattern in the UploadFS package. It might be worth seeing if the UploadFS package and cfs-autoform could work together.


Object reference not set to an instance of an object

I am getting this error when ever I try to open a profile:
Object reference not set to an instance of an object
can't find a way to solve it. I will be glad if anyone helps.

I don't have a path like this anymore
..Users<username>\Saved Games\DCS\Scripts\DCS-BIOS\doc
DCS-BIOS hub do not install such directory.
Do I have to install the fork of DCS-BIOS from here?

Installed 0.7.40 Fork but same error.

.......\Saved Games\DCS\Scripts\DCS-BIOS\doc*json*
should the correct path for the Fork BIOS.
HUB should generate a folder  with json files in .....\Saved Games\DCS\DCS-BIOS

I manually extracted the fork (DCS BIOS 0.7.40) like this ..\Saved Games\DCS\Scripts\DCS-BIOS\doc\json
I installed the HUB but I don't have this ..\Saved Games\DCS\DCS-BIOS

Hello, At any risk, test the solution that I describe in post # 118
Sincerely, Didier

Check https://github.com/DCSFlightpanels/DCSFlightpanels/releases/tag/4.2.30