Hash collisions are super easy to generate

Our hashing function has some issues:
Turns out that for all the values I tested if an index is (x, y, z) then hash(x, y, z) == hash(x, -y, -z)
This means we will see a massive amount of hash collisions when exploring near the origin

It seems that you are computing the combined hash value with the XOR operator.  The XOR operator is commutative. In addition to what you have already mentioned, it is also true that has(x,y,z) == has(x,z,y). XOR is probably the wrong choice when dealing with coordinates.

I looked more into this and it seems while the above issue with negatives is true, we are highly unlikely to get more than two voxels with the same hash. This means things are still O(1) and as long as we check the index rather than the hash when comparing locations its not a big problem. Closing.


[New Resource] xPfxExport

User Story: Need to be able to use certificates that are either already in the Windows Certificate Mahine Store (or have been requested via xCertReq) in software that does not use the Windows Certificate Store to store certificates. An example is Java Key Store (JKS) files.
This would require a certificate to be found in the machine store (via Thumbprint, Subject, Serial Number, KU, Issuer etc) and then exported to a file (if the file does not exist), encrypting with a password or user/group combination (for Windows Server 2012+).
Export of the Private key would only be able to occur if it was available and exportable for the certificate identified for export.
If no one else is working on this I'll look at getting this one implemented (if there are no objections).

No PR's currently for review and no other open issues regarding this. So I say, go for it. Sounds like a great addition.
Great idea @PlagueHO

Thanks @johlju  - I'll get this done.

I'm going to close this issue because it will get rolled into #41.


Can you make a example fetching api?

Thank you very much

Hi @Trung15010802
Here is an example of fetching and decoding a JSON:
// Function
@pragma('vm:entry-point')
Future<Map<String, dynamic>> fetchAndDecode(String url) async {
  final response = await http.Client().get(Uri.parse(url));
  return jsonDecode(response.body);
}

// Create an IsolateManager instance
final isolateFetchAndDecode = IsolateManager.create(fetchAndDecode);

Then you can send and receive a message like this:
const url = 'https://path/to/your/json.json';
final result = await isolateFetchAndDecode.sendMessage(url);

Or like this if you want to use it in a StreamBuilder:
StreamBuilder(
  stream: isolateFetchAndDecode.stream,
  builder: (_, snapshot) {
    if (!snapshot.hasData) {
      return const Text('No Data');
    }

    return Text(snapshot.data!.toString());
  },
),

I try to fetch data from API in Flutter WEB. But it's seem have no effect. Could you help me please?
  Future<void> fetchDataFromIsolate() async {
    state = const AsyncLoading();
    final isolateManager = IsolateManager.create(
      (_) async {
        return await fetchCameraList();
      },
    );
    if (mounted) {
      state = await AsyncValue.guard(() => isolateManager.compute(null));
    }
    await isolateManager.stop();
  }

   @pragma('vm:entry-point')
  Future<List<CameraModel>> fetchCameraList() async {
    return await cameraRepository.fetchCameraList();
    // return testCameraList;
  }

@Trung15010802 Here is a better way to do it:
Future<void> fetchDataFromIsolate() async {
  state = const AsyncLoading();
  final isolateManager = IsolateManager.create(fetchCameraList);

  if (mounted) {
    state = await AsyncValue.guard(() => isolateManager.compute(null));
  }
  await isolateManager.stop();
}

@pragma('vm:entry-point')
Future<List<CameraModel>> fetchCameraList(_) async {
  return await cameraRepository.fetchCameraList();
  // return testCameraList;
}

Please note that in Flutter Web, some APIs will cause CORs issues if you don't configure them correctly. And if you want to have the best result of using Isolate Manager on the Flutter Web, you need to create a Worker to use.

Feel free to re-open this issue if you have any related problem. Thanks.


Option to extend web instance (like simpleapi)

For most people it will be better to extend a web instance because that can handle auto certificate generation.

Done. Only it's not an option ;)

Done. Only it's not an option ;)


VR Unable to fully display the Earth

After activating the VR function, the Earth in the right interface is cropped by the Earth on the left, resulting in incomplete display;
You can activate VR in the sandbox and perform Earth level reduction, but the right ball cannot be fully displayed;

4996addbf53c21eb6cd3968a67b2cf46cac077d2
This commit produced this problem since the version 1.93
@sanjeetsuhag
And I report it one year ago. 😭
#10692

@ggetz @zxj5470
I tried to track the rendering of the Earth, but when calculating the world coordinates of the current tile in the shader, I did not consider the starting point position of the viewport

Let me use the code analogy of webxr.
For example, in the webgl setting viewport code of webXR, suppose the binocular resolution is 3000*1500,
and the x of viewport needs to be set to start at 1500 before the screen of the right eye is drawn,
so it is reasonable to subtract the position of xy from the texture coordinates in cesium.


Convert ES6 to Typescript

This PR translates all the ES6 files to Typescript, and reorganizes the directory structure to accommodate that.  Unfortunately the renaming causes extra complication in the file differences, since they include the removal of the old ones as well as the creation of the new ones.  Not sure how to make that work better at this point.
There is now a directory mathjax2 for the legacy code (non-typescript), and some glue files for accessing it (these weren't translated to Typescript, as they will be going away when everything is converted to v3 anyway).  The mathjax3 directory contains the compiled javascript output of the typescript, and mathjax-ts is where all the typescript files live.

@zorkow, I believe that I have addressed your comments in the branch ts-core-review1, except for the issue of separating FindMath from InputJax, which we still need to discuss.
You can see the differences, but I haven't merged the changes in hopes that this comparison will be easier to view than if I merged the changes to ts-core first.  If you would rather that I make the merge first, let me know.

OK, I've merged the changes that I've made, so you can do round 2 of review when you get the time.

Still two minor requests/questions. O/w it's good to go.

Hooray!


Possible mistake

Something looks strange in here: "However, let's increase the HealthCheckStableDuration to 60 seconds (so that the services are healthy for at least 20 seconds before the upgrade proceeds to the next update domain)."
Is not supposed to wait for 60 seconds instead of 20? If not, what happens with the 40s left?

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

ID: b5364ed0-1848-8e92-6117-5706f96fe78d
Version Independent ID: 7285affb-291b-489b-d6b0-d6f852da52b1
Content: Service Fabric App upgrade using PowerShell
Content Source: articles/service-fabric/service-fabric-application-upgrade-tutorial-powershell.md
Service: service-fabric
GitHub Login: @mani-ramaswamy
Microsoft Alias: subramar

Thanks for the feedback! We are currently investigating and will update you shortly.

@mani-ramaswamy could you clarify on this one?

@thiago-vivas
This document is incorrect.
Specifically the default (recommendation) is 120 seconds, so setting to 60 is decreasing the default.
Ref:
https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-application-upgrade-parameters
Further the UpgradeHealthCheckInterval is "The frequency of health status checks during a monitored application upgrades" with a default (recommendation) value of 60 seconds.
Ref
https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-cluster-fabric-settings#clustermanager
Also when an upgrade occurs, while validating health stability for some period of time, the user needs to take into consideration how frequent testing of health is occurring (The value of UpgradeHealthCheckInterval).
Thus this:
"However, let's increase the HealthCheckStableDuration to 60 seconds (so that the services are healthy for at least 20 seconds before the upgrade proceeds to the next update domain)."
Will change to:
"However, let's increase the HealthCheckStableDuration to 180 seconds (so that the services are validated to be healthy for at least 120 seconds before the upgrade proceeds to the next update domain)."
Change Comment:
The first health passing test doesn't validate health, it starts the HealthCheckStableDuration timer, and the UpgradeHealthCheckInterval by default will check again every 60 seconds; so its validated to be healthy during the second check 60 seconds later (then Re-validated a second time to reduce false positive health and subsequently reduce livesite issues by validating twice).
@MicahMcKittrick-MSFT
If the user and you both feel the proposed change resolves the issue, please close this, and submit the PR (or let me know its resolved the issue and ill submit the PR)

@thiago-vivas I submitted the fix :)
Once the PR merges this issue will close and the changes will go live in a few hours.


python310Packages.sensorpush-ble: 1.5.1 -> 1.5.2

Description of changes
https://github.com/Bluetooth-Devices/sensorpush-ble/releases/tag/v1.5.2

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

19 packages built:

home-assistant-component-tests.govee_ble
home-assistant-component-tests.inkbird
home-assistant-component-tests.moat
home-assistant-component-tests.sensorpush
home-assistant-component-tests.xiaomi_ble
python310Packages.bluetooth-sensor-state-data
python310Packages.govee-ble
python310Packages.inkbird-ble
python310Packages.moat-ble
python310Packages.qingping-ble
python310Packages.sensorpush-ble
python310Packages.xiaomi-ble
python39Packages.bluetooth-sensor-state-data
python39Packages.govee-ble
python39Packages.inkbird-ble
python39Packages.moat-ble
python39Packages.qingping-ble
python39Packages.sensorpush-ble
python39Packages.xiaomi-ble


Refactor bot as class

Closed by (4)[https://github.com/alexandermjones/overwatch-queue-discord-bot/pull/4]

Closed by (4)[https://github.com/alexandermjones/overwatch-queue-discord-bot/pull/4]


enable SCTP module in kernel config

This enables SCTP support in kernel

What is the specific need for this?  Mostly I would like to have a documented reason for enabling it.
In any case, it seems fair to enable Stream_Control_Transmission_Protocol, since these two flags are also enabled in debian: https://sources.debian.net/src/linux/4.7.5-1/debian/config/config/#L6278 and https://sources.debian.net/src/linux/4.7.5-1/debian/config/config/#L6576.
cc @tianon

Agreed -- I'm +1 on adding it, but would love to hear a bit more about the specific use case that's prompted this addition (even if only for posterity). :+1:

As you may know SCTP is a transport level protocol like TCP and UDP.
All major distribution have SCTP enabled.
I use SCTP everyday during development, coding both in C and Erlang.
To use SCTP I need the host to have SCTP enabled in kernel.
This is my specific need, and since I use boot2docker I would see this harmless change to go mainstream and forget about building my boot2docker iso every time there is a release or use another vm (with another distro) as docker host.
This PR is related and enable SCTP in official erlang image:
https://github.com/c0b/docker-erlang-otp/pull/16

Excellent, thanks @colrack!


Let's improve documentation to clarify ambiguity about names/paths

I've had lingering confusion due to fact that routes can have separate names and paths, and going back through the documentation didn't really clarify it for me.
For example...

If I don't specify a name for a route, does it remain nameless, or does it use the path for the name? (for example, how would I Link or transitionTo() it?

Can I make a Link to a path instead of a name?

and especially, can I make a Link to a relative path?  I get the impression the answer is no, though I think such a feature would be handy in some scenarios.

can I transitionTo() a relative path?

Or a path relative to the site root, if not a full URL?
can I put params/query in the path argument string, if need be?

transitionTo() accepts a routeNameOrPath argument.  How exactly is potential ambiguity in this resolved?  For example if I have the following routes (contrived example for the sake of clarification):

    <Route name="foo" path="bar" ... />
    <Route name="bar" path="foo" ... />

and I transitionTo('foo'), which route will it pick?

I agree :)
In 1.0 there are no more named routes, only paths.

ahhhhh glad to hear it.  How stable is 1.0 beta?  If it's pretty stable now I'll switch to it for my company project.


Add possibility to add multiple volumes

Other than the working directory, one might need to add more volumes to the job (eg: for reference data).
This would require the following:

be able to add more than one volume in the workflow config
be able to define the path where to mount the volume?

To me the second option seems cleanest, because there is no redundant 'volume' keyword?

Also you'll be able to "mix" both configurations
jobs {
        volume = { type "host" name "scratchdir" }

        volumes = [
            { type "host" name "one" path "/var/data1"},
            { type "host" name "two" path "/var/data2" }
        ]
    }

in this case, scratchdir will be mounted as workDir by default
Or you can specify which volume mount as workdir :
{ type "host" name "second" workdir true }


Review Rules one more time before launch

https://www.synapse.org/#!Synapse:syn22277124/wiki/604831

Doc link can be found here. Main feedback:
**Organization and Structure **
Lots of good content here. Tried to group relevant bullet points together.
(1) Registering for NLP Sandbox: more tactical rules that affect developer workflow
(2) Publication Considerations: concerning publications
(3)  Service Agreement: less tactical rules, more the obligatory legal jargon for scenarios of conflict or malicious intent
Consolidating some sections

Wondering if we can consolidate some of the below sections, comments in line

Terms of Use

Terms of Use currently says, not sure if this is a placeholder for a template we need from governance?  “BE SURE TO CONTACT GOVERNANCE ABOUT THIS PROJECT”

A point of consideration for the future: per Meg Doerr's readability guidelines, I used Microsoft word to get some readability statistics.
The NLP Sandbox rules are currently readable for a college student or graduate.
If we want to make them readable to a wider audience of Americans, we may want to do a deeper dive on breaking down some of the language. However, given that there is a lot of obligatory legal language here, it may be challenging.
No action required, just a consideration for the future.


Add 2FA support

Great project and works really well.
2FA would be a great addition!

I'm glad you're liking the project so far!
Awesome addition! Surely added to next milestone.

Issue frozen until having an email server.


Generated source files from :pre_build: command hook aren't compiled

Hi,
Firstly, I want to say thanks for all your work on Ceedling.
I have a (probably niche) issue when using Ceedling with the command_hooks plugin, specifically with the pre_build hook.
Due to the nature of the project, we have some source files (.c and .h) that need to be generated before running our tests. I've setup a command hook which just calls a script to generate these files. They are created in the test/support folder.
:command_hooks:
  :pre_build:
    :executable: cmd.exe
    :arguments:
      - /c
      - "scripts\\pre_build.bat"

This all works as expected and the hook runs successfully. However, it appears that Ceedling generates its list of source/support files before the pre_build hook is run. This means that the generated files aren't compiled as they weren't picked up earlier.
This only happens the first time you run the Ceedling project with a clean repository. By the second time you run it, then the source files have already been generated, and everything works as expected.
Without being familiar with the code base, an easy fix for this seems to be just to move the invocation of the pre_build hook to somewhere closer to Ceedling starting up.
Thanks,
Adam

Environment:

Windows 11
Ceedling Version:

ceedling version
Welcome to Ceedling!

  Ceedling => 1.0.0-f899db8
  ----------------------
  C:/tools/ruby33/lib/ruby/gems/3.3.0/gems/ceedling-1.0.0/

  Build Frameworks
  ----------------------
       CMock => 2.6.0
       Unity => 2.6.0
  CException => 1.3.4

This is a good use-case that we'll have to look at after this upcoming release. It's going to require some thought. There's a catch-22 here. The pre_build and pre_release hooks are meant to fire just before a build or release... which we need to know what we're building/releasing before that point. Your usage makes a lot of sense, though. We'll just have to think of how to support both.


release-21.2: ui: create error Boundary component

https://github.com/cockroachdb/cockroach/pull/72796 --- Release note (ui change): New page for when something goes wrong and the page crashes.

Closing - no docs needed


Use get_nvml() in tests

Use more sophisticated check for nvml in tests.
Thanks to @flx42 for the report!

Looks good to me!


Updating names of tasks and their descriptions

provision desc is so loooong that the output is not looking so elegant...
there is also --summary option to put more text...


Fix tests for openstack runs

With this, I can run tox -epy38 -- --pdb tests/test_fixtures.py::test_kubernetes_node_join on OpenStack and libvirt and both pass


Update the "Sending doesn't work!" section of the first tutorial

Describe the bug
The first tutorial has a section titled "Sending doesn't work!" with advices regarding some possible errors when first using RabbitMQ. The section states that "Maybe the broker was started without enough free disk space (by default it needs at least 200 MB free) and [...]".
However, according to the docs, RabbitMQ actually needs 50 MB of free disk space at a minimum (not 200 MB as the section states).
Reproduction steps
.
Expected behavior
.
Additional context
The .NET tutorial is correct.
If possible, I'd like to work on this issue.

It doesn't really matter what the absolute minimum is (to not trigger a disk alarm). The point is that the user should pay attention to what partition the data directory is on, and asking for at least 200 MiB is perfectly fine.
A node that just has enough to not go into resource alarmed state won't be of much use in practice. Most workloads, in fact, nearly all of them, require at least some disk space.
As of 4.0, transient entities (e.g. transient queues) will be a thing of the past, so if anything, I think the low free disk space watermark will be revisited higher, for example, to 200 MiB.

Oh, right, thanks for explaining. I just found it strange and conflicting that the .NET tutorial states 50 MB as the default, while the other languages tutorials say the default is 200 MB.

The .NET tutorial may be the oldest one there is (or at least the second oldest, after the Java one).
I will take a look.


Raise errors on tagging buckets with aws:*

Cannot tag S3 buckets with reserved tag key space aws:

Coverage decreased (-0.05%) to 94.241% when pulling 0c191ac33b3f38a05bd41ed8ee1e082c926de3d4 on mikegrima:s3tags into acfdfb7d7c08be05a02101ff10cf467015bb0158 on spulec:master.


Parse Error on one server but not another

I have pushed code to production and I'm seeing this error on an upload picture to EPS:
array(1) { [0]=> array(6) { ["ShortMessage"]=> string(16) "XML Parse error." ["LongMessage"]=> string(22) "XML Error Text: "{0}"." ["ErrorCode"]=> string(1) "5" ["SeverityCode"]=> string(5) "Error" ["ErrorParameters"]=> array(1) { [0]=> array(1) { ["ParamID"]=> string(1) "0" } } ["ErrorClassification"]=> string(12) "RequestError" } }

I can make requests to get the official time and get seller dashboard, both work and return a successful response.
I've never seen this before and I'm stumped. It's a simple image upload to EPS which works on dev (using the exact same image) but fails on a production server. I'm wondering, am I missing a specific PHP library or something else? Both are using php 7.2
I have compared the output of $request->toRequestXml() on both servers and they're the same string that's being sent leading me to believe it's a config error.

Turns out it is https://github.com/davidtsadler/ebay-sdk-php/issues/180
This is a serious problem!

Because people don't close issues about existing problems :thinking:

I love developers who are childishly passive aggressive. What exactly are you contributing to this issue?


Setting could not be opened for GoogleFonts v2.0.7

Do you by chance have the ability to open the developer console (CTRL+SHIFT+I)?
If so, would you be able to report any errors that show up after you attempt to open the settings panel?
If not, I'll have to look into this tomorrow.

@notmike101 nope, i have no idea how to work on it lol, i dont even have NodeJS installed, if it's simple i can try it.

and also i deleted the .config json as you suggested

Fixed this, will be pushing up a fix in a few minutes.
For context, the issue was the lack of import { React } from 'betterdiscord/bdapi' in the code, so the typescript compiler assumed to use normal React.

Please download v2.0.8 to resolve this issue
https://github.com/notmike101/betterdiscord-google-fonts/releases/download/v2.0.8/betterdiscord-google-fonts.plugin.js

@notmike101 thank you for fixing the issue!


fix: Broken backlink alert sends double messages

Fix #51

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

v1.3.4
GitHub release

Your semantic-release bot :package::rocket:


lc trie: fix memory leak.

Risk Level: Low
Testing: curl -s <IP_ADDRESS>:<admin_port>/stats | grep server.memory
Docs Changes: n/a
Release Notes: n/a
Signed-off-by: Piotr Sikora<EMAIL_ADDRESS>

cc @rshriram @mattklein123 @brian-pane

Thanks for the fix. Given some more time to hack on this, I think we could eliminate the need to pre-allocate such a large buffer in the first place. But for now, this one-line fix should be a big win.

@rshriram I think that the 100 Trie's will get built serially, so memory usage will balloon by 16MB above what's needed and then shrink down.

Ah its built sequentially? Then this is fine.
So I validated the this branch against my local setup. The RSS was back at at normal levels.. the heap profiles also stayed at the normal (pre bug-introduction) levels.

You can validate memory use by looking at tcmalloc stats, e.g. https://github.com/envoyproxy/envoy/blob/master/source/common/memory/stats.cc#L14.

If you want to file an issue for the test as a followup, I will merge this, as it seems harmless and Istio needs this for a release.


Pydantic allows weird types on field with nested model

Currently i have this script:
from typing import List
from uuid import uuid4, UUID

from pydantic import BaseModel, Field

class Id(BaseModel):
    value:UUID = Field(default_factory=uuid4)

class User(BaseModel):
    id:Id = Field(default_factory=Id)
    roles:List[str] = ['student']

class Project(BaseModel):
    id:Id = Field(default_factory=Id)
    creator:User

id = Id()
print(id)

user = User()
print(user)

project = Project(creator={})
print(project)

The script should run as it is
When i instantiate a project, pydantic allows me to put non User objects, like dicts, strings, Id(objects of type Id) and some others, but i feel this is not expected. The weirdest thing is the fields are auto filled with an object of the correct type when a bad value is passed
what is happening here? Thanks in advance!

This looks like desired behavior to me. What exactly is the issue?
Could you please modify the python script to add an assertion that would pass if the behavior matched what you want, and doesn't pass with today's existing behavior.
e.g. you say "put non User objects". Where? How? (.parse_obj() or User(attr=val))
Note that the default behavior of pydantic is to coerce values into the schema. So if you give a list of ints to roles, those ints will be cast into strings.  (This can be changed per-model if required.)

Note that something like User(x='x') will pass, and that's designed behavior, even though you're passing an unexpected argument and omitting the usual ones.
What's happening is that by default pydantic discards unexpected arguments (e.g. x). If you want the whole model to fail validation when unexpected additional fields are added, set extra to forbid..
All of your fields in all or your models have default values, so pydantic will use default values if they're missing.

Sorry for the delay, i studied Pydantic and learned that it does not enforce types on model instantiation, i managed to go around this, it was more a unexpected behavior from my side, but it is ok, thanks!


Can't disable Elytra on dark steel armor

Works fine with gliders, I can enable and disable those. With Elytra they stay on no matter what. A bit annoying when you try to fly with angel wings and you have no choice but to enter elytra mode when you want to do a quick landing ;)
Luv EnderIO <3
Affected Versions (Do not use "latest"):

EnderIO: <IP_ADDRESS>
EnderCore: <IP_ADDRESS>
Minecraft: 1.10.2
Forge: <IP_ADDRESS>8

The Elytra-Upgrade uses the vanilla mechanics
Sent from my iPhone

On 16 Sep 2016, at 12:10, mrtomwolf<EMAIL_ADDRESS>wrote:
Warning: This message has had one or more attachments removed
Warning: (msg-19095-4.html).
Warning: Please read the "j-e-b-Attachment-Warning.txt" attachment(s) for more information.
Works fine with gliders, I can enable and disable those. With Elytra they stay on no matter what. A bit annoying when you try to fly with angel wings and you have no choice but to enter elytra mode when you want to do a quick landing ;)
Luv EnderIO <3
Affected Versions (Do not use "latest"):

EnderIO: <IP_ADDRESS>
EnderCore: <IP_ADDRESS>
Minecraft: 1.10.2
Forge: <IP_ADDRESS>8

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/SleepyTrousers/EnderIO/issues/3696
This is a message from the MailScanner E-Mail Virus Protection Service
The original e-mail message contained potentially dangerous content,
which has been removed for your safety.
The content is dangerous as it is often used to spread viruses or to gain
personal or confidential information from you, such as passwords or credit
card numbers.
Due to limitations placed on us by the Regulation of Investigatory Powers
Act 2000, we were unable to keep a copy of the original attachment.
The content filters found this:
MailScanner: Found a script in HTML message
--
Postmaster
MailScanner thanks transtec Computers for their support

Was confused since the controll options said enable/disable glider/elytra. When I press the button for disabling it says "Enabled Elytra".
I figured it was a bug since it seemed like I should be able to disable them. Guess I have to stick to gliders.

That is anoverride for disabling when it's stuck and for people who have problems with the "jump while falling" like me. ..
Sent from my iPhone

On 16 Sep 2016, at 12:23, mrtomwolf<EMAIL_ADDRESS>wrote:
Warning: This message has had one or more attachments removed
Warning: (msg-19095-14.html).
Warning: Please read the "j-e-b-Attachment-Warning.txt" attachment(s) for more information.
Was confused since the controll options said enable/disable glider/elytra. When I press the button for disabling it says "Enabled Elytra".
I figured it was a bug since it seemed like I should be able to disable them. Guess I have to stick to gliders.
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/SleepyTrousers/EnderIO/issues/3696#issuecomment-247567576
This is a message from the MailScanner E-Mail Virus Protection Service
The original e-mail message contained potentially dangerous content,
which has been removed for your safety.
The content is dangerous as it is often used to spread viruses or to gain
personal or confidential information from you, such as passwords or credit
card numbers.
Due to limitations placed on us by the Regulation of Investigatory Powers
Act 2000, we were unable to keep a copy of the original attachment.
The content filters found this:
MailScanner: Found a script in HTML message
--
Postmaster
MailScanner thanks transtec Computers for their support


Unfortunately this library plain doesn't work (2024) - it looks for cert.pem even if you're using PATs

per the doco:
var apn_options = {
	token: {
		key: p8key,  // try file, string, etc
		keyId: "PL62KVXW34",
		teamId: "BGP36TQ2W7"
	},
	production: true
}
var apnProvider = new apn.Provider(apn_options)

result ...
node:fs:581
  return binding.open(
                 ^

Error: ENOENT: no such file or directory, open 'cert.pem'
    at Object.openSync (node:fs:581:18)
    at Object.readFileSync (node:fs:460:35)
    at resolveCredential (/home/ubuntu/main/node_modules/apn/lib/credentials/resolve.js:16:13)
    at loadCredentials (/home/ubuntu/main/node_modules/apn/lib/credentials/load.js:11:13)
    at loadAndValidate (/home/ubuntu/main/node_modules/apn/lib/credentials/prepare.js:11:20)
    at config (/home/ubuntu/main/node_modules/apn/lib/config.js:38:20)
    at new Client (/home/ubuntu/main/node_modules/apn/lib/client.js:11:19)
    at new Provider (/home/ubuntu/main/node_modules/apn/lib/provider.js:12:19)
    at Object.<anonymous> (/home/ubuntu/main/main.js:545:19)
    at Module._compile (node:internal/modules/cjs/loader:1378:14) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: 'cert.pem'
}

I tried setting cert to null, etc .. no good.
Shame!

I'm still using this library. I set the value of key to the base64 value of the p8 file generated from my Apple developer account.
So run base64 <filename>.p8 and put the resulting value at apn_options.token.key


rnn_cell.py improvements

Motivation
Current implementation of rnn_cell.py does not support custom initialization on a gate and input-to-hidden/hidden-to-hidden level (like setting forgetgate bias to 0 while leaving updategate bias at 1, etc.). Further when debugging the gates in TensorBoard, the matrix is represented as one large matrix, which makes it difficult to see whats happening inside a specific matrix (e.g. hidden-to-hidden) of a specific gate (e.g. forgetgate).
In recurrent neural networks (RNNs) GRU and LSTM uses various gates with separate weights, for both hidden-to-hidden and input-to-hidden, to computing steps in a recurrent sequence.
To optimize computational speed these gates, and their separate weights, are often stacked and computed simultaneous at every step.
In TensorFlows rnn_cell.py the GRUCell and BasicLSTMCell are implemented using linear to handle weigths and computation hereof.
However, the implementation of linear does not initialize separate matrices for each gate, but initializes the gates, and their input-to-hidden/hidden-to-hidden matrices, as one big matrix for weights and bias.
Proposal
Implementing separate initialization of gates, and their input-to-hidden/hidden-to-hidden matrices, and concatenating these gates. This allows custom initializatio, TensorBoard information on hid_in/hid_hid/bias for every gate and still retains the advantage of weights in a large matrix.
Implementation
With minimal rewriting of the current structure in rnn_cell.py I implemented a lasagne-gate like structure, made a new linear function and made some minor changes to GRUCell.
All of these changes should, with minor modifications, work for BasicLSTM as well.
Do notice that code below is for my own purpose, it is not rigorously tested yet.
First, the gate to hold initialization for every weight matrix/bias in a gate (notice it also handles LSTM's by having W_cell)
class Gate(object):
  """Gate to handle to handle initialization"""

  def __init__(self, W_in=init_ops.random_normal_initializer(stddev=0.1),
               W_hid=init_ops.random_normal_initializer(stddev=0.1),
               W_cell=init_ops.random_normal_initializer(stddev=0.1),
               b=init_ops.constant_initializer(0.),
               activation=None):
    self.W_in = W_in
    self.W_hid = W_hid
    # Don't store a cell weight vector when cell is None
    if W_cell is not None:
        self.W_cell = W_cell
    if b is not None:
      self.b = b
    # For the activation, if None is supplied, use identity
    if activation is None:
        self.activation = control_flow_ops.identity
    else:
        self.activation = activation

A modified GRU cell to handle weigths (took only minimal modification to init and call)
class GRUCell(rnn_cell.RNNCell):
  """Gated Recurrent Unit cell (cf. http://arxiv.org/abs/1406.1078)."""

  def __init__(self, num_units, input_size=None,
               resetgate=Gate(W_cell=None, activation=sigmoid),
               updategate=Gate(W_cell=None, activation=sigmoid),
               candidategate=Gate(W_cell=None, activation=tanh)):
    self._num_units = num_units
    self._input_size = num_units if input_size is None else input_size
    self._resetgate = resetgate
    self._updategate = updategate
    self._candidategate = candidategate

  @property
  def input_size(self):
    return self._input_size

  @property
  def output_size(self):
    return self._num_units

  @property
  def state_size(self):
    return self._num_units

  def __call__(self, inputs, state, scope=None):
    """Gated recurrent unit (GRU) with nunits cells."""
    with vs.variable_scope(scope or type(self).__name__):  # "GRUCell"
      with vs.variable_scope("Gates"):  # Reset gate and update gate.
        # We start with bias of 1.0 to not reset and not update.
        r, u = array_ops.split(1, 2, Modified_linear([inputs, state],
          [(self._num_units, "Reset", self._resetgate),
           (self._num_units, "Update", self._updategate)]))
        r, u = self._resetgate.activation(r), self._updategate.activation(u)
      with vs.variable_scope("Candidate"):
        c = Modified_linear([inputs, r * state],
          (self._num_units, "Candidate", self._candidategate))
        c = self._candidategate.activation(c)
      new_h = u * state + (1 - u) * c
    return new_h, new_h

I found linear required the largest amount of rewriting, however I have tried to keep the original structure and functionality intact.
def Modified_linear(args, output, scope=None):
  """Modified linear takes args and output.
     Args is same as in linear, but output is a tuple consisting of:
     output_size, name of gate, gate object (with all initializations)
  """
  if args is None or (isinstance(args, (list, tuple)) and not args):
    raise ValueError("`args` must be specified")
  if not isinstance(args, (list, tuple)):
    args = [args]
  if not isinstance(output, list):
    output = [output]
  shapes = [a.get_shape().as_list() for a in args]
  for shape in shapes:
    if len(shape) != 2:
      raise ValueError("Linear is expecting 2D arguments: %s" % str(shapes))
    if not shape[1]:
      raise ValueError("Linear expects shape[1] of arguments: %s" % str(shapes))

  matrices = []
  biases = []
  with vs.variable_scope(scope or "Linear"):
    for output_size, name, gate in output: # loops over every gate
      with vs.variable_scope(name):
        W_in = vs.get_variable("W_in", [args[0].get_shape()[1], output_size],
          initializer=gate.W_in)
        W_hid = vs.get_variable("W_hid", [args[1].get_shape()[1], output_size],
          initializer=gate.W_hid)
        if hasattr(gate, 'b'):
          b = vs.get_variable("Bias", [output_size],
            initializer=gate.b)
          biases.append(b)
        if hasattr(gate, "W_cell"):
          pass
          # do some LSTM stuff ...
        else:
          matrix = array_ops.concat(0, [W_in, W_hid]) # concats all matrices
        matrices.append(matrix)

  total_matrix = array_ops.concat(1, matrices) # concats across gates
  res = math_ops.matmul(array_ops.concat(1, args), total_matrix) # computes the results

  if biases is not []:
    total_bias = array_ops.concat(0, biases) # concats across gates biases
    if total_matrix.get_shape()[1] != total_bias.get_shape()[0]:
      raise ValueError('Must have same output dimensions for W and b')
    res += total_bias
  return res

Questions

Would this be of interest for a PR to rnn_cell.py? (given further development of code and BasicLSTMCell implementation)
General comments/thoughts would be much appreciated

Two comments:

Gate seems like an overkill class.
If you want to allow setting the nonlinearity and initialization of a particular gate, why not just accept and store these parameters directly?

We're deprecating linear() in favor of the layers functions, like fully_connected. In fact, linear can and will disappear as soon as can make this happen.

@ebrevdo I agree with the gates, also they would only work for basic GRU/LSTM.
I have updated the code based on your comments (though omitted fail-safes for clarifying functionality).
Would this be compatible with your plans on GRUCell and BasicLSTMCell?
class GRUCell(rnn_cell.RNNCell):
  """Gated Recurrent Unit cell (cf. http://arxiv.org/abs/1406.1078)."""

  def __init__(self, num_units, input_size=None,
               reset_W_in = init_ops.random_normal_initializer(stddev=0.1),
               reset_W_hid = init_ops.random_normal_initializer(stddev=0.1),
               reset_b = init_ops.constant_initializer(0.),
               reset_activation = sigmoid,
               update_W_in = init_ops.random_normal_initializer(stddev=0.1),
               update_W_hid = init_ops.random_normal_initializer(stddev=0.1),
               update_b = init_ops.constant_initializer(0.),
               update_activation = sigmoid,
               candidate_W_in = init_ops.random_normal_initializer(stddev=0.1),
               candidate_W_hid = init_ops.random_normal_initializer(stddev=0.1),
               candidate_b = init_ops.constant_initializer(0.),
               candidate_activation = tanh):
    self._num_units = num_units
    self._input_size = num_units if input_size is None else input_size
    self._reset_W_in = reset_W_in
    self._reset_W_hid = reset_W_hid
    self._reset_b = reset_b
    self._reset_activation = reset_activation
    self._update_W_in = update_W_in
    self._update_W_hid = update_W_hid
    self._update_b = update_b
    self._update_activation = update_activation
    self._candidate_W_in = candidate_W_in
    self._candidate_W_hid = candidate_W_hid
    self._candidate_b = candidate_b
    self._candidate_activation = candidate_activation

  @property
  def input_size(self):
    return self._input_size

  @property
  def output_size(self):
    return self._num_units

  @property
  def state_size(self):
    return self._num_units

  def _compute(self, args, gate):
    name, W_in_init, W_hid_init, b_init = gate
    with vs.variable_scope(name):
      W_in = vs.get_variable("W_in",
        [args[0].get_shape()[1], self._num_units],
        initializer=W_in_init)
      W_hid = vs.get_variable("W_hid",
        [args[1].get_shape()[1], self._num_units],
        initializer=W_hid_init)
      b = vs.get_variable("Bias", [self._num_units],
        initializer=b_init)
      matrix = array_ops.concat(0, [W_in, W_hid])
    return matrix, b

  def __call__(self, inputs, state, scope=None):
    """Gated recurrent unit (GRU) with nunits cells."""
    args = [inputs, state]
    with vs.variable_scope(scope or type(self).__name__):  # "GRUCell"
      with vs.variable_scope("Gates"):  # Reset gate and update gate.
        matrices = []
        biases = []
        gates = [
          ("Reset", self._reset_W_in, self._reset_W_hid, self._reset_b),
          ("Update", self._update_W_in, self._update_W_hid, self._update_b)]
        for gate in gates:
          matrix, bias = self._compute(args, gate)
          matrices.append(matrix)
          biases.append(bias)
        total_matrix = array_ops.concat(1, matrices)
        total_bias = array_ops.concat(0, biases)
        res_gates = math_ops.matmul(array_ops.concat(1, args), total_matrix)
        res_gates += total_bias
        r, u = array_ops.split(1, 2, res_gates)
        r, u = self._reset_activation(r), self._update_activation(u)
      with vs.variable_scope("Candidate"):
        candidate = ("Candidate", self._candidate_W_in, self._candidate_W_hid,
            self._candidate_b)
        matrix, bias = self._compute([inputs, r * state], candidate)
        c = math_ops.matmul(array_ops.concat(1, args), matrix) + bias
        c = self._candidate_activation(c)
      new_h = u * state + (1 - u) * c
    return new_h, new_h

Any news @ebrevdo?

We currently use single fused weight and bias variables in order to speed up computation.  Having separate initializers would lead to a significant slowdown.  You are welcome to write your own RNNCell which creates separate weight variables, each with their own initializers.

Closing this for now; as it sounds straightforward for you to create this custom LSTMCell in your own repository.

So does anyone test the implementation above? It would be very useful if we could easily implement our own RNNCells. :)


[syn] Rename INTERACTIVE to RUN_INTERACTIVE

INTERACTIVE collides with other environment setup.
If you do not like this approach, we can run dvsim with:
INTERACTIVE= ./util/dvsim/dvsim.py ...
CC: @jeoongp

Thanks @eunchan!


Unions can't have records?

As far as I know, the following schema is perfectly acceptable in avro:
{
  "type": "record",
  "name": "Test",
  "fields": [
    {
      "name": "event",
      "type": [
        "null",
        {
          "type": "record",
          "name": "Updated",
          "fields": [
            {
              "name": "id",
              "type": "string"
            },
          ]
        }
      ]
    }

But, the fact that I have a union where one type is not a primitive type (i.e. a record), seems to return "unsupported Union" errors when trying to deserialize (it builds the schema just fine, apparently).
Looking at the code, it indeed has:

            Value::Union(u) => match **u {
                Value::Null => visitor.visit_unit(),
                Value::Boolean(b) => visitor.visit_bool(b),
                Value::Int(i) => visitor.visit_i32(i),
                Value::Long(i) => visitor.visit_i64(i),
                Value::Float(f) => visitor.visit_f32(f),
                Value::Double(d) => visitor.visit_f64(d),
                _ => Err(de::Error::custom("Unsupported union")),
            },

in the deserialize code (deserialize_any).
Why isn't this supported?  Wouldn't it be easy to just have the Record type run a recursive parser on the body underneath?
Edit:
I found some code in deserialize struct that has:
        match *self.input {
            Value::Record(ref fields) => visitor.visit_map(StructDeserializer::new(fields)),
            Value::Union(ref inner) => match **inner {
                Value::Record(ref fields) => visitor.visit_map(StructDeserializer::new(fields)),
                _ => Err(de::Error::custom("not a record")),
            },
            _ => Err(de::Error::custom("not a record")),
        }

Is this saying that I have to deserialize unions with records straight into a struct (or, I assume, an Option to handle unions with "null"?)?  So, basically, skip the enum in the deserialized data structure and go straight to the one variant that matches the struct?

Just FYI this should match your schema:
#[derive(Debug, PartialEq, Clone, serde::Deserialize, serde::Serialize)]
#[serde(default)]
pub struct Updated {
    pub id: String,
}

impl Default for Updated {
    fn default() -> Updated {
        Updated {
            id: String::default(),
        }
    }
}

#[derive(Debug, PartialEq, Clone, serde::Deserialize, serde::Serialize)]
#[serde(default)]
pub struct Test {
    pub event: Option<Updated>,
}

impl Default for Test {
    fn default() -> Test {
        Test {
            event: None,
        }
    }
}


Response Code 404 (Not Found) with Node >= 14.19.x

Hello all,
I have recently installed Node version 14.19.2 in my WSL. Since upgrading to this version, I can no longer create a licence report because the command returns a 404 Not Found Error.
In another WSL instance with Node version 14.18.2, the creation of the report works perfectly. I have attached 2 screenshots with debug logging.
Can you please take a look at the problem? Downgrading my WSL instance to a version <14.19 is unfortunately not an option.
Many thanks and kind regards
Johannes
Output on Node 14.19.2:

Output on Node 14.18.2:

I have the same issue. Can we just skip the 404 package?

I tried to reproduce the issue, but without success.
My configuration:
OS:                 Windows 10 Pro 21H2
Patch Level:    Latests patches installed
Description:    Ubuntu 20.04.4 LTS
Release-Nr:     4.4.0-19041-Microsoft
I ran 'license-report' via npx as I did not install any package.
Here are the results of my test (I reduced the prompt to '>>>' to improve readability:
>>>$ export DEBUG=license-report*
>>>$ npx license-report --only=prod
npx: installed 47 in 6.353s
  license-report loading /home/entwicklung/license-report-issues/085/test-project/package.json +0ms
  license-report loading /home/entwicklung/license-report-issues/085/test-project/package-lock.json +6ms
Warning: the file '/home/entwicklung/license-report-issues/085/test-project/package-lock.json' is required to get installed versions of packages
  license-report:getPackageJson getPackageJson - REQUEST https://registry.npmjs.org/@kessler/tableify +0ms
  license-report:getPackageJson getPackageJson - REQUEST https://registry.npmjs.org/debug +4ms
  license-report:getPackageJson getPackageJson - REQUEST https://registry.npmjs.org/eol +1ms
  license-report:getPackageJson getPackageJson - REQUEST https://registry.npmjs.org/got +1ms
  license-report:getPackageJson getPackageJson - REQUEST https://registry.npmjs.org/rc +0ms
  license-report:getPackageJson getPackageJson - REQUEST https://registry.npmjs.org/semver +1ms
  license-report:getPackageJson getPackageJson - REQUEST https://registry.npmjs.org/text-table +1ms
  license-report:getPackageJson getPackageJson - REQUEST https://registry.npmjs.org/visit-values +0ms
[{"department":"kessler","relatedTo":"stuff","name":"@kessler/tableify","licensePeriod":"perpetual","material":"material","licenseType":"MIT","link":"git+https://github.com/kessler/node-tableify.git","remoteVersion":"1.0.2","installedVersion":"1.0.2","definedVersion":"^1.0.2","author":"Dan VerWeire, Yaniv Kessler"},{"department":"kessler","relatedTo":"stuff","name":"debug","licensePeriod":"perpetual","material":"material","licenseType":"MIT","link":"git://github.com/debug-js/debug.git","remoteVersion":"4.3.4","installedVersion":"4.3.4","definedVersion":"^4.3.4","author":"Josh Junon josh.junon@protonmail.com"},{"department":"kessler","relatedTo":"stuff","name":"eol","licensePeriod":"perpetual","material":"material","licenseType":"MIT","link":"git+https://github.com/ryanve/eol.git","remoteVersion":"0.9.1","installedVersion":"0.9.1","definedVersion":"^0.9.1","author":"Ryan Van Etten"},{"department":"kessler","relatedTo":"stuff","name":"got","licensePeriod":"perpetual","material":"material","licenseType":"MIT","link":"git+https://github.com/sindresorhus/got.git","remoteVersion":"11.8.5","installedVersion":"11.8.3","definedVersion":"^11.8.3","author":""},{"department":"kessler","relatedTo":"stuff","name":"rc","licensePeriod":"perpetual","material":"material","licenseType":"(BSD-2-Clause OR MIT OR Apache-2.0)","link":"git+https://github.com/dominictarr/rc.git","remoteVersion":"1.2.8","installedVersion":"1.2.8","definedVersion":"^1.2.8","author":"Dominic Tarr<EMAIL_ADDRESS>dominictarr.com"},{"department":"kessler","relatedTo":"stuff","name":"semver","licensePeriod":"perpetual","material":"material","licenseType":"ISC","link":"git+https://github.com/npm/node-semver.git","remoteVersion":"7.3.7","installedVersion":"7.3.7","definedVersion":"^7.3.7","author":"GitHub Inc."},{"department":"kessler","relatedTo":"stuff","name":"text-table","licensePeriod":"perpetual","material":"material","licenseType":"MIT","link":"git://github.com/substack/text-table.git","remoteVersion":"0.2.0","installedVersion":"0.2.0","definedVersion":"^0.2.0","author":"James Halliday<EMAIL_ADDRESS>http://substack.net"},{"department":"kessler","relatedTo":"stuff","name":"visit-values","licensePeriod":"perpetual","material":"material","licenseType":"MIT","link":"https://github.com/kessler/node-visit-values","remoteVersion":"2.0.0","installedVersion":"2.0.0","definedVersion":"^2.0.0","author":"Yaniv Kessler"}]
>>>$ node --version
v14.19.2
```

So can you see any differences on running the tests compared  to your way?

So can you see any differences on running the tests compared to your way?

The only difference is that I am using Debian 10.

OK, let's see, if I can try it on debian too.
But looking at the screenshots, it looks to me as so some package cannot be found on npmjs. Could you get me a package.json (I only need the dependencies and the devDependencies)?

I found the problem. I have a local dependency in my package.json file:
... "local-dep": "file:ext/local-dep-2.13.6.tgz", ...
When I remove this dependency the generation of the license report works perfectly.
I also tried it with a colleague in his WSL, who was still using an older version of licence-report. The generation worked perfectly for him. After updating to the current version of license-report, he also got the "404 Not Found" error.
The problem has nothing to do with the Node version. The problem seems to be that license-report in the new version can no longer handle local dependencies. Unfortunately, I had installed different versions of license-report on my WSL instances and therefore immediately suspected that it was due to the Node version.
Can you confirm this? Then I would adjust the description and the title of the issue.

to be honest: I never tested with local dependencies :-) Should have done that and will do that at once.
In one of the latest releases we changed to got as a replacement for the outdated request. So 'got' may handle file: sources other than 'request'.
I will fix this the next days.

Is it possible to not fail with 404?

That's the minimum requirement - I just didn't expect that there are "invalid" uris in the package-lock.json as I did not think about local dependencies - my fault.
So Step 1 is to catch errors and step 2 is to evaluate local dependencies.
But give me some days - I do not have the time for that the next few hours 😄

Solved by pr #87

When will 5.1.0 we available on npm?

I hope that it is done this evening.

Hi @BePo65,
can you please look into this again? The latest version on npm is still 5.0.2. Thank you.
Kind regards
Johannes


Remove firebase dependency

Description
Demoing auth was great in theory but doesn't easily allow others to test the core functionality of the iOS SDK. This removes that dependency and associated logic so that users can use the app without needing Firebase credentials.

Looks good! Small nits: for <textField>s we could move the placeholder property as the first so its easier to make this a bit more readable from a PR perspective.

Thanks! Those files are auto-generated by Storyboard, so unfortunately not much we can do as far as I'm aware. Moving to SwiftUI would for sure fix that, though :)

Looks good! Small nits: for <textField>s we could move the placeholder property as the first so its easier to make this a bit more readable from a PR perspective.

Thanks! Those files are auto-generated by Storyboard, so unfortunately not much we can do as far as I'm aware. Moving to SwiftUI would for sure fix that, though :)

Oh dang did not know these were auto generated!


dynalink in nashorn causing memory leak

Hi @szegedi ,
We have an enterprise application running on wildfly and using open JDK 11.0.22. We use Camunda for business process flow and Camunda uses Nashorn. In perf test we see a leak. Upon research we suspect our problem is same as
https://stackoverflow.com/questions/62593886/java-memory-leak-why-are-unreachable-objects-not-collected
This in turn takes us to JDK bug https://bugs.openjdk.org/browse/JDK-8229011
I know, Nashhorn is no longer part of open jdk since 15. We picked up the lib from github project https://github.com/openjdk/nashorn and we are using this as we are currently moving to use JDK 17.
So our question is if the bug raised in OpenJDK is still present in the latest release of nashorn 15.4? If yes, is there a plan to fix this
P.S. I did not see 'Issues' section under https://github.com/openjdk/nashorn. Hence posting this question here.

Hi @subijayb there's a known memory leak in Dynalink that was fixed in Java 16. This could be the same. Can you re-test your app with Java 16 or later (e.g. Java 17 you want to move to) and let me know if the problem still persists?

Thanks @szegedi for the quick reply.
It will take a little bit more time for our application to fully run on JDK 17 as we are doing jakarta upgrade also :)
Meanwhile, I tested with the reproducer attached in JDK-8229011, against 11 and 17. In 11 I see same stack in the hapdump once I import in MAT. But when I run on 17, the heap dump is clean.
I think this should fix our problem. Thanks !!
One change I did in the reproducer code to make it run on 17 is to use
final ScriptEngine scriptEngine = new org.openjdk.nashorn.api.scripting.NashornScriptEngineFactory()
      .getScriptEngine();

instead of
final ScriptEngine scriptEngine = new ScriptEngineManager().getEngineByName("nashorn");


PDF not compiling without code block

The commit Fixed issue with line breaks in code blocks introduces an error when there are no code blocks present in the file to compile. The error is as follows:
user@kali-64:~/Documents/OSCP-Exam-Report-Template$ ./generate-report.sh lab-report abc.pdf
[*] Preparing lab report for generation
[*] Stitching together targets, appendices and exercises
[*] Correcting relative image URLs
[*] Generating report
[*] Creating PDF
Error producing PDF.
! LaTeX Error: Command \AlertTok undefined.

See the LaTeX manual or LaTeX Companion for explanation.
Type  H <return>  for immediate help.
 ...

l.328 \begin{document}

This showed up when running generate-report.sh from the original source. I work around it by creating a code block in the file Lab/Exercises/1.1.1 Exercise.md as follows:
# Exercise

Exercise here

```c
code here
```

I can create a pull request for this workaround but I would prefer a more elegant solution

Fixed by adding check before reassigning the code block functions.


Add upper bound to next major version for aws and azure dependencies.

Fixes: https://github.com/apache/beam/issues/25655

R: @AnandInguva


Development: Modernize chart visualization of active students and performance in exercises

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Motivation and Context

This PR changes the line chart implementation of the Active Students chart in each course card on the course management page and the Performance in Exercises chart in the student statistics tab of every course from Chart.js to ngx-charts.
If you take a detailed look at both representations of Active Students, you might notice that the old one provides data labels displaying the absolute value for every point, which the new one doesn't. I'm aware of this difference, but wasn't able to find a possibility to implement this with ngx-charts so far.
Description

The html templates as well as the underlying data processing logics in the components have been simplified and adapted in order to fit ngx-charts requirements.
Steps for Testing

Prerequisites:
Important: before deploying this PR to a test server, take a look at the current implementation. This includes hovering over the line, adjusting screen size, clicking on nodes in the chart (Performance in Exercises only) and clicking on parts of the legend (Performance in Exercises only).

1 Instructor
1 Course with students (at least one) and exercises
1 student participating on this course

Log in to Artemis as instructor
Navigate to Course Administration
Inspect the new implementation of Active Students. Again, hover over the line and adjust the screen size. The chart should now be responsive to the screen width.
Log out and log in as student again
Click on the course card for the corresponding course and enter the statistics tab
Scroll down to the Performance in Exercises chart and again, interact with it. Check whether the interaction is similar to the old implementation. The chart should be responsive to the screen width as well now.

Review Progress

Code Review

[x] Review 1
[x] Review 2

Manual Tests

[x] Test 1
[x] Test 2

Test Coverage

course-management-overview-statistics.component.ts: 66.67% | 90.32%
Screenshots

Old design of Performance in Exercises:

Old design of Active Students:

New design of Performance in Exercises:

New design of Active Students:

There is a merge conflict now, please resolve it


create inverse EntEntRelationTypes on the fly

use a Listener to create reverse Relation types when relation types are saved


Update grid client dependency to use rmb relays

grid client dependency should be updated when the client is ready https://github.com/threefoldtech/grid3_client_ts/issues/479
this should be only applied once the upgrade reaches mainnet

Given this will wait for a mainnet release of 3.9, it needs to be tracked on 3.10

verified and deployed successfully using relay


Split parsing+formatting to a separate library

Elixir v1.8 will include timezone support, which will allow Timex to shrink a bit. As Elixir calendar support improves, we want to ideally support it by an ecosystem of focused libraries. With this in mind, I would like to propose for timex to be split into:

Timex "core" - duration, interval, shifting
Timex parse+format - parsing, formatting (and the tokenization backbone)

This will allow people to cherry pick the features they need and complement with what Elixir core provides.

I'm +1 on this! I will plan to make this split between now and 1.8, as a new major version, 4.0. Definitely a fan of moving to a more a la carte approach to libraries, and with much of what Timex does increasingly moving to the standard library, the monolithic approach originally followed is less of a benefit.
I think the split you've defined here is a good one, though I may split out Timex's timezone support into a separate lib as well, so that between now and 1.8 there is still coverage there, but post 1.8, that dependency can just be dropped (or implement the new behavior if needed, but I suspect tzdata will do that, and Timex can drop it as a direct dependency).

I think there's a decision to be made about what formatting language should be used.
As far as I can see we have 3 options:

strftime - which is widely known, but very confusing and generally hard to maintain
"mnemonics" that Timex provides as the default formatting
CLDR format - it's the base for the new Java datetime library, it also provides an easy way for internationalizing the formatting - all the data is already available. The downside is that the format is not that well known.

@michalmuskala I would say whatever timex is doing, is what timex should continue to do in terms of backwards compatibility. Of course, there is nothing stopping a separate library from exploring another option, which is ultimately what solving this issue will enable. If parsing+formatting is out of timex, then the user can pick up any other option they prefer. :)

Yeah, I think initially I'd preserve the way things are done at present (though I want to swap out my old combine lib for nimble_parsec); but I would be open to the idea of adding CLDR formatting at some future point to the split-out parsing/formatting lib, I think it may ultimately be better off as a separate lib, as I doubt there is occasion where more than one style is chosen for a given application.


add wildcard support for ServiceMemberSkipDomains

resolve https://github.com/AthenZ/athenz/issues/2201
I'm going to add tests.

@havetisyan
Thank you for your review. I've updated the documentation

I forgot to commit with signed-off. I will rebase later.


Add tweaks/preference to disable amination and fade on windows 11

An additional step I have to do to speedup a little my Windows 11 installation (in particular on old computers) apart running tweaks from the script is to disable all animations, fades and shades from System properties - Advenced system configuration.
Could be useful to add this feature as 1 click option: disable animation or everything is fine, maybe with some context or suggestion, in tweaks section, Essential tweaks or Preferences I don't know where it fits better.
This could help older Computer to run faster, Windows 11 debloated isn't light anyway for some machines.

Already exsists in Essential-Tweaks "Set Display for Performance"

Got it, closed.
Thanks


Update small_network handshake to use chainspec hash

The small_network::Message::Handshake should have the network_name and protocol_version fields replaced with a new field chainspec_hash: Digest

@georgepisaltu please add points


Redirecting to: /api/routes when adding a new route

When I am trying to add a new route using the given CURL command. The route is not getting added to the ceryx and a message as Redirecting to: /api/routes is being shown, where would my installation could have gone wrong?

Can you provide the following to better understand what might be happening:

Ceryx version you are using (since 0.6.0 trailing slashes are mandatory)
The full command you are running

When I am trying to add a new route using the given CURL command. The route is not getting added to the ceryx and a message as Redirecting to: /api/routes is being shown, where would my installation could have gone wrong?

I got the same error, the issue is that you need to add an / at the end of the url, ie: /api/routes/, same goes for PUT

@Alevsk Thank you for the reply, issue is with the /

maybe an update of the documentation is required for this.


Engine has already been stopped

Every once in a while, I get the following error message in my graphql-yoga server:
Error: Error in Photon:
Engine has already been stopped
    at PhotonFetcher.<anonymous> (/Users/.../node_modules/@generated/photon/index.js:35:23)
    at Generator.throw (<anonymous>)
    at rejected (/Users/.../node_modules/@generated/photon/index.js:5:65)
    at processTicksAndRejections (internal/process/task_queues.js:86:5)

Restarting the graphql-yoga server solves the problem.

Hey everyone 👋, I cannot reproduce this issue anymore. I tried @gihrig's reproduction (by removing the yup's validation middlewares), photon does show an error, but sending a valid query again does work properly.
@pantharshit00 There's never been some code that would call .disconnect() when an error is thrown. Maybe something has been fixed on photon's side by then.
I'm closing for now, if someone experiences that issue again, please post a comment and I'll reopen

I can confirm: The error is not reproducible with preview-8.
I did not test with earlier versions.
I was not able to test with preview-10 due to issues with #372, but a new project created with preview-10, prisma2 init and graphql-auth example works as expected when run on the error reproduction routine above.
When I get my dev project working under preview-10, I'll post back with results.

Same error occurred for me when I had included the following to the main index.js file:
// Begin GraphQLServer
server
  .start(options)
  .catch(e => {
    console.error(e)
  })
  .finally(async () => {
    await photon.disconnect()
  })

Once I removed the photon.disconnect(), I stopped getting this error message. Basically, an error throw already disconnects the engine, and there’s no need to explicitly disconnect it again.


Can you support for ios 8. many thank @akosma.

Can you support for ios 8. many thank @akosma.

No, sorry, that won't be possible.

thanks


Unable to set borders around Annotation

As seen from the example picture, there should be a border around but after looking through lsp UI customization, I tried
local orig_util_open_floating_preview = vim.lsp.util.open_floating_preview
function vim.lsp.util.open_floating_preview(contents, syntax, opts, ...)
  opts = opts or {}
  opts.border = opts.border or border
  return orig_util_open_floating_preview(contents, syntax, opts, ...)
end

but it still ends up like this.

Is there something I'm missing?

The problem was I didn't set the borders when usinng folke/noice.nvim.  This was addressed in this commit https://github.com/folke/noice.nvim/commit/e34db671490ebfdc4268fb83602e1a7e06b91e69


I want to create a new Organization into existing Organization

My current docker plan is team and i have only 2 organization current running i cannot create 3 organization.
So, can u tell me how many orgnization do it support in docker team plan and also docker business plan for both plan
Please tell me
That’s it!

Hi mohdrashid1,
I'm trying to understand your issue better. When you say you can't create three organizations, do you mean three teams under your organization? We have a limit of 2 teams for organizations without an active subscription. There's no limit to the number of organizations you can create. Each organization has its own subscription.
It will speed up identifying the issue if you share your docker id with us.
Thank you,
Serj Karneichyk

Closing due to lack of engagement - if this is still relevant, pls leave a comment with more details as it can help us reconsider/investigate further 🙏


DON'T SEAL Unpublish special doc

Trello Card
##This is a PR opened to get more comments and is not feature-complete.
##Questions

Shall we call it 'withdraw' or 'unpublish' ?

master branch uses withdraw
trello card has been updated to call it 'unpublish'

By sending a 'gone' format, the document is now lost and unretrievable in publishing-api. I believe this requires an extension of publishing-api.

This pull request has been migrated to a new repo: https://github.com/alphagov/specialist-publisher-rebuild/pull/696


docs(tabs): migrate to /elements

Closes #829

Opened a couple of PRs to fix bugs that were spotted while reviewing this PR:

https://github.com/patternfly/patternfly-elements/pull/2446
https://github.com/RedHat-UX/red-hat-design-system/pull/866

Neither of which are blocking but wanted to reference them here.


When the code comes?

I'd like to try this on my own data.

I am really sorry for the delay, I was not watching the repo.
The files are here now!


tensorflow:libtensorflow_cc.so fails to build for --cpu=armeabi-v7a

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04.3
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): current master branch(28.06.2019)
Python version: 3.6
Bazel version (if compiling from source): 0.24.1
GCC/Compiler version (if compiling from source):7.4.0
CUDA/cuDNN version:Not Used
GPU model and memory:
Exact command to reproduce:
bazel build -c opt //tensorflow:libtensorflow_cc.so --crosstool_top=//external:android/crosstool --host_crosstool_top=@bazel_tools//tools/cpp:toolchain --cpu=armeabi-v7a --verbose_failures --cxxopt="-DTENSROFLOW_DISABLE_META" --jobs 8 --copt="-funsafe-math-optimizations" --copt="-ftree-vectorize" --copt="-fomit-frame-pointer" --copt=-DNO_LOCAL_MEM --copt=-DEIGEN_DONT_VECTORIZE_SYCL --copt=-DMDB_USE_ROBUST=0 --cxxopt="-fPIC" --cxxopt="-std=c++11"

Describe the problem
Trying to compile //tensorflow:libtensorflow_cc.so for armeabi-v7a and/or for arm64-v8a, both failing with the output:
ERROR: ../tensorflow/tensorflow/cc/saved_model/BUILD:40:1: no such target '//tensorflow/core:saved_model_portable_proto': target 'saved_model_portable_proto' not declared in package 'tensorflow/core' defined by ../tensorflow/tensorflow/core/BUILD and referenced by '//tensorflow/cc/saved_model:reader

TensorFlow (Mobile) for Android is deprecated. Is there a reason you cannot use TensorFlow Lite?

@jdduke thanks for replying.
I am fairly new to Tensorflow, and I am using it with C++ API, and since the tensorflow has more ops, I need to use it instead of the Lite.

@jdduke do we have any news in this !!!

You can use certain whitelisted TensorFlow operators from TensorFlow Lite using this approach. The label_image example shows how to use the C++ API.


Add link to http API

This package isn't the easiest to set up: getting BSD-DB installed is finicky, building the metadata store takes a long time and takes up a lot of disk space, etc.
To remedy this, I set up a hosted version of the library: gutenberg-http (demo page).
Mentioning all the core-contributors @hugovk @MasterOdin @sethwoodworth in case you'd like to take a look and maybe help out :)

Great stuff, thanks for setting this up!
(I have an occasionally-updated dump of some metadata here: https://github.com/hugovk/gutenberg-metadata)


Ability to send native graphQL queries

Is your feature request related to a problem? Please describe.
For advanced users, it would be nice to be able to send raw graphQL queries, e.g. for nested queries, that would never be part of 'core' Lagoon CLI.
Describe the solution you'd like
Perhaps a new command.

Do you think the output should just be the raw JSON response you would get from the API?

Yeah, raw JSON is fine. These users are expected to be advanced, so they can pipe jq if needed etc.

Adding to rewrite to-do list


Modify or change apply for badge button

the apply for badge button is the same when navigating the Project Badging or Event Badging Pages. This is confusing because if someone comes to the website with a motive of only applying they may end up clicking the wrong button since it is in the same location for both pages. the goal here is to strategize on where to place this badge.

@LamiWilliams can we see how to modify this?

I was at the DEI working group and I want to fix this issue

Sure @kaxada
Thanks, @ElizabethN (and Adeyinka). I'll work with that

I was in the DEI working group and I want to fix this issue

Oops, looks like we replied at the same time. Are you a designer? We can work on it together

I was in the DEI working group and I want to fix this issue

Oops, looks like we replied at the same time. Are you a designer? We can work on it together

I am a developer, I want to effect the change  after the design team is done

I was in the DEI working group and I want to fix this issue

Oops, looks like we replied at the same time. Are you a designer? We can work on it together

I am a developer, I want to effect the change after the design team is done

Oh, alright

@kaxada Please, find the design here
With this new direction, it might be worth considering putting the "Apply for Badge" button on the landing page too; since users no longer need to go the Event Badging or Project Badging page before finding the button to apply for either of them
cc: @OgoNelson

@kaxada Please, find the design here
With this new direction, it might be worth considering putting the "Apply for Badge" button on the landing page too; since users no longer need to go the Event Badging or Project Badging page before finding the button to apply for either of them
cc: @OgoNelson

Nice 👍, I agree with you, I think it's good that way

@LamiWilliams the implementation is awesome. Thanks, @adeyinkaoresanya and @ElizabethN for the contribution and inputs. @OgoNelson please go ahead and create a PR for this. I am happy to merge this when it is ready

@LamiWilliams the implementation is awesome. Thanks, @adeyinkaoresanya and @ElizabethN for the contribution and inputs. @OgoNelson please go ahead and create a PR for this. I am happy to merge this when it is ready

Done and I have created a PR


Reduces the scope of requested JIRA permissions to only those currently in use

What this PR changes
I've removed the ADMIN scope from the requested JIRA permissions and added DELETE.
Why
I'm trying to get this implemented in our organization and InfoSec is balking at the wide-ranging scope of permissions requested for JIRA. I took a look through the JIRA REST API calls being made and none of them are documented as needing ADMIN in order to function. Additionally it looks like some of the API calls requires a scope (DELETE) that isn't specifically requested, so I added that.
Below are the JIRA REST API calls from this integration, along with their API documentation link and required scope. Please let me know if I missed any of them, or whether there is an undocumented reason why the ADMIN permission is needed.
 We definitely need to better document what's required here. I need to double check with Atlassian on the new endpoints that were added for the migration from DVCS. Since that disables the DVCS connector and enables this integration, I think it requires ADMIN solely to perform that migration task. I'll research this and get back to you.

@tcbyrd, thanks for following up and please let me know what Atlassian says regarding the DVCS-migration endpoints.

Thanks @tpechacek! I tested this out and all functionality including migration is still working without ADMIN. I'm deploying this now, so the descriptor should be updated on the Marketplace by Monday morning.
Thanks for doing the research on this @calvincestari. I'll get this added to our documentation as well.


Integrate wasi-nn

This has the very beginnings of wasi-nn support integrated into Wasmtime. I marked this as a draft because I need help working through some of the generation issues:

on @pchickey's suggestion, I am trying to only bring in the wasi_ephemeral_nn.witx with no external dependencies
after pulling in the repository containing wasi_ephemeral_nn.witx in 97d03e1 (this is likely not the right way to do things--just a temporary hack), I attempt to use the wasmtime_wiggle::wasmtime_integration! and wiggle::from_witx! in c18d421 to create the necessary structures
I get errors but it is unclear to me what is causing them:

wasmtime/crates/wasi-common$ cargo build
   Compiling wasi-common v0.19.1 (wasmtime/crates/wasi-common)
error[E0106]: missing lifetime specifier
 --> crates/wasi-common/src/wasi.rs:6:1
  |
6 | / wiggle::from_witx!({
7 | |     witx: ["WASI/phases/snapshot/witx/wasi_snapshot_preview1.witx", "WASI/phases/ephemeral/witx/wasi_ephemeral_nn.witx"],
8 | |     ctx: WasiCtx,
9 | | });
  | |___^
  |
  = note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)
help: consider using the named lifetime
  |
6 | wiggle::from_witx!({
7 |     witx: ["WASI/phases/snapshot/witx/wasi_snapshot_preview1.witx", "WASI/phases/ephemeral/witx/wasi_ephemeral_nn.witx"],
8 |     ctx: WasiCtx,
9 | });<'a>
  |

error[E0392]: parameter `'a` is never used
 --> crates/wasi-common/src/wasi.rs:6:1
  |
6 | / wiggle::from_witx!({
7 | |     witx: ["WASI/phases/snapshot/witx/wasi_snapshot_preview1.witx", "WASI/phases/ephemeral/witx/wasi_ephemeral_nn.witx"],
8 | |     ctx: WasiCtx,
9 | | });
  | |___^ unused parameter
  |
  = help: consider removing `'a`, referring to it in a field, or using a marker such as `std::marker::PhantomData`
  = note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)

error: aborting due to 2 previous errors

Some errors have detailed explanations: E0106, E0392.
For more information about an error, try `rustc --explain E0106`.
error: could not compile `wasi-common`.

To learn more, run the command again with --verbose.

I can attach the macro-expanded wasi.rs output if that would help but I'm not too sure where to start here.

I didn't update the Git submodules, 97d03e1 isn't going to work here (only locally). This is the wasi-nn state that I am using: https://github.com/WebAssembly/wasi-nn/compare/integration.

From a quick glance at the generated error messages, I think you might have just hit a bug in wiggle or your witx definition file specifies some interface functions in conflict with the expected lifetimes. Either way, I think this is something at the level of the generated trait by wiggle, and the lifetimes mismatch. For instance, it could be that the generated trait method is something like this:
trait WasiNN {
    fn do_stuff<'a>(&self, ptr: GuestPtr<'_, A>);
}

where 'a is unusued.
If you're not in super hurry, I can take a closer look tomorrow and help you debug it.


Account for prefix in migrate:generate #22

If a db table prefix has been set in config/app.php attempt to strip
that from generated migrations to avoid duplicate prefix when migrating
the generated files.

I will test this out this weekend :) Apologies for the delay

This seems to work yes, however, I found one issue. If you have a table with a part of the prefix in, it will get replaced. For example, if you have a table: 'user_permissions', and a prefix of 'r_', you will end up with a table 'r_user_permissions'.
However, because you replace all occurrences of it, it will actually rename the table to usepermissions, removing the _r in the middle as well. We will have to look at if the prefix occurs at the start of the string.


No loaders are configured to process this file

Hello Alex, good job!
I got the following error on:
const shader = createShaderProgram(gl, vertSrc, fragSrc)
Error: Module parse failed: Unexpected token (52:56)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|       identifier
|     } = declaration;
>     const attributeSize = Number(type.match(/([234])$/)?.[1]) || 1;
|     const location = gl.getAttribLocation(program, identifier);
|     handleGlError(gl, `ShaderProgramObject gl.getAttribLocation() ${identifier}`);
    at<EMAIL_ADDRESS>
My shaders are declared as a const string as you emphasize in your documentation and I do no longer use glsl files.
I have also reduced them to the minimum:
const vertSrc = /* glsl */ `
attribute vec2 aVertexPosition;
attribute vec4 aVertexColor;

void main(void) {
    vec2 position = aVertexPosition;
    vColor = aVertexColor;
}
` as const

const fragSrc = /* glsl */ `
varying lowp vec4 vColor;
uniform lowp float opacity;

void main(void) {
    gl_FragColor = vColor * opacity;
}
` as const

However, my code is running in storybook and I suspect something on that side...
Any idea ?

Thanks for taking a look at the lib!
Looks like webpack is choking on the optional ?. chaining operator. I think I have too modern of a compilation target set. I'll have an update on this soon.

Thanks to keep me updated me as soon as possible. Your approach is quite interesting. I plan to embed your lib in mine (on going work, not yet published on GitHub) - I have also other questions regarding Mat4 use, but let's fix this one first.

I just published 0.1.2 which actually respects the compilation target of ES2019.
That line now looks like this in the compiled output:
const attributeSize = Number((_a = type.match(/([234])$/)) == null ? void 0 : _a[1]) || 1;

which should play nicer with more babel/webpack configs.
Let me know if that helps!

Thanks Alex, works fine now !


update troubleshooting on mac

Summary
adds troubleshooting for mac on catalina
Ticket Link
fixes #3698

Thanks, @Willyfrog, this is really helpful. Some stylistic feedback but otherwise this is great. :) I appreciate you updating the docs!

Definitely better, thanks!

I changed, and committed, the order of "can" and "may". Otherwise it's good to go. :)


Configure the Service Ingress IP Controller (AutoExternalIPs)

This adds a Network configuration observer that populates the value in ingress.ingressIPNetworkCIDR.
This is used to automatically allocate the Service.ExternalIP field for services of type LoadBalancer.

Blocked until
https://github.com/openshift/api/pull/347
https://github.com/openshift/library-go/pull/465
are merged.
@danwinship PTAL

@squeed isn't this something that the kubernetes controller manager operator needs to consume?
fyi @openshift/openshift-team-master

@squeed isn't this something that the kubernetes controller manager operator needs to consume?

@adambkaplan No, I don't think so. This is exclusively an OpenShift feature. It's only on the OpenShiftControllerManagerConfig type in the controlplane api.

/approve

@adambkaplan It seems the build tooling for this repo doesn't check /vendor :-). Might want to add that.

OK, API and library-go changes have merged. Rebased, ready for final review. PTAL.

/hold cancel

Have you tested that this works? Is there a jira card or something to ensure we get QE testing of this feature so we don't break it again?

Looks like CI is broken -
make: *** No rule to make target `verify-deps'.

and that openshift/api has some kubebulder-breaking changes upstream.
go run ./vendor/github.com/openshift/library-go/cmd/crd-schema-gen/main.go --domain openshift.io --apis-dir vendor/github.com/openshift/api --verify-only Derived repo "github.com/openshift/cluster-openshift-controller-manager-operator" from GOPATH and working directory. Copying vendor/github.com/openshift/api/config to temporary pkg/apis... Creating kubebuilder manifests ... --- manifests/02_config.crd.yaml 2019-07-02 16:17:10.000000000 +0000 +++ /go/src/github.com/openshift/cluster-openshift-controller-manager-operator/280371287/02_config.crd.yaml 2019-07-02 16:19:09.861661792 +0000 @@ -40,7 +40,6 @@ managementState: description: managementState indicates whether and how the operator should manage the component - pattern: ^(Managed|Unmanaged|Force|Removed)$ type: string observedConfig: description: observedConfig holds a sparse config that controller has verification failed exit status 1 make: *** [verify-codegen-crds] Error 1

Have you tested that this works?
Need to play around with the CI environment.

Is there a jira card or something to ensure we get QE testing of this feature so we don't break it again?

It turns out the CI testing for this configures the controller directly. We need to lift that in to CI as well as updating the acceptance suite. I'll add a task.

/test e2e-aws

/skip verify-deps

@openshift/openshift-team-master can you please look into why the verify crds task fails (doesn't like that the Management state regex that was previously there).

Wait until process is sorted out in https://github.com/openshift/cluster-kube-apiserver-operator/pull/514

relevant to crd-gen issue: https://github.com/openshift/cluster-openshift-controller-manager-operator/pull/105

rebased, tested, ready for final review.

hey @sttts @danwinship can I get a final review?

@openshift/openshift-team-master can I get a final lgtm? This has been green for weeks and is ready to go.

CI failure seems unrelated...
/retest

/retest

/retest
argh flakes

/retest


EVM: remove chain ID check

for #305
remove the chain ID check. it's arguably part of verifying an transaction, which we don't do in the indexer for runtimes
this makes the decodeEthRawTx function a little more footgun if anyone else discovers it though

If the analyzer chainId is misconfigured, will the indexer error when connecting to the node?

which, the oasis-3 one?

Nvm I see that it was hardcoded before tx, err := decodeEthRawTx(utx.Body, 42262)


GH4032: Add missing ILMergeSettings

Using the documentation available here:
https://github.com/dotnet/ILMerge/blob/master/ilmerge-manual.md
added the missing settings that can be set when calling ilmerge.exe
Fixes #4032

@gep13 your changes have been merged, thanks for your contribution 👍


Updating dependencies to the actual version

I'm constantly keeping track of the growing number of packages in my project, and I noticed an increase in duplicate versions of dependencies that async-graphql uses. So I took a little liberty and updated dependencies that have had a new version for a long time. I looked at the changelog of all the updated packages, it shouldn't break anything 😁

This PR also requires updating opentelemetry in the examples. So I also made a PR to update the dependencies in the examples. https://github.com/async-graphql/examples/pull/82

thanks 🙂


Support order argument in toarray method of csc and csr

It supports order argument in toarray method of sparse matrixes.

LGTM!


Invalid yaml format (comma at end of line)

https://github.com/mateuszjansowa/callpage-gtm-template/blob/b5660a7e655338a03f661df4231876c04a8674f2/metadata.yaml#L16
The comma at the end of the documentation line is causing the template review process to fail.

Thank you @philpott
Resolved with the latest commit


Make SearchInputField uncontrolled and debouncing its value.

This does not show the loading indicator when someone is typing but rather debouncing the whole onChange handler of the SearchInputField whenever a new value was entered.
This also means the SearchInputField is uncontrolled and will not response to changes in the URL once mounted. I tried another way of storing a timestamp in the state of the input component to make sure changes coming from outside are distinguishable from setState changes inside the component. But that felt really hacky and at same time makes it hard to understand what is happening in SearchInputField component excatly.
Looking forward to hear your thoughts on this.

Due to changes on how the route name and the search bar expanded status will be connected with my next PR this uncontrolled search input field will show invalid values when switching between routes.


Grep function randomly skips tests

Description
Probably this issue is my specific, but this behavior is strange  related to grep by regex. I am using mocha programmatically.
when I am using "grep" function.
The the problem is in this method:
var match = self._grep.test(test.fullTitle());
(node_modules\mocha\lib\runner.js )
This method outputs false, while it should be true, but only for some 'it' (tests). My describe has 2 it, one is passed, the other not visible because of condition 'false'.
Example:
this one 'it' visible:
'this is some description @testing, @testing-smoke Ensure dashFeeSensitivity exist'
while this 'it' not:
'this is some description @testing, @testing-smoke Ensure dashMaxFloor exist'
bot of them should return 'true', but within mocha, the second example returns 'false':
/@testing |@testing$|@testing-smoke |@testing-smoke$/g.test('this is some description @testing @testing-smoke Ensure dashFeeSensitivity exist')
/@testing |@testing$|@testing-smoke |@testing-smoke$/g.test('this is some description @testing @testing-smoke Ensure dashMaxFloor exist')
for testing purpose (I know my regex),  I changed  self._grep on /@testing |@testing$|@testing-smoke |@testing-smoke$/g and it works
Versions
mocha - 8.4.0
node - 14.16.0, 10.15.3
OS - windows 10, 64
@babel/core - 7.8.4

@TAVkiev can you add a small test example, please?
Consisting of two files, one test and another containing the programmatic part.
Thank you.

@juergba Hay, I added those files as small git project
https://github.com/TAVkiev/mocha-grep-issue
From 3 tests only one is running

@TAVkiev yes, thats' a weird case! It's the global flag g which produces this true/false flipping.
let reIgnore = /a/g;

console.log(reIgnore.test("a")); // true
console.log(reIgnore.test("a")); // false
console.log(reIgnore.test("a")); // true
console.log(reIgnore.test("a")); // false

see docs: mozilla
Using test() on a regex with the "global" flag
When a regex has the global flag set, test() will advance the lastIndex of the regex. (RegExp.prototype.exec() also advances the lastIndex property.)

Further calls to test(str) will resume searching str starting from lastIndex. The lastIndex property will continue to increase each time test() returns true.

Note: As long as test() returns true, lastIndex will not reset—even when testing a different string!

Remove the global flag, then it works:

var mocha = new Mocha().grep(/@smoke |@smoke$/);
or via Mocha constructor: var mocha = new Mocha({grep: /@smoke |@smoke$/});

Thank you for your issue and test example. I will open a PR and fix it.

Indeed, loveliest regex :)
Thanks for your reply, in my case /g flag is really redundant, 'tag' should be only one


rzc discover exited with code 1

This is just for a reference, not sure if it is an issue or cross referencing. My application build started to fail when I added my TagHelper class library. This is working in 2.2.
Describe the bug
Build fails on this error: rzc discover exited with code 1
Other Info
Error: rzc discover exited with code 1.
C:\Program Files\dotnet\sdk\3.0.100-preview5-011568\Sdks\Microsoft.NET.Sdk.Razor\build\netstandard2.0\Microsoft.NET.Sdk.Razor.CodeGeneration.targets	99
Error is pointing here

My project
  <PropertyGroup>
    <TargetFramework>netcoreapp3.0</TargetFramework>
  </PropertyGroup>

My TagHelper class library
  <PropertyGroup>
    <TargetFrameworks>netcoreapp2.2;netstandard2.0</TargetFrameworks>
  </PropertyGroup>

To Reproduce
[HtmlAttributeName(DictionaryAttributePrefix = AspClassPrefix)]
public IDictionary<string, bool> Condition { get; } = new Dictionary<string, bool>();

Expected behavior
Not failing on build
Fix
Not easy to figure out from the error what the problem was but I have added set; and all is working ok.
[HtmlAttributeName(DictionaryAttributePrefix = AspClassPrefix)]
public IDictionary<string, bool> Condition { get; set; } = new Dictionary<string, bool>();

@NTaylorMullen, do we required tag helper properties to be settable?

@NTaylorMullen, do we required tag helper properties to be settable?

Yes

@VaclavElias this is by design. Tag helper properties are required to have a set;.

@ajaybhargavb does the IDE provide correct guidance when this fails? Requiring the setter is fine, but I think the primary complaint was that there was no indication that this was broken.

I was lucky to find the problem as the error "rzc discover exited with code 1" didn't really tell me what the problem was and that I had to dig in TagHelper :) If IDE could suggest it, that would be great but not sure how rare is this issue..

Got it. Yes, I agree rzc shouldn't hide the actual error message. I am going to track this work as part of https://github.com/aspnet/AspNetCore/issues/12661 as they both have the same underlying issue.

@VaclavElias I was able to repro your issue. But when I do it, I see a null-ref displayed in the output in white before rzc exited with code 1 in red. Do you see this as well? If not, I might need to dig more into why that wasn't printed for you.
If you do see the error, this was a bug we fixed later and I've confirmed it is no longer happening on the latest SDK.
Object reference not set to an instance of an object.
     at Microsoft.CodeAnalysis.Razor.BindTagHelperDescriptorProvider.CreateComponentBindTagHelpers(ICollection`1 tagHelpers)
     at Microsoft.CodeAnalysis.Razor.BindTagHelperDescriptorProvider.Execute(TagHelperDescriptorProviderContext context)
     at Microsoft.CodeAnalysis.Razor.CompilationTagHelperFeature.GetDescriptors()
     at Microsoft.AspNetCore.Razor.Tools.DiscoverCommand.ExecuteCore(RazorConfiguration configuration, String projectDirectory, String outputFilePath, String[] assemblies)
     at Microsoft.AspNetCore.Razor.Tools.DiscoverCommand.ExecuteCoreAsync()
     at Microsoft.AspNetCore.Razor.Tools.CommandBase.ExecuteAsync()
C:\Program Files\dotnet\sdk\3.0.100-preview5-011568\Sdks\Microsoft.NET.Sdk.Razor\build\netstandard2.0\Microsoft.NET.Sdk.Razor.CodeGeneration.targets(99,5): error : rzc discover exited with code 1. [C:\GitHub\temp\RazorPagesApp\RazorPagesApp\RazorPagesApp.csproj]

@ajaybhargavb does it show up in VS?

@NTaylorMullen, not the error list but it is in the Output window

@NTaylorMullen, not the error list but it is in the Output window

Hmm, is there anything we can do to also include the error in the output window? I know MSBuild looks over the output window meticulously to find logs of a certain format.

Talked offline, things work as I expect already. I was confused 😄

Yes, I get the same error but it doesn't tell me where it is 😶
Output Window:
Object reference not set to an instance of an object.
4>   at Microsoft.CodeAnalysis.Razor.BindTagHelperDescriptorProvider.CreateComponentBindTagHelpers(ICollection`1 tagHelpers)
4>   at Microsoft.CodeAnalysis.Razor.BindTagHelperDescriptorProvider.Execute(TagHelperDescriptorProviderContext context)
4>   at Microsoft.CodeAnalysis.Razor.CompilationTagHelperFeature.GetDescriptors()
4>   at Microsoft.AspNetCore.Razor.Tools.DiscoverCommand.ExecuteCore(RazorConfiguration configuration, String projectDirectory, String outputFilePath, String[] assemblies)
4>   at Microsoft.AspNetCore.Razor.Tools.DiscoverCommand.ExecuteCoreAsync()
4>   at Microsoft.AspNetCore.Razor.Tools.CommandBase.ExecuteAsync()
4>C:\Program Files\dotnet\sdk\3.0.100-preview6-012264\Sdks\Microsoft.NET.Sdk.Razor\build\netstandard2.0\Microsoft.NET.Sdk.Razor.CodeGeneration.targets(99,5): error : rzc discover exited with code 1.

Error List:

You have to have Sherlock Holmes skills to find it 😂🔎.

In my case I solved this error: "rzc discover exited with code 1.", by turning of "Real-time protection" in in Windows Security.
I exclude "dotnet.exe" from "Windows Defender Antivirus" scan and/or allow it through "Controlled folder access".