How to remove link title when post using post_with_media

when i get new DM, i have download and process the image then upload with twit using post_with_media. but after success upload i got a link after twit
here we go https://twitter.com/MenfessWhisper/status/1214579879343034368?s=20

Can you show the relevant code? Tweepy does not have a post_with_media method.
If you mean API.update_with_media, this is likely something you're including in the status itself.

Can you show the relevant code? Tweepy does not have a post_with_media method.
If you mean API.update_with_media, this is likely something you're including in the status itself.

im sorry, my bad. i mean is API.update_with_media
`    def read_dm(self):
print("Get direct messages...")
dms = list()
try:
api = self.api
dm = api.list_direct_messages()
for x in range(len(dm)):
sender_id = dm[x].message_create['sender_id']
message = dm[x].message_create['message_data']['text']
message_data = str(dm[x].message_create['message_data'])
json_data = _json.encode_basestring(message_data)
print("Getting message -> "+str(message)+" by sender id "+str(sender_id))
            if "attachment" not in json_data:
                print("Dm does not have any media...")
                d = dict(message=message, sender_id=sender_id, id=dm[x].id, media = None)
                dms.append(d)
                dms.reverse()

            else:
                print("Dm have an attachment..")
                attachment = dm[x].message_create['message_data']['attachment']
                d = dict(message=message, sender_id=sender_id, id=dm[x].id, media = attachment['media']['media_url'])
                dms.append(d)
                dms.reverse()

        print(str(len(dms)) + " collected")
        time.sleep(60)
        return dms

    except Exception as ex:
        print(ex)
        time.sleep(60)
        pass

def download_image(self,media_url):
    arr = str(media_url).split('\n', 1)
    auth = OAuth1(client_key=constant.CONSUMER_KEY,
                  client_secret=constant.CONSUMER_SECRET,
                  resource_owner_secret=constant.ACCESS_SECRET,
                  resource_owner_key=constant.ACCESS_KEY)
    r = requests.get(media_url, auth=auth)

    with open(arr[9], 'wb') as f:
        f.write(r.content)
    image = Image.open(arr[9])
    image.save("ready.png")
    print("Media downloaded successfully!")

def post_tweet_with_media(self, message):
    print("Downloading media...")
    filename = 'ready.png'
    # tweet = tweet.replace("https://twitter.com/messages/media/1200393979625263109")
    self.api.update_with_media(filename, status = message )
    os.remove(filename)
    print("Upload with media success!")

`

Where do you use post_tweet_with_media and what is being passed as message?

post_tweet_with_media
message is string of twit.
from twitter import Twitter
import time

#test pancing ke heroku supaya keluar dyno
#test kedua pancing worker supaya muncul...

tw = Twitter()
def start():
    print("Starting program...")
    dms = list()
    while True:
        if len(dms) is not 0:
            for i in range(len(dms)):
                message = dms[i]['message']
                sender_id = dms[i]['sender_id']
                id = dms[i]['id']

                if len(message) is not 0 and len(message) < 280:
                    if "[whisp]" in message:
                        # message = message.replace("[whisp]", "")
                        if len(message) is not 0:
                            if dms[i]['media'] is None:
                                print("DM will be posted")
                                tw.post_tweet(message)
                                tw.delete_dm(id)
                            else:
                                print("DM will be posted with media")
                                tw.download_image(dms[i]['media'])
                                tw.post_tweet_with_media(message,dms[i]['media_ids'])
                                tw.delete_dm(id)

                        else:
                            print("DM deleted because its empty..")
                            tw.delete_dm(id)

                    else:
                        print("DM will be deleted because does not contains keyword..")
                        tw.delete_dm(id)

            dms = list()

        else:
            print("Direct message is empty...")
            dms = tw.read_dm()
            if len(dms) is 0:
                time.sleep(60)

if __name__ == "__main__":
    start()

You're passing in 2 arguments for post_tweet_with_media when only one parameter is expected?

You're passing in 2 arguments for post_tweet_with_media when only one parameter is expected?

sorry code is updated. my coding is fine i have already run but how i can delete link at the end

Ah, Twitter appends the media URL to the text of the DM object, similar to the way it does for Tweets.
You should be able to use the entities object in the message_data, which should contain the URL object that has the relevant indices to handle it, in the same way as for Tweets.

so how i can remove the link?

You can simply use the given indices to remove or replace that portion of the string.

You can simply use the given indices to remove or replace that portion of the string.

already did. thank you so much
re.sub(r"http\S+", "", str(tweet))


Watermark overlaps möbius-band

Show the watermark
resize the window
🐛 it doesn't look nice, watermark overlaps

actually not with the latest latest anymore


Collaboration

Identifiers:

https://github.com/dsw/proquint - Proquints: Identifiers that are Readable, Spellable, and Pronounceable.
https://observablehq.com/@keckelt/team-name-generator

Visuals:

https://jdenticon.com/
https://mantine.dev/core/avatar/
https://www.npmjs.com/package/react-user-avatar

Email: Create an identifier and store it in the extension settings, i.e., one identifier per Jupyter Client. Can be overwritten to use mail
Avatar: Create a jdenticon based on the identifier and a random color (as in the icon designer)
Can use the gravatar api by SHA256 hasing the idetnfier:

https://www.gravatar.com/avatar/estimated caterpillars?d=identicon
https://www.gravatar.com/avatar/7bc04b95eacfc50cb6b5f425fcf835c9b274b50f67617d3fd5fa7b3f59188ec2?d=identicon

Also JupyterLab has a way to setup users and access them. It’s mentioned on the extension development docs: https://jupyterlab.readthedocs.io/en/stable/extension/identity.html


Parameter substitution does not work for content.xml inside the web apps' META-INF folder

Hi,
good documentation here, however, does not make it clear that the e.g. ${db.url} substitution parameters do not work for a Java/Tomcat in an App Serevice when the context.xml is 'inside the .war' file in the META-INF folder.
Unless there is a way to get this to work??
I simply get errors such as
"Cannot create JDBC driver of class 'org.gjt.mm.mysql.Driver' for connect URL '${MYSQLCONNSTR_ecommerceDB}'

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

ID: 6f0940f3-b32a-cba8-3ff8-b30aec424b42
Version Independent ID: 6d40912c-076c-91d8-84f0-bbe5b4b3182c
Content: Migrate Tomcat applications to Tomcat on Azure App Service
Content Source: articles/java/migration/migrate-tomcat-to-tomcat-app-service.md
Service: azure-java
GitHub Login: @KarlErickson
Microsoft Alias: karler

@selvasingh or @mnriem can you comment here? Thanks!

See https://tomcat.apache.org/tomcat-9.0-doc/config/index.html

Exactly my point.
The server.xml and context.xml located in the tomcat/conf folder DOES get parameter substitution. as per the tomcat 9 docs.
However, your help page on migrating to Azure incorrently implies above that this will also occur for any context.xml file WITHIN the META-INF folder inside the deployed .war file. This is not correct.

@DaveFoxOpenGI @mnriem thanks for the feedback and info! I will create a work item to address this in the article.

Hi @DaveFoxOpenGI - @mnriem has tested, and has reproduced the error. He's determined that setting CATALINA_OPTS to "-Dorg.apache.tomcat.util.digester.PROPERTY_SOURCE=org.apache.tomcat.util.digester.EnvironmentPropertySource" makes it work. I will update the documentation with this information.

thank you.


Show other results from GOV.UK in a scoped search

Pivotal: https://www.pivotaltracker.com/story/show/
What
This PR exposes the top three search results from the whole of GOV.UK in a search that has been scoped to a single document.
Before:

After:

Summary of commits:

6c3d93f, dc90a5e, f9c24d7 and 318731b (the first three commits) are refactoring done as part of the prep work for adding this feature.
5b7049b modifies search_api.rb to add an the additional call to rummager for three GOV.UK wide search results. Tests for this request are also included in this commit.
804af74 refactors out scoped search functionality to a new class ScopedSearchResultsPresenter and adds tests for this new presenter.
e4c0154 is where the bulk of the changes are. This commit takes the new search results returned by the changes to search_api.rb in 5b7049b, and formats them properly and displays them in the view.
248d2e8 is CSS for the new results pull out.

How to review this PR
I've done quite a lot of hindsight based refactoring so each commit should make sense in isolation. Review however you prefer.
Who should review this PR

Anyone in the core team is especially encouraged to pile in.
I paired with @evilstreak, @jennyd, and @binaryberry on different bits of this at various times.
@rivalee / @markhurrell if you could check you're happy with the design and the micro-copy is what was agreed with the content team, I know it changed quite a bit and I'm not sure I got the right stuff in the end.
Anyone else with an interest in search/rummager/manuals

This PR is too advanced for me to review but :+1:  on the detailed commit messages!

I have had a quick look and this looks great to me. Thanks for taking the time to make the commits really easy to follow.

I'm very excited to see this, it looks shiny. :sparkles:.

Design looks good to me and the copy is correct!

Pedantic style comments, but otherwise seems to do the trick :+1:

OK I think I've responded to all the comments now, thanks everyone!

Looks good to me :+1:

:+1:


Add DAG to report reported media pending review

Fixes

Fixes #354
Description

Adds a report_pending_reported_images DAG that runs weekly and for each media type:

checks to see if there are any reported records in the pending_review status, meaning they require manual review through the Django Admin
reports the number of distinct records needing review to Slack, as well as a breakdown of the reasons for reporting (dmca, mature, other) and links to the admin page for that media

Additional Notes

I verified that if a single image has multiple reports in pending_review associated with it, when you update any of these records all the other records associated to that image are updated as well. (This is why I think it's best to report distinct images requiring review)
When new media types are added, this will require you to add the name of the reports table in the configuration.

Testing Instructions

This requires a connection to the API DB (rather than the Catalog), so you'll need to create the connection env variables. You can use the values set in env.template in this PR. Make sure you also have the API running locally. To test the slack messages you can create the Airflow variable slack_message_override and set it to true.
Run the report_pending_reported_media DAG locally. If you're running sample data, there should be no reports and you should see the No records to report message.
To create sample data to test the report count, you can report some images and audio (locally!) through the API. I went to localhost:8000/v1/images and grabbed the id for a couple of the listed images. For each I then navigated to localhost:8000/v1/<id>/report and POSTed a report. You can verify that these went through by checking out http://localhost:8000/admin/api/imagereport/? Likewise for Audio. Report a mix of different reasons (some 'mature', some 'other', etc).
Run the DAG again and make sure the correct numbers are reported. I also played around with manually reviewing some of the media reports (updating their status to something other than pending_review in the Admin) and verifying they were no longer included in the count; and also reporting the same records multiple times and verifying that it is only counted once in the output.
Checklist

[ ] My pull request has a descriptive title (not a vague title like Update index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

However, what is actionable about this information? Do we have documentation about how to handle these reports or guidelines on expectations (like SLAs)?
Until we have those things I don't think they should go into the alerts channel, which should ideally only include actionable information. That can either mean not enabling the DAG until we've developed those processes or temporarily putting them into our notifications channel until they're actually actionable.

These are really important questions and to my knowledge we don't have the answers right now. It's possible that depending on what processes we introduce, the alerts may even need to be directed to an entirely different channel eventually. For the moment I think there's value to at least gaining insight into the problem as a first step.

Would it be distracting to redirect them into the notifications channel for the time being then? It'd be nice if things in the alerts channel were things that needed immediate attention and had clear actionability

I see your point, but it doesn't feel quite right to me to move this to notifications. Reported media is a problem and does require attention, and while the processes/SLAs for acting on them aren't clear, just moving it out of the channel seems like it would increase the likelihood of it getting lost and unaddressed.  The notifications channel is also much more busy than alerts.
I think I may have the unpopular opinion here 😄 I can remove the notification for when no reports are found, and move the other one to the notifications channel. We would just need to be very deliberate about following up. Alternatively we can just wait on this PR until the process is solidified, since it's certain to change.

Reported media is a problem and does require attention, and while the processes/SLAs for acting on them aren't clear, just moving it out of the channel seems like it would increase the likelihood of it getting lost and unaddressed.

This does make it sound like there's some action you're expecting to take when these come through :slightly_smiling_face:  Even if the actions today are temporary "we're still figuring out what this means/documenting it/etc" it'd be nice to write that down and link to it from the notification so anyone is immediately able to either help progress that forward or know that they can ignore it and someone else who is keeping tabs on the particular project will come around to it after them.

I personally feel like the particularities of the channels and response to these notifications is an implementation detail we can handle outside of the repository. To that end, I've started a dialog with WP Photo directory folks about potentially sharing moderators and moderator policies. Until then I can personally handle these reports weekly.

I created #550 to track writing a runbook for handling reported media, and adjusting Slack notifications as necessary.


Scrutinizer CI

With a few recent documentation and code style discussions, I believe it would be valuable to use Scrutinizer CI. Unfortunately I don't have admin access to the repo so someone else will need to get it going. @ramsey
This is the .scrutinizer.yml file that I've used in the past, which I believe will be a good starting point.
filter:
    excluded_paths:
        - 'test/*'

tools:
    php_code_sniffer:
        config:
            standard: "PSR2"

I don't have admin access to the repo, either, so we'll need to pester one of the repo @thephpleague/owners to help set it up.

Nevermind. I was trying to sign up with "league/oauth2-client" instead of "thephpleague/oauth2-client." I apparently do have admin access to the repo.

Thanks, @philsturgeon, for today's fortune cookie reading!
@rtheunissen This is now working.

Thanks @ramsey. I'll see what I can do to resolve some of the issues that came up. PR's incoming. :)


Improve UI of project

Improvements to the overall UI of the website would be appreciated

Can you assign this issue to me ? @Arnavsmayan


Fixed variable used to set Yeelight color

availability was the text of the status which cause an exception.
busy is the color hex

Good catch, thanks for the PR!

It would be even better if I had actually looked at the value of the string that was being passed into the function instead of tracing every part of the configuration reading. But it was new code to me.


Maybe fix macro error

It worked for triangle example for me, maybe it fixes everything?

Thanks!

Unfortunately, this is not a solution, judging by the errors.


teams and high chain links

When playing new links on a team's sigchain, add a couple fields to the TeamSigChainState so we always know which is the last high link. Then we can start signing over and sending up the most recently seen high link for all new links. Performance improvements here we come. 🔥

Since the cache is persistent after this patch clients will be able to load things out of the cache that either do or don't have lastHighSeqno set in TeamSigChainState. Simplest thing would be to bust the cache but then everyone would have to reload their teams so.. nope. You could keep this in mind and any code written in the future should be aware that if GetLatestHigh{Seqno,LinkID}() returns 0 it should turn around and bust that team out of the cache and try again.
Also are we supposed to be storing LinkIDs or SigIDs? Whichever the hprevs are.

hprevs are linkIDs

For some reason it seems the prev hash is called a LinkID but I'm not sure why

linkID is the sha256 hash of the outer v2 link

This function gets called an unfortunate 3810 times for keybasefriends. prevAdmins eats 400ms and wasHighLink 200ms on my laptop when cold-loading keybasefriends.
Sorry, I had a hunch and measured
I suspect using sanityCheckMembers or its output roleUpdates and checking whether there's any admins or owners involved there would be faster. Checking for promotion is just checking whether there's the new owners or admin lists are populated. Checking for demotion though still involves fetching from the UserLog. But only for the number of removals when there are removals, vs now where it's once per admin/owner link.

@mlsteele i'll go down this path if you can give me another performance test in a little bit. one quick thought though: changing a user from owner to admin (or vice versa) wouldn't be a high link, so I'm going to need to check that user's previous role anyway. :/ splatting the logic into the case statement instead of doing it all at once should dramatically reduce the number of times it runs. The big tradeoff will be obviousness of the code, but I'll see if some comments and descriptive variable names can't mitigate that. we might also be able to accomplish the same performance improvement with a couple extra if statements. i can explore both.

Yep I can do that or share some setup.
It's true that a link the changes an owner to an admin is not technically in the minimal set of high links. I think it would be ideal to match the server. Which considers such links high. I'm hoping that the server can check the client by asserting that they exactly agree on the 'observed high link set'. Idk whether we should tighten the server or use that same check here on the client. @heronhaye
It seems like it will be pretty clear but easier to forget if another link type is added in the future that admins or owners. I think the if sprinkling alternative would result in sprawling control flow.

Total time when cold-loading keybasefriends. Solid 1000x improvement.
chain.isHighLink [time=21.919µs]
chain.roleUpdateChangedHighSet [time=654.135µs]


Update README

Rev 1

Fix wrong element being used in example
Add info about default onErrorMessage behavior

More to follow this month if time permits me, for instance, support for nested error messages when having a nested object structure which requires validating.

Thanks for the updates


Check if a library repo has updates but no release.

After the initial addition of a library to the bundle, any new release of that library will generate an update to the bundle. However, there are sometimes where a library has received some updates, but no release is done. This could be intentional or accidental.
Add some kind of check that can warn when this situation exists for any library included in the bundle.

I've got an early iteration running for this. It's using two API requests:

Request the latest release.

If no release is returned, the repo is flagged for needing a release.
If a release is returned, we use the published date for use in # 2.

Request any commits since the last release date.

If no commits are returned, we're all good and the repo isn't flagged.
If commits are returned, the repo is flagged for needing a release.

This is the only approach I could discern from the available API requests. The downsides are that it can increase the API requests by two for each repo (approx 800 total for 100+ repos; ~600 total without this), and also increases the time it takes adabot to run. Not huge deal breakers...
Now, with # 2, I'd like to filter the commits and ignore things that don't need releases; like /docs and repo management stuff (.travis.yml, etc). That will require an API request for each commit found, and balloon API requests and time-to-run.
Wanted to ask if those are acceptable trade-offs... @kattni @tannewt

I was hoping this would be a single call to the API. I wonder how they do it for the releases page.
You are right that it takes multiple calls as is. I'd probably use the git commit hash instead because dates can get funny with commit ordering (merge and rebase can do funny things). That'll take three calls I think. release -> tag -> commit list.
I wouldn't bother with trying to detect commits we don't need to release. Instead, lets make releasing so easy we can do it on everything as needed.

I was hoping this would be a single call to the API.

So was I. But it turns out, we're in a little bit of luck. I didn't look close enough, or maybe outside of the box enough.
Compare two commits to the rescue!
Examples: https://api.github.com/repos/adafruit/Adafruit_CircuitPython_Bundle/compare/master...20181030 (today) will give us this little gem:
},
  "status": "identical",
  "ahead_by": 0,
  "behind_by": 0,
  "total_commits": 0,
  "commits": [

  ],

Back the compare up to the release-tag-before-last: https://api.github.com/repos/adafruit/Adafruit_CircuitPython_Bundle/compare/master...20181024
  "status": "behind",
  "ahead_by": 0,
  "behind_by": 10,
  "total_commits": 0,
  "commits": [

  ],

Ca ching! Keeps us at two API requests: 1) get latest release tag, 2) compare to master. I might be able to get the list of commits; docs say something about passing the correct 'media_type'. Will keep digging.

I wonder how they do it for the releases page.
Yeah, that's what lead me to believe there was some one-request-magic. Looking at a release page's source...there is a lot of buried JS on the server side, so they're keeping that secret to themselves.

Two is awesome! Great find @sommersoft


[SA2] PS5 controller not working as Launcher is not bypassed

When clicking Save and Play, the default Sonic Adventure 2 Launcher is brought up (Meaning I also have to reset resolution settings despite setting them in the Mod Manager), and since SA2 doesn't natively support the PS5 controller the Launcher doesn't detect a gamepad and force sets it to Keyboard.
I can go into the keyboard cfg file to manually change this, but this resets when the Launcher opens.
Honestly just a way to bypass the launcher would solve this issue right away, is there a way to? As I cannot find one within the Mod Manager.
The old way would have been to rename sonic2app to Launcher, and rename the existing Launcher to something else, but the new Mod Manager seems to bring up errors and crash when the filenames are changed. Any way around this?

When you click on Save and play, the Mod Manager starts the game directly (sonic2app.exe). If it starts the launcher, that means you still have the exe files renamed which you aren't supposed to, renamed exe is overall bad practice.
That said, if you want to bypass the launcher (like when starting the game from stream), you can add -q to the commands in Steam, that will bypass the launcher.

Oh huh, I could have sworn I'd reset them properly initially but it works now! I know that's generally bad practice anyway. Thanks for the info about the command to bypass it on steam though! That's handy to know. Thanks for answering my silly query


make the default formatter overridable

simply makes the default formatter overridable, so users can define their own
time formats and customize other tokens

+1


feature: include details under sources for all devices on the network

Relates to https://github.com/SignalK/n2k-signalk/pull/154

I'd like to try this out for real, will take a few days.
I'd like to include this data in the multiplexed data log - what do you think?

Since the PGNs are logged, this does get into the the multiplexed logs.

🤦♂


use coq_makefile Fixes #35

Something I'm not sure about is how to check that dependencies are up to date. Seems like a job for CI.
What I would like is a CI check that fails if rerunning coq_makefile -f _CoqProject -o Makefile changes anything.

coqdep is called to calculate module dependencies everytime, so that is not a worry. But it would still be nice to fail a check if the dependency change was not committed.
> make
COQDEP VFILES
COQC matches_pred.v
COQC inductive_predicate_experiment.v

This is very cool.
@paulcadman has setup github actions as our CI.  So maybe he can help?
But I don't think that should be a blocker in merging this, only another incremental improvement later.
This only important thing is that we check that the CI is still calling the makefile to do actual checks of the proofs.

And this seems to be the case :)

Looks like the dependencies won't be a problem. They are generated everytime.


Error Project Load

Dear DomCR,
I see you has miss test in project, can upload, thanks you.I see project useful.

Hi @chuongmep,
The tests are in a early stage, in which I'm looking for the best way to test the library, I will publish the tests project as soon as they are useful.


Issue 1218 beta function and gradients

Summary
As discussed in #1218, several functions call exp(lbeta(x,y)) which in turn relies on autodiff for the gradients.
This PR adds the beta function and it's gradients. It also adds the rev implementation of lbeta and associated unit test
Tests
Unit tests for beta have been added, testing the values and checking that the same gradients/results are returned by exp(lbeta())
Unit tests for the new rev implementation of lbeta also added
Side Effects
N/A
Checklist

[X] Math issue #1218

[X] Copyright holder: Andrew Johnson
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[X] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[X] the code is written in idiomatic C++ and changes are documented in the doxygen

[X] the new changes are tested

@andrjohns --- please ping me when this is ready to review again.  Thanks.

@bob-carpenter Ready to review again
Jenkins is failing, but I don't think it's related to this pull:
org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:

WorkflowScript: 42: Method calls on objects not allowed outside "script" blocks. @ line 42, column 20.

               when { branch 'master' }

Hey @serban-nicusor-toptal - looks like this PR had testing issues, could you make sure it gets tested correctly? Thanks!

Refactored and fixed Jenkinsfile.
Doing a final test on the master Jenkins and will replay this build.

Everything should be back to normal now.
I'm sorry for the false-positive.
If anything else occurs please let me know.

@serban-nicusor-toptal looks like this has caused some more issues sorry!
git submodule foreach --recursive git clean -xffd

Entering 'cmdstan'

Entering 'cmdstan/stan'

Entering 'cmdstan/stan/lib/stan_math'

Entering 'example-models'

Entering 'stat_comp_benchmarks'

+ cd cmdstan

++ git status --porcelain

+ dirty=' M stan'

+ '[' ' M stan' '!=' '' ']'

+ echo 'ERROR: Git repo isn'\''t clean - I'\''d recommend you make a separate recursive clone of CmdStan for this.'

ERROR: Git repo isn't clean - I'd recommend you make a separate recursive clone of CmdStan for this.

+ exit

+ mv performance.xml develop.xml

mv: rename performance.xml to develop.xml: No such file or directory

sh: line 1:  9201 Terminated: 15          sleep 3

script returned exit code 1

@andrjohns Looking into it!

@andrjohns
Jenkins is good now. I've restarted the Job.
Please let me know if anything else happens.
Thanks!

@serban-nicusor-toptal  Thanks!

Hey @andrjohns @seantalts
The build failed because of the change in CmdStan Performance Tests.
See: http://d1m1s1b1.stat.columbia.edu:8080/job/CmdStan Performance Tests/job/downstream_tests/19/console
Explanation:
If you look at: https://github.com/stan-dev/performance-tests-cmdstan/blob/master/Jenkinsfile
perfReport compareBuildPrevious: true,

                    relativeFailedThresholdPositive: 15,

                    relativeUnstableThresholdPositive: 10,

                    errorFailedThreshold: 1,

                    failBuildIfNoResultFile: false,
                    modePerformancePerTestCase: true,
                    modeOfThreshold: true,
                    sourceDataFiles: '*.xml',
                    modeThroughput: false,
                    configType: 'PRT'

If there is a performance decrease that exceeds relativeFailedThresholdPositive: 15%  it will be marked as failure.
I now need your call on how to proceed with this one.
Thank you.

This is nice that it's checked... just the presentation of it is a bit not obvious... need to get used to it.
Just as a FYI: I saw that the SIR model ran 15% slower. I have merged earlier today into develop a PR which speeds up the SIR model by 18%. So maybe you just need to merge develop into this PR to "fix" this performance regression (assuming this PR has not yet merged these changes and that the comparison is made against develop).

@wds15 Thanks for the tip, looks like that was the issue. I ran the performance benchmarks locally and everything lined up after I merged develop in
@serban-nicusor-toptal Thanks for notifying me, things should pass now that I've merged develop in

This has been approved and passed tests, safe to merge


Fix Issue #11384 - Explaining how to load external backends in the documentation

Summary
Created additional documentation for the "Switching from one backend to another" section of the documentation, including an example of an edited keras.json configuration file.
Related Issues
Solves issue #11384
PR Overview

[ ] This PR requires new unit tests [y/n] (make sure tests are included)
[x] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[x] This PR is backwards compatible [y/n]
[ ] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Not sure why the most recent commit failed the travis CI testing, as the commit just before that managed to pass. The latest commit passes all tests except for Python 3.6 KERAS_BACKEND=cntk PYTHONWARNINGS=ignore. If anyone has any solutions please let me know

@gabrieldemarmiesse  Its supposed to be a module from a package rather than a standalone file. Like "backend": "mxnet.keras", assuming mxnet guys have added a keras sub module which is a valid backend.

@farizrahman4u do you have an example of backend which is working is ready to be loaded with this method?

@gabrieldemarmiesse Not open source (yet) :(

Ok I suppose that we can suppose that we have a backend called my_package.my_backend.
@JoshuaRM Can you replace mxnet by this dummy backend my_package.my_backend? Thank you.

@farizrahman4u  Could you review? I pushed a commit so I can't review this PR.


Organizr theme issues

Describe the bug
I'm getting error 404 saying that Oragnizr.css file is not found when using any of the themes and looking up in the repo it is in fact missing from the organizr folder. Also when using the nginx subfilter method with the nginx variable described in the docs, i can't use the commands set $app <app>; include /config/nginx/theme-park.conf; for the organizr location block since it messes up with the rest of the applications and the theming is not working as i'm accessing them as subfolders and not as subdomains. I have to manually add the subfilter block for organizr in the location block in order to get it working as it should be.
Theme Install Method
Webserver subfilter Config
Application Info
Version: 2.1.426
Branch: Develop
Container: Windows Setup
To Reproduce
Steps to reproduce the behavior:

Go to 'Organizr'
Open 'Developer Tools'
Check in the network tab for any 404 errors about organizr.css

Describe the bug
I'm getting error 404 saying that Oragnizr.css file is not found when using any of the themes and looking up in the repo it is in fact missing from the organizr folder. Also when using the nginx subfilter method with the nginx variable described in the docs, i can't use the commands set $app <app>; include /config/nginx/theme-park.conf; for the organizr location block since it messes up with the rest of the applications and the theming is not working as i'm accessing them as subfolders and not as subdomains. I have to manually add the subfilter block for organizr in the location block in order to get it working as it should be.
Theme Install Method
Webserver subfilter Config
Application Info
Version: 2.1.426
Branch: Develop
Container: Windows Setup
To Reproduce
Steps to reproduce the behavior:

Go to 'Organizr'
Open 'Developer Tools'
Check in the network tab for any 404 errors about organizr.css

The themes can be found in the Theme Marketplace menu in Organizr.

https://docs.theme-park.dev/themes/organizr/#installation


Cannot run nn.Recurrent(8,nn.LookupTable(10000,10),nn.Linear(10,10),nn.Sigmoid(),5) recently

Before I reinstall torch and rnn module, everything is OK. But recently I reinstalled torch and rnn module for some reason, then I found that the following error would raise when I run code
require 'rnn'
r=nn.Recurrent(8,nn.LookupTable(10000,10),nn.Linear(10,10),nn.Sigmoid(),5)
error message:
[string "r = nn.Recurrent(8,nn.LookupTable(10000,10),n..."]:1: attempt to call field 'Recurrent' (a nil value)
stack traceback:
[string "r = nn.Recurrent(8,nn.LookupTable(10000,10),n..."]:1: in main chunk
[C]: in function 'xpcall'
/home/xxx/torch/install/share/lua/5.1/trepl/init.lua:679: in function 'repl'
...ngli/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:204: in main chunk
[C]: at 0x00405d50
anybody can tell me how to fix it?

I believe nn.Recurrent is not supported here anymore I believe. You should be able to achieve similar results by using the nn.Recurrence module though I think. Check this out. https://github.com/torch/rnn/blob/master/doc/recurrent.md#recurrence


Refactor to more readily support custom handling of individual (mainly external) commands

Up to now, the "commands" resholved sub-resolves arguments for is mainly focused on builtins (though it does handle a few external commands with similar call patterns), most of which are handled by a small number of patterns.
The slowly-expanding list of cases of external commands (xargs, sudo, tar, find, etc.) likely to have arguments that should ideally be resolved is already making it obvious that resholved needs better support for mapping commands to both re-usable generic handlers, and bespoke single-command handlers.

This refactoring is complete as of the 0.6.0 release.


Update antd-open-browser

Description

change https://ant-design.antgroup.com to https://ant.design
edit data.json, sync component

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

ref: https://github.com/raycast/extensions/pull/8733


ElasticProxyFilter with Thai Character

Hi, I got error message below when trying to search with "Thai character".

"{"error":{"root_cause":[{"type":"parsing_exception","reason":"[terms] failed to parse field [field]","line":1,"col":1388}],"type":"parsing_exception","reason":"[terms] failed to parse field [field]","line":1,"col":1388,"caused_by":{"type":"json_e_o_f_exception","reason":"Unexpected end-of-input in VALUE_STRING\n at [Source: org.elasticsearch.transport.netty4.ByteBufStreamInput@63953f89; line: 1, column: 2813]"}},"status":400}"
But it'll work fine when i login as Administrator

It working fine with normal search and English character.

I try to debug a source code and found that it occur from ElasticProxyFilter.java
If code run into this red rectangle zone for modify request parameter with "English character". It work fine. But if content contain "Thai Character" It will return above error.

If i login as administrator account it will work both Thai and English case. Because it will skip all of those lines and sent request with original request that contain "Thai character".
Do you have any idea or guide me to fix these case. (I already try to save these file with encoding utf-8 but still not work.)
Thank you.

Solved by create new endpoint.


Crowdsale token price rate is fixed

Users should be able to customize the price function for Crowdsale contracts.

See #426 and #527 for proposals to solve this issue.


ClusterClient.TxPipeline does not execute MULTI/EXEC

Expected Behavior
// TxPipeline acts like Pipeline, but wraps queued commands with MULTI/EXEC.
func (c *ClusterClient) TxPipeline() Pipeliner {
	pipe := Pipeline{
		ctx:  c.ctx,
		exec: c.processTxPipeline,
	}
	pipe.init()
	return &pipe
}

Current Behavior
output:
get key
set key value ex 1
func main() {
	ctx := context.TODO()
	client := redis.NewClusterClient(&redis.ClusterOptions{})

	client.AddHook(hook(func(cmd redis.Cmder) {
		fmt.Println(cmd)
	}))

	pipe := client.TxPipeline()
	pipe.Get(ctx, "key")
	pipe.Set(ctx, "key", "value", 1 * time.Second)
	pipe.Exec(ctx)
}

type hook func(cmd redis.Cmder)

func (h hook) BeforeProcess(ctx context.Context, cmd redis.Cmder) (context.Context, error) {
	h(cmd)
	return ctx, nil
}
func (h hook) BeforeProcessPipeline(ctx context.Context, cmds []redis.Cmder) (context.Context, error) {
	for _, c := range cmds {
		h(c)
	}
	return ctx, nil
}
func (h hook) AfterProcess(ctx context.Context, cmd redis.Cmder) error {return nil}
func (h hook) AfterProcessPipeline(ctx context.Context, cmds []redis.Cmder) error {return nil}

Possible Solution
ClusterClient is not friendly to use TxPipeline, we can consider deleting it?

these commands are executed.

@monkey92t sorry for the delay and thanks for the working on Redis mock :+1:
Tx pipeline was working properly but hooks were not. This should be fixed by https://github.com/go-redis/redis/pull/1596

@monkey92t sorry for the delay and thanks for the working on Redis mock :+1:
Tx pipeline was working properly but hooks were not. This should be fixed by https://github.com/go-redis/redis/pull/1596


Update npm6 to allow for better semver handling

Often when using git, github or gitlab style semver dependencies with npm (ex, github:Brightspace/d2l-activity-alignments#semver:^2) the tags/branches associated with the repository can be for the form v[major].[minor].[patch] (common if following standard semver set-up s and automated tooling such as semantic-release) or [major].[minor].[patch]. The semver keyword allows for variance of branch name allowing v3.0.0 and 3.0.0 to be treated as the same version allowing these dependency checks to succeed. When not using the semver keyword in the dependency install command the resolved version will be treated as needing to be an exact match for a branch/tag name. This change adjust the install command used to update package lock file to include the semver keyword whenever it's present in the existing requirement which allows this variance in branch naming to be allowed. Once this change is done we also need to repair the from properties of the installed dependencies since they will no longer match the existing requirement in the package.json file (repair github:Brightspace/d2l-activity-alignments#semver:3.0.0 to github:Brightspace/d2l-activity-alignments#semver:^3).
Example of something that wouldn't work before
The github:Brightspace/d2l-activity-alignments#semver:^2 dependency is attempting to be updated to github:Brightspace/d2l-activity-alignments#semver:^3 which results in the install command of
npm install d2l-activity-alignments@github:Brightspace/d2l-activity-alignments#3.0.0

This causes a git error since the branch on the target repository is v3.0.0.
After change
The github:Brightspace/d2l-activity-alignments#semver:^2 dependency is attempting to be updated to github:Brightspace/d2l-activity-alignments#semver:^3 which results in the install command of
npm install d2l-activity-alignments@github:Brightspace/d2l-activity-alignments#semver:3.0.0

This succeeds even though the tag name is v3.0.0.
Notes
Repository used for testing of changes along side existing tests https://github.com/devpow112/dependabot-core-test. This would just throw errors when trying to run dependabot

@lseppala for sure. I'll work on adding some tests to the updater file stuff as well as looking why the latest test runs failed. Thanks for taking a look.

With regards to the test failure, there was an external dependency we used in testing that had drifted. We corrected this in #4466. Update your branch with the latest in main (or click the "Update branch" button below) to fix :)

#4485 #4482 #4460 #4483 #4482


Enable unittest jobs for windows

Great! Let me know when this is ready for review :)
The plan will be to merge #604 and close #493, right?

@peterjc123 Could we completely separate windows and linux unites scripts?
It appears to me that mixing them makes it harder to maintain.
I can move the current scripts from .circleci/unittest/scripts to .cicleci/unittest/scripts/linux. And windows specific files should go to .circleci/unittest/scripts/windows.

@mthrok okay, I'll make the change if you move the existing script to that place.

Great! Let me know when this is ready for review :)
The plan will be to merge #604 and close #493, right?

Yes, and It's ready for review but I'll consider @mthrok 's suggestion.

The unit test scripts for linux are designed to work together with the underlying Docker image. This new Windows unit test does not follow the same structure, so it's better to separate Windows scripts and linux scripts for a long term maintainability.

I'll do it a.s.a.p

@peterjc123 #605 I moved all the linux unit test tools to .circleci/unittest/linux. (slight change from what originally suggested) Can you approve? then I can merge it. Also you can put all the necessary scripts under .circleci/unittest/windows.

@mthrok approved and will do that.

Yes, and It's ready for review but I'll consider @mthrok 's suggestion.

sounds good!
nit: the PR is still marked both as draft and [WIP] :)

@vincentqb Marked as ready in both ways.

@peterjc123 -- do you want to also enable nightlies for windows?

@vincentqb Will do that later.


prefix lookup implementation

add prefixLength as a config option to the DHT
lookups: if prefixLength is set, only lookup prefix of hash(CID)
queried nodes: if the query key is a prefix (ie. <32 bytes), look up and return providers via prefix
also return the full key the provider providers, so the lookup node knows what peers to discard
had to modify the Message_Peer message type to include the key the peer provides (maybe there's a nicer way to do this?)

Would it make things easier to add a Prefix type? It may be easier to manage prefix length and could be more efficient than bit strings in the map[string][]*peer.AddrInfo


Revert "fix: disable votes ddos protection to test steps syncing"

Revert #1933

prnet cleaned-up successfully


CORS settings enabling PUT do not work

Describe the bug
When configuring CORS settings at the Ambassador module level, enabling PUT is not honored, PUT requests stll fail CORS preflight.  POST, GET, and OPTIONS all succeed.
To Reproduce
Steps to reproduce the behavior:

install ambassador from helm chart as follows:
helm install -n ambassador -f helm-ambassador-defaults-local.yaml stable/ambassador --version 3.1.0

For the above, use this values file: https://gist.githubusercontent.com/digressionary/09e3419b5d3d9b24ef3e6ab001a387c4/raw/bc64731274348d725c229a3390860ee81f740788/values.yaml
Expected behavior
Making a PUT request from Origin A to source B should be permitted by CORS, preflight checks should pass, and AJAX requests should be able to complete.
Versions (please complete the following information):

Ambassador: 0.73
Kubernetes environment [Docker for Desktop and self-maintained cluster deployed on AWS]
Version [1.12.0]

Additional context
Add any other context about the problem here.

@digressionary I have not been able to replicate an issue with CORS.
Using this ambassador Module (CRD):
---
apiVersion: getambassador.io/v1
kind: Module
metadata:
  name: ambassador
spec:
  config:
    cors:
      origins: "*"
      methods: POST, GET, OPTIONS, PUT

I tested by manually sending a pre-flight options request with curl to see the headers returned which look correct.
$ curl \
> -H "Origin: http://example.com" \
> -H "Access-Control-Request-Method: POST" \
> -X OPTIONS \
> -v http://<IP_ADDRESS>:30942/backend/debug/
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 30942 (#0)
> OPTIONS /backend/debug/ HTTP/1.1
> Host: <IP_ADDRESS>:30942
> User-Agent: curl/7.63.0
> Accept: */*
> Origin: http://example.com
> Access-Control-Request-Method: POST
>
< HTTP/1.1 200 OK
< access-control-allow-origin: http://example.com
< access-control-allow-methods: POST, GET, OPTIONS, PUT
< date: Fri, 16 Aug 2019 13:40:23 GMT
< server: envoy
< content-length: 0
<
* Connection #0 to host <IP_ADDRESS> left intact

Then used this tool http://www.test-cors.org/ to send a request from my browser where I found requests with methods, GET, POST, and PUT succeeded but a DELETE request was denied correctly.
Maybe your backend is denying the request from some reason? I would need a replicator from you to fully figure out what is going on.


Move the IsoBibliographicItem class to the IsoBib gem.

@ronaldtse @opoudjis since we extract BibliographicItem class from the IsoBibGem seems this gem is used only on the IsoBib. Suggest to move IsoBibliographicIcem class to IsoBib and delete this gem.

Agree! Just want to be sure that other gems that depend on IsoBibliographicItem will switch their dependencies to IsoBib gem.
I’m actually wondering if we should rename those to “relaton-iso” etc

@ronaldtse if we have more than one gem uses the IsoBiblographicItem then this gem should remain.
Renaming this gem cause to update all gems dependent on it. But since we extract BibliogtaphicItem we will have to update these gems anyway. So we can make a new "relaton-iso" gem and move IsoBibliographicIem class to it.

I think GbBib uses IsoBiblographicItem, not sure if others do. Probably not?

I just checked and find out IsoBib, GbBib, IecBib, and ItuBib use IsoBibItem.

IsoBibItem is technically only for ISO and GB models. IEC is somewhat similar, but ITU’s model is quite different.

We use a BibliographicItem class in 2 gems, IsoBib and GbBib. So we should move the class to a separated gem.

@andrew2net isn’t the BibliographicItem class used in relaton-bib?

@ronaldtse we split iso-bib-item into relaton-bib and relation-iso. So they use BibliographicItem and IsoBibliographicItem classes respectively.


Feature/hosted workflows

This updates the my workflows/workflow pages to properly display hosted workflows. This does not add any form of editing.

Codecov Report

Merging #285 into develop will decrease coverage by 0.04%.
The diff coverage is 60%.

@@             Coverage Diff             @@
##           develop     #285      +/-   ##
===========================================
- Coverage    63.37%   63.32%   -0.05%
===========================================
  Files          122      122
  Lines         3664     3670       +6
  Branches       405      406       +1
===========================================
+ Hits          2322     2324       +2
- Misses        1252     1255       +3
- Partials        90       91       +1

Impacted Files
Coverage Δ

src/app/workflow/versions/versions.component.ts
83.67% <100%> (+0.69%)
:arrow_up:

...pp/workflow/version-modal/version-modal.service.ts
74.19% <55.55%> (-5.12%)
: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 f6aa644...e0cd52d. Read the comment docs.


nginx-ingress 0.9.0-final timeline

Sorry to ask for timelines in a GH Issue, but I'm anxious to know when I can get the nginx-ingress helm chart onto a future 0.9.0 release of nginx-ingress.
Thanks for any info.

@colemickens https://github.com/kubernetes/ingress/projects/1

Cool, thank you! Looked for milestones but hadn't seen anyone using Projects. Very nice, thanks again.


Make rand and rayon optional

(Update of https://github.com/sdd/kiddo/pull/136 to rebase on top of master - thanks @clbarnes!)
Both are only used in test utilities and examples, and both can interfere with WASM compilation.
Now both are behind a test_utils feature.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (0826d95) 94.12% compared to head (b543a3c) 94.14%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #143      +/-   ##
==========================================
+ Coverage   94.12%   94.14%   +0.01%
==========================================
  Files          52       52
  Lines        6029     6029
  Branches     6029     6029
==========================================
+ Hits         5675     5676       +1
+ Misses        333      332       -1
  Partials       21       21

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


cEUR support for CLI

CLI use cases that need updating

[ ] Transfer celocli transfer:euros
[ ] Exchange celocli exchange:euros
[ ] Gas currency celocli config:set --gasCurrency cEUR
[ ] Check no cEUR balance before destroying ReleaseGold contracts celocli releasegold:refund-and-finalize

Detailed breakdown:

Add cEUR to GasOption enum

https://github.com/celo-org/celo-monorepo/blob/master/packages/cli/src/base.ts#L14

Add logic for using cEUR as a gas currency

https://github.com/celo-org/celo-monorepo/blob/master/packages/cli/src/base.ts#L177

Create a transfer command for cEUR, similar to the existing one for cUSD

https://github.com/celo-org/celo-monorepo/blob/master/packages/cli/src/commands/transfer/dollars.ts
May be good to figure out a way to share code between the dollars & euros command

Add cEUR ↔ CELO (& vice versa) exchange rates for the exchange show command

https://github.com/celo-org/celo-monorepo/blob/master/packages/cli/src/commands/exchange/show.ts

Add cEUR exchange:euros command

https://github.com/celo-org/celo-monorepo/blob/master/packages/cli/src/commands/exchange/dollars.ts
Generalize function checkNotDangerousExchange to work for a parametrized stable token

https://github.com/celo-org/celo-monorepo/blob/master/packages/cli/src/utils/exchange.ts#L22

Modify exchange:celo and exchange:gold to work with cEUR as well

https://github.com/celo-org/celo-monorepo/blob/master/packages/cli/src/commands/exchange/celo.ts and https://github.com/celo-org/celo-monorepo/blob/master/packages/cli/src/commands/exchange/gold.ts
Currently, these commands assume you are exchanging CELO → cUSD. We could add a flag to indicate what currency you wish to buy. If backward compatibility is desired, we can consider making the flag optional and default to cUSD

Update description for account:balance command to include cEUR

https://github.com/celo-org/celo-monorepo/blob/master/packages/cli/src/commands/account/balance.ts#L6
With the relevant contractkit change already made, cEUR will print with no changes in the CLI required

(Optional) add cEUR to list of examples for setting gas currency config

https://github.com/celo-org/celo-monorepo/blob/master/packages/cli/src/commands/config/set.ts#L22

Add StableTokenEUR as an option for oracle related commands

Would be good to consolidate all the different places that the options are defined into one variable
oracle:list https://github.com/celo-org/celo-monorepo/blob/master/packages/cli/src/commands/oracle/list.ts#L16
oracle:remove-expired-reports https://github.com/celo-org/celo-monorepo/blob/master/packages/cli/src/commands/oracle/remove-expired-reports.ts#L16
oracle:report https://github.com/celo-org/celo-monorepo/blob/master/packages/cli/src/commands/oracle/report.ts#L18
oracle:reports https://github.com/celo-org/celo-monorepo/blob/master/packages/cli/src/commands/oracle/reports.ts#L18

CLI use cases that need updating

[ ] Transfer celocli transfer:euros
[ ] Exchange celocli exchange:euros
[ ] Gas currency celocli config:set --gasCurrency cEUR
[ ] Check no cEUR balance before destroying ReleaseGold contracts celocli releasegold:refund-and-finalize

Check no cEUR balance before destroying ReleaseGold contracts celocli releasegold:refund-and-finalize

Is this needed? IIUC this was to prevent contracts from being destroyed that may have accumulated cUSD validator rewards

What about account:balance?

seems like economics team has taken most of this on, but not done in a generic stabletoken fashion. will create a separate ticket to generalize it.
@gastonponti will scope remaining work needed

It is included on the ticket, but here's an example of how the cli handles oracle:list for an stableToken today
./packages/cli/bin/run oracle:list 0xcCA78ae41fe8F83eE7F37B0110eb6FAe3E9Ca0F0
(node:3956) ExperimentalWarning: The fs.promises API is experimental
Failed to close the connection: Error: Expected 0xcCA78ae41fe8F83eE7F37B0110eb6FAe3E9Ca0F0 to be one of: StableToken
See more help with --help
 ›   Error: Expected 0xcCA78ae41fe8F83eE7F37B0110eb6FAe3E9Ca0F0 to be one of: StableToken
 ›   See more help with --help

We should also update this commands:

celocli network:contracts
celocli network:parameters

@gastonponti is this item in the description complete?
Check no cEUR balance before destroying ReleaseGold contracts celocli releasegold:refund-and-finalize

Removed https://app.zenhub.com/workspaces/cap-backlog-board-600598462807be0011921c65/issues/celo-org/celo-monorepo/7157 from this epic as it is low priority and closing Epic as complete.


XSS with HTML entities

With the sanitize option on it is possible to create a link with a javascript: protocol with the following: [URL](javascript&#58document;alert(1)).
HTML entities in the browser are not strict and parse what they can and leaving the rest behind. For example &#xNNanything; would parse the NN hex values but leave behind the string "anything;".
"javascript&#58document;"  with the regex /&([#\w]+);/ returns "58document" and is parsed by String.fromCharCode to "". Because of this the later tests only sees the javascript keyword without the :. However the browser parses this to: "javascript:document;".

:+1: This seems pretty significant.

I assume this project is dead. I cant see letting an XSS go for this long otherwise. I have moved to a new Lib in my project.

The solution is right there dude.

I created a fork and this PR, but the real solution is to use something else. I am recommending that the node security project mark this project as vulnerable and abandoned.

@matt- any recommendation for alternatives?

https://github.com/jonschlinkert/remarkable has been great for me.

@matt- thanks a ton!

That moment when you realize you didn't comment on an open issue, but rather an open PR.  Whelp.

when will this be pulled?

+1 Can this please be merged in so we can remove the advisory?

+1

+1

@matt- Remarkable looks great (very very clean) but it's about 6x the size of Marked.  For some use-cases (client-side) it seems like Marked would be preferable on those grounds (though for anything else I'm probably going to be using remarkable given the focus on performance).

the question is, where is the repo maintainer?

Perhaps on vacation?

+1

Vacation for a year? Must be nice.

@matt-  Any chance you would be willing to help out Modernizr (https://github.com/Modernizr/Modernizr)  swap out its dependency from Marked to Remarkable? I'm too new at programming javascript to take  on the task myself.

This appears to be no longer reproducible. I believe the author fixed it and just ignored this PR. Can anyone still reproduce it with the latest version of Marked?

mvhenten/new:~/test $ npm install marked
npm WARN package.json<EMAIL_ADDRESS>No description
npm WARN package.json<EMAIL_ADDRESS>No repository field.
npm WARN package.json<EMAIL_ADDRESS>No README data
npm WARN prefer global<EMAIL_ADDRESS>should be installed with -g
<EMAIL_ADDRESS>node_modules/marked
mvhenten/new:~/test $ node
> var md = require("marked");
undefined
> md("[URL](javascript&#58document;alert(1))");
'
URL
\n'

Please close this PR. The security issue no longer exists, but causes a lot of FUD nowadays.
I thinkt he package author fixed this bug independently of this PR and judging from the open issue count in in general doesn't care much for the bugtracker.

"Latest commit 88ce4df  on Jul 31, 2015"  This issue is not resolved and has not been even been touched.
The exact example you gave is the XSS issue. (Put that HTML in a browser and click on it.)  javascript: (in any form even with html entities) should be blocked in sanitize mode.
Example where marked works correctly:
> var md = require("marked");
> md.setOptions({sanitize: true})
> md("[URL](javascript:alert(1))");
'

\n'

Example this PR resolves with bad entities:
> var md = require("marked");
> md.setOptions({sanitize: true})
> md("[URL](javascript&#58document;alert(1))");
'
URL
\n'

If you think this is still some how resolved please read this blog to better understand the issue:
https://snyk.io/blog/marked-xss-vulnerability/

From the first sentence in this PR: "With the sanitize option on"   This lib has a sanitize mode that is intended to block normal HTML and prevent xss. https://github.com/chjj/marked#sanitize
It also filters "javascript:"  and "vbscript:" as intended in this mode. This is an example of bypassing that with html entities.
Executing javascript (an XSS) is much MUCH worse than "plain old hyperlinks".

@matt- I was able to reproduce this in indeed. I may have been confused due to the fact that post-process links and images, mitigating this issue.

@matt- sorry I work with @mvhenten and we thought it was resolved but it was only not affecting us due to post-processing. This is still a valid issue.

I've dropped an old-fashoned e-mail. maybe it helps.

@matt- You said that you made a fork - have you published in on npm? I agree with you that this module should never be used for any project if it has such a poor record of dealing with security issues, but there are already 1600 modules on npm that depend on it and having a version on npm with your patch applied would make it possible to quickly fix them (and who knows how many other modules that depend on those 1600) with a simple change of require('marked') to require('marked-secure') (or however your fork would be named) as a temporary fix before they can all be updated to use better markdown parsers.
Having a security vulnerability in at least 1600 modules on npm even though a fix has been available for over a year now but the maintainer can't be bothered to click a merge button is a serious problem. The project may be dead in a sense that its original author doesn't care about it any more but it's anything but dead when you consider its usage:

It's being used all over the place - including the official Node.js website - it has 1.5 million downloads per month and growing.
I found out about this issue today by a coincidence because I saw "dependencies: insecure" badge in the readme of the https://nodejs.org/ website's GitHub project, which was linked to the nodejs.org entry on david-dm.org that had a big red "SECURITY VULNERABILITIES IN DEPENDENCIES" and a link to the marked content-injection advisory on the Node Security Platform - which in turn included a link to this very pull request. If I hadn't clicked that badge I would probably never know about any security problems with that module - I would certainly never go through all the issues and PRs to find this one or #724 and know that this projects is dead and insecure, because there is not even a hint about any problem with that module in its readme on GitHub and npm

Especially @matt- please check out https://github.com/chjj/marked/issues/756

Interesting POV from the showdown project https://github.com/showdownjs/showdown/wiki/Markdown's-XSS-Vulnerability-(and-how-to-mitigate-it)

Is there any precedent at npm for transferring an insecure package to a new maintainer who can be trusted to move the project forward? I would assume some sort of process would need to be in place in case of a disappearing/deceased/missing maintainer of any sufficiently popular package.

@STRML I believe there must be a process - but I am not aware of any. That said - have a look at #756. I guess we just need someone that steps up.

Besides the Remarkable project, there is also markdown-it. The description of this project says:

markdown-it is the result of the decision of the authors who contributed to 99% of the Remarkable code to move to a project with the same authorship but new leadership (Vitaly and Alex). It's not a fork.

The API for markdown-it looks very similar to Remarkable. I don't understand how it's Not a fork.
I have not tried Remarkable or markdown-it. Any comments on the differences are welcome. Other people are going to continue to end up at this pull request because it's referenced by a security notice:
https://nodesecurity.io/advisories/101
Since the best "fix" seems to be switching to a new project, it seems on-topic to discuss the best alternatives here.

markdown-it has no open bug issues, with 214 closed (only 2 feature requests are open)

remarkable has 48 open issues and 120 closed. I'm not sure how many open issues are bugs.

Both projects show commit activity in the last month.

Neither project has open vulnerabilities nodesecurity.io. remarkable had one in the past. https://nodesecurity.io/advisories/module/remarkable https://nodesecurity.io/advisories/module/markdown-it

@chjj responded to a call for maintainers https://github.com/chjj/marked/issues/756

@matt- now that you're a maintainer can you merge / release this please :)

I would like to have someone else sign off on the PR (as I am the one that added it). Anyone?

Hang on, test 10 is failing:
#10. def_blocks.text failed at offset 20. Near: "

hello

<ahref="foo">1:hell".

Got:

hello

<ahref="foo">1:hell

Expected:

hello[1]:hello

Would be good to address #321, #474 first so we have test output in the PR.

Actually can confirm test 10 is failing on master, so it's not a regression. :+1: LGTM, did you want to address the semicolon concern in the line comment?

I think we are good on this one?

Any plans to publish this patch on npm?
marked still triggers "SECURITY VULNERABILITIES IN DEPENDENCIES" warning for nodejs.org at https://david-dm.org/nodejs/nodejs.org

No reason to keep delaying on this, please merge @matt-

I thought we were waiting for the test 10 is fail.

See https://github.com/chjj/marked/pull/592#issuecomment-232400724
Anyway, thanks for the merge!

Yep, that was my bad. Happy to finally get this in.

@matt- Can you npm version patch && npm publish yourself or does it still have to be done by @chjj? The version of npm is still 0.3.5 published 12 months ago: https://www.npmjs.com/package/marked

That is all @chjj
On Friday, July 29, 2016, Rafał Pocztarski<EMAIL_ADDRESS>wrote:

@matt- https://github.com/matt- Can you npm version patch && npm publish
yourself or does it still have to be done by @chjj
https://github.com/chjj? The version of npm is still 0.3.5 published 12
months ago: https://www.npmjs.com/package/marked
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chjj/marked/pull/592#issuecomment-236280610, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAbr4sQX5-C4C_BdSAvsUq5sjg32a6Tdks5qal1agaJpZM4Eg7pd
.

--
../matt

@matt-, published. Great work guys.


chore: add security plugin project

Add New Relic Security Plugin for Pixie repo and new security tag.

@aswanson-nr Is this PR something you could review? I'm not seeing many PR reviews in the repo history and we were hoping to get this in by the end of the week.

Hey @hmstepanek! This looks pretty much good to go, although you might want to add a description since this is how it currently shows up:

Awesome! Thanks! Description added. It looks like I don't have permission to merge this so might need some help there.


Plone deployment training minor fix proposal

some minor fix to ease following the training istructions (IMHO)
added a troubleshooting paragraph with sone other reported issues

thanks @stevepiercy , I've committed your changes. Have I to Close with comment (which seems the only option I have from github) and make a new pull request?

Would you please read and follow Contributing to Plone? I assumed you had already signed the agreement.

I signed the agreement yesterday, just before accepting and committing the changes
I realize the process is more complex than I thought: sorry for the inconvenience. By the way: next time I will create a new branch even for the (supposed) minor fix, I promise!

@deruosi this looks good. Please let me know when your Plone Contributor Agreement is approved, then I can merge this PR. Thank you!

@stevepiercy my Plone developer agreement was processed a few hours ago: I just joined the Plone Contributors team. Thanks!

@deruosi thank you!


16bit fade rework

coverage: 32.053% (+0.06%) from 31.992%
when pulling 3188c91a18bf07a6fc57b5c74aecb997d4525e83 on 16bitfade
into 3bc67b90cdbe019b85fde19bf59744621f242369 on master.


AQCU-1044 UV Hydro Aging band sometimes truncated

Coverage increased (+0.002%) to 99.047% when pulling c965c84df9b4e1dcee14bb5cacecf4ad4ce77129 on mhines-usgs:master into a59b279e4ef28b385ee459a7420dc1a7a0bac219 on USGS-R:master.


feat: refactor list data structure by using doubly linked list and support HintKeyAndRAMIdxMode

By issue #354, even in HintKeyAndRAMIdxMode, the current List data structure stores all values in memory, which is not very reasonable. I fixed this issue by storing Record in the index.In the HintKeyAndRAMIdxMode, it is only necessary to store the Hint, and there is no need to store the Entry.
I refactored the List using a doubly linked list because using arrays would result in significant copying overhead when performing operations like LPush, LPop, and others.

I made a simple benchmark:
notice: SyncEnable is disable and operating in HintKeyValAndRAMIdxMode mode.

LPush: I pushed 100,000 records using LPush, with each record having a value of 512 bytes. Prior to the optimization, it took approximately 50 seconds, whereas after the optimization, it only required 700ms!
LPop: I poped 100,000 records using LPop, with each record having a value of 512 bytes. Prior to the optimization, it took approximately 20 seconds, whereas after the optimization, it only required 190ms!
LRemByIndex: Due to the nature of linked lists, the delete performance is slightly inferior. In the test of deleting by index, when randomly removing 1000 entries from a List of 100,000 data points, it takes 250 us before refactoring and 4 ms after refactoring.

I made a simple benchmark:
notice: SyncEnable is disable and operating in HintKeyValAndRAMIdxMode mode.

LPush: I pushed 100,000 records using LPush, with each record having a value of 512 bytes. Prior to the optimization, it took approximately 50 seconds, whereas after the optimization, it only required 700ms!
LPop: I poped 100,000 records using LPop, with each record having a value of 512 bytes. Prior to the optimization, it took approximately 20 seconds, whereas after the optimization, it only required 190ms!
LRemByIndex: Due to the nature of linked lists, the delete performance is slightly inferior. In the test of deleting by index, when randomly removing 1000 entries from a List of 100,000 data points, it takes 250 us before refactoring and 4 ms after refactoring.

Coding master.

I'm way off.😫

Merged. Thanks for your great contributions.


Special characters are rendered with an extra space

Problem
Special characters like emojis and accent-bearing letters are followed by an extra space.
Steps To Reproduce

Replace the "Hello World!" string in the examples with something containing special characters, like "I🧡piñadas"
cargo run

Expected behavior
Emojis are just nice, but localization is a requirement for some projects. So it would be nice to have accent bearing letters rendered in the space of one character.
Screenshots

Environment:

Dioxus version: f63220019ddaac5a8bc44db855723c245d53632e, master
Rust version: 1.70.0
OS info: Windows 11, PowerShell 7.3.4
App platform: TUI

Questionnaire

[x] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

I believe the issue here is that the x offest is determined by the byte index here we should be able to fix this by using chars().enumerate() instead of char_indecies()

Typically when I see that error it is because one or more Dioxus libraries use a different source. If the Dioxus and renderer library don't use the same source then you can have places where identical types come from different versions of the dependency which causes this error message. In your example, I'm guessing that you pointed dioxus-tui to your new branch, but Dioxus is still using the main git branch.

Interesting. That's exactly right! Now I'm using my fork for both dioxus and dioxus-tui dependencies, and it compiles and runs fine. New problem, thou:

In the source, I have a p right after the heart. The ñ is perfect, so that's a step forward!

It looks like the heart takes up more than one terminal character of space. Instead of the number of characters, we can map each character to it's width (given by https://docs.rs/unicode-width/latest/unicode_width/trait.UnicodeWidthChar.html#tymethod.width) and sum the with of all the previous characters

hi, i meet the same problem just now:
the code is copy from helloworld example: https://dioxuslabs.com/docs/0.3/guide/en/getting_started/mobile.html
src/lib.rs
use dioxus::prelude::*;

fn main() {
    dioxus_desktop::launch(app);
}

fn app(cx: Scope) -> Element {
    cx.render(rsx!{
        div {
            "hello world!"
        }
    })
}

Cargo.toml
use dioxus::prelude::*;

fn main() {
    dioxus_desktop::launch(app);
}

fn app(cx: Scope) -> Element {
    cx.render(rsx!{
        div {
            "hello world!"
        }
    })
}

error message:
error[E0308]: mismatched types
  --> src/lib.rs:4:28
   |
4  |     dioxus_desktop::launch(app);
   |     ---------------------- ^^^ expected fn pointer, found fn item
   |     |
   |     arguments to this function are incorrect
   |
   = note: expected fn pointer `for<'a> fn(dioxus_core::scopes::Scope<'a>) -> Option<dioxus_core::nodes::VNode<'a>>`
                 found fn item `for<'a> fn(&'a Scoped<'a>) -> Option<VNode<'a>> {app}`
   = note: when the arguments and return types match, functions can be coerced to function pointers

You may need to run cargo update to get the latest versions of the dioxus and dioxus-desktop crates

You may need to run cargo update to get the latest versions of the dioxus and dioxus-desktop crates

i have tried before. it's nothing change in Cargo.toml

if i edit like this:
[dependencies]
mobile-entry-point = "0.1.0"
dioxus = { version = "0.3.2"}
dioxus-desktop = { version = "0.3.0" }
simple_logger = "*"
atk-sys = "*"
webkit2gtk-sys = "*"

then "cargo update" and "cargo run", the error message is:
warning: function `main` is never used
 --> src/lib.rs:3:4
  |
3 | fn main() {
  |    ^^^^
  |
  = note: `#[warn(dead_code)]` on by default

warning: function `app` is never used
 --> src/lib.rs:7:4
  |
7 | fn app(cx: Scope) -> Element {
  |    ^^^

warning: `mobile-caller-blocker` (lib) generated 2 warnings
   Compiling mobile-caller-blocker v0.1.0 (/media/creatxr/WORK/DEV/APPS-CROSS-PLATFORM/mobile-caller-blocker)
error[E0425]: cannot find function `start_app` in crate `mobile_caller_blocker`
 --> gen/bin/desktop.rs:4:28
  |
4 |     mobile_caller_blocker::start_app();
  |                            ^^^^^^^^^ not found in `mobile_caller_blocker`

For more information about this error, try `rustc --explain E0425`.

TUI has been put on ice while we overhaul its architecture based on http://github.com/dioxusLabs/blitz - lots of bugs will come out of its eventual re-release.


Fetch topic info for custom DCs from KG only.

Closing this PR - decided to go with a different approach.


remove unused imports from test code

The changes look good.  Just waiting for the build to complete.


MSUnmerged: Test certificate issues  and re-enable the service  at T2_CH_CERN

Impact of the new feature
MSUnmerged for T2_CH_CERN
Is your feature request related to a problem? Please describe.
This  was a long discussion and several wrong paths taken in order to fix those Access issues at T2_CH_CERN. The full details are in the following Jira: CMSVOC-491  and SNOW: RQF2634963 tickets
Now after those are finally solved thanks  to the effort of many people involved into those two tickets above (Thanks to all of them btw) we are to test  the access and re-enable the MSUnmerged Service for T2_CH_CERN.
Describe the solution you'd like
Enable T2_CH_CERN in the list of RSE cleaned by MSUnmerged
Describe alternatives you've considered
None
Additional context
None

The latest checks of the logs of MSUnmerged related to T2_CH_CERN turned  out to show negative results:
2024-05-13 11:18:30,679:DEBUG:MSUnmerged: RSE: T2_CH_CERN, Dir: /store/unmerged/Run3Summer23NanoAODv12/TTto4Q-2Jets_TuneCP5_13p6TeV_amcatnloFXFX-pythia8/NANOAODSIM/130X_mcRun3_2023_realistic_v14-v3, delResult: [GError('DavPosix::unlink  HTTP 403 : Permission refused ', 1),
 GError('DavPosix::unlink  HTTP 403 : Permission refused ', 1),
 GError('DavPosix::unlink  HTTP 403 : Permission refused ', 1),
 GError('DavPosix::unlink  HTTP 403 : Permission refused ', 1),
 GError('DavPosix::unlink  HTTP 403 : Permission refused ', 1),
 GError('DavPosix::unlink  HTTP 403 : Permission refused ', 1),
 GError('DavPosix::unlink  HTTP 403 : Permission refused ', 1),
 GError('DavPosix::unlink  HTTP 403 : Permission refused ', 1),

I am now setting up yet another manual cleanup in order to figure out if it is  a certificate or a proxy issue (meaning if it is the DN of the cert that is not allowed access or it is a missing role in the proxy itself involved here.

@todor-ivanov as discussed in today's WMCore meeting - and now with Andrea/Dario - we think we can come back to this item in October (unless Ops reach out to us asking for a urgent cleanup). So I am moving this back to ToDo for now. Thanks

ok, Thanks @amaltaro


The schema version of 'Microsoft.EntityFrameworkCore.Tools.DotNet' is incompatible with version <IP_ADDRESS>5 of NuGet

Edited by @rowanmiller 11-29-2016
Workaround
If you hit this issue, you can resolve it by installing the latest beta of the NuGet Package Manager.
The issue arises because of a combination of requirements in pre-release tooling.

This one is really confusing to me but...
I installed the new Tools.DotNet package:
project.json:
"Microsoft.EntityFrameworkCore.Tools.Dotnet": "1.0.0-preview3-final",
And now my package manager inside visual studio throws an error: Error Occured......Show errors in output/Dismiss. I click Show errors in output, and I get:
The schema version of 'Microsoft.EntityFrameworkCore.Tools.DotNet' is incompatible with version <IP_ADDRESS>5 of NuGet. Please upgrade NuGet to the latest version from http://go.microsoft.com/fwlink/?LinkId=213942.
Nuget appears to be up to date in package manager.
Jump to the package manager console and type $host:
Name             : Package Manager Host
Version          : <IP_ADDRESS>4
InstanceId       : 015141c8-ed0c-4346-8cb1-ff76f59fddc4
UI               : System.Management.Automation.Internal.Host.InternalHostUserInterface
CurrentCulture   : en-US
CurrentUICulture : en-US
PrivateData      : NuGetConsole.Host.PowerShell.Implementation.NuGetPSHost+Commander
DebuggerEnabled  : True
IsRunspacePushed : False
Runspace         : System.Management.Automation.Runspaces.LocalRunspace
Visual Studio -> Help -> About
NuGet Package Manager 3.5.0
Naturally I am thinking there is another version of NuGet on my system But I can't figure this one out...

Can you make sure you have the latest preview of the .NET Core tooling installed - https://www.microsoft.com/net/core#windows. We think it may be an outdated version on NuGet included in an old version of the tooling.

.Net Core 1.0.1 Tooling Preview 2 was already installed. I tried a repair install and no luck.

We're not really sure what is happening here. It's not an issue we have seen before. One idea we had was to try using Fusion logs to try and work out what is loading the older version of NuGet, and where the older assemblies are located.

Am having the same issue here. Tried using Fusion Logs but all the bindings seemed to be with NuGet 3.5. If I remove Microsoft.EntityFrameworkCore.Tools.DotNet from my project.json everything returns to normal. The update and browse tabs of the manager seem to work fine too. The package manager console command Get-Package works just fine.

Are you still seeing this with the latest tooling release 1.1.0-preview4?

Yes. it's still happening with 1.1.0-preview4. If I remove  "Microsoft.EntityFrameworkCore.Tools.DotNet": "1.1.0-preview4-final", from project.json nuget starts working again.

cc @yishaigalatzer to see if you have any ideas how to debug this one

In the meantime, using the nuget package works: Install-Package Microsoft.EntityFrameworkCore.Tools -Pre
PMC commands are documented here: https://docs.microsoft.com/en-us/ef/core/miscellaneous/cli/powershell

@bdm4 @lnaie I suspect it is possible that this issue surfaces for solutions that were originally created with Visual Studio 2013 and configured to use the MSBuild-integrated package restore functionality that was the default before NuGet 2.7. If so, your solution could contain a .nuget folder and MSBuild targets that redirect to a copy of nuget.exe in that folder, which would likely be out of date.
In case this helps, the process to migrate to the newer automatic implementation of package restore is described here: http://docs.nuget.org/ndocs/consume-packages/package-restore#migrating-to-automatic-restore

Some how you are required to install the latest beta of the Nuget Manager. Afterwards you can install the EF Tools package.
Please refer to: http://docs.nuget.org/ndocs/guides/install-nuget

Hope everyone had a good holiday. So I decided to create a blank .net core project based on the template.
Tools -> Manage Nuget... -> Installed -> Error Occurred -> Show errors in output:
The schema version of 'Microsoft.AspNetCore.Server.IISIntegration.Tools' is incompatible with version <IP_ADDRESS>5 of NuGet
Click updates 5 are available to push NETCore.App and others like it to 1.1.0, Select all, update. Click Installed. Same error as above.
🕐 hour goes by... google google google.
Find this: http://docs.nuget.org/ndocs/Guides/Install-NuGet.md
See this: NuGet Beta Channel, ok what the heck. I've tried everything else. Install the Beta Version of Nuget and restart VS.
It's working. 🍯

Added the workaround/resolution to the top of this issue.
Closing this out as it is a combination of pre-release tooling and will resolve as our pre-releases RTM and propagate.

Had the same issue. Closing my project and then running dotnet restore on the directory seemed to solve it. I did not install the beta of nuget.

Spent half an hour searching my system for an old version of nuget (2.12) that I didn't have anywhere because the error message completely threw me off.  If you can't display accurate information in an error then don't show anything.  Sheesh.


DataTables doesn't support colspan

I had a problem with DataTables on a table which has a TH element with colspan. The part where it breaks is when applying search on columns. I haven't tested if script fails when the search is off.
I edited my script to fit my needs, but it would be good to have an option to add colspan on some THs

Please link to a test case showing the issue.
DataTables does not support colspan or row span in the tbody. Additionally, in the thead it requires that there be a unique cell for each column. See also this example.

Thanks for the answer, but that wasn't acceptable in my case. The code i tried is e.g.:
http://jsbin.com/bunebezelo/edit

Hi,
Thanks for the test case. Unfortunately that is a form that DataTables does not support at this time. Sorry.


Outdatedd information

Two things:

The "Fields" pane is now "Data"
Instructions before the "Build your report" step is unclear.  https://learn.microsoft.com/en-us/power-bi/create-reports/desktop-excel-stunning-report#build-your-report

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

ID: 1e0083e3-c11c-2462-6f0f-a166759f14c0
Version Independent ID: 927cc95e-d442-db4d-75ae-d4a2fa64ae71
Content: Tutorial: From Excel workbook to stunning report in Power BI Desktop - Power BI
Content Source: powerbi-docs/create-reports/desktop-excel-stunning-report.md
Service: powerbi
Sub-service: pbi-reports-dashboards
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Thanks for your comment, @PbBarney. This issue has been assigned to the doc author for review.

Hi, @PbBarney -- I've added another screenshot to show the table you create.
https://review.learn.microsoft.com/en-us/power-bi/create-reports/desktop-excel-stunning-report?branch=pr-en-us-11920#create-a-new-table
I hope that helps! If not, you can add more feedback by using the feedback link on the article.
Thanks!


added intan recording interface

@bendichter I'll add the get_metadata() funciton for this later today, just getting this up for now

@bendichter I'll add the get_metadata() funciton for this later today, just getting this up for now

Cool. You can use the Draft PR for work in progress PRs

@bendichter Right you are; changed that. I'll try to be better about listing PR's as drafts while they are still in progress!

@bendichter OK, added and debugged the get_metadata() part as it relates at least to the Syntalos project.

@CodyCBakerPhD Looks good. Does this PR address the int16 vs float32 issue?

@bendichter No, waiting on @alejoe91 to make those core changes to either pyintan or the intanRecordingExtractor get_traces(). I think most of that issue can be resolved entirely outside of nwb-conversion-tools


Space - Becca

React Tic Tac Toe
Congratulations!  Please reflect on the assignment with these questions.
Reflection

Prompt
Response

How are events / event handlers and useState connected?
When the browser detects an event from the user, such as a button click on an html element, an event handler is triggered in response. The code in the event handler can update the state of a functional component using React's useState hook, while the current state can be accessed with the state's variable name. In the tic-tac-toe game, a click event on a particular Square component alerts the App component's onClickCallback function with the square's id, and the onClickCallback updates the square's value to display an x or o on the board, as well as the state of all the squares, the number of moves played, and who the current player is.

What are two ways to do "dynamic styling" with React? When should they be used?
One way to apply "dynamic styling" with React is to use the ternary operator in the return statement of our App functional component, allowing for conditional rendering. The ternary operator is great for concise logic that can easily fit into one line or so. We can also use JSX, either in setting a className property on the html object we want styled with the the name of a class in a CSS stylesheet we import (best practice for longer styles), or by setting a style property by referencing a style object with JSX key-value pairs if our styles are minimal.

Much like Rails works with the HTTP request->response cycle, React works with the browser's input->output cycle. Describe React's cycle from receiving user input to outputting different page content.
React intelligently reacts to changes in the browser and/or user events because of its tracking and modification of a virtual DOM, ReactDOM, and its render method. In index.js, when we render the <App /> component inside the root html element, we call the App function (including event handler functions defined inside), and the return statement executes to display the contents inside the return's single container element. If that container element contains child Components, such as Board in this project, any props supplied, such as an onClickCallback function, will be passed down. This tree-like structure of components and nested components allows data to flow to specified nodes. Every time a user takes an action that one of these nodes recognizes, such as a click or mouseOver event detected by the browser, the child component alerts its root parent component through state, which the parent component is programmed to handle -- in this case, through the useState hook imported into App. A change of state will often times cause the page to re-render, where different page content is displayed to the user depending on the action taken, such as displaying an x or o to a certain square on the board.

CS Fundamentals Questions

Question
Answer

What do you think is the BigO complexity of the method you use to compute a winner?
Though the space complexity of checkForWinner would be constant (no new data structures required), the time complexity would likely be O(n^2), where n is the number of rows in the tic-tac-toe square grid. Since there will always be the same number of rows and columns for this game, the addition of one more row would require us to perform not just one more comparison, but one more comparison per each column in the row.

Consider what happens when React processes a state change from setState -- it must re-render all of the components that now have different content because of that change.What kind of data structure are the components in, and what sort of algorithms would be appropriate for React's code to "traverse" those components?Speculate wildly about what the Big-O time complexity of that code might be.
Components within other components resembles a tree structure similar to the DOM; all the data of a React application might be housed within a single tree data structure. If it were organized as a binary tree, where each node has between 0 and 2 children, binary search algorithms would be efficient with a time complexity of log n, where n is the number of nodes to traverse. However, if the tree structure wasn't binary, search algorithms would be slower - perhaps log n * m, where n is the number of node levels (vertical direction) and m the highest number of children nodes required to traverse horizontally.

React Tic Tac Toe

Major Learning Goals/Code Review

Criteria
yes/no, and optionally any details/lines of code to reference

Demonstrates proper JavaScript coding style.
✔

Correctly passes props to child components.
✔

Correctly passes callback functions to child components and calls them to respond to user events.)
✔

Maintains the status of the game in state.
✔

Practices git with at least 6 small commits and meaningful commit messages
✔

Uses existing stylesheets to render components
✔

Functional Requirements

Functional Requirement
yes/no

The Square component renders properly and executes the callback on a click event.
✔

The Board component renders a collection of squares
✔

The App component renders a board and uses state to maintain the status of the game.
✔

Utilizes callbacks to UI events to update state
✔

Overall Feedback

Overall Feedback
Criteria
yes/no

Green (Meets/Exceeds Standards)
5+ in Code Review && 3+ in Functional Requirements

Code Style Bonus Awards

Was the code particularly impressive in code style for any of these reasons (or more...?)

Quality
Yes?

Perfect Indentation
✅

Elegant/Clever
✅

Descriptive/Readable
✅

Concise
✅

Logical/Organized
✅

Summary
Really nicely done, you hit all the learning goals here.  Well done.


chore: update to ring 0.17.5

Closes https://github.com/fedimint/fedimint/issues/2583
wasm32_c feature went away, I removed it, and everything compiled. @Maan2003 is this OK?

LGTM


Plans for an Ubuntu Linux 22.04, CIS 2.0.0 version?

Hello, are there any plans for an implementación for the Ubuntu Linux 22.04, version 2.0.0?

hi @jvleminc
This is currently in progress, there are a number of changes and reordering that take place in the major release upgrades. Along with other client requests unfortunately this has taken longer that we would like.
We hope to release this to subscribers in the next couple of weeks, it should be released to public approx 12 weeks after that release.
Hope that helps
uk-bolly

Ok, my team is also working on this in parallel, as we need it rather urgently. We will try to contribute once we have something relevant.

hi @jvleminc
I hope to complete the subscription only release of this along with the related audit this week. If you were interested in getting early access to all repo releases before being made public as well as not having to maintain your own versions. Further information about subscriptions and all the benefits that it provides can be found at https://www.lockdownenterprise.com.
many thanks
uk-bolly

hi @jvleminc
V2.0.0 is now written and releases along with the associated audit. This has now been released into private access areas for subscribers, it will be approx 3months until this is released to the public repos.
There are ALOT of changes with controls and tests changing dramatically, with sectoins moving around and new sections created. So i would suggest it is treated a a brand new benchmark as migrating from 1.0.0 would be a gigantic piece of work to align.
Many thanks
uk-bolly

Hello @uk-bolly,
Do we have any visibility on the release date ?
Thanks a lot

hi @Rafouf69
I've started to tidy up the last of the issues outstanding and hopefully out this week. As always it depends on client work that comes through.
Kindest regards
uk-bolly


Implement GET /callsets/{id}

First pass.  Still need to add tests.  Feedback welcome.
$ python client_dev.py -O callsets-get dataset1:1kg-phase1:HG00533 http://localhost:8000/current
dataset1:1kg-phase1:HG00533

Issue #527

Looks good so far (except for the somethingOrSomethingElse parameter).

@jeromekelleher ready for review

+1. Merging.
I'm about to land a fairly big refactor, so it would be good if we could leave the rest of these until afterwards.