Revert "Add certs:generate command"

Reverts heroku/heroku#1346
@brentdax looks like those tests are a bit flaky https://travis-ci.org/heroku/heroku/jobs/46520477

Oof. ensure_openssl_installed! includes some code to make sure it only actually performs its test once; this wasn't always cleared between tests, so running the tests in the wrong order would cause the test that confirms the method actually calls out to openssl(1) to fail.
I've pushed a quick patch to my branch at dd5e7f64 which fixes this issue.

You'll need to make a new PR
On Fri, Jan 9, 2015 at 6:14 PM, Brent Royal-Gordon
<EMAIL_ADDRESS>wrote:

Oof. ensure_openssl_installed! includes some code to make sure it only actually performs its test once; this wasn't always cleared between tests, so running the tests in the wrong order would cause the test that confirms the method actually calls out to openssl(1) to fail.
I've pushed a quick patch to my branch at dd5e7f64 which fixes this issue.
Reply to this email directly or view it on GitHub:
https://github.com/heroku/heroku/pull/1351#issuecomment-69433848

@dickeyxxx Done (after a false start) as #1353. Thanks!


Updates: Newer MSU Version from Lenovo and auto wget utility

Hi!
Impressive work here!  I stumbled over this repo and gave it a whirl, it achieves quite the excellent accomplishment for Marvell RAID MSU.
I made a few improvements/fixes/updates in my fork: https://github.com/fermulator/marvell_msu_docker
(feel free to pull in if you want ;o)

Hi!
Fantastic! I soon as I have a little time left, I will merge in you PR and give it a try on my server.
Thank you for sharing your changes!

I merged your PR with some slightly changes:

I use a multistage build to avoid installing packages in the final image which aren't needed afterwards (unzip/wget)
I comment out the COPY because the build fails if no local file exists (I also change the documentation to this).

Thank you very much for the update.


Bxmsprod 753 7.39.x

@mbiarnes they need a different jobName or folde, otherwise they will collide with the master's ones
second thing, we should rebase https://github.com/kiegroup/kie-jenkins-scripts/pull/768 once it's merged

don't they have different folder on Jenkins? @Ginxo

@Ginxo the jobs (a-seed-job) for 7.39.x is in this folder KIE/7.39.x - so everything created will be created from KIE/7.39.x as root dir. So nothing will collide because the big difference here is master or 7.39.x. If you look now you will see the same folder in master or 7.39.x.

@Ginxo the changes of the additionalArtifacts are the same as on master


Too much column headers generated when using non working timeframes in cropped mode

When using cropped for non working period, the headers generator generates too much column. The first "non working" column is hide but the following are shown :

The column 20,21,22,23 should be hidden.
I will take a look in the header generation code and create a PR if I found the bug.
My timeframes configuration :
beforeDay:{
                        start: moment('00:00', 'HH:mm'),
                        end: moment('07:00', 'HH:mm'),
                        color: 'red',
                        magnet: false, // This will disable magnet snapping
                        working: false, // This is a resting period
                        default: true // It will be used for each day
                    },
                    day: {
                        start: moment('07:00', 'HH:mm'),
                        end: moment('19:00', 'HH:mm'),
                        color:'green',
                        working: true, // This is a working period
                        default: true // It will be used for each day
                    },
                    afterDay: {
                        start: moment('19:00', 'HH:mm'),
                        end: moment('23:59', 'HH:mm'),
                        color: 'red',
                        magnet: false, // This will disable magnet snapping
                        working: false, // This is a resting period
                        default: true // It will be used for each day
                    }

Same problem then issue https://github.com/angular-gantt/angular-gantt/issues/639


[gameone] Remove exctractor

[x] I've verified and I assure that I'm running youtube-dl 2018.01.14

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[x] Other

Site is down for a while and content is to and end a long ago too:
https://forum.kodi.tv/showthread.php?tid=186309

@dstftw Please review.


how do items and additionalItems interact in strict mode?

Maybe this is just a question: for schema type 'array' how are 'items' and 'additionalItems' expected to work together? I have yet to hit on a valid combination in strict mode.
<EMAIL_ADDRESS>(latest)
options
const options = {strict: true};

JSON Schema
const schema = {
  $id: 'x',
  type: 'array',
  items: [{ type: 'string' }, { type: 'string' }],
  additionalItems: { type: 'string', maxLength: 4 },
  minItems: 2,
  maxItems: 10
};

Sample data
data = ['a', 'b', 'c', 'd'];

Your code
const ajv = new Ajv(options);
ajv.addSchema(schema);
// throws on next line
const validator = ajv.getSchema('x');

// message: strict mode: "items" is 2-tuple, but minItems or maxItems/additionalItems are not specified or different at path "#"

Validation result, data AFTER validation, error messages
N/A
What results did you expect?
I expected it to succeed. items validations are ok, minItems passes, maxItems passes. and additionalItems seems like it should pass - the additional items are strings less than the length limit.
I understand that I can add the strictTuples: false option, but am wondering if there is any valid combination of items and additionalItems with option strict: true.

duplicate of #1417

if there is any valid combination of items and additionalItems with option strict: true

the only valid combination is additionalItems: false
see the comment in #1417

thanks. i had read 1417 but just didn't understand that the only valid setting was false. i appreciate your quick and concise clarification.


fix(authority-source-files): Add new error code to handle authority source file deletion

Purpose
Add error handling for authority source file deletion after authority was deleted
Approach
Add a new error code and a constrain for it in ApiErrorHandler
Changes Checklist

[ ] API Changes: Document any API paths, methods, request or response bodies changed, added, or removed.
[ ] Database Schema Changes: Indicate any database schema changes and their impact. Confirm that migration scripts were created.
[ ] Interface Version Changes: Indicate any changes to interface versions.
[ ] Interface Dependencies: Document added or removed dependencies.
[ ] Permissions: Document any changes to permissions.
[ ] Logging: Confirm that logging is appropriately handled.
[ ] Unit Testing: Confirm that changed classes were covered by unit tests.
[ ] Integration Testing: Confirm that changed logic was covered by integration tests.
[ ] Manual Testing: Confirm that changes were tested on local or dev environment.
[ ] NEWS: Confirm that the NEWS file is updated with relevant information about the changes made in this pull request.

Related Issues
MODELINKS-210

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Update and improve aesthetics on UI/UX

[ ] Logo not centered and insufficient spacing
[ ] Lack of contrast in search button
[ ] Incorrect sizing of search button
[ ] Large white space above footer
[ ] Insufficient spacing in footer

In spatial view, the background jumps around when different polygons in the map are hovered over.

@pieterprovoost can you check all this again (https://dev.oih.iode.org) and close ticket if ok please?


Completion for imports has regressed

Imports used to offer completions.
e.g. import 'dart:
                  ^^^^^^
After the move to Analysis Server (or possibly before) this has broken.
It doesn't work for package: either.
It doesn't work for part '.../' (I don't know if this used to work or not)

If it's still broken on Windows we should open a separate issue for it, but I'm going to close this because the basic support was added.

If it's still broken on Windows we should open a separate issue for it, but I'm going to close this because the basic support was added.


Toolbar plugin selection should indicate radio buttons

The plugin selection in the toolbar essentially functions as a group of radio buttons, as only one plugin can be selected at a time. However, the graphic for when a plugin is selected is a white check in a green circle, which indicates they might function as checkboxes, allowing multiple plugins to be selected simultaneously. When I used tota11y for the first time, I thought I could select multiple plugins because of this. Maybe it would be clearer to remove the check and just have a green circle, or something else that suggests radio button more than checkbox.

It's not a radio because you can uncheck, and it's not a checkbox because you can only have one selected at a time - so we met in the middle :) A radio with a checkmark!
I agree that the check mark can probably be safely removed, but I'm hesitant to change it right now. I'm doing some a11y work on a separate branch, which includes a bunch of focus styling so I may include it there.


Update python vspk gen to use bambou2 and isasync

This is for the 6.0 Nuage timeline where we want to support Python 3.7 and we move to a new nuagenetworks/bambou2 library.
This PR updates Python VSPK generation from monolithe perspective
PR on VSPK Generator: nuagenetworks/vspkgenerator#35
PR on Bambou2: nuagenetworks/bambou2#1

Close & reopen to trigger travis

Closing in favor of proper solution


Issue with searchable PDF

As of 8/31/2018, if a PDF is generated from the current image without setting the Input Name, the PDF is corrupt. The code sets up all the pointers to the PDF image resource, both in the content stream and the page resources but skips storing the encoded image itself which causes the indirect reference from the image to point to the next object in the file (in my case it was the PDF info dictionary). The problem can be prevented by setting the InputName to the name of the image being processed. I think this requirement can be relaxed as follows.
The current code in pdfrenderer.cpp:
  if (!filename)
    return false;

  L_Compressed_Data *cid = nullptr;
  const int kJpegQuality = 85;

  int format, sad;
  findFileFormat(filename, &format);
  if (pixGetSpp(pix) == 4 && format == IFF_PNG) {
    Pix *p1 = pixAlphaBlendUniform(pix, 0xffffff00);
    sad = pixGenerateCIData(p1, L_FLATE_ENCODE, 0, 0, &cid);
    pixDestroy(&p1);
  } else {
    sad = l_generateCIDataForPdf(filename, pix, kJpegQuality, &cid);
  }

And here is a simple fix (Leptonica needs either the filename or the pix):
  L_Compressed_Data *cid = nullptr;
  const int kJpegQuality = 85;

  int format, sad;
  if (filename) {
    findFileFormat(filename, &format);
    if (pixGetSpp(pix) == 4 && format == IFF_PNG) {
      Pix *p1 = pixAlphaBlendUniform(pix, 0xffffff00);
      sad = pixGenerateCIData(p1, L_FLATE_ENCODE, 0, 0, &cid);
      pixDestroy(&p1);
    }
  }
  if (!cid) {
    sad = l_generateCIDataForPdf(filename, pix, kJpegQuality, &cid);
  }

This code change looks fine to me. But I am curious how you ran into this problem. The filename is set from api->GetInputName(). In what scenario is that empty?

My understanding from the documentation was that the InputName was optional except for certain special uses, such as in training. I had a Pix in memory which I had preprocessed with Leptonica functions and wanted to convert to a searchable PDF. I didn't know that I had to call api->SetInputName with the name of the image file (what if there is no file name).
We could probably also eliminate the first part of this logic which uses Deflate if the original image was a PNG file and let Leptonica decide on the encoding based on the color depth and the alpha channel.

The overall philosophy of the PDF rendering module is to be as hands off as possible with the images. The less we do, the less trouble we can get into ("You made my PDF file too big!") and the less pressure for continuous development ("Leptonica needs better transcode heuristics!" "Change the pixel values in the image!"). Forcing with Flate for PNG input is part of the philosophy of minimal image meddling. Why change that?
PS. I'd still like to encourage use of api->SetInputName whenever possible. Do you have any documentation suggestions?

Thanks for all your efforts and I totally agree with your philosophy! Tesseract is about OCR and not PDF generation. A lot of libraries can take OCR data and create fancy PDF files. I will be building one since the use of zero-glyph CID font is not going to work for us. We need a switch to toggle between "image with hidden text behind" and "no image with recognized text visible". The issue I reported was not on the approach but the fact that if the input name was not detected, a corrupt PDF was being generated. Calling api->SetInputName corrects that problem. Keep up the good work!

@jbreiden : I can commit to source code. Should I?

Yes please, if you could. My workload is crazy right now and I am not that
familiar with pull requests. Thanks!
On Wed, Sep 12, 2018 at 11:04 AM zdenop<EMAIL_ADDRESS>wrote:

@jbreiden https://github.com/jbreiden : I can commit to source code.
Should I?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/tesseract/issues/1889#issuecomment-420723488,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIGOXUVkw9vYU5hnkrRQERF6rpiZDyOCks5uaT6JgaJpZM4WVuH8
.

Yes, approved.

@dthornley In your commit, I believe you forgot to remove (at the beginning of the module)
if (!filename)
return false;
which is needed for the fix to work.

@FarhadKhalafi :
IMO your proposed change is not safe because l_generateCIDataForPdf needs filename or pix. So we should change it this way:
  if (!filename && !pix)
    return false;

Can you test it please?

The code was working fine so I do not see reason why to eliminate usage of filename.
At least not in this situation when we are so close to release of new stable release.

I am OK with your comment. Originally I got into this situation when I was not setting the filename and the code was generating a corrupt PDF where the indirect reference to the image stream was pointing to the PDF info stream instead without complaining. Now that I know about the filename, I can set it prior to calling PDF renderer and everything is OK. Sorry if I created a diversion and thank you for all your efforts!


Starting a TF session (and nothing else) uses over 350MB of GPU memory

On TF 1.13.1, the code below:
with tf.Session(config=config) as sess:
    while True:
        print("Session is open!")
        time.sleep(1)
Uses 363MiB / 16280MiB according to nvidia-smi.

Am I missing something or is this normal? Shouldn't GPU usage be 0MiB at this point? If not, why?

Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary?
Make sure you also include the exact command if possible to produce the output included in your test case. If you are unclear what to include see the issue template displayed in the Github new issue template.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


update!: bump libs to latest and deprecation of k8s metadata options and configs

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area build
/area engine
What this PR does / why we need it:
This bumps the libs dependency (which we haven't done in a while), and inherently deprecates the -k, -K, --k8s-node options of Falco that are no more functional (alongside all config options related to k8s) as tracked in https://github.com/falcosecurity/falco/issues/2763.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
update!: bump libs to latest and deprecation of k8s metadata options and configs

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.37.0

LGTM label has been added.  Git tree hash: e336f48c2e84d8a45cf363e4a32ffc3a664faa09

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Script Step to run pip (python 3)

I've got a workflow where I added a Brew install Step to install Python, which works well.
The next Step that runs after this is a Script Step that I use to run pip (one of the Python tools), in order to install a public plugin.
It contains something like this (I've added a generic name for the plugin, myPlugin):
python3 -m pip --version
python3 -m ensurepip --default-pip
alias pip='pip3'
pip install myPlugin

The problem is that it always fails saying that it doesn't know what pip is (in the 4th line).
Surprisingly, if I run the workflow with Remote Access and I use the terminal to run the same commands it works. So I'm not sure what the issue is that is causing the pip command not being recognised.
Is this an issue known when it comes to work with Python and the package manager pip?

Hey Fernando, have you tried skipping the alias and just calling pip3 directly? I know this is not optimal, but it could help me better understand the problem better if this works.
I should also note that pip3 is already installed on the system.

Hey @matrangam I tried that already and didn't work.
When pip3 tries to install any package (using pip3 without the alias), it seems like there's no way to see the latest logs after running that command nor seeing the actual issue.
The next logs are showed after running the script with the alias included:
pip 21.2.4 from /usr/local/lib/python3.9/site-packages/pip (python 3.9)
DEPRECATION: Configuring installation scheme with distutils config files is deprecated and will no longer work in the near future. If you are using a Homebrew or Linuxbrew Python, please see discussion at https://github.com/Homebrew/homebrew-core/issues/76621
Looking in links: /var/folders/6q/wgy6jtp12w5gzgm9lzcglpqw0000gn/T/tmp1imahec8
Requirement already satisfied: setuptools in /usr/local/lib/python3.9/site-packages (57.4.0)
Requirement already satisfied: pip in /usr/local/lib/python3.9/site-packages (21.2.4)
/var/folders/6q/wgy6jtp12w5gzgm9lzcglpqw0000gn/T/bitrise949048163/step_src/._script_cont: line 4: pip: command not found

ok, that helps a little I think.
Is it possible for you to share the step repo with me so I can dig into this a little more?

I don't have the Step built yet. I was still playing around with the commands I will use in the Step

Hello @ferPrieto,
python3 is also preinstalled, so you should not need to install it. I tried out the xcode-13.4 Stack and could install a pip package:
+ pip3 install celluloid
DEPRECATION: Configuring installation scheme with distutils config files is deprecated and will no longer work in the near future. If you are using a Homebrew or Linuxbrew Python, please see discussion at https://github.com/Homebrew/homebrew-core/issues/76621
Collecting celluloid
  Downloading celluloid-0.2.0-py3-none-any.whl (5.4 kB)
Collecting matplotlib
  Downloading matplotlib-3.5.2-cp39-cp39-macosx_10_9_x86_64.whl (7.3 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 7.3/7.3 MB 22.8 MB/s eta 0:00:00
Collecting packaging>=20.0
  Downloading packaging-21.3-py3-none-any.whl (40 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 40.8/40.8 KB 4.5 MB/s eta 0:00:00
Collecting fonttools>=4.22.0
  Downloading fonttools-4.33.3-py3-none-any.whl (930 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 930.9/930.9 KB 36.1 MB/s eta 0:00:00
Collecting pillow>=6.2.0
  Downloading Pillow-9.1.1-cp39-cp39-macosx_10_10_x86_64.whl (3.1 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 3.1/3.1 MB 41.9 MB/s eta 0:00:00
Collecting pyparsing>=2.2.1
  Downloading pyparsing-3.0.9-py3-none-any.whl (98 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 98.3/98.3 KB 10.3 MB/s eta 0:00:00
Collecting numpy>=1.17
  Downloading numpy-1.22.4-cp39-cp39-macosx_10_15_x86_64.whl (17.7 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 17.7/17.7 MB 35.4 MB/s eta 0:00:00
Collecting python-dateutil>=2.7
  Downloading python_dateutil-2.8.2-py2.py3-none-any.whl (247 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 247.7/247.7 KB 19.6 MB/s eta 0:00:00
Collecting cycler>=0.10
  Downloading cycler-0.11.0-py3-none-any.whl (6.4 kB)
Collecting kiwisolver>=1.0.1
  Downloading kiwisolver-1.4.2-cp39-cp39-macosx_10_9_x86_64.whl (65 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 65.3/65.3 KB 5.9 MB/s eta 0:00:00
Requirement already satisfied: six>=1.5 in /usr/local/lib/python3.9/site-packages (from python-dateutil>=2.7->matplotlib->celluloid) (1.16.0)
Installing collected packages: python-dateutil, pyparsing, pillow, numpy, kiwisolver, fonttools, cycler, packaging, matplotlib, celluloid
  DEPRECATION: Configuring installation scheme with distutils config files is deprecated and will no longer work in the near future. If you are using a Homebrew or Linuxbrew Python, please see discussion at https://github.com/Homebrew/homebrew-core/issues/76621
  DEPRECATION: Configuring installation scheme with distutils config files is deprecated and will no longer work in the near future. If you are using a Homebrew or Linuxbrew Python, please see discussion at https://github.com/Homebrew/homebrew-core/issues/76621
  DEPRECATION: Configuring installation scheme with distutils config files is deprecated and will no longer work in the near future. If you are using a Homebrew or Linuxbrew Python, please see discussion at https://github.com/Homebrew/homebrew-core/issues/76621
  DEPRECATION: Configuring installation scheme with distutils config files is deprecated and will no longer work in the near future. If you are using a Homebrew or Linuxbrew Python, please see discussion at https://github.com/Homebrew/homebrew-core/issues/76621
  DEPRECATION: Configuring installation scheme with distutils config files is deprecated and will no longer work in the near future. If you are using a Homebrew or Linuxbrew Python, please see discussion at https://github.com/Homebrew/homebrew-core/issues/76621
  DEPRECATION: Configuring installation scheme with distutils config files is deprecated and will no longer work in the near future. If you are using a Homebrew or Linuxbrew Python, please see discussion at https://github.com/Homebrew/homebrew-core/issues/76621
  DEPRECATION: Configuring installation scheme with distutils config files is deprecated and will no longer work in the near future. If you are using a Homebrew or Linuxbrew Python, please see discussion at https://github.com/Homebrew/homebrew-core/issues/76621
  DEPRECATION: Configuring installation scheme with distutils config files is deprecated and will no longer work in the near future. If you are using a Homebrew or Linuxbrew Python, please see discussion at https://github.com/Homebrew/homebrew-core/issues/76621
  DEPRECATION: Configuring installation scheme with distutils config files is deprecated and will no longer work in the near future. If you are using a Homebrew or Linuxbrew Python, please see discussion at https://github.com/Homebrew/homebrew-core/issues/76621
  DEPRECATION: Configuring installation scheme with distutils config files is deprecated and will no longer work in the near future. If you are using a Homebrew or Linuxbrew Python, please see discussion at https://github.com/Homebrew/homebrew-core/issues/76621
DEPRECATION: Configuring installation scheme with distutils config files is deprecated and will no longer work in the near future. If you are using a Homebrew or Linuxbrew Python, please see discussion at https://github.com/Homebrew/homebrew-core/issues/76621
Successfully installed celluloid-0.2.0 cycler-0.11.0 fonttools-4.33.3 kiwisolver-1.4.2 matplotlib-3.5.2 numpy-1.22.4 packaging-21.3 pillow-9.1.1 pyparsing-3.0.9 python-dateutil-2.8.2
WARNING: You are using pip version 22.0.4; however, version 22.1.2 is available.
You should consider upgrading via the<EMAIL_ADDRESS>-m pip install --upgrade pip' command.


Major Rework

A major message handling overhaul is needed. Right now, message handling is quite slow. Not sure why, but the leading culprit is the passing between threads on crossbeam channels. I see one path forward so far:

A major API rework to move towards a more GLFW/Winit style, with a callback instead of a for loop.

Right now I'm playing around in the rework branch with a WindowProcedure trait that would be implemented on a callback object and passed into the window to be run. I dislike this approach as my goal with this project was to not take over the execution with any sort of "run()" function. I am actively looking for help and support in fixing this, with anyone welcome to give their thoughts here on how to move forwards.

Done!


does it exist already ?

Hi,
you talk about Jai for inspiration (maybe you could add Odin or even more Zig), but is Zax like Jai and only visible to a few and won't be public before years or does it exist somewhere ? or are you just creating it ..?
well, the actual doc is interesting, but if there's no code.....

Had the project on the back burner for years but I'm literally just starting to actually "do it". Too many projects, too little time. I'm writing up the documentation so I can know the scope of what I want to code, and provide myself some needed documentation for the parser/generator and what features I will prioritize. The inspiration from Jai was mostly to just do it; I do also like some of the concepts as I think they solve some issues in clever ways but I would want a few things differently since I don't primarily do game coding so my priorities are a bit different. Code will be fully visible as I do it. First pass will function first and like all code will always suck in its first iteration as I sort out the design. So yes, no code yet. I didn't expect anyone to notice it for a bit so I took it for granted I'd have some time to prove it out. Should be fun to write the compiler but I wouldn't expect anyone to take it serious until the code is there to back it up, even though I'm serious about putting code against it. Thanks!

well, github search function is really bad, but sometimes with a little chance you can discover new things.. :-)
ok, thanks.

Just as an update to anyone who might find this issue. The prototype compiler is being developed here:
Zax Compiler
It's extremely early stages right now as it's being actively built.


Built in Database backup no longer works

The built in database backup no longer works since adodb has been replaced.
Implement a solution that uses mysqldump and fails gracefully if that system command is not able to be executed.

I think mysqldump is a good stopgap solution, but I think that if we want to think a little further out we should be removing these sort of database engine dependencies if possible. It's not infrequent someone asks about using Concrete5 with their MSSQL database or postgres database and while that's not a target for c5 (yet), I think implementing a database dependent solution without more thought on it might end up wasting resources.

https://github.com/heybigname/backup-manager maybe this could help ?

That looks really handy. That'd definitely be the way to go.

Any update on this as far as ETA / Priority of this feature?

The backup manager requires php >= 5.4 it looks like

Additionally, the backup manager requires mysqldump command.

Sorry, not sure if we're talking about the same thing or not... I'm talking about /dashboard/system/backup/backup/run_backup
Which has this:
            //@TODO this backup stuff needs to be reworked since we're not using adodb anymore
            $this->set('message', t('This has not been implemented in 5.7'));
            return;

Which doesn't appear to be implemented at all.

@TimDix same issue. If you read up there was a suggestion to make use of the backup-manager to do the backups and @aembler seemed to think this was the way to go.

How about this library? It's lightweight, no requirements, run without mysqldump command.
https://github.com/ifsnop/mysqldump-php

At a glance, it looks great

@hissy it does look like a pretty decent library, my biggest concern are timeouts. The library from what little bit i've read doesn't really provide a good way for concrete5 to backup 1 table per request or similar to better prevent timeout situations.
I do think, however, that it might be an ideal candidate for a CLI command, with the hopes that those are less prone to timeout issues.

I think cli command to backup would be a good solution to timeout issue.

Also.. .this may be scope creep on this issue.. But I think a backup level option would be a great improvement too... I'm thinking something like

Database Only
Database & Config
Database & Application Files
Database, Application & Package Files

Far to often someone attempts to backup their database, thinking that's all they need to migrate their site from one host to another. Also, allowing for a fuller approach to backups where one could restore to a specific point in time could have great advantages from a Disaster Recovery perspective.

@joe-meyer
These are excellent ideas.
Having true full backup and a timeline backup within the UI would be a real asset to users.

I've just started working of backup stuff.
https://github.com/hissy/concrete5-5.7/tree/enhancement/database-backup
I'd like to push it If we get feature/backup brunch into this repository. I think we need more discussion and more contributing from community.

I'd rather have on database backup that worked well and had a couple
options to exclude certain tables (logs, statistics, etc...) and simply
prompted the user to immediately download a zipped file when complete, then
a bunch of crazy options for backing up files, etc... I think a separate
package would work well for something like that.
On Tue, Aug 25, 2015 at 6:54 AM, Takuro Hishikawa<EMAIL_ADDRESS>wrote:

I've just started working of backup stuff.
https://github.com/hissy/concrete5-5.7/tree/enhancement/database-backup
I'd like to push it If we get feature/backup brunch into this repository.
I think we need more discussion and more contributing from community.
—
Reply to this email directly or view it on GitHub
https://github.com/concrete5/concrete5/issues/571#issuecomment-134593381
.

A package that does what Joe Meyer suggests would be great.
I've got a shell script I got from somewhere, for files + db, runs at X time each day, I just can't get it to clean up after itself -- ie, delete backups older than X days.  Any chance you guys would like to see if you could clean that up? At least people could download it, make a cron job and have continuously working backups going all the time.

@tduncandesign I think the goal here is to build something that's deliverable from a concrete5 interface. A shell script kind of falls outside of that, especially considering that some people might run on different environments where a shell script isn't an option (think windows hosts or shared hosts that don't allow them). That said, I'm sure if you posted what you had to the concrete5 forums there would be others interested and possibly even willing to help you refine it, especially with the lack of a solid backup option right now.

Is this something that the core team is going to make a decision on, or is it up to the community to make a decision, do the work, and then request approval from the core team? I ask because Andrew's comments seem supportive, but hands off...

Built-in backup ought to cover 70% of the cases, be easy to understand, and work well for sites of most size.

This has to work completely from a web interface.
This has to work without installing other libraries. If a common binary like mysqldump is required that might not be a dealbreaker, but it has to be checked for and the feature disabled if we can't find it.
This has to work without a lot of configuration.
I don't like the idea of .sql files in the file system. In 5.6 we encrypted them but I think it's just too much. I'd much rather hit a button, and immediately get a file to download. It'd also be cool if we could zip them up the way we do files in the file manager when you select bulk files to download.
I think having a couple options (exclude statistics, exclude logs) would be handy.
I don't want a full site backup. There are lots of options for that. If someone really wants that from concrete5, they can use an add-on. Or S3, or cPanel. This began as a Database Backup feature, let's just keep it that for now.

@aembler  +1 to that

Drupal CMS provides >local and >server-sides backups for the "default database", "public files directory" and "entire site (code,files,database)". Additionaly ..notes/text can be attached any of the backup files.  ("Entire-site" does not quite work/timeout issues? Backup/restore the database just works quickly)

Is backup manager still an option? If so, we should just go with that. We can enable backing up to external locations with config for power users and just expose the basics to end users with a button to download. It's 5.5.9+ so we'd need to do this in 8.0.0+
https://github.com/backup-manager/backup-manager

Backup Manager is a good solution, but it requires mysqldump and mysql command-line binaries.

@hissy I'm not aware of a good solution that doesn't require command line executables (And I frankly wouldn't trust any).

Here is a one.
https://github.com/ifsnop/mysqldump-php
I agree that mysqldump is more reliable way, but we can fallback to no requirements (and less reliability) option. Because, I think users who really want this feature can not use mysqldump command on their shared hosting.

I'm not sure I believe that everyone who'd use this tool is only using it because mysqldump is specifically unavailable on their system. I imagine a large number of people a) don't know what mysqldump is b) don't know how to ssh into their server even if the server allows it c) just want to back up their database.
I originally liked the idea of this being mysqldump based because it's a rock solid utility and it's fast. We work with some large databases. If mysqldump-php is really as reliable then a pure PHP solution is definitely compelling – but I don't think one that is based off of mysqldump is bad. Far from it.

@aembler agreed on all points. Lets do some testing and figure out where mysqldump won't work.

FWIW, I used mysqldump to do the dirty work in the 'Backup Database' job in this add-on. I'll probably get feedback about circumstances in which it won't work.

give this a try: https://datatainment.de/this-n-that/concrete5-backup-and-restore-database


Create NuGet packages for MSBuild 14.0

Package should contain the following assemblies:

ref
netstandard1.3
net45
lib
net45

Check in stuff under build\pkg
Ensure everything is signed correctly.
Package version will be 14.3.0 where 3 is the VS update number.  Will use -prerelease tag until everything is tested and signed off.  Will rev the build number when we service the package.
Push packages to NuGet.org

Related to #490

@jeffkl, is it published? Can this and #490 issues be closed?

Yes, I've just pushed them to NuGet.org.  I'll leave them as -prerelease until probably next week when enough people have tried them out and then I'll remove the tag.
https://www.nuget.org/packages/Microsoft.Build/
https://www.nuget.org/packages/Microsoft.Build.Framework/
https://www.nuget.org/packages/Microsoft.Build.Utilities.Core/
https://www.nuget.org/packages/Microsoft.Build.Tasks.Core/
https://www.nuget.org/packages/Microsoft.Build.Conversion/
https://www.nuget.org/packages/Microsoft.Build.Engine/

Great! Thank you! :tada:

@jeffkl,
https://www.nuget.org/packages/Microsoft.Build.Conversion/ is actually https://www.nuget.org/packages/Microsoft.Build.Conversion.Core/
And both https://www.nuget.org/packages/Microsoft.Build.Conversion.Core/ and https://www.nuget.org/packages/Microsoft.Build.Engine/ are only supported for net45 TFM. Are they coming to netstandard1.3 like rest of the packages? Is there an issue tracking their port to .NET Core?

Thanks I've updated my comment with the correct link to Microsoft.Build.Conversion.Core.  I was told that both Microsoft.Build.Engine and Microsoft.Build.Conversion.Core are considered "legacy" and we don't really want people to consume them anymore.  Do you need to reference them for anything where you want to compile something against .NET Core?

Thanks @jeffkl. I just found out that StrongNameKeyPair was not exposed in System.Reflection from BCL, and yet the features are guarded by #if FEATURE_STRONG_NAMES in MSBuild. I was under the impression that Engine has it defined somewhere. But I guess we will have to wait for 1.2.0: https://github.com/dotnet/corefx/issues/10530 then remove that guard in StrongNameHelpers.cs: (with define constant in dir.prop). :)


Karaf 4.2.x whitelist

Adding a whitelist for repositories. Useful if new repositories overlay actual used versions without warning.

Please, open PR only on master (the PR is cherry picked on target branches if needed).


--cspec is applying custom specification values that are not defined in vCenter's customization specification manager

Versions:

Version of knife-vsphere: 2.0.1
Version of chef: 12.19.36

Platform Details

Version of vSphere: 6.5.0
Version of ESXi: 6.5.0

Scenario:

I am trying to spin a VM using the following command
knife vsphere vm clone testVM4 --dest-folder Chef_Test --template TEST_TMPT --start --datastore my_datastore --cips <IP_ADDRESS> --cspec winspec
The scenario is that the VM has to take the IP address value from --cips and the rest of the values such as subnet mask , default gateway, DNS settings has to be taken from the --cspec which is holding the customization specification manager.
But the VM is getting built with the Ip address passed in --cips but thedefault gateway and the subnet mask values are a random values that are never a part of the customization specification manager.
Please paste the command and output (run with -VV). Don't forget to X out any passwords or sensitive information:
[root@chefworkstation .chef]# knife vsphere vm clone testVM4 --dest-folder Chef_Test --template TEST_TMPT --start --datastore my_datastore --cips <IP_ADDRESS> --cspec winspec -VV
Cloning template TEST_TMPT to new VM testVM4
VirtualMachineCloneSpec(
  config: VirtualMachineConfigSpec( deviceChange: [] ),
  customization: CustomizationSpec(
    encryptionKey: [48,
     *
     * # Too many number were here, removing them to make the thread smaller
     *
     *
     30],
    globalIPSettings: CustomizationGlobalIPSettings(
      dnsServerList: [],
      dnsSuffixList: []
    ),
    identity: CustomizationSysprep(
      guiRunOnce: CustomizationGuiRunOnce(
        commandList: ["net stop myservice",
         "reg delete my_reg_key"]
      ),
      guiUnattended: CustomizationGuiUnattended(
        autoLogon: true,
        autoLogonCount: 1,
        password: CustomizationPassword(
          plainText: false,
          value: " xxx"
        ),
        timeZone: 33
      ),
      identification: CustomizationIdentification( joinWorkgroup: "WORKGROUP"
      ),
      userData: CustomizationUserData(
        computerName: CustomizationVirtualMachineName(  ),
        fullName: "IT Department",
        orgName: "My Organization",
        productId: ""
      )
    ),
    nicSettingMap: [CustomizationAdapterMapping(
       adapter: CustomizationIPSettings(
         gateway: ["10.*.*.*.1"],               # This was having the gateway value that is not specified in the vCenter's customization specification manager
         ip: CustomizationFixedIp( ipAddress: "<IP_ADDRESS>" ),
         subnetMask: "255.*.*.*"		# This was having the subnet mask value that is not specified in the vcenter's customization specification manager
       )
     )],
    options: CustomizationWinOptions( changeSID: true, deleteAccounts: false )
  ),
  location: VirtualMachineRelocateSpec(
    datastore: Datastore("datastore-151"),
    pool: ResourcePool("resgroup-8")
  ),
  powerOn: false,
  template: false
)
Finished creating virtual machine testVM4
Powered on virtual machine testVM4

Expected Result:

Vm to build with the IP address passed in --cips and other values as default gateway,netmask and dns should be taken from the --cspec which is the customization specification manager
Actual Result:

VM is built with the IP address passed --cips but the gateway,netmask are generated randomly by ignoring the values specified in the customization specification manager

Hi, we expect the netmask to be passed in CIDR notation. Try --cips <IP_ADDRESS>/24 instead, and don't try to override in the cspec. The IP and gateway go on the same object we send to vsphere, so merging a CLI IP and a cspec netmask ends up being difficult.

The IP and gateway go on the same object we send to vsphere, so merging a CLI IP and a cspec netmask ends up being difficult.
Sorry I couldnt understand the above statement. Kindly add few more information to this.
So do you meant to say that I have to build a VM with my --cips as <IP_ADDRESS>/subnetmask_range. So what about my DNS and gateway settings, as they are part of the cspec and I would need them to be exactly assigning the same value as specified in the customization specification manager for successful VM build

Hi, sorry, my comment may have been more technical detail than necessary. Cloning a VM involves building a big data structure based on what's passed on the command line, then it's merged with what's stored on the server cspec if it was used. The IP and subnet "go together", so we'd need to override them in the cspec as a unit. It would be difficult to get the IP from the command line and the netmask from the cspec, plus it would end up being confusing for all.
The DNS and gateway are separate objects so it's no problem to pass those in the cspec. You can also use --cdnsips and --cgw on the CLI if you'd prefer. If your gateway is the .1 of the subnet then you can omit it, FWIW.

Meant to add that what we do at work is put as much stuff as possible on the command line rather than the cspec. So for Linux clones it's all on the command line (no cspec), and for Windows, the cspec just has the DNS and AD info so sysprep can join the box to the domain.

Hi Swalberg,
Yes, I tried to pass the IP and the subnet mask like <IP_ADDRESS>/mysubnet_range and it successfully assigned the ip,netmask and default gateway. But it didnt take my DNS settings value from the cspec and the DNS settings are blank on the VM which is built.
I tried passing the DNS values using --cdnsips and then they took it, but I want them to take it from the cspec. Any suggestion on this

Hm, can you run it with the proper --cips and no dns/gw CLI opts, and show me the output when using -VV please? This should work; we do something similar here for Windows nodes.

Hi Sean,
Please find the logs below
[root@chefworkstaion .chef]# knife vsphere vm clone testVM3 --dest-folder Chef_Test --template NewTemplate -f Chef_Test --datastore my_datastore --cips <IP_ADDRESS>/23 --cspec win12 --start -VV
Cloning template NewTemplate to new VM testVM3
VirtualMachineCloneSpec(
config: VirtualMachineConfigSpec( deviceChange: [] ),
customization: CustomizationSpec(
encryptionKey: [48,
-126,
4,
26,
48,
-126,
3,
*
*
# Too many numbers, removing them to make the thread look smaller
-114,
120,
94,
-68,
-13,
16,
30],
globalIPSettings: CustomizationGlobalIPSettings(
dnsServerList: [],
dnsSuffixList: []
),
identity: CustomizationSysprep(
guiRunOnce: CustomizationGuiRunOnce(
commandList: ["net stop myservice",
"reg delete myregkey"]
),
guiUnattended: CustomizationGuiUnattended(
autoLogon: true,
autoLogonCount: 1,
password: CustomizationPassword(
plainText: false,
value: "FwIuXmNRTmtXpgNlruE3UNi3MAO1Fz2yVKqEmSQwd/3buP6VbQZp3nAJZoxaFo9AoPgrB8LY                     4XEtAkUHQcQMkq6pdKRXtBX+WzMSrnnbWqFSL+54gd+gM3gAkYkqBEpD3/p99pB6pnaAvvGlEqRkZAR2T3PvWev5BK                     BFw2jfnMWAjuabi+ferV0t4BKxSD8/Ec5K8kncCjnBcOFMGD6OL7MRUDMeqkHhaD3wRlPtVd3MLPndX+jueqHkx5Qx                     3ICXOOJ50beGHhSJRkz6NVrnMCX5wO0HpM9izBrnjHpqMCEa2o30+9PA7XjY/G682uRyCpCne5YxROSqCvD6us8opA                     =="
),
timeZone: 33
),
identification: CustomizationIdentification( joinWorkgroup: "WORKGROUP"
),
userData: CustomizationUserData(
computerName: CustomizationVirtualMachineName(  ),
fullName: "IT Department",
orgName: "My Organization",
productId: ""
)
),
nicSettingMap: [CustomizationAdapterMapping(
adapter: CustomizationIPSettings(
gateway: ["10...1"],
ip: CustomizationFixedIp( ipAddress: "<IP_ADDRESS>" ),
subnetMask: "255...0"
)
)],
options: CustomizationWinOptions( changeSID: true, deleteAccounts: false )
),
location: VirtualMachineRelocateSpec(
datastore: Datastore("datastore-151"),
pool: ResourcePool("resgroup-8")
),
powerOn: false,
template: false
)
Finished creating virtual machine testVM3
Powered on virtual machine testVM3
On the same command if I pass the dns settings using --cdnsips, I can see in the logs showing my dns values
dnsServerList: [<IP_ADDRESS>,<IP_ADDRESS>]
The same command if I run without --cips and --cdnsips and only use --cspec, I am getting a dnsserverlist right after the line  nicSettingMap: , which is coming from the customization specification manager.

Hm, this one is puzzling. It does look like it's set along with the NIC in the customization so we're going to override it when we set the IP.
Not sure if this helps, but can you pass it on the command line and later enforce it with group policy/chef/whatever? Many of the customization options are just there to get the box running and we expect that once the server is built that something else will take over managing it.
One option would be to create a plugin that pulls down the cspec and "fixes up" the spec prior to it getting sent to vsphere.

Hi Sean,
Yeah I'll look at the custom plugins options , but for now my --cdnsips are working fine. So I am good with that as of now, and hence closing this issue. Thanks for all your assistance on this. Much appreciated :+1:


krun error on Windows 10

I'm trying to run the lessons in 1_lambda and I'm getting the following error when running krun:
D:\github\grosu\k\k-distribution\tutorial\1_k\1_lambda\lesson_1>kompile lambda.k

[Warning] Compiler: Could not find main syntax module with name LAMBDA-SYNTAX
in definition.  Use --syntax-module to specify one. Using LAMBDA as default.
D:\github\grosu\k\k-distribution\tutorial\1_k\1_lambda\lesson_1>krun tests\ident
ity.lambda
[Error] Critical: Error while running process:Cannot run program "kast" (in
directory "."): CreateProcess error=2, The system cannot find the file
specified
D:\github\grosu\k\k-distribution\tutorial\1_k\1_lambda\lesson_1>

Note that I have not run K for a long time, but I would expect the above to work.

@laurayuwen , @daejunpark , did you guys have problems with this?

This is what I get with --debug:
D:\github\grosu\k\k-distribution\tutorial\1_k\1_lambda\lesson_1>krun --debug tes
ts\identity.lambda
org.kframework.utils.errorsystem.KEMException: [Error] Critical: Error while run
ning process:Cannot run program "kast" (in directory "."): CreateProcess error=2
, The system cannot find the file specified
        at org.kframework.utils.errorsystem.KEMException.create(KEMException.jav
a:101)
        at org.kframework.utils.errorsystem.KEMException.criticalError(KEMExcept
ion.java:34)
        at org.kframework.krun.RunProcess.execute(RunProcess.java:73)
        at org.kframework.krun.KRun.externalParse(KRun.java:315)
        at org.kframework.krun.KRun.parseConfigVars(KRun.java:263)
        at org.kframework.krun.KRun.run(KRun.java:70)
        at org.kframework.krun.KRunFrontEnd.run(KRunFrontEnd.java:91)
        at org.kframework.main.FrontEnd.main(FrontEnd.java:52)
        at org.kframework.main.Main.runApplication(Main.java:110)
        at org.kframework.main.Main.runApplication(Main.java:100)
        at org.kframework.main.Main.main(Main.java:52)
Caused by: java.io.IOException: Cannot run program "kast" (in directory "."): Cr
eateProcess error=2, The system cannot find the file specified
        at java.lang.ProcessBuilder.start(ProcessBuilder.java:1048)
        at org.kframework.krun.RunProcess.execute(RunProcess.java:43)
        ... 8 more
Caused by: java.io.IOException: CreateProcess error=2, The system cannot find th
e file specified
        at java.lang.ProcessImpl.create(Native Method)
        at java.lang.ProcessImpl.<init>(ProcessImpl.java:386)
        at java.lang.ProcessImpl.start(ProcessImpl.java:137)
        at java.lang.ProcessBuilder.start(ProcessBuilder.java:1029)
        ... 9 more
java.io.IOException: Cannot run program "kast" (in directory "."): CreateProcess
 error=2, The system cannot find the file specified
        at java.lang.ProcessBuilder.start(ProcessBuilder.java:1048)
        at org.kframework.krun.RunProcess.execute(RunProcess.java:43)
        at org.kframework.krun.KRun.externalParse(KRun.java:315)
        at org.kframework.krun.KRun.parseConfigVars(KRun.java:263)
        at org.kframework.krun.KRun.run(KRun.java:70)
        at org.kframework.krun.KRunFrontEnd.run(KRunFrontEnd.java:91)
        at org.kframework.main.FrontEnd.main(FrontEnd.java:52)
        at org.kframework.main.Main.runApplication(Main.java:110)
        at org.kframework.main.Main.runApplication(Main.java:100)
        at org.kframework.main.Main.main(Main.java:52)
Caused by: java.io.IOException: CreateProcess error=2, The system cannot find th
e file specified
        at java.lang.ProcessImpl.create(Native Method)
        at java.lang.ProcessImpl.<init>(ProcessImpl.java:386)
        at java.lang.ProcessImpl.start(ProcessImpl.java:137)
        at java.lang.ProcessBuilder.start(ProcessBuilder.java:1029)
        ... 9 more
[Error] Critical: Error while running process:Cannot run program "kast" (in
directory "."): CreateProcess error=2, The system cannot find the file
specified

Pretty sure this is because we are now using external parsers for everything and you are not running from a cygwin environment so it doesn't know how to interpret and execute the bash script. The simple fix should be to add a .bat to the default on windows. I will take care of this.

And, of course, kast works in command line, otherwise I would have reported a different bug:
D:\github\grosu\k\k-distribution\tutorial\1_k\1_lambda\lesson_1>kast tests\identity.lambda
lambda_._(#token("x","Id"),#token("x","Id"))

Thanks, @dwightguth!  I think we are already doing that for all our RV tools, so that should work.

@daejunpark but that means it's not limited only to windows 10, but to all windows platforms.
I know that Java has this issue where if you try to execute a .bat file, you have to specify the extension. And it's not required in the command line.
Another thing. Why aren't you calling the parser directly from krun? It's more efficient, and you avoid all kinds of problems.

@radumereuta the old implementation was extremely quirky as a result of what we did. I'm not at all convinced that the performance difference associated with a new process is particularly noticeable with nailgun. If you want to try to test it and compare and let me know if you still see a performance difference, let me know. But I suspect it will be about the same either way.
I am choosing to go a different route this time by making the default be an absolute path to the  script on disk so that we avoid all the problems with path issues that we had before while still maintaining a much cleaner implementation.

@dwightguth, regarding the second issue, you may want to merge my other PR then.

Regarding the first, I agree with @radumereuta and the two of us had this discussion before. You may like the command line as the "API" of the tool, but it isn't the way it's usually done and can lead to problems like this.

I actually agree with @dwightguth on this one.
The only problem I can foresee is having a good enough intermediate language to represent the terms and error messages. In this case is KORE, and we are gaining confidence in the implementation every day.

@radumereuta, the issue I'm referring to is having the command line as the API of K. This is a bad idea, and I'm not even going to argue about it anymore.

If you can come up with a way to have krun call an API for kast in a way that allows support for external parsers and is cleaner than the way it was implemented in the old implementation, I have no particular objection. It's just not remotely a priority for me to work on since we don't use kast at all in the C semantics.

We do not have time for this now, please let's worry about it after Jan 18.  The simple bug fix is good enough for now.

fixed by #1994


Added smarthost support

Smarthost support added; this wil solve issue #18

I hope you agree with it now. If it's still not right, I'll leave it up to you to accept and adjust or deny completely.
As mentioned before, I'm not a coder so I don't know all ins and outs...

All looks good, thanks!


Persian Translation

Hi,
Symfony has a large community of Iranian developers and I suggest adding translation for the Persian language.

Feel free to open a PR 👍🏻

@javadsaeid yes, it'd be great to provide a Persian translation of this Demo application. Sadly none of the maintainers can speak that language, so please, provide a Pull Request or try to ask some people from the community to help you with this. We'll be happy to help you with the contribution process until this is merged. Thanks!

@javiereguiluz Thank you for your response, sure ill send PR.

@javadsaeid any news ?


[PLAT-8819] Add leaveNetworkRequestBreadcrumbForTask:metrics:

Goal
Facilitate leaving network request breadcrumbs without relying on the plugin's Objective-C method swizzling.

https://github.com/bugsnag/bugsnag-cocoa/issues/1420

Design
+[Bugsnag leaveNetworkRequestBreadcrumbForTask:metrics:] provides a public API for leaving a network request breadcrumb based on an NSURLSessionTask and NSURLSessionTaskMetrics.
Changeset
Adds +[Bugsnag leaveNetworkRequestBreadcrumbForTask:metrics:] and -[BugsnagClient leaveNetworkRequestBreadcrumbForTask:metrics:].
Moves breadcrumb message & metadata computation from BSGURLSessionTracingDelegate to BSGNetworkBreadcrumb.m, called from -[BugsnagClient leaveNetworkRequestBreadcrumbForTask:metrics:].
Calls -[BugsnagClient leaveNetworkRequestBreadcrumbForTask:metrics:] from -[BSGURLSessionTracingDelegate URLSession:task:didFinishCollectingMetrics:].
Testing
Migrates and modifies existing unit tests to continue passing.
Verified by existing E2E scenario.

Thank you for doing this! A great addition!


Update TE-14.1

Changes:

Parameterize scale numbers as input variables.
Increase readability.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 57.219%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1181

Relevant Lines:
2064

💛  - Coveralls


Added repository field

Noticed that my npm install complained about tether not having a repository field, so I added one, as per https://docs.npmjs.com/files/package.json#repository

Closed via https://github.com/HubSpot/tether/pull/100


Replace type of x in ActiveSet

Problem
ActiveSet is initiated as active_set = FrankWolfe.ActiveSet([(1.0, x0)]) in away_frank_wolfe and blended_pairwise_conditional_gradient. If x0 is of type Vector{BigFloat} an UndefRefError is thrown.
Test
using FrankWolfe
using LinearAlgebra
import Random
using MultivariatePolynomials
using DynamicPolynomials

n = Int(1e2)

xpi = rand(n);
total = sum(xpi);
const xp = xpi # ./ total;

f(x) = LinearAlgebra.norm(x - xp)^2
function grad!(storage, x)
    @. storage = 2 * (x - xp)
end

DynamicPolynomials.@polyvar X[1:15]
const max_degree = 4
coefficient_magnitude = 10

const var_monomials = MultivariatePolynomials.monomials(X, 0:max_degree)
Random.seed!(42)

const all_coeffs = map(var_monomials) do m
    d = MultivariatePolynomials.degree(m)
    return coefficient_magnitude * rand() .* (rand() .> 0.95 * d / max_degree)
end

lmo = FrankWolfe.LpNormLMO{BigFloat,1}(0.95 * norm(all_coeffs, 1))

x0 = Vector(FrankWolfe.compute_extreme_point(lmo, zeros(n)))

# add the first vertex to active set from initialization
active_set = FrankWolfe.ActiveSet([(1.0, x0)])

Solution
Ensure that type of x is of the same type as the input.

using FrankWolfe
using LinearAlgebra
using SparseArrays

n = Int(1e2)

lmo = FrankWolfe.LpNormLMO{BigFloat,1}(rand())

x0 = Vector(FrankWolfe.compute_extreme_point(lmo, zeros(n)))

# add the first vertex to active set from initialization
active_set = FrankWolfe.ActiveSet([(1.0, x0)])

I think this is sufficient as a test. What line was erroring? Was it the ActiveSet constructor?

The error is due to the ActiveSet constructor. Your example is sufficient. The rest of the test is necessary for calling away_frank_wolfe.

Codecov Report

Merging #275 (03c3ba4) into master (e1ac81f) will decrease coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #275      +/-   ##
==========================================
- Coverage   71.29%   71.17%   -0.12%
==========================================
  Files          14       14
  Lines        1961     1939      -22
==========================================
- Hits         1398     1380      -18
+ Misses        563      559       -4

Impacted Files
Coverage Δ

src/active_set.jl
84.21% <100.00%> (-0.24%)
:arrow_down:

src/types.jl
67.02% <0.00%> (-1.92%)
:arrow_down:

src/oracles.jl
86.59% <0.00%> (-0.79%)
:arrow_down:

src/utils.jl
55.64% <0.00%> (-0.25%)
:arrow_down:

src/norm_oracles.jl
95.79% <0.00%> (-0.07%)
:arrow_down:

src/polytope_oracles.jl
98.87% <0.00%> (-0.02%)
:arrow_down:

src/blended_cg.jl
50.93% <0.00%> (+0.79%)
:arrow_up:

Continue to review full report at Codecov.

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


improve caching for globbed * resources

right now you can end up with stale resources, since they are no longer in the source code but still in the cache! and thus they still end up in the .geode

might be fixed in dd3081497475d2ed998b570b43aa772fc4fda11b but im not sure


Feature/dependencies

Missing code lint - completes #105

Sorry, It was my fault merging quickly last PR

@apozohue10 - it's not your fault - you are simply too efficient.

Is this PR prepared to be merged?
If so, I will accept it and generate the new release

@apozohue10 - Yes, please merge and test. You'll need to check that:

Docker build works
Docker secrets work.

They were working on the branch before merge - I can't vouch for the other changes you have merged though.

@apozohue10 - I've been testing this Dockerized locally and the async library is both unnecessary and problematic. I have replaced it with the start-up code "stolen" from Keyrock  d854782 and it works fine for me now.
I'm not sure why async was added in the first place when setTimeout() works just as well, seems like unnecessary bloat to me.

@jason-fox Yes, I think if you can code something without relying on a library is better (unless It is something difficult). If you are interested, yesterday, I tested the PR and I had to turn the function "retryCheck" into an asynchronous function in order to run the PEP Proxy.
Anyways, setTimeout works perfectly. I am going to merge the PR.
Thank you!


Handle raw scrolling events on macOS

There's a limitation of SDL that it rounds coordinates to whole numbers. Finer scroll gestures get lost due to this.

SDL 2.0.18 added preciseX and preciseY to the scroll event.


REL: v0.10.1

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found some lint.
Here's what I've got...
For recipe/meta.yaml:

❌ This recipe is using a compiler, which now requires adding a build dependence on {{ stdlib("c") }} as well. Note that this rule applies to each output of the recipe using a compiler. For further details, please see https://github.com/conda-forge/conda-forge.github.io/issues/2102.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11981611083. Examine the logs at this URL for more detail.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@ken-lauer, thanks for updating it!


Fix router not working in Cypress tests

While adding Cypress tests to my Next.js app, we discovered the Next.js router doesn't work in these tests because they run in an iFrame. Next.js sees this and disables the router.
Even though the test runs in an iFrame, Cypress preserves full browser history support, so the router shouldn't be disabled in this case.
This change bypasses the iFrame check when in the Cypress testing environment.
Let me know if you want any changes!
I've verified this change works on this pull request: https://github.com/BeeDesignLLC/GlutenProject.com/pull/151

Hey @flybayer could you try #3880? It's a more generic solution to disable history when the browser doesn't support it.

Did @timneutkens fix work you and your setup @flybayer ? Running into the same problem on "next": "^5.0.1-canary.11"

@lasseborly I haven't tested it yet, but I think it'll work from looking at the code.

Hmm okay @flybayer.. This is till the screen I'm meet with after having cypress click a link.

I am on canary.15 and still have the same issue. So it seems not to work.


No dropdowns on the nav bar

Research so far:
* There might be help from this template https://lenpaul.github.io/Millennial/, but it's incredibly simplistic in comparison to our current template
* This poster got a version working but it's rittled with bugs https://github.com/mmistakes/minimal-mistakes/issues/1801
* This is how 'collections' works in Jekyll. Which is likely the best way to create heirarchies https://jekyllrb.com/docs/collections/
* This is how collections in MM works https://mmistakes.github.io/minimal-mistakes/docs/collections/

This isn't like, a good solution. But I did get a version working based on what that poster did. Seems to work.

Good enough.


LightGBM produces identical leaf values for majority of the leaves

Description
Hi! I noticed the most of the leaf values in the trees share identical values. I am using trees_to_dataframe() function to inspect the leave values. The identical leaf values either equal to the minimal leaf value or the maximum leaf value of the tree without exception. I'm curious what is the cause of this behavior. These leaves have different numbers of samples and weights. I would expect some differences in their leaf values. I've tested different datasets and saw the same behavior. Thank you!
Reproducible example
Below is the params I'm using and the results I got:
{'boosting': 'gbdt',
   'metric': 'rmse',
   'bagging_freq': 1,
   'verbosity': -1,
   'n_jobs': -1,
   'early_stopping_round': 10,
   'num_rounds': 10000,
   'seed': 0,
   'deterministic': True,
   'force_col_wise': True,
   'bagging_fraction': 0.9745239331700798,
   'cat_l2': 17.0,
   'cat_smooth': 285.0,
   'feature_fraction': 0.5012123832040147,
   'learning_rate': 0.07649878768592239,
   'max_cat_threshold': 74,
   'max_depth': 5,
   'min_data_in_leaf': 119,
   'min_data_per_group': 136,
   'num_leaves': 56,
   'max_delta_step': 0.43872434313624487}

Environment info
LightGBM version or commit hash:
LightGBM '3.3.4'
Command(s) you used to install LightGBM
pip install lightgbm

Additional Comments

I've tested different datasets and saw the same behavior

Are you able to provide a minimal, reproducible example, some self-contained code we could run which replicates this?
For example, like this
import lightgbm as lgb
from sklearn.datasets import make_regression

X, y = make_regression(n_samples=10_000)
dtrain = lgb.Dataset(X, label=y)
bst = lgb.train(train_set=dtrain, params={"objective": "rmse"})

You're right to note that this is unexpected, but it's not impossible. I suspect this outcome is highly dependent on your data, for example on the distribution of the target variable.
It's going to be difficult to investigate this given only the parameters and a screenshot of that dataframe representation of the Booster.

Also, is it easy for you to test with lightgbm v4.0.0? That release contains a LOT of bugfixes and other changes relative to the version you're using (v3.3.4). Perhaps the issue is already fixed in v4.0.0.

Hi! Thank you for the quick response! I just tested on V4.0.0 and saw the same behavior. Unfortunately I'm working on private datasets so I couldn't share here. But let me see if I can replicate this behavior on public datasets and get back to you.
I think your suspicion is correct though. The datasets I'm working on have very noisy target distributions. I'm curious what ensures the identical leave values only equal to the min or max leaf values but never in-between.
Based on this doc, the value in a leaf is the predicted value for this leaf node, multiplied by the learning rate. Is the "predicted value" here simply the average value of the labels among the samples within the same leaf? Should I expect two leaves with different samples/weights and average labels to have different leaf values?

Is the "predicted value" here simply the average value of the labels among the samples within the same leaf?

The value in a leaf node is the the one that minimizes the aggregate loss for all examples that fall into that leaf node.
To create a prediction for one sample, LightGBM passes that sample through each tree, then sums the values from all leaf nodes the sample falls into.
So those values you see in the value column are in terms of the gradients (which you can think of conceptually as model residuals), not the actual target. So for example, the predicted value for a leaf node being 1.1 does not mean that the average of all the values of the label for observations in that leaf node.
The only time that would be true is if you've passed the parameter boost_from_average=False, and if you're looking at the structure of the first tree in the ensemble.

Should I expect two leaves with different samples/weights and average labels to have different leaf values?

You should expect that two leaves with the same parent node have different leaf values. If they didn't, then the gain for that split would be 0 and it might as well not have been made.
If you find a situation like that, that sounds like a bug in LightGBM to me unless you are using forced splits and forcing LightGBM to split a certain way even if that way generates 0 gain.
It's technically possible, though unlikely in my experience, for leaf nodes with different parents to happen to have identical predicted values.

Thank you for the thorough explanation! It was very helpful. I was able to recalculate the leave value and matched the output above. The cause was the combination of max_delta_step and learning_rate. Since both are set to relatively small values in my dataset, and I have a pretty noisy label distribution, a lot of the leave values exceeded learning_rate * max_delta_step = 0.034 in the example above. Thus, their leave values were clipped at [mean labels - 0.034, mean labels + 0.034], which are the [1.112882, 1.180005] values in the screenshot above.
I believe the package is working as intended. Please feel free to close this thread. Appreciate your team maintaining such a great package!

Ahhhh interesting! I did not notice max_delta_step in the parameters you provided. That makes sense to me.
Thanks for using LightGBM and for being so responsive. Come back any time 👋🏻


First pass at CONTRIBUTING guidelines

"Pull Requests" section still needs filling out and maybe some discussion.
Closes #248

Codecov Report

Merging #567 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #567      +/-   ##
==========================================
- Coverage   94.55%   94.54%   -0.02%
==========================================
  Files         235      235
  Lines        8502     8481      -21
==========================================
- Hits         8039     8018      -21
  Misses        463      463

Flag
Coverage Δ

#dotnet
94.54% <ø> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/Bicep.Core/TypeSystem/TypeValidator.cs
95.13% <0.00%> (-0.04%)
:arrow_down:

src/Bicep.Cli.UnitTests/PathHelperTests.cs
100.00% <0.00%> (ø)


rpi-ws281x broken on the latest ArchLinux

With the latest ArchLinux release, running on an A+, unicornd is not working.
After digging a bit with gdb I found that the library stucks here
    while (!(cm_pwm->ctl & CM_PWM_CTL_BUSY))

which is at python/rpi-ws281x/lib/ws2811.c:261.

Last five commits should fix issue with ArchLinux and GCC >= 5.1.0
Sorry, that is not a one fix commit, but I made it in hurry direct on github.

Any word on this or the related PR @bendo?
What needs to be done to run unicornd on Arch linux ARM?
On your fork I can get c/unicorn/unicorn.c compiled and working but unicornd and the python test scripts don't work for me -- unicornd occasionally'll throw DMA error codes.

@mil Back then, I made a quick fix. I can look at unicornd and python test scripts, but probably on weekend. No time these days.

@bendo can't wait -- most thanks for your help

I'm labeling this as such so this issue is open for future Arch users, but not on my "radar" so to speak.
A PR which fixes Arch compatibility without breaking Raspbian, which is my primary concern, would be very welcome. I really don't have the time to work on it myself :(

What's the current state of building on Arch? It might be worth submitting a patch upstream, since that's where I inherit the rpi_ws281x code from, and I plan to completely remove rpi_ws281x from this repository as soon as possible, to rely on the original version.
https://github.com/jgarff/rpi_ws281x

@Gadgetoid I think my fork should work fine. To switch rpi_ws281x to original version is good idea. I wanted to do it as well in my fork, but there is no time... Two little kids. :-) You can do changes which you want, and when you will be ready I can try it on Arch. If there will be needed some correction, I will send you pull request.

I can't seem to install this either. This is the last error message I get.

As mentioned above, but with some progress now, I plan to switch over to https://github.com/pimoroni/rpi_ws281x-python soon, which includes the library directly from the upstream dev branch (with PCM and SPI support for driving LEDs) which can be found here: https://github.com/richardghirst/rpi_ws281x


Spelling of authorization and source of diagrams?

I noticed one instance of 'authorisation' rather than 'authorization' in the text - in 2.0. APIs.md.
There are several instances of it in the three diagrams. However, they are PNGs... I tried loading them into draw.io to see if the files had embedded editable objects, but that didn't seem to be the case.
I think it would be better to have editable diagrams in the repo?

grep -r -i authorisation nmos-authorization-1.0-dev
nmos-authorization-1.0-dev/docs/2.0. APIs.md:*   **/register** - Used for the manual or dynamic registration of an OAuth 2.0 client with the Authorisation

grep -r -i authorise nmos-authorization-1.0-dev
nmos-authorization-1.0-dev/docs/1.0. Overview.md:is not concerned with the security of the connection used to carry out authorization or subsequently authorised

Who has the original sources for these images?

According to the slack you should be able to import them into draw.io and edit them there

Giving draw.io a shot...

Edited one diagram in draw.io, re-drew the other in draw.io


Optimize connection close logic to resolve timeout delay issue

Description

Introduced a configuration option useRstOnTimeout in Http1Config to allow connections to be closed with an RST (Reset) signal when a timeout occurs.
Updated HttpRequestExecutor to respect this configuration and invoke the appropriate close mode (CloseMode.IMMEDIATE for RST or Closer.closeQuietly for FIN).
Maintained backward compatibility by defaulting to FIN-based connection closure.

This change improves flexibility and allows faster resource cleanup in timeout scenarios, aligning with the HTTP/1.1 recommendations for abnormal connection termination.
Problem Description
There is an issue where the connection close operation takes as long as the SocketTimeout value set in the SocketConfig after a Socket timeout occurs. This behavior arises because the NioSocketImpl performs an additional poll for the duration of the SocketTimeout during the close operation. Consequently, HTTP requests may experience delays of up to twice the configured SocketTimeout value.
Proposed Solution
This Pull Request introduces the useRstOnTimeout configuration option to mitigate this issue by enabling connections to be closed using an RST signal instead of a FIN-based closure.

Key benefits:

Avoids unnecessary delays during connection closure.
Ensures efficient resource cleanup.
Maintains backward compatibility by defaulting to FIN-based closure.

The following image shows the test results with the socket timeout value set to 5 seconds

Example Workflow Before Fix

Socket timeout occurs:
The socket reaches the timeout value specified in the SocketTimeout configuration.
Connection close:
During the close operation, the NioSocketImpl performs an additional poll for the same timeout duration.
Total wait time:
HTTP requests may end up waiting for up to twice the configured SocketTimeout value.

Resolves: HTTPCLIENT-2324

@ooeunz Almost there. Please fix the style check violations and update the title of PR and will merge the change-set.

@ooeunz Cherry-picked to 5.3.x


Bluetooth: Mesh: remove 20ms tx delay in adv bearer

The recommendation to have 20ms is fair for two
consecutive messages over a single bearer.
When mesh sends two messages it can be interpreted as two
bearers working in parallel. No need to keep
an artificial 20ms delay for that. Delay was
removed and all related bsim tests were fixed.

Will these bsim changes be "fixed" when we implement randomization for beacons? If so, could you please move these changes to a separate commit so that we could revert it later?

It might be fixed, but might be not. I'm pretty sure my changes are correct to fix test scenario in general manner. Test scenario waits for the subnetwork beacon and then sends a beacon back. If a subnetwork is the beginning of the list, the two devices send beacons to meet each other. Test will fall from time to time, that depends on added or removed changes that adds\removes some delays here and there. My changes guarantee that there will not be collisions since rx\tx beacon sending are planned in different time.

Will these bsim changes be "fixed" when we implement randomization for beacons? If so, could you please move these changes to a separate commit so that we could revert it later?

It might be fixed, but might be not. I'm pretty sure my changes are correct to fix test scenario in general manner. Test scenario waits for the subnetwork beacon and then sends a beacon back. If a subnetwork is at the beginning of the list, the two devices send beacons to meet each other. Test will fall from time to time, that depends on added or removed changes that adds\removes some delays here and there. My changes guarantee that there will not be collisions since rx\tx beacon sending are planned in different time.

Well, it was working before because beacons from multiple subnets were split in time. Now, as per my understanding, in some tests, the dut is not able to receive a beacon from the tester as it doesn't listens to as it sends next beacon (right?), in other tests the receiver is not able to receive beacon, so BEACON_INTERVAL +  just increases the probability. Once we add a delay between beacons, the behavior should be the same as it was before this change. If we change the tests, how can we be sure that it works like it worked before?

Will these bsim changes be "fixed" when we implement randomization for beacons? If so, could you please move these changes to a separate commit so that we could revert it later?

It might be fixed, but might be not. I'm pretty sure my changes are correct to fix test scenario in general manner. Test scenario waits for the subnetwork beacon and then sends a beacon back. If a subnetwork is at the beginning of the list, the two devices send beacons to meet each other. Test will fall from time to time, that depends on added or removed changes that adds\removes some delays here and there. My changes guarantee that there will not be collisions since rx\tx beacon sending are planned in different time.

Well, it was working before because beacons from multiple subnets were split in time. Now, as per my understanding, in some tests, the dut is not able to receive a beacon from the tester as it doesn't listens to as it sends next beacon (right?), in other tests the receiver is not able to receive beacon, so BEACON_INTERVAL +  just increases the probability. Once we add a delay between beacons, the behavior should be the same as it was before this change. If we change the tests, how can we be sure that it works like it worked before?

I've already described above the reason of failed beacon tests with multiple subnetwork. Any artificial delays do not guarantee that the test will pass. Probability of failed test will always be > 0. This is the nature of an implemented test scenario.
Everywhere I added BEACON_INTERVAL +  I have already fixed since these are not adv beacons. It just relaxes waiting conditions for beacons over gatt.
Ok, I'll make the separate commit if you insist, but I do not see a lot of sense in this. Any artificial beacon delay in between will not return behavior previous back, just add the new one with the new implications and broken bsim tests.


Failed to resolve workspace dependency present in 'optimizeDeps.include'

Describe the bug
It seems that optimizeDeps.include is able to resolve only dependencies that are defined in the root workspace and its package.json. In other words, you can optimize deps only from a single workspace.
After opening the repro, you will see this warning:
Failed to resolve dependency: jquery, present in 'optimizeDeps.include'

while react gets optimized just fine because it's in the root workspace.
Expectation
optimizeDeps.include should work for all dependencies across multiple workspaces. If these deps can be later resolved and served by Vite anyway I don't see a reason why they couldn't be prebundled as well.
Reproduction
https://stackblitz.com/edit/node-whfueq?file=app%2Fpackage.json
System Info
Stackblitz

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Here I found a solution from this guy
in several words - use Vite inside the workspace, not in the root
https://github.com/adiun/vite-monorepo

The example doesn't use optimizeDeps (it was removed)?

in several words - use Vite inside the workspace, not in the root
That's the case in my example as well.

I've the same issue, With the following schema:
Package A:

The one running Vite
Has as dependency and uses B

Package B:

Has as dependency a CJS package called C

When adding C to A's Vite optimizeDeps, Vite says it can't resolve the dependency. The more I see how Vite treats Workspaces, the less I want to use it.

@phiberber Have you tried optimizeDeps.include: ['B > C'] as documented here?
@tajo The warning seems to be correct as you're starting the Vite server with the base set to pkg-a. Vite would then resolve dependencies from the root, which react exists in pkg-a, but not jquery.

I've the same issue, With the following schema:
Package A:

The one running Vite
Has and uses B as dependency

Package B:

Has as dependency a CJS package called C

When adding C to A's Vite optimizeDeps, Vite says it can't resolve the dependency. The more I see how Vite treats Workspaces, the less I want to use it.

i got this issue too!

Going to close this following my comment above.


Grammar allows infer statement on the left side in a nested conditional

🔎 Search Terms
infer on the left side, nested conditional
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about common bugs.

⏯ Playground Link
https://www.typescriptlang.org/play/?#code/C4TwDgpgBAygPAQQHxQLxQVCAPYEB2AJgM5QAUAlvgGYQBOUAQlrgSVPgK4C2ARvVAD8UYsDpUA5lABcHCADd6ASiFMZUagEMANsQgBuIA
💻 Code
type S<A> = A extends (infer B extends number ? string : never) ? B : false;

🙁 Actual behavior
It was allowed.
🙂 Expected behavior
This seems off since the constraint on A changes based on the inference of B, but this would create a cyclic type.
Additional information about the issue
No response

You might find the information contained here interesting

I think this is part of a broader range of grammar, allowing infer to be used inside constructs: playground
Most of these do make sense, but a conditional doesn't seem to have much use.

Those are especially interesting:
type I<T> = T extends (infer X extends { x: unknown })["x"] ? X : never;
type J<T> = T extends { x: string, y: number }[infer X extends "x" | "y"] ? X : never;

type I1 = I<{ x: null }>
type J1 = J<"">

Those are... weird but creative :P It kinda makes sense, TS is able to decompose this and infer into a check type (and true type and false type).  If those happen to be conditional types... well that still works.

I'll readily admit that the original example seems wrong at first sight. But while the infer B doesn't occur in the extends position of the inner conditional type, it does occur the the extends position of the outer conditional, so it isn't per se incorrect. Indeed, the compiler correctly scopes B to the outer conditional. That said, it is a pretty useless construct. You would think that an instantiation S<X extends number ? string : never> for some type variable X would infer X for B, but that never happens because we defer the entire outer conditional type when either of the check or extends types is generic. And manifest conditional types are by definition generic, as otherwise they would have resolved to their true or false type.
The comment here about why infer declarations should be permitted anywhere in conditional types (and scoped to the innermost enclosing extends clause) still holds, So I don't think we want to change anything here.


Predictions are calculated twice when radios are toggled

Things to check

Have you forced a refresh in your browser (e.g. ctrl+F5 in Chrome/Firefox)?

yes

Have you made sure the first time buyer option is correct? If you're unsure, try it both ways

yes

Have you time-travelled this week? Travelling backwards resets the prices, and therefore you must not put prices in from both before & after the time-travel

no

Describe the bug
It appears that changing a radio toggle (First-Time Buyer, Previous Pattern) triggers updateContent() twice - once on the input event, another on the change event.
To Reproduce
Steps to reproduce the behavior:

Set a breakpoint on updateContent()
1a. Alternatively, have both the radio buttons and the chart in view (without having to scroll)
Change either First-Time Buyer or Previous Pattern options
If using the debugger, note that the updateContent() function is fired twice
3a. Alternatively, if viewing the chart, notice that the chart is rendered twice in quick sequence

Permalink to your prices
https://turnipprophet.io?prices=.<IP_ADDRESS>........
Expected behavior
updateContent() should only be fired once.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Ubuntu 19.10
Browser: Firefox
Version: 75.0 64-bit

Additional context
It doesn't appear that listening to the change event in addition to the input event is necessary - radios all seem to trigger just fine on just input. I think this can be fixed by simply removing this line.

Thanks for the report. Seems the recently added denounce is what revealed the visible double rendering, because one of them is delayed 500ms.


It is not necessary to trigger fetchs for reset-style-per-node when node itself launches one when it is ready/finished

Related ticket: https://github.com/CartoDB/cartodb/issues/12085
When we introduced https://github.com/CartoDB/cartodb/pull/11989, we didn't realize we f***** up reset-styles thing. Because it was resetting the status of the query objects from the deep-insights-integration node changes (https://github.com/CartoDB/cartodb/blob/master/lib/assets/core/javascripts/cartodb3/deep-insights-integrations.js#L347-L375).
In the future, we will remove this and we will apply the specific style for each type of analysis (cc @saleiva @makella).
In order to test this, I'd check that when a new analysis is added, reset styles works properly. Visit styles panel and check that the simple one is selected when the analysis is finished.

NEWS?

Added!

Hey, I am testing this and I cannot reproduce the issue in production with the steps from the ticket. Is this something that happens always? Only some of the time?

Oh! 😱 , let me check it again.

Well, it happens. I've tried this time with 'Filter by column value' and I have added more info about the problem in the issue description.

I have been able to reproduce with a bigger dataset and a more complex analysis. Continuing..., thanks!

Looking good. I tested with multiple analysis, including those which change geometry type. Also with different initial geometries (points, lines, polygons). It resetted styles every time. ✨ Great job, guys :)


Add UT and IT for 2+ level aggregations PPL command

Description
No additional effort needed to support 2+ level aggregations PPL command. This PR is adding some unit tests and integ-tests for this feature.
Issues Resolved
Resolves https://github.com/opensearch-project/opensearch-spark/issues/488
Check List

[x] Updated documentation (ppl-spark-integration/README.md)
[x] Implemented unit tests
[x] Implemented tests for combination with other commands
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Gently ping @penghuo @vamsi-amazon @YANG-DB


keras lstm shape mismatch

I am running keras over tensorflow, trying to implement a multi-dimensional LSTM network to predict a linear continuous target variable , a single value for each example(return_sequences = False).
My sequence length is 10 and number of features (dim) is 11.
This is what I run:
import pprint, pickle
import numpy as np
from keras.models import Sequential
from keras.layers import Dense, Activation
from keras.layers import LSTM
Input sequence
wholeSequence = [[0,0,0,0,0,0,0,0,0,2,1],
[0,0,0,0,0,0,0,0,2,1,0],
[0,0,0,0,0,0,0,2,1,0,0],
[0,0,0,0,0,0,2,1,0,0,0],
[0,0,0,0,0,2,1,0,0,0,0],
[0,0,0,0,2,1,0,0,0,0,0],
[0,0,0,2,1,0,0,0,0,0,0],
[0,0,2,1,0,0,0,0,0,0,0],
[0,2,1,0,0,0,0,0,0,0,0],
[2,1,0,0,0,0,0,0,0,0,0]]
Preprocess Data:
wholeSequence = np.array(wholeSequence, dtype=float) # Convert to NP array.
data = wholeSequence
target = np.array([20])
Reshape training data for Keras LSTM model
data = data.reshape(1, 10, 11)
target = target.reshape(1, 1, 1)
Build Model
model = Sequential()
model.add(LSTM(11, input_shape=(10, 11), unroll=True, return_sequences=False))
model.add(Dense(11))
model.add(Activation('linear'))
model.compile(loss='mean_squared_error', optimizer='adam')
model.fit(data, target, nb_epoch=1, batch_size=1, verbose=2)
and get the error ValueError: Error when checking target: expected activation_1 to have 2 dimensions, but got array with shape (1, 1, 1)
Not sure what should the activation layer should get (shape wise)
Any help appreciated
thanks

Hi @nirsisense ,
I think I came across your same problem just now.
Can you try to set unroll=False in your LSTM layer? Strangely enough, in my case, if I do that the network works.
It would be good if we can pinpoint the problem.

I just started to install and try keras and tensorflow on MacBook Pro this week and i got the same problem. I still could not find out solution although i searched everywhere I could.
Any help would be very much appreciated.

Hi @phanphuong6,
what is the keras code you wrote?
Did you try to set unroll=False in your LSTM layer? Does it still give errors?
Can you post the keras code you used to define your network?


Multipart-post should be locked in to ~> 2.0

With versions < 2.0 of multipart-post, there's a significant code difference in lib/multipartable.rb that prevents headers for individual parts from being properly parsed when an article is being saved.
NoMethodError: undefined method `strip' for {"metadata"=>{"Content-Type"=>"application/json"}}:Hash
  from /Users/ravenstine/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/net/http/header.rb:19:in `block in initialize_http_header'
  from /Users/ravenstine/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/net/http/header.rb:17:in `each'
  from /Users/ravenstine/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/net/http/header.rb:17:in `initialize_http_header'
  from /Users/ravenstine/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/net/http/generic_request.rb:44:in `initialize'
  from /Users/ravenstine/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/net/http/request.rb:15:in `initialize'
  from /Users/ravenstine/.rvm/gems/ruby-2.3.0/gems/multipart-post-1.2.0/lib/multipartable.rb:11:in `initialize'
  from /Users/ravenstine/workspace/apple-news-rb/lib/apple-news/requests/post.rb:29:in `new'
  from /Users/ravenstine/workspace/apple-news-rb/lib/apple-news/requests/post.rb:29:in `multipart'
  from /Users/ravenstine/workspace/apple-news-rb/lib/apple-news/requests/post.rb:39:in `content_body'
  from /Users/ravenstine/workspace/apple-news-rb/lib/apple-news/requests/post.rb:22:in `call'
  from /Users/ravenstine/workspace/apple-news-rb/lib/apple-news/article/persistence.rb:14:in `save!'
  from (irb):2
  from /Users/ravenstine/.rvm/gems/ruby-2.3.0/gems/railties-4.2.3/lib/rails/commands/console.rb:110:in `start'
  from /Users/ravenstine/.rvm/gems/ruby-2.3.0/gems/railties-4.2.3/lib/rails/commands/console.rb:9:in `start'
  from /Users/ravenstine/.rvm/gems/ruby-2.3.0/gems/railties-4.2.3/lib/rails/commands/commands_tasks.rb:68:in `console'
  from /Users/ravenstine/.rvm/gems/ruby-2.3.0/gems/railties-4.2.3/lib/rails/commands/commands_tasks.rb:39:in `run_command!'
  from /Users/ravenstine/.rvm/gems/ruby-2.3.0/gems/railties-4.2.3/lib/rails/commands.rb:17:in `<top (required)>'
  from bin/rails:4:in `require'
  from bin/rails:4:in `<main>'2.3.0 :003 > quit

This could be prevented by only permitting the right version(s) of multipart-post.  I'm not sure there is another fix for this because I don't think that attachments to articles work without their own content-type headers.

Yeah that entire thing was a nightmare. The Apple News API is extremely specific and none of the libraries I tried were able to match the format they needed out of the box. I'm not immediately aware of any good solutions, but am open to any suggestions.

I think it's a perfectly valid solution.  It's just that I had to wrestle with my dependencies to try and figure out why that failure was happening, and too many of the gems I'm using depend on older versions of multipart-post.  It's more a result of one of the aspects of Ruby dependency management I despise.  Apple's crappy "gem" uses rest-client, though I haven't yet tested it with multipart files.  It does appear to construct headers for each part, though.


[BugFix] cherry-pick fix for spill

#26738 #25694 #25191

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)


Module build failed: SyntaxError packages/wxc-button/index.vue: Unexpected token (43:8)

I copy the code from the sample tab-bar
[372] multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8081 ./temp/_mods/category.js?entry=true 40 bytes {2} [built]
+ 81 hidden modules
ERROR in ./~/babel-loader/lib!./~/vue-loader/lib/selector.js?type=script&index=0!./~/weex-ui/packages/wxc-button/index.vue
Module build failed: SyntaxError: C:/app/codes/dmaio_niu/niu-app/node_modules/weex-ui/packages/wxc-button/index.vue: Unexpected token (43:8)
41 |
42 |       const mrBtnStyle = {

43 |         ...STYLE_MAP[type],
|         ^
44 |         ...btnStyle,
45 |         ...BUTTON_STYLE_MAP[size]
46 |       };

It seems very easy to get error ,not easy to use.
no document.

I got it
it is because I didn't install babel-preset-stage-0
use this command
npm i babel-plugin-component babel-preset-stage-0 -D
and modify .babelrc to
{
"presets": ["es2015", "stage-0"],
"plugins": [
[
"component",
{
"libraryName": "weex-ui",
"libDir": "packages",
"style": false
}
]
]
}

I have the same problem.
use this command
npm i babel-plugin-component babel-preset-stage-0 -D
and modify .babelrc
but  it's still a mistake.
development environment:  centos 6.8 ,node(6.9.2 ~ 10.+)
seek the answer of the God.

If babel-loader in webpack.config.js has a exclude for node_modules, please turn on for week-ui as exclude: /node_modules(?!(/|\).(weex).)/.


Archives – confirm correct 'Categorized As' type

Rob,
PMC Archive collection records appear to either be classified as "Archives" (see https://lux-front-sbx.collections.yale.edu/view/set/7aaacfd9-3df5-4e79-8355-71946a1e2085) or "Physical Objects" (see https://lux-front-sbx.collections.yale.edu/view/object/ce373124-a009-42cc-a5bc-81c59ee6556b)
Yale archive collections use the categorisation "archival materials" (see https://lux-front-sbx.collections.yale.edu/view/object/647f74c4-338f-400a-a2c2-0f0cf3a97722).
Which is preferred?

HMOs from the archive should have a classified_as of archival materials, yes.
The class of "Physical Object" is just that it's an HMO, rather than a Digital Object.

Current logic
https://github.com/tomcrane/linked-art-net/blob/pmc/LinkedArt/PmcTransformer/Archive/Helpers.cs#L24
All Sets are "archives (groupings)", https://vocab.getty.edu/aat/300375748
HMOs with level Item are all "Archival Materials", https://vocab.getty.edu/aat/300379505
Sets with level Sub-Series are additionally https://vocab.getty.edu/aat/300404023
Sets with level Series are additionally https://vocab.getty.edu/aat/300404022
Sets with level Collection are additionally https://vocab.getty.edu/aat/300025976
So I think I am emitting the right stuff? I don't see "Archives" or "Physical Objects" in the two linked resources in the description.

https://github.com/tomcrane/linked-art-net/commit/ad256964d8f69765f6b7aa01f070ec58f475e610


dotenv support and documentation added

Hey @fyalavuz, I updated the doc for giving more info. If it's okay for you (or not, please change wording), I will merge it.

Hi Ilker, you can merge this version.
Thank you.


Handle null dimension sets on SetDimensions

Issue #, if available:
#17
Description of changes:
Using Linq to remove any dimension sets passed as null.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Fixed by #37


Update go to 1.19

It appears that updating go to 1.19 for playwright was missed when we updated to go 1.19 elsewhere.
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Should this also be 1.19?
https://github.com/go-gitea/gitea/blob/bc532563169b14cf6e51db6ca7190289fd81c0f1/go.mod#L3

@yardenshoham this is kept at 1.18 as that is the lowest version we currently support.


Need to document escape sequences

Strings and bytes have escape sequences that should be documented.

This appears to be done.


Implement isHex(), isBinary(), and isOctal()

To make user inputs foolproof, we need to implement functions isHex(), isBinary(), and isOctal() which returns true or false, depending on whether the user's input for hex, binary, and octal are all valid. See the TODO's in functions.h on each function prototype for further detail.

I'm planning on working on these functions

Sure, go ahead