Point to similar projects/inspiration from the readme

There is a references section of the main website to point to important resources. However, we should note, in the readme or similar documentation, the various tools and examples that gave us inspiration during this project. (prairielearn, relate, rosalind, etc.).

Which repo should this live in?  Or better, which is the "central" repo?


[AutoDiff] Gardening.

Fix AdjointEmitter doc comments to use accumulation when appropriate.
Remove unnecessary newline from two debug logs.
Minor doc comment fixes/edits.

@swift-ci Please test tensorflow

Not sure if that's the case if it's a SIL argument.

True, though I don't think we'd ever clean up arguments. Could you confirm?

macOS CI flaked, merging.


Fix build script warnings for windows

Got some messages while runing npm run build on windows

Error parsing arguments for flag 'c': 'warnings=false'
Windows may use only double quotes in cmd, and unix can use both. So changing to double will make both happy

output.file will be deprecated: Use 'output.filename' instead
Used webpack v1.12.14, pretty obvious message

Thanks @standy, sorry about the windows issues.  I wonder if there is an incompatibility between 1.x versions of webpack.  When I run npm run build using 1.10.1 on this branch, I get an error:
> npm run build

><EMAIL_ADDRESS>build /Users/brian/Projects/cujojs/most
> webpack --output-library most --output-library-target umd ./most.js --output-filename dist/most.js && uglifyjs dist/most.js -c "warnings=false" -m -o dist/most.min.js

Hash: beb375f99a89e67306ba
Version: webpack 1.10.1
Time: 13ms
ERROR: can't read file: dist/most.js

... more npm error output here ...

We may need to also bump the minimum version of webpack.  When I run node_modules/.bin/webpack --help, here is a screenshot of the command line options that it supports.  Unfortunately --output-filename isn't listed, only --output-file:

That's fairly convincing that webpack likely introduced an incompatible command line API change in 1.x at some point.  I don't mind updating to a newer webpack ... could you do that in this PR?

I tried webpack 1.12.14.  All I did was npm install --save-dev<EMAIL_ADDRESS>and then npm run build worked again.  Seems like the right thing to go ahead and update the webpack minimum version in this PR :+1:

Sure! Coming soon

Awesome, thank you.  I pulled the latest and it worked.  Merging, and will get a release out soon.


Missing information about SDK-style MSBuild project

The section Regenerating the code automatically is missing information about SDK-style project reference to Text Templating.

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

ID: 1b79a889-79f3-3e2b-41b0-4d3b33783cda
Version Independent ID: b4fcb3b9-4fac-79e1-95bd-34247324ae5d
Content: Design-Time Code Generation by using T4 Text Templates - Visual Studio (Windows)
Content Source: docs/modeling/design-time-code-generation-by-using-t4-text-templates.md
Product: visual-studio-windows
Technology: vs-ide-modeling
GitHub Login: @mgoertz-msft
Microsoft Alias: mgoertz

@stan-sz -- Stan, thank you for your feedback.  Please consider these resources:

Ask a question or find an answer on the Microsoft Q&A page for Visual Studio.
Check the Visual Studio MSDN Forum.
Stack Overflow
Reddit

@mgoertz-msft -- Marco, please look into this issue.

This issue hasn't been addressed yet.

#assign:@ghogen

@stan-sz
The steps in the article do work with SDK-style projects. I'm updating the doc a little to mention this. You don't put Text Templating as an SDK. The SDK would be, as usual, Microsoft.Net.Sdk.

I think we can close this issue after clarifying the potential confusion around the SDK style project. If you feel this is closed in error, please reactivate it.


Update Set-SPMetadataServiceApplication.md

Format examples, fill in placeholder text.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Set-SPMetadataServiceApplication.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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


Draft: Use Jakarta EE 11 BOM

I'll open this one as a draft only, to see if it works ok on CI as well.
this passed locally, but I guess to make it all actually pass and use the Jakarta EE 11 specs, those have to be available in WildFly. I suppose we could try patching some of the modules, but that's probably going to be a pain.
Also bumped the expressly version to 6, but since it is only available in the Jakarta staging repository ... I guess we would need to wait with a final release of HV until there's a version of expressly available in the maven central 😕

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on licensing, please check here.

I’m on my phone but their BOM looks a bit odd with some exclusions. I’m not really sure it’s a good idea to use it.

Yeah, maybe... I was looking for something where it is clear what's in EE11 and which specification version to use... browsing through spec list wasn't too helpful 🥲. Some specs referenced in HV build seems to not have a 11 variant, but then I found them in this BOM... so this definitely would need some more thinking 😃 before it goes out of draft 😃

their BOM looks a bit odd with some exclusions

I looked at jakarta.enterprise.cdi-api as an example, and from what I can see, CDI API depends, among other libs, on jakarta.inject-api. With that exclusion in BOM, it forces us to add jakarta.inject-api explicitly if our code is using @Inject which I guess makes sense (we shouldn't include things that come from transitive dependencies directly in our code).
That BOM only manages jakarta.* dependencies, which is nice 😄.
So maybe it's not that terrible to use that BOM after all...
Now, how do we think we should move forward? I'm thinking:

bump Jakarta specs, either with this BOM, or manually, but to versions from this BOM,
since WildFly is not including EE11 specs yet and the build with bumped versions passed, keep things as is and wait for WildFly, i.e. do not add more patching to the HV build to adjust WF for these newer specs ...
ping someone on Expressly to ask them for a release to Central

if we can get an Expressly release compatible with EL 6, then it's all ok. This way, we can try releasing HV 9 Beta (as planned before)
if we cannot get an Expressly release, keep it (and EL API) at 5. This way, we do an Alpha release and mention that EL/Expressly will be updated before we get to the Beta/CR

@gsmet @yrodiere WDYT?

I looked at jakarta.enterprise.cdi-api as an example, and from what I can see, CDI API depends, among other libs, on jakarta.inject-api. With that exclusion in BOM, it forces us to add jakarta.inject-api explicitly if our code is using @Inject which I guess makes sense (we shouldn't include things that come from transitive dependencies directly in our code).

Wouldn't go as far as saying it makes sense, but it is the way things are usually done in the Jakarta EE world -- or at least, in the WildFly world. I'm honestly more surprised exclusions are not done consistently in this BOM -- some dependencies have no exclusions.
These exclusions drive me mad, but they're very common and have reasons to be. The idea is that (as you said) your code should declare explicitly all its dependencies, and implementations of your dependencies will take care of pulling in all necessary transitive dependencies. As you know, in the app server world, implementations aren't necessarily known at compile time.

So maybe it's not that terrible to use that BOM after all...

I think it's tolerable, if you are sure the BOM won't arbitrarily remove some transitive dependencies and cause Java SE applications to miss them. I.e. your tests' dependencies need to be extra clean, so that test dependencies don't offset problems in the main dependencies.

since WildFly is not including EE11 specs yet and the build with bumped versions passed, keep things as is and wait for WildFly, i.e. do not add more patching to the HV build to adjust WF for these newer specs ...

Getting WildFly to work is definitely a second step. Let's release a Beta first.

ping someone on Expressly to ask them for a release to Central

+1 this is very odd, let's get some more information.

@gsmet @yrodiere WDYT?

My main concern is about the timeline. We can't have non-final dependencies in HV, and the Jakarta EE 11 BOM is tagged "M4", so I guess non-final.
Are you sure we can wait until the Jakarta EE 11 BOM gets released before we release HV 9.0.0.Final?
Or maybe that's a moot point because POM flattening hides the Jakarta EE 11 BOM anyway?

+1 this is very odd, let's get some more information.

I've created a GH issue, and Arjan replied that he'll push it to Central, so I'll keep an eye on it. But to your point:

We can't have non-final dependencies in HV

it would still mean that HV is relying on an M1 dependency, even though EL is an optional dependency and HV can work without it, but then it means that HV is missing one of the spec features: https://docs.jboss.org/hibernate/stable/validator/reference/en-US/html_single/#non-el-message-interpolator
I think that from the HV standpoint, it needs Jakarta Validation, CDI, and EL specs; it mentions the persistence spec as well, but it is the persistence spec that integrates the BV, so we may not care about it as a dependency from a spec perspective, but we do have some HV features based on it (JPATraversableResolver) so we do need it too.

Or maybe that's a moot point because POM flattening hides the Jakarta EE 11 BOM anyway?

This is also true; the published poms are flattened, so we don't really include that BOM in what we publish.

it would still mean that HV is relying on an M1 dependency

Expressly not being Final is IMO a blocker for a Final release of HV. We can still release an Alpha/Beta, but as long as our dependencies are not Final, we're stuck.
FWIW even if we made an exception just this once, I'm pretty sure the (transitive) dependency to a non-final version would be a blocker for integration into WildFly or Quarkus.

Are you flattening the pom file? If  not, I think it would be worth checking this patch doesn't cause any problem downstream.
It could be done in Quarkus with a draft PR once you have a Beta1 one out.

(And TBH, I wonder if it would make sense to flatten the POM files for the Hibernate libraries)

hey 😃,
Yes the poms a flattend. Here's an example of the engine POM based on this PR:
<?xml version="1.0" encoding="UTF-8"?>
<!--
  ~ Hibernate Validator, declare and validate application constraints
  ~
  ~ License: Apache License, Version 2.0
  ~ See the license.txt file in the root directory or <http://www.apache.org/licenses/LICENSE-2.0>.
  -->
<project xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd" xmlns="http://maven.apache.org/POM/4.0.0"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
  <modelVersion>4.0.0</modelVersion>
  <groupId>org.hibernate.validator</groupId>
  <artifactId>hibernate-validator</artifactId>
  <version>9.0.0-SNAPSHOT</version>
  <name>Hibernate Validator Engine</name>
  <description>Hibernate's Jakarta Validation reference implementation.</description>
  <url>http://hibernate.org/validator/hibernate-validator-public-parent/hibernate-validator</url>
  <licenses>
    <license>
      <name>Apache License 2.0</name>
      <url>http://www.apache.org/licenses/LICENSE-2.0.txt</url>
    </license>
  </licenses>
  <developers>
    <developer>
      <id>epbernard</id>
      <name>Emmanuel Bernard</name>
<EMAIL_ADDRESS>      <url>http://in.relation.to/emmanuel-bernard/</url>
      <organization>Red Hat, Inc.</organization>
    </developer>
    <developer>
      <id>hardy.ferentschik</id>
      <name>Hardy Ferentschik</name>
<EMAIL_ADDRESS>      <url>http://in.relation.to/hardy-ferentschik/</url>
      <organization>Red Hat, Inc.</organization>
    </developer>
    <developer>
      <id>gunnar.morling</id>
      <name>Gunnar Morling</name>
<EMAIL_ADDRESS>      <url>http://in.relation.to/gunnar-morling/</url>
      <organization>Red Hat, Inc.</organization>
    </developer>
    <developer>
      <id>kevinpollet</id>
      <name>Kevin Pollet</name>
<EMAIL_ADDRESS>      <url>http://www.serli.com/</url>
      <organization>SERLI</organization>
    </developer>
    <developer>
      <id>davide.dalto</id>
      <name>Davide D'Alto</name>
<EMAIL_ADDRESS>      <url>http://in.relation.to/davide-dalto/</url>
      <organization>Red Hat, Inc.</organization>
    </developer>
    <developer>
      <id>guillaume.smet</id>
      <name>Guillaume Smet</name>
<EMAIL_ADDRESS>      <url>http://in.relation.to/guillaume-smet/</url>
      <organization>Red Hat, Inc.</organization>
    </developer>
    <developer>
      <id>marko.bekhta</id>
      <name>Marko Bekhta</name>
<EMAIL_ADDRESS>      <url>http://in.relation.to/marko-bekhta/</url>
    </developer>
  </developers>
  <scm>
    <connection>scm:git:git://github.com/hibernate/hibernate-validator.git/hibernate-validator-public-parent/hibernate-validator</connection>
    <developerConnection>scm:git:git@github.com:hibernate/hibernate-validator.git/hibernate-validator-public-parent/hibernate-validator</developerConnection>
    <url>http://github.com/hibernate/hibernate-validator/hibernate-validator-public-parent/hibernate-validator</url>
  </scm>
  <dependencies>
    <dependency>
      <groupId>jakarta.validation</groupId>
      <artifactId>jakarta.validation-api</artifactId>
      <version>3.1.0</version>
      <scope>compile</scope>
      <exclusions>
        <exclusion>
          <groupId>*</groupId>
          <artifactId>*</artifactId>
        </exclusion>
      </exclusions>
    </dependency>
    <dependency>
      <groupId>org.jboss.logging</groupId>
      <artifactId>jboss-logging</artifactId>
      <version>3.6.0.Final</version>
      <scope>compile</scope>
    </dependency>
    <dependency>
      <groupId>com.fasterxml</groupId>
      <artifactId>classmate</artifactId>
      <version>1.7.0</version>
      <scope>compile</scope>
    </dependency>
    <dependency>
      <groupId>org.glassfish.expressly</groupId>
      <artifactId>expressly</artifactId>
      <version>6.0.0-M1</version>
      <scope>provided</scope>
      <optional>true</optional>
    </dependency>
    <dependency>
      <groupId>org.jboss.logging</groupId>
      <artifactId>jboss-logging-annotations</artifactId>
      <version>3.0.1.Final</version>
      <scope>provided</scope>
    </dependency>
    <dependency>
      <groupId>jakarta.persistence</groupId>
      <artifactId>jakarta.persistence-api</artifactId>
      <version>3.2.0</version>
      <scope>compile</scope>
      <optional>true</optional>
    </dependency>
    <dependency>
      <groupId>joda-time</groupId>
      <artifactId>joda-time</artifactId>
      <version>2.12.7</version>
      <scope>compile</scope>
      <optional>true</optional>
    </dependency>
    <dependency>
      <groupId>com.thoughtworks.paranamer</groupId>
      <artifactId>paranamer</artifactId>
      <version>2.8</version>
      <scope>compile</scope>
      <optional>true</optional>
    </dependency>
    <dependency>
      <groupId>javax.money</groupId>
      <artifactId>money-api</artifactId>
      <version>1.1</version>
      <scope>compile</scope>
      <optional>true</optional>
    </dependency>
    <dependency>
      <groupId>org.openjfx</groupId>
      <artifactId>javafx-base</artifactId>
      <version>17.0.12</version>
      <scope>compile</scope>
      <optional>true</optional>
    </dependency>
  </dependencies>
</project>

And we do the same for Search.

    <developer>
      <id>marko.bekhta</id>
      <name>Marko Bekhta</name>
<EMAIL_ADDRESS>      <url>http://in.relation.to/marko-bekhta/</url>
    </developer>

Missing a       <organization>Red Hat, Inc.</organization> here :P

Missing a       Red Hat, Inc. here :P

Who's that guy anyway? 🤔 😅 😆

I was looking at all the poms to make sure they look fine. And that made me thinking. Currently in engine we have a
        <dependency>
            <groupId>org.glassfish.expressly</groupId>
            <artifactId>expressly</artifactId>
            <scope>provided</scope>
            <optional>true</optional>
        </dependency>

I'd assume that we really need only a dependency on the EL API, and let the user pick the implementation....
But ... there are these lines:
https://github.com/hibernate/hibernate-validator/blob/33d821c6fd25112ff1eee4a448058b2c251180cf/engine/src/main/java/org/hibernate/validator/messageinterpolation/ResourceBundleMessageInterpolator.java#L198-L205
Where an impl class is used to get a CL to make things work in OSGi... so we can't remove the dependency without removing this block of code ...

do we care about OSGi ?
maybe there's a different way to get a correct CL?
don't bother and leave things as they are 😃

OK I couldn't see any obvious issues with the flattened pom/module info files, let's merge this one 🤞🏻
Thank you all for taking a look 🏆

Just created https://hibernate.atlassian.net/browse/HV-2020 to "track" this  😃

I'd say it's acceptable to just drop OSGi support in HV 9.

I guess it already is not tested 🙈, we have  the OSGi directory, but it has been excluded from the build https://github.com/hibernate/hibernate-validator/commit/7b7e6799760b9df7f6a93a19ccbfcac92629ce8b
And from what I can see 7.0 was the last version which published something OSGi related: https://repo1.maven.org/maven2/org/hibernate/validator/hibernate-validator-osgi-karaf-features/

Just created https://hibernate.atlassian.net/browse/HV-2020 to "track" this 😃

I'd say it's acceptable to just drop OSGi support in HV 9.

I guess it already is not tested 🙈, we have the OSGi directory, but it has been excluded from the build 7b7e679 And from what I can see 7.0 was the last version which published something OSGi related: https://repo1.maven.org/maven2/org/hibernate/validator/hibernate-validator-osgi-karaf-features/

I hear you say "someone needs to update OSGi tests" and I can only answer "not me".
If we're already not testing, and not publishing OSGi features, and nobody's been complaining, IMO it's time to create an issue to drop OSGi support, go through code and documentation to remove any reference, and add an entry to the migration guide.


Include Places API - JS

This might help: https://royfloresnyc.medium.com/how-to-render-photos-of-any-city-in-the-world-in-your-app-740325fa6ff5


CSS not working

Page is displaying very rudimentary style. The one shown in demo is not working.

react-js-pagination provides just markup. you have to wire css by yourself.
I agree, this is a bad design. I think I will remove classes for the next version and apply some default styles


Service level example does not handle list method

If the first method in a service is a list method, the sample generated in the service documentation at the top of the file for that service will be incorrect.

@michaelbausor Is this Java-specific, or more general?

This is Java only as far as I know, should have written this in the description, editted now.

The code now picks the first flattened method, so this is fixed.


[Subtask] Connectors for Hive SQL Query Engine -> Iceberg

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the subtask
Iceberg Hive Doc
Parent issue
https://github.com/apache/incubator-kyuubi/issues/3059
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

I will take this.


Make vasiliy-ul an approver for kubevirt/kubevirt

What this PR does / why we need it:
Proposing Vasily as an approver for kubevirt/kubevirt.
Since his first contribution in Dez. 2020 (https://github.com/kubevirt/kubevirt/pull/4671), Vasiliy has been a steady and reliable contributor to KubeVirt.
He has more than 50 PRs merged: https://github.com/kubevirt/kubevirt/pulls?q=is%3Aclosed+is%3Apr+author%3Avasiliy-ul+
He helped reviewing and bringing in more than 50 PRs : https://github.com/kubevirt/kubevirt/pulls?page=2&q=is%3Apr+commenter%3Avasiliy-ul+is%3Aclosed+-author%3Avasiliy-ul
When you go through his contributions, you will see that he has deep knowledge on almost all areas of KubeVirt. He helped on the lowest layers (with container disk mount points, cgropus v1/v2), on the user-facing layers (e.g. rancher proxy, SEV, affinity validation, ...) and also tackled the complex and still ongoing implementation of SEV (basics merged).
I know him as a reliable, cautious, skillful and respectful engineer who has a big positive impact on all features which he touches as a code writer or reviewer.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:
NONE

/cc @mhenriks
/cc @dhiller
/cc @rthallisey
/cc @davidvossel
/cc @vladikr
/cc @stu-gott
/cc @jean-edouard
/cc @fabiand
/cc @xpivarc
/cc @enp0s3

@vasiliy-ul FYI

/hold
adding a hold to allow people to add lgtm

/lgtm

/lgtm

/lgtm

7 lgtm + me makes 8. I would say that is enough. Thanks everyone. 👍
/unhold

Wow! Thank you, @rmohr, for bringing this up. Thanks a lot to everyone for the support! Really appreciate that and glad to help with moving the project further.


Add non-strict-config flag

Bring back strict config parsing.
Non-strict parsing was introduced in https://github.com/bottlepay/lnmux/pull/48.

LGTM!

Updated PR to make it a global flag rather than a per-command one.


Add an ability to round to custom number of decimal points

Hi and thank you!
I think it would be a great addition to have be able to round to a specific number of decimal places:
now:
var smallDuration = 499;
var viewValue = humanizeDuration(smallDuration); // -> "0.499 seconds"
var viewValueRounded = humanizeDuration(smallDuration, {round: true}); // -> "0 seconds"

would be nice:
var smallDuration = 499;
var viewValueRounded = humanizeDuration(smallDuration, {round: 1}); // -> "0.5 seconds"

Could this just proxy to toFixed?
(0.499).toFixed(1)  // => "0.5"
(0.499).toFixed(2)  // => "0.50"
(0.499).toFixed(3)  // => "0.499"

If I proxy it to .toFixed() I would lose the time units.

If you'd like to submit a PR, feel free!

That would be awesome :)

Perhaps round param could take a function instead of boolean value. This would be far more flexible as I need for example to cut off seconds, not to round them.

@topr Agreed. One thing that makes this tricky is how rounding affects other numbers. For example, what happens if you do something like this?
humanizeDuration(123456, {
  round: function () {
    return 0
  }
})

Would it be an issue? Not sure but I guess result should then be the same
as if 0 would be passed in instead of 123456, shouldn't it?
I don't think the lib should care or prevent such a misuse. If someone
passes a custom impl then it's his responsibility to make it right.
I was mostly thinking about use cases like humanizeDuration(123456, { round: Math.floor })
On 2 Feb 2017 22:58, "Evan Hahn"<EMAIL_ADDRESS>wrote:
@topr https://github.com/topr Agreed. One thing that makes this tricky is
how rounding affects other numbers. For example, what happens if you do
something like this?
humanizeDuration(123456, {
round: function () {
return 0
}
})
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/EvanHahn/HumanizeDuration.js/issues/73#issuecomment-277111919,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1TIpAvF0-3Xsd6UjHiXpK98VDnSJv2ks5rYl-igaJpZM4FckB4
.

What if you're trying to humanize the largest 2 units and then you choose to round down? For example, your result might start like this before you round and do your cutoff:

2 weeks, 6 days, 23 hours, 59 minutes

Depending on how you round, the results can be different. If you round up, you could get "3 weeks". If you round down, you should get "2 weeks, 6 days". These rounding issues can cascade, which is why it's tricky (but not impossible).

+1. This would be nice.

Do it.

Lots of people seem to want this feature and things around it. I want to do it but I'm worried it gets really hairy with lost of corner cases. What do people think about my comment from February?

{
  units: ['d','m','s'],
  unitsConfig: {
    m: {
      measure: 30,  // unitMeasures
      round: value => 1
    }
  }
}

This library is in maintenance mode, so I won't be adding new features, sadly. I'm going to close this issue. See #120 for more.

Use pretty-ms instead.


📝 Update README with OpenSSF best practices

update read me with Openssf best practices currently met

is this information required in the README?


hello, I have some problems about "run.sh"

       Every file could be run individually except **"attention_vis.py"** whose traceback was IndexError: list index out of range at line 40, **"RF = int(sys.argv[1])"**.
       I have already downloaded the Ninapro DB1 data set. And I got the data by runing **"create_dataset.py"** before I ran the **"run.sh"**. However, when I ran the "run.sh", the Pycharm told me that **"ModuleNotFoundError: No module named 'tensorflow'"** in **File "script_experiment.py", line 3**. Because "script_experiment.py" could be run successfully and individually, I didn't know what had I done wrongly. Could you give me some help? thanks.

I have solved the second problem by running "run.sh" in Terminal instead of Pycharm.

For the error in "attention_vis.py" you should run it with two arguments the receptive field (300 or 2500) and the subject id. For example, run:
python3 attention_vis.py 300 25
to visualize the attention weights of the subject 25 for receptive field = 300

For the error in "attention_vis.py" you should run it with two arguments the receptive field (300 or 2500) and the subject id. For example, run:
python3 attention_vis.py 300 25
to visualize the attention weights of the subject 25 for receptive field = 300

Thanks for your patiently reply. I have solved this problem, but another problem is bothering me.
After executing bash run.sh, 30 epoches of every subject were training finished, and the error, NameError: name 'MaskedConv1D' id not defined, occurred. Although there was the error, the coding could still continute to run. If I should loading a pre-training models?


ocamlPackages.toml: 6.0.0 -> 7.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://opam.ocaml.org/packages/toml/toml.7.0.0.
meta.description for ocamlPackages.toml is: "Implementation in OCaml of the Toml minimal langage"
meta.homepage for ocamlPackages.toml is: "http://ocaml-toml.github.io/To.ml"
meta.changelog for ocamlPackages.toml is: ""
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 7.0.0 with grep in /nix/store/0yjhn338kfbipqvr2w6n11kyg5806wnz-ocaml4.13.1-toml-7.0.0

found 7.0.0 in filename of file in /nix/store/0yjhn338kfbipqvr2w6n11kyg5806wnz-ocaml4.13.1-toml-7.0.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

0 x86_64-linux rebuild(s)
0 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
ocamlPackages.toml

Instructions to test this update (click to expand)

Build yourself:
nix-build -A ocamlPackages.toml https://github.com/StridBot/nixpkgs/archive/63e88f3d6a8dfaf0e636153a0e1b1b634fdf4ec7.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/0yjhn338kfbipqvr2w6n11kyg5806wnz-ocaml4.13.1-toml-7.0.0
ls -la /nix/store/0yjhn338kfbipqvr2w6n11kyg5806wnz-ocaml4.13.1-toml-7.0.0/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @vbgl for testing.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 165676 run on x86_64-linux 1

1 package failed to build and are new build failure:

comby: log was empty

@GrahamcOfBorg build comby


First unread message marked as read when the tab reloads.

As @laurynmm pointed out here in CZO  when a pinned tab (not on focus) that is set to a narrowed view is reloaded, the first unread message in the list is marked as read. This is because the code does not set the selected message ID in the message list before the messages are processed.
I'll try to explain this issue along with the screen recording I am uploading.
To reproduce this issue :

Log in to the development environment with two users.
Have both users open the same conversation view that can mark messages as read.
Have one user send two or more messages.
Stop the development server.
Start the development server again.
The user with the unread messages will see that the first unread message is marked as read, but the selected message / blue box will not move.

I am also attaching the screen recording in which this bug can be seen
https://github.com/zulip/zulip/assets/91622060/a8857f67-4b45-4bd2-ba08-2fb95042d5c2
So if my user is Shiva and my tab not active right now and Desdemona send me 4 messages, if I close the browser and reopen it the browser restores my session ( thus reloading the inactive zulip tab ) and after reloading the first unread message is now marked as read and only the last 3 messages are unread and the first message is selected message ( has a blue box around it ).

@zulipbot claim

@nimishmedatwal - Thanks for opening an issue for this bug!
I think it's going to be worthwhile to discuss how best to fix this in that CZO conversation you linked to above as the message_list.select_id() calls in the frontend are a bit complicated, so I'm not going to mark as "help wanted" quite yet. But I would definitely encourage you to share your thoughts on a fix in that discussion.

Fixed in #31890. Verified manually.


Fixing max_iters for all learners

Fixed Linear.train_one to receive the max_iter from Parameter object
Changes are listed below:

For solvers that where using Tron class (L2R_LR, L2R_L2LOSS_SVC,
L2R_L2LOSS_SVR):
the class already had a constructor that would receive the max_iter as
argument. But, it wasn't used. Therefore the learner would automatically
just use 1000.

For rest of the solvers(L2R_LR_DUAL, L1R_LR, L1R_L2LOSS_SVC,
L2R_L1LOSS_SVC_DUAL, L2R_L2LOSS_SVC_DUAL):
the function changed to receive the max_iter as argument. All the
function where defining a constant at the begining of the function.

L2R_L2LOSS_SVR_DUAL and L2R_L1LOSS_SVR_DUAL were already handling it,
so no changes was applied to this solver type

Just wanted to double check if you could see this
Thanks

Thanks

available in version 2.11

Great.
Thanks,
Salim
On Mon, Jul 10, 2017 at 3:47 PM, Benedikt Waldvogel <
<EMAIL_ADDRESS>wrote:

available in version 2.11
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bwaldvogel/liblinear-java/pull/21#issuecomment-314214721,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-bI3h5J2vB-8O5nX7wA6wo2ZGgoC7-ks5sMn_HgaJpZM4NYcO9
.


FormCollection class added. Updated the Form and Application class

Thanks for the PR.
A few issues:

FormCollection isn't actually used, it is probably intended to be the type for Application.OpenForms.
Forms are never removed from the OpenForms collection when they are closed.

I couldn't commit changes to your fork branch, so I rolled it into https://github.com/modern-forms/Modern.Forms/pull/49 and committed the changes there.
Thanks for the PR!


Only close an issue if all attached tasks are in the same list

Issue created with imdone.io

TODO: Make the +enhancement to only close an issue if all attached tasks are in the same list id:66
plan/plan-imdone.md:19
@imdone Issue Creator. The developer's task board. imdone.io

TODO: Make the +enhancement to only close an issue if all attached tasks are in the same list id:66 gh:208
plan/plan-imdone.md:19
@imdone commenter. The developer's task board. imdone.io

TODO: Make the +enhancement to only close an issue if all attached tasks are in the same list. id:66 gh:208
plan/plan-imdone.md:19
@imdone commenter. The developer's task board. imdone.io

DOING: Make the +enhancement to only close an issue if all attached tasks are in the same list. id:66 gh:208
plan/plan-imdone.md:19
@imdone commenter. The developer's task board. imdone.io

TODO: Make the +enhancement to only close an issue if all attached tasks are in the same list. id:66 gh:208
plan/plan-imdone.md:19
@imdone commenter. The developer's task board. imdone.io

TODO: Make the +enhancement to only close an issue if all attached tasks are in the same list. id:66 gh:208 +imdone
plan/plan-imdone.md:19
@imdone commenter. The developer's task board. imdone.io

BACKLOG: Make the +enhancement to only close an issue if all attached tasks are in the same list. id:66 gh:208 +imdone
plan/plan-imdone.md:19
@imdone commenter. The developer's task board. imdone.io

BACKLOG: Make the +enhancement to only close an issue if all attached tasks are in the same list. id:66 gh:208 +imdone
plan/plan-imdone-2017-1-23.md:22
@imdone commenter. The developer's task board. imdone.io

BACKLOG: Make the +enhancement to only close an issue if all attached tasks are in the same list. id:66 gh:208 +imdone
plan/plan-imdone-2017-1-30.md:23
@imdone commenter. The developer's task board. imdone.io

BACKLOG: Make the +enhancement to only close an issue if all attached tasks are in the same list. id:66 gh:208 +imdone
plan/plan-imdone-2017-1-30.md:20
@imdone Issue Closer. The developer's task board. imdone.io


Add negative tests

One problem I've been seeing with our test suite is that they all test for positive outcomes. Let's make some tests that intentionally fail and test to make sure they fail.

@AlpacaMax once you sort out the test suite bug, it would be great for you to add some negative tests to the suite. I'll comment here with some ideas, or feel free to brainstorm some of your own.

I think starting with some mmap tests would be good. Read the man page and check the error section to get some ideas.

After talking with @justyoyo123 and @AlpacaMax
@justyoyo123 is going to get started with some negative tests for mmap()/munmap()
and @AlpacaMax is going to add some negative tests for fork() and exec()
A good way to think of tests here is to checkout out the manpages for these syscalls, and see what errors are expected in different failure cases. Then designing the test to purposely hit those error codes.

After talking with @justyoyo123 and @AlpacaMax
@justyoyo123 is going to get started with some negative tests for mmap()/munmap()
and @AlpacaMax is going to add some negative tests for fork() and exec()
A good way to think of tests here is to checkout out the manpages for these syscalls, and see what errors are expected in different failure cases. Then designing the test to purposely hit those error codes.

Sounds good. Do we just add negative cases as individual files or add them into the files we already have?

Yeah, these should be new testfiles. If you want to test for multiple errors inside these new files and it makes sense to do so, you can.


TypeError: format.replace is not a function

I'm getting this after latest pull
2017-06-07 20:10:05  2710.70 BTC-USD   +0.0%        1   ---       -0.1845   0.1000           0.00038 BTC  107.24 USD    +8.3%   -6.2%/home/rebootfin/zenbot/node_modules/numbro/numbro.js:183
            escapedFormat = format.replace(/\{[^\{\}]*\}/g, '');
                                   ^

TypeError: format.replace is not a function
    at formatNumbro (/home/rebootfin/zenbot/node_modules/numbro/numbro.js:183:36)
    at Object.format (/home/rebootfin/zenbot/node_modules/numbro/numbro.js:1144:20)
    at /home/rebootfin/zenbot/lib/engine.js:432:82
    at /home/rebootfin/zenbot/lib/engine.js:341:11
    at /home/rebootfin/zenbot/extensions/exchanges/bitfinex/exchange.js:82:9
    at Request._callback (/home/rebootfin/zenbot/extensions/exchanges/bitfinex/node_modules/bitfinex-api-node/rest.js:95:16)
    at Request.self.callback (/home/rebootfin/zenbot/extensions/exchanges/bitfinex/node_modules/request/request.js:188:22)
    at emitTwo (events.js:106:13)
    at Request.emit (events.js:191:7)
    at Request.<anonymous> (/home/rebootfin/zenbot/extensions/exchanges/bitfinex/node_modules/request/request.js:1171:10)
rebootfin@zenbot:~/zenbot$

code part:
    getQuote: function (opts, cb) {
      var client = publicClient()
      var pair = joinProduct(opts.product_id)
      client.ticker(pair, function (err, body) {
        if (err) return cb(err)
        cb(null, {bid: body.bid, ask: body.ask}) //line 82
      })
    },

What's the solution?

I have the same problem with the bitstamp implementation

With some debugging I found the problem.
In lib/engine.js:
line  398
price = n(mid_price).subtract(n(mid_price).multiply(so.markup_pct / 100)).format(s.product.increment, Math.floor)
line 432
price = n(mid_price).add(n(mid_price).multiply(so.markup_pct / 100)).format(s.product.increment, Math.ceil)
In both lines the "format(s.product.increment, Math.ceil)" expects a number as a string
In both extensions/exchanges/bitfinex/products.json
and extensions/exchanges/bitfinex/products.json
are used numeric values for increment and other parameters,
but in extensions/exchanges/gdax/products.json
are used string values.
The question is, should it be fixed in products.json or should engine.js accept both forms?

I think it's easier to apply String() to products.json as I see all other exchanges have string values in there.

I'm proud to announce it solved! Thanks to @eigilb

It is not solved 😢 . It happens again. I tried this {{ new Date() | moment( 'ddd [at] h:mm' )}}
Any solution?

Import MatMomentDateModule.


Question URL at responsive-web-design seems inconsistent

Describe the Issue

I am using gitpod and running the repository using localhost.
Facing issue in url that the following shared section redirects to wrong question from 1 till 44th question
Not facing this issue at official website url : https://www.freecodecamp.org/learn/2022/responsive-web-design/ , but just opened this issue if its a bug for others as well.

Affected Page
/learn/2022/responsive-web-design/ url of localhost
Steps to Reproduce

Go to 'https://8000-[YOUR_USERNAME]-freecodecam-a5iepuqc08u.ws-us94.gitpod.io/learn/2022/responsive-web-design/#learn-html-by-building-a-cat-photo-app'
Scroll down to 'Courses'
Visit any question before 45 ie from 1 to 44.

Expected behavior

Question 1 should redirect to question 1, now its going to 44th question's url.
From question 2 till 44, its redirecting to n - 1th question ie 2nd is opening question 1 and so on.

Screenshots
Images on hover to show the url

System

Device: [e.g. iPhone 6, Laptop]
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context

Facing this issue only on localhost and not on official url. I didn't make any changes which could lead to such behaviour.

I couldn't duplicate this on gitpod on my end, can you make sure that you are at latest commit in the upstream/main.

@Sboonny Hey you could close this issue, as I am not facing the same issue again. Not sure why I was having such bug back then.


iops: init at 0.1

successfully built on NixOS x86_64.
I asked Folkert for the license via E-Mail (GPL2).

@jagajaga i have fixed that.

Lol nice, I've written a very similar tool in C. I wonder when it does make sense to flush the cache when reading from raw devices. Perhaps to avoid something like bcache?

@lethalman i think to get the read performance from disk and not from cache.

@davidak there's no cache when dealing with raw device blocks


Failed to register the server's public key

Required Info

Camera Model
D455

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
Ubuntu 18

Kernel Version (Linux Only)
4.9.201

Platform
NVIDIA Jetson

SDK Version
2.47.0

Language
C++

Segment
Devolopment

Issue Description
https://github.com/IntelRealSense/librealsense/blob/master/doc/distribution_linux.md
When installing packages from the following website, I have an  error when trying to register the server's public key.
Please let me know if you can help

Hi @JJONSOO   As a first step in investigating your case, I would recommend checking whether the problem is occurring because of a firewall, as described in the link below.
https://github.com/IntelRealSense/librealsense/issues/1276

Hi, I finished everything on the site you pointed out and tried to execute the example file, but I have an error
api version mismatch : librealsense.so was compiled with API version 2.41.0 but the application was compiled with 2.42.0 make sure correct version of the library is installed
What should I do?

The error suggests that the example program was created in a version of librealsense that is newer than the one that you have installed from packages.
There was a recent case in which a RealSense user was installing librealsense packages for their Jetson but it was only librealsense 2.41.0 instead of the latest 2.47.0 version.  Doronhi the RealSense ROS wrapper developer commented on this in the link below.
https://github.com/IntelRealSense/realsense-ros/issues/1938#issuecomment-867379211
The RealSense user in that case instead built librealsense for their Jetson from source code using the instructions in doronhi's comment.

Hi @MartyG-RealSense,
I encountered the same issue. It seems like the instructions in section 5. Register the server's public key in installation_jetson is the same as the one in distribution_linux.
The bottleneck is on this command: sudo apt-key adv --keyserver keys.gnupg.net --recv-key F6E65AC044F831AC80A06380C8B3A55A6F3EFCDE  It will lead to the error gpg: keyserver receive failed: No name on my Jetson Xavier NX device.
PS. I used to successfully installed everything on a Jetson AGX Xavier device following the same instructions as in distribution_linux about half a year ago.

Hi @hanzheteng   I recommend using the Jetson installation page's package instructions for installing packages on Jetson (section 4.  Install with Debian Packages) as the Debian packages on the distribution_linux page are designed for use with x86 / x64 computers.
My research of your question found though that problems with using sudo apt-key adv --keyserver keys.gnupg.net --recv key have been reported recently at the end of June 2021 at the link below, which offers advice for resolving the problem.
https://stackoverflow.com/questions/66217436/gpg-keyserver-receive-failed-no-name/68132500#68132500

Hi @MartyG-RealSense ,
As I mentioned earlier, this error also occurred on amd64 machines, so it is not related to Jetson platform. (Also, there is no actual instruction under 4. Install with Debian Packages if you take a look.)
The main issue here is that the key server keys.gnupg.net is not functioning anymore. Someone reported this recent issue here.
The solution is to replace the key server keys.gnupg.net by keyserver.ubuntu.com or other functioning ones. In both instructions (installation_jetson and distribution_linux), use the following command to register the server's public key should fix this problem.
sudo apt-key adv --keyserver keyserver.ubuntu.com --recv-key F6E65AC044F831AC80A06380C8B3A55A6F3EFCDE || sudo apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv-key F6E65AC044F831AC80A06380C8B3A55A6F3EFCDE
Please double check and update the instructions accordingly.

https://github.com/IntelRealSense/librealsense/issues/9315#issuecomment-871250227 was not only related to Jetson, so it was a valid suggestion for a problem that could occur on both x86/x64 and the Arm-based Jetson.
The link about keys.gnupg.net was my original message to you, which I edited out because I believed that https://github.com/IntelRealSense/librealsense/issues/9315#issuecomment-871250227 was worth looking at first as a more likely possibility.  So it's great to hear that the original suggestion worked for you after all.
I will highlight the above information to Intel.  Thanks very much for the update!

Thank you for the time and effort for helping out. The original message did help. Please convey the message to Intel about the possibly deprecated key server then.

Hi @hanzheteng  I have received a response from Intel about this issue.  They said that the official instruction pages were updated several months ago with the alternate '||' keyserver address.
distribution_linux.md
https://github.com/IntelRealSense/librealsense/blob/master/doc/distribution_linux.md
sudo apt-key adv --keyserver keys.gnupg.net --recv-key F6E65AC044F831AC80A06380C8B3A55A6F3EFCDE || sudo apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv-key F6E65AC044F831AC80A06380C8B3A55A6F3EFCDE
Jetson
https://github.com/IntelRealSense/librealsense/blob/master/doc/installation_jetson.md
sudo apt-key adv --keyserver keys.gnupg.net --recv-key F6E65AC044F831AC80A06380C8B3A55A6F3EFCDE || sudo apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv-key F6E65AC044F831AC80A06380C8B3A55A6F3EFCDE

Thank you for the clarification. I just verified on a clean Ubuntu 18 virtual machine that the second part also works.
The behavior of the first part varies on different machines. On some machines it returns immediately, but on some other machines it can take 10s to return the No Name error. The first time I ran this line of command, the first part was not responding and interrupted by me, so the second part was not actually being executed.

Thanks very much @hanzheteng for your test feedback!

Hi @JJONSOO  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


chore: move deprecated scripts

Main improvements in this PR:
Moved deprecated files to separate folder to avoid confusion. Also clarified with README's the purpose of each script/data folder.
I hereby confirm that I have:

[x] Tested my code with all requirements for running the model
[x] Selected devel as a target branch (top left drop-down menu)
[x] If needed, asked first in the Gitter chat room about this PR

Do we have somewhere documented which script was used for which model version? This could be dealt with by having subfolders like deprecated/sincev8.1.0/, deprecated/sincev8.1.1 etc. Perhaps not stricly required, if we consistenly move old scripts to the deprecated folder when making a new release as soon as a particular script is no longer required, and mention this explicitly in the PR message.

@edkerk thanks for your feedback!

Do we have somewhere documented which script was used for which model version?

Good point, we don't. I will document it here for all the ones that were moved in this PR:

Folder
File
Model Version

modelCuration
addiSce926changes.m
yeast 8.0.0

fixBiomassComp.m
yeast 8.1.0

makeFormulasCompliant.m
yeast 8.1.0

modelCorrections.m
yeast 7.7.0

takeOutFromFormula.m
yeast 8.1.0

updateMetaboliteFormula.m
yeast 8.3.4

otherChanges
addSLIMErxns.m
yeast 8.1.0

clusterBiomass.m
yeast 7.8.0

convertYmn2FBC2.m
yeast 7.7.0

[...] if we consistenly move old scripts to the deprecated folder when making a new release as soon as a particular script is no longer required, and mention this explicitly in the PR message.

I think this solution is preferable, to avoid having a very complex sub-folder structure. We can also mention it in the commit description for all new scripts.


Update for dark mode  (script.js)

Hey Buddy, as I said you are doing a good work but the project doesn't need dark mode as of now and additions will make it inconsistent.


Make dispatch create seed-images idempotent

Feature Request
Make dispatch create seed-images idempotent. It will be worthwhile to address #394 first and leverage the functionality for this command.
Detailed Description

Context

Possible Implementation

Complexity

[ ] Low - Simple enhancement or bug fix, no architectural changes or refactoring
[X] Medium - Change requires some thought, but is relatively isolated
[ ] High - Significant architectural change or large refactor

I would probably leverage the "apply" command.  Also add support for "delete"

fix in #785


Libcamera apps improvements

This branch contains a few changes for the rpi-libcamera-apps package.
It fixes a runtime issue with the raspberrypi machine, adds support for DRM, and makes the package hardware-agnostic as it should be.

@kraj @agherzan It seems like meta-multimedia isn't included your CI.

The error is ERROR: Nothing PROVIDES 'libcamera' (but /work/recipes-multimedia/libcamera-apps/libcamera-apps_git.bb DEPENDS on or otherwise requires it). and libcamera is part of meta-multimedia.

The error is ERROR: Nothing PROVIDES 'libcamera' (but /work/recipes-multimedia/libcamera-apps/libcamera-apps_git.bb DEPENDS on or otherwise requires it). and libcamera is part of meta-multimedia. I have no idea how the CI handled changes targeting rpi-libcamera-apps before.

I think the dependency on meta-multimedia should be added here.

I think this recipe can be made conditional. There are couple of ways.

Remove it from parsing if multimedia layer is absent

python() {
    if 'multimedia-layer' not in d.getVar('BBFILE_COLLECTIONS').split():
        raise bb.parse.SkipRecipe('Requires meta-multimedia to be present.')
}

or use the dynamic-layers technique, which means move the recipe into
dynamic-layers/multimedia-layer/recipes-multimedia
I think second approach is better.

rpi-libcamera-apps/1.1.2+gitAUTOINC+12098520a3-r0/git/core/libcamera_app.hpp:55:40: error: 'StreamRoles' in namespace 'libcamera' does not name a type; did you mean 'StreamRole'?
   55 |         using StreamRoles = libcamera::StreamRoles;
      |                                        ^~~~~~~~~~~
      |                                        StreamRole

?
I've just noticed it in my world builds.

We need to bump SRCREV to get a fix for current libcamera from meta-oe:
https://github.com/raspberrypi/libcamera-apps/commit/3d9ac1087b3d6aa9cf3758c088414d7cf518c395

Thanks for update, I've tried to build it for rpi4-64 and fails with:
aarch64-oe-linux-g++: error: unrecognized command-line option '-mfpu=neon-fp-armv8'

Because even for aarch64 rpi4-64 it sets cpu to armv8-neon now (as raspberrypi4-64 OVERRIDES contain raspberrypi4 as well).

Thanks, now it built OK in my world builds. LGTM
Next achievement is for switching the build to meson as CMake files were removed in latest:
https://github.com/raspberrypi/libcamera-apps/commit/9e172655e257972e5da45d59a720192516f7d769 :)

bec7182 requires https://github.com/openembedded/meta-openembedded/pull/727 to be merged first.

bec7182 requires openembedded/meta-openembedded#727 to be merged first.

Did this land?

bec7182 requires openembedded/meta-openembedded#727 to be merged first.

Did this land?

It's in master-next now

bec7182 requires openembedded/meta-openembedded#727 to be merged first.

Did this land?

now yes.

This will get automerged on CI pass. Thanks @vivien.


TypeScript: Fix generic paramater Ref value getByRef() bug

Fixes #444
Implemented and tested via new and existing tsd tests

@uguraslan Let's hold back from merging this one. While doing some other typing work I discovered an issue with this that doesn't seem to be being detected by the tests.

Fixed the issue and added many more tests that should help solidify the whole tag/getByRef type chain. Explanation is a bit difficult to explain but here's an attempt:
Using tag() on a Component that has a lower-case property of instance type Element or Component in its TemplateSpec
resulted in tag() causing a very strange error due to "circular references". The new getByRef() implementation is based
on how tag() originally worked and saw the same exact issue.
The error was tracked back to IsTerminus returning false for any instance type of Element or Component and the fact that
SpecToTagPaths did not filter out lower-case properties. This resulted in SpecToTagPaths recursively entering a lower-case
Element/Component property (which is where it should have actually terminated recursion at) and then further recursively enter
the instance's stage, application, ctx, etc properties ultimately resulting in a circular reference.
SpecToTagPaths now only returns paths for upper case ValidRef properties. And IsTerminus returns true for
Element and Component.
I've also moved many of the internal type functions that existed in Element.d.mts into internalTypes.d.mts and added comprehensive
unit tests for each of them. I've also added a test for the fixed scenario that was originally untested in components-strong-typing.test-d.ts.
This has also been tested with the TypeScript version of TMDB which is how I originally found the issue.


AWS meter-based pricing

For deployments listed in AWS Marketplace, utilize AWS Marketplace Metering Service integration  as an alternative to Bring Your Own License (BYOL).

Code examples:
Python
response = client.meter_usage(
    ProductCode='string',
    Timestamp=datetime(2015, 1, 1),
    UsageDimension='string',
    UsageQuantity=123,
    DryRun=True|False,
    UsageAllocations=[
        {
            'AllocatedUsageQuantity': 123,
            'Tags': [
                {
                    'Key': 'string',
                    'Value': 'string'
                },
            ]
        },
    ]
)

https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/meteringmarketplace.html#MarketplaceMetering.Client.meter_usage

default MeterUsageResponse meterUsage(MeterUsageRequest meterUsageRequest)

https://sdk.amazonaws.com/java/api/latest/software/amazon/awssdk/services/marketplacemetering/MarketplaceMeteringClient.html#meterUsage-software.amazon.awssdk.services.marketplacemetering.model.MeterUsageRequest-
Custom: https://docs.aws.amazon.com/marketplace/latest/userguide/java-integration-example-meterusage.html
Hourly: https://docs.aws.amazon.com/marketplace/latest/userguide/java-integration-example-registerusage.html

Entitlements in boto3:  https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/marketplace-entitlement.html

A spike on using the Docker HEALTHCHECK as a means of driving metering:
https://github.com/Senzing/accountant-aws-marketplace


Convert to model should have a way to export material with absolute bitmap paths

-convert -format obj -absolutebitmap

May I ask why?

Yes of course, to open it up in the 3D editing tools and preview the textured model! You could then do what ever you want with it, relit manually and render a movie/image or, what is also could be a possibility, to export it further in other game editors without the need to retexture it manually. Now remember that model in this case can be represented as the whole map.

Alright, thank you :)


Use inlinecss twig tag with a variable

Hi,
Is there a way to use the inlinecss twig tag with a variable, like this:
{% inlinecss styleSheet %}
Thanks

up

up

+1

done in https://github.com/robertotru/ToInlineStyleEmailBundle/pull/30

https://github.com/robertotru/ToInlineStyleEmailBundle/pull/36/files

#36


HandleClickOutside fires when already closed

Just a small issue I found so far, handleClickOutside event should check if it is visible before emitting an event.
I had a hook on @closed that began firing on every click on the app, making api requests like every time.
P.S. The plugin is nicely done, cheers!

This is fixed in 1.7.1.
https://github.com/MikaelEdebro/vue-airbnb-style-datepicker/releases/tag/v1.7.1


[3.2.0] API Products are not listed in Devportal

Description:
API Products created from Publisher are not listed in Devportal. After removing the lines in [1], this works.
[1] https://github.com/wso2/carbon-apimgt/blob/master/components/apimgt/org.wso2.carbon.apimgt.rest.api.store.v1/src/main/java/org/wso2/carbon/apimgt/rest/api/store/v1/impl/ApisApiServiceImpl.java#L111-L112

This issue does not occur in the latest built pack. hence closing the issue.


Numba warning on initialization

I get the following warning when initializing an EVoC class object.
import evoc

clusterer = evoc.EVoC()

[c:\Users\Me\AppData\Local\miniconda3\envs\my-env\lib\site-packages\evoc\float_nndescent.py:287](file:///C:/Users/Me/AppData/Local/miniconda3/envs/my-env/lib/site-packages/evoc/float_nndescent.py:287): NumbaTypeSafetyWarning: unsafe cast from uint64 to int64. Precision may be lost.
  points = point_indices[i]

Is this anything to be concerned about?
Numba version 0.60.0

No, I don't think it is anything to worry about. I've tried tracing the
warning and the lines/details it gives are no help at all. I think it
maybin fact be a spurious warning.
On Mon, Jul 29, 2024 at 6:23 PM zilch42 @.***> wrote:

I get the following warning when initializing an EVoC class object.
import evoc
clusterer = evoc.EVoC()
[c:\Users\Me\AppData\Local\miniconda3\envs\my-env\lib\site-packages\evoc\float_nndescent.py:287](file:///C:/Users/Me/AppData/Local/miniconda3/envs/my-env/lib/site-packages/evoc/float_nndescent.py:287): NumbaTypeSafetyWarning: unsafe cast from uint64 to int64. Precision may be lost.
points = point_indices[i]
Is this anything to be concerned about?
Numba version 0.60.0
—
Reply to this email directly, view it on GitHub
https://github.com/TutteInstitute/evoc/issues/9, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC3IUBMKU2RGQ7MXUWZC4YDZO3THXAVCNFSM6AAAAABLVLAAKSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQZTMNRYG42DEMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


fix(datetime): support min/max for 12-hour clock

Short description of what this resolves:
Fixes: Setting min/max on datetime component does not work with 12-hour clock; am/pm column.
There were a few ways to implement. I took the path that would have the least impact to the existing code. Let me know your thoughts as to the approach.
Changes proposed in this pull request:

12-hour clock now starts with 12 and ends with 11, e.g. 12am, 1am, 2am, ... 11am
am/pm is added to dateSortValue(..)

Ionic Version: 2.2
Fixes: #10797

Hi @manucorporat. Can you take a look? Thanks.

Hello and thank you for contributing to Ionic! We have been working on porting all of the Ionic components to web components and have recently updated master to reflect this. This significant change has caused this pull request to break. While we really appreciate the time and effort you put into creating this, we are not able to merge it because of the newly introduced conflicts. We are extremely sorry about this. We will not be merging any more features in to v3. If this is a feature and you have the time, please resubmit this PR against the master branch. If this is a critical security issue in v3, we would greatly appreciate it if you would resubmit the PR against the new v3 branch. Thanks so much for your time!

Hey guys
I'm using<EMAIL_ADDRESS>and do not see this fix there.
Is it possible to add these updates to<EMAIL_ADDRESS>?
Thank you in advance.

Hey team. I am using Ionic 5. I am still facing this issue. Please help.
My code is :-
<ion-datetime displayFormat="DD/MM/YYYY HH:mm" displayFormat="DD/MM/YYYY h:mm A" formControlName="startDateTime" [min]="minDate" autocomplete="off"></ion-datetime>
this.minDate = moment().format("YYYY-MM-DDTHH:mm:ss")


ES portable not launching retroarch

Hi I'm trying to make a portable ES+Retroarch Windows installation.Everything works fine if I use the non portable exe run from windows but if I try and use the portable bat and the .emulationstation folder on the extrenal HD it is unable to launch retroarch
I am using the %HOMEPATH% address in my cfg and using the portable ES bat

Hi, i did the same thing. here are my files:
Portable.bat:
set HOME=%~dp0
emulationstation.exe
SNES:

snes
Super Nintendo Entertainment System
~.emulationstation\roms\snes
.smc .sfc .SMC .SFC
%HOME%retroarch\retroarch.exe -L %HOME%retroarch\cores\snes9x_libretro.dll "%ROM_RAW%"
snes
snes

i placed retroarch into the main folder.
Hope that helps...

HI thanks for the reply
I don't understand
Portable.bat:
set HOME=%~dp0
emulationstation.exe
ES is finding the roms but not launching them
heres my cfg file

change the %HOMEPATH% to the drive letter G: and also placed retroarch on root of folder as above emulation is now loading but this now means it is not portable

The portable .bat file sets %HOME%, not %HOMEPATH%. Try using %HOME% in the launch command.

thanks Aloshi that works perfectly.I got a bit confused and thought %HOME% was not used on windows only android

Hey guys, I am also having trouble with the portable setup. In fact I am trying to set it up on Google Drive, to share across computers. I am also using the Portable.bat: set HOME=%~dp0 emulationstation.exe
However, it is just not working for me (using Windows 7 64bit). Using absolute paths, I can get it to work perfectly:

Now, once I substitute either %HOME$ or "%HOME%" or %HOMEPATH% or "%HOMEPATH%" I get the dreaded ...launch terminated with nonzero exit code 1! in the log.
For example:

Any ideas? The program looks great, but I just can't figure this out.
Thank you so much, Best,
Philipp


Documentation for when prerelease images are available

Describe the bug
I can't find a description for when/how prerelease images are rolled out, or if/how prelease images can be used in my actions before they roll out.
Area for Triage:

Question, Bug, or Feature?:
Question
Virtual environments affected

[ ] macOS 10.15
[ ] Ubuntu 16.04 LTS
[x] Ubuntu 18.04 LTS
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

(possibly others)
Expected behavior
I'd like some way to know where in the pipeline a given release is in, and how likely/soon that release will be available.
The original specific problem I had was that Azure CLI 2.3.1 has a bug where it won't include error details in its output; this was fixed >20 days ago and is not present in the current (2.5.1) release of Azure CLI.
However, only Azure CLI 2.3.1 is available on ubuntu-latest (20200406.2), which is a month old at this point. The Ubuntu build just takes the latest Azure CLI version, and in fact the newest Ubuntu build (20200430.1) does have Azure CLI 2.5.1.
So the problem is that it's not clear where the environment releases are in the rollout process. For this specific problem, it's not clear what's keeping 20200430.1 from being available and used for ubuntu-latest.

Hello @StephenCleary ,
Actually, you can't switch between image versions. Let me clarify our deployment process a bit.
We have a huge number of VMs that we need to update and usually it takes 3-4 days to propagate new image to all VMs. Pre-release is created when deployment is started. And then pre-release is converted to release when the deployment is finished. Current deployment is in progress and we are expecting 20200430.1 to deploy to all VMs on Tue next week (Friday is deployment freeze).
As you can see from the previous releases, usually we deploy both Ubuntu and Windows image on weekly basis. The reason why the current image is not updated for a long time is that previously we had an incident with free space on Ubuntu images. Also we had deployment freeze the most of this week due to GitHub Satellite.
As I mentioned above, the new image with Azure CLI 2.5.1 should be available early next week. Thank you for your patience.
If it is your blocker, you can update Azure CLI in runtime using this code: curl -L https://aka.ms/InstallAzureCli | bash based on official documentation.
Please let me know if you have any other questions.

Ah, that makes perfect sense. Thanks!


Manual override the SessionDuration param if "Apply the SessionDuration requested" is set to No

Our AWS has a default of 1 hour session but allows up to 12 hours. The current SessionDuration doesn't seem to collect this maximum of 12 hours, goes to null, and AWS defaults to all generated credentials to one hour.
https://github.com/prolane/samltoawsstskeys/blob/master/background/script.js#L109
It would be nice if when we set Apply the SessionDuration requested to no, an input box could pop up to set an override value or is there a better way to dynamically get the maximum SessionDuration allowed by AWS?

This is the error when I try to go above 1 hour.  I have to set the Extension option to "No" for it to work with Roles that have been set to higher than 1 hour.  Since Okta is giving us a Role already, the assume role gets limited to 1 hour for all other roles you assume.  But the first role from your IdP can be higher than 1 hour.  https://forums.aws.amazon.com/thread.jspa?threadID=242680
"ValidationError: The requested DurationSeconds exceeds the 1 hour session limit for roles assumed by role chaining."


Error when generating proxy classes

I get this error about halfway through generating proxy classes:
ParseError thrown with message "syntax error, unexpected 'if' (T_IF), expecting identifier (T_STRING) or '{'"

Stacktrace:
#15 ParseError in {root}/httpdocs/init.php:820
#14 IPS\IPS:monkeyPatch in {root}/httpdocs/init.php:797
#13 IPS\IPS:autoloader in [internal]:0
#12 spl_autoload_call in [internal]:0
#11 property_exists in {root}/httpdocs/applications/toolbox/sources/Proxy/Generator/Proxy.php:240
#10 IPS\toolbox\Proxy\Generator\_Proxy:create in {root}/httpdocs/applications/toolbox/sources/Proxy/Proxyclass/Proxyclass.php:307
#9 IPS\toolbox\Proxy\_Proxyclass:build in {root}/httpdocs/applications/toolbox/sources/Proxy/Proxyclass/Proxyclass.php:205
#8 IPS\toolbox\Proxy\_Proxyclass:run in {root}/httpdocs/applications/toolbox/modules/admin/proxy/proxy.php:67
#7 IPS\toolbox\modules\admin\proxy\_proxy:IPS\toolbox\modules\admin\proxy\{closure} in {root}/httpdocs/system/Helpers/MultipleRedirect/MultipleRedirect.php:92
#6 IPS\Helpers\_MultipleRedirect:__construct in {root}/httpdocs/init.php:820
#5 IPS\Helpers\toolbox_hook_MultiRedirect:__construct in {root}/httpdocs/applications/toolbox/modules/admin/proxy/proxy.php:132
#4 IPS\toolbox\modules\admin\proxy\_proxy:manage in {root}/httpdocs/system/Dispatcher/Controller.php:96
#3 IPS\Dispatcher\_Controller:execute in {root}/httpdocs/applications/toolbox/modules/admin/proxy/proxy.php:45
#2 IPS\toolbox\modules\admin\proxy\_proxy:execute in {root}/httpdocs/system/Dispatcher/Dispatcher.php:152
#1 IPS\_Dispatcher:run in {root}/httpdocs/init.php:820
#0 IPS\toolbox_hook_Dispatcher:run in {root}/httpdocs/admin/index.php:14

Any ideas what might be causing this? Looks like something in init.php isn't being parsed correctly, but it's a stock install.

it doesn't appear to be a problem with proxyclass generator, it appears to be whatever file it is processing when it is hitting this.
if (property_exists( $dbClass, 'databaseTable' ) && class_exists($dbClass) && method_exists($dbClass, 'db'))
the error is being thrown by whatever class property_exists is loading to check for a property. #12 of the back trace shows it autoloading the class, then it failing. unfortunately its not showing what file its trying to load, it could be a hook or a class to a 3rd party app (to test, i ran it on a full suite for 4.4 and 4.5 and this error didn't happen to either one).

Any ideas for how I could debug that for you? I could probably have it write out each class as it loads.

if you switch to the 4.5 branch (it is tweaks that have been made to toolbox for 4.5, but it works as for 4.4.x as well as most of the changes for 4.5 are cosmetic when it comes to the toolbox).
i've added a catch for ParseErrors, it will add it to the ips system logs if you have DEBUG_LOG set to true in your constants.php (or if you use the toolbox's profiler and have debug+ajax debug enabled it will put the error there). they will have have the type of 'ParseErrorFile' (ParseError is the actual error, ProxyClass type are generalized things that might need fixed, but not always).
but there really isn't anything for me to debug, as the error is in the file it is trying to process, not with proxyclass generator, i've put a catch in there so the proxyclass can complete.

Hi @codingjungle . I'm on 4.5 branch try to generate proxy classes. They didn't work and only returns popup 'foo' message after refreshing page on click 'save' event. There someting 'in progress' state, is it?)

if you don't use the profiler with debug enabled, it will use the ips log, you can follow the images above to find where that is located to find the error (note: in these images i simulated an error in one of the files for toolbox, so this error isn't actually present when parsing for proxyclass).

Hi @codingjungle . I'm on 4.5 branch try to generate proxy classes. They didn't work and only returns popup 'foo' message after refreshing page on click 'save' event. There someting 'in progress' state, is it?)
upd. I proceed to the uri https://localhost/admin/?app=toolbox&module=proxy&controller=proxy and proxy classes generates well

not sure, sounds like you have a bit more going on that a problem with proxyclass,  the only one that pops up with a "foo" is when you save settings, proxyclass will say 'Proxy Classes have been Generated'.

Thanks @codingjungle, appreciate it. I'm gonna close this and try that out.


Fatal error: Uncaught exception 'LogicException' with message 'The specified directory path "vendor/louislam/louislam-crud/view/theme/AdminLTE

Fatal error: Uncaught exception 'LogicException' with message 'The specified directory path "vendor/louislam/louislam-crud/view/theme/AdminLTE" does not exist.' in /volume1/web/crud/vendor/league/plates/src/Template/Folder.php:72 Stack trace: #0 /volume1/web/crud/vendor/league/plates/src/Template/Folder.php(39): League\Plates\Template\Folder->setPath('vendor/louislam...') #1 /volume1/web/crud/vendor/league/plates/src/Template/Folders.php(31): League\Plates\Template\Folder->__construct('adminlte', 'vendor/louislam...', false) #2 /volume1/web/crud/vendor/league/plates/src/Engine.php(115): League\Plates\Template\Folders->add('adminlte', 'vendor/louislam...', false) #3 /volume1/web/crud/src/LouisLam/CRUD/LouisCRUD.php(292): League\Plates\Engine->addFolder('adminlte', 'vendor/louislam...') #4 /volume1/web/crud/src/LouisLam/CRUD/LouisCRUD.php(132): LouisLam\CRUD\LouisCRUD->addTheme('adminlte', 'vendor/louislam...') #5 /volume1/web/crud/src/LouisLam/CRUD/SlimLouisCRUD.php(57): LouisLam\CRUD\LouisCRUD->__construct() #6 /volume1/w in /volume1/web/crud/vendor/league/plates/src/Template/Folder.php on line 72
have followed the example and can't seem to get past this error.
Any Idea what could be wrong?

HI,
This project is a library, you should not run the library. So the project should inside the "vendor" folder.
Solution:
You start a new project with louislam-crud-bootstrap
https://github.com/louislam/louislam-crud-bootstrap


egressgw: tests: wait for initial sync reconciliation

make sure we wait for the first reconciliation (i.e. the initial k8s sync) in unit tests, as otherwise we might miss a reconciliation event and all the subsequent calls to waitForReconciliationRun might return before the reconciliation has actually run

/test

/test

Change only affects unit tests; safe to merge as- is.


How to assign a hostname through DNS

How do i include my domain in kingphisher, i dont know what nameserver to point to and how to host it in kingphisher, any help is appreciated.

Assuming you own the domain, you'd just go into the configuration and point one or more A records to the IP address of the King Phisher server.
For example if you bought the domain through GoDaddy you'd do this through your GoDaddy account.


Support composite primary keys

Current implementation assumes only one PK per entity.

Probably not worth the effort for now.


Can't use Magic in flutter web

Will Magic flutter sdk support web ? The package is using webview_flutter which is not supported in web, but there is webview_flutter_web. As I sow, webview_flutter_web is limited, but maybe it's enough to make magic-sdk work in web.

Unfortunately, we're not currently supporting Flutter on web.


Client doesn't register dependency paths for version managers based on shims

Reported in https://github.com/Shopify/ruby-lsp/discussions/2701
When we configure the client, we use the environment's GEM_PATH to register the language server for handling dependency files, which allows users to enjoy all of our features even inside gems.
However, that approach will not work properly for version managers based on shims, because those will not export a GEM_PATH when running the activation script.
I believe that the solution is to return another field from the activation script for the gem paths (which can be printed with Gem.path). That value will exist even for version managers based on shims, so we can rely on that to register the client paths instead of using the environment variable.

The ruby-lsp server runs in the container.
vscode ➜ /workspaces/abc $ ps aux | grep ruby-lsp
vscode       569  0.0  0.0   2324  1280 ?        S    14:55   0:00 /bin/sh -c ruby-lsp
vscode       570 50.2  1.7<PHONE_NUMBER>24 ?       Sl   14:55   0:03 /home/vscode/.rbenv/versions/3.3.5/bin/ruby-lsp
vscode       681 35.7  0.5<PHONE_NUMBER>2 ?        Rl   14:55   0:00 /home/vscode/.rbenv/versions/3.3.5/bin/ruby bin/rails runner /home/vscode/.rbenv/versions/3.3.5/lib/ruby/gems/3.3.0/gems/ruby-lsp-rails-0.3.20/lib/ruby_lsp/ruby_lsp_rails/server.rb start

ruby -e 'puts Gem.path'
/home/vscode/.local/share/gem/ruby/3.3.0
/home/vscode/.rbenv/versions/3.3.5/lib/ruby/gems/3.3.0

bundle show rails
/home/vscode/.rbenv/versions/3.3.5/lib/ruby/gems/3.3.0/gems/rails-<IP_ADDRESS>

Are you able to reproduce it on your machine? Because it runs in a devcontainer I would expect that it will won't work on your machine as well.


stock_zh_a_hist 月线返回多余的非月底的记录

遇到任何问题，请先确认您的 AKShare 版本是最新的
pip install akshare --upgrade  # Python 版本需要大于等于 3.8

描述遇到的问题

操作系统版本: windows 10

Python 版本: python3.10

AKShare 版本: 1.11.91

接口的名称和代码: stock_zh_a_hist

接口报错的截图或代码:
symbol = "600000"
period = "monthly"
start_date = "20240101"
end_date = "20240131"
adjust = "bfq"
"exchange","symbol","date","open","close","high","low","volume","amount","amplitude","change_pct","change","turnover_rate"
"SSE","600000","2024-01-31",6.63,6.83,6.97,6.36,9535270,6341026211.0,9.21,3.17,0.21,3.25
"SSE","600000","2024-02-02",6.83,6.84,6.92,6.72,1037353,709575270.0,2.93,0.15,0.01,0.35
不应该返回"2024-02-02"的记录

期望获得的正确结果
只能返回"2024-01-31"的记录

5. 20240101

按文档填入相应参数，注意 adjust 参数

需要计算结束日期是月末。和adjust无关。


Google Signin failing for deployed app on mobile

This is fixed in v0.3.4 and above. Closing as fixed


Search Posts API

Our API for browse posts should accept a query option which will perform a full text search on the post model. This should use the title, content, and possibly meta_* fields of a post.
At the moment, posts are self contained, but in the near future we will be adding additional tables such as tags/categories which will also need to be searchable.
Search should return a paginated set of matching posts, using the existing settings for limit, offset etc.

You might want to check out my new project that adds full text search to the Ghost platform.
It is rather simple and only supports SQLite, but it works :)
https://github.com/seesharper/GhostSearch

I think having a stand alone db for the search index would be best. This way you don't have to deal with fuzzy text searching inconsistencies (or nonexistence) of the various dbms's out there.
A possible option would be using a LevelDB backed index (file based key/value store) using the search-index module.
https://github.com/fergiemcdowall/search-index
Yes it adds another file to the mix that could grow quite large, but I believe it would have a lower memory footprint than Lunr.js.

@dwstevens Tell me if I'm wrong, but I really don't think levelDB is an option. As far as I am aware it'll add a new dependency that is far more complicated even than sqlite3 to get installed (and that has the wonderful node-pre-gyp feature), meaning it's not suitable for our user base.

Ah, my bad. You are correct.

Closing this issue in favour of #5321 which has plenty of discussion & traction. Having 2 issues is just confusing at this point.


Riot show dialog for battery optimization even if the system setting is already ok

It happens when user disconnect and reconnect. The SharedPref are cleared, but the system setting for this application remains set. So the dialog is displayed again, even if this is not necessary

Fixed in Riot 0.8.15


Fixed domain option matching

@pmezard Thanks very much for your adblock library, it's been very useful!
There is a problem with how you're handling domain matching options. The domain option should be based on the OriginDomain, not the Domain. See https://adblockplus.org/filter-cheatsheet#options.
This PR fixes that, including updating the tests.

Hello,
Thank you for the PR.

About checking OriginDomain, I think you are correct, I have to check it a bit more.
About testing all domain options at once, you may be correct too but I have another fix in mind. It is currently implemented as an "and" on all opts conditions. Maybe it should be an "or", once we match a rule, we look whether one of its option group satisfies the context. I feel this is equivalent to your fix in the domain case, and extends the semantics to other options in a logical way. Because, why treat "domains" in a specific way and not "third-party". The "genericblock" logic should be adapted to apply after each validated opts.
So:

||s3.amazonaws.com^$domain=imgclick.net,$image
||s3.amazonaws.com^$domain=gaybeeg.info

would be
||s3.amazonaws.com^(($domain=imgclick.net and $image) or ($domain=gaybeeg.info))

and not
||s3.amazonaws.com^(($domain=imgclick.net or $domain=gaybeeg.info) and $image)

What do you think?

Yes, good catch!

Fixed a bit differently by https://github.com/pmezard/adblock/pull/2 . Thanks for reporting this!


Aivu not progressing correctly?

So I'll be honest and say I don't know if you intended for this or not, I can only read what you put in the release notes and compare things from there.
A lot of things you say that Aivu gets at later mythic ranks, Aivu gets them all at mythic rank 6.
For example:
Aivu gets 32 spell resistance upon reaching mythic level 6, but you say that this is not suppose to happen until mythic 9.
Aivu's tail gets the crit multiplier at mythic 6, but you say this occurs at level 7.
Aivu gets delirium breath at mythic 6, again this is suppose to occur at level 7?
See screenshots below for evidence of this:

Also, I don't know if Aivu gets this visual back at later mythic ranks, but I've noticed she has lost the blue azata glow that she usually gets at mythic rank 6 in vanilla (or with the setting off), see screenshots below for what I mean. Again, not sure if this is what you intended.
Aivu upgrade setting on:

Aivu upgrade setting off:

And last but not least. It's mentioned that Aivu's stats now upgrade as mythic rank increases, but turning the "Aivu upgrade" setting on and off and I don't see any difference in stats (aside from the breath weapon attack), only abilities. Again, this might be intended or a quirk or some other thing, but I just thought I'd make you aware.
Let me know if you need any more info!

Do you have any other mods in use
On Tue, Aug 29, 2023, 22:52 Sutton618 @.***> wrote:

So I'll be honest and say I don't know if you intended for this or not, I
can only read what you put in the release notes and compare things from
there.
A lot of things you say that Aivu gets at later mythic ranks, Aivu gets
them all at mythic rank 6.
For example:
Aivu gets 32 spell resistance upon reaching mythic level 6, but you say
that this is not suppose to happen until mythic 9.
Aivu's tail gets the crit multiplier at mythic 6, but you say this occurs
at level 7.
Aivu gets delirium breath at mythic 6, again this is suppose to occur at
level 7?
See screenshots below for evidence of this:
[image:<PHONE_NUMBER>3237_1]
https://user-images.githubusercontent.com/67255571/264219372-da62fe79-b65e-47d5-ac16-46257e5dbd5a.jpg
[image:<PHONE_NUMBER>3244_1]
https://user-images.githubusercontent.com/67255571/264219476-1ae01282-e5a5-493a-a6b7-cbfe3f0abc58.jpg
Also, I don't know if Aivu gets this visual back at later mythic ranks,
but I've noticed she has lost the blue azata glow that she usually gets at
mythic rank 6 in vanilla (or with the setting off), see screenshots below
for what I mean. Again, not sure if this is what you intended.
Aivu upgrade setting on:
[image:<PHONE_NUMBER>4023_1]
https://user-images.githubusercontent.com/67255571/264220064-12da5391-84e6-4753-a70c-84f0e5e7c7f9.jpg
Aivu upgrade setting off:
[image:<PHONE_NUMBER>4507_1]
https://user-images.githubusercontent.com/67255571/264220859-33077cba-709a-4dba-b659-1028637c4161.jpg
And last but not least. It's mentioned that Aivu's stats now upgrade as
mythic rank increases, but turning the "Aivu upgrade" setting on and off
and I don't see any difference in stats (aside from the breath weapon
attack), only abilities. Again, this might be intended or a quirk or some
other thing, but I just thought I'd make you aware.
Let me know if you need any more info!
—
Reply to this email directly, view it on GitHub
https://github.com/Vek17/TabletopTweaks-Reworks/issues/20, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWSQPHANF2A2HULL456GLLXX22IVANCNFSM6AAAAAA4D34SR4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Fixed
https://github.com/Vek17/TabletopTweaks-Reworks/releases/tag/v1.2.9

Ah, good to hear you found the issue so promptly!
I considered as much about the stat thing, and I did see the mythic draconic growth racial bonus. I also was testing it going from mythic 5 -> 6 (as in, I load a game with me at mythic rank 5 and then level up to 6) with both the aivu upgrade setting on and then off, and there was no difference in the amount of stat increase. Maybe it was just a coincidence that this lines up with the vanilla stat increase? From mythic 5 to mythic 6, for example, Aivu goes from +6 racial bonus to strength, to +8. Again, it's probably nothing, but just wanna make sure!
Also, Aivu doesn't get to have blue aura? :<

The aura was part of a bugged effect I removed technically so it isn't
there anymore
On Wed, Aug 30, 2023, 07:18 Sutton618 @.***> wrote:

Ah, good to hear you found the issue so promptly!
I considered as much about the stat thing, and I did see the mythic
draconic growth racial bonus. I also was testing it going from mythic 5 ->
6 (as in, I load a game with me at mythic rank 5 and then level up to 6)
with both the aivu upgrade setting on and then off, and there was no
difference in the amount of stat increase. Maybe it was just a coincidence
that this lines up with the vanilla stat increase? From mythic 5 to mythic
6, for example, Aivu goes from +6 racial bonus to strength, to +8. Again,
it's probably nothing, but just wanna make sure!
Also, Aivu doesn't get to have blue aura? :<
—
Reply to this email directly, view it on GitHub
https://github.com/Vek17/TabletopTweaks-Reworks/issues/20#issuecomment-1699057522,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWSQPBE2IMM6O2CZBUNAITXX4VR7ANCNFSM6AAAAAA4D34SR4
.
You are receiving this because you commented.Message ID:
@.***>


Add BERT Base to Composer

Motivation
Composer currently contains autoregressive language models, but masked language models are just as important for speeding up training. This PR serves to add the first of masked language models (BERT) to Composer.
Merging Criterion

[x] Demonstrate that BERT achieves acceptable downstream GLUE performance on all benchmarks
[ ] Demonstrate average GLUE performance on validation set and test set
[x] Code quality should be up-to-par, with well-commented abstractions
[ ] New TorchMetrics that have been added should include tests
[x] New YAML files should be clean and not contain any unnecessary hparams

As a heads up, I'm ripping out the duration abstraction that I created. It's useful for internal experimentation, but I think Ravi's Time abstraction will be a superset of its features. If anyone thinks that it should be included, feel free to mention it and I can include it

Cool, this is ready for review!
To the reviewers: Let me know if you want any WandB notebooks on hyperparameters that I have swept for pre-training. I ended up sweeping learning rate and Adam's beta2, so we have WandB experiments for that. Additionally, we have WandB experiments for all of the GLUE downstream tasks. It might be good to verify that I copied the right hyperparameters from there.
I ended up leaving the duration abstraction in, and fixed the unit tests for that. This will be ripped out soon in favor of Ravi's time abstraction, but it could be useful until then.
Let me know if you have any questions! The final performance is available in the #r-nlp channel in Slack if anyone wants to view it.

Mostly reviewed new changes to metrics. LGTM!


PackedScene won't save nodes added to non-root nodes

Tested versions
v4.2.stable.official [46dc27791]
System information
Godot v4.2.stable - Windows 10.0.22631 - GLES3 (Compatibility) - Intel(R) Iris(R) Plus Graphics (Intel Corporation; <IP_ADDRESS>64) - Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz (8 Threads)
Issue description
When saving with PackedScene.pack(), child nodes that are added at run time to nodes which are not the root of a scene will not be saved regardless of what owner they have.
Code in MRP:
func _on_button_add_feather_pressed():
	for c in get_children():
		if c.scene_file_path == "res://duck.tscn":
			var label = c.find_child("Sprite2D")

			var new1 = scn_feather.instantiate()
			var new2 = scn_feather.instantiate()

			c.add_child(new1) ##this adds the node to the root of duck.tscn
			new1.text = "this will be saved"

			label.add_child(new2) ##this adds the node to a non-root node inside duck.tscn
			new2.text = "this won't be saved"

Note that in the MRP, the owner of all saved nodes is set by a recursive call in main.gd, but the exact way the owner is set does not matter.
Steps to reproduce
with MRP:

open and run
click "make pond"
click "spawn ducks" on the pond
click "add feather" on the pond
click "save" then "load"
you will see that the feather (label) added to the root of duck.tscn will appear in the copy but the feather added to the sprite of the duck will not.

Minimal reproduction project (MRP)
PackedScene save test.zip

@aaronp64 using var should_set := new_owner == null || new_owner.is_ancestor_of(node) in the MRP does work, but this code seems like it just naively sets all descendants of the new_owner as owned by new_owner, which I recall not working in my actual production.
I will be verifying this in the main project, but in the meantime, can you explain why new_owner == null is a part of the should_set condition? It seems like if that part of the statement is true the whole operation would be pointless since the owner is null, is this a typo?

Okay, I found a problem with changing _set_owner_recursive to use var should_set := new_owner == null || new_owner.is_ancestor_of(node): it creates orphaned nodes after a savefile is loaded
You can verify this using the "print orphan" button. With the MRP it seems to create an orphan of the sprite. With my main project it creates a huge cascade of orphans probably because the node tree is deeper with more inappropriate set owner operations. I remember someone warning me once that if a node already has an owner, setting it to something else will cause orphans to be created when the save file is loaded so naively setting owner for all descendants was probably a mistake.

So it looks like the orphans are due to the PackedScene seeing that the Duck object came from Duck.tscn.  When it loads back in, it uses that file, and loads its children in from there.  But if we set the owner to the Pond before saving, the PackedScene also saves the Duck's child nodes separately, so there's two copies to load, and one gets orphaned.
It might be possible to have PackedScene handle the orphan case, or be able to skip a node and include its children, which would work with your original MRP, but I don't know enough about it to tell if either of those options makes sense - someone more familiar with the PackedScene logic would have to advise on that.
I did find a way to get everything saved/loaded without orphans (at least, it seems to work for the MRP), though it feels like there should be a simpler way to do this.  Changing _set_owner_recursive to the below, and passing in an empty dictionary for old_paths:
func _set_owner_recursive(node : Node, new_owner : Node, old_paths : Dictionary):
	if new_owner == null:
		return

	var should_set := new_owner.is_ancestor_of(node)

	if should_set:
		node.owner = new_owner
		if node.scene_file_path:
			old_paths[new_owner.get_path_to(node)] = node.scene_file_path
			node.scene_file_path = ""
	for c in node.get_children():
		_set_owner_recursive(c, new_owner, old_paths)

If you don't need the original scene_file_path values, you can leave out the old_paths dictionary, and just clear out the old values.  If you want to set them back after saving, you can loop through the dictionary keys and do something like pond.get_node(key).scene_file_path = value.  To set them back after loading, you would need to also save/load the dictionary.

so it seems like a node having a non-empty scene_file_path is causing the duplication. I haven't had time to verify if the fix scales to bigger projects or whether setting scene_file_path to empty causes other problems elsewhere but it's definitely unintuitive that it has this side effect.
Not sure if I should close this issue and create a new one


Add a using/1 macro to the Connection module

Hey there @fishcakez, I'm not sure we're supposed to start contributing to this project (given its newborn nature) but nobody can get hurt trying, right? :)
Similarly to how GenServer in Elixir provides a __using__/1 macro that defines default implementation for the :gen_server behaviour callbacks, I added a __using__/1 macro to Connection that does exactly the same.
The implementation for the :gen_server behaviour functions has been taken verbatim from Elixir's GenServer. I only added the default implementation for connect/2 and disconnect/2.
Long story short, with this PR you're able to do:
defmodule TCPConn do
  use Connection

  # only define functions I care about
end

and not get warnings for not having implemented, say, terminate/2 or code_change/3. Let me know what you think :)

Please feel free to contribute. I hope it is feature complete already - except test cases.

@fishcakez fixed the things you mentioned and push --force'd :) Let me know what you think!

This is perfect, thanks. I will think about reverting the :noconnect returns from init/1 to :ok.


chore: cleaning warnings

Changes

#748  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @AngelOnFira and the rest of your teammates on  Graphite


Ng-packagr can't read property 'toString()' of undefined

Type of Issue

[x] Bug Report
[ ] Feature Request

Description
ng-packagr experiences an exception in fs.js:12 -> resolve(buffer.toString()); can't read property 'toString()' of undefined.
My guess is that fs is failing, but resolve is still attempting to execute after reject in the fs.ts file. This operates under the assumption that buffer is undefined when an error has occurred. If that assumption is not correct, then an additional check would need to be added to the follow fix:
if (err) {
    reject(err);
} else {
    resolve(buffer.toString());
}

How To Reproduce
Clone my repo, npm install, then npm run build:lib
Expected Behaviour
ng-packagr should either succeed or print a helpful error message.
Version Information
ng-packagr: v1.1.0
node: v8.5.0
@angular: v4.4.1
rxjs: v5.4.3
zone.js: v0.8.17

Yes, you found the right code place.
There's a bug when the files does not exist (maybe a mis-spelled path) and then it errors because buffer is undefined.


Ignore _ver#s and _ver#c$ properties

while building merged HTML URL
quick fix for ignoring Palindrom meta data https://github.com/Starcounter/starcounter-include/issues/37

2.4.1 released.
@warpech Should I merge also https://github.com/Starcounter/level1/pull/4232 to https://github.com/Starcounter/level1/tree/feature/v2.3/level1/4017?

Why would that be needed?

because there's already released version. Anyways, I already merged it.


Game crashes on startup

I've wrote a simple demo to check if the module was working but when I launch the game it crashes.
This is the content of the logcat:
java.lang.ClassNotFoundException: Didn't find class "org/godotengine/godot/GodotGooglePlayGameServices"
on path: DexPathList[[zip file "/data/app/org.godotengine.mygame-2/base.apk"],
nativeLibraryDirectories=[/data/app/org.godotengine.mygame-2/lib/arm,
/data/app/org.godotengine.mygame-2/base.apk!/lib/armeabi-v7a, /system/lib, /vendor/lib]]
 at dalvik.system.BaseDexClassLoader.findClass(BaseDexClassLoader.java:56)
 at java.lang.ClassLoader.loadClass(ClassLoader.java:380)
 at java.lang.ClassLoader.loadClass(ClassLoader.java:312)
 at org.godotengine.godot.GodotLib.setup(Native Method)
 at org.godotengine.godot.Godot$2.run(Godot.java:299)
 at android.opengl.GLSurfaceView$GLThread.guardedRun(GLSurfaceView.java:1477)
 at android.opengl.GLSurfaceView$GLThread.run(GLSurfaceView.java:1259)

And this is the code I've attached to my main (and only) scene:
extends Node2D

var gps = null

func _ready():
	if Globals.has_singleton("GodotGooglePlayGameServices"):
		gpgs = Globals.get_singleton("GodotGooglePlayGameServices")
		gpgs.init(get_instance_id())
		gpgs.signIn()

And this is what I've added to the project.godot file
[android]
modules="org/godotengine/godot/GodotGooglePlayGameServices"

same problem :(


Uncaught Exception: Command failed:<PHONE_NUMBER>

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.0.0
Time: Sun Nov 24 2019 13:09:25 GMT 0200 (FLE Standard Time)
Error:
Command failed:<PHONE_NUMBER>
Error: Command failed:<PHONE_NUMBER>
System.AggregateException: One or more errors occurred. ---> System.Net.WebException: The remote server returned an error: (500) Internal Server Error.
   at System.Net.HttpWebRequest.EndGetResponse(IAsyncResult asyncResult)
   at System.Net.WebClient.GetWebResponse(WebRequest request, IAsyncResult result)
   at System.Net.WebClient.DownloadBitsResponseCallback(IAsyncResult result)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.Utility.<LogIfThrows>d__4d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.FileDownloader.<DownloadFile>d__9.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwait
...Some of the stack is missing

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #2799


Mac OS X "Invalid key shortcut sound" on ⌃⌘↓

VSCode Version: 1.20.1
OS Version: 10.11.6

Steps to Reproduce:

Press ⌃⌘↓ in the text editor. The "invalid key shortcut" sound will be played. (This is also the case for ⌃⌘← and ⌃⌘→.
Press ⌃⌘↑. The "invalid key shortcut" sound will not be played.

There's clearly something going funny here, because up and down have different behaviour. These sounds play even if they are valid key shortcuts.

Blocked on electron/electron#2617

Yeah this also affects move to next/previous terminal pane keys I found recently.

+1

Both Electron and Chromium issues were closed as won't fix 😢  https://bugs.chromium.org/p/chromium/issues/detail?id=916460

FYI for anyone else stumbling into this.
The workaround given in the upstream Chromium issue is to setup key bindings at the OS-level that are just noops.
$ mkdir -p ~/Library/KeyBindings
$ cat > ~/Library/KeyBindings/DefaultKeyBinding.dict <<EOF
{
  "^@\UF701" = "noop";
  "^@\UF702" = "noop";
  "^@\UF703" = "noop";
}
EOF

Updates FAQ in terminal to include the workaround, thanks! https://github.com/microsoft/vscode-docs/issues/4395

so is this an issue with whatever upstream codebase vscode depends on?  I get this annoying beeping sound every time I move a line down via a keyboard shortcut.

so is this an issue with whatever upstream codebase vscode depends on?

Yes. This is an issue in Chromium and thus Electron. The upstream bugs in both Chromium and Electron have been closed as "won't fix."

electron/electron#2617
https://bugs.chromium.org/p/chromium/issues/detail?id=916460

Is this workaround no longer working for anyone else on macOS 14.4? Two machines, same effect and same keys as original report (i.e. ⌃⌘↑ is fine).

I think this bug came back after a few years

Here is the new workaround from the upstream Electron bug:
{
    "@^\UF700" = "noop:";
    "@^\UF701" = "noop:";
    "@^\UF702" = "noop:";
    "@^\UF703" = "noop:";
    "@~^\UF700" = "noop:";
    "@~^\UF701" = "noop:";
    "@~^\UF702" = "noop:";
    "@~^\UF703" = "noop:";
}

Basically, you needed to add the : at the end of noop and the order of the modifier symbols is different.
I've included the codes for cmd-ctrl-arrow and cmd-opt-ctrl-arrow above.
The codes for the up arrow (700) are probably not needed.
Details for what these codes mean are at: https://gist.github.com/trusktr/1e5e516df4e8032cbc3d
I'm AFK, so I haven't tested this yet.

This is working great for me on two 14.4 Macs.


Blurry Labels

Hi there.
When using Hyprlock, around 80% of the time labels appear scaled and blurry. The other 20% of the time it works how it should.
(The odds seem to shift in favour of normal labels when waiting a bit longer before locking again?)
I don't really understand what's happening, especially since it is changing constantly.
I guess it is somewhat related to those other scaling issues that are around with Wayland and Hyprland?
Thanks.

I also have this exact issue :(

I fixed the issue by reinstalling Hyprland using hyprland-git (AUR). Hopefully this help :)


Issue with audio playback.

Description

I am running a Lavalink server off of a raspberry pi. The server works for some songs but for other songs it gives me an error. It also is prone to long pauses on any device I run it off of.
Version info
Client used:
Wavelink.py
Output of java -version:

Error

We do not support ARM CPUs. Duplicate of #279