CSV export - date and time fields are in excel format

Dear all,
we use the "CSV Writer" class to transfer a file.
In the source table we have date and time fields.
The CSV Writer exports the fields in excel format.
e.g. Date 17.08.2023 => 45155
We found a possible solution for that:
When the field lv_data_type is set in the method SET_CELL of the class ZCL_EXCEL_WORKSHEET to "D" or "T" the exporter class handles the fields correct.
If this might by a solution, I will create a pull request
thanks
Matthias

Sorry for the delay. Are you sure there is a bug in the CSV writer? Could you post a minimal reproducible example so that we can check ourselves? Thank you!

z_abap2xlsx_issue_1134.txt
Here is an example for the problem.
In XLSX download the date field is set with 45155, which is converted by Excel to real date 17.08.2023. In the csv export the value is also set to 45155, which is no date. If the csv is opened without excel the field is a number and no date.
I think in csv export, the date field should by a date in real format (not the excel internally format).

Thank you. I agree 45155 is buggy in CSV. Your program will help me analyze, and I'll confirm after week-end, based on your latest comments, probably I'll agree on your PR and merge it.

Hi,
after your comment yesterday I debugged again.
Maybe another solution could help, but here I'm not familiar with the data which abap2xlsx is using.
in the method CREATE_CSV of class ZCL_EXCEL_WRITER_CSV there is a coding which is called "Retrieve Cell Style format and data type"
The field cell_style is filled in the structure but this coding did not fill the variable ls_numfmt.
Maybe we get here a correct style then the data_type is set here and must not be set in the method SET_CELL.

This is my suggestion:
The NUMFMT Field is retrieved from the dynamic styles. (excel->get_styles_iterator( ) and following WHILE Loop)
At the begining of zcl_excel_writer_2007->create( ) are the following lines

Start of insertion # issue 139 - Dateretention of cellstyles
me->excel->add_static_styles( ).
End of insertion # issue 139 - Dateretention of cellstyles

to create a dynamic style for any static style if not done so far.
Else some dynamic styles could be missing.
This method call is still not used at the begining of zcl_excel_writer_csv->create( ) and should be included there.

Hi All,
I changed the coding like darnoc mentioned.
This will solve my problem.
But nevertheless: Maybe for an future release. we should check, why the field data_type is not set correctly.
Happy to hear comments.
Matthias

Data_type is set correctly,
This field is the Excel data type with the following valid values:

From: https://itecnote.com/tecnote/excel-what-do-excel-xml-cell-attribute-values-mean/
Currently in abap2xlsx only space and 's' are used (space leads to default 'n').
The settings for this field in zcl_excel_writer_csv->create_csv differ from this list.
These are local settings for local purpose in a local copy of table zcl_excel_worksheet->sheet content and perhaps they look like ABAP Inttype (And the query for upper case letters seems to be unnecessary).
But if the destination is a XML Output the value of data type set by  zcl_excel_worksheet->set_cell must be according to the list above.

Starting the review. Posting another minimal example to reproduce + testing all supported ABAP types:
REPORT z_abap2xlsx_issue_1134.
DATA gc_save_file_name TYPE string VALUE 'abap2xlsx_issue_1134.csv'.
INCLUDE zdemo_excel_outputopt_incl.
PARAMETERS: p_csv  RADIOBUTTON GROUP rad1 DEFAULT 'X',
            p_xlsx RADIOBUTTON GROUP rad1.
START-OF-SELECTION.
  TRY.
      DATA(lo_excel) = NEW zcl_excel( ).
      DATA(lo_worksheet) = lo_excel->get_active_worksheet( ).
      types ty_ref_to_values type standard TABLE OF ref to data with EMPTY KEY.
      LOOP AT VALUE ty_ref_to_values( ( REF #( 'Hello' ) )
                                      ( NEW d( '20230930' ) )
                                      ( NEW decfloat16( '-3.14' ) )
                                      ( NEW f( '.3' ) )
                                      ( NEW int1( 3 ) )
                                      ( NEW int2( -314 ) )
                                      ( REF #( -314 ) )
                                      ( NEW sflight-price( '314.16' ) )
                                      ( REF #( `Hello` ) )
                                      ( NEW t( '151416' ) )
                                      ( NEW syhex02( 'FFFE' ) )
                                    ) INTO DATA(ref_to_value).
         ASSIGN ref_to_value->* TO FIELD-SYMBOL(<value>).
         lo_worksheet->set_cell( ip_column = zcl_excel_common=>convert_column2alpha( ip_column = sy-tabix + 0 ) " pass by value
                                 ip_row    = 1
                                 ip_value  = <value> ).
      ENDLOOP.
      DATA(lv_writerclass_name) = COND #( WHEN p_csv = 'X' THEN 'ZCL_EXCEL_WRITER_CSV' ).
      IF p_xlsx = 'X'.
        REPLACE '.csv' IN gc_save_file_name WITH '.xlsx'.
      ENDIF.
      lcl_output=>output( cl_excel            = lo_excel
                          iv_writerclass_name = lv_writerclass_name ).
    CATCH cx_root INTO DATA(lx_root).
      MESSAGE lx_root TYPE 'I' DISPLAY LIKE 'E'.
  ENDTRY.

Current bug:

After applying the proposed fix (#1140):

For information, XLSX format:


Add github templates and a CONTRIBUTING.md

Issue and pull-request templates to keep things clean and clear.

Coverage remained the same at 76.606% when pulling 3ee51cdd75f22404d7af8a394bb571feb283a102 on github-templates into 5bc0ea2ecccb64d175cff70b0a1e76a6f8362db6 on master.


Enable es3 extension methods for WWA

See github issue #618

Looks good.

So we are not getting rid of SupportES3Entensions altogether ?

annex b only have 2 String.prototype methods: substr and anchor.
What's the backward compat impact for existing wwa apps?

@akroshg SupportES3Entensions still guards getVarDate which is not a standard.
@Yongqu This is a known bug with existing WWA App which fails due to non-availability of  these string built-in. This issue\pull request is to address the same. I agree, this has a potential to break existing apps, though chances are remote. Is there a way to test the compat?

It is not worth testing compat for these. These exist on the web everywhere, so we're talking about some code breaking that exists in WWA only that is polyfilling these methods but in a way that isn't web compatible. I think this is fine to go in as-is.


If parent cant be found try creating prop again

Improvements

Setting parentPath will be more robust on most props

Kinda worried about the overhead on this, do we think using Find to look for the parent is expensive? We'd be doing it for every prop ever and in most cases it will not matter.

xen wanted me to review this, but my brain is too tired rn. will review in a few hours

grahhhh


fix sqlalchemy bulk item insertion

Related Issue(s):

#355
#356
#196

Description:
SQLAlchemy bulk item insertion failed because the Items iterator returns the keys for the dict it contains, rather than the Item objects themselves as it's docstring says it should (and that it would make sense to).
This fixes that problem, and documents what the format of the JSON object that is the input to the bulk insert endpoint is.
PR Checklist:

[X] Code is formatted and linted (run pre-commit run --all-files)
[X] Tests pass (run make test)
[X] Documentation has been updated to reflect changes, if applicable, and docs build successfully (run make docs)
[X] Changes are added to the CHANGELOG.

Created an issue here: https://github.com/stac-utils/stac-fastapi/issues/362

@geospatial-jeff looks like you'll need to merge, I don't have write perms.


System.IO.FileNotFoundException

Hello :)
When I try to run my application in heroku I get the exception below, do you know how to fix it?
The version of ASP.net MVC I am using is 5.
System.IO.FileNotFoundException
Could not load file or assembly 'System.Data.Entity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089' or one of its dependencies. The system cannot find the file specified.
Description: HTTP 500.Error processing request.
Details: Non-web exception. Exception origin (name of application or object): mscorlib.
Exception stack trace:
at System.AppDomain.Load (System.String assemblyString, System.Security.Policy.Evidence assemblySecurity, Boolean refonly) [0x00000] in <filename unknown>:0
at System.AppDomain.Load (System.String assemblyString) [0x00000] in <filename unknown>:0
at (wrapper remoting-invoke-with-check) System.AppDomain:Load (string)
at System.Reflection.Assembly.Load (System.String assemblyString) [0x00000] in <filename unknown>:0
at System.Web.UI.TemplateParser.AddAssemblyByName (System.String name) [0x00000] in <filename unknown>:0

Strange issue, Mono is supposed to have Entity Framework built-in.
@DerKnerd, have you tried using NuGet for EF?


[main > release/v2int/2.0]: Cache ops only on first snapshot fetch (#12439)

Description
Cache ops only on first snapshot fetch when the container loads as they could always be required. We don't want to override them with ops from later snapshot fetches/refreshes by summarizer.
Above change got reverted in this PR when we reverted whole change instead of just adding back blobs fetch:
https://github.com/microsoft/FluidFramework/pull/12405

In the description, can you link the commit in main that this is cherry-picked from.


I am confused with the dataset arenas.

Dear Authors,
I found from the anchor pairs file: "arenas_edges-mapping-permutation.txt" , see below records:

anchor = pickle.load(open('data/arenas_edges-mapping-permutation.txt','rb'))
anchor[0]
334
anchor[334]
1120
anchor[1120]
1103

Does it mean the pairs (0,334), (334,1120), (1120,1103)..... are all anchors ?
Also, I found the from the edge list file "arenas_combined_edges.txt", there are 2270 nodes totally (node ids range from 0 to 2269),  and there are 1135 pairs of anchors, but all the anchor node ids are range from 0 to 1134, no one is in the range from 1135 to 2269.
Could you please help to make it clear?
Regards,
Bryan Chen

Hi Bryan,
Thank you for your interest in our work and for getting in touch.  So in that example, node 0 in graph 1 aligns to node 334 in graph 2.  (And for 334 in graph 1, you see that it aligns to 1120 in graph 2).  In this setup, each graph has 1135 nodes, so for each graph there are IDs 0-1134.  However, our implementation embeds the graphs jointly by combining the two graphs into one big adjacency matrix, so in that combined graph nodes 0 - 1134 represent nodes 0 - 1134 in graph 1, and nodes 1135-2269 in the combined graph represent nodes 0-1134 in graph 2.  Hope this helps, and sorry for any confusion you may have experienced.
Best,
Mark

On Jan 7, 2020, at 8:55 PM, ChenBaiyang<EMAIL_ADDRESS>wrote:
Dear Authors,
I found from the anchor pairs file: "arenas_edges-mapping-permutation.txt" , see below records:
anchor = pickle.load(open('data/arenas_edges-mapping-permutation.txt','rb'))
anchor[0]
334
anchor[334]
1120
anchor[1120]
1103
Does it mean the pairs (0,334), (334,1120), (1120,1103)..... are all anchors ?
Also, I found the from the edge list file "arenas_combined_edges.txt", there are 2270 nodes totally (node ids range from 0 to 2269), and there are 1135 pairs of anchors, but all the anchor node ids are range from 0 to 1134, no one is in the range from 1135 to 2269.
Could you please help to make it clear?
Regards,
Bryan Chen
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/GemsLab/REGAL/issues/7?email_source=notifications&email_token=ABCQJQY34QZXN2REUU64VALQ4UXBXA5CNFSM4KEBTAV2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IEUPK6A, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABCQJQ3VO2NFUF4BOG6QLE3Q4UXBXANCNFSM4KEBTAVQ.

Hi Mark,
Thanks for your prompt reply!  It does help.
Regards,
Bryan


Update a new UI && scene lora training and infer.

New UI in Photo Infer.

Train scene Lora

Hey! I have a problem with installing EasyPhoto, i'm using cagliostro-colab, and after installing extension and restarting - there is no EasyPhoto tab. May you help, please?

Bravo PR

A fantastic UI redesign that has clarified the functional division between T2I and I2I after integrating SCENE Lora
The inference and training of Scene LoRa have already been completed, and the results are excellent.

分别是礼帽/果风/cyber-reddead2 三种风格！太棒了

Hey! I have a problem with installing EasyPhoto, i'm using cagliostro-colab, and after installing extension and restarting - there is no EasyPhoto tab. May you help, please?

please discuss in ISSUE, try to use lanrui-ai or Aliyun DSW, we dont support colab for now

test pass with main-IPA feature， good

infer api test

train api test


fixing master error

:+1:

Changes Unknown when pulling 59ed4ec6eceaa2f6f30b368578e51b8955cdb611 on antgonza:fix-master into ** on biocore:master**.


fix: paid in full credit card assist does not respect color blind mode

GitHub Issue (if applicable): #1549
Trello Link (if applicable):
Explanation of Bugfix/Feature/Modification:
Updated colour blind mode to change theme variables rather than overriding specific elements to give greater compatibility with other features. Expanded the range of elements colour blind mode covers and improved interaction with dark theme.
Tweaked warning colour to hopefully provide greater contrast for yellow/blue colour blindness (old/new):

This is certainly the better approach! These variables didn't exist way back when this was implemented afaik.
Would you mind putting a palette in your description to show the colors you went with? I'm not certain they're matching what we used to have?

I've added a palette image to the original description. I took the positive and negative colours from the original feature's style sheet, however I don't think the positive colour override is working in the current feature.
I did change the warning colour from blue to purple, as when I was testing with Chrome's colour blindness simulator I noticed there wasn't much contrast between the positive and warning colours when set to Tritanopia (blue/yellow). If you think it'd be better to leave it as is I can revert it, I'd just need to edit the accent colours to match.
I've been working on a way to set custom colour schemes (#2393) but I might need to reach out on Reddit or the support forums to get feedback as I'm not colour blind myself and am not sure how much to trust the simulators.

Awesome, thanks for that! Feel free to join our discord too. I doubt we have many members who will have much of an opinion but might be a good place to be anyway since you're contributing so much (which is greatly appreciated, by the way!)


Refactor CI/CD using release-manager

There's lot of SBT-specific magic in CI/CD right now, which is not consistent with other our projects.
Note: don't forget to re-enable tests in assembly.

Worth doing in this release.

Hey @rzats, if you don't know how to do this you can use snowplow/snowplow (with multiple apps) or Scala App Template (single app, as sauna) as examples.


Added method to check if a class exists

This PR fixes #232.

To avoid any confusion in the future about your contribution to Weaviate, we work with a Contributor License Agreement. If you agree, you can simply add a comment to this PR that you agree with the CLA so that we can merge.beep boop - the Weaviate bot 👋🤖PS:Are you already a member of the Weaviate Slack channel?

I agree with the Contributor License Agreement.

Since this was a priority we added this feature already, closing.

@StefanBogdan which PR implemented this feature?

@hsm207 this one: https://github.com/weaviate/weaviate-python-client/pull/311


Add React Router to Pages

For google analytics and to see if people are actually generating schedules, have more pages so that when a user hits generate it will redirect them to a new page.

Landing page should be one page, download page should go to a different page, generate should go to a different page, by that I mean it should to a different url.


Best Practices > Anti-patterns: Added 'feature-coupled' step definition example for JS.

Added a 'feature-coupled' step definition to provide an example of the anti-pattern's implementation, in Javascript, to resolve: https://github.com/cucumber/docs.cucumber.io/issues/113

This change is

Hi @charlierudolph - Would you please review this javascript example?

this bit moved to anti-patterns.md

@xtrasimplicity If you don't mind, would you process @charlierudolph's comment and also, move this to the antipatterns.md (that change has been merged to master). Thnx for your help!

All done! :)


(Lumen) Publising issue

Configuration and user providers are not published.
php artisan vendor:publish  --provider="KeycloakGuard\KeycloakGuardServiceProvider"
"Nothing to publish for tag []."

What is your Laravel/Lumen version?

Hi, I'm using Lumen 5.7, I must say I'm new to Laravel/Lumen.
I read php artisan config:clear would help, ben this command does not exists for lumen à guess.
I'ld like to provide a merge request for config_path usage but I can't test it.

I am sorry, but Lumen has a different setup. Just tested this package on Laravel.

From my experience, I recommend you use Laravel even for API Backend project. Lumen has many caveats and missing blocks.
Just edit RouteServiceProvide for “api” default routes and you are done.

Thank you, I did the job in AuthServiceProvider.
If it's ok with you I'ld like to make a pull request for token validation (HTTP call to keycloak) in KeycloakGuard::validateResources as a configurable feature.

Yes, you can do it! Please, just provide a related test (see folder /tests).


DBZ-6689 make kafka signal consumer group configurable

Closes: https://issues.redhat.com/browse/DBZ-6689

@vsantona Applied, thanks!


[AMORO-2] Flink support for setting/altering Amoro table properties

Why are the changes needed?

Close https://github.com/NetEase/amoro/issues/2.
Brief change log

Implement the ArcticCatalog.alterTable method.
Add testAlterTable UTs in TestCatalog.
Affected Flink versions: flink1.15/flink1.17.

How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before making a pull request

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (docs)

It seems you have some check style problems, you could use mvn spotless:apply -pl flink to fix them due to spotless plugin.

Thanks for your contributions, I left some comment

ci unpass

The PR is conflict with master.  Would you please to fix it ?

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 3 committers have signed the CLA.:x: yangbinbin:x: FWLamb:x: shidayangyangbinbin seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 3 committers have signed the CLA.:x: yangbinbin:x: shidayang:x: FWLambyangbinbin seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@FWLamb
The removal of support for Flink 1.15 in the Flink module has resulted in some conflicts that need to be fixed in this PR.

@FWLamb
The removal of support for Flink 1.15 in the Flink module has resulted in some conflicts that need to be fixed in this PR.

done

@FWLamb Are you still tracking this part of the code? If not reimplement the closure.


Why does :type show a NAME COLLISION?

If our models have a property called type it shows this NAME COLLISION error. However since the properties for a model go under attribute it shouldn't conflict with the jsonapi spec (and also seems to be serializing properly in my tests.)
def check_reserved_attribute_name(name)
  # Allow :id since it can be used to specify the format. Since it is a method on the base Resource
  # an attribute method won't be created for it.
  if [:type, :href, :links, :model].include?(name.to_sym)
    warn "[NAME COLLISION] `#{name}` is a reserved key in #{@@resource_types[_type]}."
  end
end

It's not jsonapi-compliant to allow type attributes. See http://jsonapi.org/format/#document-resource-object-fields
The rationale is that attributes, relationships, type, and id all occupy the same namespace. There are situations in which they are addressed as part of this single namespace such as sorting, filtering, and specifying sparse fieldsets.
With that said, that list is old and should probably be revised to include just type.

Ok, I was hoping it wasn't that way (we'll need to rename our attributes.) Thanks for the info.


Agilent33250a.py fix

Fixing a small leftover from python 2 times. In python 3 string.atof() is not supported anymore and can be replaced by float().

Thanks for the PR! As you can see, flake8 is failing because now the imported string is unused. Could you remove the import? This should fix the flake8 job

Updated!

Very nice, thank you for contributing!


allow device-on-pipe stacking

About the PR
allows you to stack a device on top of a pipe or a pipe on top of a device
Why / Balance
as per popular demand, makes atmos lives less hell
i have ran a poll on discord and a considerable amount of people was in favor of this or even fully readding pipestacking, i decided to go with this as the safer solution
Technical details
applies a few patches to C# code to allow stacking pipe on top of devices, otherwise it does not work
Media
extensively tested locally, trust

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
need to watch out for future changes involving pipe prototypes
Changelog

:cl:

tweak: Inline devices are back. Atmos devices are now exempt from the pipe stacking check.

I'm for this tbh, only because of burn chambers


One neat trick to make our builds run twice as fast

Currently our Scala build pipelines go something like:

run autoformat, wait
build libraries, wait
build apps

This is a holdover from when we built things in Travis and we had limited concurrency. Running builds we knew would fail was really bad – it wasted a worker and pushed other builds back.
Now we run in Buildkite with our own instances, we're no longer limited by concurrency. What if we built libraries and apps at the same time (once we knew autoformatting was okay)?
Pros:

Slice build times in half, less time waiting for builds and deployments. Currently the minimum build time is $(max library build time) + $(max app build time), depending on how jobs are assigned. Now the minimum would be $(max library/app build times).

Cons:

Potentially more expensive – we'd be running jobs we might already know couldn't possibly succeed, because an earlier job had failed. (Although you can configure fast fail for Buildkite.) Our current Buildkite instance bill is ~$75/mo, so not exorbitant.

What do other devs think of this?

I would be pro- this, it's not as expensive as all of us sitting around twiddling our thumbs waiting for builds to pass...

@alexwlchan we could use a plugin like this to avoid using resource when we know it's going to fail?

we could use a plugin like this to avoid using resource when we know it's going to fail?

Yup, although I'd prefer an approach that doesn't require us to annotate every build pipeline.
Although tbh the Buildkite bill is sufficiently small right now I don't care much about cost reduction measures.

Done!


needs to split with -

What does this PR do?
Fixes the lack of the split of string representing alias tensor indices.

close this one following the removal -- ref: https://github.com/Lightning-AI/lightning-thunder/pull/1451


Build a Pokémon Search App Project - element id typo

Describe the bug
There's typo in two hints. Instead of #types element it mentions #type.

This is something that can be fixed by "first time" code contributors to this repository. Here are the files that you should be looking at to work on a fix:
List of files:

https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/english/15-javascript-algorithms-and-data-structures-22/build-a-pokemon-search-app-project/build-a-pokemon-search-app.md

Please make sure you read our guidelines for contributing, we prioritize contributors following the instructions in our guides. Join us in our chat room or the forum if you need help contributing, our moderators will guide you through this.
Sometimes we may get more than one pull requests. We typically accept the most quality contribution followed by the one that is made first.
Happy contributing.

I was making a PR for a different issue and saw this, so I added the fix to the PR.


[OCPCLOUD-1771] Add header to generated manifests

To indicate that some manifests are automatically generated, we need to add a related header to them.

Seems reasonable
/lgtm
/approve


Power Spectral Density

Hi, I am a beginner in learning Python. In 'Power Spectral Density’.In my opinion,for monthly data, the results are divided by 12 to calculate annual cycles like this  'plt.plot(1.0/f1[1:47]/12, pxx1[1:47], label='welch')',but for annual data,how to deal with it？I tried not to divide by 12，but
the maximum period of abscissa is 250 years，this may be something wrong .Could you help me to solve this problem? Look forward for your reply.

In fact, you can plot the result of the Welch method directly. Thus, you can find the maximum  PSD and its frequency. Then convert the frequency to years.
plt.semilogy(f1, pxx1)
plt.ylim([0.5e-3, 1])
plt.xlabel('frequency [Hz]')
plt.ylabel('PSD [V**2/Hz]')
plt.show()
They follow some relationships. Let fs be the sampling rate, thus, the sampling time is dt = 1/fs. If the sampling point is N, then the total duration is T = N* dt and the interval of the spectral points is df = 1/T.

Got it，thank you very much.Confidence test is often needed for power spectral density signals like  statistical significance at a confidence level better than 95%. How to do this?

https://www.researchgate.net/post/How_to_calculate_the_confidence_interval_of_a_given_PSD_with_Python2


Scroll to top button

Implement scroll to top button based on new design

The button should be displayed in a fixed position on the screen
When clicking the button, the page should smoothly scroll back to the top of the page
The button should only appear when the user has scrolled down a certain distance from the top of the page
The button should be hidden when the user is already at the top of the page or when the page content does not exceed the viewport height

The following icon should be used:

Navigation/Arrow Up

I’ve got a direct link to the item in the design.

@kesiah please refer to this component in the Core Design Kit. You can implement the hover, pressed and, focus state.

confirmed as done


feat: add FlatCallTracer

Closes https://github.com/paradigmxyz/reth/issues/10991
still wip but creating draft pr to get some guidance on whether this is the right path
some things to note:

we first convert the config into a FlatCallConfig
create the inspector using the from_flat_call_config (this change has been proposed here: https://github.com/paradigmxyz/revm-inspectors/pull/203) , this allows us to use the precompile flags
then use the inspector to trace the transaction, instead of initializing a default parity tracer since that would ignore the precompile flag

trying to understand if this is the right approach to adding flatcalltracer or if i'm overcomplicating things

quick attempt at actually doing the tracing:

use inspect with the inspector to actually produce the trace
initialize a default tx
convert inspector into a parity builder and give it the default tx (lmk if there is a tx hash that is relevant even tho this is a call and not a real tx)
pull first element out of vector, return

added other, need from_flat_call_config here, will wait for bump on that

yeah can use #11152 to remove the pop or include the alloy bump in the PR if needed


weechat: Out of memory when connecting to a server

Connecting to any irc server with weechat is not possibledue to a "not enough memory" error.
To reproduce:

Install weechat sudo apt-get install weechat
Start weechat
Connect to any irc server /connect irc.freenode.net

Adding a link to #92.
Would it be possible for you to take an strace of weechat so I can make sure it's using the same procfs files to determine how much memory the system has?

I'm not really familiar with strace, I ran strace -o file.txt weechat:
http://pastebin.com/DCKtzKgx
After weechat starts, I try to connect to an irc server, then exit after the out of memory message comes (and the connecting attempt is interrupted)

Thanks that was helpful.  It looks like weechat may be using /proc/meminfo to determine the amount of free memory. This is the same file top uses to display the info.  I'm working on a fix for this now and will see if this unblocks weechat.

It looks like this might actually be due to a different error, us not supporting PF_LOCAL datagram sockets.
socketpair(PF_LOCAL, SOCK_DGRAM, 0, 0x7ff5fffede20) = -1 ESOCKTNOSUPPORT (Socket type not supported)

benhillis, are you planning on supporting /proc/meminfo and PF_Local datagram sockets?  Weechat is one of the main pieces of software I got really excited about having available in Windows (the cygwin builds just don't work right).  Also, do you have anything planned like an Lxsvchost for running the Ubuntu Daemons as Windows services?

@fpqc - /proc/meminfo is an interesting one. Unix and NT have different memory manager semantics so it will be hard (or impossible) to report accurate numbers for every field.  That being said, I recently made a change to at least report the following fields: MemTotal, MemFree, SwapTotal, SwapFree.  Note Swap* numbers will be reporting the NT page file size and usage.  These are two of the main fields top and htop use to display memory usage information.
Let me sync up with the developer that worked on unix (PF_LOCAL) sockets and see what his thoughts are on adding support for datagrams.

@benhillis I know how busy you guys are, but did you find anything else out about getting weechat to run (the PF_local datagrams and the /proc/meminfo)?  Also, very happy that in the latest insider build, zsh installs properly.

@peterjuras - I am looking into AF_UNIX\SOCK_DGRAM support in WSL. With the initial code, it looks like I am able to get past the above error in 'weechat'. But, the connect command seems to be timing out. I also tried weechat on my native Ubuntu setup and I am not able to get it to connect as well. Here is what I am trying, please let me know if I am missing something:
sudo apt-get install weechat weechat /connect irc.freenode.net
I even tried adding a server by using the /server add freenode chat.freenode.net command and then connect, but no luck.
17:03:06 freenode  -- | irc: connecting to server chat.freenode.net/6667... 17:03:54     oftc =!= | irc: timeout 17:03:54     oftc  -- | irc: reconnecting to server in 10 seconds 17:04:04     oftc  -- | irc: reconnecting to server... 17:04:04     oftc  -- | irc: connecting to server irc.oftc.net/6697 (SSL)... 17:04:06 freenode =!= | irc: timeout 17:04:06 freenode  -- | irc: reconnecting to server in 10 seconds 17:04:16 freenode  -- | irc: reconnecting to server... 17:04:16 freenode  -- | irc: connecting to server chat.freenode.net/6667...

@sunilmut I made a new shell account on my vps, which is running CentOS 7 with a git-built version of Weechat 1.6-dev from last month, and here is how I connected:
/server add freenode chat.freenode.net
/connect freenode
and it worked alright.
I don't have a copy of Ubuntu 14.04 with their version installed anywhere, but I expect it should work the same.  If you can't get it to connect on your native Ubuntu setup, maybe see if you can get it to start on the git-devel version?

@sunilmut So by the way, I know this was one of your test cases for AF_UNIX, SOCK_DGRAM, and I am aware you have the big fix on the way.  Just wondering if you got Weechat running, or if it ended up failing somewhere else.

@russalex Hi, this is now fixed in 14915.  Just notifying you =).

Thanks!


(BKR-1156) Modularize google compute hypervisor

Separate vagrant hypervisor to an external module: beaker-google.

[x] Review and publish beaker-google
[ ] CI should pass on this PR once the module is published

[ci skip]

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/3995/

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/4018/

@puppetlabs-jenkins retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/4022/


Improve discoverability of your work on HF

Hi,
Niels here from the open-source team at Hugging Face. I discovered your work through the paper page: https://huggingface.co/papers/2405.19707 (feel free to claim the paper so that it appears under your HF account!).
To improve the discoverability of your work, it would be great to make the dataset available on the 🤗 hub.
See here for a guide: https://huggingface.co/docs/datasets/loading. This way, people would be able to do:
from datasets import load_dataset

dataset = load_dataset("your-hf-organization/demamba-dataset")

to load it in 2 lines of code. The dataset would also come with a viewer such that people can see the first few rows in the browser.
It can then also be linked to the paper as explained here: https://huggingface.co/docs/hub/en/datasets-cards#linking-a-paper
Let me know if you need any help regarding this!
Cheers,
Niels
ML Engineer @ HF 🤗

This would be awesome 😎

Expect this dataset to be available in Hugging Face


Example with RTSP inputs and individual RTSP output

Hi all,
thanks for this incredible library, I would like to know if you have some sample about creating dynamic pipelines of RTSP streams , having different RSTP streams as input, processing them with a pgie and some sgies, and a different RTSP ouput for each stream.
I'm trying to do it with Deepstream 6.1, but when adding/deleting new streams I got some issues with Deepstream and I have to restart the pipelines.
Regards.

@micuentadecasa re: "I'm trying to do it with Deepstream 6.1, but when adding/deleting new streams I got some issues with Deepstream and I have to restart the pipelines."
issues with DSL/DeepStream?  or issues just trying with just DeepStream?  Also, are you developing in Python or C/C++ ?

Hi @rjhowell44 , I'm doing it in Python and Deepstream 6.1, I started from this code, https://github.com/NVIDIA-AI-IOT/deepstream_python_apps/tree/master/apps/runtime_source_add_delete

@micuentadecasa There are at least a couple of users that hang out on our Discord site using DSL exactly as you mention.  Please join us if you're interested.
Unfortunately, we don't have the use case you've described covered in a single example. You will need to pull  from the following.

here's an example of how to create you're RTSP Sources and Recording Taps (not sure if you need recording) https://github.com/prominenceai/deepstream-services-library/blob/master/examples/python/ode_occurrence_4rtsp_start_record_tap_action.py
here's an example of dynamically adding/removing sources... although it uses a Tiler which doesn't really work well for source remove.  https://github.com/prominenceai/deepstream-services-library/blob/master/examples/python/dyn_uri_file_pgie_ktl_tiler_osd_window.py
and here's an example of how to create an RTSP Sink https://github.com/prominenceai/deepstream-services-library/blob/master/examples/python/1csi_live_pgie_osd_overlay_rtsp_h254.py
finally, here's an example of creating a demuxer with multiple branches and sinks - https://github.com/prominenceai/deepstream-services-library/blob/master/examples/python/2rtsp_splitter_demuxer_pgie_ktl_tiler_osd_window_2_file.py

Happy to help with any problems you might run into.

thanks @rjhowell44 , I will take a look at the samples and will connect in the Discord site
Regards.

@micuentadecasa also, one of the import services you'll need will be dsl_source_rtsp_state_change_listener_add which may not be in the above examples... and see the RTSP Stream Connection Management as well.

many thanks @rjhowell44


Resource mounting problems on resource that supports tags

As also reported in #168, a problem exists where resource mounting and recursive searching were causing a problem on resources that support tags where we should not either do recursive searching or resource mounting.
This PR will disable resource mounting on scenarios where resources that support tags are used.

Pull Request Test Coverage Report for Build 621

32 of 35 (91.43%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 78.366%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

terraform_compliance/common/helper.py
10
11
90.91%

terraform_compliance/extensions/terraform.py
19
21
90.48%

Totals

Change from base Build 619:
0.3%

Covered Lines:
710

Relevant Lines:
906

💛  - Coveralls


#845 Added a FIFA 19 Dataset analysis

DESCRIPTION
This is a regression problem where we analyze the FiFA 19 Data and predict the value. We use Linear Regression, Decision Tree  and Random Forest Regressor.
WHAT I HAD DONE

Perfromed exploratory data analysis (EDA) on the given dataset
It starts with loading the dataset and viewing the top 5 rows
We calculate statistical data in the dataset
Then comes finding correlation between the features and also finding statistical values related to the dataset
Data visualization is done with libraries such as matplotlib and seaborn
Finally 3 different algorithms are used to find the best algorithm
Also accuracy score of each algorithm is calculated for comparison purpose with other algorithms

There are multiple projects' files in this repo. Please look into that. (Each PR's files must be committed to a unique branch)

@priyankeshh ?


add more information in README.md by SinaLale

Checklist

[ ] Closing issues: #issue
[ ] Mark this if you consider it ready to merge
[ ] I've added tests (optional)
[x] I wrote some documentation

Description

I hope this is useful to you my friend.
I pull request again. if I can be useful

Thank you for response

i have updated the readme, hope the instructions are clearer now :)
thanks for the feedback and testing!


[<IP_ADDRESS>] Heat Exchanger Fluids are broken

Lava is able to enter the water and lava slot, and water is able to enter the lava and water slot. Basically there is no restrictions with the built in tanks. Pretty sure it broke with the new api change for liquids

Check with Advanced Generators <IP_ADDRESS> and BDLib <IP_ADDRESS>, i'm pretty sure this is already fixed.


Make Cards Clickable

make Example cards clickable

###Are you working on it (NO)

Hey, can I work on this?

I would like to commit a change as shown below

@mahmodHammad  Please suggest if any further modification is required

@Vishruth-S
this hover effect is good
but it would be great if you could add this line
transition: all 0.3s ease;

@mahmodHammad
Is this alright?

@nitin10s
Please take a look at this

Reduce the size of the circuit box when hovered, like in the picture @mahmodHammad sent.

@Cinderace should work on this issue if available.

@nitin10s Okay! I'm on it!

@Cinderace Really sorry I didn't notice your earlier comment. I am no longer working on this, You can take it up :)

@chaudharirohit2810 I am currently working on it. Please find another issue. Thank you.

Okay

@Cinderace  any updates?

Is this issue closed, or can i take it up?
Thanks!

 I want to make contributions, i am new to this org. Please let me know from where I can start! Thanks! :)


Update source lib readme

Resolves #2119.
I simplified the long list of included libs into a table, removed the ones that weren't there anymore, and added entries for the ones that didn't have entries.
At first I had a column for upstream branch, but it was "master" or "N/A" in all cases, so I dropped it.

Codecov Report

Merging #2124 into develop will decrease coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #2124      +/-   ##
===========================================
- Coverage    68.98%   68.97%   -0.01%
===========================================
  Files          684      684
  Lines        50394    50394
===========================================
- Hits         34764    34759       -5
- Misses       15630    15635       +5

Impacted Files
Coverage Δ

src/ripple/basics/DecayingSample.h
77.77% <0%> (-8.34%)
:arrow_down:

src/ripple/server/impl/BaseWSPeer.h
67.5% <0%> (-0.63%)
:arrow_down:

src/ripple/app/main/Application.cpp
60.82% <0%> (-0.13%)
:arrow_down:

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 24e1b99...ccfc265. Read the comment docs.

Merged as 6e889e6a185b0c0673861301bc2a53b113e1b729


update package.json scripts, fixed #30

Please makes sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's Contributing Guide.
[x] Rebase before creating a PR to keep commit history clear.
[x] Add some descriptions and refer relative issues for you PR.

好像并不太 clear，前三个 commit 可以合并成一个，甚至 rebase 到没有 merge 的 commit。


Public Lambda Layers ARN instead of SAR App

Original author: @ecolban

Runtime e.g. Python, Java, all of them. All of them
Is your feature request related to a problem? Please describe.

As described in the Lambda Powertools Python discussion, Layers are currently available as a Serverless Application Repository (SAR) App and not as a public Lambda Layer ARN.
While this allows customers to selectively deploy the semantic version they're interested in it does require deploying a SAR App before they can use the Lambda Layer.
Describe the solution you'd like

Make Lambda Layer ARN publicly available per region. This will make it easier to add Lambda Powertools to existing projects.
Some examples: https://github.com/mthenw/awesome-layers
Describe alternatives you've considered

Building and publishing own Lambda Layer though that adds operational overhead in maintaining.
Is this something you'd like to contribute if you had guidance?
Additional context

Would love to have this as a layer.  Been using Serverless Framework but I see this appears to be only available in eu-west-1.  I am in the US so having layers here as well would be great.

Absolutely - one misconception though is that we're using Serverless Application Repository (SAR) Application as a layer, and while its ARN is EU-West-1 it can be deployed anywhere globally.
Once deployed, it'll create a Lambda Layer in the region you deployed, and that's the ARN you can use.
I'll create a sample in the docs on how to do it for Serverless framework, as @am29d continurs to work on the public ARN folks will be able to use directly in Lambda with any framework

Ahh gotcha.  Yah sadly I have not used a ton of SAR yet but I guess I will need to check this out.
I am using Serverless Framework for this and being able to add it as a layer would be by far the easy path.

I guess maybe a follow up question then now that I think about it.  Since I am using a container for my lambda, are there some install commands where I can pull this in since a layer is not going to work for a container deployment.

@stephenbawks doesn't help with your container question, but here's how you can configure this through serverless:
resources:
  Transform: AWS::Serverless-2016-10-31
  Resources:
    AwsLambdaPowertoolsPythonLayer:
      Type: AWS::Serverless::Application
      Properties:
        Location:
          ApplicationId: arn:aws:serverlessrepo:eu-west-1:057560766410:applications/aws-lambda-powertools-python-layer
          SemanticVersion: # Add latest version from https://github.com/awslabs/aws-lambda-powertools-python/releases

Which can then be used to define the layer:
functions:
  main:
    handler: lambda_function.lambda_handler
    layers:
      - !GetAtt AwsLambdaPowertoolsPythonLayer.Outputs.LayerVersionArn

What about publishing it as an extension in the newly released CloudFormation Registry? The resource (layer) could be deployed by Customers in their own account and we could also consider the Stack to accept a parameter from the user to specify semver of the layer to deploy.

That’s a neat idea, Andrea!!!
I think that still requires additional steps to use it (activate), and also
requires extra steps for us to publish it — As Alex frees up bandwidth, his
work in creating a CI/CD Pipeline for publishing layers in every region
will also allow us to bring Lambda Powertools Layer in the Console at some
point … hence why it’s taking much longer than expected
On Mon, 28 Jun 2021 at 12:04, Andrea Amorosi @.***>
wrote:

What about publishing it as an extension in the newly released
CloudFormation Registry
https://aws.amazon.com/blogs/aws/introducing-a-public-registry-for-aws-cloudformation/?
The resource (layer) could be deployed by Customers in their own account
and we could also consider the Stack to accept a parameter from the user to
specify semver of the layer to deploy.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-lambda-powertools-roadmap/issues/2#issuecomment-869552272,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZPQBFDS2ML5L6PKFECSM3TVBCJHANCNFSM442OIG7A
.

What about publishing it as an extension in the newly released CloudFormation Registry? The resource (layer) could be deployed by Customers in their own account and we could also consider the Stack to accept a parameter from the user to specify semver of the layer to deploy.

i guess this is a nice idea but what about those who are on TF land?
imo just  publish public layers will cover a wider audience rather than cloudformation only.

Good point, Lambda Layers seems to reach a wider audience at the moment; we can re-evaluate in the future in case any Customer mentions it.

Is there any estimation if this will be implemented and when? We have been using third-party layer from https://github.com/keithrozario/Klayers but that is obviously not as good as having these officially published by AWS. As we are using terraform to deploy this without SAM, we can't use the layers from SAR or CloudFormation registry.

Hey @kimmoahokas thanks for the follow up, I am currently working on shipping this feature. I can't provide any dates, because there are still things out of my control or influence that might delay it. I can't wait to release it asap and make layers more accessable.

Yes @kimmoahokas - @am29d is on it and we expect it sometime next month, November the latest
We've had some challenges on making this work flawlessly across all regions, hence the delay

Thank you very much @am29d and @heitorlessa for the status update. We appreciate your effort on this and all of the lambda powertools :)

@heitorlessa  can we mark this one as done?


Move yew/services to yew-services crate

Description
Move yew/services to a new yew-services crate. This is one part of the work needed to be done for https://github.com/yewstack/yew/issues/1670.
Checklist

[x] I have run cargo make pr-flow
Note: I've had some of the stdweb tests fail, don't know why
[x] I have reviewed my own code
[ ] I have added tests
Not applicable, only refactoring

Note: I ensured that this code compiles with both web_sys and stdweb but I ran the tests only with web_sys

Thanks @hamza1311! I'm wondering if it would be nice to keep the services feature and use it to re-export yew-services at yew::services

Thanks @hamza1311! I'm wondering if it would be nice to keep the services feature and use it to re-export yew-services at yew::services

@jstarry, that won't work right now because yew-services depends on yew so there will be circular dependencies if yew depends on yew-services
Maybe we could that in a future PR when yew is broken up even more so we can avoid the circular dependency

@jstarry, that won't work right now because yew-services depends on yew so there will be circular dependencies if yew depends on yew-services
Maybe we could that in a future PR when yew is broken up even more so we can avoid the circular dependency

Ah right, thanks. Yeah that can come later, it probably will make sense to remove yew-services' dependency on yew altogether

Ah right, thanks. Yeah that can come later, it probably will make sense to remove yew-services' dependency on yew altogether

Just bumping this.
I have resolved the conflicts so can this PR be merged now?

Just bumping this.
I have resolved the conflicts so can this PR be merged now?

Apparently the chrome driver won't start. Any idea what might have gone wrong?

Apparently the chrome driver won't start. Any idea what might have gone wrong?

Apparently the chrome driver won't start. Any idea what might have gone wrong?

Hmm, not really sure. I just retried the job

Apparently the chrome driver won't start. Any idea what might have gone wrong?

Hmm, not really sure. I just retried the job

I have resolved the conflicts so can this PR be merged now?

Yeah, let's get it in!

I have resolved the conflicts so can this PR be merged now?

Yeah, let's get it in!

There's still quite a bit of cleanup left to do.

What you pointed out seems to be because IntelliJ's merge tool somewhat messed up. I fixed up.

Maybe we should remove yew-stdweb first?

I think if we can get this done now, it should happen. Maybe yew-agent and yew-format should happen before yew-stdweb is removed. The next steps (like yew-core as mentioned in #1670, removing yewtil, etc) should happen after we get rid of yew-stdweb. What do you think?

There's still quite a bit of cleanup left to do.

What you pointed out seems to be because IntelliJ's merge tool somewhat messed up. I fixed up.

Maybe we should remove yew-stdweb first?

I think if we can get this done now, it should happen. Maybe yew-agent and yew-format should happen before yew-stdweb is removed. The next steps (like yew-core as mentioned in #1670, removing yewtil, etc) should happen after we get rid of yew-stdweb. What do you think?

I think if we can get this done now, it should happen.

Ok, great :)

Maybe yew-agent and yew-format should happen before yew-stdweb is removed.

Why is that?

The next steps (like yew-core as mentioned in #1670, removing yewtil, etc) should happen after we get rid of yew-stdweb. What do you think?

Yeah, that sounds good

I think if we can get this done now, it should happen.

Ok, great :)

Maybe yew-agent and yew-format should happen before yew-stdweb is removed.

Why is that?

The next steps (like yew-core as mentioned in #1670, removing yewtil, etc) should happen after we get rid of yew-stdweb. What do you think?

Yeah, that sounds good

Maybe yew-agent and yew-format should happen before yew-stdweb is removed.

Why is that?

Because those don't have a specific dependence on yew-stdweb. agent uses in one place and format doesn't use it at all. If it can be done without messing around too much with stdweb now then why not do it? If there's a reason not to, we can delay it.

Maybe yew-agent and yew-format should happen before yew-stdweb is removed.

Why is that?

Because those don't have a specific dependence on yew-stdweb. agent uses in one place and format doesn't use it at all. If it can be done without messing around too much with stdweb now then why not do it? If there's a reason not to, we can delay it.

Because those don't have a specific dependence on yew-stdweb. agent uses in one place and format doesn't use it at all. If it can be done without messing around too much with stdweb now then why not do it? If there's a reason not to, we can delay it.

Ah, I see. It's ok, yew-stdweb removal is almost done anyways

Because those don't have a specific dependence on yew-stdweb. agent uses in one place and format doesn't use it at all. If it can be done without messing around too much with stdweb now then why not do it? If there's a reason not to, we can delay it.

Ah, I see. It's ok, yew-stdweb removal is almost done anyways

Looks great, @hamza1311! I just added a few small tweaks as well and will merge if CI passes. Thank you!

Looks great, @hamza1311! I just added a few small tweaks as well and will merge if CI passes. Thank you!


fix(azs): Leverage CommonStorageServiceDAOConfig in AZS

We were getting a bunch of 'Bean' exceptions without this
Related to https://github.com/spinnaker/front50/pull/224

@ajordens @robzienert @ewiseblatt PTAL

Oops, sorry about that!

No problem - this is why we have tests :)


Windows: conda-standalone may fail with no OPENSSL_Applink with official Python installed

Checklist

[X] I added a descriptive title
[X] I searched open reports and couldn't find a duplicate

What happened?
Some users report failures with Miniconda 24.5.0, which uses conda-standalone 24.5.0 as the bootstrapper. One example can be found here: https://github.com/ContinuumIO/anaconda-issues/issues/13408
Further debugging showed that the release binaries for conda-standalone 24.5.0 produce the same error:
OPENSSL_Uplink(00007FFB01AE2FE8,08): no OPENSSL_Applink

There appears to be a correlation with users having an official Python installed. I cannot reproduce the problem though.
Miniconda 24.4.0, which uses conda-standalone 24.3.0, works fine though.
Conda Info
N/A

Conda Config
N/A

Conda list
N/A

Additional Context
A lot of things have changed between conda-standalone 24.3.0 and 24.5.0, most notably the Python and pyinstaller versions got updated.
However, the packages on defaults were both built with pyinstaller 5.13.2.

Thanks to @guitarpicker, we learnt that this problem occurs when the SSLKEYLOGFILE environment variable is set. This is used to dump debugging information from SSL transactions to the chosen file.
After some debugging @marcoesters and I got to the conclusion that this is indirectly caused by a change introduced in requests 2.32 (note this is going to be reverted, though).
That PR calls this function on import:
from urllib3.util.ssl_ import create_urllib3_context

create_urllib3_context()

And that causes the reported crash. We are only observing this in the latest 24.5.0 build because of the requests update. But if we run the above code in any other conda-standalone version, it crashes in the same way:
> conda-standalone-24.4.0-Windows-x86_64.exe python -c "from urllib3.util.ssl_ import create_urllib3_context as c; c()"
OPENSSL_Uplink(00007FFCF30D0FE8,08): no OPENSSL_Applink

> conda-standalone-24.3.0-Windows-x86_64.exe python -c "from urllib3.util.ssl_ import create_urllib3_context as c; c()"
OPENSSL_Uplink(00007FFCF3240FE8,08): no OPENSSL_Applink

> conda-standalone-24.1.2-Windows-x86_64.exe python -c "from urllib3.util.ssl_ import create_urllib3_context as c; c()"
OPENSSL_Uplink(00007FFCF323AFE8,08): no OPENSSL_Applink

The root cause is not well understood (possibly in this CPython C code block, plus some interaction with PyInstaller?), but we can workaround the issue by unsetting the SSLKEYLOGFILE before it reaches the requests import. This is what we are doing in https://github.com/conda/conda-standalone/pull/89.

The root cause is not well understood (possibly in this CPython C code block, p

I'm out of my lane and don't feel confident about posting in the upstream repo, but after taking a cursory looking at that block, I notice it's using BIO_new_fp(fp, BIO_CLOSE | BIO_FP_TEXT); and there's this note in the man page that:

When wrapping stdout, stdin or stderr the underlying stream should not normally be closed so the BIO_NOCLOSE flag should be set."

I'm not sure if this is the same thing, but from my recent experience I can tell you that the SSLKEYLOGFILE target file is usually being referenced by some process or another where I can't do any sort of deletion or modification.  Mine is still locked by MS Edge (and probably Chrome and potentially any other log aware program as well) since I haven't gotten around to restarting my user session since reporting the issue.  Perhaps BIO_CLOSE allows some file operations to be attempted that will fail where perhaps BIO_NOCLOSE would succeed.

I currently think it's much more likely that it is due to how conda-standalone is built. You can convince yourself by running the following:
conda-standalone-24.5.0-Windows-x86_64.exe python -c "from urllib3.util.ssl_ import create_urllib3_context; c
reate_urllib3_context()"

And do the same thing with vanilla python or even a conda-installed python. Only conda-standalone will fail. We even tried packaging that command into a pyinstaller-created binary to see if it's a pyinstaller bug and it still succeeds. So, if the code itself was problematic, I would expect it to fail consistently.
I would also expect a different error message if it was some kind of access violation.


feat(consensus): add validate proposal flow to sequencer context

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1195
#1194  👈
#1193
#1192
main

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

Join @matan-starkware and the rest of your teammates on  Graphite


Solutions engineer Task - Johan Esteves

Hello,
I have completed the Solution Engineer task and added my responses in answers.md. I have also included the scripts I used.
Thanks,
Johan

Hello,
I've committed my updated answers for the solutions engineer exercise.
Thanks,
Johan


Regression in SearchBox Button

Since the button is used in two ways (clear gazetteer resuklts and executing a server search) we have to introduce a value that distinguish between these two modes

see also https://github.com/cismet/wupp/issues/1419


Fix the license in fdk-aac

WARNING: NOOB AT THE WHEEL, PLEASE SCREAM IF WHEN I DO SOMETHING VERY WRONG
Description of changes
Instead of using the Apache 2.0 license, add and use a custom fraunhofer-fdk license to prevent future confusion.
Questions: Does this license count as "free"? Am I using the correct name? Is there a viable domain link for this?

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Is there a viable domain link for this?

What about https://sourceforge.net/p/opencore-amr/fdk-aac/ci/master/tree/NOTICE?

Does this license count as "free"?

Looks like it, see e.g. https://www.gnu.org/licenses/license-list.en.html#fdk
Fedora seems to be changing their mind about it due to the lack of patent license, but that would also apply to e.g. CC0, which we consider to be free.

Please make sure your commit messages are formatted according to CONTRIBUTING.md.

apparently you're meant to ignore the fact there isn't a license in the name... ok then.
reopening since this should be fixed now.

Oh I see now what you meant.  I don't know... it's weird that they don't have it, but I think it's OK to keep it the way they named it 🤷🏽


TestMeasureBitRate sometimes fails

=== RUN   TestMeasureBitRateDynamic
##[error]    TestMeasureBitRateDynamic: measurement_test.go:95: expected: 25600.000000 (with 8.000000 precision), but got 25585.959280
--- FAIL: TestMeasureBitRateDynamic (5.00s)

https://github.com/pion/mediadevices/pull/139/checks?check_run_id=598870347 (restarted)

Hmm, we probably should relax the precision maybe? maybe like 64 bits (8 bytes)?


Make heat_map module public to expose Smoothing enum.

Hi,
I noticed there doesn't seem to be a way to access the Smoothing enum in the heat_map module, which is needed as a parameter to the HeatMap.zsmooth() method because the heat_map module is private. I followed what has been done for other modules to expose this enum publicly.
Feel very free to modify if there's a better way to do this. I'm still pretty new to rust.

@aureliegrebe, will merge it asap. Thanks!


[LAB] Week 2_Problem 1_노영래

Problem
Week 2_Problem 1
Source Code
from random import random
from typing import Callable
import numpy as np

##                         Problem 1                          ##
##                                                            ##
##            Arbitary x_train, y_train are given.            ##
##   Suppose that x and y have linear correlation, y=wx+b.    ##
##     In function training(), you should return [w, b].      ##
##          In function predict(), you should return          ##
##            list y_test corresponding to x_test.            ##
##                  Made by @jangyoujin0917                   ##
##                                                            ##

# NOTE : Feel free to use torch.optim and tensor.
theta_best = []

def training(x_train : list[float], y_train : list[float]) -> list[float]: # DO NOT MODIFY FUNCTION NAME
    # Data normalization code (prevents overflow when calculating MSE, prevents underfitting)
    # Note that you need to convert [w, b] to the original scale before returning value
    # w = w * (y_max - y_min)
    # b = b * (y_max - y_min) + y_min
    y_min = min(y_train)
    y_max = max(y_train)
    normalize = lambda y : (y - y_min)/(y_max - y_min)

    x_m = np.c_[np.ones((len(x_train), 1)), x_train]

    theta_best = np.linalg.inv(x_m.T.dot(x_m)).dot(x_m.T).dot(y_train)
    ### IMPLEMENT FROM HERE

    return theta_best.ravel()

def predict(x_train : list[float], y_train : list[float], x_test : list[float]) -> list[float]: # DO NOT MODIFY FUNCTION NAME
    x_m = np.c_[np.ones((len(x_train), 1)), x_train]

    theta_best = np.linalg.inv(x_m.T.dot(x_m)).dot(x_m.T).dot(y_train)
    x = np.c_[np.ones((len(x_test), 1)), x_test]
    y_predict = x.dot(theta_best)

    return y_predict

if __name__ == "__main__":
    x_train = [0.0, 1.0, 2.0, 3.0, 4.0]
    y_train = [2.0, 4.0, 6.0, 8.0, 10.0] # Note that not all test cases give clear line.
    x_test = [5.0, 10.0, 8.0]

    w, b = training(x_train, y_train)
    y_test = predict(x_train, y_train, x_test)

    print(w, b)
    print(y_test)

Description
Matrix product
Output (Optional)
No response

This issue is closed now because of the lack of progress.


[Task]: Headless Connect API Backend Implementation

SubTasks
This is an unknown task for now as the Headless API from Paragon is still under development. We can add a ballpark estimate for this for now.
https://www.notion.so/appsmith/Paragon-commitments-and-timelines-14bfe271b0e280839062e97ba64a9709?pvs=4#14bfe271b0e280e59366dd6bcf6ca88a

Hey team! Please add your planning poker estimate with Zenhub @AmanAgarwal041 @sneha122

Setting the final estimate as 3, as there are multiple unknowns here. Also as per the planning poker estimates.


LDAP Setting not working with Artifactory v7.57 and upper

Describe the bug
We were using the provider to configure artifactory_ldap_setting.
We upgrade to v7.59.9 and this is not working anymore.
Seems to be related to https://jfrog.com/help/r/jfrog-release-information/artifactory-7.57.1-cloud.
ldapsettings are not returned by API system/configuration anymore.
Artifactory version : 7.59.9
Terraform version 1.4.6
Provider version 7.11.2 or 8.0.0

@jcoste-orange, it's related to the change in the Artifactory, we are planning to add a new resource, which will use the new API for LDAP and LDAP group settings. The Artifactory team is also aware of the breaking change.

Ok thanks

I'm closing this one. The new resource is a planned work.

Hi @jcoste-orange, new resources were added in v8.4.0.

Thank you very much.


Pre-trained models for other datasets

Thanks for providing the flow-only OCLR model for syn data.
Do you have the pre-trained models for other datasets like DAVIS2016, DAVIS2017-motion, SegTrackv2, FBMS-59 and MoCA_filter etc.?

Thanks for your interest in our work. Our model is synthetically trained and can be generalized to real-world datasets. Therefore, the synthetically trained model checkpoint can be directly applied to real-world video datasets.


Add amp-consent (for GDPR) and ClientID (for analytics)

Try it out... hope you like it!

That's an interesting idea - I'll leave it to sbenz to say whether he wants
to have the samples rewritten in this way.
It would take a bit of time.  Would it make sense to unleash this cookie
notice on the rest of the site while this is being discussed - or would you
prefer to wait?
On Wed, Aug 15, 2018 at 12:38 PM ericlindley-g<EMAIL_ADDRESS>wrote:

@ericlindley-g commented on this pull request.
In templates/index.html
https://github.com/ampproject/amp-by-example/pull/1445#discussion_r210329717
:

@@ -6,12 +6,16 @@

I would say that if possible we should have the pages validate and have
the notice site-wide. Is there an option that does both? In the sample can
we just point to code that actually implements the consent for the page,
rather than a faked example?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ampproject/amp-by-example/pull/1445#discussion_r210329717,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcDn6ozJRZJFet0DvcApxAdq0Z0jB4hXks5uRE6igaJpZM4VzFES
.

Sounds good — I assume it's better to have the consent notifications on every other page than not, so I personally certainly wouldn't block on this.

+1 better most of the pages than nothing.

Can you take a look at the validation errors?

friendly ping "-)

Yes, sorry! I'm traveling and haven't gotten the chance to look at this.
Hopefully soon....
On Wed, Aug 22, 2018, 4:49 PM Sebastian Benz<EMAIL_ADDRESS>wrote:

friendly ping "-)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ampproject/amp-by-example/pull/1445#issuecomment-415157129,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcDn6hqPziKFGFoSQwmTA5ZxefSZ5Q6Yks5uTbXBgaJpZM4VzFES
.

Ok, I'm back and looking at this now.  There's quite a lot here.  Looks like stories require special treatment.  And in some cases the CSS is now just a bit too big....

@sebastianbenz , on pages like /stories/introduction/amp_story_hello_world/index.html, the <amp-consent> JS seems to be getting magically removed during the build process.  Currently head.html contains this:
{{^includesGeo}}
{{^includesConsent}}

{{/includesConsent}}
{{/includesGeo}}

If I put anything else in that HTML - say, a <meta> tag - it gets included. But the amp-consent bit gets replaced by a blank line.
I thought I'd defer to your expertise and experience here :)

More of the validation errors are occurring on pages like stories/introduction/amp_story_hello_world/preview/embed/index.html, which I believe only appear in the wild in iframes on pages like https://ampbyexample.com/stories/introduction/amp_story_hello_world/.
I haven't found the process by which these embed pages get built and by which the unnecessary  gets added.  Can you perhaps explain?  I could at least make these validate.  Or let me know if I'm missing something else.

And, finally... on pages like samples_templates/product_page/index.html, the CSS is now just a bit too large.  (On that page, it's 50919 bytes.)
Looks like example.html contains two additional CSS files:
      {{> css/example.css}}
      {{{exampleStyles}}}

it also appears that the extra CSS is not minified... if I'm right here?  From gulpfile.js:
const sampleTemplates = Templates.get(config.templates.root, /* minify */ false,
    '<% %>');

Is it ok to minify that extra CSS?  Or do you recommend another remedy here?

Oh dear. Will slowly work my way through your comments.

You have to update templates/preview.html to insert consent at the right position. It also needs to be a new template for amp-story-consent:

    {{{elementsAfterBody}}}
    {{^isEmbed}}

          <svg fill="#000000" height="24" viewBox="0 0 24 24" width="24" xmlns="http://www.w3.org/2000/svg">
              <path d="M19 6.41L17.59 5 12 10.59 6.41 5 5 6.41 10.59 12 5 17.59 6.41 19 12 13.41 17.59 19 19 17.59 13.41 12z"/>
              <path d="M0 0h24v24H0z" fill="none"/>
            </svg>

Demo: {{subHeading}}

          {{#metadata.experiments.length}}

            {{> experiment.html}}

          {{/metadata.experiments.length}}
          {{/isEmbed}}
          {{#document.sections}}
          {{{preview}}}
          {{#isAmpStoryTag}}
          {{> analytics.html}}
          {{> story-consent.html}}
          {{/isAmpStoryTag}}
          {{/document.sections}}
          {{^isEmbed}}

      {{/isEmbed}}
      {{^document.isAmpStory}}
      {{> analytics.html}}
      {{> consent.html}}
      {{> serviceworker.html}}
      {{/document.isAmpStory}}

CSS should now be minified: https://github.com/ampproject/amp-by-example/pull/1489 PTAL

@sebastianbenz , just merged your changes into my fork.
In the morning I'll look for a few minutes to see if I can find a neat way of fixing the problem you mention in #3 above. If I don't spot one, I believe I do see the mechanism for skipping validation:
"metadata": {
  "skipValidation": "true"
}

@sebastianbenz , looking at this further, I don't get why replaceAmpStoryRuntime() gets run on /stories/introduction/amp_story_hello_world/ in the first place.   It's there to remove certain tags from AMP Stories.  But the page that contains this example is not a story.

I'm looking now to fix the cases where a story embed appears as its own page, as in https://ampbyexample.com/stories/introduction/amp_story_hello_world/preview/embed/. There, I need to use <amp-story-consent> instead of <amp-story>, and both <amp-geo> and <amp-story-consent> need to be direct children of <amp-story>.
I imagine that to do this I'll need to change generateStoryPreviewEmbed() so that it includes my consent-story.html template inside <amp-story>.  Is that right?  Or is there an easier way?

MSTM (makes sense to me)!
Did these things, committed, and I see the build is still generating "CSS > 50000 bytes" errors on these two pages:
samples_templates/product_page/index.html
samples_templates/product_page/embed/index.html

Strangely, I don't see this error on my local machine.  In fact, for me, the CSS on the first of those pages is a paltry 45833 bytes.
Any idea what's going on here?  Is there a way to see the files Travis has built?

just wanted to ping this back up... ping!

I could reproduce it. You can fix it by replacing the content of templates/css/product.css with this version: https://gist.github.com/sebastianbenz/aa0d87bd2a3f10f3c880bd6d5764a88c

Whoop Whoop!

Too early - afaik you only need to disable validation for the story-consent sample. All other story samples should validate.

The build passes, but you can you try re-enabling validation for the stories (except amp-story-consent).

Sure.  Reenabled validation for two stories, as a test.
I also removed "us", which I don't think I'd ever left in there inadvertently before...

Yeah, sadly, the story embed pages don't work because they need <amp-story-consent>, which brings us back to the build issues mentioned above.

Woo!
Thanks for your persistence 😎


Actions spam redux dev tools

First off, thank you very much for an excellent library.
During development, I often use redux dev tools. When interacting with a form, a lot of "library-actions" are dispatched, and it becomes difficult to see my "app-actions" getting dispatched.
From a dev perspective, I'm thinking that library-implementation details should not leak out to the consuming app.

Is this a problem anyone else recognizes and would like to have solved as well?
Anyone know of an obvious solution? I'm willing to make a PR if I can get pointed in the right direction.

Hi @gurbraj, what are you worried with, exactly?
I could say you would like to see less/no @@redux-form/ actions among yours.
If that is the case, this would be a pretty major breaking change, and if you are to incorporate any other Redux library, it would bring the same problem.
I'm happy to see similar issues/solutions with any other libraries, though.

Hey @gustavohenke,

I could say you would like to see less/no @@redux-form/ actions among yours.

Yes, that is correct.
And I agree with you, if anything, this type of disabling functionality should come from the redux-dev-tools library, I'll have a look over there.

Okay, thanks @gurbraj. Let us know if you find something interesting. I'm always happy to learn more!

https://github.com/zalmoxisus/redux-devtools-extension/blob/master/docs/API/Arguments.md#actionsblacklist--actionswhitelist


Shield usage with villager recruits

When using with villager recruits and when they have a shield and a weapon they only use their shield. Only after the shield is broken they start to use their weapons.

Fix will be in the soon-to-be-released 1.1.2 version.


Sync master into develop

This PR syncs the latest changes from the master branch into the develop branch.

Failure


Add ConditionalDETR to HuggingFace Transformers

Hi!
As ConditionalDETR seems like some (relatively) minor modifications to the original DETR, it might make sense to add ConditionalDETR to HuggingFace Transformers to increase visibility and adoption. We do have the original DETR in the library, found here: https://huggingface.co/docs/transformers/model_doc/detr. This also comes with nice inference widgets on the hub, check out this one for instance (on the right -> you can directly try out DETR in the browser!): https://huggingface.co/facebook/detr-resnet-50.
The Python implementation is made in a single python script, found here: https://github.com/huggingface/transformers/blob/main/src/transformers/models/detr/modeling_detr.py.
So, if we want to add ConditionalDETR, it would have to be implemented in modeling_conditional_detr.py, which includes the modifications compared to modeling_detr.py.
Are you interested in adding this model to the library?
Kind regards,
Niels,
ML Engineer @ HuggingFace

Hi Niels,
Thank you for your message! That will be great, we are very interested in. I am kind of busy today. I will check out the implementation of DETR in HuggingFace and implement Conditional DETR based on it in the next couple days.
If I understand correctly, we will need to provide both the single-script implementation and the pretrained weights, right?
Thanks,
Depu

Yes that's correct! Maybe it might be easiest to set up a slack channel for easier communication. Is that ok for you?

Sure. I am not very familiar with Slack. Can you invite me to a slack channel? My email address
is<EMAIL_ADDRESS>


fix: width of contributors on Firefox

On Firefox right now

After the fix:

Thanks for the quick fix @manniL


hashchange: Handle when history object has been marked dead.

Observed in prod.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Strange. Merged, thanks @alexmv!


sc-2013 add code generation to workflow

Codecov Report

Merging #317 (ec4d37b) into main (74b22c5) will decrease coverage by 23.45%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #317       +/-   ##
===========================================
- Coverage   45.98%   22.53%   -23.46%
===========================================
  Files          61       26       -35
  Lines        7491     4158     -3333
===========================================
- Hits         3445      937     -2508
+ Misses       3403     3078      -325
+ Partials      643      143      -500

Impacted Files
Coverage Δ

pkg/trtl/trtl.go

pkg/gds/admin.go

pkg/sectigo/creds.go

pkg/gds/models/v1/contacts.go

pkg/gds/client/editor.go

pkg/gds/gds.go

pkg/gds/tokens/mock.go

pkg/gds/models/v1/models.pb.go

pkg/trtl/internal/pagination.pb.go

pkg/gds/config/config.go

... and 77 more

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 74b22c5...ec4d37b. Read the comment docs.


DeviceBootloader::flashUserBootloader() fails yet it returns success

On my oak-d-pro-poe, I have factory 0.0.17 bootloader.
I've written c++ code that uses the Bootloader apis to allow my users to update their firmware.
Using depthai-core 2.25.0 which has embedded bootloader 0.0.26.
The firmware update API returned true for success, yet the bootloader actually did not update to 0.0.26. It remained on 0.0.17.
Likely related to https://github.com/luxonis/depthai-core/issues/988
Pseudocode
deviceBootloader = std::make_unique<dai::DeviceBootloader>(candidate, true);
if (deviceBootloader->isUserBootloaderSupported()) {   // this returns true
    std::tie(success, resultMessage) = deviceBootloader->flashUserBootloader(localFlashCallback); // this returns {true, "blahblah"}
}
if (!success)
    throw
deviceBootloader.reset();
// wait
deviceBootloader = std::make_unique<dai::DeviceBootloader>(candidate);
assert(deviceBootloader.getVersion() == {0, 0, 17});

Unplugging/plugging mains power to POE does not help. The bootloader continues to be 0.0.17.
Suspicion
I suspect that the rules of https://docs.luxonis.com/projects/api/en/latest/components/bootloader/#danger-zoneg and bug #988 are interacting to result in the failure of this bug.
isUserBootloaderSupported() is coded to check against bootloader version 0.0.21. Since #988 returns the wrong version, the compare is 0.0.26 > 0.0.21 so Supported() returns true. But should it return false instead?
Also, as can be seen from the DEPTHAI debug log, the firmware update process did SOMETHING for 18 seconds and then declared it did the flash Success flashing the configuration userBlSize to '3824072' and userBlChecksum to '2142766317'
Below is the log, I add comments with // at the end of some lines
[2024-04-01 01:35:47.042] [depthai] [info] Search for 1 devices with criteria DeviceInfo(name=<IP_ADDRESS>, mxid=18443010318EF50800, X_LINK_BOOTLOADER, X_LINK_TCP_IP, X_LINK_MYRIAD_X, X_LINK_SUCCESS) + hint=false
[2024-04-01 01:35:50.212] [depthai] [debug] Found name=<IP_ADDRESS>, mxid=18443010318EF50800, X_LINK_BOOTLOADER, X_LINK_TCP_IP, X_LINK_MYRIAD_X, X_LINK_SUCCESS, hint=false
[2024-04-01 01:35:50.349] [depthai] [debug] Connected bootloader version 0.0.26   // this is std::make_unique<dai::DeviceBootloader>(candidate, true);
progress_callback() 35  // this is flashUserBootloader(localFlashCallback);
progress_callback() 35  // ignore these numbers, I am scaling them for a dialog box
progress_callback() 36
progress_callback() 37
progress_callback() 39
progress_callback() 40
progress_callback() 41
progress_callback() 42
progress_callback() 43
progress_callback() 44
progress_callback() 45
progress_callback() 46
progress_callback() 47
progress_callback() 49
progress_callback() 50
progress_callback() 51
progress_callback() 52
progress_callback() 53
progress_callback() 54
progress_callback() 55
progress_callback() 57
progress_callback() 58
progress_callback() 59
progress_callback() 60
progress_callback() 61
progress_callback() 62
progress_callback() 63
progress_callback() 64
progress_callback() 65
progress_callback() 67
progress_callback() 78
Exception thrown at 0x00007FFA9CB95B0C in Max.exe: Microsoft C++ exception: std::runtime_error at memory location 0x00000000007D9AC8.
[2024-04-01 01:36:08.087] [depthai] [debug] Error while trying to read existing bootloader configuration: No config available
[2024-04-01 01:36:08.753] [depthai] [debug] Success flashing the configuration userBlSize to '3824072' and userBlChecksum to '2142766317'
// here occurred if (!success) throw
progress_callback() 81
// here occurred deviceBootloader.reset();
[2024-04-01 01:36:08.758] [depthai] [debug] DeviceBootloader about to be closed...
[2024-04-01 01:36:08.762] [depthai] [debug] XLinkResetRemoteTimeout(5)
Exception thrown at 0x00007FFA9CB95B0C in Max.exe: Microsoft C++ exception: dai::XLinkWriteError at memory location 0x000000003B97FB20.
[2024-04-01 01:36:09.748] [depthai] [debug] DeviceBootloader closed, 989

Also possible this is a lack/ambiguous depthai documentation. Maybe since bug #988 and I have bootloader 0.0.17, that I need to first get the non-flashing bootloader version (which is 0.0.17) and then open a 2nd bootloader connection with flash=yes and ignore (don't call isUserBootloaderSupported() because it is wrong) and instead do my own logic compare of the bootloader version from the 1st connection (0.0.17) against a hardcoded api version constant (0.0.21) and then use that to choose flash to FACTORY or USER. And then with 0.0.26 (maybe?) now as the factory firmware I can do the same thing all over again with two bootloader connections to now detect with my own logic that a user bootloader is supported and I can then use flashUserBootloader() in the future.
I am suspicious what happened to my device's flash since the log shows 18 seconds of activity and Success flashing the configuration userBlSize to '3824072' and userBlChecksum to '2142766317'. It seems to work ok -- I get depth, color, and imu data.

@zrezke would you mind checking this out alongside #988 when you'll get to it?

@zrezke, I see PR #997 that will change the logic so that isUserBootloaderSupported() will use the new getFlashedVersion(). Looks straightforward. Now code that is written will fail the bool isUserBootloaderSupported() test and not call flashUserBootloader(). A flash to factory is the only possibility on such devices.
Does your team know the state of my 0.0.17 device? Since the code did call flashUserBootloader() due to the two bugs, and did something to flash as in the OP logs.
Can I ignore that activity?
Can I factory flash now? My goal is to get off 0.0.17 to see if PoE is more reliable on multiple restarts (other bugs I'm tracking)

@diablodale Yes - flashing factory bl now is fine - other than the soft-brick that could happen anyways, if the device gets disconnected during flashing the factory BL.


Statemachine check

Added dependency analysis and semantic checking to state machines Phase 1.
Closes #436. Closes #437.

Per discussion, we need to check the following rules:

A component containing a state machine instance must be active or queued.
An active component containing a state machine instance and no other async element is valid.

We also need to check that each state machine instance specifier in a component definition has a distinct name. I added this rule to the spec for component definitions.

I created a state_machine_instance directory for state machine tests
I added checks for incorrect symbols in state machine instances.


Better serverdetails and datetime command + :thru

Better serverdetails

The main serverdetail comamnd is now :serverinfo, much less confusing than the :details command
Serverdetails now display [ON] and [OFF] instead of Enabled and Disabled
Serverdetail field names now include spaces

Datetime command
The datetime command is a list command that returns os.date() values from os.time() in a table with formatting and easier understanding.
Thru command
A short alternative to the unofficial :to me that comes from the /thru in worldedit commands in minecraft that lets players pass through blocks.

Funny intercom go brrrr

e

Thru's description states it lets you pass through objects and such, but to me this just looks like another version of :to so why not just add thru as an alias for :to instead?

I stated that :thru is identical to :to me, it uses the :to me, it isn't made to teleport to someone

It's basically identical but more practical

Im thinking about adding a notification instead of an hint when sending a message, might do it in another pull request

I stated that :thru is identical to :to me, it uses the :to me, it isn't made to teleport to someone

yeah it just noticed the "me" part of that lol

I stated that :thru is identical to :to me, it uses the :to me, it isn't made to teleport to someone

yeah it just noticed the "me" part of that lol

aight

Also, maybe i'll do a pr, or if that's up to you, you can do it yourself, since Displaynames are rolling out, you could add a @ to things that display the .Name of a player, such as the pms, i also added a notification bit instead of the hint when someone pms

cool

shut


rails 5 support

@henders seems to work when playing with it on https://github.com/zendesk/samson/pull/972

Nice! 👍

Will create a new version in a min.


Functional configuration

There are a number of rendering options that might be useful to be exposed to the user.
Probably best way to do this is similar to go-spew: have a Config struct which has the Map function and then a global default options struct which is used by the package level Map function.

Do you need any help on this? Would like to contribute, you were a lifesaver this week. :)

Hey @nadilas, yeah that'd be great! What level would you be happy to contribute at?
If you want I can set up the basic framework (something along the lines of:
https://github.com/bradleyjkemp/cupaloy/blob/master/cupaloy.go#L13
and
https://github.com/bradleyjkemp/cupaloy/blob/master/config.go) and then you can work on the more interesting bit of actually adding config options.
Or you're welcome to do this boilerplate too if you want :slightly_smiling_face:

Hi @bradleyjkemp, If you can lay down the framework, I'd be happy to work on the configs. That way you'll have the base structure in your lib as you wanted to design it ;-)

Just added a basic config framework in https://github.com/bradleyjkemp/memviz/commit/1d8faf13bbfcc3a19622f9234b1837512a28f523 so I think you're good to go 🙂

https://github.com/bradleyjkemp/memviz/pull/41 is a baseline for this, let's discuss where you want to take it ☺


Add walletAddress to /ssx-nonce request

Description
This PR updates the /ssx-nonce request params to send the walletAddress. This change is required for early detection of a multisig login.
Type

[x] New feature (non-breaking change which adds functionality)

Diligence Checklist

[x] This change requires a documentation update
[x] I have performed a self-review of my 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] Any dependent changes have been merged and published in downstream modules

Codecov Report
Base: 72.10% // Head: 72.08% // Decreases project coverage by -0.03% :warning:

Coverage data is based on head (34b9150) compared to base (073a499).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
- Coverage   72.10%   72.08%   -0.03%
==========================================
  Files          25       25
  Lines        3334     3335       +1
  Branches      194      194
==========================================
  Hits         2404     2404
- Misses        930      931       +1

Impacted Files
Coverage Δ

packages/ssx-sdk/src/core.ts
24.51% <0.00%> (-0.07%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


osiris crash for add features from pull requests

only add fast stop and viewmodel weapon XYZ location changer

use debug.


Error retrieving next row on Spark Thrift Server when create table or view

Description
Connecting to Thriftserver with Hive driver, every time I try to create a table or view I get this error:

DBeaver Version
22.3.0
Operating System
Linux/Windows
Database and driver
org.apache.hive.jdbc.HiveDriver <IP_ADDRESS>.6.5.0-292
Steps to reproduce
Connect to a Thrift server, then:
create table tbl_test(id integer, code string) using delta;
or
create table tbl_test(id integer, code string);
Additional context
No response

Hello @andreachecchi
Could you please check the latest DBeaver version?
And please provide log files. You can find instructions here: https://dbeaver.com/docs/wiki/Log-files/. Thank you.

Thanks for your quick reply!
I have the latest version:

The log windows shows the same error I posted before:

You can find the debug log below:
head_dbeaver-debug.log
tail_dbeaver-debug.log
Hope this helps!
Thanks in advance

For me same for insert too. seems that sql queries which does not return any data they raise this exception.
It is the issue of the driver I belive.

Any news about this issue?
Thanks in advance

It is unclear for now how we can fix it from DBeaver's side.
Maybe changing this preference will help somehow?

It didn't work :(
Can I help you doing any other test?
Thanks


Invalidate current editor detail if backlinks do not match

Fixes https://github.com/subconsciousnetwork/subconscious/issues/857
Check if backlinks have changed as part of diffing the detail. This change perhaps suggests that we should decouple loading backlinks from the note contents itself in the editor?

After chatting to @gordonbrander we think it's more sensible to hold off merging this and split apart backlink refresh from editor content refresh.


Feature: Add fallback option in case of any failure

This is a feature request to have a fallback option in case any error occurred during migration.

As discussed in huddle, a few points to consider:

If there is a failure, abort the tool so multiple failures will not be hit at the same time across multiple PVC conversions.
Rolling back a failure is a non-goal due to the significantly increased complexity of the tool.
Logging and documentation of the tool is the most critical part to deal with failure. We want to guide users through a failure, but we don't need to automate it.


Misleading error message when no proper access for docker

Hi,
I was struggling yesterday with pulling an image from the GCR registry and I got the following error message multiple times:
Error response from daemon: unauthorized: You don't have the needed permissions to perform this operation, and you may have invalid credentials. To authenticate your request, follow the steps in: https://cloud.google.com/container-registry/docs/advanced-authentication
However, this is not the case, because the problem was actually that my system was not able to run docker without sudo. And with sudo I was not able to use the gcloud CLI.
I used this stackoverflow as a solution: https://stackoverflow.com/questions/48957195/how-to-fix-docker-got-permission-denied-issue/48957722#48957722
Just sending this out here so people in the future won't have the same problem and you might want to change the error message since the problem is not in the authentication of Google Cloud.

This should be updated in the Googlle Container Registry docs. It's an important note to mention. https://cloud.google.com/sdk/gcloud/reference/auth/configure-docker

what OS are you running this from?

However, this is not the case, because the problem was actually that my system was not able to run docker without sudo. And with sudo I was not able to use the gcloud CLI.

I think you could argue that it was the case, because you were acting as the root user, who did not have the needed permissions because you had not configured the root user's credential helper. I'd agree this is confusing, though.

What an important note to mention. I think this should be updated in the Googlle Container Registry docs

The docs on gcr.io mention this: https://cloud.google.com/container-registry/docs/advanced-authentication#:~:text=Note%3A If you normally,auth configure-docker instead.

I am facing this strange issue in one of my GKE cluster. In the same namespace, 2 pods are running for the same application - both the pods are trying to pull the same image but one is able to successfully pull the image and other one is failing with authentication failure -

Error msg is -
rpc error: code = Unknown desc = Error response from daemon: unauthorized: You don't have the needed permissions to perform this operation, and you may have invalid credentials. To authenticate your request, follow the steps in: https://cloud.google.com/container-registry/docs/advanced-authentication
Could somebody help me here. Also all my other pods in a different namespace are running without any issue.
How is it possible that one pod is running without any issue while other one is not having permissions to pull the image? Why GCR is allowing one pod to pull the same image and not the other pod? If I delete the running pod, it sometimes fails with permissions issue and sometimes not.


Documentation question

Could you tell me what tools you used to draw the following architectural diagram:

Thank you in advance
Adrien C.

The Icons are from -> https://docs.microsoft.com/azure/architecture/icons/
@pbousan did you use figma or something else for the diagram itself?

@jamesmontemagno @AClerbois  yes, we use Figma to draw the diagram.

Thank you @jamesmontemagno & @pbousan for your responses.
You rocks! 🤘🤘🤘🤘


add myhss

Hubert-Sternberg-Schule Wiesloch

@TodesRitter please provide us with:

the university official website URL, if different from the domain you are submitting
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the university
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the university recognizes the domain which you are submitting as an official email domain for the students.

Thank you!

school website : https://www.hss-wiesloch.de/
it related : https://www.hss-wiesloch.de/schularten-2/ausbildung/ausbildung-elektrotechnik/
it is a apprenticeship as a Software Developer which goes 3 years in germany
i hope this is enough for a proof:
https://puu.sh/w9sTq/cc0c0a3233.PNG
https://puu.sh/w9sTv/ca63818e01.PNG
https://puu.sh/w9sTp/7a4d212c54.PNG

@TodesRitter Thank you for the clarification! Actually, you have an address<EMAIL_ADDRESS>from the very same domain listed at https://www.hss-wiesloch.de, your official website, thus it's the proof. My problem was to find out what your official website is. Pull request merged now. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!

thank you!