Add docs for TIME_AXIS_MARKERS and markerSideDeltas

Hello, I'm using your package for a use-case that requires 4 digit time markers and a colon (i.e., 02:00 for 2am). I'm running into issues when I zoom with the numbers running together at the beginning of a new zoom breakpoint:

I started problem-solving by attempting to add marginLeft to the markers being mapped in the component, but that didn't do anything to the individual divs. It just added margin to the time axis as a whole (at least everything right of the sidebar). I'm also concerned that messing with the marker spacing and those div widths could make the time cursor inaccurate.
I've been playing with a custom usePanStrategy deltaY multiplier to give it more room between time markers when a user scrolls, tho this is a little janky as an experience, and I've also looked at eliminating some time markers from the default TIME_AXIS_MARKERS array, but I don't feel confident enough that I understand what they are to do that.
I'm pretty confused with what is happening between the TIME_AXIS_MARKERS values and how the markers are being calculated in the TimeAxis component, particularly multiplier stuff in the for loop. I'm really having to slow down and take it line by line, console.logging all of the vars and zooming to different levels to understand how these are correlating to what is on the screen.
Writing this issue out has helped me grasp it a little better:

timeframe represents the start and end times of the current zoom level
delta is the smallest value (in milliseconds) of the TIME_AXIS_MARKERS array

why is startTime calculated the way it is with the flooring and references to the delta? I noticed endTime doesn't .ceil or reference the delta

the for loop is grabbing the first TIME_AXIS_MARKER entry whose value is an even dividend of the floored start time and is within the marker's maxTimeframeSize and minTimeframeSize if both are present, are above the min or below the max if only one is present, or are an element where neither are present

why do some TIME_AXIS_MARKER entries not have a max/minTimeframeSize?

the rest of the multiplier stuff is to get the marker to the correct position from the start time of the zoomed timeframe

I'm still curious about the above questions, as well as the need in my use-case to have both the date labeled as well as 00:00. I'm thinking I may need to have two time-axes in my app, one for just dates (above the hourly markers) that sticks as you scroll through a 24 hr period. Here's the design I'm working off of.

Thanks for considering my suggestions and answering any questions. I'm also happy to help contribute to this request if you'd like any assistance

Oh, another question I had about zoom and pan is if there's a way to have a max/min zoom? For example, I don't want users to be able to pan to less than 15 min markers or more than 2 days
Again assuming that's somewhere in the TIME_AXIS_MARKERS values, but also thinking the limiting would be something I need to change via css too

Hey!

The time axis is a private implmentation that uses the library but is not a part of it. Therefore, there is no documentation for it. That being said, I have already implemented in the past a time axis that looks and behaves just like the design you are working off. If you want, I can try and replicate it and send you the code.
You can very simply set limits for zooming and panning via the setTimeframe function: when zooming or panning, the onTimeframeChanged callback is being called with an updater function. therfore, you can very simply check if the updated timeframe is valid, and only if valid actaully procceed with the state object.

Feel free to contact me if you need further help :)

Hi, Samuel!

Oh, that would be awesome.
Sweet – thanks for the info there.

Thank you!

Cole Hart
Senior Front End Developer // Hypergiant Galactic
Hypergiant.comhttps://www.hypergiant.com/ // Know Tomorrow, Act Today™
303.330.9966 // LinkedIn: Hypergianthttps://www.linkedin.com/company/hypergiant/
---- [ END // OFFICIAL TRANSCRIPT VERIFICATION ] ----
From: Samuel Arbibe @.>
Date: Tuesday, June 4, 2024 at 10:53 AM
To: samuelarbibe/dnd-timeline @.>
Cc: Cole Hart @.>, Author @.>
Subject: Re: [samuelarbibe/dnd-timeline] Add docs for TIME_AXIS_MARKERS and markerSideDeltas (Issue #21)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
Hey!

The time axis is a private implmentation that uses the library but is not a part of it. Therefore, there is no documentation for it. That being said, I have already implemented in the past a time axis that looks and behaves just like the design you are working off. If you want, I can try and replicate it and send you the code.
You can very simply set limits for zooming and panning via the setTimeframe function: when zooming or panning, the onTimeframeChanged callback is being called with an updater function. therfore, you can very simply check if the updated timeframe is valid, and only if valid actaully procceed with the state object.

Feel free to contact me if you need further help :)
—
Reply to this email directly, view it on GitHubhttps://github.com/samuelarbibe/dnd-timeline/issues/21#issuecomment-2147991191, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIYS4CVV5CF2VV7EARIFY2LZFXWHTAVCNFSM6AAAAABIW4KB5GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBXHE4TCMJZGE.
You are receiving this because you authored the thread.Message ID: @.***>

 const markers = useMemo(() => {
    // sort the markers from large to small.
    const sortedMarkers = [...props.markers];
    sortedMarkers.sort((a, b) => b.value - a.value);

    // the smallest marker size, in ms
    const delta = sortedMarkers[sortedMarkers.length - 1].value;

    // the total timeframe size/
    const timeframeSize = timeframe.end.getTime() - timeframe.start.getTime();

    const startTime = Math.floor(timeframe.start.getTime() / delta) * delta;

    const endTime = timeframe.end.getTime();
    const timezoneOffset = minutesToMilliseconds(
      new Date().getTimezoneOffset(),
    );

    const markerSideDeltas: Marker[] = [];

    // Here, we "walk" from the start of the timeframe to the end of it,
    // and on every step we check if a marker should be placed there.
    // A marker "should" be placed in a certain location if it divides the current location's time
    // without remainder.
    // using the findIndex function, we find the largest matching marker,
    // because the sortedMarkers is sorted from large to small.
    // that way, if on 24:00 we have both an hour marker and a day marker, we only display the day.
    for (let time = startTime; time <= endTime; time += delta) {
      const multiplierIndex = sortedMarkers.findIndex(
        (marker) =>
          (time - timezoneOffset) % marker.value === 0 &&
          (!marker.maxTimeframeSize ||
            timeframeSize <= marker.maxTimeframeSize) &&
          (!marker.minTimeframeSize ||
            timeframeSize >= marker.minTimeframeSize),
      );

      if (multiplierIndex === -1) continue;

      // optional: use the index of the selected marker to adjust it's height relativly to other markers.
      const multiplier = sortedMarkers[multiplierIndex];

      const label = multiplier.getLabel?.(new Date(time));

      // push the marker to the array
      markerSideDeltas.push({
        label,
        heightMultiplier: 1 / (multiplierIndex + 1),
        sideDelta: millisecondsToPixels(time - timeframe.start.getTime()),
      });
    }

    return markerSideDeltas;
  }, [timeframe, millisecondsToPixels, props.markers]);

Hello, I'm using your package for a use-case that requires 4 digit time markers and a colon (i.e., 02:00 for 2am). I'm running into issues when I zoom with the numbers running together at the beginning of a new zoom breakpoint:

I started problem-solving by attempting to add marginLeft to the markers being mapped in the component, but that didn't do anything to the individual divs. It just added margin to the time axis as a whole (at least everything right of the sidebar). I'm also concerned that messing with the marker spacing and those div widths could make the time cursor inaccurate.
I've been playing with a custom usePanStrategy deltaY multiplier to give it more room between time markers when a user scrolls, tho this is a little janky as an experience, and I've also looked at eliminating some time markers from the default TIME_AXIS_MARKERS array, but I don't feel confident enough that I understand what they are to do that.
I'm pretty confused with what is happening between the TIME_AXIS_MARKERS values and how the markers are being calculated in the TimeAxis component, particularly multiplier stuff in the for loop. I'm really having to slow down and take it line by line, console.logging all of the vars and zooming to different levels to understand how these are correlating to what is on the screen.
Writing this issue out has helped me grasp it a little better:

timeframe represents the start and end times of the current zoom level
delta is the smallest value (in milliseconds) of the TIME_AXIS_MARKERS array

why is startTime calculated the way it is with the flooring and references to the delta? I noticed endTime doesn't .ceil or reference the delta

the for loop is grabbing the first TIME_AXIS_MARKER entry whose value is an even dividend of the floored start time and is within the marker's maxTimeframeSize and minTimeframeSize if both are present, are above the min or below the max if only one is present, or are an element where neither are present

why do some TIME_AXIS_MARKER entries not have a max/minTimeframeSize?

the rest of the multiplier stuff is to get the marker to the correct position from the start time of the zoomed timeframe

I'm still curious about the above questions, as well as the need in my use-case to have both the date labeled as well as 00:00. I'm thinking I may need to have two time-axes in my app, one for just dates (above the hourly markers) that sticks as you scroll through a 24 hr period. Here's the design I'm working off of.
Thanks for considering my suggestions and answering any questions. I'm also happy to help contribute to this request if you'd like any assistance

In your design, I see that you need to render the day and the hour marker differently.
In that case, I would deffinitly separate them into 2 different time axis.
The simplest solution I can think of, rendering the TimeAxis twice, with different timeAxisMarkers, and also pass through some styling.
You can also separate them into 2 different TimeAxis components, up to you!

(there is also a "fake" hour marker in the daily because of a bug I just discovered... the % method doesn't work with markers greater of equal then days because of changing UTC offsets...)
And it looks something like that:

Add a bit of styling... and thats it!

Dope - thank you!

How would you make the date "sticky" to whatever pan level is shown?

You have 2 options:

Use the CSS position: sticky attribute. Should work just fine.
Use a different component to display the current day statically.


Refactor docs after recent changes

To do:

[x] Use intersphinx as much as possible.
[ ] See if we can make -h in the command-line not show reStructuredText syntax.

Codecov Report

Merging #67 (a08dd85) into main (f36cab9) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head a08dd85 differs from pull request most recent head 0ce744c. Consider uploading reports for the commit 0ce744c to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #67   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           17        17
  Lines          470       478    +8
  Branches        58        58
=========================================
+ Hits           470       478    +8

Files
Coverage Δ

zyte_api/__init__.py
100.00% <100.00%> (ø)

zyte_api/__main__.py
100.00% <100.00%> (ø)

zyte_api/_async.py
100.00% <100.00%> (ø)

zyte_api/_errors.py
100.00% <100.00%> (ø)

zyte_api/_retry.py
100.00% <ø> (ø)

zyte_api/_sync.py
100.00% <ø> (ø)

zyte_api/errors.py
100.00% <ø> (ø)


DBZ-660 Fix for CONSTRAINT UNIQUE syntax

https://issues.jboss.org/browse/DBZ-660

LGTM, merged. Thanks, @jpechane!


Turtlebot3 - Burger Error 'cdc_acm 1-1.2:1.0: failed to set dtr/rts'

ISSUE TEMPLATE ver. 0.4.0

Which TurtleBot3 platform do you use?

[ v] Burger
[ ] Waffle
[ ] Waffle Pi

Which ROS is working with TurtleBot3?

[ ] ROS 1 Kinetic Kame
[v ] ROS 1 Melodic Morenia
[ ] ROS 1 Noetic Ninjemys
[ ] ROS 2 Dashing Diademata
[ ] ROS 2 Eloquent Elusor
[ ] ROS 2 Foxy Fitzroy
[ ] etc (Please specify your ROS Version here)

Which SBC(Single Board Computer) is working on TurtleBot3?

[ ] Intel Joule 570x
[ ] Raspberry Pi 3B+
[ ] Raspberry Pi 4
[v ] etc (Raspberry Pi 3B)

Which OS you installed on SBC?

[ ] Raspbian distributed by ROBOTIS
[ ] Ubuntu MATE (16.04/18.04/20.04)
[ ] Ubuntu preinstalled server (18.04/20.04)
[v] etc (ubuntu-18.04.5-desktop-amd64)

Which OS you installed on Remote PC?

[ ] Ubuntu 16.04 LTS (Xenial Xerus)
[v ] Ubuntu 18.04 LTS (Bionic Beaver)
[ ] Ubuntu 20.04 LTS (Focal Fossa)
[ ] Windows 10
[ ] MAC OS X (Specify version)
[ ] etc (Please specify your OS here)

Specify the software and firmware version(Can be found from Bringup messages)

Software version: [x.x.x]
Firmware version: [x.x.x]

Specify the commands or instructions to reproduce the issue.

$ cd ./opencr_update
$ ./update.sh $OPENCR_PORT $OPENCR_MODEL.opencr

Copy and Paste the error messages on terminal.

cdc_acm 1-1.2:1.0: failed to set dtr/rts

Please describe the issue in detail.

I followed this link(https://emanual.robotis.com/docs/en/platform/turtlebot3/opencr_setup/#opencr-setup).
In 'Step 3.3 OpenCR Setup', I proceeded as shown in Step. 5.
Then, the error message appears.
I entered the recovery mode as shown in Step. 7.
And I repeat the Step. 1~5. But It doesn't work. Same error appears.

Hi @junu5121
Have you set up the USB Port in the SBC section as below?
$ rosrun turtlebot3_bringup create_udev_rules

Hi @junu5121
Have you set up the USB Port in the SBC section as below?
$ rosrun turtlebot3_bringup create_udev_rules

Once you see the [OK] jump_to_fw, the firmware is successfully uploaded to OpenCR.
I think there's something wrong with your serial port setting with Linux.

Yes, I did. But It didn't work....

@junu5121
Sorry about the delayed response.
Have you tried disabling the modem manager?
$ sudo apt-get purge modemmanager

Same Issue, Is this issue fixed or not?

@Kuo-Feng
Please give us more information about your system:

Which OS you installed on SBC?
Which ROS version are you using?
Which RPi model do you have?
Have you tried with a provided SBC image from ROBOTIS eManual?

@Kuo-Feng
Please give us more information about your system:

Which OS you installed on SBC?
Which ROS version are you using?
Which RPi model do you have?
Have you tried with a provided SBC image from ROBOTIS eManual?

I installed Ubuntu 20.04 on SBC, from the image that provided on the tutorials. And the ROS version is noetic. The RPi model is the one that Turtlebot 3 burger used.
There will have that message, but I think the firmware is uploaded to OpenCR sucessfully, because I can remote control the burger through teleop_key package.
Not sure that failure will have impact or not.
But currently, it works just fine.
Thank you.

@Kuo-Feng
If Raspberry Pi fails to access the ttyACM0 port to communicate with OpenCR, it could cause malfunction in operating TurtleBot3.
Since you have successfully operated with teleop_key node, it looks fine, but the cause of the problem looks like the modemmanager as described in the RPi forum below.
https://raspberrypi.stackexchange.com/questions/102022/bad-serial-connection-on-usb-otg-serial-on-pi-zero
Thank you.

Same Issue here on TurtleBot3 Burger ROS2 (FOXY)
It says [OK] jump_to_fw, can I proceed, or I will get some problems in the future?

@Lopetex I'm also getting the same error. Did you find a fix? or did you not face any other issues related to this?

Hi @keerthanamanivannan, I'm sorry but I couldn't find a fix, but as far as I know this message might not be crucial to make the robot work. In my experience, I kept encountering problems working with ROS2 and I arrived to one dead point where I couldn't communicate with the robot. The communication system changes between ROS and ROS2 so I tried ROS Noetic instead and worked amazing.
Good luck!

Apologies for the delayed response. We have assembled a TF team to address TurtleBot3 issues and are actively working on resolutions. Please note that we do not support EOL (End-of-Life) ROS versions; our support currently extends to Noetic and Humble versions. We kindly ask that you try again with the latest version. If the same issue arises, please feel free to resubmit it, and we will address it promptly.


FEATURE REQUEST: Support for structured response

I was just wondering if you had any plans to integrate the option to request a structured response in the form of JSON from Gemini by providing schema?
I have used schemas in the past using the node js library and they can be really effective when you need to force a specific response as sometimes Gemini can waffle on even when you ask it to be specific.  Link to Gemini Structured Response
I could possibly look into adding it myself but I didn't want to waste my time if it's already been tried and abandoned for some reason I have not yet realised.

Hi Brett - a useful request. I've pushed an update to a branch for you to test it hasn't broken any of your existing uses of the GeminiApp library. Here is the updated library code and a test you can try

Thanks very much.  I've tested the new library code and it doesn't seem to have affected any old usage.  I am getting an error though when trying to run the test code your provided.
Request to Gemini failed with response code 400 - {
"error": {
"code": 400,
"message": "Invalid JSON payload received. Unknown name "responseMimeType" at 'generation_config': Cannot find field.\nInvalid JSON payload received. Unknown name "responseSchema" at 'generation_config': Cannot find field.",
"status": "INVALID_ARGUMENT",
"details": [
{
"@type": "type.googleapis.com/google.rpc.BadRequest",
"fieldViolations": [
{
"field": "generation_config",
"description": "Invalid JSON payload received. Unknown name "responseMimeType" at 'generation_config': Cannot find field."
},
{
"field": "generation_config",
"description": "Invalid JSON payload received. Unknown name "responseSchema" at 'generation_config': Cannot find field."
}
]
}
]
}
}
I'm not sure if it has something to do with the generationConfig object being in camel case compared to the generation_config object mentioned in the error message being in snake case.
The documentation seems a little unclear about this and when I tried other generationConfig options like "maxOutputTokens" it worked fine.  It just doesn't like "responseMimeType" and "responseSchema".
I also tried different Gemini models but I got the same error message.

ah - if you are using a api key from Google AI Studio then the issue is likely to be you need to declare apiVersion: 'v1beta' like this (catches everyone out as the documentation isn't clear)
  const model = genAI.getGenerativeModel({
    model: "gemini-1.5-pro",
    apiVersion: 'v1beta',
    generationConfig: {
      responseMimeType: "application/json",
      responseSchema: schema
    }
  });

That almost worked but it did put me on the right track.  Where you put the apiVersion:'v1beta was in modelParams rather than requestOptions.
This worked for me:
    model: "gemini-1.5-pro",
    generationConfig: {
      responseMimeType: "application/json",
      responseSchema: schema
    }
  }, {
    apiVersion: 'v1beta',
  });```


Cleanup task to delete old ACM certs

What
This was left over from when we switched the concourse machines to use
the wildcard system domain cert. These old certs have all been cleaned
up, so this can be removed.
I've checked in the ACM console, and we no longer have any certs left to cleanup.
How to review
Code review is probably enough.
Who can review
Not me.

Shouldn't we also delete ./paas-bootstrap/concourse/scripts/delete_acm_cert.sh?


How to clear sort during data refresh

I'm hoping to clear both search and sort after loading new data (using v2 and Svelte 5). The following properly clears search. However, if a column is sorted prior to calling setRows(), that sort persists after setRows():
$effect(() => {
    data;
    table.clearSearch();
    table.setRows(data)
})

clearSort() is not listed as an available method on the docs site though it looks like #40 introduced it. Assuming that was in an older version though (am a new user to this library). Trying to use it yields table.clearSort is not a function.
Is there a different way to clear sort in v2? Maybe I'm off here

Indeed, there is no way in V2. Thx for pointing it out.
clearSort() can be added easily.

Just published on npm, 2.2.0
Doc: https://vincjo.fr/datatables/api/client/methods~clearSort


jf cli limited to 255 characters during debian upload

Describe the bug
We are currently trying to upload a single debian package, but with debian/ubuntu metadata to allow it to upload for multiple distributions..
jf rt u "$FILE" "deb/pool/main/a/xxxxxx/$(basename ${FILE});deb.architecture=all;deb.component=main;deb.distribution="trixie";deb.distribution="bookworm";deb.distribution="bullseye";deb.distribution="buster";deb.distribution="noble";deb.distribution="jammy";deb.distribution="focal";deb.distribution="bionic"" --flat=true
Is there a way to upload a debian package and then update the deb.distributions as a 2nd actions using the jf cli?
Current behavior
This worked fine before trixie and noble were added, but now its violating a field size property, as shown below
Error: 7 [Error] Failed uploading 1 artifacts.
Error: 7 [Error] server response: 500
{
  "errors" : [ {
    "status" : 500,
    "message" : "org.postgresql.util.PSQLException: ERROR: value too long for type character varying(255)"
  } ]
}

Reproduction steps
Upload a .deb with metadata for several distributions, as shown in the command example.
Expected behavior
No response
JFrog CLI version
2.56.0
Operating system type and version
Fedora 39
JFrog Artifactory version
Jfrog Cloud ( packages.adoptium.net )
JFrog Xray version
No response

@steelhead31
Thank you for using the JFrog CLI.
You could attempt uploading the artifact and configuring the property as two separate steps. For instance:
jf rt u <source> <target>

jf rt sp <target> "deb.architecture=all;deb.component=main;deb.distribution=trixie;deb.distribution=bookworm;deb.distribution=bullseye;deb.distribution=buster;deb.distribution=noble;deb.distribution=jammy;deb.distribution=focal;deb.distribution=bionic"

If you're still facing the 255 characters limitation, think about dividing the process into smaller segments by executing the jf rt sp command multiple times.
Please let me know if that helped.


make syntax less verbose

make attrs.Props a Node
when parsing new element check if first is props and if it is use it if else empty props.
some builders are now not 100% correct like TextArea can take any Node, this can be fixed though with some simple guards inside that one to allow only 1 textNode and props.
fixed tests to remove nil

if you want to send in more Props here you could even allow merging them all in newElement.

or you could even create a method to generate a single Prop and write code like
Div(Name("foo"), With("100px"), TextNode("foo")) not sure if that is a good idea though.

@bjartek If you are looking for that Div(... syntax, you might want to check out https://www.gomponents.com, which is an alternative to this library. I have competing prototypes in development using this library and https://www.gomponents.com with a few of the + packages (see header on website).
This is a great library, as is the other. There are pros and cons to each.

@bjartek, I'm intrigued by removing the nil concept for the attrs argument. One potential hiccup might be if someone places attrs in the middle of element children like:
content := elem.Div(
    elem.H1(nil, elem.Text("Hello, Elem!")),
    attrs.Props{
        attrs.ID:    "container",
        attrs.Class: "my-class",
    },
    elem.H2(nil, elem.Text("Subheading")),
    elem.P(nil, elem.Text("This is a paragraph.")),
)

I guess technically, this syntax still works, but I'm curious if it feels strange. Also, I'm wondering if this constitutes a breaking change. Moving some functions or attributes to an options subpackage might be.
Regarding the Div syntax that @chrismalek is discussing, one can eliminate the elem. package prefix by using the following dot import syntax, if they prefer a cleaner look:
import (
    . "github.com/chasefleming/elem-go"
    "github.com/chasefleming/elem-go/attrs"
    "github.com/chasefleming/elem-go/styles"
)

content := Div(attrs.Props{
    attrs.ID:    "container",
    attrs.Class: "my-class",
},
    H1(nil, Text("Hello, Elem!")),
    H2(nil, Text("Subheading")),
    P(nil, Text("This is a paragraph.")),
)


[flutter_tools] teach flutter drive to uninstall if install fails

Description
Work towards https://github.com/flutter/flutter/issues/39925
Currently flutter run will uninstall and reinstall if the initial install fails and the APK was previously installed. Allow drive to share this same logic by moving it into installApp and out of startApp.
This should reduce the occurrence of the error in the devicelab.

https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/commands/drive.dart#L472

Maybe we remove those checks too, then it would only be using startApp - it can definitely fail before that point though.

https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/commands/drive.dart#L472

Oops, not sure how my usages search missed that.

Can I do a follow up with the drive changes? It turns out we have 2 separate ways to install packages, and drive does not know about prebuilt applications

I removed the install logic because it is wrong, I will do a follow up to make this work with --use-application-binary


Approximate nearest neighbors suggestion

Hello and thanks a lot for making this open-source !
I was wondering if you had considered the NMSLIB-HNSW approximate nearest neighbor algorithm for use with these embeddings, as according to (here) it is faster than Annoy ?

Hello! thanks so much for the suggestion, I'll take a look at the lib!

There is also Scann that looks even faster, but I haven't tried it. I used NMSLIB-HNSW in the context of searching for neighbours in embeddings space from contrastive learning using the tensorflow-similarity lib, which is similar to this repo's use-case. Tensorflow-similarity doesn't use Scann so maybe there's a subtle blocker !


Added the context feature to server side only. Client can't access th…

…e context (i.e) RequestEvent type

Added some new changes today, this pr might break it


refactor: remove dead code in test_should_return_evm_address_in_bytes32 (new)

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Resolves #1308
What is the new behavior?

This change is

Thank you for the PR, please update the branch


[Doc enhancement][Update configure-identity-provider-acro-score][]

Fix the acrolinx score

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/concepts/authentication/configure-identity-provider.md
89
4
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit c699ff4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/authentication/configure-identity-provider.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Create Account

Hi, its possible to you can add to the library, the possibility to can create playstation accounts, xbox accounts and finaly ea accounts?

As this cannot be done via the FUT WebApp / Companion App i don't think this should be implemented in this framework. (Beside of I don't know how to do this without buying a FIFA game for the corresponding platform)

Hi tringler thanks for you answer, but its necessary if you dont want to create accounts

Whats with player packs? How i can open player packs or see if a player has a contract?

Feel free to open a pull request :) - Open player packs is not implemented to my knowledge - You can check the contracts on itemdata which is attached to auctioninfo object.

The prob is i dont know how :D

Here you go: https://github.com/trydis/FIFA-Ultimate-Team-Toolkit/issues/208#issuecomment-165779443

No i dont know how, i must do it programatically. To access to the player packs.

Sorry, but I don't get you - What do you want to do exactly? What is the issue?

The issue was at first about the accounts. Then i have asked if its possible to access to player packs

Yes and I explained to you how to create pull requests for these implementations,

the second point

Ok - I'm basically open to raise a pull request regarding opening packs for the toolkit, but I don't see the feature to create FUT accounts definitely **not ** in the toolkit as long as the WebApp give us that possibility.

@tringler do you know how to get consumables from club? for now we can only get players from club.

Pls don't open the 3rd topic on this issue thread. Pls open a new one.

@ExtremeG: Can you pls open a new issue thread regarding "Open Packs" and close this one regarding "Create Accounts", because it should not part of the toolkit, I will take care of the "Open Packs" issue the next weeks.

@trydis : Could you pls close that issue? I assume "Create Origin accounts" should not be a part of the toolkit.


feat(cli): accepts options corresponding to rules

-c|--current, -p|--plugin, -a|--all-available, -u|--unused
BREAKING: updated package name

@ta2edchimp could you take a look?

:+1:
You should enable this repo on https://travis-ci.org

Oh... And when renaming the plugin, I guess the repository field in package.json should be updated to point here.


kernel/modules-closure.sh v2: fix the modules closure

Prior to this commit the modules closure could reference nonsensical
pieces of data in to form of module parameters. These were treated
as seperate modules.
"modprobe --show-depends" outputs a line of the form
"insmod module_name module_args", we are only interested in the
module name, so we extract it with sed.
Also fixes a bug in the original implementation where the check for missing modules did not work as indended (the exit status of
modprobe was ignored due to a pipe).

I kind of forgot about this one, although the bug still triggers. Can / should we get more testing for this?

closed in favour of #92081


Implement Google Tag Manager and/or Analytics

Done in https://github.com/memobank/growth-guide/pull/18. Closing.


Bug: The new item dialog cannot be canceled

Description
When I press Ctrl+Shift+I, the new item dialog appears. But once creating a new item, this dialog cannot be canceled even if I press the Cancel button and a dialog to enter a new name will appear.
Steps To Reproduce

Press Ctrl+Shift+I and create a new item.
Press Ctrl+Shift+I again.
Press Cancel on the dialog.
The operation is not canceled and a dialog to enter a new name appears.

Requirements
The operation should be canceled when Cancel is pressed.
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.2506
Log File
n/a

Thanks for the report.

I can't repro this issue

I can't repro this issue

I've updated the Steps To Reproduce because I've found that there are conditions under which the problem can occur.

I found the problem if I could be assigned to this issue so nobody else starts looking into it please :)


max_tokens must be at least 1, got -160

Initial Checks

[ ] I have searched GitHub for a duplicate issue and I'm sure this is something new
[ ] I have read and followed the docs & demos and still think this is a bug
[ ] I am confident that the issue is with modelscope-agent (not my code, or another library in the ecosystem)

What happened + What you expected to happen
max_tokens must be at least 1, got -160
Versions / Dependencies
max_tokens must be at least 1, got -160
Reproduction script
max_tokens must be at least 1, got -160
Issue Severity
None

could you please provide the code to repeat this error, thanks.


WIP: Vue3 support

[x] make tests pass with Vue 3
[ ] make tests pass both for Vue 2 and Vue 3
[ ] bundle both for Vue 2 and Vue 3

❤


fixsupport uncorrelated subquery in join condition

pick from master https://github.com/apache/doris/pull/26672
commit id: 17b110863578a805a73205093b745f030c588fbf
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Atlas Ship

Atlas is a low population ship based off the goonstation map of the same name. All credit for original layout of the ship go to Gannets. This map has been modified for ss14 and changes were made in various places although the overall layout should feel similar. Passive ship power lasts around 10 minutes. This ship runs off AME/solar and can sustain off 1 core AME for around 30 minutes. Dual solars sustains for a bit short, and combined can be net positive power return. Atmos is a 2 tank setup for bare minimum performance due to ship size. The tanks are on board the ship so I have included an 'oh shit' button in case a window to a miner is broken open which will expose the tanks to vacuum. This station is fully functional as a ship however I did not include gyroscopes or a shuttle console to start to prevent salvage roundstart CBT. These will have to be built and assembled to make it functional.
you know the drill heres some candid screenshots
bridge

medbay

hydroponics

science

south hangar bay

engineering

thank you for reading and hope you enjoy
:cl: Emisse

add: Atlas, a low-pop ship map adapted from Goonstation (SS13). Major credit goes to all those who worked on the original layout of Atlas on Goonstation.

new development env map discovered

i wish renderer would do the decals that arent tiled snapped properly, all the plants look like theyre spilling out lol

Could shift this a bit to not be a 1-wide hall.

ill write that down for later i been mapping all day and need a break lol

ill write that down for later i been mapping all day and need a break lol

No problem. Do you want it merged now for testing, it otherwise lgtm and would let us get issue reports.

Yea that would be nice but it'll probably only be testable during low pop hours due to size.

Yea that would be nice but it'll probably only be testable during low pop hours due to size.

Bet. (we can just run two)

Hey it's that map I made.

giving the render a quick look. am i mistaken or are the north dockingports the wrong way around?

giving the render a quick look. am i mistaken or are the north dockingports the wrong way around?

They're fine ingame. Map renderer doesn't like rotations.


Update cors.md

Added the words "in response" and "from the request" under the section "How CORS works" to improve clarity.

@elayamathy Thanks for clarifying this section of the doc. Congratulations on your 1st contribution to the docs as well!


.net core app 3.1 session problem ???

I cannot access .net core app 3.1 IHttpContextAccessor HttpContext  Session. HttpContext always comes null....

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

ID: 76e22150-f1f1-5e4f-0e97-39b1ac089297
Version Independent ID: 687172c3-b7d8-525d-39dc-658b4a0f526f
Content: Session and app state in ASP.NET Core
Content Source: aspnetcore/fundamentals/app-state.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello! Thanks for contacting us. This sounds like a general question about using ASP.NET Core. While we try to look at and respond to all issues, for questions like this we recommend posting to a community support group like Stack Overflow with the asp.net-core tag.

Stack Overflow
ASP.NET Core Slack
ASP.NET Gitter

Potential bugs in the ASP.NET Core framework and product feedback can be reported at dotnet/aspnetcore issues (include full repro steps and cross-link to a repro project in GH when appropriate). For Visual Studio, use the Report a Problem gesture or Suggest a Feature gesture within VS, which open internal issues for the VS team. For Visual Studio Code, you can open an issue at microsoft/vscode issues.


examples/getting-started: revert bind mount for /var/lib/cilium

PR #11022 broke running Cilium in Docker on master. The bind mount is
actually not needed as the files are installed within the container.
This reverts commit eca8c985aab1 ("examples/getting-started: add bind
mount for /var/lib/cilium to docker-compose.yml")
Verified that the gsg still runs on master (using CILIUM_TAG=latest CILIUM_VERSION=latest vagrant up) and on 1.7 with this change.

test-me-please

Coverage decreased (-0.01%) to 46.823% when pulling a91847e925b2b3e4d23dfc83b4655a382b78c783 on pr/tklauser/docker-gsg-fix-master into 9fbd82092def7ced2d520408435f61d6f6408848 on master.

@joestringer @tklauser the original patch in the fixes was backported to v1.7 do we need to backport this as well?

@joestringer @tklauser the original patch in the fixes was backported to v1.7 do we need to backport this as well?

Discussed this on Slack with @jrfastab, the original patch shouldn't have been backported, so this is not needed as well. He's dropping the original patch now.


MEN-6544: Integrate mender-setup and mender-snapshot repositories

I was not very sure on where to place them. I put them at the end of the 2x series: after mender-connect and mender-configure-module and before mender-artifact and mender-cli (3x series).


Release @google-cloud/firestore v1.2.0

This pull request was generated using releasetool.

Release is at https://github.com/googleapis/nodejs-firestore/releases/tag/v1.2.0

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


Sensor#virutal?

a convenience method for testing if a sensor is virtual or not. also included in a sensor's json representation

Coverage remained the same at 100.0% when pulling 7900d87f709f0e592eb2365a908792b4778136a9 on feature/sensor-virtual into 783ee54d024feb2bfd13a9b282f69738292d253f on master.

merging this...


Make changes to CI so Docker image is reused.

Did coverage drop because we are no longer running the lcov with the new flags?
It's probably more accurate, but let's figure out what happened.


Add support for Okta integration

As part of multiple IDPs support with APIcast, we should be able to integrate Okta.
Okta setup

Create a free trial account on https://www.okta.com/
Create an application and add redirect_uri
Create a user

Follow Okta documentation on how to create the above https://developer.okta.com/documentation/
For our integration, I have used implicit grant flow https://developer.okta.com/authentication-guide/implementing-authentication/implicit

Send /authorize call to Okta auth server. Example below.

https://example.okta.com/oauth2/v1/authorize?client_id=<your_client_id>&response_type=token&scope=openid&redirect_uri=http%3A%2F%2Flocalhost%3
A3000%2Fcallback&state=state-296bc9a0-a2a2-4a57-be1a-d0e2fd9bb601&nonce=foo'

The auth server redirects to login page. Enter the user credentials.

The access_token is redirected back to the redirect_uri of the application. Example below.

http://localhost:3000/callback#access_token=eyJraWQiOiJ3UmRQTG0xU294MnIySHNNbTBjcUExcTNyOHhjamNyTzFzaWVSbHBqNG9vIiwiYWxnIjoiUlMyNTYifQ.eyJ2ZXIiOjEsImp0aSI6IkFULnIxd293OVVjaE9xM2Y5OXlBb3o3SVdGTU9RZ2prUUNvOU45aXBkZUxadTAiLCJpc3MiOiJodHRwczovL3JlZGhhdC0zc2NhbGUub2t0YS5jb20iLCJhdWQiOiJodHRwczovL3JlZGhhdC0zc2NhbGUub2t0YS5jb20iLCJzdWIiOiIwb2ExdTZ2c3hlTzBkMktldTM1NiIsImlhdCI6MTUzNzkzODcwMiwiZXhwIjoxNTM3OTQyMzAyLCJjaWQiOiIwb2ExdTZ2c3hlTzBkMktldTM1NiIsInVpZCI6IjAwdTF1OHo0NDNCbFpsTHk4MzU2Iiwic2NwIjpbIm9wZW5pZCJdfQ.E9pgfJ8xrgEishu4nEprwtUBzOobB88OAwoGAB48pCGkJXyHYHIMRLFPdD6w86rCGiheUgYUnjAPXm2njjbCSOnKJi79HEl83ohiUzj0YUHMjrw-P_NwVtnnKEVwNw_z5K4_UYbaWDS-Y3vlhunVkOdkISaMH5ezLy1pbkh3sKdG7WKSpQulA9Y0YGzlUUzxsqM-UgCAECyGzQpEiUGRvvTSo4qY-micuQrV7cPHbyEaXJDaWE7QTdgAGrG9JBuv-hy2qbfF2eMfAK-N7mxy0Sh6lRMdU3GhTST1xwKMcbjK9Vn7aH3bljWIuVLnKEw2V21GQfqbTWpLOtJyRWWMqg&token_type=Bearer&expires_in=3600&scope=openid&state=state-296bc9a0-a2a2-4a57-be1a-d0e2fd9bb601

Important links

OpenID connect configurations for Okta
https://example.okta.com/.well-known/openid-configuration

JWKS_URI
https://example.okta.com/oauth2/v1/keys

We can reuse the existing module of Forgerock to verify the incoming access_token against the public_key.
Let me know if you need any more information.

Have you verified this does not work already?

@mikz The above flow is verified on my test account - yes. If you need access to that, let me know. It will save your setup time.
One thing I couldn't verify is verification of access_token since it requires conversion of jwks to pem and then pem to public_key. I hope it shouldn't be a problem since it's the standard openID provider medata https://openid.net/specs/openid-connect-discovery-1_0.html#ProviderMetadata,  and we are already doing that as part of forgerock.

@VinayBhalerao APIcast supports JWK, so it should work.
If it does not work, we can try to fix it, but right now I assume it works, because it should.

I can confirm that Okta integration works. I'll write a post as earlier on how to do the integration step by step.
The JWK work done is simply awesome <3. Making life much easier :)
Closing the issue.

can you advise service to service integration using 3scale and OKTA


Create new branch dunfell-l4t-r32.7.2

Describe the bug
I am getting the error message:
nvidia-container-toolkit-1.0.5-r0 do_fetch: Fetcher failure: Unable to find revision 60f165ad6901f85b0c3acbf7ce2c66cd759c4fb8 in branch master even from upstream
when trying to build the Meta tegra Dunfell branch.
I think it's because "branch=main" is missing at the end of
SRC_URI = "git://github.com/NVIDIA/nvidia-container-toolkit.git;protocol=https"
But I am not sure, can someone advise on how to solve the fetcher error?
I need this to work for L4T 32.7.2 on Dunfell branch, so I guess the best way is for me to create a new Dunfell branch dunfell-l4t-r32.7.2. with the necessary changes for nvidia-container-toolkit.
Steps to reproduce
add
  nvidia-docker \   nvidia-container-runtime \
to your IMAGE_BASE_INSTALL
Thanks!

That got fixed a couple of weeks ago.  Had you not updated your local copy of the repo recently?

Yes sorry, I just saw you made the changes to the main dunfell branch but I was on an ealier commit since I need this to work for L4T 32.7.2. Is it possible to create a new branch dunfell-l4t-r32.7.2 which only has L4T 32.7.2 versions? I think you can create a branch from commit c69687d2f4fae77e016f8646351d6eb345d9f7f4, then I can create a PR to update the SRC_URI link for nvidia-toolkit-container for that branch.
Thanks!

@ervgan you might just want to make your own branch for this.

Thanks for the quick reply! I created my own dunfell-l4t-r32.7.2 branch but it is actually more desirable to have it in the main repo, otherwise the whole build process will have one layer depend on my private repo and we would need to clone from two separate repos. I think maintaining it would not be a big problem since it will mostly be updating links, paths and hashes, what do you think?

You don't actually need to fork and maintain a private branch of the meta-tegra repo to fix this particular problem - just add a bbappend in your distro layer (or some layer you do control) to override the SRC_URI setting.   I've had to do this from time to time for other recipes, when I haven't been able to update layers in my projects.

I think maintaining it would not be a big problem since it will mostly be updating links, paths and hashes, what do you think?

There are several active branches already, and I just don't have the bandwidth to add yet another, unfortunately.

Oh you mean I can override just the SRC_URI variable from the nvidia-toolkit-container.bb file? This would indeed be a better solution I'm not sure how I would use bbappend, is there an example somewhere in the repo? This is what my .bb file looks like now:
CORE_IMAGE_BASE_INSTALL += " \   cuda-libraries \   cuda-samples \   cuda-toolkit \   tensorrt-core \   tensorrt-plugins \   tensorrt-samples \   tensorrt-trtexec \   cudnn \   libvisionworks \   gstreamer1.0-plugins-nvvideo4linux2 \   nvidia-docker \   nvidia-container-runtime \ "

@ervgan that might work but I think what Matt was suggesting was for you to add the .bbappend in your own layer.  See https://docs.yoctoproject.org/dev-manual/common-tasks.html?highlight=override+src_uri#understanding-and-creating-layers for an overview.  This way you don't need to add to BBFILES and you can track other layer customizations you'll ultimately need.


Add tests to HELM building mechanism

We may add next type of tests for HELM charts:

Profiles, that describe variants of HELM values file
Tests cases for each profiles, those validate each profile using asserts (e.g. services.edi.spec.image == '....')

Obsolete


Combined predict methods of NetMHC/NetMHCpan/NetMHCIIpan/NetMHCcons

Moved predict to BaseCommandlinePredictor. Aside from cleaning up fragile redundant code, some extra benefits:

multiple alleles now work for NetMHCIIpan (fixes https://github.com/hammerlab/topiary/issues/29)
multiple lengths now work for NetMHCIIpan (fixes problem we're having with Topiary)

Also fixed NetMHCIIpan tests and added a test for multiple alleles.

Looks solid!

Reviewed 7 of 7 files at r1.
Review status: all files reviewed at latest revision, 4 unresolved discussions, some commit checks failed.

mhctools/base_commandline_predictor.py, line 43 [r1] (raw file):
Can we avoid these defaults, since the flag are different for each predictor?

mhctools/base_commandline_predictor.py, line 55 [r1] (raw file):
Lowercase netMHCIIpan?

mhctools/base_commandline_predictor.py, line 91 [r1] (raw file):
Unnecessary newline

mhctools/file_formats.py, line 203 [r1] (raw file):
Don't actually like this variable name, as it looks like it should contain a list of the class2 columns. is_class2_columns or contains_class2_columns?

Comments from the review on Reviewable.io

Review status: 6 of 7 files reviewed at latest revision, 2 unresolved discussions, some commit checks pending.

mhctools/base_commandline_predictor.py, line 43 [r1] (raw file):
Sure

Comments from the review on Reviewable.io

Merge away

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks pending.

Comments from the review on Reviewable.io


tf.sparse: limited functionality

tf.sparse.sparse_dense_matmul only support matrix of rank 2, why cannot it support higher ranks?
tf.sparse.to_dense cannot support back-propagation at least in and before TF 1.13
there is no element-wise multiplication for sparse tensors similar to tf.multiply
there is no matmul between two sparse tensors

By the way, before deprecating old components, would you please complete the current functionality? It seems that TF is very good at deprecating widely used functions, but poor at improving its weak points

Sparse tensors have been added recently and they're under constant work. The deprecation needed for TF2.0 is orthogonal to this, thus unrelated. Let's try to keep things on topic, please.

See #6391
The issues about sparse tensors have been there for at least 2.5 years

My bad, I was wrong

use tf.reshape unroll the batch matix to rank 2 matrix, then use tf.sparse.sparse_dense_matmul


Add a note related to usage in Windows environment

I have a Windows 10 Enterprise 1909 version. I installed the latest VS Code, Rust and rust-analyzer plugins. I also have the latest rust gnu and msvc toolchains. However, I have an older VC++ compiler (installed with VS 2013).
After several installation/uninstallation attempts, I just couldn't get the rust-analyzer plugin working and I kept getting this - The Rust Language Service server crashed 5 times in the last 3 minutes. The server will not be restarted. I did not know where to look for the logs. So I downloaded the latest rust-analyzer-windows.exe from the releases and just double clicked to see if it ran. Unfortunately, I got this error - The application was unable to start correctly (0xc00007b) and this is why the rust-analyzer server was crashing.
A quick googling for 0xc00007b hinted at installing the latest MSVC++ Redistributable package. I had an older version of this installed from the VS 2013 installation.
After installing the latest redistributable package for VS 2015, 2017 and 2019 (both x86 and x64 versions to be on a safer side), the problem got resolved and rust-analyzer is now working fine.
This should help those who are trying to use rust-analyzer on Windows with an older/missing MSVC++ runtime and can't figure out what on earth is going on.

SHould go to this secion of manual: https://github.com/rust-analyzer/rust-analyzer/blob/master/docs/user/manual.adoc#rust-analyzer-language-server-binary


Task/sis import

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 30 (16.67%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-4.4%) to 87.243%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sis_provisioner/management/commands/import_file.py
0
6
0.0%

sis_provisioner/dao/handshake.py
3
12
25.0%

sis_provisioner/models.py
2
12
16.67%

Totals

Change from base Build<PHONE_NUMBER>:
-4.4%

Covered Lines:
424

Relevant Lines:
486

💛  - Coveralls


test(routes/admin/access): assign undefined instead of deleting property

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 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
479

Relevant Lines:
479

💛  - Coveralls


Fix #753: undefined local variable or method "name" in archive_repository.rb

Fix #753: undefined local variable or method "name" in archive_repository.rb

we have received the CLA, thank you!

Hi @cyrille-leclerc, thanks for PR. We squashed 2 commits and merged them into develop branch: https://github.com/cloudfoundry/bosh/commit/fb822dc87709b11acfc0cd54330b58cff5bafb6b


Mobile version can request the DarkSky data?

Hello,
thanks for your excellent article and code, I just start the study of the PWAs.
My first step was to run on a Linux PC browser: to make it work I needed to install the dotenv plugin for nodeJS, just to make possible the read of the .env file - otherwise the DARKSKY_API_KEY variable was always undefined.
In this way I can request the current forecast data.
Second step was the deploy on Firebase and the installation on my mobile.
This is working well but, on the mobile, I never receive the current forecast data, only fake data.
After I studied the architecture of the project and I discovered that the data request is done by the node server on the PC!
So, on the mobile solution, how the app can works ??
Thanks!

Unfortunately - due to the CORS requirements of the Dark Sky API, you need to use the node server to proxy the data.


Nextjs@13 not supported dynamic href in the /app directory

First, If you are considering this, I am sorry for registering the issue.
When I run the examples/app folder(nextjs version is 13.3.0), I get the following error.

Link: Next.js Error Occurred
If you go to the link provided above, next.js says that url mapping is no longer available for the app directory structure.
[example code]
<Link href={{ pathname: "/[store]", query: { store: "tate" } }}>
  Tate's Store
</Link>

I think we need to figure out a way to use it in the app directory of Next.js@13.
If there's anything I made a mistake, please give me some advice!

Yes, the nextjs team removed support for the UrlObject form for Link with the app directory. Are you using the pages directory, the app directory, or both in your project?

i'm using the app directory only.

Have you tried https://nextjs.org/blog/next-13-2#statically-typed-links? This library supports the pages directory, but nextjs now ships typescript support for the app directory

Have you tried https://nextjs.org/blog/next-13-2#statically-typed-links? This library supports the pages directory, but nextjs now ships typescript support for the app directory

Thank you for your help. The "typed Routes" function works very well.

Now supported in 2.2.2


Preprocessor Directives in Launch file

Why is it in the app.json file you have to define preprocessor directives symbole and not in the launch configuration it make no sens to me you have to modify the app manifest to define preprocessor directives

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

ID: a1d090c6-31b0-9d48-1c14-6d332babdc04
Version Independent ID: 84c9558d-397f-12b5-7547-984610c8a9c1
Content: Preprocessor Directives in AL - Business Central
Content Source: dev-itpro/developer/directives/devenv-directives-in-al.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Hello, thanks for your feedback. The short answer to your question is that this is based on how the compiler works.

Closing this issue.


When create ambassador-cacert, here will throw exception.

https://github.com/datawire/ambassador/blob/32eec960f2da5f9abea1ff6a14985421020c5525/ambassador/kubewatch.py#L305
None type will throw exception for key

Hitting the same error trying to do client auth. Please fix this.


Docs Team's Company Logo Design

Logo design for the Docs Team's for the company "Ubn-Jr".
The size of the logo should be 1080x1080.
Color codes.

#FFFFFF
#D0D0D0
#A1A1A1
#585858
#353535
#090909


Queues for adding templates

Modifies the insertion of the templates to use the queue system. This will help to avoid the problem in which "floating" samples are added and cannot be automatically removed.

Looks great, only thing is: Do you think we could add a test to ensure the problem we were having doesn't come up again? Seems like a tricky test to design but would be nice to have it.

I will try to add a test, although as you said, is going to be tricky...

:+1: when tests pass.

Does somebody knows a specific case where this issue was showing up? I'm having problems on defining that situation.

Changes Unknown when pulling e581a6ff45d90d8d137c4dba6725b4c0a2ed0f65 on josenavas:queues-for-adding-templates into ** on biocore:master**.

I do not, sorry -- I think the queues should eliminate the possibility. Is deletion of md templates also taken care of via queue? If creation and deletion are both atomic operations (via queue), then the problematic situation should not be possible.

Changes Unknown when pulling e581a6ff45d90d8d137c4dba6725b4c0a2ed0f65 on josenavas:queues-for-adding-templates into ** on biocore:master**.

Yeah, I think you may be right. Do you agree @josenavas or did you find
a way? :O
On (Feb-18-15|13:17), adamrp wrote:

I do not, sorry -- I think the queues should eliminate the possibility. Is deletion of md templates also taken care of via queue? If creation and deletion are both atomic operations (via queue), then the problematic situation should not be possible.

Reply to this email directly or view it on GitHub:
https://github.com/biocore/qiita/pull/866#issuecomment-74949621

@antgonza helped me to find a way. Tests are added now

BTW, I agree that it should not happen, but better to be sure with tests!

Changes Unknown when pulling 839d819db9bc65f224d12f18868169b73a1ec8fc on josenavas:queues-for-adding-templates into ** on biocore:master**.

This should be ready to go!

:+1:

@ElDeveloper can you take a quick look and merge if you think it is correct?

Looks good to me!

Thanks for figuring out a way to test this!


I get a missing dependence error

error: bundling failed: Error: Unable to resolve module @react-native-community/viewpager from src\components\auth\GetStarted.js: @react-native-community/viewpager could not be found within the project.

Hi,
I tried the "Example" project now.
I deleted the folder "node_modules" and "yarn.lock" then I reinstalled everything with the command "yarn install".
I tried only the App.js file of the project but it works correctly, can you give more information about what doesn't work for you? Or when the error occurs

The error occurs when i use AppIntro,
But will try what you just did I deleted the folder "node_modules" and "yarn.lock" then I reinstalled everything with the command "yarn install". and see if it goes well.
Will provide feedback
Thanks

I still got the same error screen

I probably understood, with the new versions of React Native that component "viewpager" is deprecated.
I think I should update my fork that I'm using for this project:
"react-native-swiper": "git+https://github.com/Richi2293/react-native-swiper.git"
I'm using a customized version of "react-native-swiper" because I had made changes to it.
I think that for React native version like in the Example project it works well, otherwise it would have given me error too.

Yeah,
An update will fix that issue.
Thanks, Alot

Hi,
I made an update, please try again with version 1.2.5.
I did tests, and for me the Example project works correctly on both Android and iOS.
Let me know, thanks

Great will try it out...
Thanks


Latest VUnit master breaks netlist builds

Description
Due to changes in VUnit internals, netlist builds are broken as of https://github.com/VUnit/vunit/pull/1025, which likely will be included in VUnit v4.8.0.
Traceback (most recent call last):
  File "/root/git/vunit-vc/build.py", line 38, in <module>
    main()
  File "/root/git/vunit-vc/build.py", line 29, in main
    setup_and_run(
  File "/root/git/vunit-vc/tsfpga/tsfpga/examples/build_fpga_utils.py", line 218, in setup_and_run
    build_ok = projects.build(
  File "/root/git/vunit-vc/tsfpga/tsfpga/build_project_list.py", line 218, in build
    return self._run_build_wrappers(
  File "/root/git/vunit-vc/tsfpga/tsfpga/build_project_list.py", line 311, in _run_build_wrappers
    report.print_str()
  File "/root/git/vunit-vc/vunit/vunit/test/report.py", line 164, in print_str
    test_result.print_status(self._printer, padding=max_len, max_time=max_time)
TypeError: BuildResult.print_status() got an unexpected keyword argument 'max_time'

The fix seems easy enough though, maybe add **kwargs or add specific max_time argument.
Not required until VUnit v.4.8.0 is released though.

For future reference:
https://github.com/VUnit/vunit/pull/1025 is not included in the latest VUnit release as of this date, which is 4.7.0.  It is however merged to the VUnit master branch, so it will be included in VUnit > 4.7.0.
To solve the situation I have:

Made a tsfpga release from main with VUnit pegged to <= 4.7.0.
Updated current tsfpga main to handle the API on VUnit master.
Pegged VUnit to > 4.7.0 (which is currently not released) on tsfpga main.

Bullet 3 means that we can not make a new tsfpga release before VUnit makes their next release.

Fixed on main now!

I temporarily reverted the fix in order to make an emergency PyPI release. The fix is back on main now, so that our main works with lates VUnit master.


Hyrax 2.5 and solr_wrapped update

Updates:

update hyrax dependency to 2.5
set solr version to 7.7.1 in .solr_wrapper

Looks good, will merge this.


Sometime On question creation question title don't appear

Exemple

Work Around
Refresh the page

Completely new version of add-on since February, so I'm closing.


Use more clusters in Kmeans benchmarks

Increased number of clusters in Kmeans benchmarks. This will help gain insight into the performance of K-means with higher number of clusters, which will be helpful for evaluating future changes to Kmeans.

Codecov Report

Merging #106 (fcff9a0) into master (6c16c7c) will increase coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #106      +/-   ##
==========================================
+ Coverage   56.16%   56.23%   +0.07%
==========================================
  Files          66       66
  Lines        5537     5537
==========================================
+ Hits         3110     3114       +4
+ Misses       2427     2423       -4

Impacted Files
Coverage Δ

algorithms/linfa-hierarchical/src/lib.rs
51.42% <0.00%> (-0.75%)
:arrow_down:

algorithms/linfa-kernel/src/lib.rs
71.25% <0.00%> (ø)

algorithms/linfa-pls/src/pls_generic.rs
68.92% <0.00%> (ø)

algorithms/linfa-svm/src/classification.rs
85.18% <0.00%> (ø)

...rithms/linfa-trees/src/decision_trees/algorithm.rs
51.94% <0.00%> (+1.41%)
:arrow_up:

algorithms/linfa-pls/src/utils.rs
86.84% <0.00%> (+2.22%)
: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 6c16c7c...fcff9a0. Read the comment docs.

Do you have the PR in draft mode intentionally?

Yes. I'm planning to change the benchmarks to use Normal distribution instead of Uniform

Just realized that the benchmarks actually use Normal distributions to generate blobs, so this is ready to merge as-is


Add MacOS 10 wheel and improve README

Yeah, they're listed in the Dependencies section. It's just cmake, C++14 compiler, and python.


Support passwordless pihole installations

What feature would you like added?
Would you consider supporting pihole installations that have no password set?
My pihole is running inside a docker container and outside access to its web UI is handled via a Traefik proxy which also handles authorization (i.e. there is no unauthorized access). To avoid unnecessary "double-authentication" I disabled the password in pihole itself.
This would also mean that orbital-sync running inside docker could directly access the web UI without a password (bypassing Traefik).
The secondary pihole(s) would then be running on another host, and would be using passwords (arbitrarily complex ones, since a real user would never need to enter them).
In theory one could also run the secondary ones passwordless and just never expose their web UI to a (non-docker-internal) network at all, in that case the primary one should be password protected. That's not my usecase, but that could also be enabled via this feature.
Unfortunately I didn't find a way to configure this and a quick peek at the code suggests that it's not intended (I did a few minor experiments to see if I could hack this together myself, but didn't get anywhere).
Version
No response
Log output
No response

This just worked fine for me with PiHole 1 and PiHole 2 both running password less.  Just configure like this.
PRIMARY_HOST_PASSWORD: ''

I've implemented this using
PRIMARY_HOST_PASSWORD: ''
If this isn't intended, let me know, otherwise, it's working so far!

Sorry, I can confirm that this works as mentioned above. I don't know why this didn't work for me previously. I probably mis-configured something else when I tried that.


Question

Hi !
I saw you implementation and you are executing the OCR to entire table. I was wondering if it is better or worse than doing OCR one by one for each cell. I am interested in your reasoning.

Hey!
The OCR processing is delegated to Tesseract OCR. And since Tesseract OCR was mainly developed for document OCR, and not simply word by word OCR, it performs better when applied to a whole document or table in this case.
You can run Tesseract in different modes (page segmentation modes) though. PSM 4 and 6 gave me the most accurate results.
Although Tesseract OCR is a good choice for OCR, I would recommend trying out PaddleOCR. It has several advantages, such as: multi lingual support, layout analysis, table recognition, etc.

Thanks for your reply! I see, I will try the PaddleOCR!

You're welcome :)


using IpAddressFamily based on specified server address

Had an issue with this library in a private network which does not support IPv6. For some reason the address family of a socket object always defaulted to InterNetworkV6. When I set the address family explicitly to InterNetwork everything worked as expected. I think this change makes the implementation more robust.

probably need to account for cases where not an IP address but a domain name is specified, right?

Hi, thank you for your contribution. I believe you are just a couple of days too late 😄 A similar approach (which also supports non-IP-Adresses is added in the latest develop branch. You can now pass the address family in the TCP options.
Please let me know if this approach also fits your needs.
Best regards
Christian

Thanks Christian, that sounds great!
When will it be available as Nuget package?
Best Regards
Vitaliy

I expect this weekend.


unable to prefix route with period

The following route cannot be found
Route::group(['prefix' => '1.11', 'middleware' => 'web'], function () {
	Route::get('/verify', array('uses' => 'ProxyController@getVerification', 'as' => 'verify'));
});

However, this route will work
Route::group(['prefix' => '1-11', 'middleware' => 'web'], function () {
	Route::get('/verify', array('uses' => 'ProxyController@getVerification', 'as' => 'verify'));
});

The difference is in the route prefix. Cannot use a period.

OK, I now realise this is a duplicate of https://github.com/laravel/framework/issues/6180


some bindsym exec commands do not work as expected

Hello.
I was trying to specify colors for dmenu in bindsym and:
bindsym $mod+a exec dmenu_run -b -i -nb "#FDF6E3" -nf "#657B83" -sb "#EEE8D5" -sf "#CB4B16"

does not work in sway
works in i3

bindsym $mod+d exec dmenu_run -b -i -nb '#FDF6E3' -nf '#657B83' -sb '#EEE8D5' -sf '#CB4B16'

does not work in sway
works in i3

bindsym $mod+g exec dmenu_run -b -i -nb "'#FDF6E3'" -nf "'#657B83'" -sb "'#EEE8D5'" -sf "'#CB4B16'"

works in sway
does not work in i3

Note the following work from a terminal:

dmenu_run -b -i -nb #FDF6E3 -nf #657B83 -sb #EEE8D5 -sf #CB4B16
dmenu_run -b -i -nb '#FDF6E3' -nf '#657B83' -sb '#EEE8D5' -sf '#CB4B16'
dmenu_run -b -i -nb "#FDF6E3" -nf "#657B83" -sb "#EEE8D5" -sf "#CB4B16"

sway -d -V output http://pastebin.com/raw/5F1hruns
Thanks.

Commit 99f26c6 breaks the following config:
bindsym $mod+x exec "dmenu_run -i -nb '#181818' -nf '#d8d8d8' -sb '#181818' -sf '#a1b56c'"

This works in i3, but doesn't in sway.
I'm missing why strip_quotes strips all quotes and not just starting and trailing quotes?

This has been discussed a bit here: https://github.com/SirCmpwn/sway/issues/521
Basically in any sane implementation anything wrapped in "" means a single argument, if you start to make special cases like i3 does, it gets complicated and confusing. We like to avoid that if possible.

To clarify: this is a place where I am willing to break backwards compatability in exchange for saner behaviour. The correct command is:
bindsym $mod+x exec dmenu_run -i -nb '#181818' -nf '#d8d8d8' -sb '#181818' -sf '#a1b56c'


Preference-Variables not working...

It just keep show me Unable to run “Fmt”.
I have add those variables to the settings. And add those to the system env file then run mate some.go, still not working.
macOS sierra
textmate2

You are getting an error about not finding the gofmt command? If it is in any of the following locations it shouldn't even be necessary to setup the variable:

/opt/local/bin/gofmt
/usr/local/bin/gofmt
/usr/local/go/bin/gofmt

Please give an exact list of steps performed and the result expected vs what happens.

Hi, @infininight here is my setting:
https://drive.google.com/open?id=0Bz0MK78BvgY1ZnFuT2hzVFhNRXc
and the error
https://drive.google.com/open?id=0Bz0MK78BvgY1enpxaGVHbEhLSUE
and the env
https://drive.google.com/open?id=0Bz0MK78BvgY1NS1PbXZxbGlORWc
and Bundles-Go-run
https://drive.google.com/open?id=0Bz0MK78BvgY1SFdNclhjX2tnSHM
Thanks!

OK, based on the final image it seems that go is being run successfully by TextMate. The error then would be not with defining the path inside TextMate but some setting requirement internal to go. It would seem it is looking for GOROOT in the wrong location, but would need someone familiar with go to answer that. I would have expected go to set GOROOT itself but you could try setting it yourself in settings?
It would be weird though as I have go installed through Homebrew and it works fine without setting any settings at all. (Since it's in a location that TextMate expects by default.)

It seems like the editor didn't read the setting at all.But when set TM_GOFMT to goimports, it works.
Really weird.
One thing I'm sure about is that GOPATH and GOROOT  is correct.

TextMate does not source any bash/etc startup files, so the only variables that are set are the ones in preferences. So GOROOT would be the default.

TextMate does not inherit the users environment unless it is launched from the command line.
@infininight it can read env.I tried and failed.

I think you may be experiencing 2 separate issues.
First is that TextMate can't find gofmt. You mentioned that it works when you set TM_GOFMT to point to goimports. Can you confirm that gofmt exists at ~/app/go/bin/gofmt? Does it work if you manually call it on a go file? Do you have issues with running lint?
If TextMate is properly calling goimports when you set the variable, it should be able to properly call gofmt. The gofmt command in TextMate is powered by a simple bash expression: (eval "${TM_GOFMT:-gofmt}"). If TM_GOFMT is set, call that, otherwise call gofmt generically. gofmt should work independently of having GOROOT set, but I may be wrong.
Second is finding GOROOT. How did you install go? The go utility embeds GOROOT internally because it is just a frontend. It has to find the correct compiler within GOROOT to actually do work. If the embedded GOROOT doesn't match the actual location of GOROOT, then you have to set the environment variable. We don't handle TM_GOROOT specifically, but you should be able to make it work by setting plain GOROOT inside of TextMate.
If none of that works, can you post the output of go env?

@syscrusher Thanks!
~/app/go/bin/gofmt this path is correct and it can work.
Second, after I set GOROOT inside TextMate, everything works fine now.
Thanks for your work.


JBEHAVE-1411: add mechanism to manipulate passed parameter value

@maurotalevi please, review!

@maurotalevi  please, take a look! Thanks in advance

Pulled in 4.7.


Working with translated elements.

Hi @ypnos-web,
I have a simple question regarding text search in the datatable.
I have a datatable listing species, species have a translatable name and a latin name.
Let's say there's a species named "Long-beaked common dolphin" which latin name is "Delphinus capensis".
When i'm text searching dolphin notthing is found.
Should I make joins to find the translations in my method returning results or is there something included in the plugin ?
for now this is what i'm using in the controller method :
$this->DataTables->find('Species')->all()
Thank you again,
Stefan.

How is the translation implemented? Is it in a different row of the table or in a secondary table / via a behavior?
What is your datatables configuration (columns)?

Translations are implemented with the i18n table, as specified in the cookbook.
I have a field "name" in the species table and all is made with the translateTrait.
I'm going to check the solutions you pointed.
Thank you.

How did you end up solving the issue?

HI @ypnos-web ,
really sorry for not getting back at you with my solution yet, I'm late with the project including your plugin but as soon as i've finished i'll publish my works here to let you know.
For now it's just adding the joins and including conditions but it might change for something better if I can work it more.
Stefan.

Hi @ypnos-web ,
Here's what i've set up to work with translations.
Translations are all set up in one tablme for all objects as described in the cookbook.
Te example features a simple model "Zones" which has its name translated :
Column definition :
[ 			'title' => __('Name'), 			'name' => 'Zones_name_translation.content', 			'data' => '_translations.' . $current_locale . '.name', 			'width' => '15%', 		],
$current_locale is a variable containing the locale currently used by the App ( in my case it can be en or fr ).
Controller method :
`public function datatable(){
$data = $this->DataTables->find('Zones' , 'translations' , [
'contain' => [
'Structures'
]
])->group([
'zones.id',
'zones.name',
'zones.modified',
'zones.deleted',
'zones_name_translation.id',
'zones_name_translation.content',
'zones_name_translation.locale',
'zones_name_translation.model',
'zones_name_translation.foreign_key',
'zones_name_translation.field',
]);
	$this->set([
		'data' => $data,
		'_serialize' => array_merge($this->viewVars['_serialize'], ['data'])
	]);
}`

As your plugin executes a a finder it's easy to bind a translation finder which works directly without more coding than required.
Due to Postgresql restrictions on count function i had to specify each columns of the query in a group by clause , it can be really a pain if you link multiple models ( example a bit below ).
Here's an example featuring a model (Species) associated with more than one model :
$data = $this->DataTables->find('Species' , 'translations' , [ 			'contain' => [ 				'Groupes', 				'Caracteristics', 			], 			'conditions' => $DTT_conditions, 		])->group([ 			'species.id' , 			'species.latin_name', 			'species.name' , 			'species.description' , 			'species.modified' , 			'species.deleted' , 			'species.default_image' , 			'species.groupe_id', 			'species.caracteristic_id', 			'species_name_translation.id' , 			'species_name_translation.content' , 			'species_name_translation.locale', 			'species_name_translation.model', 			'species_name_translation.foreign_key', 			'species_name_translation.field', 			'species_description_translation.id', 			'species_description_translation.content', 			'species_description_translation.locale', 			'species_description_translation.model', 			'species_description_translation.foreign_key', 			'species_description_translation.field', 			'groupes.id', 			'groupes.nom', 			'groupes.deleted', 			'groupes.modified', 			'groupes.caracteristic', 			'groupes.picto', 			'groupes.youngsters', 			'groupes.exact_counts', 			'groupes_nom_translation.id', 			'groupes_nom_translation.content', 			'groupes_nom_translation.locale', 			'groupes_nom_translation.model', 			'groupes_nom_translation.foreign_key', 			'groupes_nom_translation.field', 			'groupes_caracteristic_translation.id', 			'groupes_caracteristic_translation.locale', 			'groupes_caracteristic_translation.content', 			'groupes_caracteristic_translation.model', 			'groupes_caracteristic_translation.foreign_key', 			'groupes_caracteristic_translation.field', 			'caracteristics.id', 			'caracteristics.groupe_id', 			'caracteristics.name', 			'caracteristics.modified', 			'caracteristics.deleted', 			'caracteristics_name_translation.id', 			'caracteristics_name_translation.locale', 			'caracteristics_name_translation.content', 			'caracteristics_name_translation.model', 			'caracteristics_name_translation.foreign_key', 			'caracteristics_name_translation.field', 		]);
Again i had to bind a group by due to postgresql but it would be nice to know if anyone encountered this problem on a mysql DB.
Hope it helps.
Stefan.

The code preview is a bit buggy or do i need to format it a particular way ? :(

Thanks for the insights. Maybe we could add this to the wiki as an example/howto. I didn't get it though, what do you need a count for? Is it used in the translations finder?
I formatted your comment: For multiline code, use this markup:

´´´php

´´´

Thanks for the insight on the multiline code.
Regarding the count i was just supposing the results number shown in the datatable were coming from a count but i might be wrong.
Feel free to add these examples to the wiki :)
In fact when working with translations you just have to make sure you use the correct alias to display/filter the datatable.

Yes, the DataTables helper uses count() on the Query (twice when filtering). With MySQL this leads to a simple COUNT(*) statement and nothing needs to be added to the query to make it work.
See this: http://book.cakephp.org/3.0/en/orm/query-builder.html#query-count

Well this looks diferrent for Postgresql, a count with joins requires all fields to be in a group by clause.
I'm not used to Postgre, i have to work with it for a project.

I've just checked a bit stackoverflow and the postgresql documentation and as count is an agreggate function it requires a group by clause if there's other fields in the query.
this example work :
select count(*) from my_schema.my_table
this one is not working :
select column_a, count(*) from my_schema.my_table;
but this one will :
SELECT column_a, COUNT(*) FROM my_schema.my_table GROUP BY column_a;
Might be great to also put this in the wiki for postgre users.
Stefan.

This sounds very much like how I learned SQL at university and is not bound to postgresql. I would assume it is the same for MySQL. Note however that this should not interfere with the use of $query->count(); (as done by this plugin).
See, here are examples of SQL that result by that use of count:
SELECT
  (
    COUNT(*)
  ) AS `count`
FROM
  materials Materials
  LEFT JOIN material_datas OrderingProperty4 ON (
    OrderingProperty4.material_id = Materials.id
    AND OrderingProperty4.materialtypefield_id = '4270'
  )
  LEFT JOIN material_datas OrderingProperty5 ON (
    OrderingProperty5.material_id = Materials.id
    AND OrderingProperty5.materialtypefield_id = '5507'
  )
  LEFT JOIN materialtypes Materialtypes ON Materialtypes.id = (Materials.materialtype_id)
  LEFT JOIN units Units ON Units.id = (Materialtypes.unit_id)
WHERE
  (
    NOT (
      Materials.amount_left = 0
      AND Materials.amount > 0
    )
    AND Materials.status = 'active'
    AND materialtype_id = 1007
  )

and (filtered):
SELECT
  (
    COUNT(*)
  ) AS `count`
FROM
  materials Materials
WHERE
  (
    NOT (
      Materials.amount_left = 0
      AND Materials.amount > 0
    )
    AND Materials.status = 'active'
    AND materialtype_id = 1007
    AND Materials.id in (
      9777, 9778, 9779, 9780, 9781, 9782, 9783,
      9784, 9785, 9786, 9787, 9788, 9789,
      10596, 10597, 16535, 16670, 16736,
      17016, 17073, 17356, 17525, 17070,
      17075
    )
  )

Note that I have some custom stuff going on there, but the message is: It is a simple SELECT COUNT(*) FROM query.
So you might need to check again why you need the grouping in your case as it should not be needed for the simple record counting.

I think i've found what's going on with the group by, it looks like it happens with the limit method.
these two examples can give a bit more details :
This would work :
´´´php
$species = $this->Species->find('translations',[
'contain' => [
'Caracteristics'  => [
'finder' => 'translations',
],
'Groupes' => [
'finder' => 'translations',
]
],
'order' => 'Species_name_translation.content ASC'
])
´´´
But this is not working and requires grouping :
´´´php
$species = $this->Species->find('translations',[
'contain' => [
'Caracteristics'  => [
'finder' => 'translations',
],
'Groupes' => [
'finder' => 'translations',
]
],
'order' => 'Species_name_translation.content ASC'
])->limit(10)
´´´
I've just tested this in a REST call.

That's a very good find!
Could you use Debug console or debug($query) to obtain and post the SQL querys generated by these two runs? I wonder if Cake is doing something special here.


Removed Face::id field

The field was attracting bugs and missuses (for example, comparison with ids) and moreover useless (in the meaning that it can be safely removed from the code with some fixes applied so that we neither lose performance nor code readability), so I removed it
(PR also contains some other minor improvements)

Sorry, I have a small mess up with branches, I will open a new PR with the same content later


What is OAuth authentication_token used for?

At OneDrive authentication and sign-in page, in Code flow section.
"Step 2. Redeem the code for access tokens" returns response that contains "authentication_token" along with other data like access and refresh token.
I might be asking silly question here but as I am not aware yet, I would like to know - what is "authentication_token" used for?
Context: access_token is used for authenticated calls to API and refresh_token used for fetching new access_token when the current one expires. authentication_token is used for .... ?
Thanks

It's a legacy token that the Microsoft Account OAuth flow returns that isn't that useful for the way apps use OAuth. I'd recommend just ignoring it.

@rgregg thanks for the information.
Does that probably mean that "authentication_token" has nothing to do with anything that is offered as part of OneDrive API Version 2.0 with OAuth 2.0 (on web, application etc.)?
If that is not the case, I would request you to provide me at least one use case where "authentication_token" is part of or used somehow.

Yes, that's correct. authentication_token has nothing to do with anything we offer in the OneDrive API.

Thanks again, that resolves my query.

Is it fine if I can send a PR that adds a note regarding this clearification on "msa_oauth" page. It would be helpful for anyone who keeps wondering like me

Sure.


"should" issue with chai

there's the "should" issue with chai
http://chaijs.com/guide/styles/

@zeethree50 if you want to extend global Object interface, do not place Object interface in chai module.

I agree with @vvakame. That's what I've done with #4155.

Can you please fix the failing Travis CI tests and update the PR!

Hello and thank you for your contribution!
Due to an excessively long queue of pull requests that have become stale, we are "declaring bankruptcy" and closing all PRs opened before May 1, 2016. If you'd still like to merge this code in, please open a new PR that has been merged and rebased with the master branch.
Going forward, we are committing to review or merge all PRs on a regular basis so this bankruptcy will not occur again. We apologize for the incovenience and hope you will continue to contribute to DefinitelyTyped in the future.
Thanks
The TypeScript and DefinitelyTyped teams


Error in "domain/block.py" "invalid syntax"

Executing
python export_blocks_and_transactions.py --start-block 0 --end-block 500000 --ipc-path ~/.ethereum/geth.ipc --blocks-output blocks.csv --transactions-output transactions.csv
unfortunately results in an error
File "export_blocks_and_transactions.py", line 5, in
from ethereumetl.jobs.export_blocks_job import ExportBlocksJob
File "/home/vn/ethereum-etl/ethereumetl/jobs/export_blocks_job.py", line 7, in
from ethereumetl.mappers.block_mapper import EthBlockMapper
File "/home/vn/ethereum-etl/ethereumetl/mappers/block_mapper.py", line 1, in
from ethereumetl.domain.block import EthBlock
File "/home/vn/ethereum-etl/ethereumetl/domain/block.py", line 8
self.number: int = None
^

Hi @Victornovikov. What is the version of python that you use? Also please include the full error log.

thanks a lot for a prompt response.
python --version
3.5.2
it is pretty much the full log.. just in case pasting everything from my terminal window
(eth) vn@eth-node:~/ethereum-etl$ python export_blocks_and_transactions.py --start-block 0 --end-block 500000 --ipc-path ~/.ethereum/geth.ipc --blocks-output blocks.csv --transactions-output transactions.csv
Traceback (most recent call last):
File "export_blocks_and_transactions.py", line 5, in
from ethereumetl.jobs.export_blocks_job import ExportBlocksJob
File "/home/vn/ethereum-etl/ethereumetl/jobs/export_blocks_job.py", line 7, in
from ethereumetl.mappers.block_mapper import EthBlockMapper
File "/home/vn/ethereum-etl/ethereumetl/mappers/block_mapper.py", line 1, in
from ethereumetl.domain.block import EthBlock
File "/home/vn/ethereum-etl/ethereumetl/domain/block.py", line 8
self.number: int = None
^
SyntaxError: invalid syntax

Try upgrading to Python 3.6. Type annotations are not supported in python 3.5. I'll add support for python 3.5 later.

Yep, all works now. Thanks!
Should have paid more attention to README.


sc2: stalwart war council upgrade

Adding the Stalwart war council upgrade -- duality charge. Nerfed unit has -4 range (6 -> 2), item adds the +4 range back.
The name is a pun on being twice as good as singularity charge. I placed the passive icon on the bottom row to match the style of immortals.
Progress on #152
Pairs with client PR #266
Without upgrade

With upgrade

(forgot to push the trigger commit, but it's unchanged from when I tested it)


Collaborative mode cannot sync picture

self-host excalidraw and excalidraw-room

If I drag this picture(same file with left) to  the right one. then two pictures would show

self-host excalidraw and excalidraw-room

I notice there is an error
https://firebasestorage.googleapis.com/v0/b/excalidraw-oss-dev.apps34pot.com/o/files%2Frooms%2F7a969aa9b801ba8a6d0f%2Fa95a13b9ba7c9ecf3fb5497a6075557f28e35397?alt=media

I notice there is an error
https://firebasestorage.googleapis.com/v0/b/excalidraw-oss-dev.apps34pot.com/o/files%2Frooms%2F7a969aa9b801ba8a6d0f%2Fa95a13b9ba7c9ecf3fb5497a6075557f28e35397?alt=media

excalidraw/src/excalidraw-app/data/firebase.ts
export const loadFilesFromFirebase = async (
  prefix: string,
  decryptionKey: string,
  filesIds: readonly FileId[],
) => {
  const loadedFiles: BinaryFileData[] = [];
  const erroredFiles = new Map<FileId, true>();

  await Promise.all(
    [...new Set(filesIds)].map(async (id) => {
      try {
        const url = `https://firebasestorage.googleapis.com/v0/b/${
          FIREBASE_CONFIG.storageBucket
        }/o/${encodeURIComponent(prefix.replace(/^\//, ""))}%2F${id}`;
        const response = await fetch(`${url}?alt=media`);
        if (response.status < 400) {
          const arrayBuffer = await response.arrayBuffer();

          const { data, metadata } = await decompressData<BinaryFileMetadata>(
            new Uint8Array(arrayBuffer),
            {
              decryptionKey,
            },
          );

          const dataURL = new TextDecoder().decode(data) as DataURL;

          loadedFiles.push({
            mimeType: metadata.mimeType || MIME_TYPES.binary,
            id,
            dataURL,
            created: metadata?.created || Date.now(),
            lastRetrieved: metadata?.created || Date.now(),
          });
        } else {
          erroredFiles.set(id, true);
        }
      } catch (error: any) {
        erroredFiles.set(id, true);
        console.error(error);
      }
    }),
  );

  return { loadedFiles, erroredFiles };
};

Hi. In collaboration we sync/save images using a Firebase Storage server, so that needs to be set up. We should support different providers (including your own) or other solutions as well, though.

Hi. In collaboration we sync/save images using a Firebase Storage server, so that needs to be set up. We should support different providers (including your own) or other solutions as well, though.

Yeah, I noticed that. So modify code(https://github.com/TaoChan1005/excalidraw) and use docker(kiliandeca/excalidraw-storage-backend) as my backend.


Publish 7.4-20220129031141+0000

Publish 7.4-20220129031141+0000

@bot-gradle test and merge

OK, I've already triggered a build for you.


elixir: 1.11.4 -> 1.12.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/elixir-lang/elixir/releases.
meta.description for elixir is: "A functional, meta-programming aware language built on top of the Erlang VM"
meta.homepage for elixir is: "https://elixir-lang.org/"
meta.changelog for elixir is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/cqarx00c22i45fj1y1092pnghp84gipx-elixir-1.12.0/bin/elixir passed the binary check.

/nix/store/cqarx00c22i45fj1y1092pnghp84gipx-elixir-1.12.0/bin/elixirc passed the binary check.

/nix/store/cqarx00c22i45fj1y1092pnghp84gipx-elixir-1.12.0/bin/iex passed the binary check.

Warning: no invocation of /nix/store/cqarx00c22i45fj1y1092pnghp84gipx-elixir-1.12.0/bin/mix had a zero exit code or showed the expected version

/nix/store/cqarx00c22i45fj1y1092pnghp84gipx-elixir-1.12.0/bin/.elixir-wrapped passed the binary check.

/nix/store/cqarx00c22i45fj1y1092pnghp84gipx-elixir-1.12.0/bin/.elixirc-wrapped passed the binary check.

/nix/store/cqarx00c22i45fj1y1092pnghp84gipx-elixir-1.12.0/bin/.iex-wrapped passed the binary check.

3 of 7 passed binary check by having a zero exit code.

6 of 7 passed binary check by having the new version present in output.

found 1.12.0 with grep in /nix/store/cqarx00c22i45fj1y1092pnghp84gipx-elixir-1.12.0

found 1.12.0 in filename of file in /nix/store/cqarx00c22i45fj1y1092pnghp84gipx-elixir-1.12.0

directory tree listing: https://gist.github.com/516c765930e5ffd34f6232bbfe277aa7

du listing: https://gist.github.com/8fb1bb448116de18f7115fdffc145683

Rebuild report (if merged into master) (click to expand)

13 total rebuild path(s)

4 package rebuild(s)

4 x86_64-linux rebuild(s)
3 i686-linux rebuild(s)
3 x86_64-darwin rebuild(s)
3 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
elixir
elixir_1_11
elixir_ls
mix2nix

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cqarx00c22i45fj1y1092pnghp84gipx-elixir-1.12.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A elixir https://github.com/r-ryantm/nixpkgs/archive/3c9aae48675161012c2c3cbff3458c6b83a2ed39.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cqarx00c22i45fj1y1092pnghp84gipx-elixir-1.12.0
ls -la /nix/store/cqarx00c22i45fj1y1092pnghp84gipx-elixir-1.12.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

elixir
elixir_ls
mix2nix

Maintainer pings
cc @ankhers @Br1ght0ne @DianaOlympos @gleber @happysalada @yurrriq for testing.

Result of nixpkgs-review pr 124115 at 3c9aae48 run on x86_64-linux 1

3 packages built successfully:

elixir (elixir_1_11)
elixir_ls
mix2nix

1 suggestion:

warning: name-and-version
Did you mean to pass pname instead of name to mkDerivation?
Near pkgs/development/interpreters/elixir/generic-builder.nix:19:5:
   |
19 |     name = "${baseName}-${version}";
   |     ^

Near pkgs/development/interpreters/elixir/generic-builder.nix:21:12:
   |
21 |     inherit src version;
   |            ^

There is another PR pending. A little more patience little bot.


only trigger ci on master branch and pull requests

Description of changes
Things done

[ ] Tested the changes in your own NixOS Configuration
[ ] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

@mergify queue


Please publish your package bbfc:scss for Windows

Hi!
This package has binary dependencies, and we wanted to give you a heads up that Meteor 1.1, with support for Windows will be released in a few days.
Now is the time to publish your package for Windows!
It should be very straightforward -- we've added new build machines that run Windows that you can access with the meteor admin get-machine command. This is the same as you've done before for the other architectures we support.
For complete directions on how to publish your package for Windows, visit https://github.com/meteor/meteor/wiki/Publishing-a-package-with-binary-dependencies-for-the-Windows-preview
On behalf of the Meteor team,
@stubailo, @Slava and @avital.
P.S. This is an automated message based on our database of package authors.

Hah there's only so much I can do, this is a fork of the package I maintain so I can't really be much help. Won't fix :wink:


[Feature Suggestion] Stabilize execution environment by fixing PyTorch, CUDA, TensorRT, and ONNX versions

First, I installed the environment using requirements.txt according to the instructions in the README, and then ran a test inference. However, I encountered a segmentation fault in Conv2D caused by a combination problem of PyTorch v2.3.0 + CUDA 11.8.

This problem has been reported in a PyTorch issue.

Segmentation fault (core dumped) when using pytorch Conv layers #126963

Since we often find that users spend a great deal of time in their environments trying to maintain the YOLOv9-MIT runtime environment due to the lack of fixed versions of PyTorch and other modules, we thought it would be a good idea to fix the runtime environment with a Docker container.
I understand that this repository is in the process of being improved quite frequently, so if it gets in the way of your work, feel free to ignore it, but I would like to issue a pull request for suggestions on creating a Dockerfile environment.

Verified Fork is located at
https://github.com/PINTO0309/yolov9mit/tree/feat_dockerfile

The main points of the proposed improvements are as follows

Add build instructions for CPU version of Docker container and CUDA+TensorRT version of Docker container to README
Build the container as a general user, not as a privileged user
Include TensorRT Execution Provider in onnxruntime-gpu (It will be possible to run TensorRT via onnx)
Fix all versions of a minimum set of required packages to a specific version
Enables rapid debugging environment from Dockerfile using VSCode's devcontainer, etc.

Pull request: https://github.com/WongKinYiu/yolov9mit/pull/29/

Hi,
Thank you for your suggestion and pull request. I will review the PR after I take a look at Docker!
In the next updates, I plan to provide the suggested versions of each package and separate the requirements for different levels.
Best regards,
Hao-Tang, Tsui

@PINTO0309 Do these version combinations in your dockerfile test a distributed GPU training (DDP)? I've noticed that if the pytorch versions are mismatched from what the developer originally intended it will cause issues.
@henrytsui000 Do you use/test multi gpu training with pytorch 1.12?
Thanks,

Do these version combinations in your dockerfile test a distributed GPU training (DDP)?

No, I am not. You can rewrite the Dockerfile and rebuild it yourself. In the first place, no one can give a clear reply to your question because the details of what kind of problem will arise have not been shared.


Write-Progress can destroy command line output

It seems that "Write-Progress" used in the Mounting script is not compatible with other frameworks like "Invoke-Build". The output will be interleaved because of a wrong cursor position. Hence the Write-Progress command should not be used.

Changes pushed to unstable branch.


Client tool

It says here that there is a client tool - but no link to it and i can't find it .. is it not done yet ?

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

ID: 9a558a9e-10d0-ea34-8780-d9084f87f952
Version Independent ID: 60dda4e0-7630-f358-aa3a-6a992eae61e2
Content: Managing the Azure Storage Lifecycle
Content Source: articles/storage/blobs/storage-lifecycle-management-concepts.md
Service: storage
Sub-service: common
GitHub Login: @yzheng-msft
Microsoft Alias: yzheng

@ohadbitt, Thanks for the feedback!I have assigned the issue to the content author to investigate further and update the document as appropriate.

@ohadbitt CLI extension storage-preview 0.2.5 (https://docs.microsoft.com/en-us/cli/azure/azure-cli-extensions-list?view=azure-cli-latest)  has support for lifecycle management. We will move this support to CLI core in the next several weeks. We will update this doc with a link then.

@ohadbitt, we will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.