[Bug]: org.apache.beam.sdk.io.gcp.spanner.SpannerIOWriteExceptionHandlingTest.testExceptionHandlingForWriteGrouped flaky

What happened?
Example: https://ci-beam.apache.org/job/beam_PreCommit_Java_Phrase/5316/
@johnjcasey - Who could own this issue?
Issue Priority
Priority: 1
Issue Component
Component: io-java-gcp

Oh! I was looking at this one. I was trying to decide if it was an off-by-one error. It looks like the default back off formula is 15000*1.5^k and I think k=10 would be the last retry for a max back off of 900000?

Oh! I was looking at this one. I was trying to decide if it was an off-by-one error. It looks like the default back off formula is 15000*1.5^k and I think k=10 would be the last retry for a max back off of 900000?

I agree, that formula sounds off.
Could you add a code link? I am not sure where is that backoff code.

Sure, it's in FluentBackoff.java. You'll note that there's also a bit of randomness added in each step so what could very well be happening is that sometimes it maxes out at 9 retries and other times it maxes out at 10. With no randomness it would always hit 10 I think. It was failing pretty consistently for me in local testing.
That code also has things like max retries, but they don't come into play because the defaults are high (INT_MAX and 1000 days of cumulative backlog time).
Assuming this test really does need to care about retries it seems like there are several fairly straightforward options:

Test for retries > 1 instead of a specific number. This means changes to those defaults would go uncaught, but not sure why this particular test should care about the specific semantics of the backoff code (which has its own tests)
Explicitly set the max retries and an appropriate cumulative backoff to go with it
Seed Random's RNG with a fixed value before the test and use the number of retries consistent with that RNG value (if it is indeed switching between 9 and 10)

Sure, it's in FluentBackoff.java. You'll note that there's also a bit of randomness added in each step so what could very well be happening is that sometimes it maxes out at 9 retries and other times it maxes out at 10. With no randomness it would always hit 10 I think. It was failing pretty consistently for me in local testing.
That code also has things like max retries, but they don't come into play because the defaults are high (INT_MAX and 1000 days of cumulative backlog time).
Assuming this test really does need to care about retries it seems like there are several fairly straightforward options:

Test for retries > 1 instead of a specific number. This means changes to those defaults would go uncaught, but not sure why this particular test should care about the specific semantics of the backoff code (which has its own tests)
Explicitly set the max retries and an appropriate cumulative backoff to go with it
Seed Random's RNG with a fixed value before the test and use the number of retries consistent with that RNG value (if it is indeed switching between 9 and 10)

Ack. All 3 sounds like reasonable options.

.take-issue


[PE-D][Tester E] DG: Use Cases contain not externally visible behavior of app

As per the textbook, internal implementation details should not be shown in use cases.
Use Case 1: Add a client

Step 2 and Step 3 should not be part of use case.
Suggestion: Use Case could just be steps 1 and 4, with Extension 2a and 2b moved to 1a and 1b.
Use Case 2: Select a client
Step 2 should not be part of use case.
Extension 2a could be more along the lines of "User tries to select a non-existing user."
Use Case 3: Delete a client
Suggestion: Instead of having an Extension 2a, can put a Precondition in the use case e.g. Precondition: There is at least more than one client existing in the system.

Labels: type.DocumentationBug severity.Medium
original: youngseopark05/ped#15

Good bug!


Shift+Backspace

shift+backspace to clear text in current chat box

Related: ctrl+backspace to clear up to last space

I don't think we want to provide any enhancements to chatbox typing, correct me if I am wrong. Also, fix your title to be more descriptive.

Already implemented


adds support for raw URLs in request builders

fixes #410
Generation diff
https://github.com/microsoft/kiota-samples/pull/225
Todo

[x] replicate in dotnet
[x] replicate in java
[x] replicate in go
[x] replicate in ruby
[x] update unit tests
[x] bump abstractions versions

Todo doc comment new method request info

Can you please give some examples of isRawUrl will be used? I checked the generated files the meaning of isRawURL is 'not clear.

@nikithauc
The generated code for the fluent API will generate new RequestBuilder(currentPath, httpCore, false).
Somebody using the request builder with a raw URL (next link, delta link, ...) will end up writing new RequestBuilder(rawURl, httpCore) where the default value of the parameter is true.
That parameter is assigned to a property on the request builder which is then is passed to the requestInfo.setUri method (generate request method).
This parameter is used in the setUri method to know whether we should:

Append the current path and the segment to use as a URI
Parse the current path (which is the complete URL in this case) to get the query parameters into the dedicated dictionary, and use the rest as the uri.

Basically of of that is done to avoid appending a path segment when not needed and avoid "loosing" any query parameter that might already be present.
I hope this clarifies things?


Rolling/Jazzy updates

https://github.com/ros-controls/ros2_control/pull/1643
https://github.com/ros-controls/ros2_control/pull/1256

Hey! Are these 2 PRs reflected yet in this repo? Planning to loop this in on a Jazzy project and wondering if this is something we'll have to look out for


The Bitcoin Rollup #3

Need someone to complete the Bitcoin Rollup #3 for me :)
It's a newsletter that should document all Bitcoin rollup-related news as of September 12th (my gosh I didn't know it had been that long).
See the latest edition for the format. https://mirror.xyz/0xE4dF449bDC1ec8f7688F68F7E839f1370617Ac73/iYUQZMIexJB_bPB_YBGsoXWBf28LYqztRS9Q1NzLT4U
Join the tg chat if you have questions on how to do this:

This is drafted. Posting in the AM.
here's the draft: https://docs.google.com/document/d/1vfMIxlf-gtP4HlHjG22ZrxkVwgpUinzkTbpbnNAvb7I/edit#heading=h.hdo80ngje5ze

Newsletter posted https://twitter.com/BitcoinRollups/status/1717886639287017688


Macos Vibrancy isn't really showing.

While there is a certain color change, I don't really see it.
I am using Firefox 102.0

Hello. Do you have a system theme selected in your browser?

Yes.

What is vibrancy? and how do i enable it?


feat: use explicit extensions for main, module, and browser

:tada: This PR is included in version 0.3.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support database-map-mode = tmpfs

EOSIO chains which have a lot of state change activity need a lot of RAM and takes a long time to start/stop (database-map-mode = heap); or they use a lot of disk I/O (database-map-mode = mapped).
There is a hybrid solution documented here where statedb is kept on tmpfs.
https://cc32d9.medium.com/wax-blockchain-node-performance-challenges-c5c7a10cad
Maybe there is some opportunity to include this sort of functionality OOB.  The long term solution is probably RocksDB, but the performance is not yet acceptable (see https://github.com/EOSIO/eos/issues/10408)

A mode such as thing seems tricky to explain to users: when using this mode and stopping nodeos cleanly, shutting down the box will still lose state. Unless the user performs some other manual action to persist state (doing it automatically is literally heap mode). So maybe a nodeos -d datadir --backup-the-tmpfs-state-to-disk whenever the user wants to persist to disk after using this mode? Any other reasonable options?
Personally, I would rather see improvements to "heap" mode, such as quick start/stop (sub 5s I guess) and elimination of the environment check & pinned builds.

Speed improvements on "heap" mode are definitely desirable.   Maybe needs a --yes-i-know-what-im-doing-ignore-checks-flag. LOL.

There are times where you care about saving the state when nodeos stops, there are other times where you don't (ie start from snapshot is fine).  So making it optional to save state would be cool to save on disk i/o.


cross platform: enable release binary basic testing

current xplat release binary was broken and CI could not detect that
This PR enables very basic testing to make sure the compiled binary is functioning
/cc @dilijev @digitalinfinity @jianchun

note: Merge #1325 before this one.

LGTM


[Search] Fix tests

Allow name keys to not be sanitized in request bodies.

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


Module compiled with Swift 4.0 cannot be imported in Swift 4.0.2

Module compiled with Swift 4.0 cannot be imported in Swift 4.0.2. I use Xcode 9.1.

You use Carthage, right ?
Try deleting the "Carthage" directory of your project and rebuilding it with carthage update.

@frederic-adda Thanks! It work for me!


Create FFI for getting first value for header key

We currently do not expose any FFI function for getting the first value for a given header key. We have something like this in Rust and JS, would be useful to expose to FFI as well.
Context: https://github.com/subconsciousnetwork/subconscious/pull/389#discussion_r1069813766.

Originally filed in #199, but we can take these instead.

Fixed by https://github.com/subconsciousnetwork/noosphere/pull/202/files#diff-1efb4c78223a5de922b225414cce89a317e9cddec9c7fecb0e5c9bc177fe8acaR248


[BUG] Does this work with 3.3 stable ?

Describe the bug
On my project I have firebase and facebook (DrMoriarty), it build and run but every time I add this plugin I get this error :
com.android.builder.dexing.DexArchiveMergerException: Error while merging dex archives:
The number of method references in a .dex file cannot exceed 64K.
Smartphone (please complete the following information):
I'm building on an android S9 using macos

Thanks for opening the issue, I finished the Github Actions yesterday but i have not test yet because I was too tired, I will do this soon and give a feedback for you.

I tested here and it's working fine on Android

Can you test with the example project located here?
https://github.com/Poing-Studios/Godot-AdMob-Android-iOS/tree/master/example

thank you bro :)
the issue was with the godot gradle config the minsdk was 16 had to change it to 21 to enable multidex

@anisc Awesome! This is my config.gradle just in case if you want to compare (Was automatic generated by Godot):

Did you use this plugin with an other one because to use multidex minSdk have to be higher than 20 or you will have to use "multiDexEnabled true" with the default config
https://developer.android.com/studio/build/multidex

No, i have not tried yet, i only use with this plugin.
If the issue still persists i recommend you open an issue on Godot's Github project: https://github.com/godotengine/godot. I think theres nothing i can do with the plugin.

@gustavottc no worries bro the issue is already fix (ticket is closed) I Was just curious, if you used it with other plugin without the multidex enabled.


this not correctly captured when using it inside a lambda that is a super constructor parameter (still on 2.1.5)

Duplicated from #12889, still not fixed

TypeScript Version:  2.1.5
Code
// A *self-contained* demonstration of the problem follows...
class A {
    constructor(func: () => void) {
        func();
    }
}

class B extends A {
    constructor() {
        super(() => console.log(this.name));
    }
    public name : string;
}

//An empty console log should be written:
B();

Expected behavior:
(this works on 2.0.xxx)
//The super call line should translate to:
 var _this = this;
_super.call(this, function () { return console.log(_this.name); });

Actual behavior:
//Variable this is not available on the lambda, this results on a referenceError, this is not defined
//exception at runtime
var _this = _super.call(this, function () { return console.log(_this.name); }) || this;

This works as expected. this should not be used before the super contractor is done. this is an error in ES6.

The compiler can not know if the function will be immediately invoked, so it errors on the side of being permissive, but the runtime error is not something we can fix.


Increase size of unrestrictedstring class

The dropbox uploader operation can have lengths for UnrestrictedString input types (i.e. the path) that exceed the maximum length defined in data_structures.attribute_types.UnrestrictedStringAttribute

Increased to 1000 chars in 52747bbfafe23d92d6a62e6cc5018cc6d9d531fc


Discord bot unable to log in to discord from any computer

I suspect upgrading our dependencies might resolve this

This actually appears to have been a transient error. Still going to upgrade the discord.js dependency to make sure


Add an user to a group automatically after SAML authentication

Hello,
I'm trying to figure out a solution how to add an user to a group after this user is authenticated via SAML.
I was reading through the documentation and it seems overwriting this method might work:
from djangosaml2.backends import Saml2Backend

class MySaml2Backend(Saml2Backend):
    def save_user(self, user, *args, **kwargs):
        # Add custom logic here
        return super().save_user(user, *args, **kwargs)

My question is where should I add this piece of code in my Django project to get it working? I was trying to add it to my settings.py file, but I got an error: django.core.exceptions.AppRegistryNotReady: Apps aren't loaded yet.
Or is there any better solution how to add users automatically to Django groups after authentication?
Thanks in advance.

I was able to figure it out. I added the modified class to my authentication.py file where I'm storing my authentication backends.
class ModifiedSaml2Backend(Saml2Backend):     def save_user(self, user, *args, **kwargs):         user.save()         user_group = Group.objects.get(name='Default')          user.groups.add(user_group)         return super().save_user(user, *args, **kwargs)
Then I just referenced the modified Saml2Backend in settings.py file:
AUTHENTICATION_BACKENDS = [     'netbox.authentication.ModifiedSaml2Backend', ]
Works like a charm.

@tkloda
take a look to documentation, find the place that you think it would be better to extend it with the experience you shared here, do a PR (please to dev branch), I'll merge it asap
thank you for contribution, we're community

Thank you @tkloda
https://github.com/IdentityPython/djangosaml2/commit/c9ac2e7aa4da9819e15dab030e45468a5712dff6


Feature request: Add preview to development environment mapping

Hi,
At companies I've worked at recently, the word 'preview' is used regularly to describe environments where we deploy code before being accepted into the codebase. (preview deployments)
I was wondering if it might be possible to add preview or review to the array of development env mapping: https://github.com/atlassian/github-for-jira/blob/main/src/transforms/transform-deployment.ts#L141-L146
It looks like they are reworking the gitlab integration, but https://gitlab.com/gitlab-org/gitlab/-/merge_requests/80539/diffs#4868aa0ea6e830fd12667a79037bb1969b769a59_30_23 gitlab jira has review under development as well.

Hey @mwisner. Please see this comment on one of our other issues for an update on deployment mappings. https://github.com/atlassian/github-for-jira/issues/1303#issuecomment-1226715001

Hey @mwisner. I hope that our deployment mappings were able to help address your needs. I'm going to go ahead and close this issue. If you still need further assistance, and the deployment mappings don't quite address what you are after, please feel free to reopen this issue.


[Jenkins] Download the pipeline definition instead of cloning it from Git

Currently, each step in the pipeline clones common scripts from the Github repository. The credentials used to access the repository are the same as the Git credentials used to access the source project's repository. While acceptable for the sample projects and unmodified SCP sources, this is not a valid scenario in some corporate environments. Besides, there is no control of which version of the SCP is used.
The proposal is to download release archive from the Git repo instead of cloning it.

Implemented for Jenkins Job DSL


UnicodeDecodeError during installation

I get the following error during installation:

UnicodeDecodeError: 'cp949' codec can't decode byte 0xec in position 263: illegal multibyte sequence
An error occured.
IndexError: list index out of range

Environment is as follows:

OS: Windows 10 pro, latest build
Python: Anaconda 3.8

Hi @liganega are you sitll having this issue? I think this was an issue with jupyterlab itself which should be fixed now.

Thank you @ianhi for your kind reply.
Honestly, I don't remember anymore where I had that problem. But I'll let you know when I encounter the same problem again.

Hi @liganega, I understand that this has been resolved. Please reopen or open a new issue if you're still having this issue.


Style: Change 'String' to 'string' for simple references in variable declaration

PR Summary
Change 'String' to 'string' for simple references in variable declaration
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[x] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[ ] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[x] Add [feature] if the change is significant or affects feature tests

I resolved a merge conflict.


⚠ Thumbnails API Endpoint has degraded performance

In 0c4385e, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 5570 ms

Resolved: Thumbnails API Endpoint performance has improved in a3bced5 after 13 minutes.


UnitTestFrameworkPkg: Use TianoCore mirror of subhook submodule

Use git insteadOf to change subhook url from
https://github.com/Zeex/subhook to
https://github.com/tianocore/edk2-subhook because
old url is no longer available.
Description of changes

Testing

Looks good.
Maybe also adding NVIDIA issue into comment https://github.com/NVIDIA/edk2-nvidia/issues/109 ?


Add :only: option for code cells

I made this primarily for myself but thought should make a PR if there would be any interest in merging. I'm happy to add tests/documentation, but I'd like to get an OK that the implementation is fine first.
Also, as far as I can tell, NbParserConfig.builder_name isn't used by anything, so I just co-opted it.

I'm not sure yet if this will be widely used for notebooks, but cannot hurt the addition. Pending that tests and documentation will be added to this PR.

While adding the test, I added a get_latex function to access the built file. For some reason though the built latex file is named python.tex instead of {filename}.tex. I could not figure out why, so if anyone has an idea I'd be interested to know.

There are two errors, 1 is a typo I made and the other is about that Tex file naming issue as far as I can tell. Will take a closer look later.

I haven't had a long think about this PR, but my initial sense is one of reluctance to merge it. The code-cell directive in MyST-NB is part of an (implicit) standard, and any new features should be something we carefully introduce. MyST-MD is an example of another project that implements the MyST spec.
I wonder here whether a post-process tool would be better suited, e.g. https://github.com/agoose77/sphinx-builder-classes

I understand if there may be good reasons not to merge this, but as far as I understand :load: is also not a part of the spec, and it has been sitting here for a while.
About the link, post-processing doesn't always help when you need the code outputs to be dependent on the builder. I appreciate that this is not a straightforward thing to implement in the best way, but maybe something flagged as experimental may be ok, following :load:s example. I'll be using it myself either way.


Unable to Pan (vertical and inconsistently horizontal) - vs code

Holding Alt and then trying to pan does not work.  It can sometimes work for horizontal scrolling, but never works on vertical scrolling.

Hello @StephenOTT
Actually there exist a known issue with mediators, specially with panning, which prevents me from properly testing it. We have this issue already reported [1] and it's under our scope for fixing in the short term. Not sure if you're also facing with it, and maybe causing what you mention?
Although I guess what you describe is a different topic. If so, can you please share some reproducer video or screenshots, it will help us to properly understand the root for the wrong behavior.
Thanks!
[1] https://issues.redhat.com/browse/JBPM-8589

issue 8631 seems like it is more the problem i have describing.  The lack of canvas auto-growth when panning creates weird behaviour as you expect to to move but it does not and it feels "frozen" / stuck.

Agree, its behavior is not so unituitive as it should be. I am prioritizing those issues. Please keep an eye on further releases.
Thanks again for your feedback!

For better backlog organization, we are now tracking this issue only at https://issues.redhat.com/browse/JBPM-8631


fix minor typos in docs

and also test GitHub action setup

(I want to merge this but I don't have a permission to do it.)


Update iPXE binaries

Automated iPXE binaries update.

@jacobweinstock how should I verify these shas and binaries?

Good question. For the moment, I guess you could checkout the branch locally and verify them. These ipxe binaries are autogenerated following this process: https://github.com/tinkerbell/ipxedust/blob/main/docs/IPXE.md. The build output can be seen in the logs of this GitHub Action run: https://github.com/tinkerbell/ipxedust/runs/5041986724?check_suite_focus=true The PR was also autogenerated in that GitHub action run. It's currently using my user to do this. See this issue for more details on that. https://github.com/tinkerbell/ipxedust/issues/13


Feature Request: Renter should know and report (in API) if a file is apparently lost

When a file has been deleted locally and at least some part of the file is not available on the network, the renter should be able to derive that this file appears to be lost, mark that internally and report that in the API. If there is a contract/host still covering the missing piece(s), repair may still go on so that we can recover the file if the right offline host(s) come back. If that chance doesn't exist, the renter may even give up and stop to repair it as it costs money and we'll never be able to retrieve the whole file again anyhow.
The UI or someone else interacting with the API may decide to delete all files marked as "lost" in that way or re-upload them if they can get to a local copy in some other way.

Task would be to recognize when redundancy is <1x and file can not be found on local disk


oil: 0.8.7 -> 0.8.8

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/oil/versions.
meta.description for oil is: "A new unix shell"
meta.homepage for oil is: "https://www.oilshell.org/"
meta.changelog for oil is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/yrgczpn0cj82qxzrmdy3sr9plhq1yc2p-oil-0.8.8/bin/oil.ovm passed the binary check.

/nix/store/yrgczpn0cj82qxzrmdy3sr9plhq1yc2p-oil-0.8.8/bin/osh passed the binary check.

/nix/store/yrgczpn0cj82qxzrmdy3sr9plhq1yc2p-oil-0.8.8/bin/oil passed the binary check.

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

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

found 0.8.8 with grep in /nix/store/yrgczpn0cj82qxzrmdy3sr9plhq1yc2p-oil-0.8.8

found 0.8.8 in filename of file in /nix/store/yrgczpn0cj82qxzrmdy3sr9plhq1yc2p-oil-0.8.8

directory tree listing: https://gist.github.com/4619a6e80c611a74c10b0e11d4796b7c

du listing: https://gist.github.com/90b375cf6769472dbb92570f9ac63f2e

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

1 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
0 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
oil

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/yrgczpn0cj82qxzrmdy3sr9plhq1yc2p-oil-0.8.8 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A oil https://github.com/r-ryantm/nixpkgs/archive/5638ff9d2610e367620a1f7d658d9d35f9b68db3.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/yrgczpn0cj82qxzrmdy3sr9plhq1yc2p-oil-0.8.8
ls -la /nix/store/yrgczpn0cj82qxzrmdy3sr9plhq1yc2p-oil-0.8.8/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

oil

Maintainer pings
cc @lheckemann @fjallarefur for testing.

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

1 package built:

oil


Nanopolish runing with long time and lots of memory

Hi,
I have a sequence which contains 8.5G bases which is about 100X coverage of the sample. While I am running PolyA with Nanopolish it looks would take forever to process and occupied almost all of my memory. I decided to extract 200 or 500 reads for PolyA estimation in order to avoid the core dumped. But it makes no sense for such low amount of reads.
BTW, my virtual machine contains 48 treads, 380G memory.
Thank you

Hi,
Which version of nanopolish are you using? There was a bug that caused high memory usage, it is now fixed in 0.13.2.
Jared


Request specific exchange rates by setting the symbols or currencies parameter

Request specific exchange rates by setting the symbols or currencies parameter.
GET http://api.fixer.io/latest?symbols=USD,GBP
{
    "base": "EUR",
    "date": "2016-05-19",
    "rates": {
        "GBP": 0.76585,
        "USD": 1.1197
    }
}

Same approach as #8. I am going to add two ways of change symbols parameter:

Passing a symbols list to client constructor.
Passing a symbols list to each method. This argument will take priority over previous argument.


Adding support for sharing an image with imgur.

There needs to be some integration done to complete this.

amazing.


Exception when FIPS enabled

Hello,
When FIPS is enabled on computer registry, we get an exception when trying to log in.

The fix would require to change, in a future version, in OAuthFlowImpl :: getHash, the SHA256Managed instantiation with SHA256CryptoServiceProvider.

Hello @diacristian,
Thanks for raising the issue. The contributors will try to get to this, but you can also make your own PR for review if you would like to try and get this into a new version sooner. Hope you have a great day!


Hotfix Checklist on develop branch CONTRIBUTING.md

Current Hotfix checklist assumes merge into master and develop can be done by users or developers other than the commit master.
Instead of finishing the hotfix branch - which will merge the branch to master and develop in their local repositories - people should follow the path of pushing and pull requesting.

You are perfectly right. Gone fix the documentation in next release.


[ZEPPELIN-354] - List Notebook REST API implementation, Test & documentation

Implementation of List Notebook REST API including JUnit and documentation.
The return JSON object include only the name, id and some info fields and exclude the paragraph content and angualrObject collection.
Ready for review.

Are you able to figure why the link to json files are broken on the doc site?


issue to recieve e-ticket

I would like to receive an industry pass to attend the Bitcoin 2024.
Thank you.


possible to translate the component to brazilian portuguese ?

I would like to translate the names of the months and the day of the week to Brazilian standard

https://icehaunter.github.io/vue3-datepicker/config.html#locale prop is available just for that.
Example for your case:
<script setup lang="ts">
import { ptBR } from 'date-fns/locale'
import Datepicker from 'vue3-datepicker'
</script>

<template>
  <Datepicker :locale="ptBR">
</template>


Doc cli download timeout

Update doc to modify the huggingface-cli download timeout

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Great, thanks @Wauplin


Fix for changing the Forecast Offset type to a float64

Changed the type for Forecast struct's Offset var to a float64.
This is a fix for issue #26 to support timezones like , for eg., India, which have an offset of 5.5

Whoa crazy thanks


Projects in Todoist no longer being autopopulated

OS version: macOS
Browser version: Firefox 80
Extension version: 1.66.3

Relevant integration (if any):

todoist.com
🐛 Describe the bug

Toggl button shows on Todoist but projects must be manually selected.
Expected behaviour

Project should be automatically populated
Steps to reproduce

Steps to reproduce the behaviour:

Go to todoist
Make sure to have the same project in Todoist and in Toggl Track
The Project needs to be manually selected in Firefox (while it gets autopopulated in Chrome)

Other details or context

User mentioned they've switched to chrome in the meantime but that stopped working for them. I was able to reproduce it in Firefox but not in Chrome, case reference

Case reference

case reference

case reference


test: add tests for pytorch-lightning bagua strategy

fixes: https://github.com/BaguaSys/bagua/issues/480

@NOBLES5E  https://github.com/BaguaSys/bagua/pull/528 needs to be merged before this one, to solve CI failure.


Add a link to GitHub notifications page

Both the logo link and the GitHub icon link to this repo. It would be nice if the GitHub one linked to https://github.com/notifications.

This feature would be really nice!

This feature would be really nice!

I'm on v4.2.0 and I don't see the button @manosim added via #473. Is there something special I need to do??


Overline and underline bounding boxes are larger than necessary

Description
$
  #box(stroke:.1pt,$ overline(y) $)  #box(stroke:.1pt,$ macron(y) $) \
  #box(stroke:.1pt,$ underline(y) $)
$

renders as

As you can see, there is empty space above the overline, and below the underline. This can e.g. cause delimiters to be scaled larger than they need to.
Reproduction URL
No response
Operating system
No response
Typst version

[X] I am using the latest version of Typst

Fair enough, though I don't actually understand why the space should be there unless something is typeset above it. It just says "reserved".


How to use my saved folder with just one command?

Hello,  great extension! But one thing that would really make it faster for people like me that like to use keybinds, is a way of executing the command FFS.createFolderStructure just by doing a setted keybind like "ctrl+8" (in my case). When i tried to do this I created this command in my keybindings.json{    "key": "ctrl+8",    "command": "FFS.createFolderStructure", } , but it doesn't work the way it should :(, if do the keybind, the input to put the name of the file appears:

but then a error occurs if I press enter:

how can I use this? :/

Hey @njoppi2,
Thanks for reaching out.
which version of FFS are you using? I think it should work with the command via shortcut.
If you do not use the mouse to rightclick on a folder in the treeview and click "create folder" from there, VS Code does not give a path of the folder in which the templated folders should be created.
So what I did is an extra prompt before entering the component name where you specify the path in which the new folder should be created.

and after you entered the path you are asked for the component name.
Is this not happening for you?

My version is v2.0.0
I just tested now and it worked just like you said, not sure what happened before.
But I'd like to know, is it possible to make this mid-step automatic, so that we don't have to write the current folder? Maybe making it possible to pass an arg like this? {    "key": "ctrl+8",    "command": "FFS.createFolderStructure",    "arg": { "current": true }   },
just a sugestion, it'd be really awesome!

I will take a look when I find some time. But just to be sure: What would you expect current to be? The extension has no context from where you enter the command (as far as I know at least). Is current the file that you at that moment have open in the editor?

I was thinking about it being the current folder, not file, because as far as I know it would not work to create a folder inside the file, right? Btw, one thing that I really have no idea if it can help you are the ">copy path" commands that you can write in the ctrl+shift+p input, and they'll return to you your current path, maybe you could use something like this?

I'm still not sure I understand.
Current folder would be what exactly in your case?

Root folder of the project? (Doesn't make sense, I think )
Current folder the active file is in?
This might also be bad because if you have a components/<component>/index.js format then you would create the next component inside the components/ folder next to the index.js.

I'm thinking that instead of passing an arg that is called current you could pass something like
{parentFolder: 'src/components'} instead of having to enter it everytime.
This has the caveat that you would have to create multiple keybinds for different folders but it's pretty much the only solution I can think of right now that would work without issues and has the highest success rate.

Current folder the active file is in?
This might also be bad because if you have a components/<component>/index.js format then you would create the next component inside the components/ folder next to the index.js.

exactly, at least for me, all the times that I create components with FFS, I want them simply to be inside the current folder that I'm in, next to my index.ts file (if it's still not clear I can send you a print when I get home)

I never create folders/files with FFS in a outside component like src/components, this could be useful for someone, but at least for me it's not something I'd do

Alright, I understand your usecase now.
I will implement this once I find time although I think that your usecase is a somewhat rare one.

omg thank you!! :D

@njoppi2 you can now pass the argument "__current" and it should do as you requested.
For more information check the readme :)


Pick 1804 to release v1.24

Description

pick https://github.com/tigera/operator/pull/1804
For PR author

[ ] Tests for change.
[ ] If changing pkg/apis/, run make gen-files
[ ] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[ ] Milestone set according to targeted release.
[ ] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

closing, think we don't need this anymore - will reopen if things change
cc: @caseydavenport


Misconfigured environment

Building on of the existing examples leads to misconfigured environment error.
Steps to reproduce on mac OS X 10.15.4:

Checkout master using tutorial Getting Started Guide
( in my case: West version: v0.7.2 and master of Zephyr's project )

Run build of an existing example project:
west build -b nucleo_l432kc samples/hello_world

Get the error:

[0/1] Re-running CMake...

Including boilerplate (Zephyr base (cached)): /Users/<your_user_name_here>/Documents/DevProjects/zephyrproject/zephyr/cmake/app/boilerplate.cmake
-- Application: /Users/<your_user_name_here>/Documents/DevProjects/zephyrproject/zephyr/samples/hello_world
-- Zephyr version: 2.2.99 (/Users/<your_user_name_here>/Documents/DevProjects/zephyrproject/zephyr)
-- Board: nucleo_l432kc
-- Found west: /usr/local/bin/west (found suitable version "0.7.2", minimum required is "0.7.1")
-- Found toolchain: gnuarmemb (/usr/local/bin)
CMake Error at /Users/<your_user_name_here>/Documents/DevProjects/zephyrproject/zephyr/cmake/compiler/gcc/generic.cmake:8 (message):
  Zephyr was unable to find the toolchain.  Is the environment misconfigured?

User-configuration:

  ZEPHYR_TOOLCHAIN_VARIANT: gnuarmemb

  Internal variables:

  CROSS_COMPILE: /usr/local/bin/arm-none-eabi-

  TOOLCHAIN_HOME: /usr/local/bin

Call Stack (most recent call first):
  /Users/<your_user_name_here>/Documents/DevProjects/zephyrproject/zephyr/cmake/generic_toolchain.cmake:70 (include)
  /Users/<your_user_name_here>/Documents/DevProjects/zephyrproject/zephyr/cmake/app/boilerplate.cmake:502 (include)
  /Users/<your_user_name_here>/Documents/DevProjects/zephyrproject/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:24 (include)
  /Users/<your_user_name_here>/Documents/DevProjects/zephyrproject/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:40 (include_boilerplate)
  CMakeLists.txt:5 (find_package)

-- Configuring incomplete, errors occurred!
See also "/Users/<your_user_name_here>/Documents/DevProjects/zephyrproject/zephyr/build/CMakeFiles/CMakeOutput.log".
See also "/Users/<your_user_name_here>/Documents/DevProjects/zephyrproject/zephyr/build/CMakeFiles/CMakeError.log".
FAILED: build.ninja
/usr/local/Cellar/cmake/3.17.1/bin/cmake --regenerate-during-build -S/Users/<your_user_name_here>/Documents/DevProjects/zephyrproject/zephyr/sampl
es/hello_world -B/Users/<your_user_name_here>/Documents/DevProjects/zephyrproject/zephyr/build
ninja: error: rebuilding 'build.ninja': subcommand failed
FATAL ERROR: command exited with status 1: /usr/local/bin/cmake --build /Users/<your_user_name_here>/Documents/DevProjects/zephyrproject/zephyr/bu
ild

Obviously the error is in environment variable:

CROSS_COMPILE: /usr/local/bin/bin/arm-none-eabi-

Following temporar fix works for me:
In the file:

/Users/<your_user_name_here>/Documents/DevProjects/zephyrproject/zephyr/cmake/compiler/gcc/generic.cmake

add following command:

set(CROSS_COMPILE "/usr/local/bin/arm-none-eabi-")

@Nukersson is your GNU Arm Embedded toolchain installed in /usr/local/bin ? We don't support generic installations of GNU Arm Embedded, you need to download it manually from:
https://developer.arm.com/tools-and-software/open-source-software/developer-tools/gnu-toolchain/gnu-rm
and then install it in a fixed directory and point to it via the GNUARMEMB_TOOLCHAIN_PATH environment variable.

is your GNU Arm Embedded toolchain installed in /usr/local/bin ?
Yes it is.
But what I am confused of is:

CROSS_COMPILE: /usr/local/bin/bin/arm-none-eabi-
has bin twice while TOOLCHAIN_HOME: /usr/local/bin. How can it be?

@Nukersson how did you install GNU Arm Embedded?

My GNU Arm Embedded environment installed according to toolchain-gnuarmemb

P.S: I've updated west/zephyr today and the behavior is no more there.

Thanks, will close the issue then.

HI I have the same issue.

CROSS_COMPILE: /Users/alex/toolchains/arm/bin/bin/arm-none-eabi-
Why is it bin/bin?

CROSS_COMPILE: /Users/alex/toolchains/arm/bin/bin/arm-none-eabi-
Why is it bin/bin?

Oh no, just copy and paste mistake. 😄 I have edited it.

Hi,
Here are my env vars:
(dsp-quantized-too-big-fix) benchmark-fw $ echo $GNUARMEMB_TOOLCHAIN_PATH
/Users/alex/toolchains/arm
Is there some way to reset any cached things in the build system?  I did have a different env var at the start.  Here is the full output, showing the vesrion of Zephry
(dsp-quantized-too-big-fix) benchmark-fw $ ZEPHYR_BASE=~/work/tool-repos/zephyr/zephyr/ west build -p auto -b stm32f4_disco .
-- west build: generating a build system
Including boilerplate (Zephyr base (cached)): /Users/alex/work/tool-repos/zephyr/zephyr/cmake/app/boilerplate.cmake
CMake Deprecation Warning at /Users/alex/work/tool-repos/zephyr/zephyr/cmake/app/boilerplate.cmake:37 (cmake_policy):
The OLD behavior for policy CMP0079 will be removed from a future version
of CMake.
The cmake-policies(7) manual explains that the OLD behaviors of all
policies are deprecated and that a policy should be set to OLD only under
specific short-term circumstances.  Projects should be ported to the NEW
behavior and not rely on setting a policy to OLD.
Call Stack (most recent call first):
/Users/alex/work/tool-repos/zephyr/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:24 (include)
/Users/alex/work/tool-repos/zephyr/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:40 (include_boilerplate)
CMakeLists.txt:10 (find_package)
-- Application: /Users/alex/work/edgeimpulse/studio/dsp-pipeline/server/benchmark-fw
-- Zephyr version: 2.6.99 (/Users/alex/work/tool-repos/zephyr/zephyr), build: zephyr-v2.6.0-2865-g74cc8bee7c56
-- Found west (found suitable version "0.11.1", minimum required is "0.7.1")
-- Board: stm32f4_disco
-- Cache files will be written to: /Users/alex/Library/Caches/zephyr
-- Found dtc: /usr/local/bin/dtc (found suitable version "1.6.1", minimum required is "1.4.6")
-- Found toolchain: gnuarmemb (/Users/alex/toolchains/arm/bin)
CMake Error at /Users/alex/work/tool-repos/zephyr/zephyr/cmake/compiler/gcc/generic.cmake:8 (message):
Zephyr was unable to find the toolchain.  Is the environment misconfigured?
User-configuration:
ZEPHYR_TOOLCHAIN_VARIANT: gnuarmemb
Internal variables:
CROSS_COMPILE: /Users/alex/toolchains/arm/bin/bin/arm-none-eabi-
TOOLCHAIN_HOME: /Users/alex/toolchains/arm/bin
Call Stack (most recent call first):
/Users/alex/work/tool-repos/zephyr/zephyr/cmake/generic_toolchain.cmake:42 (include)
/Users/alex/work/tool-repos/zephyr/zephyr/cmake/app/boilerplate.cmake:552 (include)
/Users/alex/work/tool-repos/zephyr/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:24 (include)
/Users/alex/work/tool-repos/zephyr/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:40 (include_boilerplate)
CMakeLists.txt:10 (find_package)

Ok, I figured it out.  If you need to change your toolchain path (GNUARMEMB_TOOLCHAIN_PATH) you need to delete your build folder first.  Otherwise, west uses some cached copy of that path

It's not just a good idea, is necessary

Ran into issue:

west build -p always -b adafruit_feather_stm32f405 samples/basic/blinky
-- west build: making build dir /Users/sidp/zephyrproject/zephyr/build pristine
-- west build: generating a build system
Loading Zephyr default modules (Zephyr base).
-- Application: /Users/sidp/zephyrproject/zephyr/samples/basic/blinky
-- CMake version: 3.27.2
-- Found Python3: /Users/sidp/.virtualenvs/.venv/bin/python3.11 (found suitable exact version "3.11.4") found components: Interpreter
-- Cache files will be written to: /Users/sidp/Library/Caches/zephyr
-- Zephyr version: 3.4.99 (/Users/sidp/zephyrproject/zephyr)
-- Found west (found suitable version "1.1.0", minimum required is "0.14.0")
-- Board: adafruit_feather_stm32f405
-- Found toolchain: gnuarmemb (/opt/homebrew/bin/arm-none-eabi-as)
CMake Error at /Users/sidp/zephyrproject/zephyr/cmake/compiler/gcc/generic.cmake:9 (message):
Zephyr was unable to find the toolchain.  Is the environment misconfigured?
User-configuration:
ZEPHYR_TOOLCHAIN_VARIANT: gnuarmemb
Internal variables:
CROSS_COMPILE: /opt/homebrew/bin/arm-none-eabi-as/bin/arm-none-eabi-
TOOLCHAIN_HOME: /opt/homebrew/bin/arm-none-eabi-as
Call Stack (most recent call first):
/Users/sidp/zephyrproject/zephyr/cmake/modules/FindHostTools.cmake:107 (include)
/Users/sidp/zephyrproject/zephyr/cmake/modules/dts.cmake:9 (find_package)
/Users/sidp/zephyrproject/zephyr/cmake/modules/zephyr_default.cmake:115 (include)
/Users/sidp/zephyrproject/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:66 (include)
/Users/sidp/zephyrproject/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:92 (include_boilerplate)
CMakeLists.txt:4 (find_package)
-- Configuring incomplete, errors occurred!
FATAL ERROR: command exited with status 1: /opt/homebrew/bin/cmake -DWEST_PYTHON=/Users/sidp/.virtualenvs/.venv/bin/python -B/Users/sidp/zephyrproject/zephyr/build -GNinja -DBOARD=adafruit_feather_stm32f405 -S/Users/sidp/zephyrproject/zephyr/samples/basic/blinky

Fixed by setting GNUARMEMB_TOOLCHAIN_PATH=/opt/homebrew, as my toolchain was installed through homebrew.


Merge serving docker related modifications

Description

1. Why the change?

2. User API changes

3. Summary of the change

4. How to test?

[ ] N/A
[ ] Unit test: Please manually trigger the PR Validation here by inputting the PR number (e.g., 1234). And paste your action link here once it has been successfully finished.
[ ] Application test
[ ] Document test
[ ] ...

5. New dependencies

[ ] New Python dependencies
- Dependency1
- Dependency2
- ...
[ ] New Java/Scala dependencies and their license
- Dependency1 and license1
- Dependency2 and license2
- ...

To test the docker image build function, we will need to first launch ipex-llm


Readme is outdated

Hi,
the readme is entirely outdated. You cannot install airwave with those instructions anymore.

The listed wine version is outdated twice.
The SDK instructions are outdated, see https://github.com/phantom-code/airwave/issues/92

Airwave is not working for me on Ubuntu 18.04 anymore and I don't know why. Updated instructions would be a good start, though.
It would be a pitty if this repo just deprecates, because airwave is an awesome tool.

I'm willing to maintain the project if needed.


drivers: flash: nrf5: Use faster write interval with BT sync

Use a lower slot duration than the used erase slot duration
to speed up the interval of write operations when in sync
with active Bluetooth connections.
Signed-off-by: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master   #10111   +/-   ##
=======================================
  Coverage   52.48%   52.48%
=======================================
  Files         213      213
  Lines       26135    26135
  Branches     5633     5633
=======================================
  Hits        13718    13718
  Misses      10160    10160
  Partials     2257     2257

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 b6a65a9...55f8189. Read the comment docs.

@joerchan please address my one comment.

Shippable seems to be failing because of this kind of stuff:
-:66: ERROR:CODE_INDENT: code indent should use tabs where possible
#66: FILE: drivers/flash/soc_flash_nrf.c:342:
+^I^I^I   ^Ip_flash_op_desc->context->interval),$

I.e. you have some tabs, then a space, then another tab

@nvlsianpu @cvinayak Fixed comments.


[fix] Fix hostname verification

Motivation
If ValidateHostName is set to true, handshake always fails.
INFO  [] ClientConnection:375 | [ -> ] Connected to broker
ERROR [] ClientConnection:463 | [ -> ] Handshake failed: certificate verify failed
INFO  [] ClientConnection:1560 | [ -> ] Connection closed

Modifications

Verify serviceUrl.host(), not physicalAddress.

physicalAddress is serviceUrl, which contains protocol (e.g. pulsar+ssl) and port number.

Use ssl::stream::set_verify_callback instead of ssl::context::set_verify_callback.

Verification should work with ssl::context::set_verify_callback, but somehow it doesn't work.

Verifying this change

[x] Make sure that the change passes the CI checks.

Documentation

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

Rebased master

@merlimat PTAL, thanks

After I reverted your changes in ClientConnection.cc, these tests could still pass.

testTlsDetectPulsarSslWithHostNameValidationMissingCertsFile tests whether the result of createProducer is a connection error when certsFile is missing. Hostname verification of master always fails, so this test could still pass.
This PR fixes testTlsDetectPulsarSslWithHostNameValidation. When certsFile is set corretly, the result of createProducer should be ok.
https://github.com/apache/pulsar-client-cpp/blob/v3.1.0/tests/AuthPluginTest.cc#L152
https://github.com/apache/pulsar-client-cpp/pull/126/files#diff-43a1460bfe2216170e3d7a5f634e8d0422b78946b018f81118dc275cbba2d04fR152

@RobertIndie

Could you also add a test case that demonstrates the failed case of hostname verification?

The test requires changing configurations of a broker, but mocking is difficult.
I don't think it is a good idea to set up another standalone cluster.
Do you have any ideas?

I don't think it is a good idea to set up another standalone cluster.

I think you can change the current config directly. Will it affect other tests? If yes, I think setting up another standalone is okay.

I added tests for invalid broker using another standalone.

The fix LGTM, but could you describe how to generate this certificates? It would not block this PR but I think it could be improved to make tests clear. It seems that when hn-verification/cacert.pem is used, setValidateHostName must be set true.

could you describe how to generate this certificates?

The certificates I added for man-in-the-middle attacks is the same as the certificates in pulsar repository.
https://github.com/apache/pulsar/tree/v2.11.0/pulsar-broker/src/test/resources/authentication/tls/hn-verification
This is also true for other existing certificates in pulsar-client-cpp repository.


Fix bookinfo samples

Please provide a description of this PR:
Follow https://istio.io/latest/docs/ops/ambient/getting-started/#control gives me:
 kubectl apply -f samples/bookinfo/gateway-api/route-reviews-90-10.yaml
Error from server (BadRequest): error when creating "samples/bookinfo/gateway-api/route-reviews-90-10.yaml": HTTPRoute in version "v1beta1" cannot be handled as a HTTPRoute: strict decoding error: unknown field "spec.parentRefs[0].port"

Applying without the port works well for the traffic shifts.

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/cc @frankbu

@hanxiaop It should work with or without the port. Are you sure you are using the experimental (not standard channel) CRDs?
kubectl kustomize "github.com/kubernetes-sigs/gateway-api/config/crd/experimental?ref=v1.0.0" | kubectl apply -f -
"github.com/kubernetes-sigs/gateway-api/config/crd/experimental?ref=v1.0.0"

@hanxiaop It should work with or without the port. Are you sure you are using the experimental (not standard channel) CRDs?
kubectl kustomize "github.com/kubernetes-sigs/gateway-api/config/crd/experimental?ref=v1.0.0" | kubectl apply -f -

I see, I'm using the standard one. Thanks


Abstract class results in wrong generated TS code.

Hey,
with 11.12.10, apparently 'abstract' was added to some generated TS classes. This results in invalid code:
/** Base class for payloads that may be validated. */
export abstract class AbstractPayload implements IAbstractPayload {

    constructor(data?: IAbstractPayload) {
        if (data) {
            for (var property in data) {
                if (data.hasOwnProperty(property))
                    (<any>this)[property] = (<any>data)[property];
            }
        }
    }

    init(data?: any) {
        if (data) {
        }
    }

    static fromJS(data: any): AbstractPayload {
        let result = new AbstractPayload();
        result.init(data);
        return result;
    }

    toJSON(data?: any) {
        data = typeof data === 'object' ? data : {};
        return data;
    }
}

which results in
error TS2511: Cannot create an instance of the abstract class 'AbstractPayload'.
on line
let result = new AbstractPayload();
11.12.9. works fine.

Can you try this template which should fix the issue?
https://raw.githubusercontent.com/RSuter/NJsonSchema/master/src/NJsonSchema.CodeGeneration.TypeScript/Templates/Class.liquid
(use the TemplateDirectory setting and replace Class.liquid)

Hi,
I having this error using nswagstudio v<IP_ADDRESS>
There is some way to correct it using this template?
TIA

So, you just want to disable the generation of abstract classes?

It´s required for me to generate the code for abstract classed.
I'll illustrate it better with two examples:
Examples:
1.
export class EntitiesDifference extends EntitiesOperation implements IEntitiesDifference {
 source: QueryNode = null;
 except: QueryNode = null;

 constructor(data?: IEntitiesDifference) {
     super(data);
     this._discriminator = "EntitiesDifference";
 }

 init(data?: any) {
     super.init(data);
     if (data) {
         this.source = data["Source"] ? QueryNode.fromJS(data["Source"]) : new QueryNode();

NOTE: QueryNode is abstract. I replaced the code with null to continue working.
2.
static fromJS(data: any): PathPredicate {
        data = typeof data === 'object' ? data : {};
        if (data["discriminator"] === "InRelativePeriod") {
            let result = new InRelativePeriod();
            result.init(data);
            return result;
        }

NOTE: InRelativePeriod is abstract too. And is not possible to receive a discriminator with this value.

Ok, I see - two more "special cases".

See https://github.com/RSuter/NSwag/issues/1093#issuecomment-363476113


Use poetry for dependencies management

See: #190
Starting this is an intial draft to move to poetry
TODO:

[ ] Test Dockerfile changes
[ ] Make cognita a publishable package (#191) - We do not want our users to worry about poetry

[ ] Refactor to put source code in a cognita module - Changing imports
[ ] Add cli to invoke serve and index - Update entrypoints in pyproject.toml
[ ] Update Dockerfile
[ ] Remove requirements.txt

[ ] Add a workflow to publish package
[ ] Update Docs

Later:

[ ] Add a workflow to publish docker images
[ ] Setup pytest and add a workflow

I'll re-open this later, we already have conflicts with recent changes


Bug: wxSimpleHtmlListBox defaults to 0 by 0 dimension (invisible)

Description:

The wxSimpleHtmlListBox won't show up by default unless both the expand flag is set and proportion is set to at least 1. The other option is to set the minimum size. Without anything telling the box to expand, it will be invisible because it takes up no space.

This isn't the only control that needs expand and/or proportion set to get a reasonable size. We need a general purpose solution which can then be applied individually as needed.


[MINOR] remove akka.version in pom.xml

Doesn't seem to be used. This version is very out of date if it is - especially since Akka now has a Category X license.

Atals has not been updated in the past two years, is it not maintained
Atals has not added the latest computing engines such as Flink and Doris, as well as a database that integrates lakes and warehouses

Is there a road map for Atlas releases or this project dead?


[d3d9] Fix strange type in dynamic vertex upload

Will take some time to review this one i think. Quite big as PRs go


Broken rendering (or encoding?) of EXAMPLEs (possibly NOTEs?) in terms

In this document:

https://github.com/metanorma/bipm-si-brochure/blob/main/sources/bipm-outcomes-en.adoc

The EXAMPLEs are rendered without proper style and runs into the example content.

A typical EXAMPLE in BIPM looks like this (from the same document):

I'm not sure whether this is an encoding or rendering issue, hence tagging both of you.

This is a rendering issue due the wrong styles for termexample element. I don't figure out why I've missed it. Will fix.

Thank you @Intelligent2013 !

@Intelligent2013 can you also review the termnote style? Thanks.

@Intelligent2013 can you also review the termnote style? Thanks.

@ronaldtse currently, in the BIPM brochure, note's render in three manner:

in the main section - as side notes (on the right page margin),
in the preface section - as main text with 'NOTE:' title:

in the quote text - as main text with 'Note:' title:

I think enough to fix the bottom margin (in XSLT) and add the colon after to entry (in metanorma gem by @opoudjis):
Currently:

After changes:

Right?

Agree. Thanks @Intelligent2013 !

@opoudjis currently, in BIPM presentation XML, colon is missing in termnote/name at the end:
<termnote id="_"><name>Note 1 to entry</name>
Information resource can refer to an asset, a record, a dataset, a
document or a component part of a document.

</termnote>

Could you add it? Thanks.

BIPM XSLT updated for termexample and termnote rendering:

(: after Note 1 to entry added manually just for demo)

I've added the colon; is there anything else needed for this ticket?

Confirm fixed.


Space-separated test classes are not executed during deployment in GitLab pipeline

Summary
spece-separated test classes specified using --tests are not executed for sf project deploy start command
Steps To Reproduce
Execute sf project deploy start command with space-separated test classes for --tests argument. (More than one test classes)
Expected result
Test methods in the specified test classes should be executed and deployment must be successful with successful test class execution or failed with one or more failed test methods
Actual result
Tests have not been executed and ended up with deployment failure due to 0% code coverage. All were previously working fine.
System Information
bash
{
"cliVersion"<EMAIL_ADDRESS>"architecture": "linux-x64",
"nodeVersion": "node-v19.9.0",
"osVersion": "Linux 5.15.0-1039-aws",
"shell": "bash",
"rootPath": "/usr/local/lib/node_modules/@salesforce/cli",
"pluginVersions": [
"@oclif/plugin-autocomplete 2.3.2 (core)",
"@oclif/plugin-commands 2.2.18 (core)",
"@oclif/plugin-help 5.2.13 (core)",
"@oclif/plugin-not-found 2.3.31 (core)",
"@oclif/plugin-plugins 3.1.6 (core)",
"@oclif/plugin-search 0.0.20 (core)",
"@oclif/plugin-update 3.1.26 (core)",
"@oclif/plugin-version 1.3.7 (core)",
"@oclif/plugin-warn-if-update-available 2.0.43 (core)",
"@oclif/plugin-which 2.2.25 (core)",
"@salesforce/cli 2.1.7 (core)",
"apex 2.3.6 (core)",
"auth 2.8.9 (core)",
"data 2.5.1 (core)",
"deploy-retrieve 1.16.0 (core)",
"info 2.6.31 (core)",
"limits 2.3.26 (core)",
"login 1.2.20 (core)",
"org 2.9.25 (core)",
"schema 2.3.20 (core)",
"settings 1.4.20 (core)",
"sobject 0.1.37 (core)",
"source 2.10.25 (core)",
"telemetry 2.2.3 (core)",
"templates 55.5.5 (core)",
"trust 2.4.32 (core)",
"user 2.3.24 (core)"
]
}
PASTE_VERSION_OUTPUT_HERE

Additional information

Hello @shabu74, a few questions

Could you share the exact command you are running?
Is this only failing in GitLab? Does it work for you locally?
Can you try passing the --tests flag multiple times?

Instead of --tests Test1 Test2
...use --tests Test1 --tests Test2

sf project deploy start --async --dry-run --ignore-conflicts --manifest releasePackage/package.xml --post-destructive-changes releasePackage/destructiveChangesPost.xml --target-org $targetusername --test-level RunSpecifiedTests --tests $run_specified_test --json --api-version $SF_API_VERSION > deploy-result.json
I did not try it locally. It worked perfectly for me in GitLab till last week. Also I have never tried --tests Test1 --tests Test2. I will try that. But my concern is that it worked fine before

You can see that those test classes have not been considered @iowillhoit

--tests Test1 --tests Test2... This approach worked. But if we need to execute 50 test classes, we need to specify 50 --tests. I don't prefer that

sf project deploy start --async --dry-run --ignore-conflicts --manifest releasePackage/package.xml --post-destructive-changes releasePackage/destructiveChangesPost.xml --target-org $targetusername --test-level RunSpecifiedTests --tests $run_specified_test --json --api-version $SF_API_VERSION > deploy-result.json

@shabu74 can you check the value of $run_specified_test in CI logs?
I see this works locally:
➜  dreamhouse-lwc git:(main) ✗ sf project deploy start --test-level RunSpecifiedTests --tests GeocodingServiceTest TestPropertyController --metadata ApexClass:Geocoding
ServiceTest --metadata ApexClass:TestPropertyController
Deploying v58.0 metadata to<EMAIL_ADDRESS>using the v58.0 SOAP API.
Deploy ID: 0Af03000016LQlDCAW
Status: Failed | ████████████████████████████████████████ | 2/2 Components (Errors:0) | 4/6 Tests (Errors:2)

Test Failures [2]
• GeocodingServiceTest.successResponse
  message: System.AssertException: Assertion Failed: Expected: 0, Actual: 1
  stacktrace:
    Class.GeocodingServiceTest.successResponse: line 32, column 1

• TestPropertyController.testGetPagedPropertyList
  message: System.AssertException: Assertion Failed: Expected: 4, Actual: 5
  stacktrace:
    Class.TestPropertyController.testGetPagedPropertyList: line 70, column 1

Test Results Summary
Passing: 4
Failing: 2
Total: 6
Time: 2693


fix: Media type selection not reflecting what is saved

When opening an existing TV Show rule, the media type would be incorrectly set to Shows instead of what was saved.

/release-pr

/release-pr

:tada: This PR is included in version 2.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Logspout crashes when route uri contains non-default query parameters

$ docker run -v=/var/run/docker.sock:/var/run/docker.sock gliderlabs/logspout:master syslog://xxx/?hello=kitty
# logspout v3-master by gliderlabs
# adapters: raw udp tcp syslog
# options : persist:/mnt/routes
panic: runtime error: assignment to entry in nil map

goroutine 16 [running]:
runtime.panic(0x7af1c0, 0xa1e5d3)
	/usr/lib/go/src/pkg/runtime/panic.c:279 +0xf5
github.com/gliderlabs/logspout/router.(*RouteManager).AddFromUri(0xc208010d50, 0x7fff3c88ff6d, 0x19, 0x0, 0x0)
	/go/src/github.com/gliderlabs/logspout/router/routes.go:102 +0x4b3
github.com/gliderlabs/logspout/router.(*RouteManager).Setup(0xc208010d50, 0x0, 0x0)
	/go/src/github.com/gliderlabs/logspout/router/routes.go:184 +0x1bb
main.main()
	/go/src/github.com/gliderlabs/logspout/logspout.go:39 +0x646

goroutine 19 [finalizer wait]:
runtime.park(0x414ad0, 0xa22c58, 0xa20ea9)
	/usr/lib/go/src/pkg/runtime/proc.c:1369 +0x89
runtime.parkunlock(0xa22c58, 0xa20ea9)
	/usr/lib/go/src/pkg/runtime/proc.c:1385 +0x3b
runfinq()
	/usr/lib/go/src/pkg/runtime/mgc0.c:2644 +0xcf
runtime.goexit()
	/usr/lib/go/src/pkg/runtime/proc.c:1445

Ran into this while working on an HTTP adapter. This adapter will buffer messages to not to make too many roundtrips. I was hoping that I could expose the configuration of the buffer (size, timeout) via options (like the filter.id, ... stuff). when trying, i ran into the crash above.
The fix appears to be simple to me (but i am not a really very experienced when it comes to go...) - in routes.go, change line 83 from this:
	r := &Route{
		Address: u.Host,
		Adapter: u.Scheme,
	}

to this:
	r := &Route{
		Address: u.Host,
		Adapter: u.Scheme,
		Options: make(map[string]string),
	}

This works for me, but i am not sure if this is the best thing to do. If it is, happy to make a PR.

This has been called out in @nagliyvred's issue #75 and pull request #76 takes the same solution you've come up with. It's worked for me for well over a week.

Thanks, @dylanmei - i should have spent the time to look at all the open issues and PRs. FWIW, the fix has been working for me as well while developing the HTTP adapter over the last couple of days. Closing this now in anticipation of #76.

Glad to hear it's working for you as well. I look forward to checking out your adapter. :)


updates to GSoC Page after selection into the program

This updates page after the selection Kubeflow into GSoc mainly highlighting

how to participate in kubeflow's gsoc
how to reach out to mentors
how to reach out to mentors in community calls
what to do if student did not get selected in gsoc program

@thesuperzapper can you review this pls

/lgtm
/approve


Close dropdowns when the container that opened them scrolls

As title says. This seems to commonly be the default behavior in UIs.

Fixed in e68fcbbcc3af13a579bfcc72873d9b25a7d11b28


add .flake8 config

Pull Request summary:
Instead using command line arguments both for pre-commit and linting github action, provide single .flake8 config which will be used for both.
Also this is only way to apply per file ignores.
Checklist:

[x] Make sure to run pre-commit before committing changes
[x] Make sure all checks have passed
[x] PR description is clear and comprehensive
[x] Mentioned the issue that this PR solves (if applicable)
[x] Make sure you test the changes

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/pull/450


TypeError beta_geo_beta_binom_fitter.py

Using the plotting.plot_period_transactions with a condensed RFM matrix for the BGBB fitter and received this error.
~\AppData\Local\Continuum\anaconda3\lib\site-packages\lifetimes\fitters\beta_geo_beta_binom_fitter.py in <lambda>(size)
    167         self.generate_new_data = lambda size=1: beta_geometric_beta_binom_model(
    168             # Making a large array replicating n by n_custs having n.
--> 169             np.array(sum([n_] * n_cust for (n_, n_cust) in zip(n_periods, weights))),
    170             *self._unload_params("alpha", "beta", "gamma", "delta"),
    171             size=size

TypeError: unsupported operand type(s) for +: 'int' and 'list'

Original:
            np.array(sum([n_] * n_cust for (n_, n_cust) in zip(n_periods, weights))),
Updated:
            np.array(sum(([n_] * n_cust for (n_, n_cust) in zip(n_periods, weights)),[])),

hm, I wonder if this is a new bug (0.11+), or has always been around. What version of lifetimes, python and Pandas are you using?

It was around in version 10 of lifetimes. I meant to mention it earlier in the week when I started fooling with the BGBB, but it just slipped my mind. I'm using 11.1 now and pandas 0.24.1.

Awesome, thanks! I'll fix and write some regression tests for it

NP. QQ for you. Are the calibration graphs meant to be used with with a BGBB model that's been trained on a condensed RFM matrix? I had to make some changes to get plot_period_transactions to behave as intended with the RFM.

hi quick Q on a fix for this issue?


NameError: name 'wr' is not defined in chapter03/01-prep_data.ipynb

Hi @michaelhsieh42 ,
First of all, thank you for the amazing content in this book (😄 I'm just a few chapters in, but feels pretty good so far)
I've only begun to try a few examples from this book, and I've run into an error NameError: name 'wr' is not defined with chapter 3. Can you take a look?

Thanks for reporting this issue. I have addressed it in the notebook.


Increase session duration

The client noticed the session does not last long enough.
Add a remember me feature. [Here] (http://laravel.com/docs/4.2/security) is the documentation for laravel's 4.2 auth library.

Files that should require modifications:
model/User.php
controllers/UsersController.php
views/users/login.php
views/users/signin.php

add a checkbox "remember me" (se souvenir de moi) in every singin/login form

changing the session driver may also solve this issue
http://laravel.com/docs/4.2/session


Expands documentation and adds examples to circuit.initialize

Fixes #5400

@Cryoris Reverting back broke the branch so I force reset and cleaned it up. This however broke the PR and thus I closed the previous one and opened this one.

@Cryoris I apologize, I am not sure how this happens. I only pushed the changes you suggested. Is there something I can do to change back to 3cc9d65?

Besides the issues with the files, the code blocks should probably be jupyter-execute blocks so that they are executed when the docs are built.


Error

After traveling to the UK from EU i can no longer connect to the server.
I can conecct whit an 1.12 launcher.

Do you know what DNS Server you are using?
Somehow it is resolving logon.everlook.org to an IPv62IPv4 prefix


Regression test for assist backups

We recently had a scenario where assist backup [1] logic regressed while a PR chain was being merged.
This tracks adding a regression test for this case. We don't necessarily need an e2e test here (it might be hard to simulate failures).
[1]
assist backup: Backup operation that leverages a previously failed kopia snapshots data

we may be able to leverage tests that are similar to TestExtensionsIncrementals but add some checks for the number of cached/uncached files. Now that the logic for details merging has been revamped a little, kopia assist cached items are actually counted as cached instead of written so that could be a decent indicator


Tabs > support manual activation

WCAG distinguish between automatic activation (tab panel displays as soon as the corresponding tab receives focus) and manual activation (click, enter, or space required to display a tabpanel after the corresponding tab receives focus). Manual activation is typically used where the content of tab panels is not immediately available, e.g. they rely on a network request.
The current tab implementation only supports automatic activation.
ref - https://www.w3.org/TR/wai-aria-practices-1.1/examples/tabs/tabs-1/tabs.html

Closed with https://github.com/stormid/components/pull/186


Data pipelines

The base Converter class should have responsibilities for filtering, standardising and producing splits removed from it.  These should be implemented individually for datasets.
That said, a default pipeline should be provided, that will be used for all the datasets at the moment, but it may not be appropriate for all datasets!

Completed a couple commits ago.  The data module will need a rethink though.


oss-fuzz for at least font-types, read-fonts and skrifa

Bonus points for write-fonts. https://github.com/google/oss-fuzz/tree/master/projects/woff2 might be a good starting point.

https://google.github.io/oss-fuzz/getting-started/new-project-guide/rust-lang/ has instructions

For skrifa https://github.com/google/oss-fuzz/tree/master/projects/freetype2 might be useful too, should be testing somewhat similar ops

Per discussion with @garretrieger timeouts, very slow reads due to exploding graphs, seem likely to apply to Rust. Hopefully fewer use after free and out of bounds reads :)

We'll need a font corpus to feed it. After a quick glance at https://github.com/google/oss-fuzz/blob/master/projects/freetype2/Dockerfile, perhaps *.[ot]t[cf] from:

https://github.com/unicode-org/text-rendering-tests
https://github.com/googlefonts/color-fonts
https://github.com/harfbuzz/harfbuzz/tree/main/test

https://github.com/freetype/freetype2-testing/tree/master/fuzzing/corpora - in particular the COLRv1 corpus is something I derived for FT from https://github.com/googlefonts/color-fonts. The fonts in the corpora should be small and probably make sense to subset from actual font collections.


Category slug doesn't get updated

Issue Summary
When changing the title of a category in the dashboard, the slug doesn't get regenerated.
Technical details

Python 3.6.10
Django version: 2.2.15
Oscar version: 2.1

I think it's the expected behaviour. If you update the slug it will have a new url and the previous one will raise 404. And it's bad thing for seo.
Maybe could be implemented this feature with django redirects. What do you think @solarissmoke? i could work for that as i implemented this feature in a couple of projects.

Oscar doesn't change the slug if it already exists to avoid altering URLs - this was because original the default URLs for category views used only slugs, and altering the slug would break old URLs. Now though, the URLs include the category ID and the view will redirect automatically if the slug portion is wrong, so it would be less problematic to change them.
That said, I don't think slugs should be updated automatically, and would prefer instead to address #561 - i.e., allow manual editing of slugs. This shouldn't be particularly difficult - just requires exposing the slug field in the dashboard form. PR for this is welcome.


Revise documentation about basic authentication

What is this PR for?
Revised documentation about nginx authentication after receive few comments about it. Especially for users who using old version of nginx (older than 1.3) facing websocket failures when using zeppelin after follow instructions. I put more information for those users to encourage use of latest version of nginx if possible + fix few grammars for readability.
What type of PR is it?
Documentation

Thanks @yoonjs2 for the nice improvement on documentation.
LGTM and merge if there're no more discussions!


Create Pickpocket Safe Guard

Pickpocket Safe Guard allows users to specify a HP Threshold in the settings. Once this threshold is reached (current HP equal to or less than the threshold), it prevents the user from performing the "Pickpocket" option and alerts them with a game message stating to eat some food. Designed originally for hardcore ironmen to help prevent deaths while pickpocketing.

Closing this as I figured it might be better to add it to the existing Pickpocket Helper plugin instead. Created PR https://github.com/RDieleman/runelite-pickpocket-helper/pull/8 there for it


Github Norms Document

Create a document with proposed Github norms around creating and maintaining issues to share with the team ahead of 1/4 discussion.

Notes from 12/4 1:1 with Jason about this:

Issue guidance:

Naming issues such that they are outcome/deliverable based
Descriptions should be pointed towards specific outcomes as well
How we assign issues
Filling in priority/size - how to appropriately size issues

Have a github norms conversation with the DSE team

How we want to handle issues and how to articulate that with stakeholders
*Josh: Have a first pass recommendation doc to run by the team

Talk to Ian about his process for creating and filling in Issue details
We should be able to look at a list of things by dataset

Structure sprint call to go through by dataset/milestone and discuss things
Then have a cleanup discussion where we rank things by priority

*Schedule a “spring cleaning” type call when everyone is back from PTO

Identify what are the questions we want to ask for every issue and milestone

Is this the right size? Right Priority? Assigned correctly? Needs breaking into smaller pieces
Standardize the issues for each milestone

*In sprint planning and mid-sprint internal calls start using new Milestone view in Github
Areas of improvement around issues:

Not in correct sprint
No size/priority
Not in the right project
No due date
Not assigned a milestone
Assigned to multiple people
Title/description too open-ended and/or issue should be broken into achievable smaller issues
Come up with a way to track these things:

Github views
Coda view - can see milestones not assigned for example

Next steps:

Start using milestone view in weekly and mid-sprint calls internal
Josh to draft a issue guidance document to share with team
DONE - *Schedule call for 1/4/24 when everyone back from PTO to do agree on github norms document (30 min), then go through spring cleanup on existing issues

1/4: Github norms 10:00-10:30 (send document around for review ahead of time, last week of December)
1/4: Spring cleanup  10:45-11:30

Create views in Github/Coda for checking these going forward


Looking for Monthly occurrence scheduling options.  Every 3rd Saturday etc..

Hi, I have fully deployed the Start/Stop VMs during off-hours overview (V2) solution in my environment.  However I am not seeing any options for properly scheduling a monthly start stop occurrence.  The options seems to be greatly limited.  I'm only seeing an option to select a particular number value (say the 17th of each month) rather than say the 3rd or 4th Saturday of each month.  Is there any option for this?  Also if we need to schedule a start stop within less than 24 hours I'm not seeing a method to create a single occurrence.  Is there any way to schedule a single occurrence (say I need to schedule a start/stop to run within next 4 hours)?  The only way I'm seeing now is to manually trigger the start/stop logical app.  I have also looked into trying to trigger the logical app using a scheduled task via PowerShell but have not been able to figure out how to get it to trigger automatically.  Please note, I have a lot of PowerShell programming knowledge.  I'm using the sequence start/stop (using tags) and this is working great.  I'm only having issues with the scheduling.

@ISI-kgozdzik - Thanks for using Start/Stop V2 solution. Currently we don't have much flexibility to customize the existing functionality of recurrence task. You can certainly reach Azure logic app support via portal to get further inputs on this.
Below are few links to check on the Logic apps recurrence task.
Example 1
Example 2

Closing this for now and please reach out if you need further assistance


updated 3.17.2

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Sorry for noise, didnt know it would auto ping you.

@conda-forge-admin, please rerender


Proofs made using unshelve are forgotten afterward

See the following minimal example
open FStar.Tactics

assume val predicate: int -> Type0
// If we add the SMTPat, then `test` succeed
assume val predicate_is_true_lemma: x:int -> Lemma (predicate x) //[SMTPat (predicate x)]

// The dumps allow us to see that the tactic is called correctly in every case
val prove_predicate: unit -> Tac unit
let prove_predicate () =
  dump "goal";
  apply_lemma (`predicate_is_true_lemma);
  dump "goal solved"

assume val test_function1: x:int -> squash (predicate x) -> int

// Subtyping check failed; expected type squash (predicate 42); got type unit;
[@@expect_failure]
let test: int = (_ by (
  let proof = uvar_env (top_env ()) (Some (`(squash (predicate 42)))) in
  unshelve proof;
  focus (fun () ->
    prove_predicate ()
  );
  exact (mk_e_app (`test_function1) [(`42); proof])
))

// Found by Lucas Franceschino
let test_workaround: int = (_ by (
  exact (mk_e_app (`test_function1) [(`42); (`(synth_by_tactic prove_predicate))])
))

The workaround is a bit weird because the tactic generates a term, which then executes another tactic.

After a bit of debugging, it appears that what is going on in the issue of @TWal is that the unification variable loses its refinement. When solving the goal with prove_predicate, at some point, the unification variable is retyped as unit, with it's refinement (predicate x) as an additional proof obligation.
Here are some examples showing a bit more this behavior:
open FStar.Tactics

// (Some utilities)
// Are two term equal?
let eq_term t1 t2 = compare_term t1 t2 = FStar.Order.Eq
// Asserts the string representation of a term matches a given string
let assert_term_is (t: term) (s: string): Tac unit =
  let t = term_to_string t in
  if t = s then ()
  else fail ("[assert_eq_string]: expected [" ^ s ^ "], but got [" ^ t ^ " ]")
// Same thing for multiple terms
let assert_terms_are (t: list term) (s: list string): Tac unit
  = if List.Tot.length t = List.Tot.length s
    then iter (fun (t, s) -> assert_term_is t s) (zip t s)
    else fail "[assert_terms_are] mismatch lengths"
// Same thing for proof states
let assert_ps_is (s: list string): Tac unit
  = assert_terms_are (List.Tot.map goal_type (goals ())) s

// First, we solve a goal [x:'a{ref x}] using [exact] with a value that has the exact type [x:'a{ref x}]
// This work correctly, the unification variable [uv] carries the refinement after the goal being solved
let f1 (ref: 'a -> Type0) (wit: 'a {ref wit})
  = assert True by (
  let t = quote (x:'a {ref x}) in
  let uv = uvar_env (cur_env ()) (Some t) in
  let getType uv: Tac _ = tc (cur_env ()) uv in
  unshelve uv;
  guard (eq_term (getType uv) t);
  exact (quote wit);
  assert_ps_is ["Prims.squash Prims.l_True"];
  assert_term_is (getType uv) "wit: 'a{ref wit}"
)

// Here however, the proof that [wit] respects the refinement [ref] is given by [proof]
// We can observe that [uv] is retyped with type [a], while [ref wit] is added as a new goal
let f2 (ref: 'a -> Type0) (wit: 'a) (proof: ref wit)
  = assert True by (
  let t = quote (x:'a {ref x}) in
  let uv = uvar_env (cur_env ()) (Some t) in
  let getType uv: Tac _ = tc (cur_env ()) uv in
  unshelve uv;
  guard (eq_term (getType uv) t);
  assert_ps_is ["x: 'a{ref x}"; "Prims.squash Prims.l_True"];
  exact (quote wit);
  assert_ps_is ["Prims.squash (ref wit)"; "Prims.squash Prims.l_True"];
  assert_term_is (getType uv) "'a"
)

Thus, following this observation, the module from @TWal can be rewrote as:
assume val predicate: int -> Type0
// If we add the SMTPat, then `test` succeed
assume val predicate_is_true_lemma: x:int -> squash (predicate x) //[SMTPat (predicate x)]

val prove_predicate: unit -> Tac unit
let prove_predicate () =
  dump "goal";
  exact (quote (predicate_is_true_lemma 42));
  // apply_lemma (`predicate_is_true_lemma);
  dump "goal solved"

assume val test_function1: x:int -> squash (predicate x) -> int

// Subtyping check failed; expected type squash (predicate 42); got type unit;
let test: int = (_ by (
  let proof = uvar_env (top_env ()) (Some (`(squash (predicate 42)))) in
  unshelve proof;
  focus (fun () ->
    prove_predicate ()
  );
  exact (mk_e_app (`test_function1) [(`42); proof])
))

Though it's not very intuitive.


[do not merge] Supersede previous unpublished/published items when unpublishing/publishing

This fixes a bug whereby the unpublish command was
finding the first ‘published’ or ‘unpublished’
item rather than the one with the highest
user-facing version.
This fixes the Specialist Publisher bug here:
https://trello.com/c/Ur355YbL/174-bug-cannot-unpublish-something-more-than-once-small-mediumish

Looks good to me

Closed in favour of https://github.com/alphagov/publishing-api/pull/405.


Adding unity catalogs

Fixing #11

Codecov Report
Patch coverage: 77.02% and project coverage change: -0.52 :warning:

Comparison is base (f02ff35) 81.39% compared to head (b1ff0dc) 80.88%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #22      +/-   ##
==========================================
- Coverage   81.39%   80.88%   -0.52%
==========================================
  Files          16       18       +2
  Lines         559      633      +74
==========================================
+ Hits          455      512      +57
- Misses        104      121      +17

Impacted Files
Coverage Δ

...thon/resources/workspace/unity_catalog/catalogs.py
65.71% <65.71%> (ø)

...hon/api_client/workspace/unity_catalog/catalogs.py
86.48% <86.48%> (ø)

...ython/api_client/workspace/unity_catalog/client.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Add minimally configurable constants

Constants can be set by changing the exported file in src/constants/index.ts.
Added a helper script to simplify this process for CI, etc.
usage example: ./scripts/set_constants minimal

TBH this feels dirty and I don't think it will work on long term. Consider scenario where later on you would like to roll-out custom testnet configuration from toml file. We won't be able to overwrite these constants.
Ideally we would have these constants in config singleton that would on start load values from toml file (mainet.toml, minimal.toml or user provided toml) and fill rest with default values. I know this includes a lot of refactoring because of how we use constants now so I'm fine with this PR. But I assume it will be harder to fix these as we add more code.

It would definitely be nice to support a wider range of user-configurable constants. I went in this direction for now because it requires very little change to our structure, and very little rework is required if we want a different solution.
With our current structure, having constants updating at runtime unfortunately hurts the ergonomics of lots of our code, but most critically our ssz types. In the case of the ssz types, several constants are being used as compile time constants, not a configurable singleton object. Changing constants to be a singleton would require the types to become at minimum a configurable singleton object as well.
At the most extreme, it would require us to import types via a function call and/or pass around a run-time configuration object to our pure functions that use types/constants.
Eg:
import constants from "./constants";
import getSSZTypes from "./types";

function pureFunction(buf: Buffer) {
  use(constants.CONSTANT);
  return deserialize(buf, getSSZTypes().BeaconBlock);
}

// or

function pureFunction(buf: Buffer, c: ChainConfig) {
  use(c.CONSTANT);
  return deserialize(buf, c.BeaconBlock);
}

Perhaps this isn't so bad, esp passing around a chain config.
But practically, all constants are known at compile time, and will not (also should not) be changing at run time. There will probably be a few known presets that we want to support (for testing, testnet and internal purposes), but all production users of the canonical chains will be using the same constants.

travis hung?


Textfield submit

im going to close this since we have an alternate method in core now - thanks though!


Use poetry to manage package and make it easy to publish to PyPI

This library is exactly what I exactly wanted and exactly what I was trying to create. The design philosophy and motivation are very close to mine. I hope I can help in some small way.
Main points of modification
Added some files to deal with poetry tests and measuring code coverage
Added files:

pyproject.toml: a config file for poetry
poetry.lock: auto-generated lock file by execute poetry lock
tox.ini: a config file for tests and measuring code coverage

Usage
# 1. Run tests using tox through poetry env
$ poetry run tox

# 2. (Optional) Run codecov
$ poetry run codecov -t ${CODECOV_TOKEN}

# 3. (Optional) Publish to PyPI
$ poetry publish --build -u ${PYPI_USERNAME} -p ${PYPI_PASSWORD}

Execution history
$ poetry run tox
py38 installed: appdirs==1.4.4,attrs==20.2.0,CacheControl==0.12.6,cachy==0.3.0,certifi==2020.6.20,chardet==3.0.4,-e git+git@github.com:peinan/chika.git@cd93439c059dbe264bad5c2760a71c6ae63c2287#egg=chika,cleo==0.7.6,clikit==0.4.3,codecov==2.1.9,coverage==5.3,distlib==0.3.1,filelock==3.0.12,html5lib==1.1,idna==2.10,iniconfig==1.0.1,jsonschema==3.2.0,keyring==20.0.1,lockfile==0.12.2,more-itertools==8.5.0,msgpack==1.0.0,packaging==20.4,pastel==0.2.1,pexpect==4.8.0,pkginfo==<IP_ADDRESS>,pluggy==0.13.1,poetry==1.0.10,ptyprocess==0.6.0,py==1.9.0,pylev==1.3.0,pyparsing==2.4.7,pyrsistent==0.14.11,pytest==6.0.2,pytest-cov==2.10.1,PyYAML==5.3.1,requests==2.24.0,requests-toolbelt==0.8.0,shellingham==1.3.2,six==1.15.0,toml==0.10.1,tomlkit==0.5.11,tox==3.20.0,urllib3==1.25.10,virtualenv==20.0.31,webencodings==0.5.1
py38 run-test-pre: PYTHONHASHSEED='1062298837'
py38 run-test: commands[0] | poetry install
Installing dependencies from lock file

No dependencies to install or update

  - Installing chika (0.1.0)
py38 run-test: commands[1] | poetry run pytest --cov=chika tests
============================================ test session starts =============================================
platform darwin -- Python 3.8.5, pytest-6.0.2, py-1.9.0, pluggy-0.13.1
cachedir: .tox/py38/.pytest_cache
rootdir: /path/to/chika
plugins: cov-2.10.1
collected 13 items

tests/test_config.py ..                                                                                [ 15%]
tests/test_io.py .                                                                                [ 23%]
tests/test_main.py ..                                                                                [ 38%]
tests/test_parser.py ........                                                                          [100%]

---------- coverage: platform darwin, python 3.8.5-final-0 -----------
Name                Stmts   Miss  Cover
---------------------------------------
chika/__init__.py       1      0   100%
chika/chika.py        221     28    87%
---------------------------------------
TOTAL                 222     28    87%

============================================= 13 passed in 0.16s =============================================
__________________________________________________ summary ___________________________________________________
  py38: commands succeeded
  congratulations :)

Modified class name Config to BaseConfig in README.md
In README.md, BaseConfig was used in the explanation:

Then, wrap the main function with chika.main(BaseConfig),

but Config was used in the code example:
@chika.main(Config)
def main(cfg: Config):

So I unified them to use BaseConfig.

Thank you for your contribution.

BaseConfig: Thanks, I noticed but didn't fix it.
poetry: I'm not familiar with poetry, so I have some questions.

Does it work with conda? For PyTorch, I usually use conda, and I don't want to change this for while.
Isn't poetry.lock for your environement? It seems to include a lot of unnecessary dependencies of chika.

Thank you for your response!

BaseConfig: Thanks, I noticed but didn't fix it.

Oh, is there any reason for this ?

Does it work with conda ?

Sorry, I'm not familiar with conda...As far as I searched, it seems work with each other, but maybe not a good way due to the mess of paths.

Isn't poetry.lock for your environement? It seems to include a lot of unnecessary dependencies of chika.

I added some packages such as pytest, tox, and so on to pyproject.toml's dev-dependency for testing. Poetry locks all the dependencies of the required packages. Maybe that's the reason why you feel the required packages are more than the original. Poetry can show the dependency tree; I attach the result image just in case.

Overall, I'm start thinking that this PR is not a good idea, that affect the environment too much. Do you mind if I withdraw this PR ?

BasicConfig: Sorry, I meant I forgot fixing it.
poetry: I will also try in my environment if it works. Let's discuss it after that.

I'm happy to have your PR and don't want you to withdraw it.

Hi, @peinan! I really appreciate your PR, but because of my concern about poetry, I decided to employ another way. Now, chika is available on pypi, i.e., you can install it by pip install chika.
Again, thank you for your PR, and if you have any feature requests or improvements, don't hesitate to raise issues or send PRs.


Enhance Interaction handling

validate fields, optimize storage, and … improve error handling

Throw an error when an unknown field is provided in CreateConversation or CreateInteraction.
Skip saving empty fields in interactions to optimize storage and improve efficiency.
Throw an exception if all fields in an interaction are empty or null.
Add unit test cases to cover the new validation, error handling, and storage optimization logic.

Description
[Describe what this change achieves]
Related Issues
Resolves #3250

Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[ ] API changes companion pull request created.
[ ] Commits are signed per the DCO using --signoff.
[ ] Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

DCO is missing
Apply spotless


Losing active scroll when dragging the scrollbar

Hi there, firstly thank you for creating this repo it's really useful!
I've noticed that once you click an anchor link, and then try to drag the scrollbar using a 'click and drag' with your mouse, the active link doesn't update anymore when you're scrolling the page. It starts behaving correctly again if you then click another link, or use a mouse wheel or touchpad drag instead. I can reproduce this on the demo page, as well as on my own integration.
Reproduced on Mac using Chrome and Firefox. Interestingly on Firefox the issue only occurs if the content has scrolled fully to the start or end of the page.

Hi @patrick-hurley, I am having troubles trying to reproduce this:

and then try to drag the scrollbar using a 'click and drag' with your mouse

Can you please capture a screen recording using the Demo website?

xHi @patrick-hurley, sorry but I cannot reproduce this:

and then try to drag the scrollbar using a 'click and drag' with your mouse

Can you please capture a screen recording using the Demo website?

Hi @smastrom. Sure, here's a video recording on Mac with Chrome. Hopefully this file type is filetype is ok, if not let me know and I can send a different one.
https://github.com/user-attachments/assets/8b1c8220-7032-47bc-a27f-0d4c4b517c59

Thank you, I'll release a fix soon.


Integration Bones and UI

This fixes small UI issues I caught, has a merge I had to do where I didn't change anything, and adds very bare bones for integration testing suites.

Coverage remained the same at 35.411% when pulling 7d4187f35df6c517bac97cc93f1d57bf7b05eee1 on TheKLARKEN:uitest into f4943376c9ec661e317f691bc07cc0c2e2b5615a on supergiant:master.


Add preview for textures in the inspector

I have added a preview for different format of textures in the inspector.
The DDS format is now render in a BabylonJS Canvas so that you can see it fully if it is a cube texture.
The feature request : https://github.com/BabylonJS/Babylon.js/issues/4589

I'm taking care of that this week!

Still ok?

Sorry, I was on holidays.
I'm looking at this before Friday!


updating error messages

Summary:
Updates error messages on initialization.
Intended Effect:
Trying to make the 3 error messages clearer:

Old:
Rejecting proposed initial value using method user because of gradient failure.
Rejecting proposed initial value using method random because of gradient failure.
Rejecting proposed initial value using method zero because of gradient failure.
Rejecting proposed initial value using method user partially because of gradient failure.

New:
Rejecting initial value:
  Error evaluating the log probability at the initial value.

Old:
Rejecting proposed initial value using method user because of vanishing density.
Rejecting proposed initial value using method random because of vanishing density.
Rejecting proposed initial value using method zero because of vanishing density.
Rejecting proposed initial value using method user partially because of vanishing density.

New:
Rejecting initial value:
 Log probability evaluates to log(0), i.e. negative infinity.
 Stan can't start sampling from this initial value.

Old:
Rejecting proposed initial value using method user because of divergent gradient.
Rejecting proposed initial value using method random because of divergent gradient.
Rejecting proposed initial value using method zero because of divergent gradient.
Rejecting proposed initial value using method user partially because of divergent gradient.

New:
Rejecting initial value:
  Gradient evaluated at the initial value is not finite.
  Stan can't start sampling from this initial value.

New tests added checking the error message coming out of the initialize methods.
How to Verify:
Look at code. Run tests.
Side Effects:
This pull request will force CmdStan's tests to change.
Documentation:
None required.
Reviewer Suggestions:
Anyone.

@bob-carpenter, @rtrangucci, @jgabry, @maverickg: any thoughts on these error messages? It would be easy to change now. I omitted the type of initialization -- it didn't seem important in the error message.

"can not" -> "cannot" or "can't" or "is unable"

Bob

On Mar 25, 2015, at 3:26 AM, Stan buildbot<EMAIL_ADDRESS>wrote:
Refer to this link for build results (access rights to CI server needed):
http://d1m1s1b1.stat.columbia.edu:8080/job/Stan Pull Request/299/
Test PASSed.
—
Reply to this email directly or view it on GitHub.

I think they're better than the current error messages and probability as good as you can do unless there's a way to provide the user with useful suggestions for troubleshooting (but that's hard).

I think the function name initialize_state_value can be changed too as what it's really doing is some kind of validation.  I kind of didn't realized that when I consolidate all those checks into one function.  Also if init_how is not used, we can leave it out.

@bob-carpenter: changed to "can't"
@maverickg: I removed init_how and removed the redundant validation (only in one of the methods)

Are we good with this error message?

Looks good to me


Feature/mysql 8 support

This PR is a copy of #696 but from a local branch.
This PR is to add support for mysql 8.0 to BBR. We would like access to CI so we can make changes as necessary.

Compiling mysql 8 for xenial is not support at the moment. We present the user with an error message if they do try to backup a mysql 8.0 database with bbr compiled against a xenial stemcell.
reference slack thread 1
reference slack thread 2

Thanks to @ohkyle who opened the original PR.

Note for future auditors:
This was safe to merge even without the approval of the CLA bot because Kyle has actually signed the CLA, he just hasn't managed to get that fact into the system yet. There is an easyCLA process running which should have him in the automated system within the next week or so.