chore: remove unnessessary heartbeating readable code

No longer required as we already heartbeat on a 10 second interval while syncs are running.

As discussed, we should emit heartbeats regularly on the read portion of the workflow as well.

This actually does this already. writeObjects does the reads and writes, and we set up the heartbeating before we call that.


Create 0000-01-02-rholdorp.md

bla

Add a description of the changes you've made in the comment box. Feel free to add a description of what you’ve accomplished so far. As a reminder, you have: created a branch, created a file and made a commit, and opened a pull request


pure-prompt: 1.20.4 -> 1.21.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/sindresorhus/pure/releases.
meta.description for pure-prompt is: Pretty, minimal and fast ZSH prompt
meta.homepage for pure-prompt is: https://github.com/sindresorhus/pure
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.21.0 with grep in /nix/store/mys4ac3k424gq73l8fq964axxxd2dpfz-pure-prompt-1.21.0
found 1.21.0 in filename of file in /nix/store/mys4ac3k424gq73l8fq964axxxd2dpfz-pure-prompt-1.21.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
pure-prompt

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

pure-prompt

Maintainer pings
cc @pacien @Pablo1107 for testing.

@ofborg build pure-prompt pure-prompt.passthru.tests


12 Hour - AM/PM

Hey @Ozencb , love the /r/startpage you have created here.
I have ended up building a website using this repo of yours, and it is now my standard homepage.
I did want to share that I have noticed one issue with the AM/PM switch-over around noon (and perhaps midnight, but I have not paid attention to it at that time of day).

Pictures are attached, explanation of what I have noticed is below.

So essentially, once 12 noon hits, the website continues to show AM until 1:00 then it switches over to PM.

I just am not sure what I would need to change in the .js to fix this myself, so figured I would reach out and see what you think.

Thanks!

Thank you for reporting the bug. It seems to be working now.

Thank you for reporting the bug. It seems to be working now properly now.

Not a problem.
Awesome! I missed the time frame today, but I will keep an eye on it tonight/tomorrow and report back if I see anything else.
Thanks again @Ozencb!

Great! If you want to reproduce the bug at once you can simply change your OS time settings and that should do it.

Good Point. Just put this same fix into my site, and seems to be all set after I just tested! Thanks again.


Why does my bot freeze once i add it to a group chat

I created the simple IO test code for my bot and it works pretty well... until i add it to a group chat. After that it's just completly frosen. I can create a new bot with the same code and it works good but again after i add it to group chat it became broken.
It just probably something with my code, i am newbye with tg bots stuff.
static void Main(string[] args)
        {
            var client = new TelegramBotClient("5663476931:AAE2lipsUvo6zUYyHPKHEcQl5oxU_UxOI_Y");
            client.StartReceiving(Update, Error);
            Console.ReadLine();
        }

async private static Task Update(ITelegramBotClient bot, Update update, CancellationToken token)
        {
            var message = update.Message;
            if (message.Text != null)
            {
                if (message.Text.ToLower().Contains("test"))
                {
                    //ChatId id = 449873501;
                    await bot.SendTextMessageAsync(message.Chat.Id, "Answer");
                    return;
                }
            }

        }

I attached my code, so anyone can simply create a new random bot and replace API key to test the problem

Please follow issue template
If you want to chat, feel free to join our group.


navigationBarTitleImage not working in v4

Version

react-native-router-flux v4.0.0-beta.17
react-native v0.47.1

Expected behaviour
The image should appear in the navBar
Actual behaviour
The app crashes. Seems like the import for 'Image' is missing in the NavigationStore.js

same here


Context menu (Right Click)

Add refresh (F5) and a create submenu>>  NEW FOLDER, NEW TEXT FILE ETC....

those options are already in the context menu

I see it now.. must click on empty space to pull up that menu...
Great product..
Thanks!
On Sat, Jun 27, 2020 at 12:59 PM Richard<EMAIL_ADDRESS>wrote:

those options are already in the context menu
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/files-community/files-uwp/issues/1269#issuecomment-650586107,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIC7PT52ZV3BLHV5KYG4KPDRYYQQVANCNFSM4OKDLYJA
.


Contexts + CdnInfoContext & Gw2ClientContext

Added base structure for new ContextsService.  Also added two contexts CdnInfoContext and Gw2ClientContext which demonstrate detecting the Client version of the application using the Mumble Link service to detect the current build ID and then the asset CDNs to check which client we are using based on that current build ID.
This provides some of the potential implementation currently being discussed in #62.
Additionally, if accepted, this contains contexts that would allow us to more easily solve #50.

Resolved bug with CdnInfo where parsedSuccessfully could be true if the number of values was not 5. https://github.com/blish-hud/Blish-HUD/pull/106#discussion_r318001999
Reduced complexity of "Is______ClientType" function.  https://github.com/blish-hud/Blish-HUD/pull/106#discussion_r318009487 & https://github.com/blish-hud/Blish-HUD/pull/106#discussion_r318009731
Fixed bugs in CdnInfoContext as well as Gw2ClientContext.
Contexts can now be unloaded.
Gw2MumbleService now has an event "BuildIdChanged"
OverlayService now, in addition to TacO, checks to see if the running client is the standard client or the Chinese client (moves the CornerIcons over if it is the Chinese client).

@greaka While I think the primary focus of this PR is the Contexts implementation, I think it'd be good to have Lei test the artifact that is created from this build and ensure the Blish HUD CornerIcon moves over as expected on a real Chinese version of the client.  I did a good amount of testing with Fiddler to fake the build ID that is returned and was able to confirm it should work.  Also ensured that it would fallback gracefully if something fails with the request (and logs are much more detailed with why it failed, now).
If you like this latest implementation, I can finish writing up the XML docs for it and mark it ready for a true review.
Once we get this merged, we can start adding things like the FestivalContext, which I am fairly excited about.  🎉

@greaka Please re-review the implementation when you have a chance.  I believe I have implemented everything now and have put in XML documentation where appropriate.  I've also added a State member to Contexts which indicates if the Context is loading, ready, etc.
To prevent anybody from unregistering a Context, I added a ContextHandle which can be used to expire a Context.  The ContextHandle associated with a Context type is returned when you call RegisterContext.  The full workflow would look something like:
// Register the context when the module is loaded
var myContextHandle = GameService.Contexts.RegisterContext(new ExampleContext());

// Expire context when the module is unloaded
myContextHandle.Expire();

Also, I received confirmation that this implementation has fixed Lei's problem!  🙂
https://cdn.discordapp.com/attachments/568470477543178261/617894300327477249/jt_2019-09-02_09-30-59.png

I have multiple nitpicks in case the contexts are meant to be thread safe.
Having them threadsafe is probably a nice to have for now though

Agreed!


setWorldOrigin method fails

[x] Review the documentation: https://docs.viromedia.com/
[x] Search for existing issues: https://github.com/viromedia/viro
[x] Use the latest ViroReact release: https://docs.viromedia.com/docs/releases

Environment
Please provide the following information about your environment:

Development OS: macOS 10.13.6
Device OS & Version: iOS 11.4.1
Version: ViroReact 1.0, react-viro 2.9.1, React Native 0.55.1
Device(s): iPhone X

Description
Calling setWorldOrigin with an array of 3 numbers for the ViroARSceneNavigator doesn't work according to the documentation.
Reproducible Demo
Create a new Viro app with react-viro-cli, add this to HelloWorldSceneAR.js:
componentDidMount() {
  setTimeout(() => {
    this.props.sceneNavigator.setWorldOrigin([0.0, 0.0, -2.0]);
  }, 5000);
}

After 5 seconds, there is an error saying "JSON value '(
0,
0,
"-2"
)' of type NSMutableArray cannot be converted to NSDictionary"

Thanks, that works!


Unable to build

14:18:47 $ env GOOS=linux GOARCH=amd64 go build -o gs .
# github.com/ceocoder/apt-gcs
../../../goworkspace/src/github.com/ceocoder/apt-gcs/init.go:93:22: undefined: google.CredentialsFromJSON

I did a go get on all imports listed in init.go but same error, any ideas?

nvm, followed your github actions workflow and it worked :)


expanding arithmetic block

as a user I would like to use one block to chain similar expressions

var1 * var2 * var3
var1 + var2 + var3

to reduce total number of blocks needed to write longer math expressions arithmetic block should be able to extend to provide additional inputs for certain well formed expressions.

I could imagine a button on the right side the block maybe a little + sign. when you add another input maybe a - sign would appear to remove the input. Maybe the would both be there but they would be grayed out when not available

Seems like an enhancement to me (albeit a good one), so I added that label.

Going through the code and exploring ideas for this, one question that has popped up, if we add the enhancement to chain multiple expressions how will this effect the "Commutative Vector Math Rules" that currently exist?

Alternate proposal:

Pro: extensible
Con: order of operations confusion

Option number 4 and 5 both are going to be viable in a measurable way of both feasibility and importance to user, and can cater to both Content Creators and Program Manipulators. I will build both blocks to implement and test both use case.

It would appear that @eah13 idea for the expanding arithmetic block is not currently possible with the Blockly build. While i believe this would be the prime solution and that Google should consider implementing it as well, it is not possible to create a block with a combination of an 'inline' style input (the inset block inside the outer block) and an 'external' style block (the outside connection between two blocks, commonly seen when connecting an input and output block)

@dube1503 bummer.  We should talk today about whether we can put our heads together to make this possible in our fork or fall back on another approach.


output

someone asked about saving data frames to specific cell locations which you can do in Stata with putexel
maybe this is something people would need more often and we could add to data output?

we can totally backlog this

It doesn't look like there is a solution exactly like putexcel, but this is maybe the closest I could find: https://stackoverflow.com/questions/29260914/export-data-into-specific-cells-in-excel-sheet


Add tests on sending 0x00

Seems that during packet cmd sending i faced issue when sending 0x00 and got rx_err triggered to 1:

We have to ensure that there is no error while sending a valid set of bits (data bits + parity)

Test was added but no error got in this test


Update pipeline compatibility and fix output object types

PSGSuite already supports pipeline, but parts of it can be somewhat non-intuitive to use (i.e. piping Get-GSUser -Filter "IsAdmin -eq '$false'" | Update-GSUser -Suspended:$false would not actually unsuspend all the admins, as it would attempt to pass the entire User object and not just the PrimaryEmail)
Also, this will simultaneously remove the Select-Object calls after each object, instead using Add-Member to not break the object type.
All in all, here's what's planned:

Replace Select-Object with Add-Member:

# From this...
$userObj |
	Select-Object @{N = "User";E = {$User}},*

# To this...
$userObj |
	Add-Member -MemberType NoteProperty -Name 'User' -Value $User -PassThru

Override the built-in ToString() method on the object:

$userObj |
	Add-Member -MemberType NoteProperty -Name 'User' -Value $User -PassThru  |
	Add-Member -MemberType ScriptMethod -Name ToString -Value {$this.PrimaryEmail} -PassThru -Force

updates deployed in v2.6.0


App crashes when run from cmd-line (as opposed to start)

Describe the bug
App crashes when run from cmd-line (as opposed to start)
Steps to reproduce the bug

Steps to reproduce the behavior:

Create a new C# project using instructions at https://aka.ms/winui (Try WinUI3).
Use MSIX packaged app template
Launch sample app from Start Menu <- OK
Right click sample app in Task Mgr "Open File Location" to find .exe path
In cmd-prompt run app .exe - e.g. c:\Path\To\MyApp.exe <- PROBLEM app will crash
Dump show crash not in app code but in WinUI code

0d coreclr!RaiseTheExceptionInternalOnly+0x29a
0e coreclr!RealCOMPlusThrowWorker+0x31
0f coreclr!RealCOMPlusThrow+0x8d
10 coreclr!RealCOMPlusThrow+0x99
11 coreclr!TryCallMethod+0x1160f6
12 coreclr!RuntimeTypeHandle::CreateInstance+0x26d
13 System_Private_CoreLib!System.RuntimeType.CreateInstanceDefaultCtorSlow+0x48
14 System_Private_CoreLib!System.RuntimeType.CreateInstanceDefaultCtor+0xd3
15 System_Private_CoreLib!System.Activator.CreateInstance<System.__Canon>+0x3e
16 Microsoft_WinUI!WinRT.WeakLazy<_IApplicationStatics>.get_Value+0xb0
17 Microsoft_WinUI!Microsoft.UI.Xaml.Application._IApplicationStatics.get_Instance+0x38
18 Microsoft_WinUI!Microsoft.UI.Xaml.Application.Start+0x19
19 MyApp!MyApp.Program.Main+0xf2
This blocks automation and GUI Unit Testing say with something like WinAppDriver & Appium to test WinUI3 app
Expected behavior
AppSDK app launched from command prompt does not crash
Version Info
0.5.7
NuGet package version:
0.5.7

Windows app type:

UWP
Win32

Yes

Windows 11 version
Saw the problem?

Insider Build (22141.1000)
Yes

Device form factor
Saw the problem?

Desktop
Yes

Additional context

This is currently not supported, the exe generated by the build is not an unpackaged app, which is a requirement for running an exe from the command line. You should be able to use the IApplicationActivationManager interface to launch your apps though.

This will also work for desktop packaged apps: https://docs.microsoft.com/en-us/windows/uwp/xbox-apps/automate-launching-uwp-apps

This @StephenLPeters this info definitely helps to work around the issue. I would suggest the framework not crash in this scenario, but instead simply exit. Best would be to show window stating info on how to launch it.


Error: When loading with Browserify

I am getting the error with Moment 2.9.0 and 2.10.0 when loading this library via browserify:
Uncaught TypeError: Cannot set property 'format' of undefined
From: https://github.com/jsmreese/moment-duration-format/blob/master/lib/moment-duration-format.js#L202
Thoughts?
Thanks for the work on this, it works wonderfully on the server side!

Seems to work great when loading in the browser using plain ol

The problem here is that when using browserify or webpack, moment doesn't attach to window.
Adding window.moment = require('moment'); or var moment = require('moment'); according to https://github.com/jsmreese/moment-duration-format/blob/master/lib/moment-duration-format.js#L193 prior to requiring this plugin will fix it.
This should be documented better since it is not very intuitive.

Way to pick up on this relatively old issue. Thanks @andreialecu, I'll have to give it a shot.

Sorry for the delay. Changed my github notifications when I started a new job in March and didn't realize I've been ignoring this project for 8 months.
I'm pretty sure that what you need is on my dev branch added in 5698eccd2c1e8717e736146b7a6b464b2625508d. An initializer function is exported so that you can initialize moment-duration-format on any moment instance you wish.
I'll try to get a new version wrapped up and published soonish.

Perfect, thanks @jsmreese

Any chance of getting a new version of this library up on npm that resolves the above issues? (see the above issue from @navtej)


Fix Error in Console in motion-detail

Describe the bug
If the meeting's language is not english and a motion-detail page is reloaded then there is running an Error through the console
How to Reproduce

Create a meeting in the language German (or non English language)
Navigate into the meeting
Create a motion
On the motion-detail page reload the page
Following Error is running through the console

motion-view.component.ts:206 ERROR Error: Parameter "key" is required and cannot be empty
    at _TranslateService.instant (ngx-translate-core.mjs:696:13)
    at _MotionViewComponent.setTitle (base.component.ts:97:49)
    at _MotionViewComponent.onMotionUpdated (motion-view.component.ts:234:15)

Expected behavior
What ever is causing this Error should be fixed

I cannot reproduce the failure. Tested with German meeting, motion, motion w. amendments.

I cannot reproduce it.
So I looked at the code. I'm not sure why this happen: It seems like the ViewMotion.getTitle() delivers a undefined, null or perhaps empty string. That should not happen, as title is required. Perhaps broken data?
I still needs a way to reproduce this.

Tested some more, if you enter only one space into the title and save it, you get this error.

I have seen the error message few times with empty spaces in motion view, but not every time. So, I am sure why this happens.

This could also be a problem with the "new" translate service.

The problem is not singularity in the motion-detail and a bit conceptual.
The same problem also occurs in topic-detail, assignment-detail, participant-detail. And there is a very similar problem in the autopilot.
I'll update the Issue according to the new findings


Victor to follow up.

Hi Victor! Not sure if our devs told you before, we have a fee-sharing program where trades through alpha wallet can earn some fees in KNC. Have you registered an address for alpha wallet?  https://developer.kyber.network/docs/Integrations-FeeSharing/#docsNav
Cool, once you registered, could you pass me your registered fee-sharing program wallet ID address? that will help us track dapp volume growth in reports like this: https://blog.kyber.network/kyber-ecosystem-report-2-bb0fdff5d962  thanks:)

moved to more relevant repo: https://github.com/AlphaWallet/web-service/issues/31


Output Template conditional fields

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
I can see in the Output Template of the documentation that it is possible to set Replace and Default values for the current field. This can be useful to include some values (and additional punctuation or separators) if there is a value, or skip the field entirely if it is blank.
However I'm looking to put some downloads into folders for some extractors but not others. For example, if from YouTube I'd like to include the Uploader in the Output Template, but for Facebook I want to ignore the Uploader.
My current Output Template, in my config file, looks like this:
--output "~/Videos/yt-dlp/%(extractor)s/%(uploader_id)s/%(upload_date>%Y-%m-%d&{} - |)s%(title)s - %(duration>%M&{}mins - |)s%(id)s.%(ext)s"
But I sort of want it to include the %(uploader_id)s/ only if it's YouTube.
Is this possible? I have a feeling it's not, but then the curly brackets/braces are confusing me a little.
Everything else works exactly how I want it to.
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

It's not directly possible. But you can do:
--parse-metadata "%(extractor_key)s %(uploader_id)s:youtube %(yt_uploader_id)s"

to create a new field that contains uploader_id only when extractor_key=youtube. Then you can use that field in -o as %(yt_uploader_id|)s

Thanks. I'll play with this tomorrow and see if I can work it in.

Okay, I was a little delayed there.
Having read up about --parse-metadata and seeing how that worked, I adapted it a little, mainly because "extractor_key" didn't seem to exist but "extractor" did. So now the config file looks like this:
--parse-metadata "%(extractor)s %(uploader_id)s:youtube %(yt_uploader_id)s"
--output "~/Videos/yt-dlp/%(extractor)s/%(yt_uploader_id&{}/|)s%(upload_date>%Y-%m-%d&{} - |)s%(title)s - %(duration*0.01666|)d%(duration&mins - |)s%(id)s.%(ext)s""

I know I can probably smarten up the "mins" bit, but that's because I can't format the duration as minutes easily. For now it works and the question has been answered.
Thanks again for the help!


[Bug Report] Select do not update label when multiple options have the same value

Element UI version
2.2.0
OS/Browsers version
ArchLinux / Firefox 60.0a1 (Nightly)
Vue version
2.5.13
Reproduction Link
https://jsfiddle.net/mmx38qxw/326/
Steps to reproduce

Make a select element with options having same value but different labels (Like country codes provided in the jsfiddle).
Selecting any of the options with same values, won't update the label (many countries have similar country code, e.g. U.S. has country code of  1 but also Virgin Islands, Trinidad and Tobago, Turks and Caicos Islands and they are considered different countries with different iso codes)

What is Expected?
Even when the values are the same, the select element should update label to the chosen one.
What is actually happening?
The label is fixed, and won't get updated upon users select.

Sorry, value must be unique. Select's result will only be controlled by value.
Otherwise the component's perceived state is not controlled solely by data, but also requires previous state (user input). This is against the principles of Vue / React / MVVM frameworks.
For example, if you set countryCode to 1 programmatically,  which one should the component highlight if there are multiple options with value 1.

@wacky6 is right.
@arashm A possible solution for you is https://jsfiddle.net/mmx38qxw/333/ , where value can be an object.


Add calendar device action

Description:
Related issue (if applicable): fixes #26983
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hi @shalini-s,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

You have just committed the scaffold without making any changes. This is incorrect.

hey, i looked through the calendar docs and it seemed like the relevant actions were the same as the default ones - turn on and turn off - are there others i should include?

All the TO DOs are still in the code and the calendar integration has no turn on or turn off commands.


Update Composer dependencies (2024-01-09-00-15)

Visual regression test passed!
View the visual regression test report


Add CV Example

from https://www.youtube.com/watch?v=EHRw_cOSs00

Done with https://github.com/azettl/printcss.examples/commit/dc61a4f5978dc172dfb990e817cd10a8b30af9b7


(internal) release_patch: build v5.9.347

Test plan: automated release PR, CI will perform additional checks

:green_circle: Internal release is ready for promotion!
:warning: Do not close/merge the pull request or delete the associated branch if you intend to promote it.

:green_circle: Release has been promoted, see tag: https://github.com/sourcegraph/deploy-sourcegraph-k8s/tree/v5.9.347.
:no_entry: Do not under any circumstance delete the branch holding the tagged commit (i.e. release-v5.9.347).
:arrow_right: You can safely close the PR and delete its a associated branch.


Feature/customer orders test

DBT Build Status 📖success
Show DBT Build Output

Pushed by: @abhr1994, Action: pull_request


Duplicate VRP served out

Running StayRTR v0.5.0 and noticed a resetting RTR session today:
27076 2023/02/28 20:03:03.344 CET MINOR: RPKI #2001 management RPKI
"Rpki Session state on x.x.x.x changed to down due to fatalErrorCode"

27075 2023/02/28 20:03:03.344 CET CRITICAL: LOGGER #2002 Base A:RPKI:UNUSUAL_ERROR
"Slot A: rpkiValidatePrefix: x.x.x.x:Duplicate V6 prefix 2a02:88d:4004:: prefix len 48"

The output of rpki-client shows one entry for the mentioned prefix in the json file. When issuing a rtrdump (./rtrdump -file debug.json -connect <IP_ADDRESS>:3323) towards StayRTR we can see a duplicate entry in the debug.json file:
{"prefix":"2a02:88d:4004::/48","maxLength":48,"asn":48695},{"prefix":"2a02:88d:4004::/48","maxLength":48,"asn":48695}
We have not been able to reproduce this issue so far.. :)

Are you running v0.5.0 as tagged on docker?  I'm waiting for the v0.5.0 instance to send out the duplicate VRP to be able to determine if the v0.5.1 RTR session remains up. We'll keep it running for a while.
Feel free to close this issue. If we see a re-occurrence with v0.5.1 i'll open up a new one. Thanks again. :)


Testing a network connection?

Would this work at all for showing whether a browser is connected to a local network, as opposed to the Internet?

Please close.


Custom font not working

When i set custom font for MaterialApp not working, in this situation AdaptiveTheme is parent of MaterialApp
When i remove AdaptiveTheme custom font work as well

@hosseinkhojany Didn't understand your issue. Can you please provide more information and some code snippets too?

I Imported your code to my project and edit it and fix it

@hosseinkhojany Can't help you without the sample reproducible code & configuration.

Closing this issue because of inactivity. Feel free to re-open if you're still facing the issue.


Fixed TSLint 'prefer-for-of' and 'no-consecutive-blank-lines' warnings in MSBot

Double blank lines in two different files were reduced to only one for fixing 'no-consecutive-blank-lines'.
Two for structures were replaced with for-of structures as suggested by TSLint for fixing 'prefer-for-of' warnings in MSBot.

Replaced with https://github.com/Microsoft/botbuilder-tools/pull/361


Can't save without deployment URL while adding Projects.

Description
While adding projects the deployed website URL field should be optional in the profile page.
now the save button remains disabled without the deployed URL.
Screenshots

Additional information
No response

I would like to work on it.

@RajKanani1206 Assigned you.

@Debajyoti14

Also, there is another bug, It's working without Project Name. Fixing that too.

@RajKanani1206 Thanks for finding. Fix it.

@Debajyoti14 Fixed already and created PR too.


Include next chunk file

In each file, near the beginning, an encrypted version of the next file path should be included.

Finished


Allow optional validation of the value of the User-Agent and X-Forwarded-For headers during authentication

I added two new options to configure the validation of the User-Agent and X-Forwarded-For headers during authentication.
Mailing list: https://groups.google.com/forum/#!msg/mod_auth_openidc/iockk1U7UUY/CCMtXwEOAQAJ

I'd rather see the state input headers configured through a single primitive that can have multiple values (e.g. "user-agent", "x-forwarded-for") and perhaps make it free format to accommodate for other headers in the future, with a backwards compatible default.

@zandbelt Thank you for the comment.

the state input headers configured through a single primitive that can have multiple values (e.g. "user-agent", "x-forwarded-for")

How about the following a new option?
# HTTP header names that are used as the input for calculating the fingerprint of the state during authentication.
# When not defined the default of "user-agent x-forwarded-for" is used
OIDCStateInputHeaders "<header(s)-separated-by-spaces-and-enclosed-in-double-quotes>

However, I have a concern that the x-forwarded-for header needs to be treated as a special header even if we introduce this new generic option because it needs to pick up the leftmost value as in the current logic below:
https://github.com/zmartzone/mod_auth_openidc/blob/aad5dec6a4d367d9cf630d569923e9964c8e4bf7/src/util.c#L2408-L2411

that's exactly what I had in mind :-) the leftmost header handling doesn't matter in this case: for the state calculation we can use the full header (and avoid using the convenience functions)

@zandbelt

for the state calculation we can use the full header (and avoid using the convenience functions)

If a DNS round-robin load balancer like AWS ELB is put in front of it, wouldn't the IP on the right change during the OAuth2 dance?
In that case, I think if we use the full header value of X-Forwarded-For to calculate the hash, the authentication will fail.

I guess you're right; perhaps always take the leftmost value of every header then, or make it configurable as part of the header value e.g. ":leftmost"

@zandbelt

perhaps always take the leftmost value of every header then, or make it configurable as part of the header value e.g. ":leftmost"

The value of X-Forwarded-For header uses comma + space separator. But other headers might use another separator (it depends on the spec of the header). I think it's hard to configure them in a generic way.
So, how do you think to introduce special handling only for X-Forwarded-For like current implementation?
Or how about introducing the following simple one option for this enhancement?
# Defines whether the value of the header will be used for calculating the fingerprint of the state during authentication.
# When not defined the default is On.
#OIDCStateInputHeader [On|Off]

I would then prefer "fixed" options for x-forwarded-for and user-agent that can be configured with "x-forwarded-for", "user-agent", "none" or "both", with "both" as the default and using the convenience function to access x-forwarded-for

@zandbelt

I would then prefer "fixed" options for x-forwarded-for and user-agent that can be configured with "x-forwarded-for", "user-agent", "none" or "both", with "both" as the default and using the convenience function to access x-forwarded-for

Okay. I modified this commit. Could you check it?

I like the option and the coding is clean but I do have one more comment: instead of seeing two config fields :
	int state_input_header_use_user_agent;
	int state_input_header_use_x_forwarded_for;

it could be just a single byte value:
   apr_byte_t state_input_headers;

with "OR"-ed values:
   #define OIDC_STATE_INPUT_HEADER_USER_AGENT 1
   #define OIDC_STATE_INPUT_HEADER_X_FORWARDED_FOR 2

and default:
  #define OIDC_DEFAULT_STATE_INPUT_HEADERS
 OIDC_STATE_INPUT_HEADER_USER_AGENT && OIDC_STATE_INPUT_HEADER_X_FORWARDED_FOR

much likeoidc_parse_pass_idtoken_as https://github.com/zmartzone/mod_auth_openidc/blob/master/src/parse.c#L701-L731
that style is a bit more terse and it clears the way for adding more headers in the future; if it isn't too much trouble I'd appreciate it if you'd change it.

@zandbelt Thank you for the comment. I revised the commit. Could you review again?

looks good! if you confirm that you've tested this through then please add  add a log entry to the ChangeLog and update the version number in configure.ac to 2.4.3rc0 (you're already listed in the AUTHORS file 👍)

@zandbelt I confirmed this new option in my test environment (base on this Dockerfile). I updated ChangeLog and configure.ac too.


Enforce max zoom

Some requests for high zooms [1] end up producing overflows and generate errors [2]. Can we start enforcing a max zoom, and return 404s for zooms higher than that?
@nvkelso what would be reasonable? 20? 25?
[1] https://tile.mapzen.com/mapzen/vector/v1/all/9999/1/1.json
[2]
File "/usr/local/lib/python2.7/dist-packages/tilequeue/tile.py", line 319, in calc_meters_per_pixel_dim (2 ** (zoom + 8))) | OverflowError: long int too large to convert to float

20
That's the max zoom we set in Leaflet for house styles and usage on 20 is very low:
https://cloud.githubusercontent.com/assets/853051/20083180/a9ebc9a4-a50f-11e6-9884-59a6f03b8c9b.png

(If someday we did indoor mapping tiles this might change, but overstuffing a zoom 18 or 20 tile would probably be fine.)


Pie labels should be outside

Originally created by Tom Smyth at 2016-10-25 11:59, migrated from redmine-#5332

Status changed from New to Closed

By Tom Smyth at 2016-10-25 15:59


List package on conda-forge

Dear SAP team and maintainers @jarus,
Currently, sqlalchemy-hana is available from PyPi. This works fine for everyone managing their dependencies with pip but arguably excludes those working with conda (or makes their lives harder, at least; see also https://pythonspeed.com/articles/conda-vs-pip/).
It would be great if the package, ideally together with hdbcli due to dependencies, could also be listed on the conda-forge repository.
I'd be happy to help with preparing recipes and PRs for conda-forge (more information about the process can also be found here) - please feel free to reach out to me about this.
Thank you for your consideration. 🙏

@JakobEverdingQC I'm sorry for the late reply. conda is not a release channel which is supported by SAP open source projects, therefore for now we don't have any plans to support it


Support to compare Error objects

I tried to support Error object. It is easy to cover almost all cases.
But it is too difficult to make sure that supporting current supported browsers and nodejs.
Refs:

Node.js v9.11.1 Documentation#Errors
Error | MDN web docs

I felt we should avoid to support to compare Error objects.
We might better to put some comments on README.md about that.

Almost same solution: https://github.com/substack/node-deep-equal/pull/48


Failing parsing wikidata ttl dump file

I'm trying to parse the current Wikidata dump and I get:
Unexpected character U+23 at index 102:
http://www.univ-paris3.fr/adminsite/webservices/export_rss.jsp?NOMBRE=10&CODE_RUBRIQUE=&LANGUE=0#KLINK#KLINK [line 10093827]
Can you tell me if this is an rdf4j bug or a Wikidata problem? In the second case I would report to the community ....
Maybe related to issue #1017

Looks like a wikidata issue, since there are two '#'
The RFC (https://tools.ietf.org/html/rfc3986) appendix A mentions that a fragment (the part after the #) cannot contain another '#' (unless it is encoded like %23)

ok, I will report this to the community ....

If you want to parse it, even with errors, you can disable verification by setting the Rio parser setting to false
See https://rdf4j.eclipse.org/javadoc/latest/org/eclipse/rdf4j/rio/helpers/BasicParserSettings.html#VERIFY_URI_SYNTAX

I will try this. Thank you very much!


Adding note about kubectl 1.6 to the dev guide

Also fix line 71.

@MHBauer done

Given that the only thing done before the get is installing the charts with helm, I don't understand why my get would not work with a v1.5 kubectl.
Does helm use kubectl in the background?

@MHBauer answers below:

Given that the only thing done before the get is installing the charts with helm, I don't understand why my get would not work with a v1.5 kubectl.

Get does work with a v1.5 kubectl. Only creates need 1.6. @pmorie has requested that we use 1.6 for all operations to keep things simpler for the user

Does helm use kubectl in the background?

No. It sends rendered templates over gRPC to Tiller, which uses the Go SDK to send operations to the Kubernetes core server.

Totally fine with requiring v1.6 kubectl as client. Still attempting the walkthrough.

@MHBauer cool. Note that this PR isn't intended to fix the walkthrough completely. https://github.com/kubernetes-incubator/service-catalog/pull/593 is, intended to do so.

Assuming consensus to simply require kubectl 1.6 or greater, the oldest commit needs updating to be consistent with that decision.


Proposal of showcase project - PROJECT_NAME

Showcase proposal template
Note: "*" fields are mandatory.

Project Name: The name of the project that will be used as title of the section of the showcase.

Project Name *:  Stellar Duel

Project Description: The description of the project that will be used as a paragraph after the title of the showcase.

Project Description *: The game is a smart contracts on Signum blockchain. Players can choose a fleet and fight each others. The web page connects the players and the smart contract, making transactions much easier.

Project Author: The author of the project that will be informed after the project description. (Keep if blank to don't show the author)

Project Author: Rui Deleterium

Project Links: Links to access the project showcase. These links will be shown after project description.

Project Links: https://stellarduel.space/

Project Images: Images of the project (can be the images with legends, or the links to it).

Project Images *:
https://user-images.githubusercontent.com/54009773/170266796-5aa8c548-5403-4a93-b98e-a1e8c8a1b185.png
https://user-images.githubusercontent.com/54009773/170266813-91f8e119-509e-4669-a1f6-7a0a34821d97.png
https://user-images.githubusercontent.com/54009773/170266826-ca15d811-1f99-40d8-97af-8589ab8722cb.png

Additional Information: Other relevant information.

Additional Information: TuiCSS was a great fit for this project that aimed to be a very simple game to remember the beginning of multiplayer games on the internet during the late 80's.

Feedback: If you want to give a feedback to the project, or a suggestion, please, enjoy this field!

Feedback: Some measures are different when rendering is on mobile or desktop. I tweaked some to be always in 'ch' units. Probably changing most of them to 'ch' would make the design consistent between mobile and desktop.

Thanks! I will open a new issue about your feedback and check the best solution for it in the next releases. Please, just check the showcase page if you liked the way I put the content there. If you have some suggestion, or wants to change something there, say here ;).


Group messages in the same bubble

Hello,
I'm making an iOS tchat application using JSQMessagesViewController 7.2.0, and I'm looking for grouping messages in the same bubble, I found this old issue [https://github.com/jessesquires/JSQMessagesViewController/issues/203] talking about it.
But I'm totally lost on how I can do that.
I known it not really an issue, it's a help request, could you give me some hint on how to do that with UICollectionView.
Thanks in advance.
Jerome

@simpleg For this version of the library, you have a couple of options:

If you want to group messages in the same bubble you will probably need to create your own media item, manually combine messages, and lay them out in a UIView that you return.
If you want to have separate bubbles that appear close together, you remove the top / bottom labels for any cells that should appear grouped by overriding the various delegate methods. You can move them even closer together by changing the collectionView's flowLayout minimumLineSpacing
If you want to have separate bubbles that are grouped and have distinctive shapes, you can create additional JSQMessagesBubbleImageFactory objects and return different bubble images for different index paths in messageBubbleImageDataForItemAtIndexPath

Thank you for your answer I will try :)


Clarify that Get Started is how to download OpenSearch

Description
Existing website makes it hard to know where OpenSearch can be downloaded from, so updating the text.  Note, its kind of wordy, but I am open to suggestions.
Before

After

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

Let's continue discussion on the forums:
https://discuss.opendistrocommunity.dev/t/download-or-get-started-on-opensearch-org/7317/5?u=searchymcsearchface

@stockholmux closing out this pull request that is 2 months old so there is one less thing for me to track, doesn't seem to be movement on this issue.

@peternied I'm just now getting a chance to work on this part, so reopening.

Hi! Any progress on this pull request?


Segmentation fault while counting human genome

I am trying to use squeakr to count kmers from the human genome assembly GRCh38. As suggested in #31 I have taken each chromosome and added dummy quality values to convert it into FASTQ.
I am currently on master 346f581 and I run the following command
./squeaker count -k 20 -t 1 -o ./output human.fq

where human.fq is the FASTQ converted file of the human genome assembly. The output that I get is:
[2019-01-07 11:22:17.136] [squeakr_console] [info] Reading from the fastq file and inserting in the CQF.
Segmentation fault (core dumped)

Any ideas for what I can do to fix this? Thanks!

Hi @Colelyman,
Apologies for the delay in getting around to this.  We know what the cause is here.  The FASTA/Q parsing strategy we use parses the file in "blocks", with the assumption that each block will contain at least one complete record.  Since blocks are large (they can be MB), this is true with sequencing reads (even long reads).  However, when the "reads" are a reference genome, as is the case here, that can cause a problem.  One solution is to make the block size large enough to cover the biggest contig in the assembly (@prashantpandey can point you at the line of code that would need to be changed for that).  A better solution would be to "fix" our parser to avoid this assumption, but we've not had a chance to do that yet.
Best,
Rob

Thanks, @rob-p. @Colelyman, you can increase the block size in reader.h
diff --git a/include/reader.h b/include/reader.h
index 679bf0d..c35669e 100644
--- a/include/reader.h
+++ b/include/reader.h
@@ -99,7 +99,7 @@ bool reader::fastq_read_parts(int mode, file_pointer *fp) {
        reader& file_reader = *(fp->freader.get());

        uint32_t OVERHEAD_SIZE = 65535;
-       uint64_t part_size = 1ULL << 25;
+       uint64_t part_size = 1ULL << 29;
        char *part = (char *)malloc((part_size + OVERHEAD_SIZE)*sizeof(char));
        memcpy(part, part_buffer, part_filled);

I have tried the code with the above change and it didn't crash.
We will notify you when we push the new parser changes in the repo.
Thanks,
Prashant

@rob-p @prashantpandey thank you so much! I will try this out.

I'm experiencing the same problem for a genome dataset with commit 695edb8a389f5f80f0622774364d7dca9a09abf5.
I can confirm the suggested change/workaround by prashantpandey is working.
Please reopen this ticket (so its easier to find) until you get a chance to actually fix the parsing strategy and/or add the information to the projects README.md
Sincerely tkranz


Add CRIM3S - Lost (XVA Remix) / Low pitch by XVA

Song Name
CRIM3S - Lost (XVA Remix) / Low pitch
Artist Name
XVA
Source
Youtube
Youtube Link
T52Nd5p73ym-3eZR
Direct File Link
No response
Song ID
945695
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 108057153

@AJXPRO your not a mod lol

reject
invalid YouTube link

@AJXPRO your not a mod lol
How can I be mod?


New version: SIMIlluminationPatterns v0.4.0

Registering package: SIMIlluminationPatterns
Repository: https://github.com/kunzaatko/SIMIlluminationPatterns.jl
Created by: @kunzaatko
Version: v0.4.0
Commit: ecba6bc5f5960c9c24908b7aaa654e141e24d2af
Reviewed by: @kunzaatko
Reference: https://github.com/kunzaatko/SIMIlluminationPatterns.jl/commit/ecba6bc5f5960c9c24908b7aaa654e141e24d2af#commitcomment-150695619
Description: Julia package implementing illumination patterns used in structured illumination microscopy

Stop


Allow mutable ALTREP

At least there can be three strategies on modification

immutable

always duplicate and modify the new one

mutable

modify the materialized version of SEXP in data2 (or somewhere linked to data2)
modify the underlying *mut T, if available

I haven't seen an implementation that chose 2-ii, but it might be the case with Rust.

It seems this is not possible. Closing.
https://github.com/yutannihilation/savvy/pull/223#issuecomment-2089829689


Standardise files with files in sous-chefs/repo-management

Signed-off-by: Dan Webb<EMAIL_ADDRESS>

Released as: 1.0.5


build: add Ferdi snap package

Description of Change

add target snap to electron-builder.yml

This will require some in-depth testing if different functionality works as expected, e.g. audio/video.
Worst case, we might need to change the default confinement level: https://snapcraft.io/docs/snap-confinement
@vraravam We still need an officially registered account and published snap to have users receive auto-updates: https://snapcraft.io/docs/releasing-to-the-snap-store.
How to test this

run npm run build
cd into /out
run snap install --dangerous --name ferdi_test <ferdi-version>.snap
might be necessary to run snap set system experimental.parallel-instances=true before
run snap run ferdi_test
check application to work correctly

Motivation and Context
Closes https://github.com/getferdi/ferdi/issues/1749
Checklist

[x] My pull request is properly named
[x] The changes respect the code style of the project (npm run prepare-code)
[x] npm test passes
[x] I tested/previewed my changes locally

Release Notes

@mhatvan - I am not sure if I can register for a new snap package. I will not be able to maintain it going forward. Afaik, some other enthusiast has created a similar registration for an AUR package (not a core contributor). We can proceed in that manner. Else, if the snap target allows for a downloadable artefact from the GH repo - which can then be installed onto the OS (ie not via the snap store) - then this PR can be merged imo. Just that it'll not be an official installation from the snap-store.

We can contribute it as 'snapcrafters', then everyone is able to update it. It doesn't make sense to only have it as download, defeats the whole purpose of self-updating snaps.

And what I actually meant is not that you make an account personally, but was wondering if there is official email account that I can use from Ferdi, also the Logo.
Don't we have some kind of press/branding information?

Agreed, but that info is still with @kytwb - I dont have that insight (yet)

Do you have any clue when you would get that information?
I would need a Ferdi email adress, full name, username and password to set on https://login.ubuntu.com/dWDzbZYNqYPDxmrw/+decide
Then we could proceed and get the snap package released asap.

This can be merged now, we can manage the rest (stable, beta, nightly releases) from the snapcraft interface.


Tom/lidar embedding integration

Just an fyi 'lidar embedding integration' is a horrible name for this branch as that is what I intended to do, but ended up having to fix other things


Host request overview label

Subject: Host request overview label
Description:
For a cancelled (by guest) req it says “you have cancelled this request”, even though it was the guest who did it.
Results:
Backend version: develop-54e41e19
Frontend version: develop-54e41e19
User Agent: Mozilla/5.0 (iPhone; CPU iPhone OS 17_4_1 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/17.4.1 Mobile/15E148 Safari/604.1
Page: https://couchers.org/messages/hosting
User: aapeli (1)

Confirmed to still be the case.


CAMEL-14798: Separate Avro RPC functionality out of camel-avro

With this PR camel-avro component implements only Avro dataformat for camel. All rpc functionality was separated into new component camel-avro-rpc. Documentation of components was changed accordingly.

parent/pom.xml may need to add the new JAR
And in apache-camel there are some descriptor files too.
fx just do a git grep camel-avro and see where its in use, and add also the camel-avro-rpc too


Are the images in V2X-I dataset the subset of the images in Rope3D?

I'd like to expand the Rope3D datatset in the object detection task, while I wonder if the V2X-I is helpful.

Yes i think so，I used the hash function to compare the images between DAIR-V2X-I and Rope3d. Found that most of the images  in DAIR-V2X-I has its copy in Rope3d.

请问您能分享下Rope3D的数据集文件吗？官方网站似乎无法下载了


Change Navigation bar color on narrowing to stream.

Change background color of navigation bar to the color associated with stream.
Fix:#245
Used foregroundColorFromBackground to determine the color of the title and drawer icons.

Automated message from Dropbox CLA bot
@vishwesh3, it looks like you've already signed the Dropbox CLA. Thanks!

Thanks for the PR @vishwesh3 !
Instead of passing the color for a stream to doNarrow, it might be cleaner to look up the stream operator in state.chat.narrow and use that to find the current stream's color in state.subscriptions. That way you wouldn't need to pass the color variable along in doNarrow (which feels a little out of place).
Does that explanation make sense?

@neerajwahi Done, used state.chat.narrow and state.subscriptions

@neerajwahi  Done

Thanks @vishwesh3 ! This looks great overall.
Looks like there's one last issue. The Home screen (which has BRAND_COLOR as the background) has white text but black icons:

Now icons and text color is decided from foregroundColorFromBackground

@neerajwahi Please have a final look.

Looks great, thanks @vishwesh3 ! Merging now


Add GoDaddy DynDNS API support

Based on the following documentation https://developer.godaddy.com/
This adds basic support for GoDaddy DynDNS REST API.
I could test the following :

A record
AAAA record
Invalid and Valid tokens produce the correct output in logs
Incorrect domain name produce the correct output in logs

More error messages and features could be handled as the API is richer than what I handled here, but this is a start

Known issues with this implementation:

To support all cases GoDaddy might require an additional field to support root and subdomain names together. This comes from the fact that the GoDaddy rest API uses the URL to split domain and host part.
This merge request specifically supports sub domains with a hot part or a wildcard, only.

If you want a better integration, please advise on what approach would you like to see implemented:

An extra field (domain, name, etc. ?)
An analysis of the provided hostname to detect if it's a [hostname].[root] or [root] with all the cases that this includes "example.co.uk" << example is a root, etc.

+1 on this PR, I used this feature in pfSense and I'd love it here.

Hi everyone.
Sorry for this very late reply.
I didn’t have any free time available to update this branch with the feedback.
I’ll see if I can add it properly this weekend.
Best regards,

Hi @fichtner ,
Everything should be back to normal again.
But I would still advise to either add a warning to this feature based on my comments above or to complete it properly. I don't have the time to invest into this feature in the near future sadly.
Best,

Merged, thanks a lot! ❤


[SV] added variations

@larsdunemark It seems like the automatic tests are not running for this PR, do you see anything to approve this to run? The first run of test of a new contributor seems to be started manually.
@edgein666 Have you got the environment set up to run the tests locally? Usually good to add some test sentences for new words to try catch as much as possible :) But the code and logic seems good to me.

@Robin-St yep, figured out how to do it now. 🙂

[SV] Added "dim to half"
This was not meant to be added now. Will see if I can remove it.

Not able to remove my latest commit, "dim to half", I wanted to discuss it first but managed to commit anyway.

thanks @larsdunemark, I tried, are we good to go now?

will create new PR.


Add Identity Map Codec (please read the notes to reviewers)

also has tests
Follows on from this closed PR on core - https://github.com/elastic/logstash/pull/4104
NOTES TO REVIEWERS:

This pseudo codec is to be used along with the multiline codec in the file input initially.
When a stream identity/codec is evicted, it is also flushed using the block given to the decode method which closes over the input-filter queue.
There will always be one or more buffered lines trapped in the multiline codec because we don't flush on close on any buffered codecs. But if the stream/codec is evicted then the trapped lines will be released.

If you have read or commented on the first PR, these are the important differences:

There is only one ThreadSafe::Hash that hold both the codec and its eviction time.
EVICT_TIMEOUT is now 1 hour, read this as - if the stream/codec was last seen more than an hour ago it can be evicted from the map. This needs to be long enough to allow all the multilines to be read and short enough to ensure flushing.
MAX_IDENTITIES is now 20000
Cleaner thread only runs when the are stream/codecs mapped

Fixes #10

So, to clarify, the intended usage is to be able to call encode(event, identity) as an extension to the current Codec API?
This currently looks great to me code-wise btw!

@guyboertje I'd like to hop on a zoom with you monday just to make sure I have all the details straight, but it LGTM!

Minor comments,  code LGTM!
I have tested this with concurrent-ruby 0.9.1 and tests passes

@guyboertje thanks for answering my questions, for concurrent-ruby just make an issue for the migration path. I expect both gem thread_safe and concurrent-ruby to concurrently work ;)

Lgtm sorry my last comment wasn't clear enough


octal: Rewrite tests to use hspec with fail-fast.

Related to #211.

Interesting (haven't seen quickcheck with hspec), 👍


Fix Shader and ShaderInclude resource loading

Fixes https://github.com/godotengine/godot/issues/76183

Possibly also helps with some other issues like https://github.com/godotengine/godot/issues/79324. There have been several reports of deleted or renamed Shader resources somehow "haunting" a project, reappearing or otherwise causing some strange issues. These are possibly caused by the Shader resource loading logic which never fails even if you try to load a completely nonexistent Shader or ShaderInclude file, see https://github.com/godotengine/godot/pull/73975 for more details.
This PR adds error checking when loading these resources. In the linked issue MRP the crash happens because the scene file contains invalid references to a deleted .gdshader file:
[ext_resource type="Shader" path="res://Shaders/planetShader.gdshader" id="1_3kkjf"]
and
shader = ExtResource("1_3kkjf")
Because currently loading this invalid resource actually succeeds it also makes the editor think that Shaders directory exists and when it tries to scan this invalid directory Godot crashes (because DirAccess:open() return value is not checked).
After this PR, trying to open the scene will correctly fail with an error message pointing to the missing file instead of silently creating invalid resources and then crashing Godot when trying to save the scene. If people have similar dangling resource references in their projects, they can no longer open and edit these scenes in the editor as the loading will fail. To fix this, they can either manually edit the .tscn file using an external text editor or create a temporary shader file where the missing one should be, then edit and adjust the scene inside the Godot editor. edit: Turns out there is also a scene dependency fixer functionality in the editor that can be used to fix this, live and learn.
This PR will also trigger some relatively untested code branches because previously Shader resource loading never failed. There are some conditions in the codebase that never failed previously, but after some testing they seem to work okay. Still, it's something to keep an eye out for.

Thanks!

Cherry-picked for 4.1.2.


fmt(cstyle): cstyle fixes in istgt/src/istgt_iscsi.h

Fixed style issues reported by cstyle.pl in istgt/src/istgt_iscsi.h.

For issue: https://github.com/openebs/openebs/issues/2010

@payes Can you review the changes?

@payes Any update on this?

@payes Resolved Merge Conflicts.

Closing this PR due to inactivity. @shubhambhattar  Feel free to reach out us.


Update database interface

Hi , when I use the Sugar ORM ,I find one issue that when I want to do some update DB operation and maybe it is very tough to write the sql script , so can u provide one interface for the update DB operation ?

@LNeway The problem is when you try to do a migration?

Yes

Do you want that we add this as a feature?


Write tests for @tinyhttp/send

[ ] send

[ ] send

[ ] remove headers for 301 and 204 statuses (codecov)
[ ] set Content-Type to application/octet-stream if the header hasn't been set before

[ ] sendFile

[ ] throw if path is not absolute (codecov)
[ ] Content-Type header (based on filename) is set properly
[ ] options.headers sets all required headers

Thanks, @talentlessguy! I'll start working on it soon!

Thanks, @talentlessguy! I'll start working on it soon!

@talentlessguy, I'm trying to make the 2nd send test pass, but I think the module may be missing a res.end for buffers. Adding it seems to solve the problem, but, since I'm new to the codebase, it'd be great if you could confirm that it makes sense :sweat_smile:

@alcar how is it going?

@talentlessguy only managed to actually take a look at the code yesterday, and now I'm writing the final 2 tests, so should have a PR ready in a bit! Sorry for the delay!


Facebook connect?

Will you support facebook connect officially? or are there any extensions to do that?

@wunki, I am using django-socialregistration too. How do you integrate both?
I tried to use some signals, but I am affraid I need to copy almost all create_user's code inside socialregistration's receivers to make sure it works as expected.
What do you suggest?

It doesn't take that much work. I have created a gist as demo. It's not copy-paste and it works, but you should be able to see what's needed to get userena working.
Connecting userena to django-socialregistration

right!
I used ASSIGNED_PERMISSIONS instead of PERMISSIONS. Your gist's PERMISSIONS description is outdated.
See my fork.
Don't you think it would be better if we split UserenaManager.create_user into something like create_user and create_user_extra?
So any app could create users in a easy way without breaking the code. It would be something like this:
from socialregistration.signals import connect as profile_connect
from userena.managers import create_user_extra

@receiver(socialregistration_signals.connect, sender = FacebookProfile, dispatch_uid = 'facebook.connect')
def social_connect_callback(sender, user, profile, client, **kwargs):
    create_user_extra(user)


Direct user to follow the upgrade steps after installation

As discussed on the community forums, the appstarter doesn't work out of the box after a fresh installation. This was determined to be due to the composer.lock file being out of date, running composer update installs the up to date version of the framework.
Description
Updated the installation via Composer documentation to direct users to follow the steps in the "Upgrading" section even after a fresh installation. This ensures the framework is up to date.
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPdocs: N/A
[ ] Unit testing, with >80% coverage: N/A
[x] User guide updated
[ ] Conforms to style guide: N/A

Oooh - you may have hit the nail on the head ... the appstarter ships with a composer.lock file that it shouldn't! Let me play with that ... this PR might not be needed if the problem can be fixed at the root :)

composer.lock is git ignored, and I don't know how it got into the repo :(
I have deleted it, and re-created the beta.3 release, but it still shows up with alpha.4 and needs updating :(
I am going to leave your comment in the installation instructions, and try to figure out this puzzle later.
Thanks!

@jim-parry Sorry if this is a dumb question, but did you remove the file or clean it from the history? .gitignore has weird interactions with files already committed and git rm.

Good question. I probably didn't do it right, and need to fix it :(


fix(tenant): use rand org bucket id generator over snowflake in tenant service

Closes #19017
This move tenant back to using the rand packages id generator when generating IDs for orgs and buckets.
Prior to this change, it was possible for harmful IDs to be generated for buckets and organizations. Which could leave buckets or even entire orgs unable to read their data (empty responses).

Hey @alespour, just circling back on this with the intention to merge it and I noticed that the additional test doesn't pass in the context of influxdb. If you're unaware, we run all the tests against an influxdb launcher. The driver is in influxdb/query/stdlib/testing/. To make it run I think you just need to eliminate the now option and add a range liike
    |> range(start: 2020-01-01T00:00:00Z )


fix: unsubscribe from watch order input

Related issues 🔗
Closes #3041
Description ℹ
A description of what this PR does. This should be non-dev readable. Technical stuff should go in the technical section below.
Demo 📺
Gif/video/images of new/corrected functionality if applicable
Technical 👨🔧
Details of technical implementation that reviewers may need to be aware of, if applicable.

contuinuing on fix/order-store-watch


don't generate empty aria-labels

Currently, sre may produce empty speech, e.g., https://github.com/zorkow/speech-rule-engine/issues/559 https://github.com/zorkow/speech-rule-engine/issues/560. While the first one looks like a bug, the second issue does not so we can't simply drop them. Maybe we should populate them with a fallback string, maybe we should look at other SRE attributes and vary our response.

As per F2F with @zorkow neither of these are bugs.
Let's add "blank" as a fallback string.


better xml building by using xmlbuilder directly

XML (and the freshbooks api), supports child nodes with the same name. XMLBuilder already does a good job building XML request objects, no need to use the limited XMLJS version. I have updated the examples file to show how contacts (and similar objects) should be created (prior to this change, you couldn't add more than one contact). Also updated version to 0.2.0 as changes may not be backwards compatible (depending on how end users implemented this).

@parisholley: thanks so much for the PR. Unfortunately, I don't have time to maintain this repo. If you're interesting in taking it over, let me know and we can talk transfer. :)


Is it possible to make the callBackFunction dependable on the current cumulVar?

No.  The solver expects this to be consistent.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le ven. 17 mai 2019 à 14:22, EliasBA<EMAIL_ADDRESS>a écrit :

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1285?email_source=notifications&email_token=ACUPL3NWVWQGZX5GXE5XOCTPV2PPPA5CNFSM4HNUZNR2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GUMQBTQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACUPL3MDMVWETQD2OHF6KSTPV2PPPANCNFSM4HNUZNRQ
.


Remove the PDFPageView.{error, stats} properties, and some related viewer clean-up

Please refer to the individual commit messages for additional details.

/botio-linux preview

Nice clean-up; thanks!


Settings API - Use all authentication options

This PR updates the Settings API to both use server2server auth in addition to Okta auth.
Test Steps:

Run the smoke test
Verify the code makes sense

Changes

Updated SettingsFunctions to use both Okta and Server2Server during the authentication process
Made all tests using copied Okta auth functions pull from a new location, with a new name to stop a function name conflict

Linked Issues

#1946

I believe this needs an update to prime-router/docs/api/settings.yml

I believe this needs an update to prime-router/docs/api/settings.yml

It did! Updated it.

Made changes so that the tests remain as before, only with the change that the bearer token is fetched via the new method

Added smoke tests for Settings Auth

All tests pass in staging! Ready to review

@luis-pabon-tf I think some of the smoketests need a little refactoring.  It looks like one of the tests changes in server2serverauth requires being authenticated with okta

Added a wrapper, which has fixed the local issue.


Satisfy error interface for errors and add FromError mapper

What changed?

Satisfy error interface for error related internal types.
Add common thrift.FromError mapper.

Keeping them in separate files, to not touch autogenerated code for now.

Why?
Once we start changing types to internal, error related change becomes problematic as errors propagate up everywhere. Hence created this thrift.FromError mapper that checks if error is in internal form and if so - converts it to thrift equivalent. Next we could add Thrift handler wrapper, that would use this mapper as the last step. All internal propagates errors will be converted to thrift there. Once this is done changes switch errors to internal types can be started.

How did you test it?

Potential risks

Coverage decreased (-0.09%) to 68.773% when pulling 2baceafc3dc1b6259c543ebe8f741c93fcf936a7 on vytautas-karpavicius:internal-errors into 9f58fc4c48dd2fb23e66dc42571a992ff5967048 on uber:master.


Error while starting with Docker

Hi @vdmitriyev,
I'm trying to start the DB using docker but when i try to send an http request to it, the server refuses the connection.
While if I try to start the application in debug from VSCode, there's no problem.
Have you tried to start it from docker ?

Fixed here -> https://github.com/dmarro89/dare-db/pull/31 @vdmitriyev

Before accepting your previous PR I had tried the TLS version of the Docker and it worked well

I am going to try our your latest PR and let you know

@dmarro89 I am going to try our your latest PR and let you know

The problem was on the http server.
Let me fix the pipeline and then you can try it.

@vdmitriyev fixed

@dmarro89 thanks. It runs and tests work fine. However, I have some concerts regarding <IP_ADDRESS> as default value for host and environmental variables. Please, see other comments attached to your code.

@dmarro89 thanks. It runs and tests work fine. However, I have some concerts regarding <IP_ADDRESS> as default value for host and environmental variables. Please, see other comments attached to your code.

Ok thanks

@dmarro89 thanks. It runs and tests work fine. However, I have some concerns regarding <IP_ADDRESS> as a default value for host and usages of environmental variables. Please, see other comments attached to your code.

Hi @vdmitriyev I don't see any comment on the code, am I wrong?

@dmarro89 thanks. It runs and tests work fine. However, I have some concerns regarding <IP_ADDRESS> as a default value for host and usages of environmental variables. Please, see other comments attached to your code.

Hi @vdmitriyev I don't see any comment on the code, am I wrong?

It is strange. Because I provided some comments.

@dmarro89 thanks. It runs and tests work fine. However, I have some concerns regarding <IP_ADDRESS> as a default value for host and usages of environmental variables. Please, see other comments attached to your code.

Hi @vdmitriyev I don't see any comment on the code, am I wrong?

It is strange. Because I provided some comments.

Maybe you have started a review but non clicked on comment at the end of it.

@dmarro89 thanks. It runs and tests work fine. However, I have some concerns regarding <IP_ADDRESS> as a default value for host and usages of environmental variables. Please, see other comments attached to your code.

Hi @vdmitriyev I don't see any comment on the code, am I wrong?

It is strange. Because I provided some comments.

Maybe you have started a review but non clicked on comment at the end of it.

Now I mentioned you explicitly in the comments

PR was merged https://github.com/dmarro89/dare-db/pull/31
Issue will be closed


BRUH

Traceback (most recent call last):
File "/home/bamboozled/Mizogg-Tools/Miz_tools_OLD_version/Miz_Tools_ice_colour.py", line 11, in
import secp256k1 as ice
ModuleNotFoundError: No module named 'secp256k1'
bamboozled@FuckeryBox:~/Mizogg-Tools/Miz_tools_OLD_version$ python
Python 2.7.18 (default, Jul 14 2021, 08:11:37)
[GCC 10.2.1 20210110] on linux2
Type "help", "copyright", "credits" or "license" for more information.

import secp256k1 as ice
Traceback (most recent call last):
File "", line 1, in
ImportError: No module named secp256k1
import ice
Traceback (most recent call last):
File "", line 1, in
ImportError: No module named ice

I'm clueless as to what to do next my guy, I already tried importing the module on python; as well as tried to install it thru pip AND pip3 but to no avail.
does thou maybe know thy solution to this perplexing puzzle?

You need to have these files in the same folder to run. You can get them from https://github.com/iceland2k14/secp256k1


Server crash when using Textile Backup

The backup mod Textile Backup causes a crash whenever a backup is attempted, either timed or manual. I attached two crash reports.
crash-2021-06-16_12.16.27-server.txt
crash-2021-06-16_12.22.49-server.txt

Looking into it further, C2ME works just fine with Textile Backup.
I had forgotten to remove Dimensional Threads when re-installing C2ME. That's the cause for the crash. Uninstalled DimThread and all is peachy.


VM throws incorrect exception for implicit downcast

Consider the following code:
class A {}
class B extends A {}
void main() {
  B b;
  A a = new A();
  b = a; // (1)
}

Strong mode requires a TypeError to be thrown at (1); accordingly, the front end compiles this as b = a as B; (this functionality was added in 6f9d6965fec8285d275f13eff9df3ca1db97b9d2).
Normally, an "as" check throws a CastError when an "as" check fails.  So to signal that this "as" check should throw a TypeError, the front end sets the AsExpression.isTypeError flag (see https://github.com/dart-lang/sdk/blob/6f9d6965fec8285d275f13eff9df3ca1db97b9d2/pkg/kernel/binary.md#binary-format).  The VM needs to use this flag to determine what kind of exception to throw when the "as" check fails.
I'll upload a failing test illustrating this bug shortly.

Test is here: https://dart-review.googlesource.com/c/sdk/+/17042

@sjindel-google could you take a look at fixing Kernel->IL translation to throw the right error?

I can't reproduce this error. I've cherry-picked your revision, removed the line from language_2/language_2_kernel.status marking the test as failure, and the test still passes.

My CI: I wasn't running with --strong. I'll look into it.

Thanks!  Sorry I wasn't clearer about how to repro the bug.


[feature request] allow to disable scan frequency

It's not possible to disable the automatic scan frequency. Allow config value to be set to 0 to disable the refresh.

Hi @tjorim,
Thank you for your valuable feedback! 😊
I’m happy to let you know that in the latest release (v1.04), we’ve implemented exactly what you suggested. You can now set the scan_frequency to 0 to completely disable the automatic refresh. This gives you full control to manually trigger automation suggestions whenever you need them via Developer Tools -> Actions.
Thanks again for your suggestion—it’s helped make the integration even more flexible and user-friendly. If you have any more feedback or encounter any issues, feel free to reach out!


Support for cosine similarity search on sparse vectors

By default k-NN plugin uses dense vector for Cosine similarity measure. NMS library has support for sparse vector space type. We could enable this support in k-NN plugin.

In order to implement Sparse Vector Support, we would need to make a modification to our object creation in JNI code. Currently, the dataset is represented as float * and that pointer is passed in to create the corresponding object. However, for sparse vector support, we would need to represent the Object's data set as SparseVectElem.
Additionally, we need to think of the appropriate UX for users to input data for this format. For example, users could input data in the following format:
{: ,..}

I can take on this issue if it is not blocking. Together with #109

Additionally, we need to think of the appropriate UX for users to input data for this format.

@vamshin @jmazanec15 Shall we come up with a design of an appropriate sparse vector input format or there is UX designer doing the job?

BTW, in nmslib, sparse space type does not support optmized index, either(#161 )

IMO, we should add a new field type knn_sparse_vector. This is worth a design doc before execution. also applies to #109


[FEATURE] Publish snapshots to maven for 1.3 line

This issue was created because the publish snapshots to maven workflow was never successfully backported into the 1.3 branch. This means that maven artifacts for 1.3 doesn't exist. We should manually backport to ensure that going forward 1.3.x artifacts exist.
https://github.com/opensearch-project/security/pull/2438#issuecomment-1819588107

@derek-ho This seems closed out to me with that code change merged - can you confirm?


Merge dev-2021.1 into Master

Created release for 2020.4
Update the readme and PR template in dev-2021.1
Will send out another PR for gh-pages merging and post a blog post of the release updates after this
Let me know if I am missing something.

Hi, @saugatt! Thanks for opening this PR!

In the file connector-packager/connector_packager/xml_parser.py:

Line 135:121: E501 line too long (134 > 120 characters)

In the file connector-packager/connector_packager/xsd_validator.py:

Line 177:149: W291 trailing whitespace
Line 177:121: E501 line too long (148 > 120 characters)
Line 176:144: W291 trailing whitespace
Line 176:121: E501 line too long (143 > 120 characters)
Line 18:121: E501 line too long (142 > 120 characters)

For reference: Error Code Reference, PEP8, Python code style


Opening closing webview containing webchat will cause the webchat to fail to connect

On Version 4.5.2
We have an Android application that contains a webview which hosting and webchat, so exiting and entering the webview activity multiple times causes the webchat to fail to connect and all post activity be rejected from directline
Steps to reproduce

Host appservice containing webchat connected to an chatbot that have a welcome screen
Create an app with two activities the first opens the second
In the second add webview which opens the hosted app
Open application and add navigation from activity 1 to activity.
Run app navigate from activity 1 to activity 2.
Go back.
Repeat steps 5-6 until you welcome screen is not presented.
After this step any post activity for a period of time will not work.

Expected behavior
Application work properly
[Bug]

Hi @muminasaad I apologize, but I'm not quite following everything here on the steps.
When you are talking activity, you are talking activities within the app, not within the bot or webchat, correct? (i'm not clear on what you are describing regarding app navigation).
If you can provide more info, code or even a full mockup that would be very helpful. Can this be reproduced in a scenario outside of an Android application?

Hi Dana,
So let me explain, I have an android solution that hosts a WebView, in the WebView I am opening a WebPage that contains the WebChat.
I preserve the conversation Id, I connected to using a localstorage and so on, so that each time you open you connect to the same conversation.
As part of the application of the applications testing what we did is that we navigate to the WebChat multiple times, opening the Webview then clicking back and reopening the Webview.
Each time we exit we dispose the Webview and everything, and open the Webview will create everything.
So the navigation to the Page is successful and works perfectly. Like opening new tab on browser and closing previous.
Everything goes great until you do it 4-10 times in a row.
What happens is that the application loads and the WebPage loads, React loads, I receive Connection_Fulfilled action and everything seems fine, but the previous message history is not recovered.
Upon investigating we see that all communication start to fail for a small period of time through the DirectLine WSS and stops the retrieval.
We test the token and we test the webchat after this point it works, but the history is gone. which shows blank screen to the customer which is a bad customer experience.

Thank you @muminasaad

Everything goes great until you do it 4-10 times in a row.

Can you explain more about this? Specifically 'in a row'. Does it matter if messages are sent between these times that you close/reopen? Does matter if these times in a row happen within a certain time window?

Closing due to inactivity. Please provide more information if this issue is still occurring.


[Bug] Loading textures command always refreshes all cars

Mode should be 1 when specifying a car_idx.
As of now right the fn car() still refreshes all cars in the session.
See reference: https://github.com/SIMRacingApps/SIMRacingAppsSIMPluginiRacing/blob/master/src/com/SIMRacingApps/SIMPlugins/iRacing/BroadcastMsg.java#L190
Should be fixed in this PR:
https://github.com/adnanademovic/simetry/pull/3

Released the fix as 0.2.2.
Thanks again and sorry for the long wait.


nrf52/ieee802.15.4/6lowpan server crash

Hello,
I have done some testing with more heavy duty sending (100 Hz). The server occasionally crashes in the hpwork queue thread. I have increased stack size of the thread with no noticeable improvements.
I am testing with two nrf52840-dk boards.
Tagging @raiden00pl since you provided the original work.
Thanks!
Originally posted by @maxikrie in https://github.com/apache/nuttx/issues/11617#issuecomment-2251450978

This is probably a problem with the ieee802154 stack in NuttX because I observed similar crashes with mrf24j40 (connected to nrf52).


Add "All" or "All/None" option to dropdown filters.

According to #49

I'm not sure if this is a good idea to change "All" to "None" when selecting all. For me it looks strange to have a "None" selected and all items visible.


Maintain expand/collapse state of items in the variables panel when stepping through code in debug mode

When using the debugger, everything in the variables panel gets collapsed on every step. This is especially annoying when trying to examine a nested structure.
I had initially thought this was a default VS Code behavior, but according to https://github.com/Microsoft/vscode/issues/32199 the state is determined by the debug server itself. Can dig bit deeper into this and see if this can be fixed in the extension debug adapter or by HHVM.

That's interesting that they said that - I am not sure what VS Code is keying off of in the UI.  There is no expansion state maintained by the debug adapter, or communicated in the protocol.
Things stay expanded as expected in Nuclide, so this must be an interaction with VS Code.
One thought off the top of my head is the IDs of Scopes and Variables change with each ScopesResponse and VariablesResponse - maybe VS Code is tracking expansion state by object ID instead of path?

Yeah the ID change is most likely causing it

Let me see what I can do here...

I pushed 83e2be29c4c771cf2ce5c158c2c43af1d518329d which will keep variableReference IDs for the same object stable within a request.
I'm not certain how VS Code is tracking expansion state, but if it's by variableReference like we thought above, I think this should take care of it for you


Gradle could not find 'com.jakewharton.rxbinding:rxbinding:0.1.0-SNAPSHOT'

Hi!
It seems this: compile 'com.jakewharton.rxbinding:rxbinding:0.1.0-SNAPSHOT' don't work :(
Is there a valid link?

Did you add Sonatype's Snapshots repository?
repositories {
        maven { url "https://oss.sonatype.org/content/repositories/snapshots/" }
}

dependencies {
        compile 'com.jakewharton.rxbinding:rxbinding:0.1.0-SNAPSHOT'
}

@beamishjabberwock I didn't :( Thank you!


Unable to build

At some point my intellij stopped building projects that have meteor snapshot in them.
A problem occurred configuring root project 'meteor-addon-template'.
> Could not find meteor-client-0.5.1-SNAPSHOT.jar (meteordevelopment:meteor-client:0.5.1-SNAPSHOT:20220926.112654-59).
  Searched in the following locations:
      https://maven.meteordev.org/snapshots/meteordevelopment/meteor-client/0.5.1-SNAPSHOT/meteor-client-0.5.1-20220926.112654-59.jar

Tried re-cloning repositories and cleaning caches in intellij but error persists.
Anyone knows how to fix it?

I ended up resetting my settings to default ones but that's not a great solution

Replace modImplementation "meteordevelopment:meteor-client:0.5.2-SNAPSHOT" with
modImplementation("meteordevelopment:meteor-client") {
        version {
            strictly "0.5.2-SNAPSHOT"
        }
    }

I'm having the same issue:
Could not find meteor-client-0.5.2-SNAPSHOT.jar (meteordevelopment:meteor-client:0.5.2-SNAPSHOT:20230123.224945-43).
Searched in the following locations:
https://maven.meteordev.org/snapshots/meteordevelopment/meteor-client/0.5.2-SNAPSHOT/meteor-client-0.5.2-20230123.224945-43.jar

Seems to be gradle/intellij issue.
Solved by refreshing dependencies of the project:


Take a tour

The "take a tour pop-up" should only appear once for every new visitor. Returning visitors shouldn't be getting the same pop-up again.

It does pop up only for new users. The button remains there however if a user ever wants to take the tour again

What do you think?

I still get the pop-up after visiting several times. Perhaps you can check using your normal browser. If the issue persists then we might need to fix it as soon as possible.

I will check now
null

It doesn't show for old users. Closing this now


Error while using xslx readFile method

var spans = node.attributes.spans.split(':').map(function(span) { return parseInt(span); });
                                       ^

TypeError: Cannot read property 'split' of undefined
    at utils.inherits.parseOpen (/home/carmelo12341/dev/nodejs/excel/node_modules/exceljs/lib/xlsx/xform/sheet/row-xform.js:97:40)
    at utils.inherits.parseOpen (/home/carmelo12341/dev/nodejs/excel/node_modules/exceljs/lib/xlsx/xform/list-xform.js:71:31)
    at utils.inherits.parseOpen (/home/carmelo12341/dev/nodejs/excel/node_modules/exceljs/lib/xlsx/xform/sheet/worksheet-xform.js:109:19)
    at SAXStream.<anonymous> (/home/carmelo12341/dev/nodejs/excel/node_modules/exceljs/lib/xlsx/xform/base-xform.js:65:14)
    at emitOne (events.js:77:13)
    at SAXStream.emit (events.js:169:7)
    at Object.me._parser.(anonymous function) [as onopentag] (/home/carmelo12341/dev/nodejs/excel/node_modules/exceljs/node_modules/sax/lib/sax.js:245:15)
    at emit (/home/carmelo12341/dev/nodejs/excel/node_modules/exceljs/node_modules/sax/lib/sax.js:615:33)
    at emitNode (/home/carmelo12341/dev/nodejs/excel/node_modules/exceljs/node_modules/sax/lib/sax.js:620:3)
    at openTag (/home/carmelo12341/dev/nodejs/excel/node_modules/exceljs/node_modules/sax/lib/sax.js:801:3)
    at Object.write (/home/carmelo12341/dev/nodejs/excel/node_modules/exceljs/node_modules/sax/lib/sax.js:1181:11)
    at SAXStream.write (/home/carmelo12341/dev/nodejs/excel/node_modules/exceljs/node_modules/sax/lib/sax.js:227:16)
    at Entry.ondata (_stream_readable.js:542:20)
    at emitOne (events.js:77:13)
    at Entry.emit (events.js:169:7)
    at readableAddChunk (_stream_readable.js:153:18)

In node 6.3.1 I get an additional error:
(node:2898) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.

The error disappears if I use the LTS version (4.4.7)
This is my code:
var Excel = require('exceljs');

var workbook = new Excel.Workbook();
var filename = 'test.xlsx';

workbook.xlsx.readFile(filename).then(function () {

});

Is there a way to fix this?

I had the same issue than you. I was able to 'fix' it by manually editing the /lib/xlsx/xform/sheet/row-xform.js file and replacing the line 95 with the following:
var spans =  node.attributes.spans ? node.attributes.spans.split(':').map(function(span) { return parseInt(span); }) : [];
Unfortunately, the content of the variable spans is used several lines below and since it's empty, it might cause issues that I am not aware of.

Thanks for the workaround, apparently it is failing at the last element because it is always undefined (apparently). Istead of asigning it to an empty array, I assigned it to the last not-undefined value, which is just [57, 67]
      var spans = node.attributes.spans ? node.attributes.spans.split(':').map(function(span) { return parseInt(span); }) : [57, 67];

After doing that, another error appeared, which is this one:
/home/carmelo12341/dev/nodejs/excel/node_modules/exceljs/lib/xlsx/xform/style/border-xform.js:101
          this.model.color = this.map.color.model;
                           ^

TypeError: Cannot set property 'color' of undefined

Fixed it with yet another workaround (damn it)...
Changed that part from this:
if (this.map.color.model) {
    this.model.color = this.map.color.model;
}

to this
if (this.map.color.model && this.model) {
    this.model.color = this.map.color.model;
}

But still, another error appeared sigh
Unhandled rejection TypeError: Cannot read property 'workbookViewId' of undefined
    at /home/carmelo12341/dev/nodejs/excel/node_modules/exceljs/lib/xlsx/xform/book/workbook-xform.js:184:41

After some console.logs, I discovered that those lines aren't important, simply commenting them or deleting them will fix that error. This ones (around line 181)
// reconcile bookViews
    _.each(model.worksheets, function(sheet) {
      _.each(sheet.views, function(view) {
        var bookView =  model.views[view.workbookViewId];
        _.extend(view, bookView);
        delete view.workbookViewId;
      });
    });

I don't like workarounds and maybe what I did broke something but for now, it works! Maybe it was just an additional loop somewhere in this code, but this code changes fixed all the issues (for now) and... I'm happy with it.
I would appreciate if guyonroche fix this, someday. And also add some more examples in the docs (just saying)
c:

Arturas' PR is now published in 0.2.31 - this should fix this issue.

I believe the issue #186 should also be closed as duplicate.


Fixes to samples to make them build, plus Github Action to prevent breaks

Fix solution project reference
Target netcoreapp3.1 instead of netcoreapp3.0
Depend on EF Core 5.0.0-preview3 for new annotations
Fix typo in type name

@roji I was planning a similar PR to this. Question: should we use the previews everywhere? Or only where the sample has 5.0 code?

I think using preview for preview feature and latest stable for all others would be good compromise. It would also make sure we update our samples from 3.1 to 5.0 when we release 5.0, can catch some breaking changes in existing functionality if any. Also allows people downloading sample to not have preview setup unless they are trying out preview features.

Changed to only depend on 5.0 preview3 in the DataAnnotations project which requires it.


Document list: parts should respect remove_underline

The links to parts/"subdocuments" on the document list component are currently always rendered without underlines, regardless of whether remove_underline is actually set on the component itself. This is probably because it's only used in Finder Frontend's search results page, which always uses remove_underline.
What

Make links to parts render with underlines unless remove_underline is set on the component
Remove custom CSS for removing underlines when remove_underline is set, and just use govuk-link--no-underline instead

Why
We are starting to implement a UI overhaul of the search results page which has underlines on all links in the document list component, including those for parts.
Visual Changes
Before

After

This is probably because it's only used in Finder Frontend's search results page, which always uses remove_underline.

See Github Code Search for uses of this component – all in Finder Frontend or search-related internal apps (some of which are decommissioned anyway)


BaseModel.construct slower than doing validation

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Turns out that validation in our code base is taking a signification chunk of the time, so I proposed to use .model_construct(...) whenever validation is not needed. Surprisingly, after doing some simple benchmarks, it seems construct is actually slower.
In the docs I read:

pydantic also provides the construct() method which allows models to be created without validation this can be useful when data has already been validated or comes from a trusted source and you want to create a model as efficiently as possible (construct() is generally around 30x faster than creating a model with full validation).

I am observing that it is 5x slower in a micro benchmark (below).
Example Code
from dataclasses import dataclass

@dataclass
class A:
    x: int
    y: int

from pydantic import BaseModel

class B(BaseModel):
    x: int
    y: int

%timeit A(x=1, y=2)
%timeit B(x=1, y=2)
%timeit B.model_construct(x=1, y=2)

which prints:
136 ns ± 3.77 ns per loop (mean ± std. dev. of 7 runs, 10,000,000 loops each)
549 ns ± 27.2 ns per loop (mean ± std. dev. of 7 runs, 1,000,000 loops each)
2.54 μs ± 19.7 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)

Python, Pydantic & OS Version
at 10:25:23 ❯ python -c "import pydantic.version; print(pydantic.version.version_info())"
             pydantic version: 2.8.2
        pydantic-core version: 2.20.1
          pydantic-core build: profile=release pgo=false
                 install path: /Users/bas.nijholt/micromamba/envs/myenv/lib/python3.11/site-packages/pydantic
               python version: 3.11.9 | packaged by conda-forge | (main, Apr 19 2024, 18:34:54) [Clang 16.0.6 ]
                     platform: macOS-14.6.1-arm64-arm-64bit
             related packages: fastapi-0.113.0 typing_extensions-4.12.2
                       commit: unknown

You are mixing V1 documentation with V2 code. See the updated docs section for V2.


fix: update @apideck/better-ajv-errors to ^0.3.1

R: @jeffposnick @tropicadri
Closes #2981
Fixes https://github.com/advisories/GHSA-896r-f27r-55mw (json-schema) that was fixed in better-ajv-errors with 0.3.1 #9 that is outside of the current minor range specified in workbox-build

Thanks for the contribution!

This hasn't been released yet, I believe (6.4.1 depends on @apideck/better-ajv-errors^0.2.7). Do you know if we expect a 6.4.2 soon?
Comparison is here: https://github.com/GoogleChrome/workbox/compare/v6.4.1...v6

It's out: https://github.com/GoogleChrome/workbox/releases/tag/v6.4.2


Add Feature Request template

Add a feature request template for the whole organization use and/or take inspiration from.

these kinds of templates are generally not respected, wdyt?

Like this one #6.
My bad I should've created a single PR with all the issue templates instead of creating separate ones. Should I close these PRs & create another with all Issue Templates instead?

these kinds of templates are generally not respected, wdyt?

Like this one #6.
My bad I should've created a single PR with all the issue templates instead of creating separate ones. Should I close these PRs & create another with all Issue Templates instead?

Yes YAML templates would be nice
We can modify each PRs no problem with that 👍🏽

All Issue Templates conform with the new YAML templates right now. Do let me know if there's something to be changed.


sst_import: run size check after ingest sst file

What is changed and how it works?

Issue Number: Close #14752
What's Changed:

run size check after ingest sst file

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test

Side effects
Release note
run size check after ingest sst file

/merge

/merge

/merge


Improve model performance

The decision tree is 85% accurate. Improve this.

Model performance is improved now.


PR: Move README to md

Codecov Report

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

@@           Coverage Diff           @@
##           master     #107   +/-   ##
=======================================
  Coverage   19.77%   19.77%
=======================================
  Files          11       11
  Lines         617      617
=======================================
  Hits          122      122
  Misses        495      495

Continue to review full report at Codecov.

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


Go DS API needs helpful error message when creating a DS in a CDN without DNSSEC keys

If you have DNSSEC enabled on your CDN but the DNSSEC keys have not been generated for your CDN yet, it is impossible to create Delivery Services for that CDN. However, the error message returned when trying to create the Delivery Service is just "Internal Server Error".
In this case the error message returned to the user should state that the CDN has DNSSEC enabled but no DNSSEC keys yet, which prevents the creation of a new Delivery Service. IMO this should be a 400 Bad Request rather than a 500 Internal Server Error.

There appears to be a workaround for this so I've added the minor label. The workaround is to just make sure the CDN has DNSSEC keys right @rawlinp ?

I wouldn't really call it a workaround because there's no information given to the user to indicate that the issues is the CDN's DNSSEC keys, but I would agree it's minor.

@rawlinp - it's a super secret workaround :)

We now have a helpful error message for this issue.


Fixed compile issues for WPF.

I believe this compile issue was introduced in the previous update for this file 18 days ago.
I have added in an attempt at the WPF-equivalent code however I am not sure how to test.

Fixes compiling errors from https://github.com/unvell/ReoGrid/pull/200


Update app-insights-proactive-detection.md

Some minor changes to the documentation to reflect some recent changes

Hi @antonfrMSFT, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

replicated changes to azure-content-pr per editors. Anton, please close this one.

closing per Alan's request.


How can i add push notifications in android for rocket chat ?

How can i add push notifications in android for rocket chat ?
I want to notify user through push when message gets received and app is not in open.

Now discussing at #77


SBT 1.4 support

Hello 👋🏽,
does this plugging works in sbt 1.4.9?
I couldn't find any official statement about it, but I'd also like to know if someone is running sbt 1.4 with this plugin successfully.
Thanks!

Hi @diraol,
Yeah I see how that could be confusing from the README.md page. But YES it works fine for 1.4, 1.5 and 1.6 also. I'm currently on 1.6.2 for some of my projects without any problems.

@diraol I attempted to clarify this on the README.md page.

Perfect, thanks for the clarification! :)