Support .yaml and .yml extensions

wskdeploy will only find yaml files with a .yaml extension.  It's more flexible to have support for .yml also.

Close as merged.


chore(linter): shorten eslint/eqeqeq rule error message's span

#3193  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mysteryven and the rest of your teammates on  Graphite


Renaming a device is inconsistent

Description
I noticed that renaming a device using PUT /api/devices/UID isn't consistent.
Sometimes, the device will not be renamed. Sometimes a 409 will be thrown when you try to rename a device with a name already used, but not always.
To reproduce, just have a shellhub instance with one instance on it and spam a rename with a new name on it. The name will not change directly, and once it finally does, then it will not always throw a 409
Edition
Community
Version
0.14.1

Can confirm this has been fixed as of 0.14.3-rc1 according to my testing


Make the Servlet Session ID configurable

Closes #1894
This should enable folks to use custom identifiers for different session backends in Jetty and other container servers. Want to confirm this works before merging it, but wanted to go ahead and open for feedback.

Confirmed this works!
Example project at: https://github.com/farmdawgnation/jetty-with-mongo-sessions
I was even able to dig into the Mongo database and poke it with a stick:
> db.jettySessions.findOne()
{
	"_id" : ObjectId("598ccb7eefcb1867bc913596"),
	"id" : "node0134ohhfrhqmhxzna5g9wvg0",
	"created" : NumberLong("1502399357551"),
	"valid" : true,
	"context" : {
		"0_0_0_0:_jetty-with-mongo-sessions" : {
			"__metadata__" : {
				"version" : NumberLong(2),
				"lastSaved" : NumberLong("1502399358075"),
				"lastNode" : "node0"
			},
			"nice_lift_session_thingy" : BinData(0,"rO0ABXlzcgA4bmV0LmxpZnR3ZWIuaHR0cC5wcm92aWRlci5zZXJ2bGV0LlNlc3Npb25Ub1NlcnZsZXRCcmlkZ2UKLbT8mRLBMgIAAUwACHVuaXF1ZUlkdAASTGphdmEvbGFuZy9TdHJpbmc7eHB0ABxub2RlMDEzNG9oaGZyaHFtaHh6bmE1Zzl3dmcw")
		}
	},
	"maxIdle" : NumberLong(1200000),
	"expiry" : NumberLong("1502400558056"),
	"accessed" : NumberLong("1502399358056")
}

Ah, probably. Let me try removing it and see if it still fixes the bug.

Regular val works just as well. Provided no issues crop up by tomorrow I'll probably merge this at some point tomorrow afternoon.

Huzzah!


"Double tap to rename files" function

Is it possible to delay the rename action when the file is clicked?
Because currently when you click on the name of a file in list view, it immediately changes to "rename mode", so the best solution would be to delay the rename action by 500 or more milliseconds

remove this annoying thing entirely!
renaming is not so common thing we need that kind of shortcut. explicit button/hotkey/menu action is more than enough

This issue should be resolved with #2142.


Azure Portal link is redirecting to https://ms.portal.azure.com/#create/Microsoft.EventHubCluster instead of https://ms.portal.azure.com/#create/Microsoft.EventHub for Event Hubs Premium (Preview)

[Enter feedback here]

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

ID: 50a5c504-bd11-6f5e-c96d-658e045fa6c2
Version Independent ID: 34d0ff20-e2bb-2a96-7db1-a36fba0bbe03
Content: Overview of Event Hubs Premium (Preview) - Azure Event Hubs
Content Source: articles/event-hubs/event-hubs-premium-overview.md
Service: event-hubs
GitHub Login: @spelluru
Microsoft Alias: spelluru

@ppgovekar Thank you for your feedback! We will review and update as appropriate.

Hi @ppgovekar ,
Thanks for reaching out. Could you please help provide the steps that you are following? Are you trying to create Event Hubs Premium tier from Azure portal or some-other way? I tried from Azure portal and don't see any issues:

Please provide few additional details to further assist on this query.
Thanks

Hi @ppgovekar ,
Thanks for reaching out. Could you please help provide the steps that you are following? Are you trying to create Event Hubs Premium tier from Azure portal or some-other way? I tried from Azure portal and don't see any issues:

Please provide few additional details to further assist on this query.
Thanks

So if you go to https://docs.microsoft.com/en-us/azure/event-hubs/event-hubs-premium-overview and check Portal link at bottom(https://aka.ms/eventhubsclusterquickstart) it's redirects to https://ms.portal.azure.com/#create/Microsoft.EventHubCluster instead of https://ms.portal.azure.com/#create/Microsoft.EventHub.
Let me know if you need more details.

@KranthiPakala-MSFT - The Azure portal link in the Next Steps sections needs to be fixed as Prathamesh suggested. Thanks.

@KranthiPakala-MSFT - and, the URL shouldn't have "ms.portal", which is internal. Please use https://portal.azure.com/#create/Microsoft.EventHub.

Thank you @spelluru and @ppgovekar for additional info. Will go head and submit a PR to fix this issue.
@ppgovekar - Thanks much for providing the details. Will let you know once the changes are made.

@ppgovekar
The PR for the fix has been created and it will be merged soon (approx. within 24hrs). Thanks again for pointing this out, I am closing the issue based on this update, please let us know if there is something else that we can help you with.


[FEATURE] make debian distribution available via apt/apt-get

Description
Installing with .deb is not very convenient hence the fact the project.wbp is always saved where to tool is executed from. With apt the installation path is set and execution is global from there.
Acceptance Criteria

[ ] I can download ck3-workbench via apt install ck3-workbench
[ ] I can download ck3-workbench via apt-get install ck3-workbench
[ ] after installation I can execute the workbench from everywhere and the last project state is always loaded

That's a good idea.
It´s more comfortable & attractive, if the tool is easy to get and install.


Compiling Build Options for *.DLL and OpenSLL.EXE file only

Fir the minimum distribution of Openssl Component in our package, maybe this is documented somewhere, but I would like to build for the following:

To build only the *.dll, *.lib files and the OpenSSL.EXE program only, and

Ideally, just have the *.dll without the "-1_1" version numbers.

Is this possible with the current makefile generated?
I have been using "nmake build_libs"  to get the *.dll and *.lib.   I think you can do "nmake build_programs" to get openssl.exe but it makes the fuzz and test programs.  Probably not a big deal. to make them all, but I only need openssl.exe.
It would be better for me to just have *.dll with the versions since these images can be updated more often than we will need to update our own images.  If the numbers change, are we expected to have an code interface change as well?  I can only another major change like v2, but 1.x update should not require an API change in my opinion.
Thanks

I would like to suggest a 3rd item:

XP/2003 support.   I understanding its EOL with Microsoft, but don't tell major companies who may not need to update XP/2003 for legacy mission critical machines, the SSL security updates are needed.  No technical reason to deny them an openssl update unless Openssl itself is not using advanced Win32/64 API files beyond XP/2003 (which I don't think it is, yet).     Anyway, with OpenSSL v1.0, all I had do was edit the vc-32.pl to set /subsystem:console,"5.01" for the linker command.

For v1.1, I don't know which *.pl, to do this in, but I see the generated makefile has the BIN_LDFLAGS which can be set.
32 bit:
BIN_LDFLAGS=/subsystem:console,"5.01" /opt:ref $(CNF_LDFLAGS) $(LDFLAGS)
64 bit:
BIN_LDFLAGS=/subsystem:console,"5.02" /opt:ref $(CNF_LDFLAGS) $(LDFLAGS)
Maybe an LINK_VER option can be passed (default none) to do this so developers don't need modified the auto-generated make?
Thanks

Re building just the libraries, build_libs is exactly right.
For building just openssl.exe, you can nmake apps\openssl.exe (but you must at least do nmake build_generated first).
Regarding dll versions, it's correct that when the number changes, the ABI has changed. The next where this is expected is with OpenSSL 3.0.0, where you will see libcrypto-3.dll and libssl-3.dll.
Regarding changing or adding to the diverse flags, it's in the .conf files in Configurations. You can create your own .conf files and have them inherit other items, so to make your own for XP/2003, you could have one that has something like this (untested, but I'm willing to help figure it out) :
(
    "WIN32-XP-VC" => {
        inherit_from => [ "VC-WIN32" ],
        lflags       => add('/subsystem:console,"5.01"', '/opt:ref'),
    },
);

Perfect.   Good info to have for overall OpenSSL "KB"    I'll go ahead and test the ,conf change with using the new *-XP-VC labels now.  Thanks

Ok, I got it. :)
I created a "xp-support.conf" file and placed in the same location as configure. I could of placed in the configurations folder and the perl --config switch would not be required.
04/21/19 08:03 am, SSI/HLS
Adds XP/2003 support
usage from within build folder:
perl ..\configure VC-WIN32-XP --config=..\xp-support.conf
my %targets = (
"VC-WIN32-XP" => {
inherit_from => [ "VC-WIN32" ],
lflags       => '/subsystem:console,"5.01" /opt:ref',
bin_lflags   => '/subsystem:console,"5.01" /opt:ref',
},
"VC-WIN64A-XP" => {
    inherit_from => [ "VC-WIN64A" ],
    lflags       => '/subsystem:console,"5.02" /opt:ref',
    bin_lflags   => '/subsystem:console,"5.02" /opt:ref',
},

);
The issues?  Not much,

I had to override the above two fields: lflags and bin_lflags.  Apparently, lflags is used to establish the LDFLAGS macro in makefile to link the *.dll files only.  The bin_lflags is used to establish the BIN_LDFLAGS macro in the makefile and it is only used to create the EXE files (programs).

There is some duplicity in the link macros as they are used for the link commands used for both *.DLL and *.EXE actions.  This can use "some clean up" but it was harmless. The linker did not complain of the duplicate switches.

Tip:  Since OpenSSL v11 is not dependent on any advanced WIN32 API beyond XP/2003,  you can avoid the recompile of all the source code.  All we need here is a relink to stamp the PE header.  So I copied the already built "x32"  folder to a new "x32xp",  deleted all the *.dll and *.lib, created the new makefile from within the x32xp folder:
perl ..\configure VC-WIN32-XP --config=..\xp-support.conf
and then ran
nmake build_libs
nmake apps\openssl.exe
To create the images with the proper PE stamps:
C:\Files\openssl\openssl-1_1_1-stable\x32xp>fileosver *.dll apps*.exe
32bit 05.01 XP <IP_ADDRESS>    WINDOWS_CUI 2019-04-21 10:01:20 .\libcrypto-1_1.dll
32bit 05.01 XP <IP_ADDRESS>    WINDOWS_CUI 2019-04-21 10:01:26 .\libssl-1_1.dll
32bit 05.01 XP <IP_ADDRESS>    WINDOWS_CUI 2019-04-21 10:07:07 apps\openssl.exe
Thanks

Maybe one of the OpenSSL cogs can decide to take the minor cleanup of the makefile link macros and promote it to a minor bug issue, It is harmless but we never know if MS will get picky with their tools in the future.
Other than that, I will close this issue.  Thanks @levitte

Since you set lflags, i.e. appied those flags to everything, another option is to configure like this:
perl Configure VC-WIN32 LDFLAGS="/subsystem:console,\"5.01\" /opt:ref"

and
perl Configure VC-WIN64A LDFLAGS="/subsystem:console,\"5.02\" /opt:ref"


Error while loading tracks.csv

Hello,
I am getting an error while trying to load tracks.csv.

Any ideas?

This behavior was deprecated in pandas 0.25:

Removed the previously deprecated ordered and categories keyword arguments in astype (GH17742)

Install an older version of pandas (pandas==0.21.0 was used when developing this code).

Actually a duplicate of #34.


Header widths on IE are off a little when no scrollbar

IE can be a pain....

i'm seeing misaligned headers as well ( chrome / mac / latest )
from this page

Header widths also seem to be a mess when grid is in a container div... Look at better sync options


Speeding up PyTorchContinousGenerator by copying multiple features into GPU memory

Greetings,
I'm currently using DeepTrack for a project to create a convolutional model for particle tracking. I started with the in-package model builder class, but I now need to create a similar model using PyTorch. Since I still want to use the feature system of DeepTrack I'm using the PyTorchContinousGenerator class to generate my data. So far this has worked quite well, unfortunately now I'm training a quantized version of this model with Brevitas and the training loop is much slower than the original version. My thoughts are that this is due to the fact that the generator does not make a copy directly in the GPU memory hence for each batch I need to copy the data from CPU to GPU memory. My question would be: would it be possible to create a new generator class or adapt the existing one so that feature data is directly copied into GPU memory after being generated?
Thank you.

Hi!
This should not be difficult to implement. In fact, I think it's something we should consider doing per default. The Pytorch generator is almost identical to the standard ContinuousGenerator, with the only difference being the getitem being overridden to convert the internal numpy arrays to torch tensors.
Instead, one could hook into the construct_datapoint function which is called immediately after generating the sample. I will have a quick look.

Not tested, but I imagine something like:
class PyTorchContinuousGenerator(ContinuousGenerator):
    """Extends the ContinuousGenerator to support PyTorch models.

    This class is used to generate batches of data for PyTorch models."""

    @staticmethod
    def image_to_torch(image):

        device = "gpu" if torch.cuda.is_available() else "cpu"

        array = image.to_numpy()._value
        torch_array = torch.from_numpy(array).to(device)
        torch_image = Image(torch_array, copy=False)

        torch_image.merge_properties_from(image)

        return torch_image

    def construct_datapoint(self, image: Image):

        if isinstance(image, list):
            image = [self.image_to_torch(x) for x in image]
        else:
            image = self.image_to_torch(x)

        return super().construct_datapoint(image)

    def __getitem__(self, idx):

        batch_size = self._batch_size

        subset = self.current_data[idx * batch_size : (idx + 1) * batch_size]

        for d in subset:
            d["usage"] += 1

        data = [self.batch_function(d["data"]) for d in subset]
        labels = [self.label_function(d["data"]) for d in subset]

        return torch.stack(data), torch.stack(labels)

should work.
Let me know if you need any more help with the implementation based on this.

Hi @BenjaminMidtvedt, thank you for the quick reply, really appreciate it!
I just tested the generator, a couple of things to correct:

device = "gpu" if torch.cuda.is_available() else "cpu" -> "gpu" should actually be "cuda"; on a more general case this is the RuntimeError I get: RuntimeError: Expected one of cpu, cuda, ipu, xpu, mkldnn, opengl, opencl, ideep, hip, ve, ort, mps, xla, lazy, vulkan, meta, hpu, privateuseone device type at start of device string: gpu; but usually cuda I think it's the most used one.
there's a typo in the construct_datapoint method: image = self.image_to_torch(x) -> this should image = self.image_to_torch(image)

After correct these 2 things data is generated correctly, but then the __getitem__ method fails with this traceback:
TypeError: no implementation found for 'torch.stack' on types that implement __torch_function__: [<class 'deeptrack.image.Image'>]
I'm assuming that the Image feature requires the implementation of this method, I just don't know in detail what the implementation would required to be.

Right! Forgot that we don't have bindings for torch functions yet. Will be added.
You can simply do torch.stack([x._value for x in data]) to expose the underlying torch tensors. You no longer need the Image wrapper

You can simply do torch.stack([x._value for x in data]) to expose the underlying torch tensors. You no longer need the Image wrapper

Could you be more specific? I tried applying your suggested change but I get AttributeError: 'numpy.ndarray' object has no attribute '_value'. I'm sure I'm forgetting something but it's not clear what.

I see. You have a mixture of numpy arrays and torch tensors at this point?
You can do
data = [self.batch_function(dt.image.strip(d["data"])) for d in subset]
labels = [self.label_function(dt.image.strip(d["data"])) for d in subset]

return torch.stack(data), torch.stack(labels)

which should be more stable. Then you just need to ensure that the batch_function / label_function you provide accept and return torch tensors. This will depend on the specifics of your application.

Hi @BenjaminMidtvedt , sorry for the late reply. Your suggestion helped me figure out what to do. In the end this is the example code snippet I used:
from deeptrack.image import Image

class PyTorchContinuousGenerator(ContinuousGenerator):
    """Extends the ContinuousGenerator to support PyTorch models.

    This class is used to generate batches of data for PyTorch models."""

    @staticmethod
    def image_to_torch(image):

        device = "cuda" if torch.cuda.is_available() else "cpu"

        array = image.to_numpy()._value
        torch_array = torch.from_numpy(array).to(device)
        torch_image = Image(torch_array, copy=False)

        torch_image.merge_properties_from(image)

        return torch_image

    def construct_datapoint(self, image: Image):

        if isinstance(image, list):
            image = [self.image_to_torch(x) for x in image]
        else:
            image = self.image_to_torch(image)

        return super().construct_datapoint(image)

    def __getitem__(self, idx):

        batch_size = self._batch_size

        subset = self.current_data[idx * batch_size : (idx + 1) * batch_size]

        for d in subset:
            d["usage"] += 1

        data = [self.batch_function(dt.image.strip(d["data"])) for d in subset]
        labels = [self.label_function(d["data"]) for d in subset]

        return torch.stack(data), torch.stack(labels)

def get_normalized_torch_label(image: dt.Image) -> torch.Tensor:
    return torch.from_numpy(image.get_property("position") / IMAGE_SIZE)

torch_data_generator = PyTorchContinuousGenerator(
    feature = imaged_particle,
    label_function = get_normalized_torch_label,
    min_data_size = 10,
    max_data_size = 20,
    batch_size = batch_size,
    shuffle_batch = True
)

Training is now much faster, thank you very much. Unfortunately the implementation is a bit unbalanced but for what I need is more than enough. I guess that you now just need to implement the __torch_function__ in the Image feature to make it more easily adaptable for PyTorch generators. I'll close the issue.


Books amend list

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

this is a list of books. we don't list lists

So, Would I have just given you a link to the pdf?
Sent from Gmail Mobile
On Fri, Oct 20, 2023 at 9:03 PM Eric Hellman @.***>
wrote:

this is a list of books. we don't list lists
—
Reply to this email directly, view it on GitHub
https://github.com/EbookFoundation/free-programming-books/pull/10527#issuecomment-1773576718,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOKVD2FHUPB77X4NQDGCOCTYAMNORAVCNFSM6AAAAAA6IC2HEGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZTGU3TMNZRHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Strict mode

🍩 Feature Request
Is your feature request related to a problem? Please describe.

In a few of our apps the testing setup is a little incomplete which sometimes causes "svg could not be found" warnings on the console. I'd like to ensure that such warnings are not ignored, by making the test fail when an inlined SVG could not be found.
Describe the solution you'd like

I was thinking that maybe a strict: true/false option could be added to the svgJar object in the ember-cli-build.js file. To not make this a breaking change it would be false by default, but people could set it to true for their dev and test environment builds.
The makeSvg() helper would then look at the state of that flag and either console.warn() like we do today, or throw new Error() if strict is true.
Describe alternatives you've considered

export function enableStrictSvgJar() {
  let originalWarn = console.warn;
  console.warn = function (...args) {
    if (args.length > 0 && args[0].startsWith('ember-svg-jar: ')) {
      throw new Error(args[0]);
    }

    originalWarn.call(console, ...args);
  };
}

I've hacked it together for now, but this is probably not a good long-term solution to the problem.
Additional context

That makes sense. Please feel free to make a PR for the feature.

hmm, thinking some more about this, I guess it would also be nice if in production mode we could trigger a Sentry error report instead of showing the warnings, so maybe it would be better to generally be able to overwrite the warning handler. 🤔


Add GPS data?

Hi, your script is probably the closest thing I've seen to achieve what I want to do so I'm wondering if you can help me.
Does your script add GPS data as well in the .tcx file? If not, is it possible to write a version that does?
I uses strava that records gps and exports in .gpx (that I can convert to .tcx), and I have Sigma ROX 9.1 that records all the other stuff (hr, cadence, speed....etc) in .csv. I'm wondering if I can merge the gps info from .gpx and have everything in the converted .tcx?

Hi @timyjen,
You could certainly use a modified version of the script to read from two files and merge the data points into one. You'd need a distinct sync point where the two data files are considered as starting from and both files would need to agree on how often they log data. So that the data points can be merged correctly. Usually the lemond only logs once per second for example. Another feed could log every 10ms or every 1/2second etc.
This script though is really designed to take a lemond csv file which does not have any GPS coordinates (as it is a dumb trainer) and doesn't have any simulator built in that would generate those points.
Feel free to fork the project and try out your own changes to achieve your desired result. Currently, I can't invest time to help you directly to achieve what you want. I'm closing this issue as it is out of scope wrt the goals of this particular software as is. Best of luck with the code!


It doesn't work

0% Minimum Life in practice mode does not work since JSE version 1.16.0

this is the most confusing shit ive ever seen, what


Icons to the right of + shake when mousedown on new terminal button

Notice how the split and kill icons shake when mouse down is fired, it's just for a frame so some don't show up in the gif:

This feels like a grayscale issue due to the zoom on click, I can't repro since I don't have any non-hidpi screens.

This feels like a grayscale issue due to the zoom on click, I can't repro since I don't have any non-hidpi screens.

I also do not see this on my machine. @Tyriar do you see this across all action bars or only for those exact icons?

I also do not see this on my machine. @Tyriar do you see this across all action bars or only for those exact icons?

It happens in output as well, it seems to start from the 4th+ icon from the right. It does not happen in the explorer or editor top right.

It happens in output as well, it seems to start from the 4th+ icon from the right. It does not happen in the explorer or editor top right.

I have no plans to fix this and neither do I know how. Unassigning myself

@Tyriar can you confirm if this still happens since we removed the scale on click?

I think the scale was the cause


Installation fails with PHP 8.0.22 on windows

Bug Description
Installation of php_codesniffer fails with this error message:
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - dealerdirect/phpcodesniffer-composer-installer[v0.5.0, ..., v0.6.2] require php ^5.3|^7 -> your php version (8.0.22) does not satisfy that requirement.
    - Root composer.json requires dealerdirect/phpcodesniffer-composer-installer ^0.5 || ^0.6 -> satisfiable by dealerdirect/phpcodesniffer-composer-installer[v0.5.0, v0.6.0, v0.6.1, v0.6.2].

Minimal Code Snippet

The issue happens when running this command:
composer create-project wp-coding-standards/wpcs --no-dev
Environment

Question
Answer

PHP version
8.0.22

composer
2.4.4

Windows
10

Additional Context (optional)

Tested Against develop branch?

[x ] I have verified the issue still exists in the develop branch of WPCS.

Closing as a duplicate of #1979, #2000, #2068, #2111.
Please search existing (open and closed) issues before opening a new issue.


Discord Group

So my idea, add a thing in the config files to make a Voice Chat group linked to a discord Voice Chanel, so then the people in the VC group in game and in the Discord VC can hear each other...

So people not in the game would be able to chat? This might be possible, except for the big problem that there is no player to use to play audio in the group since the people in the VC wouldn't be on Minecraft.
Also, you might as well just use a normal VC

yup

maybe you could use somthing similar to https://modrinth.com/mod/audioplayer

I don’t think I’ll be adding this. It would be hard to add because groups require player / entity members, so we would have to create a fake player / entity. It’s also pretty pointless because you can just use a discord VC.
Also, both of those mods are SVC addons, which use the same API this mod does.


fix: implement getStructure method for S3 plugin

Description

Add getStructure implementation for S3 to return a list of accessible buckets.
Add command templates for S3.

List files
Read file
Create file
Delete file

Remove getDatasourceMetadata implementation as getStructure method can be directly used now.
TBD: Some client side implementations / fixes are required before this code can be merged:

Currently the template is rendered correctly but the default value for command UQI attribute is not picked up by the client.

Update: Isolating this issue out into a new issue: #9455.

Need to support picking up values from ActionConfiguration object in the Template object.
Update client to use getStructure method call for S3 plugin instead of getDatasourceMetadata.

Fixes #8857
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

JUnit TC
Manual

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: fix/s3_get_structure

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
55.05 (0)
36.49 (0.01)
34.45 (0)
55.58 (0.01)

:green_circle:
app/client/src/pages/Editor/GeneratePage/components/GeneratePageForm/GeneratePageForm.tsx
18.42 (0.24)
10.53 (0.06)
0 (0)
20.59 (0.3)

:green_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.94 (0.23)
41.67 (0.84)
36.21 (0)
56.99 (0.25)

Reopening because this was being worked on actively. @Irongade can you confirm whether the changes on your end are ready to be reviewed?

Thanks @nidhi-nair . This is being worked upon actively. @Irongade had added some changes for this earlier. We need to do integration testing for this now.

Reopening because this was being worked on actively. @Irongade can you confirm whether the changes on your end are ready to be reviewed?

Yes they are.

/ok-to-test sha=d540650

/ok-to-test sha=ed04b20

/ok-to-test sha=ed04b20

/ok-to-test sha=ed04b20

/ok-to-test sha=ed04b20

/ok-to-test sha=905984b

/ok-to-test sha=905984b

/ok-to-test sha=6068ba0

/ok-to-test sha=48a2b03

/ok-to-test sha=48a2b03

/ok-to-test sha=e6129fb


IL2CPP Build Issue

Hi, I have an app that uses this simple graphql package. The package seems to be working perfectly on Mono on both unity editor and android device but when i switch to scripting backend il2cpp I can only see the data on the editor but not on the android device.

Hello!
I have a feeling as to what could be causing this, though I don't remember any issues with IL2CPP when I actively used SimpleGraphQL.
I have updated the project to use Unity's Newtonsoft-JSON version instead of the internal version. As they say they already support IL2CPP, this may resolve your issue. Give 1.1.3 a try and let me know how it goes.

Hi, I have an app that uses this simple graphql package. The package seems to be working perfectly on Mono on both unity editor and android device but when i switch to scripting backend il2cpp I can only see the data on the editor but not on the android device.

Hi. Were you able to get the build issue resolved and test it out on android devices?
I am also planning to try this library on IL2CPP platforms for mobile AR, Magic Leap headsets and hoping it will not have any issues and make graphQL integration simple and easy!
@NavidK0 Thanks to the devs for this effort because I remember struggling with IL2CPP issues and graphQL in the past. Most of the other libraries are using C# dynamics in the parsers (newtonsoft, simplejson) which is the problem on IL2CPP AOT platforms.

Hi. Thanks for the quick reply. I have updated the package to the latest and re-builder my app but I had no luck. The problem though does not seems to be with this package since I have created a fresh project and implemented some of my app's functions to test it and its working great. Some other package seems to be the problem. Thanks for looking into it so quickly.


Serve static content from CDN URL instead of root

What problem does this feature solve?
I'd like to serve static files (e.g. /favicon.ico, /nuxt/*.js files, etc) from my CDN instead of through the /root. So:

Becomes

I'm not sure if this is possible yet, but it would greatly improve PageSpeed scores.
What does the proposed changes look like?
Not sure. Perhaps setting a /root/ path in Nuxt.config that equals '/' but could also be 'https://cdn.domain.com/'

This feature request is available on Nuxt community (#c10722)

Seems to have been answered here: https://cmty.app/nuxt/nuxt.js/issues/c2446#comment-5abbc4df86a34ad2198b6a07 🎉

I tried setting publicPath to this:
build: {
	publicPath: 'https://cdn.domain.com/_nuxt/',

I see that URLs are now:
<script src="https://cdn.domain.com/_nuxt/b3ee6aca9944f00e82f3.js" defer></script>

However, if I go to the page I get a 404.
It seems to me that https://cdn.domain.com/b3ee6aca9944f00e82f3.js is found, but https://cdn.domain.com/_nuxt/b3ee6aca9944f00e82f3.js is not.
So I am unsure how to proceed here; should publicPath simply be publicPath: 'https://cdn.domain.com/?

Also, setting publicPath does not seem to give "favicon" my CDN url.

It had to be: publicPath: 'https://cdn.domain.com/ without /_nuxt/. Favicon still does not have CDN url as I would think it ought to have.

@simplenotezy You specify the favicon URL yourself in your nuxt config; Nuxt doesn't transform it. If you want to specify your CDN URI for your favicon, you can do so there. Does that work for you?

I guess that would work @danielroe! 😊


Converting byte sequences (e.g., strings) to/from arbitrary bytes (or other numbers)

There was a recent discussion on the IM channels by cancer about "the preferred way to iterate over bytes from a byte pipeline in Elvish?". Which resulted in that individual opening PR#1649 to implement two new commands: to-nums and from-nums. The problem is that the proposed new commands are very similar to the existing https://elv.sh/ref/str.html#str:to-utf8-bytes and https://elv.sh/ref/str.html#str:from-utf8-bytes commands. Rather than introduce two new commands I think there should be a discussion about how to handle strings, which includes arbitrary values on the byte stream, which we might want to interpret as arbitrary bytes (not just UTF-8 sequences).
Among the shortcomings of the existing str:str:to-utf8-bytes and str:from-utf8-bytes commands are their reliance on hexadecimal string encoding rather than the num type. More significant is that str:from-utf8-bytes throws an exception if its input is not valid UTF-8 while str:str:to-utf8-bytes will silently, and correctly, encode invalid UTF-8:
> str:to-utf8-bytes "\xF0\xFF你"
▶ 0xf0
▶ 0xff
▶ 0xe4
▶ 0xbd
▶ 0xa0
> str:from-utf8-bytes 0xF0 0xFF 0xe4 0xbd 0xa0
Exception: bad value: arguments to str:from-utf8-bytes must be valid UTF-8 sequence, but is [240 255 228 189 160]
[tty 157], line 1: str:from-utf8-bytes 0xF0 0xFF 0xe4 0xbd 0xa0

I do not believe the commands introduced by #1649 should be introduced. I also think the existing str:from-utf8-bytes and str:to-utf8-bytes commands should be replaced by different commands that only optionally (in both directions) validate the sequence as valid UTF-8 but by default handle arbitrary byte sequences.

FWIW, I couldn't find a single, published, use of str:to-utf8-bytes or str:from-utf-bytes. Obviously there may be private uses of those two commands, but it seems to me they are likely to be in the single digits of Elvish users (possibly zero). So the sooner we replace those commands with something more flexible the better for the Elvish eco-system. I suggest replacing those two commands with str:from-nums and str:to-nums; each having a &utf8 option that raises an exception if the data is not valid UTF-8, but otherwise allowing invalid UTF-8 sequences. The str:to-nums command outputting num types and str:from-nums accepting anything that is a valid binary byte (e.g., when converted from a string).
The new str:from-nums command should support a &max option that specifies the maximum number of bytes it processes. It is not obvious the same option has any useful sem

Looking at str:from-utf8-bytes, another thing that should be changed is for a new str:from-bytes to use the standard convention for taking value inputs rather than variadic arguments.
I'd like to make this change to str:from-codepoints too, but the latter would be a bit more involved as there are more real-world usages.


feat: add torus auth method

Added Torus as a default auth method for mobile and desktop browsers.
This enables a user to log in with their Google, Facebook, Twitch, Reddit or Discord account via bnc-onboard widget.
Here is an awesome preview:

License: MIT
Signed-off-by: Vaibhav Saini<EMAIL_ADDRESS>

You may need to tweak the get() method of the balance part.
Right now, the decimal point is not in the right place.

Hi there @aaronbarnardsound. Would love to get this merged in, got several dapps keen on using blocknative with us in the pipeline. Let me know what we can do to help!
@vasa-develop Thanks for the PR, appreciate it

Thank you @vasa-develop for the PR. @YZhenY we are taking a look and expect to merge early next week. (We're a little slower last and this week due to holiday schedules).

@vasa-develop @YZhenY I will be checking this out today, will keep you updated.

@vasa-develop The Torus implementation is looking pretty good. I took the opportunity to refactor some of the interfaces to allow more flexibility when adding wallets as it was needed in this PR for it all to work nicely. I sorted out the balance issue and added some validation.
@YZhenY There is one problem I am having with Torus, when the logout function is called on the Torus instance, it throws an error: User has not logged in yet even though this is called after logging in successfully. Could you look in to that issue on your end as that will need to be resolved before we can merge this PR.


Add EasyBoard from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of Dizor, to add EasyBoard to CKAN.
Mod details:
name = /mod/655/EasyBoard
author = Dizor
abstract = EasyBoard is a KSP mod that simplifies boarding a vessel for kerbals. With this mod board key "B" works as a toggle. When "B" pressed in EVA your kerbal will wait until you take him to any crew cabin or command seat. Once airlock or command seat reached your kerbal will automatically go in.
license = GPLv3
Homepage = https://github.com/IDizor/EasyBoard/releases/
description = This is your mod listing! You can edit it as much as you like before you make it public.
To edit this text, you can click on the "Edit this Mod" button up there.
By the way, you have a lot of flexibility here. You can embed YouTube videos or screenshots. Be creative.
You can check out the SpaceDock markdown documentation for tips.
Thanks for hosting your mod on SpaceDock!
Please direct questions about this pull request to Dizor.

Sounds handy. I want this.

Ooh, I never knew I wanted this until now!

Should be good to go, now.

Easyboard, welcome to my KSP install. :D


Remove reference to OriginalTreeAttachment in Scala 2.12 bridge

Ref https://github.com/sbt/zinc/issues/993

Just saw https://github.com/sbt/zinc/pull/985/commits/d15228951f3de0ae07c0da5f34b84be5f0e7a4bb#r667629152


Links Double Open

This issue is back, must've been from a refactor.
The documents and external links are opening twice. I have a suspicion that it's caused by event handling, registering the click with the link and the element it's inside.
Research and hopefully fix tonight.

This has been corrected in version 1.1.0


some problem  on experience  drogue cloud

i had install drogue-cloud on my machine, and i can create app/dev by drg.but I ran into a few problems:

i can’t publish msg by http (also can’t connect to mqtt-endpoint), but i can publish in the http-endpoint pod. is there something not configure?

huangjm@DESKTOP-06B3E7S:~$ echo '{"temp":42}' | http --auth 'mqtt_dev@some-app:mqtt_dev' POST https://http-endpoint.<IP_ADDRESS>.nip.io:30443/v1/foo

http: error: ConnectionError: ('Connection aborted.', OSError(0, 'Error')) while doing POST request to URL: https://http-endpoint.<IP_ADDRESS>.nip.io:30443/v1/foo

how can i adjust kafka's log level?

If you have used the installer script, then a local CA has been created and Drogue Cloud has been configured to use this. However, you local system does not recognize this CA as a trust anchor. The default example commands should all include the --verify (for http, or respective option for the other commands) to enable this per request.
I noticed this option is missing from your example command.
If you run ./scripts/drgadm examples … you should the get complete arguments printed out as an example:
Publish data:
---------------

After you created a device, try these commands at a shell prompt:

  http --auth device1@example-app:foobar --verify build/certs/endpoints/root-cert.pem POST https://http-endpoint.<IP_ADDRESS>.nip.io:30443/v1/foo temp:=42
  mqtt pub -v -h mqtt-endpoint.<IP_ADDRESS>.nip.io -p 30001 -u device1@example-app -pw foobar -s --cafile build/certs/endpoints/root-cert.pem -t temp -m '{"temp":42}'

Thanks! The problem has been solved.  The reason is that I installed it on WSL and did not set hosts.

Regarding Kafka logging: Drogue Cloud uses Strimzi by default to provision Kafka.
So you can configure Kafka logging using Strimzi: https://strimzi.io/docs/operators/latest/using.html#external-logging_str … if that is what you had in mind. If not, please let me know what you are looking for.

I was trying to figure out the architecture of Drogue-Cloud.As described in the documentation, it's based on Knative, but why can't I find any trigger/subscription?  And pod does not autoscaling.  I don't think it makes much sense to be based on Knative, right?


Flyway 7.x should accept classpath locations separated by '.', just like 6.x did

I've just spent over two hours upgrading Flyway 6.x to 7.x (latest). The 6.x version worked like a charm, the 7.x version refused to find my Java migrations. After debugging the migration resolution in each version side-by-side (in an otherwise identical codebase) I've finally found the issue.
In Flyway 6.x, the following API configuration line works:
 Flyway.configure()
    ....
    .locations("classpath:com.example.myapp.db.migration.scripts")
    ...
    .load()

However, this produces nothing (i.e. 0 migration scripts) in 7.x. After a lot of debugging, I've found the solution. The equivalent for 7.x is:
 Flyway.configure()
    ....
    .locations("classpath:com/example/myapp/db/migration/scripts")
    ...
    .load()

The new separator has to be /, whereas before also . was accepted (which is the more natural way to write a Java package name). I haven't found any documentation on this change in particular.
I don't know why this has been done this way, I just hope that this issue will save someone the hours I've wasted on this.

This was an intentional change for version 7.
Unless there is further interest we will not be reopening this at this time.
Closing for now.


Add extension to verify role bindings for user

This PR adds the extension VerifyRoleBindingsForUser which validates the corresponding role bindings have been created for a user.

@igomez06 this was part of the rbac validations we moved from validation suite to extension suite as part of this task rancher/qa-tasks#892

This should only be the case if the verification is being used outside of the package, if not we're unnecessarily bloating the framework when this func should just be in the test. So you're saying VerifyRoleBindingsForUser is being used outside of tests/v2/validation/rbac?

sorry @igomez06 for the delayed response. Yes, this is to make use of the role bindings verification outside of VerifyRoleBindingsForUser as well. We have a few tests in rbac additional for which we can make use of this function.
However, considering this involves actions rather than verification, it is appropriate for actions. Since we already have a PR for this, we can merge it now and later move it to actions once that is established. Let me know if that sounds good.

sorry @igomez06 for the delayed response. Yes, this is to make use of the role bindings verification outside of VerifyRoleBindingsForUser as well. We have a few tests in rbac additional for which we can make use of this function.
However, considering this involves actions rather than verification, it is appropriate for actions. Since we already have a PR for this, we can merge it now and later move it to actions once that is established. Let me know if that sounds good.

Yes this would be moved any way, but my question remains. Is this function going to be used outside of tests/v2/validation/rbac? If not, then I would argue that this should just go into tests/v2/validation/rbac/rbac.go

Yes, once the RBAC tests are relocated to the integration suite, they will be referenced outside of the tests/v2/validation/ directory. In that scenario, this should be maintained in actions.

Yes @igomez06 will wait on actions to be implemented and once thats done this can be ported over to actions. We can then close this PR.

This code has been ported over to actions #47447
Closing this since code will reside in rancher/rancher actions.


Storing keys from master

Is it possible to store keys in the master and have child processes access the data?  I have a long running process I only want run as one instance, and in a multi-cpu server I would have that process running multiple times unnecessarily consuming resources.

Hi @somecallmemike
I'm not sure I understand your question.
The whole point of this module is to store info un master and have child processes access to it. When you store a key from a child process, that key gets stored in the master process and, when it's read, it's also read from the master process.
This is true if you're using nodejs cluster module.
If you run different master processes, then this nodule is not an option because you will need some kind of ICP (Inter Process Protocol) channel to communicate between them.
Please, feel free to reopen this issue if this answer is not clear.
Thanks.


Feature request : set / get the GCP Project of the Script

Hi there,
Do you think it could be possible to implement a clasp command to set the GCP project like it can be done over the UI (cf screen shot) ?
Also, a clasp command to get the current GCP project would be nice with "default" value if the GCP project script is still the default project affected when the script is created.
Would be happy to help but really not sure if I have enough skills  ...
Thanks !

i was wondering same thing that you.... but looking to App Ascript WebAPI... this things seems not supported.... So The first thing is to ask to add this feature on the API itself...

Yep, want to add this but it's still blocked by the underlying API.

OK ! Thanks for the feedback


react-native init fails on 0.57 with missing babel plugin

One of React Native dependencies, Metro, depends on a package that it does not specify in its package.json.
It is causing the following error:
Loading dependency graph, done.
error: bundling failed: Error: Cannot find module '@babel/plugin-external-helpers'
    at Function.Module._resolveFilename (module.js:547:15)
    at Function.Module._load (module.js:474:25)
    at Module.require (module.js:596:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/Users/grabbou/Repositories/DemoNewNew/node_modules/metro/src/reactNativeTransformer.js:15:31)

when running a new project.
To reproduce, run react-native init Test and then, attempt running the project.
For now, add this package as a dependency of your project explicitly and wait for a new release.

#21214

Closing in favor of that one.


Support non-double feature values in analyzers

Currently the document and analyzer class assumes that counts are stored as double values always. This isn't the case, particularly for inverted_index classes.
The change here will be to make analyzer be analyzer<FeatureValue>, with FeatureValue taking values double (for forward_index) and uint64_t. Tokenizing a document will return a std::unordered_map<std::string, FeatureValue> instead of the counts being stored directly in the document class.
This will make it more readily obvious what kinds of analyzers are appropriate for what kinds of index classes. Adding a feature for classification that sets real-valued features is fine, but it is not appropriate for creating inverted_index instances. We can now allow for this and error out appropriately if someone uses an inappropriate analyzer for their chosen index type. Analyzers that output integer values feature counts could be used for either inverted_index or forward_index creation.
While analyzer will become a template class, since we only will have two template instantiations for most of our analyzer classes, we can keep the analyzers compiled as regular static libraries as before (rather than becoming fully header-only and reparsed for each translation unit that includes them).

Fixed with the merge of PR #108.


Added nodejs v4

How come you removed iojs?

io.js complicate support https://github.com/ncb000gt/node.bcrypt.js/commit/ebd1d5e89d381d5f7c774663c216567b4568596c
maybe there are other


fix(font): Styles should not include HTML entities

Maybe I'm crazy or missing something significant, but the Font component generates markup that is full of HTML entities and doesn't seem to work with those entities. I suspect something similar is probably going on for https://github.com/resendlabs/react-email/issues/442.
What am I missing? I can't find any indication that style blocks are supposed to use HTML entities, but it's a big internet out there. HTML 4 spec is pretty explicit that style elements should not contain character references.
I'm happy to pursue a solution that uses something other than dangerouslySetInnerHTML, but I'm not sure what that would look like.
Here's a semi-related comment that seems to argue that dangerouslySetInnerHtml is the right choice in this scenario: https://github.com/facebook/react/issues/13838#issuecomment-470296607
Any thoughts or guidance is welcome.

I think this is the right fix for this issue because it relates to a <style> element. HTML entities are supported inline, but not in style blocks as I've tried to demonstrate with this JSFiddle: https://jsfiddle.net/c7dxbset/. As such, for populating the body of the <style> element, dangerouslySetInnerHtml seems to be the right solution.

That's great. This bug was driving me crazy haha Finally fixed

it is not fixed


Adds a log expectation for a test case that matches behavior seen in a deployment

Test-only change to surface a log message seen in a deployment.
Commit Message:
Additional Description:
Risk Level: none, test only
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:

/assign @RyanTheOptimist


Add TransferParticipant to sip and sip internal services

Current implementation is for the RPC to wait until the transfer succeeds or fails (ie until the new peer picks up the phone). This means that there is no transfer status API/attributes on the participant. The transfer can take longer than the default RPC timeout though, so this may or may not be the way we want to go eventually.


Not working with Django 1.10

Hello,
I know that Django 1.10 just poped out, but i wanted to see if it was already working. It seems it's not :smile:
I installed django-filer, configured a simple app which make use of it, and updated django to 1.10 after that. Here is the trace :
Unhandled exception in thread started by <function check_errors.<locals>.wrapper at 0x7ff993492ea0>
Traceback (most recent call last):
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/django/utils/autoreload.py", line 226, in wrapper
    fn(*args, **kwargs)
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/django/core/management/commands/runserver.py", line 113, in inner_run
    autoreload.raise_last_exception()
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/django/utils/autoreload.py", line 249, in raise_last_exception
    six.reraise(*_exception)
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/django/utils/six.py", line 685, in reraise
    raise value.with_traceback(tb)
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/django/utils/autoreload.py", line 226, in wrapper
    fn(*args, **kwargs)
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/django/__init__.py", line 27, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/django/apps/registry.py", line 108, in populate
    app_config.import_models(all_models)
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/django/apps/config.py", line 199, in import_models
    self.models_module = import_module(models_module_name)
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 986, in _gcd_import
  File "<frozen importlib._bootstrap>", line 969, in _find_and_load
  File "<frozen importlib._bootstrap>", line 958, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 673, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 665, in exec_module
  File "<frozen importlib._bootstrap>", line 222, in _call_with_frames_removed
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/filer/models/__init__.py", line 3, in <module>
    from .clipboardmodels import *  # flake8: noqa
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/filer/models/clipboardmodels.py", line 9, in <module>
    from . import filemodels
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/filer/models/filemodels.py", line 18, in <module>
    from .foldermodels import Folder
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/filer/models/foldermodels.py", line 240, in <module>
    mptt.register(Folder)
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/mptt/__init__.py", line 12, in register
    from mptt.models import MPTTModelBase
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/mptt/models.py", line 385, in <module>
    class MPTTModel(six.with_metaclass(MPTTModelBase, models.Model)):
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/django/utils/six.py", line 808, in __new__
    return meta(name, bases, d)
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/mptt/models.py", line 263, in __new__
    cls = super_new(meta, class_name, bases, class_dict)
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/django/db/models/base.py", line 157, in __new__
    new_class.add_to_class(obj_name, obj)
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/django/db/models/base.py", line 316, in add_to_class
    value.contribute_to_class(cls, name)
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/mptt/managers.py", line 82, in contribute_to_class
    super(TreeManager, self).contribute_to_class(model, name)
  File "/home/nicolas/.virtualenvs/tmp/lib/python3.5/site-packages/django/db/models/manager.py", line 120, in contribute_to_class
    setattr(model, name, ManagerDescriptor(self))
AttributeError: can't set attribute

I'm not yet experienced to Python/Django to find what is the problem and fix, so i leave it to the experts hoping it will be fixed soon.
Good luck! :+1:

The issue seems related to https://github.com/django-mptt/django-mptt/issues/469#issuecomment-232152830

Require that _default_manager always is a TreeManager instance.
Do not touch _base_manager at all.

Django: 1.10, Python: 3.4.2, Mptt: 0.8.6
Unhandled exception in thread started by <function check_errors.<locals>.wrapper at 0x7f7a05feb9d8>
Traceback (most recent call last):
  File "/home/shaman/Work/envs/ufanet_ru/lib/python3.4/site-packages/django/utils/autoreload.py", line 226, in wrapper
    fn(*args, **kwargs)
  File "/home/shaman/Work/envs/ufanet_ru/lib/python3.4/site-packages/django/core/management/commands/runserver.py", line 113, in inner_run
    autoreload.raise_last_exception()
  File "/home/shaman/Work/envs/ufanet_ru/lib/python3.4/site-packages/django/utils/autoreload.py", line 249, in raise_last_exception
    six.reraise(*_exception)
  File "/home/shaman/Work/envs/ufanet_ru/lib/python3.4/site-packages/django/utils/six.py", line 685, in reraise
    raise value.with_traceback(tb)
  File "/home/shaman/Work/envs/ufanet_ru/lib/python3.4/site-packages/django/utils/autoreload.py", line 226, in wrapper
    fn(*args, **kwargs)
  File "/home/shaman/Work/envs/ufanet_ru/lib/python3.4/site-packages/django/__init__.py", line 27, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/home/shaman/Work/envs/ufanet_ru/lib/python3.4/site-packages/django/apps/registry.py", line 108, in populate
    app_config.import_models(all_models)
  File "/home/shaman/Work/envs/ufanet_ru/lib/python3.4/site-packages/django/apps/config.py", line 199, in import_models
    self.models_module = import_module(models_module_name)
  File "/home/shaman/Work/envs/ufanet_ru/lib/python3.4/importlib/__init__.py", line 109, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 2254, in _gcd_import
  File "<frozen importlib._bootstrap>", line 2237, in _find_and_load
  File "<frozen importlib._bootstrap>", line 2226, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 1200, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 1129, in _exec
  File "<frozen importlib._bootstrap>", line 1471, in exec_module
  File "<frozen importlib._bootstrap>", line 321, in _call_with_frames_removed
  File "/home/shaman/Work/envs/ufanet_ru/lib/python3.4/site-packages/filer/models/__init__.py", line 3, in <module>
    from .clipboardmodels import *  # flake8: noqa
  File "/home/shaman/Work/envs/ufanet_ru/lib/python3.4/site-packages/filer/models/clipboardmodels.py", line 9, in <module>
    from . import filemodels
  File "/home/shaman/Work/envs/ufanet_ru/lib/python3.4/site-packages/filer/models/filemodels.py", line 43, in <module>
    class File(PolymorphicModel, mixins.IconsMixin):
  File "/home/shaman/Work/envs/ufanet_ru/lib/python3.4/site-packages/polymorphic/base.py", line 74, in __new__
    inherited_managers = new_class.get_inherited_managers(attrs)
  File "/home/shaman/Work/envs/ufanet_ru/lib/python3.4/site-packages/polymorphic/base.py", line 148, in get_inherited_managers
    if manager._inherited:
AttributeError: 'Manager' object has no attribute '_inherited'

I think that I ran into the same problems.  I am a python/django newbie, so I have no confidence in my solution, but I will put some notes here to help other people...

The core problem appears to be with the relationship between django-polymorphic and django 1.10
I ended up using django-polymorphic 1.0b1, and django 1.10.1.
  From reading related speculation, it seems as though the problem occurs because BaseImage is an abstract model that derives from File which is a concrete polymorphic model (with polymorphic manager).  In other words, a problem with concrete-abstract-concrete model inheritance in the presence of polymorphism.  I suspect that the correct solution is for django-filer to rejig their model hierarchy, but I have no idea what the right answer is.

The solution that got me up and running (and I am not suggesting anyone else do this - it probably has some nasty polymorphic side-effects) is that I forked django-filer and then in the BaseImage model I added the member 'objects = FileManaqer()' (i.e. same as the manager defined in File).  It felt dirty, but that got me up and running (this is just a play project where I wanted to play with django-filer).
Hope this helps someone.

Latest version 1.2.5 still does not work with Django 1.10. Аn error message is not changed.

realy?? :open_mouth:
My gratitude does not use it for my project under Django 1.10

This is now fixed in master.

Fixed by #918
Thanks @jschneier for contributing this


connectDragPreview not work when it is directly inside of connectDropTarget

Describe the bug
If connectDragPreview is directly inside of connectDropTarget, it works bad. But the old version 7.7.0 works well.
connectDropTarget(connectDragPreview(/*...*/))
Otherwise if I wrap a DOM node(eg. ) with connectDragPreview, it works well again:
connectDropTarget({connectDragPreview(/*...*/)})
Maybe it's my fault, look forward your reviewing.
Reproduction
Live  Reproduction
Steps to reproduce the behavior:

Click on 'Drag me by the handle GOOD', it works well
Click on 'Drag me by the handle BAD', see error

Expected behavior
Click on 'Drag me by the handle BAD' works well
Screenshots

Desktop:

OS:  macOS 10.14.5
Browser: chrome
Version:
react: 16.8.4
react-dom: 16.8.4
react-dnd: 9.3.1
react-dnd-html5-backend: 9.3.1

I’ll see if I can tackle this next week. As a workaround - chaining works if you use refs:
let ref = useRef()
connectDragSource(connectDragPreview(ref))
return

@darthtrevino  The workaround works well, and waiting for your tackling. Thank you!

The problem is not solved in version 9.3.2.connectDragPreview not work when it is directly inside of connectDropTarget


improving seo, deadlinks

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #100
[x] Closes #95
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[ ] Fill up your DOT address: Payout

Community participation

[ ] Are you at KodaDot Discord?

Screenshot 📸

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

This link didn't work
copilot:summary
copilot:poem

Check your robots.txt file and meta tags: Ensure they're not inadvertently blocking Google from indexing your site or any of its important pages.

I don't see robotss.txt here nor meta tags

https://github.com/kodadot/docs/blob/main/developers/.vitepress/config.ts

Polkadot NFT marketplace

I don't see any mention of it?

Improve your SEO: This might include using relevant keywords in your content, optimizing your meta descriptions and alt tags, using semantic HTML, etc.

Still poor when I've checked with checkbot.io https://deploy-preview-104--developers-kodadot.netlify.app/

Inspire yourself

https://github.com/kodadot/subwork/blob/main/docs/.vitepress/config.ts#L49

@yangwao


Duplicate files being create on ftp during data relese

User Alistair Rust reported that two directories of files are available for the DR 14th Sept.
The same thing seems to have occurred in August, and possibly other times.

Comment from @sprintell:
We have recently had issues with sync project. This looks like something caused by the sync project during DR deployment.
The sync tool failed once in d last DR, and was re-triggered by Sajo after TSC fixed it, could it have been that for each of those instances, the ftp release was successful resulting in 2 copies.
I also noticed that it seems similar things have happened a couple of times in the past
Btw 14th - 29th May. 2018 - http://ftp.ebi.ac.uk/pub/databases/gwas/releases/2018/05/
Btw 14th - 29th Aug. 2018 - http://ftp.ebi.ac.uk/pub/databases/gwas/releases/2018/08/
This files should be synced to the ftp once for every data release


[Modules] Updated OperationalInsights/workspaces to new dependencies approach #2177

Description
Updated Log Analytics Workspace to new dependencies approach.
Close #2177 Update OperationalInsights/workspaces to new dependencies approach
Pipeline references

For module/pipeline changes, please create and attach the status badge of your successful run.

Pipeline

Type of Change
Please delete options that are not relevant.

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Update to documentation

Checklist

[x] I'm sure there are no other open Pull Requests for the same update/change
[x] My corresponding pipelines / checks run clean and green without any errors or warnings
[x] My code follows the style guidelines of this project
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (readme)
[x] I did format my code

Thank you @eriqua , all comments are resolved and i've enabled the auto-merge.
Thank you for you support


Lower/remove newtonsoft.json requirement

Hi,
I'm wanting to add this to an old project, but because of the dependency chain starting at newtonsoft.json everything wants to upgrade
Is there a practical reason there could not be lower minimum version for newtonsoft.json?
(in my case 6.0.6 or less would be good)

No practical reason. It will probably work with a lower Json version, It uses JSON.Net to deserialize the response from Google server validation.

Lowered to 6.0.1 on 1.6.0


feat(cardeditor): initial drop of experimental component

Summary
This PR provides an initial implementation that covers work for the following issues:
#1560
#1561
#1562
#1563
We'll provide further improvements and additions to this component over the next 2 weeks, but the component delivered in this PR is functional enough to start being imported and used by applications
Change List (commits, features, bugs, etc)

Added new Experimental component <CardEditor>

Acceptance Test (how to verify the PR)

Jest tests pass
Click around the storybook <CardEditor>interactive story.  The JSON editor allows you to fully modify the JSON definition of the card, and the card edit form will provide inputs to modify card size and title.  If no card definition is passed to CardEditor, the component opens in "gallery mode" where the user can select a card type from a list.

The interactive story also renders the card definition JSON on the left side of the screen, updated in response to the onChange event defined on CardEditor.

Yeah, in fact I was thinking about removing the showGallery state from the component completely and replacing with a callback (onShowGallery) so that the implementer can manage the "mode" of the editor component (add (gallery) or edit (form)). I'll do that for now.
In proposed "Dashboard Editor" patterns I've seen, the Card Editor would default to the "gallery" state when no cards are selected on the preview dashboard. Then, when either A) a user selects an existing card, or B) a user selects a card type from the gallery, the editor would switch over to the "form" state. With this behavior, selecting "View gallery" would essentially deselect the currently-editing card.
(I wouldn't be surprised if the "view gallery" button gets removed by the time we are finished)

:tada: This PR is included in version 2.120.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


refactor: only fallback to approximate if tiktoken fails

minor refactor to make the code more readable

Thank you!


Module resolution: Is it possible to build a repo without building its dependencies?

Demo Repo
https://github.com/yf-yang/ts-bundler-bug
Which of the following problems are you reporting?
Something else more complicated which I'll explain in more detail
Demonstrate the defect described above with a code sample.
import type { Generic, InterFaceB, InterFaceC } from 'base';

Run tsc --showConfig and paste its output here
{
    "compilerOptions": {
        "allowUnreachableCode": false,
        "allowUnusedLabels": false,
        "declaration": true,
        "forceConsistentCasingInFileNames": true,
        "module": "esnext",
        "noEmitOnError": true,
        "noFallthroughCasesInSwitch": false,
        "noImplicitReturns": true,
        "sourceMap": true,
        "noUncheckedIndexedAccess": true,
        "strict": true,
        "target": "esnext",
        "esModuleInterop": true,
        "skipLibCheck": true,
        "declarationMap": true,
        "extendedDiagnostics": true,
        "jsx": "react-jsx",
        "moduleResolution": "bundler",
        "composite": true,
        "rootDir": "./src",
        "baseUrl": "./src",
        "outDir": "./dist",
        "paths": {
            "@": [
                "."
            ],
            "@/*": [
                "*"
            ]
        }
    },
    "references": [
        {
            "path": "../../packages/base"
        }
    ],
    "files": [
        "./src/index.ts"
    ],
    "include": [
        "src/**/*.ts"
    ],
    "exclude": [
        "../../node_modules"
    ]
}

Run tsc --traceResolution and paste its output here
Too long, I'll skip this part since I've figured out how tsc executes and mention that in the comments below.
Paste the package.json of the importing module, if it exists
{
  "name": "derived",
  "version": "1.0.0",
  "private": true,
  "type": "module",
  "main": "./dist/index.cjs",
  "module": "./dist/index.js",
  "types": "./dist/index.d.ts",
  "exports": "./src/index.ts",
  "files": [
    "dist"
  ],
  "scripts": {
    "build": "tsc"
  },
  "dependencies": {
    "base": "workspace:^1.0.0"
  }
}

Paste the package.json of the target module, if it exists
{
  "name": "base",
  "version": "1.0.0",
  "private": true,
  "type": "module",
  "main": "./dist/index.cjs",
  "module": "./dist/index.js",
  "types": "./dist/index.d.ts",
  "exports": "./src/index.ts",
  "files": [
    "dist"
  ],
  "scripts": {
    "build": "tsc"
  }
}

Any other comments can go here
What I am expecting
I am developing a monorepo (specifically, that is for web, with webpack as the bundler). In the example, there are two projects in the monorepo, base and derived, and base is a dependency of derived.
A good feature of the bundler is live update. Whenever I change anything in the base, the change is captured by the dev server provided by the bundler, so I don't need to build base again to view the change in the browser.
Therefore, I am expecting the same stuff with moduleResolution: bundler option. To be specifically, when I change a type annotation of something in base, I can instantly find the type imported in derived is also changed without building base. That is also implemented by tsserver.
However, after reading https://github.com/microsoft/TypeScript/pull/51669, it seems apply change to packages that importing this one is not what moduleResolution: bundler is originally designed for. Therefore, Without building base first, I am unable to run tsc in derived repo.
That is OK for development, since the build step can only be executed once after everything is done with the language server. However, the typescript-eslint plugin is also using the typechecker, and the way it using lib/typescript.js is pretty similar to tsc, which means without compiling the base first, the typescript-eslint plugin will be broken. Every imported types fall back to any and trigger many rules that prevents any. That's pretty painful for development.
Why this is happening
After some debugging, I figure out the root cause is here:
https://github.com/microsoft/TypeScript/blob/16a36a71f13893bb0d99be45b9b0b3f53fa56a91/src/compiler/program.ts#L3661-L3678
Since base is a project reference folder from derived's tsconfig's perspective, its files are always redirected. After reading the code, I find the file name is either directly be converted to .d.ts if outDir is not specified or be converted to a .d.ts file in the outDir. Therefore, it is not viable to make typescript-eslint work merely via directly setting exports to src/index.ts.
The question
After reading the code and the original PR, I want to confirm one question:
Is it possible to build derived (actually, is it possible to make typescript-eslint get the correct type signature from derived) without having base built? How can I achieve the goal, or is it something that the typescript maintenance team cares about?

We're not the owners of typescript-eslint. They'd need to provide the useSourceOfProjectReferenceRedirect method in their createProgram host in order to surface this behavior, which is up to them.

So you mean this behavior is customizable for downstream libraries 🤔? Let me discuss the idea with them.

Alright, typescript-eslint has already supported that with flag EXPERIMENTAL_useProjectService. See https://github.com/typescript-eslint/typescript-eslint/pull/6754


Support for SemVer pre-releases

semver allows versions to include a "pre-release" (eg. -beta), and that would be useful for FreeBSD versioning, which has not only -RC1 and such, but also uses -RELEASE for stable versions, which unfortunately all end up as Version::Custom today.
I understand that adding an optional component is likely to have some impact, switching enum type from a tuple to a struct with an Option.  This also raises the issue that many OS versions indeed are not strictly SemVer-compliant (eg. no patchlevel or even no minor version), and this could be a path towards supporting them more exactly.
Note SemVer also has a concept of "build metadata" (eg. +20130313144700), but I'm not sure how much that one would be useful here.

Closing this in favor of #280


Scrollbars being shown

Describe the bug
Vertical and horizontal scrollbars are being shown even if there is no scrolling content.

closed in #393


Add DoctrinePaginatorAdapter to interface with Doctrine pagination

All fair points, agreed.
Unfortunately it is required to bring in the 'doctrine/orm' package to allow the tests to run but it is only require-dev.

Yeah, I commented before realizing it was require-dev. You're all good! Thank you!


Cannot create multiple remote connections with same canonical name and different ports

Describe the bug
A clear and concise description of what the bug is.
I have encountered a bug when trying to use WaveTerm to connect to multiple machines on the same url with the same credentials, but different ports. For example, I want to connect to user@domain:22 and user@domain:82. However, WaveTerm does not allow me to create new remote connections that use the same canonical name but a different port. It gives me an error message saying “cannot create remote “user@domain”: duplicate canonical name “user@domain”: cannot create new remote”. This prevents me from accessing different machines that share the same url and credentials.
To Reproduce
Steps to reproduce the behavior:

try to connect to multiple machines on same url with same credentials, port 22 and port 82.
See error
Error: cannot create remote "user@domain": duplicate canonical name "user@domain": cannot create new remote

Expected behavior
A clear and concise description of what you expected to happen.
The ability to create new remote connections that use the same canonical name but a different port.
Desktop (please complete the following information):

OS: [e.g. MacOS/Linux, x64 or arm64] Linux - Ubuntu - WSL

thanks for submitting.  as canonicalname is the key for remotes this isn't possible right now.  i think we can fix this by adding non-standard port numbers to the canonical name.
as a workaround for now, you can add an entry in your /etc/hosts file to alias domain and then you'll be able to add a new connection.

Closing since this has been fixed with the merge of pull request #209

as a note, this will not be available in the next release (should be 0.6.0), but it can be found in the main github branch until then


[bug]Variable substitutions

I'm trying to use env vars but I keep running into problems:
When trying to substitute in the "command" I get this problem:
"cr.io/${PKG_NAME}" -> cr.io/${PKG_NAME}
'cr.io/${PKG_NAME}' -> cr.io/${PKG_NAME}
cr.io/${PKG_NAME} -> cr.io/${PKG_NAME}
cr.io/ ${PKG_NAME} -> cr.io/ package_name

When trying to use variable substitutions it doesn't seem to pick up project level env vars.
PKG_NAME: "boilerplate_rust"
TAG_NAME: "cr.io/${PKG_NAME}"

TAG_NAME just becomes cr.io/
It's the same if I put TAG_NAME in task level env.
That leaves me without any options to do something like this:
docker push cr.io/${PKG_NAME}
or
docker push ${TAG_NAME}
I expect I've just missed something but can't figure out what.

@macthestack What's your task config look like?

Right now it's just hard-coded values similar to this:
env:
  PKG_NAME: "package_name"
  TAG_NAME: "cr.io/package_name"

tasks:
  docker_build:
    command: docker build --build-arg PKG_NAME -t ${TAG_NAME} .

I would prefer to be able to write something like this:
env:
  PKG_NAME: "package_name"

tasks:
  docker_build:
    command: docker build --build-arg pkg=${PKG_NAME} -t cr.io/${PKG_NAME} .

or:
env:
  PKG_NAME: "package_name"
  TAG_NAME: "cr.io/${PKG_NAME}"

tasks:
  docker_build:
    command: docker build --build-arg pkg=${PKG_NAME} -t ${TAG_NAME} .

or maybe even:
env:
  PKG_NAME: cargo metadata --format-version 1 | jq -r '.packages[] | select(.source == null) | .name'

tasks:
  docker_build:
    command: docker build --build-arg pkg=${PKG_NAME} -t cr.io/${PKG_NAME} .

...although the last one probably needs a separate discussion.

@macthestack At first glance, I think this is just how Bash works? You can test it locally.
$ FOO=bar echo $FOO

It echos a default string. But if we set the env var in another command, it works.
$ FOO=bar
$ echo $FOO
bar

The project-level env vars are passed to the child process: https://doc.rust-lang.org/std/process/struct.Command.html#method.env But the parent is the one doing the expansion.

Landed in v1.11 https://moonrepo.dev/blog/moon-v1.11#unlocked-features


Scheduler running out of memory when launching a larger (100+) number of nodes simultaneously

CycleCloud version: 8.6.2-3276
Slurm version: 22.05.11
AFAIK, CycleCloud's prolog script calls get_acct_info.sh which calls azslurm accounting_info and this happens for each launched node. I'm observing that each launch of azslurm accounting_info takes ~150MB of memory, so when launching hundreds of nodes simultaneously, the scheduler can easily get out of memory.
Currently I'm mitigating by commenting out the call to get_acct_info.sh in the prolog script.

We are definitely working on improving that prolog script in the next release. But i just wanted to clarify that it runs on every job launch and not node launch. Also if you are not reliant on the azslurm cost feature or you dont use it then we suggest just commenting out the PrologSlurmctld line in /etc/slurm/slurm.conf. That will just not run the script.

Thanks for confirming! Looking forward to the next release.

But i just wanted to clarify that it runs on every job launch and not node launch.

That makes sense, I'm running 1-node jobs, so the distinction wasn't clear to me.


V5.9 pinout with otherwise up-to-date code

Not to be merged, this PR is to create a patch for PCBversion 5.9 only.

The release generated correctly. Closing this PR and deleting this branch to prevent further accidental releases. This code is not intended to be maintained separately to the main branch.


Create Test-ADCredential.ps1

Thanks @AnthonyKersten !
Would you mind renaming the folder to something like: AD-TOOL-Test-ADCredential ?

I've renamed the folder

Thanks @AnthonyKersten ! :+1:


Put more talks online

cc @christiansherry

I'll sort them out Monday, will need to clear with jenny putting hers up

We also need to look at a redesign of the website which will help with this too

Micheal Cullums talk has no video unfortunately, the file was corrupt when I came to get it off the card :(

Jenny Wong - Dec (with video, pending approval)
Michael Cullum - Feb
Any others missing?


About the evaluation metrics

Hi,
Your repo said that there are v1 and v2 as evaluation metrics, what is it exactly? Which part of the paper can I find?

Hi! Please see Appendix E for an explanation on the choice of evaluation metric. 'v1' evaluates ACC on 'Old' and 'New' subsets the same way 'Legacy NCD' does in Figure 6 in the appendix, while 'v2' is 'Ours'.
Hope this helps!


Backport state bootstrap API to v0.37.x

Feature Request
Summary
The original description of the problem and the need for is can be found in issue #884 and ADR 101 from #801.
 At the moment, the stores are bootstrapped with code within the SDK which was copied from CometBFT's statesync. However, there are some concerns with whether this leaves the stores in a correct state. Furthermore, they should not be required to maintain and fix bugs in code that should utlimately be done by Comet.
As there are already users using this feature, we want to roll out a API for them to use ASAP to avoid any problems with offline state sync.
This needs to be done in a non-breaking way (including non API breaking).
Proposal
Backport the PR https://github.com/cometbft/cometbft/pull/1057

Closed with #1514


People leaving causes ICE to go completely nuts

I've noticed some people have a way of leaving that causes either a desync or for ICE to break in really weird ways a good portion of the time. While they are disconnected in the game, I see other players as quiet and when they eventually get kicked from the game, I will then have a perfect connection to everyone except the P2P table shows for some reason I'm only getting data from other players every few seconds.
Changing coturn and enabling/disabling IPv6 doesnt help but debug does show the following which moves with the lag:
2024-05-12T23:49:46.803+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'Bottleneck' '["ack","3215","45474,115577","5179.4"]'
2024-05-12T23:49:52.744+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'Bottleneck' '["ack","3215","45474","11119.7"]'
2024-05-12T23:49:57.794+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'Bottleneck' '["ack","3215","45474","16169.8"]'
2024-05-12T23:50:01.430+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'BottleneckCleared' '[]'
2024-05-12T23:50:01.540+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'Bottleneck' '["ack","3234","101807","19916.3"]'
2024-05-12T23:50:01.545+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'BottleneckCleared' '[]'
2024-05-12T23:50:08.605+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'Bottleneck' '["ack","3235","45474,115577","5659.5"]'
2024-05-12T23:50:13.864+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'Bottleneck' '["ack","3235","45474","10918.0"]'
2024-05-12T23:50:19.350+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'Bottleneck' '["ack","3235","45474","16404.4"]'
2024-05-12T23:50:24.350+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'Bottleneck' '["ack","3235","45474","21404.7"]'
2024-05-12T23:50:25.049+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'BottleneckCleared' '[]'
2024-05-12T23:50:25.049+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'Bottleneck' '["ack","3236","24","22103.7"]'
2024-05-12T23:50:25.063+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'BottleneckCleared' '[]'
2024-05-12T23:50:30.898+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'Bottleneck' '["ack","3236","24","5840.6"]'
2024-05-12T23:50:35.752+02:00 DEBUG 830658 --- [FAF Client] [      JJsonPeer] c.f.c.fa.relay.ice.IceAdapterCallbacks   : Message from game: 'BottleneckCleared' '[]'

Not sure if this is known or if anyone wants help testing, but this is happening more and more lately.

Should be fixed by #56 (tests confirmed that CPU load massively decreased)


[all] Bump npm version to match export version

This bumps the versions of each npm package to match the exported version.
After we publish these changes, a follow up PR will be used to "lock" builder versions in now deployments as well as now dev.

Related to #3298
Related to #3254

This PR is no longer necessary


Migrate to Quarkus 3.0

to synchronize with upgrade in other repository
"root" migration: https://github.com/com-pas/compas-core/pull/230

I took a look into why the Cim Mapping was failing.
Turns out that the Cim-mapping has a dependency on RDF4J for reading RDF (?) data.
This is still using javax. Once this has been updated to Jakarta, we can safely upgrade Cim Mapping to Quarkus 3.
There is an active issue in the RDF4J repo:
https://github.com/eclipse-rdf4j/rdf4j/issues/4603
part of their Milestone 5:
https://github.com/eclipse-rdf4j/rdf4j/milestone/80

https://github.com/eclipse-rdf4j/rdf4j/pull/4604

Currently the open PRs from @apupier in both repos present challenges that prevent us from merging and creating new releases.
For compas-cim-mapping there is a build error here: https://github.com/com-pas/compas-cim-mapping/actions/runs/6405355292/job/17394667679?pr=338
And for the compas-scl-validator we have Sonarcloud complaining about the logging approach here: SonarCloud - compas-scl-validator #206 – Upgrade to Quarkus 3.0


DCOS-57982: component for rendering badges in a text input

Testing
You should be able to add badges by hitting "Enter" after you type some value into the input
You should be able to add badges using the Typeahead variants
You can't add duplicates
Screenshots
Default:

Pre-filled:

w/ a Typeahead:

Some small things I'd like to address before merging, but probably aren't blockers:

[ ] The vertical centering of the text in the badges is a little off when the <input> isn't on the same line as the badges
[ ] I'd like to add some handler for when a user tries to add a duplicate badge
[x] I want to reconsider "sanitizing" the string that goes into the value property of the badge data

Good catch @GeorgiSTodorov - I can fix that


Error on compile time (Installed plugin from git)

Commit: 41dd398
Build Platform : Linux (Ubuntu in CT)
UE Version: 5.3.2 (From source)
Error on compile time (Installed plugin from git)  , Other plugins are built without problems
You just need to place () in the condition
Win and Mac (OSX) build correct

请问一下,Linux,应该如何修复改段代码,
if ((TouchIndexs[i] && bIsCurrentlyPressed == false) || IsClamp(Vector, Vector2) == false)
修改为这样有用吗?

请问一下，Linux，应该如何修复改段代码，
if （（TouchIndexs[i] && bIsCurrentPressed == false） || IsClamp（Vector， Vector2） == false）
修改为这样有用吗？

已修复!

Commit: 41dd398 Build Platform : Linux (Ubuntu in CT) UE Version: 5.3.2 (From source)
Error on compile time (Installed plugin from git) , Other plugins are built without problems You just need to place () in the condition Win and Mac (OSX) build correct

感谢提供错误描述


failed to reload a modified plugin when the .so file path retained the same

Problem
Here is what I have done:

I loaded a plugin path "/home/1.so" (say name is "test1") and then unload it.
I changed the name() function's return to "test2" and then compile a new plugin retains the same plugin path "/home/1.so"
Load the new plugin and check the plugin list, the result is "test1" but not "test2"

I believe the first plugin is not be unloaded from the memory even the plugin manager has executed self._drop_plugin(idx).
The libloading::Library::new function actually using "RTLD_LAZY | RTLD_LOCAL" flag which may retrieve the original plugin back to the plugin vector.

Duplicate #3620.


Module 'understand' has no attribute 'open'.

hello,
thanks for your src.
when I run use_understand_to_extract_entity.py , the error shows :  Module 'understand' has no attribute 'open'.
Could you please give me some help?
thanks，so much.

how to  install understand package in python?

Which version of understand do you use? I used understand with version 'build 1097' and ran without error. If you are using a new version, python API could be changed and you can find the corresponding document to find the correct command.
In addition, to import the module 'understand', I place the use_understand_to_extract_entity.py in the python folder of understand (C:\SciTools\bin\pc-win64\Python) and run the script.  Maybe you can try to put the use_understand_to_extract_entity.py in the corresponding folder and rerun it.

I use understand with version 'build 1093'.
I put 'understand.pyd' and 'use_understand_to_extract_entity.py' in the python folder of understand (D:\some_softwares\uuunderstand_6_1093\SciTools\bin\pc-win64), then the before error Module 'understand' has no attribute 'open' disappears.
The Understand can analysis the src, but when run use_understand_to_extract_entity.py, the new error is "understand.UnderstandError: NoApiLicense Automatic generation of trial licenses is not enabled for this product."
Is there something wrong with my understand ?
Can you share the software download Baidu Netdisk Link ？

The understand requires a license for running. Students can ask for an educational license for usage. I am using a student license to run the understand. If you are a student, I think you can apply for one.


Ensure linked-dist is built for lint CI

Pull Request Description
Attempts to fix Lint CI by ensuring IDE build artifacts always exist, because TS relies on build.json and linked-dist for typechecking.
Important Notes
None
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

wait what?
 INFO ide_ci::program::command: npmℹ
 INFO ide_ci::program::command: npmℹ ><EMAIL_ADDRESS>lint
 INFO ide_ci::program::command: npmℹ > npx --yes eslint src
 INFO ide_ci::program::command: npmℹ
 INFO ide_ci::program::command: npm⚠ npm ERR! code ENOTEMPTY
 INFO ide_ci::program::command: npm⚠ npm ERR! syscall rename
 INFO ide_ci::program::command: npm⚠ npm ERR! path /runner/_work/enso/enso/app/ide-desktop/node_modules/iconv-lite
 INFO ide_ci::program::command: npm⚠ npm ERR! dest /runner/_work/enso/enso/app/ide-desktop/node_modules/.iconv-lite-jSVWoePl
 INFO ide_ci::program::command: npm⚠ npm ERR! errno -39
 INFO ide_ci::program::command: npm⚠ npm ERR! ENOTEMPTY: directory not empty, rename '/runner/_work/enso/enso/app/ide-desktop/node_modules/iconv-lite' -> '/runner/_work/enso/enso/app/ide-desktop/node_modules/.iconv-lite-jSVWoePl'
 INFO ide_ci::program::command: npm⚠
 INFO ide_ci::program::command: npm⚠ npm ERR! A complete log of this run can be found in:
 INFO ide_ci::program::command: npm⚠ npm ERR!     /home/ci/.npm/_logs/2023-05-05T20_02_39_594Z-debug-0.log

so this doesn't work properly. now lint ci is uploading artifacts...
it is green, and only takes ~12m including build, which doesn't even sound too bad...
... but if lint ci is going to upload artifacts every time that's probably not a good thing.
i'm explicitly running ide build but afaict only ide upload is supposed to upload artifacts...?
                let ide_cli = Cli::parse_from([
                    "./run",
                    "ide",
                    "build",
                    "--skip-version-check",
                    "--skip-wasm-opt",
                    "--wasm-profile=dev",
                    "--backend-source",
                    "release",
                    "--backend-release",
                    "nightly",
                ]);

... ah, never mind, ide unconditionally uploads an artifact. in this case it shouldn't be too bad as lint ci should only build ide if build.json or linked-dist are missing
https://github.com/enso-org/enso/blob/608c5d8945d1c9931bc3df42699e6c8453ddeedd/build/cli/src/lib.rs#L565-L586

i'm just going to assume the iconv-lite issue was a one-off thing

not sure how i feel about this solution but it seems to work?

again though, not familiar with build so idk if, say, ./run gui is better, or whether this should use a different source (probably...), or whether there's a better way to do this in general
i tried using current-ci-run as the source but that seemed to break CI, i'm assuming because the artifact needs to have already been built. so instead I just used the regular command-line that everyone uses

@somebody1234

i'm explicitly running ide build but afaict only ide upload is supposed to upload artifacts...?

To clarify the build-script nomenclature: "artifact" is something uploaded to a CI run (which shows in the summary when run is complete), while "asset" is a file that is uploaded to a release.
ide upload uploads an asset to a release, it is meant to be used only as a part of the release pipeline. (Create release, upload assets, publish the release.)
All builds will upload the artifact by default (see below).

... ah, never mind, ide unconditionally uploads an artifact. in this case it shouldn't be too bad as lint ci should only build ide if build.json or linked-dist are missing

This could be adjusted, but the core assumption in the CI was that we want to avoid duplicating any heavy jobs. As such, if something is built, it will be uploaded as an artifact, so any other jobs that require build artifacts can use it.

i tried using current-ci-run as the source but that seemed to break CI, i'm assuming because the artifact needs to have already been built. so instead I just used the regular command-line that everyone uses.

The current-ci-run downloads the artifact from the ongoing run. (In contrast with ci-run that works only with completed CI run's artifacts.) To use it reliably, you need to have dependencies between CI jobs, so if a job needs an artifact, the job that provides the artifact is completed beforehand.

The core issue seems to be that the lint command was never assumed to require having the GUI built. The lint was supposed to be a (relatively) quick check, like cargo check vs cargo build. Why do we need to build the whole IDE just to type-check the TS code? To me this looks like the core problem.

Also, if you have any questions about the build script, don't hesitate to ask me on our Discord.

@mwu-tow the issue is that typescript depends on linked-dist and build.json to be present, since it imports from both of those locations. there is an alternative PR open that imports the types from the source files instead:
#6583
it's worth noting that i think linked-dist and build.json are only absent on clean runners, which explains why they succeeded on the original PR: link to original successful lint action
re: having to build the whole IDE, some notes:

technically we only need build.json and linked-dist
i definitely think that a restructure would be a better solution; it is very weird to have to import * as linkedDist from '../../../../somewhere/outside/of/root/npm/workspace'. i'm not familiar enough with our setup (especially on the ensogl/pack side to know a good solution to this though

superseded by #6602


New Contribution - Suspended

Contributor: PenumbraEnthusiast
SSH Public key: ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIAuBPxaMwe7msbpr/01OFlDO9MtpoZapZiqTaqPT5MlL
Attestation: [any form of attestation for contribution (e.g., signature) - Optional]

Can I start? Did I miss the opportunity?


Fix oauth refresh timeouts

https://github.com/department-of-veterans-affairs/vets-contrib/issues/4132
See https://lighthouseva.slack.com/archives/CMT4MFPS6/p1580154255007800 and https://lighthouseva.slack.com/archives/CGG5E8CUU/p1580220425011000 for additional context.
This PR fixes a small programming error (using document.state.S when document might be undefined) and ensures that we will actually send a 500 response when encountering an unexpected error. Additionally, it modifies the refresh handler so that errors retrieving the state param from DynamoDB will only prevent the state from being included in the response, rather than sending an error to the user.

You mentioned a conversation with Drew about why testing this satisfactorily would be difficult. Can you share a bit more about what the problems are?

@nfasulo I was just referring to this comment, which captures the difficulty I was having in writing an e2e test for this component. It's not insurmountable, just more difficult than the simple thing I was anticipating!


Versioning

We called this thing api-v1, but I expect when this is stable enough for an actual v1.x release we won't be finished. We really can't do major bumps in the Web api bc all clients would have to update. We also can't really create a new repository for every api major version, since most likely the breaking changes will not be that big, or will only be in one part (eg only in the websocket server).
Instead, I think we should rename this to u-wave-http-api or something similar. All major versions will live in this repository side by side, and they'll all remain available on side by side routes. I have to look more into how other people do it, but right now I'm thinking that different Web api versions can call into the same controller files and use different controller methods only where the versions differ.
(brief bc on mobile)

+1

maybe http-api?


uiua: init unstable-2023-09-28

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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.

This is a bit ugly I'll clean it up in asap

Result of nixpkgs-review pr 257673 run on x86_64-linux 1

1 package built:

uiua

This is ready for review.

I see you bumped the version, could you also show this in the commit message and PR title?

I see you bumped the version, could you also show this in the commit message and PR title?

Yes, ofc, done!

I did some testing, and it turns out you only need alsa-lib in buildInputs

Ahh nice, thanks, then we can throw out udev.

Also, I don't really know what the right name for a flag like this should be. The linked derivation used enableSound but I think enableAudio or audioSupport would also be correct.

I personally prefer audioSupport over enable sound. The term enable sound seems like it automatically produces sound, while audioSupport suggests it provides the capability to produce sound.

Comments are addressed now

Result of nixpkgs-review pr 257673 run on x86_64-linux 1

1 package built:

uiua


[Add code] : Want to add code for Z-algorithm of string matching.

Hi, I want to add code for string matching using Z-algorithm in C++. The time complexity of Z-algo is O(length(text) + length(pattern)).

Hey,
Great! yes you can start working on..