CRD examples: Replace invalid TracingPolicy names

Many metadata.name of the TracingPolicy examples contained underscores in their name, and unfortunately, that makes it impossible to deploy them on Kubernetes. I assumed they were passed to Tetragon via flags using --config-file or similar.
Maybe it would be interesting to implement the regex check in Tetragon to avoid this pitfall?
Kubernetes objects names comply with lowercase RFC 1123 subdomain convention: they must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is [a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*)

Maybe it would be interesting to implement the regex check in Tetragon to avoid this pitfall?

Yes makes sense! next task? ;-)


TERM=foot-direct doesn't work with micro 2.0.14

# micro --version
Version: 2.0.14
Commit hash: 04c57704
Compiled on August 28, 2024

It takes around 20 seconds to micro to bring up the TUI and when it does it looks godawful.

TERM=foot micro ## this works
TERM=foot-direct micro ## this doesn't

I had to search what foot and foot-direct are used for and came across:

https://codeberg.org/dnkl/foot/src/commit/1925593a374b3e3b4cead86abc4ae352260a372e/doc/foot.1.scd#L490-L494
https://codeberg.org/dnkl/foot/wiki/Home.md#24-bit-color-support-in-emacs

So it's a option to change the builtin terminfo, which causes trouble in our very old depending version of tcell, already known to have some limitations over the upstream one.
Even if it would have worked to forward the truelcolor support to tcell and micro, there is still the fact that micro disables truecolor by default over the automatic detection done by tcell (see this discussion resp. comment).


On iOS Chrome, article pages can be moved sideways

Expected behavior
Just like the homepage, the article pages should stay fixed horizontally.  As the user tap and swipe up and down, the pages should only scroll up and down but not move left and right.
Actual behavior
When the user’s finger moves sideways as they swipe vertically, the article pages are moving left and right, making it a poor user experience.
How to reproduce

Browser: 65.0.3325.152
Operating system: iOS 11.2.6 (15D100)

Load https://busy.org/ and start scrolling up and down, as you scroll, the page is scrolling: good
Try moving the page left and right by taping and swiping horizontally, the page does not move sideways: good
Click on an article preview to get to it’s own page.
When the article page isn’t loaded, try to move it left/right, the page will move: not great.
As you scroll the page up and down, the page will also follow your finger left and right. Having not much margin/padding around the text, the user might not be able to continuously read as the scroll up because the text might just be hidden if the page gets moved too much.

Notes
This bug does not happen if you load the article page directly (if you know the URL, reload the page after coming from the homepage, or open it on a new tab from a link on the homepage). This is because opening it from the homepage does not actually load a new page but loads the article in an overlay on top of the homepage content (seescreenshot).
Screenshots and comments
A page being moved to the right, moving the page sideways reveals a faded out content in the background. It looks like the previous page is still there and the article page is opened in an overlay on top. That overlay is not fixed horizontally.

What breaks it for me is the movement as you scroll up/down. I often scroll slowly and read as I scroll and the text is often hidden away. It also make me feel a little dizzy as the page ondulates as it moves up/down.

Posted on Utopian.io -  Rewarding Open Source Contributors

Related to #1589


Reading then updating throws an Error

I'm trying to chain read and update operations. First I read (fetch) data from the database based on its id, then I change it and want to update the data in the database as well but I get a Unhandled rejection Error: There is no pool defined on the current client error.
index.js
var id = args.id;
var proj = new Project(id);

proj.read(knex).then(function(rows) {
  proj.set(args);
  proj.update(knex).then(console.log); // <== ERROR THROWN HERE
});

Project.js (only related functions)
read(knex) {
  var self = this;

  return knex('projects').where('id', this.id).then(function(rows) {
    if (rows.length == 0)
      throw '[panda] Read project: no project with id = ' + self.id;
    self.set(rows[0]);

    return self;
  });
}

update(knex) {
  console.log(knex('projects').where('id', this.id).update(this).toString());

  return knex('projects').where('id', this.id).update(this);
}

Command
$ node index.js projects update --id=5 --name=Test-update
update "projects" set "completion" = 0, "currentIteration" = NULL, "id" = 5, "name" = 'Test-update' where "id" = 5
Unhandled rejection Error: There is no pool defined on the current client
    at /home/flawyte/development/projects/js/panda/node_modules/knex/lib/client.js:202:25
    at tryCatcher (/home/flawyte/development/projects/js/panda/node_modules/bluebird/js/main/util.js:26:23)
    at Promise._resolveFromResolver (/home/flawyte/development/projects/js/panda/node_modules/bluebird/js/main/promise.js:480:31)
    at new Promise (/home/flawyte/development/projects/js/panda/node_modules/bluebird/js/main/promise.js:70:37)
    at Client.acquireConnection (/home/flawyte/development/projects/js/panda/node_modules/knex/lib/client.js:200:12)
    at /home/flawyte/development/projects/js/panda/node_modules/knex/lib/runner.js:136:49
    at tryCatcher (/home/flawyte/development/projects/js/panda/node_modules/bluebird/js/main/util.js:26:23)
    at Function.Promise.attempt.Promise.try (/home/flawyte/development/projects/js/panda/node_modules/bluebird/js/main/method.js:31:24)
    at Runner.ensureConnection (/home/flawyte/development/projects/js/panda/node_modules/knex/lib/runner.js:135:26)
    at Runner.run (/home/flawyte/development/projects/js/panda/node_modules/knex/lib/runner.js:30:31)
    at QueryBuilder.Target.then (/home/flawyte/development/projects/js/panda/node_modules/knex/lib/interface.js:27:43)
    at /home/flawyte/development/projects/js/panda/index.js:80:31
    at tryCatcher (/home/flawyte/development/projects/js/panda/node_modules/bluebird/js/main/util.js:26:23)
    at Promise._settlePromiseFromHandler (/home/flawyte/development/projects/js/panda/node_modules/bluebird/js/main/promise.js:507:31)
    at Promise._settlePromiseAt (/home/flawyte/development/projects/js/panda/node_modules/bluebird/js/main/promise.js:581:18)
    at Promise._settlePromises (/home/flawyte/development/projects/js/panda/node_modules/bluebird/js/main/promise.js:697:14)

If I remove the then(console.log) in index.js no error is thrown but data in database remains unchanged after script ending. Why ?

Disclaimer: I'm not that familiar with the Knex codebase.
I see the following code in Knex's client.js:
  acquireConnection: function() {
    var client = this
    return new Promise(function(resolver, rejecter) {
      if (!client.pool) {
        return rejecter(new Error('There is no pool defined on the current client'))
      }
    })
  },

You can tap into that function and see what this is set to.
If this is not a Client object, then it's been bound incorrectly somwhere.
If this is a Clientobject, it looks likeClient's destroy` has been called, setting the pool to undefined. Why that happens I don't know, but it's a place to start looking :-)

If I remove the then(console.log) in index.js no error is thrown but data in database remains unchanged after script ending. Why ?

Because the query is only issued to the database upon .then() (or one of Bluebird's other promise runners) being called. No .then(), no database interaction.
Also to avoid the unhandled rejection part, you should always return your promises and catch (or provide a rejection handler function to the second argument of .then()) them at some point. If an error occurs within the promise chain and no handler is provided it results in an unhandled rejection.
I am not sure why you're getting the error you're getting. Can you produce an SSCCE that reproduces it? Also what DB, OS, versions, etc. are you using?

Problem solved !
I had a line knex.destroy(); at the very end of my script and as knex is asynchronous, the line was being executed after the read instruction (due to quick execution) but before update, causing the error above. I noticed the error while writing the SSCCE :)
Thanks for your help.


Remove node 13 tests 2020-06-01

Would be nice to have issue to remove this at EOL (2020-06-01).
Originally posted by @mayurkale22 in https://github.com/open-telemetry/opentelemetry-js/pull/1026#issuecomment-624831329

bump since its tomorrow :)

I would like to take this one.

Thank you!

You're welcome😊
On Tue 2 Jun, 2020, 1:32 AM Daniel Dyla<EMAIL_ADDRESS>wrote:

Thank you!
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/open-telemetry/opentelemetry-js/issues/1030#issuecomment-637072652,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO4NOHKZIQP6G43IKHE2KP3RUQCMTANCNFSM4M2WZJUA
.


Update instructions to point folks to template issue

I created a template for proposing GSoC projects. Maybe it's not a great idea, but I wanted to give it a shot. I was thinking folks could put their project proposals in as tickets to this repo, instead of having them spread around different places.

It is a good idea! We need to post links to past good proposals as examples in there.

Changes to make to: https://github.com/ioos/gsoc/blob/main/.github/ISSUE_TEMPLATE/gsoc-project-proposal.md as part of this:

[ ] Update Labels front matter/header to use GSoC23
[ ] Include estimate project duration (175 or 350?)
[ ] Any other info we need to collect?

Ok, I think my most recent commits fd8ada7b6fc414d47223909cb89940cf1a8ce4f0 and 60cf9551d065a5ce61c863b2687fc3809d010b7e addressed this.  Hopefully we're all set to accept new project ideas from mentors now.  Let's get the word out!


updateUser UU:08 “The time zone provided is not valid”

Apparently the <Timezone> value returned from getUser is not a valid input back into updateUser. Why?

Closed by #36


Relax the bounds on the "base" dependency to support GHC 7.10

Thanks! I'll release a new version on Hackage later today!

OK, version <IP_ADDRESS> has been released on hackage. Please feel free to report any further issues!

Thank you!


Transactions cause fcl to switch currentUser to ServiceAccount (0.0.78)

Note: I'm cross-posting this from https://github.com/onflow/fcl-js/issues/1017 at the request of @chasefleming
Problem
While working on a project which implements functionality from KittyItems, I noticed that transactions appeared to be forcing currentUser to automatically swap to the ServiceAccount. This is with @onflow/fcl 0.0.78, fcl-dev-wallet:0.3.1, and emulator:latest. This did not happen with fcl-js 0.0.77.
Steps to Reproduce

Clone https://github.com/onflow/kitty-items
(If you already have it) Clear any cached "web"-related images in Docker.
Update web package.json to "@onflow/fcl": "0.0.78"
Add BASE_URL=http://localhost:8701 to dev-wallet env (in docker-compose.yml)
Deploy the project locally
Login as ServiceAccount, Initialize, and Mint an item
Log out, Fund Account A, login as Account A
Attempt to purchase an item

You will see the following warning in dev console:
You are manually enabling a very experimental feature that is not yet standard, use at your own risk.
         You can disable this warning by setting fcl.warning.suppress.redir to true in your config

You'll also see that fcl.currentUser().subscribe has updated the current user back to ServiceAccount. Additionally, you can see that even the Initialize and Mint transactions called when logged in as ServiceAccount caused the warning to fire and currentUser to swap (though, since you're logged in as ServiceAccount at the time, it doesn't impact anything).
This warning is coming from: https://github.com/onflow/fcl-js/blob/master/packages/fcl/src/current-user/index.js#L116

Hey @cbcomdev, KittyItems was just updated to use @onflow/fcl:0.0.79-alpha.3 and fcl-dev-wallet:0.3.2. I couldn't reproduce the problem in the latest version, could you give it a try?


Does this run from VM?

I tried to run this from within a 64bit VirtualBox Kali (stock) VM on Mac OS X 10.12.5 but it crashed. I'm not sure if this is supported but would be great to have this tool work from within a VM.
Installation
# ./autochrome.rb
[???] Detected Linux (x86_64)
[---] Determining latest Chromium version for Linux_x64
[---] Got latest version: Linux_x64 483973
[---] Downloading Chromium from https://commondatastorage.googleapis.com/chromium-browser-snapshots/Linux_x64/483973/chrome-linux.zip
[???] Saved  96370455 of  96370455 bytes (100%)
[---] Installed Chromium
[---] Installing extension /root/Desktop/Engagement/autochrome/data/themes/Red.crx
[---] Installing extension /root/Desktop/Engagement/autochrome/data/themes/Yellow.crx
[---] Installing extension /root/Desktop/Engagement/autochrome/data/themes/Blue.crx
[---] Installing extension /root/Desktop/Engagement/autochrome/data/extensions/settingsreset.crx
[---] Installing extension /root/Desktop/Engagement/autochrome/data/extensions/autochrome_junk_drawer.crx
[---] Installed user profiles

To launch Chromium, run '/root/.local/autochrome/chrome-wrapper' in a terminal.
The application may not appear in your launcher until you log out.

Click on the "Getting Started" bookmark after launching Chromium.

Execution
# /root/.local/autochrome/chrome-wrapper
[1820:1820:0703/062723.175746:FATAL:zygote_host_impl_linux.cc(107)] No usable sandbox! Update your kernel or see https://chromium.googlesource.com/chromium/src/+/master/docs/linux_suid_sandbox_development.md for more information on developing with the SUID sandbox.  Core file will not be generated.

I was able to reproduce this using both root and a regular user account on a Kali VM.
As a workaround, you can call chrome-wrapper with the --no-sandbox argument on the command line, or add that value to the OPTS array in /root/.local/autochrome/chrome
Looking at the Chromium source here, the error occurs when neither the namespace or suid sandboxes are available.  (The suid sandbox seems to be mostly deprecated, so the namespace sandbox not working is the issue.)
I think the failing bit might be here, but I'll have to dig a bit more to see if there's a better way to fix this than just disabling the sandbox entirely.

Looks like running sysctl kernel.unprivileged_userns_clone=1 as root will allow the namespace sandbox to work, though chromium still refuses to run as root unless --no-sandbox is set.
My tentative plan for a fix is:

If Autochrome is run as root, warn the user but continue
In Autochrome, check for unprivileged_userns_clone = 0 and prompt the user to set it to 1.
In the wrapper, use --no-sandbox if the user is root or unprivileged_userns_clone is 0.


Updates for SS4 compatibility

This was quickly done and should be reviewed & tested by someone else before merging and releasing

Thanks Adrian, looks good to me. Are you using these changes in a site somewhere already?

Their currently in a development site which won't be live for quite a while unfortunately.
Just made a quick change to the admin extension as it wasn't filtering the set


How to set two different color for dates?

I displayed two different types of dates. so how to indicate one list contains red color highlights and another list contains blue color,

This is not supported.
This is also sufficiently specific that I don't intend to add it.
You can use this library as a basis to implement it. Take a look at how the highlighted days are implemented.


Reach author

Hey @ReeceM, how can i reach you? email, slack?

Hi @Qoraiche, I usually use email for most communication.
I have slack as well.

@ReeceM can you share it with me so I can add you to my slack channel?

Hi @Qoraiche, it is<EMAIL_ADDRESS>

Invitation Sent! check your email

Got it, 👍


Stop slideshow after all pictures are loaded

I like to stop the slideshow after all pictures are loaded. Iam lost. Any help with a hint of where the little .stop() goes is appreciated.

Hi mat5hias,
You'd want to modify the code near setInterval, and use clearInterval once you've made your way through all of the images:
http://www.w3schools.com/jsref/met_win_setinterval.asp
Hope that helps.
–Scott


generic_key is not attaching the correct key to the label/descriptor

Describe the bug
The generic_key label in emissary-ingress is not attaching the correct key as a descriptor.
According to the docs you would expect:

But when I apply this mapping:
apiVersion: getambassador.io/v3alpha1
kind: Mapping
metadata:
  name: quote-backend
  namespace: localservices
  labels:
    maps-to-host-label: main-host
spec:
  ambassador_id: ["ambassador-3"]
  hostname: "*"
  prefix: /backend/
  service: quote
  timeout_ms: 29000
  labels:
    ambassador:
      - default_label: #apply to all mappings
        - remote_address:
            key: remote_address
      - group_label_1:
        - generic_key:
            key: testKey
            value: testValue
        - request_headers:
            header_name: Authorization
            key: identifier

And I look at the logs of my rate limiter:

You can see that it is getting (generic_key, testValue), you would expect: (testKey, testValue). The request_headers mapping is working as expected.
To Reproduce
Steps to reproduce the behavior:

Install emissary
Add the quote service the same as in the tutorial
Add the mapping as shown above (maybe without the ambassador-id and remove the namespace)
Add your RateLimiter of choice (I used the rate limit of Lyft) to see the output of the keys.

Expected behavior
I'd expect (testKey, testValue).
Versions (please complete the following information):

Ambassador: 3.1
Minikube with kubernetes V1.21

Additional context
For me this fortunately is not a big issue, as I only wanted to have 1 generic key anyway, but I'd imagine that this can be a very annoying bug if you want multiple generic keys.

I figured out yesterday that if you create this mapping inline, by using an annotation it does work as expected.


Issues running bot immediately after configuration

After configuring the bot for the first time and running it, I'm running into this error no matter what I try:

I thought the issue was related to the heroes in the config already being listed for sale so I de-listed them, attempted to run the bot again, same issue. I'm unsure if the issue is something on my end or if there's a problem with the code. Let me know if any more info is needed!

I'm sorry, I think the config file for this public repo was wrong, I'm going to update my current version of the code now. Also this code only supports the level 0 quests only (fishing, foraging, mining, gardening), it seems like dfk have created a new contract for the level 1 stats quests so I will need to look into it before implementing.

No problem at all, thanks so much for the update, pete!

Hey Pete, still running into some issues unfortunately. Upon testing the bot with the updated code, I'm getting this error relating to the auction system now:

Let me know if you need any more info relating to the issue, appreciate all your work!

Hi
This seems to be related to the "heroes" section in the config.json file. Below is what is what the format should be like:
the hero contains another nested json object that consists of heroId as the key, and another nested object of forsale and the price you want to list your hero. "forSale": false is when you dont want to list your hero and true when you want to put them on auction.
From your screenshot it seems like the code couldnt find neither true nor false for the heroId it found. If putting your heros in the format below doesnt work maybe you can try sending me your config file. I hope you find it useful!

Sorry about that, should've included the config file to start with haha. See below for the heroes auction section of the config, let me know if you need the entire config file itself and I'll send that over!

That looks fine to me, I've ran the exact same code and I had no problem listing my heroes as shown below

A possible problem is that you might have not listed all your questing heroes under the "heroes" key?, the idea for the code is that it checks all the heroId you put into the "quests" section to see which quest each hero should do when there is enough stamina and when the heroes are regenerating stamina, it will check if you want it forSale and the listing price for that heroId under the "heroes" section of config.  This means that you need every heroId that is in "quests" section to also be under the "heroes" section, if you dont want to list that for sale you just need to write "forsale": false.
If this does not fix it please send me the whole config file (maybe take out your wallet address for privacy),  thanks.

Ah, that fixed it! Sorry about that, I wasn't aware all questing heroes needed to also be listed under the auction section even if they weren't for sale. Thank you very much for helping me out with this :)
One last small thing I wanted to ask though, is it possible to adjust the default starting stamina for JEWEL mining from 15 to 20 and if so, would that be done by adjusting line 221 (minStamina = 25;) in routine_runner.js to 20, or somewhere else? Appreciate all your amazing work once again!

Sorry, one more thing to add... After running the bot for a couple hours, I noticed it's not starting non-professional quests automatically despite some heroes having enough stamina to do so. In config.json under the Foraging quest section, I tried setting "profession": true, to "false" but unfortunately the issue still occurs despite the heroes having full stamina. I attached my config just in-case you want to take a look, let me know if you need any additional info!
configgithub.txt

Ah, that fixed it! Sorry about that, I wasn't aware all questing heroes needed to also be listed under the auction section even if they weren't for sale. Thank you very much for helping me out with this :)
One last small thing I wanted to ask though, is it possible to adjust the default starting stamina for JEWEL mining from 15 to 20 and if so, would that be done by adjusting line 221 (minStamina = 25;) in routine_runner.js to 20, or somewhere else? Appreciate all your amazing work once again!

thats right, ive added minstamina value in the config file now

I'm sorry about this but I will have to check this again as I've never tested the non-professional part of the code as I've always let my hero do what they are good at - I will let you know when Ive look into it. A good news is that ive added the level 1 training quests now, you should be able to assign the heroes to do those quests professionally instead (since its always 5 stamina) which will likely give a higher xp efficiency. (unless you want the professional items, in that case you unfortunately have to manually do those quests first).

Sorry, one more thing to add... After running the bot for a couple hours, I noticed it's not starting non-professional quests automatically despite some heroes having enough stamina to do so. In config.json under the Foraging quest section, I tried setting "professional": true, to "false" but unfortunately the issue still occurs despite the heroes having full stamina. I attached my config just in-case you want to take a look, let me know if you need any additional info!
configgithub.txt

I'm sorry about this but I will have to check this again as I've never tested the non-professional part of the code as I've always let my hero do what they are good at - I will let you know when Ive look into it. A good news is that ive added the level 1 training quests now, you should be able to assign the heroes to do those quests professionally instead (since its always 5 stamina) which will likely give a higher xp efficiency. (unless you want the professional items, in that case you unfortunately have to manually do those quests first).

Awesome, thanks so much for pushing out that minstamina update and looking into all of this! Truly appreciated :)

Awesome, thanks so much for pushing out that minstamina update and looking into all of this! Truly appreciated :)

i've pushed it on the config but forgot to assign to the minstamina value in the code so it became undefined lol, ive pushed the most recent update to fix this.


Pass token to tfe provider

I had hoped that setting the credentials block in the home directory would be sufficient for the tfe provider, but this does not seem to be the case.
Seems that the credentials block might only be used to run plans against tf cloud backends, so that block might not really be doing anything.
Error example

Error: Invalid provider configuration
Provider "registry.terraform.io/hashicorp/tfe" requires explicit
configuration. Add a provider block to the root module and configure the
provider's required arguments as described in the provider documentation.
Error: Required token could not be found. Please set the token using an input variable in the provider configuration block or by using the TFE_TOKEN environment variable.
with provider["registry.terraform.io/hashicorp/tfe"],
on  line 0:
(source code not available)

ref

See https://github.com/hashicorp/terraform-provider-tfe/issues/146
Best to pass this in. Supporting .terraformrc is more than just loading static creds, you have to execute credential helper plugins.

Alternatively, you could set TFE_TOKEN

When yo'ure poking through github and a wild ben appears

considered it as an env var, but I figured everything else is passed as configuration so might as well stick to that for now unless theres a good reason not to

Always expect the Drucker inquisition


NO matches found on every search or autotag

From<EMAIL_ADDRESS>on February 20, 2014 17:28:25
File "\Users\tony/Dropbox/tagger/comictagger\comictaggerlib\volumeselectionwindow.py", line 378, in currentItemChanged
File "\Users\tony/Dropbox/tagger/comictagger\comictaggerlib\coverimagewidget.py", line 136, in setURL
File "\Users\tony/Dropbox/tagger/comictagger\comictaggerlib\coverimagewidget.py", line 193, in updateContent
File "\Users\tony/Dropbox/tagger/comictagger\comictaggerlib\coverimagewidget.py", line 200, in updateImage
File "\Users\tony/Dropbox/tagger/comictagger\comictaggerlib\coverimagewidget.py", line 235, in loadURL
File "\Users\tony/Dropbox/tagger/comictagger\comictaggerlib\imagefetcher.py", line 84, in fetch
File "\Users\tony/Dropbox/tagger/comictagger\comictaggerlib\imagefetcher.py", line 178, in get_image_from_cache
sqlite3.OperationalError: no such table: Images
version 1.1.10
No matches found on every search or autotag
Original issue: http://code.google.com/p/comictagger/issues/detail?id=41

From jbellanca on February 21, 2014 11:12:22
Yes, there's odd results being returned that started this week.  For example, if you search for "Wonder Woman", and find the current 2011 series, there should be 28 issues, but only the first 11 are returned.  The same thing happens for all searches.  Many titles actually return no results.

From<EMAIL_ADDRESS>on February 21, 2014 23:26:03
Getting the same issue.
"The Sensational She-Hulk" always returns the trade paperback "Sensational She-Hulk by John Byrne," and thus no matches.

From<EMAIL_ADDRESS>on February 23, 2014 15:19:11
Yeah I have problems with several series. the only work around that I have found is that if you want the current All-New X-Men Series you have to rename it to something more generic like X-Men manually search it and then the actual series appear. I had problem with X-Men: Legacy too.

From<EMAIL_ADDRESS>on March 22, 2014 11:43:26
Russ, are you still getting this error?

From<EMAIL_ADDRESS>on March 25, 2014 15:26:42
russmcgraw777, There's another user with similar errors.  I have a test build if this is a still an issue.
If don't hear back, I'll close the issue as "can't reproduce".  Thanks!

From<EMAIL_ADDRESS>on April 03, 2014 23:19:57
Tried all new builds including test have same problem with no such table inages and no match

From<EMAIL_ADDRESS>on April 04, 2014 15:14:54
Other questions:
Did ComicTagger ever work for you?
What version of windows are you on?
I'm curious if you tried removing the ComicTagger setting folder, what would happen.  It should be something like : C:\Users\usrname\AppData\roaming\ComicTagger

From<EMAIL_ADDRESS>on April 06, 2014 02:17:29
Yes
Windows 7 64bit
Deleted the folder and reinstalled fixed the problem
Thank You

From<EMAIL_ADDRESS>on April 06, 2014 07:46:26
Cool. I'm glad that fixed it, but I wish I knew the root cause of the corruption.  There have been occasional reports like yours, mostly on windows, I think.  If it happens again, before you purge anything, can you zip up the settings folder and email it to me at<EMAIL_ADDRESS>Thanks!

From<EMAIL_ADDRESS>on June 23, 2014 07:55:06
I am also getting the same problem, I purged all data around the ComicTagger Software.
I am running ComicTagger v1.1.15-beta
Windows 7 64Bit
The Tags on the Comics are blank but it looks like the the naming structure is missing with it maybe.
Example in the screenshot, My file name is
Agent X - Volume 001 - Issue 11 - (June - 2003).cbz
It gets Issue 11 but it is grabbing the series as "Agent X - Volume 001 - Issue"
If I remove all but "Agent X" I get back some information. Is there a method for identifying the parser params. I have 800 GB of comics in this format and can't practically change it.
Attachment: Screenshot.jpg

From<EMAIL_ADDRESS>on June 23, 2014 09:20:03
Cybercrawler, when you say you have the same problem, do you mean you're seeing the same errors as in the original post?  Or are just having issues with the file name parser?
For the latter, unfortunately you can't tune the way the filename parser works.  Information on what sort of file naming schemes works best is here: https://code.google.com/p/comictagger/wiki/UserGuide#Filename_Parsing If your files are consistently named, you should be able to use a bulk renaming utility to convert them all to a format that CT likes. Once the files are tagged, you can use the renaming function in CT to go back to whatever format you like.  In this case you could:
replace "- Volume" with "Vol"
replace "- Issue " with "#"
which would yield "Agent X Vol 001 #11  - (June - 2003).cbz"

From<EMAIL_ADDRESS>on June 23, 2014 10:39:13
If the tags are blank, it returns no results. If I manually change the series, Volume and issue then it will find the correct metadata.
I can change the surface archive name but this would not change what is in the archive.
Current Structure is
Agent X - Volume 001 - Issue 1 - (September - 2002).cbz
->Agent X - Volume 001 - Issue 1 - (September - 2002) - Page 000.jpg
->Agent X - Volume 001 - Issue 1 - (September - 2002) - Page 001.jpg
->Agent X - Volume 001 - Issue 1 - (September - 2002) - Page 002.jpg
I could write a script to unpak and rename all of the structure but it will simply take to long to run on a collection nearing a 850 GB.
I was really looking for a method to tag the old collection and the new collection coming in
As of now only the issue # can be padding from what I am looking at in the renamer. and this does not rename the files in the archive.
Lol it looks like again I have named things to specifically, I am having the same issues with other media (movies and Music) but any way this really does not belong in this section, maybe development ideas -
Thanks for your time.

From<EMAIL_ADDRESS>on June 23, 2014 11:05:34
One or both of us is misunderstanding the other.  I was suggesting that you use a 3rd party renaming utility such as this: http://www.advancedrenamer.com/ That would allow you to change, in bulk, the filenames to something that ComicTagger can parse.  If all the files are consistently named, it should be easy.
CT doesn't care about the image names inside the archive (as long as they sort properly)  There is no need to rename the interior images in the archive.

From<EMAIL_ADDRESS>on June 25, 2014 11:37:08
Is there any way for the re-namer to get the comics in SeriesName - Volume ### - Issue ### - (MonthText - Year) Or do I need to use the 3rd party tool again to get it in this format? It looks like the tool only pads the Issue field.

From<EMAIL_ADDRESS>on June 26, 2014 11:43:46
Sorry, right now, there is no zero padding for the volume in the renaming function.  You might be able to do this with a third party renamer.
If you want to make a formal feature request, please make a new thread here. That'll help me find it when I get back in dev mode.


Adding Pull request Template Guide

Hey!
I have added a file which guides repository owners on how they can add
pull requests template to their repository which is very helpful and adds uniformity in pull requests.
I believe this is a good practice which most of the individuals ignore or may not know about
I’ve created a detailed PR template in the pull-request-template.md file for the repository owners to use it in their repositories
Hope you guys like it :smile:

Thanks but this isn't a good fit for this project as it seems to be a very generic template for a code repo. I'm closing the PR.


enh: add more monitoring around data source upserts in connector

Adds an attempt log and counter + histogram of duration for success and errors
Adds documentUrl and documentLength to the loggerArgs for the upsert logs

post-merge review welcome


How to set the RMap put timeout

Hello  I'm testing the redisson with Master/Slave module .
When redis memory is exhaustion the RMap.put will be wait.
There is no any Exception or someting.
Is anyway to fix this?

does redis have error in this situation?

When available/mappable memory runs out, Redis simply blocks the write requests for every each client/connection, since its event/request loop is single threaded.
Depending on the default configured timeouts, you should get read or write io timeout exception from the underlying socket implementation (Netty in this case).

@wmz7year you may tune LRU eviction mechanism which Redis supports.

@wmz7year it's done in #195 !


cookies don't work

I don't think cookies work as of now, we need to investigate.

Hello guys, any news on this ? It's quite urgent to be fixed on my end. It prevents users to stay logged in after testing requests on my platform

+1

+1

all cookies are deleted after I open scalar page.

Seeing the same issue of all cookies being deleted after the scalar app bootstraps

Facing the same problem, cookies doesn't set. Even I set it manually on request headers, cookies still can't be sent!


Extract environment settings from appengine-web.xml and set them for devappserver1 runs

What do you think of this approach?
If ok, I'll go ahead and add unit tests.
I tested this with devappserver1 and IntelliJ.

The fact that the environment variables can clash between services looks
like a problem for the devappserver1 multi-module model.
On Wed, Apr 19, 2017 at 2:24 PM, Etan Shaul<EMAIL_ADDRESS>wrote:

@etanshaul commented on this pull request.
In src/main/java/com/google/cloud/tools/appengine/cloudsdk/
CloudSdkAppEngineDevServer1.java
https://github.com/GoogleCloudPlatform/appengine-plugins-core/pull/378#discussion_r112278540
:

@@ -194,4 +199,22 @@ boolean isJava8(List services) {
}
return java8Detected;
}

@VisibleForTesting
Map<String, String> getAllAppEngineEnvironment(List services) {
Map<String, String> allAppEngineEnvironment = Maps.newHashMap();
for (File serviceDirectory : services) {

 Path appengineWebXml = serviceDirectory.toPath().resolve("WEB-INF/appengine-web.xml");

I know theres some duplication here with isJava8; looking at refactoring a
little.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/appengine-plugins-core/pull/378#pullrequestreview-33581071,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHf5HRecWder1GtvSMGUclEMky6QU8n7ks5rxlFKgaJpZM4NCDy9
.

That's true. However isn't that a problem for devappserver2 as well? I wonder how it handles it.

devappserver2 uses a different jvm for each service.
In any case this seems better than what we have now. Even if there's
potential for a clash.
On Wed, Apr 19, 2017 at 2:28 PM, Etan Shaul<EMAIL_ADDRESS>wrote:

@etanshaul commented on this pull request.
In src/main/java/com/google/cloud/tools/appengine/AppEngineDescriptor.java
https://github.com/GoogleCloudPlatform/appengine-plugins-core/pull/378#discussion_r112279586
:

@@ -99,16 +105,55 @@ public boolean isJava8() {
&& getRuntime().startsWith("java8");
}

private static String getTopLevelValue(Document doc, String parentTagName, String childTagName) {

public Map<String, String> getEnvironment() {
Node environmentParentNode = getTargetNode(document, "appengine-web-app", "env-variables");
return getAttributeMap(environmentParentNode, "name", "value");
}

private static String getText(Node node) {
if (node != null) {

 return node.getTextContent();

}

return null;
}

private static Map<String, String> getAttributeMap(Node node, String key, String value) {

I realized I need to update this to also search for a target child node by
name. E.g.:

The logic should probably search for the tag 'env-var'.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/appengine-plugins-core/pull/378#pullrequestreview-33582207,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHf5HTo-UTc-Laos-rrL6H1tMHaj0bhdks5rxlJHgaJpZM4NCDy9
.

oh, I see. yeah the effect of the clash here is simply to use the value from the last service in the list.

that's probably worth warning on.
On Apr 19, 2017 2:31 PM, "Etan Shaul"<EMAIL_ADDRESS>wrote:

oh, I see. yeah the effect of the clash here is simply to use the value
from the last service in the list.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/appengine-plugins-core/pull/378#issuecomment-295380776,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHf5HQU7rTU1ok1aOPEosJXu5h44dNjaks5rxlMIgaJpZM4NCDy9
.

Codecov Report

Merging #378 into master will decrease coverage by 0.27%.
The diff coverage is 61.22%.

@@            Coverage Diff             @@
##           master     #378      +/-   ##
==========================================
- Coverage   63.82%   63.55%   -0.28%
==========================================
  Files          66       66
  Lines        1703     1734      +31
  Branches      252      261       +9
==========================================
+ Hits         1087     1102      +15
- Misses        508      521      +13
- Partials      108      111       +3

Impacted Files
Coverage Δ

...oogle/cloud/tools/appengine/cloudsdk/CloudSdk.java
40.92% <ø> (+0.15%)
:arrow_up:

...gle/cloud/tools/appengine/AppEngineDescriptor.java
61.7% <57.14%> (-20.45%)
:arrow_down:

...ppengine/cloudsdk/CloudSdkAppEngineDevServer1.java
82.24% <71.42%> (-1.8%)
: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 814aca9...841a027. Read the comment docs.

@loosebazooka @patflynn and anyone else that's interested, please review. This change affects all tooling updated to use devappserverv1, allowing it to properly set environment variables configured in appengine-web.xml.

@loosebazooka added unit tests for AppEngineDescriptor

@elharo since you previously commented, are you good with this now?

@elharo @briandealwis please take a look to see if I've addressed all your comments.

@briandealwis / @elharo do I have the ok? I'd love to merge this in today if you have no further comments.

@etanshaul
@elharo
@briandealwis
What do you guys think of a log.info message tell the user we're setting these environment variables?

+1: that would help the developer figure out where values are coming from.

@patflynn, added a logging statement ca8678e69677ceb65a768d6923855087d5ffae6c

discussed with @elharo offline and we are good to go with this. will merge


Enumerable

In This PR we did :

[ ] correct (GitHub Flow.
[ ] Check offical documentation about the following 3 methods in Enumerable.
[ ] Create a class MyList that has an instance variable @list.
[ ] In MyList implement a method #each that yields successive members of @list and uses the MyEnumerable module.
[ ] Create a module MyEnumerable that implements the following methods (they should have the same funcionality as methods in Enumerable):
[ ] Each class and module should has a separate .rb file.
[ ] Verify  solution:

Thank You


Revamp to avoid conflicts

The same as the PR Alice had already gone through

Unban account free

Unban account id fre fre
Bhai hlep me


Problem running on docker image cypress/included:3.8.0

Hi,
I'm trying to use cy.matchImageSnapshot(key) on GitLab CI with docker image, but I'm having the following error:
CypressError: cy.task('Matching image snapshot') failed with the following error:

The 'task' event has not been registered in the plugins file. You must register it before using cy.task()
Fix this in your plugins file here:
/builds/project/cypress/plugins/index.js
https://on.cypress.io/api/task

It works perfectly when I'm running the specs locally but strangely it fail when they are runned on the docker image cypress/included:3.8.0 by the gitlab CI.
I have in plugins\index.js :
const {
  addMatchImageSnapshotPlugin,
} = require('cypress-image-snapshot/plugin');

module.exports = (on, config) => {
  addMatchImageSnapshotPlugin(on, config);
};

in support\commands.js :
import 'cypress-wait-until';
import {addMatchImageSnapshotCommand} from 'cypress-image-snapshot/command';

addMatchImageSnapshotCommand();

Package.json
"cypress": "^3.8.0",
"cypress-image-snapshot": "^3.1.1",
"cypress-wait-until": "^1.4.2",

I can't find what I missed.
Thanks

I solved it, the plugins\index.js file was dismissed and not taken into account.


Fix typo in Readme.md

Another typo :)  If you put value in options object it's not set the value. Value must be outside the options as a component prop value.

It seems funny but this typo already create two issues :) #29 and another closed where I tell them that this is typo.
@benrlodge please apply this simple PR and wait for another which will fix server-side rendering issues.

@benrlodge Since I add this changes to my next PR. Please close that one.

thanks @RIP21


[DOCUMENTATION] Control Github Wiki page

There must be a controls page on the Github wiki, showing the controls for every single action in the game.

In progress

Please check over to ensure accuracy


update generated files in target folder

Im not sure the target directory should be checked in, in the first place.
@mrdavidlaing ?

@simonjohansson Actually, but convention we do checkin the target/ folder; makes consuming the logstash-filters easier for downstream users since they don't have to get all the bin/build dependancies configured.

@yatzek - you seem to have built using an old version of the source.
For example, you are missing changed from https://github.com/logsearch/logsearch-for-cloudfoundry/commit/45041f4cfda4c082dcd1b749dc4b1320811171a8
Is your fork up to date?

Closing, since this has been superceded by #67


Scan aborted - <StartButton_Clicked>b__13_0()

Current Behavior
When I press "Scan Genshin", it switches to Genshin Impact window (which is in the Paimon menu) and then reverts back to Inventory Kamera with the following error:
at InventoryKamera.InventoryKamera.GatherData()
at InventoryKamera.MainForm.<StartButton_Clicked>b__13_0()
InventoryKamera.log
Expected Behavior
The scanner continues to scan
Reproducing a Behavior
Just click on "Scan Genshin" while making you sure you have the correct resolution, up-to-date Microsoft Visual C++.... version, keybind to default, and game versions up-to-date.
https://user-images.githubusercontent.com/33529271/213145531-284732d6-26df-482c-a712-0d5eff60a0c5.mp4
Device OS
Windows 11
Genshin Impact Version
3.4
Inventory Kamera Version
1.3.6
Screen resolution, screen mode, and UI scale
1280x720
Additional notes and remarks
I have tried all resolutions. Restarted both applications. Tried just scanning Artifacts and different variations. I have used also the correct Traveler's and Wnaderer's name.
Evidence
InventoryKamera.log
InventoryKamera.debug.log
Preflight Checklist

[X] I have checked for similar issues in the issue page, and I have not found any similar to mine.
[X] I am using the latest version of the scanner.
[X] I tried setting the scanning delay to maximum, and it still doesn't work.
[X] I have attached the video recording to the bug report.
[X] I have attached the error log to the bug report.
[X] I am available to later explains the issue.
[X] I have disabled all screen filtering software or plugins (i.e. HDR, Image Sharpening) while scanning.
[X] I use default in-game keybindings or have configured keybindings for the scanner.

same bug :(

Had the same issue, fixed by changing my nickname, guess Traveler and Wanderer nickname cannot be the same

And the nickname of the traveler must not have a name from another playable character (e.g. Tartaglia).

And the nickname of the traveler must not have a name from another playable character (e.g. Tartaglia).

thank you @EinUnknownCoder , this solved the issue for me. My Traveler's Name was the same as an in-game character. When I changed it, it work without any issue.


gvcf: rename field MIN to MIN_DP

MIN_DP is the field name produced by a couple other callers and appears in the draft VCF 4.3 spec... https://github.com/samtools/hts-specs/blob/master/VCFv4.3.tex

@mlin https://github.com/zeeev/freebayes , https://github.com/zeeev/freebayes/blob/master/src/AlleleParser.cpp
Done.


Crash in FirebaseDatabase -> FSRWebSocket.m - Line 601

Step 0: Are you in the right place?

For issues or feature requests related to the code in this repository
file a Github issue.

If this is a feature request please use the Feature Request template.

For general technical questions, post a question on StackOverflow
with the firebase tag.
For general (non-iOS) Firebase discussion, use the firebase-talk
google group.
For backend issues, console issues, and other non-SDK help that does not fall under one
of the above categories, reach out to
Firebase Support.
Once you've read this section and determined that your issue is appropriate for
this repository, please delete this section.

[REQUIRED] Step 1: Describe your environment

Xcode version: 12.4
Firebase SDK version: 7.9.0
Installation method: CocoaPods
Firebase Component: Database

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
What happened? How can we make the problem occur?
I was not able to reproduce it myself but do have the crash report. We started noticing it after updating to 7.9.0. No code change was done we just updated Firebase version from 6.23.0 to 7.9.0 and started using XCode 12.4. We were using XCode 11.4.1 before.
100% iPhone
88% iOS 14
12% iOS 13
If you have a downloadable sample project that reproduces the bug you're reporting, you will
likely receive a faster response on your issue.
Relevant Code:
Crashed: FirebaseWorker
0  libsystem_kernel.dylib         0x1a061b0dc __pthread_kill
1  libsystem_pthread.dylib        0x1a0694094 pthread_kill$VARIANT$mp
2  libsystem_c.dylib              0x1a0573ea8 abort
3  libsystem_malloc.dylib         0x1a066e684 _malloc_put
4  libsystem_malloc.dylib         0x1a066e83c malloc_report
5  libsystem_malloc.dylib         0x1a066149c free
6  libobjc.A.dylib                0x19fc66724 object_dispose
7  CFNetwork                      0x1a103572c -[CONNECTION_SessionTask dealloc]
8  �(�n                          0x1034ac1d4 (Missing)
9  CFNetwork                      0x1a1064a58 _CFSocketStreamCreatePair
10 FirebaseDatabase               0x1023066a8 -[FSRWebSocket _initializeStreams] + 601 (FSRWebSocket.m:601)
11 FirebaseDatabase               0x102305754 -[FSRWebSocket initWithURLRequest:protocols:queue:googleAppID:andUserAgent:] + 325 (FSRWebSocket.m:325)
12 FirebaseDatabase               0x102305878 -[FSRWebSocket initWithURLRequest:queue:googleAppID:andUserAgent:] + 338 (FSRWebSocket.m:338)
13 FirebaseDatabase               0x1023225a0 -[FWebSocketConnection initWith:andQueue:googleAppID:lastSessionID:] + 119 (FWebSocketConnection.m:119)
14 FirebaseDatabase               0x1022c8c5c -[FConnection initWith:andDispatchQueue:googleAppID:lastSessionID:] + 56 (FConnection.m:56)
15 FirebaseDatabase               0x1022ee53c -[FPersistentConnection openNetworkConnectionWithToken:] + 540 (FPersistentConnection.m:540)
16 FirebaseDatabase               0x1022ee40c __45-[FPersistentConnection tryScheduleReconnect]_block_invoke_2 + 508 (FPersistentConnection.m:508)
17 libdispatch.dylib              0x1a04bca38 _dispatch_call_block_and_release
18 libdispatch.dylib              0x1a04bd7d4 _dispatch_client_callout
19 libdispatch.dylib              0x1a0466324 _dispatch_lane_serial_drain$VARIANT$mp
20 libdispatch.dylib              0x1a0466e40 _dispatch_lane_invoke$VARIANT$mp
21 libdispatch.dylib              0x1a046f4ac _dispatch_workloop_worker_thread
22 libsystem_pthread.dylib        0x1a069e114 _pthread_wqthread
23 libsystem_pthread.dylib        0x1a06a0cd4 start_wqthread

This looks like another instance of #3667 which we weren't able to solve.  Any information about reproducing would be helpful.

In the issue that @paulb777 refers above (#3667) , there is a mention of 'hookkeyboard.dylib' in what corresponds to stackframe 2 in the crashed thread.
As mentioned elsewhere, it does appear to be a bit suspicious - and it might be doing some swizzling of low level functionality.
It could be something that is only seen on jailbroken devices - altough that would of course be good to verify.

@Tulakshana Please check if the crash is only occurring on jail-broken devices.

@paulb777 nope they are not Jailbroken

@paulb777 hey in all the crash reports I have I have a line that I don't recognize. Class name is not readable and the log says '(Missing)' instead of showing the effected line. Any idea why this might be?
7  CFNetwork                      0x20fa1972c -[CONNECTION_SessionTask dealloc]
8  �h[o                          0x10f5f01d4 (Missing)
9  CFNetwork                      0x20fa48a58 _CFSocketStreamCreatePair
10 FirebaseDatabase               0x101e1e698 -[FSRWebSocket _initializeStreams] + 601 (FSRWebSocket.m:601)

I think that line is precisely why @paulb777 might have asked if the devices are jailbroken, because it shouldn't be possible to inject code from one installed app to another, but if the device is jailbroken, then such a situation is possible.

I think that line is precisely why @paulb777 might have asked if the devices are jailbroken, because it shouldn't be possible to inject code from one installed app to another, but if the device is jailbroken, then such a situation is possible.

I see. Firebase didn't indicate them to be Jailbroken though. I saw the following today.

Closing for staleness. We can reopen with more details about how to reproduce.


Restrict JobAPI & Modify jobs API

Description

api/job/id is now restricted to only a GET request
api/company/jobs/ now accepts a POST request (replaces /api/addjob/)
api/company/jobs/ requires authentication, and checks request field 'company' to ensure id matches the id of the authenticated user. Otherwise it returns a 403 Forbidden.

Fixes # (#94 )
Adds reset messages / errors on Login.js and Register.js
Adds null=True to company_logo in models.py
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Postman tests of new api functionality:
POST to api/job/:id

GET to api/company/jobs/ without authentication:

GET to api/company/jobs/ with authentication:

POST to api/company/jobs/ with incorrectly matched  company id field / user id:

POST to api/company/jobs/ with correctly matched company id / user id

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My code has been reviewed by at least one peer
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Great work on documentation.  merging request.


Fix the quick start in README

Describe the bug
The current README has an outdated quick start about the realtime todo app.
Expected behaviour
The latest quick start in the docs is based on explorer in mission control. The README should reflect this latest way of quick start
Actual behaviour
The readme shows the incorrect oudated realtime todo app example for quick start

If this bug restricts your use of space-cloud, give it a thumbs up reaction, so that we can determine which bugs need to be fixed immediately.
👍

Fixed


Twitter Image Pipeline should be updated to latest

TIP in XamarinComponents is on 2.2.2 but https://github.com/twitter/ios-twitter-image-pipeline is up to 2.7.2 with some significant improvements.  Should consider keeping this component up to date.

@NSProgrammer  i have added this to the backlog
thanks


Draft: [SPARK] Proof of concept for handling lineage with DataStreamWriter#foreachBatch

…reamWriter#foreachBatch(Dataset[Row], Long)
Problem
👋 Thanks for opening a pull request! Please include a brief summary of the problem your change is trying to solve, or bug fix. If your change fixes a bug or you'd like to provide context on why you're making the change, please link the issue as follows:
Closes: #ISSUE-NUMBER
Solution
Please describe your change as it relates to the problem, or bug fix, as well as any dependencies. If your change requires a schema change, please describe the schema modification(s) and whether it's a backwards-incompatible or backwards-compatible change, then select one of the following:

Note: All schema changes require discussion. Please link the issue for context.

[ ] Your change modifies the core OpenLineage model
[ ] Your change modifies one or more OpenLineage facets

If you're contributing a new integration, please specify the scope of the integration and how/where it has been tested (e.g., Apache Spark integration supports S3 and GCS filesystem operations, tested with AWS EMR).
One-line summary:
Checklist

[ ] You've signed-off your work
[ ] Your pull request title follows our guidelines
[ ] Your changes are accompanied by tests (if relevant)
[ ] Your change contains a small diff and is self-contained
[ ] You've updated any relevant documentation (if relevant)
[ ] Your comment includes a one-liner for the changelog about the specific purpose of the change (if necessary)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[ ] You've added a header to source files (if relevant)

SPDX-License-Identifier: Apache-2.0
Copyright 2018-2023 contributors to the OpenLineage project

@d-m-h closing, because @Imbruced's PR solved that already.


Attribute error

I am suddenly getting this error when I run gdax.PublicClient() and gdax.AuthenticatedClient(API_KEY, API_SECRET,API_PASS) - it didnt happen before
AttributeError: module 'gdax' has no attribute 'PublicClient'

This must be an issue with pip.  Please update your directory with the master branch here on Github and follow PR #260 for an idea of when PIP will be updated.


Use HTTPS where possible

This avoids mixed content warnings on https://ucnv.github.io/pnglitch/.

I don't understand why it should be capable with https.
Please let me know where the https url is actually used.

HTTPS Everywhere users (like me) automatically get redirected to the HTTPS version, which then fails to load the HTTP resources.
Let’s turn that question around: why wouldn’t you want the site to work over HTTPS? There’s several good reasons to use HTTPS, and no reason at all to prefer HTTP.

OK, I will consider this.
I will close this request once.
Thanks anyway.

Please merge this PR. It has no downside, and improves the security of your site.


Use explicit relative imports

Bad idea to use the old relative imports since if you have a duplicate
name somewhere in your module path it might try and import the wrong
module causing an error.

Do you mean like
from soccer import leagueids, teammates
or
from soccer.writers import get_writer
I see the leading dot style a lot so I just did that. I don't think that style is totally discouraged but there is possibly situations when the absolute import is necessary.

i see those alot too but apparently its in the PEP8 too where absolute imports are preferred:
http://stackoverflow.com/questions/4209641/absolute-vs-explicit-relative-import-of-python-module

Seems like not everyone likes them but now that the syntax is more standardised a lot of projects do the relative import.
So if it is a matter of style your preference is for absolute imports?

that is my preference but not a necessity...its more important we follow a consistent format :)

That is true.
@architv can pick a style he prefers.

@Saturn @ueg1990 I think we should stick with absolute imports since that's the format followed in the package.

@Saturn Let me know if there is any other issue or we can close the PR.

Changed to use absolute imports. It seems to work for me.


Add BRD to processing team mapping 761

BRD was missed in the initial breakdown for 761. BRD should be part of central team
#761
Description
Summarize issue
Dependencies
Are there any dependencies?
DOD

[ ] Add BRD to Central Team ministries
[ ]
[ ]
[ ]
[ ]

@arielleandrews will confirm if we still need this ministry

BRDO ceased to exist and is now CABRO which falls under Crown Agencies Secretariat under the Ministry of Finance - @divyav-aot do we need any changes to reflect this or are we good to close this ticket? cc @nkan-aot

CAS already has its own checkbox so i believe this can be closed


Suggestion: Community contributor, SIG info, reviewers split

During review of https://github.com/o3de/community/pull/158, it became clear that information about community, SIGs, and available reviewer/maintainer roles is duplicated or split in a hard-to-discover way across multiple files. This issue is for awareness of the problem to produce a full RFC if needed, or at least discussion within the project.
As a starting point, I would suggest the following be how the information is maintained and organized:

README.md - Should not just be an index of the SIGs/repos - should be a short summary of the project, who we are, etc.
CONTRIBUTING.md - The current community-membership.md page should be merged into CONTRIBUTING.md, excepting the role descriptions.
ROLES.md - An expanded description of community roles (currently maintained in the docs affected by #158) including chairperson, TSC/TAC roles and how they are staffed, as well as links to any SIG-specific roles (such as sig-docs-community requiring a "Website reviewer" role)

We just pushed an update to the README which I think gives more of an overview view of the project/repro.
Agree that the how to contribute sections of community-membership.md should be merged with CONTRIBUTING.md
Would not agree with renaming ``community-membership.md` but instead have that be a list of roles ie contributor, maintainer, reviewer, chair and co-chair and TSC membership.


SEO improvement: change rename image file

Is your feature request related to a problem? Please describe.
SEO improvement: change rename image file.
Describe the solution you'd like
I propose to replace the renaming of the image files like this: original-name-random-number.extension.
Can anyone tell me where to look in the files?
Thanks.
Describe alternatives you've considered
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that asks for the same feature to avoid creating a duplicate.

The UploadsController handles storing images.
How does your proposition improve SEO?

SEO specialists argue that the filename must match the keyword.
Then:
original-name = keyword
random-number = avoid replacing an existing file.
A better solution would be:
title-post / keyword.ext.

Thanks for the suggestion. Drop any links here that support that hypothesis.
I'll see about adding this to the v7 features.

I tried this solution.
    public function store()
    {
        $payload = request()->file();

        if (! $payload) {
            return response()->json(null, 400);
        }

        // Only grab the first element because single file uploads
        // are not supported at this time
        $file = reset($payload);

        $name = time() . '-' . $file->getClientOriginalName();

        //$path = $file->storePublicly(Canvas::baseStoragePath(), [
        //    'disk' => config('canvas.storage_disk'),
        //]);

        $path = $file->storeAs(Canvas::baseStoragePath(), $name);

        return Storage::disk(config('canvas.storage_disk'))->url($path);
    }

Search Engine Optimization (SEO) Starter Guide


[vcpkg] Support cross compile arm64-linux, arm-linux triplets on a linux x86_64 host.

Describe the pull request

What does your PR fix?
Support cross compile arm64-linux, arm-linux triplets on a linux x86_64 host.

Which triplets are supported/not supported? Have you updated the CI baseline?
Support arm64-linux and arm-linux triplets.

Does your PR follow the maintainer guide?
Yes

Personal opinion: All those crosscompiling toolchains should have seperate toolchains and not be cramped into a single universal linux one.

I did something related to this in #11535 but only for an autoconf system for that specific port. Not sure if that would be related though. In that case I depend on the cross-compiler toolchain file.

Where should we put the cross-compiler toolchain file? If we ask user to supply it, user might not know how to generate it. Ideally, in most of cases, user only need few additional efforts to make cross compiling work.

I'm referring to cmake .. -DCMAKE_TOOLCHAIN_FILE=path/to/vcpkg/scripts/buildsystems/vcpkg.cmake -DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=some/cmake/toolchain/for/crosscompiler

But how does it work with ./vcpkg install ...? Seem's we can't pass -DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=some/cmake/toolchain/for/crosscompiler to vcpkg install.

But how does it work with ./vcpkg install ...? Seems we can't pass -DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=some/cmake/toolchain/for/crosscompiler to vcpkg install.

You add it into the triplet.
Simply use set(VCPKG_CHAINLOAD_TOOLCHAIN_FILE <sometoolchainpath>) in the triplet

/azp run

I think this is the right incremental improvement.
Once complexity increases enough, it may make sense to have separate files for each of the different X-to-Y cross targets; however right now we support all Linux targets out of a single file (linux.cmake). Furthermore, the detection of which X-to-Y file to select still would need to exist somewhere. Finally, our toolchain hashing system currently supports only hashing one file for the toolchain and doesn't have a mechanism to determine which specific combo was loaded.
For all these reasons, this looks like a good improvement for our current state and will help inform more improvements into the future.

/azp run

Thanks for your contribution!


Error creating new Foundation Apps project

When trying to create a new Foundation Apps project I am seeing the following message:
Downloading the Foundation for Apps template...
Done downloading!
Installing dependencies...
npm  =>  bower  =>  bundler
C:\Users\r269651\AppData\Roaming\npm\node_modules\foundation-cli\lib\foundationC
LI.js:130
if (err instanceof Error) throw err;
^
Error: spawn ENOENT
at errnoException (child_process.js:998:11)
at Process.ChildProcess._handle.onexit (child_process.js:789:34)
I'm unclear what the cause of this error would be, so any suggestions would be greatly appreciated.

I am getting below issue
C:/> cordova plugin add https://git-wip-us.apache.org/repos/asf/cordova-plugin-device.git
Fetching plugin "https://git-wip-us.apache.org/repos/asf/cordova-plugin-device.git" via git clone
Error: spawn ENOENT
at errnoException (child_process.js:1001:11)
at Process.ChildProcess._handle.onexit (child_process.js:792:34)
any solution?

@manjunathsoftwares You're running a Cordova command, which unfortunately we can't help you with. But to give you a hint, ENOENT in conjunction with Node's spawn() command sometimes means it can't find the command it's trying to run, which in this case appears to be Git. Make sure you have Git installed on your machine.


Attributes as selector

Is it possible to target the attribute instead of the element?
For example: i have this html; Manage Products
I want to target the date-test-id instead of the classes which prone to break when refactoring. Thanks

this work 		attachTo: { element: "[data-test-id='chart-title']" },


Change Id number to account number

No more Id number is needed, It'll be replaced by user account number (random string of 8 numbers).

Fixed in de0c5b54e8c176868c377669a11f16d1da638620


Google maps plugin error, Failed to handle method call

App crashing on google map load
release apk only debug is working well
2020-11-11 20:44:58.995 25431-25431/? E/MethodChannel#flutter/platform_views: Failed to handle method call     java.lang.NullPointerException: Attempt to invoke interface method 'java.lang.Object java.util.Map.get(java.lang.Object)' on a null object reference         at io.flutter.plugins.googlemaps.e.s(Unknown Source:10)         at io.flutter.plugins.googlemaps.g.a(Unknown Source:28)         at io.flutter.plugin.platform.SingleViewPresentation.onCreate(Unknown Source:116)         at android.app.Dialog.dispatchOnCreate(Dialog.java:423)         at android.app.Dialog.show(Dialog.java:317)         at android.app.Presentation.show(Presentation.java:259)         at io.flutter.plugin.platform.l.<init>(Unknown Source:55)         at io.flutter.plugin.platform.l.a(Unknown Source:65)         at io.flutter.plugin.platform.k$a.g(Unknown Source:110)         at io.flutter.embedding.engine.i.i$a.b(Unknown Source:152)         at io.flutter.embedding.engine.i.i$a.G(Unknown Source:144)         at g.a.d.a.j$a.a(Unknown Source:17)         at io.flutter.embedding.engine.e.b.d(Unknown Source:57)         at io.flutter.embedding.engine.FlutterJNI.handlePlatformMessage(Unknown Source:4)         at android.os.MessageQueue.nativePollOnce(Native Method)         at android.os.MessageQueue.next(MessageQueue.java:336)         at android.os.Looper.loop(Looper.java:182)         at android.app.ActivityThread.main(ActivityThread.java:7564)         at java.lang.reflect.Method.invoke(Native Method)         at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:539)         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:950) 2020-11-11 20:44:58.997 25431-25491/? E/flutter: [ERROR:flutter/lib/ui/ui_dart_state.cc(177)] Unhandled Exception: PlatformException(error, Attempt to invoke interface method 'java.lang.Object java.util.Map.get(java.lang.Object)' on a null object reference, null, java.lang.NullPointerException: Attempt to invoke interface method 'java.lang.Object java.util.Map.get(java.lang.Object)' on a null object reference         at io.flutter.plugins.googlemaps.e.s(Unknown Source:10)         at io.flutter.plugins.googlemaps.g.a(Unknown Source:28)         at io.flutter.plugin.platform.SingleViewPresentation.onCreate(Unknown Source:116)         at android.app.Dialog.dispatchOnCreate(Dialog.java:423)         at android.app.Dialog.show(Dialog.java:317)         at android.app.Presentation.show(Presentation.java:259)         at io.flutter.plugin.platform.l.<init>(Unknown Source:55)         at io.flutter.plugin.platform.l.a(Unknown Source:65)         at io.flutter.plugin.platform.k$a.g(Unknown Source:110)         at io.flutter.embedding.engine.i.i$a.b(Unknown Source:152)         at io.flutter.embedding.engine.i.i$a.G(Unknown Source:144)         at g.a.d.a.j$a.a(Unknown Source:17)         at io.flutter.embedding.engine.e.b.d(Unknown Source:57)         at io.flutter.embedding.engine.FlutterJNI.handlePlatformMessage(Unknown Source:4)         at android.os.MessageQueue.nativePollOnce(Native Method)         at android.os.MessageQueue.next(MessageQueue.java:336)         at android.os.Looper.loop(Looper.java:182)         at android.app.ActivityThread.main(ActivityThread.java:7564)         at java.lang.reflect.Method.invoke(Native Method)         at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:539)         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:950)     )     #0      StandardMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:582)     #1      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:159)     <asynchronous suspension>     #2      TextureAndroidViewController._sendCreateMessage (package:flutter/src/services/platform_views.dart:1039)     <asynchronous suspension>     #3      AndroidViewController.create (package:flutter/src/services/platform_views.dart:749)     <asynchronous suspension>     #4      RenderAndroidView._sizePlatformView (package:flutter/src/rendering/platform_view.dart:174)

@giorgi-abashidze  Are you solved this bug ?
this happened with me and I can't solve it.

@giorgi-abashidze  Are you solved this bug ?
this happened with me and I can't solve it.


added version checks for wildcard queries

unit tests pass and manually running a job against an xsearch endpoint works.


Jesse: Watch invaat.com

Jesse requests the watch of the watch_keyword invaat\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

invaat\.com has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by Nisse Engström in Charcoal HQ


Pull request after adding madlibs

We got another conflict!

Conflict resolved!
Now our story should be completed.


Regex for keywords?

Hi,
First of all - great scraper! really, truly, awesome work! It's a joy playing around with this!
So, my question is in regard to the use of regex for keywords. I wanted to use regex for specialized search string patterns, but I can't figure out how to get this to work. If I wanted all keywords that had 0-5 characters (a-z, 0-9) and ended with the characters 'ej6', how would I make that work?
I attempted:
let config = {
    keywords: ['/^[a-z0-9]{0,5}$/ej6'],

let config = {
    keywords: ['/^[a-z0-9]{0,5}$/' + 'ej6'],

... along with a few other attempts, but I honestly have no idea how to make it work with regex, and maybe there is an even better way using JS (?).
Best,
Simon

Do you want to search all possible words with the pattern /[a-z0-9]{0,5}ej6/ ?
Then just generate them via script.
For example in python:
import string
import itertools

keywords = list(itertools.combinations(string.ascii_lowercase + string.digits, 5))

with open('keywords.txt', 'w') as f:
    for kw in keywords:
        f.write(''.join(kw)+'ej6\n')


Added timer command

This pull request changes the following things:
Added a timer command for synchronizing time between users. Moderators can start (or restart) timers with timer start. All users can get the current timestamp with timer. Timers are unique by each channel.
I went with your implementation storing the timestamps for each channel, specifically stored in a map.
Let me know if you think steps for concurrency are warranted here. I didn't bother with it since I couldn't imagine a situation where a race condition would cause an issue.
Implements #52
By submitting this Pull Request I agree to have done the following (check all that apply):

[x] Run go fmt on all the files I've changed
[x] Tested the bot on my own server to verify the feature works
[x] Verified with CamD67 that the feature is desired (Check out the Projects page, or submit an issue for new features)
[x] Submitted PR to the develop branch not master!

Golang detects for concurrent map read and write (or write and write) and causes a fatal error: concurrent map read and map write.
https://golang.org/doc/go1.8#mapiter
I'll update the code to use sync.Mutex for concurrency.

One big thing that this in-memory solution doesn't have is safety against crashes and reboots. All timers will be blown away if that happens. I'd prefer it to not happen very often, but it does happen occasionally.
This works for now, and when we update more of the infrastructure to be more easily reboot-able we should revisit this.
Still, looks good to me 👍


Add --gem option to update a specific gem

Thanks for the PR.
What benefit does adding this option provide vs. just having someone run bundle update #{gemname}?

bummr generates a commit message with details on the old & new versions of the gem.
It does that well, so I wanted to use it instead of manually bundle update #{gem} && git commit -m "Update #{gem} gem from XXX to YYY".


Change whatbrowser.org link to HTTPS

Changing http://www.whatbrowser.org/intl/ to https://www.whatbrowser.org/intl/ because the website supports HTTPS now. HTTPS provides security and authenticity, and I think it is something that CloudFlare officially supports. Moreover, it makes no sense to have websites running on CloudFlare support HTTPS and then have them link to a non-secure location. (At least that is why I don't use "A Better Browser" right now.)

@xPaw Do you know how long it usually takes for the code to update on production servers? Thanks!

I have to push a new version first.

I see. Thanks!


Support custom Bundles in RiveFile convenience init

Problem
By now RiveFile+Extensions.swift does not support user-defined modules. This results in boilerplate code when using Rive animations from SPM projects with a single bundle for each target/module.
Consider the following example:
Repo layout:
Sources/ModuleA/Resources/Animation.riv
Sources/ModuleA/A.swift
Sources/ModuleA/Resources/Animation.riv
Sources/ModuleA/B.swift

Code inside A.swift:
struct AView: View {
  var animation = { () -> RiveViewModel? in
    guard
      let url = Bundle.module.url(forResource: "Animation", withExtension: "riv"),
      var data = try? Data(contentsOf: url),
      let file = try? data.withUnsafeMutableBytes({ [count=data.count] (bytes) -> RiveFile? in
        guard let baseAddress = bytes.baseAddress else { return nil }
        return try RiveFile(
          bytes: baseAddress.bindMemory(to: UInt8.self, capacity: count),
          byteLength: UInt64(count)
        )
      })
    else { return nil }
    let model = RiveModel(riveFile: file)
    return RiveViewModel(model)
  }()
}

Proposal
Add bundle parameter to RiveFile.init(name:extension:), like this:
convenience init(name fileName: String, extension ext: String = ".riv", bundle: Bundle = .main) throws {

I agree that it’s currently suboptimal for multiple bundles, something better is needed. I use a similar workaround for data.
For safer and shorter code when initializing from data, I just cast the data.
let data = Data(whatever)
let riveFile = RiveFile(byteArray: [UInt8](data))

Was implemented here: https://github.com/rive-app/rive-ios/pull/250


Add warning label to README.md to notify users NativeWalletSDK is WIP

Summary
Add warning label to README.md to notify users NativeWalletSDK is WIP

Review Error for erin-at-work @ 2022-07-26 23:16:16 UTC
User failed mfa authentication, public email is not set on your github profile. see go/mfa-help


Make hardcoded ROCm path configurable

During code compilation in pytorch, I get the error that the linker ld.lld cannot be found.
This is due to the the hardcoded paths here
https://github.com/ROCmSoftwarePlatform/triton/blob/797737bccf61074ecdce450b72b677dfc26eee29/lib/Target/HSACO/HSACOTranslation.cpp#L137-L138
and here
https://github.com/ROCmSoftwarePlatform/triton/blob/797737bccf61074ecdce450b72b677dfc26eee29/scripts/amd/check_llvm_src.sh#L2
It would be nice if these could be made configurable via environment variables or during package installation.
I am using the triton pypi package that is installed with pytorch 2.0. I don't know which branch this corresponds to (in this repository?)

Hi,
Thanks for finding that oversight.
We have a PR with a fix targeted at our main dev branch triton-mlir.
The branch that is used for Pytorch 2.0 is triton-mlir-pyt20.
Once the PR lands in the main dev branch, we will cherry-pick the commit into the Pytorch 2.0 branch.

The fix for this has landed so closing this.


SslProtocols.None not working in .NET 4.7.2

FTP OS:  Windows
Computer OS: Windows
When trying to connect to a FTPS Server we are forced to provide the list of protocols that are accepted. This should not be the case as per .NET 4.7.2.
In .NET 4.7.2 the choice of protocol was delegated OS and if we just select Default the OS will pick the highest and most relevant one based on the OS setting.
WRONG
ftpClient.SslProtocols = Security.Authentication.SslProtocols.Tls Or Security.Authentication.SslProtocols.Tls11 Or Security.Authentication.SslProtocols.Tls12
Right
ftpClient.SslProtocols = Security.Authentication.SslProtocols.None
Reference link SslProtocols Enum
Can this be addressed soon please.
Thanks

Related to #176.
Which solution should be done,  conn.SslProtocols = SslProtocols.Default | SslProtocols.Tls11 | SslProtocols.Tls12  or None?

Can you please reply?

Can you please reply?

It should be Security.Authentication.SslProtocols.None

I would appreciate it if the default, as recommended by microsoft, is changed to Security.Authentication.SslProtocols.None.


Revert emphasis back to inline code

Revert emphasis back to inline code (changed by bba38e9b8d57d71c463c6efff0966c851964e41a), per Markdown convention.

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

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-configuration.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/param-persist-howto.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/param-persist-tutorial.md
:white_check_mark:Succeeded
View

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

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

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-configuration.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/param-persist-howto.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/param-persist-tutorial.md
:white_check_mark:Succeeded
View

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

Using bold font for commands is not an industry convention.
For example, for gcloud CLI:
https://cloud.google.com/sdk/gcloud/reference/config/set

and AWS CLI:
https://docs.aws.amazon.com/cli/latest/reference/configure/set.html

PowerShell's doc Editor's checklist - Formatting also suggests commands should be marked as inline code.

Backtick syntax elements that appear, inline, within a paragraph

Cmdlet names Verb-Noun

Indeed this is something Azure CLI should align with PowerShell.

Using bold font for commands is not an industry convention.
For example, for gcloud CLI:
https://cloud.google.com/sdk/gcloud/reference/config/set

and AWS CLI:
https://docs.aws.amazon.com/cli/latest/reference/configure/set.html

PowerShell's doc Editor's checklist - Formatting also suggests commands should be marked as inline code.

Backtick syntax elements that appear, inline, within a paragraph

Cmdlet names Verb-Noun

Indeed this is something Azure CLI should align with PowerShell.


[CheckJs Error]: type inference from octokit request not working

Details:
{
	"resource": "/Users/oscardom/dev/open-source/octoherd/script-setup-renovate/script.js",
	"owner": "typescript",
	"code": "2339",
	"severity": 8,
	"message": "Property 'content' does not exist on type '{ type: string; size: number; name: string; path: string; content?: string; sha: string; url: string; git_url: string; html_url: string; download_url: string; _links: { git: string; html: string; self: string; }; }[] | { ...; } | { ...; } | { ...; }'.\n  Property 'content' does not exist on type '{ type: string; size: number; name: string; path: string; content?: string; sha: string; url: string; git_url: string; html_url: string; download_url: string; _links: { git: string; html: string; self: string; }; }[]'.",
	"source": "ts",
	"startLineNumber": 35,
	"startColumn": 39,
	"endLineNumber": 35,
	"endColumn": 46
}

{
	"resource": "/Users/oscardom/dev/open-source/octoherd/script-setup-renovate/script.js",
	"owner": "typescript",
	"code": "2339",
	"severity": 8,
	"message": "Property 'sha' does not exist on type '{ type: string; size: number; name: string; path: string; content?: string; sha: string; url: string; git_url: string; html_url: string; download_url: string; _links: { git: string; html: string; self: string; }; }[] | { ...; } | { ...; } | { ...; }'.\n  Property 'sha' does not exist on type '{ type: string; size: number; name: string; path: string; content?: string; sha: string; url: string; git_url: string; html_url: string; download_url: string; _links: { git: string; html: string; self: string; }; }[]'.",
	"source": "ts",
	"startLineNumber": 37,
	"startColumn": 30,
	"endLineNumber": 37,
	"endColumn": 33
}

Agh, I think it's because of this Octokit issue: https://github.com/octokit/rest.js/issues/1971
What's your opinion in using //@ts-expect-error here?


Add rationales for every criterion

Is your feature request related to a problem? Please describe.
There are many criteria, but it's not always clear why those criteria are present. Why should something be pinned (for example)? In particular, sometimes criteria aren't appropriate, but it's hard to determine that without knowing its rationale.
Describe the solution you'd like
Provide for each criteria the rationale.

E.g., for pinning,
"Pinning prevents a malicious later version (that wasn’t necessarily reviewed) revealing secrets or creating artifacts that end up in a production version, and makes it easier to determine exactly what was changed when a problem is detected."
This rationale suggests that you don’t need to pin something if an unpinned version cannot reveal secrets AND it cannot change a generated artifact that could be used in production. For example, it might be okay to have a test unpinned - it might try to create cryptocurrency, or produce an unexpected result (e.g.,  "always passes"), but its damage would be limited.

@olivekl @laurentsimon assigned it to all of us. Will setup a sync for this in the coming weeks.

E.g., for pinning, "Pinning prevents a malicious later version (that wasn’t necessarily reviewed) revealing secrets or creating artifacts that end up in a production version, and makes it easier to determine exactly what was changed when a problem is detected."

that's not exactly true. Not pinning dependencies is akin to not knowing what you have in your repo.  When codecov hack happened, you could have quickly looked at your commit history and pin point precisely which versions may have run a malicious version; and it would speed up incidence response. Instead of looking for the past 3 years, you only look at this exact 3 weeks, for example.

This rationale suggests that you don’t need to pin something if an unpinned version cannot reveal secrets AND it cannot change a generated artifact that could be used in production. For example, it might be okay to have a test unpinned - it might try to create cryptocurrency, or produce an unexpected result (e.g., "always passes"), but its damage would be limited.

Giving more example is useful for sure.

I would recommend using the GitHub Actions hardening guide as a reference. That has good rationale for pinning of action and token permissions.
FYI - I have also created a PR to add reference to scorecard in that guide. Please do share feedback in the PR so that scorecard is easy to discover when one reads the hardening guide.
https://github.com/github/docs/pull/14856#issuecomment-1026820809


Proposal: delete typedefs

The use cases for typedefs are:
type Meters = f64;
type Feet = f64;
const a: Meters = 10;
const b: feet = 13;
const c = a + b; // error: incompatible types

type c_style_void = u8;
fn doSomething(opaque: &c_style_void) {

}

The first use case can be solved by wrapping the thing in a struct. Second use case can be solved with public/private fields.

By wrapping it in a struct do you mean
const Meters = struct {
    value: f32
}

This feels like unecessary nesting.
type Feet = f64;

const Distance = enum {
    meters: Meters,
    feet: Feet,
}

const d = Distance.meters { Meters { .value = 1.0 } };
// vs
const e = Distance.feet { 1.0 }

I also don't understand what you mean in the second case, could you explain further or show an example of a solution?

That's a reasonable argument about unnecessary nesting. On the other hand you could make a shortcut function:
fn meters(value: f64) -> Distance {
    Distance.meters { Meters { .value = value, } }
}

const d = meters(1.0);

Also, if you have this enum that's really all you need to distinguish:
const Distance = enum {
    Meters: f64,
    Feet: f64,
};
const d = Distance.Meters { 1.0 };

In the second case, for example:
const MyThing = struct {
    // private fields
    a: i32,
    b: i32,

    pub fn init() -> MyThing {
        return MyThing { .a = 1, .b = 2, };
    }
    pub fn doSomething(self: &MyThing) {
    }
}

There is one more thing, that I neglected to mention, that we use typedefs for, and that is when we do a C import and we see something like:
struct Foo;
void do_something(struct Foo *foo);
struct Bar;
void do_something_bar(struct Bar *bar);

We need Foo to now represent an opaque type that can't accidentally be passed to do_something_bar.
We also need a way to represent this type in zig even if we don't parse a C import.
However the way we use typedefs for this is suboptimal. Currently we do:
type Foo = u8;

The problem with this is that the underlying type of Foo is not actually u8, it is opaque. So this typedef lets you do something like foo += 1 which should be a compile error.
So this proposal would be to delete typedefs and then introduce a way to create a truly opaque type.

Zig doesn't actually have private fields; it just has fields, right?
I agree with getting rid of the awful Feet vs Meters nonsense. I have a long rant prepared about how putting units of measurement into the type system of a programming language is too hard to be worth while, but i won't dump that here (unless someone asks).
The ability to create opaque types seems valuable, especially for expressing C header concepts in Zig. How about:
const Foo = @OpaqueType();
fn do_something(foo: &Foo) {}

Another usecase for typedefs that haven't been discussed yet is creating variations on existing types that have different attributes. For example, making a non-copyable version of a struct. I can try to think of a more fleshed out example, but I'm actually in favor of removing typedefs instead of trying to come up with a problem to fit the solution. I don't think the existing usecases for typedefs are very valuable.

Zig doesn't actually have private fields; it just has fields, right?

Status quo is we parse the word pub in front of fields, but then do nothing with the information, and there are no compile errors for accessing private fields.


key values are not saved to shapefile/graphml

After saving the graph to shapefile or graphml the "key" field is set to 0 for every element.

Can you provide a minimal working example and your package/python version info? The key should be 0 for every edge except for parallel edges, which are rare. And there should be no key attribute for nodes. So if your graph has no parallel edges, you should see a 0 for every key attribute value.
From the networkx docs:

For MultiGraph/MultiDiGraph we use a dict-of-dicts-of-dicts-of-dicts where the third dictionary is keyed by an edge key identifier to the fourth dictionary which contains the edge attributes for that edge between the two nodes.

Thanks @gboeing, I misunderstood the key field.

Hi @gboeing,
First wanted to thank you for creating this package - it's been extremely helpful for analysis!
Just wanted to point out that this issue actually prevents Gephi from correctly reading the saved graphML properly, as it's unable to detect different edges due to the duplicate id.
Simple code example:
G = ox.graph_from_place('Berkeley, California', network_type = "drive")
ox.save_load.save_graphml(G, filename="berkeley.graphml", folder=None)

Resulting edges in graphML (excerpt, truncated for conciseness)
<edge id="0" source="53030508" target="53030507">
</edge>
<edge id="0" source="53030510" target="53033490">
</edge>
<edge id="0" source="53030510" target="53030514">
</edge>

When trying to open the graphML file in Gephi:

Thanks @gboeing
As a quick workaround using osmnx, in case anyone is interested in exporting the graphs to work in Gephi:
Using some osmnx graph G:
G = ox.graph_from_point(center_point=[37.870288, -122.259479], distance=500, network_type="bike")

Split the graph into nodes and edges gdf:
G_nodes = ox.graph_to_gdfs(G, nodes = True, edges = False)
G_edges = ox.graph_to_gdfs(G, nodes = False, edges = True)

Re-assign key values so each edge gets a unique key:
G_edges.key = range(G_edges.shape[0])

Convert the gdf's back into a graph, and save it as graphml:
H = ox.gdfs_to_graph(gdf_edges=G_edges, gdf_nodes=G_nodes) # WORKAROUND WORKS!!
ox.save_load.save_graphml(H, filename="singapore.graphml", folder=None)

Resulting graph can be opened in Gephi:

@tauhlim Gephi-compatible graphml saving is now added as a workaround in b5f30c203f504354a25597f70900a713a9f1bc0a including preserving node coordinates so you can run Gephi's GeoLayout.


[Support] Fix and improve some codeblocks and documentations

I found 2 mistakes on the french documentation and take some times to improve the codeblocks ; inspired from #8050

Merci pour le PR @JeromeBotineau !! 👍

في الثلاثاء، ١٨ يوليو ٢٠٢٣ ٧:٤٢ م Nic @.***> كتب:

Merged #9917 https://github.com/cloudflare/cloudflare-docs/pull/9917
into production.
—
Reply to this email directly, view it on GitHub
https://github.com/cloudflare/cloudflare-docs/pull/9917#event-9852854261,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWV6NCXMIK4QM34ZYE4WJSTXQ24GDANCNFSM6AAAAAA2ORLANQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
com>


Set up basic CI infrastructure

@simonechiocchetti I have set up a basic CI testing infrastructure (+ spell checking). Please have a look if you feel comfortable with the changes I made. If you are, feel free to merge at any time - otherwise we can discuss it later 😊


docs: add Community Maintainers Guidelines

Description
This PR adds Community Maintainers Guideline with changes as follow:
In the contributing folder inside docs:

[x] Create a subfolder called opensauced-maintainers-guide.
[x] Create a community-maintainers-guide.md and add the guidelines content here.
[x] Move maintainers-guide.md into the new folder.
[x] Update the sidebars.js.

What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[ ] 🐛 Bug Fix
[x] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents
Closes #242

Mobile & Desktop Screenshots/Recordings

Steps to QA

Added to documentation?

[ ] 📜 README.md
[x] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

@BekahHW I adjusted some wording, so I need your review for all and please let me know if I need to fix anything. Thank you!

@adiati98 what does this mean - Issues outside the good first issue can be assigned max. 2 times per contributor. Suggest they check other repositories for more issues.

@BekahHW after worked on this, I have these ideas:

Change the title and sidebar_label in the maintainers-guide.md from "OpenSauced Maintainers Guide" to "Becoming OpenSauced Maintainer". Because most content in this page are information about how to become OpenSauced maintainer.
https://github.com/open-sauced/docs/blob/943d2e41f9bf6bcab1e484303c9736c787ed9ff6/docs/contributing/maintainers-guide.md?plain=1#L3-L4

Remove the "Committing and Merging Changes" section in the maintainers-guide.md because this is not necessary here. It's already covered in the "Community Maintainers Guidelines".
https://github.com/open-sauced/docs/blob/943d2e41f9bf6bcab1e484303c9736c787ed9ff6/docs/contributing/maintainers-guide.md?plain=1#L47-L52

Looking at the chapters, maybe we can change the title of "Contributing Guide" to "OpenSauced Guide" or "OpenSauced Guidelines"?

@adiati98 what does this mean - Issues outside the good first issue can be assigned max. 2 times per contributor. Suggest they check other repositories for more issues.

Issues that are not good first issue can only be assigned max. 2 times per contributor in the community repos. And we can suggest to other repos for more issues. I refer to your reply here. Or maybe I get it wrong? 😅

@adiati98 what does this mean - Issues outside the good first issue can be assigned max. 2 times per contributor. Suggest they check other repositories for more issues.

@BekahHW Issues that are not good first issue can only be assigned max. 2 times per contributor in the community repos. And we can suggest to other repos for more issues. I refer to your reply here. Or maybe I get it wrong? 😅

Ah, ok. People can take on more than two issues but not at a time. So, for example if they have an issue in guestbook and one in intro, they can't have one in pizza-verse.
My biggest concern is that contributors ask to take on issues across repos and don't finish them. I want them to contribute. But I want them to complete them.
So maybe more like:
Issues outside the good first issue can be assigned max. 2 times per contributor at a time. We love contributions, but we want to equip contributors to succeed by giving them time to complete issues. Through experience, we've found that two issues at a time allows them to focus on completing the issue and supports maintainers by not having issues open for long periods of time.

I update the wording as per your suggestion.
One last thing, should I change the file name as what you mentioned earlier from community-maintainers-guide.md to community-maintainers.md?
Once it confirmed, I can fix it and let you know or you can merge this PR.


Remove unnecessary else block

If the if let statement is true, we're guaranteed to break out of the loop, so there's no point in having an else block
Fixes #93

Better visualized when ignoring whitespace:
https://github.com/frewsxcv/tiny-http/pull/94/files?w=1


add new cb client: consumer/resume-profile

hi team,
we try to call https://api.careerbuilder.com/consumer/resume-profile to convert job seeker profile into resume, but the API end pint does not include within ruby-cb-api repository now.
could your team to help code this api into repository, or we code it and make pull request to you?
thanks,
yong

Hi @yongzhu, I'll reach out to you on Hipchat


Remove error logger pushing code from parsing routine

As per this comment by @dsyme I was curious if the code could just be removed. I actually have no clue why or why not this would or would not work.  But:

Assertion is no longer present
In a small test project things seemed to work
In VF# things seemed to work
Setting breakpoints in VF# seemed the same

I tried to run tests locally but my machine got a BSOD so I'll just let CI do that instead.

@dotnet-bot Windows_NT Release_ci_part2

Timeout

@dotnet-bot test this please

I've been playing around with this + typing plenty of code for the entire morning and the editor feels the same.  Not exactly a descriptive test, but it "feels" the same for me so I suppose that's something.


Can't use #html_safe inside collection decorators (?)

I have the following
class ThreadDecorator < Draper::Decorator
  delegate_all
  decorates_association :tags
  # bla bla...
end

class TagsDecorator < Draper::CollectionDecorator
  def to_s
    decorated_collection.map do |tag|
      h.content_tag(:span, tag.name, class: 'label label-info')
    end.join(' ')
  end
end

The 'issue' is that appending .html_safe at the end of TagsDecorator#to_s won't escape the result, so I have to put decorated_thread.tags.to_s.html_safe. which looks a little bit akward in the views.
Not a life or death issue, but does this something to do with draper?

Please reopen if you have any new info.


Instructions on how to apply the affect in your own project?

That would be helpful.

Player/sprite 's material and WorldEnvironment in the main scene is basically everything about this effect.Others is about controll or animation or something else.The player's material sets specific hdr color to part of sprite,and the worldenvironment makes all the hdr color glow.If it's still unclear you can check Brackeys Unity 2D GLOW tutorial.It's for unity but the main idea is same.

Cool. Maybe put that in the README?

Cool. Maybe put that in the README?

Done.


R Tree Implementation

Is this Module https://sqlite.org/rtree.html implemented in iOS and Android, or only in one, or in none of them?

it looks like it's by default only support Rtree on iOS.
https://github.com/andpor/react-native-sqlite-storage/issues/328