nixos/murmur: Add overrideable package switch

Motivation for this change
I introduced an overrideable package switch for murmur, mainly because it can be build with ice or grpc support and different users might want to use different builds on their servers. This makes it easy for everybody to choose.
Tested with the mumble test in nixos.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

different users might want to use different builds on their servers.

You mean you want to use a different version?

different users might want to use different builds on their servers.

You mean you want to use a different version?

murmur has overrides, with this switch they are easy to use in the package. Also there is umurmur, an alternative mumble server that works with similar configuration files.

murmur has overrides, with this switch they are easy to use in the package. Also there is umurmur, an alternative mumble server that works with similar configuration files.

@GrahamcOfBorg test mumble

@GrahamcOfBorg test mumble

@Infinisil is there a chance to get this merged into master? Are there some formal criterion's except that it would save me a deal of pain and I don't see how it could break anything for anyone?

@Infinisil is there a chance to get this merged into master? Are there some formal criterion's except that it would save me a deal of pain and I don't see how it could break anything for anyone?

No problem, pushed it to 20.09 in 6d06166fb72c1adabd93450c49fe525ddd172644 :)

No problem, pushed it to 20.09 in 6d06166fb72c1adabd93450c49fe525ddd172644 :)


Fixed uninstall methods on teams-for-vdi.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/4585
The script was removed 2 months ago:
https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/commit/e3b56d74f88d7af82534a069e8e979af1663959e
And the article was replaced by https://docs.microsoft.com/microsoftteams/msi-deployment. I think we cannot remove MSI from VDI using this method.
Could you confirm @LolaJacobsen @LanaChin? Thanks!

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

This issue has also been reported in https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/4179.

Following up with the PM.

Looks great, @dariomws. Merged.


chore: docs and ci

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


Bugfix - AttributeError: 'list' object has no attribute 'x'

Fixes the following bug (on MacOS)
Traceback (most recent call last):
  File "/Users/Hahner/anaconda3/envs/DENSE/lib/python3.7/site-packages/pyqtgraph/opengl/GLViewWidget.py", line 189, in paintGL
    self.setModelview()
  File "/Users/Hahner/anaconda3/envs/DENSE/lib/python3.7/site-packages/pyqtgraph/opengl/GLViewWidget.py", line 142, in setModelview
    m = self.viewMatrix()
  File "/Users/Hahner/anaconda3/envs/DENSE/lib/python3.7/site-packages/pyqtgraph/opengl/GLViewWidget.py", line 152, in viewMatrix
    tr.translate(-center.x(), -center.y(), -center.z())
AttributeError: 'list' object has no attribute 'x'

Process finished with exit code 1

#15
I have tried it, and it worked for me.


[CSBindings] Look through optional types when trying to validate l-va…

…lueness of the new bindings
When bindings are picked for particular type variable, right-hand
side of the binding might be another type variable wrapped into optional
type, when trying to determine if both sides of the binding have the
same l-valueness it's imperative to look throught optional type of the
right-hand side. Otherwise new binding might be effectively unsolvable.
Resolves: rdar://problem/37291371

@swift-ci please smoke test

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please test source compatibility

@swift-ci please smoke test

@swift-ci please test source compatibility


DOC: Improved docs for jnp.nanmin and jnp.nanmax

Part of #21461

Modified the docstrings and squashed the commits into one.


Details panel spinner looks odd

See video: http://recordit.co/U86TtCFcbA

Looks ok for me.

I am using Chrome. @rade Are you on Firefox?

ha! it's broken for me in chrome. works in FF.


Added a test framework

I was trying to get testing working with async but @TRManderson gave the impression that what I was trying to do isn't possible. So I've done this instead.
Not super happy with it, and very open to suggestions.
Also fixed a bunch of PEP8 stuff while I was there.
Progresses on #254

Just a general comment - this isn't Polymathian. UQCSBot doesn't necessarily follow the Polymathian style guide, and "Because PyCharm" isn't necessarily a good enough reason like it is at work.

I'm all for the style guide changes, but it heavily pollutes the PR-- could it be separated into its own PR?

@TRManderson all of the style changes are PEP8 apart from the requirements.txt. I'd say PEP8 should apply everywhere.
As for the requirements.txt file, it was more a combo of "it messes with pycharm" and "there's no documented reason this needs to exist". Idno what it's for, but I can undo that part if it actually has a use.
@mitchmcdee how keen are you on that? I did all this in one commit, so I'd have to split the changes by hand which would suck a bit

well they're just style changes with no dependencies-- should be easy to just copy paste each file into a new branch and then git checkout -- <file>. Would just make reviewing a heck of a lot easier :^)

@mitchmcdee after many screw ups with interactive rebasing, I've managed to split it into two branches. This one only has the test stuff now.

kek 😂 coulda just done it in a new commit!

Could you add a little blurb to the README explaining how to run tests?

Added readme


Treechop does not work in survival

Hello- No clue what's causing this, surely an incompatibility but I don't quite know which..
latest.log
If I could get help I'd be very glad-

Hi! Could you update to treechop 0.17.8? I’m not sure if it will fix it but there’s a good chance

Hi! Could you update to treechop 0.17.8? I’m not sure if it will fix it but there’s a good chance

0.17.8 is only for 1.19.2 though? And I'm running version 0.18.8, latest on Modrinth at this time-
I could try to rollback.

Sorry, I meant 0.18.8*, oops!

0.17.8 is only for 1.19.2 though? And I'm running version 0.18.8, latest on Modrinth at this time-
I could try to rollback.

The log file showed you were using 0.18.7. Maybe you had both versions in your mods folder?

I checked, I didn't have duplicates..
But, all 0.18.7 to 0.18.8 (available versions) does not work in my modpack for some reason, only 0.18.3

Still a problem to this day, even on latest.. Unsure what's still causing it but, heck
Something in 0.18.7 must've broken-
latest.log

Same here! Works perfectly fine in creative though

Root cause found; Incompatibility with "MultiMine" as mentioned in #293


Features with value side effects not used in explanation search

If you add features that add value side effects to the state they don't seem to be considered in the explanation search process. They bottom out in constants. Reproduce instructions to follow soon...

This may still be an issue but it is related to the FO_Planner which we have plans to deprecate.

Duplicate of #61


Fix the scroll space to right of site

Not sure where it's coming from.  But open /map and scroll right to reproduce.

@saikat Checked the latest version and the issues not there anymore. I think this one's fixed

@saikat can we close this?


Add ability to modify installed program wrap bounds

The wrap source and target cannot change after installing the program
into the PIO at present, even though this is possible with the C/C++
HAL. This is useful when using the same program across multiple state
machines that have different wrap sources and targets.

This is useful when using the same program across multiple state
machines that have different wrap sources and targets.

I have the feeling that changing the state on the installed program isn't the right thing.
Maybe an Installed pogram should remain unchanged and the state machines take a ref type (to the program) that holds the SM specific wrap points ?

This is useful when using the same program across multiple state
machines that have different wrap sources and targets.

I have the feeling that changing the state on the installed program isn't the right thing.
Maybe an InstalledProgram should remain unchanged and the state machines take a ref type (to the program) that holds the SM specific wrap points ?

I agree - have only done it like this since each state machine requires its own instance of InstalledProgram when using PIOBuilder (via unsafe fn share()). So would be interested to see what others think.

I agree with @ithinuel that the set_wrap on InstalledProgram feels a little bit out of place. But at the same time, I don't know a clearly better alternative. @ithinuel, could you explain "the state machines take a ref type (to the program) that holds the SM specific wrap points" a little more? I don't understand what you are suggesting.
And, thinking about it a little longer, I didn't find a reason why this feels wrong. In fact, it seems to be the logical place to do it. After all, an InstalledProgram is just a Program which "was placed in the PIO's instruction memory". It's not yet assigned to a state machine, it's not yet running. So why not provide the same set_wrap method on InstalledProgram as on Program? It doesn't affect the instructions, so it can be done after installing the program in instruction memory.
I wonder if the feeling is caused by some mismatch between the way  the hardware works and our abstraction. When looking at the registers, the wrap points are not part of the program (ie. some part of the instruction space), but of the state machine. Multiple state machines can share a program, but they always have their own wrap points. But in the API, we configure the wrap points as part of the program, because most of the time, for a given program, there is only one set of wrap points which makes sense.

Yes, I think you pointed at the right thing in

the feeling is caused by some mismatch between the way the hardware works and our abstraction

I probably won't have the time to have a serious think about it before jan' 23 so I think we can merge this as a first step towards inabling the usecase.


Add peekable to Rows

I couldn't find a way to peek at the next row without advancing the Rows iterator outside the crate, so I thought I'd submit this PR to expose this functionality.

I didn't check the open PR's beforehand, seems like https://github.com/jgallagher/rusqlite/pull/478 solves this better. I'm closing this PR


Cargo apk returns Error: Failed to parse config when Cargo version inherits from workspace

Since Rust 1.64.0, it is possible to inherit some package properties from the workspace.
This feature seems not yet supported by Cargo apk for the package.version property.
For example, running the command cargo apk run --package my_package in a workspace returns the following error:
Error: Failed to parse config.

Caused by:
    invalid type: map, expected a string for key `package.version` at line 3 column 21

Where the Cargo.toml file of my_package is:
[package]
name = "my_package"
version.workspace = true

We just started using these internally too, and are working on parser support for it.

https://github.com/dvc94ch/cargo-subcommand/pull/23
https://github.com/dvc94ch/cargo-subcommand/pull/24
https://github.com/rust-windowing/android-ndk-rs/pull/360


Updates the missing item types for security API related schemas

Description
This PR updates missing items property for parent properties of array type in the security schema.
Issues Resolved

Related comment: https://github.com/opensearch-project/opensearch-api-specification/issues/233#issuecomment-2228802285

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.

@dblock the recent PRs fail with this

@dblock the recent PRs fail with this

I'm looking into it.

@dblock I have a work-in-progress item for tests. I'll raise a separate PR to add tests for all security related APIs.

@dblock I have a work-in-progress item for tests. I'll raise a separate PR to add tests for all security related APIs.

We want to avoid regressions, so maybe PR the tests first?
Either way this PR should be rebased and pass existing tests.

i'll mark this as draft meanwhile.

Closing this as changes will be covered by #439


Page index issue

When i want to show current index, rightToLeft swipe is not working. If i enter to some text (like Text("Test header"), it's working. Really interesting.
`import Pages
struct WelcomeView: View {
@State var index: Int = 0

var body: some View {
  VStack {
    Text("\(index)")
    Spacer()
    Pages(currentPage: $index) {
         Text("Welcome! This is Page 1")
         Text("This is Page 2")
         Text("...and this is Page 3")
         Circle() // The 4th page is a Circle
    }
  }
}

}`

I've had some time to look at the issue, but if I understood you guys correctly it's working on my end. For example this works and lets me swipe right to left:
import SwiftUI
import Pages

struct HomeView: View {

    @State var index = 0

    var body: some View {
        VStack {
            Text("The current index is: \(self.index)")
            Pages(currentPage: $index) {
                Text("Page 1")
                Text("Page 2")
                Text("Page \(self.index + 1)")
            }
        }
    }
}

struct HomeView_Previews: PreviewProvider {
    static var previews: some View {
        HomeView()
    }
}

Does this solve your problem?


angle limit

Hello, thank you for sharing this amazing project!
I have a question about the function updateVerticalMeasuring(cameraPose: Pose) in the MeasureActivity class.
Specifically, at line 548, I noticed the following condition:
if (normalizedCameraAngle < -pi2 * 2 / 3 || normalizedCameraAngle > +pi2 * 1 / 2) {

Could you please explain the reasoning behind this angle restriction?
Is this meant to address specific edge cases, or does it serve a particular purpose in the measurement logic?
Your clarification would help me better understand this part of the code.
Thank you for your time and for maintaining this excellent project!

If I remember correctly, the limit is chosen arbitrarily, but there needs to be a reasonable limit. The steeper the angle, the higher the imprecision because even the smallest change in angle will result in a big difference in measured length.

Only 5° difference but 1/3 length difference. It gets more and more extreme the closer you are to 90°.


URI policy

COOS currently defines all its artifacts in the http://id.unece.org/def/coos space.
Other possibilities can be considered. For example, Insee uses a different namespace for RDF constructs (ontology components, graphs) and for individuals.
In all cases, it must be guaranteed that the namespaces used are durably reserved for RDF resources.
A document describing the URI policy has to be produced as a deliverable of the COOS activity.

A first version of URI policy is available at https://github.com/linked-statistics/COOS/blob/master/uri-policy.md for discussion.

Dear Franck,
GSBPM is organised in phases (Level 1), split in sub-processes (Level 2)
Example:
Analyse > 6.3 Interpret and explain output (http://id.unece.org/activity/subProcess/6.3)
GAMSO is organised in _Activity area_s split in activities
Example:
Corporate Support > 3.7 Manage Finance (http://id.unece.org/activity/activity/3.7)
How are you representing the URI for the upper levels:

GSBPM Phases;
GAMSO Activity Areas ?

If we apply the domain http://id.unece.org/activity/ to GAMSO and GSBPM resources, it won't obvious to know what resource comes from GAMSO or GSPM. A solution might be to define an generic URI for GAMSO or GSBPM as a conceptScheme and assign the instance resources to their respective scheme.
Best regards,
Christine

We read this suggested URI policy and we agree with this version from the point of view of both content and detail.

I'm not an expert on URI's, so please take this with a grain of salt: I don't like the idea of encoding part of the ontology structure in the domain, as implied here:
domain denotes the business domain relevant for the resource, for example: activities, products, organizations, etc.
Type should probably be good enough to link it to the ontology. Domain then could be used to encode something beyond of what is already modeled in the ontology, e.g. additional information about it, context, etc.
Perhaps something like this: http://id.unece.org/coos/subProcess/6.3

For the domains, it seems our classes can be classified in the three high-level Prov classes: activity, entity and agent. We may want to start with those three domains, see whether they work and if we need further classification.

Yes, that's also a possibility. For now we only have activities, so let's go with that. Rather 'activities', to avoid repetition with type Activity and make it clearer that it is a naming context. I change the policy and Turtle file and close the issue.


Question/Error alert type

Are you going to add support for question and error alert types?

Feel free to create a merge request!


region `UNPLACED_SECTIONS' overflowed by 4 bytes

I'm trying to run ThreadX on a stm32f0 (cortex-m0) chip. And the final step, an error occurs.
region 'UNPLACED_SECTIONS' overflowed by 4 bytes
Here is the sample LD file from official code, why there is something related to CM3???
MEMORY
{
  UNPLACED_SECTIONS (wx) : ORIGIN = 0x100000000, LENGTH = 0
  CM3_System_Control_Space (wx) : ORIGIN = 0xe000e000, LENGTH = 0x00001000
  AHB_Peripherals (wx) : ORIGIN = 0x50000000, LENGTH = 0x00200000
  APB1_Peripherals (wx) : ORIGIN = 0x40080000, LENGTH = 0x00080000
  APB0_Peripherals (wx) : ORIGIN = 0x40000000, LENGTH = 0x00080000
  GPIO (wx) : ORIGIN = 0x2009c000, LENGTH = 0x00004000
  AHBSRAM1 (wx) : ORIGIN = 0x20080000, LENGTH = 0x00004000
  AHBSRAM0 (wx) : ORIGIN = 0x2007c000, LENGTH = 0x00004000
  RAM (wx) : ORIGIN = 0x10000000, LENGTH = 0x00008000
  FLASH (rx) : ORIGIN = 0x00000000, LENGTH = 0x00080000
}

Sorry, I didn't read README carefully.
build_threadx.bat and build_threadx_sample.bat works, I should figure out how to compile with my IDE

Hi @sanbrother - we used the same (M3) linker script for the M0/M3/M4. We're planning on cleaning this up and making much simpler scripts soon!

@goldscott Thank you so much, I am glad to hear that.


date between minValue & maxValue, but can't be selected

I downloaded the lastest version.
This is a very feature-rich widget.
I've been using it for a while.
Now I have a scenarios:
I set minValue: new Date(2016, 02, 06) & maxValue: new Date(2017, 01, 31)
and sure, I can select the day before 2016-02-06 or after 2017-01-31,
but when I want to select 2017-01-01, I'm in trouble.
I try to use a variety of methods but still can't do it.
I want to know how can I select 2017-01-31 ?
Thank you for browsing.

Thank you for reporting issue @Aus0049. We will work on it and provide you with a solution within a week time.

Sorry, with existing implementation of AnyPicker, supporting these minValue and maxValue is not possible. We will try to include logic to support this in future versions. But cannot implement it right now.

thank u very much.
I will always support this widget.
It is indeed very easy to use.
👍 : )

hi have you resolve this bug ?


Change self to this

Don't need self = this inside arrow functions that maintain parent scope.

@robinchesterman I want to merge this, bug a couple of things had to be done. So I applied the changes myself in stead of bothering you: https://github.com/SpoonX/aurelia-auth/pull/26
Thanks for pointing this out!


SHARK Stable Release Tracker (12/2023)

Adding a list of TODOs:

[ ] MultiControlNet

[ ]  resize input image(s) to desired output size
[x] verify correctness of pipeline structure -- are we using the right unet weights?
[ ] enable "controlnet t2i adaptation", i.e. t2i with controlnet adapters ( ? )
[x] zoedepth import doesn't work
[ ] stencil canvas isn't centered in image editor

[ ] SDXL

[ ] LoRAs
[ ] vulkan issues with SDXL-1.0 1024x1024
[ ] controlnet

[ ] fix nod logo

[ ] fix UX regressions caused by gradio version update

can we have sdxl turbo too added?

Done. https://github.com/nod-ai/SHARK/releases/tag/20231205.1064

i get this on rx 7900xtx on default settings on sd xl turbo

@monorimet

When switching to sdxl-turbo, it disables the positive prompt, leaving only the negative one. Instead, I have to write the prompt while in sdxl-base mode, then switch to turbo.

The sdxl-turbo model works mostly fine for me, but trying to run sdxl-base results in
SystemExit: SharkifyStableDiffusionModel.get_unet_sdxl.<locals>.UnetModel.forward() missing 2 required positional arguments: 'time_ids' and 'guidance_scale'
OS: EndeavourOS/Arch Linux
GPU: RX 7900 XT (using amdvlk as mesa/vulkan-radeon is broken)
Settings: 512x512 50 steps

When switching to sdxl-turbo, it disables the positive prompt, leaving only the negative one. Instead, I have to write the prompt while in sdxl-base mode, then switch to turbo.

There's something goofy about the inputs/guidance scale for the SDXL implementation. This is actually intended behavior as a quick'n'dirty workaround but it's not a long term solution. I'll add a task for it.


Make crossterm support controlling multiple screen buffers

Task Description
A terminal has screen buffers to which you can write. Buffers are separated from each other and won't change one and other contents. The main buffer is visible by default. We can go to the alternate screen by using the 'crossterm_screen' crate. When we change stuff on the alternate screen nothing will change on the main screen and in reverse.
This is all nice and well, however, we are limited to the main, and alternate screen. And can't have a second alternate screen. I want cross-term to allow controlling multiple screen buffers.
Task Todo

[ ] Allow the user to Create as many screen buffer as he wants to
[ ] Allow the user to change the active screen buffer
[ ] Allow the user to delete buffers.

Notices

Windows allows us to easily support this because with winapi you can create  a new screen buffer and make it the active  (visible) one. Research needs to be done to accomplish this with Linux.

This functionality should be placed in the crossterm_screen create.

Given the fact that a single alternate screen behavior is baked into xterm, one possible option is to create various Read/Write buffers and update the alternate screen based on what is stored. This would effectively emulate multiple "screen"s by then having the user select which "buffer" to render. This might be less of a library concern rather than a user implementation consideration. Or did you have something else in mind?

No that is a good solution, I was thinking of that as well. We can mimic the behavior of output, input, error streams.


Missing sources for some javascript files

I'm working on getting OmniDB into Debian. To be considered free software, Debian wants source code for everything in a program. That includes the original source for bundled *.min.js javascript files. OmniDB is already pretty good at providing that [*], but we are not quite there yet:
lintian ../omnidb_2.14.0+ds-1_source.changes
E: omnidb source: source-is-missing OmniDB/OmniDB_app/static/OmniDB_app/js/chartist-plugin-axistitle.min.js
E: omnidb source: source-is-missing OmniDB/OmniDB_app/static/OmniDB_app/js/chartist.min.js
E: omnidb source: source-is-missing OmniDB/OmniDB_app/static/OmniDB_app/js/flatpickr.min.js
E: omnidb source: source-is-missing OmniDB/OmniDB_app/static/OmniDB_app/js/jquery-1.11.2.min.js
E: omnidb source: source-is-missing OmniDB/OmniDB_app/static/OmniDB_app/js/jquery-ui.min.js
E: omnidb source: source-is-missing OmniDB/OmniDB_app/static/OmniDB_app/js/jquery.dd.min.js
E: omnidb source: source-is-missing OmniDB/OmniDB_app/static/OmniDB_app/js/popper.min.js
E: omnidb source: source-is-missing OmniDB/OmniDB_app/static/OmniDB_app/js/tooltip.min.js
E: omnidb source: source-is-missing OmniDB/OmniDB_app/static/OmniDB_app/lib/chart/chart.min.js
E: omnidb source: source-is-missing OmniDB/OmniDB_app/static/OmniDB_app/lib/cytoscape/cytoscape.min.js

Would it be possible to provide the files in the next version? Thanks!
[*] which btw is a great difference from pgAdmin4, which has only pointers to the origins of gazillions of minified javascript files it is bundling. Furthermore, the build system and minifier used there is not packaged for Debian, so getting pgAdmin4 into Debian is impossible for now.

Hi @ChristophBerg ,
Thank you very much for this. I will include the missing files and let you know.
Best regards,
William

Hi @df7cb ,
This fix reached master branch now that we released 2.15.
Please let us know if you need any further assistance. Closing this ticket now.
Thanks and regards,
William


Int new

The following shouldn't mutate int:
x = 5
x.__init__(7)
assert x == 5

In general I think __init__ should do nothing for built-in types.


[travis] Save rebar3 cache between builds

This was a big help on chef-backend but doesn't seem to be doing much here.  Might be related to the version of rebar we are on, unsure atm.

:+1:


[SPARK-39784][SQL][FOLLOW-UP] Use BinaryComparison instead of Predicate (if) for type check

What changes were proposed in this pull request?
follow up this comment
Why are the changes needed?
code simplification
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing test

Merged to master. The TPC-DS integration test seems to be still flaky.

Thanks @dongjoon-hyun @cloud-fan


Is maximum length == 1024?

Hi!
I'm using https://huggingface.co/transformers/training.html  to fine tune your model.
At trainer.train() step I'm getting error that input size should be 1024.
After I changed:
def tokenize_function(examples):
    return tokenizer(examples["text_a"], padding="max_length", truncation=True, max_length=4096)

tokenized_datasets = dataset_dict.map(tokenize_function, batched=True)

to
def tokenize_function(examples):
    return tokenizer(examples["text_a"], padding="max_length", truncation=True, max_length=1024)

tokenized_datasets = dataset_dict.map(tokenize_function, batched=True)

It worked

Sorry
I pretrained the chinese-bigbird-tiny and chinese-bigbird-small with block 1024。
https://github.com/LowinLi/chinese-bigbird#模型训练参数
So the parameter of max_length can not be set out of 1024.
Moreover the shape of model position_embeddings is just 1024, it can not work with sequence length exceed 1024.
I am pretraining chinese-bigbird-base with block 4096 and  release it at next month.
Thanks for your attention:)

Hi!
I'm using https://huggingface.co/transformers/training.html to fine tune your model.
At trainer.train() step I'm getting error that input size should be 1024.
After I changed:
def tokenize_function(examples):
    return tokenizer(examples["text_a"], padding="max_length", truncation=True, max_length=4096)

tokenized_datasets = dataset_dict.map(tokenize_function, batched=True)

to
def tokenize_function(examples):
    return tokenizer(examples["text_a"], padding="max_length", truncation=True, max_length=1024)

tokenized_datasets = dataset_dict.map(tokenize_function, batched=True)

It worked

hello, I have release chinese-bigbird-base-4096 and chinese-bigbird-wwm-base-4096 for maximum length == 4096
Thanks for your star

Oh! Great job! Thank you for your effort 👍👍👍


Stops discovering types after encountering generated files

I have discovered that Sourcery seems to stop discovering types after it encounters source files generated by Sourcery.
Given the following file list, sourcery only discovers types in files files A and B.
/source

A.swift
B.swift
/generated (folder containing files generated by sourcery)
H.swift

This is easily worked around by making sure that your generated files are not located under the input directory.

oh crap, that's my bad I was adding ignoring generated sources and made mistake that it prunes old types instead of just continue, fixing now

I've a fix but unfortunately can't release it right now because github keeps rejecting zip file, can you try version: https://dl.dropboxusercontent.com/u/11523469/sourcery-0.3.7.zip

Yes, that fixed it.  Thanks.

zip uploading worked through chrome, 0.3.7 is released now :)
Thank you for reporting the issue and let me know if you find anything else or need a feature


Added explicit start = 0 modifiers to parameters without default binding

This fixes #171.

Will be substituted by a PR on #171 from @tobolar.  (see discussion)
Thank you for suggesting a first solutions and make us think about it.

Not sure #171 is good either, see comment.

I will not accept this PR as is but your latest suggestion of #171 with the assert statement seems good to me.
I have reopened this PR in case you want to do the changes. I can do it as well but I won't do it before Friday. Now I feel my jetlag. I fail to write English... I shall not code Modelica...

substituted by #201 according to discussion in #171


Do not specify timezone in iso8601

datetime function. We do not know if the datetime
is in UTC or not, so we should not assume that it is.
I noticed that inspect also uses the iso8601 function and adds a Z.
This signals to the world that the datetime is UTC even though we do not know if it is or not.

Thank you! The docs were pretty clear though that it expects a datetime in UTC as argument. So I think we should keep the iso function as is BUT we should change inspect to use to_string. Thoughts?

We don't need the timezone specification, so let's remove it.


Recursive collects non-source files

E.g., doctest . -recursive enters .git (which is perhaps fine, although we might want to skip dotdirs?) and attempts to test all sort of files (which is not so fine :smile:).

presumably even without recursion it tests all sorts of files in . (?)
+1 for skipping dotdirs.

I think that's right (probably .git was just the first directory entry).


Exit with error status code when plan preview failed

What this PR does / why we need it:
Plan preview CI send error comment but workflow is still succeeded.
So we have to exit with error status code when exception case.
Which issue(s) this PR fixes:
Fixes #4660
Does this PR introduce a user-facing change?:

How are users affected by this change:

I'm expecting to plan-preview workflow failed when exception case.

Is this breaking change:
How to migrate (if breaking change):

@TakumaKurosawa Thanks for the great contributions to plan-preview codebase 😄
There are some conflicts on this PR, could you update 🙏

@khanhtc1202
I'm not sure why this pull request is closed automatically.
I updated local master branch, then I rebased master branch into a head branch.

@khanhtc1202
Sorry.
Now I understand why😇
There is no changes on my branch, right? 😂


Add option to automatically apply BuyXGetY discount rewards

This PR is a first pass at adding an option to automatically add BuyXGetY discount rewards.
It adds a toggle to the discount create/edit screen to automatically add.
When selected we work out if any rewards are available that haven't yet been added to the cart and if we so we add them. Along the way we keep track of how many we added, so we can decrement/increment if cart values are changed, and if necessary remove the item from the cart when it no longer is eligible.
Devs can check for whether an item was added by discount by checking the meta->added_by_discount array values ... if the sum is the same as the cart line quantity then it was added by a discount and can be displayed differently (if required).
Closes #876

@alecritson can you fix the merge conflict and merge?

@ryanmitchell I had a go at resolving the PR conflicts, if you're happy it's all good then we can get this merged!

@ryanmitchell Looks like something has cropped up from when $affectedLines was added in another PR. Do you think this would be an easy fix?

I think this one needs some work to be brought into link with discount breakdowns - can we leave this for the next release?

I think this one needs some work to be brought into line with discount breakdowns - can we leave this for the next release?

That would be 0.3 now, when we look to update for L10

Thats fine, it just needs refactored a bit and I dont want to rush it and introduce an error

@alecritson when you get a chance could I get a 2nd pair of eyes on this. I've update it to work but I'm not super thrilled about some of how it is working - particularly having to duplicate the pipeline functionality cart line actions. No rush, but would be good to hear if you can think of a better way to make it work.

Currently testing and encountering DivisionByZeroError with a 2+1, quantity in Cart 7 .

Opened a PR for part of the issue: #1235, the PR also fixed the case of 2+1 with linesQuantity 7, maxReward 10. While it doesn't include the exact testcase, it does include a test for cases where 2+1 will end up as a float.
Perhaps when testing the AutomaticReward, we might want to add 2+1 with linesQuantity 7, maxReward 10 to check for the DivisionByZero again.

@ryanmitchell do you think this PR is likely to be continued?

Yeah I'll try and get it updated this week.


Development

Codecov Report
Base: 96.49% // Head: 96.45% // Decreases project coverage by -0.04% :warning:

Coverage data is based on head (c38a605) compared to base (5d3cd28).
Patch coverage: 99.01% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #22      +/-   ##
==========================================
- Coverage   96.49%   96.45%   -0.04%
==========================================
  Files           9       10       +1
  Lines         913      960      +47
==========================================
+ Hits          881      926      +45
- Misses         32       34       +2

Impacted Files
Coverage Δ

now_lms/misc.py
100.00% <ø> (ø)

now_lms/__init__.py
99.55% <95.00%> (-0.45%)
:arrow_down:

now_lms/db/__init__.py
100.00% <100.00%> (ø)

now_lms/db/tools.py
90.05% <100.00%> (ø)

now_lms/forms.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 at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Refactor to make sure all our condition types/statuses/reasons are defined in the appropriate API package.

We have some controllers such as the Supervisor "upstream watcher" controllers that defined condition "reason" codes. These are effectively part of our public API. We should make sure that all these constants are exported from some API package so that anyone importing our API definitions can reliably interact with these fields.
We could also do the same for the Supervisor header names and parameter values related to password logins.

Reviewed on 12-7-21. Seems to be a good first issue for contributors.


Consider using a different color palette under dark mode

The Fortran logo and most of the accent color uses #553276, which is fine with the white background. But under dark mode, the purple color used (#553276) and the background (#121212) does not create a good contrast (see contrast checker). Maybe consider using the color palette from One Dark Pro?

@awvwgk @fortran-lang Please suggest colour theme/pallette for dark theme.
Thanks and Regards,
Henil Shalin Panchal

@han190 Please feel free to recommend palette for dark theme.
Thanks and Regards,
Henil Shalin Panchal

I think #C0ACD2 this colour would be great in dark theme too


firebase_messaging 9.0.1 in background generating two time notification

i am using new version of firebase_messaging 9.0.1. in that i am getting two time notification while app is in background. i have check using print it is calling background handler one time still showing two time notification. then i have removed background handler still two time.

@HetalGohil i face the same issue, how you fix it ? thanks :)


vpcTemplates: ability/documentation to do isolated VPCs with VPC templates (i.e. conneected to Central/shared services vpc, but isolated from each other)

looking at vpcTemplatedConfig, i could not figure out a way to template an isolated VPC that would routhe trugh transient gateway and provide isolated templated networks.   I certainly did not find an example of that under references.
Would certainly like to have such a feature, or, if exist, have an example how to use it (I could not figure out how to configure needed propagations with existing abstracts. )
Thanks!

Hello @snemir2, Thank you for using the Landing Zone Accelerator on AWS solution. The LZA solution currently supports VpcConfig to define a VPC that is deployed to a singled account and region. This might helps you implement a isolated VPCs based upon your requirements.
If you have any other questions, please do not hesitate to reach out to us.


optimize: modify no available server error more clearly, and fixed NP…

Ⅰ. Describe what this PR did

Print expect server service name instant of no available server to connect. when could found server in registry center
Fixed issue of NPE when get server service instances return empty.

Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it

Modify server service group name not default.
Start a service with using registry center.
When service can't find default service, will print log like no available service 'default' found, please make sure registry config correct

Ⅴ. Special notes for reviews

Codecov Report

Merging #1956 into develop will increase coverage by 0.02%.
The diff coverage is 61.11%.

@@              Coverage Diff              @@
##             develop    #1956      +/-   ##
=============================================
+ Coverage      55.29%   55.32%   +0.02%
- Complexity      2408     2409       +1
=============================================
  Files            429      429
  Lines          14279    14283       +4
  Branches        1676     1676
=============================================
+ Hits            7896     7902       +6
- Misses          5675     5676       +1
+ Partials         708      705       -3

Impacted Files
Coverage Δ
Complexity Δ

...ery/registry/consul/ConsulRegistryServiceImpl.java
0.97% <ø> (ø)
1 <0> (ø)
:arrow_down:

...ta/discovery/registry/FileRegistryServiceImpl.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...covery/registry/etcd3/EtcdRegistryServiceImpl.java
11.27% <0%> (-0.09%)
3 <0> (ø)

...very/registry/zk/ZookeeperRegisterServiceImpl.java
64.28% <100%> (-0.29%)
25 <0> (ø)

...eata/core/rpc/netty/NettyClientChannelManager.java
71.05% <76.92%> (-0.12%)
21 <2> (-1)

...o/seata/server/coordinator/DefaultCoordinator.java
48.41% <0%> (+0.39%)
28% <0%> (+1%)
:arrow_up:

...in/java/io/seata/server/session/GlobalSession.java
85.02% <0%> (+0.48%)
68% <0%> (+1%)
:arrow_up:

...server/store/file/FileTransactionStoreManager.java
57.46% <0%> (+0.95%)
29% <0%> (ø)
:arrow_down:


Update registration-json to reconcile regularly

Originating issue: IBMPrivateCloud/roadmap#58510

Update the registration-json ConfigMap on each reconcile loop instead of only creating the ConfigMap if it doesn't already exist.
Reorder handlers so ConfigMap handling runs after Secret handling.
Convert registration-json JSON generation to use Go template instead of string replacement.
Refactor redirect URI-related code for clarity.

/lgtm


Zap mobile from public IP and RTL

Hi I built the bitcoin/ lnd node on raspberry PI (I am Stadicus compliant :-)) and also use the RTL user interface (saubyk)
Got the. Zap IOS app to work on my local network at home, with the following setting in lnd.conf
restlisten=<IP_ADDRESS>:8080 # for RTL
rpclisten=<IP_ADDRESS>:10009
tlsextraip=<IP_ADDRESS>
I used to be able to access with Zap IOS my pi remotely ( with tlsextraip=publicipaddress ) and portforward of 10009 to my Pi, but than RTL does not work
Let me know of a route to a solution you can see, Also tries to open two tlsextraip but that did not fly

I had a similar problem trying to get Joule and Zap iOS to both work. I think the problem was similar to yours that if I added both tlsextraip=<IP_ADDRESS> and =externalipaddress then only one of them would work. What I found is that when looking at my tls.cert file in a viewer it did not contain both ipaddresses. No idea why this happens but as soon as I removed one of the two lines in lnd.conf it would work.

tlsextraip=<IP_ADDRESS> is incorrect. You should add a tlsextraip entry for every IP address that you want to be able to connect with. Eg, if you want to conenect to youer node using <IP_ADDRESS>' and '<IP_ADDRESS> then you should add the following entries:
tlsextraip=<IP_ADDRESS>
tlsextraip=<IP_ADDRESS>

After making the changes, you should delete your tls.key and tls.cert files and restart the node. Then, you must update your connections params to use the new tls cert.

hi, I followed your suggestion but it still does not work from externally accessing (wallet does not connect, have port 10009 forwarded on my router
tlsextraip=external_ip
tlsextraip=internal_ip
rpclisten=<IP_ADDRESS>:10009
restlisten=<IP_ADDRESS>:8080
tlsextraip=<IP_ADDRESS>
Wallet does connect from within WIFI but not from public internet

@walterkok please hop into Slack and ping me there. Will be far easier to debug that way.


[1.x] Adds documentation about named routes

This pull request adds documentation about named routes.

https://github.com/laravel/folio/pull/80


Did Not Connect: Potential Security Issue

Firefox detected a potential security threat and did not continue to qrcodescan.in because this website requires a secure connection.
What can you do about it?
qrcodescan.in has a security policy called HTTP Strict Transport Security (HSTS), which means that Firefox can only connect to it securely. You can’t add an exception to visit this site.
The issue is most likely with the website, and there is nothing you can do to resolve it. You can notify the website’s administrator about the problem.

@HKalbasi Thanks for reporting 👍 . There was an issue with SSL/TLS certificate. Its been resolved now. Let me know if you still face the issue else we can close this.

Fixed. Thank you for quick response!


Accessing markers inside a cluster

I'm trying to hover a document, and pulse the cluster that the points from that document are in, I know I can find the markers inside a cluster if I have the cluster already, but for each document I am just saving the markers. How can I look up the cluster if I know the markers?

HI, Is there any solution without PruneCluster ? I need help as for two location which are separated by 1 mile or less , the cluster for them doesn't zoom in.
Thank In Advance

Hei @anshudaur. I don't really understand, could you open another issue with more details, such as a screenshot for example ?


Update dump_pll.c

Here are the main changes I made:

I added error handling for the open() and mmap() functions. If either of these functions fail, the code will print an error message and terminate the program. This will provide more useful information for debugging if the program fails to run correctly.

I changed the while(1) loop to include a return statement at the end of the program. This is to avoid infinite looping and to ensure that the program terminates correctly.
I added a call to close(fd) at the end of the program to close the file descriptor for /dev/mem and release the resources associated with it.
I added a return 0 statement at the end of the main function to indicate successful termination of the program. These changes improve the overall robustness and readability of the code.

Thanks, but this is a one-off debugging script.


Can't stub $router in unit tests

So I'm writing a test that does something like:
  t.stub($store);
  const $router = { push: t.spy() };
  const data = { email<EMAIL_ADDRESS>password: 'password' };
  const wrapper = mount(Login, { data, globals: { $router, $store } });
  t.stub($store, 'getters').value({ isLoggedIn: true });

And while the $store stub is available on this in the component the $router one is not set. I'm assuming there is some special behaviour with the $router global?
This is a unit test so I'm not installing vue-router or vuex in the tests, I'm stubbing them out using Sinon instead.

If you're using the vue webpage template, I'm the files are parsed by karma which causes vue router to be installed. This is my edited version of the file https://github.com/eddyerburgh/stub-route-in-vue-unit-tests/blob/master/test/unit/index.js. the comments weren't changed, so they are a bit confusing. In the template they require all src files except main, which includes router files that install vue router and set an immutable $router object. Ive changed the code to require all files in  the components dirextory.
Hopefully this solves your issue.

Accidentally closed (I'm  on mobile) 🙃

Interestingly enough, I'm actually running it as an isolated unit test. So no Karma, or PhantomJS, just browserEnv() and tape. this.$router is actually undefined in the tests.

Would you be able to post a minimal reproduction of the test and component?

Yeah I'll try to get something together next week.

Sorry I haven't had time to get back into this. I'm definitely having a vue-router related problem. I'm also unable to stub $route. I don't have this problem with any other global variable name. I can call it route and it works.
As far as I know, I'm not installing vue-router as the only files that are compiled are test files. However it's possible that is happening, but the result of wrapper.vm.$route and wrapper.vm.$router is that they are undefined.

Bah, I just realized I am installing the router at some point in the testing because I have tests that test the router and I'm using tape, so the environment is not entirely reset between tests.
I'll have to play with this a bit more. I wonder if Avoriaz could isolate itself from the global Vue but I'm guessing that would be hard.

Ok I've ensured I don't install Router now I get errors about router-link not being registered.

Ok I read the other issues and figured it out. I need to register a component stub. No problem. I get it now.

I'll write a guide for this at some point

I'm currently unable to coax avoriaz to accept my mock router or router-link stubs, and so the tests insist that the router-link element is unknown. I've tried shallow, stubbing a router-link component manually and other functions all to no avail. If there's a guide that can walk me through this I'd be happy to give it a read.

@chrisnicola what other issues?

@eddyerburgh do you have that guide written o maybe just drop a couple lines in here to help unblock us. THANKS! 😄

This is a guide for vue-test-utils but it's the same concepts — https://vue-test-utils.vuejs.org/en/guides/using-with-vue-router.html


Wiki page suggestions

A couple questions/suggestions about the followign text on https://github.com/TranslatorSRI/CQS/wiki:

Transform the Results such that a new result is added, where CQS ("infores:cqs") is the infores:primary-knowledge-source and Auxiliary (Support) Graphs are added.

First. retrieval source metadata is added to edges, not Results - so I assume this is saying that the CQS will be added as the primary knowledge source of the 'treats' prediction edge created by the CQS.
Second, recall that we had tentatively decided that for KP-defined one-hop predictions, the KP should be the primary source, not the CQS.  Instead, the CQS would be captured as an aggregator knowledge source.   See the table here . . . are we all good with this?

Your comments make sense to me. I updated the wiki accordingly. Thanks, Matt!


[QUESTION] LiteDB 5.0.9 how connection=shared works

Hi,
I need to implement a control to count the number of running occurences of a windows form app written in C# running locally on my PC (.Net Framework 4.5). I thought to use LiteDB , where each instance of MyApp  implements a polling (using Timers) to access a db file for reading and writing its PID (process ID) and last time update.

Here's the code:
class Instance
		{
			public int Id { get; set; }
			public int PID { get; set; }
			public DateTime lastupdate { get; set; }
		}
...
Public void myPolling() {

String CONNECTION= "Filename=c:\temp\mydbfile;Password=abc;Connection=shared";

using (var db = new LiteDatabase(DBFILENAME))
{

	var col = db.GetCollection<Instance>("instances");

	//remove all the zombie instances
	DateTime TimeOut = DateTime.Now.AddMinutes(-1);
	col.DeleteMany(x => x.lastupdate <= TimeOut);

	col = db.GetCollection<Instance>("instances");

	//look for the current PID, if found , udpdates the lastupdate and return
	var result = col.FindOne(x => x.PID == Process.GetCurrentProcess().Id);
	if (result != null)
		{
			result.lastupdate = DateTime.Now;
			col.Update(result);
			return true ;
		}

	//count the remaining istances
	col = db.GetCollection<Instance>("instances");
	if (col!=null && col.Count() >= 2)
	{
		MessageBox.Show("Max instances exceeded:" + col.Count(), "", MessageBoxButtons.OK, MessageBoxIcon.Error);
		return false;
	}

	//create the new istance
	var inst = new Instance{
		PID = Process.GetCurrentProcess().Id,
		lastupdate = DateTime.Now
	};

	col.Insert(inst);
	col.EnsureIndex(x => x.PID);

       return true;
}

In the connection string I set connection=shared but it seems that any instance of MyApp.exe points to its own mydbfile dabatase instead of sharing the same mydbfile.
If I try to change the connection to connection=direct or removing the connection parameter from CONNECTION string, each MyApp instance can share the same mydbfile and I can see the instance inside the db (using LiteDB Studio)

but after launcing other MyApp.exe I get the following error
System.IO.IOException: The process cannot access the file because it is being used by another process
(realexception)
System.IO.IOException: Il processo non può accedere al file 'C:\Users\bnf2\AppData\Local\Temp\xrvtmp' perché è in uso da un altro processo.
in System.IO._Error.WinIOError(Int32 errorCode, String maybeFullPath)
in System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
in System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options)
in LiteDB.Engine.FileStreamFactory.GetStream(Boolean canWrite, Boolean sequencial)
...
Where I'm wrong ? Can somebody help me ?
Thanks :)

@bnfgrg
please edit your question to support syntax highlight by using

```    -your-code-- ```
but you should use
```C#   -your-code--   ```

first, LiteDB can handle multiple connections in shared mode, and you can not do that in direct mode, because of file locks,
so this why you got that exception
second, this is a simple code to handle multiple connections in the same app
 public class Issue1911
    {
        class Instance
        {
            public int Id { get; set; }
            public int Pid { get; set; }
            public DateTime LastUpdate { get; set; }

            public Instance()
            {
                LastUpdate = DateTime.Now;
            }
        }

        [Fact]
        public void ShouldHaveTheSameDbVersionOfSharedConnection()
        {
            if (File.Exists("db.ldb"))
            {
                File.Delete("db.ldb");
            }

            var db = "Filename=./db.ldb;Connection=shared";
            var db1 = new LiteDatabase(db);
            var db2 = new LiteDatabase(db);

            var instances1 = db1.GetCollection<Instance>("Instances");
            var instances2 = db2.GetCollection<Instance>("Instances");

            instances1.Insert(new Instance
            {
                Pid = 12234
            });

            instances2.Insert(new Instance
            {
                Pid = 22234
            });

            var all1 = instances1.FindAll().ToList();
            var all2 = instances2.FindAll().ToList();

            all1.Should().HaveCount(2);
            all2.Should().HaveCount(2);
        }
    }

third, multiple connections from many different apps will cause an issue, because if the instances run so quickly, LiteDB may not have time to commit transactions, so you will think that you have the same old DB version
but the problem is that CPU cycle is incredibly fast compared to Disk IO cycles,
so you just used the wrong option,
the problem you try to solve,  should not use any IO operation, It is very dangerous,
so, you have a few options
use service
create a services project, and connect that service to LiteDB, only that service, then use TCP local connection, to track your instances
and you will use InMemory LiteDB instance, then every 10 seconds or any period you want commit that changes to a file
and only load this file when service starts for the first time, so now you will not wast IO cycles

it is very easy to implement
the usage of LiteDB is for persistence only, if you just need to use fresh versions of instances every time, you run the service,
there is no need to use LiteDB

please read https://github.com/mbdavid/LiteDB/issues/1873 , https://github.com/mbdavid/LiteDB/issues/1752 , https://github.com/mbdavid/LiteDB/issues/1614 , https://github.com/mbdavid/LiteDB/issues/635 , https://github.com/mbdavid/LiteDB/issues/1614 ,
and finally, save your DB to file https://github.com/mbdavid/LiteDB/issues/1646
use Mutex
this is an advanced topic, but you can use it, it is very fast, and it the best if you understand exactly what you try to achieve
there are other options, but it is very advanced, and this is not the place where we should discuss, that, this is the LiteDB repository.

@bnfgrg
please edit your question to support syntax highlight by using

```    -your-code-- ```
but you should use
```C#   -your-code--   ```

first, LiteDB can handle multiple connections in shared mode, and you can not do that in direct mode, because of file locks,
so this why you got that exception
second, this is a simple code to handle multiple connections in the same app
 public class Issue1911
    {
        class Instance
        {
            public int Id { get; set; }
            public int Pid { get; set; }
            public DateTime LastUpdate { get; set; }

            public Instance()
            {
                LastUpdate = DateTime.Now;
            }
        }

        [Fact]
        public void ShouldHaveTheSameDbVersionOfSharedConnection()
        {
            if (File.Exists("db.ldb"))
            {
                File.Delete("db.ldb");
            }

            var db = "Filename=./db.ldb;Connection=shared";
            var db1 = new LiteDatabase(db);
            var db2 = new LiteDatabase(db);

            var instances1 = db1.GetCollection<Instance>("Instances");
            var instances2 = db2.GetCollection<Instance>("Instances");

            instances1.Insert(new Instance
            {
                Pid = 12234
            });

            instances2.Insert(new Instance
            {
                Pid = 22234
            });

            var all1 = instances1.FindAll().ToList();
            var all2 = instances2.FindAll().ToList();

            all1.Should().HaveCount(2);
            all2.Should().HaveCount(2);
        }
    }

third, multiple connections from many different apps will cause an issue, because if the instances run so quickly, LiteDB may not have time to commit transactions, so you will think that you have the same old DB version
but the problem is that CPU cycle is incredibly fast compared to Disk IO cycles,
so you just used the wrong option,
the problem you try to solve,  should not use any IO operation, It is very dangerous,
so, you have a few options
use service
create a services project, and connect that service to LiteDB, only that service, then use TCP local connection, to track your instances
and you will use InMemory LiteDB instance, then every 10 seconds or any period you want commit that changes to a file
and only load this file when service starts for the first time, so now you will not wast IO cycles

it is very easy to implement
the usage of LiteDB is for persistence only, if you just need to use fresh versions of instances every time, you run the service,
there is no need to use LiteDB

please read https://github.com/mbdavid/LiteDB/issues/1873 , https://github.com/mbdavid/LiteDB/issues/1752 , https://github.com/mbdavid/LiteDB/issues/1614 , https://github.com/mbdavid/LiteDB/issues/635 , https://github.com/mbdavid/LiteDB/issues/1614 ,
and finally, save your DB to file https://github.com/mbdavid/LiteDB/issues/1646
use Mutex
this is an advanced topic, but you can use it, it is very fast, and it the best if you understand exactly what you try to achieve
there are other options, but it is very advanced, and this is not the place where we should discuss, that, this is the LiteDB repository.

thanks @AlBannaTechno for your answer. However, if I use connection=shared where each MyApp.exe writes to mydbfile adding a single Instance document , even if I wait for one or two minutes, I cant' see the Instance added in the instances on mydbfile. Should I see the instances added by the MyApp.exe applications ? Why I can't see them ?

thanks @AlBannaTechno for your answer. However, if I use connection=shared where each MyApp.exe writes to mydbfile adding a single Instance document , even if I wait for one or two minutes, I cant' see the Instance added in the instances on mydbfile. Should I see the instances added by the MyApp.exe applications ? Why I can't see them ?


Timed-based moderation commands, e.g. suspend, mute, should override the previous punishment if executed again

Current practice is for staff to check for current active/previous punishments and escalate based on that.  If I am understanding, we should provide a prompt to extend a given punishment if a staff is suspending someone already suspended.  Should also probably mention number of extensions, in case it's like a 3-suspend = ban policy.

Sure, I like the prompt idea. That's actually what I had in mind (a prompt that asks you to confirm extension of times punishment by either pressing the EXTEND button or OVERRIDE button).
I'm also okay with the idea of listing the number of times the punishment has been extended/overridden. I don't want to include any policy in the bot regarding number of extensions or whatever with a higher punishment because then we'll be asked to add some customizability regarding this, and well you know that's tedious; imo if it becomes more severe, staff can handle it themselves.


v1.10.0-rc1

Epinio UI [OPTIONAL]

[x] ( 📝 Manual step ) Tag the epinio/ui. git tag -a vX.Y.Z-a.b.c -m 'vX.Y.Z-a.b.c' git push origin vX.Y.Z-a.b.c LINK

[ ] ( 📝 Manual step ) Tag the epinio/ui-backend (with the same tag) and check the release action. git tag -a vX.Y.Z-a.b.c -m 'vX.Y.Z-a.b.c' git push origin vX.Y.Z-a.b.c LINK

[x] ( 📝 Manual step ) Check the epinio/helm-charts pull requests for the latest updates. Merge the PR. LINK

Epinio

[x] Check the CI status in epinio/epinio

[x] ( 📝 Manual step ) Tag epinio and check the release action. git tag -a vX.Y.Z-rc -m 'vX.Y.Z-rc' git push origin vX.Y.Z-rc LINK

[x] Check the release page for the latest assets and changelog. LINK

[x] Check the epinio/homebrew-tap for the latest version. LINK

Helm Charts

[x] ( 📝 Manual step ) Check the epinio/helm-charts pull requests for the latest updates.  Update the version of the Chart.yaml adding the -rc suffix, and set the artifacthub.io/prerelease annotation to "true". Merge the PR. LINK

[x] ( 📝 Manual step ) Run the epinio/helm-charts release action to publish the latest chart. LINK

[ ] ( 📝 Manual step ) Check that the publication of the chart triggered the epinio/helm-charts epinio docker action to generate a docker image for deploying the latest epiniod image. LINK Trigger manually if needed. Check that the resulting docker image is available in ghcr.io, with the chosen tags LINK.

Release

[x] ( 📝 Manual step ) Edit the latest draft release, set it as "pre-relase" and publish the release. LINK

Yep! Done here https://github.com/epinio/epinio/pull/2553


Added eslint and formatted package

Merge after https://github.com/koajs/koa-router/pull/1 @dead-horse

I'd like to keep the style as original for now, people can migrate PR's from koa-router much easier

Okay I'll make the changes

@dead-horse there was very little consistency in the codebase so it's going to be tricky to create the perfect set of eslint rules. For now, I've added back semi-colons so the pull request should be much more readable. Let me know your thoughts.

Can we get this merged so all pull requests going forward have a formatting style? @dead-horse

@dead-horse ?

Thanks for PR - I will be integrating prettier/xo setup from https://lass.js.org into this project.  If you're interested in this approach and get time before I do, you could practically clone my work/setup I have done in projects like https://github.com/cabinjs/parse-logs.


react-native init can't use in China

react-native init hello
This will walk you through creating a new React Native project in /Users/username/hello

downloading all the time

Is there any other way?

You need to be able to access:
https://www.npmjs.com/
https://bintray.com/bintray/jcenter
Can you please use Stack Overflow to ask questions and tag them with react-native? http://stackoverflow.com/questions/tagged/react-native
This helps us keep track of bugs that need to be fixed in Github issues.

sorry , i spent half hour to download it , and it's ok now , npmjs and bintray is normal to visit


Feature request regarding UE4

I am dying to make a game using Steam Audio, and want to use Unreal Engine 4.19 as the base for all of my changes, I do not wish to upgrade any further, the feature set is perfect for me.
Unfortunately, it tends to take Epic a while to upgrade to each new Steam Audio version and I'm worried that since I want to use 4.19 that I won't get to take advantage of all the sweet new updates.
I was hoping that the latest release with Embree support would be made available for UE4? I can integrate it myself if need be, but it would be nice if there was a way to let people use it for "outdated" UE4 versions as well.
If this isn't feasible then I understand.

https://github.com/EpicGames/UnrealEngine/pull/5940

We have updated the Steam Audio plugin for Unreal 4.24. This updates Steam Audio to 2.0-beta.17. The pull request is at: https://github.com/EpicGames/UnrealEngine/pull/6203. You can grab these changes and merge them back into the Unreal 4.19 branch to use the latest Steam Audio features.
For more information, see https://steamcommunity.com/games/596420/announcements/detail/1589135317217982691.


Account Converter should copy contact's primary address type to address objects and not make work address the default

hub thread here:  https://powerofus.force.com/0D580000027H65N
relevant part:
Caroline Renard (Technology Interpreter for Arts Groups)
It would be useful if the Primary Address Type on Contact was applied during conversion. Have just converted an org with several contacts that had been directly related to organizations (so the address on their Contact record was "Work") and the HH address was blank. Now that they are converted, in the new Household Accounts, the same address appears as "Home" and Default in the Address object. I've used Apsona to update the Address Type on the Address object back to "Work" and to uncheck the Default, but it would be nice not to have to do this bit of post-processing - and it would have completely confused my client if they had noticed (or NOT noticed!) before I did. Thanks!

Included in beta release version 3.43 (Beta 8) and higher

Included in production release version 3.43


unix: check watcher queue in uv_backend_timeout()

Return 0 in uv_backend_timeout() if the watcher queue is not empty.
See issue #1565.

I've read #1565 but I don't understand why this is necessary.
loop->watcher_queue are the file descriptors to add before the next call to epoll_wait() or kevent().
Why should uv_backend_timeout() return 0 when file descriptors are added?  It seems unrelated.

When new file descriptors will be added you need to call uv_run again without blocking. Otherwise you can't get notified of new events (through uv_backend_fd). For example, consider the following main loop:
while (is_running) {
    uv_run(loop, UV_RUN_NOWAIT);
    int timeout = uv_backend_timeout(loop);
    // Run main event loop with timeout as max timeout.
}

Consider what happens if you call uv_read_start() from within a uv callback. Now you really need to call uv_run() again immediately (hence why you should return 0 from uv_backend_timeout).

Sorry for the delay, the notification got buried in my inbox.
Okay, I see what you mean now but this change as it stands would busy-loop on Solaris/Illumos because of these three lines.
I don't care much about that platform but if we ever added a select(2) back-end, it would probably have the same issue, so I think we need to address this.
What might work is to claim a bit in uv_loop_t.flags and set it in uv__io_start() and uv__io_stop() when a watcher is inserted into loop->watcher_queue.  If the bit is set, uv_backend_timeout() returns zero.
It's not completely watertight because all watchers might get stopped again before uv_backend_timeout() is called.  In that case the bit should be cleared again but it might be difficult to implement that without also implementing some kind of reference counting.

This PR hasn't seen action in almost two years so I'll take the liberty of closing it out.


allows gossip pull requests with new contact-info

Problem
Current code is only allowing gossip pull requests with legacy contact-info:
https://github.com/anza-xyz/agave/blob/8c5a33a81/gossip/src/cluster_info.rs#L1958-L1966
Summary of Changes
Working towards migrating to the new contact-info, the commit allows gossip pull requests with both legacy and new contact-infos.

Codecov Report
Attention: Patch coverage is 50.00000% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 81.9%. Comparing base (8c5a33a) to head (c25a53e).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #803   +/-   ##
=======================================
  Coverage    81.9%    81.9%
=======================================
  Files         851      851
  Lines      231480   231479    -1
=======================================
+ Hits       189745   189747    +2
+ Misses      41735    41732    -3

Will the LegacyContactInfo be deprecated?

Will the LegacyContactInfo be deprecated?

Yes that is the plan. The new ContactInfo wraps a couple of the gossip messages into one, including LegacyContactInfo.

Awesome, thank you!


External composable libraries reactivity broken

Describe the bug
Composable libraries that rely on @vue/runtime-core have their reactivity system broken.
For example vue-composable uses @vue/runtime-core as peerDependency.
When used with vite, because of vite uses the runtime.esm-browser it will include all the reactivity system in one file, making the third-party libraries that rely on runtime-core broken.
System Info

required vite version: 0.11.1
required Operating System: Windows 10
required Node version: 14.1
Optional:

yarn version: 1.22.4
Installed vue version: beta.9
Installed @vue/compiler-sfc version: beta.9

Reproduction
Install vite and use this component as App.vue
Fixes
This can be fixed by changing the import on the third-party to vue instead of runtime-core.
Ideally I would expect the runtime-core to be a proxy of vue

vite actually can directly support the esm-bundler builds, so we should just use it.

Now it works ⚡ thanks ⚡


Based on Helm Chart , how to configure the information to connect to S3?

Based on Helm Chart , how to configure the information to connect to S3?
Thanks.

you'll need to configure the s3 plugin

I have successfully used S3, thank you.


Allow for time sway

Some users for various reasons may have their device's time set to a different time than the world clocks time. Normally this would get blocked by AWS unless you specify the offset in the policy expiration and x-aws-date header. This PR allows you to pass the timeDelta option to allow device time to sway.

Sweet... looks good to me, but I'll have to pull this down and check it out later today.
Also, do you mind removing the version bump? I'll probably do that in a separate commit.

Removed the version bump. If you need anything else, let me know.

@benjreinhart have you had a change to test this? Would love to get back to your npm package.

Hey Chad,
Looking back on this, do you mind explaining to me how this helps your situation with users' clocks being different than 'the world clock's time?'
This PR only affects the expiration date sent to Amazon, not the x-amz-date field in the policy. Also, it appears only to shrink the timeframe that is allowable by amazon by setting the expiration date to a time closer to the present (unless you passed negative values in the timeDelta option).
For instance, before this PR, we were taking the current date and adding 5 minutes to it to derive our expiration date, thereby saying 'the request will expire in 5 minutes from now.' After this PR, we do the same thing, except now we're also subtracting the timeDelta option from the 5 minutes, thereby saying 'the request will expire in 5 minutes from now minus the timeDelta option's value (or 0).' So if we gave a value of 60*1000 (1 minute in MS) for the timeDelta option, we'd effectively be saying 'this request expires in 5 minutes minus 1 minute from now.' Our expiration window would be shorter at this point. I'm not sure how this solves the offset from world clock issue you were talking about above.
Do you mind showing me an example of how you're using this option and how it solves your problem?

If you are passing negative values as the offset in order to expand the expiration window, we should probably rephrase this or add some other option to configure the expiration because otherwise the source is a bit confusing (to me anyways). I'd be happy to help sort that out.

I am going off of memory here but from what I remember you want to modify the expiration date because it was using 5 minutes from the devices time. If the device's time is in the past more than 5 minutes then the expiration date will have already lapsed.
The code I am using is:
const deviceTime = Math.round(new Date().getTime() / 1000);
        const serverTime = response.current_time;

        dispatch({
          type: DEVICE_TIME_SWAY_CHANGED,
          timeDelta: (deviceTime - serverTime) * 1000,
        });

Where response.current_time is the current UTC time. Then we pass this number into RNS3.put. Some of our users have their phone set in the future and some in the past (for reasons unknown).


'github-pr-review' reporter is not making suggestion for blank line addition

related to https://github.com/mdn/content/pull/29332#issuecomment-1740072618

I've reproduced it in a test environment. In this test PR for the commit in workflow lint-and-review-docs Prettier is suggesting one blank line addition but reviewdog is not able to make a review comment about it.

I created https://github.com/massongit/content/pull/1 with the same content as https://github.com/OnkarRuikar/content/pull/26.
In this PR, there is no difference in https://github.com/massongit/content/blob/fd57312e70913e5219883ad3455a6caec5c600ee/files/en-us/games/index.md?plain=1#L18.
Prettier expects a difference below to apply this line.

## Port native games to the Web

However, it seems that reviewdog does not add a review comment unless it is an additional line.

I found that it should be due to -filter-mode=added (default).
The inserted blank line is not within added lines. That's why reviewdog doesn't report the problem.
To fix the problem, you can use -filter-mode=diff_context.
Or if you use https://github.com/reviewdog/action-suggester, then it set -filter-mode=diff_context internally by default.


Add "related_notes" slot to "Antibody" class and delete the "AntibodyStatement" class.

These suggested changes were necessitated by the implementation of the new "Note" class, which is intended to replace the "EntityStatement" class.
Even though they are no longer referenced by anything, I left the antibody_note_type_set enum there for future reference. I suspect these will need to become ControlledVocabularyTerm objects so that the proper Note types can be created.

@edwong57 @gildossantos, does this have repercussions on the Expression model notes and statements?

Yes @draciti, changes will be required, but I think you'll still have all the note types you want. They'll just modeled slightly differently on the backend.

👍🏻👍🏻

From: Edith @.>
Sent: Monday, February 14, 2022 11:37 AM
To: alliance-genome/agr_curation_schema @.>
Cc: Raciti, Daniela @.>; Mention @.>
Subject: Re: [alliance-genome/agr_curation_schema] Add "related_notes" slot to "Antibody" class and delete the "AntibodyStatement" class. (PR #82)
@dracitihttps://github.com/draciti , I may have to modify the expression.yaml file.
—
Reply to this email directly, view it on GitHubhttps://github.com/alliance-genome/agr_curation_schema/pull/82#issuecomment-1039478118, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAG4A66BRWY7B4OKGWJBFH3U3FKYVANCNFSM5OMKCYIA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>


Don't support IE?

Don't not support IE?
If yes, Do you have a plan to support it?

Don't not support IE?
If yes, Do you have a plan to support it?


MEG file extensions are not correct

The line https://github.com/INCF/bids-validator/blob/ac2a221b579ec6ca0253f1c0700768a54ad2f972/utils/type.js#L203 contains the file name extensions that are considered valid for MEG files. Some of them are not correct (like _meg.ctf and _meg.itab), whereas others that are valid are not listed (like  .raw and .raw.mhd for ITAB).
For 4D/BTi data it is even more problematic, as those don't use file extensions. I have these example datasets
bti148:
total 69540
-rw-rw-r-- 1 roboos fieldtrip 70814400 Apr 11  2011 c,rfhp0.1Hz
-rw-rw-r-- 1 roboos fieldtrip    14445 Apr 11  2011 c,rfhp0.1Hz.m4d
-rw-rw-r-- 1 roboos fieldtrip     3911 Apr 11  2011 c,rfhp0.1Hz.xyz
-rw-rw-r-- 1 roboos fieldtrip    83344 Apr 11  2011 hs_file

bti248:
total 68704
-rw-rw-r-- 1 roboos fieldtrip   195256 Apr 11  2011 config
-rw-rw-r-- 1 roboos fieldtrip 67725824 Apr 11  2011 e,rfDC
-rw-rw-r-- 1 roboos fieldtrip  2135416 Apr 11  2011 e,rfDC,F,a
-rw-rw-r-- 1 roboos fieldtrip      136 Apr 11  2011 hs_file

bti248grad:
total 2220
-rw-rw-r-- 1 roboos fieldtrip  219608 Oct  5  2010 config
-rw-rw-r-- 1 roboos fieldtrip 1997600 Oct  5  2010 e,rfhp1.0Hz,COH
-rw-rw-r-- 1 roboos fieldtrip   32992 Oct  5  2010 hs_file

Thanks for the report. I will look into it in a few days if nobody else beats me to it

I would appreciate if you could provide some guidance for file-name and file-content handling, as we also need to improve this for bids-eeg, where we have BrainVision vhdr/vmrk/[eeg|seg|dat] triplets. I really I would want to check that the triplet is complete, and that the content is OK (at least, not totally wrong).

I guess the issue of checking the triplet is easy. However, checking the content might be harder if there are no javascript readers. This is an issue that @choldgraf has also been facing for iEEG part of the validator. I am able to help more concretely after 2nd June since I am traveling until then ...

a superficial check on the content might be implemented based on file header magic. The code in https://github.com/fieldtrip/fieldtrip/blob/master/fileio/ft_filetype.m https://github.com/fieldtrip/fieldtrip/blob/master/fileio/ft_filetype.m documents some of the header details for quite some files. This would not allow for a content-wise check, which requires more proper parsing.

On 28 May 2018, at 17:57, Mainak Jas<EMAIL_ADDRESS>wrote:
I guess the issue of checking the triplet is easy. However, checking the content might be harder if there are no javascript readers. This is an issue that @choldgraf https://github.com/choldgraf has also been facing for iEEG part of the validator. I am able to help more concretely after 2nd June since I am traveling until then ...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/INCF/bids-validator/issues/473#issuecomment-392560875, or mute the thread https://github.com/notifications/unsubscribe-auth/AA234_J9qhIO1pVQY2UbVxZ6DdbWxYGvks5t3B5rgaJpZM4T36FJ.


feat(audit): Remove use of storage gaps in upgradeable contracts

closes: #1068
To Test:

[x] Run Local Testnet with deployed contracts by first running anvil_deploy_aligned_contract before anvil_start_with_block_time
[x] Upgrade the contracts by running anvil_upgrade_aligned_contracts & anvil_upgrade_batcher_payment_service
[x] Re run Local testnet and ensure nothing breaks

Closing as issue is deprecated

Issue is not deprectaed

Closing


Error while running scan on Jenkins.

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
Works fine on local machine. Fails when run on Jenkins server.
2017-07-13 13:38:24.851 xcodebuild[27056:33883811] Error Domain=com.apple.CoreSimulator.SimError Code=163 "Invalid device state" UserInfo={NSLocalizedDescription=Invalid device state, NSUnderlyingError=0x7f8d8cfd8990 {Error Domain=NSMachErrorDomain Code=-308 "(ipc/mig) server died"}}
2017-07-13 13:38:24.851 xcodebuild[27056:33883811] Error Domain=IDETestOperationsObserverErrorDomain Code=6 "Early unexpected exit, operation never finished bootstrapping - no restart will be attempted" UserInfo={NSLocalizedDescription=Early unexpected exit, operation never finished bootstrapping - no restart will be attempted}

Testing failed:
Test target MyTests encountered an error (Early unexpected exit, operation never finished bootstrapping - no restart will be attempted)
** TEST FAILED **
[13:38:25]: Exit status: 65

I have the following line in my Fastfile.
ENV['FASTLANE_EXPLICIT_OPEN_SIMULATOR'] = '2'

Similar issues posted with troubleshooting being discussed:
https://github.com/fastlane/fastlane/issues/6968
https://stackoverflow.com/q/34990175/7054813
https://medium.com/@greesce/xcode-framework-testing-headaches-c41db4004930

@PetrosIchor I looked into all three links and I added the $(FRAMEWORK_SEARCH_PATHS) in my test targets. I still keep getting same error while running scan on Jenkins.

I have the same issue, Im trying to find solution, but like @RishabhTayal I tried all of the fixes, but the error still appear. I found the app is install first time, but after that the error start to appear. For some reason the application cant be installed on the simulator.
✅ fastlane environment ✅
Stack

Key
Value

OS
10.12.5

Ruby
2.2.4

Bundler?
false

Git
git version 2.11.0 (Apple Git-81)

Installation Source
~/.fastlane/bin/bundle/bin/fastlane

Host
Mac OS X 10.12.5 (16F73)

Ruby Lib Dir
~/.fastlane/bin/bundle/lib

OpenSSL Version
OpenSSL 1.0.2g  1 Mar 2016

Is contained
false

Is homebrew
true

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
8.3.3

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL
en_US.UTF-8
✅

LANGUAGE
en_US.UTF-8
✅

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.46.1
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

slack-notifier
1.5.1

CFPropertyList
2.3.5

claide
1.0.1

colored2
3.1.2

nanaimo
0.2.3

xcodeproj
1.4.4

rouge
1.11.1

xcpretty
0.2.6

terminal-notifier
1.7.1

unicode-display_width
1.1.3

terminal-table
1.7.3

plist
3.2.0

public_suffix
2.0.5

addressable
2.5.1

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.5.0

babosa
1.0.2

colored
1.2

highline
1.7.8

commander-fastlane
4.4.5

excon
0.55.0

faraday
0.12.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170404

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.0

gh_inspector
1.0.3

uber
0.0.15

declarative
0.0.9

declarative-option
0.1.0

representable
3.0.4

retriable
2.1.0

mime-types-data
3.2016.0521

mime-types
3.1

little-plugger
1.1.4

multi_json
1.12.1

logging
2.2.2

jwt
1.5.6

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
2.8.3

google-api-client
0.12.0

json
1.8.1

mini_magick
4.5.1

multi_xml
0.6.0

rubyzip
1.2.1

security
0.1.3

xcpretty-travis-formatter
0.0.4

dotenv
2.2.0

bundler
1.14.6

faraday_middleware
<IP_ADDRESS>

generated on: 2017-07-13

Even am running into this issue on fast lane.
But this looks to be a bug with Xcode 8.3.2 because running xcodebuild from command line on unit tests target is also failing intermittently with same error.

This error has now been occurring for 99% of the time I run scan on Jenkins.

The same issue for me.
The build fails is using scan:
Testing failed:
Test target MyTests encountered an error (Early unexpected exit, operation never finished bootstrapping - no restart will be attempted)
** TEST FAILED **
[13:38:25]: Exit status: 65

And if I copy paste the xcodebuild command that scan generated - the tests run.

@RishabhTayal have you tried removing the FASTLANE_EXPLICIT_OPEN_SIMULATOR='2' line (as @Antondomashnev did above)? Curious to see if that makes things work any better for you. 🚀

@mpirri I tried the ENV['FASTLANE_EXPLICIT_OPEN_SIMULATOR'] = '2' option in my Fastfile (Also described in the description). Doesn't work.
I believe I had 3 parallel builds enabled on my Jenkins server which might be causing this issue. I reduced the number of parallel builds to 2 now and I don't get the error while running scan anymore.

Glad to hear it is working for you after that Jenkins change!
For what it is worth, the FASTLANE_EXPLICIT_OPEN_SIMULATOR environment variable only needs to be a true/false value ("2" is interpreted as true). Having this set might be causing the Simulator to be killed while running tests by a concurrent build happening at the same time, so if you have difficulty, definitely try the false setting.

Got it. What would be the interpretation for false?

In this case false disables the explicit killing and launching of a new simulator before running the tests.

I mean in Fastfile what would I need to put for false? ENV['FASTLANE_EXPLICIT_OPEN_SIMULATOR'] = '1'?

You could either change it to ENV['FASTLANE_EXPLICIT_OPEN_SIMULATOR'] = 'false' or just delete the line entirely (since the value defaults to false).

Cool. I think we can close this issue now. However I still think we should look into why scan fails when more than 2 jobs are enabled on Jenkins. The build fails even if only one job is running on Jenkins.

@RishabhTayal I'm curious how did you manage to have two parallel jobs on Jenkins as they both need the simulator? I mean if the first job requires iPhone 6 and the second iPad Air 2 for example, how they can work simultaneously?

That's a good question. I am not exactly sure how it was working but it recently started breaking.

From my experience we've dropped parallel jobs because there were plenty of build failures. But if it's not your case I will appreciate any knowledge sharing 😄

I still have a problem and never added the line FASTLANE_EXPLICIT_OPEN_SIMULATOR. Getting the same issue though.

Adding: ENV["FASTLANE_EXPLICIT_OPEN_SIMULATOR"] = "0"  to 'before_all' in the Fastfile fixed this issue for me.


Introduce PySpark Session support ( enables the adapter usage for job clusters)

Resolves #
Resolves: [dbt-spark Issue #272] ,[dbt-databricks Issue #575]
Description
Pull Request Description
Summary
This PR introduces support for defining a PySpark-based connection when using the adapter. This enhancement allows dbt to run as part of a running Databricks job cluster, expanding its usage beyond SQL warehouses or all-purpose clusters.
Background
The Spark session functionality referenced here was first discussed in [dbt-spark Issue #272]. Specifically for databricks ,  the issue was raised here :[dbt-databricks Issue #575]
Key Features

PySpark-Based Connection:
A new environment variable, DBT_DATABRICKS_SESSION_CONNECTION, has been introduced.

When this variable is set to True, a new DatabricksSessionConnectionManager is initialized.
This manager assumes that the dbt code is being executed in the context of an existing Spark session, making it possible to integrate with running Databricks job clusters.

Testing:

A new pytest  matrix feature called session_support was introduced in to the unit tests. When the session support is enabled , the DBT_DATABRICKS_SESSION_CONNECTION env var is set to true and the unit tests are being executed against the new DatabricksSessionConnectionManager

Functional Testing:
Functional tests were conducted using a Databricks notebook.

The notebook programmatically triggered dbt while ensuring the DBT_DATABRICKS_SESSION_CONNECTION variable was set to True.

These tests confirmed that dbt works seamlessly within a running Spark session.

example notebook  code:
`os.environ["DBT_DATABRICKS_SESSION_CONNECTION"] = "True"
res = dbtRunner().invoke(["run","--profiles-dir","/Workspace/Users/doron.kruh@yotpo.com/dbt-dbx-session-test/","--project-dir","/Workspace/Users/user@databrciks.com/dbt-models","--target","prod", "--select","model_to_execute"] )`

Why This Matters

Enables running dbt within existing Spark sessions, providing more flexibility for advanced Databricks workflows.
Expands the range of cluster types supported by dbt.
Supports integration with Databricks job clusters, ensuring compatibility with real-world use cases.

Next Steps

Document this feature for users who may need it.
Verify compatibility with additional Databricks environments as needed.

Checklist

[X ] I have run this code in development and it appears to resolve the stated issue
[X ] This PR includes tests, or tests are not required/relevant for this PR
[X ] I have updated the CHANGELOG.md and added information about my change to the "dbt-databricks next" section.

@dkruh1, we need to discuss internally if we want to take this feature.  I appreciate the effort, and I understand why this feature would be valuable to users, but we need to decide whether we want to take on the maintenance burden of an additional connection mechanism.  Will get back to you shortly.

@dkruh1 after discussion, we will not be taking this feature at this time.  We are focused on ensuring that dbt-databricks provides the best experience for interacting with SQL Warehouses and serverless compute.  As this is OSS, you are free to fork our repo and use your implementation that way.


Add Template for Pull Request

The template must have the following parts:

Title: Pull Request Overview
Summary
Changes Made
Tests Results
Next Steps

Use the following photo as an example:

Is this issue still available? I would like to contribute to this ETA  3days

Can I jump on this task?

Could I try solving this?
Kindly assign

Can I be assigned to this?

Can I try solving this issue?

I would love to take on this issue. I am a member of the Coding Dojo. I am immediately available. @jimenezz22

I'm a Full Stack Developer with two years of experience. I'm also a member of Dojo Coding, where I'm expanding my expertise in Cairo and Web3 technologies. I'm enthusiastic about learning, and becoming a part of this project.
To address this issue, I would take the following steps:

Create the Template File:
I would create a new file in the .github directory named PULL_REQUEST_TEMPLATE.md.

Define the Structure of the Template:
I will adjust the PULL_REQUEST_TEMPLATE.md file to include the required sections.

Test the Template:
I would ensure that the template works by creating a new pull request in the repository and verifying that it automatically populates the description area, allowing contributors to fill in the necessary details.

Let me handle this issue!
I'd like to make my first contribution to this amazing project, I've been on an intesive road map with the help of Dojo Coding CR comunity

Hello, I’m Derian, a software engineer focused on full-stack web development, and I’m currently part of Dojo Coding. I’d like to work on this issue.
Here’s how I would solve the issue:
Create the .github folder in the root of the repository.
Add the template within that folder with the parts described in the issue.
Save and upload the file to the repository.
If I have any questions, I’ll ask in the Telegram group as soon as possible.
I’ll open a PR and wait for your feedback.
I’ll make the necessary changes according to the PR review.

Hi team!
I’d like to work on this issue and contribute to the project. To approach it, I propose adding a checklist under the "Summary" section to help the team quickly identify if the change falls under [Bug fix, New feature, Breaking change, or Documentation update required post-merge]. This is just a proposal, and I’m excited to tackle the issue in a way that best aligns with the project’s structure and goals.
Looking forward to any guidance or feedback you might have!

Hi, I'm Josué Brenes and I'll be working on issue.
I'm Dojo Coding member.
I estimate this will take 1 day to complete.
To resolve this, I would create a Markdown template file (pull_request_template.md) in the .github directory of the repository. This file would include the required sections in Markdown format, making it automatically available in each Pull Request.

# Pull Request Overview

## 📝 Summary
- Closes #(issue)
- Brief description of the purpose of the Pull Request.

## 🔄 Changes Made
- List of code changes, specifying the modules or sections affected.

## 🧪 Tests Results
- Results of the tests executed.
  - Example: "Unit tests: 15/15 passed"
  - Example: "Integration tests: 10/10 passed"

## 🚀 Next Steps
- Pending actions or next steps if applicable (e.g., additional reviews or tests by other teams).

I’d like to help with this. I am very familiar with the documentation architecture. Thank you so much. ETA <6 hrs

I would love to work on this. I have implemented various Pull Request templates. My ETA is 24hrs.

I’d like to work on this

i'd love to work on this, im very enthusiastic about making PR templates, i love doing that on my free time

im a newbie contributor, i'd be grateful if you'd give me the opportunity to work in this.

assigned to you @nov0ax


Could you Vs VQGAN-LC benchmark?

see VQGAN-LC is also big vocab_size quantification. Is it possible to construct an evaluation comparison?

We release the evaluation script and report the standard evaluation metrics (e.g. Reconstruction FID) in Table 1 of README.md. They receive the optimal results of 2.62 rFID at 16 downsample ratio and 1.29 rFID at 8 downsample ratio , while we obtain 1.53 rFID and 0.39 rFID respectively.


docs: copy-constructor vs copy-assignment

https://github.com/cmu-db/15445-bootcamp/blob/e2d462b821e29440da1f038e61589866c32d9dad/src/shared_ptr.cpp#L79-L80
Might use copy-constructor here?

// copy-constructed
std::shared_ptr<Point> s4 = s3;

// copy-assigned
std::shared_ptr<Point> s4;
s4 = s3;

I totally agree with you too.
"std::shared_ptr s4 = s3; " should be copy-construction, because it is the first time s4 appears.
"std::shared_ptr s4;
s4 = s3;"
and this should be copy-assignment.

yep, this looks like a typo. would you please submit a pull request? thanks!


[Release-2.0] Fix 9829 : do not report error using import, export, module augmentation in d.t.s

Fix #9829; this is the PR #9870 before I merge "master"

@mhegazy and @sheetalkamat  have approved


Automated cherry pick of #11146

Cherry pick of #11146 on release-7.7.
/cc  @AbhinavVihan
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: 9fc1a535ba79a1a7882e7707a1a24aa9dd7158f2


[SPARK-7402][ML] JSON SerDe for standard param types

This PR implements the JSON SerDe for the following param types: Boolean, Int, Long, Float, Double, String, Array[Int], Array[Double], and Array[String]. The implementation of Float, Double, and Array[Double] are specialized to handle NaN and Infs. This will be used in pipeline persistence. @jkbradley

Merged build triggered.

Merged build started.

Test build #43623 has started for   PR 9090 at commit 697bd24.

Test build #43623 has finished for   PR 9090 at commit 697bd24.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43623/
Test PASSed.

Making a pass now

Nice, I can't really see anything to correct.  LGTM except for the indentation nit.

Merged build triggered.

Merged build started.

Merged build triggered.

Merged build started.

Test build #43651 has started for   PR 9090 at commit a890d79.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43650/
Test FAILed.

test this please

Merged build triggered.

Merged build started.

Test build #43655 has started for   PR 9090 at commit a890d79.

Test build #43651 has finished for   PR 9090 at commit a890d79.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43651/
Test PASSed.

Merged build finished. Test PASSed.

Test build #43655 has finished for   PR 9090 at commit a890d79.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43655/
Test PASSed.

Merged build finished. Test PASSed.

Merged into master.


src/plugin/regions.js: fix regions display on zoom out in next

Short description of changes:
merge 4ee2b29 in next branch to fix region updateRender on zoom out.
Breaking in the external API:
None
Breaking changes in the internal API:
None
Todos/Notes:
None
Related Issues and other PRs:
see #825

Thanks! 👍


[Candidate 1v1 Bozic Destruction ZH v6] Map uses houses with missing fire ports

Observed behavior
Map uses houses with missing fire ports. Soldiers shoot from 0,0,0. Soldiers in these buildings consistently damage themselves.
Expected behavior
Map uses houses with good fire port setups only or is setup in a way where those houses with broken fire ports do not cause problems.
Screenshots
1

2

3

4

5

6

House 1 was replaced with Tower.
House 2 was removed.
House 4 can still damage itself.
Houses 3, 5, 6 appear to be ok now. They look to be lifted a bit above ground.

updated

Houses 3, 5, 6 appear to be ok now. They look to be lifted a bit above ground.

yes flaten ground did not change the problem the houses are elevated by 0,5


add /v1 for okta issuer

Issue #, if available:
Description of changes:
Checklist:

[ ] Have you successfully deployed to an AWS account with your changes?
[ ] Have you written new tests for your core changes, as applicable?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

i

yeah it is confusing. I will make it simple for now.