Scaffolder - GithubResponseError: Pull request creation failed; caused by HttpError: Not Found

Expected Behavior
I'm able to scaffold without error
Actual Behavior
Receiving GithubResponseError: Pull request creation failed; caused by HttpError: Not Found consistently with my build. Have tried upgrading. Have tried giving more permissions to the token. Unfortunately, the error message doesn't give a whole lot of info on what's actually going wrong.
https://github.com/backstage/backstage/issues/8263 seems related, but this is not intermitent.
Steps to Reproduce
Hard to say, I've never been able to get it working in my local or staging instances.
Context
We currently can't do any kind of scaffolding.
Your Environment

Browser Information: Firefox 100.0 (64-bit)

Output of yarn backstage-cli info:

OS:   Linux 5.4.0-110-generic - linux/x64
node: v14.18.3
yarn: 1.22.18
cli:  0.17.1 (installed)

Dependencies:
  @backstage/app-defaults                               1.0.2
  @backstage/backend-common                             0.13.5
  @backstage/backend-tasks                              0.3.1
  @backstage/catalog-client                             0.8.0, 0.9.0, 1.0.2
  @backstage/catalog-model                              0.12.1, 0.13.0, 1.0.2
  @backstage/cli-common                                 0.1.9
  @backstage/cli                                        0.17.1
  @backstage/config-loader                              1.1.1
  @backstage/config                                     0.1.15, 1.0.1
  @backstage/core-api                                   0.2.23
  @backstage/core-app-api                               1.0.2
  @backstage/core-components                            0.9.4
  @backstage/core-plugin-api                            0.1.13, 0.8.0, 1.0.2
  @backstage/core                                       0.7.14
  @backstage/dev-utils                                  1.0.2
  @backstage/errors                                     0.1.5, 0.2.2, 1.0.0
  @backstage/integration-react                          0.1.25, 1.1.0
  @backstage/integration                                0.8.0, 1.2.0
  @backstage/plugin-api-docs                            0.8.5
  @backstage/plugin-app-backend                         0.3.32
  @backstage/plugin-auth-backend                        0.14.0
  @backstage/plugin-auth-node                           0.2.1
  @backstage/plugin-catalog-backend                     1.1.2
  @backstage/plugin-catalog-common                      1.0.2
  @backstage/plugin-catalog-graph                       0.2.17
  @backstage/plugin-catalog-import                      0.8.8
  @backstage/plugin-catalog-react                       0.8.1, 0.9.0, 1.1.0
  @backstage/plugin-catalog                             1.2.0
  @backstage/plugin-circleci                            0.3.5
  @backstage/plugin-github-actions                      0.5.5
  @backstage/plugin-org                                 0.5.5
  @backstage/plugin-permission-common                   0.5.3, 0.6.1
  @backstage/plugin-permission-node                     0.6.1
  @backstage/plugin-permission-react                    0.3.4, 0.4.1
  @backstage/plugin-proxy-backend                       0.2.26
  @backstage/plugin-scaffolder-backend                  1.2.0
  @backstage/plugin-scaffolder-common                   1.1.0
  @backstage/plugin-scaffolder                          1.2.0
  @backstage/plugin-search-backend-module-elasticsearch 0.1.4
  @backstage/plugin-search-backend-module-pg            0.3.3
  @backstage/plugin-search-backend-node                 0.6.1
  @backstage/plugin-search-backend                      0.5.2
  @backstage/plugin-search-common                       0.3.4
  @backstage/plugin-search-react                        0.2.0
  @backstage/plugin-search                              0.8.1
  @backstage/plugin-shortcuts                           0.2.6
  @backstage/plugin-tech-radar                          0.5.12
  @backstage/plugin-techdocs-backend                    1.1.1
  @backstage/plugin-techdocs-module-addons-contrib      1.0.0
  @backstage/plugin-techdocs-node                       1.1.1
  @backstage/plugin-techdocs-react                      1.0.0
  @backstage/plugin-techdocs                            1.1.1
  @backstage/plugin-user-settings                       0.4.4
  @backstage/release-manifests                          0.0.3
  @backstage/search-common                              0.3.4
  @backstage/test-utils                                 1.1.0
  @backstage/theme                                      0.2.15
  @backstage/types                                      0.1.3, 1.0.0
  @backstage/version-bridge                             0.1.2, 1.0.1

Ok sorry about that, now that I have the access I need I can verify that we're not using OAuth but are just using the basic token integrations:
integrations:
  github:
    - host: github.com
      token: ${GITHUB_TOKEN}

I'll try elevating the permissions of the token to see if that alleviates the issues we're having and report back.

Alright, I added the following perms and it fixed the issue. I'm assuming the org_hooks are what's important here. Is there documentation that goes over any of this that I may have missed?

Can we see the template.yaml that you're trying to use? Have you tried using any of the example ones. What do you fill in for the RepoUrlPicker field where it says owner and name?

@benjdlambert as mentioned the problem was fixed with the permissions change. If you have docs that go over what permissions to set in GitHub I wasnt able to find them. I did follow other examples out there, but none of them go over the perms needed for the token.

Hi
There is mentions of permissions for the token at https://backstage.io/docs/getting-started/configuration#setting-up-a-github-integration and some in this section which the getting started guide links to https://backstage.io/docs/integrations/github/github-apps
@voodooGQ I'm sure we can do better and if you have some suggestions please suggest them in a PR 🙏
I'm going to close this issue as the problem seems to be fixed, feel free to reopen if there are outstanding issues :)

I have the same issue, I posted in Discord
I think that before when we were using the GitHub token it was working but now with the GitHub APP is broken and I have this error
GithubResponseError: Pull request creation failed; caused by HttpError: Not Found
15    at Object.handler (/app/node_modules/@backstage/plugin-scaffolder-backend-module-github/dist/index.cjs.js:2175:15)

This is our app-config
integrations:
  bitbucketCloud:
    - username: ${BITBUCKET_CLOUD_USERNAME}
      appPassword: ${BITBUCKET_CLOUD_PASSWORD}
  okta:
    apiToken: ${OKTA_API_TOKEN}
  awsS3:
    - accessKeyId: ${AWS_ACCESS_KEY_ID}
      secretAccessKey: ${AWS_SECRET_ACCESS_KEY}
  github:
    - host: github.com
      apps:
        - $include: backstage-app-credentials.yaml

...
catalog:
  import:
    entityFilename: catalog-info.yaml
    pullRequestBranchName: backstage-integration
  providers:
    github:
      mainOrg:
        organization: 'ORG_NAME'
        catalogPath: '**/catalog-info.yaml'
        validateLocationsExist: false
        filters:
          branch: 'main'
          repository: '.*'


Support for CloudWatch Events as event source

I am using this as event source, this is a scheduled event on cloud watch:
event_sources:
  - arn: arn:aws:events:us-east-1:XXXX:rule/some-rule

And getting this error:
ValueError: Unknown event source: arn:aws:events:us-east-1:XXXX:rule/some-rule

I checked the code and CloudWatch events seems to not be implemented as a type of event source.
I can help with a PR if you are able to give me the directions on what I need to do to implement it.

I'm interested in this feature too and may take a crack at it on my fork.  I'll post an update here if I have something working if others want to help test it.

Got something working today. Does kappa own the event sources specified in the file?  If so it seems odd to specify the arn in the yml file up front.  I deviated slightly from the existing pattern in Context._create_event_sources() and introduced a notion of type - here's the example yaml:
---
name: kappa-cron
environments:
  dev:
    profile: xyz
    region: us-west-2
    event_sources:
      - type: cloudwatch
        name: hello-cron-dev
        schedule: rate(1 minute)
        description: cron to run this lambda function every minute
        enabled: true
lambda:
  description: Kappa sample lambda that runs every minute
  handler: simple.handler
  runtime: python2.7
  memory_size: 128
  timeout: 3

The add() method does three things:

events.put_rule
lambda.add_permission
events.put_targets

The assumption is that the rule is owned by kappa and can be removed if/when the lambda function is removed.
Any comments on the above?  I could revert to the arn convention, but keeping things like the account id out of the yaml file seems good if we can avoid it.
The alternative would look like this (I removed some irrelevant sections for comparison):
---
name: kappa-cron
environments:
  dev:
    event_sources:
      - arn: arn:aws:events:us-west-2:1234567890:rule/hello-cron-dev
        schedule: rate(1 minute)
        description: cron to run this lambda function every minute
        enabled: true

Ok - PR has been filed:
https://github.com/garnaat/kappa/pull/69


Change Automatic Price Refresh Interval

Hi hunter, could we please have a setting that let´s us choose the interval we want for all prices to be automatically refreshed? I'm guessing all settings will be "safe", but would be nice to know what is the minimum we can go.
Thanks!

I was going to post the same thing. My items keep listing for the same price before it updates.

You can go as low as you want, but then you would be constantly refreshing prices and never bidding!

To add to this, maybe it could have the option to refresh the price before every time it lists someone. One time today the bot refreshed the price when a snipe was up, so it got rid of all my investments for half price. I think this option would help prevent that.

@WillowWillowWillow will be tackling this feature.


Include legacy values mapped to array subproperties in Mapped legacy fields

This would be accomplished by replacing sub_prop_name here with prop_path here:

                        self.report_legacy_mapping(
                            self.legacy_basic_property(sub_prop_name), True, True
                        )

I wonder if there would be any downside to consistently using prop_path as the argument for self.legacy_basic_property?

I think implementing this would also allow us to close #79

.... and also close #84


Updated to BIDS 1.0.0-rc3

Looks good to me except I think you used the wrong issue code on the slice encoding check. They're both 34 currently. I'm responding from a phone so I'm not able to test it, but everything looks fine to me if you want to merge after checking the issue code.


feat: add ad-hoc pipeline just for push br-cse to tiup stagine

Now we use the new CD flows, no need to merge it any more!


Cannot generate seed with randomised music, if other game isn't installed but not selected as a BGM source

I set up BioRand the other day with my installation of RE2 SourceNext, and didn't have RE1 installed.
I noticed that I couldn't generate a seed with randomised music - I was getting the error message "RE1 installation must be enabled to use RE1 assets.". However I didn't actually have RE1 selected as a BGM source so I should be able to generate a seed with randomised music as long as the other game isn't selected.
Looked at making a fix before creating this issue - I noticed that, before generating a seed, there is a check to see if the other game's BGM is selected, and if it is and that game is not installed, then the seed will not be generated and that error message will be displayed.
However, this check doesn't actually check which game BGMs are actually selected, it just checks if that game's name (either RE1 or RE2) is present in a hash set composed of RE1, RE2 and the names of any BGM folders. RE1 and RE2 are always present in this hash set so it will always think that you selected RE1 or RE2.
Going to link a fix shortly that should fix this issue - essentially just need to check RandoConfig.EnabledBGMs to see if the BGM is actually selected.

First attempt at an actual code contribution to open source, so let me know if I missed anything - not sure if I need to have this issue assigned to myself or add myself to CONTRIBUTORS.md if my changes make it through?


file not available ceaser-easing

Please help me
While using SASS file how can i resolve dependency for
@import "compass/CSS3";
@import "ceaser-easing";
when i am running my gulp task to convert all SASS to CSS it is showing some error
Error in plugin 'sass'
Message:
app\lib\angular-material\css\ADM-dateTimePicker.scss
Error: no mixin named box-shadow
Backtrace:
app/lib/angular-material/css/ADM-dateTimePicker.scss:127
on line 127 of app/lib/angular-material/css/ADM-dateTimePicker.scss

        @include box-shadow(inset 0 1px 1px rgba(black,.075));

Install these packages first:
gem install compass
gem install ceaser-easing


python3Packages.md-toc: 8.1.2 -> 8.1.3, python3Packages.mdurl: 0.1.0 -> 0.1.1

Description of changes
https://blog.franco.net.eu.org/software/CHANGELOG-md-toc.html#813---2022-04-20

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 169535 run on x86_64-linux 1

2 packages failed to build:

python310Packages.jupytext
python39Packages.jupytext

11 packages built:

python310Packages.markdown-it-py
python310Packages.md-toc
python310Packages.mdformat
python310Packages.mdit-py-plugins
python310Packages.mdurl
python310Packages.mypy-boto3-builder
python39Packages.markdown-it-py
python39Packages.md-toc
python39Packages.mdformat
python39Packages.mdit-py-plugins
python39Packages.mdurl


addon isn't loading when dropped in <Guild Wars 2>/addons

are there other dependencies that the compass relies on that i need to have installed? i'm currently using GW2 Addon Manager for my other addons and just dropped this one in the /addons folder, but I can't seem to get it to load.

Hello,
loading Compass through Addon manager is not and will not be supported. Compass is a Nexus' addon - you can read more here raidcore.gg/Nexus.
There is highly likely for other addons to be ported to Nexus as well in the future either by their authors or by raidcore/nexus team.
Nexus is compatible with arcdps in following configuration:

nexus: d3d11.dll
addon loader: d3d11_chainload.dll
arcdps: put it's DLL into addons

Or most other combinations of above. Please take note that addon loader is NOT chainloading, therefore Nexus cannot be loaded by it.

I see - I'm rather new to this but is there a link somewhere that walks you through how to chainload Nexus > AddOn Manager > ReShade

Get the Nexus Installer from the URL in Sognus' reply. The installer should automatically handle all of that.
In case it asks you to download .NET, you will have to do a manual installation of Nexus.
For a fully manual installation of Nexus + AddonLoader do the following:

Rename AddonLoader's d3d11.dll to d3d11_chainload.dll (.dll is the file extension, if it doesn't have one, leave it out)
Download Nexus' d3d11.dll and place it into the main directory of Guild Wars 2. https://api.raidcore.gg/d3d11.dll
You might have to unblock Nexus, to do this Right-Click -> Properties -> if there's an Unblock checkbox near Read-Only and Hidden, make sure it's checked.
(Optional) If using ArcDPS, move it into the addons directory to have it loaded by Nexus instead of Addon Loader.

Thank you, your step-by-step worked perfectly 👍

Another question - how do you edit the settings / position of the compass? I see a settings.json file that's in the Compass folder but all it says is
{
	"CompassIndicatorLong": null,
	"CompassIndicatorOutline": null,
	"CompassIndicatorTextPrefix": null,
	"CompassStripOffsetV": null,
	"IsCompassIndicatorLocked": null,
	"IsCompassIndicatorVisible": null,
	"IsCompassStripVisible": null,
	"IsCompassWorldVisible": null
}

You can access the options in Nexus by default via CTRL+O (O as in Options) or by clicking the Nexus icon in the top left and then selecting Options from the menu. There you should be able to set the values.

If you need further help I recommend you to ask via Discord https://discord.gg/raidcore. There I can give you some live support. :)


Unable to decompiled file!

when I open then decompiled file it occurs error like below.
java.lang.IllegalStateException: Wrong magic number:<PHONE_NUMBER>
at com.strobel.assembler.metadata.ClassFileReader.readClass(ClassFileReader.java:338)
at com.strobel.assembler.metadata.MetadataSystem.resolveType(MetadataSystem.java:129)
at com.strobel.assembler.metadata.MetadataSystem.lookupTypeCore(MetadataSystem.java:86)
at com.strobel.assembler.metadata.MetadataResolver.lookupType(MetadataResolver.java:46)
at us.deathmarine.luyten.Model.openEntryByTreePath(Model.java:338)
at us.deathmarine.luyten.Model$TreeListener$1.run(Model.java:266)

I would say the jar is either corrupt, or the class could be encrypted, the software could be decrypting classes and loading them later as a sort of obfuscation. Either way I can only reproduce this intentionally, by renaming a binary file to class and putting it in a jar.
You may have to do a little more investigation, look for a class extending Classloader and pulling resources from the jar. I once wrote a wrapper that would decrypt and load each class and after phoning home and authenticating using a public and private key. Either way there is something else going on here.


update linux download link

point to haxe.org linux packages page

Hi @andyli
There has been problems in the past with Haxe being partially installed on Linux, but missing either proper Neko support (any library that uses CFFI fails due to missing libraries) or being an old version that is incompatible.
I'm hesitant (but excited) to point developer to PPAs and other kinds of official paths rather than our own shell script, but only if it is reliable. Is Haxe 3.2.1 available in all these channels? Have they been tested with libraries like OpenFL?
Is haxelib setup with user read/write permissions after install on these, and is it haxelib as a Neko binary, not as a Haxe interp?
Thank you :smile:

partially installed on Linux, but missing either proper Neko support (any library that uses CFFI fails due to missing libraries) or being an old version that is incompatible

That's exactly what package systems are trying to solve - they allow us to properly define software dependencies, such that installing haxe will always install a compatible version of neko.
Having a custom install script for OpenFL is just going to make things worse: one more way of installing haxe/neko and we have to ask people what way they're used when they have trouble. And, OpenFL's script doesn't provide a way to uninstall/upgrade, which all Linux package managers support out of the box.

Is Haxe 3.2.1 available in all these channels?

Yes. Haxe 3.2.1 is available for all the distros listed in http://haxe.org/download/linux.

Have they been tested with libraries like OpenFL?

I've manually tested with the OpenFL DisplayingABitmap sample. Works as expected.
If there is anything that doesn't work with our linux packages, please report to us.

Is haxelib setup with user read/write permissions after install on these

As documented in http://haxe.org/download/linux, haxelib setup should be run manually after installation. We suggested using ~/haxelib, which is writable by the user by default. One can still use /usr/lib/haxe/lib, but shouldn't change its permission since it is indeed a system directory.

is it haxelib as a Neko binary, not as a Haxe interp?

It is a nekotools booted binary. There are a few issues with Haxe --interp, so we decided to move away from it.

Hi Andy,
I have tested your PPA for Ubuntu, and it appears to work properly. Prior to your ongoing effort to maintain current and stable Haxe releases for different *nix systems, either the official packages for Haxe and Neko were hopelessly out-of-date, or as mentioned previously, the packages were broken, such that using Neko native shared libraries would result in a crash.
I am prepared to recommend your Linux packages instead of our historic install script. You may not know our script started a very long time ago when there was no Haxe installer for Linux, and packages were non-existant or unreliable. We tested official packages as recently as Haxe 3.4.2 (I believe), but that was not using the PPA.
It was recently noted that there was a "passive aggressive" tussle between the Haxe Linux download page and our own. I would like to see if the **WARNING** message on the Haxe Linux download page could be removed? I would be happy to merge your pull request afterward.
Thanks again for all your work

@andyli Do you think it would be possible to remove that warning message from the Linux download page?

Do you think it would be possible to remove that warning message from the Linux download page?

Sure. As soon as OpenFL's installer script is removed, I will take care of removing the haxe.org download page warning.

Closing this PR since there is a conflict anyway. Feel free to ping me whenever the installer script is removed.


Bible Stories-not displaying for languages after Nagamese in the drop down

-Bible stories : In the language drop down, some languages that used to appear after Nagamese such as Urdu ,etc does not display .(high)

Bible stories do display for languages beyond Nagamese in the scroll bar


Documentation typo fixes

3 words were mistyped, likely due to autocomplete.  This is just a quick fix for those typos

Thanks!


(Some but not all countries) Program metrics different between country page and program page

This is a release blocker because it appear to be a global issue, affecting the accurate display of metrics.
On Dev2, I'm seeing a discrepancy between program metrics displayed on the Country page and the Program page. Here is an example with a program in Afghanistan.
Country page

Program page

Here's a program in Haiti
Country page

Program page

@camgnostic I poked around a bit more and it's not a global issue. For example, I checked a few programs in Afghanistan and Tolaland that were consistent.

QA - FAILED

Though I did find in the reported in this ticket (afghanistan) programs matched 100% between home/program page, I am still seeing a slight difference like this one in Haiti:

Update: this issue revolves around an edge case: we have an indicator in this program for which all targets are 0.  That means that while we can calculate a target (0) for the program to date, and an actual value (in this case, 14) for the program to date, we can't calculate the % over/under to determine if it's on track.  So it falls through the cracks, where it doesn't get counted as unavailable (it's got at least one completed target period and has results) and it doesn't get counted as over/under/on-target, so it doesn't get counted at all.  @jennyhavah is going to make the call on where to put these indicators on the graph, then I'll get this ticket closed out.

@camgnostic so whats the resolution on this, since this was moved back the dev2 environment has been updated and im still seeing this minor difference

@camgnostic  and I came to the resolution that in the very edgy edge case where the sum of periodic targets = 0, the indicator is treated as "unavailable."
As to why you're still seeing the discrepancy on Dev2, not sure. Cameron?

@jennyhavah @camgnostic Perhaps the test environment was not up to date as expected at the time of test, but checking this morning - it looks to be this has been resolved..

@camgnostic What's the best way for us to validate that this is fixed across all programs?
I know one way to reproduce the problem is to modify an indicator with results, so that the targets are all zero.
Is there something else we can check?

@jennyhavah the haiti program in production is having the same issue, as we do in dev2, just an fyi

For reference - there are three issues here:

Haiti Ann Viv Ansam program: if all your targets are zero the system was having trouble categorizing the indicator as "reporting" or "on target" or what.  We fixed that (it shows up in the "unavailable" category now)
Afghanistan program: Papaya-related soft-delete was breaking query logic for generating the home page data rollups, resulting in it counting a ton of deleted indicators when working out the percentage complete/targets-assigned/results-assigned/evidence-assigned values, so it was artificially deflating those numbers.  This is now fixed.
the 12/13 % differential.  Front end calculations were working out the number of indicators withOUT targets, backend was working out the number WITH targets, which usually results in the same percentage, but in the case of 2 indicators missing targets out of 16 indicators, you get 2/16 = 12.5, which rounds up to 13% (and then 100-13 means 87% with targets) or by the other logic you get 14/16 = 87.5 which rounds up to 88% (and then 100-88 means 12% without targets).  This is also fixed now.

QA - PASSED

Awesome, checked all the reported issues in this ticket, and all of the metrics matched.
Checked numerous other programs in other countries for continuity, and was not able to find anymore discrepancies.


vision-only input

Hi,
Thanks for your solid work! I am trying this single-modality support and seems that audio-only input works fine, as suggested by #77.
May I ask if vision-only inputs are supported for video-salmonn? I am encountering this shape error in #81. Details are as follows,
RuntimeError: Given normalized_shape=[2304], expected input with shape [*, 2304], but got input of size[1, 320, 768]

Any suggestion on how we should address this? Thanks.

Hi. Sorry for the confusion. We trained the system to have both inputs even when the question is only about the visual. Please just send one irrelevant or silent audio and ask questions about the image/video. This bug is to say that the audio part is missing since there are 2304-768=1536-dim missing.

Hi @BriansIDP
I have tried input of silent audio and find video-salmonn works well. Thanks for your kind reply.


Add rust_eh_personality to blacklist

Thanks! This is actually a bug in the Rust compiler rather than the linker implementaiton (which is what the other ones are here for). That being said I don't mind having this for now and once we fix the bug in the Rust compiler we can delete this!


In multi-line @param tag, part of type is parsed as documentation

(Reserving the issue number)

Not an issue any more; was in my branch but I made some changes.


Import from DnD Beyond

Hi there,
I have one more request.
Actually, your module not only works for the stat blocks from the srd document, but also for the DnD Beyond stat blocks, if you copy&paste them to your importer. A few manual adjustments need to be made though, and maybe you could implement them to your module.
That's how a stat block copied from DnD Beyond looks like:

Black Bear
Medium beast , unaligned
Armor Class 11 (Natural Armor)
Hit Points 19 (3d8 + 6)
Speed 40 ft., climb 30 ft.
STR
15 (+2)
DEX
10 (+0)
CON
14 (+2)
INT
2 (-4)
WIS
12 (+1)
CHA
7 (-2)
Skills Perception +3
Senses Passive Perception 13
Languages --
Challenge 1/2 (100 XP)
Proficiency Bonus +2
Keen Smell. The bear has advantage on Wisdom (Perception) checks that rely on smell.
Actions
Multiattack. The bear makes two attacks: one with its bite and one with its claws.
Bite. Melee Weapon Attack: +4 to hit, reach 5 ft., one target. Hit: 5 (1d6 + 2) piercing damage.
Claws. Melee Weapon Attack: +4 to hit, reach 5 ft., one target. Hit: 7 (2d4 + 2) slashing damage.

Manual adjustments that currently need to be made:

Delete the line with the Proficiency bonus
Delete the blank lines

Fixed with v1.1.1.


AtomicCell is unsound on types containing niches

I believe it's incorrect to implement AtomicCell<T> in terms of UnsafeCell<T>, and it needs to be UnsafeCell<MaybeUninit<T>> instead, to prevent code outside the cell from observing partially initialized state.
Here is an example of safe code that reproduces UB:
use crossbeam_utils::atomic::AtomicCell;
use std::num::NonZeroU128;
use std::thread;

enum Enum {
    NeverConstructed,
    Cell(AtomicCell<NonZeroU128>),
}

static STATIC: Enum = Enum::Cell(AtomicCell::new(match NonZeroU128::new(1) {
    Some(nonzero) => nonzero,
    None => unreachable!(),
}));

fn main() {
    thread::spawn(|| {
        let cell = match &STATIC {
            Enum::NeverConstructed => unreachable!(),
            Enum::Cell(cell) => cell,
        };
        let x = NonZeroU128::new(0xFFFFFFFF_FFFFFFFF_00000000_00000000).unwrap();
        let y = NonZeroU128::new(0x00000000_00000000_FFFFFFFF_FFFFFFFF).unwrap();
        loop {
            cell.store(x);
            cell.store(y);
        }
    });

    loop {
        if let Enum::NeverConstructed = STATIC {
            unreachable!(":(");
        }
    }
}

$ cargo run

warning: variant is never constructed: `NeverConstructed`
 --> src/main.rs:6:5
  |
6 |     NeverConstructed,
  |     ^^^^^^^^^^^^^^^^
  |
  = note: `#[warn(dead_code)]` on by default

warning: `repro` (bin "repro") generated 1 warning
    Finished dev [unoptimized + debuginfo] target(s) in 0.27s
     Running `target/debug/repro`

thread 'main' panicked at 'internal error: entered unreachable code: :(', src/main.rs:31:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Thanks for the report! I've filed #834 to fix this.

Honestly, the fact the we can do a non-atomic load on STATIC in the main thread is imo the problem here since this causes UB (due to interleaved atomic/non-atomic access). I think this happens because AtomicCell is repr(transparent) which allows the compiler to use the niche of the inner value. I don’t think we should do that… In theory removing repr(transparent) would have fixed this since accessing the discriminant of the enum doesn’t touch the AtomicCell.

Since https://github.com/rust-lang/rust/pull/99011 (a few months after this issue), UnsafeCell<T>  suppresses niches, exactly due to problems like this. So I don't think there's any need for crossbeam to do anything itself here any more.


Rename num_xpu to num_xpus

Description of changes:
Rename num_xpu to num_xpus
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #58 into r2.0 will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             r2.0      #58   +/-   ##
=======================================
  Coverage   98.51%   98.51%
=======================================
  Files          14       14
  Lines         470      470
  Branches       27       27
=======================================
  Hits          463      463
  Misses          4        4
  Partials        3        3

Impacted Files
Coverage Δ

src/sagemaker_containers/env.py
99.44% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Web Interface Error

Describe the bug
When I try to run from the web interface it says 'NoneType' object is not subscriptable.
To Reproduce
Steps to reproduce the behavior:

Access the Web interface
Select all Containers, All Networks and All Tests
See error

Expected behavior
Tests to run I guess and some sort of feedback provided
Screenshots

Desktop (please complete the following information):

OS: Ubuntu Server 20.04
Browser: Brave and Firefox

@The-Inamati  This issue has been fixed in the latest REL, can you specify the version of DockerENT you are using.

I cloned the repo the same day I posted. The current version is v0.1.15

Just run pip install DockerENT --upgrader once to check if everything is
okay. I am running it, and it is running fine.
On Mon, Sep 28, 2020 at 2:34 PM Inamati<EMAIL_ADDRESS>wrote:

I cloned the repo the same day I posted. The current version is v0.1.15
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/r0hi7/DockerENT/issues/35#issuecomment-699879743, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFBMCRX5WC3J5G4OGGJVRPDSIBGQFANCNFSM4RZN2F3Q
.

Closing this issue, please open if you feel this issue still exit.


[jointJacobian] different behavior between C++ and python

Python method jointJacobian DOES NOT call its corresponding C++ function. Instead, it internally calls computeJointJacobians followed by getJointJacobian. This causes three differences in the behavior:

first, the unimportant, obvious ones: at the end of the python method, the overall Jacobian is computed, while at the end of the C++ method it is not. Also, the python method defeats the purpose of being more efficient, but who cares about efficiency in python
second: the python method allows to specify a Reference frame (LOCAL/WORLD), while the C++ method does not and only offers the local version.
third: computeJointJacobians internally does the equivalent of forwardKinematics, which C++ jointJacobian does not. This is particularly bad, because the doxygen C++ comment to computeJointJacobians(model, data, q) explicitly states:

This function computes also the forwardKinematics of the model.

while the one for jointJacobian says:

This function is equivalent to call first computeJacobians(model,data,q) and then call getJointJacobian<LOCAL>(model,data,jointId,J).

but this is not true. If you think this is not relevant, well, it was causing problems in one example Nicolas prepared in PR #541.
Should this be fixed? In which way? Or, should we leave things as they are in order not to break everybody's code, but explicitly document the difference in behavior?

Update: actually, python's jointJacobian behavior is still different than that: it takes a boolean update_kinematics parameter (for which no default value is provided) and if this is true, then computeJointJacobians, otherwise not, and  and then getJointJacobian is called. This is explicitly documented in the python docstring which says (true = update the value of the total jacobian).
So, basically, the C++ and the python version are two different methods, which were thought to operate and be employed differently from each other. The python version is a convenience method which allows to follow the usual computeJointJacobians + one ore more getJointJacobian procedure while skipping the explicit call to computeJointJacobians, while the C++ version is a truly different algorithm which is made for one-off use.
I don't know, should I make the python behavior identical to the C++ one and maybe add an option to C++ to include kinematics computation, or leave things as they are and just be really clear in the documentation about the different behavior?

I don't have time to investigate further, but the main idea is that Python follows C++, not the reverse.

But, I would suggest to let open this issue and to solve it after the release.

The main 2 functions should be computeJointJacobians (compute data.J from root to leaves) and getJointJacobian (get some columns of data.J). These 2 should be bind in Python.
The secondary function should be jointJacobian --possibly with another name-- that compute a single jacobian from leaf to root. It is more user friendly. Two solutions might be proposed to implement it: either compute the jacobian in data.J and return a reference to it (rnea style) ; or let the user allocates its own matrix J and compute in it. The first solution is bad because data.J should be used for computeJointJacobians ; the second is bad because it is asking the user to allocate memory, which is not friendly and not following the style of Pinocchio. I do not have a good solution.
Then Python API should follow C++ API.

I would not say asking the user to allocate memory is not following the style of Pinocchio: that's exactly what getJointJacobian does.
Writing the result in data.J (which corresponds to the behavior of a deprecated method) is particularly bad, since data.J would represent different things depending on what methods are called. I'm not only talking about the overall Jacobian vs specific Jacobians, but also of specific Jacobians vs each other. Consider what would happen in the following (buggy) case:
se3::Data::Matrix6x & J1 = se3::jointJacobian(model, data, q, jointId_1)
se3::Data::Matrix6x & J2 = se3::jointJacobian(model, data, q, jointId_2)

Ok, you should not call jointJacobian twice, and instead call computeJointJacobians first and then getJointJacobian twice, but choosing to write into data.J seems to encourage particularly error-prone code. The solution
se3::Data::Matrix6x J1 = se3::jointJacobian(model, data, q, jointId_1)
se3::Data::Matrix6x J2 = se3::jointJacobian(model, data, q, jointId_2)

is not good, because it will amount to a copy into J1 and J2, i.e. a hidden allocation.
I suggest to leave the C++ method as it is. Only, in the doxygen comment, it should be made clear that this function is NOT equivalent to  computeJointJacobians + getJointJacobian, and in particular forwardKinematics is NOT called.
Then the python method would be made identical to C++, with the only difference that the output matrix, instead of being given, would be created, set to zero and returned (I think this is acceptable in python). Its signature would be jointJacobian(model,data,q,JOINT_ID). Therefore, the python function will lose the capability to return the Jacobian in the world frame. Also, it will then become necessary to explicitly call forwardKinematics if data.oMi is to be read, as it is in C++.
Usage of the python function as se3.jointJacobian(model,data,q,JOINT_ID,ReferenceFrame,update_kinematics) should be kept for compatibility for a while, keeping the same behavior, but be deprecated. It will require some tweaking to deprecate the function only when it is called in a certain way but not in others, but it can certainly be done. An alternative is to rename jointJacobian (the current version in C++ and the new one in python) to something else, like computeJointJacobian.
If we want to wait for version 2.0 in order to implement these changes but also keep working on the doc in #541, then I suggest to employ the computeJointJacobians + getJointJacobian paradigm in the python examples too.

The point is that algo in Pinocchio should not return their results but should take as input their returns.
For instance, jointJacobianor computeJointJacobianshould take at the last argument const Eigen::MatrixBase<Matrix6Like> & Jout.

This makes the following approximate code:
MatrixXd J;
jointJacobian(model,data,q,J);
Sorry, but I prefer:
J = jointJacobian(model,data,q);
especially in Python. Let's say my taste is not relevant.

I also prefer the second version.
In python, it is highly unconventional to update one of the input of a function, return nothing, and expect the user to understand what happened.

But to be efficient, the first approach is expected.
New algorithms in Pinocchio have been developed following this paradigm.
For Python, you may have a different API for the return type, no matter the solution in C++.

I agree with @jcarpent. In C++ the users gives the memory, but in python the matrix is returned. It is just a different way of returning the matrix.
As a compromise, we might have the C++ method zero out the provided matrix and return a reference to the same matrix which was passed as input. This way we could write
MatrixXd J(6,model.nv);
doStuff(jointJacobian(model,data,q,J));

which is more compact than
MatrixXd J(6,model.nv);
J.setZero();
jointJacobian(model,data,q,J);
doStuff(J);

How many other algorithms in Pinocchio have the same behavior? I guess all other derivative algorithms?
As Jacobian is a very standard algo, to be used by newcommers, what about adding another matrix field in data to avoid the preallocation of J by the user in the "more compact" form proposed by @gabrielebndn ?

@nmansard I advise against it. At first I thought it could be a solution, but then I realise it has its own problems, because the same matrix would potentially represent different joint Jacobians and this in not good either. Again, consider the problems potential users could incur to if they did:
se3::Data::Matrix6x & J1 = se3::jointJacobian(model, data, q, jointId_1);
se3::Data::Matrix6x & J2 = se3::jointJacobian(model, data, q, jointId_2);

The only clean way to do so would be to add a whole new vector field containing one Jacobian for each joint, which would be a total waste of space (we should do the same for frameJacobian too).
My compromise suggestion was different: basically, the matrix would still be provided by the user, only pinocchio would set it to zero and return a reference to the same matrix which was passed as input, so to be able to use the expression in nice one-liners

I agree with @gabrielebndn, the whole idea of Jacobian is initially to not allocate memory for each joint.
We cannot revert it because it is the correct things should go.
Adding new data.Jjoint would be inefficient in terms of memory footprints for instance when we need to have all the data in the L1 cache for instance.
The correct way to handle the sparsity through MotionSet. it is also what I suggested in https://github.com/stack-of-tasks/pinocchio/projects/2

The same behavior of jointJacobian is followed by all other derivative algorithms, plus a couple of recent convenience methods called getFrameVelocity and getFrameAcceleration (but I think we could easily switch to a more user-friendly behaviour for those two, I will investigate this further).

An easy solution is to rely on computeJointJacobians and getJointJacobian.

@jcarpent that would not address Nicolas' comments, since getJointJacobian also asks the user to allocate for J. Indeed, I think that whatever we choose for jointJacobian w.r.t. the return policy, we should make the same in getJointJacobian, frameJacobian, and getFrameJacobian.
If you were referring to PR#540, then I agree, I think this discussion is blocking the progress, and to avoid any inconsistency we should use computeJointJacobians and getJointJacobian in the examples, until we have made a decision on this.
(BTW, frameJacobian is not provided in C++, it is only provided as a Python binding, where it is implemented in the same way as the one for jointJacobian. We should fix this, it's on the to-do list of the python bindings project. But we should only fix this after the decision is final on whether to modify jointJacobian and/or rename it).

No, when you ask the user, you only require that the user allocates just needed memory for his stuff, not for all the joints.

@jcarpent I am not following you. Right now, jointJacobian and getJointJacobian both ask the user to allocate a single J, where the result will be put. They are identical w.r.t. that aspect, as for now.

If the user calls several times jointJacobian for the same joint, then you will have at each call memory allocation. While this is not true when first allocating, then calling getJointJacobian

What? I do no think this is the case right now. The following two pieces of code should make the same number of memory allocations:
se3::Data::Matrix6x J(6,model.nv); J.fill(0);
for (int i=0 ; i<100 ; ++i) {
      se3::computeJointJacobians(model,data,q);
      se3::getJointJacobian<se3::LOCAL>(model,data,JOINT_ID,J);
}

se3::Data::Matrix6x J(6,model.nv); J.fill(0);
for (int i=0 ; i<100 ; ++i) {
      se3::jointJacobian(model,data,q,JOINT_ID,J);
}

Yes, but not the one suggested 'J = se3::jointJacobian(model,data,q,JOINT_ID)'

Exactly. That's why I think we should not do that.

@gabrielebndn Ping. What is the status of this issue?

We haven't talked about it, but it's still in my mental list of TODOs. I am reluctant to take action because I feel any change might potentially change some behavior and I feel any decision on this should be shared. If you agree, I will leave C++ untouched and I will change Python's jointJacobian to reflect C++ . Then I will probably take similar measures with the frameJacobian

I do agree. You can then move forward.

Will do

Solved by #698 .


About the case that v-model and checked can not be used together in IE

In the document, v-model ignores the checked property, but IE does not ignore it and outputs an error.

v-model will ignore the initial value, checked or selected attributes found on any form elements.
Form Input Bindings

v-model generates a checked property in domProps.
Therefore, IE outputs errors because it is a specification that does not allow duplicate properties.
I think that it is better to have the following notes in the document.
When using v-model, do not use the checked property. It is an error in IE.
This seems to be the same as the selected property.

Can you provide a reproduction of the issue?

I created a sample repository.
https://github.com/tarotaro080808/checked-polyfill
Actually, running this project causes an error in IE.
And, you can confirm that it works by deleting the checked property.
The location of the checked property is below.
https://github.com/tarotaro080808/checked-polyfill/blob/master/src/App.vue#L9
This problem seems to depend on @babel/polyfill.
I do not know if this is a problem to be addressed in the document on the Vue.js side.
Thank you.


Setting runtime EDID on MS2109

I decided to create this issue to keep from spamming #10.
The only place I was able to find the EDID was in the CODE region. RAM and EEPROM did not seem to contain the EDID.
The following returns the edid on an MS2109.
$ sudo ./cli read CODE 0x7738 256

Trying to write an EDID to it however returns the following:
$ sudo ./cli write-file CODE 0x7738 /tmp/edid.bin
cli: error: Memory can't be written

I assume this is because it's the CODE region and cannot be written to.
Am I just not looking at the RAM region the right way to find the EDID in it?

Got your email. I'll test it tomorrow.

It did work! Thank you! It took some messing around with to get it working. I finally took the edid from your edid_change.bin and slightly modified it and it took.
Since this exists in CODE I'm assuming it is not possible to change the edid without flashing the EEPROM and resetting the device?

Hello,
No, it is possible to do it at runtime as well, but I wanted to try a
simple method first. I will try to document it some time in the future.
Sincerely,
Bertold
Op wo 25 okt. 2023 22:14 schreef Ayrton Sparling @.***>:

It did work! Thank you! It took some messing around with to get it
working. I finally took the edid from your edid_change.bin and slightly
modified it and it took.
Since this exists in CODE I'm assuming it is not possible to change the
edid without flashing the EEPROM and resetting the device?
—
Reply to this email directly, view it on GitHub
https://github.com/BertoldVdb/ms-tools/issues/13#issuecomment-1779986210,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQILGOYBB2AOSW2ZNBYGATYBFXL5AVCNFSM6AAAAAA5ALSIP2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZZHE4DMMRRGA
.
You are receiving this because you commented.Message ID:
@.***>

Awesome, thank you for taking the time to work on this Bertold!

您不能像在 ROM 中那样编辑 CODE 区域，我会通过电子邮件发送给您一些尝试。

Send me a thank you<EMAIL_ADDRESS>


Possible incorrect dependencies on conda-forge for tsflex 0.3.0

Looking at the dependencies for tsflex 0.3.0 on conda-forge vs what is in the pyproject.toml file, it looks like the dependencies for the conda package are likely incorrect (in particular I'm noticing the fastparquet dependency still in conda-forge, whereas it is no longer present for tsflex 0.3.0 pyproject.toml).

I faced the same issue, I got around it by specifying tsflex as a pip dependency in the conda env file and installed it using mamba.


GH-X: [JS] Remove WideValue

I think we can use bigint now.

We actually need this because decimals are stored as typed arrays.


Fixing FN:  attachment_html_recipient_in_javascript_identifiers.yml

Fixing issue where the rule would fail to flag if the recipient and the javascript terms were found in separate exploded pieces of the same file.

Mimic showing multiple pickups and no regressions. Many of these were only firing "Any HTML" rules.
New Flag. EML 24000
New Flag. EML 22757
New Flag. EML 33987
New Flag. EML 56868
New Flag. EML 3845
New Flag. EML 20089
New Flag. EML 6435
New Flag. EML 5239
New Flag. EML 20088
New Flag. EML 22872
New Flag. EML 20087
New Flag. EML 23996
New Flag. EML 34067


Support Pipelines in Pipelines [TEP-0056]

Feature request
We need to add support for TEP-0056 - Pipelines in Pipelines.
Use case
User should be able list all pipelineruns which are embedded as part of PiP.

/assign @khrm

Refer pipeline#7055
and https://github.com/tektoncd/community/blob/main/teps/0056-pipelines-in-pipelines.md


docs(codepush): first pass updating hotfix docs for codepush

This PR resolves PHIRE-403
Description

Updates the hot fix docs to account for new codepush process.
We will review together and try a dry run in mobile office hours before merging.
PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Add docs for codepush hot fix process - Brian

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (Add docs for codepush hot fix process - Brian)
undefined (changes)

  Generated by :no_entry_sign: dangerJS against 61e43db56a54ca6df0aec0151c81c005245022b4


Segfault when parsing a List (Str, Dec)

I'm trying to add the rest-api exercise to the roc track on exercism. It involves parsing a JSON object containing a Dict, but unfortunately this is blocked by #33 . I thought I could work around this by replacing the Dict Str Dec with a List (Str, Dec). Unfortunately, I'm now running into a segfault.
Here's a code example to reproduce the issue (I'm using roc nightly pre-release, built from commit 54cd967 on Fri Sep  6 09:02:04 UTC 2024):
app [main] {
    pf: platform "https://github.com/roc-lang/basic-cli/releases/download/0.15.0/SlwdbJ-3GR7uBWQo6zlmYWNYOxnvo8r6YABXD-45UOw.tar.br",
    json: "https://github.com/lukewilliamboswell/roc-json/releases/download/0.10.2/FH4N0Sw-JSFXJfG3j54VEDPtXOoN-6I9v_IA8S18IGk.tar.br",
}

import json.Json

main =
    Task.ok {}

ListOfTuples: List (Str, Dec)

expect
    jsonStr = "[[\"Alice\", 1.1], [\"Bob\", 2.2], [\"Charles\", 3.3]]"
    result : Result ListOfTuples _
    result = jsonStr |> Str.toUtf8 |> Decode.fromBytes Json.utf8
    result == Ok [("Alice", 1.1), ("Bob", 2.2), ("Charles", 3.3)]

Thanks for reporting this @ageron!
I've converted this to an app to facilitate investigation:
app [main] {
    pf: platform "https://github.com/roc-lang/basic-cli/releases/download/0.15.0/SlwdbJ-3GR7uBWQo6zlmYWNYOxnvo8r6YABXD-45UOw.tar.br",
    json: "https://github.com/lukewilliamboswell/roc-json/releases/download/0.10.2/FH4N0Sw-JSFXJfG3j54VEDPtXOoN-6I9v_IA8S18IGk.tar.br"
}

import pf.Stdout
import json.Json

ListOfTuples: List (Str, Dec)

jsonStr = "[[\"Alice\", 1.1], [\"Bob\", 2.2], [\"Charles\", 3.3]]"

result : Result ListOfTuples _
result = jsonStr |> Str.toUtf8 |> Decode.fromBytes Json.utf8

main =
    Stdout.line! (Inspect.toStr result)

Valgrind output with the legacy linker:
❯ valgrind ./examples/helloWorld
==21969== Memcheck, a memory error detector
==21969== Copyright (C) 2002-2022, and GNU GPL'd, by Julian Seward et al.
==21969== Using Valgrind-3.22.0 and LibVEX; rerun with -h for copyright info
==21969== Command: ./examples/helloWorld
==21969==
==21969== Invalid read of size 8
==21969==    at 0x147E33: ??? (roc_app:0)
==21969==    by 0x148F70: ??? (roc_app:0)
==21969==    by 0x144B8F: List_walkHelp_dbefccae6de790f8e3497ad3c6c1c58a12a744edb0d65ec1ec4ade8b1151a59b (roc_app:0)
==21969==    by 0x13AB00: List_walk_52aff1341cf42f5e6559a2cf028663f7bbbc7576ac1948fc58784a0613b79 (roc_app:0)
==21969==    by 0x12FEAA: Inspect_158_d9e2d7d318b97522751e8d862a897cd552d26040fa25d41678f9ac7b36cd7 (roc_app:0)
==21969==    by 0x145C00: Inspect_156_35d6cb78c74f84df82ab12c37bc71bbe5193f93e9d36506abc7c9c8ca124d1ab (roc_app:0)
==21969==    by 0x136BB3: Inspect_apply_5eb6a2599d3097c754d93922b84522fd22c626afbfca9a48d724fa1945e3ca9 (roc_app:0)
==21969==    by 0x12FDD0: #Derived_custom8_52459ae5e05017996bb4298dd9ac3944ffe997fa2e2ad98ba6fd7348395f63 (roc_app:0)
==21969==    by 0x13CB5E: Inspect_apply_8e7a40fb7cb2175e9c8b7aee60f44cef84959b742d3a14c483b6e3b14f05c2f (roc_app:0)
==21969==    by 0x13458D: Inspect_213_3994ebd10847f51a1ba443e4f3b9fb75da3f81a354da59de9bd34aaa2e927d (roc_app:0)
==21969==    by 0x139E8A: Inspect_210_a51293a4c3ce80beb92fd22c82b6b69bd26ee8bc815b483e3cf291f486236c (roc_app:0)
==21969==    by 0x1399EC: List_walkHelp_1971ed175c5339d8a493ee2a719f3ca8f50fbcc2a26feaf7b54a27898e3f (roc_app:0)
==21969==  Address 0xfffffffffffffff8 is not stack'd, malloc'd or (recently) free'd
==21969==
==21969==
==21969== Process terminating with default action of signal 11 (SIGSEGV): dumping core
==21969==  Access not within mapped region at address 0xFFFFFFFFFFFFFFF8
==21969==    at 0x147E33: ??? (roc_app:0)
==21969==    by 0x148F70: ??? (roc_app:0)
==21969==    by 0x144B8F: List_walkHelp_dbefccae6de790f8e3497ad3c6c1c58a12a744edb0d65ec1ec4ade8b1151a59b (roc_app:0)
==21969==    by 0x13AB00: List_walk_52aff1341cf42f5e6559a2cf028663f7bbbc7576ac1948fc58784a0613b79 (roc_app:0)
==21969==    by 0x12FEAA: Inspect_158_d9e2d7d318b97522751e8d862a897cd552d26040fa25d41678f9ac7b36cd7 (roc_app:0)
==21969==    by 0x145C00: Inspect_156_35d6cb78c74f84df82ab12c37bc71bbe5193f93e9d36506abc7c9c8ca124d1ab (roc_app:0)
==21969==    by 0x136BB3: Inspect_apply_5eb6a2599d3097c754d93922b84522fd22c626afbfca9a48d724fa1945e3ca9 (roc_app:0)
==21969==    by 0x12FDD0: #Derived_custom8_52459ae5e05017996bb4298dd9ac3944ffe997fa2e2ad98ba6fd7348395f63 (roc_app:0)
==21969==    by 0x13CB5E: Inspect_apply_8e7a40fb7cb2175e9c8b7aee60f44cef84959b742d3a14c483b6e3b14f05c2f (roc_app:0)
==21969==    by 0x13458D: Inspect_213_3994ebd10847f51a1ba443e4f3b9fb75da3f81a354da59de9bd34aaa2e927d (roc_app:0)
==21969==    by 0x139E8A: Inspect_210_a51293a4c3ce80beb92fd22c82b6b69bd26ee8bc815b483e3cf291f486236c (roc_app:0)
==21969==    by 0x1399EC: List_walkHelp_1971ed175c5339d8a493ee2a719f3ca8f50fbcc2a26feaf7b54a27898e3f (roc_app:0)
==21969==  If you believe this happened as a result of a stack
==21969==  overflow in your program's main thread (unlikely but
==21969==  possible), you can try to increase the size of the
==21969==  main thread stack using the --main-stacksize= flag.
==21969==  The main thread stack size used in this run was 8388608.
==21969==
==21969== HEAP SUMMARY:
==21969==     in use at exit: 504 bytes in 6 blocks
==21969==   total heap usage: 12 allocs, 6 frees, 1,008 bytes allocated
==21969==
==21969== LEAK SUMMARY:
==21969==    definitely lost: 0 bytes in 0 blocks
==21969==    indirectly lost: 0 bytes in 0 blocks
==21969==      possibly lost: 480 bytes in 5 blocks
==21969==    still reachable: 24 bytes in 1 blocks
==21969==         suppressed: 0 bytes in 0 blocks
==21969== Rerun with --leak-check=full to see details of leaked memory
==21969==
==21969== For lists of detected and suppressed errors, rerun with: -s
==21969== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)
Segmentation fault (core dumped)

This valgrind output is similar to parts of https://github.com/roc-lang/roc/issues/6813, these issues could have the same cause.


Buttons for Redo and Undo are absent in the desktop version, looks like they smashed into the "Reset Zoom" button

When using Excalidraw in desktop browser, in the lower-left corner we can see three buttons: the "+" and "-" and one button that when clicked will reset zoom to 100%, but on it you will see the "anticlockwise arrow" icon which is usually for the "undo" action I believe. In fact, I can see no button for "undo" or "redo" unlike in the mobile version (we still can use Ctrl+Z, Ctrl+Y). So I am wondering if it is deliberate not to have these two buttons on desktop version, or did they become hidden/collapsed into the reset zoom button.

No, we don't expose undo/redo on purpose because the shortcuts for these are pretty well known, and we try to ensure the UI isn't unnecessarily and overly complex to navigate.

No, we don't expose undo/redo on purpose because the shortcuts for these are pretty well known, and we try to ensure the UI isn't unnecessarily and overly complex to navigate.

I've opened a PR to add ctrl-Y to redo which was missing. https://github.com/excalidraw/excalidraw/pull/2831

I've opened a PR to add ctrl-Y to redo which was missing. https://github.com/excalidraw/excalidraw/pull/2831

Btw, if you come up with a better icon for zoom-reset, we're open to PRs.

Btw, if you come up with a better icon for zoom-reset, we're open to PRs.

When using a tablet, there is no way to undo because there is no keyboard to perform CTRL+Z


Possibility to always return all fields after query execution

Is it possible to always return all fields after query execution ('*') instead of using .returning('*') if database is postgres ?
Maybe a model property can be added and set to true (false by default) ?
It's useful when using GraphQL as it typically returns full types.

It's not a good idea for objection, but you can easily implement this yourself. The easiest way to do it is this:
class BaseModel extends Model {
  query(...args) {
    const query = super.query(...args);

    if (!query.isFindQuery()) {
      query.returning('*');
    }

    return query;
  }
}

isFindQuery is not documented yet, but will be in the next version.

@koskimas maybe there are edge cases when it can cause troubles?

Not that I can think of. Performance should't be an issue. The insert/update itself will surely dominate the runtime.


Update LivelinkConnector.java

Hi Karl Wright,
There was a bug in this file since version 1.8, when we try to add a Livelink folder in Path tab, it was breaking and today I found that this was because, you missed seqPrefix+ in line no 3265, Please consider changing this in the next release.
Now since, I became more familiar with ManifoldCF architecture, I may be try to find more bug in livelink connector here on.
Thanks for such a great tool to index.

Hi!
They dont use github. You might make a issue and put uor patch at Jira.
Here it is.
https://issues.apache.org/jira/browse/CONNECTORS/

s/uor/your/g

On Feb 8, 2015, at 12:43 AM, Kishore Kumar<EMAIL_ADDRESS>wrote:
Hi @kwright
There was a bug in this file since version 1.8, when we try to add a Livelink folder in Path tab, it was breaking and today I found that this was because, you missed seqPrefix+ in line no 3265, Please consider changing this in the next release.
Now since, I became more familiar with ManifoldCF architecture, I may be try to find more bug in livelink connector here on.
Thanks for such a great tool to index.
You can view, comment on, or merge this pull request online at:
https://github.com/apache/manifoldcf/pull/3
Commit Summary
• Update LivelinkConnector.java
File Changes
• M connectors/livelink/connector/src/main/java/org/apache/manifoldcf/crawler/connectors/livelink/LivelinkConnector.java (2)
Patch Links:
• https://github.com/apache/manifoldcf/pull/3.patch
• https://github.com/apache/manifoldcf/pull/3.diff
—
Reply to this email directly or view it on GitHub.

Hello Kishar,
I looked at the link and the code was written by Karl Wright and not me. Sorry I couldn't help you.
--Kevin

Thanks @yoshi0309
I submitted a issue with JIRA.

@kwright
No problem, I submitted via JIRA and was accepted by ManifodCF Team.


Moves global CSS back into base.css

We were loading global CSS through glamor, but the insert of it is failing on the client. We were able to get it to work, but it would require mucking up the server files. At the end of this thing, there will be very little global CSS so we can just embed it into a style tag within the head. For now, this commit just adds those global styles back into the base.css file.
Finishes: #141335023

          1 Message

:book:
Nice, more deletions than insertions :red_circle:

  Generated by :no_entry_sign: danger


Fix github ci clippy tests on nonminimal bool test

The symmetrical test here fails a nonminimal bool test, but it looks fine to skip in this case

@nyxtom Just needs a rebase


Troubleshooting

@NekR
As discussed in #209, here is an initial list of things to troubleshoot. It's based on what users seems to have problems with by going thru recent issues.
It was a bit hard to format it in a decent way. Please provide any feedback of things you want changed / removed / added and I will fix it accordingly.

Hey! I was a bit busy yesterday and kind of today. I'll try to review it later today. Thanks :-)

All looks good to me, thank you @GGAlanSmithee! Very great work 💯 Merging!

Thanks @NekR


Resolve inconsistency with import paths between development and production environments

We are encountering an issue with module import paths that behave differently in our development and production environments. This inconsistency affects the reliability of our deployment process and could potentially lead to runtime errors.
Issue Details:
When using absolute import statements such as:
from backend.batch.utilities.helpers.EnvHelper import EnvHelper
the application functions correctly in the local development environment. However, in the production environment, the same path causes import errors, and the system requires a relative import path, like:
from ..helpers.EnvHelper import EnvHelper
Tasks:

[ ] Determine why absolute paths fail in the production environment but work in development.
[ ] Implement a solution that supports both absolute and relative import paths in all environments, thereby standardizing our codebase and deployment process.

As per the discussion with @adamdougal , we will continue to investigate on this issue and will see if any solution arise in the future that supports both absolute and relative import paths in all environments, will keep this issue open and update in case if any solutions found. FYI @Roopan-Microsoft


refactor: typescript code

This PR will add code for TypeScript.
I couldn't make a small PR as it was giving errors. That's why PR is a little long.
Please review this PR and give your feedback.

Code is Clean Everything looks Good

Thankyou for reviewing.


Improve Security Manager module

Currently SecurityManager and KeyManager use 2 databases for storing keys
we need to review this implementation and possible use only one database for storing keys.
Also we have many KeyManager classes:
io.subutai.common.security.crypto.key.KeyManager
io.subutai.core.security.impl.crypto.KeyManagerImpl
io.subutai.core.security.api.crypto.KeyManager
we need to leave only one
Also remove duplicate methods

check all places where it can be called and update clients if needed

Duplicate Key-Management REST services removed.
PGP KeyPairs use , separate db tables. This was implemented to secure , Secret (private) key pair.


chore(docs): supplement errorMessage behaviour in input

Closes
📝 Description

Add a brief description

⛳ Current behavior (updates)

Please describe the current behavior that you are modifying

🚀 New behavior

Please describe the behavior or changes this PR adds

💣 Is this a breaking change (Yes/No):

📝 Additional Information

Closing - mixed up with the branch name


Fix null pointer when getting primary clip text

Task/Issue URL: https://app.asana.com/0/1202552961248957/1208362038343242/f
Description
Tries to fix NullPointer when accessing primaryClip text
Steps to test this PR
Feature 1

[ ] Smoke test the feature
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

#5077  👈
develop

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

Join @cmonfortep and the rest of your teammates on  Graphite


Deploy new terraform version to prd

Summary | Résumé
Reseting the scan file timeout alarm to initial value after applied f…
…ix (https://github.com/cds-snc/notification-terraform/pull/760)
Fixing dkim validation DNS records to use proper domains (https://github.com/cds-snc/notification-terraform/pull/763)
Everything up to lambdas building (https://github.com/cds-snc/notification-terraform/pull/762)
* Everything up to lambdas building
Adding back manual TXT records for legacy domain validation. (https://github.com/cds-snc/notification-terraform/pull/765)
fix: update s3_scan_object to be async (https://github.com/cds-snc/notification-terraform/pull/766)
Update the module to the latest version which sends
Lambda merge (https://github.com/cds-snc/notification-terraform/pull/764)
* Everything up to lambdas building
fix: update waf rules for new GCA routes (https://github.com/cds-snc/notification-terraform/pull/767)

this was merged in another PR


Getting error 'uninitialized constant PaperTrail::Version'

On Rails 3.0.3 with PaperTrail 4.0.0.rc.
Ran the following commands and was greeted with the error
bundle
bundle exec rails generate paper_trail:install
bundle exec rake db:migrate
rails c

The generator created one migration file create  db/migrate/20150514161738_create_versions.rb

Do you have a custom initializer for PaperTrail in your config/initializers directory?

Thanks for the quick reply!  I do not.  I tried creating one and placing the copying lib/paper_trail/frameworks/active_record/models/paper_trail/version.rb  into the file.  That changed my error to uninitialized constant PaperTrail::VersionAssociation (NameError).

Can you provide the full stack trace please?
rails c --trace

Sure thing!
https://gist.github.com/alexclarkofficial/7184d199ddedf05fab3f
We were actually able to get the app running with the following initializer:
class PaperTrail::VersionAssociation < ActiveRecord::Base
  include PaperTrail::VersionAssociationConcern
end

class PaperTrail::Version < ActiveRecord::Base
  include PaperTrail::VersionConcern
end

What happens if you change the initializer to this?:
require 'paper_trail/frameworks/active_record'

I'm wondering if the Rails::Engine eager loading doesn't work the same way on Rails 3.0.x as it does on 3.1+...
Also curious if you could give this initializer a whirl to see what happens?
PaperTrail::Rails::Engine.eager_load!

lastly, could you try booting up the console and seeing what returns from this command?
p  PaperTrail::Rails::Engine.paths['app/models'].instance_variable_get(:@paths)

Thanks for working with me. I'm wondering if this class isn't performing how it normally does with that version of Rails for some reason.

Not a problem.  The first initializer gave this error when I started the console:
.rvm/gems/ruby-1.9.3-p551/gems/paper_trail-4.0.0.rc1/lib/paper_trail/config.rb:33:in `track_associations': uninitialized constant PaperTrail::VersionAssociation (NameError)

The second gave this:
config/initializers/paper_trail.rb:1:in `<top (required)>': undefined method `eager_load!' for PaperTrail::Rails::Engine:Class (NoMethodError)

And lastly here is the result for the console command(after using the initializer I sent over:
1.9.3-p551 :001 > PaperTrail::Rails::Engine.paths['app/models'].instance_variable_get(:@paths)
 => ["app/models"]

Interesting, seems like the models directory is not being loaded in properly in that case. I'm wondering if it's a Rails version number. Will push a commit up soon trying to address this.

Will push a commit up soon trying to address this.

Ben, did that commit happen?  Were you able to reproduce the issue?

#554, which was merged, should fix the issue encountered that raised
uninitialized constant PaperTrail::VersionAssociation (NameError)
I'm still not certain what was causing the original issue. I need to setup a dummy app with an older version of Rails (3.0.x) to try to see if I can reproduce it.

Alex, I can reproduce this issue with rails 3.0.3 and paper_trail 4.0.0.rc2.
bundle
bundle exec rails generate paper_trail:install
bundle exec rake db:migrate
rails c
PaperTrail::Version
# NameError: uninitialized constant PaperTrail::Version

To clarify, rails 3.0.3 is not able to autoload the PaperTrail::Version constant.
Ben, given that rails 3.0 is EOL, and our test suite passes on rails 3.2, my instinct here is for paper_trail 4.0 to drop support for rails < 3.2.
Alex, is there any reason you need to use paper_trail 4.0 now, or can you just use paper_trail 3.x until you can upgrade to a non-EOL version of rails?

@jaredbeck - Not against that, however I think we could probably continue to support ActiveRecord 3.x by modifying this block here to read more like this:
if defined?(::Rails) && ActiveRecord::VERSION::STRING >= '3.2'
  require 'paper_trail/frameworks/rails'
else
  require 'paper_trail/frameworks/active_record'
end

I don't really see a lot of harm in continuing to support it for the time being, but perhaps it would be asking for trouble, but I guess my thinking is sort of "until there is something that will definitely break it, why drop support". That being said I think there is a line that should be drawn somewhere...  I mean the gem also still supports Ruby 1.8 which is probably unnecessary as well, but again, I didn't see much reason to drop support since it generally mostly boils down to using the hash rocket syntax for hashes. :wink:

.. I think we could probably continue to support ActiveRecord 3.x by modifying this block ..

Great!  Yeah, if there's a simple fix, I don't see any reason not to keep supporting rails 3.0.

It's worth a shot @alexclarkofficial, can you give cb10579 a try and see if that fixes your issue?

@batter Yes, can do! If I don't confirm by the Monday feel to ping me again.

@alexclarkofficial - Can you try the 4.0.0 release and confirm it fixes the issue you were seeing?

@batter Sorry for the hold up! Also sorry to report it failed even earlier... Simply having 'paper_trail' in my Gemfile stopped Rails from booting. Here's the log trying to run the generator:
Alexanders-Air:mackey alexanderclark$ bundle exec rails generate paper_trail:install
Bundler is using a binstub that was created for a different gem.
This is deprecated, in future versions you may need to `bundle binstub railties` to work around a system/bundle conflict.
/Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activerecord-3.0.3/lib/active_record/connection_adapters/abstract/connection_pool.rb:316:in `retrieve_connection': ActiveRecord::ConnectionNotEstablished (ActiveRecord::ConnectionNotEstablished)
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activerecord-3.0.3/lib/active_record/connection_adapters/abstract/connection_specification.rb:97:in `retrieve_connection'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activerecord-3.0.3/lib/active_record/connection_adapters/abstract/connection_specification.rb:89:in `connection'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activerecord-3.0.3/lib/active_record/base.rb:673:in `table_exists?'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/paper_trail-4.0.0/lib/paper_trail/config.rb:35:in `track_associations'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/paper_trail-4.0.0/lib/paper_trail/version_concern.rb:14:in `block in <module:VersionConcern>'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activesupport-3.0.3/lib/active_support/concern.rb:52:in `class_eval'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activesupport-3.0.3/lib/active_support/concern.rb:52:in `append_features'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/paper_trail-4.0.0/lib/paper_trail/frameworks/active_record/models/paper_trail/version.rb:5:in `include'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/paper_trail-4.0.0/lib/paper_trail/frameworks/active_record/models/paper_trail/version.rb:5:in `<class:Version>'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/paper_trail-4.0.0/lib/paper_trail/frameworks/active_record/models/paper_trail/version.rb:4:in `<module:PaperTrail>'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/paper_trail-4.0.0/lib/paper_trail/frameworks/active_record/models/paper_trail/version.rb:3:in `<top (required)>'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activesupport-3.0.3/lib/active_support/dependencies.rb:239:in `require'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activesupport-3.0.3/lib/active_support/dependencies.rb:239:in `block in require'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activesupport-3.0.3/lib/active_support/dependencies.rb:225:in `block in load_dependency'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activesupport-3.0.3/lib/active_support/dependencies.rb:596:in `new_constants_in'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activesupport-3.0.3/lib/active_support/dependencies.rb:225:in `load_dependency'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activesupport-3.0.3/lib/active_support/dependencies.rb:239:in `require'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/paper_trail-4.0.0/lib/paper_trail/frameworks/active_record.rb:4:in `<top (required)>'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activesupport-3.0.3/lib/active_support/dependencies.rb:239:in `require'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activesupport-3.0.3/lib/active_support/dependencies.rb:239:in `block in require'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activesupport-3.0.3/lib/active_support/dependencies.rb:225:in `block in load_dependency'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activesupport-3.0.3/lib/active_support/dependencies.rb:596:in `new_constants_in'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activesupport-3.0.3/lib/active_support/dependencies.rb:225:in `load_dependency'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/activesupport-3.0.3/lib/active_support/dependencies.rb:239:in `require'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/paper_trail-4.0.0/lib/paper_trail.rb:155:in `<top (required)>'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/bundler-1.10.5/lib/bundler/runtime.rb:76:in `require'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/bundler-1.10.5/lib/bundler/runtime.rb:76:in `block (2 levels) in require'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/bundler-1.10.5/lib/bundler/runtime.rb:72:in `each'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/bundler-1.10.5/lib/bundler/runtime.rb:72:in `block in require'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/bundler-1.10.5/lib/bundler/runtime.rb:61:in `each'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/bundler-1.10.5/lib/bundler/runtime.rb:61:in `require'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/bundler-1.10.5/lib/bundler.rb:134:in `require'
	from /Users/alexanderclark/mackeyRMS/mackey/config/application.rb:6:in `<top (required)>'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/railties-3.0.3/lib/rails/commands.rb:15:in `require'
	from /Users/alexanderclark/.rvm/gems/ruby-1.9.3-p551/gems/railties-3.0.3/lib/rails/commands.rb:15:in `<top (required)>'
	from script/rails:6:in `require'
	from script/rails:6:in `<main>'

Simply having 'paper_trail' in my Gemfile stopped Rails from booting.

Can you reproduce this in a brand new rails 3.0.3 app, with papertrail 4.0.0?

Any new ideas here? Can we reproduce this in a brand new rails 3.0.3 app, with papertrail 4.0.0?
Given the limited amount of time we have, I'd prefer to close this, and focus on versions of rails that are not EOL. However, Alex, if you have a solution, I think we'd love to have a PR.  Thanks.

Sorry, I actually changed jobs over the weekend and don't have a computer right now. Feel free to close.
Sent from my iPhone

On Aug 21, 2015, at 2:17 AM, Jared Beck<EMAIL_ADDRESS>wrote:
Any new ideas here? Can we reproduce this in a brand new rails 3.0.3 app, with papertrail 4.0.0?
Given the limited amount of time we have, I'd prefer to close this, and focus on versions of rails that are not EOL. However, Alex, if you have a solution, I think we'd love to have a PR. Thanks.
—
Reply to this email directly or view it on GitHub.


com/github/tomakehurst/wiremock/servlet/WireMockHttpServletRequestAdapter.java NullPointerException on newer than 2.20

I havn't gotten it down to a simple test case yet, but I dug into it,
StubMappingJsonRecorder#buildRequestPatternFrom has a call to request.getParts(), which returns null. This seems to be because WireMockHttpServletRequestAdapter#cachedMultiparts is defined, but size of 0. I'm not sure how it gets populated because I don't see it ever being null.
When i downgraded to 2.20 the same code ran again.
Nothing stands out to me looking at https://github.com/tomakehurst/wiremock/compare/2.20.0...2.22.0
Code is configured using the WireMockRule
@Rule
    public WireMockRule bitbucketApi = new WireMockRule(wireMockConfig().
            dynamicPort().dynamicHttpsPort()
            .usingFilesUnderClasspath("api/cloud")
    );

https://github.com/jenkinsci/blueocean-plugin/blob/340402489d16646128b5d75f205e0a241ca88761/blueocean-bitbucket-pipeline/src/test/java/io/jenkins/blueocean/blueocean_bitbucket_pipeline/cloud/BbCloudWireMock.java#L15-L18
I didn't set this up so I'm not entirely sure how everything works or what other info is useful.

Oh, I added the logger
.notifier(new ConsoleNotifier(true))

not sure if that matters

If you're still having this problem, please could you post a failing test case + your dependencies?


[Civl] Cleaned up the ticket sample

Constantin suggested using the ticket example to be the primary illustrator of all Civl machinery except permissions in the upcoming POPL 2024 tutorial. He made slides explaining the example. I was trying to change the code to become closer to the exposition. In particular, I dropped all the allocation of thread identifiers from the sample because permissions (specifically redistribution) will be explained in a more comprehensive using other examples (to be determined) and there is plenty going on in this example already.
Adding main back might still be useful.
Suggestions about the tutorial section on permissions are welcome.

@bkragl : BTW, I stopped writing PR summaries since I didn't think anybody was paying attention :-). But every once in a while you pop in with some insightful question or comment. Do you actually look at the Civl PRs regularly?

I do go over my github notifications, but often this happens in bursts. I try and see if I have the necessary context to contribute something, which unfortunately does not seem that often these days. I guess it's more likely for examples than it is for changes in the code. I understand that this is not a big motivation to write elaborate PR descriptions.

Here is a PR: https://github.com/boogie-org/boogie/pull/828


Is this expected behaviour? Error response from daemon: Conflict: The name ... is already assigned.

My use case is on our docker swarm cluster. The application container needs to sit behind nginx container so it needs to run nginx container on every host. Also it needs to be the same name on all hosts because our automated script needs to inject nginx configuration every new code deployment so we expected name should be the same on all hosts.
$ docker-machine -v
docker-machine version 0.5.6, build 61388e9

$ docker -v
Docker version 1.9.1, build a34a1d5

$ docker-machine ls
NAME             ACTIVE      URL       STATE     URL                                  SWARM                     DOCKER   ERRORS
node-01   * (swarm)   generic   Running   tcp://node-01:2376   node-01 (master)   v1.9.1
node-02   -           generic   Running   tcp://node-02:2376   node-01            v1.9.1

$ docker exec -it node-01/swarm-agent-master /swarm -v
swarm version 1.0.1 (744e3a3)

$ eval $(docker-machine env --swarm node-01)
$ docker run -d -e constraint:node==node-01 --name nginx -p 80:80 -p 443:443 nginx
a3322b355cd86305c99a00752f3ed948e0b795e61a21a9089dbcb18babd9c718

$ docker run -d -e constraint:node==node-02 --name nginx -p 80:80 -p 443:443 nginx
Error response from daemon: Conflict: The name nginx is already assigned. You have to delete (or rename) that container to be able to assign nginx to a container again.

$ eval $(docker-machine env node-02)
$ docker run -d --name nginx -p 80:80 -p 443:443 nginx
944ebdacd2b0997ecffef74936ae45606044691a4975dc522c691ecb43fdd431

$ eval $(docker-machine env --swarm node-01)
$ docker ps
CONTAINER ID        IMAGE                      COMMAND             CREATED             STATUS              PORTS                                                NAMES
944ebdacd2b0        nginx   "/start.sh"         4 seconds ago       Up 12 seconds       <IP_ADDRESS>:80->80/tcp, <IP_ADDRESS>:443->443/tcp   node-02/nginx
a3322b355cd8        nginx   "/start.sh"         2 minutes ago       Up 3 minutes        <IP_ADDRESS>:80->80/tcp, <IP_ADDRESS>:443->443/tcp   node-01/nginx

The question will be do you intend to have different container name on the whole docker swarm cluster? I expected it should count the swarm host name into container name also so it can still can be unique across whole docker swarm cluster.

@winggundamth Yes, the container name must be unique for the whole cluster by design.
In addition, when you're going to use the network feature introduced since 1.9, each container must be resolvable. So their names must be unique.
To make a container selectable, you may try using label in conjunction with --filter.


Clarify "default" feature

This PR extracts out core refactoring exposed by https://github.com/microsoft/vcpkg-tool/pull/295. Specifically, this PR seeks to clarify where in the code we are using "parsed-notation" (no features implies defaults, [core] implies not defaults) versus internal notation ([default] implies defaults).
The primary change to accomplish this is the introduction of struct InternalFeatureSet : std::vector<std::string> to denote when this handoff occurs. By type:

Dependency is parse-notation
ParsedQualifiedSpecifier is parse-notation
FullPackageSpec is internal-notation
FeatureSpec is internal-notation

Therefore, the core conversions to internal notation happen as part of Dependency::to_full_spec() and ParsedQualifiedSpecifier::to_full_spec() (shared via normalize_feature_list()). filter_dependencies() uses Dependency::to_full_spec().

/azp run

/azp run


chips: SPACE separator does not work with virtual keyboards on Android

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Separate chips on space when SPACE is set in MatChipInput.matChipInputSeparatorKeyCodes
What is the current behavior?
On Android devices it does not. (I personally tried Nesus X5 and pixel xl v1)
What are the steps to reproduce?
https://stackblitz.com/edit/chipinput
"android friendly chip input" uses keyup and looks at the last char of the input's value, to get around Android's "bad" keyboard event w/o keys and indistinguishable keycode 229.
What is the use-case or motivation for changing an existing behavior?
To be able to use MatChipInput on Android.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
All but on Android devices
Is there anything else we should know?
Android produces keycode 229 on space and other special characters.
Related to https://stackoverflow.com/questions/36753548/keycode-on-android-is-always-229/42552368

I'm also looking for a solution to this. Is there any workaround for using mat chips on Android?

Same issue here for me.

For the time being, I modified the MatChipInput.prototype._emitChipEnd method on line 1739 in the file

<EMAIL_ADDRESS>
to emit the chipEnd event by manually checking the value of the input field.
This is just a temporary hack until we get a fix.

Issue still occures in v7.3.6 :(

From looking into this a bit, it looks like this is caused by the fact that Android browsers always return 229 as the keycode regardless of what key is pressed.
One options for us would be to get the charCode of the previous character that was added, we would need affordances to ensure that we don't recheck the previous character twice in a row.

As I have continued to investigate here is a bit more information on the state of things.

The getCharCodeAt() method on String returns the ASCII charCode which is different than the keyCode that is provided in keyboard events.
The current recommendation (from mdn) for filtering events like this is actually to use key rather than keyCode, which returns the "result" of the keypress, i.e. shift + 1 produces event.key === '!' // true, while 1 produces event.key === '1' // true
Currently we do our checks in the keydown event which means that the value of the input does not yet include the key that was pressed, so we would have to also look for the value in keypress or in keyup

I faced the same issue and found a workaround.
Let say that "chipList" is the id of .
The following input is used to insert values into the mat-list, for example emails :
<input             [(ngModel)]="emailsInput"             (ngModelChange)="onEmailsInputChange()"             placeholder="insert an email..."             [matChipInputFor]="chipList">
Here we use the two way binding to trigger the onEmailsInputChange() function and reflect it to the mat-chip-list.
Here's the function content :
if (this.emailsInput.substr(-1) === ' ') this.add();
The add function will add the element to the mat-chip-list and so on.
The idea here is to use text based event instead of keyCode.

Hi, thanks for your help your link has helped me fix this issue which i was getting @BERRADA-Omar

Also had to use a workaround like those suggested here such as checking every keydown whether a space was entered. How is this still an issue 5 years later?


⚠ Bofrost.es has degraded performance

In 917d100, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1521 ms

Resolved: Bofrost.es performance has improved in b599434 after 13 minutes.


Constant Crashing With EXCEPTION_ACCESS_VIOLATION

hs_err_pid5564.log
Can't really run this mod as it randomly crashes in the process, usually when I'm editing a pallet, but I can't figure out the source of the incompatibility. I would love to offer more information to help, but outside of a mod list (which has 24 mods that do almost nothing hefty), I have no idea what would assist. I have no idea how anyone got this to work.

No offense intended, but it's obvious the author of the port doesn't really care about this. Dead mod is dead.


Is sound support possible?

it would be very nice to have sound support.

It's possible but not so easy to implement; some help may be nice :)


Persistant database in gitpod

Please describe the feature you would like to see implemented.
Adding this folder option to the docker init. It stores the MySQL database to the gitpod workspace folder and because it resides in the workspace folder it is persistent.
docker run --restart always -d --name=mysql -v /workspace/mysql:/var/lib/mysql -p <IP_ADDRESS>:3306:3306 -e MYSQL_ROOT_PASSWORD=root mysql:8

feel free to make a PR

Addopted it https://github.com/FriendsOfShopware/FroshTools/commit/b618937ead68fd46e024efa9edb7fb22d7581d4d

Next time will do it by PR


Move Sleep process definition

Overview
Moves the Sleep process class definition with other test processes to facilitate reuse.
Related Issue / Discussion
In response to comments in #657
Additional Information
Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[x] I'd like to contribute [feature X|bugfix Y|docs|something else] to PyWPS. I confirm that my contributions to PyWPS will be compatible with the PyWPS license guidelines at the time of contribution.
[x] I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

Coverage increased (+0.03%) to 81.087% when pulling 53ba8ac8910cba430d874066fc7986e7cae6a125 on huard:refactor_sleep_test into 85ca8191f1440e5906210408d2fa48c41f0ca679 on geopython:main.

@huard looks good. Thanks :)


During performance test got "The request channel timed out attempting to send after 00:04:00"

During performance test at 50 concurrent users we got the following error:
The request channel timed out attempting to send after 00:04:00. Increase the timeout value passed to the call to Request or increase the SendTimeout value on the Binding. The time allotted to this operation may have been a portion of a longer timeout. The HTTP request to 'https://irisoncology-uat.crm.dynamics.com/XRMServices/2011/Organization.svc/web?SDKClientVersion=<IP_ADDRESS>84' has exceeded the allotted timeout of 00:04:00. The time allotted to this operation may have been a portion of a longer timeout.
The related operation is creating an appointment emr entity in Dataverse. Most of the performance test calls succeeded and had response times of around 12 secs ( slow ). But one of them reported the above error.
I found the thread below that suggests setting appropriate values for Max Concurrent Calls, Max Concurrent Instances and Max Concurrent Sessions ( look for 1) Adjust Service Throttling Settings ):
https://social.msdn.microsoft.com/Forums/vstudio/en-US/692f5631-c40c-40d7-9a98-8d0d7eb0994b/systemtimeoutexception-the-request-channel-timed-out-while-waiting-for-a-reply-after?forum=wcf
How can we modify those values in our system?
Thanks!

Take a read though this: Azure Functions Best Practices
There is a good deal of guidance there on how to deal with those sorts of items.

Thanks Matt!
I'm wondering how the following is related to SetMinThreads
new property "RecommendedDegreesOfParallelism".
This property will report the recommended number of threads for communicating with Dataverse.
I can see in my system that I get a RecommendedDegreesOfParallelism =32
is it fair to say that setMinThreads(32, 32)
would ensure that when communicating with Dataverse that the 32 threads would be created on demand.
How is the RecommendedDegreesOfParallelism  calculated in your side?

Generally, that number is used to set the max concurrency number for any parallel operation.
See : (Parallel programming in .NET)[https://docs.microsoft.com/en-us/dotnet/standard/parallel-programming/]
Insofar as how the number is calculated, that is a combination of a number of factors on the platform side ranging from infrastructure to number of nodes in the supporting cluster and size of host's


Thread decoding support: Update pNzc (none-zero count) for precise deblocking

update pNzc (none-zero count) for deblocking in threaded mode: non-zero count vector should also be set for the current MB' neighborMBs. This is achieved by adding pNzc allocation in PICTURE.

closed due to a test case failure in clang c++.

closed due to a test case failure in clang c++.


Unable to find any of pip2, pip to use.  pip needs to be installed. Debian 9

Describe the bug
Installation error python-netaddr
TASK [dj-wasabi.zabbix-agent : Install local python-netaddr package] **********************************************************************************************************************************************
FAILED - RETRYING: Install local python-netaddr package (3 retries left).
FAILED - RETRYING: Install local python-netaddr package (2 retries left).
FAILED - RETRYING: Install local python-netaddr package (1 retries left).
fatal: [test-hw -> localhost]: FAILED! => {"attempts": 3, "changed": false, "msg": "Unable to find any of pip2, pip to use.  pip needs to be installed."}

Installation method/version

Ansible Galaxy, role version: 1.6.0

Ansible Version
ansible 2.5.1
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/home/user/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python2.7/dist-packages/ansible
  executable location = /usr/bin/ansible
  python version = 2.7.15rc1 (default, Nov 12 2018, 14:31:15) [GCC 7.3.0]

Targetted hosts
Concerns the following OS(es):

Debian 9.4

Expected behavior
Work role without this error. Or exclude netaddr package from installation

Fixed with the latest commits. Thanks so much

The issue still exists in dj-wasabi.zabbix-agent, 1.7.1 from ansible-galaxy, on Debian 9.9. pip is actually installed and the path is correct. Which version should I use? (Sorry, I'm still a bit new to Ansible.)


Update README & version as part of 0.9.5 Bijection release

Published artifacts for Bijection 0.9.5 - https://github.com/twitter/bijection/releases/tag/0.9.5

+1

Thanks for taking a look Oscar & Ben!


Is weld still under research/development?

Hi guys, @sppalkia @deepakn94 @mateiz  , we're considering to found a numeric computation WG in rust community and just wondering if weld project is still under research/development since there is no update for a few months? Maybe we can point out some new directions? Guess weld would be an important role in math computation field.

Hello everyone, I am also very interested in weld. I read two papers related to weld and ran several test cases using WeldNumpy. I did see performance improvements in some scenarios. During my use, I found that the current version does not support python3. I made some minor modifications to make it support python3. I noticed that some operations need to be added to WeldNumpy, and I hope to contribute some code. I am also very interested in the potential of weld in the parallel development of multiple machines. I hope to do some work for the weld community. Looking forward to your reply.

Thanks for your interest in Weld! Currently, none of us are actively working on the project, though I hope to return to it as a side project at some point.
When I was last working on it, I was "modernizing" the Python bindings and integrations. The python directory contains the old code with hacky NumPy and Pandas integrations that only work with Python 2.7 (plan is to eventually delete this). The weld-python directory contains the new code and a new version of Grizzly that works only with Python 3.x, and also contains many more tests, documentation on the functions, etc. This is where all the active development was last happening, but it does not have a NumPy integration yet.

I am exploring Weld for gpu to optimize LLM models written in Pytorch. But first, I need gpu support in Weld. Please see  https://www.modular.com/ and their Mojo lang to speed up ML workloads. I think Weld is a better tool than Modular.

In our private repo of Weld, my company has modified Weld as follows.

Refactored hardcoded llvm backend so that common backend code sits in the codegen directory and the llvm2 directory contains only LLVM specific code. So now, new backends can be added with ease.
Modified Weld to use LLVM 14.0.
Modified Weld to build and test on Apple Mac machine.

A question, what app do folks want to run on gpu?


Unable to start RAMA on port 8080

I am getting this logs but unable to start RAMA
cmd -  mvn spring-boot:run -Dspring-boot.run.jvmArguments="-Xss4m"

I am unable to run RAMA on port : 8080

Rama isn't running on port 8080. It's the Spring API server that runs on that port.
Without more information I'm unable to help you any further. I would take a look at any WARN or ERROR logs, or if there's something else on your machine running on that port.


Adding the missing field "bet_outcome"

Hey mate,
According the model definition at betfair docs for the model ClearedOrderSummary, it should contain also the bet_outcome.  I have added it to save you the pain. Works just fine for me  ;)

I believe those test failing have something to do with my PR

Thanks!