EditorSettings's Show Hidden Files affects user FileDialog nodes as well

Godot version
3.5.2.stable, 4.0.1.stable
System information
Arch Linux
Issue description
When the Show Hidden Files property is enabled in the Editor Settings:

Any FileDialog nodes from the project will inherit this setting, and their Show Hidden Files property is set to true everytime the scene is re-opened:
https://user-images.githubusercontent.com/6501975/226909135-a8fafdf7-ca36-40aa-b69a-a7baf2d7ae4f.mp4
This occurs on 3.5.2 as well.
Steps to reproduce

Enable Show Hidden Files property in the Editor Settings:

Reload Godot for the bug to occur

Create a FileDialog node, and set it's Show Hidden Files property to false
Save and close the scene, and reopen it
The Show Hidden Files property is reset to true.

Minimal reproduction project
N/A

This is by design but I'm not sure why

It can be set in scenes, though not entirely sure why that would be useful as it's something you can change from the dialog itself, but the current behavior clashes with the scene saving so needs some change
I would assume that the setting in EditorNode is legacy, and should be removed

It can be set in scenes

Yes, but it's overwritten with the static variable, which can't be modified from the editor. It could be a project setting maybe.


SimpleViewer using Xtion2 error

Hi, I am trying to run SimpleViewer in Ubuntu 14.04 using Xtion 2 ASUS, but I get below error

Video Camera at /dev/video1 Detected
Depth Camera at /dev/video2 Detected
After initialization:
Creating Depth stream
Depth Stream start
Creating color stream
VID_STREAMON error 28, No space left on device

I saw https://answers.ros.org/question/12582/usb_cam-vidioc_streamon-error-28/
and I did it, but this error appeared.
Could  you please help me?

you must you port for USB2.0


How can I get the ObjectClass form a RLMArray  *property

I define a property in a class, like this RLMArray *propertyName
and i want to get its real type -> ObjectClass
// this method using runtime to get the type of property
// in the case , it return @"RLMArray<ObjectClass>"
- (NSString *)getPropertyTypeString:(NSString *)propertyName

after this , using Class class = NSClassFromString(@"RLMArray") to get its ObjectClass. But it return null;
besides this, i got JSON from server, then use [self setValue:JSON[jsonKey] forKey:propertyName];
it throws BAD_ACCESS;

after this statement, shows propertyName is nil;
JSON[jsonKey] has many items (array).

@mrackwitz the reason seems that realm can not parse null.

If you want to retrieve the related class of a property, you can go over the object schema.
let object : RLMObject = …
let property = object.objectSchema[propertyName]
if property == .Array {
     let objectClassName = object.objectSchema[propertyName]
     …
}

What do you want to achieve with the element type?

From what you showed so far; I'd guess that the EXC_BAD_ACCESS originates from a memory management issue in your code. If you need further help with that, please show us more context around that, where is the method called, what is self?

@mrackwitz my fault,  thank you for the example, i'll check my code again, carefully.


Allow isHTML to be overridden

This exposes isHTML as a session variable to allow overriding the function used in the same manner as Turbo.session.drive = false can be used to disable turbo functionality by default and placing it in opt-in mode. To modify the default isHTML check one can simply do the following.
Turbo.session.isHTML = (url) => { return true; }

I attempted to run the tests locally before submitting this, was able to resolve the chromedriver error by installing the chromedriver cask on my Mac and setting an environment variable for CHROMEVER that pointed at it, and there were a number of failures. It appears that might be an issue on the main branch as well, but do let me know if this PR is worth consideration and I will dig in and attempt to sort out my local environment test issues if not global.

We're exploring this on https://github.com/hotwired/turbo/pull/519.


Model decodes different sentences identically

I am attempting to create a part-of-speech/prepositional super-sense tagger using the readme walk through. I created my own data generator/problem and let training run for about 6500 steps on transformer_base. I put a few sentences in the decode file and run the walk through decode lines. Each sentence is decoded into the same set of tags.
INFO:tensorflow:Restoring parameters from /root/t2t_train/sequence_tagger/transformer-transformer_base/model.ckpt-6721
INFO:tensorflow:Inference results INPUT: Michael Jordan played basketball for the Chicago Bulls.
INFO:tensorflow:Inference results OUTPUT: O O-GROUP O-Function O-social O O-GROUP B-COMMUNICATION Ī O-Co-Agent O O-PERSON O-Possessor O O-GROUP O
INFO:tensorflow:Inference results INPUT: I killed Godzilla with a laser beam.
INFO:tensorflow:Inference results OUTPUT: O O-GROUP O-Function O-social O O-GROUP B-COMMUNICATION Ī O-Co-Agent O O-PERSON O-Possessor O O-GROUP O
INFO:tensorflow:Inference results INPUT: Bob ate pizza in the park.
INFO:tensorflow:Inference results OUTPUT: O O-GROUP O-Function O-social O O-GROUP B-COMMUNICATION Ī O-Co-Agent O O-PERSON O-Possessor O O-GROUP O
INFO:tensorflow:Writing decodes into /root/t2t_data/decode_this.txt.transformer.transformer_base.beam8.alpha2.0.decodes
The decoded line doesn't match up with any of the inputs. The model was getting a checkpoint accuracy of around 80% which makes me think there's just something going wonky with the decoder. If I re-run the decode line, it outputs a different decoded sentence (still doesn't match up with any inputs). A similar issue was posted and closed stating that T2T 1.0.11 fixed this but it did not in my case.
Thanks in advance!

@jackperkins98  my decoding also have some problem. T2T.1.0.9.  The total loss is 2.1 but BLEU reaches only at 12.42 which is too low. I find that the ratio is 2.002, which is too high.
BLEU = 12.42, 29.1/15.3/9.2/5.8 (BP=1.000, ratio=2.002, hyp_len=126937, ref_len=63412)

It's very hard to reproduce your issue if you're using a private data-set or an older version of T2T. Can you see the issue with a new version of T2T, or add your data-set? Also, I believe 6500 steps is simply not enough to train, esp. with warmup_steps set to a high number. Could you try at least 20-30k steps? I'm closing this for now as unreproducible, but please reopen if there's a way for us to reproduce.

Hey - have you had any luck with this?  I've got a transformer model doing the same thing after ~27,000 iterations

It was just the amount of steps. After 30k it worked fine. Depending on how
bug your data set is, you probably just need more steps.
On Wed, Sep 6, 2017 at 5:47 PM, Benjamin Cohen<EMAIL_ADDRESS>wrote:

Hey - have you had any luck with this? I've got a transformer model doing
the same thing after ~27,000 iterations
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensor2tensor/issues/130#issuecomment-327621428,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcDR293blfydsHz1ifK8bIIsDMm7Az5Xks5sfxLVgaJpZM4OTeUo
.


Update plugin-ascentialtest-cmd-line.yml

updated the user name of the developer.

Description
https://github.com/jenkinsci/ascentialtest-cmd-line-plugin
Submitter checklist for adding or changing permissions

Always

[x ] Add link to plugin/component Git repository in description above

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

Is that your github username or Jenkins infra username? It needs to be the latter.


User Agent spoofing resets after WebContent crash

Steps I took:

Set Spoof User Agent to Firefox Linux Desktop
Visit a page that triggers a WebContent crash (I tested with the HTML from #514 )
Visit a page that checks the user agent (e.g. https://whatsmyua.info)

The user agent displayed is "Mozilla/5.0 (Linux; x86_64) Ladybird/1.0" rather than the expected Firefox UA

This happens because overriding the user agent is not saved to the settings like other values (such as initial tab page) are.
Is the spoof user agent setting saved on exit by any other browser? I was under the impression that they were always per-tab, transient settings.

to be clear, I'm referring to WebContent crashes, not a whole browser crash

Oh! Yes, this is definitely something we should re-sync when launching a new web content process for new tabs


Include selected activation key info

Add the available RHEL 9+ features to the activation keys step per the mocks:  https://marvelapp.com/prototype/6g7dja9/screen/76602546

The activation key information was added in https://github.com/RedHatInsights/image-builder-frontend/pull/886 and the Registration step was updated by https://github.com/RedHatInsights/image-builder-frontend/pull/968


Lookout API needs to use a bespoke REST API instead of grpc generated rest

Discussions with GR have made it clear that we want to no longer support REST API in Armada.  With GRPC clients for python and C#, we are able to migrate clients to use GRPC.
However, lookout still uses REST API for both the Armada endpoints and their own endpoints.
We should convert lookout.proto into a rest api definition.

Hey team! Please add your planning poker estimate with Zenhub @dejanzele @jayofdoom @kannon92 @richscott

Please add your planning poker estimate with Zenhub @suprjinx


hal: dma::controller::Registers trait

Defines a Registers trait which abstracts over the DMAC registers, allowing DMA drivers to be written targeting this trait, and then used with any relevant PAC which already maps them with svd2rust.
The trait is defined differently for SAMD21 vs SAMD51 as they have different DMAC registers, but usable with both. An additional higher-level abstraction trait will have to be written to abstract
across the differences between the two.
This approach is relatively macro-heavy, and per this definition needs to be wired up (via a macro) on a board-by-board basis, but the result should enable the rest of the DMA subsystem to be written sans macros by leveraging the newly defined trait.

Why are you defining things at the board level? Shouldn't it all be HAL level? Also, I wonder if we should modify the SVD to define the registers, so they are PAC level.

@sajattack the DMAC registers are defined by the PAC. It'd be great if we could do this at the PAC level, but as far as I can tell the PAC is 100% svd2rust, and doesn't directly depend on the HAL


Support Stretch with fw 2.7.18

Fixes https://github.com/plugwise/plugwise-beta/issues/228

User feedback is positive, merging into main-branch


Provide workaround for POR-3116

Currently GBIF always assigns new datasets a GBIF DOI during registration, unless the DOI assigned to the dataset is explicitly specified in the API request as a parameter.
During indexing GBIF parses the dataset's EML, and can find a user-assigned DOI. For example, it may have been specified in the citation identifier.
Ideally GBIF would parse the dataset's EML at the time of registration to try and detect if the dataset already has a DOI. Issue POR-3116 has been opened up to try and make this enhancement.
As a workaround, the IPT can simply include the user-assigned DOI in the API request to prevent GBIF from assigning the dataset an extra unnecessary DOI.

Issue verified.


I configured the GPT432K with .\Configure.ps1, but when I asked, the answer was still GPT3, what do I need to change?

I configured the GPT432K with .\Configure.ps1, but when I asked, the answer was still GPT3, what do I need to change?

Edit the value under Settings, Configuration of the WebApi service that you deployed to in the Azure Portal.

感谢，已经解决
获取Outlook for Androidhttps://aka.ms/AAb9ysg

From: Darth Coder @.>
Sent: Tuesday, November 28, 2023 4:04:14 AM
To: microsoft/chat-copilot @.>
Cc: 刘海斌 @.>; Author @.>
Subject: Re: [microsoft/chat-copilot] I configured the GPT432K with .\Configure.ps1, but when I asked, the answer was still GPT3, what do I need to change? (Issue #657)
Edit the value under Settings, Configuration of the WebApi service that you deployed to in the Azure Portal.
―
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/chat-copilot/issues/657#issuecomment-1828507632, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALX3UTGDTRG5CFNDRVFIF2LYGTW35AVCNFSM6AAAAAA72YTYKSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRYGUYDONRTGI.
You are receiving this because you authored the thread.Message ID: @.***>

Had to pull out my translator.  Apologies for the late engagement and big thanks to @DarthCodingMaster for the community engagement!


Add a test for ES6 Modules

#Would perhaps make sense to add ES6 Modules if HTML Imports are removed. ES Modules are what Mozilla want to go with instead for Web Components, and Edge is currently implementing too. According to https://www.chromestatus.com/features/5365692190687232 they have wide vendor consensus. There are at least two ES features: I18n API and Promises, so it wouldn't be totally out of the ordinary to include a ES feature on this test.

Any idea how this should be feature detected?
/cc @kangax

Do you want to test <script type="module" /> support specifically?

Yes, that is the idea.

You could use the fact that not supported type's won't load the src, and thus no onload event will be fired:
Here's my guess, but I have no browser that supports it to test on:
<body>
<script>

	var done = function () {
		// we're done testing
	};
	var success = function () {
		console.log(this.getAttribute('type') + ' is supported');
	};

	document.body.addEventListener("load", done, false);
</script>

<!-- works IF type=module is supported -->
<script type="module" onload="success.call(this)" src="about:blank"></script>

<!-- should NOT work -->
<script type="a-invalid-type" onload="success.call(this)" src="about:blank"></script>

<!-- should work -->
<script type="text/javascript" onload="success.call(this)" src="about:blank"></script>

<!-- should work -->
<script onload="success.call(this)" src="about:blank"></script>

</body>

Should be able to test any test using Edge 14 with flags enabled (I believe it works with both import/export and

You could just have it set to false by default and have a
<script type="module" src="module_test.js"></script>

which just sets it to true. It's an extra file but no fuss.

Yeah, I was thinking about something similar. But I don't want it to be dependant on the network, so I guess using a data URL will work too.

I've added a test for ES6 modules. Works in Edge 14 with Experimental JavaScript features enabled.

Added, now available on http://alpha.html5test.com


Modify the callbacks to use the new external param classes

See https://github.com/DiamondLightSource/hyperion/issues/1092 and https://github.com/DiamondLightSource/hyperion/issues/1275
Acceptance Criteria

The callbacks use the new parameters and have no dependency on the old models
Tests all pass
In doing this think about callback structure, what parameters it needs and spin out new issues if there are ways to simplify

Depends on https://github.com/DiamondLightSource/hyperion/pull/1279 and https://github.com/DiamondLightSource/hyperion/issues/698

We think a M


'NoneType' object is not subscriptable

Within home-assistant, the current status of a jellyfin instance is not updated. The logs show the following:
ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/pyemby/server.py", line 287, in socket_connection
    self.process_msg(msg.data)
  File "/usr/local/lib/python3.8/site-packages/pyemby/server.py", line 319, in process_msg
    self.update_device_list(self._sessions)
  File "/usr/local/lib/python3.8/site-packages/pyemby/server.py", line 342, in update_device_list
    dev_name, device['NowPlayingItem']['Type'],
TypeError: 'NoneType' object is not subscriptable

This looks to me like an issue in pyemby, perhaps just some mapping, or is this an issue by jellyfin not implementing the API correclty?
This has been reported in home-assistant/core#38501

I asked for help at reddit, will hopefully get some response.

This would be because the Jellyfin serializer started returning null values instead of not including the key in the response. It caused some problems for me too in jellyfin-mpv-shim.
This code would have to be updated to deal with a null value for device['NowPlayingItem'] in addition to the key not being present.
It was changed all over, so more changes for where keys historically weren't present now need to also deal with null values. I've gotten really defensive about null values coming back from Jellyfin accordingly.

iwalton's correct.  What's happening is that some of the responses from the server are coming back as None.  As an example, here's the device profile I'm getting from Home Assistant.
{'AdditionalUsers': [],
 'ApplicationVersion': '1.6',
 'Capabilities': {'AppStoreUrl': None,
                  'DeviceProfile': None,
                  'IconUrl': None,
                  'MessageCallbackUrl': None,
                  'PlayableMediaTypes': [],
                  'SupportedCommands': [],
                  'SupportsContentUploading': False,
                  'SupportsMediaControl': False,
                  'SupportsPersistentIdentifier': True,
                  'SupportsSync': False},
 'Client': 'Home-Assistant',
 'DeviceId': '229044588395818',
 'DeviceName': 'HomeAssistant',
 'DeviceType': None,
 'FullNowPlayingItem': None,
 'HasCustomDeviceName': False,
 'Id': 'b8a8f09816c76aa04e04dbd3151f4cff',
 'IsActive': True,
 'LastActivityDate': '2020-09-20T16:04:15.3467043Z',
 'LastPlaybackCheckIn': '0001-01-01T00:00:00',
 'NowPlayingItem': None,
 'NowPlayingQueue': None,
 'NowViewingItem': None,
 'PlayState': {'AudioStreamIndex': None,
               'CanSeek': False,
               'IsMuted': False,
               'IsPaused': False,
               'MediaSourceId': None,
               'PlayMethod': None,
               'PositionTicks': None,
               'RepeatMode': 'RepeatNone',
               'SubtitleStreamIndex': None,
               'VolumeLevel': None},
 'PlayableMediaTypes': [],
 'PlaylistItemId': None,
 'RemoteEndPoint': '::ffff:<IP_ADDRESS>',
 'ServerId': '0e23a5ed08c84a159ea7d7aea7ff5e67',
 'SupportedCommands': [],
 'SupportsMediaControl': False,
 'SupportsRemoteControl': False,
 'TranscodingInfo': None,
 'UserId': '00000000-0000-0000-0000-000000000000',
 'UserName': None,
 'UserPrimaryImageTag': None}

Relatively easy fix, but then there's several other places in the codebase that are doing similar things where it assumes if the key exists, it will be a dict.  For example: https://github.com/mezz64/pyEmby/blob/6bb621e4e25bf1b9b0aba2c38b588e68f8816226/pyemby/server.py#L409 and https://github.com/mezz64/pyEmby/blob/6bb621e4e25bf1b9b0aba2c38b588e68f8816226/pyemby/device.py#L124-L125
I'm working through and trying to find each of them right now, should have a PR up in a bit.

Further update: In it's current state basically every function in device.py that involves the NowPlayingItem needs refactored.  eg:
@property
def media_id(self):
    """ Return title currently playing."""
    try:
        return self.session['NowPlayingItem']['Id']
    except KeyError:
        return None

would need to become something along the lines of
@property
def media_id(self):
    """ Return title currently playing."""
    if self.session['NowPlayingItem']:
        return self.session['NowPlayingItem']['Id']
    else:
        return None

Obviously this would be a lot of changes in that file that may not be entirely wanted.  An alternative method would be something that we implemented in our Kodi addon that solves the problem by simply removing all keys with values of None.  https://github.com/jellyfin/jellyfin-kodi/blob/master/jellyfin_kodi/jellyfin/utils.py
@mezz64 let me know which direction you'd rather go with it.

@mcarlton00 I'd prefer the remove None valued keys approach.  Seems much more elegant than checking everywhere.


Welcome message with userlist

We need a second message to be sent out to the user who just logged in saying something like this:
Users online(4): user1, user2, BBBuserwithID3, AAAuserwithID4

Also, make it into a /online command. Idea is to order people by ID, not by name.
After that message, another message with this should come up for the user who just logged in:
Welcome to the server! Use "/help" to see all the commands you have access to.

Note: It should be "/help"

Idea is to order people by ID, not by name.
I'd suggest the following order: Type (Admins first, then Mods, then normal players). They are then sorted by id. (Thus join date)

Note: It should be "/help"
Including quotes?

Including quotes?
Excluding quotes, sorry. Will fix.

Okii, done that


Fix PrimitiveArralList/TransformlingList test file names

The test classes for PrimitiveArrayList and TransformingList follow the '_Test' naming convention but the file names were missing the underscore. It doesn't really matter, but it looked a little weird in the IDE.
Check List:

Fixes # : just cosmetic
Unit tests : just file rename
Javadoc with a code example (on API only) : NA
PR meets the contributing guidelines

Following the contributing guidelines will make it easier for us to review and accept your PR.

Merged, good catch @etellman!


Contract identity issue

We found a problem with the lib api. When we have multiple checks we have an error that is considered an execution error. You can find the error in the source.
We thought that maybe the error was being added to the logs, but the code after the log fixed the issues, so the logs shouldn't exist.
For now the solution will be to add the field identity_suffix  into the check. But I think this should be fixed on SODA side, or using more field to create the ID or not log the error and use the approach with index.
contract.py
logs.error(f"Duplicate check identity '{check_identity}': {other_check.location} and {check_location}")

Another question:
Multiple checks need the field metric to be defined; what is the use for that field? Will that field always be mandatory?

SAS-3274

Goal: The goal is create a better correlation mechanism between checks in the files and checks in Soda Cloud.  We want to impose the least amount of burden on the user and avoid the need for users to configure correlation ids themselves.  But we also want to ensure that users can edit contract files freely and move checks around while still preserving the identity.  Not preserving the identity causes a new check on Soda Cloud being created, loosing the metric history and historic check results.
Proposed solution:
We switch to a new concept of a check id.   A check id is composed of

scope

warehouse_name
schema_name
dataset_name
column_name
check type

check type specific correlation properties

expression_sql hash for metric_expression checks
query_sql hash for metric_query checks

user defined correlation_id

Users then only have to specify a correlation_id if the scope and check type specific correlation properties do not distinctly identify a check in the source YAML file.

In the hellofresh use case, the name is handled as a unique ID, and the user must fill it.
I think the solution 2 can be the best one. We can write our docs explaining that the name is also linked to the soda cloud, and by changing the name, the history will be lost. But I think the users expect that because it is a unique ID for us.

More analysis notes:
Solution 2 would also work best on the Soda Cloud backend.
Based on the correlation properties, contracts lib should create an identity property in the genrated SodaCL using a hash (and not the full property serialized text)
We can offer a renaming check workflow through an name_deprecated property.  That should translate to an identity_deprecated in SodaCL.  Soda Cloud backend work should be planned to support the identity migration.  The changes needed in Soda Cloud backend should not be too big as the data model is matching this strategy.


Changing cosmology in tutorial-hod.ipynb

If I run the unmodified tutorial.ipynb notebook it works fine and runs through to the end, producing all the plots. However, if I change any of the cosmological parameters in the first line of In[5]:
cparam = np.array([0.02225,0.1198,0.6744,3.094,0.9645,-1.])

I get the following error:
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
<ipython-input-5-9026577a174b> in <module>
      1 cparam = np.array([0.02225,0.1198,0.6744,3.094,0.9645,-1.])
----> 2 hod.set_cosmology(cparam)
      3
      4 gparam = {"logMmin":13.13, "sigma_sq":0.22, "logM1": 14.21, "alpha": 1.13, "kappa": 1.25, # HOD parameters
      5            "poff": 0.2, "Roff": 0.1, # off-centering parameters p_off is the fraction of off-centered galaxies. Roff is the typical off-centered scale with respect to R200m.

/usr/local/lib/python3.9/site-packages/dark_emulator/model_hod/hod_interface.py in set_cosmology(self, cparams)
    148             if self.do_linear_correction:
    149                 self.massfunc.set_cosmology(self.cosmo_orig)
--> 150             self.rho_m = (1. - cparams[0][2])*rho_cr
    151             self.R200 = (3*self.Mh/(4.*np.pi*self.rho_m)/200.)**(1./3.)
    152             self.R200_mat = np.tile(self.R200, (len(self.fftlog_2h.k), 1)).transpose()

IndexError: invalid index to scalar variable.

Am I doing something wrong? If so, how do I change the cosmology within the hod part of the emulator?

FYI: Constance just provided me with this work around, changing line 2 to:
hod.set_cosmology(np.atleast_2d(cparam))

works, but seems like a strange thing to have to do.

Yes. The function assumes a 2D array. I will modify it to be more flexible. Thanks for the comments!


[tune/WandbLogger] tune.WandbLogger could use the tune.run name as the default run display name

Weights and Biases version: 0.8.22
Python version: 3.7.5
Operating System: Darwin

Description
It would be nice if tune.WandbLogger would use the tune.run name as the run display name by default. Currently, when run with something like:
from ray import tune
from wandb.ray import WandbLogger

tune.run(
    ExperimentRunner,
    name='this could be the default display name for wandb run',
    loggers=[WandbLogger],
    ...,
)

the display name gets randomly chosen (e.g. fanciful-wildflower-3). I was expecting the tune.run name to get picked up by the logger and be used as the display name by default.

+1 to this

@lavanyashukla, here are examples of tune Trainable and RayTrialExecutor, which should make wandb work with Ray. WandbTrainable enables trial-level logging and WandbExecutor enables experiment-level logging. These are modified versions of the scripts @richardliaw gave me.
from ray.tune import Trainable  # DurableTrainable?
import wandb

class WandbTrainable(Trainable):
    def _setup(self, config, *args, **kwargs):
        if self._use_wandb:
            wandb.init(
                project=config.get("experiment_name"),
                name=self.trial_id,
                id=self.trial_id,
                resume=self.trial_id,
                reinit=True)
            wandb.config.update(self.config)

        return super(WandbTrainable, self)._setup(config, *args, **kwargs)

    def _log_result(self, result, *args, **kwargs):
        if self._use_wandb:
            wandb.log({
                str(key): value
                for key, value in result.items()
                if (value and 'config' not in key
                    and not isinstance(value, str))
            }, step=result["training_iteration"])

        return super(WandbTrainable, self)._setup(result, *args, **kwargs)

from ray.tune.ray_trial_executor import RayTrialExecutor
import wandb

class WandbExecutor(RayTrialExecutor):
    def __init__(self, *args, use_wandb=False, name="WandbExecutor", **kwargs):
        self._use_wandb = use_wandb
        self._name = name
        self._step = 0
        if self._use_wandb:
            wandb.init(
                project=self._name,
                name="executor",
                reinit=True,
                resume=True)
        return super(WandbExecutor, self).__init__(*args, **kwargs)

    def on_step_begin(self, trial_runner, *args, **kwargs):
        if self._use_wandb and trial_runner.get_trials():
            self.log_wandb(trial_runner)
        self._step += 1
        return super(WandbExecutor, self).on_step_begin(
            trial_runner, *args, **kwargs)

    def log_wandb(self, trial_runner):
        """Log experiment level metrics for trials on result.

        WARNING: currently only works for sequential due to total timesteps
        """
        if not hasattr(trial_runner.scheduler_alg, '_mode'):
            return

        if trial_runner.scheduler_alg._mode == "max":
            metric_val = max((
                trial.last_result.get(
                    trial_runner.scheduler_alg._metric, -float('inf'))
                for trial in trial_runner.get_trials()))
        else:
            metric_val = min((
                trial.last_result.get(
                    trial_runner.scheduler_alg._metric, float('inf'))
                for trial in trial_runner.get_trials()))

        wandb.log({
            f"top_{trial_runner.scheduler_alg._metric}": metric_val
        }, step=self._step)


Create architecture diagram

Diagrams as Code! "as code" all the things!!!

![](https://imgflip.com/i/7eibl0


Rename isTypeParameter to isTypeParameterDeclaration

This is technically a public API breaking change, but it's only been there since #16121.
This makes the function more in line with isConstructorDeclaration, isTypePredicateNode, and others. We should use the name of the type, not the name of the syntax kind.
@mhegazy I'd like to get this in for ts2.4.1.

Looks like this breaks the build in services: https://travis-ci.org/Microsoft/TypeScript/jobs/242197871


CustomRender for Shell

Hello Mobile Blazor Team
Shall I ask how to Make CustomRender for Shell in Mobile Blazor which typeof here to use to effect on the Shell
assembly: ExportRenderer(typeof(Xaminals.AppShell), typeof(Xaminals.Droid.MyShellRenderer))]

Hi @SuperRealTeam , I'm not certain, but I think that custom renderers should work exactly the same as they do in any other Xamarin.Forms app. Mobile Blazor Bindings just creates the regular Xamarin.Forms components, so if you have custom renderers, they should get loaded as usual.
If something isn't working, can you show what you tried, what you expected to happen, and what actually happened?


do not pretend to return a value. it's always nil at this point

Line 342 reads as if we could actually return a value for "newKey" but the flow guarantees that it is nil at this point. It mainly a cosmetic change to aid in reading / porting the code.

PS: unless it hides an issue with the flow :)

Merging in by hand, thanks!


[CodeCompletion] Completion for UnresolvedMember via CodeCompletionExpr

Using dummy UnresolvedMemberExpr doesn't give us much benefit. Instead, use    CodeCompletionExpr which is type checked as type variable so we can use CodeCompletionTypeContextAnalyzer to infer context types.
This way, we can eliminate most of special logic for CompletionKind::UnresolvedMember.
rdar://problem/39098974
rdar://problem/42828673

Nice!

I have recently talked to @nathawes about this too!

This pull request introduced a -Wself-assign-overload warning (Status |= Status in ParseStmt.cpp). Was this intentional?

@davezarzycki Ah, thank you! it's not intentional. I'll fix it.

https://github.com/apple/swift/pull/18666


Escaped serial port name on Linux

Type: Bug Report
Describe the bug

OS and Version: Arch Linux
VS Code Version: 1.68.0
Serial Monitor Extension Version: 0.1.0
Target embedded device: N/A, UART bridge: CH340
Other extensions you installed (and if the issue persists after disabling them):
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

Start and stop serial monitoring
The serial port name is displayed as /dev/ttyUSB0

Expected behavior

The serial port name should be displayed as /dev/ttyUSB0
Screenshots

Additional context

Thanks for reporting this! We'll look into this and get it fixed for the next release.

@imi415 Thanks for your feedback! I wanted to let you know that we do not currently have a date in mind, but that we have a fix for this issue ready to ship in the next release.


Quake mode does not consider vertical taskbar width

Windows Terminal version (or Windows build number)
10.0.19042.1165, 1.9.1942.0
Other Software
No response
Steps to reproduce

Make sure you have at least one monitor with a vertical task bar.
Open a Quake mode window on the monitor with the vertical task bar.

Expected Behavior
The Quake mode window should not be partially hidden behind the vertical task bar. It should recognize a vertical task bar is used and resize and reposition accordingly.
Actual Behavior
One side of the terminal looks cut off by the sidebar and it seeems to be positioned behind the task bar.

Good catch! I guess I assumed that the rectangle I was using was the work rect, not the monitor one.


Does not work for Tablets

I am using ScreenUtil and it works most of the time for Phones, however, it does not work for Tablets & iPads. Is this a problem for all or I am not correctly using the package.
Can someone suggest the right way to use design draft? How to get the 360 and 690 for my phone as in example below & if I set this for my phone that I am using while developing the App, will this work for Tablet as well?
//The size of the equipment in the design draft(360,690) ScreenUtil.init(constraints, designSize: Size(360, 690));

Under normal circumstances it is possible,
designSize: the size of the device in the design draft, in dp.
And I updated readme

Under normal circumstances it is possible,
designSize: the size of the device in the design draft, in dp.
And I updated readme

Thanks. If I run code on my dev phone, I get this
I/flutter (24600): Device width dp:411.42857142857144dp
I/flutter (24600): Device height dp:820.5714285714286dp
I/flutter (24600): Device pixel density:2.625
I/flutter (24600): Bottom safe zone distance dp:0.0dp
I/flutter (24600): Status bar height dp:33.904761904761905dp
I/flutter (24600): The ratio of actual width to UI design:0.380952380952381
I/flutter (24600): The ratio of actual height to UI design:0.38095238095238093
I/flutter (24600): System font scaling:1.0
I/flutter (24600): 0.5 times the screen width:205.71428571428572dp
I/flutter (24600): 0.5 times the screen height:410.2857142857143dp

So should I set
designSize: Size(411, 820)
If I do and develop my App, i.e. set Font/Widget height/width/Padding and so on accordingly which will work perfectly for my dev phone - will this work on Tablet of different size without any problem?

Thanks. If I run code on my dev phone, I get this
I/flutter (24600): Device width dp:411.42857142857144dp
I/flutter (24600): Device height dp:820.5714285714286dp
I/flutter (24600): Device pixel density:2.625
I/flutter (24600): Bottom safe zone distance dp:0.0dp
I/flutter (24600): Status bar height dp:33.904761904761905dp
I/flutter (24600): The ratio of actual width to UI design:0.380952380952381
I/flutter (24600): The ratio of actual height to UI design:0.38095238095238093
I/flutter (24600): System font scaling:1.0
I/flutter (24600): 0.5 times the screen width:205.71428571428572dp
I/flutter (24600): 0.5 times the screen height:410.2857142857143dp

So should I set
designSize: Size(411, 820)
If I do and develop my App, i.e. set Font/Widget height/width/Padding and so on accordingly which will work perfectly for my dev phone - will this work on Tablet of different size without any problem?

no no no no
When developing, there is a UI design draft，The phone size of the design draft is what we need to fill in.
If there is no design draft for the time being, please assume a mobile phone size, such as Size(320,480)

no no no no
When developing, there is a UI design draft，The phone size of the design draft is what we need to fill in.
If there is no design draft for the time being, please assume a mobile phone size, such as Size(320,480)

I am confused now :-( Sorry.
With above information, what do you suggest I should set below?
Earlier I had set Size(1080, 2154), & it's kind of worked, but well not on Tablets
    return ScreenUtilInit(
      designSize: Size(1080, 2154),
      allowFontScaling: false,

I am confused now :-( Sorry.
With above information, what do you suggest I should set below?
Earlier I had set Size(1080, 2154), & it's kind of worked, but well not on Tablets
    return ScreenUtilInit(
      designSize: Size(1080, 2154),
      allowFontScaling: false,

So, now , do you have a UI design?

So, now , do you have a UI design?

What does that mean? Does that mean a device/dimensions that we'll use to design/develop an App the Development cycle?

What does that mean? Does that mean a device/dimensions that we'll use to design/develop an App the Development cycle?

You don’t need to fill in the size of the actual device, just fill in the developed benchmark size.
But
If you are based on the size of this phone throughout the process, you can fill in this way.

You don’t need to fill in the size of the actual device, just fill in the developed benchmark size.
But
If you are based on the size of this phone throughout the process, you can fill in this way.

Okay. Just to make sure I understood (and probably help beginners like me) correctly - If I am going to use the below device through out the dev cycle, I can set Design Size as designSize: Size(411, 820) & continue the development, but if the design draft is not known - I can set something like Size(320,480) or (360,720) as you suggested earlier and continue the App development and make App responsive.
I/flutter (24600): Device width dp:411.42857142857144dp
I/flutter (24600): Device height dp:820.5714285714286dp

Couple more question: If I continue with Size(320,480) when the design draft is not known, at any point in time, do I need to re-do the UI to match new Size OR
Because I have already developed the Responsive UI based on 320,480, it will work for other sizes as well.
Please correct me if I am wrong.

Okay. Just to make sure I understood (and probably help beginners like me) correctly - If I am going to use the below device through out the dev cycle, I can set Design Size as designSize: Size(411, 820) & continue the development, but if the design draft is not known - I can set something like Size(320,480) or (360,720) as you suggested earlier and continue the App development and make App responsive.
I/flutter (24600): Device width dp:411.42857142857144dp
I/flutter (24600): Device height dp:820.5714285714286dp

Couple more question: If I continue with Size(320,480) when the design draft is not known, at any point in time, do I need to re-do the UI to match new Size OR
Because I have already developed the Responsive UI based on 320,480, it will work for other sizes as well.
Please correct me if I am wrong.

The size of the design sketch is not important. The important thing is to try not to change it, otherwise there will be UI abnormalities, and if it is not changed, the initial value will be no problem.

The size of the design sketch is not important. The important thing is to try not to change it, otherwise there will be UI abnormalities, and if it is not changed, the initial value will be no problem.

Okay. Got it. When I had first set it, I had set it as Size(1080, 2154) since then there's no change. So why do you think in this case it doesn't work very well on Tablet / iPad?

Okay. Got it. When I had first set it, I had set it as Size(1080, 2154) since then there's no change. So why do you think in this case it doesn't work very well on Tablet / iPad?

Which widget does not perform well, can you show the diagram and code?
Generally speaking, yes

Which widget does not perform well, can you show the diagram and code?
Generally speaking, yes

Okay, I will try to share more data related to this. Thanks a lot for your help with this so far!

Okay, I will try to share more data related to this. Thanks a lot for your help with this so far!

Welcome to feedback more questions in time

Welcome to feedback more questions in time


How to disable a snippet?

Say I don't like Emacs to insert #include "xx.h" when I create xx.cpp. What are the ways to disable the behavior?

See the file-templates module.
;; add to config.el
(set-file-template! 'c++-mode :ignore t)


Documentation

Is there any documentation that list for example, all the functions and variables in jspreadsheet-ce? This is a really great project, having this would be very helpful

https://bossanova.uk/jspreadsheet/v4/docs/quick-reference


[Help]How to use the AutoLayoutSizeSource with a cell view from nib?

func albumProvider(withInsets insets: UIEdgeInsets = .zero) -> BasicProvider<Album, AlbumTableCell> {
        let albumProvider = BasicProvider<Album, AlbumTableCell>(
            identifier: nil,
            dataSource: RealmResultsDataSource(data: self.albumEntry),
            viewSource: ClosureViewSource<Album, AlbumTableCell>(viewGenerator: { (_, index) -> AlbumTableCell in
                return AlbumTableCell.loadFromNib(named: "AlbumTableCell", bundle: nil) as! AlbumTableCell
            }, viewUpdater: { [weak self] (albumV: AlbumTableCell, data, index) in
                albumV.backgroundColor = .white
                albumV.numberOfLines = 6
                albumV.data = data.album
            }),
            sizeSource: AutoLayoutSizeSource(dummyView: AlbumTableCell.self, viewUpdater: { (albumV, data, idx) in
                // Need to do anything?
            }),
            layout: FlowLayout(lineSpacing: onePixalLineHeight, interitemSpacing: 0, justifyContent: .start, alignItems: .start, alignContent: .start),
            animator: nil,
            tapHandler: { [weak self] (tapContext) in
                self?.toAlbumDetail(album: tapContext.data.album!)
        })
        return albumProvider
    }

I have a proposal here to make a second constructor which will accept a pointer to the view constructor method that you need.
public init(dummyView: (() -> View)

Then you just create view like this:
self.dummyView = dummyView()

And then use it like this:
let view: (() -> MyView) = {
  return UIView.loadFromNib(MyView.self)
}

let xibSizeSource = XibSizeSource(dummyView: view...)


[DOC] Add sub-page for performance/benchmark runs in the main page for remote store feature

What do you want to do?

[x] Request a change to existing documentation
[x] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
## Benchmarks

We have run benchmark for multiple workload options available within [opensearch benchmark](https://opensearch.org/docs/latest/benchmark/index/) tool and multiple cluster configuration. Below, we have summarised the benchmark results for [StackOverflow](https://github.com/opensearch-project/opensearch-benchmark-workloads/tree/main/so), [HTTP logs](https://github.com/opensearch-project/opensearch-benchmark-workloads/tree/main/http_logs) and [NYC taxis](https://github.com/opensearch-project/opensearch-benchmark-workloads/tree/main/nyc_taxis), for multiple configurations of bulk indexing client which simulates concurrency of requests.

>Your results may vary based on the cluster topology, hardware used, shard count, and merge settings.

### Cluster, shard & test configuration

* 3 Nodes with data, ingest, master role
* Node instance: AWS EC2 r6g.xlarge
* OpenSearch benchmark host: Single AWS EC2 m5.2xlarge
* Shard configuration: 3 shards with 1 replica
* repository-s3 plugin installed with default S3 settings

### StackOverflow

The following table lists the benchmarking results for the `so` workload with the mentioned configuration and remote translog buffer interval of 250ms.

|	|	|8 bulk indexing clients (Default)	|16 bulk indexing clients	|24 bulk indexing clients	|
|---	|---	|---	|---	|---	|
|	|	|Document Replication	|remote enabled	|Percent difference	|Document Replication	|remote enabled	|Percent difference	|Document Replication	|remote enabled	|Percent difference	|
|Indexing Throughput	|Mean	|29582.5	|40667.4	|37.47	|31154.9	|47862.3	|53.63	|31777.2	|51123.2	|60.88	|
|P50	|28915.4	|40343.4	|39.52	|30406.4	|47472.5	|56.13	|30852.1	|50547.2	|63.84	|
|Indexing Latency	|P90	|1716.34	|1469.5	|-14.38	|3709.77	|2799.82	|-24.53	|5768.68	|3794.13	|-34.23	|

### HTTP logs

The following table lists the benchmarking results for the `http_logs` workload with the mentioned configuration and remote translog buffer interval of 200ms.

|	|	|8 bulk indexing clients (Default)	|16 bulk indexing clients	|24 bulk indexing clients	|
|---	|---	|---	|---	|---	|
|	|	|Document Replication	|remote enabled	|Percent difference	|Document Replication	|remote enabled	|Percent difference	|Document Replication	|remote enabled	|Percent difference	|
|Indexing Throughput	|Mean	|149062	|82198.7	|-44.86	|134696	|148749	|10.43	|133050	|197239	|48.24	|
|P50	|148123	|81656.1	|-44.87	|133591	|148859	|11.43	|132872	|197455	|48.61	|
|Indexing Latency	|P90	|327.011	|610.036	|86.55	|751.705	|669.073	|-10.99	|1145.19	|817.185	|-28.64	|

### NYC taxis

The following table lists the benchmarking results for the `http_logs` workload with the mentioned configuration and remote translog buffer interval of 250ms.

|	|	|8 bulk indexing clients (Default)	|16 bulk indexing clients	|24 bulk indexing clients	|
|---	|---	|---	|---	|---	|
|	|	|Document Replication	|remote enabled	|Percent difference	|Document Replication	|remote enabled	|Percent difference	|Document Replication	|remote enabled	|Percent difference	|
|Indexing Throughput	|Mean	|93383.9	|94186.1	|0.86	|91624.8	|125770	|37.27	|93627.7	|132006	|40.99	|
|P50	|91645.1	|93906.7	|2.47	|89659.8	|125443	|39.91	|91120.3	|132166	|45.05	|
|Indexing Latency	|P90	|995.217	|1014.01	|1.89	|2236.33	|1750.06	|-21.74	|3353.45	|2472	|-26.28	|

We see consistent gains for cases where the indexing latency is more than the average remote upload time. When we increase the bulk indexing clients, remote enabled configuration provides indexing throughput gains upto 60-65%. Detailed results are present on GitHub - https://github.com/opensearch-project/OpenSearch/issues/9790

What other resources are available? Provide links to related issues, POCs, steps for testing, etc.

@Naarcha-AWS - Can we close this?

Closing this as this is done already.


Added more ideas and made readme more lively

I am new to Hacktoberfest and was trying to contribute in the simplest way possible. Hopefully I will be of more help later as well.

Thanks!


Question: WinUI 3 and external libraries

In our currently developed project (UWP) we are using some external libraries, eg.: ReactiveUI, Win2D, CompositionProToolkit, Microsoft.Toolkit.*
We are very eager to upgrade to WinUI 3 when the time has come and the project has matured enough.
But the question is, there would have to be a special WinUI version for all of our dependencies, right? This could mean a fair amount of extra work being that behind those dependencies of course is another tree of (maybe UWP related) dependencies, which would have to be converted to WinUI 3 themselfes.
Am I right in my assesment of the situation and what are the team's thoughts on this?

https://github.com/microsoft/microsoft-ui-xaml/issues/717

Am I right in my assesment of the situation and what are the team's thoughts on this?

Yes, that's correct for where we're at with WinUI 3. We'd love your feedback on the threads that @kmgallahan linked. Especially if you have ideas about how to make this easier for scenarios like this. :)


Bug? selecting a column for GroupBy

I am reading the book "Python for Data Analysis" and try some examples. But I found something strange.
On page 256, it says:
Quote:
Indexing a GroupBy object created from a DataFrame with a column name or array of column names has the effect of selecting those columns for aggregation. This means that:
df.groupby('key1')['data1']
df.groupby('key1')[['data2']]

are syntactic sugar for:
df['data1'].groupby(df['key1'])
df[['data2']].groupby(df['key1'])

I think this makes sense, but when I tried a testing code, it didn't work as expected.
df.groupby('key1')[['data2']]

Here, it still returns the whole DataFrame. It does not perform the same as
df[['data2']].groupby(df['key1'])

I used "for" loop to iterate the object, maybe this is the reason?
 Iterating just give you effectively a split of the frame. See the docs here


 Last update dcc6811...b8fc413

or @mfussenegger ?

@mfussenegger fixup pushed


Switch codebase to typescript

Is your feature request related to a problem? Please describe.
Switching to typescript or adding type hints can increase the usability in the long run.
Describe the solution you'd like
Convert all the files one by one

Hi @yashshah1
thank you for opening this feature request, but at the moment there's no plan to switch the codebase to typescript.
But feel free to open a PR to add the type definitions if you want

I'm closing this


Adds a flag to hub browse to choose whether to open the current or …

…default branch.
This is a start for #1107 to let the user control a bit more which page  is open when running "hub browse" in a tracked branch.

Looks good, but I'm reluctant to add a new flag that doesn't seem that it will get much use. What's the use-case for this?

I agree, I don't find much value in it. I was merely following the suggestion in #1107.
"If you want the default behavior of hub browse to stay unchanged, we could provide switches like -p for opening up the linked pull request, -b for the branch etc."
I'm just trying to help out here, so if you don't like it, just push back in #1107 and let's drop this PR.

I want this feature to see the code reviews on the pull request related with the branch I'm working on.

@npcode That's why I ended up doing in bash .aliases:
echo $(hub browse -u)| sed "s/tree/pulls/g" | xargs open
I didn't find any way to use the hub browse -- [sub-page] to replace the "tree" segment of the URL for "pulls" given the current directory.


Frontier [testing]: Run distributed pytorch benchmark

Use built conda env (#37)
Benchmark from discover benchmarking
MNIST benchmark
Develop slurm scripts for Frontier slurm queue
Benchmark w/ deepspeed

https://github.com/microsoft/DeepSpeedExamples/tree/master/benchmarks/communication

export MIOPEN_USER_DB_PATH="/mnt/bb/${USER}/my-miopen-cache"
export MIOPEN_CUSTOM_CACHE_DIR=${MIOPEN_USER_DB_PATH}
rm -rf ${MIOPEN_USER_DB_PATH}
mkdir -p ${MIOPEN_USER_DB_PATH}

rocm needs this to work with rccl and msccl

Passed these benchmark tests


Project Week 1

Here is my part of the video:
https://www.loom.com/share/418f00c2862741788ac6f8f61ff6fd23

Your video is quite long and should be more brief. Your group was given a 4 minute max time and you spoke for more than half of it. Also in regards to your group, I will grade this separately but will not do that for NotM, you have to do a better job at collaborating with your group to hit deadlines. That being said you do seem to have a good direction of where you want your code to go. Hopefully this planning will make it easier for you when you begin your coding.
Score 4.3


How to upgrade from sync openssl to asynchronous openssl

Can you provide a simple demo for reference?
How to upgrade from sync openssl to asynchronous openssl

Please clarify your question:

What OpenSSL version are you using?
Do you have an asynchronous capable engine?
By "asynchronous openssl" do you mean you want to do asynchronous SSL/TLS, or asynchronous crypto?

I want to use openssl 1.1.1 and intel QAT Engine for asynchronous ssl/tls

The main differences between normal SSL/TLS operations and async SSL/TLS is that you need to:

Load your async engine e.g. typically via a call to ENGINE_by_id()
https://www.openssl.org/docs/man1.1.1/man3/ENGINE_by_id.html

use SSL_MODE_ASYNC and be prepared to handle SSL_ERROR_WANT_ASYNC in any calls to SSL_get_error().
https://www.openssl.org/docs/man1.1.1/man3/SSL_CTX_set_mode.html
https://www.openssl.org/docs/man1.1.1/man3/SSL_get_error.html

Optionally you may want to check whether the async engine is ready to resume an operation. You can use SSL_get_all_async_fds() for that in conjunction with a call such as select or poll.
https://www.openssl.org/docs/man1.1.1/man3/SSL_get_all_async_fds.html


Remux outdated

Remux is currently on version 0.1.7 (you can see for yourself at https://github.com/rmkit-dev/rmkit/tree/master/src/remux). This is a current stable release, however opkg has this version neither at the stable nor testing repo...

next release will be 0.1.7, i haven't made the PR on toltec yet

next release will be 0.1.7, i haven't made the PR on toltec yet

Ah I see, thx

Ah I see, thx

you can grab the latest remux binary from build.rmkit.dev/master/latest/files in the meanwhile

you can grab the latest remux binary from build.rmkit.dev/master/latest/files in the meanwhile

Thanks -- Will do that actually.

Thanks -- Will do that actually.


Request for ability to focus the repo-map on code that matters

In my project I have several parts which should be ignored by the universal ctags indexing, meaning the repo-map should not spend any attention on those parts. A good example is node_modules which is huge and which will just be a distraction. Essentially I want to be able to pass a parameter to aider telling it which folder to build up the repo-map from.

Thanks for trying aider!
Aider only uses files which have been checked into the git repo in the repo map. It would be unusual to add node_modules to your git repo.
So I guess I am wondering what makes you think the node_modules files are being included in the repo map?
You can see the map by running with the --show-repo-map switch.

I see, make sense.
It was package.json.lock that distracted me I think.
But then I have the desire to focus the attention on part of the code base
and due to limited amounts of token, it seems useful to be able to control
the attention.
man. 31. jul. 2023 kl. 13.16 skrev paul-gauthier @.***>:

Thanks for trying aider!
Aider only uses files which have been checked into the git repo in the
repo map. It would be unusual to add node_modules to your git repo.
So I guess I am wondering what makes you think the node_modules files are
being included in the repo map?
You can see the map by running with the --show-repo-map switch.
—
Reply to this email directly, view it on GitHub
https://github.com/paul-gauthier/aider/issues/160#issuecomment-1658169233,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAERNXDK3PNMJ77YHZHULTXS6HXXANCNFSM6AAAAAA25AG7TQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Vh Nikola Schou
2068 4899

if you run aider with --map-tokens  set to 0 it will disable the repo map and save you some tokens - then you have complete control on focusing attention based on what you add to the context.
the repo map is only turned on when using gpt-4 so if you use aider -3 you'll get a 16k window and no repo map, but also a less powerful coder.

Ok good
tir. 1. aug. 2023 kl. 22.59 skrev Joshua Vial @.***>:

if you run aider with --map-tokens set to 0 it will disable the repo map
and save you some tokens - then you have complete control on focusing
attention based on what you add to the context.
the repo map is only turned on when using gpt-4 so if you use aider -3
you'll get a 16k window and no repo map, but also a less powerful coder.
—
Reply to this email directly, view it on GitHub
https://github.com/paul-gauthier/aider/issues/160#issuecomment-1661084374,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAERNUTAD32MT6VLFUYRM3XTFUZXANCNFSM6AAAAAA25AG7TQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Vh Nikola Schou
2068 4899

Also, you can run aider with --show-repo-map to see which parts of your repository are being included in the map. Please note though, that aider dynamically adjusts the repo map depending on which files you have added to the chat. So provide the files you want to chat about on the command line along with --show-repo-map.

I'm going to close this issue, since it's very close to existing issue #92. Let's continue any discussion over there.


Edit monitor environment object in unit1 handson

added render_mode='rgb_array' in the first time a monitoring environment gets initiated, making it ready to be pushed to the hub

Thanks I merge it


[WIP] GaiaX CI Test

What do these changes do?

Related issue number

Fixes

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.4 out of 5 committers have signed the CLA.:white_check_mark: BingqingLyu:white_check_mark: youyangy:white_check_mark: shirly121:white_check_mark: bmmcq:x: longbin.lailblongbin.lailb seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Open another pr #812


New-DbaAvailabilityGroup should enable and start the AlwaysOn_health extended event session

Summary of new feature
Quote from the dokumentation:
"The alwayson_health extended events session is created automatically when you create the availability group, and captures a subset of the availability group related events. This session is preconfigured as a useful and convenient tool to help you get started quickly while troubleshooting an availability group. The Create Availability Group Wizard automatically starts the session on every participating availability replica configured in the wizard."
So I think it would be nice to do this configuration with New-DbaAvailabilityGroup
Proposed technical details (if applicable)
I use this line of code to do this:
Get-DbaXESession -SqlInstance $SqlInstances -Session AlwaysOn_health | ForEach-Object -Process { $_.AutoStart = $true ; $_.Alter() ; $_ | Start-DbaXESession }

Latest version of dbatools as of writing
I currently use Version 1.0.113.

This would be something tied to a switch but will not be default to enable.
We are simply performing what SMO does to create an AG. An additional configuration of this nature should be done via parameter switch, with the default action of false not enabling that session.
If the session does not exists, it should ignore and just through warning stating as much.


⚠ Science Web Services - QState has degraded performance

In cce942c, Science Web Services - QState (https://sws.labbricks.com/qstate/) experienced degraded performance:

HTTP code: 523
Response time: 1360 ms

Resolved: Science Web Services - QState performance has improved in 8806b83 after 12 minutes.


Controlled sources results

Describe the bug
I'm not sure if controlled sources are fully supported already. If they are, the semi-symbolic DC analysis returns strange results. The values aren't fully evaluated, and there are odd keys like "'v(*shortVsense)': -100.000".
To Reproduce
from symcirc import *

netlist = """
*GEEC circuit number #1850
V1 ctrl1 0 DC 500m AC 0 0
Vsense 2 1 DC 0 AC 0 0
R1 1 0 1k
R2 3 0 1k
E1 ctrl2 0 ctrl1 0 2
G1 2 0 ctrl2 0 100m
F1 3 0 Vsense -5
H1 7 0 Vsense 100
"""
circuit = AnalyseCircuit(netlist, "DC", symbolic=False, method="tableau")
results = circuit.node_voltages()
results.update(circuit.component_values("all"))

print(results)

Output
{'v(ctrl1)': 0.500000, 'v(2)': -100.000, 'v(1)': -100.000, 'v(3)': -50.0*v(7) - 500.0, 'v(ctrl2)': 1.00000, 'v(7)': None, 'v(*shortVsense)': -100.000, 'v(V1)': 0.500000, 'i(V1)': 0, 'v(Vsense)': 0, 'i(Vsense)': -0.100000, 'v(R1)': -100.000, 'i(R1)': -0.100000, 'v(R2)': -50.0*v(7) - 500.0, 'i(R2)': -0.05*v(7) - 0.5, 'v(E1)': 1.00000, 'i(E1)': 0, 'v(G1)': -100.000, 'i(G1)': 0.100000, 'v(F1)': -50.0*v(7) - 500.0,
'i(F1)': 0.05*v(7) + 0.5, 'v(H1)': v(7), 'i(H1)': 0}

Sorry for the long delay. I'm working on identifying the source of this issue, it's probably an error in the CCVS implementation.

Ok, so I think I've located two issue sources:

*short<source_name> is a virtual node generated for the tableau method matrix, it is supposed to be filtered out of the results after analysis, this will be fixed soon
There's a bug which happens when multiple current controlled sources are assigned the same Vsense. The results are correct after adding a second Vsense. This bug should therefore affect only circuits where multiple current controlled sources (CCVS, CCCS) have the same Vsense.

Workaround before fix:
netlist = """ *GEEC circuit number #1850 G1 2 0 ctrl2 0 100m F2 3 0 Vsense1 -5 E1 ctrl2 0 ctrl1 0 2 H2 7 0 Vsense2 100 V1 ctrl1 0 dc 500m R1 1 0 1k R2 3 0 1k Vsense2 2 4 dc 0 Vsense1 4 1 dc 0 """

@OndrejBukovsky Update: I've finally managed to solve this issue, it took more time than I initially expected. I will run some more tests and then publish a new version of SymCirc on PyPi. (v0.1.3)


xml2kvp: sibling_hash are unique across records, do not need to be

sibling_hashs were introduced to faciliate the roundtripping between tabular data (CSV) and XML.
For example, the following 6-char alphanumerics indicate which of these are siblings:
 'mods|mods(0b4a20)___mods|location(b9af27)___mods|url(7d1c82)___@access=preview': 'http://digital.library.wayne.edu/item/wayne:Adventur1860b50081974/thumbnail',
 'mods|mods(0b4a20)___mods|location(b9af27)___mods|url(ac6147)___@usage=primary': 'http://digital.library.wayne.edu/item/wayne:Adventur1860b50081974',

But, applying these sibling hashes makes for columns that are unique to the point of not being workable.  But, they don't need to be that unique.
Sequential numbering might help with this, as opposed to 6-char uuids?

Closing - deemed to difficult to implement right now while maintaing roundtrip-ability of colums to XML.


Migrate navigation to typesafe routes

See. https://github.com/bitcoindevkit/bdk-kotlin-example-wallet/commit/7aab086f505999d1df4357eb02eba06366471924 for an example of this type of migration.

Fixed with 7fbe1a0f8c1808983ac5db4ad976b46e6a36633e.


data_failure

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
on my computer there is a bug that says:
could not convert string to float: 'Germany'
could not convert string to float: 'France'

It's amazing ,I never meet a question like that.

First check ur code again, perhaps you miss something like LabelEncoder()...
Second, maybe you can find more at https://stackoverflow.com/questions/30384995/randomforestclassfier-fit-valueerror-could-not-convert-string-to-float
Hope this would help :)


[1.3.6-prepare][API] Improvement the duration field modify the display to a time-based minute-second format (e.g. 1d 10h 20m 1s) #4752

+1

pr #4752
issue #4751


RMT peripheral missing for ESP32-S3

The SVD file for the ESP32-S3 is missing the RMT peripheral.

Thanks for letting me know. I'll try to hunt down the source file for this peripheral and get a new SVD out as soon as possible.

Thanks a lot! This is a perfect starting point for patching rather than having to start from scratch.

A new SVD containing the RMT peripheral has been pushed to main.


Normalised the reads count

Hi,
I would like to quantify the gene expression in treated and untreated condition. The total reads count is low in my treated condition. So I am wondering would the nanocount take in consideration the difference in the total reads count or do I need to down samples the total reads count in both conditions? Please see the attached

Thanks in advance

Hi, keep in mind NanoCount is for transcript expression not gene expression. However, you can still quantify the transcripts in your samples. The output contains a column called 'tpm' which is normalised (transcripts per million) but if you're hoping to run a differential expression analysis, please use the 'est_count' column for the counts as most differential expression programs do their own normalisation for depth. Hope this helps.


Support TLS for lakeFS

Close #5321

@nopcoder I think we should add documentation on how to configure it...

@nopcoder I think we should add documentation on how to configure it...

it is best practice to have termination when possible


"zhmc ldap delete": fails with TypeError #460

Changes made in file ../zhmccli/_cmd_ldap_server__definations.py _ldadef_delete() function  was updated
parameters name LDAPSD updated instead of ldapdef

I first thought this change was ok to go, but pylint fails on the argument name:
zhmccli/_cmd_ldap_server_definition.py:217:28: C0103: Argument name "LDAPSD" doesn't conform to '([a-z][a-z0-9_]{0,28}[a-z0-9]|[a-z][a-zA-Z0-9]{1,29})_{0,1}$' pattern (invalid-name)

Renaming the parameter to lower case ldapsd should address this. The click.argument() name can stay upper case. See the other uses of click.argument() in the project, they are all like that.
Can you please do this @samirgorai ?
Also, you can run a local test by running make all.


Confusion about displaying error due to using images that are attached to other pages

Environment
Host
Growi.cloud
Client

item
version

OS
Windows

browser
Chrome

How to reproduce? (再現手順)

See "What happens?".

What happens? (症状)
If you paste an image from a group-restricted page into another public page (perhaps as a Markdown source), the image may not be displayed because it belongs to the group-restricted page.
This is also a case where the images are displayed inconsistently depending on the person viewing the public page. The images are displayed fine if the person viewing the public page is a group member of the other page, but not if the person is not a member of that group.
The matter is that the images are displayed differently depending on the person who confirmed the above.
What is the expected result? (期待される動作)

See my suggestion below.

Note (Suggestion)
I hope that one of the following improvements will avoid confusion.
A. Add a function that duplicates and re-registers all images that refer to images on external pages as attached images on the page in question so that users who belong to the appropriate groups or administrators can use it from the settings of each page.
B. Make the JSON error messages when accessing images a bit more friendly.
For B, the current error message is:
{"ok":false, "error": "Forbidden to access the attachment '5fbf8ae98a53b50048774d34'"}

My suggestion is like:
{"ok":false, "error": "Forbidden to access the attachment '5fbf8ae98a53b50048774d34'. This attachment is belong to the page /foo/boo ."}

or
{"ok":false, "error": "Forbidden to access the attachment '5fbf8ae98a53b50048774d34'. This attachment might belong to other pages."}

@emadurandal
For the minimum fix at the moment, I added the description in the suggestion B to API error message.
https://github.com/weseek/growi/commit/900757cc12922b3360be413a3a83f12638f8bdca
Wait the next release v4.2.10. Thanks.


[Parquet] Null count in list arrays incorrect?

The minimal example below reproduces the point:

import pyarrow as pa  # pyarrow==7
import pyarrow.parquet

path = "bla.parquet"
data = [[0, 1], None, [2, None, 3], [4, 5, 6], [], [7, 8, 9], None, [10]]

t = pa.table(
    [pa.array(data)],
    schema=pa.schema([pa.field("int64", pa.list_(pa.int64()), nullable=True)]),
)

pyarrow.parquet.write_table(t,path)

parquet_file = pyarrow.parquet.ParquetFile(path)
nulls = parquet_file.metadata.row_group(0).column(0).statistics.null_count
assert nulls == 1, nulls

the null count should be equal to 1 because the inner array only has one null value (right?)
Reporter: Jorge Leitão / @jorgecarleitao
Note: This issue was originally created as ARROW-16299. Please see the migration documentation for further details.

I guess this is expected, because nulls doesn't represent null-value here.
  /** Number of NULL values, in this data page.
      Number of non-null = num_values - num_nulls which is also the number of values in the data section **/
  2: required i32 num_nulls

In fact, it means num-of-levels - num-of-values


[FEATURE] Timeout component export future based on the provided max_execution_time

Affected project(s)

[ ] documentation
[ ] examples
[X] wasmCloud host
[ ] wasmCloud CLI (wash)
[ ] wasmCloud dashboard UI (washboard)
[ ] capability providers
[ ] provider bindgen
[ ] control interface client
[ ] other / not sure

Is your feature request related to a problem? Please describe.
We currently have tuning for the max_execution_time parameter for the host since #2439, but we don't currently ensure that we are timing out the future for handling a request if it takes longer than the max execution time.
Describe the solution you'd like
The future at https://github.com/wasmCloud/wasmCloud/blob/main/crates/host/src/wasmbus/mod.rs#L1216 should be wrapped in a tokio::time::timeout to ensure that if it takes longer than the requested max execution time, we stop polling that future & shutdown the component.
We should return a reasonable error if a component handler takes longer than the max execution time to execute.
Describe alternatives you've considered
No response
Additional context
No response

Hey @brooksmtownsend , Can I pick this issue up ?


[feature request] copying of VRC Station components

this tool has been a god send for copying components, however i have noticed it will not copy over VRC Station components on avatars, could this be added?
https://docs.vrchat.com/docs/vrc_station

Added in the latest WIP release. Check the discord to get it.
Will post to a main release here later.


liana: 5.0 -> 6.0

Description of changes
Diff: https://github.com/wizardsardine/liana/compare/v5.0...v6.0
This update will also address the package failing to build with Rust 1.80 (#332957).
Still broken on aarch64-linux.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Hmm. I've found a few issues I'm going to address first.

Needed some new runtime libs


Waveforms on audio previews

Description of the new feature / enhancement
Having waveforms showing audio amplitude when previewing audio type files. Similar on the waveform displays on Audacity or Adobe Audition, but in a quick peek.
Scenario when this would be used?
For easier search for audio to mix down in a video or sound design.
Supporting information
A lot of audio/video editing softwares don't have previews when importing in audio files, making it hard to choose the right audio without having to go through the files.

/dup #33283


Upgrade ember-cli-sass to 4.0.0-beta.4

This fixes a dependency error described here
https://github.com/aexmachina/ember-cli-sass/issues/45

Looks like this is failing again but another fix is in place... I'll create a PR with the fix documented in the latest update to the issue.


Impossible to make a pivot with a created_at column but no updated_at

Laravel Version: 5.6.3
PHP Version: 7.2.2

Description:
With a normal model it's possible to define CREATED_AT but set UPDATED_AT to null when there is only the created_at column and no updated_at column in my table. But in a many to many relation with a pivot it's impossible to use withTimestamps with only one parameter to have the same comportement.
Since there is no setter for pivotCreatedAt in \Illuminate\Database\Eloquent\Relations\BelongsToMany I had to use this workaround to make my usecase work:
$relation = $this->belongsToMany(User::class, 'table_name')
	->withPivot('pivot_col');

$withCreatedAt = function(string $createdAt)
{
	$this->pivotCreatedAt = $createdAt;

	return $this->withPivot($this->createdAt());
};

$withCreatedAt->call($relation, 'date_added');

return $relation;

Do you think it would be possible to make this easier by for example create the methodes withCreatedAt et withUpdatedAt ?

For suggestions and feature requests please use the https://github.com/laravel/internals repo.


Syntax Error: Unexpected token R

C:\>npm-windows-upgrade
npm-windows-upgrade v3.1.0
undefined:1
Run `auto` to initialize
^

SyntaxError: Unexpected token R
    at Object.parse (native)
    at C:\Users\xxx\AppData\Roaming\npm\node_modules\npm-windows-upgrade\lib\
versions.js:39:20
    at ChildProcess.exithandler (child_process.js:204:7)
    at emitTwo (events.js:100:13)
    at ChildProcess.emit (events.js:185:7)
    at maybeClose (internal/child_process.js:827:16)
    at Socket.<anonymous> (internal/child_process.js:319:11)
    at emitOne (events.js:90:13)
    at Socket.emit (events.js:182:7)
    at Pipe._onclose (net.js:471:12)

The pertinent line in versions.js is:
exec('npm view npm versions --json', function (err, stdout) {

and npm view npm versions --json returns:
C:\temp>npm view npm versions --json
[
  "1.1.25",
  "1.1.70",
  "1.1.71",
  "1.2.19",
  "1.2.20",
  "1.2.21",
  "1.2.22",
  "1.2.23",
  "1.2.24",
  "1.2.25",
  "1.2.27",
  "1.2.28",
  "1.2.30",
  "1.2.31",
  "1.2.32",
  "1.2.8000",
  "1.3.0",
... all the way to
  "3.10.9"
]

OK, it seems I had an AutoRun command (HKEY_CURRENT_USER\Software\Microsoft\Command Processor\AutoRun) which was executed every time a new command was entered. It output a message beginning with "R".


memory leak?

The memory allocated by FastLock does not appear to be released?

Thanks for reporting this, good catch!


[Logging] Error "unable to close channel" although channel is actually successfully closed

Background
Following the tutorial on simnet, I'm trying to close the Alice <-> Bob channel from Alice's side.
Alice manages to close the channel (A subsequent listchannels afterwards does not list the channel).
However Bob has an ERR log with
unable to respond to remote close msg: unable to close channel, ChannelID(6bb6bb69537f1cbf75db75eec97facbef4ccc19729bd759ef782036000f38090) is unknown
I believe this ERR log should perhaps be a WARN or INFO instead, because the channel is indeed successfully closed (ERR logs should indicate ERROR conditions, not normal flow).
Your environment

lnd version 0.4.1-beta commit=097fd50747356a39593e5ac7625bf0b0fb1305a1
Windows 10
btcd compiled from source today from roasbeef branch commit a03db407e40d3b66ea29984263bbc8bf4d2f04c4

It's a benign log statement, just that both sides echo back the last message during the close negotiation process. Typically the other side has already ack'd and then accepted the last closing message and cleaned up the state. As a result, the state machine responsible for processing this message has already exited.

So, if it's a benign message ... shouldn't it be something other than ERR?

Yep, not disagreeing!


External Deployments Operator IAM Role

create external deployments iam role using new resource under existing irsa module

@dmcwhorter-ddl can you add a plan test here with external_deployments_operator enabled


Finer granularity with LOC Graph

I think it'd be good to introduce a bit more distinction in the LOC graph.
This is what mine looks like:

Yes, we will update this soon. Thank you!

Hi @alichtman!
We temporarily removed the chart, the new version of it is coming soon.


Don't send statement email to real users from fake data script

Originally created by Tom Smyth at 2017-09-05 17:57, migrated from redmine-#7063

Status changed from New to Closed

By Tom Smyth at 2017-09-10 18:20

Version set to 4.5

By Tom Smyth at 2017-09-24 21:40


Added UR5 robot descriptions with smaller table and without table

Added a new robot description with a smaller table (ur5_robotiq_robot_limited.urdf.xacro)
Added a new robot description without table (ur5_robotiq_robot_limited_notable.urdf.xacro)
Added a new robot semantic description without table (ur5_robotiq_notable.srdf)

Also, should there be a YAML configuration for the default UR5 without a table?

Yeah that already exists, it just did not show up because it did not change, (Now it should since I fixed the .xacro extension).
Don't merge this yet, I want to run some more tests to make sure everything is in order

Performed several tests and it look ok


Proposal for new website design :)

Recently I stumbled over this site again and I thought that there was some room for improvement here.
Some features I missed:

Built-in search
Better side navigation with active-indicator to quickly see where one is
Dark Theme support 😎
Better mobile support - for those who like to read documentation on the go

First I thought I would only create a simple mockup to show you how it could look, but I really liked the initial look and feel, so I rebuilt the complete documentation site 🙈 - hopefully it was not all for nothing. But in the end, you are the deciders and if does not meet your expectations at all, no hard feelings 😅.
VuePress as new site technology
Although I recently contributed to the DSAG ADT Guide - sorry, it is only available in German - I never really warmed up towards Jekyll. I always felt more comfortable with javascript, and so I moved away from jekyll and used VuePress instead. At the end we get a SPA but each markdown file is still rendered into an HTML file, so not that different from jekyll. But because of the whole setup one can easily use custom VueJS components directly in markdown. But I won't bore you too much with pros and cons, as there is existing documentation where the frameworks are described better and in more detail. I would much rather show you the site.
Impressions of the new design
So here we go, this would be new intro page of the abapGit documentation. I moved the different entry points of the documentation in the top navbar, so "User-Guide" and "Development-Guide". For the primary theme color I used the orange from the abapGit logo, but we can choose any other color as well.

Next up we have a standard content page by which I will use to explain the most important feature of the used theme (VuePress Theme Hope).

Side bar which displays the contents of the guide that is currently in focus. Depending on the screen width, the content of the right side bar (2.) will be moved here.
Right Side bar which shows the headings of the current page
Actions to

navigate the GitHub page of the abapGit Docs
change the theme
call up a search dialog to search the site content

Breadcrumbs to show where the current site is located
Information about:

author (currently hardcoded to abapGit)
the date of publishing
reading duration
category (used from the frontmatter of the markdown page)

Furthermore the following actions are located at the footer of a content page

Link to jump directly into edit mode on GitHub
Information about last page update and contributors to this page
Navigation Links to previous/next page

Special features
It is very to easy to create special info sections to warn, inform or give tips to the user.

Such sections can be inserted via:
:::tip
<your tip>
:::

For a detailed overview of the available categorys look here.
I already mentioned the missing dark theme, so of course the new site comes with it 😊.

I gues that's enough for a cursory overview of the capabilities that are built into the site.
The site is currently deployed on a gh-pages branch of my fork of this repository and you can visit via https://abapgitdocs.devepos.com.
Please let me know your thoughts and if you want to move into this direction.
P.S. I also did some restructuring of the markdown files during the move to VuePress
The new folder structure looks like this:
└─ src
   ├─ .vuepress # configuration files for vuepress
   ├─ development-guide
   │  ├─ api
   │  ├─ read-first
   │  ├─ serializers
   │  ├─ technical-details
   │  ├─ testing
   │  └─ user-interface
   │
   ├─ user-guide
   │  ├─ getting-started
   │  ├─ other
   │  ├─ projects
   │  │    ├─ offline
   │  │    └─ online
   │  ├─ reference
   │  ├─ repo-settings
   │  └─ setup
   └─ README.md

Cheers,
Ludwig

Thanks so much, Ludwig!
I had been looking into a more modern Jekyll theme with search but this is so much better. I approve!

Yes, ABAP is fully supported in code blocks

Looks really nice and clean :+1: :+1:
I think this is the docs AG deserves :)

Wow 😍

Alright, bring on the PR 🙏 😃

Done 😁.
https://github.com/abapGit/docs.abapgit.org/pull/127


Fibaro ubs

Description:
Added support for Fibaro Universal Binary Sensors, when configured to flood or motion sensing.
Checklist:

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

While it ends out correctly, this PR has a lot of unrelated commits because it was not branched off from dev. You can clean it up like this:
git checkout fibaro_ubs
git reset --hard dev
git cherry-pick 45d2df5c61f9157684aa7f90dce8ab851b83c46e
git push -f origin HEAD


Error while using :Typstwatch in vim

Hi,
when I try to use :TypstWatch in gvim I have the following error message:
.

Thank you for the issue! Do you mean that this error shows up after closing the PDF-reader and reruning TypstWatch? In that case, the problem is easily solved and I will try to fix it as soon as possible. Otherwise..?

One of the two errors should be resolved in main. For the other error (the main one, TypstWatch already running) you can test out in the watcher branch.
Please let me know if it works!


slugs to posts

Rather than using the index in the stream (which may change!) we need to use slugs for stories.

So, for now, we have ids. Not quite slugs, but progress anyway!


Fix Dependencies

Can't compile the safe-contracts dependency until this is fixed: https://github.com/ApeWorX/ape/issues/1221

For now, use this: https://gist.github.com/fubuloubu/3b8daabb2918aa28496459ca50c71095

Is this still an issue?

I am getting an error after a plugin install:
Traceback (most recent call last):
  File "/Users/dev/multisig/.venv/bin/ape", line 5, in <module>
    from ape._cli import cli
  File "/Users/dev/multisig/.venv/lib/python3.9/site-packages/ape/__init__.py", line 7, in <module>
    from ape.managers.project import ProjectManager as Project
  File "/Users/dev/multisig/.venv/lib/python3.9/site-packages/ape/managers/__init__.py", line 3, in <module>
    from ape.plugins import PluginManager
  File "/Users/dev/multisig/.venv/lib/python3.9/site-packages/ape/plugins/__init__.py", line 8, in <module>
    from ape.exceptions import ApeAttributeError
  File "/Users/dev/multisig/.venv/lib/python3.9/site-packages/ape/exceptions.py", line 14, in <module>
    from ethpm_types import ContractType
  File "/Users/dev/multisig/.venv/lib/python3.9/site-packages/ethpm_types/__init__.py", line 1, in <module>
    from .abi import ABI
  File "/Users/dev/multisig/.venv/lib/python3.9/site-packages/ethpm_types/abi.py", line 3, in <module>
    from pydantic import Extra, Field
  File "pydantic/__init__.py", line 2, in init pydantic.__init__
  File "pydantic/dataclasses.py", line 41, in init pydantic.dataclasses
    # | <blank> | No action: no method is added.          |
ImportError: cannot import name dataclass_transform

Can you share pip freeze? Maybe it's using Pydantic 2.0

@dtdang @antazoey can you confirm this was resolved with the NPM dep work?

@fubuloubu Checked and confirmed that the issue was resolved with the NPM dep feature.


Adding one from the NAN samples back agian.

Checking the NAN samples and adding one. Leaving the others out, because they were not good.

@xiaoyuisrain I just asked for your review here, Could you quick;y check of you would agree with this example? I annotated it myself.

@RedHilarious Is 'receive' in the input sentence considered metaphorical? If so, then all four paraphrases are still metaphorical?

@xiaoyuisrain Hm yeah, good point. So the first paraphrase ("He was received warmly") was from the original Biizoni dataset, so this was considered the correct literal paraphrase. So I guess they targeted the "interest and affection" and left "received" untouched.
I also updated the PR with a couple of more samples for the other task (literal to metaphorical). Would you have a look at this as well?

@RedHilarious I do not understand why with interest and affection is metaphorical >< But it is difficult to say that He was received warmly is literal: even if we ignore receive, warmly is definitely used metaphorically.
I added some comments to the lit2met examples (please see the comments above). Sorry that I did not know they would appear here as well. Looks a bit messy ...


Automatically slugify site_name

Currently, this plugin fails if an embedded mkdocs site_name field does not match the regex ^[a-zA-Z0-9_\-/]+$.
This PR change this behavior to automatically generate a slug based on site_name, this will allow to maintain user-friendly names and still be able to embed.

hey @camilaibs, thanks for the review
I did some changes, it should works now


Is there a "prettier"

Hi Gregg!
No urgency on this because my code works, it's mostly a stylistic question:
    g = RDF::Graph.load("https://w3id.org/duchenne-proms")
    RDF::Vocabulary.from_graph(g, url: "https://w3id.org/duchenne-proms", class_name: "DDP")
    t = DDP.find_term("https://w3id.org/duchenne-proms#DDP_00064")
    pref = DDP.find("http://purl.org/dc/elements/1.1/").__prefix__
    t.properties[(pref.to_s + ":description").to_sym]

those last two lines are really ugly... I am sure there must be a better way to do it!?  Your APIs are usually more elegant than that :-)
Any suggestions welcome, but it's not urgent.
Cheers!

Closing as a duplicate of #424.


Fix packer build SPN objectId is not available issue

fetch SPN object id from graph endpoint if it is not present in service endpoint

When is this fix expected to become available for TFS 2017? Is it included in Update 3?

@helderpinto this fix is available in TFS 2018


updated the 'getting started' instruction about adding Bootstrap v3.3.7 instead of v.4 on Codepen

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (replace XXXX with an issue no): Closes #13787

Description

Since the default dropdown on codepen adds the v.4 of Bootstrap, I have amended the text in the "get set for our front-end development project" as follow:

Click the gear in the upper left hand corner of the CSS box, then copy this link https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap.min.css inside the text box. Now give your h1 element the class of text-primary to change its color and prove that Bootstrap is now available. Note that by using the dropdown menu and selecting "Bootstrap", the Alpha v.4 will be added instead.

And updated the gif with a new one that reflect the new functionality, also uploaded on imgur here
NOTE: This fix will also require to update the available translation for this challenge seed: *spanish - russian - portughese-

Code LGTM.

@Marmiz I think this needs a better gif imho. The one here is a low resolution. Any way to improve it? I'll merge it in if you can get a higher res gif.

@dhcodes I am merging this in and opening a new issue for tracking the higher resolution gif.
@Marmiz Thanks for the pull request!
Happy fixing!

I'm too much of a noob to know if I'm missing something... but this seems to be still not fixed? I checked and the instructions are still not amended to the above and the issue keeps popping up in the forum...
https://forum.freecodecamp.com/t/bootstrap-issues-with-codepen/118274
https://forum.freecodecamp.com/t/text-not-centering-ughh/118006
Just, um,  wondering :)

Hi @Unabashedley This PR has been merged into the beta website (staging) branch. I believe you are pointing out the discussion on the main website?
Can you open a fresh issue just in case?

Also just to note that the changes will NOT reflect in the main website until the new curriculum is out of beta.


Allow astropy Time objects to be retrieved from LightCurve

Over in #147, @christinahedges is proposing that we allow timeobj (astropy.time.Time) to be passed on to KeplerLightCurve, which would make it much easier to access time in a variety of formats, but it carries the problem that lightcurve.time and lightcurve.timeobj may become inconsistent.
As an alternative, I wonder if we should pass the time_format argument to any LightCurve object.  This would allow astropy Time objects to be generated dynamically at any time.  (See PR attached.)
The main drawback is that time_format = 'bkjd' or time_format = 'bjd' is not supported by astropy right now.  The conversion from bkjd to bjd is trivial, but the conversion from bjd to jd is not.  The correction is pre-computed and made available in the Kepler data products, but that is not necessarily passed on to a LightCurve object.
Any thoughts?

Ok I've changed my mind, we shouldn't have timeobj be inherited. We should just use TIME and correct it to BJD by adding 2454833.0.
I'll undo this!

I've updated #147 to remove the timeobj inheritance

Codecov Report

Merging #159 into master will decrease coverage by 0.07%.
The diff coverage is 93.75%.

@@            Coverage Diff             @@
##           master     #159      +/-   ##
==========================================
- Coverage   92.32%   92.25%   -0.08%
==========================================
  Files          17       17
  Lines        2189     2194       +5
==========================================
+ Hits         2021     2024       +3
- Misses        168      170       +2

Impacted Files
Coverage Δ

lightkurve/targetpixelfile.py
89% <100%> (ø)
:arrow_up:

lightkurve/utils.py
100% <100%> (ø)
:arrow_up:

lightkurve/tests/test_lightcurve.py
97.69% <100%> (ø)
:arrow_up:

lightkurve/tests/test_targetpixelfile.py
90.57% <100%> (-0.14%)
:arrow_down:

lightkurve/lightcurvefile.py
94.82% <100%> (ø)
:arrow_up:

lightkurve/lightcurve.py
91.8% <84.61%> (-0.37%)
:arrow_down:

Continue to review full report at Codecov.

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

@christinahedges I am happy with this now.  How do you feel about dealing with time as proposed in this PR now?

Yeah this seems like a good way to deal with it. So long as astropy_time is never required or breaks my lightcurve when I want to initialize one without a frame
lc = lightkurve.LightCurve(x, y, e)
Shouldn't break.

@christinahedges Agreed!
@mirca How do you feel about these changes?

This looks good to me! I like the name astropy_time :D


Support unicode encoding for foreign languages

Should be able to support foreign languages in unicode format
Should be supported from LibreOffice and from raw text files
Tests should cover both
Example
* Hello world in Chinese?
** 你好，世界

Closed by #38


Transfer learning for shape predictor

Hi,
First of all, thanks for the library and ABOVE all the examples and the quality pretrainning models that help a lot on the understanding of the library possibilities.
I am a newbie on dlib, it is really my first contact with it, so I am sorry if the question seems naive or a little off scope, but I tried to find the answer on the forums and I didn't managed to.
What I want to do is to use old models to enable transfer learning to the shape predictor, I first though that would be something simple,  but I didn't find a "parameter" on the API to do so. So I was happy so thought I could contribute to the project adding this parameter to the API and proposing it to the project.
Looking into the code I identified the shape_predictor train in the shape_predictor_trainer.h https://github.com/davisking/dlib/blob/170877da88fdd887b9404aec78646e604d8368d8/dlib/image_processing/shape_predictor_trainer.h#L242 with makes the real training. I was hopping be able pass as parameter a pre-training model, such as shape_predictor_68_face_landmark.dat, and do something like net1 = net2; as explained on this thread https://sourceforge.net/p/dclib/discussion/442518/thread/40667e18/#4ce8/11e7.
Well sure, collecting from the model all the parameters needed for the training function, such as the number of trees per level, the depth of the cascade, etc.
However, if I understood well for the shape predictor, the forest definition is not iterative, i.e. it does not reuse the previous values as basis for the next training iteration. It goes over each level and over each tree "in one shot", so even if it had another pre-calculated model,
where we had trees already calculated to use as initial values, it wouldn't really help, would it?
Am I missing something?
Best,

Correct. Reusing a model to train a new one doesn’t really make sense for this algorithm. Just train from scratch. Training is fast anyway.


Include config file in the image for bundling with server installers

Is your feature request related to a problem? Please describe.
I am looking to leverage a software installer application that dynamically generates docker compose containers. Our process does not allow for config file generation or external files outside the image. If the the this feature is added, there is a strong possibility that NPM will be added to our software repository.
Describe the solution you'd like
I would like to see the config.json included in the image itself, and copies itself to the config folder, which is mounted via a bind volume. I would also like to see the json file include environment variables that are referenced via docker compose for easy modification of credentials.
Describe alternatives you've considered
We discussed dynamically generating the json file, but this is a process that is outside the intent of the installer program. We would prefer an integrated config file, so the image comes whole.
Additional context
Here is an example of what possibly can be leveraged:
DOCKER-COMPOSE.YML
services:
# NGINX Proxy Manager with LetsEncrypt https://github.com/jc21/nginx-proxy-manager
  proxymanager:
    image: jc21/nginx-proxy-manager:latest
    container_name: proxymanager
    labels:
      - "com.dockstarter.appinfo.description: NGINX Proxy Manager with LetsEncrypt included"
      - "com.dockstarter.appinfo.nicename: NGINX Proxy Manager"
    logging:
      driver: json-file
      options:
        max-file: ${DOCKERLOGGING_MAXFILE}
        max-size: ${DOCKERLOGGING_MAXSIZE}
    ports:
      - "80:80"
      - "81:81"
      - "443:443"
    environment:
      - engine=mysql
      - host=mariadb
      - name=proxymgr
      - user=usernamehere
      - password=passwordhere
      - port=port
      - FORCE_COLOR=1
    volumes:
      - ${DOCKERCONFDIR}/proxymanager/config:/app/config/
      - ${DOCKERCONFDIR}/proxymanager/data:/data
      - ${DOCKERCONFDIR}/proxymanager/letsencrypt:/etc/letsencrypt
      - ${DOCKERSHAREDDIR}:/shared
    depends_on:
      - mariadb
    restart: unless-stopped

NOTE: ${DOCKERCONFDIR} is the compose variable for the config directory for our application. This is just an example of how it could work.
CONFIG.JSON
{
  "database": {
    "engine": "${engine}",
    "host": "${host}",
    "name": "${name}",
    "user": "${user}",
    "password": "${password}",
    "port": "${port}"
  }
}

I think this is already possible by using the NODE_CONFIG environment variable ( https://github.com/lorenwest/node-config/wiki/Environment-Variables ). Though I haven't tested it.


Add some custom g:var

Problem:  there is no option for custom base directory #174 ,  <esc> can not cancel imports #175
Solution: add some g:var for custom setting,and make the plugin make the
plugin be more configurable

hi @artur-shaik
I have finished testting,everthing works well,safe to be merged.
but there are something need to be dicussed.

where should we put custom g:var,plugin or autoload?if we put them in autoload,make sure it will be loaded before other JC2 vimscript
should we change g:FILE_SEP to s:f_sep,also with g:PATH_SEP,Avoid the impact of other plug-ins

put g:var into autoload wiil make vim startup faster in other filetype

I think FILE_SEP and PATH_SEP should be global. May be it is better to prefix it with JavaComplete.


💡[FEATURE] Simultaneous PIO Host and Device

Describe the feature you want to implement here:

I have a project that currently uses the built-in USB as device and your PIO USB library as host. If possible, I would like to use your PIO USB library as an additional device.
Is it currently possible to use your PIO library for device and host simultaneously?
If it is not currently possible, could it be possible in the future? I realize that there may be hardware constraints.
Assuming it is possible, my request is to have simultaneous PIO USB host and device.

I think this example shows what you are looking for: host_hid_to_device_cdc.c