Packet decode error, Beta60 - FeatherESP32S3psram2mb with SEN5x and 1 neopixel (onboard)

Similar to before, I'm using the onboard neopixel on the feather S3 2MB PSRAM, running from main branch as of an hour ago. I add the neopixel first, then on subsequent reboot I add the SEN5x with 7 of the 8 metrics (not fahrenheit), and get i2c decode error with a total of 8metrics. I initially saw it sooner with the onboard lipo charger metrics active, but I culled them.
I naively assume there was a regression related to packet size @brentru

We could naively increase the overall buffer size, it’s currently pretty small (even compared to the size of the filesystem overhead)

Though it might be worth it to find the size of each sensorEvent message, and calculate an estimated maximum for the amount of packed sensorEvent messages  (within message, sensorEvents).

This should be resolved by https://github.com/adafruit/Adafruit_MQTT_Library/pull/227 and will be included in WipperSnapper Beta 71 (released later today).
Please try beta 71 and let me know in this issue if the packet decode error has been resolved or is still persistent.


Not rendering

I understand this project is pretty old, but it's awesome and the best one (out of 2) out there. I simply cannnot get the portals to render in my world, following everything there is to be seen in the example and fixing up some outdated parts from the code (mainly MeshFaceMaterial being removed.)
No matter WHAT I try (making the portals super large, changing position, changing rotation, etc.) nothing renders. I'm not expecting a reply (or a fast reply) but maybe somebody would come across this and be able to help.

Can you share the code?
What do you see in the portals? Are they back? White?
Can you revert back to an earlier version of three and check if it works there?

Absolutely nothong at all renders, as if I never created them. I followed the same exact code in the example. Honestly, I even downloaded the example and nothing rendered when using the latest threejs, so I downloaded the threejs used on the example website and that allowed my local copy of the example to render and function properly. The same could not be said for my actual game code, switching to the old threejs yields no portal rendering whatsoever. I can put logs everywhere and they all get called: the portals are created, the scene is registered, the scene is set, and the portal render functions even get called. I can log the coordinates and those all check out, everything looks perfectly fine; just nothing shows up! I'm not home right now but i can share code samples when I'm back, just know its essentially the same principal of whats in example.js and nothing new or fancy added yet (that comes after it actually works)

// Init
                this.renderer = new THREE.WebGLRenderer({
			antialias: true
			//stencil: true
		});

		this.renderer.setPixelRatio(window.devicePixelRatio);
		this.renderer.setSize(window.innerWidth, window.innerHeight);

		document.body.appendChild(this.renderer.domElement);

		this.camera = new THREE.PerspectiveCamera(75, window.innerWidth / window.innerHeight, 1, 1000);
		this.createScene();

// createScene
                this.scene = new THREE.Scene();
		this.scene.background = new THREE.Color(0xffffff);
		this.scene.fog = new THREE.Fog(0xffffff, 0, 750);

		const light = new THREE.HemisphereLight(0xeeeeff, 0x777788, 0.75);
		light.position.set(0.5, 1, 0.75);
		this.scene.add(light);

// create walls (called in init)

                const loader = new THREE.TextureLoader();

		loader.load('./assets/image/wallTexture.jpg', map => {
			map.wrapS = map.wrapT = THREE.RepeatWrapping;
			map.offset.set(0, 0);

			map.repeat.set(2, 2);

			const wallSize = 50;
			const wallPos = 50;

			const geometry = new THREE.BoxGeometry(wallSize, wallSize, 5);
			const material = new THREE.MeshBasicMaterial({ map });

			const frontWall = new THREE.Mesh(geometry, material);
			frontWall.position.set(0, 20, -wallPos);
			this.add(frontWall);

			const backWall = new THREE.Mesh(geometry, material);
			backWall.position.set(0, 20, wallPos);
			backWall.rotation.set(0, Math.PI, 0);
			this.add(backWall);

			this.addPortals(frontWall, backWall);
		});

		loader.load('./assets/image/wallTexture.jpg', map => {
			map.wrapS = map.wrapT = THREE.RepeatWrapping;
			map.offset.set(0, 0);

			map.repeat.set(40, 40);

			const floorGeometry = new THREE.PlaneGeometry(1000, 1000, 100, 100);
			const floorMaterial = new THREE.MeshBasicMaterial({ map });
			floorGeometry.rotateX(-Math.PI / 2);

			const floor = new THREE.Mesh(floorGeometry, floorMaterial);

			this.scene.add(floor);
		});

// And finally portals

                this.portalController = new THREE.PortalController({
			renderer: this.renderer,
			camera: this.camera
		});

		// this.controls = this.portalController.cameraControls;

		this.portalController.registerScene('scene1', this.scene);

		const portal1 = this.portalController.createPortal(20, 20, 'scene1');
		portal1.position.copy(frontWall.position);

		const portal2 = this.portalController.createPortal(20, 20, 'scene1');
		portal2.position.copy(backWall.position);
		portal2.rotateY(Math.PI);

		portal1.setDestinationPortal(portal2);
		portal2.setDestinationPortal(portal1);

		this.portalController.setCurrentScene('scene1');

using the same portal code (with position.copy) & wall code, I can get this to work in the example but it won't work in my actual game client.

I decided to just code my own portals basing it off yours and another portal library that I found. Now I've got recursive rendering portals (seeing portals through portals) that render perfectly (except teleportation... guess they aren't portals yet. I'll get to that soon.) Thank's for the help anyways though.


Add the ability to include/exclude namespaces via label selectors

The title is self explanatory. Today only an array of static namespaces is available as a configuration options.

Right now we have a static inclusion list. Are you just looking to dynamically define that list by labels? Or are you also looking to define it in the opposite direction and say "only these namespaces should be considered" (unless overridden by the ClusterSPIFFEID spec)?


Fuse the --strong option into --preview-dart-2 so that we get strong mode bu default

Integrate strong mode option into the preview-dart-2 mode so that we run flutter code by default in strong mode.

@a-siva fyi: an unforeseen consequence of PR #14392 is that it also silenced all "invalid override errors" reported by Fasta (because forwarding stubs that were triggering it) are no longer inserted. I think this means we can move forward with this PR because it does not change the set of reported errors.
[Note: the absence of override errors is only intermittent because according to @stereotype441 our override checking will start reporting them again at some point, unless underlying mixin issue is resolved conclusively].

@a-siva scratch my previous comment actually even though the errors are silence the --preview-dart-2 --strong is completely broken by #14392 in runtime - because checks fail in runtime :-/
Can we revert #14392?

Landed this from another PR


Ordered network

Connections are currently brittle, specifically with regard to the exact timing of messages.
We use an orderedDelivery function to ensure that inbound messages are delivered to the Connection in the right order; even if messages come in order 0-2-1 they're delivered in order 0-1-2. But for example if we get message #1 and we never received message #0 -- which can easily happen, for example, if a message was received on the network before the Connection had started up -- we'd just sit there forever waiting for #0 which would never come.
This PR lays the groundwork for making connections more robust in general. We replace the orderedDelivery function with a stateful OrderedNetwork class, which maintains a register of all the inbound and outbound messages it has seen. It can request that a missing message be resent, and on the other send can respond to a resend request.
This PR is essentially a refactoring of the Connection class's internal workings, with no outward behavior changes. Specifically, the connection doesn't yet use the resend functionality.

Current dependencies on/for this PR:

main

PR #85   👈

PR #86

PR #87

This stack of pull requests is managed by Graphite.


Positioning with new CSS changes

@lukejacksonn ^ ^

Was afk yesterday.. what is the issue here? I like your setup!

haha thanks! It just seems like there is a bit more padding than necessary on top. Could just be me though 😂


Examples

Added very basic streaming examples to readme and a fuller one to an examples/ folder.
I also added a small opus file to go with the example.
This makes me feel less bad about https://github.com/Rantanen/node-opus/issues/6 :P

Thanks. :)


Architecture and code organisation

inselect is currently a single python package but it really should be (at least) two:

logic for segmenting images;
a gui application that depends upon 1.

The current architecture means that when we come to segment images on servers (to be expected, when one considers mass-digitisation) we are forced to install a large GUI library - QT - an unnecessary dependency. Some work has done towards this by structuring most of the source files into the gui and lib directories.

As a result of closing #102 , inselect/lib no longer depends on QT


Getting Query timed out on AWS EC2 instance

We are using AWS EC2 instance and have implemented node.js server written in hapi js which is deployed using docker.
On our development server public-ip library is calling dns-socket and is giving proper result.
Now we have setup staging server with similar configuration. Here we are getting below error.
error:  Error: Query timed out
    at DNS._ontimeout (/usr/src/app/node_modules/public-ip/node_modules/dns-socket/index.js:98:18)
    at ontimeout (/usr/src/app/node_modules/public-ip/node_modules/dns-socket/index.js:52:10)
    at wrapper (timers.js:275:11)
    at Timer.listOnTimeout (timers.js:92:15)

Can you please suggest some way out of this issue? Let me know if any further information required.

Looks like a network issue, see https://github.com/sindresorhus/public-ip/issues/15#issuecomment-299233250. Closing as there's nothing we can do about that.


[scalardb-cluster] Add ScalarDB Cluster chart

This PR adds a new chart of ScalarDB Cluster.
I copied this chart from ScalarDB Server's chart first, and there are a lot of fixes in the following PRs.

#185
#191
#192
#193
#194
#195
#196
#197
#198
#207
#210
#211
#213
#214
#225

After this PR will be merged, I will release v1.0.0.
Please take a look!

@brfrn169
Thank you for your review!

Are you going to include the SQL metrics in v1.0.0?

Yes. I will add the SQL metrics in another PR and after that, I will release v1.0.0.


message view: Fix hiding of connection-error message on narrowing streams.

When there is some error in connecting to server(more specifically to the
tornado server) the "Unable to connect to Zulip" connection error message
gets cleared as Django server could send the response of "get" request of
old messages and hence get_old_messages_success hides the error message
even though the connection is not properly established.
Fixes: #5599.
Is my understanding of issue and event system is correct?
@timabbott FYI.

I think it's a bit more complicated; we have 2 connection errors (one with a "Try again" messages, for get_events, and the other without, for fetching historical messages ).  I think we just want to hide the second in this code path.

Making it more clear:

We only want to stop clearing of above message whenever we try to fetch an older message.

we just want to hide the second in this code path.

$('#connection-error').removeClass("show") in message_fetch->get_old_messages_success()->process_result()  only clears that banner whenever we try to fetch old message. It don't do anything with connection-error in server_events->get_events.
Hence it only deals with second change, what am I missing here?

The problem here is that we have two independent code paths for fetching data from the server that are using the same UI widget to display their state:
(zulip-py3-venv) tabbott@zaset:~/zulip$ git grep connection-error
static/js/message_fetch.js:    $('#connection-error').removeClass("show");
static/js/message_fetch.js:            $('#connection-error').addClass("show");
static/js/server_events.js:                ui_report.hide_error($("#connection-error"));
static/js/server_events.js:                    ui_report.hide_error($("#connection-error"));
static/js/server_events.js:                    ui_report.show_error($("#connection-error"));
static/js/server_events.js:                    ui_report.hide_error($("#connection-error"));
templates/zerver/index.html:        <div class="alert alert_sidebar alert-error home-error-bar" id="connection-error">

The text and everything in the actual template is for the server_events.js code path.  We should make a copy of the UI widget for the message_fetch code path, without the "try now" button (since I don't think we can easily implement what such a button to do -- text should be something like "Error fetching messages from server; try again later").
Does that make sense?

@timabbott, Yeah, This clears everything, this will solve the problem mentioned in issue, but having different warnings for these code path will show two different warnings when both event and message fetch systems aren't working i.e. we aren't connected to the internet. Is this okay?

Yeah, that's ugly.  I think basically the get_events code path has priority -- that one is what's really important.  The other one is less important (happens when you get an error fetching messages from the server, which is in practice quite rare, and usually results in more minor issues).  I think there's a couple options:
(1) Move the "get_messages" error to be displayed visually attached to the little "Z" icon where we show the loading indicator for fetching new messages.  This might be a cleaner display.
(2) Do a display algorithm that basically prioritizes the "get_events" one:

If the get_events message is shown, don't show the other one.  When that is dismissed, clear the state for both (makes sense because usually this means you're done with your network interruption).
If the get_messages one shows, but not the get_events one, clear that when the next get_messages call succeeds.

Thoughts on which is better?

Both sounds reasonable, For (1) I think, it shows what is happening here(fetching the messages) but if there is an error in network connectivity then this seems not be informative to user that what is happening here.
I've also thought about (2) I think this makes more sense as compared to (1), I can check for whatever is more feasible.
Another possiblity, I think we should implement (1) with "Error fetching messages from server; try again later" message for get_message error with the logic of (2). Is this is okay?

@timabbott Adding simple class for get_events error works, and to make the tests pass I've to add removeClass to stub_out_jquery which I've added in a separate commit. Maybe both can be squash.(I've ordered removeClass commit before the fix here because each commit should pass tests individually).

@timabbott updated and opened PR for eslint rule.

Finally got around to merging this, huge thanks for doing it @shubhamdhama!
Also added 83b127cec12907aa04b33c3be706e034e20de3ad to make the code a bit more readable.


Home Assistant captcha token link is missing

Describe the issue
Hi,
i´m trying to install BMW Connected Drive → region: rest_of_world.
I don't get a link for the captcha token.

Expected behavior
Get a link to receive a captcha token.
Which Home Assistant version are you using?
core-2024.12.5
What was the last working version of Home Assistant Core?
No response
What is your region?
Rest of world
MyBMW website

[x] I can still successfully login to the BMW MyBMW website and the car status is available there.
[X] I have MyBMW enabled for my vehicle.

Number of cars

[ ] I have 2 or more cars linked to the MyBMW account.
[ ] I have a Mini vehicle linked to my account.
[ ] I have a Toyota Supra vehicle linked to my account.

Output of bimmer_connected fingerprint
No response
Anything in the logs that might be useful for us?
No response
Additional information
No response

You are using the custom component (as shown by the BETA part in your screenshot). That version is not up to date so please make sure you use the version from Home Assistant --> delete bmw_connected_drive folder in the config/custom_components folder and restart HA.
Then you can follow the documentation.

Thank you, it worked.


[SPARK-28568][SHUFFLE][DOCS] Make Javadoc in org.apache.spark.shuffle.api visible

What changes were proposed in this pull request?
This PR proposes to make Javadoc in org.apache.spark.shuffle.api visible.
How was this patch tested?
Manually built the doc and checked:

@mccheah, @vanzin, @squito, @jerryshao,  I happened to take a look at the JIRA. The doc will be visible with this change and able to verify the doc output.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13603/
Test PASSed.

Test build #108509 has started for PR 25323 at commit 4233dfa.

Test build #108509 has finished for PR 25323 at commit 4233dfa.

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

Merged build finished. Test PASSed.

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


Problem adding site

I’m trying to add this site and even if it’s correct it’s impossible to add

There may or may not be a bug here. The feed may not be findable. But, just in case, let’s check.

This website does not provide feeds.


release 1.3.4

Some of the unit tests failed, would that be okay?

@ywang-nr let me fix those issues.


Add bindings for function and method parameters, returns. [phab:D1227]

Created by fromberger at 2017-01-06 17:45:10:
Also updates the unit tests for function parameters.

fromberger wrote at 2017-01-06 23:01:23:

Fix handling of bindings for blank identifiers.

zarko accepted this revision at 2017-01-07 00:00:58

fromberger closed via commit 10c47765d0a4647f3e30a3661a422ac441da90c8 at 2017-01-07 00:19:44


Front end does not bounds check constructor generic arguments

Consider the following code:
class A<T extends int> {}
dynamic foo<T>() {
  return new A<T>(); // (1)
}
dynamic bar() {
  return new A<String>(); // (2)
}

A compile-time error should be reported at both (1) and (2) (in strong mode), because neither T nor String is a subtype of int.  The front end does not currently report the necessary errors.

@sjindel-google FYI

Closing as duplicate of https://github.com/dart-lang/sdk/issues/33308


RE2022-240: Add simple dynamic configuration system

Needed to configure the arangosearch view name for filtering operations
Also adds a __len__ method to FilterSet

test failed on codecov. But I think that should be ok.


remove some allocations in Serialize

I have know reason to think this is in the hot path, but don't allocate if we don't have to.

r? @matklad
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit 8a4be7892e30284587867c833d9b0d69f8cd92d4 has been approved by alexcrichton

:hourglass: Testing commit 8a4be7892e30284587867c833d9b0d69f8cd92d4 with merge 0785b54581d668740366f88f6497e26803c6a9db...

:broken_heart: Test failed - status-appveyor

@bors retry

:hourglass: Testing commit 8a4be7892e30284587867c833d9b0d69f8cd92d4 with merge 07a5546b1ad0cb778552bf22e3858e5c79d679f8...

:broken_heart: Test failed - status-appveyor

Looks like appveyor should have called in sick today.

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 7386d438c8a184ebd24c19755ba3a86b92915755 to master...


a11y: fix table-forms keyboard navigation

Description
The PR aims to address the following issues found on pages with lists containing nested forms, including: "Bot Responses", "Knowledge (QnA)" and "Knowledge Base" pages.
Here is the list of issues addressed:

Unable to navigate to fields nested in DetailsList cells using keyboard
Some buttons are hidden while navigating via keyboard ("Clear field" button, "Add new" item button)
Save via keyboard Enter key doesn't work under several circumstances
Shift+Enter should persist caret position and the field in focus while adding a new line
Visual focus indication for focused fields works unstable
No error announcement
No way to blur selected fields when the focus is on the DetailsList cell field

Highlights on changes made:

Implement a CellFocusZone component to handle focus in DetailsLists' cells having form elements
Move all related logic to the component, so it can: focus form elements, handle Escape and Enter key and handle focus loss cases
Update EditField to play well with the new component
Add a callback to be called on EditField when a new line gets added
Handle Shift+Enter properly: from now it adds a new row to input and resets focus back to the input after it got switched to the textarea element
Update CSS rules to handle focus for tables-view and EditableField components
Refactor styles: move shared styles to the CellFocusZone
Refactor table-views: introduce CellFocusZone and improve keyboard handling
Knowledge Base: change the way alternative questions added (see the last gif: from now Shift+Enter equals clicking "Add new" button)
Fix EditField styles to handle field height correctly
Add announcements for EdtiField errors

Task Item

VSTS 70470
VSTS 70527

Screenshots

#minor

Looks like the server package's typecheck is failing:
Error: [typecheck:client] src/pages/knowledge-base/table-view.tsx(546,47): error TS2345: Argument of type 'number | undefined' is not assignable to parameter of type 'SetStateAction<number>'.
[typecheck:client]   Type 'undefined' is not assignable to type 'SetStateAction<number>'.

@tonyanziano strange. It passes locally:

But npm run typechec:client does. Let me fix this

Coverage decreased (-0.09%) to 55.867% when pulling dcbe7453f2ecd37109b4168048c99df4dd8d22de on OEvgeny:a11y/table-inputs into 8bbf851cb8a2194bdfa877d242d7c822a93a8674 on microsoft:main.


What's the meaning of frequency about optimizer

The ‘frequency’ value is an int corresponding to the number of sequential batches optimized with the specific optimizer. It should be given to none or to all of the optimizers. There is a difference between passing multiple optimizers in a list, and passing multiple optimizers in dictionaries with a frequency of 1: In the former case, all optimizers will operate on the given batch in each optimization step. In the latter, only one optimizer will operate on the given batch at every step.

I don't know "only one optimizer will operate on the given batch at every step", if all optimizers's frequency is 1, which optimizer will be used?

if you don't use frequency then all the optimizers with take a step in each training_step.
but if you define frequencies then it will follow the frequency intervals.
for eg:
if opt_1_freq = 1 and opt_2_freq = 2
the it will be like:
batch1 -> opt_1.step()
batch2 -> opt_2.step()
batch3 -> opt_2.step()
batch4 -> opt_1.step()
batch5 -> opt_2.step()
batch6 -> opt_2.step()
...

if all optimizers's frequency is 1, which optimizer will be used?

in this case if will be like
batch1 -> opt_1.step()
batch2 -> opt_2.step()
batch3 -> opt_1.step()
batch4 -> opt_2.step()
...

it is sometimes used while training GANs.
see training procedure in Improved Training of Wasserstein GANs, Algorithm 1
paper: https://arxiv.org/abs/1704.00028

if you don't use frequency then all the optimizers with take a step in each training_step.
but if you define frequencies then it will follow the frequency intervals.
for eg:
if opt_1_freq = 1 and opt_2_freq = 2
the it will be like:
batch1 -> opt_1.step()
batch2 -> opt_2.step()
batch3 -> opt_2.step()
batch4 -> opt_1.step()
batch5 -> opt_2.step()
batch6 -> opt_2.step()
...

if all optimizers's frequency is 1, which optimizer will be used?

in this case if will be like
batch1 -> opt_1.step()
batch2 -> opt_2.step()
batch3 -> opt_1.step()
batch4 -> opt_2.step()
...

it is sometimes used while training GANs.
see training procedure in Improved Training of Wasserstein GANs, Algorithm 1
paper: https://arxiv.org/abs/1704.00028

Thanks, if i set frequency in lr schedule, it works like optimizer?

Somewhat similar but not exactly. In case of scheduler if you set a frequency then scheduler.step() will take place at frequency interval and all the schedulers with be independent of each other.
For eg
scheduler_1_freq = 2
scheduler_2_freq = 1
interval = 'step' for both

then it will be like
batch1 -> scheduler_2.step()
batch2 -> scheduler_1.step() and scheduler_2.step()
batch3 -> scheduler_2.step()
batch4 -> scheduler_1.step() and scheduler_2.step()
batch5 -> scheduler_2.step()
...

Somewhat similar but not exactly. In case of scheduler if you set a frequency then scheduler.step() will take place at frequency interval and all the schedulers with be independent of each other.
For eg
scheduler_1_freq = 2
scheduler_2_freq = 1
interval = 'step' for both

then it will be like
batch1 -> scheduler_2.step()
batch2 -> scheduler_1.step() and scheduler_2.step()
batch3 -> scheduler_2.step()
batch4 -> scheduler_1.step() and scheduler_2.step()
batch5 -> scheduler_2.step()
...

Thanks!

Somewhat similar but not exactly. In case of scheduler if you set a frequency then scheduler.step() will take place at frequency interval and all the schedulers with be independent of each other.
For eg
scheduler_1_freq = 2
scheduler_2_freq = 1
interval = 'step' for both

then it will be like
batch1 -> scheduler_2.step()
batch2 -> scheduler_1.step() and scheduler_2.step()
batch3 -> scheduler_2.step()
batch4 -> scheduler_1.step() and scheduler_2.step()
batch5 -> scheduler_2.step()
...

If i only use one lr scheduler, set frequency = 1 is same as frequency = any number?

nope... in the case of schedulers all are independent. so for eg if you set frequency=3 and interval='step'
then
batch1 -> no scheduler step
batch2 -> no scheduler step
batch3 -> scheduler.step()
batch4 -> no scheduler step
batch5 -> no scheduler step
batch6 -> scheduler.step()
...

nope... in the case of schedulers all are independent. so for eg if you set frequency=3 and interval='step'
then
batch1 -> no scheduler step
batch2 -> no scheduler step
batch3 -> scheduler.step()
batch4 -> no scheduler step
batch5 -> no scheduler step
batch6 -> scheduler.step()
...

Wow~ i see, i think these explanation should be added to documents...

already there for optimizers. https://pytorch-lightning.readthedocs.io/en/latest/lightning_module.html#configure-optimizers
but I think can be improved for schedulers. Want to submit a PR?

already there for optimizers. https://pytorch-lightning.readthedocs.io/en/latest/lightning_module.html#configure-optimizers
but I think can be improved for schedulers. Want to submit a PR?

How to improve？I think the document is not very clear.....

Just checked again, I think it's already there for both optimizers and schedulers. Check both the NOTE sections.

Just checked again, I think it's already there for both optimizers and schedulers. Check both the NOTE sections.

Maybe my English is not very good..

@rohitgr7 I do not find the documentation all that clear and I also noticed that while personally coding it by adding a loop like in the WGAN paper it works as expected, but using the frequency parameter, which according to the documentation should do the same (and according to you, something slightly different in my understanding), it seems it does not.
Is it possible to have the discriminator just do n sweeps over the same batch for every sweep the generator does?

@aegonwolf did you manage to find a solution? Currently, want to implement multiple discriminator updates per gen update and am unsure if I should be using the frequency parameter in configuring optimizers or writing a for loop over the discriminator training.


fix(opportunities): adjust the scroll into form

https://openforge.teamwork.com/index.cfm#/projects/235303/tasks/17524761

QA Passes


Why is goframe.org not accessible anymore. Please check.

refer the title

已恢复

Th  Cz cny
l

On 24 Mar 2022, at 10:47, houseme @.***> wrote:

已恢复
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.


[gRPC] Support contract first approach for service creation

Description:
Current gRPC implement supports only code first approach. We need to support contract first where generate service code from the proto file.
Steps to reproduce:
N/A. This is an improvement. we are not support this
Affected Versions:
N/A. This is an improvement. we are not support this
OS, DB, other environment details and versions:
N/A. This is an improvement. we are not support this
Related Issues (optional):
None
Suggested Labels (optional):
improvement
Suggested Assignees (optional):
daneshk

This is fixed with https://github.com/ballerina-platform/ballerina-lang/pull/10560


Release?

Hi! I maintain matio packages for Fedora and EPEL. I see that a few CVEs are fixed in master; is a release forthcoming or should I just ship a snapshot to address these? Thanks!

an official release 1.5.18 will be welcome with official support of hdf5 1.12
Currently on macos, libmatio 1.5.17 segfault (libmatio 1.5.17 + hdf5 1.12 packaged with brew)
But libmatio current master works with hdf5 1.12 :(
see:
https://travis-ci.org/github/Nelson-numerical-software/nelson/jobs/676873956
 14/29 - [matio] test_savemat_cell                                          Fail
    run('/Users/travis/build/Nelson-numerical-software/nelson/modules/matio/tests/test_savemat_cell.nls')
    Error:
      /Users/travis/build/Nelson-numerical-software/nelson/bin/macosx64/nelson-cli: line 33: 85346 Abort trap: 6           $SCRIPT_DIRECTORY/nelson-cli-exec "$@"
 15/29 - [matio] test_savemat_char                                          Fail
    run('/Users/travis/build/Nelson-numerical-software/nelson/modules/matio/tests/test_savemat_char.nls')
    Error:
      /Users/travis/build/Nelson-numerical-software/nelson/bin/macosx64/nelson-cli: line 33: 85352 Abort trap: 6           $SCRIPT_DIRECTORY/nelson-cli-exec "$@"
 16/29 - [matio] test_savemat_double                                        Fail
    run('/Users/travis/build/Nelson-numerical-software/nelson/modules/matio/tests/test_savemat_double.nls')
    Error:
      /Users/travis/build/Nelson-numerical-software/nelson/bin/macosx64/nelson-cli: line 33: 85358 Abort trap: 6           $SCRIPT_DIRECTORY/nelson-cli-exec "$@"

I am trying my best to file a new release in May.

Hi, any updates on the expected release date?

No update except that I did not manage to release in time. I need to decide if I should release time-triggered (as I did the last years) or finish the relevant open issues I have in mind before.

Version 1.5.18 was released today.

Is there a collection of patches that could be applied to 1.5.17 that would address the security fixes without incrementing the soname?

No, there is not. There have been too many fixes (addressing the issues detected by fuzzing) even if there was no CVE number assigned.
I increased the so version because I changed some C types in mat_sparse_t, removed strdup_vprintf and strdup_printf from the public API and added new error constants MATIO_E_ABC to matio.h with functions returning different error codes.

Ok, thank you.


Initialising, [Warning] Not connected to mumble

we can talk in game, but it shows [Warning] not connected to mumble. What's the problem of this?

Did you find a fix?


Consider disabling in-app browser, or providing an option to do so

I personally am not a fan of the embedded browser in Android, and I don't think I know many people who are. The first two Google links I see for "android in-app browser" that are not about how to implement this open with things like:

Want to disable in-app browser for Android apps? Well, the majority of applications these days offer an in-app browsing feature for opening URLs and webpages directly within the app without leaving it to your phone’s default browser. This is apparently done by developers to keep users locked within their apps for as much as time possible.
Now while this offers convenient and faster access to web content, some of you may not like the lackluster approach. As a result, it becomes a two-step process to open links from the applications on your primary browser. In this article, let’s see how you can disable the in-app browser for certain Android apps.

(from How to Disable In-App Browser for Android Apps)

In olden times, when mobile apps wanted to show web content, they would open the URL in the default browser. This caused the browser to become the active application, and depending on the platform, may have also required the user to perform more than just a single tap to return to the previous app. Keep in mind that this was happening while apps were just beginning to gain traction and marketers were scrambling to identify valuable metrics in this new app space. In the effort to keep metrics going “up and to the right”, it was only natural that publishers would want users to stay in “their” app for as long as possible, and lo! The in-app browser was conceived as a means to this end.*

(from In-App Browsers: What You Need to Know)
We should either disable the in-app browser altogether, or provide an option to do so. I'm not sure which is best, I'd love some feedback from users on this.
I wasn't able to find any explanation of why we have an in-app browser currently, I'm not sure if it was just enabled by default by React Native, or if we added it. However, I wasn't able to find any commits explicitly enabling it, which I suspect means that it was just a default.

@WesleyAC

We should either disable the in-app browser altogether, or provide an option to do so. I'm not sure which is best, I'd love some feedback from users on this

links are opening in in-App browser and then there is an option to open in separate browser(open in browser), I liked the way it is implemented , as sometime I see single page and navigate back, sometime If I like the content then starts more search and like to open new tabs and so on. For this I saw there is already an option to to select and surf in separate browser and its good.

See also #4323, where a user requested this on iOS. I'm going to close this issue, and edit that one to be for both iOS and Android.


Add ArangoDB Container support

ArangoDB -> https://www.arangodb.com

Thanks!
Celery definitely needs this one for its integration suite.

Codecov Report

Merging #221 (43b84df) into master (df0c28a) will decrease coverage by 17.25%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master     #221       +/-   ##
===========================================
- Coverage   86.18%   68.93%   -17.26%
===========================================
  Files          26       27        +1
  Lines         702      721       +19
  Branches       65       65
===========================================
- Hits          605      497      -108
- Misses         76      207      +131
+ Partials       21       17        -4

Impacted Files
Coverage Δ

testcontainers/arangodb.py
100.00% <100.00%> (ø)

testcontainers/mssql.py
0.00% <0.00%> (-100.00%)
:arrow_down:

testcontainers/postgres.py
0.00% <0.00%> (-100.00%)
:arrow_down:

testcontainers/compose.py
0.00% <0.00%> (-98.37%)
:arrow_down:

testcontainers/mysql.py
0.00% <0.00%> (-84.00%)
:arrow_down:

testcontainers/oracle.py
0.00% <0.00%> (-55.56%)
:arrow_down:

testcontainers/core/generic.py
77.14% <0.00%> (-8.58%)
:arrow_down:

testcontainers/core/container.py
70.21% <0.00%> (-6.39%)
:arrow_down:

Continue to review full report at Codecov.

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

The failing tests seem to be the consequence of a recent sphinx change (https://github.com/sphinx-doc/sphinx/issues/10474). You should be able to fix it by changing language = None to language = 'en' in docs/conf.py.

Fixed

Thanks for your review.  updated.

Hi, all resolved.

@tillahoffmann - I will appreciate if you could merge and generate a new release.

Hi @thedrow / @SergeyPirogov  any chance you can step in and approve&release this one ?

@nshine thank you

Hi @SergeyPirogov
Thanks for the approve - can you please trigger a new release? i.e. 3.6.0


Traceback (most recent call last):

While running this code on CYGWIN
`$ python -m scripts.label_image \

--graph=tf_files/retrained_graph.pb
--image=tf_files/flower_photos/daisy/3475870145_685a19116d.jpg`

this error showed up
Traceback (most recent call last):  File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main     "__main__", fname, loader, pkg_name)   File "/usr/lib/python2.7/runpy.py", line 72, in _run_code     exec code in run_globals   File "/home/Admin/tensorflow-for-poets-2/scripts/label_image.py", line 24, in <module>     import numpy as np ImportError: No module named numpy

@pumpkinband it looks like you have no installed numpy. Please run the following in your terminal.
pip install numpy==1.14.5

@av8ramit  actually i tried but every time this error keeps going
Command "/usr/bin/python -u -c "import setuptools, tokenize;__file__='/tmp/pip-i                                                                                nstall-LkomBi/numpy/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f                                                                                .read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" i                                                                                nstall --record /tmp/pip-record-bAHGSs/install-record.txt --single-version-exter                                                                                nally-managed --compile" failed with error code 1 in /tmp/pip-install-LkomBi/num                                                                                py/

I'd recommend trying in a virtual environment with the latest corresponding python version. If that doesn't work maybe update your setuptools version as well.
pip install --upgrade setuptools==39.1.0

Tried both the things but 'numpy' isn't installing.

I recommend filing this issue on the numpy repository as they will have a better idea of how to help you. After you get that working please comment or open a new issue if there is any issue with the TensorFlow portion.

Thanks a lot brother.


Allow user to leave lobby even when an error occurs

https://github.com/sopra-fs23-group-07/sopra-fs23-group-07-client/commit/cb7d0da6b0e8ad977f83b05086d88b2409f63b2a


for issue #6: added some funny phrases

for issue #6: added some funny phrases.. my first pull request / contribution to an open source project.

Thanks for contributing!


feat: Support for the replace parameter in useLocation().route

Implementation https://github.com/preactjs/preact-iso/issues/6#issuecomment-1929966569
const loc = useLocation()

loc.route('/', true)

Strange, I didn't find the formatting configuration, maybe this is used?

Thanks


Allow ProgressBarIter to use a progress bar without consuming it

Edited to use Borrow instead of Deref.
The ProgressIterator trait and corresponding ProgressBarIter struct are very useful for adding a progress bar to an iterator.  The currently implementation of ProgressIterator::progress_with() takes ownership of/consumes the progress bar.  This limits control over the progress bar after it has been added to the iterator.  For example, in the code below we can't call pb.finish_and_clear() to change the default finish behavior with the current implementation.
(In theory we can still access the public field ProgressBarIter::progress, but in practice the ProgressBarIter will probably be consumed by another chained iterator at which point the field will no longer be accessible.)
Maintaining backwards-compatibility with the current API, I propose makeing ProgressIterator::progress_with() generic over std::borrow::Borrow<ProgressBar> so that it can consume a ProgressBar or take a reference &ProgressBar, Box<ProgressBar>, Arc<ProgressBar>, etc.  The user then retains ownership (if using &ProgressBar) or a reference/clonable reference (with Box or Arc) which can be used to modify the behavior of the progress bar even after it has been added to the iterator.  For example, the default behavior is for the progress bar to be finish()ed when the iterator is exhausted, but using the generic progress_with() below we can call finish_and_clear() to clear the bar after we are done with it.
use indicatif::{MultiProgress, ProgressBar, ProgressIterator, ProgressStyle};
use std::sync::Arc;
use std::thread::sleep;
use std::time::Duration;

fn main() {
    // Create a MultiProgress.
    let multi = Arc::new(MultiProgress::new());

    // Add an overall progress indicator.
    let main_pb = multi.add(ProgressBar::new(5).with_style(
        ProgressStyle::default_bar()
        .template("{msg} [{wide_bar}] {pos}/{len}")
    ));
    main_pb.set_message("Overall");
    main_pb.tick();

    // Spawn the MultiProgress animation thread.
    let multi_animation = {
        let multi = multi.clone();
        std::thread::spawn(move || {
            multi.join()
        })
    };

    // 100 ms delay.
    const DELAY: Duration = Duration::from_millis(100);

    // Create five sequential tasks with progress indicators.
    (0..5).progress_with(main_pb).for_each(|x| {
        // Progress indicator with 10 steps.
        let pb = multi.add(ProgressBar::new(10).with_style(
            ProgressStyle::default_bar()
            .template("{msg} [{wide_bar}] {pos}/{len}")
        ));
        let msg = format!("Task {:<2}", x);
        pb.set_message(&msg);

        // Wrap task in progress bar.  Note passing &pb instead of pb.
        (0..10).progress_with(&pb).for_each(|_| sleep(DELAY));

        // Clear progress bar.
        // Not possible before!!!
        pb.finish_and_clear();
    });

    // Wait for animation to finish.
    multi_animation.join().unwrap().unwrap();
}

If you think this kind of change is consistent with the design goals of indicatif then I would be happy to add it to the rayon parallel progress iterators too.

This may not be super obvious, but a ProgressBar internally has an Arc<RwLock<>> around the actual state, so passing ownership is actually not super meaningful in this case. I have also run into the problem of not being able to pick the right finish behavior, but I think we might this want a differently scoped fix for that problem. (Maybe related to #219?)

This may not be super obvious, but a ProgressBar internally has an Arc<RwLock<>> around the actual state, so passing ownership is actually not super meaningful in this case.

I had long suspected that ProgressBar uses some form of interior mutability for its methods taking &self and mutating state, but as you say this Arc is internal.  I may be overlooking some way of leveraging this in ProgressBar's API, but since ProgressIterator::progress_with() takes ownership of ProgressBar (not Arc<ProgressBar>) I don't see any way to manually tweak the bar after it has been wrapped in a ProgresBarIter in a meaningful way, for example, to change the progress style if a warning/error occurs during iteration, to modify the finish behavior, etc.
pb = ProgressBar::new();
piter = (1..10).progress_with(pb);
// technically can still access pb like this
piter.progress.set_style(/* whatever */);
piter.for_each(|x|
    // can't access pb from in here
);

With the proposed backwards-compatible change:
pb = ProgressBar::new();
piter = (1..10).progress_with(&pb);
// now can access pb directly or piter.progress if we want
piter.for_each(|x|
    // now we can access pb from in here
);
// and even out here

I have also run into the problem of not being able to pick the right finish behavior, but I think we might this want a differently scoped fix for that problem.

Agreed, I had noticed that the finish() behavior for ProgressBarIter is different from the finish_and_clear() behavior in ProgressBar's Drop trait as well.  I was going to address that in a different PR, probably by proposing an enum to specify the default finish method on Drop or completion of an iterator.

You can just clone the ProgressBar and use the clone to mutate it after setting up the iterator?

As for the finish behaviour, I'm inclined to think we should have a behavior-agnostic finish() method that users like the iter wrappers use, and APIs to configure the desired finishing behavior. That seems more in line with how the rest of the APIs work.

You can just clone the ProgressBar and use the clone to mutate it after setting up the iterator?

Ah, I see.  ProgressBar derives Clone and ProgressBar's only member, state, is an Arc, so a cloned ProgressBar actually references the same ProgressBar that is was cloned from.  This makes my pull request largely redundant since you could just do the following with the minimal expense of an unnecessary clone in some cases:
let pb = ProgressBar::new();
let cloned_pb = pb.clone();
(0..10).progress_with(pb).for_each(|x|
    pb_cloned.do_something();
);
pb_cloned.finish_and_clear();

Even though this behavior is clearly documented for the struct-level doc comments on ProgressBar, it was totally not obvious to me because ProgressBar::state is not public, and moreover I had seen examples elsewhere of people using Arc<ProgressBar>.  We don't have arbitrary self types yet, but I wonder if it would be practical to refactor like this to make the Arc explicit in the API?
pub struct ProgressBar {
    state: RwLock<ProgressState>,
}
impl ProgressBar {
    pub fn new(len: u64) -> Arc<ProgressBar> {
        Arc::new(ProgressBar { state: /* ... */ })
    }
    pub fn tick(self: &Arc<ProgressBar>) {
        self.state.update_and_draw( /* ... */ );
    }
}

Alternatively you could override clone() like this to make the behavior even more explicit in the documentation:
impl ProgressBar {
    /// Clone this `ProgressBar`.  The clone references the same bar
    /// as the original.  This is similar to invoking `clone()` on [`std::sync::Arc`].
    pub fn clone(&self) -> Self {
        <Self as Clone>::clone(self)
        // Or drop the #[derive(Clone)] and:
        // ProgressBar { state: self.state.clone() }
    }
}

As for this pull request, I don't mind if you close it without merging.  Merging would offer a little more flexibility than the current API without any performance cost, but it would make the code a little more complicated to read.  If you don't merge, it's still possible to do what I want using the special behavior of clone().

So how would you solve the finish behavior issue with the current clone() approach? I was under the impression it wasn't possible.

My apologies, it is mostly possible using the current clone approach, and this PR doesn't substantially change the behavior.  I am so daft I did not realize that ProgressBar::clone() has the same clone-by-reference behavior as Arc::clone() (as opposed to the behavior of String::clone(), which would create a totally new string).  So returning to the very first example in the PR you can do:
// Create the progress bar.
let pb: ProgressBar = multi.add(...);
// Clone the progress bar.
// The cloned bar actually controls the same bar as `pb`!
let pb_clone = pb.clone();
// Move `pb` into the `ProgressBarIter` struct,
// and do something with the resultant iterator.
(0..10).progress_with(pb).for_each(|_| sleep(DELAY));
// Here `ProgressBarIter` has called `pb.finish()`.
// Then the `ProgressBarIter is dropped so `pb` is also dropped.
// But we still have `pb_clone`, so we can do this:
pb_clone.finish_and_clear();

Now the point of this PR was to make ProgressIterator::progress_with() receive the progress bar by value/move or by reference.  But since ProgressBar already has clone-by-reference behavior, this change turns out to be unnecessary.
Now, as a separate PR it might be nice to introduce some API into ProgressStyle or else something like ProgressBar::set_finish(behavior: FinishBehavior) where FinishBehavior is some enum that lets the user choose between finish with message, finish and clear, finish at current position, etc.  And then we could modify the ProgressIterator implementation and the Drop implementation for ProgressBar to do what is says in this enum instead of some fixed, default behavior.  That would save the user from having to keep track of a clone of the progress bar after it has been wrapped in an iterator.

Right, that's what I was wondering about. Okay, I'm going to close this -- I actually think hiding the Arc inside ProgressBar is nice, although I'd be happy to take more patches documenting the behavior in places you think should talk about it that currently don't. Would also like to take a patch to set the finish behavior with an API along the lines you describe (sounds good so far).


Vim cursror is always set to the one of Windows Terminal

The cursor in Vim doesn't change and the solutions provided in issue 4335 don't work.
Environment
Windows build number: 10.0.18363.0
Windows Terminal version (if applicable):  1.4.3243.0
Vim: 8.2.2127

Steps to reproduce
Install Vim and switch from normal to insert mode. The cursor always stays the same. You can set the cursor in the Windows Terminal settings .json, but it will not change in Vim.
Expected behavior
The cursor should change when switching modes in Vim
Actual behavior
It doesn't change

WHAT VERSION OF VIM are you using. You didn't list "any other software", which is part of the template, so we can't even tell if you're using Windows vim or Linux vim in WSL.

I updated the issue

Are you using windows vim.exe or vim inside wsl.

I am using the Windows version, because I am working with a Windows-only compiler, but the behavior is the same for WSL2 in Windows Terminal. When I use WSL only the cursor is always a block and when I use Powershell only the cursor is a block in normal mode and an underscore in insert mode.
To be specific:
Powershell in Windows Terminal (normal & insert mode):

WSL in Windows Terminal (normal & insert mode):

For the pure Powershell I have made the block cursor the defaults setting in the Powershell settings, which results in a block for normal mode and an underscore for insert mode, which is acceptable. But then the Vim themes don't work well in Powershell, but they do in Windows Terminal...
Pure Powershell (normal mode):

Pure Powershell (insert mode):

Pure WSL (normal & insert mode):

Facing the same issue

Facing the same issue

I'm not sure that this is a WT issue. The shape of the cursor is configured in vim, and by default will not change when switching modes (that I am aware of). Additionally, make sure that your vim version includes the +cursorshape feature (:echo has('cursorshape') should print 1, and it should say +cursorshape in the output of vim --version). I use nvim (version v0.5.0-dev+16044ae-dirty) and vim (version 8.2) with the same .vimrc and both have no issue under WSL with the cursor changing from mode to mode. Try adding this to your .vimrc:
let &t_SI.="\e[5 q"
let &t_SR.="\e[4 q"
let &t_EI.="\e41 q"

For more info on this, see :h termcap-cursor-shape. I am unable to reproduce this issue, and by default vim will not change cursor shapes when changing modes (if you do vim -u NONE in PowerShell, you may see this behavior as well). It's likely that the default system-wide vimrc being used in PowerShell is setting this up for you.

    let &t_SI = "\<Esc>[6 q"
    let &t_SR = "\<Esc>[3 q"
    let &t_EI = "\<Esc>[2 q"

This is what I had to add to my .vimrc to make the cursor function as expected in WT.

Side subject here, anyone knows the terminal escape sequence for the openBox cursor? That would make me very happy and be an easy solution. I also have the same issue as well with neovim .05 and .06. Replicated the issue in Windows terminal with wsl and ubuntu:focal in docker. I tried the below with other cursor shapes as well same thing. My zshrc and init.vim are empty.
https://share.getcloudapp.com/mXuPEo5n

vim.exe still not working in WT.
If you try to open vim.exe' in 'cmd.exe' without any settings, the cursor shape will change while switching mode. But it don't in WT. And set t_EIlike things have a lot of glitches like not working for replace mode, and also break plugin likeultisnips, at least i didn't find any config work perfectly in all situation. So WT show support changing cursor without any config just like cmd.exe` does.


Strict typing on create() doesn't account for generated fields

Describe the bug
This change: https://github.com/sensedeep/dynamodb-onetable/pull/339 changed the typedef for Model.create() to enforce required model properties during create. This makes sense, except if you have a required property that's also generated.
For example, we have a property like this
      consumerId: { type: String, generate: true, required: true },

The new type schema requires I manually input consumerId and gives an error Property 'consumerId' is optional in type 'Partial<Consumer>' but required in type '{ consumerId: ConsumerId; ...
To Reproduce
I'm not entirely sure how to give you a repro for a type issue. I can try if this isn't clear though.
Expected behavior
Generated fields should be marked required in the Model type, but not in the input to Model.create
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: Mac OS
Node Version: 14.18
OneTable Version: 2.3.8
TypeScript Version: 4.6.3

A minimal debug.ts would save us the time of creating one.

I'm looking into this.
An earlier PR got reverted that typed the create properties as:  CreateProperties
This was not working where it would not flag any missing required field.
But the alternative is flagging missing generated properties.
What we need is a fixed CreateProperties definition.
I'm digging into the TS for this, but this has some unholy magic involved. Any help would be appreciated.

The fix may be changing
type CreateProperties<T> = Omit<T, keyof Generated<ExtractModel<T>>> | Generated<ExtractModel<T>>

to
type CreateProperties = Omit<T, Generated<ExtractModel>>

Any feedback on the above suggestion, greatly appreciated.

I'm digging more into this and I think there is some confusion regarding what does "required" mean.
Required means that you must provide the property when creating.
So you should not use required on a 'generate' property as OneTable is providing the value and you do NOT have to provide it to create.
Some general rules:

Only flag properties as required that must be provided by the caller of create
Don't set required when using 'default'. Either set required and have the caller provide the data or use default if it is optional.

With these assumptions, CreatedProperties can go away and just be replaced with T.
What am I missing?

How would you then represent nullability of a field in the schema? I'm using TypeScript and would like to be able to use onetable's Entity<> types as my domain entities and not to having to check attributes for null and undefined every time I get an entity from a DB.

Back to the original issue.
Yes, there is a bug in handling create properties that are required and generated. The TS handling requires that you provide the generated property to satisfy the type signature. This is not ideal. We're working on more comprehensive fix for this. See discussion #

Regarding nullability in general.
The Table params "nulls" controls how nulls are handled. Generally this is set to false, meaning, never write nulls to the database. With nulls == false, when a property is null and Model.update() is called, OneTable will remove the property from the item. It would be nice if our typings for Update() allows nulls and other APIs did not.
With TS, nullability gets somewhat overloaded. What does null really mean in your app? It varies from app to app. Does it mean the property is not present in the database or that it is present with a null value. i.e.  Do you want to store null in the database?
So we should separate the concerns of what gets stored in the Database, vs Entity types that your app uses.
We generally run with Table.parms.nulls == false and never store nulls in the database.  This greatly simplifies things.
Now, we need OneTable TS support to not allow nulls in Entity types (but perhaps only if params.nulls == false).

What does null really mean in your app?

I also don't use null in my app at all. I always use strictNullChecks. If I need to express absence of value I use undefined and add it to type signature accordingly: name: string | undefined or name?: string.

Do you want to store null in the database?

No. And I'm happy with the default behavior of params.nulls: false.
I would also prefer if onetable didn't return null values for fields absent in the DB including nested Object fields.
And I would rather prefer update() rely on params.remove: [name] explicit syntax to remove fields instead of fiddling with nulls.

Great, thanks.
We're toiling at the moment trying to get TS magic to work on create().

Okay, we have it working sweetly. This is some serious TS magic.  Ready for test.
The new magic does:

Fixed create(), find() and update() to correctly flag incorrect properties.
Fixed lax use of nulls. Now only permitted in update() to delete properties.
Correctly visually highlights in VS code when you have incorrect properties or types in data items.
Correctly highlights methods arguments for create(), find(), update etc. Will warn when required parameters are missing and when data types are incorrect.
Correctly autosuggests data field property names as you code.

The Schema Rules

Required in a schema model field means the field is required in data items. A model type will have these as required properties. Required means an item object MUST have the property present. Don't confuse this with the properties you give to create/update/get/find etc. Those are NOT model data types. They have special rules.
Without required, the property will be optional. Default, generated, value-templates can be marked as required.
In create() you do not have to provide generated fields, fields with default values or fields with a value template ... Even if they are flagged as required.
In find() you can provide any subset of the properties (provide it is valid for DynamoDB of course).
In update() you can provide any subset of the properties AND you can set a properties value to null to remove it.
In update() you can supply sparse nested objects if you have set partial: true to the table constructor params. Update will update only the properties you have provided, regardless of how deep.

Notes

You can now use Table.getModel in Typescript without <>. i.e.

Use
const User = table.getModel(‘User’)

rather than
const User = table.getModel<UserType>('User')

The Bad
The intellisense (IS) on methods is pretty complex. However the IS will correctly prompt when typing for data fields and the IS on data types is great.
Questions?

@mobsense seems like the updated and created timestamp fields are still required when calling Model.create(...).
Take a look at this schema:
{
  format: "onetable:1.1.0",
  version: "0.0.1",
  indexes: {
    primary: { hash: "PK", sort: "SK" },
  },
  params: {
    typeField: "_type",
    timestamps: true,
    isoDates: true,
    createdField: "createdAt",
    updatedField: "updatedAt",
  },
  models: {
    Payment: {
      PK: { type: String, value: "id#${id}", hidden: true },
      SK: { type: String, value: "id#${id}", hidden: true },
      id: { type: String, required: true },
      createdAt: { type: Date, required: true },
      updatedAt: { type: Date, required: true },
    },
  },
}

Didn't find it in the documentation: if I specify params.createdField: createdAt, should I remove this field from the models definition?

Didn't find it in the documentation: if I specify params.createdField: createdAt, should I remove this field from the models definition?

Yes, you don't need to specify timestamp fields in the model.

Your choice. You don't have to specify them in the model, but if you are using TS and want intellisense, then define them in your model.

@dev-embedthis if I remove these fields from the model, will they be considered as "metadata" and won't they be returned from methods such as Model.get?

The timestamp fields are always returned. Probably should respond to the "hidden" param and not be returned by default.
But currently, they are returned.
Changing would probably break some existing code.

Since timestamp fields are always returned, shouldn't they be in the current type definitions if they're not specified in the model (just in createdField/`updatedField)?

To do that, just define them in your model. TS definitions are auto-generated from your schema, so if you want them defined, you need to include them in your model.


cppitertools: init at 2.1

Description of changes
This PR packages CPPItertools, a port of Python's itertools module a builtin functions like enumerate(), to C++17.
I've added myself as a maintainer.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thanks @eclairevoyant! Those should be fixed now!

Thanks @eclairevoyant! Those should be fixed now!

I have validated it builds on Darwin aarch64, and I have seen Qyriad build it on x86_64-linux, pray I do not validate it any more. Going to merge it.


PyCharm support

A general support for the intellij platform would be nice: some information can be found here.

Other Jetbrains Products are supported since version 0.4.2

Thanks!


Fix customization spec

Summary
Fix failing customization spec

some tests fail because user is logged out
login each test instead of suite

Ticket Link
none, failing on daily Cypress test

This is weird 'cause I'm not seeing potential place where it will trigger user to logout.  Maybe we'll need to look at this deeper.

@saturninoabril This might be because of the IdleTimeout set to default 60s and is triggering users to auto logout after that


Curvature weighting

Currently there is an Exception thrown when calculating the distance between two edges.

Ups, hier hab ich mich leider verklickt.


Update simple-example.rst

This PR fixes grammar issues on the introduction part of the tutorial
Changes
Improved readability and grammar in the updated section.
How I tested this
Checked for proper grammar, spelling, and punctuation in the updated section.
Notes
Checklist

[x] PR has an informative and human-readable title (this will be pulled into the release notes)
[x] Changes are limited to a single goal (no scope creep)
[x] Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
[x] Any change in functionality is tested
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged / future TODOs are captured in comments
[x] Project documentation has been updated if adding/changing functionality.

@sudoevans nice catch, thanks!


Divide by zero error in run_markov

Hi,
Thanks for the great package!
I recently encountered an error in run_markov. I was trying to select  a subset of cells that are in a specific (presumed terminal/mature) cluster and run backwards.
anaconda2/envs/scanpy/lib/python3.6/site-packages/velocyto/analysis.py:1857: RuntimeWarning: invalid value encountered in true_divide
self.tr = self.tr / self.tr.sum(1)[:, None]

I think it is occurring when prepare_markov returns a tr that has NaNs in it. This occurs from self.tr = np.array(self.transition_prob[cells_ixs, :][:, cells_ixs]) if one selects cells_ixs that have a transition_prob of 0, since then when the following lines are called:
np.fill_diagonal(self.tr, self.tr.max(1))
self.tr = self.tr / self.tr.sum(1)[:, None]

self.tr gets filled with rows of NaNs in it. Subsequently, after running run_markov all of vlm.diffused is NaN. What is the best way to avoid this?
Thanks,
David

The best way is to make sure that the transition matrix is a valid one and
that each cell has a neighbour. This could be achieved in a greedy way for
example.
Otherwise you could patch the problem by replacing Nans with zeros after
division or zeros with tiny floats before division. The latter has less
guarantees to behave correctly.
In both cases you can make patched version of the method defining a
function locally that has the first argument the object. You could start by
copy pasting the source of the method modifing it and calling it on vlm


MINOR: Respect the default value of partition argument in SimpleConsumerShell

The partition argument is not marked as required, and has a default of 0, according to the tool's help message. However, if partition is not provided the command returns with Missing required argument "[partition]". This patch is to fix the required arguments of the tool by removing partition from them.

LGTM, thanks!


🧼 lint service-worker

[x] Lint files

I hope we could remove service-worker.js in month or so. probably those who hasn't visited us in month we don't care much


test: 💍 pass more ast test case

pass more css ast test,
left bootstrap.css and bootstrap.rtl.css failed


Use relative publicPath in webpack build

Have a use case where a client app needs to be hosted under a sub-path (https://client.com/subPath/clientApp) and using / as the base publicPath causes issues. We should be able to just set it to ../ for prod builds and everything will work the same since the index.html is always in an app sub-directory from the root context.

Closed via #7


MNT Update dependencies

Update Dockerfile, patch around inline optimization issues, and hack around drop of atomics on CPU to keep a working CI (by pinning CI on an old working version).

Apparently it doesn't work:
WARNING: The directory '/github/home/.cache/pip' or its parent directory is not owned or is not writable by the current user. The cache has been disabled. Check the permissions and owner of that directory. If executing pip with sudo, you should use sudo's -H flag.
WARNING: Requirement '/opt/numba_dpex/cpu-stable/numba_dpex*.whl' looks like a filename, but the file does not exist
ERROR: numba_dpex*.whl is not a valid wheel filename.

It's normal that the test don't pass yet, we must merge first so that the runner build and update the image to our repo, and re-run the pipelines after that. I'll perform a few final tests locally before.


Where is the distillation loss reflected in your code？

Where is the distillation loss reflected in your code？
The total is : loss_tot = loss + loss_exist * 0.1. I fail to find the distillation loss.
in https://github.com/cardwing/Codes-for-Lane-Detection/blob/a224eb4d38cf3294ea80b2d4181f8afe028d801e/ERFNet-CULane-PyTorch/train_erfnet.py#L151

The self distillation loss is not added in the pytorch code. You need to add it on your own.

can you show me an example of how to add the self distillation loss in the pytorch code？

You can refer to the torch code.

请问您添加成功了吗


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-29-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 4310

Questions ?
Please refer to the Backport tool documentation


Set default AnimationStrategy for AsepriteNativeParser to Loop

===:clipboard: PR Checklist :clipboard:===

[x] :pushpin: issue exists in github for these changes
[x] :microscope: existing tests still pass
[x] :see_no_evil: code conforms to the style guide
[x] :triangular_ruler: new tests written and passing / old tests updated with new scenario(s)
[x] :page_facing_up: changelog entry added (or not needed)

==================

Closes #200
Changes:

Set default AnimationStrategy for AsepriteNativeParser to Loop for Aseprite animation tags whose direction/type is Forward

@dericatienza Many thanks!

I'll bump the version soon!

@dericatienza New version published on npm v0.28.1
<EMAIL_ADDRESS>


'Mv purge all all' Results in disability to chat.

I swear, every time I do /mv purge all all noone is able to chat at all.
No error. No message. No anything.
This server is running CraftBukkit version git-Bukkit-1.1-R6-29-gd6d767e-b2029jnks (MC: 1.2.3) (Implementing API version 1.2.3-R0.1-SNAPSHOT)
01:29:03 [INFO] Multiverse-Core version 2.4-b512

Might be related to https://github.com/Multiverse/Multiverse-Core/issues/538 - apparent outcome is identical (chat doesn't work)

True, I have Mv-Core and Mv-Inv..

Probably all the buggy quick'n'dirty R6 updates.

...This is really pissing me off also...

Remember, you're on a dev build of Bukkit and MV. This means you're building a sand castle on top of a swamp which is floating in the ocean. (All of this is built on the "never has any bugs" Vanilla Minecraft). Expect turbulence and execute patience.

I've updated my other issue #538 to add some other places I've found where this bug occurs; I'm suspecting they are all linked together, since they all have the same bug as the end result.

Alright thanks, will have a look tonight.

Going to close this one in favor of #538, keep future discussion there


Issue creating one  pagers either through robyn_outputs or onepager()

Project Robyn
Describe issue
I'm able to follow the demo.r script up until the point where you use the robyn_outputs function to produce one pagers. It writes every file except the one pagers. Even if I ignore this function in the demo and try to spit out a one-pager using the onepager() function, my Rstudio instance continues to run for hours with no end in sight until I interrupt the session and it crashes. At first, I was able to get it to run by installing the rngtools and foreach libraries, but now it is spinning out again. Is it possible that robyn has additional library requirements that are not mentioned in the requirements?
Provide reproducible example
This section of code is where I get stuck. My dataset is different with many zeroes throughout.
OutputCollect <- robyn_outputs(
InputCollect, OutputModels,
pareto_fronts = "auto"
min_candidates = 100
calibration_constraint = 0.1,
csv_out = "pareto"
clusters = TRUE
export = create_files
plot_folder = robyn_directory
plot_pareto = create_files
)
print(OutputCollect)
Environment & Robyn version
Robyn Version 3.10.3
R Versions 4.3.2

Ideally, if you are able to run the outputs. The package requirements should have been met. Can you please share the OS you are using. If using windows, can you also take a look at the log file which is generated and stored on your HDD.

Repeating the above with more memory allocation stopped the above error


Add JSDoc Comments

Just wondering why you would need JSDoc comments when this repo is written entirely in TypeScript? Seems a little redundant to me.

@zmrl010  care to chime in? What are you doing the other day where this would’ve helped?

Hi @patlehmann1 ,
I was the one who suggested JSDoccomments for this lib. I made the suggestion because while we do have all the typing information through typescript, JSDoc comments are capable of providing more than just types. They can provide a description, usage examples, links to extended documentation, etc. Most major IDEs support JSDoc in some fashion. In particular I use VSCode and PyCharm a lot and they both show JSDocs on hover for just about anything that has them. You can even JSDoc on interface properties, which is something I like to do for documenting exported interfaces.
Because the library is written typescript, I was planning on omitting the types out of the JSDoc comments anyways. The main goal is to get some descriptions for most exports. I plan on getting this done sometime this week
Hope that answers it for you!


Use valid baseURI for EOL versions of OCP

The old baseURI was recently decommissioned and EOL versions of OCP are unable to install. While we don't support EOL versions, some partners have an urgent need for installing EOL versions, this change brings them this.

check dallas ocp-4.7-vanilla

check dallas ocp-4.7-vanilla openshift-vanilla:components=ocp=4.7.60

check dallas ocp-4.7-vanilla openshift-vanilla:components=ocp=4.7.60


iOS says: Accessory not responding

hi,
I am trying to get  homebridge-octoprint running but  unfortunately it fails every time... The plugin is installed  and configured  in the config.json correctly.
I testet the server name and API with postman and got a proper response but the home app on iOS is still saying: 'no response'
every time a restart/ open the home app on iOS, the homebridge log says:
3|homebridge  | [5/7/2020, 10:54:56] [OctoPrint] Getting current printing state: GET 192.xxx.xxx.xx:port/api/printer
3|homebridge  | [5/7/2020, 10:54:56] [OctoPrint] Getting current job data: GET 192.xxx.xxx.xx:port/api/job
3|homebridge  | [5/7/2020, 10:54:56] [OctoPrint] Getting current printing state: GET 192.xxx.xxx.xx:port/api/printer

I also tested to configure the "server" key with IP and correct port in config.json but without success.
thank you for any kind of idea to solve this problem.

Hi, I have the same problem.

Be sure to include the protocol (http://) and not just the address (192.xxx.xxx.xx:port)
{
    "accessory": "OctoPrint",
    "name": "OctoPrint",
    "server": "http://<IP_ADDRESS>",
    "api_key": "xxxxx"
}


Regression in  in parquet 42.0.0 : Bad parquet column indexes for All Null Columns, resulting in Parquet error: StructArrayReader out of sync on read

Describe the bug

Writing a column consisting solely of all nulls results in an empty offset index for that column
#[test]
fn test_writer_all_null() {
    let a = Int32Array::from(vec![1, 2, 3, 4, 5]);
    let b = Int32Array::new(vec![0; 5].into(), Some(NullBuffer::new_null(5)));
    let batch = RecordBatch::try_from_iter(vec![
        ("a", Arc::new(a) as ArrayRef),
        ("b", Arc::new(b) as ArrayRef),
    ])
    .unwrap();

    let mut buf = Vec::with_capacity(1024);
    let mut writer = ArrowWriter::try_new(&mut buf, batch.schema(), None).unwrap();
    writer.write(&batch).unwrap();
    writer.close().unwrap();

    let bytes = Bytes::from(buf);
    let options = ReadOptionsBuilder::new().with_page_index().build();
    let reader = SerializedFileReader::new_with_options(bytes, options).unwrap();
    let index = reader.metadata().offset_index().unwrap();

    assert_eq!(index.len(), 1);
    assert_eq!(index[0].len(), 2); // 2 columns
    assert_eq!(index[0][0].len(), 1); // 1 page
    assert_eq!(index[0][1].len(), 1); // 1 page
}

This appears to have been a bug introduced by https://github.com/apache/arrow-rs/pull/4389
In particular - https://github.com/apache/arrow-rs/pull/4389/files#diff-b1859e4da1d85e57a4185dc407458ac83a369dac132285689c27e878e3695ad6R695
To Reproduce

Expected behavior

Additional context

I added some more stuff to the title of this ticket to make it easier to find / search for

FYI We found this in our internal testing. I will post symptoms here to help anyone else who comes across this:
We found a query like this in IOx that resulted in Parquet error: StructArrayReader out of sync on read errors
$ datafusion-cli -c "SELECT col, time FROM 'data.parquet' WHERE<PHONE_NUMBER>953220316 <= time::bigint"
DataFusion CLI v27.0.0
Arrow error: External error: Arrow: Parquet argument error: Parquet error: StructArrayReader out of sync in read_records, expected 0 skipped, got 11

The workaround for datafusion is to disable using the page index:
❯ set datafusion.execution.parquet.enable_page_index = false;

As this was a writer bug, you will need to rewrite the affected files. Their isn't a read-side patch as the page indices are simply invalid


Internationalization (translations)

Probable first languages:
English (already the default)
Portuguese (logical second choice)
your language!

Let's investigate the latest available translation strategies:

external translation services (paid or free)
rails style i18n
build process based i18n
whatever else you can find


Parallelize ts-morph creating source files

Whoop-de-do.  I saved nothing over about eight seconds.


Feat/tilde datasci content

created a prequisite content for data science specific projects
Also includes:
SQL included

Excludes:

week 1 to week 4 not included

content/syllabuses/data-sci.md does not look righ to me based on the video shared on Monday from Sheena

Sorted now @ry-oc thanks, hadn't seen the changes


Auto-detect owned ENS domains and update TEXT record from UI when creating new space.

To make the process of creating a new space more seamless, we want to eradicate the necessity to move over to https://app.ens.domains and do as much as possible inside of the snapshot UI.
The ENS Subgraph can be used to easily retrieve the domain names owned by an account.
https://thegraph.com/hosted-service/subgraph/ensdomains/ens?query=Example query (hosted service should be fine for this)

(account must be lowercase)
For settings the TEXT record from the UI:
https://www.npmjs.com/package/@ensdomains/ensjs

async setText(key: String, recordValue: String) => Promise<EthersTxObject>
Sets the text record for a given key for the current ENS name.

(Outside the scope of this issue: Ideally at some point we can even support registration of new ENS domains through snapshot.)

Hi, I would like to take up this issue. I have checked out the code for @ensdomains/ensjs npm package on a local test repository and got it to work. Given this is my first issue for the repo, I'll go through the code and ask any doubts here.

Sound good! Ideally we should use the function sendTransaction from Snapshot.js to uodate the text record, ENS.js is a bit heavy.


NavButtonBehavior designer error

I'm getting the following error in the designer:
Cannot create instance of type 'Template10.Behaviors.NavButtonBehavior' [Line: 0 Position: 0]
This is in a fresh file > new project based on the Hamburger Template from the Template Pack add-in version 1.18. I updated to the 1.1.12 preview version of the Nuget package. It works fine on 1.1.11.
Despite the error, the app compiles and runs just fine.

Irritating, I know. No idea. Glad it still runs. There's nothing special about your project? Is it Visual Studio 2015 Update 3 or is it Visual Studio 15 Preview?

It's VS2015 Update 3. It's the base Template10 hamburger template. Then I've tried to update the Nuget packages. There's nothing else I've changed. I noticed similar errors in other issues here related to NavButtonBehavior and UWP Behaviors library being somewhat brittle, so I kind of wrote it off to something similar.

For me, it's VS 2015 Update 3, too. Same way of getting the error. This is reproducible.

+1 for same error

+1 (VS 2015 Community Update 3) The error is on designing time (xaml), then compiles and run Ok

I'm having the same problem but my app doest even compile. It stops at the global::System.Diagnostics.Debugger.Break(); line. I'm using VS 2017.


Add support for WAF read and update

Add support for WAF read and update
Add test coverage for ListWAFRules and ListWAFPackages functions
Update documentation for WAF endpoints

cc @patryk are to you review? 🙇

@jacobbednarz thanks for the feedback I'll make those changes and ping you when ready for re-review 🙇

@jacobbednarz @patryk ready for re-review

@patryk thanks for the feedback, I've made the change to use WAFRulesResponse


Question: Azure DevOps extension in offline environments

How to set up the Azure DevOps extension in an offline environment?
I see that the Azure CLI supports installing from a local source like this:
 az extension add –source …
… but this doesn't seem to be enough. (It fails when it runs "pip", which tries to connect to public sources…).
Any tips on how an offline / or x-copy-like deployment looks like?

Download the latest wheel (*.whl) from https://github.com/Azure/azure-devops-cli-extension/releases
In your offline environment use: az extension add -s azure_devops-0.19.0-py2.py3-none-any.whl

@fedorbirjukov The first solution is not working for me as it requires another package called keyring

We got the same issue and had to install the extension and its dependencies manually:
    # Download the azure-devops extension
    # See https://learn.microsoft.com/en-us/cli/azure/azure-cli-extensions-list
    # See https://github.com/Azure/azure-devops-cli-extension/releases/tag/20240514.1
    wget https://github.com/Azure/azure-devops-cli-extension/releases/download/20240514.1/azure_devops-1.0.1-py2.py3-none-any.whl
    # Download the dependencies of azure-devops extension
    wget https://files.pythonhosted.org/packages/f6/b1/ba5a96bccd3496241d8908164b9502a129156443cdd5acbdbf04a90b7a09/distro-1.3.0-py2.py3-none-any.whl

    # Install the azure-devops extension
    /opt/az/bin/python3 -m pip install --target ~/.azure/cliextensions/azure-devops --no-deps distro-1.3.0-py2.py3-none-any.whl
    /opt/az/bin/python3 -m pip install --target ~/.azure/cliextensions/azure-devops --no-deps azure_devops-1.0.1-py2.py3-none-any.whl
    # Verify the azure-devops extension
    az extension list
    az devops -h


Ee name sort changes - Alternative

@miwasp Hey man, I think these are more like the changes you wanted, but can you please test it and let me know how it goes?
Note - This is only currently for reference only, please do not merge.
ref:
https://github.com/EmuELEC/emuelec-emulationstation/pull/33

The problem with gamelist examples is that i have several real life examples, which are sorted correct with sortorder "names" but seem completely unordered with sortorder "sortnames". I can't reproduce this yet with those simple examples provided in the other PR.
I'm currently building this PR...

Ok, take a look at the gamelist at the end. It contains no sortname tags. If i set sortorder to "Names ascending" i get:
Capcom..., Crazy..., Dead..., Mortal..., Power..., Sonic..., Soul...
This is the correct alphabetical order. If i set sortorder to "Sortnames ascending" i expect the same result but i get:
Mortal..., Power..., Sonic..., Soul..., Crazy..., Dead..., Capcom...
If you remove the releasedate tag in all entries the sorting is ok.
<?xml version="1.0"?>
<gameList>
        <game id="143384" source="ScreenScraper.fr">
                <path>./TDCFinal2.cdi</path>
                <name>Capcom Fighting Collection [WOR](Unl.)</name>
        </game>
        <game id="13512" source="ScreenScraper.fr">
                <path>./Dead or Alive 2 (USA)[RDC].cdi</path>
                <name>Dead or Alive 2 [US]</name>
                <releasedate>20000229T000000</releasedate>
        </game>
        <game id="13534" source="ScreenScraper.fr">
                <path>./Mortal Kombat Gold (USA)[ECHELON].cdi</path>
                <name>Mortal Kombat Gold [US]</name>
                <releasedate>19990831T000000</releasedate>
        </game>
        <game id="13520" source="ScreenScraper.fr">
                <path>./Power Stone (USA)[RDC].cdi</path>
                <name>Power Stone [US]</name>
                <releasedate>19990909T000000</releasedate>
        </game>
        <game id="13489" source="ScreenScraper.fr">
                <path>./Sonic Adventure (USA) (Limited Edition)[DCCM].cdi</path>
                <name>Sonic Adventure [US]</name>
                <releasedate>19990909T000000</releasedate>
        </game>
        <game id="13500" source="ScreenScraper.fr">
                <path>./Soul Calibur (USA)[DCCM].cdi</path>
                <name>Soul Calibur [US]</name>
                <releasedate>19990908T000000</releasedate>
        </game>
        <game id="13514" source="ScreenScraper.fr">
                <path>./Crazy Taxi.cdi</path>
                <name>Crazy Taxi [EU]</name>
                <releasedate>20000218T000000</releasedate>
        </game>
</gameList>

The releasedate tag shouldn't be compared in these changes. I will have to take a further look. That's really strange. Maybe the releasedate is outputting sortName in the metadata since it's after the name. Can you re-label the tags releasedate2 and check if it does the same thing?

If i re-label the tags to releasedate2 the sorting is ok. But releasedate is a regular tag which you also get when you scrape your games and again, sorting by name works ok.

So just to clarify you are selecting by sortName (ASC or DESC), and releasedate is getting sorted instead? Very strange bug I'll keep looking.

Hey can you also in ur UI settings make sure, "IGNORE LEADING ARTICLES WHEN SORTING" is disabled.

I just made a small change, but I'm in doubt it will fix the issue.

I checked the implementation of the metadata file, but could not see any odd code that is returning releasedate instead of the games name. Might have to look at this tomorrow, but yeah this is definitely an unchecked bug in ES.

This is testing with getName commented out. If it's not sorted still we can rule out getName being the issue.

@Langerz82 what is your discord username?

@shantigilbert I'm not sure yet, I only have discord on my windows OS lol. When I switch OS I will install discord on my Linux.

@miwasp, Looks like Batocera closed the issue because they have sortName commented out in there code base. I looked for keywords all over the emulationstation code and could not find any part that might of been responsible for the releasedate affecting the sortName metadata retrieval. It's so damn odd lool.

think I may of figured the bug out, looks like it may to do with the way order the sorttypes were pushed on the vector.

Hey can you also in ur UI settings make sure, "IGNORE LEADING ARTICLES WHEN SORTING" is disabled.
Yes, its disabled.

I just saw that if i select sortorder "release date ascending" i get a list with the year as prefix. This is usually only the case with sortorder "System, Release year ascending". Something is messed up with selecting the sortorder.

yep, I fixed it with the latest change. Now I think I gotta make a change so in the sort by list, Name, and sortName are at the top instead of sortName at the bottom.

@miwasp, Give me about an hour to clean up all the bad code, and I'll make the change for proper testing. ^_^ so glad i figured out the bug lol.

Im gonna make a new clean pull for proper testing.


ArgumentNullException when git name or email not set

It might be my fault, but with the new exe i get this:
ArgumentNullException (Value cannot be null. Parameter name: author)
Everything worked fine with the latest version
I just reinstalled my windows yesterday
Setup everything from clean

Originally posted by @xxxcrow in https://github.com/geluk/pass-winmenu/issues/97#issuecomment-939500947

@xxxcrow: I suspect you haven't set your user.email and user.name configuration variables yet. If that's the case, you can work around the issue by setting them:
git config --global user.name <name>
git config --global user.email <email>

If not, can you explain what you did when you got the error?

I actually thought git config should solve the issue, but it didn't
Firstly i had "UriFormatException" error
I knew i didn't set user.name/user.email for git
In the issue "UriFormatException" you sent a new exe file that should solve it
But with the new version a new exception showed "ArgumentNullException"
After you suggested git config --global user.name <name> git config --global user.email <email>
And... it shows the same "ArgumentNullException"

Fixed in 1.12.1


No files are being downloaded

Note: for support questions, please use the Discord. This repository's issues are reserved for feature requests and bug reports.

I'm submitting a ...

[x ] Bug Report

Do you want to request a feature or report a bug?

Bug

Pastebin link of the error

No error

What is the expected behavior?

To download the script

What is the motivation / use case for changing the behavior?

It isn't downloading the script when I run curl -Ls https://api.github.com/repos/Revenact/Pterodactyl-Script/releases/latest | grep -wo "https.*install.sh" | wget -qi -

Please tell us about your environment:

  - Version: v0.0.6
  - OS: [Ubuntu 20.04 (Tried on 18.04)]

Running dir does nothing. I am using root

Note: for support questions, please use the Discord. This repository's issues are reserved for feature requests and bug reports.

I'm submitting a ...

[x ] Bug Report

Do you want to request a feature or report a bug?

Bug

Pastebin link of the error

No error

What is the expected behavior?

To download the script

What is the motivation / use case for changing the behavior?

It isn't downloading the script when I run curl -Ls https://api.github.com/repos/Revenact/Pterodactyl-Script/releases/latest | grep -wo "https.*install.sh" | wget -qi -

Please tell us about your environment:

  - Version: v0.0.6
  - OS: [Ubuntu 20.04 (Tried on 18.04)]

Running dir does nothing. I am using root

You aren't wrong. It can't find a https trail with install.sh at the end.
Maybe removed?

You aren't wrong. It can't find a https trail with install.sh at the end.
Maybe removed?

Fixed.


New Parser and Analyzer support ShowProcedureStmt（Easy）

Required work and output

Use antlr4 to complete parse statements. The generated statement must follow the original data structure, and should be compatible with the old statement.

In the original syntax file  sql_parser.cup, the new syntax needs to be compatible with the syntax of the old version, using the syntax file sql_parser.cup of the old version as the reference standard.

Remove the logic of analyze functions in the statement and move the relevant logic to the new Analyzer framework. The new Analyzer is in the SQL/Analyzer directory.

If the code related to Privilege Check exists in the original analyzer, move the code related to privilege control into PrivilegeChecker.

When submitting the PR, please make sure there is unit test coverage for related changes.

Task

[ ] ShowProcedureStmt

How to participate?

Pick an issue you are interested in, click the issue link and leave a comment under the issue to sign up.  One person can only choose one issue and the first person who picks the issue will be assigned.

After the issue is assigned, you should complete the task in 14 days. If you are unable to complete it within the specified time, the task will be withdrawn and assigned to others.

Participants are required to produce two pieces of content (forms including but not limited to mind maps, study notes, blogs, and tutorials) in the course of the activity, each of which should be at least 200 words.

Got questions? Join our  Slack channel  and find channel:  #sql-planner.

Before you start

Read  Contributing.md  carefully to understand how to submit a pull request and how to configure BE and FE development environment.

Sign the  Contributor License Agreement  before you make a pull request.

To learn more about StarRocks, please refer to  StarRocks documentation.

Pull request requirements

Create a pull request (PR) after you complete the task.

PR title format: [SR SQL Planner Summer Camp] + PR title

After submitting PR, you can comment under the pinned Issue [SR SQL Planner Summer Camp].

Comment format: "I have finished Issue #" + PR link

please assign this to me


[identity] Implement CAE

Packages impacted by this PR

@azure/identity

Issues associated with this PR

https://github.com/Azure/azure-sdk-for-net/issues/31473

Describe the problem that is addressed by this PR
Adds CAE to the Azure Identity SDK
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

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


how to fill in the text_prompt for some uncommon categories?

Thanks for your great works!
I got perfect results.
Can you provide a dictionary containing the supported categories to show me which text I can fill in？
Looking forward to your answer.
Thanks.

There isn't such a thing as a dictionary of supported categories. GroundingDino is a zero-shot model. Might be helpful to check their paper/repo https://github.com/IDEA-Research/GroundingDINO

Hi:
What is the difference between lang-segment-anything and GroundingDINO+SAM?


Getting a module 'copilot-cmp' not found with packer

Here's the config that I used
{
  "zbirenbaum/copilot.lua",
  event = { "VimEnter" },
  config = function()
    vim.defer_fn(function()
      print("copilot enter")
      require("copilot").setup()
    end, 100)
  end,
},
{
  "zbirenbaum/copilot-cmp",
  after = { "copilot.lua" },
  config = function()
    print("copilot cmp enter")
    require("copilot-cmp").config()
  end,
},

Here's the error that I got
packer.nvim: Error running config for copilot-cmp: [string "..."]:0: module 'copilot-cmp' not found:
^Ino field package.preload['copilot-cmp']
^Ino file './copilot-cmp.lua'
^Ino file '/usr/share/luajit-2.1.0-beta3/copilot-cmp.lua'
^Ino file '/usr/local/share/lua/5.1/copilot-cmp.lua'
^Ino file '/usr/local/share/lua/5.1/copilot-cmp/init.lua'
^Ino file '/usr/share/lua/5.1/copilot-cmp.lua'
^Ino file '/usr/share/lua/5.1/copilot-cmp/init.lua'
^Ino file './copilot-cmp.so'
^Ino file '/usr/local/lib/lua/5.1/copilot-cmp.so'
^Ino file '/usr/lib64/lua/5.1/copilot-cmp.so'
^Ino file '/usr/local/lib/lua/5.1/loadall.so'

Use require("copilot-cmp").setup() instead of require("copilot-cmp").config() and include either after = { "copilot.lua", "nvim-cmp "}, or module = "copilot_cmp" so it isn't called by nvim-cmp before it's loaded.
Realistically speaking, you can start it before copilot.lua, since all calling setup does is make an InsertEnter autocmd for the real setup, and there are checks in place to make sure the copilot server is on. I should probably do some testing  to ensure no possible errors and change the docs.

Use require("copilot-cmp").setup() instead of require("copilot-cmp").config() and include either after = { "copilot.lua", "nvim-cmp "}, or module = "copilot_cmp" so it isn't called by nvim-cmp before it's loaded.
Realistically speaking, you can start it before copilot.lua, since all calling setup does is make an InsertEnter autocmd for the real setup, and there are checks in place to make sure the copilot server is on. You might just not get completions until you go back to normal mode and then enter insert mode again if copilot server starts after the fact.

Replaced config with setup but the error is still there. I'm not sure if this is an error with packer or this plugin.

I changed the require to "copilot_cmp" instead of "copilot-cmp", now it works. The example codes in this repository uses "copilot-cmp" so I don't know why this worked.

I changed the require to "copilot_cmp" instead of "copilot-cmp", now it works. The example codes in this repository uses "copilot-cmp" so I don't know why this worked.

This helps! Thanks!

So sorry about that, I've updated the docs


Alpha Transparency for specific colors - seamless Desktop effect.

Is your feature request related to a problem? Please describe.
Not related to a problem, but I would like to recreate the Coherence mode vmware has had and something that works better than the RemoteApp rdp feature from microsoft as well as it is very limited in multiple ways.
Describe the solution you'd like
I'd like to take a specific color - let's say magenta pink, hexcode #FF00FF, and convert that to being 100% transparent while using xfreerdp or more specifically the windows version wfreerdp client. Even if the user cannot click below the transparent area, which would be nice too, it'd still be a major plus being able to see the rest of my host OS while using specific applications or the taskbar area of Windows.
Keying transparency to a specific color or threshold of colors would be most ideal as the user can then set their wallpaper to that color.
Describe alternatives you've considered
Creating an agent that runs and tracks the position and size of each window and convey that info the client. This could be overly complicated though where as a simple keying of color would be a better and easier implementation.
Additional context
Checkout vmware or parallels coherence modes from the past if what I am wanting is unclear. It seems they are largely deprecated now though and I imagine due to users not using the features heavily from the battery drain and additional heat heavy Windows vms cause. I realize they did not use a color and filter it out, but rather tracked each window separately I imagine and pipe that through, just seems like a lot of work and largely unneeded when a simpler approach can be used and something that is not easily broken from changing APIs or ABIs.

will not be implemented for xfreerdp but keeping this open if someone is interested to implement this.