TypeError: Cannot read property 'length' of undefined

I'm trying to communicate in Node.js with a device that uses Twofish in CBC mode.
I know the frames encrypted and unencrypted and I know that the first 16 bytes of the encrypted frame are the initialization vector, the next two are the length (to exclude the padding) and everything else is the Twofish encrypted message:
var key = new Buffer('01000000000000000000000000000000', 'hex');
var encrypted = new Buffer('b46a0260b0bc4922b5eb0785a4b7cc9e000f1425e4bb3d59f55734af263a60097901', 'hex');
var decoded = new Buffer('5ea08000000106010000000000e81c', 'hex');

If I use http://twofish.online-domain-tools.com with:
b4 6a 02 60 b0 bc 49 22 b5 eb 07 85 a4 b7 cc 9e as hex init. vector
14 25 e4 bb 3d 59 f5 57 34 af 26 3a 60 09 79 01 as hex input
01 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 as hex key
I get:
5e a0 80 00 00 01 06 01 00 00 00 00 00 e8 1c 00 as hex output (which makes sense as the length 0f means that the last 00 is just padding).
If I try to use yours then I get:
twofish.js:109
, nStrLen = sDOMStr.length
^
TypeError: Cannot read property 'length' of undefined
at Object.strToUTF8Arr [as stringToUTF8Array] (/Users/hlantinga/Documents/ATS/twofish.js:109:28)
at Object.decryptCBC [as decryptCBCMode] (/Users/hlantinga/Documents/ATS/twofish.js:920:23)

Sorry @HilkoLantinga, but coming back from Christmas holidays brings me in a "clients issues spiral of death"...
Today, after working time, I'll review the issue much more carefully.

Don't be sorry. It's great that you're willing to take a look in the first place.

So,
I haven't a solution by now but I tried to understand from your comments the scenario:
Here I tried to wrote down a test.
The ciphertext has to be an array, so I've ported all the values: are these right? If yes the outcome from decryption is different from what you have reported.
I've tried to trace the steps and all seems right. If you see any issues in the test case (I hope so :pray:) or you spot some of my mistakes please let me know...
Thanks.

The numbers you use are spot on. IV is correct, Key is correct, Expected output is correct.

I again @HilkoLantinga,
I reviewed the entire algorithm spotting some issues around typed arrays usage. Now the implementation seems good.
Could you please test again in your scenario? If the outcome will be correct I would release a new version in npm.
Thank you
Dario

Unfortunately I'm in a hotel that does not allow VPN so I can't check it against the device, will be home in 2 days and test it.

No problem man,
I'll write down the other tests scenarios from literature.

Right now, testing against cbc-d-m datas, I think the actual version has some problems...

Ok.
now here you can find the spec that refers to your issue, it passes.

tagged version 1.0.0 version is now on npm.
If you encounter future problems let me know here or in another issue.
Thanks for your contribution.


Error Binance

Hi, I am getting the following error:

C:\Users\Administrator\Desktop\bots\simple-pump-and-dump-bot-master\simple-pump-and-dump-bot-master\src>python bot-binance.py
Traceback (most recent call last):
File "C:\Users\Administrator\Desktop\bots\simple-pump-and-dump-bot-master\simple-pump-and-dump-bot-master\src\bot-binance.py", line 37, in
with open(secrets, "r") as secrets_file:
FileNotFoundError: [Errno 2] No such file or directory: './secrets-binance.yaml'

API keys have been added to the conf-binance file

Hey, please take a look at the readme again, you should have two files for the bot; A config file and a secrets file.
They're intended to keep your api keys separate from your config.
The error you have seems to indicate that you don't have a secrets-binance.yaml file

No, there is no secrets-binance.yaml file

Okay, please follow the instructions on the readme page for creating the secrets-binance.yaml file


Look for an update soon!  Thanks @happytogether


Interactive "Get Started" button ...

Hi @SaiSantosh2209 please give some more details about this issue.

@SaiSantosh2209 ?

Hello 👋  , I just wanted to make the button more interactive while selecting or hovering it ..

@gauravsingh1281 is this issue still active, would like to work on this?


fix: added empty molecule checks for sugar removal routines

resolves #386

If I may ask, why an exception? Why not simply return an empty string as the output SMILES code or some detectable placeholder string like "[empty]"?

CDK SmilesGenerator should have returned a more user-friendly message when we tried to generate SMILES from an empty IAtomContainer. We are now raising an exception to indicate that we have encountered this and give a user-friendly message. We could improve on the return error code since this is not truly an internal error but something to flag up the end user.

this is not truly an internal error but something to flag up the end user.
Exactly. So I would propose simply catching this specific "corner case" and return a useful value instead of throwing an error.

And another thing: We have the same situation with the other two sugar removal routines.


Init README.md should have project description, architecture, ..

Initial README.md should have project description, architecture, quick start, and onboarding

I've added architecture and quick start in 82221d6


BUG: Subclassed DataFrame slicing may return normal Series

[x] closes #11559
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

Failed in codecov, but it's ok as it doesn't touch codebase.

thanks!
FYI I had fixed the codecov issue.


Make top-level role index pages

/roles/engineering is blank. Add an index of the child pages below it

Released


Release artifacts are not signed

#SecuritySlam
What would you like to be added:
The project cryptographically signs release artifacts.
Why is this needed:
Signing artifacts would boost the security of the project. And since the CLOMonitor security score flags the project for not signing its artifacts, signing them would also make the score better.
Completion requirements:
These are the details from CLOMonitor around this task:
Signed-Releases OpenSSF Scorecard check

Score: 0 (check passes with score >= 1)

Reason: 0 out of 5 artifacts are signed or have provenance

Details:

Warn: release artifact v0.5.1 does not have provenance: https://api.github.com/repos/kubernetes-sigs/kueue/releases/131789880
Warn: release artifact v0.5.1 not signed: https://api.github.com/repos/kubernetes-sigs/kueue/releases/131789880
Warn: release artifact v0.5.0 does not have provenance: https://api.github.com/repos/kubernetes-sigs/kueue/releases/126674662
Warn: release artifact v0.5.0 not signed: https://api.github.com/repos/kubernetes-sigs/kueue/releases/126674662
Warn: release artifact v0.4.2 does not have provenance: https://api.github.com/repos/kubernetes-sigs/kueue/releases/124678930
Warn: release artifact v0.4.2 not signed: https://api.github.com/repos/kubernetes-sigs/kueue/releases/124678930
Warn: release artifact v0.4.1 does not have provenance: https://api.github.com/repos/kubernetes-sigs/kueue/releases/117224650
Warn: release artifact v0.4.1 not signed: https://api.github.com/repos/kubernetes-sigs/kueue/releases/117224650
Warn: release artifact v0.4.0 does not have provenance: https://api.github.com/repos/kubernetes-sigs/kueue/releases/111415075
Warn: release artifact v0.4.0 not signed: https://api.github.com/repos/kubernetes-sigs/kueue/releases/111415075

Please see the [check documentation](https://github.com/ossf/scorecard/blob/e1d3abc7fd2bdfe8819ac19b5c82815ea20890e6/docs/checks.md#signed-releases) in the ossf/scorecard repository for more details

/sig release

/retitle Release artifacts are not signed

This is a result report of the #securitySlam
cc @alculquicondor @tenzen-y

Found it https://wiki.debian.org/Creating signed GitHub releases

/lifecycle frozen


Added Desmos

@sunnya97 Will this PR make Osmosis work with Desmos even if the assetlist PR (osmosis-labs/assetlists#78) is not yet merged, or do we need to wait for that first?


2d quiver plots have arrowheads

Fixes #69
[x,y] = meshgrid(0:0.2:2,0:0.2:2);
u = cos(x).*y;
v = sin(x).*y;
quiver(x,y,u,v, 'MaxHeadSize',0.3)
fig2plotly()

See https://plot.ly/~jnj.16180340/514/

:dancer:


Allows the usage of slide index in print mode

Refers to #374
Allows to retrieve the slides index using the useDeck hook on print mode. Previously the value was always 0, because on the Print component, the slide index is not passed to the Slide component.

Even better would be to do this, which makes it possible to print the notes as well.
const Print = ({ slides, notes }) => {
  const outer = useDeck()
  const context = {
    ...outer,
    mode: modes.print,
  }

  return (
    <Context.Provider value={context}>
      {slides.map((slide, i) => (
        <Slide key={i} slide={slide} index={i} notes={notes} preview />
      ))}
    </Context.Provider>
  )
}

Would be very cool to see this merged :)


[master-next] Adjust to upstream llvm change SectionRef::getName()

I've got a similar patch here: https://github.com/apple/swift/pull/26657
I don't mind which one goes in. Glad to know others are working on this! :)

@xiaobai Oh I missed your PR.


XamarinAndroid: Ability to set output APK file name

Question, Bug, or Feature?
Type: Feature
Enter Task Name: XamarinAndroid
Issue Description
In some cases, the developer needs to specify the output APK file name, including using pipeline variables. It would be nice to add input to XamarinAndroid task for these purposes.

Any update on this?


Error in not validating array's of inputs correctly.

Subject of the issue
The error occurs in the instance of validating a form with array's of equal inputs, it only validates the first element found within the form.
Your environment

version of jquery-validate: 1.19.1
version of jquery: 3.4.1
which browser and its version: Google Chrome v81.0.4044.122 (Build oficial) (64 bits)

Steps to reproduce
http://jsfiddle.net/thejoker_dev/p6zs51v8/
Expected behaviour
It should validate all types of inputs, including those that are arrays of themselves.
Actual behaviour
Validates only the first input of the array with the same name, leaving the rest unvalidated.
Attached jquery-validate.js file modified on line 465 (checkForm function) which I leave below and it works without problems!
checkForm: function() { 		    this.prepareForm(); 		    for ( var i = 0, elements = (this.currentElements = this.elements()); elements[i]; i++ ) { 		        var checkingElements = this.findByName( elements[i].name ).not(this.settings.ignore); 		        if (checkingElements.length !== undefined && checkingElements.length > 1) { 		            for (var cnt = 0; cnt < checkingElements.length; cnt++) { 		                this.check( checkingElements[cnt] ); 		            } 		        } else { 		            this.check( elements[i] ); 		        } 		    } 		    return this.valid(); 		}
Attached jquery-validate.js file modified on line 465 (checkForm function) which I leave below and it works without problems!
jquery.validate.zip

The is a long standing issue and linked to many in the past - https://github.com/jquery-validation/jquery-validation/issues/2046 https://github.com/jquery-validation/jquery-validation/issues/2102 https://github.com/jquery-validation/jquery-validation/issues/1226
Linked PR https://github.com/jquery-validation/jquery-validation/pull/2000

JqueryValidation filtering same name elements under rules cache, I just comment the rulesCache. It works for me.
https://stackoverflow.com/a/69452906/1939258

How to add remote method validation to array of inputs using class name
`

input type="text" id="ApplyAuthTCSCDT1" name="ApplyAuthTCSCDT[1]" class="form-control scdDateTime formPadding scdDataDT" required=""
input type="text" id="ApplyAuthTCSCDT2" name="ApplyAuthTCSCDT[2]" class="form-control scdDateTime formPadding scdDataDT" required=""
input type="text" id="ApplyConfirmpnSCTCDT1" name="ApplyConfirmpnSCTCDT[1]" class="form-control scdDateTime formPadding scdDataDT" required=""
input type="text" id="ApplyConfirmpnSCTCDT2" name="ApplyConfirmpnSCTCDT[2]" class="form-control scdDateTime formPadding scdDataDT" required=""
input type="text" id="RemoveAuthTCSCDT1" name="RemoveAuthTCSCDT[1]" class="form-control scdDateTime formPadding scdDataDT" required=""
input type="text" id="RemoveAuthTCSCDT2" name="RemoveAuthTCSCDT[2]" class="form-control scdDateTime formPadding scdDataDT" required=""

`
Have tried to use the following jquery validation rule function but getting error Uncaught TypeError: e.nodeName is undefined.  Exception occurred when checking element ApplyConfirmpnSCTCDT1, check the 'remote' method. .
The following are the jquery code.
`

$(".scdDataDT").each(function() {
$(this).rules('add', {
required: true,
remote: {
url: "url.php",
type: "POST",
async: false,
data: {
scdDT: function() {
console.log($(this).val()) ;
//return $( "#"+$(this).attr("id")).val();
},
pbId: function() {
return $( "#SCDApplyRemovePBId" ).val();
},
pbNo: function() {
return $( "#SCDApplyRemovePBNo" ).val();
}
}
},
messages: {
required: "*",
remote:"#"
}
});
});`


Azure managed disk ignored when storage_account_type = "StandardSSD_ZRS"

Infracost v0.10.13
When storage types are set as StandardSSD_LRS, infracost reports storage costs correctly but if you change the type to "StandardSSD_ZRS" then the costs are ignored when hey should be 3xthe LRS cost.
This is both within azurerm_managed_disk and the OS disk created as part of azurerm_windows_virtual_machine
When the storage type is set as "StandardSSD_LRS" for both types of disks
Project: *******

 Name                                                          Monthly Qty  Unit                    Monthly Cost (GBP)

 azurerm_managed_disk.drive_e
 ├─ Storage (E50)                                                        1  months                             £292.54
 └─ Disk operations                                   Monthly cost depends on usage: £0.001731436 per 10k operations

 azurerm_managed_disk.drive_f
 ├─ Storage (E50)                                                        1  months                             £292.54
 └─ Disk operations                                   Monthly cost depends on usage: £0.001731436 per 10k operations

 azurerm_windows_virtual_machine.server
 ├─ Instance usage (pay as you go, Standard_E2ds_v5)                   730  hours                              £164.95
 └─ os_disk
    ├─ Storage (E15)                                                     1  months                              £18.28
    └─ Disk operations                                Monthly cost depends on usage: £0.001731436 per 10k operations

 OVERALL TOTAL (GBP)                                                                                £768.32

When the storage type is set as "StandardSSD_ZRS" for both types of disks
Project: *******

 Name                                                 Monthly Qty  Unit   Monthly Cost (GBP)

 azurerm_windows_virtual_machine.server
 ├─ Instance usage (pay as you go, Standard_E2ds_v5)          730  hours             £165.90

 OVERALL TOTAL (GBP)                                                                 £165.90

Hey @Marcus-James-Adams, thanks for raising this. Yes, looking at the code for azurerm_managed_disk we don't currently support ZRS disk types. I think this is something we could accomodate quite easily by the looks of it. However, just to set your expectations, we have a lot of other high priority issues in our backlog. So we might not get to this that quickly. If you want this quicker than our current roadmap, feel free to dive in and raise a PR ❤

hey @hugorut I'm going to try to get this one sorted as my first contribution. How do I add this to the project board?

I think this was fixed by https://github.com/infracost/infracost/pull/2153


implement Bezier

I made a rough prototype of a possible bezier implementation. Here is the beautified adapted output of the render_to_image example:

From a design standpoint I had the following ideas for implementation:

Bezier curves can technically have an infinite amount of control points, crates on crates.io I could find only support quadratic and cubic bezier curves. I assume this is modelled after the SVG spec, which only supports cubic and quadratic beziers.
The rough prototype just implements a generic Bezier struct with 3 control points and an Option for a fourth one
I would like to abstract away the SVG limitations in our implementation and expose a generic Bezier from an array of Points which we subdivide into quadratic and cubic parts for rendering

Are there any caveats with a generic implementation over an array of Points you can think of @Michael-F-Bryan ?

Wow, that image looks really quite pretty!

A problem I've often found when talking about splines and bezier curves is that there are loads of different variants, and it's really hard to find some "one size fits all" abstraction which each variant fits into.
For example, one library I've used has a generic Polyline type which contains a Vec<Point> and a PolylineFit which can be one of

None (straight lines between vertices)
Bulge (each vertex has an associated bulge factor)
Quadratic (a normal quadratic bezier curve)
Cubic (cubic bezier)
Spline (spline interpolation)
Round (the number attached to each vertex indicates a fillet radius)
BSpline (a b-spline)

This is logically equivalent to your idea of abstracting away the curve's order and can get quite confusing. Additionally, switching between the PolylineFit type changes the fundamental shape of your curve (i.e. the points it passes through), so it's more than just how the object gets rendered.
Also, what do the array of points signify when switching between curve types? In a generic implementation will they always be control points? How would we deal with b-splines which also have "knots"?
I'm tempted to have each curve variant be its own type, and we might use similar abstractions under the hood to reduce code duplication and ease conversion between curve types.

How do applications like Inkscape and AutoCAD deal with this sort of thing under the hood?
Based on my interactions with the DXF and DWG formats I'm inclined to not follow their lead, but the way Inkscape implements bezier curves is quite intuitive from an end user's perspective...

I was only thinking of Bezier curves for now.
Maybe in a broader sense we could have an interface which has a method for producing a renderable kurbo::BezPath? We could then group all curve types under this interface and have all individual curve types implemented as their own structs:

QuadraticBezier
CubicBezier
NDimensionalBezier
BSpline

pub trait Curve {
    fn path(&self) -> kurbo::BezPath;
}

The main 2 use cases from my experience are

construction of curves from raw control points
interpolating a curve through a given set of feature points

Interpolated splines are a bit tricky as multiple primitive backends can be used for interpolation of a set of points, so all curve primitives where we can find a nice interpolation algorithm could implement another interface
pub trait InterpolatedCurve {
    fn interpolate_points(points: Vec<Point>) -> Self;
    fn interpolate_points_with_end_tangencies(points: Vec<Point>, start_tangent: Vector, end_tangent: Vector) -> Self;

For my CAD work i mainly used a NURBS modeler, in this they abstract all curves to nurbs curves under the hood (even circles and beziers).
The documentation for the AutoCAD spline suggests they do the same for their splines.
Looking at InkScapes tools for creation and the XML-DocumentTree they seem to be rather oriented around the SVG spec


add split pseudo-voigt with split lorentzian fraction

Allows the Lorentzian fraction to be different on the left and on the right of the centroid:

Closes https://github.com/silx-kit/silx/issues/3901
Changelog: add split pseudo-voigt with split lorentzian fraction

In fact one should test to have a multiple of 6 parameters.
if len(params) and not params % 6:

In fact one should test to have a multiple of 6 parameters.
if len(params) and not params % 6:

Done


Correction for raw affiliation Canadian Rivers Institute, 28 Dineen Drive, Fredericton, NB E3B 5A3, Canada; Department of Biology, University of Waterloo, 200 University Avenue, Waterloo, ON N2L 3G1, Canada

Correction needed for raw affiliation Canadian Rivers Institute, 28 Dineen Drive, Fredericton, NB E3B 5A3, Canada; Department of Biology, University of Waterloo, 200 University Avenue, Waterloo, ON N2L 3G1, Canada
raw_affiliation_name: Canadian Rivers Institute, 28 Dineen Drive, Fredericton, NB E3B 5A3, Canada; Department of Biology, University of Waterloo, 200 University Avenue, Waterloo, ON N2L 3G1, Canada
new_rors: 01aff2v68;05nkf0n29
previous_rors: 01aff2v68
works_examples: W3188155380
contact: b8a9c7504852260faf7c8fcdd865a846:97030034 @ ourresearch.org

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


View Components

ASP.NET MVC -> Views -> View Components

What is the story on already written blog posts on the topics by the community? Are there any plans to link them relevantly or move the ones that make sense?
For example, I have a post on View Components. Of course, the community blog posts may not meet the quality bar of the official docs.

We love our community blog posts, but we want this site to have official documentation that we maintain over time. @Rick-Anderson and @MikeWasson have some initial View Component documentation here that I had expected to port over and update. Let us know though if you want take on this topic.


Gradlew Build Fails

Sorry for another issue but it looks like bintray is going away and or having a major outage as building your project no longer works
aiode@aiode00:~/aiode$ ./gradlew build
> Task :compileJava FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':compileJava'.
> Could not resolve all files for configuration ':compileClasspath'.
   > Could not resolve com.sedmelluq:lavaplayer-ext-youtube-rotator:0.2.3.
     Required by:
         project :
      > Could not resolve com.sedmelluq:lavaplayer-ext-youtube-rotator:0.2.3.
         > Could not get resource 'https://dl.bintray.com/sedmelluq/com.sedmelluq/com/sedmelluq/lavaplayer-ext-youtube-rotator/0.2.3/lavaplayer-ext-youtube-rotator-0.2.3.pom'.
            > Could not HEAD 'https://dl.bintray.com/sedmelluq/com.sedmelluq/com/sedmelluq/lavaplayer-ext-youtube-rotator/0.2.3/lavaplayer-ext-youtube-rotator-0.2.3.pom'. Received status code 502 from server: Bad Gateway
   > Could not resolve net.dv8tion:JDA:4.3.0_346.
     Required by:
         project :
      > Could not resolve net.dv8tion:JDA:4.3.0_346.
         > Could not get resource 'https://dl.bintray.com/sedmelluq/com.sedmelluq/net/dv8tion/JDA/4.3.0_346/JDA-4.3.0_346.pom'.
            > Could not HEAD 'https://dl.bintray.com/sedmelluq/com.sedmelluq/net/dv8tion/JDA/4.3.0_346/JDA-4.3.0_346.pom'. Received status code 502 from server: Bad Gateway

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/7.0/userguide/command_line_interface.html#sec:command_line_warnings

BUILD FAILED in 5s
1 actionable task: 1 executed

Update: been a busy week so I only just now had time to come back to this. I did a fresh install and got the same error. So if you are running the bot let it keep running

Running isn't the problem, building is. Seems that https://dl.bintray.com/sedmelluq/com.sedmelluq is down. I removed the repository, along with moving all jcenter dependencies to jitpack where I can since jcenter will shut down completely soon. Jda-nas is still pulled from jcenter, everything else was moved. Builds should work again.


RuntimeError in Jupyter Notebook

If I run the exmaple code for KBQA in a jupyter notebook I get the following error message:
"RuntimeError: asyncio.run() cannot be called from a running event loop"
Any idea what the issue might be / how to fix it? Thank!
---- (executed code) -----
from factool import Factool
Initialize a Factool instance with the specified keys. foundation_model could be either "gpt-3.5-turbo" or "gpt-4"
factool_instance = Factool("gpt-4")
inputs = [
{
"prompt": "Introduce Graham Neubig",
"response": "Graham Neubig is a professor at MIT",
"category": "kbqa"
},
]
response_list = factool_instance.run(inputs)
print(response_list)

Dear @ch-au, Sorry for the late reply! I accidentally missed this issue.
I think changing all asyncio.run in factool.py to await will work. Since Jupyter Notebook is already running an event loop, starting another event loop with asyncio.run() won't be possible.
Let me know if you have more questions!

Thanks, Ethan. That worked.
Von: Ethan Chern @.>
Antworten an: GAIR-NLP/factool @.>
Datum: Mittwoch, 13. September 2023 um 8:43 PM
An: GAIR-NLP/factool @.>
Cc: ch-au @.>, Mention @.***>
Betreff: Re: [GAIR-NLP/factool] RuntimeError in Jupyter Notebook (Issue #33)
Dear @ch-auhttps://github.com/ch-au, Sorry for the late reply! I accidentally missed this issue.
I think changing all asyncio.run in factool.pyhttps://github.com/GAIR-NLP/factool/blob/main/factool/factool.py to await will work. Since Jupyter Notebook is already running an event loop, starting another event loop with asyncio.run() won't be possible.
Let me know if you have more questions!
—
Reply to this email directly, view it on GitHubhttps://github.com/GAIR-NLP/factool/issues/33#issuecomment-1717467780, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGFDFJNS2FGMHET6OH4HHNDX2GL7TANCNFSM6AAAAAA4J64GZI.
You are receiving this because you were mentioned.Message ID: @.***>


Updated success and error with then and catch.

Deprecation notice: https://docs.angularjs.org/api/ng/service/$http

Deprecation notice is within the 1.5.9 documentation: https://code.angularjs.org/1.5.9/docs/api/ng/service/$http


default value of feeling seems not to be set

The default value of feeling should be shown "★★★", but it is blank.
Should set 'ng-init'?
調子

  <select class="form-control" data-ng-model="feeling">
    <option value="1">★</option>
    <option value="2">★★</option>
    <option value="3" selected>★★★</option>
    <option value="4">★★★★</option>
    <option value="5">★★★★★</option>
  </select>

Instead of using ng-init, I used ng-options and I added the new variable in temperatures.client.controller.js.
This was fixed on https://github.com/uchikoshi22/reconetsu/commit/24faedde2c08e2b79cfd2c26b090e08a0a14bff2
'''$scope.feeling = $scope.feelingChoices[2];'''
To throw POST data to mongoose, I change the Tempratures object's variable in temperatures.client.controlller.js.
from
 feeling     : this.feeling
to
 feeling     : this.feeling.value,


NLTK 3.8.1 not tagged as release

Hi, I see from PyPi and the project website that NLTK 3.8.1 was released in January, but it doesn't seem to have been tagged/released in the GitHub repo.

@ferdnyc Thanks for letting me know! I made the tag, but forgot to push it with git push --tags. I've done so now.
I think this should be resolved now, so I'll close this :)


update recognizer packages to 1.1.1

This gives us confirm prompt working in Chinese and Dutch.
We are still missing German and French.

Pull Request Test Coverage Report for Build 41020

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

Totals

Change from base Build 41010:
0.0%

Covered Lines:
4430

Relevant Lines:
9220

💛  - Coveralls


Fixing broken URL

Anchor name is worng

Docs Build status updates of commit 620c767:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/Cards/cards-actions.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add option to add a mentioned  to Google Calendar.

It would be awesome to add a mentioned time zone aware time <time xxx> to Google Calendar or the default OS calendar app. Discuss the UX for this in chat.zulip.org before starting to work on this.
Relevant docs - https://zulip.com/help/format-your-message-using-markdown#mention-a-time

@zulipbot claim

@zulipbot claim

@Uttam-Singhh I am still working on this, you can take any other issue.

@nikhilmaske-2001 Sure! No Problem

@zulipbot abandon

@zulipbot claim

@zulipbot claim

claim

@zulipbot  claim

@zulipbot claim

@zulipbot abandon

@zulipbot claim

@zulipbot abandon

@zulipbot claim

abandon @zulipbot

@zulipbot abandon

@zulipbot claim

@zulipbot  declaim. This issue already has a PR #17593

@zulipbot abandon

@zulipbot claim

claim

@zulipbot claim

@zulipbot abandon


(website) infinite redirect on https://bork.sh

$ curl -v https://bork.sh/
*   Trying [2606:4700:3033::ac43:d38f]:443...
* Connected to bork.sh (2606:4700:3033::ac43:d38f) port 443
* ALPN: curl offers h2,http/1.1
* (304) (OUT), TLS handshake, Client hello (1):
*  CAfile: /etc/ssl/cert.pem
*  CApath: none
* (304) (IN), TLS handshake, Server hello (2):
* (304) (IN), TLS handshake, Unknown (8):
* (304) (IN), TLS handshake, Certificate (11):
* (304) (IN), TLS handshake, CERT verify (15):
* (304) (IN), TLS handshake, Finished (20):
* (304) (OUT), TLS handshake, Finished (20):
* SSL connection using TLSv1.3 / AEAD-AES256-GCM-SHA384
* ALPN: server accepted h2
* Server certificate:
*  subject: CN=bork.sh
*  start date: Mar 30 19:51:15 2024 GMT
*  expire date: Jun 28 19:51:14 2024 GMT
*  subjectAltName: host "bork.sh" matched cert's "bork.sh"
*  issuer: C=US; O=Google Trust Services LLC; CN=GTS CA 1P5
*  SSL certificate verify ok.
* using HTTP/2
* [HTTP/2] [1] OPENED stream for https://bork.sh/
* [HTTP/2] [1] [:method: GET]
* [HTTP/2] [1] [:scheme: https]
* [HTTP/2] [1] [:authority: bork.sh]
* [HTTP/2] [1] [:path: /]
* [HTTP/2] [1] [user-agent: curl/8.4.0]
* [HTTP/2] [1] [accept: */*]
> GET / HTTP/2
> Host: bork.sh
> User-Agent: curl/8.4.0
> Accept: */*
>
< HTTP/2 301
< date: Fri, 05 Apr 2024 12:10:09 GMT
< content-type: text/html
< location: https://bork.sh/
< x-github-request-id: 1892:29872C:41C60CC:42DA5CA:660FEA04
< accept-ranges: bytes
< via: 1.1 varnish
< age: 29
< x-served-by: cache-lcy-eglc8600080-LCY
< x-cache: HIT
< x-cache-hits: 1
< x-timer: S1712319010.852185,VS0,VE1
< vary: Accept-Encoding
< x-fastly-request-id: fd33e0d4533a1b6d285731a6044ad970cbbca2ac
< cf-cache-status: DYNAMIC
< report-to: {"endpoints":[{"url":"https:\/\/a.nel.cloudflare.com\/report\/v4?s=7VNUiMQsRyauEGTgJ2pBA6zn5z2Akdl2nPY%2B9Oj6Oo2Q36cc4pGf%2BvMv81CoFpAMlJgu3sQ4VDsY%2FMfWoAWSDfVA77tTMrhRN46Uc085NE8Qy8mlQoOdNqv34hOrTXaMGwboEXeq"}],"group":"cf-nel","max_age":604800}
< nel: {"success_fraction":0,"report_to":"cf-nel","max_age":604800}
< server: cloudflare
< cf-ray: 86f96ef359142403-LHR
< alt-svc: h3=":443"; ma=86400
<

<title>301 Moved Permanently</title>

301 Moved Permanently

nginx

* Connection #0 to host bork.sh left intact

I blame Cloudflare…
I've grey-clouded it, hopefully that fixes it.


Should modify iisnode.yml when it is under a folder and pointed by package.json/scripts/start

The location of iisnode.yml is based on the location of the Node.js entrypoint, but not web.config.
Currently, Kudu modify the iisnode.yml in the root, instead of locating it at the same level of the Node.js entrypoint.
Repro steps.

Write a web server entrypoint named lib/app.js and it simply spill out process.env.node_env
Modify package.json and set scripts.start to node lib/app.js
Write a lib/iisnode.yml with content node_env: under-lib
Write another iisnode.yml with content node_env: root
Have a usual web.config that run the project with iisnode, for example, copy from https://gist.github.com/davidebbo/2571104
Deploy thru Project Kudu (in my case, it's Azure Web App)

Expected

lib/iisnode.yml is appended with pointer to Node.js of the latest version (7.7.4 when I deployed to Azure)
When browse to the site, it say under-lib

Actual

iisnode.yml is modified instead
When browse to the site, it say root

Project structures.
/iisnode.yml (this file should be ignored)
/package.json (this file has scripts.start pointing to /lib/app.js)
/lib/iisnode.yml (this file should be modified)
/lib/app.js (entrypoint and response with process.env.node_env)

The log/error given by the failure.
The wrong iisnode.yml is modified.
Mention any other details that might be useful.
I verified the behavior of locating iisnode.yml in an environment with IIS and iisnode, no Kudu and not on Azure.
You can take my Docker image that have iisnode preinstalled and try it out yourself easily.

Can you please share a minimal repo that demonstrates the problem? It makes it easier (and less error prone) than having to create one step by step.

@davidebbo Could you try out my repro at my repository?
You will see there are two iisnode.yml, one under root, and one under /lib/. The one at root is a dummy and it is never used by iisnode. It set NODE_ENV to root.

In my case, I use the Project Kudu on Azure Web App. You can use the "Deploy to Azure" from README.md for a quick deployment.

After you deploy it via Kudu, the site will print {"NODE_ENV":"under-lib",...}. That means iisnode is loading /lib/iisnode.yml, instead of /iisnode.yml.
But when you look at /iisnode.yml, you will see it get modified by Project Kudu:
node_env: root
nodeProcessCommandLine: "D:\Program Files (x86)\nodejs\7.7.4\node.exe"

And for the /lib/iisnode.yml that is actually loaded by iisnode, you won't see the change there. It just looks like:
node_env under-lib

It seems Project Kudu is modifying the incorrect iisnode.yml.

iisnode read the configuration file iisnode.yml based on the location of the current request, you can see the code here and the reference of GetScriptTranslated.
That means, Project Kudu would need to lookup the iisnode.yml based on web.config/system.webServer/handlers/add[name="iisnode"]/@path.

I think the issue is that the logic that the logic that deals with using the subpath is only applied when there is no web.config in the repo. Can you try deleting it and letting Kudu's web.config generation do its thing?

Possible Kudu fix: 0ef0cec5f6687a0e8e927fe7dbfbd19ae6fc9ca2. Better diff that ignores blanks: https://github.com/projectkudu/kudu/commit/0ef0cec5f6687a0e8e927fe7dbfbd19ae6fc9ca2?w=1

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


Date PIcker code structure refactor

As flagged by @Hixie in material/pickers/pickers.dart:
// TODO(ianh): Not exporting everything is unusual and we should
// probably change to just exporting everything and making sure it's
// acceptable as a public API, or, worst case, merging the parts
// that really must be public into a single file and make them
// actually private.

Which is a valid point, and here is a proposal for how to do that. The current code structure for the date pickers is:
material/pickers
├── calendar_date_picker.dart
├── calendar_date_range_picker.dart
├── date_picker_common.dart
├── date_picker_deprecated.dart
├── date_picker_dialog.dart
├── date_picker_header.dart
├── date_range_picker_dialog.dart
├── date_utils.dart
├── input_date_picker.dart
├── input_date_range_picker.dart
└── pickers.dart

I would like to combine several of these and move everything up to the top level material directory as follows:

File
Contents

material/date_picker.dart
showDatePicker/RangePicker,  date_picker_header and dialogs

material/calendar_date_picker.dart
just a move, but possibly re-expose YearPicker

material/calendar_date_range_picker.dart
move and make CalendarDateRangePicker public

material/date_picker_deprecated.dart
just a move

material/date.dart
date utilities from date_utils.dart & date_picker_common.dart

material/input_date_picker_form_field.dart
move from input_date_picker.dart

In addition, we should new test files for the top level components:
test/material/calendar_date_picker_test
test/material/calendar_date_range_picker_test
test/material/input_date_picker_form_field_test.dart
This should bring the code structure for the Date Pickers to be more inline with the rest of the flutter framework, and thus easier for people to follow. This will also remove un-exported public symbols that were still technically available for people to use.

SGTM!


Issue 6411 ROMol hasquery

Reference Issue
Fixes #6411
What does this implement/fix? Explain your changes.
Adds ROMol::hasQuery & Mol.HasQuery()
Any other comments?
This isn't cached as in the issue report

#hackathon


Add GetConstraints and GetCosts to Problem

These are useful if, for instance, I want to get the names of the constraints or get the contribution of each cost to the total cost.

Thanks!


[ReferenceError: Property 't' doesn't exist]

this error seems to be coming from index.min.js

Could you reproduce it in CodePen or CodeSandbox?

OK. v0.2.0, which does not minified, is now available.
The reason for minifying was for mangling class fields. Now I have stopped using classes and the object field names are now single characters.

update version 0.2.0 for @luncheon/simplify-svg-path on @benjeau/react-native-draw/node_moduls
i tried and ok

same as hongdthaui, that works

@hongdthaui @sickopickle
thanks for confirming that it works.

@luncheon just found out this error upgrading from 0.1.4 to 0.1.5 , nice to see that it's fixed in 0.2.0, thanks for that!
It would be nice to have either a changelog or to remove / mark as problematic the 0.1.5 release on npm


screen ratio issue

What happened?
When opening the client the resolution is 1(w):1.5(L) so when i open the menu with I i cant see the options to get into graphics options to change this as i cant see it due to it being hidden under other things as a result of the resolution
Steps to reproduce
open client with 2 screens set out one above the other

Operating System
Windows 11  64bit
Wallet
None, played as guest
CPU
AMD Ryzen Threadripper PRO 3995WX 64-Cores
GPU
NVIDIA GeForce RTX 3080 Ti 12108MB x2
RAM
261969MB
Kernel Version
1.0.0-1928584197.commit-b86632b
Unity Version
desktop-main.commit-4e9d7b9
Realm
dg
NameTag
fdgsf#74d8

Click the "P" button on your keyboard when you're in-game.

yes i understand this is where it is but the options i need are off the screen due to this resolution it gave me to start with

this PR that's been recently merged solves this problem.
https://github.com/decentraland/explorer-desktop/pull/162


Templates are notoriously hard to create due to privledges.  It seems that only project administrators can create templates.  This seems craze for projects with thousands of people on them, like bing.

[Enter feedback here]

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

ID: 1debd814-eb48-c984-a4f9-7606093e9617
Version Independent ID: f48ef796-c2e3-8fa3-6c41-326a5b252a3d
Content: Use work item templates to update stories & more - Azure Boards
Content Source: docs/boards/backlogs/work-item-template.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@JeffreyWolford Thanks for your question. It looks like you have a product question or suggestion, not a documentation issue. To make sure it gets in front of the right people, please submit your bug here.
If doc changes come up from your suggestion, we can reopen this issue.

@JeffreyWolford - To add or edit templates. a user must be a member of the team.  You can determine if a user is a member of a team by going to Project Settings>Permissions>Users, select a User and choose the Member of tab.


Add DROP OWNED BY

DOC-4770
DOC-5465  Depends on system privileges.
Blocks https://github.com/cockroachdb/docs/pull/15298

This change is

@chengxiong-ruan mind giving this another look?  taking this over since Stephanie left the co a while back.  Added some examples as well
@rafiss added you to review for the DOC-5465 piece

NB. this PR will wait for #15715 to go in and then rebase/modify atop that one per this convo
(There should be no substantial technical content changes to this PR as part of that process tho)

@ianjevans this is RFAL from a docs review POV now.  The latest changes should all be "docs review only"-level changes in the sense that they shuffle around some things that have already had technical review either in this PR or in #15715


macvim does not recognize math latex symbols anymore

I am running Macvim 8.0.1633 (146) on macos 10.13.3 Weirdly, even with syntax on, the math
highlighting is plainly wrong, see
https://vi.stackexchange.com/questions/16199/macvim-does-not-recognize-math-latex-symbols-anymore
In the screenshot above, I didn't search for equation and the file compiled just fine.
As explained, the output of echo synIDattr(synID(line("."), col("."), 1), "name") turns out to be texBadMath. Anybody knows here what is going on?

Is the issue also present in plain console Vim?  The TeX syntax highlighting file was changed fairly recently.
Can you reproduce it, in either MacVim or plain Vim, starting with [m]vim -u NONE -U NONE --noplugin?
I can't reproduce on my end.


Add recurring events

Please add recurring events (weekly, monthly, etc.), this makes it much easier to set up repeating tasks.
Recurring events are a bit difficult to implement, so here are some tips:
Define absolute and relative recurring events

Absolute recurring events are bound to a specific date, like the 20th of the month, every Tuesday, etc.
Relative recurring events are bound to relative dates, like every third Monday, last Sunday in month, etc.
Allow the user to select multiple weekdays, if a weekly recurring event is selected

Great idea for improvement, it will be added soon :D

New app verison :)

Where can you add recurring events? I can not find anything related to this


[BUG] date field is not imported

Prerequisites

[Х] Checked if your Laravel Excel version is still supported: https://docs.laravel-excel.com/3.1/getting-started/support.html#supported-versions
[Х] Able to reproduce the behaviour outside of your code, the problem is isolated to Laravel Excel.
[Х] Checked that your issue isn't already filed.
[Х] Checked if no PR was submitted that fixes this problem.
[Х] Filled in the entire issue template

Versions

PHP version: 7.2.19
Laravel version: 6.17.1
Package version: 3.1

Description
created_at field is not imported in database. Instead I see 1970-01-01 in phpmyadmin
Steps to Reproduce
namespace App\Imports;

use App\NewsPost;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Collection;
use Maatwebsite\Excel\Concerns\ToCollection;
use Maatwebsite\Excel\Concerns\WithHeadingRow;

class NewsImport implements ToCollection, WithHeadingRow

{
    public $rowCount = null;

    public function collection(Collection $rows)
    {
        News::truncate();

        foreach ($rows as $row)
        {
          if( !empty($row['category_id'])) {

            NewsPost::create([

               'category_id'=> $row['category_id'],
               'title'      => $row['title'],
               'sef'        => $row['sef'],
               'introtext'  => $row['introtext'],
               'fulltext'   => $row['fulltext'],
               'image'      => $row['image'],
               'metatitle'  => $row['metatitle'],
               'metakey'    => $row['metakey'],
               'metadesc'   => $row['metadesc'],
               'hits'       => $row['hits'],
               'created_at' => $row['created_at'],

            ]);
          }

        }

      $this->rowCount = $rows->count();

    }

}

namespace App\Admin\Controllers;

use Illuminate\Support\Facades\DB;
use App\Imports\NewsPostImport;
use Maatwebsite\Excel\Facades\Excel;

class SeedController extends BaseAdminController
{

    public function index(Content $content)
    {
        DB::statement('SET FOREIGN_KEY_CHECKS = 0');
        Excel::import($newsPost     = new NewsPostImport,       'import/news_posts.xlsm');
        DB::statement('SET FOREIGN_KEY_CHECKS = 1');

        return $content
            ->title('Import & Export')
            ->description('Laravel excel')
            ->withSuccess('Импорт завершен успешно',
               '
Импортировано '. $newsPost->rowCount.' новостей
');
    }

}

Is created_at fillable on your model?

Yes. I have all fields fillable

Try to dd() and see what's in there.

sorry for delay. I was on vacation.
I put dd here
            if (empty($row['slug'])) {
              News::create([
                 'category_id'  => $row['category_id'],
                 'title'        => $row['title'],
                 'slug'         => Str::slug($row['title'], '-'),
                 'fulltext'     => $row['fulltext'],
                 'image'        => $row['image'],
                 'metatitle'    => $row['metatitle'],
                 'metakey'      => $row['metakey'],
                 'metadesc'     => $row['metadesc'],
                 'hits'         => $row['hits'],
                 'published_at' => $row['published_at'],
                 'created_at'   => $row['created_at'],
                 'updated_at'   => $row['updated_at'],
                 'deleted_at'   => $row['deleted_at'],
              ]);
              dd($row);
            }

and I see strange date output on the page
Illuminate\Support\Collection {#1986 ▼
  #items: array:14 [▼
    "category_id" => 4
    .......
    "published_at" => null
    "created_at" => 41767
  ]
}

in my excel I have date like that

any idea?

I found problem. It was on excel side.
I put dd($row) before News::create and check what comes from excel.
And I saw 41767 again.
Then I go to excel and check formatting of created_at field. It was date. But when I change format from date to text immideately I see 41767.
Doing that I understood that before import all dates in excel are converted to text.
So I converted these fields to text manually, put dates inside and it works now.

Only once I was able to do import succesfully and then I still have some problems, which I cannot understand.
It is headache to import dates. I cannot see rule, which working always.
I tested few times how should I keep date in Excel before import and I see not stabile behaviour.
I make all date fields format = text in excel.
When I put date 08-05-2014 and format as text.
Anyway I have error during import
InvalidArgumentException
Unexpected data found.
 Unexpected data found.
 Data missing
http://localhost.test/admin/seed

pointing on \vendor\nesbot\carbon\src\Carbon\Traits\Creator.php:623
        if (static::isStrictModeEnabled()) {
            throw new InvalidArgumentException(implode(PHP_EOL, $lastErrors['errors']));
        }

But when I do dd($row) I see
 "created_at" => "08-05-2014"

What is wrong?

Seems that I found solution.
Only working way is to keep date 2014-05-08 in Excel and format as text.
Because in this order it is stored in MySQL = YYYY-MM-DD
Any of these is not working
08-05-2014 - wrong order
08.05.2014 - wrong delimiter and wrong order
2014.05.08 - wrong delimiter
If date fields formated as date in Excel in my case date 2014-05-08 was always converted to 41767 before import. I don't know why.
I tryed date to timestamp converter and it gives me
2014-05-08 =<PHONE_NUMBER>.
08-05-2014 =<PHONE_NUMBER>
Only way I could get 41767 is when I tryed in Excel formula =text(08.05.2014;0)
So I suppose it is converted by Excel.
And it is converted even if I adjust date field to YYYY-MM-DD format.
So only working solution is text format with YYYY-MM-DD text inside.
Correct me if this solution is not completely right.

I'm having the same issue. The 'date' fields are returning a number (a ref, I guess?) and not the value.

To get Carbon when having the date column formatted as a date: Carbon::instance(\PhpOffice\PhpSpreadsheet\Shared\Date:: excelToTimestamp($row['created_at']))

It works for not empty fields. But when import script reach empty field it stops with following error.
SQLSTATE[22007]: Invalid datetime format: 1292 Incorrect datetime value: '1970-01-01 00:00:00' for column 'published_at' at row 1 (SQL: insert into `news_posts` (`category_id`, `title`, `slug`, `fulltext`, `image`, `metatitle`, `metakey`, `metadesc`, `hits`, `is_published`, `published_at`, `created_at`, `updated_at`, `deleted_at`) values ...

Put it into a temporary variable

what do you mean under  "temporary variable"?

I found acceptable solution
        foreach ($rows as $row)
        {
          if (!empty($row['category_id'])) {
            $slug = empty($row['slug']) ? Str::slug($row['title'], '-') : $row['slug'];
            $published_at = empty($row['published_at']) ? $row['published_at'] : Carbon::instance(\PhpOffice\PhpSpreadsheet\Shared\Date::excelToDateTimeObject($row['published_at']));
            $created_at = empty($row['created_at']) ? $row['created_at'] : Carbon::instance(\PhpOffice\PhpSpreadsheet\Shared\Date::excelToDateTimeObject($row['created_at']));
            $updated_at = empty($row['updated_at']) ? $row['updated_at'] : Carbon::instance(\PhpOffice\PhpSpreadsheet\Shared\Date::excelToDateTimeObject($row['updated_at']));
            $deleted_at = empty($row['deleted_at']) ? $row['deleted_at'] : Carbon::instance(\PhpOffice\PhpSpreadsheet\Shared\Date::excelToDateTimeObject($row['deleted_at']));

              NewsPost::create([
                 'category_id'  => $row['category_id'],
                 'title'        => $row['title'],
                 'slug'         => $slug,
                 'fulltext'     => $row['fulltext'],
                 'image'        => $row['image'],
                 'metatitle'    => $row['metatitle'],
                 'metakey'      => $row['metakey'],
                 'metadesc'     => $row['metadesc'],
                 'hits'         => $row['hits'],
                 'is_published' => $row['is_published'],
                 'published_at' => $published_at,
                 'created_at'   => $created_at,
                 'updated_at'   => $updated_at,
                 'deleted_at'   => $deleted_at,
              ]);

          }

        }


3.0.0 Fonts

How add to image fonts?
in 4.0.0 i saw fonts to support emoji. but what do with 3.0.0?
added to base_3 fonts. built. but emoji not work

Emoji work in Ubuntu 18.04+ and 3.0.0 is build based on Ubuntu 16.04.

Closing because of inactivity.


Allow zooming in Editor Help

This is a clean, rewritten version of https://github.com/godotengine/godot/pull/37196.
Compared to that PR, it introduces a global zoom button at status bar:

(I did not add a co-author because I've almost redone it from scratch)

Closes https://github.com/godotengine/godot-proposals/issues/1914

This needs to be redone differently - by implementing a zoom factor property to the RichTextLabel. The proper change is hard to make because all BBCode elements must be complained a zoom level.


Provide our own CookieJar implementation

The JavaNetCookieJar is a hazard to use, particularly because it uses HttpCookie behind the scenes. That class unnecessarily quotes cookie values, and we need to do gross things to strip them off. This behavior is particularly bad because sometimes it doesn’t strip the quotes and we end up stripping off quotes that we shouldn’t.

Is it possible add QuotePreservingCookieJar.java as a replacement for JavaNetCookieJar.java
https://gist.github.com/swankjesse/e283f2ddf264abd9a0d45bc80e7ec2d8

Yeah, just copy it into your codebase.

Could you release a new version okhttp3? React-Native need QuotePreservingCookieJar.java to address Issue 10121.

We will not be adding QuotePreservingCookieJar.java to OkHttp. You’ll have to copy it into your own project.

Yeah for sure. The most difficult part is deciding how the persistence model works. I’d love to have Sqlite but sadly we don’t. What do you think?

Ahh, I see. So this should support automatic disk persistence, in addition to in memory.
RE: SQLite - I'm still getting up to speed so I'm not entirely sure the role there. My hunch is that would provide faster access when reading cookies from disk on a fresh load. I wonder if a simple index would work there. I don't know enough yet on the access patterns, so maybe it wouldn't be enough.

I, from my perspective as a user, would be very happy with an in-memory implementation as a first addition.

There’s a handful of interesting problems we’d be able to solve with a java.util.Properties style interface:

cookies
HPKP
remembering which hosts support HTTP/2?
remembering proxy precedence?

Maybe that’s sufficient to start?

Alright, I'll attempt to draft something using the CookieJar as a first client and see where that takes me.

I did some experimenting here and realized, would it make sense to back this with DiskLruCache? It seems any key-value like storage would behave in a similar way:

Store arbitrary data with a given key.
Set an upper bound on the size of the store.
Have a policy for evicting entries when size exceeds upper bound.
Handle faulty filesystems.

All of these are supported by DiskLruCache.

You’re blowing my mind a bit, I never thought of that as an option! Yeah, for sure.
Keys would be domain names?

In case it can be of any help you can check my implementation of a PersistentCookieJar.
Some problems that I see with it for the moment:

It uses SharedPreferences to store the cookies so is not platform independent.
It loads all cookies in memory, maybe not problematic with the majority of apps but I suppose it won't scale well with cookie intensive usage (a browser for example).

Both problems can be solved with your proposals. For example using the DiskLruCache and storing all cookies of the same domain(ignoring subdomains) into .properties files and deserializing the .properties into a collection in memoery after the first call to loadForRequestUrl() for any url of the domain.

@swankjesse yes, that's my thought right now.
Thanks @franmontiel, I'll have a look.

With the release of 4.x, where does this issue stand?

I use this implementation to store cookies in memory (Inspired by PersistentCookieJar).
class MemoryCookieJar : CookieJar {
    private val cache = mutableSetOf<WrappedCookie>()

    @Synchronized
    override fun loadForRequest(url: HttpUrl): List<Cookie> {
        val cookiesToRemove = mutableSetOf<WrappedCookie>()
        val validCookies = mutableSetOf<WrappedCookie>()

        cache.forEach { cookie ->
            if (cookie.isExpired()) {
                cookiesToRemove.add(cookie)
            } else if (cookie.matches(url)) {
                validCookies.add(cookie)
            }
        }

        cache.removeAll(cookiesToRemove)

        return validCookies.toList().map(WrappedCookie::unwrap)
    }

    @Synchronized
    override fun saveFromResponse(url: HttpUrl, cookies: List<Cookie>) {
        val cookiesToAdd = cookies.map { WrappedCookie.wrap(it) }

        cache.removeAll(cookiesToAdd)
        cache.addAll(cookiesToAdd)
    }

    @Synchronized
    fun clear() {
        cache.clear()
    }
}

class WrappedCookie private constructor(val cookie: Cookie) {
    fun unwrap() = cookie

    fun isExpired() = cookie.expiresAt < System.currentTimeMillis()

    fun matches(url: HttpUrl) = cookie.matches(url)

    override fun equals(other: Any?): Boolean {
        if (other !is WrappedCookie) return false

        return other.cookie.name == cookie.name &&
            other.cookie.domain == cookie.domain &&
            other.cookie.path == cookie.path &&
            other.cookie.secure == cookie.secure &&
            other.cookie.hostOnly == cookie.hostOnly
    }

    override fun hashCode(): Int {
        var hash = 17
        hash = 31 * hash + cookie.name.hashCode()
        hash = 31 * hash + cookie.domain.hashCode()
        hash = 31 * hash + cookie.path.hashCode()
        hash = 31 * hash + if (cookie.secure) 0 else 1
        hash = 31 * hash + if (cookie.hostOnly) 0 else 1
        return hash
    }

    companion object {
        fun wrap(cookie: Cookie) = WrappedCookie(cookie)
    }
}

Usage:
val okHttp = OkHttpClient.Builder()
    .cookieJar(MemoryCookieJar())
    .build()

I would also like to see OkHttp's own CookieJar implementation. I found another issue with JavaNetCookieJar on Android caused by an Android-only change to java's CookieManager class which forced me to create my own simple CookieJar. https://issuetracker.google.com/issues/174647435

@headsvk yikes. Looks like they inexplicably introduced a regression.

Arguably this paragraph in the spec says that a Set-Cookie with non matching path can be rejected

Cookies do not always provide isolation by path.  Although the network-level protocol does not send cookies stored for one path to another, some user agents expose cookies via non-HTTP APIs, such as HTML's document.cookie API.  Because some of these user agents (e.g., web browsers) do not isolate resources received from different paths, a resource retrieved from one path might be able to access cookies stored for another path.

We'll probably punt this beyond 3.10. Getting this right is challenging because OkHttp can't use Android-only APIs. I recommend the persistent cookie jar code linked above if you're targeting Android and not the JVM.

Any opinion as of 4.9.0 of whether you'd be happy with an okhttp-android module using Android APIs here.  I think we have test infra now (locally, not CI yet) to build against this correctly.

The feature we want is sqlite storage, or perhaps even MySQL storage. Doing a module that uses SQDelight to store cookies to a DB would be a nice win. Could also do similarly for HSTS, for example.

@swankjesse is this worthwhile for 5.0? We can now use android APIs.

Not for 5.0. Good for the backlog.


Edit mentor profile

Description
The PR is to enable mentor to edit his profile information
Fixes #298
How to test?
It can be tested with postman and by running the project
Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] This PR is ready to be merged and not breaking any other functionality

wrong thinking


How to disable the white dot in indicator?

In the layout xml you defined SliderLayout, add the line
custom:indicator_visibility="invisible"

@husayndharshi : it wasn't working that way for me. so what i did was
    @BindView(R.id.home_slider)
     SliderLayout offersSliderLayout;

and then setIndicatorVisibility
     offersSliderLayout.setIndicatorVisibility(PagerIndicator.IndicatorVisibility.Invisible);


Git commit put comments in git message

Steps to reproduce

Make some changes and stage them.
:NeogitStatus
cc
Type any text
:w
:q

Your commit name will look in the following way: asdf # Please enter the commit message for your changes. Lines starting # with '#' will be ignored, and an empty message aborts the commit.

This doesn't happen for me 🤔

Do you have a commit message template?

Wow, sorry, it also caused by https://github.com/stevearc/stickybuf.nvim
I'm so sorry, will contact the author.


Feat: Native SDK unit tests support framework

JSON engine with validator
Temporary mock transport
Dependency updates

b'## Copyright scan failure
Commit: ca3cdcde950b28cbb756dbf663b62e80d0f604d2
Report detail: https://gist.github.com/rdkcmf-jenkins/96c2fcd08b623a3f32e8e0969be02d81'

b'## Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/firebolt-openrpc/219/rdkcentral/firebolt-openrpc

Commit: ca3cdcde950b28cbb756dbf663b62e80d0f604d2

Report detail: gist'

b'## WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: ok

Commit: ca3cdcde950b28cbb756dbf663b62e80d0f604d2
'

Since this PR is outdated, moved changes to https://github.com/rdkcentral/firebolt-openrpc/pull/222 which is based off of latest next branch. All review comments in this PR are addressed under the new PR (review comments fix commit)
Closing this PR


fix: Upgrade Vue version

General

✏ Mark the necessary items without changing the structure of the PR template.

Fix: https://github.com/vbenjs/vue-vben-admin/issues/3022

[x] Pull request template structure not broken

Type

ℹ What types of changes does your code introduce?

👉 Put an x in the boxes that apply

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

Checklist

ℹ Check all checkboxes - this will indicate that you have done everything in accordance with the rules in CONTRIBUTING.

👉 Put an x in the boxes that apply.

[x] My code follows the style guidelines of this project
[x] Is the code format correct
[x] Is the git submission information standard?
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

直接升级到3.3？

我不敢确保任何地方都没有问题，我只是想尽可能的减少 Vue 版本差异。


Inconsistency of Email Notification Level and Email Address Required

Email Notification Level
List of alert levels to send email alerts in response to. Leave blank if you do not wish to receive email notifications. Must be double-quoted and comma separated.
Email Address
(Required) The email address to subscribe for alert messages.
It doesn't make sense to me to have an option in email notification level to leave it blank if we don't want notifications and then to also require that an email address be present.
also if I do try to leave it blank for Email Notification Level I get the following error:
Template format error: Unresolved resource dependencies [SNSTopic] in the Resources block of the template
so basically it an Email Notification Level appears to be required.

Hi @cougar2009, and we apologize for the inconvenience. We are currently researching the issue at our end and  working towards the resolution. Please continue to monitor the repository for updates.

Thanks for your feedback. Essentially, when you don't want email notifications, you should leave Email Notification Level and Email Address blank. However, there is bug in the template which results in an undesired dependency https://github.com/awslabs/aws-limit-monitor/blob/1dcd25b9ffbb2e8b2f230bd6650dc236fb779b5e/deployment/limit-monitor.template#L436
We need to add condition for this resource as well.
Condition: SNSTrue
We will make this change with next push.


Make sfu-to-sfu talk MSC3401 and add new features

Fixes https://github.com/matrix-org/sfu-to-sfu/issues/13
Implements MSC3401 SFU bits
Requires https://github.com/matrix-org/matrix-js-sdk/pull/2423
Requires https://github.com/mautrix/go/pull/77

We'll also want to hook up logrus or whatever people use now at some point soon, but probably not in this PR.

Yep, that has been my thoughts since the fist time I actually need to debug it based on logs :P
Sorry, for the Go newbie mistakes, hopefully, it looks better now


Create issue_report.txt

Sorry for what looks like a PR but is actuall an issue.
There is no way to report issues on your repo...
MM  v2.16.0 no longer starts for me because of your module:
 [02.07.2021 15:04.54.518] [ERROR] Error: Cannot find module 'request'                                                                                |
| Require stack:                                                                                |
| - /opt/magic_mirror/modules/MMM-homeassistant-sensors/node_helper.js

The runtime dependency request was removed from core MagicMirror in version v2.16.0 (and replaced because it is deprecated).
Can you please fix your repo accordingly?
(as a workaround I had to manually install request (npm install request)
Thanks

@lolouk44
since I don't own a magicmirror any longer, I cannot currently invest any effort in this old project.
A PR is welcome, and I will merge it to my repo.
Meanwhile installing request via npm is a good workaround

Thanks. I'll fork out and see if I can make the changes myself. I'll post a note here if I manage it in case you want to send other my way.

Closing as created an actual PR: https://github.com/leinich/MMM-homeassistant-sensors/pull/10


gazebo_ros_pkgs: 2.4.13-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository gazebo_ros_pkgs to 2.4.13-0:

upstream repository: https://github.com/ros-simulation/gazebo_ros_pkgs.git
release repository: https://github.com/ros-gbp/gazebo_ros_pkgs-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.24
previous version for package: 2.4.12-0

gazebo_msgs

No changes

gazebo_plugins
* Fix timestamp issues for rendering sensors (indigo-devel) (#551 <https://github.com/ros-simulation/gazebo_ros_pkgs/issues/551>)
* Contributors: Ian Chen, Jose Luis Rivero

gazebo_ros

No changes

gazebo_ros_control

No changes

gazebo_ros_pkgs

No changes

//cc FYI @iche033


Adapted codebase function Open to use che client PATCH api call

related to this new API call in che-starter: https://github.com/redhat-developer/che-starter/issues/134
We have an operation Open that should now call use PATCH workspace/{name}.

@aslakknutsen I'll update the PR with the last changes.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@225368a). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master    #1245   +/-   ##
=========================================
  Coverage          ?   59.51%
=========================================
  Files             ?      120
  Lines             ?    10578
  Branches          ?        0
=========================================
  Hits              ?     6296
  Misses            ?     3630
  Partials          ?      652

Impacted Files
Coverage Δ

codebase/che/client.go
0% <0%> (ø)

controller/codebase.go
27.77% <0%> (ø)

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 225368a...e9cdd7c. Read the comment docs.

@aslakknutsen this PR has been merged redhat-developer/che-starter#134


Update Reference to IETF OAuth Token Status List

Two approvals and this is a minor update. Will merge now.


Limited English speaking population

Description:  The number and percent of people 5 years and over that speak English less than very well for three time periods broken out by race and ethnicity.
Sources:  Census, ACS PUMS
Years:

[ ] 2000
[ ] 2008-2012
[ ] 2015-2019

Geographies:

[ ] PUMA
[ ] Borough

Denominator: Total pop >5 who speak a language other than English at home
Race/Ethnicity breakout: Yes
Roablock:

Issue with 2008-2012 PUMS data

This is the indicator I will use to test the addition of 2008-2012 PUMS data

This indicator is to be updated with the new custom denominator process


infinite-scroll: Cannot read properties of undefined (reading 'append')

Please ensure:

[X] The bug is caused by Refined GitHub. It doesn't happen if I disable the extension.

Example URLs
https://github.com/
Browser(s) used
Google Chrome 97.0.4692.71 (64-bit)
Description
TypeError: Cannot read properties of undefined (reading 'append')
    at chrome-extension://hlepfoohegkhhmjieoechaddaejaokhf/build/refined-github.js:5223:45
    at chrome-extension://hlepfoohegkhhmjieoechaddaejaokhf/build/refined-github.js:1923:97
    at runFeature (chrome-extension://hlepfoohegkhhmjieoechaddaejaokhf/build/refined-github.js:2924:26)
    at setupPageLoad (chrome-extension://hlepfoohegkhhmjieoechaddaejaokhf/build/refined-github.js:2934:40)
    at chrome-extension://hlepfoohegkhhmjieoechaddaejaokhf/build/refined-github.js:2956:35

Console error:
refined-github.js:2886 ❌ infinite-scroll 22.1.11 → TypeError: Cannot read properties of undefined (reading 'append')
    at refined-github.js:5223:45
    at refined-github.js:1923:97
    at runFeature (refined-github.js:2924:26)
    at setupPageLoad (refined-github.js:2934:40)
    at refined-github.js:2956:35

Include in this issue:

[ ] Screenshots/video/gif demonstrating the bug, if it’s visual
[X] Console errors, if any

It seems that the "Explore" sidebar fails to load once in a while

! -> ? then

It seems that the "Explore" sidebar fails to load once in a while

! -> ? then

That would just silence the error. I think it'd be better to add a fallback to the selector, e.g. [aria-label="Explore"], aside

It probably would look weird but OK

Weird, I was only able to reproduce this a single time. It seems that the "Explore" sidebar fails to load once in a while, which causes this line to throw an error:

Personally, the "Explorer" sidebar always worked, but yesterday, it didn't and that's how I encoutered this. Even in Incognito, no extensions, still didn't show.


Views fail to index with timeout prior to os_process_timeout elapsing

I'm running into an issue getting an instance of CouchDB stood back up after a restart. It was running fine prior to the restart, but now seems to be running into an issue when building the views from scratch. I've since played with the configuration a bit to try to debug.
You can see the full logs of my situation here, with some sensitive bits redacted out.
Some notable lines from the logs:

os_process_timeout is set to 600000, an absurdly high, not real-world value: https://gist.github.com/javamonn/ed8ee8c778471c18ba1fb13433ed3300#file-gistfile1-txt-L38
There's an early [os_mon] error, prior to indexing kicking off that seems important but I can't find much information about it: https://gist.github.com/javamonn/ed8ee8c778471c18ba1fb13433ed3300#file-gistfile1-txt-L244
Indexing seems to time out after 5 seconds, ignoring the os_process_timeout I've set. begin, end.

Expected Behavior
View indexing should respect os_process_timeout and eventually complete, or give a more appropriate error message if there is another error.
Current Behavior
View indexing times out prior to os_process_timeout, indexing in Futon diplays stuck at 0%.
Possible Solution
Is there possibly another error occurring, and CouchDB is swallowing it with "OS process timed out."? https://issues.apache.org/jira/browse/COUCHDB-2431 seems to imply this could be the case. I've tried adding some logging to handleError in main.js to no avail, is there somewhere else where added logging or a log file would increase visibility into the error? I have SSH access to the machine and seem to be able to execute the same couchjs invocations that couchdb does.
Context
This instance is deployed to Google Container Engine using an image close to the official image. I can't reproduce this locally with an image close to what I have deployed, but I haven't tried locally with a backup of the production data. The instance has 5 databases (including _users and _replicator, the largest one of which is about 59MB total, with 151993 documents.
Your Environment

Version used: 2.1.1
Browser Name and version: N/A
Operating System and version (desktop or mobile): Linux <REDACTED> 4.4.86+ #1 SMP Thu Dec 7 20:11:11 PST 2017 x86_64 GNU/Linux
Link to your project: N/A

I've just noticed the [cluster] section from the logs. I'm actually only running a single node, so the default values of n = 3 is incorrect. I've updated to n = 1 and redeployed but that doesn't seem to have affected the outcome.
I've added some logging to main readline() loop in main.js and I'm not seeing any output on view creation, which suggests that something more fundamental is messed up with my install, as the query server is just hanging.
I've found http://docs.couchdb.org/en/latest/install/troubleshooting.html and am going through it now.

@javamonn
You're right in that the cpu_sup process failing suggests that your instance is heavily CPU constrained. The 5 second timeout may mean that couchjs is failing to even start up correctly - you're not even getting an initial "hello" handshake back from that spawned process.
Are you able to increase the CPU allocation for your instance? If so, please do. CouchDB runs best with 4 or more cores allocated.
We are tracking numerous situations in which CouchDB deployed in containers is simply unreliable. We don't have enough people to debug these scenarios right now. So far, in most cases it's simply too few CPU or RAM resources being allocated.
If you need a guaranteed stable CouchDB we recommend deploying it in the traditional fashion on dedicated VMs or bare metal if possible.

@wohali incredible diagnosis, bumping the CPU allocation I was giving the container did the trick. Now I'm not sure how the instance was running at all prior to the restart taking it offline.
I was giving it about a core originally. I bumped it up to 2 cores, and the cpu_sup error went away, but indices would still timeout after a couple of seconds. I bumped it up to 4 cores and it's humming away now.
Are the hardware recommendations documented anywhere? I found similar recommendations on Couchbase, but can't find any on CouchDB proper.
Thanks for the help.


Remove unreachable-code feature

Summary
We haven't found time to flip this on, so feels like it's best to remove it for now -- can always restore from source when we get back to it.

Ah yeah, it's not motivated by anything specific, I'd just prefer to keep unused code in source rather than on main now that it's been a while.


Use expm1 in elu

expm1(x) is more accurate than exp(x) - 1 when x is nearly, but not
exactly, zero.
In the case of elu, we would compute exp(x) - 1 when x is <= 0. If x is
negative and has a very small magnitude, computing exp(x) - 1 would
round to zero.
For example, if x was -1.0E-8 then:
exp(x) - 1 is 0 but expm1(x) is -1.0E-8

Thanks for catching this!


[DOCUMENTATION] Create Manual Testing file

Also has a minor bug fix with LocateCommand

Coverage decreased (-0.01%) to 81.026% when pulling f62141f034273a8d420705701241f69d287dd9aa on jeffreygohkw:master into 586d057b29f31d18ab41b2f047be9c6f4b7e2265 on CS2103AUG2017-W09-B2:master.


Nextachiev

This closes #4

Could you provide more documentation on how this closes #4?

The initial question/request of #4 was to be able to show when buying x3 of building B was better than buying x1 of building A even if building A had a lower PP than building B. Such situations occur when achievements can be earned by buying B x3.
With the changes to colour code the x10 section green when it is better than the x1 section and the tooltip now displaying how much of a building needs to be bought for the next achievement such information is now accessible. After adding #766 all relevant information will be in the tooltip.

Ah I see.
Does cookie monster (now?) handle situations where the overall PP for
buying  a building upgrade for building A then a few buildings A is more
effective than just buying building B (although building B's PP is better
RIGHT NOW).
On Sun, Apr 11, 2021 at 2:16 PM Daniël van Noord @.***>
wrote:

The initial question/request of #4
https://github.com/Aktanusa/CookieMonster/issues/4 was to be able to
show when buying x3 of building B was better than buying x1 of building A
even if building A had a lower PP than building B. Such situations occur
when achievements can be earned by buying B x3.
With the changes to colour code the x10 section green when it is better
than the x1 section and the tooltip now displaying how much of a building
needs to be bought for the next achievement such information is now
accessible. After adding #766
https://github.com/Aktanusa/CookieMonster/issues/766 all relevant
information will be in the tooltip.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Aktanusa/CookieMonster/pull/765#issuecomment-817358096,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHST53RSBLAIAZG47UZ2ECDTIHYRRANCNFSM42VBCXRQ
.

--
Kevin Anderson

No it doesn't (sadly). Upgrade PP's are not considered when coloring PP's.


Displayed MyAccount path of a suborganization should be updated

Describe the issue:

Displayed MyAccount path of a suborganization should be updated

How to reproduce:

1.Login to the console
2.Create a suburbanization
3.Switch into a sub organization
4.Go to applications section and check the my account URL path displayed
Expected behavior:

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

This feature should be supported along with the completion of Sub Org user login support for My Account & Console.
Issue: https://github.com/wso2/product-is/issues/18554

Fixed with PR: https://github.com/wso2/identity-apps/pull/5141


Remove the cqrs abstraction

What do you mean?

Right now, cqrs is a facade over both event_store and command_bus. By grouping them it allowed for building/tracking some relations between events and commands. However, this concept was confusing to many new people and as such it brings more problems than benefits.
The whole idea of tracking events/commands dependency is still valid and needed. We just need to find other ways to do it.
Removing the cqrs object is already in progress. We replace the cqrs with both event_store and command_bus, explicitly being passed where needed.
A bit cumbersome refactoring, but thanks to good tests it's going well.

I see that the issue title could be confusing, I changed the name. Thanks!

The CQRS facade is now gone (▀̿Ĺ̯▀̿ ̿)


Make it possible to specify log file

https://github.com/Shopify/shopify-php-api/blob/0556edf274ceb83346533bd22b662d7622e58c66/src/Clients/Http.php#L275
Hi
I am using this package to integrate Shopify in one of my apps. I would prefer if it were possible to provide custom file (and path) for logging errors, maybe probably through Context object.

I am too, and just not giving the last argument (thus it being null) works just fine for me.

@MagicLegend  The file created is not a log, but a timestamp to check when the last log of a deprecated api method is sent.
Currently there is no way to disable it, or set the path.
It is created in the same folder as the Http class, which isn't great in a cloud context.
Anyway, in general we do not write in the vendor folder.


Fix default value of JobConf.ExecutorJsonPath

Since #616 is merged, the default value for named parameter JobConf.ExecutorJsonPath.class should be tests/src/main/resources/sample_executor_resources.json, but it's  src/main/resources/sample_executor_resources.json now.

Close via #797


[#2426] Hide Anymail errors

This fixes #2426 by hiding Anymail errors.
The original bug report is caused by test-amy2 attempting to send email without authorization. There's a couple of ways to fix this:

disable sending emails in test-amy2 (through AMY_LIVE_EMAIL env variable)
catch the exceptions where they are caused (this is the way I chose)
set up test-amy2 in a way that it can send real emails
switch to sandbox/test-only mode in test-amy2 + mailgun (probably solution that would take the most time).

I wanted to fix this issue quickly, and I didn't want to disable sending emails altogether since we may want to do this in future. So I decided on solution no. 2.

Actually, are there any cases where this exception might be raised in production?

Actually, are there any cases where this exception might be raised in production?

@elichad Possibly if our Mailgun account expires, or they have an outage or something.


Implement correct interrupt handling

Follow up to https://github.com/square/okhttp/pull/7185
Also related to issues like https://github.com/square/okhttp/issues/3146
Specifically while the callers thread is interacting with the socket, typically via call.execute()

@yschimke Did you try using latest version of okHttp (4.xx)?

Issue is still actual, I have tried with 4.12 and also with latest beta versions


filtering ModelSelect2MultipleWidget by field owner [Question]

I have some classes:
class AccountSearchFieldMixin(object):
    search_fields = [
        'login__icontains',
        'password__icontains',
        'phone__icontains',
    ]

class AccountSelect2TagWidget(AccountSearchFieldMixin, ModelSelect2MultipleWidget):
    model = VkAccount
    queryset = VkAccount.objects.exclude(proxy__isnull=True)

    def create_value(self, value):
        self.get_queryset().create(name=value)

class TaskVkInvitationsFriendsSearchForm(forms.ModelForm):
    # country = ModelChoiceField(Countries.objects.all(), empty_label=None, label='Страна')f

    class Meta:
        model = TaskVkInvitationsFriendsSearch
        exclude = ['type', 'date_creation', 'date_update', 'maintask', 'invited_friends']
        widgets = {
            'accounts': AccountSelect2TagWidget,
            'cities': CitiesSelect2TagWidget,
            'scheduler_data': forms.HiddenInput,
            # 'country': CountrySelect2TagWidget
        }

I want filtering widget['accounts'] by request.user as account owner (Foreignkey)
Can you explain me, how can i do this?

Hi @steeldimon,
thanks for reaching out. I hope that I can help you. Let's start with AccountSearchFieldMixin. You inherit from object which is uncommon in Python 3. Please beware that this package does not support Python 2. With that out of the way, let's get started:
What you want to achieve is not entirely easy. You see, form, fields and widgets in general don't have access to the request (by default). So if you would want to access it in a form, you would need to pass it on manually. That being said, since the options for select2 model widgets are loaded dynamically, there is an option after all, to filter by request.
You will need to create your own customer view for this and override the AutoResponseView.get_queryset, seealso: https://github.com/applegrew/django-select2/blob/82859121a6285d212d6d42ec8a9b65cf02ee5991/django_select2/views.py#L50-L57
Here an example:
views.py
from django_select2.views import AutoResponseView

class AccountJsonView(AutoResponseView):

    def get_queryset(self):
        qs = super().get_queryset()
        if not self.request.user.is_authenticated:
            raise Http404
        return qs.filter(owner=self.request.user)

forms.py
from django import forms
from django_select2.forms import ModelSelect2Widget

class MyForm(forms.Modelform):
    class Meta:
        widgets = {
            'account':  ModelSelect2Widget(data_view='account_json')
         }

urls.py
from django.conf.urls import url

from .views import AccountJsonView

urlpatterns = [
    url(r"^fields/account.json$",
        AccountJsonView.as_view(), "account_json"),
]


Merge NK branch into master

Once we finish the general expectations work, we should merge NK into master so that we start getting more "alpha testers" using that branch.

OK, updated this file.  See the pull request log for specifics.

Done by #138


ModuleNotFoundError: No module named 'myeia'

Greetings,
I am trying to implement myeia into my scripts. Starting with a test script replicating your instructions, the error message is:
ModuleNotFoundError: No module named 'myeia'
When examining the location of the files, they are placed in:
/usr/local/lib/python3.11/site-packages/src
Changing /usr/local/lib/python3.11/site-packages/src to /usr/local/lib/python3.11/site-packages/myeia, the following error is returned:
from myeia import API
ImportError: cannot import name 'API' from 'myeia' (/usr/local/lib/python3.11/site-packages/myeia/init.py)
The file init.py is empty. It should contain something like
from .myeia import API, version
No?
Much thanks for your effort and attention.
Max
p.s. my operating environment is Fedora Linux 37; except for two python modules (entsog and entsoe) installed wity pip, all the rest are installed via dnf/rpm.

Continuing ...
I've:
moved src to myeia
modified init.py to contain:
from .myeia import API, version
modified myeia.py like this:
from pandas.core.common import SettingWithCopyWarning
from pandas.core.generic import SettingWithCopyWarning
version = "0.2.0"
and now the error is:
File "/usr/local/lib/python3.11/site-packages/myeia/myeia.py", line 54, in get_data
response.raise_for_status()
File "/usr/lib/python3.11/site-packages/requests/models.py", line 1021, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 403 Client Error: Forbidden for url: https://api.eia.gov/v2/natural-gas/pri/fut/data/?api_key=None&data[]=value&frequency=daily&facets[series][]=RNGC1&sort[0][column]=period&sort[0][direction]=desc
Somehow, API key isn't being picked up; from the url " ... ?api_key=None ...".
Max

I think its fixed now, can you check if it works for you with version 0.2.2?

Here's the error that I receive:
Traceback (most recent call last):
File "/home/pyz/projects/Commodities/DataSources/EIA/Development/Electricity/./GetMyEIA.py", line 5, in
from myeia import API
ImportError: cannot import name 'API' from 'myeia' (/usr/local/lib/python3.11/site-packages/myeia/init.py)
init.py is empty (size 0).
I believe that it should have a line something like this:
from .api import API
When I run my script now, it returns the following error:
df = eia.get_data(
^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/myeia/api.py", line 48, in get_data
response.raise_for_status()
File "/usr/lib/python3.11/site-packages/requests/models.py", line 1021, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 403 Client Error: Forbidden for url: https://api.eia.gov/v2/natural-gas/pri/fut/data/?api_key=None&data[]=value&frequency=daily&facets[series][]=RNGC1&sort[0][column]=period&sort[0][direction]=desc
It fails to pass the API key to the building of the URL.
Max

Studying your work, here's my script that for the most part is successful.
#!/usr/bin/python3
import requests
import pandas as pd
eiaURLData = 'https://api.eia.gov/v2/seriesid/{}?api_key={}'
eiaAuthToken = "<YourEIAAPITokenHere"
eiaSeriesID =  'PET.MD0MP_R10-R20_1.M'
eiaSeriesID =  'INTL.81-2-ARE-TBPD.A' (For some reason this doesn't work)
url    =  eiaURLData.format(eiaSeriesID, eiaAuthToken)
print(url)
eiaData = requests.get(url)
print(eiaData.status_code)
eiaDataJSON = eiaData.json()
baseEIAData = pd.DataFrame(eiaDataJSON["response"]["data"])
print(baseEIAData.head())
baseEIAColNames = list(baseEIAData)
eiaPrDate = baseEIAColNames[0]
eiaValues = baseEIAColNames[-2]
eiaUnits = baseEIAColNames[-1]
df = baseEIAData[[eiaPrDate, eiaValues]]
print(df.head())
eiaDelimFile	= "%s.txt" % (eiaSeriesID, )
df.to_csv(eiaDelimFile, sep='\t', header=False, index=False)

from myeia import API

The empty init.py already identifies it as a package
so in that case you would need to do from myeia.api import API.
the naming is a bit confusing I understand.
For the series-id: PET.MD0MP_R10-R20_1.M
You would first need to identify the "route" for the new API v2.
I can highly recommend you to use the API dashboard to find all routes first: https://www.eia.gov/opendata/browser/
The route in that case is: "petroleum/move/pipe"
and the series: "MD0MP_R10-R20_1" <- is the series-id from the old apiv1
So your script would start like:
from myeia.api import API
eia = API()  # set your API KEY in your .env where the .env.example is located

if __name__ == "__main__":
  df = eia.get_data(route="petroleum/move/pipe", series="MD0MP_R10-R20_1", frequency="monthly", facet="series")
  print(df.head())
  df.to_csv(<your_file_path>)

My script is set up in a way that it only returns the value of a given series, so if you want to adjust that to get for example the "units" or "process" you can fork or copy my code and adjust the following part of the api.py file:
if facet == "series":
    df = base_df[["period", "value", "series-description", "series"]]

elif facet == "seriesId":
    df = base_df[["period", "value", "seriesDescription", "seriesId"]]

I hope this helped!

For the series-id: "INTL.81-2-NIC-TBPD.A"
There seems to be no data available: https://www.eia.gov/opendata/v1/qb.php?sdid=INTL.81-2-NIC-TBPD.A

Here's the error that I receive: Traceback (most recent call last): File "/home/pyz/projects/Commodities/DataSources/EIA/Development/Electricity/./GetMyEIA.py", line 5, in from myeia import API ImportError: cannot import name 'API' from 'myeia' (/usr/local/lib/python3.11/site-packages/myeia/init.py)
init.py is empty (size 0).
I believe that it should have a line something like this: from .api import API
When I run my script now, it returns the following error: df = eia.get_data( ^^^^^^^^^^^^^ File "/usr/local/lib/python3.11/site-packages/myeia/api.py", line 48, in get_data response.raise_for_status() File "/usr/lib/python3.11/site-packages/requests/models.py", line 1021, in raise_for_status raise HTTPError(http_error_msg, response=self) requests.exceptions.HTTPError: 403 Client Error: Forbidden for url: https://api.eia.gov/v2/natural-gas/pri/fut/data/?api_key=None&data[]=value&frequency=daily&facets[series][]=RNGC1&sort[0][column]=period&sort[0][direction]=desc
It fails to pass the API key to the building of the URL. Max

I tested version 0.2.2 on my WSL 20.04.5 Ubuntu with python 3.10.9 and it should pick up the API KEY from the .env file.

myeia's design requires three pieces of information:

route
seriesid
frequency.

The script that I offered only requires the seriesid. It's a little more efficient; but you need to breakdown EIA's offerings to isolate the seriesid.
And then hopefully, they won't deprecate the series ids.

This APIv1 method is now deprecated .However, if you are fond of APIv1 Series IDs, you may invoke them using the special /series/ route, as follows. Note that we use a full route, but at the end we include the APIv1 Series_ID:
https://api.eia.gov/v2/seriesid/ELEC.SALES.CO-RES.A?api_key=xxxxxx

I see they added now the old v1 structure to v2. Yeah that would work fine.
I might add that to my script soon.


dnsproxy fails with "cannot assigned requested address" under load

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
With enough dns traffic, cilium starts erroring with cannot assigned requested address
Cilium Version
1.14.6 4a4fa058 2024-01-17T20:45:58+02:00 go version go1.20.12 linux/amd64
Kernel Version
5.15.0-1052-aws #57~20.04.1-Ubuntu SMP Mon Jan 15 17:04:56 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Kubernetes Version
v1.25.16-eks-5e0fdde
Sysdump
No response
Relevant log output
No response
Anything else?
This is related to https://github.com/cilium/cilium/issues/29175
Specifically this change: https://github.com/cilium/cilium/pull/29239
Code of Conduct

[X] I agree to follow this project's Code of Conduct

How much is "enough" traffic? Do you know of a way to reproduce this in a test setup?

How much is "enough" traffic? Do you know of a way to reproduce this in a test setup?

In the previous incarnations, I wasn't able to synthetically reproduce it easily. But we have a couple clusters were it reliable happens.

P.S. Was this OK on previous versions of Cilium?
v1.14.3 + v1.14.5 are fine
v1.14.4 + v1.14.6 have issues

Given that you are calling out the new transparent proxy functionality as a likely reason for the problem, have you tried changing the value of the flag with --dnsproxy-enable-transparent-mode (or dnsProxy.enableTransparentMode in Helm values)?

Given that you are calling out the new transparent proxy functionality as a likely reason for the problem, have you tried changing the value of the flag with --dnsproxy-enable-transparent-mode (or dnsProxy.enableTransparentMode in Helm values)?

Sorry, I missed that this was behind a flag now. Let me disable TPROXY mode when rolling forward again.

Disabling transparent mode has fixed the errors.

Should the issue be closed?

Should the issue be closed?

Sorry. Yes, this should be closed.


Bad encoding on export

Saving a Profile locally results in an Encoding that is not compatible with tools like Prosys Forge. Changing the text encoding from UTF-8-BOM to UTF-8 would solve this issue.

Although the text in the XML claims UTF-8, examining the document with a tool like Notepad++ shows its not actually encoded that way.

This appears to happen when trying to download a profile that the user has created within Profile Designer. A fix has been applied and is in stage site for testing.
Note: When downloading a Cloud library profile, this does not appear to be a problem because that type of download is likely pulling the original XML file produced by Cloud Library.
Note: I did confirm in notepad++ that the encoding is correct but did not test in Prosys Forge.

Deployed to live site. Marking done.


"Florești" is listed in the drop-down menu

Description:
When user writes "Florești" the location appears in the drop-down menu next to the comune and the county which is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Florești"
Expected results
The location appears in the drop-down menu as "Comuna Florești, CLUJ"


Xcode 9 errors

*Type 'NSAttributedStringKey' (aka 'NSString') has no member 'font'
*Type 'NSAttributedStringKey' (aka 'NSString') has no member 'foregroundColor'
*Cannot convert value of type '[NSAttributedStringKey : AnyObject]' (aka 'Dictionary<NSString, AnyObject>') to expected argument type '[String : Any]?'

@arashetm you need to rollback to the version that supports your project's swift version.. can be found in setting-> swift language version. This errors are because NSAttributedStringKey was introduced in Swift 4.

I have the same problem but i got swift language version 3.2 in my settings. Is it because i have Xcode 9.1?

i have resolved my issue


Form Model Refactor

Refactor the ReportSummaryForm component to use an object as a model instead of separate props.

Merge async / await tests from master
Refactor form state
Fix tests

Looks good. Definitely an improvement. Thanks for taking it on.


Dont warn that learn isn't marked as an arduino library

I noticed that in the learn repo, ci-arduino warns that "arduino-library is not found in this repo topics." Change the way the repos are selected to run this check so that this warning won't be printed there.

FYI: repo that tags as "arduino-libraries" is for rebuild repo when there is a new BSP released e.g SAMD, nrf52 or arduino-esp32
https://github.com/adafruit/arduino-board-index/blob/gh-pages/.github/workflows/trigger_libraries.yml
for example https://github.com/adafruit/Adafruit_TinyUSB_Arduino/actions/runs/4210118239 . Maybe consider to tag the learn guide as 'arduino-libraries' as well for it to be rebuilt.

@hathach i did both!


DSND-779: added required changes for invoking chs-kafka-api before de…

…leting charge

DSND-779: Added all required changes for invoking chs-kafka-api before deleting charges from Mortgages.
Added/modified ChargesService, ChargesAPiService and Unit tests.


Make it generic

Currently some of the code is specific to a Scala or programming project. But in the future, this could be used to visualize other domains, for example a set of Docker containers, build steps, maven dependencies.
This should be made more generic, especially the node and edge style classes.
One possibility could be to scan the entire graph on first load and dynamically generate the styles based on distinct node and edge types.
Or preset templates/config for type of project.

Some notes from @matanster in gitter discussion:
So for not-domain-specificity sake, this mapping there can be a custom mapping, whereas the default mapping would just select, arbitrarily, colors for edge types ― evenly spaced on the color spectrum ― per the number of edge types there is in the data. I think there's a unique method or something like it in ES6 for getting the number of unique values from the data, for getting the number of edge types.
So if there's say 3 edges, the color difference between each type can be huge, whereas if there's 8 or 9, the color differences are smaller, basically dividing over the pleasantly visible spectrum of color, while thinking of not using colors that are indiscernible from whatever background color there is. To aid in that, d3 has color interpolation functions, which help generate colors for such a technique.
(when I mention d3 there, I don't mean anything of d3 need be used, other than a color api function of your choice from it.. which may otherwise exist in other libraries/framework/native ES6


Add notes or info to users not in the server

We should be able to leave notes or issue warnings(if we end up making warnings go through regardless of them receiving them or not) as well as add reddit accounts through .verify even if the user is not in the server.

Thought I already did this reeeee

fixed in #59


Add jquery.tag-it definition.

Wrote a definitions file for the jquery widget tag-it: http://aehlke.github.io/tag-it/

jquery.tag-it/jquery.tag-it.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/jquery.tag-it
http://bower.io/search/?q=jquery.tag-it
others?

[ ] has a test file? (jquery.tag-it/jquery.tag-it-tests.ts or others)
[ ] pass the Travis-CI test?

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


Release v1.18.0-alpha.1

Fix Windows bugs (#718)
Add module id dynamic import instrumentation (#697)
Ensure that we don't report coverage for files in tests (#716)

!merge


Proximity placement group

Adding PPG as per #145

@xpillons just added PPG creation with VM example. will plan work for VMSS with PPG.


chore(main): release 1.1.0

:robot: I have created a release beep boop
1.1.0 (2024-01-23)
Features

theme: change light theme primary (#185) (f8ec977)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Arize-ai/ui-components/releases/tag/v1.1.0 :sunflower: