Site routes not working on server

Hi,
I've created a site that works well locally. However, when I load it to Heroku, the routes don't work as expected. Linking to /search from the home page works fine, but not when navigated to directly or refreshed.
I have also tried making my own node server to access /dist/client and serve that, but then I get many errors in the browser console about being able to connect to socket-js.
Any insight would be appreciated.

Example with expressjs:
//...
//...
app.get('/*', function(req, res){
  res.sendFile('public/index.html');
});

The second question, Specify the NODE_ENV=production when compile

@OKNoah I don't use Heroku so I can't offer any detailed explanation for their service, but you most likely need to setup your web server to always serve the root index.html file regardless of the endpoint hit. If everything is already being forwarded to your express app, then you only need to do what @JuoCode posted, otherwise if that's not yet setup you'd have to configure nginx or whatever other web server it is you're using.
That said, unless you actually need the express server for something (i.e. an API) then I generally just point the web server directly at dist and bypass an app server altogether.

@davezuko Yes, I'm now having to make my own server to host the /dist stuff. However, this throws server-side rendering out the window, unless I learn how to set that up on my own as well. Still, I am running into basic problems, like things in my src/assets folder are not available after I compile as production environment.
Also, I am not using nginx. Heroku simply runs the app with whatever environmental variables you give it.

Maybe I'm misunderstanding. Is there an production server in this repo or not?

@OKNoah I got an email about a comment you posted but no longer see it. To answer it: no, there currently is not a production server in this app. Out of the box it's able to be served statically from ~/dist from a web server, but if you need to serve it from an app server like for Heroku then you'll need to set that up yourself (again, probably just what @JuoCode posted would suffice).

Closing this for now (moving it to https://github.com/davezuko/react-redux-starter-kit/issues/221). I've updated the README to include a bit more detail around deployment, and I'll continue to improve upon the documentation once https://github.com/davezuko/react-redux-starter-kit/issues/221 gets moving; feel free to discuss this further there if necessary.


Add offline mode (interact with bot through Postman etc) for easier debugging

Hi,
Firstly, thanks for this library. It made creating a simple Discord bot much simpler than I was expecting.
I noticed I managed to get temporarily restricted from the Discord API after I'd started and stopped my bot instance too many times when debugging an issue.
This got me thinking, because your API is built on top of Flask could I just call it locally with curl, Postman, etc without needing to use Discord?
The answer was yes (as below) but one problem is the incoming message signature validation - it would require updating each time you change the message which would take time.
So with this in mind I added a new DONT_VALIDATE_SIGNATURE config option. It defaults to False, and obviously shouldn't be used in production, but it will just skip validating the signature.
To solve the second problem I've added DONT_REGISTER_WITH_DISCORD. This just causes fetch_token to return with a mocked result. it also defaults to False, and would be entirely useless to use in production.
You then then use Postman or whatever to issue a content-type: application/json post request to your sever, e.g.: http://<IP_ADDRESS>:5000/interactions with a JSON body of:
{
    "id": 1,
    "channel_id": "",
    "guild_id": "",
    "token": "",
    "data": {
        "id": 1,
        "name": "ping",
        "options": [
            {
                "type": 1,
                "name": "Pong"
            }
        ]
    },
    "member": {
        "id": 1,
        "nick": "",
        "user": {
            "id": 1,
            "username": "test"
        }
    }
}

This was the most minimal set of parameters I could use. This emulates /ping with the first parameter set to pong.
I appreciate this is probably something you'll find useless, but if you want it, great!
Thanks

Thank you for taking the time to submit this PR!
This looks like a really useful feature, and I like the way you've implemented it. Also, thanks for taking the time to add an example of its use.
Before merging this, I'd like to add a page to the documentation describing the purpose of the offline mode and some general guidance on how to use it. To be honest, I probably can't get around to doing this right away, since the end of the semester is coming up, but I'll write up a page as soon as I can. (Or, if you're familiar with Sphinx or willing to learn, you could try your hand at writing some docs. No pressure though--you've already contributed plenty.)
Finally, (and I'm leaving this here as more of a reminder to myself than anything), your offline mode gave me an idea: what if this library added a module/tool to simulate calling the interactions endpoint, like a more automated version of what you did with Postman? It might be useful for running automated tests against a bot (or this library, for that matter).
Anyways, thanks again for taking the time to share your changes with this PR! I think this is a really nice feature that would be useful for a lot of people, and I'll get it merged as soon as I can.

Before merging this, I'd like to add a page to the documentation describing the purpose of the offline mode and some general guidance on how to use it. To be honest, I probably can't get around to doing this right away, since the end of the semester is coming up, but I'll write up a page as soon as I can. (Or, if you're familiar with Sphinx or willing to learn, you could try your hand at writing some docs. No pressure though--you've already contributed plenty.)

I did nearly submit documentation at the start but I was being lazy 😃
Sure, I've just pushed another commit that adds a Debugging page. You might want to rewrite it in your style but I think it covers everything you need.  I used Sphinx a very long time ago but I think I figured it out.
Good luck with the semester!

Finally, (and I'm leaving this here as more of a reminder to myself than anything), your offline mode gave me an idea: what if this library added a module/tool to simulate calling the interactions endpoint, like a more automated version of what you did with Postman? It might be useful for running automated tests against a bot (or this library, for that matter).

The same idea crossed my mind as well. It would be pretty trivial to do I think.
You could even play around with Locust to do some load testing of how many users it could handle concurrently etc.

Anyways, thanks again for taking the time to share your changes with this PR! I think this is a really nice feature that would be useful for a lot of people, and I'll get it merged as soon as I can.

No problem, I'm glad it's useful!

Awesome! Your docs page looks great, I just changed the inline code syntax (Sphinx uses double backticks for code instead of single backticks for some reason) and restructured the hierarchy a bit. Let's get this merged! 👍


Accepted yet?

Have the awesome audio-mixing capabilities of this fork been accepted into the main BlueAlsa program yet? I tried to figure it out by looking at the pull requests there, but could not figure it out.

The multi-client code has not been submitted as a PR yet. There was an earlier attempt many months (maybe years?) ago, but that had too many problems to I cancelled the request. I now use A2DP playback mixing with aptX regularly, so I know that is reasonably stable. but SCO mixing is largely untested and so is capture snooping. I am not planning to prepare a PR in the short-term, but hopefully it will happen one day. If you could try out either the multi-client branch or the main branch and give feedback here on which applications you have tried, which profiles and codecs etc, and any problems found then that might help speed up the initial testing and refinement so that readiness for submitting upstream might happen sooner.

I haven't had any problems with it, using multiple bluetooth speakers. I don't know which codeds those used, but they all worked just fine.

It seems that in the future the Raspberry Pi team will be forcing the use of audio servers:
https://github.com/raspberrypi/linux/issues/4543#issuecomment-1209648363

Raspberry Pi team will be forcing the use of audio server

I don't follow your reasoning. I think they are saying that the new firmware no longer performs mixing of IEC958 frames, so you cannot have multiple clients sending hdmi AC3 passthrough; and dmix cannot mix IEC958 either. But I don't see where they say that PCM streams are affected. I don't have a Pi4 to test with, but I assume that
application->[plug]->dmix->hw

should work with multiple clients sending PCM (ie S16_LE) and then the driver encodes the mixed PCM into IEC958? I think that how it works with other cards with hdmi or s/pdif outputs.

Hmm, looking at the vc4-hdmi driver, it looks like it only accepts SND_PCM_FORMAT_IEC958_SUBFRAME data, so I was wrong about PCM. The RPi hdmi card requires plug to convert from linear PCM formats to IEC958 format, and of course dmix cannot sit in front of plug.
So in fact the application opening the hdmi device always has exclusive access to it, and that is why you would need a proxy such as pulseaudio or pipewire to mix streams from different applications. Unfortunately I don't have an HDMI device to test how other cards deal with this, so perhaps my understanding of HDMI has always been wrong (athough I'm pretty sure I have used dmix:hdmi or dmix:iec958 as device names successfully in the past, but never tried with RPi).

I have no idea what you're saying :-) But it sounds in line with what I read.
Does this mean that the multi-client idea becomes less likely to work in future Raspberry OS versions? Will a pipewire or pulse audio solution become inevitable?
@Arkq

@flatsiedatsie bluez-alsa is somehow not effected by that. It is a standalone solution for bridging Bluetooth and ALSA. Upstream version lacks mixing capability, but @borine fork has it. If you are searching for a solution which allows ALSA applications to stream audio to Bluetooth, this will still work, I think.

For users that require HDMI output from multiple processes on the RPi with vc4-kms-v3d driver (ie certainly RPi4 and zero 2, possibly earlier models too) then pulseaudio or pipewire now seem essential, so it would make no sense for them to use bluealsa.
For users with dedicated sound cards on a RPi  (like me) this change makes no difference, and I will continue to use my own fork of bluealsa, so my mixing code will be maintained.
One downside for bluealsa is with bluealsa-aplay which requires dmix to support multiple client connections. I have no idea how common it is for bluealsa-aplay to be used with HDMI outputs on a RPi, but I suspect it is rare.
BTW I've looked at the intel HDA HDMI driver code, and that does support linear PCM formats in addition to the digital IEC958 format, so it seems my original understanding was correct and it is the RPi driver which is really letting the RPi down here.

@Arkq Yes, I'm already using it, to great delight. I was just wondering whether this news hampers any future development of BlueAlsa towards such a built-in mixing direction.

so it would make no sense for them to use bluealsa.

I don't quite understand. I have parallel audio processes (music + voice control command response). Sometimes I want this to be played over the device speakers (hdmi), but sometimes when I have people over I want to switch to a louder bluetooth speaker. Would using vc4-kms-v3d + sound server mean it is no longer possible to route audio to (Blue)alsa as an output?

so it would make no sense for them to use bluealsa.

I mean, if the system is running pulseaudio or pipewire, then the best way to use Bluetooth audio is to use the respective Bluetooth modules of those services. It is technically feasible to use bluealsa for bluetooth and also have pulseaudio or pipewire running at the same time, but the sheer waste of system resources and compromises in function and performance of the whole audio system make it most unlikely that it would ever be a "sensible" choice.
If that kind of frankenstein system interests you, take a look at the bluealsa wiki article https://github.com/Arkq/bluez-alsa/wiki/PulseAudio-integration which documents some experiments I did on this subject,

Ah, I see. Thank you. I did not know pulseaudio came with its own options for bluetooth audio output.


Fix LGTM alert

Operands hasMove['hypnosis'] and hasMove['hypnosis'] are identical.

This pull request fixes 1 alert when merging 3c6fe0e14e1bc1daabf72d65e37bbae19b115c5c into 9f5a727ad1dd89bea529d9aaf9843040774407a4 - view on LGTM.com
fixed alerts:

1 for Identical operands

Comment posted by LGTM.com


Autocomplete for HTML

I'm not able to use HTML autocomplete when I have the file set to HTML Liquid. For an example if I want to create an <section> tag, typing in sec doesn't bring up any options for the section html tag. Is there a way to configure this?

Would like to know this as well

@iansoper @summer18
HTML autocomplete in vscode is provided by emmet, you can achieve that in liquid by adding your vscode config
  "emmet.includeLanguages": {
    "liquid": "html"
  }

here is more info about the config https://code.visualstudio.com/docs/languages/html#_emmet-snippets


Parent ViewModel is not bound (null) when error occurs in child VM

General
When error occurs in binding of child viewmodel, parent ViewModel is NULL.
#Desired behavior
Parent ViewModel should not be NULL, only Child VM.
How to simulate:
public class ParentVM {
   public string FirstName { get; set; }
   public string LastName { get; set; }
   public ChildVM Xyz { get; set; }
}

public class ChildVM
    {
        public string A { get; set; }
        public string B { get; set; }
}

Request payload:
{
    "firstName": "PETER",
    "lastName": "PARKER",
    "xyz":  "kldqwjkld"  <<<< should be { "a": "qddq", "b": "dqdd" }
}

Result in controller is, VM is NULL:

Version info:
Microsoft.AspnetCore.App (2.1.1)
Win 10 x64  .NET Core runtime.

This issue was moved to aspnet/AspNetCore#5681


Added Function and Action guides

Added Console App about Action and Function with Test program.

Added Console App about Action and Function with Test program.


ToolTip crashes the app if owner gets removed from the visual tree

Describe the bug
I have a ListView containing some RichTextBlocks displaying hyperlinks, these hyperlinks have ToolTips to show the user their target URL.
If the URL is internal to the app, clicking the link will refresh the ListView loading new data.
If this happen exactly when the ToolTip is triggered by the hover action the app will crash by a NotSupportedException.
Expected behavior
The app won't crash
Screenshots

Version Info

NuGet package version:
[Microsoft.UI.Xaml 2.4.2]

Windows 10 version
Saw the problem?

May 2020 (19041)
Yes

November 2019 Update (18363)
Yes

May 2019 Update (18362)
Yes

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox

Surface Hub

IoT

Still crashing


docs: fix  tags that break building asyncapi website

Log that shows breaking build -> https://app.netlify.com/sites/asyncapi-website/deploys/61efc78618da970008ff59a5
Here PR that got merged after fixing <br> -> https://github.com/asyncapi/website/pull/567/files#diff-1699deb4ec142d39730ee1c744b0733210d41de3766efd4ec61e5f089a3acad7R517

/rtm


Warn user before leaving the page

Hi
I was studying about pendingChangesConfirmation but it does nto seem to apply to my situation.
It wont trigger, when user navigates away via menu on the left.
I need to be able to handle ANY leaving of the page and warn user about data being lost.
Any idea how to do that?
thx

nevermind, found it:

$(window).bind('beforeunload', function(){
return 'Are you sure you want to leave?';
});


Develop fix

Added and updated test cases 3MF files:

P_OPX_3002_03_production fixed the transform, that caused a misalignment.
P_XPX_0720_06.3mf. Fixed alternatives referencing itself.
P_XPX_0720_11.3mf. Fixed duplicated UUID.
P_OPX_3008_01. Removed test case. Updated test description. Pending to update PDF and thumbnail album.
P_DPX_218_06/07: New test cases
P_DPX_3222_01/01/03 material. Increased displacement height to 5mm.
P_XXM_0501_09 P_XPM_0501_09: do not override same pid in the triangle. This use case was not covered.

Update the following test cases to use the filter "linear" or the default, also "linear".

P_DPX_3200_08
P_DPX_3200_10
P_DPX_3200_12
P_DPX_3204_02
P_DPX_3204_04
P_DPX_3204_05
P_DPX_3204_06
P_DPX_3204_07
P_DPX_3208_03
P_DPX_3208_04
P_DPX_3208_05
P_DPX_3208_06
P_DPX_3220_01
P_DPX_3220_03
P_DPX_3222_01_material
P_DPX_3222_02_material
P_DPX_3222_03_material
P_DPX_3222_03_material
P_DPX_3224_01_production
P_DPX_3224_02_production
P_DPX_3226_01_boolean
P_DPX_3226_02_boolean
P_DPX_3226_03_boolean
P_DPX_3228_01
P_DPX_3228_02
P_DPX_3228_03

Include 3MF Volumetric & Implicit extensions schema


#949 Add synchronisation between Start/Stop/Delete actions

Related to https://github.com/code-ready/crc/issues/949
This PR implements cancel for machine start.
If the user clicks stop or delete in the tray, it will first abort startup sequence and then perform stop/delete.

Since you introduce a new struct, have you considered using a regular mutex to protect access to the struct, and either some booleans, or some state enum, startInProgress bool stopInProgress bool deleteInProgress bool? This would be less scary than these locks, trylock/,....

It is already hard to deal with 2 locks and one cancel function. I don't want to have to deal with more objects.
The test coverage should enable refactorings/enhancements.

Hard to know the exact behaviour you are after from the one line logs, so no idea if this is workable or not ^^

This is why I added 3 tests. It will be easy to change the implementation. If there is something missing add a test, fix the code and move on.
Hundred lines of commit log doesn't worth a single test.
I can also add: I don't trust a commit log, I trust tests.

Since you introduce a new struct, have you considered using a regular mutex to protect access to the struct, and either some booleans, or some state enum, startInProgress bool stopInProgress bool deleteInProgress bool? This would be less scary than these locks, trylock/,....

It is already hard to deal with 2 locks and one cancel function. I don't want to have to deal with more objects.

The point would be there would be a single lock protecting the instance, lock, read value you are interested in, unlock, act on the value that you read. No need for additonal locks, for trylock, ..

Hard to know the exact behaviour you are after from the one line logs, so no idea if this is workable or not ^^

This is why I added 3 tests. It will be easy to change the implementation. If there is something missing add a test, fix the code and move on.
Hundred lines of commit log don't worth a single test.
I can also add: I don't trust a commit log, I trust tests.

Yup, tests definitely will be the source of truth for the code behaviour. However, as a reviewer, I can spend time reverse engineering the intended behaviour from the test code, I might misunderstand it, or the test may not be doing what you wanted it to do, I'll have no way of knowing. And the next person interested in knowing how this code should behave will have to do exactly the same thing. A descriptive commit log will save time overall, and will make the reviews faster and better..

I can also add: I don't trust a commit log, I trust tests.

If tests are defined in a readable way, with clear intentions in code, there is no commit log that would make that clearer.
I also trust tests more; working code.

Not sure why the discussion seemingly turned into "it is either tests or commit logs, cannot have both" :)

Since you introduce a new struct, have you considered using a regular mutex to protect access to the struct, and either some booleans, or some state enum, startInProgress bool stopInProgress bool deleteInProgress bool? This would be less scary than these locks, trylock/,....

I've experimented with this, replacing the semaphores/trylock/... with a mutex, a channel and a state enum.
https://github.com/cfergeau/crc/tree/sync
https://github.com/cfergeau/crc/blob/sync/pkg/crc/machine/sync.go
The tests added in this PR pass after this change. It's more code than in the PR, and unfortunately there's still some magic going on (the async state updates through a go channel). I haven't carefully re-read the code looking for races. It's probably more idiomatic go code than using multiple semaphores+trylock. The 'state' this introduce could be useful as a high level state to be shown in the tray.

guillaumerose requested a review from cfergeau 2 hours ago

I'm not sure how to move forward here. I don't think the initial commits are that useful as most of their code is removed by Rework sync code. I can split that Rework commit along the same lines as your initial PR if needed.
Since a good chunk of the final code was written by me, I don't think that's right that I give this a review :)

I squashed most commits.
PTAL @code-ready/crc-team

Do we want debug logging in the synchronization ?

Do we want debug logging in the synchronization ?

Let's add logs if we encounter bug or someone complains. Logs are already pretty big.

Let's add logs if we encounter bug or someone complains. Logs are already pretty big.

Agree.... sorry for the delay, but have been busy today.
/lgtm
... but merged anyway ;-)


SVG

Description
When exporting a SVG file in AI using Internal CSS settings, the <style> is placed within <defs>. The css definitions are ignored when the SVG is rendered.
Steps to Reproduce
Export a SVG file using Internal CSS settings in AI (using a few different colors).
Expected Behavior
SVG getting rendered with the colors
Actual Behavior
SVG getting rendered in black
Work around
Exporting SVG using inline styles or manually removing the <defs> tag solves the problem.

Thanks to @xtergs it's implemented now. See https://github.com/luberda-molinet/FFImageLoading/pull/952
Cheers :)


add predicate filter option

Make it possible to retry only upon errors that are eligible by the predicate option.
Adapted from the original proposal by @gastonelhordoy in #27

@gastonelhordoy You made a good point in #27 regarding predicate functions.
I reworked the implementation so it always uses bluebird's filtered catch, supplying a default predicate that always succeeds if the user isn't using one. I also updated your tests slightly to verify it works if the user supplies either an error class name or a predicate function.

Thanks @demmer! I think it's pretty much what I was looking for. Are you gonna releasing a new version with this?

@gastonelhordoy Done: https://github.com/demmer/bluebird-retry/releases/tag/v0.7.0


Documentation for ServiceNotUsedForReidentification

In https://github.com/privacysandbox/attestation, I see an explanation for ServiceNotUsedForIdentifyingUserAcrossSites. However, in the attestation file, it contains ServiceNotUsedForReidentification. I do not see documentation/explanation for ServiceNotUsedForReidentification in https://github.com/privacysandbox/attestation. Could someone clarify and add an explanation for that?

Google has a PR for this since 20 June.


Extend table for 'Encryption Model and key management table' for support of "Server-side encryption using customer-managed keys in customer-controlled hardware"

The table 'Encryption Model and key management table' (https://docs.microsoft.com/en-us/azure/security/fundamentals/encryption-atrest#encryption-model-and-key-management-table) is a nice summary for encryption model for data at rest, but its missing a column for services supporting "Server-side encryption using customer-managed keys in customer-controlled hardware" (HYOK) scheme, albeit they're rare - but in certain scenarios it is an important feature to evaluate the possibility of using a certain service.

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

ID: cd0472f9-1ed9-0154-3bfd-b25e5da64cd1
Version Independent ID: 846187d4-c24c-279f-e216-80b6507228b3
Content: Microsoft Azure Data Encryption-at-Rest
Content Source: articles/security/fundamentals/encryption-atrest.md
Service: security
Sub-service: security-fundamentals
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@Drag0nR3b0rn
Thanks for your feedback! We will investigate and update as appropriate.

Please let us know if this is still an issue. #please-close


Login from android client error : Cannot read property 'id' of null

Dear developper,
I've magoware portal up and running, created a new customer, but got database error on
android client (apk included in git) when try to log in :
2021-01-22T09:46:51.361Z - error: device upsert error :  Cannot read property 'id' of null
TypeError: Cannot read property 'id' of null
at addDefaultMediaPlayer (/usr/local/src/magoware/modules/deviceapiv2/server/controllers/credentials.server.controller.js:567:47)
at process._tickCallback (internal/process/next_tick.js:68:7)
POST /apiv2/credentials/login 200 151 - 42.562 ms
In advance, thanks for your help.

@PHHENS please retrieve the latest code and run the migrations.

Troubles with subscriptions :
Server Side : subscription appears when editing user, but not in
subscription menu.
Client Side :  No subscription (cancel or buy subscription)
hope to have something work ... maybe ... one day ...
Le vendredi 26 février 2021 à 12:53 -0800, Malfor Saja a écrit :

@PHHENS please retrieve the latest code and run the migrations.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


[ArchBoard] Standardize Uploading and Downloading Methods

Reviews for standardizing these methods across languages may introduce API changes.

Standardization done


[FLINK-9975][hadoop] also shade Hadoop's io.netty dependency

What is the purpose of the change
This is to prevent user code from clashing on Netty which we relocated in the hadoop jars using its old namespace: org.jboss.netty. Hadoop >= 2.7, however, upgraded Netty and relies on the new package: io.netty.
Brief change log

relocate io.netty to org.apache.flink.hadoop.shaded.io.netty

Verifying this change
This change is already covered by existing tests, such as the yarn, hdfs, and end-to-end tests.

I also verified the jars in the lib/ folder contain the same dependencies (for hadoop 2.8.3) except the relocated io.netty.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no (modifies relocation, though!)
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

+1

rebased to master, did the same checks for jars in lib/ - now waiting for just another Travis run to be safe here, before merging


Discussed with Xin and refine all comments in TS.

Need refine the codes in TS and circom based on comments

Old and abandon.


Using multiple registry.group<>() calls gives ENTT_ASSERT "Conflicting groups"

I read your blog and I am trying to use multiple owning groups, since your blog described that it was the fastest way to iterate.
I keep getting conflicts when I try to use multiple 'owning groups'.
When I change the group to a view it works fine.
In the entt.hpp file it says:
* Groups are created on the fly and share with the registry its internal
* data structures. Feel free to discard them after the use.
* Creating and destroying a group is an incredibly cheap operation. As a
* rule of thumb, storing a group should never be an option.
My Assumption: I think the group variables are not destroyed after the method call ended.
Are groups only unique by the first type that is provided and is this causing conflicts?
Also: I don't need any sort functionality, just iterate and update / read variables of components mostly.
Anyway the simplified code looks something like this:
main() {
    init();
    loop();
}

// called once on startup
init() {
	// Fill the entt registry with a bunch of entities with components.
	// Example: A single entity now has components:
	// - TransformComponent
	// - Component3D
	loadComponents();

	// Do some other stuff...

	// Determine which entities are visible and add a RenderableComponent to those.
	auto all3DComponents = registry.group<Component3D, TransformComponent>();
	for(auto entity: all3DComponents) {
		registry.emplace<RenderableComponent>(entity);
	}

	// Example: A single entity now has components:
	// - TransformComponent
	// - Component3D
	// - RenderableComponent

	initRenderSpecific();

	// Example: A single entity now has components:
	// - TransformComponent
	// - Component3D
	// - RenderableComponent
	// - Renderer3DVulkanComponent
}

// called by init
initRenderSpecific() {
	// Determine renderer (opengl / vulkan / etc...)

	// Create specific render object for the render type.
	// So iterate all renderable components.
	// >>>>>>>>>>>>>>>>>>>>>>>>>>>> THE METHOD BELOW sometimes HAS A CONFLICT <<<<<<<<<<<<<<<<<
	auto allRenderableComponents = registry.group<TransformComponent, RenderableComponent>();
	// <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<

	for(auto entity: allRenderableComponents) {
		registry.emplace<Renderer3DVulkanComponent>(entity);

	}
}

// called by main.
loop() {
    while(true) {
        update();
        render();
    }
}

// called allot by loop
update() {
	>>>>>>>>>>>>>>>>>>>>>>>>>>>> THE METHOD BELOW HAS A CONFLICT <<<<<<<<<<<<<<<<<
	auto group = registry.group<TransformComponent, Renderer3DVulkanComponent>();
	//... loop group and update values of Renderer3DVulkanComponent based on camera matrix * TransformComponent
	<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
}

// called allot by loop
render() {
	// Apparently a group can't have a single component, so can't iterate fast, so should I use a view or use group with an additional component that we just don't use?
	auto view = registry.view<Renderer3DOpenGLComponent>();
       // or
	auto group = registry.view<Renderer3DOpenGLComponent, RenderableComponent >();

        // .. loop view/group and for each Renderer3DOpenGLComponent call renderComponent().

        // Note: RenderableComponent is just a component with bool visible, which in this case is always true, it is thus not actually used.
        // Renderer3DOpenGLComponent are only added for components that have a RenderableComponent with visible true)
}

Blog I'm referring to: https://skypjack.github.io/2019-03-21-ecs-baf-part-2-insights/
*note: all for loops are actually parallel, an example looks like this:
std::for_each(std::execution::par, std::begin(group), std::end(group), [&group, &currentLocation, &moveVector, this](auto &entity) {
	// *** loop code here ***
};

** To get this working for all platforms I had to compiled my own version of clang, libc, pstl and tbb, since windows and android do not have parallel support out of the box when compiling with clang. **

My Assumption: I think the group variables are not destroyed after the method call ended.

They are not. Groups have an initialization step that you don't want to pay all the times. Therefore, you undergo this cost once (when the pools are likely empty or almost empty) and that's it.
The error about conflicting groups is due to:
registry.group<TransformComponent, RenderableComponent>();

Versus:
registry.group<TransformComponent, Renderer3DVulkanComponent>();

Where TransformComponent can be an owned type only for a group.
That being said, I suggest you to work with views until you've a perf problem, then optimize hot paths (switching to groups) if this ever happens.

Closing as solved/starving issue. Feel free to continue the discussion here if this isn't the case.


Add Windows Media Keys to vk_codes

Added these to the library locally as they were useful for my app, and figured I may as well PR them.

thanks


Cache Operation

Environment:
@apollo/experimental-nextjs-app-support: "0.5.2"
@apollo/client: "^3.8.0-rc.2"
next.js: "14.03"
I have an SSR query initialized as follows:
function makeClient() {
  const httpLink = new HttpLink({
    uri: "http://localhost:4000/",
    fetchOptions: { cache: "no-store" },
    credentials: "include",
  });
  return new NextSSRApolloClient({
    connectToDevTools: true,
    cache: new NextSSRInMemoryCache(),
    link:
      typeof window === "undefined"
        ? ApolloLink.from([
            new SSRMultipartLink({
              stripDefer: true,
            }),
            httpLink,
          ])
        : httpLink,
  });
}

Query:
const {
  data,
  loading,
  error,
} = useQuery(GET_DATA, {
  variables: { ... },
});

I have two questions..

When I set fetchOptions: { cache: "no-store" } for the query, I noticed in Apollo Client Developer Tools that the cache still contains the data from the previous query.
When I found that Apollo Client Developer Tools showed the cache for my query, I tried to re-query, but it didn't use the cache. I attempted to add fetchPolicy: "cache-and-network" to the GET_DATA query, but I still couldn't successfully retrieve data from the cache.

Please assist me in correctly utilizing Apollo Client's caching functionality. Much appreciated!

useQuery will start a network request after the component rendered - and at that time, SSR will already be finished and only transport a loading state to the client.
All we can do from this package's perspective is ensure that we prevent an unnecessary (as there's no way of transporting it) request on the server.
You probably want to use useSuspenseQuery instead, since that will actually pause rendering on the server until data is there and then transport the data over.

Hey @luc345 👋
Just to double check, you're trying to avoid storing data in the Apollo cache correct? You'll want to do this with a fetchPolicy of no-cache, which you'd add to your useQuery hook.
const { data } = useQuery(GET_DATA, { fetchPolicy: 'no-cache' })

cache: 'no-store' is a Next.js cache API , which is probably why you're still seeing that data in the Apollo Cache (so many caches!)

@phryneas
I apologize for providing incorrect information yesterday. I realized that the fetchPolicy: "cache-and-network" I used not only prioritizes the cache but also sends requests to the server to update it, ensuring fast web page loading and accurate cache data. Yesterday, I was unaware of this behavior, so when I observed queries being made to the server during my checks, I mistakenly thought the cache wasn't being utilized.
Today, I tried using useSuspenseQuery and useQuery for caching, and the results so far indicate that the cache is working correctly.
Thank you for your help :)
@jerelmiller
I implemented the method you suggested and indeed, I no longer see the cache in the Apollo Client Devtools.
Thank you for your help :)

I think your problems are solved so far, so I'm closing this issue. Feel free to open a new issue if anything comes up :)


[Bug]

Before Reporting

[X] I found no existing issues matching my bug
[ ] My issue is not caused by the Millennium interface inside Steam (Themes, Plugins & Update Panel). If it is, make a issue report in this repo.
[X] My issue is not caused by a theme

Describe the Bug
Open Millennium is not shown in the steam interface. Steam will not appear if you switch to beta or home beta. This open millennium appears when no beta is selected. Hope that the author can be improved to fix this bug. Can be used in any beta version.
Expected Behavior
Open Millennium is not shown in the steam interface. Steam will not appear if you switch to beta or home beta. This open millennium appears when no beta is selected. Hope that the author can be improved to fix this bug. Can be used in any beta version.
Steps To Reproduce
Open Millennium is not shown in the steam interface. Steam will not appear if you switch to beta or home beta. This open millennium appears when no beta is selected. Hope that the author can be improved to fix this bug. Can be used in any beta version.
Operating System
Windows
Anything else?
Open Millennium is not shown in the steam interface. Steam will not appear if you switch to beta or home beta. This open millennium appears when no beta is selected. Hope that the author can be improved to fix this bug. Can be used in any beta version.

Use Millennium beta.
https://docs.steambrew.app/users/installing


Invisible face and DGESVD illegal parameter error

When running the attached example, I run into an error where
 ** On entry to DGESVD parameter number 11 had an illegal value

is spammed to my terminal and the model's face is invisible/nonexistent (see below)

This may have to due with me installing OpenBLAS due to CMake recommending it, or something along the lines of a broken LAPACK install, so I'm trying to fiddle around with both of those.

Yes, sorry, it turns out that OpenBLAS seemed to have some issues on my system.
Thanks for the project!


Downgrade qos to client sub if needed

Solution for #208

Yes we do! Can you use them in this test?

@mcollina I've already used the publish and subscribe helper functions in the test 👍.


🛑🛑🛑Please contact customer support🛑🛑🛑

when try command AL:GO return an error

i working in 64 bit Ubuntu 20.04.5 LTS

Please describe in more detail what your issue seems to be? Which error are you experiencing when using the AL: Go! command in the AL Language extension?

This repository no longer handles general questions and is only dedicated to bug reports and feature suggestions for the latest preview version of the AL compiler and developer tools.
To get help for your question please use one of the following resources:
Business Central Community
mibuso forum
Dynamics User Group


Decrease the For loop with x..y where (x > y)

Hi!
Rust newbie here!
I want to know if it's possible to implement the simple solution to decrease a 'For' loop for values between x and y (with x > y obviously...) please?
For example:
for x in (10..0) {
//some stuff...
}

Thanks a lot!

What does decrease mean?

See https://internals.rust-lang.org/t/step-by-on-negative-numbers/2231

Oh, thanks!
I use personnaly for i in (0..10).rev(), but i think it's borring to do that kind of thing...
Sorry for the double issue!


Black screen

When I cast something like Crucio or stupefy (spells that run people into rag-dolls) it will make the victim disappear and their screen will go black... same if I cast it on myself...

DarkRP gamemode?
Provide a detailed description of your problem please.


adding acs 2006-2010 metadata mapping, received from erica

this should address #153
Script used to generate this file:
import pandas as pd
import requests
import ast
import json

df = pd.read_csv('meta2010.csv')
df['census_variable'] = df['census_variable'].apply(ast.literal_eval)
df['rounding'] = df['rounding'].fillna('0').astype(int)
df['source'] = ''
df = df.loc[df.Notes != 'remove', ['pff_variable','census_variable', 'base_variable', 'domain', 'rounding', 'source']]
df_dict = df.to_dict('records')
with open('metadata.json', 'w') as f:
    json.dump(df_dict, f, indent=4)

Running in github actions to test out results:
https://github.com/NYCPlanning/db-factfinder/actions/runs/1052808059

Seems like we have a few failures:
⛔ FAILURE: pbwpv	city
⛔ FAILURE: pbwpv	borough

⛔ FAILURE: pu18pvu	NTA
⛔ FAILURE: pu18pvu	CDTA
⛔ FAILURE: pu18pvu	CT20
⛔ FAILURE: pu18pvu	city
⛔ FAILURE: pu18pvu	borough

⛔ FAILURE: smpntc	NTA
⛔ FAILURE: smpntc	CDTA
⛔ FAILURE: smpntc	CT20
✅ SUCCESS: smpntc	city
✅ SUCCESS: smpntc	borough

⛔ FAILURE: avgfmsz	NTA
⛔ FAILURE: avgfmsz	CDTA
⛔ FAILURE: avgfmsz	CT20
✅ SUCCESS: avgfmsz	city
✅ SUCCESS: avgfmsz	borough

⛔ FAILURE: mafr	NTA
⛔ FAILURE: mafr	CDTA
⛔ FAILURE: mafr	CT20
⛔ FAILURE: mafr	city
⛔ FAILURE: mafr	borough

⛔ FAILURE: norway	NTA
⛔ FAILURE: norway	CDTA
⛔ FAILURE: norway	CT20
⛔ FAILURE: norway	city
⛔ FAILURE: norway	borough

⛔ FAILURE: grpintc	NTA
⛔ FAILURE: grpintc	CDTA
⛔ FAILURE: grpintc	CT20
✅ SUCCESS: grpintc	city
✅ SUCCESS: grpintc	borough

⛔ FAILURE: popinfms	NTA
⛔ FAILURE: popinfms	CDTA
⛔ FAILURE: popinfms	CT20
✅ SUCCESS: popinfms	city
✅ SUCCESS: popinfms	borough

✅ SUCCESS: p65plbwpv	NTA
✅ SUCCESS: p65plbwpv	CDTA
✅ SUCCESS: p65plbwpv	CT20
⛔ FAILURE: p65plbwpv	city
⛔ FAILURE: p65plbwpv	borough

⛔ FAILURE: pu18bwpv	NTA
⛔ FAILURE: pu18bwpv	CDTA
⛔ FAILURE: pu18bwpv	CT20
⛔ FAILURE: pu18bwpv	city
⛔ FAILURE: pu18bwpv	borough

⛔ FAILURE: cameroon	NTA
⛔ FAILURE: cameroon	CDTA
⛔ FAILURE: cameroon	CT20
⛔ FAILURE: cameroon	city
⛔ FAILURE: cameroon	borough

⛔ FAILURE: sweden	NTA
⛔ FAILURE: sweden	CDTA
⛔ FAILURE: sweden	CT20
⛔ FAILURE: sweden	city
⛔ FAILURE: sweden	borough

⛔ FAILURE: fam3	NTA
⛔ FAILURE: fam3	CDTA
⛔ FAILURE: fam3	CT20
✅ SUCCESS: fam3	city
✅ SUCCESS: fam3	borough

⛔ FAILURE: nmsmpntc	NTA
⛔ FAILURE: nmsmpntc	CDTA
⛔ FAILURE: nmsmpntc	CT20
✅ SUCCESS: nmsmpntc	city
✅ SUCCESS: nmsmpntc	borough

⛔ FAILURE: omafr	NTA
⛔ FAILURE: omafr	CDTA
⛔ FAILURE: omafr	CT20
⛔ FAILURE: omafr	city
⛔ FAILURE: omafr	borough

merging for now, and issues will be addressed separately in a different PR


RuntimeError: Sizes of tensors must match except in dimension 0. Expected size 768 but got size 1024 for tensor number 1 in the list.

I am interested in your work.
And I have a question:
The batch size must be 1?
If not, how do I revise the code?

Thank you for your attention to our work.
The current default config is batch size=1, and we have added a check in the code here: https://github.com/franciszzj/VLPrompt/blob/main/kings_sgg/models/relation_heads/relation_transformer_head_v2.py#L413.
If you wish to use a larger batch size, you can refer to this part: https://github.com/franciszzj/VLPrompt/blob/main/kings_sgg/models/relation_heads/relation_transformer_head_v2.py#L421. However, my experimental experience has shown that using a larger batch size may slightly reduce the performance


Data not aligned (bad choice of font)

Screenshot
Using Superset 1.0.1
Here for dates fields :

Here for some prices

Description
When analysing data in a table, one expects them to be nicely aligned.
It's especially the case regarding money amounts.
Fonts should be monospace.

Hi squalou
I am interested in solving this issue, this is my first  PR, can you explain how to solve this issue.

I did an ugly fix on my side, and it's probably NOT the way to go, nevertheless here's what I did.
in file superset-frontend/stylesheets/less/variables.less I removed 'Inter' font entirely
< @font-family-sans-serif: 'Inter', Helvetica, Arial;
---
> @font-family-sans-serif: Helvetica, Arial;

it works but ... it has effects everywhere. (which is fine on my setup)

Are there any update ?
I'm testing 1.2.0-rc2 with my patch, would be glad to get rid of it.

can i take this issue up?

@geido I think you took a look at this quite some time ago using the monospaced variant of Inter. Do you remember where that landed? I seem to recall the difficulty being around knowing/mapping which columns to which we should apply the styling/class

@rusackas the fix was applied to the table in this PR https://github.com/apache/superset/pull/19668. There's probably more work to do to make sure this works fine everywhere but as far as I remember it should look good now in most places.

Ok, closing this for now then... @squalou please feel free to open a fresh issue with updated specifics if we need to investigate this further! Thank you!

Great news of sort :)
Is there an easy way to know in which release this PR has landed ? (if it has already).
I remember trying 2.0.1-rc1 last October, it was not yet right.

I"ll sadly open a fresh issue :/

I'll sadly have to open a fresh issue,
here is a 2.0.1 table with dates


last line of text not rendered with getImage()

I have run into a bug where if you create only a few text objects the last one does not render when using getImage().  If you draw another object like a line, all text objects render.
I found this text bug does not happen using the depreciated canvasForExport().

I just saw this same issue mentioned on the Google Group. The solution is to call  lc.tool.commit(lc);
https://groups.google.com/forum/#!topic/literallycanvas/qfXwXbGYBzo

I just saw this same issue mentioned on the Google Group. The solution is to call lc.tool.commit(lc);

Just to point out, if you switch between multiple tools you should check that lc.tool (the current tool) is the Text tool before calling lc.tool.commit(lc); Otherwise you'll end up with a TypeError.

I just saw this same issue mentioned on the Google Group. The solution is to call lc.tool.commit(lc);

Just to point out, if you switch between multiple tools you should check that lc.tool (the current tool) is the Text tool before calling lc.tool.commit(lc); Otherwise you'll end up with a TypeError.

Hi there, how to get the "current tool"? I have tried to read the documentation but cant find any clues


chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3268

Questions ?
Please refer to the Backport tool documentation


Reworked to use enum class in Qt

Thank you for the PR and sorry for the late response!
Yeah, I guess it's time to modernize the Qt variant. However, I think I would rather introduce it as another variant.
However, I wonder about the Invalid code: why did you choose -1 as value for the invalid code? Why not 0?

I guess 0 could used instead. It's just -1 a common undefined :)

Oh well... in the meantime I realized that I already defined -1 to be the "invalid" code implicitly. 😆

I implemented it as a new variant and pushed it to https://github.com/j-ulrich/http-status-codes-cpp/tree/feature/%2328-Qt-Cpp11.
What do you think?


Remove version: FelipeEFdeCastro.Klavaro version 3.13

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/klavaro/files/klavaro-3.13-win-setup.exe/download
Status Code: 200
(Automated message - build 896)

Close with reason: Package still available.;


api: add Builtin#UnpackArgs as a covenience function

This is a simple convenience function to help address a common mistake I've seen a few developers make when they want to add a skylark Builtin.
I also updated a few places in the Skylark codebase to demonstrate places where it comes in handy.
I don't feel strongly about this. Only offering it as a suggestion.

Seems reasonable to get rid of a bit of redundancy.
@alandonovan should approve, but he's out of office. Can this wait a while?

@jayconrod ya that's fine! there's no rush

that's a totally reasonable api choice! i'll withdraw this pr.


Communicating data between gdscript <-> browser

Is there a way I can send data from the browser side (javascript) to my gdscript and vice-versa?
For example: If a html textbox is typed into can i get the data typed in gdscript?

Hi @3ddelano the current way to interact with JS is given line
https://github.com/Lecrapouille/gdcef/blob/6f00652a97ad75af31e2d247bcb2ddf055ecbfc8/addons/gdcef/demos/2D/CEF.gd#L175
But you cannot get feedback from JS. This is a request https://github.com/Lecrapouille/gdcef/issues/40

Oh thanks for the reply. Right now to get feedback from js im using a http server running in gdscript and the js makes API calls to the server. This works but needs a lot more code to setup.

@3ddelano Using a HTTP server adds a lot of overhead as well. I'm working on a feature that will let you communicate easily between GDscript and the browser via a message protocol. You can use that to transfer JSON strings with instructions for each side as an example.

Oh nice

@3ddelano Using a HTTP server adds a lot of overhead as well. I'm working on a feature that will let you communicate easily between GDscript and the browser via a message protocol. You can use that to transfer JSON strings with instructions for each side as an example.

Do you have any update on this ?

@berkaycimsir for the moment no.

@berkaycimsir for the moment no concerning @pimhakkert

Can you think of any shortcout way for this right now ?
I was able to send data from godot to my browser via MessageChannel  but couldn't find a way to do it from browser to godot side.

@berkaycimsir can you go to my gdcef discord https://discord.gg/EckEwy7S5U to show me a simple example of what you have made ? I need a basic example since I'm not a webdev.

Closing, following the conversion on https://github.com/Lecrapouille/gdcef/issues/40


add RELEASE-NOTES

Pull Request Test Coverage Report for Build 776

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 68.536%

Totals

Change from base Build 526:
0.006%

Covered Lines:
11490

Relevant Lines:
16765

💛  - Coveralls


MAX_TOKEN needs to be parameterized to support various models

Describe the bug
https://github.com/sqlchat/sqlchat/blob/main/src/components/ConversationView/index.tsx#L26-L28
Currently, the MAX_TOKEN is hardcoded as 4000, which is almost the context limit of gpt-3.5-turbo [4096]
However, GPT4 can support 8192 tokens, which is double the MAX_TOKEN, the app can handle larger context in GPT4 if MAX_TOKEN is adjusted accordingly
reference: https://platform.openai.com/docs/models/overview
To reproduce
As above
Additional context
No response

Fixed in https://github.com/sqlchat/sqlchat/commit/088f0904c068907c0761a7cf606b6c5a7c484dcf


CI: coverage only for src

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (9aebe22) 81.62% compared to head (2ee3da4) 77.40%.
Report is 9 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #20      +/-   ##
==========================================
- Coverage   81.62%   77.40%   -4.23%
==========================================
  Files          11        7       -4
  Lines        1230      708     -522
==========================================
- Hits         1004      548     -456
+ Misses        226      160      -66

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


ZIP-239: v5 mempool transaction protocol versions

Motivation
When the mempool is implemented, Zebra must conform to ZIP-239 in order to properly relay transactions to the network.
This ticket focuses on activating the new inv message variant at the specified protocol version, and rejecting it beforehand.
Scheduling and Alternatives
If necessary, this ticket can be delayed until later in the mempool work.
After NU5 activates on testnet and mainnet, we can set INITIAL_MIN_NETWORK_PROTOCOL_VERSION to Nu5. If we make this change, we do not need to implement the consensus rules in this ticket.
Specifications
ZIP-239 describes the consensus rules for a new MSG_WTX inventory type, that MUST be used when relaying V5 transactions.

An inv or getdata message MUST NOT use the MSG_WTX inv type ... on peer connections that have not negotiated at least the peer protocol version specified in Deployment (170014).

it is possible for a node to receive an inv or getdata message with a MSG_WTX inv type, on a peer connection that has negotiated protocol version 170014 or later, before NU5 has activated. The node MUST handle this case in the same way that it would after NU5 activation when it has no v5 transactions to relay

Receiving a MSG_WTX inv type on a peer connection that has negotiated a protocol version before 170014, on the other hand, SHOULD be treated as a protocol error.

Related Work

#2233 and #2446 added an initial Message::Wtx variant for ZIP-239, and implemented serialization and deserialization of it.
The transaction ID for V5 transactions (with the auth_digest) is defined in ZIP-244, and implemented in #2129.
#2449 implements the remaining data structures for WTXs, and sends them for v5 transactions.

We might be able to close this ticket, because it's covered by other tickets.
Here's how we handle each network protocol rule:

An inv or getdata message MUST NOT use the MSG_WTX inv type ... on peer connections that have not negotiated at least the peer protocol version specified in Deployment (170014).

We won't send the MSG_WTX inv type until we have v5 transactions in our mempool.
We don't expect to get v5 transactions until we're connected to zcashd peers which have activated NU5.
After NU5 activation, we'll reject connections from outdated peers.
We should also eventually disconnect existing outdated peers.
Tickets:

always handle MSG_WTX inv types, regardless of network protocol version (various mempool tickets)
increment the CURRENT_NETWORK_PROTOCOL_VERSION to 170014 before NU5 activation on testnet (#1841)
reject connections from outdated peers after a network upgrade (#2519)
increment the MINIMUM_NETWORK_PROTOCOL_VERSION after NU5 testnet activation (#2532)

it is possible for a node to receive an inv or getdata message with a MSG_WTX inv type, on a peer connection that has negotiated protocol version 170014 or later, before NU5 has activated. The node MUST handle this case in the same way that it would after NU5 activation when it has no v5 transactions to relay

We don't activate the mempool until we've reached the chain tip, which means we'll act like it's empty.
We don't expect to get v5 transactions until after NU5 activation (see above).
Tickets:

disable the mempool until we've reached the chain tip (#2592)

Receiving a MSG_WTX inv type on a peer connection that has negotiated a protocol version before 170014, on the other hand, SHOULD be treated as a protocol error.

Zebra will ignore these messages before NU5 activation (see above).
After NU5 activation, it will respond as if the connection had a valid version.
This is acceptable behaviour, because the protocol rule is only a SHOULD, not a MUST.

We don't actually need to implement this ticket, because in the worst case, Zebra:

accepts but drops v5 transactions before NU5 activation
accepts and sends v5 transactions after NU5 activation

Here's some more analysis of interactions with legacy Bitcoin protocol implementations:
Currently, if Zebra receives a 36-byte WTX message, parsing will always fail. The InventoryHash parser will expect body_len + 32 bytes for WTX, but the message parser only provides body_len bytes.
The body_len limit is applied even if the message has trailing bytes.
(This assumes the peer is honest-but-buggy. A malicious peer could modify body_len as well. But if they can do that, they can send arbitrary data, so there's no additional vulnerability here.)


Use logback and offer structured logging options

install logback-classic and exclude conflicting transitive dependencies in pom.xml

add a default logback.xml file

update the README to link to the logback documentation and point out how and where to enable structured logging

add test.Dockerfile to build a deployable docker image from a locally built fatjar

Thanks for your contribution @n-oden. JDBC bridge is not actively maintained on server side, so I'd suggest you using another table function like url instead. I'll find time to implement a new module in clickhouse-jdbc repo to retire this one.

Thanks for your contribution @n-oden. JDBC bridge is not actively maintained on server side, so I'd suggest you using another table function like url instead. I'll find time to implement a new module in clickhouse-jdbc repo to retire this one.

Oh, oops. You might want to mark this repo as archived then to keep people away from it. I'll take a quick look at the clickhouse-JDBC repo and see if it's difficult to port this change over to it.


Enable html tags inside the markdown file

Hello,
I don't know if this works already but I didn't manage to make it work: I would like to add some html tags inside my markdown files so that I can add custom styling and settings. Right now, everything that is inside the html is not parsed by the parser any more. Would this be possible?
I would like for example to have this working:

![Caption](img url)

Caption

If you are using the Python Markdown parser, this is already possible with the extra extension.  You simply create your block HTML tags with the attribute markdown="1".  You are tough out of luck if you are using the Github parser as we don't control the Github parser; we simply send your content to them via their API and they return the result.

Oh, this is great! Thank you for the tip!


fix(sentry): clear name in device context

Is there anything else I need to do, here? We're not stripping out anything in the web error reporter - https://github.com/Jigsaw-Code/outline-client/blob/329df444be6ba91b212bb96727f6941ff6d750e3/src/www/app/error_reporter.ts
Also how do I test this - what are our sentry environments?

LGTM too


Redraft and update ansible README.md (Phase 1!)

Draft until https://github.com/AdoptOpenJDK/openjdk-infrastructure/pull/993 is merged as this is based on it

Above updates from will have been made in another PR

Above updates from will have been made in another PR. Merging based on Will's review


[WIP] Implemented delivery order annotation

Signed-off-by: Francesco Guardiani<EMAIL_ADDRESS>Part of #589

Proposed Changes

:gift: Implemented delivery order annotation

Release Note

Now eventing-kafka-broker implements ordered delivery. You can choose between ordered and unordered delivery through the annotation kafka.eventing.knative.dev/delivery.order

Docs

Coming soon in another PR to docs repo

I'm going to implement a basic e2e test, although I'm not sure how far I can go with testing this feature in a e2e manner

/retest

Because I would love to use reconciler-test for the e2e, I've opened this one https://github.com/knative-sandbox/eventing-kafka-broker/pull/703

/this-is-fine :heart_eyes:

If you want we can have a status condition to indicate the DeliveryOrder configured for a Trigger since it's easy to make a typo and then the DeliveryOrder configured is the default one rather than the one users wanted.

/kind enhancement

If you want we can have a status condition to indicate the DeliveryOrder configured for a Trigger since it's easy to make a typo and then the DeliveryOrder configured is the default one rather than the one users wanted. (A separate PR is fine too)

I don't see the point TBH, there's no need to tell the user the controller read the contract properly IMO

I mean an annotation is weak compared to a key in the spec because we don't have a way to validate that the key is correct for that reason a status condition would be nice to have IMO.

@pierDipi can we tackle the status issue in another PR? From my understanding (I might be wrong here) we need to setup some alternate condition set, like done in receiver_condition_test.go, right? If that's the case, i think it deserves a separate PR

/retest
Something wrong in maven?!

Seems like ordered delivery is not really ordered, will investigate

/hold

The reason why it's failing is veeeery weird. It doesn't add the delivery order in the json:
{
  "generation": "38",
  "resources": [
    {
      "uid": "87585205-e53a-442f-8e4b-90192cf2cc0b",
      "topics": [
        "knative-broker-test-oymfewtk-broker-rtbugxnl"
      ],
      "bootstrapServers": "my-cluster-kafka-bootstrap.kafka:9092",
      "ingress": {
        "path": "/test-oymfewtk/broker-rtbugxnl"
      },
      "egresses": [
        {
          "consumerGroup": "5d4fcef4-2551-4f2e-8f7b-88fdf4697015",
          "destination": "http://sink-ngsdgoqr.test-oymfewtk.svc.cluster.local/",
          "uid": "5d4fcef4-2551-4f2e-8f7b-88fdf4697015"
        }
      ]
    }
  ]
}

I've removed broker_test.go, since upstream movement around features is unstable now...
For me we're ready to go
/unhold

/hold
one sec, maybe it's better to use a label (that's what other people does :smile: )

/unhold
Ok now we use labels

I see no benefits in using a label here, basically, the difference is that using a label I can query all objects with that label without doing a full scan or building a custom index whereas an annotation requires a full scan or a custom index.
For example, we use a label in our triggers to get fast access to all triggers associated with a broker.
https://github.com/knative-sandbox/eventing-kafka-broker/blob/1258e973843974a5a7fc42aff651cab4ed2a3f56/control-plane/pkg/reconciler/trigger/controller.go#L141-L157
In this case, a query like "give me all ordered trigger" seems pretty useless.
Of course, the benefit of using a label comes with additional storage, memory, and compute costs, so my preference would be to use an annotation since this value is only useful in a single reconcile cycle.

@pierDipi the reason i went for a label is to align with kafka source: https://knative.dev/docs/eventing/samples/kafka/source/#optional-specify-the-key-deserializer
But yeah i was checking in serving and they use annotations too for "special"/"experimental" features, so i'll revert and go for the annotation

I wonder how this works given the issue on Slack :thinking:

/unhold

I wonder how this works given the issue on the partitionkey on Slack thinking

My test tests a single partition :) I'm going to develop a test with different partitions once partitionKey is fixed

My test uses a broker with a single partition :) I'm going to develop a test with different partitions once partitionKey is fixed

Oh, gotcha!


[BUGFIX]Copy file url to clipboard

Bug fixed for copying file url to clipboard

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


2.4.1

fix: Updating importlib-metadata dependency

Hi, is it possible to bump importlib-metadata version to the previous value?
or it requires additional work?


fix: don't duplicate dash in URL

As of recent Flarum versions, the id is automatically included in the slug. This fixes the wrong canonical URL being set.

Hey @askvortsov1, do you think this PR should also close this issue (https://github.com/v17development/flarum-blog/issues/105)?

Nope, that's sadly separate. The integration between blog and the SEO extension is completely broken, no canonical URLs are being applied.

I've pushed another fix to this branch that prevents an error message showing up for non-logged-in users.


Add ability to pass config via secret

Issue submitter TODO list

[X] I've looked up my issue in FAQ
[X] I've searched for an already existing issues here (legacy) and here
[X] I've tried installing latest charts and the issue still persists there
[X] I'm running a supported version of the application & chart which is listed here

Describe the bug (actual behavior)
Right now it's possible to pass pre-created config file from ConfigMap using yamlApplicationConfigConfigMap value. It's a feature request to add similar value, but getting config from secret using new yamlApplicationConfigSecret value.
This will be beneficial for GitOps based deployments. E.g. in our case we have quite a long config file (300+ lines), so having everything defined as env variables is quite tricky. Mainly since we have to store kafka credentials encrypted via SealedSecret, so in case we add/remove clusters we have to change quite a lot of variables.
With such approach it will be possible to store the config completely encrypted and handled from a single place.
I can work on this if you think this might be useful for others and have chances to be merged.
Expected behavior
No response
Your installation details
It's rather a feature request
Steps to reproduce
It's rather a feature request
Screenshots
No response
Logs
No response
Additional context
No response

Hi, ideally this could be both -- yaml config and encrypted secrets passed separately via env variables.
There's an open issue regarding the fact that config is not currently overridable by external env vars, but we plan on fixing that for sure.

There's an open issue regarding the fact that config is not currently overridable by external env vars
Oh, most probably this is what I've faced, wasn't sure if this is a bug or I messed with env variables format somehow. Could I ask you to share a link to this bug? Unfortunately wasn't able to find it neither in the new nor in the old repos.

From my point of view being able to pass config via secrets is good since you won't be forced to change variable naming if you delete a cluster. Let me share some example:
If we have 3 clusters:
---
kafka:
  clusters:
    - name: a
      ...
    - name: b
      ...
    - name: c
      ...

we should define credentials using the next variables:
KAFKA_CLUSTERS_0_PROPERTIES_SASL_JAAS_CONFIG: ...
KAFKA_CLUSTERS_1_PROPERTIES_SASL_JAAS_CONFIG: ...
KAFKA_CLUSTERS_2_PROPERTIES_SASL_JAAS_CONFIG: ...

Now, if we delete cluster a, we'll have to rename all env variables, since indentation there is based on list index. Being able to encrypt the config completely could potentially solve this issue. Does this make any sense? Or maybe I'm try to define them intially wrongly?

I can work on this if you think this might be useful for others and have chances to be merged.

@timchenko-a please do!


Make :total_degree default

It happens a lot, e.g., #163, that the :warmup method does not find roots. We should probably make the :total_degree default and recommend  :warmup for performance where it should be compared to :total_degree.

@jdelpino @jkosata Do you guys agree?


feat: remove injected iframes on unmount

Hi @rfoel, thanks for this package, I found it extremely useful today.
I noticed though that it did not unmount the iframes that are created by the zendesk script. If this is by design please feel free to ignore me and close this PR ;)

@f5io thanks for the PR!


Implement dynamic exchange fees

Feature Proposal
Implement and calculate dynamic exchange fees on Kwenta. Two deliverables:

Calculate dynamic fee and show it if in effect next to current base fee
Add a tool tip if dynamic fees is in effect (see screenshot for text example)

For the design details, please follow #263.
Rationale
SIP-184 introduces dynamic exchange fees which kick in during turbulent market periods in order to avoid frontrunning opportunities. The feature is currently in testing phase.
Implementation

Color: #FFDF6D
Icon: Contact Community PM
For the total fee cost of an exchange, see feeRateForExchange here.
exchangeFeeRate should get you the base fee here.
Tooltip text: This transaction will incur an additional dynamic fee due to market volatility.

Notes
Make sure to install contracts-interface version 2.58.0-alpha to get access to the Kovan and Optimistic Kovan deployment of the Exchanger with dynamic fees.
Note the ticket will not be merged until tested on mainnet post Futures launch
Design

Reward
75 SNX

Assigned to @LeifuChen


Change the doc-generation function to produce a well known JSON definition file as the documentation artifact, instead of markdown.

Not stale - currently lower priority

Not stale - currently lower priority


Metric Definition Duplicates

Issue #, if available: https://issues.amazon.com/issues/AIML-ADC-6788
Description of changes:

Adds a check to ensure that when built-in containers are used for BYOM, duplicate metrics aren't created
Updated unit tests to work with new testing

ORIGINAL HPO JOB:

COPIED HPO JOB:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

LGTM after all other conversations are resolved


add test for app

Add test for new app creation

Coverage decreased (-0.05%) to 37.839% when pulling a84e78ddda5692d7c5b7a9eb97ddc5405db4caca on devghosh:app_test into 5485444a70eb6b73234999f603732a86fb1fedeb on jabong:master.

Coverage decreased (-0.05%) to 37.839% when pulling b61660fd8c54162f9726c18c5c52b2f599ac996a on devghosh:app_test into 5485444a70eb6b73234999f603732a86fb1fedeb on jabong:master.

Coverage decreased (-0.05%) to 37.839% when pulling b61660fd8c54162f9726c18c5c52b2f599ac996a on devghosh:app_test into 5485444a70eb6b73234999f603732a86fb1fedeb on jabong:master.

Coverage decreased (-0.05%) to 37.839% when pulling b675f3a2ef8c81ae9074cf976ab3c0ed1209acce on devghosh:app_test into 5485444a70eb6b73234999f603732a86fb1fedeb on jabong:master.

Coverage decreased (-0.05%) to 37.839% when pulling b675f3a2ef8c81ae9074cf976ab3c0ed1209acce on devghosh:app_test into 5485444a70eb6b73234999f603732a86fb1fedeb on jabong:master.

Coverage decreased (-0.05%) to 37.839% when pulling 14043ef8a3e92327a42cdb3151d4283d0155e655 on devghosh:app_test into 5485444a70eb6b73234999f603732a86fb1fedeb on jabong:master.

Coverage decreased (-0.05%) to 37.839% when pulling 14043ef8a3e92327a42cdb3151d4283d0155e655 on devghosh:app_test into 5485444a70eb6b73234999f603732a86fb1fedeb on jabong:master.

Coverage decreased (-0.05%) to 37.839% when pulling 14043ef8a3e92327a42cdb3151d4283d0155e655 on devghosh:app_test into 5485444a70eb6b73234999f603732a86fb1fedeb on jabong:master.


Add react-router as a peer dependency

With webpack, react-router is being bundled twice when using connected-react-router. I guess it should be a peer dependency rather than a dependency, right?

Fixed in v4.4.0.


Please add an option to change the font systemwide.

📝 Provide a description of the new feature
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
We can do this by playing with registry but it only applies to desktop apps. But Windows Start Menu and UWP apps still use the default Seagoe UI font. It would be great to have an option to change the font systemwide.
If you'd like to see this feature implemented, add a 👍 reaction to this post.

@deathcrafter
feel free to add your comments to https://github.com/microsoft/PowerToys/issues/6918

@deathcrafter
feel free to add your comments to https://github.com/microsoft/PowerToys/issues/6918

Thanks for letting me know.😊

https://winaero.com/winaero-tweaker/
This program can change text and icons in various UI places.

https://winaero.com/winaero-tweaker/
This program can change text and icons in various UI places.

not found how can change PT run fonts use this program, can you share?


[MM-34336] Convert MainMenu component to TypeScript

Summary
Migrates 'components/main_menu' and tests from jsx to tsx
Ticket Link
Jira: https://mattermost.atlassian.net/browse/MM-34336
https://github.com/mattermost/mattermost-server/issues/17267
Comments
I'm still fairly new to TypeScript, so I'm also taking this as a learning opportunity.
I'm a bit unsure whether there is an existing type we could use for the pluginMenuItems
But please fell free to provide any kind of feedback!
NONE

Thanks for the PR! Unfortunately, you'll also need to resolve some conflicts since we just removed a bunch of code from those files. Hopefully that makes fixing the other issues easier 🙂

Thanks! I'll do that

For your questions:

If you change the type of state from the GlobalState defined in mattermost-redux to the one defined in types/store, that should give you the type for pluginMenuItems. The one in types/store extends the other one to add some more stuff that's specific to the web app since the other one is used elsewhere.

Yup, just find out, makes sense. And thanks for the clarification

The package-lock changes are because you're using an older version of NPM prior to NPM 7. If you don't want to upgrade NPM right now, everything should still work if you just remove the changes to the package-lock from your PR

Ah great, thanks! Yeah, I'm still some-what new to the whole npm thingy.
Your comments are much appreciated!

@iOSGeekster Can you resolve conflicts and update? Thanks 🙂

Sure thing! It is updated now 👍

Thanks for participating in Hacktoberfest! You can claim your sticker set here: https://get.printfection.com/hacktober21/4144583267 @iOSGeekster


How to pass Map type arguments to EvalWidget?

Hi, recently I successfully passed a String type variable to EvalWidget, but something went wrong when I try to pass a Map.
I tried to pass a map directly, I also tried to pass $Map.wrap(map), but none of them worked.
My Flutter version: 3.7.12
flutter_eval version: 0.6.0
Here is my code 👇
import 'package:flutter/material.dart';
import 'package:flutter_eval/widgets.dart';
import 'package:flutter_eval/flutter_eval.dart';
import 'package:dart_eval/dart_eval_bridge.dart';
import 'package:dart_eval/stdlib/core.dart';

void main()  {
  runApp(const MainApp());
}

const String code = '''
import 'package:flutter/material.dart';

class MyWidget extends StatelessWidget {
  Map<String, dynamic> map;
  // Map<String, dynamic> map = {'title': 'Hello'};

  MyWidget(this.map, {super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Text(
          map['title'],
          style: const TextStyle(fontSize: 50),
        ),
      ),
    );
  }
}
''';

const Map<String, dynamic> map = {'title': 'Hello World'};

class MainApp extends StatelessWidget {
  const MainApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: EvalWidget(
        packages: const {
          'example': {'main.dart': code}
        },
        assetPath: 'assets/001.evc',
        library: 'package:example/main.dart',
        function: 'MyWidget.',
        args: [
          $Map.wrap(map),
          null,
        ],
      ),
    );
  }
}

Looking for a solution. Thanks!

By the way, FontWeight.w100 to FontWeight.w900 worked, but Null check operator used on a null value error occur when using FontWeight.bold or FontWeight.normal

In order to pass a Map you must wrap the values it contains:
final map = {$String('title'): $String('Hello World')};

You can also use runtime.wrapRecursive() to do this automatically if the map contains only primitive values.
FontWeight issue is fixed in v0.6.1.
Custom fonts can be added in the host app's assets and then used in the flutter_eval code.


Question: Why is int char code used instead of a char?

Hey @puzrin thanks for an excellent parser!
I've been trying to understand why we use integer character codes instead of characters of type String in the code.
For example this line:
if (state.src.charCodeAt(pos++) !== 0x3E/* > */) { return false; }

wouldn't it be more concise and readable to compare the character directly, i.e.
if (state.src[pos++] !== '>') { return false; }

I'm sure there's a good reason, just couldn't understand what it is by myself. Would be great if you could shed some light on this.
Thanks in advance!

That's for speed optimizations. I'd suggest to read https://mrale.ph/ first. Because how js jit works is completely different from what you can imagine :).


the lastest code i download could't run directly

it could't find a variables

Maybe you modified imap and forgot to include https://github.com/daohu527/imap/blob/main/imap/global_var.py

No, i just download this code and run


Data problem

Hello, what do files ".x .y .tx .ty .allx .ally .graph .index" store？

All commented in data.py

All commented in data.py

I didn't find data.py in this repo


Overall doc organization

We currently have 3 major sections - User Guide, Developer Tutorials, and Samples.
A few issues and ideas to start a conversation:

Some content currently in user guide is meant for Zowe developers (those who want to develop plugins/apps for Zlux, for example). This content is in the Extending zLux section. It ilooks like SDK doc - doc that describes how to use the features of a framework to develop plugins. I think we should keep the User Guide focused on usage, not development. However, INSTALLING plug-ins applies to users, and that's what should be documented in the Extending sections.
Related to the above bullet - we have Imperative CLI Framework doc for developers in the /imperative GitHub Wiki. It is not a tutorial, but could belong in a Developer Guide along with the Zlux developer material.
We have content in dev tutorials for onboarding an api to mediation layer. This is content meant for developers, but it isn't an optional tutorial - it is really the core documentation for how to accomplish something. Does this make sense to have here? I've also heard suggestions that perhaps there needs to be a separate Admin Guide to describe this type of content. Need to discuss with Andrew J.

In conclusion, perhaps we need a SDK/developer focused area to add the zLux framework and Imperative (the cli framework) doc. We should consider use cases/personas to understand what major sections we should have on the site (Admin Guide?). We should keep the user guide focused on install/config/usage. If we have an admin guide, some of that install/config info would fit there instead.
Open to ideas/discussion.

User, Admin, Developer (documentation section and a tutorials section ) sound good to me. We have some topics (Server Configuration, User Settings and Defaults, Logging) that might be included in an admin guide.

I agree with that layout in general. A lot of our install doc would really belong in admin guide in a way - the z/OSMF configuration and installing zowe runtime are not done by the end user. We should clearly define what belongs in an Admin guide vs User guide vs Dev guide, then filter our topics into those categories accordingly. From there, we can build new TOCs. Let's see if others have input on this, as I am a bit unsure of myself as to what the best organization would be. We should run any proposals past dev too. We can discuss further in our next team meeting.

I concur with these points. We can invite our SMEs (Joe, Sean, Mark perhaps) to help go through the docs site materials and provide some input for us.
Also I found something quite interesting that Kubernets is doing for their documentation: https://kubernetes.io/docs/home/?path=users&persona=app-developer&level=foundational. They provide a way to classify documentation by personas and proficiency level. Something that we can consider for our docs too.

A good point from Joe:

One thing that came up is that in the base Zowe we let users download and install, we include the base function JES,  USS and MVS Explorer, but we don’t doc any of them in terms of their function.  This would be like a “user’s guide”,  rather than an install doc that we have now
I’m thinking this is a gap we should look at filling to help our end users.

When we have the user's guide split, we should consider expanding the current information there about how to use the base functions.

Thanks for the link to the Kubernets site Ashley! I like how they classify the documentation.

Conversation in Slack about this subject: https://project-zowe.slack.com/archives/C983BA2MD/p1534836217000100

I think that this post from Robert is insightful -
"Personas to consider

Sysprogs using Zowe for their regular work
General developers using Zowe for their regular work
Specific subsystem developers using Zowe for their regular work
Subsystem administrators using Zowe for their regular work
CI/CD people using Zowe for their regular work
All of the above, but who want to customize/extend Zowe internally
All of the above but who want to customize/extends Zowe publicly (contribute specific plugins to cli or zlux)
People who install/administer their company's Zowe infrastructure
People who want to participate in the Zowe community to influence/keep-abreast-of Zowe direction
People who want to contribute to the core of Zowe
ISV's who want to develop/sell Zowe-compatible products

As per Joe's point: Right now "developer tutorials" is for 4 and 5, and people in categories 2 and 3 don't necessarily recognise that they are "users" not "developers"."

From the Zowe CLI perspective, there are 3 main personas:
The systems admins/security admin who performs back-end maintenance, installation, and setup for z/OSMF and other component APIs that run on the mainframe to support the CLI (such as CICS APIs to support the Zowe CLI Plug-in for CICS).

Relevant topics: System requirements, z/OS requirements, Configuring z/OSMF

The end user, who is usually a developer or a CI/CD pipeline orchestrator, who uses the CLI in daily work to develop mainframe-based applications or write automation.

Relevant topics: Installing Zowe CLI, Configuring Zowe CLI, Extending Zowe CLI (how to install a plugin)

The developer who wants to develop for Zowe CLI in the form of a CLI plug-in or contribute to the existing code, add commands, etc....

Relevant topics: Coming Soon - Develop for Zowe CLI (this page will eventually have tutorials for how to develop a CLI plugin or add new commands to the core CLI)

General Note: Some topics could apply to multiple personas, for example Zowe Overview or Installation roadmap,

@BrandonJenkins14 I agree with the three general personas you've outlined, and I think it's a sums up what we've been discussing.
The one thing I have a question about would be if a lot of the guides under "end user" are actually for "developer". That's something we can figure out later though.

@adam - do you mean why did I list "Extending Zowe CLI" under the end-user persona in my notes? That's because the end-user is the person who installs plug-ins to the CLI on his/her own PC directly. While zLux plug-ins are installed on the back-end by an admin-type person.     On that note, I think that the content under Extending zLUX currently is developer-focused (how to build plugins) - https://zowe.github.io/docs-site/user-guide/mvd-creatingzluxappplugins.html . But how to install plug-ins for zLUX might belong under an admin guide.
Just thinking out loud.

@BrandonJenkins14 Sorry Brandon didn't notice they were all CLI based. I just saw most of the sections there and thought you were organizing the tutorials not the userguide. I still think of the userguide as a unit, but that would need to broken up as well I guess.
As far as the tutorials, I think all of them are currently developer role only. We'll need to add some others to satisfy the other roles.

A proposal as a starting point is under discussion here https://ibm.box.com/s/p2nunfe8qdo5i3gmg7bgjbtvlb0mzgcq and more conversations can be found on Slack: https://openmainframeproject.slack.com/archives/CC60ALD61/p1536166660000100

This issue is hard to track from my perspective. Does anyone have a status update on if new doc site designs are proposed/being worked on? I know onboarding squad meets regularly to sketch out persona paths, and there is an extender-1.0 branch in this repo with changes in progress.
I still see an issue where Zowe Application Framework is being discussed in detail in the user guide from a developer perspective. The "Using Zowe Desktop" section looks great, but I think that "Extending Zowe App Framework" will belong in an Extenders/Devs section.

@BrandonJenkins14 I have a proposal to reorganize the site here: https://github.com/zowe/docs-site/tree/docs_reorg. However, with the ongoing discussions from the onboarding squad to define Zowe 1.0, we'll need to work closely to understand what works better for users and redesign the doc site flow and structure. I'll meet with Tim/Joe this week to see how we can move forward. Stay tuned. :-)

A quick status update for this issue:
We're collaborating with the on-boarding squad to reorganize the topics and sections on the doc site. After the new structure is set, we'll proceed to change the organization and add missing topics if any.

In 0.9.4, we rolled out the new organization based on discussion. Check out the doc site. We have much further work to do to improve the current organization like adding new topics and improve navigation of several sections which we can open new issues to track. Therefore, closing this issues now. Feel free to reopen if any problem persists. Thanks!


MacOs - C# and Unity 5.5 Intellisense doesn't work

VSCode Version: 1.7.2
OS Version: MacOs 10.12.1
I use zsh. Not bash.

I can't use intellisense with C# and unity 5.5. I've tried hard but I can't solve it. Please Help!

Starting OmniSharp server at 12/11/2016, 6:41:17 PM
Target: /Users/rsl/Desktop/Jugs/Jugs.sln
OmniSharp server started wth Mono
Path: /Users/rsl/.vscode/extensions/ms-vscode.csharp-1.5.3/bin/run
PID: 881
OmniSharp: -s /Users/rsl/Desktop/Jugs/Jugs.sln --hostPID 865 --stdio DotNet:enablePackageRestore=false --encoding utf-8
{"Event":"log","Body":{"LogLevel":"INFORMATION","Name":"OmniSharp.Startup","Message":"Omnisharp server running using Stdio at location '/Users/rsl/Desktop/Jugs' on host 865."},"Seq":1,"Type":"event"}
[INFORMATION:OmniSharp.DotNet.DotNetProjectSystem] Initializing in /Users/rsl/Desktop/Jugs
[INFORMATION:OmniSharp.DotNet.DotNetProjectSystem] Auto package restore: False
[INFORMATION:OmniSharp.DotNet.DotNetProjectSystem] Update workspace context
[INFORMATION:OmniSharp.DotNet.DotNetProjectSystem] Resolving projects references
[INFORMATION:OmniSharp#MSBuild] Detecting projects in '/Users/rsl/Desktop/Jugs/Jugs.sln'.
[INFORMATION:OmniSharp#MSBuild] Loading project from '/Users/rsl/Desktop/Jugs/Assembly-CSharp.csproj'.
[WARNING:OmniSharp#ProjectFileInfo] Couldn't locate Mono, TargetFrameworkRootPath not specified
[INFORMATION:OmniSharp#ProjectFileInfo] Using tools version: 15.0
[ERROR:OmniSharp#ProjectFileInfo] The reference assemblies for framework ".NETFramework,Version=v3.5" were not found. To resolve this, install the SDK or Targeting Pack for this framework version or retarget your application to a version of the framework for which you have the SDK or Targeting Pack installed. Note that assemblies will be resolved from the Global Assembly Cache (GAC) and will be used in place of reference assemblies. Therefore your assembly may not be correctly targeted for the framework you intend.
[WARNING:OmniSharp#MSBuild] Failed to process project file '/Users/rsl/Desktop/Jugs/Assembly-CSharp.csproj'.
/Users/rsl/Desktop/Jugs/Assembly-CSharp.csproj
[INFORMATION:OmniSharp.Startup] Configuration finished.

In order to use the c# unity intellisense you have to open the file folder where your scripts are. This should work.

Moved to OmniSharp/omnisharp-vscode#1031

@Mkayxx I know this issue is closed but I'm experiencing an identical one. Do you mean I should open it in VSCode? Tried opening the "Assets" folder and navigating to my scripts folder from there - didn't help.
If you worked through this issue, would you mind explaining how you did it?
Thanks!

What I mean by opening the file folder.if your unity game project is called 2d platformer meow,try finding the 2d platformer folder and open it and intelligence should work,also you have to download some of the extensions listed at the link.

Thanks! I followed a suggestion elsewhere and installed the newest version
of Mono, which solved the problem for me.
On Wed, Dec 14, 2016 at 5:06 AM Mkayxx<EMAIL_ADDRESS>wrote:

Try everything this site tells you
https://code.visualstudio.com/Docs/runtimes/unity.
What I mean by opening the file folder.if your unity game project is
called 2d platformer meow,try finding the 2d platformer folder and open it
and intelligence should work,also you have to download some of the
extensions listed at the link.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/17049#issuecomment-267028980,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKfQLn5pInbHZqygGTEKG6RsexcAIqlOks5rH-nfgaJpZM4LJ_ov
.


Pages Won't Scroll

The pdf renders alright, however it renders only the first page and doesn't scroll. Is there anything I am missing?

Is the iOS or android ?

I also got same problem in IOS. It is working good in android... Can anyone help me quickly please.

@sibeliusseraphini Can you help to fix?

@kamalkrishna3000 take a look in this branch https://github.com/cnjon/react-native-pdf-view/tree/feature/swift
it is almost done.
You can also use another component to scroll or swipe between pdf pages, for now it only show one page

I am facing this too.

I'm having issues with iOS scrolling as well

on android works great but on ios zoom out makes pdf view looks so little on the same pdfview and scrolling not working

same problem here

Any update on this? It's been many months since it was almost done.

same problem here, any update on this please? 👍

I just gave up and rendered it using WebView on iOS.

@pedramsaleh i do the same, but i can not zoom :c

Maybe try react-native-quick-look?

Do not use this component to display PDF on  IOS .
on IOS  can  use  webview  display PDF.
this.pdfPath = RNFS.DocumentDirectoryPath + /tour.pdf;
<WebView
source={{uri: this.pdfPath}}
style={{flex: 1, backgroundColor: 'white'}}
scalesPageToFit={true}
/>
'this.pdfPath'  should  be the path in  Xcode  project.

Can anyone solved this ?
I am try using WebView but cannot open pdf  Local Directory on iOS.
my this.pdfPath = /Users/rezaarifiansyah/Library/Developer/CoreSimulator/Devices/3F9E5A72-FE37-42D0-AB99-84F86974373D/data/Containers/Data/Application/E350A5C9-212E-4EEA-9F61-6E1492810FB5/Documents/5d7d57f1abfac97dd29ff7c7ae159846
@NextChampion  can you help me to use WebView .
Thanks

1 copy the Local path;
2 Open Finder ;
3 command + shift + g  ;
4 paste ;
5 enter ;
@rezaarifian

+1
Component would be even better with scroll...

Hi!
In my project, I use https://github.com/philipshurpik/react-native-animated-tabs to make beautifully animated swiping between pages.
Here is the example of render method you can try:
render() {
	const {uri} = this.props;
	const {pageCount} = this.state;

	return <View style={commonStyles.flexContainer}>
		{uri && <AnimatedTabs
			panelWidth={Dimensions.get('window').width}
			style={styles.tabs}
			onMoveShouldSetPanResponder={(a, e) => {
				return Math.abs(e.dx) > 30 && Math.abs(e.dx) > Math.abs(e.dy) * 1.5;
			}}
		>
			{(new Array(pageCount)).fill().map((i, pageNumber) => <PDFView
				key={pageNumber}
				path={uri}
				pageNumber={pageNumber + 1}
				zoom={0.90}
				borderWidth={0}
				backgroundColor={colors.bgGray}
				style={styles.pdf}
				onLoadComplete={pageCount => this.setState({pageCount})}
			/>)}
		</AnimatedTabs>}
	</View>
}


First implementation of tripleo::database::mysql

Implement MySQL support with or without Galera.

It looks like this commit includes a lot a duplicate code. Is there a reason why we are just adding a dependency on the repository that originally included these files (e.g. [1]) ?
[1] https://github.com/stackforge/puppet-openstack-cloud/blob/master/manifests/database/sql/mysql.pp

@ghantoos thanks for your comment. We are implementing all the composition layer in TripleO itself, and puppet-openstack-cloud will die.

@dprince done


Added throughput logging for inference.

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Example output:
[2019-08-01 16:32:44 +0200] [95057] [INFO] Inference took 1.10s for 3 items, 0.37s on average.
[2019-08-01 16:32:44 +0200] [95057] [INFO] 	1 took -> 0.52s (len(target)==18).
[2019-08-01 16:32:44 +0200] [95057] [INFO] 	2 took -> 0.28s (len(target)==18).
[2019-08-01 16:32:44 +0200] [95057] [INFO] 	3 took -> 0.29s (len(target)==18).

Job PR-228/1 is complete.
Docs are uploaded to http://gluon-ts-staging.s3-accelerate.dualstack.amazonaws.com/PR-228/1/index.html

Codecov Report

Merging #228 into master will decrease coverage by 0.79%.
The diff coverage is 20%.

@@            Coverage Diff            @@
##           master     #228     +/-   ##
=========================================
- Coverage   82.04%   81.24%   -0.8%
=========================================
  Files         130      130
  Lines        7374     7397     +23
=========================================
- Hits         6050     6010     -40
- Misses       1324     1387     +63

Impacted Files
Coverage Δ

src/gluonts/shell/serve.py
73.41% <20%> (-8.81%)
:arrow_down:

src/gluonts/dataset/repository/_artificial.py
35.29% <0%> (-64.71%)
:arrow_down:

...c/gluonts/dataset/artificial/generate_synthetic.py
12.94% <0%> (-10.59%)
:arrow_down:

src/gluonts/dataset/artificial/_base.py
36.67% <0%> (-6.6%)
:arrow_down:

src/gluonts/dataset/repository/datasets.py
75% <0%> (-5.56%)
:arrow_down:


populate complete Envoy bootstrap locality

Set locality field region sub_zone other than zone

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: hzxuzhonghu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rshriram
If they are not already assigned, you can assign the PR to them by writing /assign @rshriram in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hzxuzhonghu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
ca948324959f41de64a11ecfb01a95efe6daa965
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu thanks!  With this PR, user still can't leverage locality unless we populate the values correct, agree?

yea

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hzxuzhonghu, rshriram
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [rshriram]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add test cases for Bayesian Networks using reference implementation.

The existing test suite is essentially non-existent. Given the ongoing changes to the model interface, it's essential there's a set of tests that give decent assurance that the behaviour of the models remain correct.

This should include a comparison to Scikit-Learn's Naive Bayes implementation.


[11.x] Fix incorrect typechange in Illuminate\Database\Query\Builder

In #53793 various properties in the Illuminate\Database\Query\Builder class were marked as nullable as they have no initial value, thus defaulting to null. However the wheres properties is initialized with an empty array, thus already is an array on class construction and should never be null.

@jnoordsij You're absolutely correct. The wheres property is initialized as an empty array and should not have been marked as nullable. I initially added the nullable type during development but forgot to remove it before opening the PR. Thank you for catching that!