Light/dark detection?

https://lib.rs/crates/terminal-light

Focus has shifted to anstream.  This doesn't fit within anstream and a crate exists for this space.


camera.flyTo goes underground

camera.flyTo ends up underground in a location where the globe does not have a height.
The expected result is that that camera flys to 100 meters above the ground at the given location and the vehicle is visible on the surface of the ground.
Sandcastle example
Browser: Chrome 103.0.5060.114
Operating System: MacOS

Regarding:
let terrainHeight = viewer.scene.globe.getHeight(cartographic);
console.log(`terrainHeight=${terrainHeight}`);

It's correct behavior that terrainHeight is undefined here because globe.getHeight only works with terrain tiles that are actually loaded and only suitable for querying data as-is on-screen.  You need to use the async sampleTerrainMostDetailed to get the true height of terrain at a location, which will definitely return a height.
I still think there is a bug here because flyTo should not end up underground, but you are telling the camera to fly underground so we should clarify exactly what the official behavior is supposed to be. I expect the camera terrain collission to handle this though and pop you up above grown at that location.
If the goal is to fly to the entity, you should be using viewer.flyTo(entity) but the biggest issue here is that viewer.flyTo(entity) doesn't appear to work either when heightReference is CLAMP_TO_GROUND.
The below tweaked code demonstrates what is going on here. Click FlyToModel and it will be incorrect. Click it a second time, and it will be differently incorrect. Click it a third time and it is correct.  There is clearly a race condition going on involved with clamping to ground.
const viewer = new Cesium.Viewer("cesiumContainer", {
  terrainProvider: Cesium.createWorldTerrain(),
});

const longitudeDegrees = 149.515332;
const latitudeDegrees = -34.984799;

const position = Cesium.Cartesian3.fromDegrees(
  longitudeDegrees,
  latitudeDegrees,
  0
);

const entity = viewer.entities.add({
  position: position,
  model: {
    uri: "../SampleData/models/GroundVehicle/GroundVehicle.glb",
    heightReference: Cesium.HeightReference.CLAMP_TO_GROUND,
  },
});

Sandcastle.addToolbarButton('FlyTo Model', ()=> viewer.flyTo(entity));

I'm occasionally thinking "Well, athough I'm not familar with CesiumJS, I'll have a look at this bug - it cannot be so hard..." (and I'm  regularly proven wrong...). But even at the risk of distracting others from the actual cause here, two observations:
When enabling
Cesium.ExperimentalFeatures.enableModelExperimental = true;
in the last sandcastle, then clicking the "FlyTo Model" button does reliably zoom to the model:

But... the model itself is underground.
I started diving into the rabbit hole of comparing Model and ModelExperimental, and noticed that ModelVisualizer.prototype.getBoundingSphere computes significantly different bounding spheres...

Even though the model has the same modelMatrix in both cases. I haven't been able to figure out the reason for that one, but maybe someone can narrow down the search based on that...

@javagl are you on a commit pre-https://github.com/CesiumGS/cesium/pull/10395? @j9liu might have more background here.

I'll look into this later today. It's strange that ModelExperimental doesn't seem to be accounting for heightReference at all; the model stays underground the entire time.

I labeled this next release, since ModelExperimental seems to add an additional regression on top of the existing problem.

I confirmed that heightReference works when directly passed into ModelExperimental via #10448 (local Sandcastle for proof). It works for the "Clamp to Terrain" sandcastle but changing viewer.trackedEntity = e to viewer.flyTo(e) reproduces the buggy behavior. I'll have to see if there's something in the entity API that ModelExperimental isn't interfacing with correctly

I labeled this next release, since ModelExperimental seems to add an additional regression on top of the existing problem.

@lilleyse Does it make sense to address both the model experimental "regression" and the underlying problem at the same time, or are these two distinct problems?

@j9liu is going to open a PR with the fix mentioned above so that ModelExperimental behaves like Model, but there's still a separate underlying issue.

I opened #10571 -- the camera is still bugged, but at least it behaves the same with ModelExperimental and Model now.

Here's some of my notes on this issue so far.

How does HeightReference work with ModelGraphics?

When the Entity is created, the ModelGraphics object can be assigned a HeightReference.
Per-tick, the  DataSourceDisplay calls the update function of each Visualizer, including ModelVisualizer. This is where the Model/ModelExperimental object is initialized and all relevant properties are propagated, including heightReference. The heightRefence is not passed into the constructor, instead, the public setter for the value is used.
Per-tick flow for ModelExperimental.update:
updateClamping
Check if _heightDirty, which can be set by any of the following:
heightReference public setter is used
scene.terrainProviderChanged event is raised
getUpdateHeightCallback is called

The cartographic position of the model is computed
The getUpdateHeightCallback is registered with the QuadtreePrimitive.updateHeight function to be called (from the endFrame function of QuadtreePrimitive) whenever a new tile is rendered that contains the cartographic position of the model. The value returned from updateHeight is a function to be called to remove this callback from being called whenever updateHeight is called.
A new Cartesian3 position is computed from the the current height for the cartographic position of the model, which is obtained from globe.getHeight.
The getUpdateHeightCallback is called.
The model._clampedModelMatrix is updated with the new position.
The model._heightDirty flag is set.

The updateBoundingSphereAndScale updates the bounding sphere from model._clampedModelMatrix.
The updateReferenceMatrices updates all relevant matrices from model._clampedModelMatrix.

How does flyTo work with an Entity with ModelGraphics with HeightReference?

The zoomToOrFly function cancels any existing zoom and assigns a new Viewer._zoomTarget.
Per-tick flow for the Viewer._postRender function:
If the _zoomTarget is an array of entities, loop through all the entities.
Check the DataSourceDisplay.getBoundingSphere for the entity and see if the state is BoundingSphereState.DONE. If it is BoundingSphereState.PENDING, the function returns.

Compute the bouding sphere from the combination of bounding sphere of all entities.
Clear the current zoom in flight.
Call camera.flyToBoundingSphere.

Why does flyTo go under the ground when called for an Entity with ModelGraphics with HeightReference?
If the camera is far away, the bounding sphere computed for Model/ModelExperimental will be based on the lower LOD terrain tile. This initial position computed will be enough to complete the updateZoomTarget function called in Viewer._postRender. Essentially, there's no correction for the recomputation of the bounding sphere based on the higher LOD tiles loading during the flight of the camera to the initial position.
Potential Fix 1: Store the current target bounding spheres that are being flown to and in the per-tick update of Viewer, check if they have been updated. If so, update the bounding sphere the camera is flying to. The issue here is that there needs to be some sort of termination condition i.e. at some distance, we're close enough stop receiving height update and finish the promise. I tried implementing this one and it seems kinda clunky. Having to cancel and begin a new camera flight doesn't look very nice.
Potential Fix 2: Compute an estimate of the bounding sphere in the model that uses sampleHeightMostDetailed and use this as the initial bounding sphere for the flyTo function. This is the one I am currently pursuing. The code changes here are so far limited to ModelVisualizer.


added support for trigger webhook

Webhook triggers support
This PR handles webhook triggers. This is implemented in the HassClient and will be supported in NetDaemon when this is merged.
Usage
Make a trigger template webhook sensor in HA
template:
  - trigger:
      - platform: webhook
        webhook_id: my-super-secret-id
    sensor:
      - name: "NetDaemon webhook temp sensor"
        state: "{{ trigger.json.temperature }}"
        unit_of_measurement: °C

Use following code to trigger it:
public record TriggerData
{
   [JsonPropertyName("temperature")]
   public double Temperature { get; set; }
}

var data = new TriggerData {Temperature=4.3};
// trigger using record
await client.TriggerWebhook("my-super-secret-id", data);
// trigger using anon class
await client.TriggerWebhook("my-super-secret-id", new {temperature = 4.1});

Coverage increased (+1.2%) to 77.397% when pulling b500979d4dc60e38e62154f8f131a0961a61064c on trigger-webhool into 70d159c16222a831b16d5a72d8095e00c032c242 on dev.


How do I specific multiple users, ssh keys when initiating DC/OS cluster

Hello,
When creating the cluster I would like to specify multiple administrator user name and their ssh public keys. How do I do that in the template?
I see the following in linux profile. Though "ssh" is a list, I am afraid the user name is azureuser and I don't see a way to map one username to one specific sshkey.
"linuxProfile": {
"adminUsername": "azureuser",
"ssh": {
"publicKeys": [
{
"keyData": ""
}
]
}
}
Thanks,
Sasidhar.

Only one username is allowed.  This is passed directly to the underlying virtual machine definition: https://docs.microsoft.com/en-us/rest/api/compute/virtualmachines/virtualmachines-create-or-update.


[Usable - PowerToys Run]:  Keyboard user is not able to drag the 'PowerToys Run'  Query Launcher.

Microsoft PowerToys version
0.41.3
Running as admin

[X] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
Test Environment:
OS: Windows 10 Version Dev
App version: 0.41.3
App name: PowerToys
Repro Steps:

Launch 'PowerToys' application.
Navigate to 'PowerToy Run' tab.
Then Open 'PowerToy Run' using defined Shortcut key.
Search Edit field appear and Observe the issue.

User Impact -> Priority 3
If the content is not operated through a keyboard or keyboard interface, it would not be operable by people with no vision, Some people with hand tremors find using a mouse very difficult , as well as by people who must use alternate keyboards or input devices that act as keyboard emulators. Individuals with low vision would also find it difficult to control it from the keyboard.
Observation:
For native Win32 application we can use window+arrow key to drag the application.
ScreenCapture_7-1-2021 4.55.35 PM.zip
✔ Expected Behavior
Keyboard user should be able to drag the 'PowerToys Run'  Query Launcher via keyboard.
❌ Actual Behavior
Keyboard user is not able to drag the 'PowerToys Run'  Query Launcher.
Other Software
No response

@v-amanpra ,
Upon further discussion, this is not aligned with the design goals of the project.
It doesn't make much sense to drag it.
Dragging it is not something that is done even with the mouse. It acts as a general search bar.
Please re-open it if you think otherwise.
cc: @crutkas @dedavis6797


Consider adding departure time

If the CSV input has something like a departure_seconds column, the jittered output could have this too. For each output row, the departure time would be jittered somehow -- maybe a uniform or normal distribution centered around the input time? We would need extra config and flags (with default values) to specify all of this.
@lucasccdias, is my understanding correct? How specifically would you want to jitter departure_seconds?
I'm hesitant to add this feature, because I'm not convinced that it will be easy for the user to learn and specify a bunch of extra command-line flags to say how they want to transform time. Instead, why couldn't they add departure time on their end? In other words:

Write the desire line CSV file
Call odjitter on it
Read the output GeoJSON file and add a departure time property, using whatever logic they want

One question is how departure time is determined. Does any of the input desire line data have something like this? How is it specified -- maybe just the hour range that a bunch of trips go from zone1 to zone2? If so, maybe what we should instead do is make it easy to match up the jittered GeoJSON output with the original input, and have some kind of lookup key, or just copy over the departure time property, and do the jittering on that elsewhere.
And backing up a little more, I think the motivation for this feature request was to generate A/B Street scenarios, either with abstr or not. If so, it could be helpful to understand how we want to do that. Part of odjitter input is weighted subpoints, and there's more in-progress code within A/B Street to generate these weights for the exact purpose of creating scenarios. If our ultimate aim is to create scenarios from raw desire line data, we have a spectrum of options how to do it -- some of them using odjitter, some of them directly calling this other pipeline.

Yes, it would be great to create 'scenario.json' files, worth splitting out as a separate issue at some point.

One for Tuesday afternoon?


AttributeError: module 'tensorflow' has no attribute 'variable_scope'

Hi,
I was following your article on Deep OCR when this error happened.
My  tensorflow version information:
Version: 1.4.0
My Python Version:
Version : Python 3.5.4
Operating system : Windows7

Hi,
What happens when you run these two lines as a script?
import tensorflow as tf
print(tf.variable_scope)


PyThaiNLP 3.0.7

PyThaiNLP v3.0.7 is This release is a bug fix release of PyThaiNLP 3.0.5.
Bug Fixed

Fixed nercut bug. #666 Thank you @kmining for your bug report.

You can install by pip install pythainlp or upgrade by pip install -U pythainlp.
Documentation: https://pythainlp.github.io/docs/3.0/index.html
Report bug: https://github.com/PyThaiNLP/pythainlp/issues
See PyThaiNLP 3.0 change log#545
Contributors

Thanks all the contributors. (Image made with contributors-img)

Coverage remained the same at 97.148% when pulling bcc596e77ee4a996bd1e57a5b16089c7f2fa03d9 on pythainlp-3.0 into 884daa834a4d7f88eded27ea77276f0c44030df6 on dev.


Link goes to wron place

The subscription key. To create or find your key and region, see Find keys and region.
find-keys-and-region on the target page does not exist.,

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

ID: e983d6c0-cd70-0de9-e740-7b0b79031d1f
Version Independent ID: 36326d61-f63d-8be5-8aca-2073e51c4cc1
Content: SpeechConfig.FromSubscription(String, String) Method (Microsoft.CognitiveServices.Speech) - Azure for .NET Developers
Content Source: xml/Microsoft.CognitiveServices.Speech/SpeechConfig.xml
Service: cognitive-services
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@FutureAIGuru We are looking into it and get back to you for any additional information.


Remove references  to the boost libraries, as they're not needed

In this case, the fact that it still builds is good enough.

do you not need this tested anymore? I was converting some saves. would you like me to go through with it or find to just drop it


feat(atomic): i18next proof of concept

https://coveord.atlassian.net/browse/KIT-243
Here's how localization could be implemented with i18next. This library seems to fulfill all of our needs so far.
Related documentation:
https://www.i18next.com/overview/introduction
https://github.com/i18next/i18next-http-backend
https://github.com/i18next/i18next-browser-languageDetector

I'll close this PR, thanks for the feedback, really made me think a bit more on the issue.
Next steps for Localization and other related tasks

Put engine & i18n in a shared “context” property present in every component instead of separate properties
Rework the initialization process of the atomic-search-interface in order to have a single mechanism that setups the engine, localization and all the sub-components in an elegant manner
Remove the LanguageDetector, keeping it simple, with a “lang” prop on the search-interface, that can be watched and update the language when necessary
Have a way to correctly rerender the components when the language changes
Add a loading state on the atomic components (placeholder)
Create an atomic-component that displays localized strings

I'll close this PR, thanks for the feedback, really made me think a bit more on the issue.
Next steps for Localization and other related tasks

Put engine & i18n in a shared “context” property present in every component instead of separate properties
Rework the initialization process of the atomic-search-interface in order to have a single mechanism that setups the engine, localization and all the sub-components in an elegant manner
Remove the LanguageDetector, keeping it simple, with a “lang” prop on the search-interface, that can be watched and update the language when necessary
Have a way to correctly rerender the components when the language changes
Add a loading state on the atomic components (placeholder)
Create an atomic-component that displays localized strings


broken link

The broken link was reported by a community member

I can't find link 'https://docs.devprotocol.xyz/protocol/tokenomics'
I will link to 'https://docs.devprotocol.xyz/introduction/'

The topic is here: https://docs.devprotocol.xyz/introduction/devtoken/

Thanks. Ill do


pull request for e2e test at 12/14/2016 10:15:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


MB-12535 - Functional update/dependabot/npm and yarn/react idle timer 5.4.0

Jira ticket for this change
Summary
The react idle timer api was completely rewritten from scratch and now includes a react hook that can be used instead of the interface used in the previous version.
This PR should update our LogoutOnActivity.jsx to a functional component to make use of the react-idle-timer useIdleTimer hook.
Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make storybook

Terminal 2
Start the UI locally.
make client_run

Terminal 3
Start the Go server locally.
make server_run

Additional steps

Access the
Login as a

Verification Steps for Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Screenshots

Messages

:book:
:link: MB-12535

  Generated by :no_entry_sign: dangerJS against 6d5dd435b111568a0092b28bebc9cdbcec02b844


Typing speed test added

issue #78

@ananyakaligal when you are making a pull request, also write down  the changes with the screenshot or preview

hello, i'm attaching a screenshot of the web app below.
Could you please label it as level-3 , This typing game is really good and ready for the next level because it looks cool and finished with the little favicon as well.


Convolution and dense layers default initializer

In Keras documentation, glorot_uniform says that the initializer is using Glorot Uniform from this paper. However, the Keras implementation is totally different from the equation on the paper. Also, there are some arguments such as mode ='fan_avg' is the default. It should be same as the referenced paper. 'fan_sum'. Golort uniform is shown in Equation 1, but Keras implementation is shown in Equation 2. I had hard time to produce the same or close results to Keras using Pytorch.

def __call__(self, shape, dtype=None):         fan_in, fan_out = _compute_fans(shape)         scale = self.scale         if self.mode == 'fan_in':             scale /= max(1., fan_in)         elif self.mode == 'fan_out':             scale /= max(1., fan_out)         else:             scale /= max(1., float(fan_in + fan_out) / 2)         if self.distribution == 'normal':             # 0.879... = scipy.stats.truncnorm.std(a=-2, b=2, loc=0., scale=1.)             stddev = np.sqrt(scale) / .87962566103423978             x = K.truncated_normal(shape, 0., stddev,                                    dtype=dtype, seed=self.seed)         else:             limit = np.sqrt(3. * scale)             x = K.random_uniform(shape, -limit, limit,                                  dtype=dtype, seed=self.seed)         if self.seed is not None:             self.seed += 1         return x

Suppose you have a sentence x which is a string; e.g., x = "erath is falt".
Write the Python code for taking a sentence x and producing a dict representing the following two feature templates please help to solve

@saeedalahmari3
Moving this issue to closed status as there has been no activity, in case you still face the error please create a new issue.


Plex space-grey theme update accent colors

Hello Gilbn
First of all the work here is awesome! I do however have one problem with the plex theme and that is the accent colors on new items are not that clearly visible with this theme. I suggest change it to red so it matches the radarr and sonarr configs? This is the problem I am thinking about:

Would be much easier to see if it was red instead!

Yeah, changed to color now. Clear cache

Thanks looks awesome! However it does not work in libraries only the home tab:

Also I think these could do with a more alert color:

I suggest green for these as well :)

tried brightening up the variables

Looks great now only one more small thing :)

Should be good now.


[feature request] persistent connection

Hi!
What about adding persistent connection to memcached? I tried following wrapper, which disables "close", and everything is working on my stand. But I don't believe that it could be so simple.
Could you help me understand, where is the problem with such idea?
P.S. My test suit could be found here: https://github.com/max-posedon/djmemd

from django_pylibmc.memcached import PyLibMCCache as DjangoPyLibMCCache

class DjangoPyLibMCCache2(DjangoPyLibMCCache):
    def close(self, **kwargs):
        pass

The upstream Django fix for this seems to have stalled again:
https://github.com/django/django/pull/4866
https://code.djangoproject.com/ticket/11331
At the moment I'm using the workaround mentioned here:
https://devcenter.heroku.com/articles/memcachier#django
However as the OP above suggests, we could just work around this in django-pylibmc similar to how django-bmemcached did:
https://github.com/jaysonsantos/django-bmemcached/pull/8
Would you accept such a PR? :-)

I fixed this upstream in Django 1.11:
django/django/pull/7200
When I get a chance I'll add a fix to django-pylibmc for Django 1.18/1.10.


I'm puzzled: this looks exactly what I need for Xamarin iOS storyboard editing

Hi
My Xamarin Forms project's iOS part has a launch screen in a storyboard, and since verson 16.9 I can no longer edit it.
It says 'use xcode'. I have no idea how to perform that operation in a proper manner.
Then I found this page. It looks exactly what I need (and everyone else using c# Xamarin Forms).
I am very keen that this could be made available to coders using c# not c++.
Please advise.
Thanks
Pete Midgley.

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

ID: 3c83a717-6517-b6fc-a3fe-b0859d16c8f5
Version Independent ID: 64096bcf-4469-fdb4-5e82-8dbac44a76f0
Content: Sync changes between Xcode and Visual Studio
Content Source: docs/cross-platform/sync-changes-between-xcode-and-visual-studio.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @corob-msft
Microsoft Alias: corob

@petemidg32
I'm sorry to hear you're having trouble. The Visual Studio UI for syncing a C++ iOS project sends commands to a small Node.js server program installed on your Mac called vcremote. This server applet has some specific knowledge of C++ projects and directories, so it can find and copy source and header files, and transmit debug information. I doubt that it can be used as-is to sync Xamarin app project files. My guess is that Xamarin has something similar, but since I don't know, I'm invoking my colleague @davidbritch to see if he knows of a solution.

Thanks for that Colin; I’ve seen and used some of David Britch’s stuff so have my own experience of his expertise!
Regards
Pete Midgley
From: Colin Robertson @.>
Sent: 03 May 2021 19:21
To: MicrosoftDocs/cpp-docs @.>
Cc: Peter Midgley @.>; Mention @.>
Subject: Re: [MicrosoftDocs/cpp-docs] I'm puzzled: this looks exactly what I need for Xamarin iOS storyboard editing (#3118)
@petemidg32https://github.com/petemidg32
I'm sorry to hear you're having trouble. The Visual Studio UI for syncing a C++ iOS project sends commands to a small Node.js server program installed on your Mac called vcremotehttps://www.npmjs.com/package/vcremote. This server applet has some specific knowledge of C++ projects and directories, so it can find and copy source and header files, and transmit debug information. I doubt that it can be used as-is to sync Xamarin app project files. My guess is that Xamarin has something similar, but since I don't know, I'm invoking my colleague @davidbritchhttps://github.com/davidbritch to see if he knows of a solution.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/cpp-docs/issues/3118#issuecomment-831441629, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGCS2YBNNJVXJDVE7O34TQ3TL3SRRANCNFSM436QKQ4Q.

@petemidg32
The iOS designer was removed in 16.9 and so your only option is now to use the storyboard designer in Xcode, on a Mac, to update the storyboard.

@petemidg32
Since this isn't really an issue for the C++ documentation, I'm closing it. For product issues and ideas, I recommend using the feedback tools built into the IDE. Your feedback goes to the development team that can make things happen. Best of luck!


Bug report: m365 planner plan list --ownerGroupName migth be misleading if there are multiple groups with the same displayname

Description
m365 planner plan list --ownedGroupName allows you to return the planner plans of a group with the displayName specified under --ownedGroupName.
If you have multiple groups with the same displayName, the command will take the first group it finds and return the plans or not. This might be misleading compared to other commands where we return an error message if multiple matches have been found.
Steps to reproduce

create a couple of teams with the same displayname "yourDisplayName"
create one planner plan for one team
execute m365 planner plan list--ownerGroupid "yourDisplayName"
the command will pick the planner plans of the first group it finds

Expected results
The CLI should return the error message that there are multiple Groups with the same displayname and that the user should choose from one of these by id.
Actual results
Takes the first group it finds and returns the results
Diagnostics
No response
CLI for Microsoft 365 version
latest CLI
nodejs version
latest node
Operating system (environment)
Windows
Shell
PowerShell
cli doctor
No response
Additional Info
We have the same issue with planner plan get, and planner plan add, planner task add, planner task list, planner task set

Great catch!

Assuming the problem will also occur in the following commands:

planner bucket add
planner bucket list
planner plan get
planner task add
planner task list
planner task set

They all use either a getPlan or getPlanId method to translate a groupName to an ID (the first in the array)

Great catch both! @appieschot, if we could confirm this, we should create separate issues for all these bugs to ensure that we'll fix them. We should also ensure that fixes contain a test case that proves that the issue has been fixed.

Hi @waldekmastykarz, @appieschot,
I can confirm that all these commands are affected (it has been stated under "additional info"). I am going to create the issues.
Cheers

Awesome! Thank you @plamber

Hi @plamber @waldekmastykarz @appieschot,
You can assign this one to me. I'm aware of this issue and planning to fix it in my next PR. (#2845)

Hi @Abderahman88,
thank you very much. Could we just change the wording? Please disambiguate sounds a little bit strange. Could we use something like "Please choose between the following IDs" instead?
I am going to create issues except for the planner set command.
Thank you,
Patrick

@plamber Great suggestion. I'll change it 👍

@Abderahman88: since all commands mentioned by @appieschot are using the same methods over and over (getPlanId, getGroupName, getPlan), could we create a base class and add these commands there and inherit from that base class? It would be awesome if we can reduce some code in all commands.
I am going to create the related issues for all other commands. Once the base class is created, we can work on the residual ones with less efford.
Let me know what you think

could we create a base class and add these commands there and inherit from that base class

Considering #2989 I'd like to suggest that we add the method in Utils for now and when we start working on #2989 we'll move it to a Planner-specific util module. Let's not create a separate base class that we'd need to remove later as that would introduce additional work.

@Abderahman88 are you still working on this? Not sure if you've seen that we've merged #2989 and we can have the util method stored centrally and reused across all relevant commands.

@waldekmastykarz I'll take a look at it 👍

If I'm not mistaken, I think this issue is covered when #3267 completes right?

Thank you. This topic is closed


FR: Collapse all branches

Thanks for this great extension.  When opening a large mind-map, it is hard to navigate because it is so huge.  It would be really helpful if there was a way to collapse all branches in one go, and even better if this was optionally done when first rendering the map.

Yes please! I have a pretty large mindmap too, and I'm trying to figure out if this has been implemented?

Absolutely essential, I reckon.  Would be nice to be able to specify the level to open to.  Also, if root level could be level 0 rather than at the top of level 1.

One other thing. It would also be nice if when you open a level, only that level opens, not all higher levels For example if you are on level two, and click on a 'dot' only level three opens, not level three to 'x'.

Curious the usability how folks would want this, because I'm interested also in this.

One idea is to try and listen to command-shift-p commands to fold / unfold similar to how you can in code, and keep the markmind map in sync with the code folding level. OR, have another set of shortcut commands for just the mindmap:

Another idea is to have buttons in the markmap control in the bottom right of the window

Is there another user experience folks would be interested in?

I'm interested in trying a PR of this, just seeking some feedback @markmap the best UX (and also pointers for implementation). :-)

This button is used to fold/unfold the next node and its children recursively. Maybe it was a bad UX 😂

Ohhh interesting, thank you so much. I don't think I figured out how to use that when I tried (macOS Chrome) in https://markmap.js.org/repl Clicking that nothing happened. I'm not sure where the state of the current node is indicated to recursively toggle from.

I implemented a keyboard level version of this in another library today which possibly we could share inspiration from / etc. https://github.com/lynchjames/obsidian-mind-map/pull/103


Vend multiple packages?

With frameworks specifics PRs like https://github.com/proposal-signals/signal-utils/pull/72 I wonder if it would be good to have generic core utilities in signal-utils and framework integrations in packages named like react-signal-utils, etc.

Ye! This would require we also set up script orchestration, like turbo, or something, so that packages build in the correct order (for tests, ci, etc)

Wireit is quite good at that without taking everything over.

legit, been wanting to use wireit on a project, but haven't had the opportunity yet!
are you familiar with setting that up?

Very. I use it on every project :)

want to set something up on this repo? I'll slap some :+1: on that PR


Minimize the Travis CI matrix

I would like to redefine the purpose of Travis CI to quick automatic checks for all contributors. In this regard, recent situations of Travis CI's long job queue is not desirable. This change reduces the number of jobs running on Travis CI by reducing the jobs following the redefined purpose.

Checks that are covered by Jenkins will only run for PRs (master/stable branches are removed).
Windows tests are removed (we will use our new CI infrastructure for them).
Only one combination of Ubuntu/py environment is used for each PR (namely Ubuntu16.04 py35).
macOS py27 test will run for each PR instead of macOS py35 (for wider py versions coverage combined with Ubuntu test).

Jenkins, test this please

Jenkins CI test (for commit 3524dca2708ef82cc4cf7176498c60926124d537, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Test failures are irrelevant.
LGTM


Fix RbLearn link

Hi there,
Just noticed that this link was incorrectly formatted so I fixed it! 😄

I want to contribute to this list and help the maintainers, so:

[x] I've accepted the terms of the CC0 License;

Thank you @mach-kernel!


Postgres error log full of "there is no parameter $1" errors

Seeing tons of errors like this in our postgres logs:
[485-1]  sql_error_code = 42P02 time_ms = "2023-09-13 16:13:02.707 UTC" pid="909748" proc_start_time="2023-09-13 15:35:34 UTC" session_id="6501d6c6.de1b4" vtid="89/57447" tid="0" log_line="1890" database="XXX" connection_source="XXX.XXX.XXX.XXX(XXXXX)" user="XXX" application_name="datadog-agent" ERROR:  there is no parameter $1 at character 233

I'm not sure if this is the same issue as #13840, which seems related but was supposedly fixed.
We're running agent version 7.47.0:
Agent 7.47.0 - Commit: b8c31bf - Serialization version: v5.0.90 - Go version: go1.20.6

and using heroku-buildpack-datadog a0beb72.

I am seeing these same errors as well when enabling database monitoring.
Agent 7.47.0 - Commit: b8c31bf - Serialization version: v5.0.90 - Go version: go1.20.6

and heroku-buildpack-datadog

Same here

7.49.0 is out and contains some fix about it. At first glance it works now.


add description attribute

Migrated from: https://github.com/puppetlabs/prosvcs-node_manager/issues/44
When exporting node group info from the API a group comes with an optional description field.  We should be able to populate this via a puppet type.

New attribute merged into master.  Should go out next release.
https://github.com/WhatsARanjit/prosvcs-node_manager/commit/914eaf9374a0aa704e633dc61a0ef7290d1858f6


Firefox Nightly triggers both scripts

I'm using Firefox Nightly 55.0a1 with the dom.moduleScripts.enabled flag set to true to give ES6 module loader support.
Opening your https://blog.hospodarets.com/es-modules-utils/demo/ test page, both the "supported" and the "unsupported" alert boxes are triggered.

Hi @brownieboy
This will be fixed as only the Firefox issue https://bugzilla.mozilla.org/show_bug.cgi?id=1330900 is resolved.
The plan is that none browsers ship ECMAScript modules to the stable version till nomodule attribute support is added.


Attempt to Parallelize Swagger Generation

I don't know why, but from CI it seems to be slower (maybe agent on devops has less core)?

I don't know why, but from CI it seems to be slower (maybe agent on devops has less core)?

I'm not sure what is with the timeouts, a previous commit I had cut generation time from 12 minutes to 7 minutes but this commit times out. I'm going to put this on my backlog until MQ as this isn't something that's required and is nice to have.


Add ability to update config within app

Take a look on this: https://github.com/sindresorhus/electron-store

Switched to use electron-store.


cli commands to delete projects and runs

[ ] Create new commands
[ ] Add unit tests for new commands
[ ] Add new commands to tutorial_cli notebook

@hardbyte Hasn't it been done in #265 which is merged in?


There is no official full form of 'NPM'

In our list of glossaries- I see below
Npm -	Node Package Manager. A tool for distributing JavaScript packages.
There is no official full form of NPM. NPM is just NPM. To describe, sometimes people say that it is the default package manager for the JavaScript runtime environment Node.js
Found on page https://www.itwinjs.org/learning/glossary - "Glossary of terms in iTwin.js"

Perhaps I am misunderstanding, but we do not state there is an "official full form" of NPM? What is your issue with the wording here?

yes, with the wording. My only point was there no meaning to NPM acronym. However, whenever we write NPM with Node Package Manager. Reader misunderstands that as a full form of NPM acronym.
If you think that it is not worth fixing. I will close / delete the issue right away.

https://en.wikipedia.org/wiki/NPM

Ahhhh I understand the acronym :) Sure, if you want to raise a PR with improved wording that is fine.

https://github.com/iTwin/itwinjs-core/pull/6304

https://github.com/iTwin/itwinjs-core/pull/6308


Dataset problem~

Hi,
I'm a little confused about dataset. You use cpvr2016 and nips2014 to train the model, right? During training, you also use validation dataset? Which validation dataset did you use?
Btw, where can I test the model and get an accuracy? test.py file?

You could look into Data Preparation part in README.  Most of the public dataset is provided, you could setting the validation dataset in train_MORAN.sh.

@gzhcv Thanks a lot!

You could look into Data Preparation part in README. Most of the public dataset is provided, you could setting the validation dataset in train_MORAN.sh.

How about test part? Where can I test the model and get an accuracy directly?

@JingLiJJ  I just running train_MORAN.sh to get the accuracy, it's not a good method. Maybe there is another way to get the accuracy (e.g. look into the val() function in main.py). And, it seems no exsit code to get the accuracy directly

@JingLiJJ @gzhcv It is an on-the-fly way for evaluation, following CRNN and ASTER.

@JingLiJJ I just running train_MORAN.sh to get the accuracy, it's not a good method. Maybe there is another way to get the accuracy (e.g. look into the val() function in main.py). And, it seems no exsit code to get the accuracy directly

Thanks~ I got it.


[Feature Request]: Support for template parameters

Feature description
Right now we are using templates to manage Custom Serving Runtimes, but we are using the objects field to store the spec, but not the parameters functionality built in.
We need to add support, both from a UX and UI standpoint for managing parameters to control certain aspects such as resource limits, GPU and more.

@vconzola Can you take a look at this and talk with @lucferbux for additional info

Based on this conversation templates might be deprecated in the future, we need to consider wether adding parameters to templates or just use any other technology.

Migrated to:  https://issues.redhat.com/browse/RHOAIENG-558


Why do you copy Bootstrap Scss into the repo?

Why are you copying the Bootstrap files into the CoreUI repository rather than including the files directly from bower_components? Using bower_components directly would surely make it easier to maintain CoreUI and update Bootstrap? You could then gitignore the bower_components directory to remove repo bloat so that for example you don't need to keep a copy of the entire Bootstrap Sass library within your own code. As far as I can see you're not taking full advantage of using Bower.

I don't use bower in Angular 2 Version,
I don't use bootstrap.scss file, all files are included in style.scss
I use custom variables
I don't use all bootstrap components, ex. jumbotron, because no one use jumbotron in admin panel.

Thanks @mrholek. so does this mean that you customise the actual Bootstrap files once you've copied them to the repository?
Also, I'm looking into incorporating CoreUI with my base framework for a CMS. What is the best way of installing CoreUI so that it can be easily updated as you release new versions? Normally I'd make use of Bower for this, but this doesn't appear to be an option with your theme.
I use Bootstrap in many projects and do not pull it into my repository like is being done here.
I use NPM to include Bootstrap SASS as a dependency and then only pull in the SCSS files that I require from Bootstrap (instead of including them all). There's no reason that can't be done here - just pull in bootstrap as a package.json dependency.
Overall, the Core UI is fairly impressive, but I'm worried that core Bootstrap SCSS files may get modified - making upgrades difficult.


pubsub test flakes

[x] https://ci.appveyor.com/project/GoogleCloudPlatform/google-cloud-java/build/609 (testModifyAckDeadline)

This seems to not be flaking anymore. Closing this for the time being.

We have new flakes :frowning_face: . Reopening.

Can you give links to the new failures we are seeing?

@garrettjonesgoogle It's the unchecked box in the issue description.

Fixed by #1898


⚠ SkipTheCommericals has degraded performance

In 52d7128, SkipTheCommericals ($STC) experienced degraded performance:

HTTP code: 200
Response time: 1727 ms

Resolved: SkipTheCommericals performance has improved in a2930c1.


Problem with initializing existing canvas with Pixi.Application

Im sorry for not following the template given for raising issue.
My problem start when i cannot load image from https URL due to cross origin. Checked with my superior on that, and that setting cannot be changed. I used Pixi Loader to load image like this
Loader.add([{
	        name: "display",
	        url: source,
	        crossOrigin: true
	    }])
	    .load((loader, resources) => generateImage(resources));

Where source is the https URL for that image that i tried to fetch. I also tried with crossOrigin : "" and crossOrigin : 'anonymous' but both didnt work.
Since loading image to canvas using normal approach works, so i decided to load the image to canvas like this
let ctx = imgDisplayer.current.getContext('2d'); // Im using React ref here.
		let img = new Image();
		img.onload = function(){
			imgDisplayer.current.width = img.width; // To set canvas width to image width
			imgDisplayer.current.height = img.height; // To set canvas height to image height
			ctx.drawImage(img, 0, 0);
			setCanvas(imgDisplayer.current); // Im using React here to save the loaded canvas into state
		}
		img.src = source; // This is the https URL that throws cross origin issue if i use it on Pixi Loader

And then try to initialize Pixi on top of that loaded canvas
const app = new Pixi.Application(1300, 866, canvas); // Using the canvas state here
Im not sure if the approach above is the correct way. I've seen alot examples with instantiating Application, like Application(300, 400) and also Application({ width: 300, height: 400 }).. Im not even sure which one is the correct way, but since the above didn't throw me error, so i assume is that one. If i use Application({ width: 300, height: 400 }), it will throw me error saying webgl is not supported and ask to use canvas renderer instead.
And then with gsap, i try to modify the filter for that canvas
gsap.to(app, { duration: 0.5, pixi: { hue: 180 } });
But nothing happen
I'm new to GSAP (using 3.2.6) and Pixi (using 5.2.4), so i'm not sure if im doing things right here. Please help me on the proper direction for this.

And then try to initialize Pixi on top of that loaded canvas

init webgl on top of 2d? That wont work. Even if it worked, that image could be destroyed on the first clear(), after all, you need it in texture for drawImage later, right?
Congratulations, you made my week by making the most cursed way of loading image into pixi :)
Yes, in some cases anonymous doesnt work anymore, I dont know why, maybe its browser-related. Here I use crossOrigin:true which is the same as crossOrigin:anonymousaccording toresource-loader` source (https://github.com/englercj/resource-loader/blob/release/3.x/src/Resource.js#L116)
https://www.pixiplayground.com/#/edit/DxodMaAr-18_fIzHMbC0i
@bigtimebuddy do you know anything about new crossorigin problem?
@webbahproject if you want to make your own way of loading stuff - two easy ways

draw  something to new canvas. use Texture.from(canvas) . use texture.update() every time you change that canvas

make your own resource. https://pixijs.io/examples/#/textures/gradient-resource.js . Here's how pixi treats images: https://github.com/pixijs/pixi.js/blob/dev/packages/core/src/textures/resources/ImageResource.ts
https://github.com/pixijs/pixi.js/blob/dev/packages/core/src/textures/resources/BaseImageResource.ts#L54

btw, now that I think about it, it might be that web-server on other side does something very wrong like that https://api.ok.ru/img/stub/user/male/50.png . We just reported this to OK tracker.
Maybe if pixi crossOrigin:true doesnt work in your case - you really should report it to supplier!

Actually, you can even try it without pixi first - make an image and try drawImage it to canvas getContext('2d'); If it does work for you - then its definitely a pixi problem.

Actually, you can even try it without pixi first - make an image and try drawImage it to canvas getContext('2d'); If it does work for you - then its definitely a pixi problem.

I can show the image from the https URL onto the canvas just fine like this
let ctx = imgDisplayer.current.getContext('2d');
		let img = new Image();
		img.onload = function(){
			imgDisplayer.current.width = img.width; // To set canvas width to image width
			imgDisplayer.current.height = img.height; // To set canvas height to image height
			ctx.drawImage(img, 0, 0);
			setCanvas(imgDisplayer.current);
		}
		img.src = source;

If i try to add crossOrigin setting like this, it doesnt work.
let ctx = imgDisplayer.current.getContext('2d');
		let img = new Image();
		img.onload = function(){
			imgDisplayer.current.width = img.width; // To set canvas width to image width
			imgDisplayer.current.height = img.height; // To set canvas height to image height
			ctx.drawImage(img, 0, 0);
			setCanvas(imgDisplayer.current);
		}
img.crossOrigin = true // Even 'anonymous' didnt work.
		img.src = source;

When i try to load the https URL into Pixi Loader, it didnt work and show black box.

Also sorry if my question on this thread seems out of this world.
I saw one of the thread in here asking how to initialize existing canvas with Pixi and thought existing canvas means those canvas with image loaded onto it, when in actual its initializing empty canvas.
The requirement i had is to build images filter  and i saw the GSAP 3 + Pixi 5 interaction examples and it was exactly what i need. Only problem is that, the images comes from a https server, It works fine when i test it with images from http server, but with https, its throwing me those cross origin error.

Also sorry if my question on this thread seems out of this world. My first 2-3 days of learning Webgl and Pixi was really like @@:;;;

Sometimes those crazy ideas actually work ;)

It works fine when i test it with images from http server, but with https, its throwing me those cross origin error.

Strange, usually its the other way - http images cant be loaded in https...

This is the piece of code that i wrote in NextJs
useEffect(() => {
		// Initialization of GSAP and Pixi
		Pixi = require('pixi.js');
		PixiPlugin = require('gsap/PixiPlugin').PixiPlugin;

		// register the plugin
		gsap.registerPlugin(PixiPlugin);

		// give the plugin a reference to the PIXI object
		PixiPlugin.registerPIXI(Pixi);

		// Create instances
		const Loader = new Pixi.Loader();

		const generateImage = resources => {
			const app = new Pixi.Application({
				width: 1300,
				height: 866,
				backgroundColor: 0x000000,
		    autoResize: true,
		    view: document.getElementById('img-displayer')
			});

			// const app = new Pixi.Application( 1300, 866, { view: canvas });
			const display = new Pixi.Sprite(resources.display.texture);
			display.width = 1300;
			display.height = 866,
			app.stage.addChild(display);
		}

    Loader.add([{
        name: "display",
        // url: "http://s3-us-west-2.amazonaws.com/s.cdpn.io/16327/car-dashboard-blur.jpg",
        url: "`https://s3-us-west-2.amazonaws.com/s.cdpn.io/16327/car-dashboard-blur.jpg`",
        crossOrigin: true
    }])
    .load((loader, resources) => generateImage(resources));

	}, []);

I have to require the plugins like that to make it work on SSR framework like NextJs. Please let me know if there's a proper approach for this.
Anyway that piece of code didn't work.. it shows as black box on the page.
It shows this error on the console
Access to image at 'https://s3-us-west-2.amazonaws.com/s.cdpn.io/16327/car-dashboard-blur.jpg' from origin '<origin url in here>' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. s3-us-west-2.amazonaws.com/s.cdpn.io/16327/car-dashboard-blur.jpg:1 Failed to load resource: net::ERR_FAILED
If i throw away that crossOrigin property from the Loader, it wouldn't work too.
If i use the commented URL instead
url: "http://s3-us-west-2.amazonaws.com/s.cdpn.io/16327/car-dashboard-blur.jpg",
Then it will work.
I can show the image just fine using canvas's drawImage method using both URLs (see the piece of code on my posts above)
All the resources to be displayed are series of images sent from https server to the page im working at, which is also hosted in a https server. Technically shouldn't be a problem since the URL is just a direct URL to show the image, but from what i read, Pixi uses ajax call to make a call to fetch that image.
There's very little sample of Pixi loading resources from https.. most of the samples i read generate the resources using relative path, where the samples were loaded from the project subfolder. Even the reference i used to start up my Pixi journey https://greensock.com/docs/v3/Plugins/PixiPlugin had to use http URL due to CORS issue.


Quality of writing

This is part of JOSS review.
This paper (not the software) is low on content relative to other ocean-related JOSS papers and requires prior experience with TMD 2.5 in order to be at all meaningful. As the markdown is full of great documentation, I recommend spending a small amount of time repurposing an example and/or a few key function highlights and including it in your paper.

We understand the need to ensure that the paper communicates what TMD3.0 does, without expecting readers to have prior experience with previous versions of TMD. We believe we've explained it well in the paper:

The Tide Model Driver for MATLAB version 3.0 (TMD3.0) allows users to access and calculate tidal predictions from model coefficients at arbitrary locations and times, and this version of the software represents decades of development by the tide community...TMD3.0 is issued as a free, open-source software package that is available to all.

Regarding the suggestion to expand the paper to include code examples in the paper, the JOSS guidelines state:

...the JOSS paper (the compiled PDF associated with this submission) should only include:

A list of the authors of the software and their affiliations.
A summary describing the high-level functionality and purpose of the software for a diverse, non-specialist audience.
A clear statement of need that illustrates the purpose of the software.
A description of how this software compares to other commonly-used packages in this research area.
Mentions (if applicable) of any ongoing research projects using the software or recent scholarly publications enabled by it.
A list of key references including a link to the software archive.

The guidelines also state:

Note the paper should not include software documentation...

And that makes sense, because syntax or best practices may evolve in the future, and the living documentation can always reflect any changes, but an example baked into the published paper would not be able to be updated.
Also, a little anecdote to describe a major reason we chose JOSS: In previous experiences publishing on software in traditional examples, we've been compelled to include examples, and that always presented a quandary. Any single example would be somewhat arbitrary and couldn't fully represent the scope of a toolbox. The temptation is to craft an elaborate example that showcases all the major functionality of the toolbox and shows how functions work together, but that would push readers directly into the deep end and would likely drown them in the unfamiliar syntax and function calls. Taking a more humane approach, we could craft a beginner's example of a simple tidal prediction, but even a simple example quickly becomes a tutorial if presented properly. For these reasons, examples in a static document are a bit doomed, which is likely why the JOSS style guide does  not allow for them.

I'll respond here about the JOSS paper contents. I agree with @chadagreene that JOSS papers are meant to be quite concise. @lnferris you're right that often they are not — JOSS editors have been purposefully pretty liberal with our interpretation of what is ok, preferring to let the authors choose, within the scope of the JOSS guidelines.
That said, @chadagreene can you change the links in lines 22 and 23 to be markdown links pointing to those addresses, or otherwise format them in some way to look like URLs?


Create the Doc bindings

Implement the bindings for the Doc struct based on the exposed bindings.

[x] Create Doc class definition
[x] ydoc_new
[x] ydoc_destroy
[x] ydoc_new_with_options
[x] ydoc_id
[x] ydoc_guid
[x] ydoc_collection_id
[x] ydoc_should_load
[x] ydoc_auto_load
[x] ydoc_observe_updates_v1
[x] ydoc_unobserve_updates_v1
[x] ydoc_observe_updates_v2
[x] ydoc_unobserve_updates_v2
[x] ydoc_observe_after_transaction
[x] ydoc_unobserve_after_transaction
[ ] ydoc_observe_subdocs
[ ] ydoc_unobserve_subdocs
[ ] ydoc_observe_clear
[ ] ydoc_unobserve_clear
[ ] ydoc_load
[ ] ydoc_clear
[ ] ydoc_read_transaction
[ ] ydoc_write_transaction

For each bound method, the implementation and unit tests must be written.

The following items have been skipped for the respective reasons:

ydoc_clone: cloning references in C# is straightforward

Hi, @Horusiath!
I'd like you to take a look at the work I'm doing to create the .NET bindings for Yrs.
The "core" of the work has been happening mostly on the Doc and DocChannel classes. They're the public and the native binding classes, respectively.
Everything else around them are tests or helper definitions to interact with the FFI exposed by YFFI. I've been moving slow because I can only work on this in my spare time (and this is not my only side project), but there's been steady progress here.
What do you think?

Also, I'd like to ask you about the Guid() method exposed by YFFI as y_doc_guid.
I noticed that, sometimes, the UUID v4 generated contains some internal values like 11 (decimal) that are translated to 0B. Then, when they're formatted to string, it becomes B (or b) only and the final string doesn't match the GUID format that C# expects. Is that fixable somehow? I'd not like to touch (or request changes to) the Yrs code, then if you have a less invasive suggestion, that'd be nice.

I noticed that, sometimes, the UUID v4 generated contains some internal values like 11 (decimal) that are translated to 0B. Then, when they're formatted to string, it becomes B (or b) instead of 0B and the final string doesn't match the GUID format that C# expects (it's like characters are missing). Is that fixable somehow? I'd not like to touch (or request changes to) the Yrs code, then if you have a less invasive suggestion, that'd be nice.

I think it should be doable. However keep in mind that the source of that uuid is originally in JavaScript, so it's possible that someone is using arbitrary strings in there - which is also a reason why I went with string representation in Rust.


iife bundle missing from v13

Looks like the latest published npm artifact is missing the iife build. In other words, should.js is missing from the root of the project. That means I can't use should.js in the browser without either building it manually or using v11.2 (or transpiling the es6 version). I didn't see anything that suggested this omission was intentional. Thanks!

ohh, sorry for this, will fix this in few minutes.

This should be fixed now 13.0.1


What is the Webhook JSON Schema?

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
Just testing out the webhook functionality... I can think of some good use cases for it. I've made an extremely simple Flask app to get an idea of the JSON schema:
@app.route('/v1/status', methods=['POST', 'GET'])
def status_v1():
    request_data = request.get_json()
    p = Serial(devfile='/dev/ttyUSB0', baudrate=9600, bytesize=8, parity='N', stopbits=1, timeout=1.00, dsrdtr=True)
    print(request_data)
    return make_response(jsonify({'status': 'success'}), 200)

It literally takes the POST request and prints it out in terminal. This is the output I get:
{'heartbeat': {'monitorID': 10, 'status': 0, 'time': '2022-01-21 14:06:38', 'msg': 'PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.\n\n--- <IP_ADDRESS> ping statistics ---\n10 packets transmitted, 0 received, 100% packet loss, time 400ms\n\n', 'important': True, 'duration': 0}, 'monitor': {'id': 10, 'name': 'Testing', 'url': 'https://', 'method': 'GET', 'body': None, 'headers': None, 'basic_auth_user': None, 'basic_auth_pass': None, 'hostname': '<IP_ADDRESS>', 'port': None, 'maxretries': 0, 'weight': 2000, 'active': 1, 'type': 'ping', 'interval': 60, 'retryInterval': 60, 'keyword': None, 'ignoreTls': False, 'upsideDown': False, 'maxredirects': 10, 'accepted_statuscodes': ['200-299'], 'dns_resolve_type': 'A', 'dns_resolve_server': '<IP_ADDRESS>', 'dns_last_result': None, 'pushToken': None, 'notificationIDList': {'2': True}, 'tags': []}, 'msg': '[Testing] [🔴 Down] PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.\n\n--- <IP_ADDRESS> ping statistics ---\n10 packets transmitted, 0 received, 100% packet loss, time 400ms\n\n'}

Pasted the output into Notepad++ and a couple of online tools, yet they all say the JSON is invalid. Could you please provide the JSON schema?
Note, the versions below are where Uptime Kuma is running, NOT where the Flask app is running.
🐻 Uptime-Kuma Version
1.11.3
💻 Operating System and Arch
Raspberry OS Lite armv7
🌐 Browser
Google Chrome 97.0.4692.99
🐋 Docker Version
Docker version 20.10.12, build e91ed57
🟩 NodeJS Version
No response

I also have this question. Is there a doc somewhere with the JSON schema for webhook notifications?

Not currently, but you can dump the content via https://webhook.sites or similar.
Please make sure to test all 3 different notifications types (UP/Down, Cert-Expiry, Testing) to ensure the webhooks does what you expect. Not all events have all context (for example testing does not have a monitor)


Add segementation visually

Already done here https://github.com/Tesseract-MI/prostatecancer.ai/pull/5
Closing this as Done.


[BUG] tweetLimit in community/newsroom isn't been reflected

Describe the bug.
Currently in community/newsroom page here, we are using TwitterTimelineEmbed from react-twitter-embed package to show the twitter timeline, though we have mentioned tweetLimit:2 in our code, it isn't being reflect upon.
The code block

      <TwitterTimelineEmbed sourceType='profile' screenName='AsyncAPISpec' options={{ tweetLimit: '2' }} />

Due to this many tweets are being loaded all together.
Expected behavior
Only limited amount of latest tweets should be shown, but when I looked within the react-twitter-embed package, this surely doesn't seem to be possible (as there are many issues already related to that like this ), but what we can do is to make the component scrollable by setting a specific height in the options attribute of the component, as it was mentioned in the package's source code:

Expected behaviour at the end should look something like this:
https://github.com/user-attachments/assets/e15c32e8-fc24-4b98-9ba5-8f5b22650a30
I'll make sure this aligns well with the current existing UI above just shows a probable fix to this.
Screenshots
Current Behaviour:
https://github.com/user-attachments/assets/bfef83f6-4127-4e5d-8a43-35f3826fafc6
How to Reproduce
visit this page and head over to Latest News section
🥦 Browser
Google Chrome
👀 Have you checked for similar open issues?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to work on this issue ?
Yes I am willing to submit a PR!

I am seeing this
@Saumya40-codes would you like to work on this issue

/gfi

I am seeing this @Saumya40-codes would you like to work on this issue

Yes i'll like to fix this!

Additionally should we also consider adding scrollbar to contain the amount of tweets being show?  (as I mentioned in the main issue)

i am using Firefox Developer Edition and i am not able to see the twitter section, it is working as expected in google chrome and microsoft edge

I had the same experience with the X timeline. After some research, I discovered that Twitter timelines can be generated using https://publish.twitter.com/ . However, it seems that this feature is not enabled for every account. Specifically, accounts with a blue tick have their timelines displayed correctly, while most accounts without a blue tick are not showing their tweets, there are some non-blue tick accounts that do display their timelines correctly. This discrepancy might be the root cause of the issue we're experiencing.

tbh, I don't see a problem if the last resort is to basically remove this twitter widget.  But that is just my opinion and I'm interested what others think

@asyncapi/bounty_team


Azure storage v2 Cool tier with Azure Files

Hi, I created a cool storage (v2) and set up file shares within the storage account.  Is cool storage for Azure Files supported now?

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

ID: 534b3ef3-5664-973e-b146-8c1f960bf536
Version Independent ID: 57c3ce18-2857-8ed0-10a5-5061d391a079
Content: Frequently asked questions (FAQ) for Azure Files
Content Source: articles/storage/files/storage-files-faq.md
Service: storage
GitHub Login: @RenaShahMSFT
Microsoft Alias: renash

@vicperdana Thanks for the feedback. We are actively investigating and will get back to you soon.

@vicperdana Currently, Azure Files supports only the standard storage tier. We don't have timelines to share for premium storage and cool storage support at this time.
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

Can you confirm if Azure files supports Storage General Purpose v2 with Hot access tier?
Or is Azure files supported under Azure Storage General Purpose v1 only?
Thanks


Configuration file merge

Merge and simplify configuration files.

I have addressed some concerns:

The configuration file and working directory can be specified on the command line too.
The software does best effort to determine working directory and configuration file without HOME.
Separate database sections for MySQL and SQLite in configuration file.

ACK


Add a label to run.run

With this feature we can label any remote calls we perform.  If the command fails during execution it'll print out the label so it's more apparent when looking at log output what the initial purpose of the command being ran was.  This can help us to determine if a job failure is environmental, a teuthology failure or a legit ceph test failure.  Of course, it depends on the writer of the test to properly use the new label kwarg.  See, http://tracker.ceph.com/issues/10699
Also, I've started a simple testing task I used to make sure this feature was working correctly.  I've got another branch that uses pytest to autodiscover tests in this new task but I thought that'd be best in another PR.
Here's sample output from a test run of this branch.
2015-02-03 10:04:25,981.981 INFO:teuthology.run_tasks:Running task tests...
2015-02-03 10:04:25,982.982 INFO:teuthology.task.tests:running test_command_failed_label...
2015-02-03 10:04:25,983.983 INFO:teuthology.orchestra.run.magna067:Running (working as expected, nothing to see here): "python -c 'assert False'"
2015-02-03 10:04:26,095.095 INFO:teuthology.orchestra.run.magna067.stderr:Traceback (most recent call last):
2015-02-03 10:04:26,096.096 INFO:teuthology.orchestra.run.magna067.stderr:  File "<string>", line 1, in <module>
2015-02-03 10:04:26,096.096 INFO:teuthology.orchestra.run.magna067.stderr:AssertionError
2015-02-03 10:04:26,098.098 INFO:teuthology.task.tests:***** PASSED
2015-02-03 10:04:26,098.098 INFO:teuthology.run_tasks:Running task tests.force_command_failure...
2015-02-03 10:04:26,098.098 INFO:teuthology.orchestra.run.magna067:Running (working as expected, nothing to see here): "python -c 'assert False'"
2015-02-03 10:04:26,146.146 INFO:teuthology.orchestra.run.magna067.stderr:Traceback (most recent call last):
2015-02-03 10:04:26,146.146 INFO:teuthology.orchestra.run.magna067.stderr:  File "<string>", line 1, in <module>
2015-02-03 10:04:26,147.147 INFO:teuthology.orchestra.run.magna067.stderr:AssertionError
2015-02-03 10:04:26,148.148 ERROR:teuthology.run_tasks:Saw exception from tasks.
Traceback (most recent call last):
  File "/home/andrewschoen/teuthology/teuthology/run_tasks.py", line 53, in run_tasks
    manager = run_one_task(taskname, ctx=ctx, config=config)
  File "/home/andrewschoen/teuthology/teuthology/run_tasks.py", line 41, in run_one_task
    return fn(**kwargs)
  File "/home/andrewschoen/teuthology/teuthology/task/tests.py", line 46, in force_command_failure
    label="working as expected, nothing to see here"
  File "/home/andrewschoen/teuthology/teuthology/orchestra/cluster.py", line 64, in run
    return [remote.run(**kwargs) for remote in remotes]
  File "/home/andrewschoen/teuthology/teuthology/orchestra/remote.py", line 128, in run
    r = self._runner(client=self.ssh, name=self.shortname, **kwargs)
  File "/home/andrewschoen/teuthology/teuthology/orchestra/run.py", line 378, in run
    r.wait()
  File "/home/andrewschoen/teuthology/teuthology/orchestra/run.py", line 114, in wait
    label=self.label)
CommandFailedError: Command failed on magna067 with status 1: 'working as expected, nothing to see here'
2015-02-03 10:04:26,152.152 WARNING:teuthology.run_tasks:Saw failure during task execution, going into interactive mode...

Since Jenkins is broken again I ran unit tests locally. They passed.

Tests failed for this pull request.
Refer to this link for build results: http://jenkins.ceph.com/job/teuthology-pull-requests/818/

Tests failed for this pull request.
Refer to this link for build results: http://jenkins.ceph.com/job/teuthology-pull-requests/820/


Update dependencies

Pull Request doc
Purpose
Update dependencies as new packages were introduced with the building electrification project. It was tricky. Also, docstrings have been cleaned up and missing __init__.py files have been added.
What the code is doing
N/A
Testing
Current unit test
Where to look

Impossible to generate a Pipefile.lock file  that work with all 3 Python versions we support. The Pipefile.lock file in this PR has been generated with Python 3.10.4. For this reason, dependencies for testing are installed using the requirements.txt file (see tox.ini).@jenhagg, do we want to keep the Pipefile.lock in version control? pipenv recommends not to if multiple version of python are targeted.
Skip test in prereise.gather.flexibilitydata.doe.tests.test_bus_data that most of the time fails to connect to DOE server
Add missing __init__.py files

Usage Example/Visuals
N/A
Time estimate
15min

I would keep Pipfile.lock versioned. Maybe also add the python version to Pipfile (see docs), as below. Btw, best source I found for python_full_version is in PEP 508.
[requires]
python_full_version= "3.10.4"

I think testing multiple python versions with requirements.txt is fine. Not 100% reproducible, but don't expect too much noise.
Thinking out loud a bit here, but maybe helpful:
The context for Pipfile.lock is that pipenv and lock file is designed for "deployment environments" meaning an application that will always use exactly one python version. We kinda have a hybrid situation where we publish packages which are libraries - meant to be usable in a variable number of python environments (could be multiple OSes, python versions, etc. whatever we specify is supported). And pipenv says it's not [necessarily] meant for packages. But also have users who work with the packages directly in a local environment, but not e.g. typically included in another application, the way libraries "normally" are (idk, just in my experience). I think this is somewhat a result of splitting up repositories and simplifying internal dev workflow, e.g. by setting powersimdata as a dependency in postreise. But there is also value in making the functionality accessible to anyone generally.
So anyway, since we more or less want both a reproducible environment and to support package distribution, it maybe makes sense to keep both Pipfile.lock and requirements.txt, respectively. And the CI checks will catch anything that breaks downstream users (fine print: as long as it's covered by tests/linting/etc).

I would keep Pipfile.lock versioned. Maybe also add the python version to Pipfile (see docs), as below. Btw, best source I found for python_full_version is in PEP 508.
[requires]
python_full_version= "3.10.4"

I think testing multiple python versions with requirements.txt is fine. Not 100% reproducible, but don't expect too much noise.
Thinking out loud a bit here, but maybe helpful:
The context for Pipfile.lock is that pipenv and lock file is designed for "deployment environments" meaning an application that will always use exactly one python version. We kinda have a hybrid situation where we publish packages which are libraries - meant to be usable in a variable number of python environments (could be multiple OSes, python versions, etc. whatever we specify is supported). And pipenv says it's not [necessarily] meant for packages. But also have users who work with the packages directly in a local environment, but not e.g. typically included in another application, the way libraries "normally" are (idk, just in my experience). I think this is somewhat a result of splitting up repositories and simplifying internal dev workflow, e.g. by setting powersimdata as a dependency in postreise. But there is also value in making the functionality accessible to anyone generally.
So anyway, since we more or less want both a reproducible environment and to support package distribution, it maybe makes sense to keep both Pipfile.lock and requirements.txt, respectively. And the CI checks will catch anything that breaks downstream users (fine print: as long as it's covered by tests/linting/etc).

Sounds good. I have updated the Pipfile as suggested.


Update tested AGP versions

from 8.1.0 to 8.1.1
from 8.2.0-alpha15 to 8.2.0-beta02
introducing 8.3.0-alpha02
nightly is now 8.3.0-dev
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test AFT AST APT

I've triggered the following builds for you. Click here to see all build failures.

AllFunctionalTestsPullRequestFeedback build
AllSmokeTestsPullRequestFeedback build
AllPerformanceTestsReadyForRelease build

@bot-gradle test and merge

I've triggered a build for you. Click here to see all failures if there's any.


Create who.is-app.tech.json

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.

Description
A project site of mine.
Link to Website
https://atlasl1.github.io/Miscellaneous

I have read the CLA Document and I hereby sign the CLA


Allow to define PACTs per test case instead of globally

Changes:

For ScalaTest, add trait for defining PACTs inline in each test case instead of globally for the whole suite.
Extract some common code to avoid duplication.

This allows a more local definition of PACTs as needed for each test case, which I find more readable than having one large PACT for all test cases together.
I don't know MUnit enough to add a similar feature for MUnit, but I hope - if it is accepted - that it can easily be implemented by a more knowledgeable person there, too.

Sounds like a nice improvement @danile42 , I'll give it a look in the coming days.
Could you add a word about the new possible usage in the README in the meantime?

@gaeljw: Yes, I will add to the README. Looking forward to any review comments :)

Included in v0.12.0 (available in a few minutes). Thanks for your work @danile42 :+1:


Modularization code in Quiz Game

Modularize code in Quiz Game.
This project has only one file called main.py. I think it is needed modularization to improve the code because of the following reasons:

Reduce duplicate code
Easy to maintain

What will do:

Make some functions and modularize code
Make it easy to remove or add quiz by using list

This is project link to Quiz Game: https://github.com/Mrinank-Bhowmick/python-beginner-projects/tree/main/projects/Quiz Game

I would like to work this issue. Could you assign to this issue please?
Thank you

Ya sure go for it, mention this issue in your pr 👍

hii, I would like to work on this issue


⚠ File access (beta) has degraded performance

In dd48e6c, File access (beta) (https://beta.fyle.uk/file/fish) experienced degraded performance:

HTTP code: 503
Response time: 6879 ms

Resolved: File access (beta) performance has improved in 5d9c54c after 8 minutes.


Add github issue report config

Github has a feature that allows us to setup a standard issue template, see https://help.github.com/articles/creating-an-issue-template-for-your-repository/
The tricky thing is that we don't have a central repository for users to submit issues to. Users must also be registered on GitHub before they can submit issues.
Maybe not ideal but very cost efficient for us.

It has been done


blur_new_optimizations=1 causes flashing

open an xwayland window(the example here being webcord)
have a eww bar open
turn off your internet
webcord started flashing
https://github.com/hyprwm/Hyprland/commit/a163ca92377a56c196222a48c0a5676a7bb4ac3b
this might be the commit that introduced it, haven't got time to thoroughly check tho

can't repro.
Is the window opaque?
Define flashing. What is flashing? webcord? the blurred background?
git bisect would be useful.

https://0x0.st/oRdv.mkv

After doing a git bisect, I can indeed confirm that https://github.com/hyprwm/Hyprland/commit/a163ca92377a56c196222a48c0a5676a7bb4ac3b this is the commit that introduced the bug

should be fixed with 2a5ae435e19e5ccc3a4d989497f062614307f104

It is fixed! Thanks.


Receive images and audio by telegram adapter

Rasa Core version: 0.12.0
Python version: 3.6
Operating system: osx
Issue:
As a developer i want to receive audio and images when a user sends to telegram bot
Current Status
The current state of receiving messages just handles messages and locations
rasa_core/rasa_core/channels/telegram.py

Thanks for raising this issue, @tmbo  will get back to you about it soon.

Well, the receiving part isn't that hard - the question is rather what we do with the data and how we process it. do you have any ideas around this? e.g. what is it that you want to do if a user sends you an image?

Hi @tmbo thanks for the feedback, my idea is to create an external adapter to handle the photo and audio.
The idea of the audio is to transform into plain text and then send to Rasa and the idea of the photo is to recognize QRcode, modify to text and send to rasa
But  i think its better to create this externally of rasa core like an extension because it will require more libs


SP5: WPFplot not working after WindowClosingEvent canceled (#2700)

Its same issue - #2700
Bug Report
Issue: WPFplot dispose after WindowClosingEvent canceled
Reproducing:
Hi.
What I meant by "not working" is that it is impossible to refresh the plot including interaction.
sorry for i didn't write details.
I wrote the same code as you shared and ran two applications, Scottplot4 and Scottplot5.
// Scottplot 5
public partial class MainWindow : Window
{
    public MainWindow()
    {
        InitializeComponent();

        plot.Plot.Add.Signal(ScottPlot.Generate.Sin());
        plot.Plot.Add.Signal(ScottPlot.Generate.Cos());

        this.Closing += MainWindow_ClosingEvent;
    }

    private void MainWindow_ClosingEvent(object? sender, CancelEventArgs e)
    {
        e.Cancel = true;
    }
}

// Scottplot 4
public partial class MainWindow : Window
{
    public MainWindow()
    {
        InitializeComponent();

        double[] dataX = new double[] { 1, 2, 3, 4, 5 };
        double[] dataY = new double[] { 1, 4, 9, 16, 25 };

        plot.Plot.AddScatter(dataX, dataY);
        plot.Refresh();

        this.Closing += MainWindow_ClosingEvent;
    }
    private void MainWindow_ClosingEvent(object? sender, CancelEventArgs e)
    {
        e.Cancel = true;
    }
}

and, There is a video of the problem below:
https://youtube.com/shorts/GhnFCOVjh8c?feature=share
Interaction and refresh are works well.
after, Press X to proceed with window closing and cancel the event,
It works fine on Scottplot4, Not in Scottplot5.
System Details

ScottPlot Version: 5.0.3-beta
Operating System: Windows 11
Application Type: WPF
.NET Version: .NET5

Thanks for following-up @minjjKang! I'm going to copy this information into https://github.com/ScottPlot/ScottPlot/issues/2700 to keep this discussion all in one place, and I'll respond over there 👍


monitoring: really add the monitoring ingress from #9

Forgot to include the Ingress yaml in the kustomization file in commit 2e37c91.  D'oh.

bors r+


chore(package): update kronos-message to version 2.1.4

Codecov Report

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

@@          Coverage Diff          @@
##           master   #130   +/-   ##
=====================================
  Coverage      80%    80%
=====================================
  Files           1      1
  Lines          10     10
=====================================
  Hits            8      8
  Misses          2      2

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 d324d25...afa66af. Read the comment docs.


Pod antiaffinity strategy.

Signed-off-by: ravisantoshgudimetla<EMAIL_ADDRESS>

test this please

Can one of the admins verify this patch?

ok to test

test this please

retest this please

retest this please


Unable to Run Install Script.

I am getting this error when trying to run the install script.
Traceback (most recent call last):
File "/usr/local/bin/listen-for-shutdown.py", line 4, in
import RPi.GPIO as GPIO
ImportError: No module named RPi.GPIO
Python 3.x
64bit Bullseye on a RPi 4 b
Let me know if you need my information.

I've been using this fork, been working well for me.

Thanks, but still getting the same error.

How did you try to install?


Adjust padding on secondary search probe

This seems to work okay:
padding: 10px 40px 5px 20px;

Some of these have a lot of extra space:


IntialNode Count MaxCount MinCount Validation

The validation for         initialNodeCount: 2 working weird.
This works
  nodePools:
    - metadata:
        name: my-node-pool
      spec:
        minCount: 2
        maxCount: 2
        initialNodeCount: 2

This does not work:
  nodePools:
    - metadata:
        name: my-node-pool
      spec:
        minCount: 2
        initialNodeCount: 2

This does not work:
  nodePools:
    - metadata:
        name: my-node-pool
      spec:
        initialNodeCount: 2

We need to decide exactly how it should work.  If initialNodeCount is set, but min and max are not set, what do we set them to??  How does this impact autoscaling??
@bgeesaman @robinpercy @wripley

My thoughts:
If min and max are not set, no autoscaling is to be enabled OR, set min and max to same as initialnodecount and warn?
If only min is set, validation error for not providing max.


Add CMake macro for generating pybind11 bindings

Desired behavior
Right now there is some code duplication in ign-gazebo and ign-math for finding pybind11. I assume the goal is to provide Python interfaces also for other packages in the future, thus it makes sense to have a common macro for it.
Implementation suggestion
Since PythonLibs is deprecated, the snippet should also use FindPython3 which will be provided with PR #213 .
It might be useful to extend the new IgnPython to provide several arguments as input in order to reuse the existing code but this might require some additional effort than creating an independent IgnPythonBindings macro.

Right now there is some code duplication in ign-gazebo and ign-math for finding pybind11. I assume the goal is to provide Python interfaces also for other packages in the future, thus it makes sense to have a common macro for it.

While packaging new versions of ign-gazebo/ign-math I had the same thought. It would be great if we can de-duplicate the code and use a unify macro for it.

It might be useful to extend the new IgnPython to provide several arguments as input in order to reuse the existing code but this might require some additional effort than creating an independent IgnPythonBindings macro.

Currently IgnPython works as a CMake Module working just by including it. We can add a macro in the same file and avoid the creation of a new file in the repository. That could make sense since there are probably no use cases where someone needs to use the python bindings macro but don't want to check for a python3 installation.

It might be useful to extend the new IgnPython to provide several arguments as input in order to reuse the existing code but this might require some additional effort than creating an independent IgnPythonBindings macro.

Currently IgnPython works as a CMake Module working just by including it. We can add a macro in the same file and avoid the creation of a new file in the repository. That could make sense since there are probably no use cases where someone needs to use the python bindings macro but don't want to check for a python3 installation.

I made the needed changes for pybind11 in https://github.com/ignitionrobotics/ign-math/pull/402 to fix the related issue.
There is a small difference, we need find_package(Python3 QUIET COMPONENTS Interpreter Development) instead of find_package(Python3 QUIET COMPONENTS Interpreter) = find_package(Python3 QUIET) which is used in ign-cmake. We could use a macro which also searches for Development but in my opinion it'd be nicer to use a different include statement.
macro:
include(IgnPython)
search_pybind11_library() # does a find_package(Python3) twice, once without Development and once with (+ there are even more options like numpy/different developments)

separate module:
include(IgnPythonBindings)

I don't know but might it be possible to do include(IgnPython *my_arguments_list*)? Then the items to be searched for can directly be declared within the include command.
However, for similarity between different Python 'includes' I think it would make sense to have a common way on how to use/find Python3 - a single macro for each case of finding Python3, a macro with several arguments to specify the needs or separate and specific include modules.


Better API documentation

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
The current API documentation provides a basic overview of the available endpoints and touches on pagination. However, it lacks details regarding the query parameters that are supported, as well as documentation for the response body. Enhancing the API documentation will greatly improve the developer experience.
It would be awesome to have interactive API documentation with tools like Swagger or Redoc. Offering an OpenAPI specification would be also be helpful, because it would make it easier to generate an SDK for using the Traefik API.

I was just looking for something like an openapi spec this morning, I hope there is one and maintainers can just point us to it, otherwise please prioritize this!

Hello,
Thank you for your suggestion!
We agree with you, it would be great to have a better API documentation.
If you or another community member would like to build it before that happens, let us know, and we will work with you to make sure you have all the information needed.
Let us know here before you start. We prefer to work with our community members at the beginning of the design process to ensure that we are aligned and can move quickly with the review and merge process.

Hey @rtribotte I'm very keen to work on this issue . I would be really happy if you could assign this issue to me.

Hello @rohitmore07,
Great news! We have assigned the issue to you, thank you!


format prop is described on README but it's not published

as you can see, "format" prop to tell component to generate other format (like wav) is documented on this project's README but it's not published (until this last version - 3.0.1).
I guess should publish new version with this feature or remove it from README :pray:

Yes

Everyone forks and publishes the same without even checking the issues


feat(trpc-user): Create the tRPC backend and add user procedures

This PR:

Build the tRPC backend.
Add the tRPC backed for user-related APIs.

Note that as we agreed in https://github.com/ideafast/ideafast-portal/issues/2393, this PR focuses on adding backend functions and passing the build and unit tests.
Several related backend functions are also added to help pass the build and tests; such functions will be updated again later.
Please focus on the type definitions, core functions, tRPC procedures and the unit tests of 'user'.

I have two main point to raise on this PR.

I have big concerns about the success of data migration for older versions. I can see a large number of m_* variables have been renamed amongst other things, we need to be extra careful as to how this is done.
I am not fully understanding the concept of cores in this sense. You have essentially completely reimplemented everything twice. The architecture we discussed originally leveraged an architecture where the core package would provide the execution logic for both the TRPC and GraphQL layers. Taking as an example the getUsers function: They do completely different things, and each separately called the DB driver. That is really not a scalable way to address this issue. It is perhaps worth discussing this again ?

Hi Florian,

Yes, previously the m_ variables include those that are only used in the IDEAFAST (such as subjected, and visited); I think for a more general situation we should not assume any pre-defined properties of the subject, thus I rename them. I think the necessary actions indeed are required, so it is risky but I guess I still prefer to make these changes.
I agree. In the roadmap,  what I plan to do is to keep both core functions at the early stage and merge them into one once both the two collections of APIs are working as expected (roadmap 2.17: merge tRPC and GraphQL core functions into one). What do you think of this?


complete the missing brackets in the code example

Hi Brannon,
There were some parentheses missing in the code examples of README file. I have checked all the examples and added the missing parentheses.
Cheers,
zl1n

Cheers @zl1n! Thank you for this contribution 🙏


Spike github integration token to allow for more plugin downloads via github api

Hey all,

Results in :

Ideas?
brian [11:48 PM]
The deepstream team was very helpful in getting some of my core questions answered.
I'm not sure if my API limit is because my organization pulls so many git requests or if it is deepstream needs to expand or register their github API.
sergejkurakin [11:50 PM]
You can do up to 60 unauthenticated requests per hour https://developer.github.com/v3/#rate-limiting
[11:51]
so, if you do any - you can easy go beyond the limit
----- Today May 4th, 2017 -----
yasserf [12:12 AM]
hey, interesting! I seen that happen before on travis as well
[12:14]
just looked through the code, doesn't seem like we have a way of injecting an auth token to get that always working
[12:14]
that being said, github just launched an awesome new integrations token which we could use to get over this limit
[12:15]
you can download the packages directly from github to place them in your lib folder until we get this fixed
[12:17]
option b would be for us to put everything in a public s3 bucket (edited)

This has been happening rarely enough for it not to be a priority..


Fix importing in tools files

Fixes importing files in the tools/ folder

Whoops, this isn't actually needed!


Add storm lines

Description

How to test the feature:

[ ]
[ ]

Checklist - did you ...

Test your changes with

[ ] REACT_APP_COUNTRY=rbd yarn start
[ ] REACT_APP_COUNTRY=cambodia yarn start
[ ] REACT_APP_COUNTRY=mozambique yarn start
[ ] Add / update necessary tests?
[ ] Add / update outdated documentation?

Screenshot/video of feature:

@Max-Z80 feel free to use and merge into your branch if useful

ok @ericboucher , I merge this


CSS cleanup

This touches basically everything with the goal of reducing the amount of unique code used for UI components.
At the same time, I couldn't help myself from cleaning up some somewhat unrelated things so there's some miscellaneous changes and fixes thrown in because that's definitely how this is supposed to work 🙃.
The overall gist of the reduction in UI components is that before there were 8 identifiable components:

Side Panels
UI Bubbles
Panel Articles/Sections
Lookup Dialog
System messages
Modal Dialogs
Notifications
Popups

With special cases arising for moderation tools such as user lookups, the mod sidepanel, and chatban history.
This PR reduces this to just 4 core components:

UI Bubbles (which now includes the mod sidepanel)
Panels (side panels, modal dialogs, and popups)
Panel Articles/Sections (which are used by notifications and chatban history)
Floating Panels (lookups and system messages)

Perhaps the most significant other changes are those made to buttons. The summary of this is that anything a user is expected to click on to perform an action is now a button. This is mostly just an HTML semantics things which isn't even all that related to CSS but resolving it at the same time resulted in more consistent button styling methods for traditional text-buttons.
Hopefully this can be merged without too much change, but I wouldn't be surprised to be told "go back and break it into more appropriate pieces".

I think .floating-panel should be a bit more compact:

Reduce the value of margin-top in .floating-panel .content > div:not(:first-of-type) to 5px
Change padding in body .message to padding-bottom: 10px; (If you feel it should be explicit, padding: 0 0 10px;)
Remove min-width in body .message
Remove margin-bottom in .floating-panel .content

I've done mostly as requested but kept some of the bottom margin in .floating-panel .content since this padding is still somewhat important. This is most noticeable in lookups where it originates from:

(Image with no margin)


Problems with dependencies with different folder names

Hi, I'm newbie on python, and I have a complex arduino program that uses some libraries in just one folder. I guess your program works taking the names of the folders and using to know the name of the libraries on it. But I have many libraries on just one folder of the lib, and the software can't include this files on the project. I tried put one folder for each files but some files includes other libraries. Can you help me with it? Thanks

yep, that's something I've been wanting to work on for a few months but I haven't got the time yet.
I'll try to get it fixed ASAP ;)

I would also like to do this. Here is what I am trying to do:
lib/Station.h
-- #include <XBee.h>
lib/PowerManager.h
src/H2OiQ.cpp
-- #include "Station.h"
-- #include "PowerManager.h"
The libdetection.py script that you added to the Arduino-Makefile project doesn't allow this. One additional thing to note in addition to your suggested enhancement above:

Arduino system libraries (Arduino.h, SoftwareSerial.h, XBee.h, etc..) cannot be included in library files.

Is this an easy fix? I'd love to utilize this project for my next project.

Arduino system libraries (Arduino.h, SoftwareSerial.h, XBee.h, etc..) cannot be included in library files.

@jon-whit: what do you mean? what system are you on?

In BarProject.cpp if you include SoftwareSerial.h you don't get any compilation errors:
#include <SoftwareSerial.h>

void setup() {
...
}

void loop() {
...
}

However, if you try to include the same SoftwareSerial system header in BarLib.h and then include BarLib.h in BarProject.cpp, then compilation fails with message:
In file included from BarProject.cpp:3:0:
/home/jonathan/GitHub/Bare-Arduino-Project/lib/BarLib/BarLib.h:11:28: fatal error: SoftwareSerial.h: No such file or directory
 #include <SoftwareSerial.h>

@jon-whit my apologies for the long delay, I've been pretty busy recently with my startup.
I tried what you said, including #include <SoftwareSerial.h> in the BarProject and BarLib and everything compiles just fine.
Also, if you look at the content of the library files, you'll see that I have included Arduino.h.
Could you try cloning the project again and tell me if it still doesn't work?

I am having a similar problem as @jon-whit, and I just cloned the project a day or two ago.
I haven't had a chance to look into it yet but I suspect that the code that generates the included libraries is only looking at the top level file i.e. BarProject.cpp
The work around I'm using, is that if I want to add SPI.h in a library I made I need to also add it in the top level file, even if I don't need it there.
So if in BarLib.h I want:
#include <SPI.h>
I need to also put in BarProject.cpp:
#include <SPI.h>
as well.

yep that's right -- for Arduino libs, you need to include them in your project.

@ladislas This issue still has yet to get fixed. The libdectection.py script that you created is not inclusive enough for #includes directives. Consider the simple Arduino sketch directory shown below:
/home/user/sketchbook
.... readme.txt
.... xbee_lib
........ build.xml
........ docs
........ examples
........ XBee.h
........ XBee.c

If you have a project file main.cpp in src that includes XBee.h,
// main.cpp
#include <Arduino.h>
#include <XBee.h>

void setup()
{
    Serial.begin(9600);
    XBee xbee = XBee();
    xbee.setSerial(Serial);
}

void loop()
{
}

Running the compilation target in the Makefile fails with an error indicating that XBee.h could not be found.
> avr-g++ -MMD -c -mmcu=atmega328p -DF_CPU=16000000L -DARDUINO=105  -D__PROG_TYPES_COMPAT__ -I/usr/share/arduino/hardware/arduino//cores/arduino -I/usr/share/arduino/hardware/arduino//variants/standard -Wall -ffunction-sections -fdata-sections -Os -pedantic -Wall -Wextra -fno-exceptions -std=gnu++11 main.cpp -o /home/user/dir/project-dir/bin/project-dir/uno/main.cpp.o
main.cpp:2:X: fatal error: XBee.h: No such file or directory
 #include <XBee.h>

I'm simply trying to use 3rd party libraries for some of my Arduino projects, but the only way I have found to do so is to copy the 3rd party library into the lib directory and change the headers and definition file names to names of the form xxxxLib.(h,c).

@jon-whit : do you have a repository on github that I could clone to test your settings.
I think I already raised an issue like the one you're having here: https://github.com/sudar/Arduino-Makefile/issues/249#issuecomment-54158707
There are two things that might get in the way of doing what you want:

the first is the way the Makefile works: it is based on how Arduino works and doesn't allow you to put all your libraries in a big lib folder and simply include them. It wants you to do as Arduino says, i.e. have a folder named LibName with inside you files LibName.h and LibName.cpp. You can also have other files with different name but you'll only be able to reference the one with LibName in your main.cpp
the second is that the Makefile does a lot of checking and processing to be as easy to use as possible for beginners. But it uses the Makefile language which is complicated and prone to errors. There is an idea of rewriting the full logic in python to be more easily customizable, still using make to call everything, but it will take time

my script allow you to not reference all the the headers you need to compile your sketch. It looks for other headers in your lib/LibName folders and include them automatically. It does not however look for header outside of the clone repo. Everything must be in lib. My project Moti is a good exemple of how things should work (https://github.com/WeAreLeka/moti). You can just clone it, customize the Makefile with your own paths and it should work.
I do not plan on adding the possibility to look for libs outside of your project as the initial purpose of the script was to be able to package everything in the same place and be able to share it with a team, without having to install outside libraries in complex locations.

any news on this?

i'm closing this issue, feel free to reopen if needed.